Skip to content

Conversation

@zmoon
Copy link
Owner

@zmoon zmoon commented Jan 20, 2026

Initial version. May later change the data structure of the returns (e.g. to include more info).

Towards #66 but doesn't implement everything discussed there (tags, etc.)

@codecov
Copy link

codecov bot commented Jan 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.26%. Comparing base (86c23ad) to head (6b5c1f3).
⚠️ Report is 16 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
+ Coverage   94.07%   94.26%   +0.19%     
==========================================
  Files          15       15              
  Lines        1687     1745      +58     
==========================================
+ Hits         1587     1645      +58     
  Misses        100      100              

☔ View full report in Codecov by Sentry.
📢 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.

@zmoon zmoon marked this pull request as ready for review January 20, 2026 16:27
@zmoon zmoon requested a review from Copilot January 20, 2026 16:27
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds functionality to load sets data and tune collections data from The Session web API. The changes enable fetching information about which collections contain a specific tune, retrieving individual member sets, and fetching all sets for a member with pagination support and multi-threading capabilities.

Changes:

  • Added a generic _consume function to handle paginated API endpoints with configurable multi-threading
  • Added get_tune_collections to retrieve collection information for a specific tune
  • Added get_member_set and get_member_sets to retrieve member set data with pagination support
  • Added comprehensive test coverage for the new API functions

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 10 comments.

File Description
pyabc2/sources/the_session.py Implements core API consumption logic with _consume and three new public functions for retrieving tune collections and member sets data
tests/test_sources.py Adds six new test functions covering the new API functions, including validation tests and pagination scenarios

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zmoon zmoon merged commit e2f8378 into main Jan 20, 2026
9 checks passed
@zmoon zmoon deleted the more-sesh branch January 20, 2026 17:33
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.

2 participants