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

Copyright notices from the original codebases are missing #48

Open
dgrell opened this issue Jun 4, 2024 · 1 comment
Open

Copyright notices from the original codebases are missing #48

dgrell opened this issue Jun 4, 2024 · 1 comment

Comments

@dgrell
Copy link

dgrell commented Jun 4, 2024

You have marked every file with (C) 2024 Willy Fitra Hendria, also the LICENSE file only has your name in it. This is a break with the licence of the projects that you have based yours on.

For example, visualkeras (which has substantial code similarities, and which you acknowledge as a basis), has an MIT licence with "Copyright (c) 2020 Paul Gavrikov"

The MIT licence states, as its one requirement that "The above copyright notice [...] shall be included in all copies or substantial portions of the Software.
"

Action point: the LICENSE file and any code file that has substantial pieces of code from other sources, need to include the copyright line from those original sources in addition to yours.

An example for graph.py:

# Copyright (c) 2020 Paul Gavrikov
# Copyright (C) 2024 Willy Fitra Hendria
# SPDX-License-Identifier: MIT

I have not checked the other two packages you mention as inspiration, the same should be done for those authors if there is code overlap.

https://github.com/openjournals/joss-reviews/issues/6678

@willyfh
Copy link
Owner

willyfh commented Jun 4, 2024

Hi @dgrell, thank you for your feedback. I will thoroughly check the source code and provide the appropriate copyright lines.

@willyfh willyfh mentioned this issue Jun 25, 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