Skip to content

[WIP] TY support - #7836

Closed
Jens Hedegaard Nielsen (jenshnielsen) wants to merge 20 commits into
microsoft:mainfrom
jenshnielsen:ty_support_2025
Closed

[WIP] TY support#7836
Jens Hedegaard Nielsen (jenshnielsen) wants to merge 20 commits into
microsoft:mainfrom
jenshnielsen:ty_support_2025

Conversation

@jenshnielsen

Copy link
Copy Markdown
Collaborator

This is a branch of various fixes to try to enable ty support for type checking. The aim is not to land this branch directly but to cherrypick thigs from it. Along with improvments to ty that will hopefully get things working along the way

@codecov

codecovBot commented Feb 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 63.63636% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.88%. Comparing base (eec7057) to head (193d9fa).

Files with missing linesPatch %Lines
src/qcodes/instrument/channel.py70.37%8 Missing ⚠️
src/qcodes/dataset/dond/do_nd.py33.33%2 Missing ⚠️
src/qcodes/logger/instrument_logger.py50.00%2 Missing ⚠️
src/qcodes/instrument/mockers/ami430.py0.00%1 Missing ⚠️
...codes/instrument_drivers/AlazarTech/dll_wrapper.py0.00%1 Missing ⚠️
src/qcodes/parameters/parameter_base.py50.00%1 Missing ⚠️
src/qcodes/parameters/sequence_helpers.py50.00%1 Missing ⚠️
Additional details and impacted files
@@ Coverage Diff @@## main #7836 +/- ##
==========================================
- Coverage 59.87% 58.88% -1.00% 
==========================================
Files 352 352 Lines 31872 31884 +12 ==========================================
- Hits 19083 18774 -309 - Misses 12789 13110 +321 

☔ 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.

This is a breaking change since root_instrument is now abstract on InstrumentBase and must be implemented in a subclass
@jenshnielsen

Copy link
Copy Markdown
CollaboratorAuthor

replaced by #8441 8441

Sign up for freeto 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.

1 participant

@jenshnielsen