Uh oh!
There was an error while loading. Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork 359
Add CITATION.cff#891
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
base:main
Are you sure you want to change the base?
Add CITATION.cff #891
Uh oh!
There was an error while loading. Please reload this page.
Conversation
ShadowMitia commented Oct 20, 2021 • edited
Loading Uh oh!
There was an error while loading. Please reload this page.
edited
Uh oh!
There was an error while loading. Please reload this page.
ShadowMitia commented Oct 20, 2021
The file documentation is here for those interested https://github.com/citation-file-format/citation-file-format/blob/1.1.0/README.md#identifier-objects |
Uh oh!
There was an error while loading. Please reload this page.
Amaras commented Oct 20, 2021
I think it looks good. |
ShadowMitia commented Oct 20, 2021
I'm looking into the format for that. From what I understand there are two places we can have maintainers/contributors:
|
ShadowMitia commented Oct 20, 2021
The references section has a lot of useful things for us I think. But I have NO IDEA how they show up in practice. But if I understand it correctly we could reference a lot of things association with AAA, including papers and contributors. |
Amaras commented Oct 20, 2021
Maintainers and contributors who have a substantial amount of code in there should probably be listed as authors if they want. Let's say at least 10 PRs merged for a reasonable-ish cut-off point. I think maintainers who have less than those 10 PRs should still be included as authors if they have over 5 PRs merged. |
Amaras commented Oct 20, 2021
After checking, the |
ShadowMitia commented Oct 20, 2021
Well technicallyt the website code is a dependency for the actual website 😁 |
ShadowMitia commented Oct 21, 2021
Depending on how we finalise this file, it could replace This format has a validation tool with it. I would suggest adding the corresponding github action for it, or just ask people to run the validation before submitting a new version. |
ShadowMitia commented Oct 21, 2021
One thing I'm not sure about, it considers that this repo is hosting something of type If it is not the correct way, we can use the |
ShadowMitia commented Oct 21, 2021 • edited
Loading Uh oh!
There was an error while loading. Please reload this page.
edited
Uh oh!
There was an error while loading. Please reload this page.
I think we're close to a first verison, so undrafting it. I'll clean up all the commits into one once all the reviewing is done (and don't hesitate on reminding me 😁 ) |
leios left a comment
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.
overall, I am a big fan of this! I am a bit weary to put everyone in the citation file without their explicit permission, though. I feel we might need to tag each one and ask for their permission.
If they do not accept, they should stay in the Contributors.md file.
As a note: I think moving forward, we should use the citation.cff exclusively and move away from the contributors.md file
| alias: Leios | ||
| orcid: "https://orcid.org/0000-0002-3243-8918" | ||
| - family-names: Mazzuca | ||
| given-names: Nicole |
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 know Nicole said they do not want to be officially part of this project, due to some philosophical differences with how we write idiomatic C++ code
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.
If they provided code or help for other things they still should be here 😁 (If they want)
Uh oh!
There was an error while loading. Please reload this page.
| - family-names: Boyles | ||
| given-names: William | ||
| - family-names: Weinstein | ||
| given-names: Max |
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.
How do we organize all of these names?
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.
It's definitely going to be a big mess 😅
Uh oh!
There was an error while loading. Please reload this page.
| - "open research" | ||
| - "data structures" | ||
| - "collection of data structures" | ||
| type: software |
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.
Are there other types? I don't know if software is right, but it's also not wrong
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.
So this is where the file format gets a bit weird. Apparently it assumes the repo is either for "software" or "dataset". These are the only allowed types at that level of the file.
In the "references" you can have way more types, but it's just things that reference this file I guess? So I've been assuming that we're talking about the code for the website (which you can think of as a software you can run on your machine as well), but I agree it's not the best thing.
| - "data structures" | ||
| - "collection of data structures" | ||
| type: software | ||
| commit: 16fd2180041821f8ee53ece14c535e5b25de34fe |
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.
This essentially counts as a release for citations, right? That is to say that when people cite this work, they will be citing this, specific commit?
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.
Yeah I was just trying something out. I thought that the file required a "version" to be attached, but it wasn't. But I was thinking maybe it made sense to be able to cite the website when it was in a specific state?
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 would think we should omit this for now
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 it might be a good idea to tie this to a certain release (Maybe V 0.2021.1)? That way if people do cite this work, they can easily find the correct version. I also think zenodo links will usually be related to a specific release anyway, right?
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 also think zenodo links will usually be related to a specific release anyway, right?
Correct.
ShadowMitia commented Oct 23, 2021
I never thought about asking permissions 😅 |
ShadowMitia commented Oct 23, 2021
If we have to ask for approval maybe I should empty all the names and everyone adds themselves again? |
Amaras commented Oct 23, 2021
That is probably the best idea, even if that can become quite messy indeed. |
ShadowMitia commented Oct 23, 2021 • edited
Loading Uh oh!
There was an error while loading. Please reload this page.
edited
Uh oh!
There was an error while loading. Please reload this page.
I'm looking at the citations.cff repo. I'll probably raise an issue about citing a ressource or website instead of just software and dataset, and see what they recommend. I'm listing a coupe of issues in the meantime that might be interesting: |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Co-authored-by: Nicholas Tindle <[email protected]>
Co-authored-by: Sammy Plat <[email protected]>
Update CITATION.cff
leios commented Oct 24, 2021
To the discussion about what to do with people who do not want to be cited, here's a few solutions:
In all the cases, the citation file should be opt-in, in my opinion. Also: for 1 and 2, we need a way to cite "everyone else" and link to the right page, so maybe a ORCID for the AAA organization? |
ntindle commented Oct 24, 2021
Heavy agree on those who have an orcid and have them self add. Those who care about it would need to do it in a PR if they aren’t a maintainer and we would be able to approve based on past work. The orcid limitation wouldn't really be a problem as I personally made one as did @Amaras to make sure we would appear. |
ShadowMitia commented Oct 24, 2021
I've heard zenodo mentionned several times, this looks like this could be relevant https://guides.github.com/activities/citable-code/ |
leios commented Oct 24, 2021
I was mentioning zenodo because it looks like this file creates a zenodo link. I am not sure if that link continually changes with each commit, though |
ShadowMitia commented Oct 24, 2021
I don't know about each commit but from citation-file:
(Should've added that sorry 😅) |
leios commented Oct 24, 2021
Oh, I didn't know it went the other way as well. Ok, that's nice. |
ShadowMitia commented Nov 3, 2021
I got a response. The There might be something to do with The other solution is to have a bibtex directly in the project. It won't be parsed by github, but it will still show up on the right for quick access. But won't autogenerate anything. |
ntindle commented Nov 4, 2021
I would say we are software and we can start versioning on every commit to master? |
ShadowMitia commented Nov 4, 2021 • edited
Loading Uh oh!
There was an error while loading. Please reload this page.
edited
Uh oh!
There was an error while loading. Please reload this page.
We are software, but the thing we went to cite is the content in that software. It works as a workaround, but it's still not 100% what the citation file is designed for. |
ShadowMitia commented Nov 4, 2021
I've added a small CITATION.bib for reference. This would get picked up by github but would only point to that file. |
ntindle commented Nov 5, 2021 • edited
Loading Uh oh!
There was an error while loading. Please reload this page.
edited
Uh oh!
There was an error while loading. Please reload this page.
Right on it’s not what it’s designed for but I figure it’s better than nothing? I would prefer we use the cff a bit loosely as they are still releasing new versions that a bib that’s not very flexible |
ShadowMitia commented Nov 5, 2021
Oh wait hang on, there is a section on github on how to use CITATION.cff for other things : https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-citation-files#citing-something-other-than-software.... So actually we should be good to go on CITATION.cff. It probably won't show up perfectly inside github, but then we can get zelendo and zenodo integrations, and that should work fine. So the only real remaining question is : what do we do about authors/contributors for the citation? Let me know if I'm missing anything else. |
ntindle commented Nov 15, 2021
I will answer the above based on my opinions but I think the ultimate answer falls to @leios
Add All existing people from the contributors.md to the list. Ping them all in one big notification on this repo and give them a week to opt out.
Until we define what an author is, I feel it is unfair to exclude people
This replaces the contributers.md in my mind
Add them as authors for now, ranked by contribution count. Possibly with maintainers first (selfishly, this is because I wouldn't likely be cited and would be in the et. others. If this isn't a priority, I don't mind being near the bottom) |
Amaras commented Nov 16, 2021
I don't feel right making this list opt-out, since I assume consent is not given by default (it *could" be opt-out for maintainers and "authors" once we define that term though)
It feels unfair to exclude people, but it also feels unfair that we "forcefully" include inactive people who don't read their GH notifications.
Same for me.
+1 on that: Leios first, then other optionally maintainers ranked by contributions, then authors tanked by contributions. With a possibility to move down on demand, of course |
berquist commented Nov 16, 2021
I am in camp opt-out.
Authors are those in
I abstain from this point other than to say one model you can consider that other large (chemistry) software projects adopt is to have a largely alphabetical ranking, then have the first and/or last author(s) be those that are the brainchildren. Example, Example |
ntindle commented Nov 16, 2021
Alphabetical other than X (x to be determined if just @leios) is very reasonable to me. |
jiegillet left a comment
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 didn't follow all the discussions extremely closely, but it looks like something worth doing.
Uh oh!
There was an error while loading. Please reload this page.
Co-authored-by: Jie <[email protected]>
leios commented Dec 4, 2021
For authorship order, we are currently either considering Alphabetical or Activity-based right? The other option is to have me first, then the algorithm archivists (maintainers) in alphabetical order, followed by everyone from the For this, I guess we just ping all the maintainers here and ask if they are ok with this. If not, we can take them off the list. We should maybe give them a week to respond. Afterwards, we ping all the contributors. We might want to give everyone 2 weeks to respond to this? |
To help visualise #848 I've created a branch with a sample file. The idea is to keep modifying it (by me and maintainers directly) until we're satisfied. But because of the visual aspect, we need a place to see what it looks like/generates.
All other contributors can open up PRs on my branch to suggest changes.
You can just view the result on https://github.com/ShadowMitia/algorithm-archive/tree/citation-file