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: License Conflict + Requirements File #3

Closed
gchure opened this issue Nov 4, 2024 · 2 comments
Closed

JOSS Review: License Conflict + Requirements File #3

gchure opened this issue Nov 4, 2024 · 2 comments

Comments

@gchure
Copy link

gchure commented Nov 4, 2024

Hi! I'm reviewing your project for JOSS and am opening issues as I progress through the checklist.

First, there is a conflict in licensing for PANINIpy. Your repository claims it is licensed under an MIT standard license, however your setup.py file assigns an "Unlicense" which is a legally ambiguous license (and mybe not valid in the yes of JOSS). Please converge on a license and have that unified across the project.

Second, there is no apparent requirements file for the project, leading to installation failures into a sterile venv. Please add requirements either as an external file or within the setup.py.

@baiyueh
Copy link
Owner

baiyueh commented Nov 5, 2024

Hi @gchure, thanks for bringing that up, have fixed to MIT license same as the github repo. Have added both requirments.txt and include the dependencies automatically while pip install.

@gchure
Copy link
Author

gchure commented Nov 5, 2024

Looks good, but there are some issues with the setup.py file that made installation and running the tests problematic. I will open a new issue to discuss.

@gchure gchure closed this as completed Nov 5, 2024
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