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

Amda refac and some global refac #17

Merged
merged 38 commits into from
Oct 7, 2021
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
38 commits
Select commit Hold shift + click to select a range
9bd9655
Introduced amda.TimetableIndex to replace xmlid a string
jeandet Aug 4, 2021
4c9d893
AMDA: some cleanups plus introduced DatasetIndex (same idea than Time…
jeandet Aug 5, 2021
cb41689
AMDA: same with CatalogIndex and ParameterIndex
jeandet Aug 5, 2021
308fabc
CDA: Reduce a bit pool size for // tests, this seems to block GH Actions
jeandet Aug 6, 2021
0f295e9
AMDA: Moved config entries to config module and removed SAOP support
jeandet Aug 6, 2021
650f775
AMDA: Now REST client uses http module to ensure useragent is set
jeandet Aug 6, 2021
7103d97
ADMA: some tests cleanups
jeandet Aug 6, 2021
80a69b4
AMDA: removed dataty_constructor param
jeandet Aug 6, 2021
cbd7eb6
Moved 429 and 523 HTTP status codes handling to http module
jeandet Aug 6, 2021
c41b381
New common TimeTable class which fits AMDA needs and can be used with…
jeandet Aug 8, 2021
12a2173
AMDA: Introduced new Catalog class mostly like TimeTable
jeandet Aug 9, 2021
7706a86
AMDA: since get_token is a property token is a better name
jeandet Aug 9, 2021
38400fe
AMDA: REST client refac
jeandet Aug 9, 2021
a64ccc4
AMDA: rest fix error, thx CI
jeandet Aug 9, 2021
ab932ee
AMDA: Rest module refac, indirect request basically uses regular requ…
jeandet Aug 10, 2021
423b0b5
AMDA: Fix LGTM legit warning
jeandet Aug 10, 2021
5a4975c
Try to add pytest for tox
jeandet Aug 10, 2021
22b47d7
update GH actions
jeandet Aug 10, 2021
140b441
CONFIG: Adds ability to set entries from ENV + adds tests
jeandet Aug 10, 2021
20f3cf7
AMDA: adds tests with authentication
jeandet Aug 10, 2021
81ebbbf
Forces tox to forward amda creds from env
jeandet Aug 10, 2021
35bc3f4
Second try to pass AMDA creds on GH Actions
jeandet Aug 10, 2021
58fddcb
try fix AMDA creds on GH actions and typo in http module
jeandet Aug 11, 2021
7e9507d
AMDA: uses new dataset type, skips tests when no credentials are found
jeandet Aug 12, 2021
4ab139c
AMDA: improves naming and implementation consistency
jeandet Aug 12, 2021
4e0b667
LGTM: try to mute false positives
jeandet Aug 12, 2021
79749fc
AMDA: forgot to make get_catalog compatible with CatalogIndex
jeandet Aug 12, 2021
2067e3c
AMDA: Few small refac
jeandet Aug 13, 2021
86d746b
Fix small mistake on previous variable merge function refac
jeandet Aug 15, 2021
80bfdbd
Whole API rework and reorganization + doc WIP
jeandet Aug 22, 2021
7e274c9
New user doc WIP plus some cleanups
jeandet Oct 3, 2021
2d7b6a1
[ssc] Fix regression with python 3.6
jeandet Oct 7, 2021
4527178
Moved AMDA top module from _amda to ws to make it public for API doc
jeandet Oct 7, 2021
637ce68
[GH Actions] adds numpydoc
jeandet Oct 7, 2021
8ba0eea
[Doc] some cleanups, still a lot to do
jeandet Oct 7, 2021
c2d5e21
[lgtm] fixes few actual errors and mute some false positives
jeandet Oct 7, 2021
43c9864
[GH Actions][lgtm] update codecov action and fix legit issue
jeandet Oct 7, 2021
6754c81
[Doc] makes REAMDE point to stable documentation
jeandet Oct 7, 2021
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[GH Actions][lgtm] update codecov action and fix legit issue
Signed-off-by: Alexis Jeandet <alexis.jeandet@member.fsf.org>
  • Loading branch information
jeandet committed Oct 7, 2021
commit 43c9864dffcad36762774627c532abf582b0c559
3 changes: 1 addition & 2 deletions .github/workflows/PRs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,11 +39,10 @@ jobs:
make doctest
- name: Upload coverage to Codecov
if: matrix.python-version == '3.8'
uses: codecov/codecov-action@v1
uses: codecov/codecov-action@v2
with:
token: ${{ secrets.CODECOV_TOKEN }}
file: ./coverage.xml
flags: unittests
name: codecov-umbrella
yml: ./codecov.yml
fail_ci_if_error: true
1 change: 0 additions & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,5 +45,4 @@ jobs:
file: ./coverage.xml
flags: unittests
name: codecov-umbrella
yml: ./codecov.yml
fail_ci_if_error: true
2 changes: 1 addition & 1 deletion speasy/webservices/amda/ws.py
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ def get_data(self, product, start_time=None, stop_time=None, **kwargs) -> Option
return self.get_dataset(dataset_id=product, start=start_time, stop=stop_time, **kwargs)
if product_t == ProductType.PARAMETER and start_time and stop_time:
if is_user_parameter(product):
return self.get_user_parameter(product=product, start_time=start_time, stop_time=stop_time, **kwargs)
return self.get_user_parameter(parameter_id=product, start_time=start_time, stop_time=stop_time)
else:
return self.get_parameter(product=product, start_time=start_time, stop_time=stop_time, **kwargs)
if product_t == ProductType.CATALOG:
Expand Down