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

Htdp2.0 #29

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

Htdp2.0 #29

wants to merge 7 commits into from

Conversation

bennn
Copy link
Member

@bennn bennn commented May 22, 2015

This is for discussing, not merging. It doesn't compile anyway.

Look at the type in types.rkt. There are ~100 methods. Only 13 were in the documentation.
The latest compile error is passing a higher-order type for Any in the traverse method. If we fix it, we still need to assign a type to:

(compose render-multi-mixin render-mixin)

@bennn
Copy link
Member Author

bennn commented May 22, 2015

The fully typed version now runs, but note #30. It is not faithful to the original program.

Still need to try running all variations. I'm skeptical 😱

@bennn
Copy link
Member Author

bennn commented Jun 5, 2015

These do not run. Don't merge!

@bennn
Copy link
Member Author

bennn commented Jun 5, 2015

The bad configurations are 0010 0011 1010 and 1011. The error is always the same. @takikawa do you know how to fix?

process-whole: contract violation;
 cannot subclass a row polymorphic class with disallowed members (alt-paths up-path script-path script-file search-box?)
  in: the 1st conjunct of
      the 1st argument of
      ...
      the 5th argument of
      the 2nd argument of
      (->* <TRUNCATED ... Love, Ben>

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.

1 participant