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 rework + last refac/fixes for 1.0 release candidate #49

Closed jeandet closed 1 year ago

jeandet commented 2 years ago

This PR adds the lasts few bits for 1.0 release:

codecov[bot] commented 2 years ago

Codecov Report

Base: 87.56% // Head: 78.56% // Decreases project coverage by -8.99% :warning:

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

:exclamation: Current head ce7e973 differs from pull request most recent head d9d61fd. Consider uploading reports for the commit d9d61fd to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #49 +/- ## ========================================== - Coverage 87.56% 78.56% -9.00% ========================================== Files 31 44 +13 Lines 1536 2510 +974 Branches 261 417 +156 ========================================== + Hits 1345 1972 +627 - Misses 128 423 +295 - Partials 63 115 +52 ``` | Flag | Coverage Δ | | |---|---|---| | unittests | `77.45% <73.59%> (-10.12%)` | :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/49?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop) | Coverage Δ | | |---|---|---| | [speasy/core/index/\_\_init\_\_.py](https://codecov.io/gh/SciQLop/speasy/pull/49/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L2NvcmUvaW5kZXgvX19pbml0X18ucHk=) | `0.00% <0.00%> (ø)` | | | [speasy/core/index/speasy\_index.py](https://codecov.io/gh/SciQLop/speasy/pull/49/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L2NvcmUvaW5kZXgvc3BlYXN5X2luZGV4LnB5) | `0.00% <0.00%> (ø)` | | | [...asy/webservices/cda/\_inventory\_builder/\_\_init\_\_.py](https://codecov.io/gh/SciQLop/speasy/pull/49/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L3dlYnNlcnZpY2VzL2NkYS9faW52ZW50b3J5X2J1aWxkZXIvX19pbml0X18ucHk=) | `0.00% <0.00%> (ø)` | | | [...ces/cda/\_inventory\_builder/\_xml\_catalogs\_parser.py](https://codecov.io/gh/SciQLop/speasy/pull/49/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L3dlYnNlcnZpY2VzL2NkYS9faW52ZW50b3J5X2J1aWxkZXIvX3htbF9jYXRhbG9nc19wYXJzZXIucHk=) | `0.00% <0.00%> (ø)` | | | [speasy/plotting/mpl\_backend/\_\_init\_\_.py](https://codecov.io/gh/SciQLop/speasy/pull/49/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L3Bsb3R0aW5nL21wbF9iYWNrZW5kL19faW5pdF9fLnB5) | `50.00% <50.00%> (ø)` | | | [speasy/webservices/csa/\_\_init\_\_.py](https://codecov.io/gh/SciQLop/speasy/pull/49/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L3dlYnNlcnZpY2VzL2NzYS9fX2luaXRfXy5weQ==) | `54.09% <54.09%> (ø)` | | | [speasy/core/cache/cache.py](https://codecov.io/gh/SciQLop/speasy/pull/49/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L2NvcmUvY2FjaGUvY2FjaGUucHk=) | `80.64% <62.50%> (-11.52%)` | :arrow_down: | | [speasy/plotting/\_\_init\_\_.py](https://codecov.io/gh/SciQLop/speasy/pull/49/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L3Bsb3R0aW5nL19faW5pdF9fLnB5) | `68.75% <68.75%> (ø)` | | | [speasy/webservices/ssc/\_\_init\_\_.py](https://codecov.io/gh/SciQLop/speasy/pull/49/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L3dlYnNlcnZpY2VzL3NzYy9fX2luaXRfXy5weQ==) | `81.01% <68.75%> (-4.71%)` | :arrow_down: | | [speasy/webservices/amda/inventory.py](https://codecov.io/gh/SciQLop/speasy/pull/49/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop#diff-c3BlYXN5L3dlYnNlcnZpY2VzL2FtZGEvaW52ZW50b3J5LnB5) | `80.18% <70.58%> (-16.48%)` | :arrow_down: | | ... and [39 more](https://codecov.io/gh/SciQLop/speasy/pull/49/diff?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=SciQLop) | | 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 2 years ago

This pull request introduces 5 alerts and fixes 6 when merging 97895cf3ed70de76f7b6cea6bedd27d5233d465d into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 5 alerts and fixes 6 when merging 34e48d4aff14a969a2d56a799516f6a88586c587 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 5 alerts and fixes 6 when merging 7eaf05b6fed5f006edcd91615ba2f35169702f88 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 6 alerts and fixes 6 when merging 2ba58c539b4ac4cd17c1c3d5ae3d4fadf2a8aeae into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 6 alerts and fixes 6 when merging ecc8a12ac6d5711afc480dbfd31c4db0997c5beb into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 9 alerts and fixes 6 when merging 62e14818fe73736bebebdd376c8eecfe6c8045b3 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 6 alerts and fixes 6 when merging db42a7802bd5fd988c79f25e04c62197e3313b8f into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 6 alerts and fixes 6 when merging b112302de10d5824e1df0449b110d36cf54133bc into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 6 alerts and fixes 6 when merging ae1e8214fbc6340a65150040f9bd2e3da4129f74 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 6 alerts and fixes 6 when merging c6311ce1a96db9090e15294b71dee1e9aa7d3810 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 8 alerts and fixes 6 when merging 962a35ff078a2b95fec632e4ef8e8db0c8ef6cf8 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 6 alerts and fixes 6 when merging af13dbc59fed718f8fc2518b466118f9da0deee0 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 6 alerts and fixes 6 when merging ab7a0196caa76566f220e98d4c7a3cac92fbd1ad into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 11 alerts and fixes 6 when merging 99b41ad53293dcf0138dbd5ad7459df59c501a16 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 13 alerts and fixes 6 when merging 1716bd29d6f4375bbbebed7d0fa66bf9dd2d523b into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 13 alerts and fixes 6 when merging f26c945fef1805ba37580effeca625976b203dee into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 13 alerts and fixes 6 when merging acce13c24d5b335c974700d2e374f1fc716a2e46 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 13 alerts and fixes 6 when merging b5703032011849fbbf85eae503119b1a4a3d679b into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 12 alerts and fixes 6 when merging ae7559edf6c0dab9c26da918a885d09b42ad62e7 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 12 alerts and fixes 6 when merging f969dcdb6faa0a073d6286a836a0776b2ca09c0c into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 12 alerts and fixes 6 when merging b8bc28b0e5ba3bb83b5e8b506e5543cb006165e9 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 13 alerts and fixes 6 when merging 8b939f9ac5f4e05258807f10ae122076d4422b3a into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 14 alerts and fixes 6 when merging 675c6f007246d6712d86423a974ed9493b2d8f98 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 14 alerts and fixes 6 when merging 3f6c8acbc9010045399bea68468f5d7358b3519c into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 2 alerts and fixes 6 when merging a9f70a2820a5f3660cfcae34b21e0e81d51748ac into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 6 when merging 6baec5a013e1fd564440d781c95c3af351283b41 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 6 when merging 4e141b18c5564b534c406c218ba7a5904a516350 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 6 when merging e3de3a5f323f0775468f2b977708f8ed7bc4fa0c into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 6 when merging ab67e9a9f7b5dff459aea572d7d19f22abff1931 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 6 when merging 252b625fa82e2f4537ee174668e242c1f17f1d7e into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 6 when merging e67a995db19ba5bf1a0f24e9e974e738b790d0e7 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 6 when merging a24f4c04d7d1955b5e0631e145e35539f543e916 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 6 when merging 0b7a7b3f3d3db2a04c39195c7bda7dcfba21a437 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 6 when merging b792028c074eb09bf9894d42bafba1748243739c into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 6 when merging 2c6eec82c4bfabfca21a79e27cac1cba98e8b15e into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 6 when merging 99f30ae682edfdc6a9ca288d09d68f8dcc3cbbfa into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 6 when merging ce7e973753016aad1dce0e7681762a3b3aa3bfe2 into b923e073022d2343da2c9bb0aec6335a99b5f132 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request introduces 1 alert and fixes 18 when merging d9d61fdde246ad3d0372b4dfb5b71476b7b3a711 into 0f9307a04ed1badd04e12f86819992be874c3a57 - view on LGTM.com

new alerts:

fixed alerts: