/GLnexus

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

capnp de/serialization of unified sites #114

Merged
merged 1 commit into from Jun 21, 2017
+217 −3

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@mlin @coveralls
@mlin
Owner

mlin commented Jun 19, 2017

The slowness of de/serializing unified sites in YAML has become an issue for WGS. Here is plumbing for doing so with capnp instead, while (as with discovered alleles) we still keep the YAML representation for tests and debugging.
@coveralls

coveralls commented Jun 19, 2017

Coverage Status

Coverage decreased (-0.02%) to 92.967% when pulling 2ca2527 on mlin-capnp-sites into 785b2af on master.
@coveralls

coveralls commented Jun 20, 2017

Coverage Status

Coverage decreased (-0.07%) to 92.942% when pulling 987cc8c on mlin-capnp-sites into 2e70795 on master.
@coveralls

coveralls commented Jun 21, 2017

Coverage Status

Coverage decreased (-0.04%) to 92.967% when pulling 7c7db99 on mlin-capnp-sites into 2e70795 on master.
@mlin mlin capnp de/serialization of unified sites

Some checks were not successful

1 errored and 2 successful checks
continuous-integration/travis-ci/pr — The Travis CI build could not complete due to an error
Details
continuous-integration/travis-ci/push — The Travis CI build passed
Details
coverage/coveralls — Coverage increased (+0.02%) to 92.967%
Details
3403250

@mlin mlin merged commit 9c1bdaf into master Jun 21, 2017
0 of 2 checks passed

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment