Skip to content
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

Altered data structure #13

Merged
merged 112 commits into from
Apr 13, 2016
Merged

Altered data structure #13

merged 112 commits into from
Apr 13, 2016

Conversation

abradle
Copy link
Collaborator

@abradle abradle commented Apr 13, 2016

Altered data structure so that:

  • Decoder and encoder are symmetric
  • We can accommodate different serializers, not just MessagePack
  • Can accommodate different codec strategies

abradle and others added 30 commits April 1, 2016 17:39
Integration tests pass.
…as some bioassemblies do not have an index).
… see that any missing chain is actually missing from the structure.

    If parsing usign auth ids - bioassemby info isn't available - as no way to pass indices to auth chains.
Null values are missing from the file now.
For the API 99.0 is the default for resolution and 1.0 for rfree and rwork.
They are both now int[] not List<String>. First element is the ASCII id for the character. Second elemnet is the number of elements to be repeated.
Small refactoring in other modules
Another review session, thanks everyone!
@coveralls
Copy link

Coverage Status

Coverage decreased (-32.8%) to 16.245% when pulling 407e6d4 on alter_data_struct into b09bc3a on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-32.8%) to 16.245% when pulling 403e905 on alter_data_struct into b09bc3a on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+41.7%) to 90.759% when pulling 8f43dba on alter_data_struct into b09bc3a on master.

@abradle abradle merged commit c68481c into master Apr 13, 2016
@abradle abradle deleted the alter_data_struct branch April 22, 2016 12:53
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.

None yet

3 participants