Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check _Next for more than 1 clock #576

Closed
cjchin opened this issue Jun 2, 2017 · 4 comments
Closed

Check _Next for more than 1 clock #576

cjchin opened this issue Jun 2, 2017 · 4 comments
Labels

Comments

@cjchin
Copy link
Contributor

cjchin commented Jun 2, 2017

I think it is missing a section to describe that if Start_Event is asserted for X cycles, that it will also check that expression will also need to be asserted for X cycles.

And to maybe to put a decription where the description of allowing for overlaps

image

What do you think?

@cjchin
Copy link
Contributor Author

cjchin commented Jun 2, 2017

check_next_passing_with_overlap

@cjchin
Copy link
Contributor Author

cjchin commented Jun 2, 2017

In addition, I believe it will be better to illustrate the meaning of allow_missing_start. such as having an example like:
check_next_passing_with_overlap

@LarsAsplund
Copy link
Collaborator

I think the continuously asserted start event has been described already

"check_next passes if expr = '1' num_cks active (according to active_clock_edge) and enabled (en = '1') clock edges after a start event. The start event is defined by an active and enabled clock edge for which start_event = '1'."

So if you have many start events in a row there will be many clock cycles with expr = '1' in a row.

I can add your figure for allow_missing_start. Can you provide the .json?

@cjchin
Copy link
Contributor Author

cjchin commented Jun 4, 2017

Ok I will make a pull request for this item.

Do you mind if i also change "check_next passes if expr = '1' num_cks active..." to "check_next passes if expr = '1' for the num_cks active..."?

@eine eine transferred this issue from VUnit/VUnit.github.io Oct 23, 2019
@eine eine added the Docs label Oct 23, 2019
dalex78 added a commit to dalex78/vunit that referenced this issue May 9, 2022
…ure (see VUnit#576). Indicates in the timing diagram when the options 'allow_overlapping' and 'allow_missing_start' add no influence regarding the given example.
LarsAsplund pushed a commit that referenced this issue May 9, 2022
…ure (see #576). Indicates in the timing diagram when the options 'allow_overlapping' and 'allow_missing_start' add no influence regarding the given example.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants