Skip to content

Update App.tsx#306

Open
Tirthshah3105 wants to merge 1 commit intotheforage:mainfrom
Tirthshah3105:patch-3
Open

Update App.tsx#306
Tirthshah3105 wants to merge 1 commit intotheforage:mainfrom
Tirthshah3105:patch-3

Conversation

@Tirthshah3105
Copy link
Copy Markdown

I didn't use distinct count on timestamp because you’re essentially counting how many distinct timestamps appear, which isn't typically useful for visualizing time-series data. Since the timestamp field is being used as a row-pivot, it's already grouping data by each distinct timestamp without needing to aggregate it. You'd want the graph to display the actual time points rather than a count of those time points.

However, if you need to use "timestamp": "distinct count" for a specific purpose (such as counting the number of unique timestamps or events), feel free to include it, but it may not provide meaningful insights in a line chart that shows trends over time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant