Skip to content

Conversation

@Adam-Ladd
Copy link

My previous commit added the ability to set alpha for contours. However, this caused a couple of the existing tests to fail as alpha=1 is added to the opt String. This commit fixes that as default (alpha=1 value is now added to the test opt string`

P.S. the test test_canvas_glyph_3d_and_hide_3d_grid is currently failing. However this fails on previous commits so and is unrelated to my contributions.

@cpmech
Copy link
Owner

cpmech commented Jan 17, 2026

Hi, I've just published a new version (1.20.0) with the ability to set alpha in Contour. I've added two functions: set_fill_alpha and set_line_alpha to set the transparency for the filled Contour and line contour, respectively. I've also fixed the tests.

On my end, and in the GitHub actions, test_canvas_glyph_3d_and_hide_3d_grid seems fine. If it is failing in your system, can you please check line # 561 of the test_canvas.rs file with the following check:

assert!(n > 850 && n < 910);

Depending on the Python and/or Matplotlib version and the system, such a range may vary slightly.

@Adam-Ladd
Copy link
Author

Grand, sorry for the slightly sloppy PR.

On my machine test_canvas_glyph_3d_and_hide_3d_grid is failing with a line count of 974 my development environment is a fedora container with python 3.14.2 and matplotlib 3.10.8

@cpmech
Copy link
Owner

cpmech commented Jan 17, 2026

Ok, I've increased the lower and upper limit (1000) in the main branch. So, if you update your fork, it should work now for you. Thanks again.

@Adam-Ladd Adam-Ladd closed this Jan 18, 2026
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