betodealmeida commented on PR #33559: URL: https://github.com/apache/superset/pull/33559#issuecomment-2903080322
I don't want to distract from the fact that this PR is important and fixes a bug that makes the radar visualization completely useless, but I have a strong impression that the description of this PR and some of the comments were written using AI. If that's the case, I think it's important to be transparent about the use of AI tools. The description of the issue, for example, says: > When plotting multiple metrics with vastly different scales (e.g., 208000, 2590, 9200), the Radar Chart currently uses a shared scale across all metrics. This causes small values to appear indistinguishable from large ones—rendering the chart ineffective for visual comparison. This is not true at all. There's no shared scale across all metrics, **on the contrary**: the problem we see here happens because each metric has its own scale, so that they all end maxed up. We need a shared scale across all the metrics so that big numbers are shown big, and small numbers are shown small. I spent quite some time trying to understand the mismatch between the description of the problem and the proposed solution. Please don't use AI to generate PR descriptions. If you do so, please be transparent and declare how you're using AI in the process. -- 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]
