-
Notifications
You must be signed in to change notification settings - Fork 129
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
Time aware layers #305
Comments
the answer is yes. We have been able to build this into our provider. |
Any comments on the best way to set this up as opposed to hacking it ourselves? |
@keithfraley - Looking at the code, I think the functionality it already there. FeatureServer looks in your provider's metadata to see if you have defined a
Spec pulled from here: https://developers.arcgis.com/rest/services-reference/layer-feature-service-.htm |
@keithfraley noticed you have had some success with this issue on your provider (koopjs/koop-provider-elasticsearch#11). Was adding |
It's more than that, you need to handle the query string parameters for
time, it is a separate call then a normal date filter
…On Tue, May 8, 2018, 10:54 AM Rich Gwozdz ***@***.***> wrote:
@keithfraley <https://github.com/keithfraley> noticed you have had some
success with this issue on your provider (
koopjs/koop-provider-elasticsearch#11
<koopjs/koop-provider-elasticsearch#11>). Was
adding time-info to your provider's metadata enough to enable
time-sliders in AGOL and Pro?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#305 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ADyxVLdb2XAArVfXCPgovXoefodv0_HRks5twb-_gaJpZM4QCT07>
.
|
Ok, but is that something you're handling in the provider? Trying to determine if this issue can be closed. |
This was handled at a provider level in our case, but certainly should /
could be addressed at a winnow level
…On Tue, May 8, 2018, 11:35 AM Rich Gwozdz ***@***.***> wrote:
Ok, but is that something you're handling in the provider? Trying to
determine if this issue can be closed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#305 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ADyxVAwIeoErZT81Bk_LY-m0aTyGWEguks5twclNgaJpZM4QCT07>
.
|
I'd say if it is handled at the winnow level you would have to make it optional. In some cases (like when using a big data store) it is safer and faster to handle it at the provider level to keep from returning thousands of features from the store that don't fit into a given time interval. |
Ok, cool, regardless, awesome addition
…On Tue, May 8, 2018, 2:06 PM Danny Hatcher ***@***.***> wrote:
I'd say if it is handled at the winnow level you would have to make it
optional. In some cases (like when using a big data store) it is safer and
faster to handle it at the provider level to keep from returning thousands
of features from the store that don't fit into a given time interval.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#305 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ADyxVFR_caG4W-eS-En_oB3NRiMMRIcwks5twezHgaJpZM4QCT07>
.
|
That's a a great point @dhatcher, so I'd say we keep this in the provider. I will close this, but am opening an issue in the koopjs.github.io repo to document an example of this in a provider: koopjs/koopjs.github.io#27 |
Is it possible to create a time-aware layer in KOOP so that the time-sliders in the various clients are activated when consuming the service?
We can easily handle the form-date and to-date queries in the getData method.
Thank you.
The text was updated successfully, but these errors were encountered: