Skip to content

240 architecture implement quality control pipeline orchestration - #243

Draft
jeipollack wants to merge 7 commits into
developfrom
240-architecture-implement-quality-control-pipeline-orchestration
Draft

240 architecture implement quality control pipeline orchestration#243
jeipollack wants to merge 7 commits into
developfrom
240-architecture-implement-quality-control-pipeline-orchestration

Conversation

@jeipollack

Copy link
Copy Markdown
Contributor

Summary

Provide a concise description of the proposed changes.
Include "closes #<ISSUE_NUMBER>" if this addresses an open issue

Note: Validation tests and plots should not be added to the PR; they should be documented in the dedicated issue instead.

What’s changed

Describe the main changes introduced by this PR.
Focus on user-visible changes, bug fixes, performance improvements, or internal changes.

  • Example bullet 1
  • Example bullet 2

How to test / verify

Outline steps for reviewers or maintainers to validate the changes.
Include commands, scripts, or datasets if relevant.
Full validation results should be referenced in the associated issue, not pasted here.

  • Step 1
  • Step 2

Scope

Indicate the type of PR:

  • Feature
  • Bug fix
  • Hotfix
  • Documentation / process change
  • Internal / refactor
  • Release

Optionally, note if this PR is part of a larger milestone or set of related PRs.

Changelog

Did this PR introduce user-visible changes?
If yes, a Scriv changelog fragment must be added and committed.

  • Changelog fragment added (if applicable)

Reviewer Checklist

Reviewers should confirm the following before approving and merging:

  • The PR targets the correct base branch (develop, or main for release PRs)
  • The PR is assigned to the developer
  • Appropriate labels are applied
  • The PR is included in relevant projects and/or milestones
  • Description clearly explains what has changed
  • Issue references included, if applicable
  • Code and documentation adhere to current standards (ruff)
  • Documentation updates included, if relevant
  • CI tests are passing
  • All reviewer comments have been addressed

Next Steps / Notes (if applicable)

Any follow-up actions, known issues, or reminders for maintainers.

@jeipollack jeipollack self-assigned this Aug 18, 2026
@jeipollack
jeipollack force-pushed the 240-architecture-implement-quality-control-pipeline-orchestration branch from 89a896b to ae8b21b Compare August 18, 2026 14:13
@jeipollack
jeipollack force-pushed the 240-architecture-implement-quality-control-pipeline-orchestration branch 2 times, most recently from 1c43b37 to b64ea75 Compare August 21, 2026 09:52
Jennifer Pollack added 3 commits August 24, 2026 18:30
- Replace the hard-coded threshold field with a generic policy mapping
- Update RejectionPolicyConfig to support policy-specific parameters
- Update parse_rejection_policy_config()
- Update the YAML configuration examples and fixtures
- Update rejection-policy tests
- Use backticks in error messages to highlight configuration parameter names
- Initialize the MetricsRegistry in the pipeline constructor
- Instantiate enabled quality metric implementations from configuration
- Add unit tests for valid and invalid metric configurations
@jeipollack
jeipollack force-pushed the 240-architecture-implement-quality-control-pipeline-orchestration branch from b64ea75 to 34b408d Compare August 25, 2026 15:19
Jennifer Pollack added 4 commits August 25, 2026 17:24
- Add helpers to instantiate enabled metrics and rejection policies
- Build the processing sequence for resource resolution, metric computation, and policy application
- Add unit and integration tests for the orchestration
- Clean up whitespace in the quality control configuration fixture
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant