Skip to content

Changes needed for Ocelot#17

Merged
stchang merged 24 commits intostchang:masterfrom
AlexKnauth:occurrence-concrete-predicate
Jul 11, 2017
Merged

Changes needed for Ocelot#17
stchang merged 24 commits intostchang:masterfrom
AlexKnauth:occurrence-concrete-predicate

Conversation

@AlexKnauth
Copy link
Collaborator

No description provided.

requires occurrence-orig-binding branch of turnstile
TODO: Is there a better way to restrict non-overlapping types into
bottom? Is it safe to say, no matter what, no two “base types” will
have any values that overlap?

TODO: Is there a better way to deal with known dead code (when
propositions are bottom) than what’s done here?

TODO: What should this do with Or propositions? (Right now they don’t
add any information.) What about Not propositions?
to deal with converting the prop to an env, dead code, putting the
correct scopes on the env identifiers, etc.
@stchang
Copy link
Owner

stchang commented Jul 5, 2017

Thanks. Would changing base-forms to base/forms fix the tests?

@AlexKnauth
Copy link
Collaborator Author

No, merging stchang/macrotypes#12 would fix them.

@stchang
Copy link
Owner

stchang commented Jul 5, 2017

ok great.

including fixing for/list to allow only concrete booleans for `#:when`
and `#:unless` conditions
@stchang
Copy link
Owner

stchang commented Jul 8, 2017

So the tests still fail even though stchang/macrotypes#12 is merged. Do you know what the cause is?

@stchang
Copy link
Owner

stchang commented Jul 8, 2017

Also, would you mind adding a line to travis.yml so it clones and tests your typed ocelot repo? Otherwise we're going to get out of sync

@AlexKnauth
Copy link
Collaborator Author

The failures in old versions of racket are expected. Typed Rosette relies on a change I made to how generics work (racket/racket#1644), which is currently only there in 6.9 and HEAD.

The failure on HEAD is weirder. There's just one test that's failing at runtime. It appears to be this one:
https://travis-ci.org/stchang/typed-rosette/jobs/250557454#L621-L627

(check-type (type-of s) : (C→ Any Bool) -> vector?)
--------------------
FAILURE
name:       check-equal?
location:   rosette-guide-sec7-tests.rkt:65:0
actual:     vector?
expected:   #<procedure:vector?>
--------------------

@AlexKnauth
Copy link
Collaborator Author

vector? thing fixed now.

@stchang
Copy link
Owner

stchang commented Jul 10, 2017

Hmm, I think the tests ran out of memory? Does ocelot have high requirements?

@stchang
Copy link
Owner

stchang commented Jul 10, 2017

Btw, I'm assuming this pr supercedes #9?

@stchang
Copy link
Owner

stchang commented Jul 10, 2017

Not sure what's going on with the tests. Only suggestion I could find is trying to reduce number of parallel jobs.

Can you add -j 1 or -j 2 to the line that installs ocelot? Not sure why it matters for ocelot but not others.

@stchang
Copy link
Owner

stchang commented Jul 10, 2017

And if reducing parallel jobs doesnt work, we can try to increase memory by setting sudo: required instead of sudo: false at the top of the script?

@stchang
Copy link
Owner

stchang commented Jul 10, 2017

Or you could give me access to your repo

@AlexKnauth
Copy link
Collaborator Author

I added -j 1 on the install step. I'm not sure whether that's enough or whether I'll have to do the same thing for the raco setup and raco test steps as well...

Also, I have the "Allow edits from maintainers" option turned on.

@stchang stchang merged commit 14d8ba0 into stchang:master Jul 11, 2017
@AlexKnauth AlexKnauth deleted the occurrence-concrete-predicate branch July 11, 2017 17:15
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