-
Notifications
You must be signed in to change notification settings - Fork 69
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
[WIP] - MPcule API #645
[WIP] - MPcule API #645
Conversation
…through molecule world)
…ing tests pass in emmet-core (including remaking JSON files for summary)
… names and instead create utility function to define a solvent string
…th LevelOfTheory with pre-defined solvents)
…agate to property documents
Reminder: need to regenerate "builder_XXX_set.json" test files once builders are all done
…ode to ensure long-term fixity of graph hashes.
… actually be used
Adding tests for minimal MPcules API routes
Seems there's an issue setting up the tests - @munrojm any idea what's up? |
Look like an issue with conda env install in the actions workflow. My other branch isn't having an issue so I am going to re-run and see if it persists. If it does, I will take a closer look. |
Codecov ReportBase: 90.49% // Head: 91.97% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #645 +/- ##
==========================================
+ Coverage 90.49% 91.97% +1.48%
==========================================
Files 133 101 -32
Lines 9760 8770 -990
==========================================
- Hits 8832 8066 -766
+ Misses 928 704 -224 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Think we're good here @munrojm! |
Work in progress
Creating a new API for molecular data collections, to be used both in isolation and in an upcoming Materials Project web app.
Contributor Checklist