SciQLop / speasy

Space Physics made EASY! A simple Python package to deal with main Space Physics WebServices (CDA,SSC,AMDA,..)
Other
24 stars 7 forks source link

Inventory update optimization #53

Closed jeandet closed 1 year ago

jeandet commented 1 year ago

allows to update inventory only if it has changed and also decouples SpeasyIndexes and flat inventories.

codecov[bot] commented 1 year ago

Codecov Report

Base: 85.86% // Head: 85.54% // Decreases project coverage by -0.31% :warning:

Coverage data is based on head (b2f933f) compared to base (a01a8ea). Patch coverage: 91.56% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #53 +/- ## ========================================== - Coverage 85.86% 85.54% -0.32% ========================================== Files 44 44 Lines 2518 2553 +35 Branches 374 386 +12 ========================================== + Hits 2162 2184 +22 - Misses 236 246 +10 - Partials 120 123 +3 ``` | Flag | Coverage Δ | | |---|---|---| | unittests | `84.80% <91.56%> (-1.06%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#carryforward-flags-in-the-pull-request-comment) to find out more. | [Impacted Files](https://codecov.io/gh/SciQLop/speasy/pull/53?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop) | Coverage Δ | | |---|---|---| | [speasy/core/proxy/\_\_init\_\_.py](https://codecov.io/gh/SciQLop/speasy/pull/53/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L2NvcmUvcHJveHkvX19pbml0X18ucHk=) | `82.41% <73.33%> (-2.77%)` | :arrow_down: | | [speasy/core/inventory/indexes.py](https://codecov.io/gh/SciQLop/speasy/pull/53/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L2NvcmUvaW52ZW50b3J5L2luZGV4ZXMucHk=) | `85.29% <78.57%> (-1.17%)` | :arrow_down: | | [speasy/core/dataprovider.py](https://codecov.io/gh/SciQLop/speasy/pull/53/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L2NvcmUvZGF0YXByb3ZpZGVyLnB5) | `89.47% <100.00%> (+0.90%)` | :arrow_up: | | [speasy/core/inventory/\_\_init\_\_.py](https://codecov.io/gh/SciQLop/speasy/pull/53/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L2NvcmUvaW52ZW50b3J5L19faW5pdF9fLnB5) | `100.00% <100.00%> (ø)` | | | [...asy/webservices/cda/\_inventory\_builder/\_\_init\_\_.py](https://codecov.io/gh/SciQLop/speasy/pull/53/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L3dlYnNlcnZpY2VzL2NkYS9faW52ZW50b3J5X2J1aWxkZXIvX19pbml0X18ucHk=) | `81.48% <100.00%> (-14.82%)` | :arrow_down: | | [...ces/cda/\_inventory\_builder/\_xml\_catalogs\_parser.py](https://codecov.io/gh/SciQLop/speasy/pull/53/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L3dlYnNlcnZpY2VzL2NkYS9faW52ZW50b3J5X2J1aWxkZXIvX3htbF9jYXRhbG9nc19wYXJzZXIucHk=) | `86.79% <100.00%> (+3.77%)` | :arrow_up: | | [speasy/webservices/csa/\_\_init\_\_.py](https://codecov.io/gh/SciQLop/speasy/pull/53/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L3dlYnNlcnZpY2VzL2NzYS9fX2luaXRfXy5weQ==) | `84.80% <100.00%> (+0.37%)` | :arrow_up: | | [speasy/core/index/speasy\_index.py](https://codecov.io/gh/SciQLop/speasy/pull/53/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L2NvcmUvaW5kZXgvc3BlYXN5X2luZGV4LnB5) | `90.90% <0.00%> (-9.10%)` | :arrow_down: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert when merging ccf766f99076ddb5c20f3a1d5dd6de3472bcd10f into a01a8ea41deac9e6212c09f90278d02da7c061eb - view on LGTM.com

new alerts: