-
Notifications
You must be signed in to change notification settings - Fork 4
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
start modularizing intervals + BP graph, and generate AA-esque summary file #24
base: main
Are you sure you want to change the base?
Conversation
67da804
to
dd50e4e
Compare
a70bfc8
to
ee6af40
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @suhas-r In the "cycle2bed" mode, can you remove the requirement of input --num-cycles
and by default to convert all cycles to bed format?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes this has been done on the main branch in this commit.
b44912a
to
098f96c
Compare
…Edge dataclasses and associated refactors
testing, remove unused code, clean-up static typing
a83ebe6
to
1520f98
Compare
46ddc58
to
111e7ea
Compare
941b2fa
to
90b5e37
Compare
9f67b70
to
239aaee
Compare
Currently, we generate a container class
LongReadBAMToBreakpointMetadata
that stores information such as Chimeric Alignments + Copy-Number segments from pre-processing our aligned BAM file and.cns
seeds from CNVkit segmentation. We iteratively mutate components of this class to contain different transformed versions of these inputs in confusing + brittle ways. We also rely on metadata provided by the container class when attempting to generate the ILP model corresponding to a singleBreakpointGraph
, which shouldn't be necessary. After we do the initial amplified interval search and refinement, we should be able to generate self-containedBreakpointGraph
s that can be processed independently.In terms of the final outputted
_graph.txt
file, this PR adds PathConstraint + AmplifiedInterval information for eachBreakpointGraph
. This allows us to use just a*_graph.txt
file with thecycle
entrypoint to generate/solve an ILP model (no additional usage of a BAM or CNV seed file).