Skip to content

Type inference via multithreading and ownership#84

Closed
croyzor wants to merge 193 commits intomainfrom
inference-wip/ownership
Closed

Type inference via multithreading and ownership#84
croyzor wants to merge 193 commits intomainfrom
inference-wip/ownership

Conversation

@croyzor
Copy link
Collaborator

@croyzor croyzor commented Apr 16, 2025

No description provided.

@croyzor croyzor marked this pull request as ready for review December 10, 2025 14:05
@croyzor croyzor requested a review from acl-cqc December 10, 2025 14:05
@croyzor
Copy link
Collaborator Author

croyzor commented Dec 10, 2025

We have verbally agreed to make this the new main branch for BRAT since so much else depends on it

@acl-cqc acl-cqc changed the title wip: multithreading & ownership Type inference via multithreading and ownership Dec 10, 2025
Copy link
Collaborator

@acl-cqc acl-cqc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup let's do it.
The only thing I've spotted that really I'd have liked to pull out as a separate/preliminary change is the addition of CaptureSets, but commit 85ff9c7 (and following c90845c) does not cherry-pick cleanly onto main so....OK

@acl-cqc
Copy link
Collaborator

acl-cqc commented Dec 11, 2025

Yup let's do it. The only thing I've spotted that really I'd have liked to pull out as a separate/preliminary change is the addition of CaptureSets, but commit 85ff9c7 (and following c90845c) does not cherry-pick cleanly onto main so....OK

But can you fix the CI first?!

Because they have different behaviour when there's no matches!
@croyzor croyzor force-pushed the inference-wip/ownership branch from a31400a to 727a8a2 Compare December 12, 2025 10:25
@croyzor
Copy link
Collaborator Author

croyzor commented Dec 17, 2025

Superceded (cleaned up) by #93

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