-
Notifications
You must be signed in to change notification settings - Fork 488
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
Feature/sg 000 add adapter #1471
Conversation
…ature/SG-000-add_adapter_v2
tests/integration_tests/data_adapter/test_data_adapter_on_sg.py
Outdated
Show resolved
Hide resolved
Thanks for the Colab notebook. It really helpful to see how the intended usage.
Sure, heuristics may not handle ALL use cases but would certainly save some time in 90% of the cases.
As we've already asked user about number of classes when creating adapter data loader, we should be able to pull this information from that instance:
|
Not fundamentally against the change, but I have a small concern; We could change the method to convey this idea, for instance Eventually, my point is I would like the name (of class and method combined) to include the info that this builds a "dataloader which adapts X for a task Y" with X (dataset/dataloader). |
Yeah I wasnt too sure about whether to simplify or to be more explicit. But I think you're right, with the context it is (relatively) clear that the config_path is that of the current object (i.e. adapter) I changed both |
That's an interesting idea. I will create a task for this, and maybe to include this idea to other questions (if possible/useful). |
Good point, i'll change that |
Note: the build fails because we did not release yet the latest DG version. |
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
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
Introduction of Adapter logic
Why Do We Need This?
We aim to allow seamless integration with SG for users who are already using DG.
Solution (updated)
If the user doesnt want to set anything in advance (in such case he will be asked questions) OR if he already has a cache (from DG or previous run)
If the user wants to set in advnace some parameters, he can still use
DetectionDataConfig
Now, the same is possible if the user already has a dataloader.
Notebook
https://colab.research.google.com/drive/1clgfrdCrg5cWSl7nrjY0zVndz06SIC0P?usp=sharing