Skip to content

Add test_keys_check for Select#275

Merged
ValerianRey merged 3 commits intomainfrom
improve-select-testing
Mar 26, 2025
Merged

Add test_keys_check for Select#275
ValerianRey merged 3 commits intomainfrom
improve-select-testing

Conversation

@ValerianRey
Copy link
Copy Markdown
Contributor

No description provided.

@ValerianRey ValerianRey added cc: test Conventional commit type for changes to tests. package: autojac labels Mar 26, 2025
@ValerianRey ValerianRey self-assigned this Mar 26, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 26, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

see 2 files with indirect coverage changes

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

@ValerianRey ValerianRey force-pushed the improve-select-testing branch from f66f1ae to 618ccad Compare March 26, 2025 01:15
@ValerianRey ValerianRey force-pushed the improve-select-testing branch from 618ccad to 82fcc38 Compare March 26, 2025 01:17
Copy link
Copy Markdown
Contributor

@PierreQuinton PierreQuinton left a comment

Choose a reason for hiding this comment

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

Otherwise, LGTM.

keys.
"""

all_keys = [torch.tensor(i) for i in range(10)]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why not range(2)?

@ValerianRey ValerianRey merged commit a2e77c7 into main Mar 26, 2025
14 checks passed
@ValerianRey ValerianRey deleted the improve-select-testing branch March 26, 2025 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cc: test Conventional commit type for changes to tests. package: autojac

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants