Skip to content
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

Negative values are being shown on the wrong side of radar chart axes #31348

Open
2 of 3 tasks
gilles-dataroots opened this issue Dec 9, 2024 · 5 comments
Open
2 of 3 tasks
Labels
viz:charts:radar Related to the Radar chart

Comments

@gilles-dataroots
Copy link

Bug description

Since my company updated to the latest version of Superset begin December, I'm seeing weird visuals on radar plots. As you can see the negative values are now being plotted on the opposite side of the axis.

Screenshots/recordings

image

Superset version

master / latest-dev

Python version

3.9

Node version

16

Browser

Chrome

Additional context

No response

Checklist

  • I have searched Superset docs and Slack and didn't find a solution to my problem.
  • I have searched the GitHub issue tracker and didn't find a similar bug report.
  • I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.
@dosubot dosubot bot added the viz:charts:radar Related to the Radar chart label Dec 9, 2024
Copy link

dosubot bot commented Dec 9, 2024

Hey @gilles-dataroots! I'm here to help you with any bugs, questions, or contributions you have for Superset. Let's get this sorted out together!

The issue you're experiencing with negative values being plotted on the wrong side of radar chart axes might be related to recent changes in the radar chart component. The latest updates introduced new configuration options for metrics, specifically radarMetricMaxValue and radarMetricMinValue, which allow users to specify maximum and minimum values for metrics. The radarMetricMinValue defaults to '0' if not set, which might affect how negative values are displayed [1][2].

To address this issue, you might want to check the configuration of your radar chart to ensure that the radarMetricMinValue is set appropriately for your data, especially if it includes negative values. Adjusting these settings could help correct the visual representation of negative values on the radar chart axes.

To continue talking to Dosu, mention @dosu.


Help Dosu learn! Give it feedback: Great Response | Irrelevant Answer | Incorrect Sources | Too Verbose | Hallucination | Other

@gilles-dataroots
Copy link
Author

The previous version of the radar chart would dynamically make sure all values were being plotted on the axis. Is this no longer the intended goal? A hardcoded radarMetricMinValue would not allow for an index without a hard minimum...

Copy link

dosubot bot commented Dec 9, 2024

The radar chart component in the latest version of Superset still supports dynamic adjustment of axis values based on the data range. The radarMetricMinValue and radarMetricMaxValue are optional configurations. If these values are not set, the radar chart will automatically use the minimum and maximum values from the data, allowing for dynamic scaling without a hardcoded minimum or maximum [1][2]. If you're experiencing issues with negative values, ensure that these configurations are not inadvertently set to values that could affect the dynamic scaling.

To continue talking to Dosu, mention @dosu.


Help Dosu learn! Give it feedback: Great Response | Irrelevant Answer | Incorrect Sources | Too Verbose | Hallucination | Other

@gilles-dataroots
Copy link
Author

gilles-dataroots commented Dec 10, 2024

@dosu that sounds promising, where can I actually find and update the radarMetricMinValue parameter?

@gilles-dataroots
Copy link
Author

I've found the option in the Customize tab of the edit chart page. Even after removing the 0 and setting the radarMetricMinValue for the metric Night to auto, the chart still displays some negative values on the wrong size of the origin as you can see below.
image
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
viz:charts:radar Related to the Radar chart
Projects
None yet
Development

No branches or pull requests

1 participant