NASA-IMPACT / veda-jupyterhub

VEDA JupyterHub technical planning and documentation
0 stars 1 forks source link

Figure out plan for maintenance of QGIS STAC browser plugin #12

Open batpad opened 3 months ago

batpad commented 3 months ago

We want to setup the QGIS STAC Plugin to be installed and enabled by default into our QGIS setup on VEDA hub as part of https://github.com/NASA-IMPACT/veda-jupyterhub/issues/9

Currently, we are installing the STAC plugin from @slesaad's fork, setup here.

This is because the upstream repository seems to have very little activity and we have had trouble merging in PRs with functionality that we would need (even though ideally all that functionality could and should be upstreamed).

Going forward, we would hope that the QGIS STAC plugin becomes a common way for people to browse through relevant STAC catalogs when people are using QGIS in the hub, and we probably would like to add more features to improve the browsing experience, potentially allow for different types of datasets, etc. It seems like long-term, it's probably not a great idea to continue building off this fork in @slesaad's github account, and it would be great to have a clear idea on who is responsible for maintenance, merging PRs, etc.

We should either:

Do we know who maintains https://github.com/stac-utils/qgis-stac-plugin/ and if we could offer help / try and figure this out? cc @wildintellect @geohacker

+cc @yuvipanda

wildintellect commented 3 months ago

Since the upstream is owned by the stac-utils organization, we can bring the question of maintainers and upkeep in the STAC community meetings, the owners or at least admins who could pass responsibility are in those meetings.

More details on the Plugin distribution ownership https://plugins.qgis.org/plugins/qgis_stac/#plugin-details

geohacker commented 3 months ago

Aren't @vincentsarago and @bitner members of stac-utils org? I'll also send a note to Rob.

vincentsarago commented 3 months ago

@geohacker David and I are members not owners

geohacker commented 3 months ago

I've asked Rob if he can add @batpad

yuvipanda commented 3 months ago

The PRs that have been merged are:

  1. https://github.com/stac-utils/qgis-stac-plugin/pull/218 (to support s3://)
  2. https://github.com/slesaad/qgis-stac-plugin/pull/2 (which could be submitted upstream as well I think)
  3. https://github.com/stac-utils/qgis-stac-plugin/pull/244 which I just made, to allow this package to be installed from pip.
vincentsarago commented 3 months ago

Just asked in Gitter (https://app.gitter.im/#/room/#SpatioTemporal-Asset-Catalog_Lobby:gitter.im) about owner ship of this repo