Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • May 13 22:35
    captaincoordinates edited #688
  • May 13 21:18
    captaincoordinates opened #688
  • May 13 14:12
    jmbelanger commented #686
  • May 13 11:42
    KoalaGeo edited #687
  • May 13 11:42
    KoalaGeo labeled #687
  • May 13 11:42
    KoalaGeo opened #687
  • May 11 06:39
    ldesousa commented #615
  • May 10 13:18
    ksonda commented #615
  • May 10 13:05
    ldesousa commented #615
  • May 10 11:17
    ksonda commented #615
  • May 10 11:16
    ksonda commented #615
  • May 10 08:11
    ldesousa commented #173
  • May 10 08:04
    ldesousa commented #615
  • May 08 00:17
    tomkralidis commented #686
  • May 07 15:33
    ksonda commented #615
  • May 07 04:14
    ksonda commented #173
  • May 07 04:08
    ksonda commented #173
  • May 07 04:07
    ksonda commented #173
  • May 07 04:03
    ksonda commented #173
  • May 07 02:28
    dblodgett-usgs commented #173
Tom Kralidis
@tomkralidis
@francbartoli / @pvgenuchten checking in, are we good for today ?
paul van genuchten
@pvgenuchten
i noticed some comments that you want to use mostly gitter (this one?) for questions, or are we presenting the slides along side?
Francesco Bartoli
@francbartoli
all good on my side @tomkralidis
Tom Kralidis
@tomkralidis
@pvgenuchten not following?
paul van genuchten
@pvgenuchten
what do you mean tom, who/what is not clear?
i'm in the gotomeeting
Francesco Bartoli
@francbartoli
where is the link @pvgenuchten?
Alex Kmoch
@allixender
The GotoMeeting session is not linked in the program schedule :cry:
Francesco Bartoli
@francbartoli
I found that but it is saying that the id is not valid for me
No sorry what I found is an old id for a sprint
Tom Kralidis
@tomkralidis

@pvgenuchten

i noticed some comments that you want to use mostly gitter (this one?) for questions, or are we presenting the slides along side?

We’re using the slides during the presentation. I mention Gitter if people have technical issues, does that work/make sense?

paul van genuchten
@pvgenuchten
Sure, nice!
Francesco Bartoli
@francbartoli

@pvgenuchten

i noticed some comments that you want to use mostly gitter (this one?) for questions, or are we presenting the slides along side?

We’re using the slides during the presentation. I mention Gitter if people have technical issues, does that work/make sense?

+1

paul van genuchten
@pvgenuchten
but i guess we also monitor the github discussions
Alex Kmoch
@allixender
Francesco Bartoli
@francbartoli
Thx @allixender!
Tom Kralidis
@tomkralidis
yes, makes sense @pvgenuchten
Tom Kralidis
@tomkralidis
@francbartoli given your OATiles session is the first of all 3 pygoeapi sessions, can you talk a bit (4-5 min) to slides 2-7 and that they form the basis for all pygeoapi sessions?
Francesco Bartoli
@francbartoli
Sure, it was on my plan @tomkralidis
Tom Kralidis
@tomkralidis
+1
thanks
Francesco Bartoli
@francbartoli
GitHub discussions look good, do we want to open up that feature for the pygeoapi repo?
Tom Kralidis
@tomkralidis

GitHub discussions look good, do we want to open up that feature for the pygeoapi repo?

Good idea @pvgenuchten . Should we open a discussion to discuss how we handle these discussions :)

Tom Kralidis
@tomkralidis

jokes aside, GitHub Discussions now enabled: https://github.com/geopython/pygeoapi/discussions

I kicked things off in https://github.com/geopython/pygeoapi/discussions/665

@pvgenuchten @francbartoli @kalxas @justb4 @jorgejesus and others, please help kick things off with your intros when you have a chance.

I will tweet and send to the mailing list.

jmckenna
@freenode_jmckenna:matrix.org
[m]
request for pygeoapi PSC: please add into your release process, actually creating a 'release' on Github so that all 'followers' are magically notified. eg. go here and look at bottom-right panel, empty releases section: https://github.com/geopython/pygeoapi versus go here and look at lower-right 'releases' section: https://github.com/MapServer/MapServer That automated notice of the release sent by Gihub, is now as important as the old email
notices to the mailing list. thanks!
Tom Kralidis
@tomkralidis
good idea jmckenna. Trying this out
jmckenna
@freenode_jmckenna:matrix.org
[m]
thanks
@tomkralidis: very nice, received! VIP
Tom Kralidis
@tomkralidis
thank you jmckenna for the suggestions and continuous improvement. VIP
Note I've added this step to our release management page: https://github.com/geopython/pygeoapi/wiki/ReleaseManagement#step-9
jmckenna
@freenode_jmckenna:matrix.org
[m]
perfect, great document. thanks!
Sander Schaminee
@GeoSander
Hi @tomkralidis. I guess you were too busy last week with the presentations and so on, but this is just a friendly reminder to have a look at PR #664 :)
Tom Kralidis
@tomkralidis

thanks @GeoSander I had a look and #664 looks a bit less invasive to the providers, great :). Some questions:

  • the docs would have to be updated (I am guessing you left the docs as is while we decide on which PR to move forward with)
  • in pygoeapi config, why do we break out links into en and fr as well? Shouldn’t their hreflang be enough? I’m good with doing language for string properties but for entire objects?
  • it looks like the pygeoapi.api.pre_process decorator and request header handling has been fixed/cleaned up via the APIRequest object
  • can you provide an overview of where/how to use the APIRequest object?
  • https://github.com/geopython/pygeoapi/pull/664/files#diff-f3ee4caa7dc224a51caf333d89b130fe02ff0351a77dfdd11c06d0a4b56e6732R755 : passing language to a coverage function? Do we need this?

Overall, I think we are close here, and while putting language so deep into the data codepaths is new for me, this work does move the yardstick on multilingual support. Great work!

I did add a few others to review the PR, and would want at least one of them to do a fulsome review of the PR as well.

10 replies
Rob
@robrichter

Hi all.
At first I want to thank you for a great work.
I am participating on a project where I would like to use Pygeoapi.
I have a few questions about functionality I need on my project that I was not able to find in documentation.

  1. Transactions
    PR: geopython/pygeoapi#524

  2. CQL filters
    PR: geopython/pygeoapi#478

  3. Custom projection (epsg: 5514)
    PR: geopython/pygeoapi#307

I found pull requests that didn't make it to master. Is it planned to finish them? What is the blocker? Can I help it myself?
Thanks

Francesco Bartoli
@francbartoli
Hi @robrichter, those PRs 1. and 2. were provided during the GSoC last year. They are both a good starting point for those features but definitively cannot be merged as they are. And definitively to be improved as well. About the CQL filters for the JSON dialect in the ES provider there is a PR opened geopython/pygeoapi#670. It is a working in progress yet
Sander Schaminee
@GeoSander
@tomkralidis I have updated PR #664 once again according to your suggestions earlier. I really hope someone else can have a look too...
I put some code examples in the docstring of the APIRequest class as well.
Tom Kralidis
@tomkralidis
thanks Sander. I think Francesco will review soon.
Sander Schaminee
@GeoSander
:+1: :grinning:
Chris Barrett
@christophersbarrett
Hi all, I'm having trouble with relative paths - for example I am deploying pygeoapi as a docker container to a Fargate cluster on AWS ECS. The cluster is access through ALB (load balancer) which also assigns a domain to the ip address. I'd like pygeoapi to be accessible from the path <domain>/pygeoapi. I have that working, but none of the links work in the UI. So the link is constructed as <domain>/pygeoapi/collections, but it 404s
So in short, pygeoapi image is accessible at the path we want - but none of the links work. So app routing isn't working it seems like. Server config in local config:
server:
bind:
host: 0.0.0.0
port: 80
url: /pygeoapi
mimetype: application/json; charset=UTF-8
encoding: utf-8
language: en-US
cors: true
Another clue: it can't find the static resources either.
Just van den Broecke
@justb4
@christophersbarrett are you following config conventions as described here?