search

yeatmanlab / AFQ-Browser

Browser-based visualization tools for AFQ results
BSD 3-Clause "New" or "Revised" License
33 stars 15 forks source link

Add a record to the afqvault manifest through a PR. #208

Closed arokem closed 6 years ago

arokem commented 6 years ago

Still need to figure out how to create pull requests programmatically, but I believe that's the only remaining piece for this.

arokem commented 6 years ago

Still need to deduplicate if the row is already there.

arokem commented 6 years ago

OK - this is no longer WIP. I think it works as desired (@jyeatman : sorry for all those PR emails 😜)

richford commented 6 years ago

Not sure how to test this but since you and @akeshavan were working on this, if you say it's good to go, I'll believe you.

richford commented 6 years ago

Correct me if I'm wrong here:

Resolves #166 Resolves #134

arokem commented 6 years ago

Should definitely resolve #166. We might want to leave #134 around, so that we can consider integration with something like Zenodo in the future, but I'd say we don't have to worry about that right now.

arokem commented 6 years ago

@akeshavan : could you please take a look and make sure that I am not doing something silly here.

akeshavan commented 6 years ago

looks good! merging