Skip to content

Conversation

@devmotion
Copy link
Member

Fixes #1344.

@isaacsas
Copy link
Member

isaacsas commented Dec 1, 2025

@devmotion thanks! Looks like there is some kind of CI issue currently. I'll make a release with the current master as this isn't a critical issue, and then we can make another release once this is merged. That way we stop holding people up from using Makie.

@devmotion
Copy link
Member Author

Hmmm... These CI errors seem very weird and completely unrelated: https://github.com/SciML/Catalyst.jl/actions/runs/19835077827/job/56830272189?pr=1345#step:6:449 Maybe it's a general GHA problem right now? In that case, CI would fail on master as well in the same way.

So I'm not sure if it's worth rushing a new release, I think most likely these tests will pass went they are rerun.

@devmotion
Copy link
Member Author

In particular, given that it took 3 weeks until #1337 could be merged, I don't think a few hours more will matter.

@devmotion
Copy link
Member Author

The relevant extension test set already passes, so the PR doesn't seem to cause any test failures: https://github.com/SciML/Catalyst.jl/actions/runs/19835077827/job/56830272071?pr=1345 The docs build should show whether it works as intended.

@isaacsas
Copy link
Member

isaacsas commented Dec 1, 2025

Yeah, they are quite weird errors. In any case I already started the release, but once this is merged I'll just make another. I don't think it is a big deal to lose the coloring in one (hopefully) short-lived release.

@devmotion devmotion marked this pull request as ready for review December 1, 2025 22:02
@isaacsas isaacsas merged commit 74cfd21 into master Dec 1, 2025
25 of 32 checks passed
@isaacsas
Copy link
Member

isaacsas commented Dec 1, 2025

Thanks!

@devmotion devmotion deleted the dmw/plot_complexes_edgecolors branch December 1, 2025 22:45
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.

Fix graph complex plotting color dropping

3 participants