-
Notifications
You must be signed in to change notification settings - Fork 23
Live Swapping #566
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Foifur
wants to merge
25
commits into
52X
Choose a base branch
from
52X_Live_Swapping
base: 52X
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Live Swapping #566
Conversation
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
- Added new condition instead of tricking FirstConnect
I added the value to the stash key but not the cell itself, oops
Should have been doing it from here the whole time, whoops
Now that swapping emulators without closing RTC is possible, we need to parts of the UI that only set on form shown (which only happens once)
Now that SwapImplementation is callable from anywhere, tons of buttons and methods in UICore and CorruptCore need to be able to await the call to it.
I kind of hate that it has to be done this way, but the effort of overhauling the entire message queue to work asynchronously requires way too much conversion
If it's not cancelled, it continues to run in the background, so if you try to swap implementations when it triggers it will incorrectly say it failed
This way we can call for the hiding of the progressForm and unlock the interface from anywhere
This is so the launcher can tell RTC to immediately unlock the interface after swapping, since we won't be loading any games yet
Whoops, forgot this so the build tests weren't happy
We don't need to worry about checking for mixed stockpiles on a per implementation basis anymore
I missed this because we normally have our folders as upper case, so if there isn't it will never confirm the check
We already have the lists loaded, so no need to do it again
This was in originally but accidentally got wiped away after the contextbuilder update
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR adds the ability to swap between emulators without closing down StandaloneRTC.exe. This gives us the benefit of not having to save/close any work that is currently ongoing in RTC, and opens up the ability to swap implementations gracefully without using /taskkill. It also allows cross emulator stockpiles to be created, as it can seamlessly swap between emulators and load stockpiles.