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

Datapipe Benchmarks #123

Closed

Conversation

loliverhennigh
Copy link
Collaborator

@loliverhennigh loliverhennigh commented Oct 10, 2023

Modulus Pull Request

Related to issue here #117

Description

Implement datapipe benchmarks as an example that is easy for users to run. This will both allow us to show new users a compelling reason to use modulus while also allowing us to iterate on datapipe ideas such as zarr stores.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • The CHANGELOG.md is
    up to date with these changes.

@loliverhennigh loliverhennigh changed the title added benchmarks Datapipe Benchmarks Oct 10, 2023
@NickGeneva NickGeneva added the 2 - In Progress Currently a work in progress label Oct 10, 2023
@loliverhennigh loliverhennigh added 3 - Ready for Review Ready for review by team and removed 2 - In Progress Currently a work in progress labels Oct 26, 2023
@loliverhennigh
Copy link
Collaborator Author

/blossom-ci

@loliverhennigh
Copy link
Collaborator Author

/blossom-ci

1 similar comment
@loliverhennigh
Copy link
Collaborator Author

/blossom-ci

@loliverhennigh loliverhennigh added 0 - Blocked Cannot progress due to external reasons and removed 3 - Ready for Review Ready for review by team labels Nov 3, 2023
@NickGeneva
Copy link
Collaborator

Closing since PR should move to core repo.

@NickGeneva NickGeneva closed this Nov 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 - Blocked Cannot progress due to external reasons
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants