cognitedata / gearbox.js

GearBox is a place where application developers can contribute useful, reusable components across applications
Apache License 2.0
15 stars 9 forks source link

Extend maximum recommended datapoints in TimeseriesDataExport #496

Open hernil opened 5 years ago

hernil commented 5 years ago

You hit the limit of 10000 datapoints - some data may be omitted

When selecting a broad enough span of time together with a high enough resolution of data points one may run into this limitation. It is sort of unclear where this limit originates.

Our use case requires export of hourly data, usually ten years back in time. Some back of the napkin math dictates that is pushing the ~90k mark of datapoints per timeseries.

Some explenation of the 10 000 point limit would be good as a starting point so we can help looking into potential bottlenecks!

mrtraser commented 5 years ago

SDK doesn't support ability to request more than 10 000 points for now, so it is nothing we can do here. We need to wait for backend implementation of this feature.

muradsater commented 5 years ago

Was this requested against Cognite / SDK team?

polomani commented 5 years ago

The ticket was created on DEVX board DEVX-195 but we might be blocked by api

nalaka commented 4 years ago

@hernil @polomani @mrtraser @muradsater - Since from DEVX-195 it seems that this should be handled in the JS SDK, I am closing this for now. Please reopen if we need to handle this inside Gearbox.

polomani commented 4 years ago

Please, don't close this issue, it's an ongoing problem. And since it's not prioritized on sdk yet, we might need to implement it on the component here

And it's quite important for @hernil project

nalaka commented 4 years ago

@polomani & @hernil - shall we implement doing multiple queries and aggregating the results in TimeseriesDataExport component itself then?

Also we still need a maximum that our component will handle - is 100,000 enough?

I think we may be able to incorporate this into our next sprint if confirmed.