richardfogaca commented on PR #38458:
URL: https://github.com/apache/superset/pull/38458#issuecomment-4014197945
The new regression tests cover the small-positive, ordering, negative, and
max-label paths, but I don’t see a direct assertion for the zero-value case
that the PR description calls out.
Could we add one explicit cluster test with aggregation="sum" and sum: 0,
then assert the rendered arc radius is finite and at least dotRadius / 6? That
would pin the “zero-value clusters are no longer invisible” behavior instead of
relying on nearby cases
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
