-
Notifications
You must be signed in to change notification settings - Fork 6
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
[Bug] AOI visualization disappears when map recenters in E&A #1093
Labels
P2
Find a way to fix in the next few sprints
regression
Signifies a regression issue found during testing
Comments
aboydnw
added
regression
Signifies a regression issue found during testing
P2
Find a way to fix in the next few sprints
labels
Aug 6, 2024
similar to #949 |
Closed
Probably want to address with #1289 |
AliceR
added a commit
that referenced
this issue
Jan 16, 2025
This update simplifies AOI management by removing the concept of multiple AOIs where only some are selected for analysis. Instead, there will always be a single AOI that serves as input for analysis. The AOI is now displayed using a regular LineLayer with a GeoJson Source (both react-map-gl components). There are three methods for creating an AOI: uploading a GeoJSON file, selecting a predefined boundary from a menu (or directly from the map in a future iteration), and using the Mapbox Draw Control. By leveraging the draw control solely for generating polygons, we eliminate the need for extensive custom code to sync with the library. This simplification addresses long-standing technical debt (close #710) and resolves many issues (fix #1093, fix #1258).
This was referenced Jan 20, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
P2
Find a way to fix in the next few sprints
regression
Signifies a regression issue found during testing
Context
From @sandrahoang686 's initial investigation:
URL:
Blocking a release? (y/n): n
Regression issue? (y/n): y
[If yes, add Regression label]
Priority: P2
Steps to reproduce the issue
https://www.loom.com/share/39b8d68a331443629d86910f72272614?sid=20d4ea43-5a7a-48aa-bbe9-36b51489f2d7
The text was updated successfully, but these errors were encountered: