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

Suggestion for rechdf5 output: add actual recorded location and distance to target #132

Closed
jf-huang opened this issue Aug 12, 2022 · 8 comments
Assignees

Comments

@jf-huang
Copy link
Contributor

jf-huang commented Aug 12, 2022

When SW4 outputs the receiver time histories in the USGS ASCII file format, the actual location of the receiver and the distance to the target are recorded, an example is shown below:

Station: D_01_01
Target location (WGS84 longitude, latitude) (deg): -1.223651e+02 3.769469e+01
Actual location (WGS84 longitude, latitude) (deg): -1.223652e+02 3.769462e+01
Distance from target to actual location (m): 1.131371e+01

However, in the rechdf5 file, only the target location is available which may be far away from the actual location depending on the grid size. Would be great if the two pieces of information can be added back as two additional datasets or as attributes of the station. Actual X/Y/Z coordinates or longitude/latitude/depth can be stored based on the isnsew option.

@houjun houjun self-assigned this Aug 17, 2022
@houjun
Copy link
Collaborator

houjun commented Aug 24, 2022

@jf-huang Can you check the tang/sachdf5new branch and run your test case?
Each station group will have new "ACTUALSTLA,STLO,STDP" and "DISTFROMACTUAL" datasets, which correspond to the USGS actual location and distance. The original "STLA,STLO,STDP" remains the target location.

@jf-huang
Copy link
Contributor Author

@houjun Thanks for the quick response! I have run a test case and the new branch gives the same info as the USGS format.

Besides, I was wondering if it's advisable to store the actual X/Y/Z coordinates as well when isnsew is set to 0 (false) in view that the X/Y/Z coordinates are explicitly requested by the user?

@houjun
Copy link
Collaborator

houjun commented Aug 25, 2022 via email

@jf-huang
Copy link
Contributor Author

Sorry I wasn't clear, I meant the actual X/Y/Z (i.e., mGPX/mGPY/mGPZ in TimeSeries.C). I am asking because the geographic coordinates are not easy to interpret sometimes and actual X/Y/Z may be easier in hypothetical runs.

@houjun
Copy link
Collaborator

houjun commented Aug 25, 2022

@jf-huang I just pushed an update to add "ACTUALSTX,STY,STZ" to each station group, can you try it out?

@jf-huang
Copy link
Contributor Author

@houjun It works, thanks!

@houjun
Copy link
Collaborator

houjun commented Aug 31, 2022

Updated code merged to developer

@houjun houjun closed this as completed Aug 31, 2022
@jf-huang
Copy link
Contributor Author

jf-huang commented Oct 11, 2022 via email

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

2 participants