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

Allow applying dimensions through WMS filter #185

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

arnaud-morvan
Copy link

Support dimensions in WMS sources through WMS FILTER parameter.

<source name="layer1" type="wms">
    <getmap>
        <params>
            <LAYERS>layer1</LAYERS>
            <FILTER>layer1:"floor" = '{dim:FLOOR}'</FILTER>
        </params>
    </getmap>
    ...
</source>

Useful for creating tilesets with dimensions from WMS layers without dimensions (note that QGIS server does not support WMS dimensions for the moment).

@arnaud-morvan
Copy link
Author

@tbonfort @jmckenna @rouault : What do you think about this, if accepted I would create a nem PR to update the mapserver documentation.

@arnaud-morvan
Copy link
Author

@tbonfort : Hello Thomas, I did not get feedback on this pull request openned 2 months ago, do you think this could be merged ?

@arnaud-morvan
Copy link
Author

Hello, how about merging this new feature ?

@jbo-ads
Copy link
Member

jbo-ads commented Feb 4, 2020

Sorry for the long delay, hope this is not too late.

Before merging I have some comments:

  • Removing duplicate code into _apply_dimensions() is a goot idea. Just take care that rdim->cached_value is now replaced by rdim->requested_value (see issue GetFeatureInfo: dimensions values are not passed to WMS source #212).

  • The same loop on dimensions is performed twice. Is it possible to group both computations into only one loop?

  • Template with named dimension is handled ({dim:floor}). What about template with all dimensions: {dim} ? Like in cache_sqlite.c for instance (remember to replace rdim->cached_value by rdim->requested_value).

Base automatically changed from master to main January 15, 2021 22:57
@jbo-ads
Copy link
Member

jbo-ads commented Mar 10, 2021

@arnaud-morvan - Is this pull request still relevant?

@arnaud-morvan
Copy link
Author

Seems that we do not need this anymore for QGIS which support WMS dimensions now : https://changelog.qgis.org/en/qgis/version/3.10/#wms-dimension
But this could still be interesting when using an external service as source that do not expose the dimensions we would like in mapcache.

@arnaud-morvan
Copy link
Author

I put this in my todo list and will try to find some time to spend on this rebase.

@jbo-ads
Copy link
Member

jbo-ads commented Mar 10, 2021

Great! Thank you.

@jmckenna jmckenna added this to the 1.12.0 milestone Mar 10, 2021
@jmckenna jmckenna removed this from the 1.12.0 milestone Jan 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants