-
-
Notifications
You must be signed in to change notification settings - Fork 249
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
Standardize authorship attribution #198
Comments
I propose to use option 3 with bullet points, like in https://pymc-examples.readthedocs.io/en/latest/survival_analysis/bayes_param_survival_pymc3.html. I think the dates are not necessary but don't hurt either. I propose to use the following categories:
Main open question that comes to mind is if going forward we want those bullet points to link to the PR with the changes |
@OriolAbril - Links to oriolabril.github.io in your original post all appear broken for me. But going off your explanation, I agree that option 3 is probably the best proposed. But some thoughts:
If it were me, I'd basically do an amended version of 3 - put a list of contributors at the top:
Then make the bottom more akin to a changelog with the full description like you propose:
That way people see contributors, but aren't overwhelmed with too much detail unless they want to look at the bottom. Last:
I don't see how this would hurt, only help, so I think you should. |
I would prefer having a short and synthetic sentence at the top over duplicating info. Duplicating info is much harder to maintain and it quickly ends up containing different info in each place. We could also try to get that on the left sidebar so it is easily available but doesn't "interfere" with reading the notebook 🤔 |
Right now it's not clear how authors of examples should claim authorship so they can get proper credit. There are several alternatives around:
We could even use https://github.com/sloria/sphinx-issues with the user, pr roles to simplify linking to github users and relevant PRs if using approaches 1 or 3.
The text was updated successfully, but these errors were encountered: