Eric Thompson

Results 39 comments of Eric Thompson

I like this idea, and I didn't even know that this was something that was possible. The challenge is that the internal modules are very interconnected, and it will require...

Scratch that. As a trial I re-organized the internal dependencies of `download` and `assemble`. It wasn't nearly as hard as I expected, and it makes for much better organized code...

The smaller list of dependencies that should be required to run `download` is: ``` python configobj ruamel.yaml>=0.17.16 pandas>=1.0 pytz libcomcat>=2.0.13 obspy>=1.2.1 matplotlib>=3.1.0 numpy>=1.21 scipy>=1.7 setuptools-scm>=6.3.2 ``` This results in a...

Well, it doesn't really matter because the dask import error is coming from the "assemble" module, which isn't getting called here and shouldn't be imported. So, this is an indication...

I think I finally understand how to do this in an effective way, which is to refactor as a [native namespace package](https://packaging.python.org/en/latest/guides/packaging-namespace-packages/#native-namespace-packages). We'll also get a lot of from doing...

The term "regression plot" doesn't actually make a lot of sense. I think this is a legacy term and we should avoid it. We are really just plotting the data...

Currently, the `min_usable_frequency`/`max_usable_frequency` parameters in motionfetecher are getting set based on `tr.getParameter('corner_frequencies')`, but if we don't do bandpass filtering, then these won't get set, so we should also implement min/max...

I don't know what "TT" means. @cbworden or @wald-usgs might know. I also just noticed that "PSA(03)" is a bit problematic. I assume you are getting this from the old...

Thanks @g-weatherill and sorry for not posting earlier. This will be very important for the ShakeMap team, and we plan to use this as a template to contribute more conditional...

Hi @g-weatherill and @mmpagani, I'm trying to work out how to call this class with the preferred method as indicated in @micheles's [response to an issue we had](https://github.com/gem/oq-engine/issues/9312) in which...