Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The SDF icons seem to lack antialiasing #80

Open
bchapuis opened this issue Feb 27, 2024 · 3 comments
Open

The SDF icons seem to lack antialiasing #80

bchapuis opened this issue Feb 27, 2024 · 3 comments

Comments

@bchapuis
Copy link

In the following example, the icons generated with spritezero seem to apply some antialiasing whereas the spreet one does not:

spritezero

spreet

The svg file used to generate the veterinary icon is located here:

https://github.com/apache/incubator-baremaps/blob/main/basemap/assets/icons/veterinary.svg

The following README contains instructions to generate a sprite that contains most openstreetmap-carto icons:

https://github.com/apache/incubator-baremaps/blob/main/basemap/assets/README.md

All the icons seem to be affected by this issue.

@Shikakka
Copy link

Shikakka commented Mar 25, 2024

The problem seems to be the lack of "blurring" the svg.

This is the sprite generated by Elastic/Spritezero
image

And the sprite generated by Spreet:
image

It lacks the "blur" that causes Mapbox to render it without any antialiasing.

@CommanderStorm
Copy link
Contributor

CommanderStorm commented Oct 20, 2024

And the sprite generated by Spreet

@Shikakka just to make sure. Have you used

Your output suspiciously does not quite look like an sdf to me..

@jessekrubin
Copy link

I am experiencing the same problem, however I am using deckgl to render the sdf icons.

I also see adjacent icons bleeding into each other (might be a deck problem) -- for this I think buffering/bordering each icon by 1-2 pixels would fix my issue. How involved would it be to add a cli flag for bordering icons? (can pr if I find time)

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

No branches or pull requests

4 participants