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

Violation of BSD license terms for Fatiando a Terra code #7

Open
leouieda opened this issue Aug 23, 2020 · 1 comment
Open

Violation of BSD license terms for Fatiando a Terra code #7

leouieda opened this issue Aug 23, 2020 · 1 comment

Comments

@leouieda
Copy link

Hi @igp-gravity, I found this package today and was a bit surprised to see so much code from Fatiando a Terra copied here without explicit acknowledgement and without our license and copyright notice in the source. While the BSD 3-clause license used in all our projects permits redistribution of the source code, you are required by the license to include the copyright notice and license terms along with the copied code. See https://github.com/fatiando/verde/blob/master/LICENSE.txt for example. It would be courteous if you included a notice in the documentation (docstrings) as well, though you are not required by the license to do so.

In fact, I would encourage you to not copy the code from projects like Verde here (of course, you can do so if you want and follow the license terms). Instead, installing and import verde as a dependency is the best way to do this. You wouldn’t copy the NumPy code into your repository, for example. This is not hard to do and users would get the benefit of the latest developments from Verde when we make new releases. There is a growing ecosystem of geophysics software in Python (fatiando, simpeg, gempy, pygimli) and making code that plays well with the existing packages is a great boost to everyone. geoist has some interesting functionality that would be a great complement to the existing tools without needing to try to include all of them in its own source. This also encourages contributions from experienced open-source developers, like myself and the rest of the Fatiando community who might find geoist useful.

If you have modifications to the library (Verde etc), you should contribute those back to the project so that all can benefit instead of keeping them here. We welcome new contributions and have a comprehensive guide for giving credit to contributors: https://github.com/fatiando/contributing/blob/master/AUTHORSHIP.md

If you need functionality from the fatiando package, please consider helping us port this to the new Harmonica package. We’re working hard to implement filters and modelling/inversion code but time is limited and we could use some help.

Thank you for writing open-source code and making your work public! It’s always great to see more of this. I hope you consider my suggestions above since open-source works best when it’s done as a community. If not, then please at least follow the BSD license terms.

@steveshichen
Copy link
Collaborator

steveshichen commented Aug 23, 2020 via email

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