Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Option for Timeseries Null days to be shown #487

Closed
alanmcruickshank opened this issue May 18, 2016 · 7 comments
Closed

Option for Timeseries Null days to be shown #487

alanmcruickshank opened this issue May 18, 2016 · 7 comments
Labels
enhancement:request Enhancement request submitted by anyone from the community viz:charts:timeseries Related to Timeseries

Comments

@alanmcruickshank
Copy link
Contributor

New Feature Request:

For timeseries plots, at the moment if there is a day with no data, the day is skipped and the day isn't shown at all. I'd love an option where these days could be shown even though they're blank, so that there would be an intentional gap.

Something similar to the "Show Missing Values" option available in packages like Tableau: https://onlinehelp.tableau.com/current/pro/online/mac/en-us/missing_values.html

@xrmx xrmx added the enhancement:request Enhancement request submitted by anyone from the community label Aug 9, 2016
@LAlbertalli
Copy link
Contributor

Any updates on these feature request? It is really annoying, especially for datasets with low-frequency data.

@alanmcruickshank
Copy link
Contributor Author

@LAlbertalli
This is currently possible through using the advanced options section and re-sampling using the re-sample rule which corresponds to the relevant time grain.

The UX is a bit rough because if you change the time grain you also have to change the resample rule. What would be excellent is a way of tying a resample rule to a time grain so they both change together - perhaps as an inbuilt function within certain visualisations.

@mistercrunch
Copy link
Member

There could be some auto-resampling based on Grain, but it would require tighter semantics around the way Grains are defined. Currently Grains are defined by database dialects and they're pretty loose list of names and SQL expressions. I guess we'd only have to add a 3rd configuration element as a Pandas freq expression (7D, 12H, ...).

I think it'd be pretty trivial actually. If that pandas freq is defined for that grain, we'd call the pandas resampling function automatically.

@benvogan
Copy link

benvogan commented Apr 4, 2017

@alanmcruickshank Unless I am missing something the pandas resampling methods either backfill or forward-fill. There is no fill method (that I can find) that zero fills - which is what we really want.

@mistercrunch
Copy link
Member

I think fill_method=None will fill with NA, which translates to gaps in trendlines in nvd3 as opposed to a non-linear axis. It's easy to call fillna(0) if you want to replace those NA too. Though users might want either... I'd say default should be NA.

@mistercrunch
Copy link
Member

Notice: this issue has been closed because it has been inactive for 383 days. Feel free to comment and request for this issue to be reopened.

@trtg
Copy link

trtg commented Mar 12, 2019

It would be ideal to have some way to show gaps in data, since that can often indicate some sort of system failure (broken ingestion, etc). The current approach of effectively interpolating can mask problems. Setting the How dropdown of of the pandas.resample section to sum, the Rule dropdown to whatever your interval is and leaving the Fill Method dropdown blank, seems to do what I want in this particular case. @mistercrunch @benvogan

@zhaoyongjie zhaoyongjie added the viz:charts:timeseries Related to Timeseries label Jun 18, 2021
@zhaoyongjie zhaoyongjie reopened this Jun 18, 2021
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 17, 2021
* feat(plugin-chart-word-cloud): allow minimum size

* fix: type

* fix: test
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 24, 2021
* feat(plugin-chart-word-cloud): allow minimum size

* fix: type

* fix: test
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 25, 2021
* feat(plugin-chart-word-cloud): allow minimum size

* fix: type

* fix: test
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 26, 2021
* feat(plugin-chart-word-cloud): allow minimum size

* fix: type

* fix: test
@apache apache locked and limited conversation to collaborators Feb 2, 2022
@geido geido converted this issue into discussion #18379 Feb 2, 2022

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
enhancement:request Enhancement request submitted by anyone from the community viz:charts:timeseries Related to Timeseries
Projects
None yet
Development

No branches or pull requests

7 participants