HectorUI v3.1 #85
Merged
HectorUI v3.1 #85
Conversation
This file contains 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
New variables can be selected without having to click "Load Graphs" first, by carrying over the "runs" var as r6$output instead of creating a new blank variable. This does cause the app to crash if a new SSP is selected after selecting new variables, though, so it's not a complete fix
…ad Graphs"" This reverts commit cf4de20.
Selecting a new output variable now draws the chosen SSP from the R6 class rather than the shiny input, ensuring that it uses the SSPs that were selected at the time "Load Graph" was last clicked (or when app was opened). This does impact plotting (and downloading data) -- when you change output variables, it will only plot one SSP and you have to click Load Graph to show both again.
Another update to the flow of selecting SSPs and parameters -- the app will no longer crash, but users will need to select Load Graphs when selecting new parameters or else only the first selected SSP will plot. Gets rid of disconnect issue (seemingly completely?) but does cause issue with plotting.
When a new output variable is selected, the SSPs/parameters selected the last time Load Graphs was clicked (or when the app loaded) will remain the same, until Load Graphs is clicked again.
Condensed line of code where vars are fetched when output variable is changed
Variables temporarily removed from app, as there is an issue with Hector that causes values to not be affected by parameter changes correctly. Will be added back to the app when the issue is fixed.
Fixed disconnect when output variable is chosen before "Load Graph"
FFI and LUC emissions now incorporate their XXX_uptake counterparts when graphing
Volcanic/aerosol forcing variables temporarily removed
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 Minor patch includes bug fixes from