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.

Sign up for GitHub

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

Simplified rmgdb.loop_families() #717

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
+44 −86
Draft

Simplified rmgdb.loop_families() #717

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
637c4f5
Simplified rmgdb.loop_families()
alongd Dec 4, 2023
7cfb2a6
f!
alongd Dec 5, 2023
dc45e29
Check in ReactionTest before setting cls.rmgdb.kinetics.families to None
alongd Dec 5, 2023
0b7adf4
f! rmgdb
alongd Dec 5, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
103 changes: 31 additions & 72 deletions arc/rmgdb.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -213,6 +213,24 @@
return None, False


def match_family_by_label(rmgdb: RMGDatabase,
family_label: str,
) -> Optional['KineticsFamily']:
Comment on lines +215 to +217

Check notice

Code scanning / CodeQL

Explicit returns mixed with implicit (fall through) returns Note

Mixing implicit and explicit returns may indicate an error as implicit returns always return None.
"""
Find a family in the RMG database by its label.

Args:
rmgdb (RMGDatabase): The RMG database instance.
family_label (str): The family label.

Returns: Optional[KineticsFamily]
The corresponding family object instance.
"""
for family in rmgdb.kinetics.families.values():
if family.label == family_label:
return family


def loop_families(rmgdb: RMGDatabase,
reaction: Reaction,
) -> List[Tuple['KineticsFamily', list]]:
Expand All @@ -227,79 +245,20 @@
Returns: List[Tuple['KineticsFamily', list]]
Entries are tuples of a corresponding RMG KineticsFamily instance and a list of degenerate reactions.
"""
reaction = reaction.copy() # Use a copy to avoid changing atom order in the molecules by RMG.
for spc in reaction.reactants + reaction.products:
generate_resonance_structures(spc)
fam_list = list()
for family in rmgdb.kinetics.families.values():
family.save_order = True
degenerate_reactions = list()
family_reactions_by_r = list() # Family reactions for the specified reactants.
family_reactions_by_rnp = list() # Family reactions for the specified reactants and products.

if len(reaction.reactants) == 1:
for reactant0 in reaction.reactants[0].molecule:
fam_rxn = family.generate_reactions(reactants=[reactant0],
products=reaction.products,
delete_labels=False,
rxn_copy = reaction.copy()
reactions_f = rmgdb.kinetics.generate_reactions(reactants=rxn_copy.reactants,
products=rxn_copy.products,
only_families=True,
resonance=True,
)
reactions_r = rmgdb.kinetics.generate_reactions(reactants=rxn_copy.products,
products=rxn_copy.reactants,
only_families=True,
resonance=True,
)
if fam_rxn:
family_reactions_by_r.extend(fam_rxn)
elif len(reaction.reactants) == 2:
for reactant0 in reaction.reactants[0].molecule:
for reactant1 in reaction.reactants[1].molecule:
fam_rxn = family.generate_reactions(reactants=[reactant0, reactant1],
products=reaction.products,
delete_labels=False,
)
if fam_rxn:
family_reactions_by_r.extend(fam_rxn)
elif len(reaction.reactants) == 3:
for reactant0 in reaction.reactants[0].molecule:
for reactant1 in reaction.reactants[1].molecule:
for reactant2 in reaction.reactants[2].molecule:
fam_rxn = family.generate_reactions(reactants=[reactant0, reactant1, reactant2],
products=reaction.products,
delete_labels=False,
)
if fam_rxn:
family_reactions_by_r.extend(fam_rxn)

if len(reaction.products) == 1:
for fam_rxn in family_reactions_by_r:
for product0 in reaction.products[0].molecule:
if same_species_lists([product0], fam_rxn.products, save_order=True):
family_reactions_by_rnp.append(fam_rxn)
degenerate_reactions = find_degenerate_reactions(rxn_list=family_reactions_by_rnp,
same_reactants=False,
kinetics_database=rmgdb.kinetics,
save_order=True
)
elif len(reaction.products) == 2:
for fam_rxn in family_reactions_by_r:
for product0 in reaction.products[0].molecule:
for product1 in reaction.products[1].molecule:
if same_species_lists([product0, product1], fam_rxn.products, save_order=True):
family_reactions_by_rnp.append(fam_rxn)
degenerate_reactions = find_degenerate_reactions(rxn_list=family_reactions_by_rnp,
same_reactants=False,
kinetics_database=rmgdb.kinetics,
save_order=True
)
elif len(reaction.products) == 3:
for fam_rxn in family_reactions_by_r:
for product0 in reaction.products[0].molecule:
for product1 in reaction.products[1].molecule:
for product2 in reaction.products[2].molecule:
if same_species_lists([product0, product1, product2], fam_rxn.products, save_order=True):
family_reactions_by_rnp.append(fam_rxn)
degenerate_reactions = find_degenerate_reactions(rxn_list=family_reactions_by_rnp,
same_reactants=False,
kinetics_database=rmgdb.kinetics,
save_order=True
)
if degenerate_reactions:
fam_list.append((family, degenerate_reactions))
fam_list = list()
for rxn in reactions_f + reactions_r:
fam_list.append((match_family_by_label(rmgdb=rmgdb, family_label=rxn.family), rxn))
return fam_list


Expand Down
Loading