Skip to content

feat: add grambow barrier heights benchmark - #670

Open
lwehrhan wants to merge 5 commits into
ddmms:mainfrom
lwehrhan:feat/add-grambow-reactivity-benchmark
Open

feat: add grambow barrier heights benchmark#670
lwehrhan wants to merge 5 commits into
ddmms:mainfrom
lwehrhan:feat/add-grambow-reactivity-benchmark

Conversation

@lwehrhan

@lwehrhan lwehrhan commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Pre-review checklist for PR author

PR author must check the checkboxes below when creating the PR.

Summary

The Grambow dataset of reaction comprises about 12,000 reactant/product/transition state triplets for reactions of small neutral organic molecules limited to the elements H, C, N and O, accompanied by energy labels at ωB97X-D3/def2-TZVP level of theory. This benchmark assesses if the MLIP can accurately predict the barrier heights of all reactions across the full dataset. Reaction energies are also reported, however the benchmark is specifically targeted at barrier heights.

Linked issue

Resolves #662

Progress

  • Calculations
  • Analysis
  • Application
  • Documentation

Testing

New decorators/callbacks

@ElliottKasoar ElliottKasoar added the new benchmark Proposals and suggestions for new benchmarks label Jul 8, 2026
Comment thread docs/source/user_guide/benchmarks/molecular_reactions.rst Outdated
@joehart2001

Copy link
Copy Markdown
Collaborator

Thanks for the pr! same as #658 could you send the data + have a look at the element filtering?

docs: https://ddmms.github.io/ml-peg/developer_guide/filter.html

@ElliottKasoar ElliottKasoar added the external Contribution from external framework label Jul 23, 2026
@lwehrhan
lwehrhan marked this pull request as ready for review July 23, 2026 23:04
@joehart2001
joehart2001 force-pushed the feat/add-grambow-reactivity-benchmark branch from cd4691d to 9317dfa Compare July 28, 2026 12:38
@joehart2001

Copy link
Copy Markdown
Collaborator

Hey, im pretty happy with your side of the code. ive added the structure vis here too, but there are some minor bugs regarding the point highlighting during structure vis which i need to look into. otherwise all good

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

Labels

external Contribution from external framework new benchmark Proposals and suggestions for new benchmarks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New benchmark Grambow reaction barriers

3 participants