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

Updates #16

Merged
merged 22 commits into from
Sep 26, 2024
Merged

Updates #16

merged 22 commits into from
Sep 26, 2024

Conversation

nwlandry
Copy link
Collaborator

This PR does the following:

  • Fixes Rename attr to attrs #15.
  • Moves extraneous files related to performance testing, integration testing, and other functionality to _OBS
  • Moved all of the files except for the schema and the CHANGELOG out of the schemas folder.
  • Made two requirements files: one for unit tests and one for the tutorials.
  • Updated the README.
  • Moved all of the Jupyter notebooks to the tutorials folder. Made a notebook for each library and an "integration" notebook showcasing the ability to exchange network datasets between libraries.

@nwlandry
Copy link
Collaborator Author

@colltoaction --- are you able to review this?

Copy link
Collaborator

@colltoaction colltoaction left a comment

Choose a reason for hiding this comment

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

The rest looks good!

README.md Show resolved Hide resolved
@nwlandry
Copy link
Collaborator Author

Thanks so much for the review, @colltoaction! Really appreciate it!

@nwlandry nwlandry merged commit 7aefda6 into main Sep 26, 2024
1 check passed
@nwlandry nwlandry deleted the updates-nwl branch September 26, 2024 16:26
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.

Rename attr to attrs
3 participants