enram / bird-migration-flow-visualization

Bird migration flow visualization
http://enram.github.io/bird-migration-flow-visualization/viz/
MIT License
23 stars 6 forks source link

Correct SQL requirements for v/u_speed #105

Closed peterdesmet closed 9 years ago

peterdesmet commented 9 years ago

The conditions for avg_u_speed and avg_v_speed are currently:

    CASE
        WHEN avg(b.radial_velocity_std) >= 2 AND avg(b.bird_density) >= 10 THEN round(avg(b.u_speed)::numeric,5)
        ELSE 0
    END AS avg_u_speed,
    CASE
        WHEN avg(b.radial_velocity_std) >= 2 AND avg(b.bird_density) >= 10 THEN round(avg(b.v_speed)::numeric,5)
        ELSE 0
    END AS avg_v_speed,

But those WHEN requirements should actually be stated per altitude and radar, not on the average.

peterdesmet commented 9 years ago

Should these updates be included in the current visualization?

peterdesmet commented 9 years ago

I only want to update this in the full query, not for each CartoDB call, therefore blocked by #86.

peterdesmet commented 9 years ago

@hvangasteren, please review the new query. It does not yet include bird density.

peterdesmet commented 9 years ago

Query has been reviewed and described