Conversation
…on(excluding assemblyplan MD)
…collections in sbol_doc instance var
…racted part objects in final document
|
closes #35 |
|
closes #36 |
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a9e3aac792
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| """ | ||
| backbone_def = extract_toplevel_definition(backbone_doc) | ||
| backbone_plasmid = MocloPlasmid(backbone_def.displayId, backbone_def, backbone_doc) | ||
| backbone_plasmid = Plasmid(backbone_def.displayId, backbone_def, backbone_doc) |
There was a problem hiding this comment.
translate_abstract_to_plasmids now instantiates Plasmid with backbone_def.displayId (a string) as the first argument, but Plasmid.__init__ expects a ComponentDefinition and immediately accesses definition.displayId; this raises AttributeError on every call, so abstract translation cannot run at all. The same constructor mismatch also appears in construct_plasmid_dict.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
good catch implement this fix
src/buildcompiler/buildcompiler.py
Outdated
| plasmid_dict, antibiotic_resistance=KAN | ||
| ) | ||
| else: | ||
| compatible_plasmids = get_compatible_plasmids(plasmid_dict, self.backbone) |
There was a problem hiding this comment.
When a caller provides the optional backbone parameter, this branch still calls get_compatible_plasmids(..., self.backbone), but self.backbone is never defined in BuildCompiler; this path raises AttributeError and makes user-selected backbones unusable.
Useful? React with 👍 / 👎.
src/buildcompiler/transformation.py
Outdated
| ] # TODO update with more sophisticated selection process? | ||
| plasmid_cd = plasmid.plasmid_definition | ||
|
|
||
| transformation_activity = sbol2.Activity(f"transform_{chassis_md.name}") |
There was a problem hiding this comment.
The activity URI is derived only from chassis_md.name, so every loop iteration creates the same Activity identity for different plasmids; adding the second transformed plasmid to transformation_doc will hit URI collisions and abort instead of producing one transformation record per plasmid.
Useful? React with 👍 / 👎.
Gonza10V
left a comment
Gonza10V
left a comment
There was a problem hiding this comment.
Looks good I'll merge so we use can keep the developing in new branches.
2 participants
Remaining TODO:
Optional TODO:
closes #33
closes #28
closes #35 #36