ClandininLab / brainsss2

refactor of the brainsss repo for 2-photon imaging analysis
MIT License
0 stars 0 forks source link

packaging bits and bobs #29

Open rueberger opened 2 years ago

rueberger commented 2 years ago

just a few random things I noticed...

recommended method for installing is to clone and then pip install the directory, but if you're not using the editable flag you might as well publish the project to pypi and simplify things a bit for users.

super minor thing: right now scripts are being published via the scripts argument in setup.py which is no longer best practice. may be more effort than it's worth to move things over to entry_points though, with the only discernible benefit being the satisfaction of knowing you're that much further along in taming the python packaging beast

poldrack commented 2 years ago

Thanks for poring over the code!