BlauGroup / BlauGroup/mrnet

Concerted reaction tests should also test against reaction free energy

Open
#89 0 comments 0 reactions 1 assignee Claimed by @hpatel1567 View on GitHub
Dominant language
Python
Stars
11
Forks
9
PR merge metrics
No merged PRs in 30d

Description

Currently, concerted reaction addition tests only check the number of concerted reactions identified / added. However, we recently missed a bug which incorrectly modified the free energy of concerted reactions without changing the number of concerted reactions that are generated. A new test should be added or a current test should be extended to also check the properties of the generated concerted reactions in addition to the number of concerted reactions.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.