test: reapply upstream architecture changes on synced base#4
Open
ivanmilevtues wants to merge 5 commits into
Open
test: reapply upstream architecture changes on synced base#4ivanmilevtues wants to merge 5 commits into
ivanmilevtues wants to merge 5 commits into
Conversation
Amp-Thread-ID: https://ampcode.com/threads/T-019ecb81-dc76-76cb-8bf3-22a366c9be41 Co-authored-by: Amp <amp@ampcode.com>
Architecture review · 45 components changedgraph LR
n_Experiment_Orchestration_Data_Access["Experiment Orchestration #amp; Data Access"]
n_Core_Image_Processing_Engine["Core Image Processing Engine"]
n_Feature_Detection_Decoding["Feature Detection #amp; Decoding"]
n_Spatial_Quantitation_Expression["Spatial Quantitation #amp; Expression"]
n_System_Framework_Validation["System Framework #amp; Validation"]
n_Experiment_Orchestration_Data_Access -- "provides raw tile data for stack construction" --> n_Core_Image_Processing_Engine
n_Experiment_Orchestration_Data_Access -- "orchestrates FOV processing into decoded datafr…" --> n_Spatial_Quantitation_Expression
n_Experiment_Orchestration_Data_Access -- "triggers validated pipeline transformations" --> n_System_Framework_Validation
n_Core_Image_Processing_Engine -- "requests specific tiles via lazy-loading" --> n_Experiment_Orchestration_Data_Access
n_Feature_Detection_Decoding -- "queries image metadata and axis grouping" --> n_Core_Image_Processing_Engine
n_Feature_Detection_Decoding -- "populates intensity tables with detected featur…" --> n_Spatial_Quantitation_Expression
n_Feature_Detection_Decoding -- "retrieves global configuration and logging sett…" --> n_System_Framework_Validation
n_Spatial_Quantitation_Expression -- "transforms spot intensities into expression mat…" --> n_Feature_Detection_Decoding
n_Spatial_Quantitation_Expression -- "utilizes image levels for segmentation masks" --> n_Core_Image_Processing_Engine
n_System_Framework_Validation -- "defines coordinate axes and transformation types" --> n_Spatial_Quantitation_Expression
n_Spatial_Quantitation_Expression -- "calls" --> n_Experiment_Orchestration_Data_Access
n_Core_Image_Processing_Engine -- "provides filtered images for" --> n_Feature_Detection_Decoding
n_Core_Image_Processing_Engine -- "supplies images for" --> n_Spatial_Quantitation_Expression
n_System_Framework_Validation -- "validates manifests for" --> n_Experiment_Orchestration_Data_Access
n_System_Framework_Validation -- "provides base classes for" --> n_Feature_Detection_Decoding
n_Experiment_Orchestration_Data_Access -- "calls" --> n_Feature_Detection_Decoding
n_Core_Image_Processing_Engine -- "calls" --> n_System_Framework_Validation
n_Spatial_Quantitation_Expression -- "calls" --> n_System_Framework_Validation
n_System_Framework_Validation -- "calls" --> n_Core_Image_Processing_Engine
classDef added fill:#1f883d,stroke:#0b5d23,color:#ffffff;
classDef modified fill:#bf8700,stroke:#7d4e00,color:#ffffff;
classDef deleted fill:#cf222e,stroke:#82071e,color:#ffffff,stroke-dasharray:5 3;
class n_Experiment_Orchestration_Data_Access,n_Core_Image_Processing_Engine,n_Feature_Detection_Decoding,n_Spatial_Quantitation_Expression,n_System_Framework_Validation modified;
linkStyle 10 stroke:#0b5d23,stroke-width:2px;
linkStyle 0,1,2,3,4,5,6,7,8,9 stroke:#7d4e00,stroke-width:2px;
linkStyle 11,12,13,14,15,16,17,18 stroke:#82071e,stroke-width:2px,stroke-dasharray:5 3;
Colors indicate component changes compared to Download the PR analysis artifacts from this workflow artifact. Experiment Orchestration & Data Access : 8 files changed
Core Image Processing Engine : 8 files changed
Feature Detection & Decoding : 7 files changed
Spatial Quantitation & Expression : 8 files changed
System Framework & Validation : 9 files changed
Explore this PR’s architecture in your browser or VS Code. codeboarding-action · run 27571094368 |
Amp-Thread-ID: https://ampcode.com/threads/T-019ecb81-dc76-76cb-8bf3-22a366c9be41 Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019ecb81-dc76-76cb-8bf3-22a366c9be41 Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019ecb81-dc76-76cb-8bf3-22a366c9be41 Co-authored-by: Amp <amp@ampcode.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Test PR for CodeBoarding review behavior on top of the latest synced master baseline.\n\nThis reapplies the same substantive architecture changes from the previous test PR without the retry commits.