Skip to content
This repository has been archived by the owner on Jan 18, 2020. It is now read-only.

Merge the latest internal bug fixes and features #10

Merged
merged 20 commits into from
Dec 12, 2013
Merged

Conversation

naegelyd
Copy link
Contributor

No description provided.

Since the genotype field is nullable there is no guarantee that it will be present in call cases so we need to check for its existence before attempting to access properties of it.

From internal change on 11/21
From internal merge on 12/4
From internal merge on 12/6
Fix load_hpo method to take the using argument.

From internal change on 12/6
From internal change on 12/6
This key is not always there and was causing key errors in the tests. Now, the code properly ignores the effects for variants without this key by checking if the 'EFF' key exists before trying to access and process it.

From internal change on 12/6
Without this enabled, files will be queued up to be processed every time they are scanned.

From internal change on 12/9
This prevent stopping the queue process for faulty samples

From internal change on 12/10
Harvest doesn't so neither will we for now
@coveralls
Copy link

Coverage Status

Coverage increased (+2.3%) when pulling b986464 on internal-changes into eaa8ff8 on master.

naegelyd added a commit that referenced this pull request Dec 12, 2013
Merge the latest internal bug fixes and features
@naegelyd naegelyd merged commit 1740388 into master Dec 12, 2013
@naegelyd naegelyd deleted the internal-changes branch December 12, 2013 17:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants