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

Long tut alpha equivalence #112

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

haroldcarr
Copy link
Contributor

Language Lambda/n is defined and used in a test, but the =a(lpha) function is defined in terms of language SD and uses the sd metafunction that is also defined on SD.

@wilbowma
Copy link
Collaborator

wilbowma commented Jul 4, 2017

It looks as if the intended behavior is for the test to remain as is, but that be defined for Lambda/n instead of SD so the test makes sense.

@haroldcarr
Copy link
Contributor Author

If is defined for Lambda/n then sd has to be defined for Lambda/n too.

@wilbowma
Copy link
Collaborator

wilbowma commented Jul 4, 2017

Looking carefully, there are clearly some issues with the declared languages in this section. sd/a and sd look as if they should be defined for Lambda, based on their tests. If they didn't have contracts, none of this would be a problem---redex is happy to ignore language declarations a lot of the time. We could use define-extended-metafunction to update the languages and contracts, but that hasn't been introduced in the tutorial.

@mfelleisen
Copy link
Contributor

mfelleisen commented Jul 4, 2017 via email

@haroldcarr
Copy link
Contributor Author

The tutorial gives the impression that run can run it at any point, up to the point one is reading. To me, that is useful.

For now, I will just keep sending "random" pull requests that "fix" things I run into.

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