Skip to content

Release/v1.6.0 - #123

Merged
elewis2 merged 78 commits into
mainfrom
release/v1.6.0
Oct 2, 2026
Merged

elewis2 merged 78 commits into
mainfrom
release/v1.6.0

Conversation

@elewis2

@elewis2 elewis2 commented Sep 28, 2026 •

Copy link
Copy Markdown
Collaborator

Release v1.6.0

New feature:

  • improve GraphKB caching and add optional rate limiting
    Replaced ad-hoc in-process caches (QUERY_CACHE, FEATURES_CACHE, per-connection term cache) with HTTP-level caching via requests-cache.
    Rate limiting is built on requests-ratelimiter's LimiterAdapter and is opt-in (off by default). Can be combined with caching or used standalone for scripts that only need rate limiting.
    New GraphKBConnection options: cache_name, only_if_cached, custom session, limiter_kwargs; per-request ignore_cache/force_refresh headers.

Improvements:

  • split github actions tests into a fast pass and a full pass (on Python 3.11)
  • DEVSU-3013: create_key_alterations now returns structured {variantType, variant} objects (keyed by the observed variant's key) instead of hardcoded display-name strings, so the IPR API can associate key alterations with their originating variants at report-creation time.
  • SDEV-5403: added HRD kbCategory example values to content.spec.json and expanded HRD annotation tests.
  • DEVSU-3011 (pori_python/ipr/summary.py): fixed get_ipr_analyst_comments project matching to use the new projects (list, many-to-many) field instead of the old single project field, following an IPR API schema change; added integration test suite validating the mock summary output against the live schema.

Bug fixes:

  • DEVSU-3023 (pori_python/ipr/main.py): fixed async report uploads dumping the entire nested job/report payload into the result. Async uploads now return just ident, matching the shape of sync results (which return ident + message).
  • DEVSU-2990 (pori_python/ipr/summary.py): variant-text section headers changed from 'h2' to 'h3' for consistent font sizing, and cancer types in the header are now comma-space separated.

creisle and others added 30 commits February 9, 2026 12:17
…een calls so I split the tests into two functions
Merge back main into develop
Merge branch 'feature/replace-caching' of https://github.com/bcgsc/pori_python into feature/replace-caching
- Update get_ipr_analyst_comments method with 'projects' key in queried variant-text being a list instead of single value (since projects and variant texts now have many-to-many relationship bcgsc/pori_ipr_api#499)
- Update project name matching logic to new projects data structure
@codecov-commenter

codecov-commenter commented Sep 28, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.00000% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.86%. Comparing base (da00821) to head (091d7d0).

Files with missing lines Patch % Lines
pori_python/graphkb/util.py 84.12% 10 Missing ⚠️
pori_python/ipr/main.py 88.88% 2 Missing ⚠️
pori_python/ipr/ipr.py 91.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #123      +/-   ##
==========================================
+ Coverage   84.12%   87.86%   +3.73%     
==========================================
  Files          18       18              
  Lines        2608     2636      +28     
==========================================
+ Hits         2194     2316     +122     
+ Misses        414      320      -94     
Flag Coverage Δ
unittests 87.86% <87.00%> (+3.73%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread setup.cfg Outdated
@elewis2
elewis2 merged commit d57a7de into main Oct 2, 2026
5 checks passed
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.

6 participants