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

Removed use of torch_lr_finder from notebooks #55

Merged
merged 1 commit into from
May 18, 2022

Conversation

adcroft
Copy link
Contributor

@adcroft adcroft commented May 17, 2022

Since the environment no longer had torch-lr-finder I've removed the import where it was unused in the notebooks, and commented the import and use in the one notebook that did use it..

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

- Since the environment will not have torch-lr-finder I've removed the
import where it was unused and commented it out otherwise.
@adcroft adcroft force-pushed the remove-lr-finder branch from d646573 to 7af0412 Compare May 17, 2022 21:15
@rabernat rabernat merged commit fd7ce1e into m2lines:main May 18, 2022
@adcroft adcroft deleted the remove-lr-finder branch May 19, 2022 14:52
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