-
Notifications
You must be signed in to change notification settings - Fork 83
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
Add support for getters #587
Merged
Merged
Commits on Nov 30, 2023
-
Getters are designed with the expectation that they will be called twice. On the first pass the getter is added to a queue and a callback function is set on each. Once the getter resolves the result is set in the results dict and getters are ready to be called again to return their result.
Configuration menu - View commit details
-
Copy full SHA for aec5993 - Browse repository at this point
Copy the full SHA aec5993View commit details -
Configuration menu - View commit details
-
Copy full SHA for 9a998a0 - Browse repository at this point
Copy the full SHA 9a998a0View commit details -
ENH: Listen to pre_run_cell signal
This allows us to grab the cell id and raw cell string for re-running cells with getters.
Configuration menu - View commit details
-
Copy full SHA for e1a74b7 - Browse repository at this point
Copy the full SHA e1a74b7View commit details -
ENH: Build a CellWatcher class to monitor cells before running
Intercept execute_requests (which will be all cells queued to run after the Viewer class has been instantiated. Once the itk_viewer has been created the queue is processed one request at a time. If a cell contains getters, futures are created for each and a callback function is set to set the result as they resolve. Once all of the results are set the cell is run and the queue continues to be processed.
Configuration menu - View commit details
-
Copy full SHA for 4d4464e - Browse repository at this point
Copy the full SHA 4d4464eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 0f59c64 - Browse repository at this point
Copy the full SHA 0f59c64View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6b77217 - Browse repository at this point
Copy the full SHA 6b77217View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4138f47 - Browse repository at this point
Copy the full SHA 4138f47View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6b03793 - Browse repository at this point
Copy the full SHA 6b03793View commit details -
Configuration menu - View commit details
-
Copy full SHA for 06aa218 - Browse repository at this point
Copy the full SHA 06aa218View commit details -
Configuration menu - View commit details
-
Copy full SHA for cba9c04 - Browse repository at this point
Copy the full SHA cba9c04View commit details -
Configuration menu - View commit details
-
Copy full SHA for 44e9913 - Browse repository at this point
Copy the full SHA 44e9913View commit details -
Configuration menu - View commit details
-
Copy full SHA for 1915084 - Browse repository at this point
Copy the full SHA 1915084View commit details -
Configuration menu - View commit details
-
Copy full SHA for e6f0bcc - Browse repository at this point
Copy the full SHA e6f0bccView commit details -
Configuration menu - View commit details
-
Copy full SHA for c2783ca - Browse repository at this point
Copy the full SHA c2783caView commit details -
BUG: Fix logic for executing next request
If we have a current request when the queue is empty we still need to execute that cell.
Configuration menu - View commit details
-
Copy full SHA for d0c41c2 - Browse repository at this point
Copy the full SHA d0c41c2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 916dd5c - Browse repository at this point
Copy the full SHA 916dd5cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 397fc00 - Browse repository at this point
Copy the full SHA 397fc00View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2218712 - Browse repository at this point
Copy the full SHA 2218712View commit details -
Configuration menu - View commit details
-
Copy full SHA for a0b8352 - Browse repository at this point
Copy the full SHA a0b8352View commit details
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.