-
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
#130 Add mypy and step to test imports #155
Conversation
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.
Thanks alot, just one comment :)
poetry run python -c "import baal; import baal.active.dataset; \ | ||
import baal.active.heuristics; import baal.active.active_loop; \ | ||
import baal.bayesian; import baal.calibration; import baal.modelwrapper" | ||
|
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.
why do we need to test imports? is something changing or it was for internal test?
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.
We need to be sure that pip install baal
doesnt require HuggingFace or torchvision.
* dry-run-example * Fix loop * Fix nbsphinx to 0.8.6 * #130 Add mypy and step to test imports (#155) * Fix build * Fix structlog * Update flash * Update script * Log heuristic * Seems to work now * Unfreeze backbone * Reduce number of epochs * Add wd * reset epoch * Format * Update Flash and remove custom code * Split API doc in multiple files (#158) Co-authored-by: fr.branchaud-charron <fr.branchaud-charron@servicenow.com> * Lint Co-authored-by: parmida.atighehchian <parmida.atighehchian@servicenow.com> Co-authored-by: fr.branchaud-charron <fr.branchaud-charron@servicenow.com> Co-authored-by: Frédéric Branchaud-Charron <frederic.branchaud-charron@elementai.com> Co-authored-by: Dref360 <frederic.branchaud.charron@gmail.com>
Summary:
Add Mypy and a step in Circle to test for imports.
Features:
Fixes #130
Checklist:
tests/documentation_test.py
).