-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: Updates to the alignment entity #298
Conversation
return None | ||
@classmethod | ||
def get_name_and_path(cls, metadata: dict, name: str, path: str, results: dict[str, str]) -> [str, str, dict]: | ||
paths = {filname: filname for filname in metadata.get("files", [])} |
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.
typo: filename
vs filname
format: OPTIONAL ENUM (IMOD, ARETOMO) | ||
method_type: OPTIONAL ENUM (fiducial_based, patch_tracking, projection_matching) | ||
is_portal_standard: OPTIONAL, BOOLEAN (DEFAULT FALSE) | ||
offset: OPTIONAL |
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.
thanks for adding these
@@ -34,7 +36,7 @@ def upgrade_config(data: dict[str, Any]) -> dict[str, Any]: | |||
|
|||
|
|||
def has_changes(file, config): | |||
with open(file, 'r') as file: |
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.
I think our formatters disagree
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.
wdym? 😅
@@ -817,7 +817,7 @@ classes: | |||
offset: | |||
description: *desc_alignment_offset | |||
range: AlignmentOffset | |||
volume_dimesion: | |||
volume_dimension: |
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.
good catch
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.
LGTM after the typo is fixed
* Updates to schema and template * Fixing tests * Adding files to alignments metadata * Update the alignment configs method type * Fixing typo * Adding undefined alignment_method_type
Relates to
Description
Changes to the config schema:
is_canonical
->is_portal_standard
method_type
Changes to the configs:
alignment.method_type
in the config. It follows the following conditions:S3 Workflows:
get_name_and_path
class method to the AlignmentImporter. This allows the DestinationFilterFinder to reference an existing Alignment.