Station elevations should be clamped to DEM

uafgeotools / rtm

A Python package for locating infrasound sources using reverse time migration

https://uaf-rtm.readthedocs.io/

MIT License

38 stars 13 forks source link

Station elevations should be clamped to DEM #33

Closed davidfee5 closed 4 years ago

davidfee5 commented 4 years ago

For the travel time calculation in 3-D, the station elevations should be clamped to the DEM by default. Right now the DEM elevation may differ than the provided tr.stats.elevation, resulting in an incorrect Euclidean distance and travel-time calculation. This is the default for the FDTD calculation. Might want to provide an option for a floating (e.g. aerostat!) station that is not clamped to the ground.

liamtoney commented 4 years ago

As discussed, let's make clamped-to-DEM the default with no option for floating stations — can add this in later if needed.

davidfee5 commented 4 years ago

Fixed in #34