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
Electrode Doc #149
Electrode Doc #149
Conversation
added builder to popluate URL data for references added builder to popluate URL data for references
Codecov Report
@@ Coverage Diff @@
## master #149 +/- ##
==========================================
- Coverage 93.84% 93.48% -0.37%
==========================================
Files 24 25 +1
Lines 1414 1488 +74
==========================================
+ Hits 1327 1391 +64
- Misses 87 97 +10
Continue to review full report at Codecov.
|
@shyamd this is ready to merge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some small changes. Do you need these large data dump file for testing? This shouldn't be a place where you store data as that makes the repo unwieldy.
updated prec updated prec updated prec
I think following the same cases/pattern as pymatgen here makes life easier for future development. |
…syncio-0.11.0 Bump pytest-asyncio from 0.10.0 to 0.11.0
Bare branch for defining electrode document used in the InsertionElectrode builder.