poldracklab / pydeface

defacing utility for MRI images
MIT License
110 stars 42 forks source link

Versioning and pypi upload ? #33

Closed octomike closed 4 years ago

octomike commented 4 years ago

For new users it's a bit confusing to see what version to use. Pypi only has 1.0.0 (which is old and still uses python2), Github releases only list 1.0.0 and 1.1.0 and setup.py contains version string 2.0.

A new 2.0 (pre) release(?), a tag and an upload to pypi would be super helpful for us :)

vsoch commented 4 years ago

Totally agree, +1 on this one. 2.0 also is not using semantic versioning, which is important to do.

ofgulban commented 4 years ago

+1. It would also be good to know who are the active maintainers, @chrisgorgo @effigies ? I am time to time getting emails about errors etc. although I am not a maintainer. I would be willing to help tidying things up if you add me as a collaborator.

vsoch commented 4 years ago

I left the lab when I graduated in 2016 so I wouldn't be the right fit, and I know @chrisgorgo is now at Google, so he likely can't take it on. Pinging @rwblair and @oesteban who will likely know!

effigies commented 4 years ago

Looks like @poldrack is the maintainer on PyPI, so he'll need to either upload a 1.1.0 package or add maintainers.

As someone with essentially no connection to this project, I would be :+1: for adding @ofgulban as a maintainer on the strength of #15.

poldrack commented 4 years ago

if @ofgulban is happy to do that then I'm happy to add him

On Wed, Oct 30, 2019 at 8:34 AM Chris Markiewicz notifications@github.com wrote:

Looks like @poldrack https://github.com/poldrack is the maintainer on PyPI https://pypi.org/project/pydeface/, so he'll need to either upload a 1.1.0 package or add maintainers.

As someone with essentially no connection to this project, I would be 👍 for adding @ofgulban https://github.com/ofgulban as a maintainer on the strength of #15 https://github.com/poldracklab/pydeface/pull/15.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/poldracklab/pydeface/issues/33?email_source=notifications&email_token=AAGUVECATG7TJTV35MQESA3QRGSPZA5CNFSM4JBI7TA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECUVACY#issuecomment-547966987, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGUVECRAPKRIQNHWUHOIQ3QRGSPZANCNFSM4JBI7TAQ .

-- Russell A. Poldrack Albert Ray Lang Professor of Psychology Professor (by courtesy) of Computer Science Bldg. 420, Jordan Hall Stanford University Stanford, CA 94305

poldrack@stanford.edu http://www.poldracklab.org/

vsoch commented 4 years ago

Also a note for everyone - pypi this year has added two factor auth (and are also working on other security measures) so anyone that has an account that can manage packages, in that someone getting access and fussing around would have dire consequences, please strongly consider using it!

poldrack commented 4 years ago

thanks, done!

On Wed, Oct 30, 2019 at 1:13 PM Vanessasaurus notifications@github.com wrote:

Also a note for everyone - pypi this year has added two factor auth http://pyfound.blogspot.com/2019/06/pypi-now-supports-two-factor-login-via.html (and are also working on other security measures) so anyone that has an account that can manage packages, in that someone getting access and fussing around would have dire consequences, please strongly consider using it!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/poldracklab/pydeface/issues/33?email_source=notifications&email_token=AAGUVECY4IOTAWISETCG3C3QRHTG5A5CNFSM4JBI7TA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECVTSYI#issuecomment-548092257, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGUVEFHFZK7LRQPHEZ55QDQRHTG5ANCNFSM4JBI7TAQ .

-- Russell A. Poldrack Albert Ray Lang Professor of Psychology Professor (by courtesy) of Computer Science Bldg. 420, Jordan Hall Stanford University Stanford, CA 94305

poldrack@stanford.edu http://www.poldracklab.org/

ofgulban commented 4 years ago

if @ofgulban is happy to do that then I'm happy to add him

Yes I am happy to help @poldrack .

poldrack commented 4 years ago

please send me your pypi user name...

On Wed, Oct 30, 2019 at 1:58 PM Omer Faruk Gulban notifications@github.com wrote:

if @ofgulban https://github.com/ofgulban is happy to do that then I'm happy to add him … <#m7198061341754589811>

Yes I am happy to help @poldrack https://github.com/poldrack .

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/poldracklab/pydeface/issues/33?email_source=notifications&email_token=AAGUVEANJO3DRB6QQ3OYGPLQRHYO3A5CNFSM4JBI7TA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECVXZPQ#issuecomment-548109502, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGUVEBFCD33235IARCT3CLQRHYO3ANCNFSM4JBI7TAQ .

-- Russell A. Poldrack Albert Ray Lang Professor of Psychology Professor (by courtesy) of Computer Science Bldg. 420, Jordan Hall Stanford University Stanford, CA 94305

poldrack@stanford.edu http://www.poldracklab.org/

effigies commented 4 years ago

@ofgulban Gave you write access to the repository. Let me know if you need that bumped to admin.

ofgulban commented 4 years ago

my pypi username is ofgulban (https://pypi.org/user/ofgulban/)

ofgulban commented 4 years ago

@ofgulban Gave you write access to the repository. Let me know if you need that bumped to admin.

Thanks @effigies . I think it is ok for now. If need it I will let you know.

ofgulban commented 4 years ago

@effigies could you bump me to admin? I need it to setup zenodo.

Also, I think it would be safe to limit write access to master branch after releasing 2.0.0 without further change in the code and develop in devel branch in the future. I can also handle this.

effigies commented 4 years ago

@ofgulban Done.

ofgulban commented 4 years ago

Thanks @effigies zenodo badge is ready.

@poldrack it seems that I do not have the right permissions to upload to pypi. My pypi username is ofgulban (https://pypi.org/user/ofgulban/), in case if you have missed the message above.

poldrack commented 4 years ago

ok, you've been added as a maintainer on pypi

On Wed, Oct 30, 2019 at 2:02 PM Omer Faruk Gulban notifications@github.com wrote:

my pypi username is ofgulban (https://pypi.org/user/ofgulban/)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/poldracklab/pydeface/issues/33?email_source=notifications&email_token=AAGUVEHJDOWEO2CGXEWI5TLQRHY67A5CNFSM4JBI7TA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECVYFXQ#issuecomment-548111070, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGUVEB3CI4ZHZRZFBTXSBTQRHY67ANCNFSM4JBI7TAQ .

-- Russell A. Poldrack Albert Ray Lang Professor of Psychology Professor (by courtesy) of Computer Science Bldg. 420, Jordan Hall Stanford University Stanford, CA 94305

poldrack@stanford.edu http://www.poldracklab.org/

ofgulban commented 4 years ago

@poldrack thanks. Could you also add me as a maintainer in https://test.pypi.org/ . It would be cleaner to check if everything looks alright there.

effigies commented 4 years ago

@ofgulban The first uploader becomes maintainer on regular PyPI, so that is likely to be true on the test server. Since pydeface isn't on on there, you should be able to do it yourself.

ofgulban commented 4 years ago

Oh I see.

poldrack commented 4 years ago

yep, i never added it to test

On Thu, Oct 31, 2019 at 10:04 AM Omer Faruk Gulban notifications@github.com wrote:

Oh I see.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/poldracklab/pydeface/issues/33?email_source=notifications&email_token=AAGUVECGZ2EHCFVBGC2RXS3QRMF3PA5CNFSM4JBI7TA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECYQZNY#issuecomment-548474039, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGUVEEULZ2LDQTRGKP2K63QRMF3PANCNFSM4JBI7TAQ .

-- Russell A. Poldrack Albert Ray Lang Professor of Psychology Professor (by courtesy) of Computer Science Bldg. 420, Jordan Hall Stanford University Stanford, CA 94305

poldrack@stanford.edu http://www.poldracklab.org/

ofgulban commented 4 years ago
  1. I did some cosmetic changes in readme and setup.py. Did not touch to any code for now.
  2. I have fixed the versioning. It is 2.0.0 now.
  3. Setup zenodo integration. See https://zenodo.org/record/3524401.
  4. Uploaded to pypi (https://pypi.org/project/pydeface/>). Seems to work fine

Thanks @octomike & @vsoch for highlighting these issues. Thanks @poldrack and @effigies for your quick responses.

I will have a look at the remaining issues and pull requests wheneverly, but probably after ISMRM deadline next week unless if there is sth broken.