Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JOSS review: Installation #16

Closed
DamienIrving opened this issue Jun 2, 2023 · 5 comments
Closed

JOSS review: Installation #16

DamienIrving opened this issue Jun 2, 2023 · 5 comments

Comments

@DamienIrving
Copy link
Contributor

@miniufo - well done on your submission to JOSS.

As part of my review of your submission (see openjournals/joss-reviews#5510), I have some questions/suggestions regarding software installation:

  • In the "how to install" (https://github.com/miniufo/xinvert#2-how-to-install) section of your README file it says "install via pip (not yet)" but as far as I can tell it can be installed by pip. What is the meaning of "not yet"?
  • Do you think it would be advantageous to also make the package available via conda (i.e. conda-forge) for users who manage their python environments using conda?
@miniufo
Copy link
Owner

miniufo commented Jun 2, 2023

  • that's right. I forgot to update this. (not yet) should be removed.
  • Definitely should be in conda. Right now I send a PR to conda for another package here, which is the first time I do this. I am not sure how long it would take (or maybe I do something incorrect). Maybe you can point me to the right path on how to make xinvert available via conda.

@DamienIrving
Copy link
Contributor Author

Your xgrads PR looks fine to me - hopefully they will action it soon.

Since xinvert is available via pip I don't think waiting for it to be available via conda should delay your JOSS publication. It would be nice to have it available via conda eventually, but it's not absolutely necessary for publication with JOSS.

@miniufo
Copy link
Owner

miniufo commented Jun 5, 2023

That's great. I'll follow the steps of xgrads and make a PR of adding xinvert to conda recipe.

@miniufo
Copy link
Owner

miniufo commented Jun 14, 2023

Hi @DamienIrving, I've removed 'not yet' in the README.md. Also, I've sent a PR to conda recipe, waiting it to be merged. Please see if this issue is fixed now.

@DamienIrving
Copy link
Contributor Author

Awesome. Thanks, @miniufo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants