Skip to content

Declare the platforms supported by the instrument plugins - #11

Merged
seb5g merged 2 commits into
mainfrom
feature/supported-platforms
Nov 4, 2026
Merged

seb5g merged 2 commits into
mainfrom
feature/supported-platforms

Conversation

@seb5g

@seb5g seb5g commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

Before:
Nothing told PyMoDAQ that the Omnidriver viewer only works on Windows (OmniDriver .NET driver loaded with clr).

After:
DAQ_1DViewer_Omnidriver declares supported_platforms = ('win32',): PyMoDAQ refuses to initialize it elsewhere, with a message. The Seabreeze viewer stays undeclared, so the package classifiers are unchanged.

How: one class attribute (PyMoDAQ >= 5.3.5 reads it, older versions ignore it).

Tested: file compiles. Not tested: initialization on Windows or elsewhere with a real spectrometer; the declaration comes from the code reading.


Generated by Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Tdiso1u1ZhikSutj2Qs5Cg
… template

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Tdiso1u1ZhikSutj2Qs5Cg
@seb5g
seb5g merged commit 3dbc60c into main Nov 4, 2026
7 checks passed
@seb5g
seb5g deleted the feature/supported-platforms branch November 4, 2026 15:52
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.

2 participants