Skip to content

feat: add a dropdown to select priority to plot #91

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JulesL2
Copy link
Contributor

@JulesL2 JulesL2 commented Feb 2, 2025

Description

Add a slider in the GUI that select the level of precision ie the number of roads to plot. It updates the background data when the value changes.

This should reduce the size of the poster by default and it will speed up the process to obtain the first poster. However it will be less detailed by default

UPDATE:

@ThomasParistech I did code everything with class, it is good to go now. Only thing is, the selector should be on the page before upload, but starting with medium quality makes the feature ready to be used.

Corresponding Github Issues / Pull Requests

#59

@github-actions github-actions bot added the enhancement New feature or request label Feb 2, 2025
@JulesL2 JulesL2 force-pushed the JulesL2/roads_priority_selection branch 2 times, most recently from 6558549 to bacf764 Compare February 13, 2025 17:50
@JulesL2 JulesL2 marked this pull request as draft February 14, 2025 16:21
@JulesL2 JulesL2 marked this pull request as ready for review February 14, 2025 16:21
@JulesL2
Copy link
Contributor Author

JulesL2 commented Feb 15, 2025

@ThomasParistech I did add the dropdown selector before uploading the file, the PR is ready to merge

@JulesL2 JulesL2 force-pushed the JulesL2/roads_priority_selection branch from 8ce8877 to 4f44216 Compare February 15, 2025 17:05
@JulesL2
Copy link
Contributor Author

JulesL2 commented Feb 16, 2025

With pictures :

Home Menu
Medium
Very High

@JulesL2
Copy link
Contributor Author

JulesL2 commented Mar 7, 2025

@ThomasParistech Have you got some time to check this PR ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant