Closed imanjra closed 2 months ago
The recent changes aim to streamline workspace management within the application. These updates introduce the LOAD_WORKSPACE_OPERATOR
for dynamic workspace loading and refine the method for setting spaces, enhancing code readability and operational efficiency.
File Path | Change Summary |
---|---|
.../built-in-operators.ts |
Added LOAD_WORKSPACE_OPERATOR import and updated SetSpaces class to handle workspace loading. |
.../builtin.py |
Updated resolve_input method to directly use set_spaces and removed event triggering. |
.../operations.py |
Added set_spaces method to facilitate setting spaces by name or workspace object. |
š°āØ
In the code where spaces intertwine,
A rabbit hopped, leaving marks behind.
"Let's tidy this up," it said with glee,
Now workspaces load with such simplicity!
ššš
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
What changes are proposed in this pull request?
Add
set_spaces
operation toctx.ops
available in operators executeHow is this patch tested? If it is not, please explain why.
Set workspace
Set arbitrary space
Release Notes
Is this a user-facing change that should be mentioned in the release notes?
See usage above
What areas of FiftyOne does this PR affect?
fiftyone
Python library changesSummary by CodeRabbit