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

[query] Remove HailStateManager #14842

Open
wants to merge 1 commit into
base: ehigham/move-gen-into-tests
Choose a base branch
from

Conversation

ehigham
Copy link
Member

@ehigham ehigham commented Mar 18, 2025

This change removes the need for the HailStateManager by storing the reference genome on the TLocus and parsing the reference by name when the IR is parsed.
There should be no behavioural changes herein - the ir can't compile if the refererence is unknown - all I'm doing is parsing it earlier.

This change has low security impact.

@ehigham ehigham marked this pull request as ready for review March 18, 2025 03:46
@ehigham ehigham force-pushed the ehigham/move-gen-into-tests branch from 5dfe699 to fa8b624 Compare March 19, 2025 19:13
@ehigham ehigham force-pushed the ehigham/remove-hail-state-manager branch from cbd4b7f to 94d5e03 Compare March 19, 2025 19:13
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