这是indexloc提供的服务,不要输入任何密码
Skip to content

feat: added logger in ClusterRendererMultipleItems #1519

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

Merged
merged 4 commits into from
Jun 3, 2025

Conversation

kikoso
Copy link
Collaborator

@kikoso kikoso commented Jun 3, 2025

This PR adds a mechanism to log the behavior of the ClusterRendererMultipleItems.

There seems to be a bug where an individual item is being displayed at the same time as the cluster. We have had related issues potentially related to this (maybe this and this?). But we did not get any way to reproduce them, so it is not clear why they happened.

This PR incorporates some logging that we can later use to debug the issue. Initially on the ClusterRendererMultipleItems, later on we can add it to the other renderers.

@googlemaps-bot
Copy link
Contributor

googlemaps-bot commented Jun 3, 2025

Code Coverage

Overall Project 34.3% -1.8% 🍏
Files changed 0%

File Coverage
GridBasedAlgorithm.java 0% -19.43%
RendererLogger.java 0%
ClusterRendererMultipleItems.java 0% -13.27%

@dkhawk dkhawk merged commit 65498ba into main Jun 3, 2025
11 checks passed
@dkhawk dkhawk deleted the feat/added_logger_in_cluster_renderer_multiple_items branch June 3, 2025 22:55
googlemaps-bot pushed a commit that referenced this pull request Jun 3, 2025
# [3.14.0](v3.13.0...v3.14.0) (2025-06-03)

### Features

* added centroid algorithm ([#1520](#1520)) ([8b57c3a](8b57c3a))
* added logger in ClusterRendererMultipleItems ([#1519](#1519)) ([65498ba](65498ba))
@googlemaps-bot
Copy link
Contributor

🎉 This PR is included in version 3.14.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@googlemaps-bot
Copy link
Contributor

🎉 This PR is included in version 3.14.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants