Demo clustering with ViewModel #506
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Only move the camera to the starting location when not restoring
Avoid locking and unlocking the algorithm in a tight loop when removing many cluster items
Allow the algorithm to handle its own locking, so it can be reused in another ClusterManager instance. Also allow adding/removing items with proper locking outside of ClusterManager. For best ViewModel support.
googlebot
added
the
cla: yes
jpoehnelt
added
the
type: feature request
|
@jeffdgr8 Can you split this up into a couple separate prs? |
Only move the camera to the starting location when not restoring
jpoehnelt
approved these changes
Oct 21, 2019
pauminku
added a commit
to pauminku/android-maps-utils
that referenced
this pull request
Nov 26, 2019
* googleOrigin/master: (107 commits) Remove unnecessary interface method public modifiers. (googlemaps#587) Remove unnecessary primitives boxing. (googlemaps#586) Fix googlemaps#575 - Change Travis config, use matrix of API emulators (googlemaps#585) Retain the bitmap cache (googlemaps#381) Made GeoJsonParser with parseGeometry public to use the parser standalone. (googlemaps#492) Fix googlemaps#570 - Use project GitHub URL for remote KML loading demo (googlemaps#581) Bump Gradle plugin for Android Studio v3.5.2 (googlemaps#574) Fix googlemaps#575 - Use API 26 emulator instead of API 28 (googlemaps#576) Fix googlemaps#371 - Run emulator-based unit tests on Travis (googlemaps#573) Fix googlemaps#371 - Run emulator-based unit tests on Travis (googlemaps#573) Fix googlemaps#532 - Use ContextCompat to load drawable resources for scaling (googlemaps#571) Change test methods visibility (googlemaps#569) Release 0.6.2 (googlemaps#568) Demo clustering with ViewModel (googlemaps#506) Differentiate between initial start and configuration change restore (googlemaps#567) Fix wrapping around IDL (googlemaps#564) Update Gradle wrapper to 5.6.3 (googlemaps#566) Add MaxIntensity functionality for user defined intensities on HeatMapTiles (googlemaps#499) Use float zoom to calculate visible bounds (googlemaps#483) Remove apklib publishing (googlemaps#563) ... # Conflicts: # library/src/androidTest/java/com/google/maps/android/PolyUtilTest.java
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Move algorithm lock from
ClusterManagertoAlgorithmto tie the locking to the algorithm object, to better allow reusing the algorithm in multipleClusterManagerinstances, to best support retaining the algorithm in a ViewModel.Add a demo of clustering using a view model to retain the cluster item data between configuration changes.
Add
removeItems()for optimally removing many items, without repeatedly locking and unlocking the algorithm in a tight loop.Update all demos to only move map camera on initial load, allowing configuration changes to restore the camera position.Moved to #567Update URLs to HTTPS (one no longer works with HTTP).