Skip to content

Refactor/curator app maintainability#82

Open
RonaldRonnie wants to merge 3 commits intomainfrom
refactor/curator-app-maintainability
Open

Refactor/curator app maintainability#82
RonaldRonnie wants to merge 3 commits intomainfrom
refactor/curator-app-maintainability

Conversation

@RonaldRonnie
Copy link
Collaborator

  • Config: single CONFIG dict with FEEDBACK_DIR, USER, BIOANALYZER_VERSION from env
  • Schema: OPTIONS dict for valid_states/col_feedback/true_label; dynamic _feedback_schema()
  • Data: unified _load_data(path_or_upload, is_path) with BytesIO for uploads
  • Helpers: _safe_int, _normalize_status, _priority_score (comprehension); PMID link inline
  • Normalization: assign chains, try/except for PMID and year parsing
  • Feedback: load_feedback/save_feedback/upsert_feedback use _feedback_schema(); single loop for row pred/true/col_feedback
  • UI: render_filters dict comp for status filters; display_cols from single want list; compact_cols list comp
  • Robustness: logging for load/save/errors; PMID validation warning if not in dataset; explicit TypeError/ValueError in helpers
  • Removed: OPTIONAL_COLUMNS (unused); redundant docstrings; duplicate loops"

- Config: single CONFIG dict with FEEDBACK_DIR, USER, BIOANALYZER_VERSION from env
- Schema: OPTIONS dict for valid_states/col_feedback/true_label; dynamic _feedback_schema()
- Data: unified _load_data(path_or_upload, is_path) with BytesIO for uploads
- Helpers: _safe_int, _normalize_status, _priority_score (comprehension); PMID link inline
- Normalization: assign chains, try/except for PMID and year parsing
- Feedback: load_feedback/save_feedback/upsert_feedback use _feedback_schema(); single loop for row pred/true/col_feedback
- UI: render_filters dict comp for status filters; display_cols from single want list; compact_cols list comp
- Robustness: logging for load/save/errors; PMID validation warning if not in dataset; explicit TypeError/ValueError in helpers
- Removed: OPTIONAL_COLUMNS (unused); redundant docstrings; duplicate loops
@RonaldRonnie RonaldRonnie self-assigned this Mar 2, 2026
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