Page MenuHomePhabricator

Select the domain (X-axis range) for charts before pulling data
ClosedPublic

Authored by epriestley on Apr 17 2019, 1:09 PM.

Details

Summary

Depends on D20441. Ref T13279. Currently, we pull all the data, then decide what the X-axis should look like.

Since users will reasonably want to do stuff like "show me march-april 2018" in the future, we need to move toward flipping this around so that we can support cases where the domain is specified by the user.

For actual chart functions (like "constant(3)" or "cos(x)"), we must also know the domain before we pull data, since there are an infinite number of places where we can evaluate the function "constant(3)".

See note in T13279 about continunity.

Test Plan

Screen Shot 2019-04-17 at 6.02.47 AM.png (861×976 px, 156 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

amckinley added inline comments.
src/applications/fact/controller/PhabricatorFactChartController.php
36

No setYAxis?

124–127

No list($domain_min, $domain_max)?

This revision is now accepted and ready to land.Apr 17 2019, 11:00 PM
src/applications/fact/controller/PhabricatorFactChartController.php
36

What, you think I'm just made of axes over here? That axes grow on trees?

This revision was automatically updated to reflect the committed changes.
src/applications/fact/controller/PhabricatorFactChartController.php
36

Macro gimli:  AND MY AXIS