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

task wise mbtile generation #982

Closed
wants to merge 2 commits into from
Closed

Conversation

Sujanadh
Copy link
Collaborator

@Sujanadh Sujanadh commented Nov 8, 2023

fixes #890
this changes allows to generate the mbtiles of individual task using their task_id. if task_id is not provided it will generate the mbtiles of entire project boundary.

@Sujanadh Sujanadh self-assigned this Nov 8, 2023
@github-actions github-actions bot added the backend Related to backend code label Nov 8, 2023
@spwoodcock
Copy link
Member

For this to work we would also need to update the frontend.

But please also see the comments in the linked issue & how Rob proposed we solve this.
We still want to generate the tiles for the entire project, then subdivide.
I proposed a way to do this with project level PMTiles, then dividing into task level MBTiles on request.

I will probably make a PR for this in the coming weeks.

@Sujanadh
Copy link
Collaborator Author

Sujanadh commented Nov 8, 2023

so the tile of entire project must be generated not the task wise? and allowing to download tile of required task from the generated tile of entire project?

@Sujanadh Sujanadh requested a review from robsavoye November 8, 2023 11:50
@spwoodcock
Copy link
Member

Yes exactly.

osm-fieldwork does support outputting PMTiles, so we should do that.
The subdivision from PMTiles into MBTiles won't be simple though & will require delving into the format internals.

If you want to have a go, feel free, otherwise I will be working on this soon.

@spwoodcock
Copy link
Member

Rob already gave his feedback for how to solve this: #890 (comment)

But he can always weigh in again.

@spwoodcock spwoodcock closed this Nov 20, 2023
@spwoodcock spwoodcock deleted the task_wise_mbtiles branch November 29, 2023 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Related to backend code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow downloading of mbtiles per task
2 participants