search

pontushojer / NAIBR

Novel Adjacency Identification with Barcoded Reads
MIT License
2 stars 0 forks source link

Merge with original repo #2

Open pdimens opened 1 year ago

pdimens commented 1 year ago

Good afternoon,

Thank you for taking on this work and improving on the original NAIBR. Would it be possible to nudge the original authors to merge this repo and have it hosted on bioconda? It seems development on the original repository has been stale for some time.

pontushojer commented 1 year ago

Calling on @relyanow or @PijusSimonaitis if you have any ability to help with this.

Thank you for taking on this work and improving on the original NAIBR. Would it be possible to nudge the original authors to merge this repo and have it hosted on bioconda? It seems development on the original repository has been stale for some time.

Yes that why I have moved over to this fork for now. It would be ideal to merge this but I have not have any response to my previous PRs to their repo, see https://github.com/raphael-group/NAIBR/pull/20, https://github.com/raphael-group/NAIBR/pull/22 and https://github.com/raphael-group/NAIBR/pull/23. So I don't have any high hopes ATM.

The other option is to setup a bioconda recipe based on this fork.

pdimens commented 1 year ago

I saw those open PRs and hopefully they will respond. A bioconda recipe would be helpful, but it would still be good to have their blessing.

pontushojer commented 1 year ago

From my understanding of the MIT licence allows anyone to distribute the software. This is in some sense them giving their blessing to publish this on bioconda. Of course it would be nice to maintain the original repo but If its as stale as it seems maybe this fork is the way forward.

pdimens commented 1 year ago

Do you think it would be reasonable to give it a slightly different name on bioconda until such a thing [might] happen? Something akin to NAIBR+ or what have you.

pontushojer commented 1 year ago

@pdimens NAIBR is not available from bioconda 

conda install naibr-plus

I changed the title of this issue since i figure we could leave it open until such a time it gets merged (likely not) with the main repo.

pdimens commented 1 year ago

That's awesome, thank you! I should be able to add this to the build recipe for www.github.com/pdimens/harpy now!