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

group vel tomo input file names #4

Open
dylanmikesell opened this issue Jun 11, 2020 · 0 comments
Open

group vel tomo input file names #4

dylanmikesell opened this issue Jun 11, 2020 · 0 comments

Comments

@dylanmikesell
Copy link

There is a going to be problems with folks trying to use the tomo plugin for high frequency data. For example, when running msnoise p tomo prepare_tomo, the following file is written:
TestGroupVel_0.1sGLISN.dat.

(First, the word GLISN is in there for some reason. I guess this a hold over from Aurelien's original version. I don't see a need for that. The files are likely already in a project folder.)

Second, and more important, the files are named based on period with only one decimal. I am trying to do a tomography from 10 to 30 Hz, with the following periods:
0.033,0.034,0.036,0.037,0.038,0.040,0.042,0.043,0.045,0.048,0.050,0.053,0.056,0.059,0.062,0.067,0.071,0.077,0.083,0.091,0.100,0.111,0.125,0.143,0.167,0.200

When I run the prepare_tomo command, I get three files:

TestGroupVel_0.0sGLISN.dat
TestGroupVel_0.1sGLISN.dat  
TestGroupVel_0.2sGLISN.dat

Each one related to whichever was the last file written in this period band.

0.091
0.167
0.200

Because of the 1 decimal naming convention, the files just get overwritten. It would be very useful to have at least 3 decimals in the file names if we are going to use the period in the file name.

dylanmikesell added a commit to dylanmikesell/msnoise-tomo that referenced this issue Jun 16, 2020
This addresses issue ThomasLecocq#4. We need more decimal places when going to periods below 1s. This takes the code from 1 decimal to 3 decimal places.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant