Open
Conversation
…created via swapchain image objects.
… Make graphics contexts thread-local in the implemenetation, this still needs to be done on the agnostic side. Delete tons of device code. TODO: Allow Swapchain images to present and to resolve where the queue submit logic will live.
…ring, figure out how we're gonna cache descriptor sets.
…bably should? It worked when I used it last.
We probably need a better API for this at some point...
FiniteReality
force-pushed
the
feature/ecs-time-context-refactor
branch
from
9488d61 to
fc76b03
Compare
capnkenny
reviewed
Mar 14, 2026
Member
capnkenny
left a comment
capnkenny
left a comment
There was a problem hiding this comment.
Got some questions and nits, will continue reviewing in a while
Graphics/Core/include/NovelRT/Graphics/SecondaryCmdListInfo.hpp
Outdated
Show resolved
Hide resolved
This reverts commit 92017de.
This is a workaround until we get the scheduler dependency graph implemented.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Basically #622 but based on a different concept. This is mainly just for comparison.