Skip to content

Remove redundant CLEM workflow logic - #867

Merged
tieneupin merged 4 commits into
mainfrom
redundant-clem-workflow-logic
Aug 19, 2026
Merged

Remove redundant CLEM workflow logic#867
tieneupin merged 4 commits into
mainfrom
redundant-clem-workflow-logic

Conversation

@tieneupin

@tieneupin tieneupin commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

This is a follow-up PR to PR #861, in which the LIF and TIFF API endpoints and the CLEM result registration workflow were updated so that they constructed and posted the relevant recipes to the processing_recipe service instead of loading another workflow to do so. Now that we have tested and verified that the changes in the previous PR worked, the redundant API endpoints and workflows are removed in this PR.

@tieneupin tieneupin self-assigned this Aug 19, 2026
@tieneupin tieneupin added enhancement New feature or request server Relates to the server component cryo-clem Part of the cryo-CLEM pipeline extension labels Aug 19, 2026
@codecov

codecov Bot commented Aug 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.74%. Comparing base (8aee927) to head (6fb1048).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #867      +/-   ##
==========================================
- Coverage   54.82%   54.74%   -0.08%     
==========================================
  Files         104      101       -3     
  Lines       11346    11239     -107     
  Branches     1496     1489       -7     
==========================================
- Hits         6220     6153      -67     
+ Misses       4796     4761      -35     
+ Partials      330      325       -5     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tieneupin
tieneupin force-pushed the redundant-clem-workflow-logic branch from f796de7 to 6fb1048 Compare August 19, 2026 10:55
@tieneupin
tieneupin marked this pull request as ready for review August 19, 2026 11:44

@stephen-riggs stephen-riggs left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you're confident none of this is needed it seems fine

@tieneupin
tieneupin merged commit 707a679 into main Aug 19, 2026
14 checks passed
@tieneupin
tieneupin deleted the redundant-clem-workflow-logic branch August 19, 2026 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cryo-clem Part of the cryo-CLEM pipeline extension enhancement New feature or request server Relates to the server component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants