Skip to content

add timestamp640, pulse_gen640#144

Open
thirono wants to merge 12 commits intomasterfrom
timestamp640
Open

add timestamp640, pulse_gen640#144
thirono wants to merge 12 commits intomasterfrom
timestamp640

Conversation

@thirono
Copy link
Copy Markdown
Collaborator

@thirono thirono commented Dec 3, 2020

add 2 modules

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 3, 2020

Codecov Report

❌ Patch coverage is 79.16667% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 32.39%. Comparing base (af5e949) to head (07576f8).
⚠️ Report is 249 commits behind head on master.

Files with missing lines Patch % Lines
basil/HL/pulse_gen640.py 73.68% 10 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #144      +/-   ##
==========================================
+ Coverage   31.92%   32.39%   +0.46%     
==========================================
  Files          72       74       +2     
  Lines        4780     4828      +48     
==========================================
+ Hits         1526     1564      +38     
- Misses       3254     3264      +10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@themperek
Copy link
Copy Markdown
Member

There should be files like test_SimPulseGen640.v according to documentation?

@thirono
Copy link
Copy Markdown
Collaborator Author

thirono commented Dec 3, 2020

pulse_gen640 cannot be simulated... it includes an interface from xilinx.

@themperek
Copy link
Copy Markdown
Member

Now it is available: https://github.com/Xilinx/XilinxUnisimLibrary ?

@thirono
Copy link
Copy Markdown
Collaborator Author

thirono commented Dec 11, 2020

XilinxUnisimLibrary requires SecureIP which is bundle with vivado

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants