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

Updated list of AiiDA powered research papers #60

Merged
merged 5 commits into from
Apr 22, 2024
Merged

Conversation

khsrali
Copy link
Contributor

@khsrali khsrali commented Apr 9, 2024

I did a manual search to make this updated list of papers powered by AiiDA.
The idea is to have this PR ready to go at the same time as we send questionnaire, so that people can add their research paper to this page, if not listed. (this is probable, since my manual approach was pessimistic)

khsrali and others added 2 commits April 9, 2024 17:56
…AiiDA. The idea is to have this PR ready to go at the same time as we send questionnaire, so that people can add their research paper to this page, if not listed. (this is probable, since my manual approach was pesimestic)
@sphuber
Copy link
Contributor

sphuber commented Apr 11, 2024

How did you perform the search exactly? Did you just get any paper that cites the AiiDA paper? And which paper, the original from 2016, or also the 2020 one? What about the ACWF v1.0 and ACWF v2.0?

@khsrali
Copy link
Contributor Author

khsrali commented Apr 11, 2024

How did you perform the search exactly? Did you just get any paper that cites the AiiDA paper? And which paper, the original from 2016, or also the 2020 one? What about the ACWF v1.0 and ACWF v2.0?

I fetch all papers citing at least either
Main paper: S.P. Huber et al., Scientific Data 7, 300 (2020)
AiiDA engine: M. Uhrin et al., Comp. Mat. Sci. 187 (2021)

And check them one by one, discarded those which don't actually use aiida for research.

The list is accurate but not complete, for instance it doesn't include those aiida powered research paper citing only the
First paper, ADES model: G. Pizzi et al. Comp. Mat. Sci. 111, 218-230 (2016)

That's why at some point they got too many to check manually, so I made a PR of those we are sure, and hopefully will catch the rest by sending the questionnaire.

@sphuber
Copy link
Contributor

sphuber commented Apr 11, 2024

Thanks a lot @khsrali . That makes sense. But maybe it would be useful to add a disclaimer saying that these are curated manually and so that the list is not exhaustive.

Also, probably have to update this comment:

list of publications containing calculations powered by AiiDA, selected from the [citations of the AiiDA paper](https://scholar.google.com/scholar?cites=10268089832296963062&as_sdt=2005&sciodt=0,5&hl=en).

to include all the papers that you include in the search.

Would it be possible to add the ACWF v1.0 paper perhaps? It has 20 citations currently, but might have relevant research papers

@khsrali
Copy link
Contributor Author

khsrali commented Apr 11, 2024

it would be useful to add a disclaimer saying that these are curated manually and so that the list is not exhaustive.

sure sure, makes sense

Also, probably have to update this comment:
list of publications containing calculations powered by AiiDA, selected from the citations of the AiiDA paper.
to include all the papers that you include in the search.

For this and the picture, honestly, maybe it's more convenient to wait until we collect responses from the questionnaire.

Would it be possible to add the ACWF v1.0 paper perhaps? It has 20 citations currently, but might have relevant research papers

Do you mean to add it to the left column, where we say "If you use AiiDA please cite:" ? or to check the citations?

@sphuber
Copy link
Contributor

sphuber commented Apr 11, 2024

For this and the picture, honestly, maybe it's more convenient to wait until we collect responses from the questionnaire.

Just to be clear, what I meant was to add the engine paper, because you said the selected papers also contain those that cite the engine paper.

@khsrali
Copy link
Contributor Author

khsrali commented Apr 17, 2024

Hi @sphuber , I added the disclaimer. I intentionally repeated it under each relevant section.
Later, after sending off the questionnaire, we can slowly complete the list.

@sphuber
Copy link
Contributor

sphuber commented Apr 22, 2024

Thanks @khsrali . Changes look good, but the CI is failing. Don't understand why this is happening now, but it seems that the Python version we are using is no longer compatible with pydata-sphinx-theme. I will quickly fix that and then we can merge this

@sphuber
Copy link
Contributor

sphuber commented Apr 22, 2024

#63

@sphuber
Copy link
Contributor

sphuber commented Apr 22, 2024

Ok, I fixed the CI in #63 so this should now pass and we can merge. In fixing the broken links I did notice that the team is extremely outdated. If we want to keep that page, you may want to try and find some time to update it.

@sphuber sphuber self-requested a review April 22, 2024 08:19
Copy link
Contributor

@sphuber sphuber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot @khsrali

@sphuber sphuber merged commit 2a2e8c7 into aiidateam:main Apr 22, 2024
3 checks passed
@khsrali
Copy link
Contributor Author

khsrali commented Apr 22, 2024

Super! Thanks a lot for the quick action @sphuber

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

Successfully merging this pull request may close these issues.

2 participants