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

Syntax Update -- Property/Where syntax in terms of necessary and sufficient #161

Closed
wants to merge 4 commits into from

Conversation

sidprasad
Copy link
Collaborator

Tests now take a form that more transparently shows the implicit implications. Perhaps we could call these conditional hypotheses instead of property/where tests?

assert p { ...} is necessary for q for {...bounds... / scope}
where 
{ }

assert p { ...} is sufficient for q for {...bounds... / scope}
where 
{ }

@bennn
Copy link
Collaborator

bennn commented Jan 12, 2023

I'm not ready to look at an implementation yet.

What do we want students to do with the testing form?

I'm still puzzled from our Wednesday discussion about how this is meant to be a general testing construct. Everything I've seen (including the external evaluation docs) talks about running examples / properties against a wheat. What to do if there's no wheat?

@sidprasad
Copy link
Collaborator Author

@bennn @tnelson : Could I ask you to take a look today? I'd like to get this in for TA camp

@bennn
Copy link
Collaborator

bennn commented Jan 17, 2023

I'm still waiting to see where & how this is supposed to be used by students.

I can do a call 3-4:30pm or after 6pm.

@sidprasad sidprasad closed this Jan 19, 2023
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