search

gwnrtools / nr-catalog-tools

A unified interface to various catalogs of Numerical Relativity simulations of compact binary mergers.
https://github.com/gwnrtools/nr-catalog-tools
GNU General Public License v3.0
1 stars 5 forks source link

Optimize `get_td_waveform` #34

Closed vaishakp closed 1 year ago

vaishakp commented 1 year ago

Interpolate only once, after evaluation instead of interpolating all modes before evaluation.

To be merged after #32, #35 and #33.

prayush commented 1 year ago

@vaishakp Is there a reason this should wait for #32, #33 and #35 (this seems ready to go now)?

vaishakp commented 1 year ago

@vaishakp Is there a reason this should wait for #32, #33 and #35 (this seems ready to go now)?

Actually no. This can be merged.