-
Notifications
You must be signed in to change notification settings - Fork 10
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
HDF5 Files are missing xml info #141
Comments
Alright, after inspection, I see that this is explicitly set to PyGLImER/src/pyglimer/waveform/download.py Line 912 in 038fea9
|
As a quick fix, I made a script:
I have a feeling that the inventory however is supposed to be written earlier, will continue to investigate. |
I believe I have found the issue. And it's simple not taking into account all stations at the end of the event loop. When PyGLImER/src/pyglimer/waveform/download.py Line 912 in 038fea9
It does removes a large chunk of PyGLImER/src/pyglimer/waveform/download.py Line 923 in 038fea9
is called only a fraction of the available stations have actually I think a fix would be to simple run statxml_to_hdf5(rawloc, statloc) after Please check this @PeterMakus, if you agree I'll push a fix, then merge, then I should be CCP stacking. |
Hi @lsawade , I guess we could do that. I am wondering though whether it will fix the problem. As I see it, all response information is downloaded and added to the hdf5 files here: PyGLImER/src/pyglimer/waveform/download.py Line 923 in 038fea9
after executing PyGLImER/src/pyglimer/waveform/download.py Line 920 in 038fea9
The reason for the latter line was to make sure that really all response information is added and not just a subset. For example, if station XY was active from 1970-2010, but the sensor was changed in 1990, it might download information for only one sensor if we only requested data from e.g. 1995 onwards. Have you tried changing this line and checked whether it actually fixes the problem? |
Hi, The full inventory is definitely downloaded and all the stationxml's are definitely there, but not added to the HDF5 files because this
problem remains. Because Question. Do you always download the entire station.xml, no matter what? |
Yes, the entire station.xml is always downloaded. I guess the savest bet would be to alter |
Ok, |
Testing! |
Avoid duplicates
Bug Summary
Frustratingly enough, there seems to have been an issue when downloading all the data that the response files weren't stored in the HDF5 files. Approximately a 1/4 of files have responses making the error absurd. I'm also thinking about a quick fix, because xml's have been downloaded to the station directory just not been added to the the HDF5 file grrr.
Code to Reproduce
Error Traceback
PyGLImER Version?
Operating System?
linux
Python Version?
3.10.6
Installation Method?
developer installation / from source / git checkout
The text was updated successfully, but these errors were encountered: