ledatelescope / bifrost

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

Ring memory logging #100

Closed jaycedowell closed 6 years ago

jaycedowell commented 7 years ago

Added support for logging of ring metadata (space, alignment, size, etc.) and updated several of the tools to use the new information. This includes:

jaycedowell commented 7 years ago

I'm not sure where the travis errors are coming from. Do they look familiar to anyone?

benbarsdell commented 7 years ago

The ctypesgen step failed (visible in the hidden build log), but it doesn't stop the build (I'll try to fix this soon).

Can't include string in memory.h (it has to be pure C).

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.1%) to 65.725% when pulling 4a67c8127b883b6cbe91ca624dfad1413a36f083 on jaycedowell:ring-info into 84891770f905b786d8c302695ad4928e80ee7a3a on ledatelescope:master.

codecov-io commented 7 years ago

Codecov Report

Merging #100 into master will decrease coverage by 1.28%. The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #100      +/-   ##
==========================================
- Coverage   67.17%   65.88%   -1.29%     
==========================================
  Files          39       41       +2     
  Lines        3555     3834     +279     
==========================================
+ Hits         2388     2526     +138     
- Misses       1167     1308     +141
Impacted Files Coverage Δ
python/bifrost/proclog.py 27.27% <ø> (ø) :arrow_up:
python/bifrost/ring2.py 86.66% <64.28%> (-1.8%) :arrow_down:
python/bifrost/ring.py 90.82% <68.75%> (-1.93%) :arrow_down:
python/bifrost/libbifrost.py 55.07% <0%> (-1.65%) :arrow_down:
python/bifrost/DataType.py 63.2% <0%> (-1.45%) :arrow_down:
python/bifrost/blocks/serialize.py 89.28% <0%> (-0.88%) :arrow_down:
python/bifrost/sigproc2.py 54.02% <0%> (-0.52%) :arrow_down:
python/bifrost/block.py 89.78% <0%> (-0.47%) :arrow_down:
python/bifrost/ndarray.py 77.23% <0%> (-0.25%) :arrow_down:
... and 5 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 8489177...81427a0. Read the comment docs.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.1%) to 65.725% when pulling 2d2ef06a960c7b0ef32893c08d2733c600ed6853 on jaycedowell:ring-info into 84891770f905b786d8c302695ad4928e80ee7a3a on ledatelescope:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.1%) to 65.725% when pulling 2d2ef06a960c7b0ef32893c08d2733c600ed6853 on jaycedowell:ring-info into 84891770f905b786d8c302695ad4928e80ee7a3a on ledatelescope:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.3%) to 65.941% when pulling e2da49c6f641675b3b00cc0b4414cda5be7f5eed on jaycedowell:ring-info into 84891770f905b786d8c302695ad4928e80ee7a3a on ledatelescope:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.3%) to 65.941% when pulling e2da49c6f641675b3b00cc0b4414cda5be7f5eed on jaycedowell:ring-info into 84891770f905b786d8c302695ad4928e80ee7a3a on ledatelescope:master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-3.4%) to 62.195% when pulling 81427a08ce501a0f4b563d85ac45f21af9f99015 on jaycedowell:ring-info into 84891770f905b786d8c302695ad4928e80ee7a3a on ledatelescope:master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-3.4%) to 62.195% when pulling 81427a08ce501a0f4b563d85ac45f21af9f99015 on jaycedowell:ring-info into 84891770f905b786d8c302695ad4928e80ee7a3a on ledatelescope:master.