Pdok 18835/vervanging trex v2 attributes - #43
Conversation
RoelvandenBerg
left a comment
RoelvandenBerg
left a comment
There was a problem hiding this comment.
Er gebeurt een hoop. Waaronder refactoring. Ik denk dat het goed is om in de PR goed uit te leggen waarom je doet wat je doet. Dan is beter te volgen wat er gebeurd. Ik ben gestopt na een comments. Kan ook mondeling.
| quadrantSlice := make([]Quadrant, (maxY-minY)*(maxX-minX)) | ||
| for i := range maxX - minX { | ||
| for j := range maxY - minY { | ||
| quadrantSlice := make([]Quadrant, (maxY-minY+1)*(maxX-minX+1)) |
There was a problem hiding this comment.
leest als een fix. heeft dit ook effect op andere plekken?
There was a problem hiding this comment.
Dit was een bug in een functie die ik zelf heb toegevoegd in de eerste PR. Na de fix realiseerde ik me dat het netter was om dit in de andere branch te doen. (Deze functie is overigens een placeholder en moet nog worden vervangen.)
| handle *gpkg.Handle | ||
| } | ||
| // Custom wrapper for gpkg.Handle. To be embedded in other structs | ||
| type geopackageHandle struct{ handle *gpkg.Handle } |
There was a problem hiding this comment.
Waarom hernoem je dit?
There was a problem hiding this comment.
Dit is voor code deduplicatie en om de "snap" en "mvt" commando's beter te scheiden qua afhankelijkheden.
2 participants
Description
Add command
texel mvtthat generates tiles from output oftexel(now also available astexel snap) with-encoption enabled (said option is available in base branch). New command generates tiles structure in target folder.List of changes (does not align with commits):
Changes for feature:
assemble.go. Improves ontegola/go-spatialgpkgmvt.gomvtflow ininterface.go(similar to existing)processingmvt.gomain.go(includes initialization step).Refactors:
gpkg.go, snap-specific and mvt-specific ingpkgsnap.goandgpkgmvt.go(encoded.gogone).go_spatial_encoding.-encenabled and not.Tests:
processGeometryinprocessing_test.goassemble_test.goFixes:
-encGeometriesflag properly usedType of change
Checklist: