mapbox / mapbox/mts-data-sync

Running estimate ends in error: TypeError: Cannot read property '1' of undefined

Open
#15 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
71
Forks
15
PR merge metrics
No merged PRs in 30d

Description

I'm following the documentation to create my first tileset.

I've successfully run: mtsds --config path-to-geojson and got the expected config, recipe and .jsonl files.

I get this error when running mtsds --estimate as documented:

Estimating job size... this could take a while depending on your max zoom
TypeError: Cannot read property '1' of undefined
    at lineCover (/Users/seanrennie/Development/HUMMINGBIRD/mts-data-sync/node_modules/@mapbox/tile-cover/index.js:231:30)
    at polygonCover (/Users/seanrennie/Development/HUMMINGBIRD/mts-data-sync/node_modules/@mapbox/tile-cover/index.js:154:9)
    at getTiles (/Users/seanrennie/Development/HUMMINGBIRD/mts-data-sync/node_modules/@mapbox/tile-cover/index.js:79:9)
    at Proxy.exports.indexes (/Users/seanrennie/Development/HUMMINGBIRD/mts-data-sync/node_modules/@mapbox/tile-cover/index.js:53:12)
    at DestroyableTransform.<anonymous> (/Users/seanrennie/Development/HUMMINGBIRD/mts-data-sync/src/estimate.js:59:27)
    at DestroyableTransform.emit (events.js:223:5)
    at DestroyableTransform.EventEmitter.emit (domain.js:475:20)
    at addChunk (/Users/seanrennie/Development/HUMMINGBIRD/mts-data-sync/node_modules/readable-stream/lib/_stream_readable.js:291:12)
    at readableAddChunk (/Users/seanrennie/Development/HUMMINGBIRD/mts-data-sync/node_modules/readable-stream/lib/_stream_readable.js:278:11)
    at DestroyableTransform.Readable.push (/Users/seanrennie/Development/HUMMINGBIRD/mts-data-sync/node_modules/readable-stream/lib/_stream_readable.js:245:10)

I'm happy to help debug further but not sure where to go from here?

If I run mtsds --sync it works as expected.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Reproduce the documented workflow with the generated config, recipe, and JSONL files, then inspect src/estimate.js at line 59 and the stack trace from @mapbox/tile-cover. Compare the estimate path with the successful sync path. Done means mtsds --estimate completes for the documented tileset input without this TypeError.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, node.js
Domain
cli
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.