geotab-data-platform commented on PR #30716:
URL: https://github.com/apache/superset/pull/30716#issuecomment-2458103740

   Hi @villebro, 
   Please see attached the current behaviour vs after the fix.
   
   ## Settings Used:
   Chart Type: Mixed Chart
   yhat = 1
   yhat_lower = -10
   yhat_upper = 5
   query b = 10
   
   <img width="200" alt="image" 
src="https://github.com/user-attachments/assets/bc4c5b2f-e9dd-463d-8579-20954bd684d5";>
   <img width="200" alt="image" 
src="https://github.com/user-attachments/assets/f8e00c17-c769-4cf4-984a-0c4daeaf3061";>
   <img width="200" alt="image" 
src="https://github.com/user-attachments/assets/332c7ae8-0cb4-4a3b-b6d1-4cb15251e048";>
   
   ## Current Behaviour
   
   <img width="626" alt="image" 
src="https://github.com/user-attachments/assets/20048db8-e12d-43cd-bd88-3cddab1f46f8";>
   
   As you can see, the confidence interval ranges between (0,15) instead of 
(-10,5).
   
   ## After the fix
   Setting the same values we get the following response with correct 
confidence intervals (-10,5).
   
   <img width="1190" alt="image" 
src="https://github.com/user-attachments/assets/b90d6e67-2ca2-43c7-95f2-120f9e3a834b";>
   
   When you do not reorder the props and send them to echarts as a json to 
generate the chart, the echarts package will default to an x axis that uses 
value and not time. Additionally, the props must be ordered in the that 
specific order as the behaviour will not be as expected otherwise and 
confidence bands will not be correct.
   
   


-- 
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]

Reply via email to