Conversation

emilykl

Image tests on some of the map traces are failing, apparently because Carto made some minor changes to their basemaps. These changes don't seem to be related to anything in our own codebase.

This PR updates the baseline images to match the new basemaps.

This PR also updates the font download URL for Gravitas One in the CI script, which changed due to expo/google-fonts#136. This fixes the other failing image tests. Also updates the CI script such that an error is raised if the font file fails to download, which will make future debugging much easier.

@emilykl

I've looked at all the image diffs; the only significant change is that (apparently) ocean names are no longer displayed on the basemaps. All of the other differences are minor or invisible.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this!

@archmoj

Wondering if other failing image tests

    "various_geo_projections",
    "smith_template",
    "geo_text_chart_arrays",
    "font-wishlist",
    "canada_geo_projections"

are related to #7359?

@gvwilsongvwilson added P1needed for current cyclefixfixes something brokenlabels Mar 31, 2025
@emilyklemilykl changed the title update map baselines Update map baselines and fix failing font download Mar 31, 2025
@emilykl

@archmoj I just figured out the source of those failing tests, it's because the Gravitas One font URL changed so the font download failed (see my updated PR comment).

@emilyklemilykl merged commit 40f2221 into master Mar 31, 2025
1 check passed
@emilyklemilykl deleted the update-map-baselines branch March 31, 2025 20:14
@@ -82,7 +89,7 @@ def download(repo, family, types) :
)

download(
'https://.com/expo/google-fonts/blob/master/font-packages/gravitas-one/',
'https://.com/expo/google-fonts/blob/main/font-packages/gravitas-one/400Regular/',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯 🙏 Many thanks @emilykl
Switching from master to main broke these.
I suspect a similar problem would occur again.
I feel to avoid file changes (that occur on master/main branch of these repositories) it would be best to use permanent links for all these font files e.g.
https://.com/expo/google-fonts/tree/cc65b43125023a2ad696eeec6ec10763c35ff588/font-packages/gravitas-one/400Regular instead of https://.com/expo/google-fonts/blob/main/font-packages/gravitas-one/400Regular/.
Thank you!

Sign up for free to join this conversation on . Already have an account? Sign in to comment
fixfixes something brokenP1needed for current cycle
None yet

Successfully merging this pull request may close these issues.