-
Notifications
You must be signed in to change notification settings - Fork 1
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
Full Custom Tasks Support #53
Conversation
Test Results 4 files 4 suites 1h 15m 40s ⏱️ Results for commit 67752ed. ♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Just two question:
- Now that most of the protocols were removed do we still keep the
FeaturesProtocol
? - Are the LuxonisDataset datasets created before still compatible with this newer version of luxonis-train/luxonis-ml?
That one is used in
No, only datasets created after #110 are compatible ( |
Co-authored-by: KlemenSkrlj <[email protected]>
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified Files
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't review the code, but from manual testing some custom task applications it looks good
Co-authored-by: KlemenSkrlj <[email protected]> Co-authored-by: GitHub Actions <[email protected]>
Added full support for custom task names. When using custom task names, a new field
task
has to be specified in the configuration of the corresponding head node. Thetask
gets propagated to all the modules attached to the head (losses, metrics, etc.).Example
Labels
config.yaml
Subclassing Changes
Minor breaking changes introduced for implementing custom nodes and attached modules.
__init__
arguments.BaseAttachedNode.get_label
andBaseAttachedNode.get_input_tensor
now do the checking insteaddNodes
old_node.py
new_node.py
Attached Modules
old_loss.py
new_loss.py