-
Notifications
You must be signed in to change notification settings - Fork 111
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
400 create citations using GitHub #442
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this PR, which will enable the project to be more widely cited.
Is it also possible to add a citation at the end of the readme? (https://github.com/scikit-learn/scikit-learn?tab=readme-ov-file#citation)
doi: "10.48550/arXiv.2207.12274" | ||
journal: "arXiv preprint arXiv:2207.12274" | ||
title: "MAPIE: an open-source library for distribution-free uncertainty quantification" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to quote the paper presented to COPA (as a replacement or in addition)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you remind me the link to this paper? My only concern is that this paper also presented a new insight and was not a general overview of the library.
What do you think @vincentblot28 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we must add the COPA paper as we also present MAPIE in it. https://proceedings.mlr.press/v204/cordier23a/cordier23a.pdf
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #442 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 39 41 +2
Lines 4616 4907 +291
Branches 487 523 +36
==========================================
+ Hits 4616 4907 +291 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for these changes. Once again, these are just a few harmonised changes. At the same time, I don't understand why we don't see all the citation information in the "Cite the repository" function...
Co-authored-by: Thibault Cordier <124613154+thibaultcordier@users.noreply.github.com>
Co-authored-by: Thibault Cordier <124613154+thibaultcordier@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! Let's merge them!
Description
This pull request introduces the addition of citation details to the MAPIE project by including a
CITATIONS.cff
file. The goal is to facilitate proper citation of MAPIE in academic and professional work by providing a standardized citation format. This change addresses the issue raised regarding the beneficial use of citations directly from the GitHub features. TheCITATIONS.cff
file includes author details, software title, version, release date, URL, and a preferred citation format.Fixes #400.
Type of change:
Please remove options that are irrelevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
CITATIONS.cff
file is properly formatted and recognized by GitHubChecklist
make lint
make type-check
make tests
make coverage
make doc