-
Notifications
You must be signed in to change notification settings - Fork 35
Bring the "for RMG" developments into main. #270
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
Open
rwest
wants to merge
60
commits into
main
Choose a base branch
from
for_rmg
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
60 commits
Select commit
Hold shift + click to select a range
61ca710
include electronchange in reaction reconstruction in phase
mjohnson541 f1c175b
add Marcus kinetics type and allow dGrxn dependence of kinetics
mjohnson541 61fa7f6
handle dGrxn and d dependence of reactions in interfaces and phase ca…
mjohnson541 9709465
adapt Domains to dGrxn and d dependence of forward rate coefficient c…
mjohnson541 8c308fd
add a Marcus reaction to the ORR.rms mechanism for testing
mjohnson541 f230853
fix variable naminng
mjohnson541 ae59b8c
Added reference potential to Arrheniusq
ssun30 ff0ef51
Fixed interfaces not setting phi correctly
ssun30 b2ba84f
Parser uses RMG if RDKit fails to get structure
ssun30 db6d862
Now runs CI upon pushing marcus_development
ssun30 5885823
fix getKc call in ReactiveInternalInterface
mjohnson541 2e0e612
add phi and d to newer domains
mjohnson541 2586194
phase electron change fixes
mjohnson541 f81e093
missing domain tweaks for phi/d
mjohnson541 bd94192
fix FragmentBasedReactiveFilmGrowthInterfaceConstantT interface
mjohnson541 c22423a
Added default values for wr/wp for Marcus Kinetics
ssun30 3f0396f
Fixed array input format in ORR.rms
ssun30 2cea301
Changed variable naming in TestReactors
ssun30 6dd0fe6
Fixed typos
mjohnson541 c15ad5a
loosen solver tolerances for some tests
ssun30 59823ca
ignore automatically generated files
hwpang 3ece71b
Add channels
hwpang 625498d
Remove yml file as it's not used anymore, using CondaPkg to manage env
hwpang 1673e59
Switch to CondaPkg, PythonCall, PythonPlot
hwpang a53dca0
Remove unused dependency
hwpang 060a85f
Update version
hwpang 8fdf345
bump to 1.10
hwpang 3e76a06
No need to install env beforehand
hwpang a161fe1
Install with CondaPkg
hwpang 8104f65
Switch to 1.10 and no need to create env
hwpang 0f90162
export env path for CondaPkg
hwpang 4d8f6a4
Update build to use CondaPkg
hwpang 7c4a7de
Update installation instruction
hwpang e696b75
Use PythonPlot instead of PyPlot
hwpang d687d23
Switch to PythonPlot
hwpang e6ba67d
Switch parsing to PythonCall and PythonPlot
hwpang c59c6e4
Switch plotting to pythonplot
hwpang b1264fc
Use PythonCall syntax in module
hwpang e53ade9
Don't set jacobian prototype if jacobian is not sparse
hwpang 8b70c2b
Switch to pythoncall syntax in test
hwpang 3842a93
Import necessary packages
hwpang 8e007a0
Use SciMLSensitivity
hwpang cc98ddb
Change rtol
hwpang 7686bad
Add propane mech test file
hwpang 1aa21d2
Use >= 3.9
hwpang 2fb2129
keys(CondaPkg.current_packages()) doesn't actually show the packages …
hwpang e25fa1d
Should be molecule
hwpang 65959be
skip unused line
JacksonBurns 93845bf
avoid reducing over empty edge species rates, which is an error
JacksonBurns 56fbe5e
port sucessful CI changes from rmg to rms
JacksonBurns 5e0c059
update cache version
JacksonBurns 04cd72b
port ci updates to docs action
JacksonBurns 55f9766
Update CI.yml
JacksonBurns b98949a
Update documentation.yml
JacksonBurns 5f4ee4a
let rms figure out conda on its own
JacksonBurns dc4ef01
let rms figure out conda on its own
JacksonBurns e839c05
remove defaults channel
JacksonBurns 7d3904b
add instantiate
JacksonBurns cf83540
consolidate steps to avoid repeat installs
JacksonBurns 43018b1
Merge pull request #268 from hwpang/fix_installation
JacksonBurns File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,4 +3,8 @@ | |
*.dot | ||
docs/build/ | ||
.DS_Store | ||
deps/build.log | ||
deps/build.log | ||
.CondaPkg/ | ||
rms_env/ | ||
*.cov | ||
Manifest.toml |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
channels = ["mjohnson541", "rmg", "conda-forge", "cantera"] |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,21 +1,29 @@ | ||
using PyCall | ||
using Conda | ||
packages = Conda._installed_packages() | ||
if !("rmg" in packages) && !("rmgmolecule" in packages) && (PyCall.pyversion.major != 3 || PyCall.pyversion.minor != 7) | ||
const Pkg = Base.require(Base.PkgId(Base.UUID("44cfe95a-1eb2-52ea-b672-e2afdf69b78f"), "Pkg")) | ||
Conda.rm("mamba") | ||
Conda.add("conda=4") | ||
Conda.add("mamba") | ||
Conda.update() | ||
Conda.add("python=3.7") | ||
try | ||
Conda.rm("numpy") #get around MKL problem | ||
catch e | ||
using CondaPkg | ||
using PythonCall | ||
|
||
has_rmgpy = true | ||
has_rmgmolecule = true | ||
try | ||
PythonCall.pyimport("rmgpy") | ||
catch | ||
has_rmgpy = false | ||
end | ||
try | ||
PythonCall.pyimport("molecule") | ||
catch | ||
has_rmgmolecule = false | ||
end | ||
|
||
if !has_rmgpy && !has_rmgmolecule | ||
|
||
if !(v"3.7" <= PythonCall.C.python_version() && PythonCall.C.python_version() <= v"3.9") | ||
CondaPkg.add("python"; version=">=3.9") | ||
end | ||
Conda.add("nomkl") | ||
Conda.add("numpy") | ||
Conda.add_channel("mjohnson541") | ||
Conda.add_channel("hwpang") | ||
Conda.add("rmgmolecule") | ||
Pkg.build("PyCall") | ||
end | ||
CondaPkg.add("rmgmolecule"; version=">=0.3.0", channel="mjohnson541") | ||
CondaPkg.add("matplotlib", channel="conda-forge") | ||
CondaPkg.add("rdkit", channel="conda-forge") | ||
CondaPkg.add("pydot", channel="conda-forge") | ||
end | ||
|
||
const Pkg = Base.require(Base.PkgId(Base.UUID("44cfe95a-1eb2-52ea-b672-e2afdf69b78f"), "Pkg")) | ||
Pkg.build("PythonCall") |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The installation instructions have been updated to use CondaPkg and PythonCall. Please ensure that the documentation clearly reflects these changes so that users can correctly set up their environment.
Copilot uses AI. Check for mistakes.