search

paudetseis / Telewavesim

Teleseismic body wave modeling through stacks of (submarine/anisotropic) layers
https://paudetseis.github.io/Telewavesim/
MIT License
66 stars 18 forks source link

Remove pyfftw dependency #8

Closed trichter closed 5 years ago

trichter commented 5 years ago

This PR would remove the dependency on pyfftw and replaces its function by the corresponding numpy functions. The test cases run as fast as before. If the user wants to use pyfftw for a possible speedup he can use this code snippets before importing telewavesim:

import numpy
import pyfftw
numpy.fft = pyfftw.interfaces.numpy_fft
import telewavesim
codecov-io commented 5 years ago

Codecov Report

Merging #8 into master will decrease coverage by 0.19%. The diff coverage is 56.66%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master       #8     +/-   ##
=========================================
- Coverage   56.16%   55.96%   -0.2%     
=========================================
  Files           9        9             
  Lines        1136     1131      -5     
=========================================
- Hits          638      633      -5     
  Misses        498      498
Impacted Files Coverage Δ
telewavesim/__init__.py 100% <ø> (ø) :arrow_up:
telewavesim/tests/test_0_imports.py 100% <ø> (ø) :arrow_up:
telewavesim/tests/test_routines.py 100% <100%> (ø) :arrow_up:
telewavesim/tests/test_notebooks.py 100% <100%> (ø) :arrow_up:
telewavesim/utils.py 71.09% <40.9%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update b8298a9...f0b537a. Read the comment docs.