rusackas commented on PR #36344:
URL: https://github.com/apache/superset/pull/36344#issuecomment-3619604573

   This is so cool! What a great POC! 
   
   I was able to make a couple chart columns with one of the example datasets, 
which is awesome to see:
   <img width="1137" height="578" alt="image" 
src="https://github.com/user-attachments/assets/5afcc629-e4ae-4bbe-b569-3935b3b07c3c";
 />
   
   So now I get how you've wired it up... if you add a bunch of metric columns 
to the chart, this scoops the metrics from each column into a chart for the 
row. Very cool! In this example, I can now see how each Language (row) stacks 
up for each _metric_, but also get these charts to see how the metrics stack up 
to _each other_ for the row! How cool!
   
   I think @kasiazjc and @yousoph would love to see this... it opens up a slew 
of possibilities. 
   
   1) This feature is already useful as-is, though I might have some feedback 
about re-jiggering some of the controls (which are only relevant in certain 
cases, so we can probably hide several of them most of the time. Happy to give 
this granular feedback if you want to optimize the UI a bit here :D 
   
   2) I could imagine an alternate version of controls to power the charts. The 
interpretation I _think_ might get a lot of use is to have each chart column 
use TWO inputs... a dimension and a metric (i.e. x axis and y axis). That would 
allow some interesting use cases... let's say we're using "Vehicle Sales" here. 
You could have a row for each Vehicle Type, and the normal metric columns for 
Units Sold and Total Sales ($). But then if you add a chart column, you could 
pick... Units Sold as the Metric, and then pick the Timestamp column or the 
State column for an X axis. Or, similarly have a row per state, and display a 
chart showing a metric across Vehicle Types as your chart column. This would 
allow us to deprecate/remove the "Time-Series Table" chart.
   
   3) As a POC, this implementation is useful to show the way other places, 
too! It'd be a very separate project, but we _seemingly_ could finally use 
these lil' chart types to show summary stats (i.e. histograms) at the tops of 
numeric columns in SQL Labs results table. Again, whole different scope of 
work, but it seems to share some of the same AG Grid building blocks.
   
   Again, very nice concept... thanks for this POC! I'll provide some UI 
feedback on this implementation soon, and maybe we can get this version across 
the finish line!
   


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