search

Xilinx / PYNQ

Python Productivity for ZYNQ
http://www.pynq.io/
BSD 3-Clause "New" or "Revised" License
1.99k stars 818 forks source link

Fix bug, return `XrtDevice` only for x86 machines #1337

Closed mariodruiz closed 2 years ago

mariodruiz commented 2 years ago

Check for x86 architecture to create XrtDevice, this avoids creating two devices for edge platforms

Fix Xilinx/PYNQ-Dev#356

codecov-commenter commented 2 years ago

Codecov Report

Merging #1337 (733b2ff) into image_v2.8 (b85f453) will increase coverage by 0.02%. The diff coverage is 80.00%.

Impacted file tree graph

@@              Coverage Diff               @@
##           image_v2.8    #1337      +/-   ##
==============================================
+ Coverage       46.47%   46.49%   +0.02%     
==============================================
  Files              28       28              
  Lines            4953     4955       +2     
  Branches          971      971              
==============================================
+ Hits             2302     2304       +2     
  Misses           2594     2594              
  Partials           57       57
Impacted Files Coverage Δ
pynq/pl_server/xrt_device.py 29.36% <66.66%> (+0.18%) :arrow_up:
pynq/ps.py 100.00% <100.00%> (ø)

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 b85f453...733b2ff. Read the comment docs.