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 initial species in core before seed mechanism #2382

Closed
wants to merge 2 commits into from

Conversation

hwpang
Copy link
Contributor

@hwpang hwpang commented Mar 2, 2023

Motivation or Problem

The current initialization would fail if one uses a pyrms-based reactor with seed mechanism and the seed mechanism contains species that are the initial species. It would fail with an error related to the initial species not in the phase.names.

Description of Changes

I moved the code block for initializing the bath gases and the initial species in front of the seed mechanism code block.

@hwpang hwpang self-assigned this Mar 2, 2023
@hwpang
Copy link
Contributor Author

hwpang commented Mar 2, 2023

A bit more detailed explanation for future reference. Currently, when RMG tries to add pressure dependent reactions from the seed mechanism to core that are pressure dependent on the initial species, the code crashes when it tries to make the corresponding core phase(s). This is because the core phase tries to get the index of that initial species from phase.names, but initial species are not yet in the core phase.

@mjohnson541 mjohnson541 force-pushed the debug_seed_mechanism_phase_names branch from 8574d62 to 74ba2bd Compare March 2, 2023 19:40
mjohnson541
mjohnson541 previously approved these changes Mar 2, 2023
@JacksonBurns
Copy link
Contributor

Rebasing with updated CI

@JacksonBurns JacksonBurns force-pushed the debug_seed_mechanism_phase_names branch from 74ba2bd to 0835cbf Compare April 16, 2023 00:14
@hwpang hwpang force-pushed the debug_seed_mechanism_phase_names branch from 0835cbf to d5334ac Compare June 9, 2023 18:45
@github-actions
Copy link

github-actions bot commented Sep 8, 2023

This pull request is being automatically marked as stale because it has not received any interaction in the last 90 days. Please leave a comment if this is still a relevant pull request, otherwise it will automatically be closed in 30 days.

@github-actions github-actions bot added the stale stale issue/PR as determined by actions bot label Sep 8, 2023
@JacksonBurns
Copy link
Contributor

@hwpang this looks simple enough - want to try and revive this one?

@github-actions github-actions bot removed the stale stale issue/PR as determined by actions bot label Sep 10, 2023
@hwpang hwpang force-pushed the debug_seed_mechanism_phase_names branch from d5334ac to da00ae8 Compare September 12, 2023 17:24
…re library reactions get move to core if all reactants and products are in initial species
@hwpang hwpang force-pushed the debug_seed_mechanism_phase_names branch from da00ae8 to b772183 Compare September 12, 2023 21:08
@hwpang
Copy link
Contributor Author

hwpang commented Sep 13, 2023

I will close this PR for now as I don't have time to work on it.

@hwpang hwpang closed this Sep 13, 2023
@JacksonBurns JacksonBurns reopened this Sep 19, 2023
@sevyharris sevyharris self-assigned this Sep 19, 2023
@JacksonBurns
Copy link
Contributor

Blocked by #2514

@JacksonBurns JacksonBurns linked an issue Dec 12, 2023 that may be closed by this pull request
Copy link

This pull request is being automatically marked as stale because it has not received any interaction in the last 90 days. Please leave a comment if this is still a relevant pull request, otherwise it will automatically be closed in 30 days.

@github-actions github-actions bot added the stale stale issue/PR as determined by actions bot label Jan 23, 2024
@github-actions github-actions bot added the abandoned abandoned issue/PR as determined by actions bot label Feb 23, 2024
@github-actions github-actions bot closed this Feb 23, 2024
@JacksonBurns JacksonBurns removed stale stale issue/PR as determined by actions bot abandoned abandoned issue/PR as determined by actions bot labels Feb 26, 2024
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.

AssertionError during an RMG job run
4 participants