Skip to content
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

Release 0.3 #46

Merged
merged 146 commits into from
Nov 13, 2018
Merged

Release 0.3 #46

merged 146 commits into from
Nov 13, 2018

Conversation

tkornuta-ibm
Copy link
Contributor

@tkornuta-ibm tkornuta-ibm commented Oct 31, 2018

This release adresses several bugs and brings new features:
What's new?

Misc:
#28, #33, #38, #70, #88

vmarois and others added 30 commits October 24, 2018 14:21
Should not break alexnet anymore. We still need torchvision==0.2.0 for the doc build, but no longer rely on setup.py for the doc build & use a docs/requirements.txt where we specify the version.
…n deal with both options when loaded from config or external file passes in config
[Feature] Enhancing the documentation
tkornuta-ibm and others added 3 commits November 8, 2018 23:21
… with wrongly formated values in grid-analyzer
added :py:class:miprometheus.utils.StatisticsAggregator in comments
@tkornuta-ibm
Copy link
Contributor Author

This pull request introduces 2 alerts and fixes 39 when merging d8c20f1 into b4f2442 - view on LGTM.com

new alerts:

  • 2 for Non-callable called

fixed alerts:

  • 16 for Except block handles 'BaseException'
  • 16 for Module is imported with 'import' and 'import from'
  • 4 for Variable defined multiple times
  • 1 for Unused local variable
  • 1 for 'import *' may pollute namespace
  • 1 for First argument to super() is not enclosing class

Comment posted by LGTM.com

tkornuta-ibm and others added 23 commits November 9, 2018 12:41
…e, changed trainer command line argument handling - fixes #62)
…ged in online trainer 2) added saving model after last epoch in offline trainer 3) importing aggregated test statistics in grid-analyzer
…ts at the end of training (when epoch/episode limit is hit), modifies model save. Fixes #85
Minor change in stop threshold in MNIST configs (plus test of closing issues with keywords when merging with develop)
@tkornuta-ibm
Copy link
Contributor Author

This pull request introduces 2 alerts and fixes 39 when merging 2a92103 into b4f2442 - view on LGTM.com

new alerts:

  • 2 for Non-callable called

fixed alerts:

  • 16 for Except block handles 'BaseException'
  • 16 for Module is imported with 'import' and 'import from'
  • 4 for Variable defined multiple times
  • 1 for Unused local variable
  • 1 for 'import *' may pollute namespace
  • 1 for First argument to super() is not enclosing class

Comment posted by LGTM.com

@tkornuta-ibm tkornuta-ibm changed the title Test merge of develop into master Release 0.3 Nov 13, 2018
@tkornuta-ibm tkornuta-ibm merged commit 2be45de into master Nov 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants