PYNQ icon indicating copy to clipboard operation
PYNQ copied to clipboard

Assign partial dictionary directly.

Open mariodruiz opened this issue 3 years ago • 1 comments

  • Assign partial dictionary directly.
  • There is a bug in the way phys_addr was treated. The partial HWH provides the absolute offset address not the relative offset address in respect to the partial region apertures

mariodruiz avatar Mar 21 '22 19:03 mariodruiz

Codecov Report

Merging #1352 (3cf64c8) into image_v2.8 (0cdf6b9) will increase coverage by 0.12%. The diff coverage is 0.00%.

@@              Coverage Diff               @@
##           image_v2.8    #1352      +/-   ##
==============================================
+ Coverage       46.49%   46.62%   +0.12%     
==============================================
  Files              28       28              
  Lines            4955     4942      -13     
  Branches          971      970       -1     
==============================================
  Hits             2304     2304              
+ Misses           2594     2581      -13     
  Partials           57       57              
Impacted Files Coverage Δ
pynq/pl_server/server.py 49.02% <0.00%> (+2.36%) :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 0cdf6b9...3cf64c8. Read the comment docs.

codecov-commenter avatar May 12 '22 13:05 codecov-commenter

This will be incorporated somewhere else

mariodruiz avatar Sep 14 '22 10:09 mariodruiz