Skip to content

Bin ranges#37

Merged
josiahseaman merged 5 commits intograph-genome:masterfrom
dimatr:bin_ranges
Apr 10, 2020
Merged

Bin ranges#37
josiahseaman merged 5 commits intograph-genome:masterfrom
dimatr:bin_ranges

Conversation

@dimatr
Copy link
Copy Markdown
Collaborator

@dimatr dimatr commented Apr 10, 2020

should fix #36

@subwaystation
Copy link
Copy Markdown
Member

I did more extensive testing from odgi build to odgi bin to CS. Looks good from my side.

@subwaystation
Copy link
Copy Markdown
Member

We will JSON_VERSION = 11.

Comment thread matrixcomponent/JSONparser.py Outdated
@josiahseaman
Copy link
Copy Markdown
Member

Version bump hasn't been committed yet.

@subwaystation
Copy link
Copy Markdown
Member

I still need to bring in new example data.

@josiahseaman
Copy link
Copy Markdown
Member

Checking this now. Don't worry about the merge conflict, it's minor.

@josiahseaman
Copy link
Copy Markdown
Member

The current data needs to be updated from odgi. data/run1.B1phi1.i1.seqwish.w1000.json doesn't have a header, so it never throws a version error.

@subwaystation
Copy link
Copy Markdown
Member

Oh man, these conflicts.....

@josiahseaman josiahseaman merged commit 6d28970 into graph-genome:master Apr 10, 2020
@josiahseaman
Copy link
Copy Markdown
Member

josiahseaman commented Apr 10, 2020

@subwaystation All done! In the future, please note that we won't be tracking fasta chunks beyond bin width 1.

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.

Synchronize JSON schema with odgi

4 participants