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

add TeraChem #90

Closed
wants to merge 1 commit into from
Closed

add TeraChem #90

wants to merge 1 commit into from

Conversation

euhruska
Copy link
Contributor

Updated src/data/codes.json file.

@ltalirz
Copy link
Owner

ltalirz commented Sep 23, 2021

Thanks @euhruska for adding TeraChem!

Indeed, TeraChem is on the top of the watchlist, and if one just uses the keyword "TeraChem" it does match the relevance criterion. For other codes in the list, we also add an author or the company, and "terachem" Martinez OR "PetaChem" currently falls slightly short of 100 hits.
See this post for a brief explanation of the status quo.

I suggest we keep this PR open for the moment; if we go towards dropping the author names from the query we can merge it (or we might merge it in 2022 when also the above query will likely cross the threshold).

@ltalirz
Copy link
Owner

ltalirz commented Jan 9, 2022

Rebased in #102

@ltalirz ltalirz closed this Jan 9, 2022
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