jaksmid opened a new issue #15748:
URL: https://github.com/apache/superset/issues/15748


   I am using deck gl grid visualization to show the price per square meter in 
real estates properties. I noticed that sometimes height is calculated 
incorrectly. E.g. if I want to show the max price per sq. meter and I have 
filters that limit rows used such as `price_per_sq_m < 40000`, the height that 
is shown still exceeds this limit. 
   
   I managed to reproduce the bug using the sample dataset shipped with 
superset. 
   
   ### Expected results
   
   I would expect that height respects both filters and used aggregated 
function.
   
   ### Actual results
   
   Height does not correspond to the filters and aggregate function. See 
screenshot.
   The height where the mouse is hovering is clearly above the filter. Max 
function used for aggregation does not change the maximum either and should not 
exceed the set filter.
   
   #### Screenshots
   
   <img width="1480" alt="Screenshot 2021-07-16 at 21 50 03" 
src="https://user-images.githubusercontent.com/3618465/126001562-8e8e0855-7d23-49a1-aa8d-d20a561ea61d.png";>
   
   
   #### How to reproduce the bug
   
   * Open the deck gl dashboard and explore the grid chart. 
   * Change the height metric to max(occupancy)
   * Set the filter to limit occupancy (e.g. 70).
   * Observe heigh of certain areas still exceeding 90.
   
   ### Environment
   
   (please complete the following information):
   
   - superset version: `Superset 1.2.0`
   - python version: `Python 3.9.5`
   - node.js version: `v16.3.0`
   
   ### Checklist
   
   Make sure to follow these steps before submitting your issue - thank you!
   
   - [X] I have checked the superset logs for python stacktraces and included 
it here as text if there are any.
   - [X] I have reproduced the issue with at least the latest released version 
of superset.
   - [X] I have checked the issue tracker for the same issue and I haven't 
found one similar.
   
   ### Additional context
   
   Observed this problem even before 1.0
   


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