-
Notifications
You must be signed in to change notification settings - Fork 339
Draft: Improve performance of is_function #7669
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
@eendebakpt please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
|
Thanks @eendebakpt That seems very useful. In the long term I think we should consider completely removing and simplifying the command class from parameter but until then this seems like a nice improvement. I did not look into why the tests doesn't like this as is |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7669 +/- ##
==========================================
- Coverage 59.78% 59.77% -0.02%
==========================================
Files 352 352
Lines 31659 31671 +12
==========================================
+ Hits 18927 18931 +4
- Misses 12732 12740 +8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
For instruments with many parameters (100 - 1000 range) the overhead of qcodes parameters becomes a bottleneck in startup time. In this PR we improve performance of
is_function:Testing code:
(there are more bottlenecks, but
is_fuctionis used in creation of every Parameter and is quite heavy)