Skip to content

Fix job list sorting to use numeric order#1611

Closed
aliasaria wants to merge 5 commits intomainfrom
fix/job-list-sort-order
Closed

Fix job list sorting to use numeric order#1611
aliasaria wants to merge 5 commits intomainfrom
fix/job-list-sort-order

Conversation

@aliasaria
Copy link
Member

Summary

  • Jobs with IDs > 100 were appearing after single-digit jobs because get_jobs() returned results in index order without numeric sorting
  • Added numeric sort (descending, newest first) in Experiment.get_jobs() in the SDK
  • Bumped lab-sdk version from 0.1.4 to 0.1.5

Test plan

  • Verify jobs with IDs > 100 appear above lower-numbered jobs in the Tasks list
  • Verify sweep jobs still appear correctly

Jobs with IDs > 100 were appearing after single-digit jobs because
get_jobs() returned results in index order without numeric sorting.
Sort results by job ID descending (newest first) in the SDK.

Bump lab-sdk version to 0.1.5.
@paragon-review
Copy link

Paragon Summary

This pull request review identified 1 issue across 1 category in 3 files. The review analyzed code changes, potential bugs, security vulnerabilities, performance issues, and code quality concerns using automated analysis tools.

This PR fixes a sorting bug where jobs with IDs greater than 100 incorrectly appeared after single-digit jobs in the Tasks list by adding numeric descending sort in the SDK's get_jobs() method. The lab-sdk version is bumped to 0.1.5 to ship the fix.

Key changes:

  • Fixed job list sorting bug where IDs > 100 appeared after single-digit IDs
  • Added numeric descending sort (newest first) in Experiment.get_jobs() SDK method
  • Bumped lab-sdk version from 0.1.4 to 0.1.5
  • Changed files: experiment.py, local.py, pyproject.toml

Confidence score: 4/5

  • This PR has low-moderate risk with 1 medium-priority issue identified
  • Score reflects code quality concerns and maintainability issues
  • Consider addressing medium-priority findings to improve code quality

3 files reviewed, 1 comment

Severity breakdown: Medium: 1


Tip: @paragon-run <instructions> to chat with our agent or push fixes!

Dashboard

@sentry
Copy link

sentry bot commented Mar 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@aliasaria
Copy link
Member Author

No point in merging this if Deep is converting to short ids

@aliasaria aliasaria closed this Mar 24, 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.

3 participants