Fix DMS not set on some projections

[cvanelteren]

Addresses #292 where DMS is not set for some axes that should have it set.

[cvanelteren]

Some baselines will probable fail (but maybe not). Will add a test.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

[Copilot]

Pull Request Overview

This PR ensures degree-minute-second (DMS) formatting is consistently applied to projections that require it.

• Added a test to verify DMS formatting on the Mercator projection.
• Removed explicit gridline formatter overrides so axis formatters can propagate DMS formatting.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
ultraplot/tests/test_geographic.py	Add test_dms_used_for_mercator to validate DMS formatting
ultraplot/axes/geo.py	Remove explicit gl.xformatter/gl.yformatter assignments

Comments suppressed due to low confidence (2)

ultraplot/tests/test_geographic.py:864

• The new test only verifies longitude formatting (xformatter). To fully cover DMS behavior, add assertions for latitude formatting using gridlines_major.yformatter on the lat ticks.

        a = ax[0].gridlines_major.xformatter(tick)

ultraplot/axes/geo.py:1708

• By removing the explicit assignment of gl.xformatter and gl.yformatter, gridlines may no longer pick up the DMS formatter from the axis. Consider reintroducing these assignments (after the axis formatter is configured) so gridlines use the correct DMS formatter.

        )

[beckermr]

This change confuses me. There were no cases where the code that's been removed was needed?

[cvanelteren]

Actually you are right, I thought I added this but that's not true will do some more searching why it is not working.

[cvanelteren]

It's a bit strange when I edit line 244 and remove the merc projector it is set to false and it works as it should for the mercator projection but it does not make sense since use_dms will be set to false.

[cvanelteren]

I will wait before the issuer gives some clarity. The PR now only contains a test.