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

New tutorials #53

Merged
merged 7 commits into from
Apr 27, 2023
Merged

New tutorials #53

merged 7 commits into from
Apr 27, 2023

Conversation

timnaher
Copy link
Contributor

@timnaher timnaher commented Apr 27, 2023

Summary

  • added 2 tutorials on how to use ACME in a machine learning setting
  • changed the default ESI cluster partition in run_tests.sh from DEV to 8GBDEV

Author Guidelines

  • Is the change set < 400 lines?
  • Is the code running locally and on HPC infrastructure?
  • Is the code running on all supported platforms (macOS, Linux and Windows)?

Reviewer Checklist

  • Are testing routines present? N/A
  • Do objects in the global package namespace perform proper parsing of their input? N/A
  • Are all docstrings complete and accurate?
  • Are proposed changes tracked in CHANGELOG.md?

pantaray and others added 6 commits April 17, 2023 11:15
- rewind back to start collecting changes

 On branch dev
 Changes to be committed:
	modified:   CHANGELOG.md
- changed the default slurm queue from DEV to 8GBDEV in run_tests.sh so that tests run and queue permission error is fixed

On branch new_tutorials
Changes to be committed:
	modified:   ../../CHANGELO
	modified:   run_tests.sh
Changes to be committed:
	modified:   contributing.rst
- added  example workflows for machine learning pipelines with ACME

Changes to be committed:
	new file:   classifier_tutorial.rst
	new file:   deeplearning_tutorial.rst
	new file:   images/classifier_tutorial.png
Changes to be committed:
	modified:   CHANGELOG.md
@pantaray pantaray self-requested a review April 27, 2023 11:36
@pantaray pantaray self-assigned this Apr 27, 2023
@pantaray pantaray added the documentation Improvements or additions to documentation label Apr 27, 2023
Copy link
Member

@pantaray pantaray left a comment

Choose a reason for hiding this comment

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

These are fantastic tutorials! Thanks so much @timnaher for the contribution! Where do you think would be a good place to link to them? I was thinking to put them right on the landing page as separate item in the "User Guide" section. Maybe even have them show up as separate "Tutorial" section in the top-bar menu?

Thanks again for the great work!

Changes to be committed:
	modified:   source/classifier_tutorial.rst
	modified:   source/deeplearning_tutorial.rst
@timnaher
Copy link
Contributor Author

These are fantastic tutorials! Thanks so much @timnaher for the contribution! Where do you think would be a good place to link to them? I was thinking to put them right on the landing page as separate item in the "User Guide" section. Maybe even have them show up as separate "Tutorial" section in the top-bar menu?

Thanks again for the great work!

thanks, happy to contribute! :)
Yes, I think having them as a separate section makes sense. With time we could add some more... Maybe also some neuroscience related ones?

@pantaray
Copy link
Member

Thanks for the changes - looks great! I'll put these great tutorials into a dedicated section and make them available from the top down menu. Thanks again for the fantastic work!

@pantaray pantaray merged commit dc7965d into esi-neuroscience:main Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants