@emilykl@gvwilson In my other PR the baseline tests seem to be working again, maybe we can trigger this one's action as well to see if it works?

@my-tien I did re-trigger the tests and it's still failing -- I think it will need my PR#7418 to be merged but I'm just working through some final issues with that one, hopefully will be resolved by end of day.

@my-tien I just merged#7418 which fixes theModuleNotFoundError: No module named 'plotly' error in themake-baselines CI step.

You should mergemainmaster into this branch, it will resolve some of the CI issues.

@my-tien Just making sure I understand the effect of this change — can you confirm whether the below is correct?

Previously, if you settickson='boundaries' and (ticklabelposition='inside' orticklabelposition='outside'),ticklabelposition had no effect. This PR makes it sotickson='boundaries' and (ticklabelposition='inside' orticklabelposition='outside') are both respected. Effectively this only makes a difference whenticklabelposition='inside', since'outside' is the default anyway.

@my-tien Just making sure I understand the effect of this change — can you confirm whether the below is correct?

Previously, if you settickson='boundaries' and (ticklabelposition='inside' orticklabelposition='outside'),ticklabelposition had no effect. This PR makes it sotickson='boundaries' and (ticklabelposition='inside' orticklabelposition='outside') are both respected. Effectively this only makes a difference whenticklabelposition='inside', since'outside' is the default anyway.

Yes, exactly!

Am I right to assume that the failing tests require no action from my side?

FYI:

@my-tien TheMissing X server error is definitely unrelated to this PR.

The error in the warning-counting test also seems unrelated, although I'm not sure of the root cause.@alexcjohnson Have you ever run into failures of this test before?

@my-tien Tests are passing now and I'm happy merging this as-is (unless@archmoj has any final comments?)

Thank you for the PR!

Thx a lot@emilykl and@archmoj 😸