-
Notifications
You must be signed in to change notification settings - Fork 7
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] Uses CDF_ISTP as default #101
Conversation
Signed-off-by: Alexis Jeandet <alexis.jeandet@member.fsf.org>
@brenard-irap the CI issue looks legit, with CDF output format it looks like I get a single column name for a 2 columns variable: In [1]: from speasy import amda
In [2]: first_param_index=amda.list_parameters()[0]
In [3]: print(first_param_index)
<ParameterIndex: He flux>
In [4]: first_param=amda.get_parameter(first_param_index, "2018-01-01", "2018-01-02T01")
In [5]: first_param
Out[5]: <speasy.products.variable.SpeasyVariable at 0x7f894d4698c0>
In [6]: first_param.columns
Out[6]: ['He flux']
In [7]: first_param.values
Out[7]:
array([[0.12428 , 0. ],
... Looks like an issue on ADMA side? |
For a spectrogram, we don't need to define a label for each data component... I think you have the same problem with this CDAWeb parameter
|
@brenard-irap my bad I overlooked, since there was only two columns I thought that is was a simple timeserie. |
AMDA provides different column names between CDF and CSV format Signed-off-by: Alexis Jeandet <alexis.jeandet@member.fsf.org>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #101 +/- ##
==========================================
- Coverage 86.50% 86.43% -0.07%
==========================================
Files 51 51
Lines 3045 3045
Branches 490 490
==========================================
- Hits 2634 2632 -2
- Misses 275 276 +1
- Partials 136 137 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
No description provided.