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

Requirements file and re-run of the notebooks #16

Merged
merged 2 commits into from
Nov 3, 2021
Merged

Conversation

cemsaz
Copy link
Contributor

@cemsaz cemsaz commented Nov 2, 2021

Added requirements file, tested the notebooks and updated the Autogluon notebook to the latest API.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@cemsaz cemsaz requested a review from bwerness November 2, 2021 23:39
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
Contributor

bwerness commented Nov 2, 2021

Please update the following things:

  • Remove temporary files in the hidden .ipynb_checkpoints folder
  • replace header in the Autogluon-CV notebook

After that, this PR is ready for approval!

@cemsaz
Copy link
Contributor Author

cemsaz commented Nov 3, 2021

@bwerness, I removed the checkpoints and added the figure header to the autogluon nb.

Copy link
Contributor

@bwerness bwerness left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@bwerness bwerness merged commit fc04634 into master Nov 3, 2021
@cemsaz cemsaz deleted the add-lib-versions branch December 10, 2021 22:48
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.

2 participants