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

Throw error for boundary constraints using $rand against main trace #46

Closed
grjte opened this issue Oct 28, 2022 · 1 comment · Fixed by #137
Closed

Throw error for boundary constraints using $rand against main trace #46

grjte opened this issue Oct 28, 2022 · 1 comment · Fixed by #137
Labels
enhancement New feature or request v0.2

Comments

@grjte
Copy link
Contributor

grjte commented Oct 28, 2022

The use of $rand random values by boundary constraints should be limited to auxiliary constraints and therefore to boundary constraints which are against columns from the auxiliary trace.

We need to throw an error if $rand is used in boundary constraints defined against main trace columns. This should be handled in the IR.

For example, this should be an error:

...
trace_columns:
    main: [a, b]
...
boundary_constraints:
    enf b.first = a + $rand[0]
...

See:
#36 (comment)
#36 (comment)

@grjte grjte added bug Something isn't working enhancement New feature or request and removed bug Something isn't working labels Oct 28, 2022
@tohrnii tohrnii added the v0.2 label Nov 28, 2022
@Fumuran Fumuran linked a pull request Feb 7, 2023 that will close this issue
@tohrnii
Copy link
Contributor

tohrnii commented Feb 13, 2023

Closed by #137.

@tohrnii tohrnii closed this as completed Feb 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request v0.2
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants