Skip to content
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

Increase WMS maxRefreshIntervals to 10,000 and show warning if limit is reached #7025

Open
leylines opened this issue Jan 9, 2024 · 5 comments

Comments

@leylines
Copy link

leylines commented Jan 9, 2024

Version terriajs: latest

Example: Sentinel Data from eumetsat.int

https://view.eumetsat.int/geoserver/copernicus/daily_sentinel3ab_olci_l1_rgb_fulres/ows?service=WMS&request=GetCapabilities&version=1.3.0

2020-02-17T04:03:00.000Z/2024-01-09T10:22:00.000Z/PT1H41M

The Timeslider in terriaMap show only dates from Februar to April 2020.

Regards,

Jörg

@leylines
Copy link
Author

Any news on this?

@nf-s
Copy link
Contributor

nf-s commented Mar 22, 2024

Hi @leylines

Sorry for the delay in response. This is due to the default limit of timesteps generated from time-interval strings being set to 1000

This can be increased by setting the maxRefreshIntervals property on your WMS item in your Terria JSON catalogue.

We have a ticket tracking this issue here

@nf-s
Copy link
Contributor

nf-s commented Mar 22, 2024

I think we should increase the default limit to 10,000 - and also show a warning message if this limit is reached. We could also include a button to "increase the limit" or something like that

@nf-s nf-s changed the title Incorrect Time-Dimension in WMS from Geoserver and maybe other WMS-Servers. Increase WMS maxRefreshIntervals to 10,000 and show warning if limit is reached Mar 22, 2024
@leylines
Copy link
Author

Thank you, this is very useful, i got it to work. Is there a way to define the number of intervals in reverse? For example, 1000 intervals would be sufficient if they can be defined from the last actual measurement.

Thank you and regards,

Jörg

@nf-s
Copy link
Contributor

nf-s commented Mar 26, 2024

No problem!

Currently there is no way to define the number of intervals in reverse - but I think that might be a better default - in my mind it makes sense to show the latest 1000 intervals if they list has been truncated

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

No branches or pull requests

2 participants