Closed imanjra closed 3 weeks ago
The updates primarily focus on enhancing the Operator
class functionalities and improving the flexibility of input and output handling in the OpenDataset
class. Additionally, new utility functions and unit tests have been introduced to support these changes. The modifications allow for optional dataset input, the ability to skip input/output processing, and dynamic method checking for operator configurations.
File Path | Change Summary |
---|---|
app/packages/operators/src/built-in-operators.ts |
Modified OpenDataset class to make the "dataset" input parameter optional. |
app/packages/operators/src/operators.ts |
Added skipInput and skipOutput properties to OperatorConfigOptions and OperatorConfig classes. Updated Operator class methods to respect these properties. |
fiftyone/operators/server.py |
Enhanced ListOperators class to dynamically update configurations based on overridden methods. |
fiftyone/operators/utils.py |
Introduced is_method_overridden function to check if a method is overridden in a subclass. |
tests/unittests/operators/utils_tests.py |
Added unit tests to verify the functionality of the is_method_overridden utility function. |
In the land of code where rabbits play,
Operators now have a say,
Inputs optional, outputs too,
Flexibility in what they do.
With tests to guide and methods clear,
Our code hops forward, have no fear!
🐇💻✨
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?
open_dataset
validation error when the optionaldataset_name
param is not provided (can be undefined to unselect current dataset)How is this patch tested? If it is not, please explain why.
Using an example operator in the app and unit test
Release Notes
Is this a user-facing change that should be mentioned in the release notes?
See the list above
What areas of FiftyOne does this PR affect?
fiftyone
Python library changesSummary by CodeRabbit
New Features
skipInput
andskipOutput
properties to operator configurations for enhanced control over input/output processing.Bug Fixes
required
constraint for the "dataset" input parameter in theOpenDataset
class to improve flexibility.Tests