Skip to content
This repository has been archived by the owner on Dec 10, 2024. It is now read-only.

Bugfix: Pickers: Remove api picker slowdown #114

Merged
merged 1 commit into from
Oct 31, 2024

Conversation

iopapamanoglou
Copy link
Contributor

@iopapamanoglou iopapamanoglou commented Oct 31, 2024

Bugfix: Pickers: Remove api picker slowdown

Description

Remove unnecessary double parameter check
Remove check for all candidates for compat (use shortcircuiting)

Fixes #111

Checklist

Please read and execute the following:

  • My code follows the coding guidelines of this project
  • My PR title is following the contribution guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I ran Black to format my code

Code of Conduct

By submitting this issue, you agree to follow our Code of Conduct:

Remove unnecessary double parameter check
Remove check for all candidates for compat (use shortcircuiting)
@iopapamanoglou iopapamanoglou mentioned this pull request Oct 31, 2024
5 tasks
@iopapamanoglou iopapamanoglou merged commit db0387e into main Oct 31, 2024
6 checks passed
@iopapamanoglou iopapamanoglou deleted the bugfix/remove_api_picker_slowdown branch October 31, 2024 08:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Accelerate test cases
1 participant