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

Cleanup notebooks folder #716

Merged
merged 7 commits into from
Jan 14, 2023

Conversation

andrewm4894
Copy link
Collaborator

@andrewm4894 andrewm4894 commented Jan 14, 2023

A little bit of housekeeping since notebooks seem to be coming in quite frequently. Idea here is to standardize and add a little more of a flexible folder structure to make it as easy as possible to browse notebooks in GitHub.

  • rename "data-argumentation" folder to "data-augmentation".
  • standardize file naming to be a little more pythonic.
  • one folder per notebook plus a README for easy GH viewing.
  • adjust colab badges for new structure.

May be easier to review this PR by just browsing in here: https://github.com/andrewm4894/Open-Assistant/tree/cleanup-notebooks-folder/notebooks

- standardize file naming to be a little more pythonic.
- one folder per notebook plus a README for easy GH viewing.
- adjust colab badges for new structure.
@github-actions
Copy link

pre-commit failed.
Please run pre-commit run --all-files locally and commit the changes.
Find more information in the repository's CONTRIBUTING.md

@github-actions
Copy link

pre-commit failed.
Please run pre-commit run --all-files locally and commit the changes.
Find more information in the repository's CONTRIBUTING.md

1 similar comment
@github-actions
Copy link

pre-commit failed.
Please run pre-commit run --all-files locally and commit the changes.
Find more information in the repository's CONTRIBUTING.md

@andrewm4894 andrewm4894 marked this pull request as ready for review January 14, 2023 19:39
Copy link
Collaborator

@yk yk left a comment

Choose a reason for hiding this comment

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

thank you!

@andrewm4894 andrewm4894 enabled auto-merge (squash) January 14, 2023 22:49
@andrewm4894 andrewm4894 merged commit 86cadc6 into LAION-AI:main Jan 14, 2023
@andrewm4894 andrewm4894 deleted the cleanup-notebooks-folder branch January 14, 2023 22:51
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