materialsproject / api

New API client for the Materials Project
https://materialsproject.github.io/api/
Other
105 stars 34 forks source link

Clarify top level get_entries method #848

Closed munrojm closed 9 months ago

munrojm commented 9 months ago

Clarifies the top level get_entries method to avoid confusion with what looks like multiple entries for a single material_id. Relevant issue: https://github.com/materialsproject/api/issues/841

codecov-commenter commented 9 months ago

Codecov Report

All modified lines are covered by tests :white_check_mark:

Comparison is base (ff7f643) 70.42% compared to head (965b56a) 70.42%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #848 +/- ## ======================================= Coverage 70.42% 70.42% ======================================= Files 51 51 Lines 2482 2482 ======================================= Hits 1748 1748 Misses 734 734 ``` | [Files](https://app.codecov.io/gh/materialsproject/api/pull/848?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=materialsproject) | Coverage Δ | | |---|---|---| | [mp\_api/client/mprester.py](https://app.codecov.io/gh/materialsproject/api/pull/848?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=materialsproject#diff-bXBfYXBpL2NsaWVudC9tcHJlc3Rlci5weQ==) | `65.81% <100.00%> (ø)` | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.