Last date/time label missing on chart X axis

[dennishwuh]

Hi, I managed to run a superset docker, connected to my own database and plotted some charts successfully.
However, it didn't show up the last day in X axis, has anybody faced the same problem, please help.
Thanks for your time.

Screenshots

Environment

Docker

[sfirke]

This bug was patched in #20627 in summer 2022, there was no original bug report issue I found so thank you for creating this. That patch was reverted in August 2023 because it had caused a worse problem of a strangely-formatted final datetime label. So this bug is open again and will likely be present in 2.1.1 and 3.0.0.

Those interested in patching this should see discussions on those two PRs.

[rusackas]

I've seen issues that are almost to the contrary of this, where people wanted the end-of-axis tick removed because it didn't match the granularity of the other ticks. E.g. the ticks would be Jan, Feb, Mar, 3/14/23. Do you think this ist still an issue to contend with @sfirke ?

[liany-dropit]

I'm also having this issue:

[sfirke]

@liany-dropit am I correct that the line in that chart extends beyond Sunday, so there should be a Sunday label?

@rusackas from looking at Antonio's before/after in the original PR, I believe what the PR was fixing was the case where there is one more complete time period label that is missing. His "after" video cuts off quickly but I think it shows a "May" value populating.

I agree that Jan, Feb, Mar, 3/14/23 is bad, that's why the PR got reverted. But I believe the complaint is that without the PR it's going Jan, Feb, <> and the March value is unlabeled -- Jan, Feb, Mar would be the correct behavior. Caveat: I don't have an example of this bug at hand so am going from screenshots/video/memory.

[liany-dropit]

@sfirke In this case this is a weekly chart, it should show a full week, so Sunday is the last item in this chart. I expect there to be a label for Sunday as it looks strange that it's the only one of the seven days that doesn't have a label but does have a value.

[liany-dropit]

@sfirke do you have an estimation for a fix on this issue?

[sfirke]

No idea sorry.