Skip to content

Fix tests#420

Draft
AreWeDreaming wants to merge 1 commit into
masterfrom
fix_regression
Draft

Fix tests#420
AreWeDreaming wants to merge 1 commit into
masterfrom
fix_regression

Conversation

@AreWeDreaming

Copy link
Copy Markdown
Collaborator

Pins + fixes to get regression to pass again on master.

@AreWeDreaming AreWeDreaming marked this pull request as draft June 17, 2026 00:54
@AreWeDreaming

Copy link
Copy Markdown
Collaborator Author

I cannot reproduce this error locally.

@AreWeDreaming

Copy link
Copy Markdown
Collaborator Author

Using pip I can reproduce this!

@AreWeDreaming

Copy link
Copy Markdown
Collaborator Author

While I can reproduce this now the error is very confusing even with the debugger. The floating point colors come from the matplotlib.rcparams. The strangest part is that if I test this with same matplotlib version but in a pure conda environment then I don't get this bug.
Since our primary use-case for OMAS is to map DIII-D data which needs MDSplus, I'd say we change the CI/CD to install the environment through conda. This has some downstream consequences since omas is distributed on PyPi and then propagated to conda from there (as far as I remember).

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.

1 participant