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

Clarify qcelemental/checkup_data/{physconst,periodictable}.py license #159

Closed
mbanck opened this issue Nov 3, 2019 · 1 comment · Fixed by #160
Closed

Clarify qcelemental/checkup_data/{physconst,periodictable}.py license #159

mbanck opened this issue Nov 3, 2019 · 1 comment · Fixed by #160

Comments

@mbanck
Copy link

mbanck commented Nov 3, 2019

Those two files have a Psi4 license header, is that intended? I guess it makes sense as they might have been copied over from Psi4, but maybe it would be better to check with the Psi4 project whether they/you are fine with relinquishing that copyright and just have it be the top-level LICENSE.md?

@loriab
Copy link
Collaborator

loriab commented Nov 4, 2019

Nothing in that directory gets installed. It's more of a template for other codes to sanity check their local-->qcel transition. Psi4 probably shouldn't have been slapping a license header on those data files in the first place. I'll remove it.

@loriab loriab mentioned this issue Nov 4, 2019
2 tasks
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

Successfully merging a pull request may close this issue.

2 participants