ledatelescope / bifrost

A stream processing framework for high-throughput applications.
BSD 3-Clause "New" or "Revised" License
64 stars 29 forks source link

Tools python 3 compatibility changes #151

Closed mkolopanis closed 3 years ago

mkolopanis commented 3 years ago

A few minor tweaks to the tools to make them work in python 3.

coveralls commented 3 years ago

Coverage Status

Coverage remained the same at 61.701% when pulling ada863ef83d59125f29abe6561f0cba6e5f2a53a on epic-astronomy:tools_py3_update into e1cea0b71a969f2fa73b8bc0b18a48e8e88b2059 on ledatelescope:master.

codecov-commenter commented 3 years ago

Codecov Report

Merging #151 (ccdb6f2) into master (a53b35b) will decrease coverage by 2.86%. The diff coverage is n/a.

:exclamation: Current head ccdb6f2 differs from pull request most recent head ada863e. Consider uploading reports for the commit ada863e to get more accurate results Impacted file tree graph

@@            Coverage Diff             @@
##           master     #151      +/-   ##
==========================================
- Coverage   61.62%   58.75%   -2.87%     
==========================================
  Files          64       64              
  Lines        5300     5300              
==========================================
- Hits         3266     3114     -152     
- Misses       2034     2186     +152     
Impacted Files Coverage Δ
python/bifrost/fir.py 0.00% <0.00%> (-100.00%) :arrow_down:
python/bifrost/romein.py 0.00% <0.00%> (-100.00%) :arrow_down:
python/bifrost/blocks/scrunch.py 37.03% <0.00%> (-59.26%) :arrow_down:
python/bifrost/map.py 20.83% <0.00%> (-56.25%) :arrow_down:
python/bifrost/linalg.py 36.84% <0.00%> (-52.64%) :arrow_down:
python/bifrost/fft.py 45.00% <0.00%> (-50.00%) :arrow_down:
python/bifrost/fdmt.py 50.00% <0.00%> (-50.00%) :arrow_down:
python/bifrost/transpose.py 41.66% <0.00%> (-50.00%) :arrow_down:
python/bifrost/reduce.py 55.55% <0.00%> (-33.34%) :arrow_down:
python/bifrost/libbifrost.py 52.70% <0.00%> (-25.68%) :arrow_down:
... and 2 more

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 a53b35b...ada863e. Read the comment docs.