Skip to content

Wire the confidence facets together - #78

Merged
edkerk merged 1 commit into
developfrom
feat/confidence-wire-facets
Jul 18, 2026
Merged

Wire the confidence facets together#78
edkerk merged 1 commit into
developfrom
feat/confidence-wire-facets

Conversation

@edkerk

@edkerk edkerk commented Jul 18, 2026

Copy link
Copy Markdown
Member

Wires the three confidence facets into a usable pipeline, so the score → review → curate loop works end to end.

Changes

  • annotate_confidence(model, *, proposal=None, scores=None, facets=None) — one call runs every applicable scorer and returns {facet: reactions_scored}. equation and gene_association need only the model; localization runs only when both proposal and scores are given, and is otherwise skipped rather than failing. facets=[...] restricts the set.
  • curation_priority skips curated placements — a reaction whose localization facet is mark_curated drops from the review queue (new include_curated=False); include_curated=True keeps it.
  • The no-SBO-terms warning names its remedyraven_toolbox.annotation.add_sbo_terms(model).

Base automatically changed from fix/delta-g-missing-sentinel to develop July 18, 2026 21:54
@edkerk
edkerk force-pushed the feat/confidence-wire-facets branch from 1023acc to cb9e529 Compare July 18, 2026 22:07
@edkerk edkerk changed the title Wire the confidence facets together (§10.1) Wire the confidence facets together Jul 18, 2026
@edkerk
edkerk merged commit f7a41b7 into develop Jul 18, 2026
8 checks passed
@edkerk
edkerk deleted the feat/confidence-wire-facets branch July 18, 2026 22:11
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