search

HERA-Team / matvis

Fast matrix-based visibility simulator
MIT License
2 stars 3 forks source link

Make IPython import optional #13

Closed philbull closed 3 years ago

philbull commented 3 years ago

Fix IPython import to be optional (i.e. only needed if animation function is used).

codecov[bot] commented 3 years ago

Codecov Report

Merging #13 (c6e5b1c) into main (5c537f0) will increase coverage by 5.17%. The diff coverage is 76.19%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
+ Coverage   43.37%   48.54%   +5.17%     
==========================================
  Files           6        6              
  Lines         332      344      +12     
  Branches       48       52       +4     
==========================================
+ Hits          144      167      +23     
+ Misses        175      165      -10     
+ Partials       13       12       -1
Impacted Files Coverage Δ
src/vis_cpu/plot.py 13.51% <0.00%> (-5.41%) :arrow_down:
src/vis_cpu/wrapper.py 72.41% <16.66%> (-0.67%) :arrow_down:
src/vis_cpu/vis_cpu.py 82.14% <86.36%> (+1.62%) :arrow_up:
src/vis_cpu/conversions.py 80.70% <100.00%> (+26.15%) :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 873d68a...c6e5b1c. Read the comment docs.