-
Notifications
You must be signed in to change notification settings - Fork 11
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
Updates for 1.4.0 release #17
Conversation
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #17 +/- ##
==========================================
- Coverage 68.79% 68.73% -0.06%
==========================================
Files 21 24 +3
Lines 2416 2911 +495
Branches 427 539 +112
==========================================
+ Hits 1662 2001 +339
- Misses 680 828 +148
- Partials 74 82 +8 ☔ View full report in Codecov by Sentry. |
CHANGELOG.md
Outdated
|
||
### Features | ||
|
||
- Adding support reading and processing routes |
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.
- Adding support reading and processing routes | |
- Adding support for reading and processing routes |
CHANGELOG.md
Outdated
|
||
### Trivial changes | ||
|
||
- Making help for pipeline runner more simple |
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.
- Making help for pipeline runner more simple | |
- Making help for pipeline runner simpler |
docs/routes.rst
Outdated
------- | ||
|
||
The simplest route format supported is a text file, where each reaction is written as a reaction SMILES on a line. | ||
Routes are separated by comma |
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.
We say that routes are separated by comma
but below with separate them with a new line.
docs/routes.rst
Outdated
Reading | ||
------- | ||
|
||
The simplest route format supported is a text file, where each reaction is written as a reaction SMILES on a line. |
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.
This should match the information from read_reaction_lists
docstring.
The simplest route format supported is a text file, where each reaction is written as a reaction SMILES on a line. | |
The simplest route format supported is a text file, where each reaction is written as a reaction SMILES in a line. |
docs/routes.rst
Outdated
The last line of code also make sure that the second route shares mapping with the first route. | ||
|
||
|
||
Othe readers are available |
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.
Othe readers are available | |
Other readers are available |
rxnutils/routes/base.py
Outdated
class SynthesisRoute: | ||
""" | ||
This encapsulates a synthesis route or a reaction tree. | ||
It provide convinient methods for assigning atom-mapping |
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.
It provide convinient methods for assigning atom-mapping | |
It provides convenient methods for assigning atom-mapping |
rxnutils/routes/base.py
Outdated
It is typically initiallized by one of the readers in the | ||
`rxnutils.routes.readers` module. | ||
|
||
The tree depth and the forward step is automatically assigned |
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 tree depth and the forward step is automatically assigned | |
The tree depth and the forward step are automatically assigned |
rxnutils/routes/base.py
Outdated
) -> None: | ||
""" | ||
Assign atom-mapping to each reaction in the route and | ||
ensure that is is consistent from root compound and throughout |
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.
ensure that is is consistent from root compound and throughout | |
ensure that is consistent from root compound and throughout |
rxnutils/routes/base.py
Outdated
""" | ||
Returns linear sequences or chains extracted from the route. | ||
|
||
Each chain is a list of a dictionary representing the molecules, only the most |
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.
Each chain is a list of a dictionary representing the molecules, only the most | |
Each chain is a list of dictionaries representing the molecules, only the most |
rxnutils/routes/readers.py
Outdated
|
||
def _transform_retrosynthesis_atom_mapping(tree_dict: Dict[str, Any]) -> None: | ||
""" | ||
Routes output from AiZynth has atom-mapping from the template-based model, |
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.
Routes output from AiZynth has atom-mapping from the template-based model, | |
Routes output from AiZynth have atom-mapping from the template-based model, |
@CKannas - fixed your remarks |
No description provided.