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

Type inference #2

Draft
wants to merge 63 commits into
base: master
Choose a base branch
from
Draft

Type inference #2

wants to merge 63 commits into from

Conversation

PPER
Copy link
Contributor

@PPER PPER commented Jun 16, 2020

No description provided.

@cyrus-
Copy link

cyrus- commented Jun 17, 2020

Can you empty out master and set it up so that the diff for this PR shows all of the code?

@PPER
Copy link
Contributor Author

PPER commented Jun 17, 2020

I can not clear all commits in a branch, so I left the first commit in master. The first commit is just to upload the latex template, so it's not necessary to show in PR. (If I delete master and set a new, I'm worried there will be something I cannot handle when merging)

PPER and others added 30 commits June 30, 2020 22:09
…ntermediate status denoting something is potentially solved or unsolved. added utyp stuff which essentially lets you reference the side of a hole. changed unify_one and add_result so dependencies weren't overwritten and were done in oth directions if holes exist on both sides of a constraint. WIP- doesn't compile yet
…es to be accumulated along with those for type variables; the former acts as a record for the latter
…trees of union find clusters with where children are partially informed by their parent's status (ie only if it is in their branch) and parents are entirely informed by their children's status. wouldn't need topological sorting and would be easy to resolve due to union find's clusters. a little sad if this works since it may make all the top-sort work not very useful, but its a nice abstraction I think lol. I'm thinking I should finish the top-sort implementation and shift once it works since I think it is getting close
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.

4 participants