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

RMG-Electrochem #2316

Open
wants to merge 110 commits into
base: main
Choose a base branch
from
Open

RMG-Electrochem #2316

wants to merge 110 commits into from

Commits on Jul 27, 2023

  1. Configuration menu
    Copy the full SHA
    18d15d7 View commit details
    Browse the repository at this point in the history
  2. added Faraday's Constant F

    davidfarinajr authored and mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    fed4eed View commit details
    Browse the repository at this point in the history
  3. added Potential as V quantity

    davidfarinajr authored and mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    07d5dcc View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    69cd7a9 View commit details
    Browse the repository at this point in the history
  5. added atomtype unit tests

    davidfarinajr authored and mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    7a838a9 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    fad4adf View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    5c943b7 View commit details
    Browse the repository at this point in the history
  8. added group unit tests

    davidfarinajr authored and mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    3f03438 View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    d64a296 View commit details
    Browse the repository at this point in the history
  10. Configuration menu
    Copy the full SHA
    5286352 View commit details
    Browse the repository at this point in the history
  11. added updated charge method and revised update Molecule method

    the update method now updates lone pairs before updating charge
    davidfarinajr authored and mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    9f2de34 View commit details
    Browse the repository at this point in the history
  12. Configuration menu
    Copy the full SHA
    5eae0c3 View commit details
    Browse the repository at this point in the history
  13. Configuration menu
    Copy the full SHA
    b4212fa View commit details
    Browse the repository at this point in the history
  14. Configuration menu
    Copy the full SHA
    0289b79 View commit details
    Browse the repository at this point in the history
  15. added molecule.pxd declarations

    davidfarinajr authored and mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    edd0129 View commit details
    Browse the repository at this point in the history
  16. Configuration menu
    Copy the full SHA
    378b598 View commit details
    Browse the repository at this point in the history
  17. Configuration menu
    Copy the full SHA
    9d5e587 View commit details
    Browse the repository at this point in the history
  18. Configuration menu
    Copy the full SHA
    8b8adff View commit details
    Browse the repository at this point in the history
  19. do not forbid ions

    however, we still need better thermo
    davidfarinajr authored and mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    6b31a06 View commit details
    Browse the repository at this point in the history
  20. Configuration menu
    Copy the full SHA
    8eb389a View commit details
    Browse the repository at this point in the history
  21. Configuration menu
    Copy the full SHA
    9ee890a View commit details
    Browse the repository at this point in the history
  22. added Molecule unit tests

    davidfarinajr authored and mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    eb2095b View commit details
    Browse the repository at this point in the history
  23. Configuration menu
    Copy the full SHA
    069444a View commit details
    Browse the repository at this point in the history
  24. Configuration menu
    Copy the full SHA
    811bcbc View commit details
    Browse the repository at this point in the history
  25. Configuration menu
    Copy the full SHA
    c094955 View commit details
    Browse the repository at this point in the history
  26. Configuration menu
    Copy the full SHA
    edfb710 View commit details
    Browse the repository at this point in the history
  27. Configuration menu
    Copy the full SHA
    78e7f76 View commit details
    Browse the repository at this point in the history
  28. Configuration menu
    Copy the full SHA
    9114451 View commit details
    Browse the repository at this point in the history
  29. Configuration menu
    Copy the full SHA
    456ed8e View commit details
    Browse the repository at this point in the history
  30. Configuration menu
    Copy the full SHA
    a2201ab View commit details
    Browse the repository at this point in the history
  31. added H+ and e to translator

    davidfarinajr authored and mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    536eb36 View commit details
    Browse the repository at this point in the history
  32. Configuration menu
    Copy the full SHA
    96f89e3 View commit details
    Browse the repository at this point in the history
  33. Configuration menu
    Copy the full SHA
    5a832c5 View commit details
    Browse the repository at this point in the history
  34. Configuration menu
    Copy the full SHA
    6e48b43 View commit details
    Browse the repository at this point in the history
  35. Configuration menu
    Copy the full SHA
    597bb66 View commit details
    Browse the repository at this point in the history
  36. Configuration menu
    Copy the full SHA
    33a06b9 View commit details
    Browse the repository at this point in the history
  37. add Li atom spin

    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    27f2e72 View commit details
    Browse the repository at this point in the history
  38. Configuration menu
    Copy the full SHA
    58da613 View commit details
    Browse the repository at this point in the history
  39. add Li atomtype

    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    7a57e75 View commit details
    Browse the repository at this point in the history
  40. Configuration menu
    Copy the full SHA
    3ffbfb7 View commit details
    Browse the repository at this point in the history
  41. Configuration menu
    Copy the full SHA
    30b9e4d View commit details
    Browse the repository at this point in the history
  42. Configuration menu
    Copy the full SHA
    80b5d98 View commit details
    Browse the repository at this point in the history
  43. add Li to R and R!H lists

    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    6147486 View commit details
    Browse the repository at this point in the history
  44. filter out resonance structures that try to put - charges on Li

    fix resonance bug (squash)
    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    5f413af View commit details
    Browse the repository at this point in the history
  45. add Li McGowan volume

    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    4ad20ae View commit details
    Browse the repository at this point in the history
  46. Configuration menu
    Copy the full SHA
    ffd7e22 View commit details
    Browse the repository at this point in the history
  47. add additional Li BDEs

    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    58e5061 View commit details
    Browse the repository at this point in the history
  48. Configuration menu
    Copy the full SHA
    17a3c63 View commit details
    Browse the repository at this point in the history
  49. Configuration menu
    Copy the full SHA
    2c4fbde View commit details
    Browse the repository at this point in the history
  50. Configuration menu
    Copy the full SHA
    638b864 View commit details
    Browse the repository at this point in the history
  51. Configuration menu
    Copy the full SHA
    998ffb1 View commit details
    Browse the repository at this point in the history
  52. Configuration menu
    Copy the full SHA
    a6013c5 View commit details
    Browse the repository at this point in the history
  53. Configuration menu
    Copy the full SHA
    4410294 View commit details
    Browse the repository at this point in the history
  54. Configuration menu
    Copy the full SHA
    4f06814 View commit details
    Browse the repository at this point in the history
  55. Configuration menu
    Copy the full SHA
    53f214c View commit details
    Browse the repository at this point in the history
  56. Configuration menu
    Copy the full SHA
    9cbf444 View commit details
    Browse the repository at this point in the history
  57. Revert "Remove charge transfer types from average_kinetics (to be rev…

    …erted)"
    
    This reverts commit 2180175.
    
    I also tweaked it a little bit.
    rwest authored and mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    732f057 View commit details
    Browse the repository at this point in the history
  58. Configuration menu
    Copy the full SHA
    8ec47b8 View commit details
    Browse the repository at this point in the history
  59. update rule fitting

    1) Enable fitting of ArrheniusChargeTransferBM
    2) if the BM fit is bad: E0 < 0, A > 1e30 or abs(n) > 5 just average the kinetics
    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    ba2d9f9 View commit details
    Browse the repository at this point in the history
  60. Configuration menu
    Copy the full SHA
    9697e62 View commit details
    Browse the repository at this point in the history
  61. Configuration menu
    Copy the full SHA
    3fa4dc1 View commit details
    Browse the repository at this point in the history
  62. add Li0 and Li+ atomtypes

    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    b06e552 View commit details
    Browse the repository at this point in the history
  63. Configuration menu
    Copy the full SHA
    1d0d4dd View commit details
    Browse the repository at this point in the history
  64. Configuration menu
    Copy the full SHA
    7f5d8ec View commit details
    Browse the repository at this point in the history
  65. standardize ascend option in cross validate

    ascend based on original uncertainties and compute errors at that node in cross validate
    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    4e99274 View commit details
    Browse the repository at this point in the history
  66. Configuration menu
    Copy the full SHA
    5f582ea View commit details
    Browse the repository at this point in the history
  67. add comment to averaged kinetics

    (Commit updated by rwest during rebase:
    In the original commit the  comment used to specify /why/ 
    it was averaged, but it is added in the averaging method itself.
    The /why/, if needed, should be added elsewhere.)
    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    62fcb55 View commit details
    Browse the repository at this point in the history
  68. fix BM fitting

    consistenty use dH298
    do not estimate e0 = w0/10 when  |dHrxn| > 0.4w0
    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    b550f2a View commit details
    Browse the repository at this point in the history
  69. Configuration menu
    Copy the full SHA
    c6536ce View commit details
    Browse the repository at this point in the history
  70. Configuration menu
    Copy the full SHA
    dd505f6 View commit details
    Browse the repository at this point in the history
  71. Configuration menu
    Copy the full SHA
    5d9a9ac View commit details
    Browse the repository at this point in the history
  72. Configuration menu
    Copy the full SHA
    1884255 View commit details
    Browse the repository at this point in the history
  73. Configuration menu
    Copy the full SHA
    125c9d8 View commit details
    Browse the repository at this point in the history
  74. Configuration menu
    Copy the full SHA
    1aaedc8 View commit details
    Browse the repository at this point in the history
  75. Configuration menu
    Copy the full SHA
    ceaa28e View commit details
    Browse the repository at this point in the history
  76. fix KineticsModel outputs

    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    7a95fa2 View commit details
    Browse the repository at this point in the history
  77. add SoluteTSData object

    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    814fbf7 View commit details
    Browse the repository at this point in the history
  78. Configuration menu
    Copy the full SHA
    4bc483b View commit details
    Browse the repository at this point in the history
  79. Configuration menu
    Copy the full SHA
    c76e327 View commit details
    Browse the repository at this point in the history
  80. Configuration menu
    Copy the full SHA
    425bdf7 View commit details
    Browse the repository at this point in the history
  81. Configuration menu
    Copy the full SHA
    370a966 View commit details
    Browse the repository at this point in the history
  82. Configuration menu
    Copy the full SHA
    b949bec View commit details
    Browse the repository at this point in the history
  83. only use kinetics that have arrhenius forms for tree generation

    the TS solute data is used only for rule fitting after tree is generated
    
    fix tab error
    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    f899089 View commit details
    Browse the repository at this point in the history
  84. Configuration menu
    Copy the full SHA
    1bc3a7a View commit details
    Browse the repository at this point in the history
  85. Configuration menu
    Copy the full SHA
    120e506 View commit details
    Browse the repository at this point in the history
  86. Configuration menu
    Copy the full SHA
    28473b1 View commit details
    Browse the repository at this point in the history
  87. fix arrhenius test

    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    0dba75b View commit details
    Browse the repository at this point in the history
  88. fix atomtype test

    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    837f0f1 View commit details
    Browse the repository at this point in the history
  89. Configuration menu
    Copy the full SHA
    cc3c443 View commit details
    Browse the repository at this point in the history
  90. Configuration menu
    Copy the full SHA
    5634365 View commit details
    Browse the repository at this point in the history
  91. Configuration menu
    Copy the full SHA
    a861172 View commit details
    Browse the repository at this point in the history
  92. remove test that solvent is in initial species

    this allows us to use parameters of similar solvents
    when the  exact solvent isn't available
    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    64bb21a View commit details
    Browse the repository at this point in the history
  93. adapt corrTest

    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    335228a View commit details
    Browse the repository at this point in the history
  94. Configuration menu
    Copy the full SHA
    c7c4643 View commit details
    Browse the repository at this point in the history
  95. Configuration menu
    Copy the full SHA
    572bd82 View commit details
    Browse the repository at this point in the history
  96. Configuration menu
    Copy the full SHA
    1df7896 View commit details
    Browse the repository at this point in the history
  97. Configuration menu
    Copy the full SHA
    76aef1a View commit details
    Browse the repository at this point in the history
  98. Configuration menu
    Copy the full SHA
    05e12ce View commit details
    Browse the repository at this point in the history
  99. Configuration menu
    Copy the full SHA
    a716e57 View commit details
    Browse the repository at this point in the history
  100. Configuration menu
    Copy the full SHA
    ab3678b View commit details
    Browse the repository at this point in the history
  101. Refactor some rate fitting in rule generation.

    It was::
    
        if n > 1:
            # big long block
        else:
            return None
    
    I inverted the check so we can return early, and outdent a huge block of code.
    rwest authored and mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    87d5d4d View commit details
    Browse the repository at this point in the history
  102. Minor refactor.

    rwest authored and mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    7823ad8 View commit details
    Browse the repository at this point in the history
  103. Fix ArrheniusBM.get_activation_energy unit test.

    The E0 had been changed, but the expected Ea not updated.
    rwest authored and mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    d70da49 View commit details
    Browse the repository at this point in the history
  104. Configuration menu
    Copy the full SHA
    824cbb0 View commit details
    Browse the repository at this point in the history
  105. Configuration menu
    Copy the full SHA
    e479209 View commit details
    Browse the repository at this point in the history
  106. modifed arrbm fit_to_data unit test

    Test now compares the fitted rate to the rate it was trained on to make sure they agree.
    davidfarinajr authored and mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    6c352db View commit details
    Browse the repository at this point in the history
  107. TEMPORARY: set database branch to lithium

    remove this commit before merging
    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    f805d04 View commit details
    Browse the repository at this point in the history
  108. fix atomtype charge checking

    checks that the first group_atom.atomtype is either an appropriate charged atomtype or is a general version of an appropriate charged atomtype
    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    752d413 View commit details
    Browse the repository at this point in the history
  109. fix typo in database test

    mjohnson541 committed Jul 27, 2023
    Configuration menu
    Copy the full SHA
    fd4457c View commit details
    Browse the repository at this point in the history

Commits on Jul 28, 2023

  1. allow cat of regression diff to fail, print a warning instead

    this will allow the regression test results to all be reported even if there is such a huge difference between the dynamic and baseline that the system utilities cannot print it
    
    see: #2316 (comment)
    JacksonBurns committed Jul 28, 2023
    Configuration menu
    Copy the full SHA
    5df983b View commit details
    Browse the repository at this point in the history