-
Notifications
You must be signed in to change notification settings - Fork 28
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
Cross-database compara datachecks + conflict resolution #294
Conversation
…it against the production_db.
…database must be in the tested one
Pull Request Test Coverage Report for Build 1378
💛 - Coveralls |
length() is only useful to correct the length of circular slices when start is greater than end. Here start is guaranteed to be lower or equal than end. length() involves additional queries to get the seq_region_id and check whether the slice is circular.
OK. I'll push my changes related to DNAFragCore here |
… low memory footprint - Dnafrags too can be batched together. - This works for polyploid components too.
997d5fa
to
878f7bb
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.
I have pushed all my changes here and in ensembl-compara. I have also piggybacked a fix in CheckGenomicAlignments (cc @CristiGuijarro ).
The updated DNAFragCore now runs on all species, regardless of the number of regions, but it's much quicker as I have made sure the data are fetched in a small number of queries. The entire set of Vertebrates can be checked in 30 min
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.
Nifty!
Copy of #285 that resolves merge conflicts.