-
Notifications
You must be signed in to change notification settings - Fork 99
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 paper: Contribution and authorship #343
Comments
@TeroFrondelius thanks for the suggestion. In any case, we would like to keep the list of authors is it is. @fverdugo and myself were the ones that started the project, designed its kernel, and the only developers during the first stage of the project. |
Here: https://joss.readthedocs.io/en/latest/submitting.html#substantial-scholarly-effort, it defines 1000 LOC as a major contribution. Of course, I didn't look very detailed, but @victorsndvg has contributed 5903 LOC in 78 commits. |
As far as I can see, this number is about the min scholarship effort of the project to be accepted. We consider that LOC alone do not give much information. E.g., it is not the same to create wrappers using clang than designing the kernel of Gridap. Victor created wrappers to PETSc and re-implement in Julia the COO matrices and compression algorithms in our previous project. This is not the core of the library. On the other hand, the project has had about 200K of LOCs. So, we consider that this the list of authors is appropriate and complete. |
I accept the explanation, even though I don't fully agree. Engineering and mathematics should learn from medical and physics, where the list of authors is much longer and smaller contributions to the research are also recognized. |
I think that in Engineering we have too few authors in all papers. Anyway, I checked the contributions and the best of my understanding @victorsndvg has made some major contributions and could be listed as joss-paper author.
Xref: openjournals/joss-reviews#2520
The text was updated successfully, but these errors were encountered: