Skip to content

Reimplement B14.py to fix issue #69(100% coverage w/ less vectors) an…#97

Open
sww0000 wants to merge 14 commits into
coreyqh:mainfrom
sww0000:B14
Open

Reimplement B14.py to fix issue #69(100% coverage w/ less vectors) an…#97
sww0000 wants to merge 14 commits into
coreyqh:mainfrom
sww0000:B14

Conversation

@sww0000
Copy link
Copy Markdown
Contributor

@sww0000 sww0000 commented Apr 19, 2026

Added B14.adoc + reimplemented B14.py

Comment thread src/cover_float/testgen/B14.py Outdated
sww0000 added 4 commits April 24, 2026 09:46
added @register_model("B14")
pre-commit issues fixed (hopefully)
@sww0000
Copy link
Copy Markdown
Contributor Author

sww0000 commented Apr 25, 2026

ready to be merged

@coreyqh
Copy link
Copy Markdown
Owner

coreyqh commented Apr 26, 2026

Please restore the commented out define statements in config.svh, then I will merge

@coreyqh
Copy link
Copy Markdown
Owner

coreyqh commented Apr 27, 2026

Now it looks like all lines are uncommented, which enables coverage for models that do not yet have covergroups.

Sorry that little things are blocking this, but we want to make sure that we do not accidentally break the flow for others.

I would reference the copy of config.svh in the main branch of the repo and only make a change to the line that is relevant for your covergroup.

Feel free to reach out if you have any questions :)

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.

3 participants