Page MenuHomePhabricator

Update to latest maki
Open, Stalled, Needs TriagePublic


although the documentation says: maki icons will be supported as markers, there are some maki icons not supported.

e.g. mountain, which is essential as there are many articles on mountains and this is the natural icon, I guess.
(same for volcano, stadium, castle)

See for the gaps. (generated with <maplink> iterating over all the icons)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 12 2016, 10:58 PM
MaxSem renamed this task from Not all maki icons supported to Update to latest maki.Sep 12 2016, 11:00 PM
Herzi.Pinki updated the task description. (Show Details)Sep 12 2016, 11:01 PM
MaxSem added a subscriber: MaxSem.

makizushi still uses maki 0.5.0, so we need to either contribute a patch upstream or fork (this is much more complicated than just updating version number due to massive changes in maki 1.0).

Yurik moved this task from Unsorted to General on the Maps (Kartographer) board.Sep 15 2016, 5:06 PM
Restricted Application added a project: Discovery. · View Herald TranscriptSep 16 2016, 7:25 AM
Yurik removed a project: Maps.Dec 15 2016, 4:39 AM
MSantos changed the task status from Open to Stalled.Sep 18 2018, 10:07 PM
MSantos added a subscriber: MSantos.

It seems like makizushi is only worried about mapnik upgrades and it's becoming deprecated. Therefore, makiwich is going to be the successor map marker creator.

With the current stack is not possible to have the latest maki running, unless we migrate to libraries that use makiwich. That is also a problem because we have third-party libraries relying on makizushi, like geojson-mapnikify which is a dependency for tilelive-overlay, and they are following the same path as makizushi.

It's a considerable change of codebase that we are not able to work right now.

mxn added a subscriber: mxn.Nov 10 2018, 10:00 PM
TheDJ added a subscriber: TheDJ.Mar 24 2020, 12:44 PM

This might be easier to do now that npm has aliases perhaps ?