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

Update for Materials Project v2 API #190

Merged
merged 5 commits into from
Sep 18, 2023
Merged

Update for Materials Project v2 API #190

merged 5 commits into from
Sep 18, 2023

Conversation

matthewcarbone
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Sep 18, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.81% 🎉

Comparison is base (cd1058f) 80.09% compared to head (a5bbf17) 80.90%.
Report is 2 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #190      +/-   ##
==========================================
+ Coverage   80.09%   80.90%   +0.81%     
==========================================
  Files          13       13              
  Lines        1040     1011      -29     
==========================================
- Hits          833      818      -15     
+ Misses        207      193      -14     
Files Changed Coverage Δ
lightshow/__init__.py 83.33% <100.00%> (ø)
lightshow/database.py 89.86% <100.00%> (+6.24%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@matthewcarbone
Copy link
Contributor Author

@FCMeng no review necessary, but I've updated Lightshow for the Materials Project v2! 😁

@matthewcarbone matthewcarbone added this to the v2 milestone Sep 18, 2023
@matthewcarbone matthewcarbone merged commit 5766b2c into master Sep 18, 2023
14 checks passed
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