Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    paul van genuchten
    @pvgenuchten
    hi gs-gitter, @justb4 and me are running some experiments with ogc-api-implementations, we're also including geoserver. Congratulations to the developer team, it looks really good. Today i experimented with the template override mechanism in ogc api, and was able to add a corporate identity to the html output by overriding common-header.ftl (from ~/data/templates/ogc/features). Results are at https://apitestbed.geonovum.nl/geoserver/ogc/features. I was a bit surprised though to find the default header in https://apitestbed.geonovum.nl/geoserver/ogc/features/collections/test-paul3:5a_meetlocaties_sel_wrd/items?f=text%2Fhtml&limit=50
    Andrea Aime
    @aaime
    patches welcomed :-D
    you have a colleague that has quite some expertise in managing those templates
    btw, nice customization :-D
    Looking at the code, maybe you need to customize a different header template?
    @jodygarnett attempted to document how the templates work, not sure this particular class was included in the lot, it's doing a bit its custom thing as it's dealing potentially with lots of features
    @pvgenuchten see ab ove
    Jody Garnett
    @jodygarnett
    That is cool @pvgenuchten , if you can tell me what to add to the docs it would be great.
    So getfeature-header.ftl and getfeature-footer.ftl?
    Jody Garnett
    @jodygarnett

    @aaime I see you got the http://geoserver.org/announcements/2021/06/23/geoserver-2-18-4-released.html page out, and in markdown.

    How was the writing experience? Were you able to cut and paste markdown from jira?

    aside: I think we need to post notes from PSC meeting.
    Andrea Aime
    @aaime
    I could, but did not want to all that much :-D
    getting in another meeting right now (after teaching for 4 hours...) somebody pray for my soul :-D
    Jody Garnett
    @jodygarnett
    I am also doing training for 2 hours, your soul is doomed, mine less doomed
    paul van genuchten
    @pvgenuchten
    Thanx for checking @aaime, let me try those and will update you @jodygarnett
    paul van genuchten
    @pvgenuchten

    Looking at the code, maybe you need to customize a different header template?

    that template also includes common-header.ftl

    Jody Garnett
    @jodygarnett

    @pvgenuchten I have an answer for you ... the documentation does not match the code. Not sure which is right.

    • [x] Creating a file in GEOSERVER_DATA_DIR/templates/getfeature-header.ftl works
    • [ ] The documentation indicates GEOSERVER_DATA_DIR/ogc/features/getfeature-header.ftl

    Please try the templates/getfeature-header.ftl directory and report back.

    image.png
    Here is a very simple test
    paul van genuchten
    @pvgenuchten
    i'm interested to override common-header.ftl, not getfeature-header.ftl
    (getfeature-header includes common-header)
    paul van genuchten
    @pvgenuchten
    this works fine for most of the endpoints, except for the items endpoint
    Jody Garnett
    @jodygarnett
    image.png
    @pvgenuchten placing templates/common-header.ftl alongside templates/getfeature-header.ftl works (see modified breadcrumbs below)

    If I understand correctly the #include element:

    <#global pagecrumbs="<li class='breadcrumb-item'><a href='"+serviceLink("")+"'>Home</a></li><li class='breadcrumb-item'><a href='"+serviceLink("collections")+"'>Collections</a></li><li class='breadcrumb-item active'>Feature</li>">
    <#include "common-header.ftl">
    
    TEST Test test

    Does not know about our search order, and just looks in the same location as getfeature-header.ftl

    paul van genuchten
    @pvgenuchten
    Ok, going to try that
    Jody Garnett
    @jodygarnett
    excellent, hope it works (I am building main, are you building or using a nightly build?)
    paul van genuchten
    @pvgenuchten
    I use 2.19.nightly
    Andrea Aime
    @aaime
    The OGC API implementations are dveloped only on main, the code on 2.19.x is pretty much dead (receiving no backports)
    2 replies
    Jody Garnett
    @jodygarnett

    aside: I have been having fun testing postgis/geoserver performance (eventually found the problem was old version of postgis pre TWKB).

    During testing I was surprised that using "prepared statements" was a small bit slower than using text. I understood that using prepared statements (and simplify fast ) would engage TWKB and I expected more of an effect.

    Andrea Aime
    @aaime
    do you also have an older version of postgresql?
    Prepared statements were pretty bad performance wise since version... hum... what was it? 11 or 12?
    they learned to re-plan the query in a PS based on actual value around those versions (which makes all the difference)
    James Hughes
    @jnh5y
    Are you saying that prepared statements have gotten worse recently?
    Andrea Aime
    @aaime
    they got better
    Jody Garnett
    @jodygarnett
    I updated to the latest postgresql 13 and postgis 3.2 (following the chart )
    Andrea Aime
    @aaime
    there were pretty much unusable before
    James Hughes
    @jnh5y
    good to know
    Andrea Aime
    @aaime
    @jodygarnett then dunno... my experience with recent versions is the opposite, prepared statements are somewhat faster... may be system specific, data specific... who knows
    Jody Garnett
    @jodygarnett
    The difference is small (for 2 million buildings):
    • no simplification: prepared statements on 9.3 seconds, or off 8.9 seconds, is unexpected
    • fast simplification: prepared statements on - 6.3s, 6.25s vs off - 5.87s, 5.93s
    James Hughes
    @jnh5y
    what timing is that? ingest? query?
    Jody Garnett
    @jodygarnett
    Cool just wanted to compare your recent experience, I can dig more.
    It is a getmap at a city scale, url taken from layer preview at 1000x1000, testing using curl for timing.
    James Hughes
    @jnh5y
    ok, do all the buildings have to get returned? (I imagine you are picking up some rendering time, etc)
    Andrea Aime
    @aaime
    difference is like 6%? Not that much, could be caused by other things (like, you should be doing a load test and do averages, checking nothing else is going on)
    Jody Garnett
    @jodygarnett
    Just having fun trying out the new toys; this was intended as a mean test to compare having the database in different locations.
    Yep, as indicated this result was not the primary goal, just a surprising (to me) result. Since it also sounds a small bit surprising to you I can spend some more time on it.
    Andrea Aime
    @aaime