Update simulations for larger genomes #60
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… not grow with the size of the simulation
… for speed and disk footprint when running tests on massive simulated test data sets
…operations within simreads, which causes the rng to be used in a different order and changes the random output
…-ACGT bases within fragments
…ad in the genome because otherwise it was not enough for the human genome
…eems to prevent simulating all the way through the human genome
…the int from the rng, despite the values generated by the rng being larger and the values it got assigned to being larger, they were squeezed through int
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This involved fixing a bug in the simulation code that would not generate genomic positions relative to the start of the entire genome that were larger than max val of int, meaning roughly 1G