search

catalystneuro / nwb-conversion-tools

Create NWB files by converting and combining neural data in proprietary formats and adding essential metadata.
https://nwb-conversion-tools.readthedocs.io/en/main/
BSD 3-Clause "New" or "Revised" License
25 stars 12 forks source link

pip installation #46

Closed CodyCBakerPhD closed 3 years ago

CodyCBakerPhD commented 4 years ago

Upon first installation, calling pip install nwb-conversion-tools without a cloned repository is unable to find the package. Is this posted to PyPI yet?

luiztauffer commented 4 years ago

this is because the package name on pypi is still nwbn-conversion-tools, we dropped the n recently. I can create a new project on pypi with the proper name. Now, should we stop updating the nwbn- project right away, leaving it only as legacy? @bendichter

bendichter commented 4 years ago

Yes please do that

On Fri, Jun 26, 2020, 6:41 AM Luiz Tauffer notifications@github.com wrote:

this is because the package name on pypi is still nwbn-conversion-tools, we dropped the n recently. I can create a new project on pypi with the proper name. Now, should we stop updating the nwbn- project right away, leaving it only as legacy? @bendichter https://github.com/bendichter

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/catalystneuro/nwb-conversion-tools/issues/46#issuecomment-650112942, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGOEERHCL3WGYCWSHGKRFLRYR3NXANCNFSM4OITR3QA .

luiztauffer commented 4 years ago

@CodyCBakerPhD @bendichter there were a lot of changes since I last contributed to the repo, so I'm not 100% up to date with everything. For now, we have it published on test Pypi and I think it would be wise to test it before we publish on the official pypi. You can do this:

pip install -i https://test.pypi.org/simple/ nwb-conversion-tools
bendichter commented 4 years ago

@luiztauffer test pypi is usually used for testing deployment, not testing of the package itself. If there are problems, we can always cut a new release. Is there a problem with that?

luiztauffer commented 3 years ago

so, what do you guys think, should we put out the current version already? @bendichter @CodyCBakerPhD

bendichter commented 3 years ago

Yeah go for it!

On Tue, Dec 15, 2020, 9:21 AM Luiz Tauffer notifications@github.com wrote:

so, what do you guys think, should we put out the current version already? @bendichter https://github.com/bendichter @CodyCBakerPhD https://github.com/CodyCBakerPhD

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/catalystneuro/nwb-conversion-tools/issues/46#issuecomment-745315190, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGOEESLDNPBEQ7X5YXIXGLSU5WHFANCNFSM4OITR3QA .

CodyCBakerPhD commented 3 years ago

It has been pretty stable since v0.6.0... might need to update/expand the examples in the to-nwb-template repo for demonstration however, I'll check