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

Improve citation guidelines #1905

Open
mattwthompson opened this issue Jul 10, 2024 · 0 comments
Open

Improve citation guidelines #1905

mattwthompson opened this issue Jul 10, 2024 · 0 comments

Comments

@mattwthompson
Copy link
Member

We recently came across a paper that cited the toolkit as

AM1/BCC partial charges were assigned using the OpenFF toolkit with default
settings.

Unfortunately there are two (possibly three, soon) default settings. RDKit/AmberTools and OpenEye generate slightly different charges for many molecules (#447 and many other discussions) based on what is installed and licensed, and soon NAGL will introduce a third backend. Having code associated with paper(s) helps but doesn't necessarily solve this issue if it's not clear what packages are installed at runtime.

We can't enforce citation guidelines, but I suggest we should nudge people towards including critical information like the backend used for charge generation. Are there other details that are this important (read: text that is necessary to reproduce results, even when code is included)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant