Skip to content

Version refractor#69

Open
abdelmounaimTA wants to merge 37 commits into
tariqdaouda:bloodyfrom
abdelmounaimTA:version_refractor
Open

Version refractor#69
abdelmounaimTA wants to merge 37 commits into
tariqdaouda:bloodyfrom
abdelmounaimTA:version_refractor

Conversation

@abdelmounaimTA
Copy link
Copy Markdown

No description provided.

feghalya and others added 30 commits February 13, 2020 20:59
Fix attributes dictionary in GTFEntry
In Python 3, getSequenceVariants is no longer deterministic. Polymorphisms are stored in sets. Since Python 3.3, python randomizes hash keys of sets for a security reason (https://stackoverflow.com/questions/14956313/why-is-dictionary-ordering-non-deterministic). This means that the iteration order might be different every time. 

I propose a simple sort before iteration of alleles so the output of getSequenceVariants becomes deterministic.

There is also a small change in the encode method to fix the edge case where a stop variant is present at the end of the sequence.
In this case, the sequence is SEQUENCEV/ instead of SEQUENCEV/*.
Deterministic version of getSequenceVariants
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.

6 participants