janani-gurram commented on PR #35693: URL: https://github.com/apache/superset/pull/35693#issuecomment-3453809484
> This viz doesn't have the Advanced Analytics feature, it seems that would be worth adding here, since it provides the zero-imputation feature. Advanced Analytics doesn't have an option to strip out null values however. It's probably better to use Advanced Analytics to "fix" null values OR use a Filter to remove null values right from the control panel. Stripping them out in post-processing doesn't give users the _chance_ to set them as 0... they won't even know there ARE null values, which seems dangerous. Thanks for the suggestion! I agree that adding zero-imputation (and broader Advanced Analytics support) could be valuable. For this PR, the goal is just to address the rendering issue caused by NULL values. Implementing zero-imputation or new analytics options might be better suited for a follow-up PR? -- 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]
