mapnik / node-mapnik

Bindings to mapnik for node.js
http://mapnik.org/documentation/node-mapnik
BSD 3-Clause "New" or "Revised" License
531 stars 165 forks source link

upgrade to latest mapnik master #975

Closed mathisloge closed 2 years ago

mathisloge commented 3 years ago

hi @artemp

do you plan to integrate the changes from the proj6 update or to update to the latest mapnik master?

artemp commented 3 years ago

@mathisloge can you give a bit more context here, please? node-mapnik works with latest mapnik@master, no?

mathisloge commented 3 years ago

Mostly. With the proj6 branch i think the proj_transform->source / dest functions were removed. So all logs like https://github.com/mapnik/node-mapnik/blob/f83bff5176c32a3c7819e5788e13ebcc3deed874/src/mapnik_projection.cpp#L318 are failing.

But these are just four lines which are invalid (only in mapnik_projection.cpp)

artemp commented 3 years ago

@mathisloge I see, thanks for highlighting this. I have local proj6 branch with all the fixes but never pushed, doh.. I'll need to update to latest master and to push proj6 branch, :+1:

mathisloge commented 2 years ago

@artemp if you have some spare time, could you push your proj6 branch? I would like to run the tests in https://github.com/mapnik/node-mapnik/pull/976 . Most of them are failing due to proj init grammar changes.

artemp commented 2 years ago

@mathisloge - I just pushed my local branch https://github.com/mapnik/node-mapnik/tree/proj6. Let me know if it works for you, cheers.

mathisloge commented 2 years ago

Thanks for pushing!

I'm still getting some erros with:

proj_create: init=epsg:/init=IGNF: syntax not supported in non-PROJ4 emulation mode
D:\dev\node-mapnik\test\vector-tile.composite.test.js:300
  vtile.composite([vtile2],{reencode:true});
        ^

TypeError: failed to initialize projection with: '+init=epsg:3857'
    at Test.<anonymous> (D:\dev\node-mapnik\test\vector-tile.composite.test.js:300:9)
    at Test.bound [as _cb] (D:\dev\node-mapnik\node_modules\tape\lib\test.js:88:32)
    at Test.run (D:\dev\node-mapnik\node_modules\tape\lib\test.js:105:10)
    at Test.bound [as run] (D:\dev\node-mapnik\node_modules\tape\lib\test.js:88:32)
    at Immediate.next (D:\dev\node-mapnik\node_modules\tape\lib\results.js:82:19)
artemp commented 2 years ago

Thanks for pushing!

I'm still getting some erros with:

proj_create: init=epsg:/init=IGNF: syntax not supported in non-PROJ4 emulation mode
D:\dev\node-mapnik\test\vector-tile.composite.test.js:300
  vtile.composite([vtile2],{reencode:true});
        ^

TypeError: failed to initialize projection with: '+init=epsg:3857'
    at Test.<anonymous> (D:\dev\node-mapnik\test\vector-tile.composite.test.js:300:9)
    at Test.bound [as _cb] (D:\dev\node-mapnik\node_modules\tape\lib\test.js:88:32)
    at Test.run (D:\dev\node-mapnik\node_modules\tape\lib\test.js:105:10)
    at Test.bound [as run] (D:\dev\node-mapnik\node_modules\tape\lib\test.js:88:32)
    at Immediate.next (D:\dev\node-mapnik\node_modules\tape\lib\results.js:82:19)

@mathisloge yep, proj6 branch needs updated mapnik-vector-tile https://github.com/mapbox/mapnik-vector-tile I just pushed https://github.com/mapbox/mapnik-vector-tile/tree/proj6 <---- It might need some tweaking but should work

mathisloge commented 2 years ago

@artemp thanks! Now all tests are running 👍