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

Convert diagonal clock constraints in simulation when possible #123

Open
seblund opened this issue Dec 16, 2022 · 1 comment · Fixed by #82
Open

Convert diagonal clock constraints in simulation when possible #123

seblund opened this issue Dec 16, 2022 · 1 comment · Fixed by #82
Labels
simulation Functionality is related to the simulation part of the system

Comments

@seblund
Copy link
Member

seblund commented Dec 16, 2022

Here is how clock constraints are shown in the simulation tool now (SW5):
208062192-515749d9-1977-46d9-99a1-dab8a6a11f9c~2
We can and should convert expression with cycles like this to x=y=z, to avoid as many diagonal constraints (x-y<=c) as possible because they are generally confusing.

We should also convert 0-x<=c to -c<=x (if that is not already done)

@Nielswps Nielswps added the simulation Functionality is related to the simulation part of the system label Feb 24, 2023
@Nielswps Nielswps linked a pull request Jun 28, 2023 that will close this issue
@Nielswps
Copy link
Contributor

This has been implemented on the simulator branch but does not account for circular connections

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
simulation Functionality is related to the simulation part of the system
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants