-
Notifications
You must be signed in to change notification settings - Fork 37
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
Postgis driver support #1032
Postgis driver support #1032
Conversation
* [pre-commit.ci] pre-commit autoupdate updates: - [github.com/PyCQA/pylint: v3.1.0 → v3.2.3](pylint-dev/pylint@v3.1.0...v3.2.3) * Lintage for Pylint upgrade. * Non-pre-commit pylint too old. --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Paul Haesler <[email protected]> (cherry picked from commit bca8a8e)
…viour for integration tests.
…chema-ed both dbs.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop-1.9 #1032 +/- ##
===============================================
- Coverage 93.77% 93.24% -0.54%
===============================================
Files 46 53 +7
Lines 6573 6999 +426
===============================================
+ Hits 6164 6526 +362
- Misses 409 473 +64
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable as far as I can tell, but could probably benefit from some more comments
Support for postgis driver. Uses native postgis tables for dataset search and populating range tables.
Multidb test framework - one postgres test index and one postgis one.
Wow, it's actually working now! Need to update documentation, review test coverage and probably some code cleanup before ready for review, but getting very close.
📚 Documentation preview 📚: https://datacube-ows--1032.org.readthedocs.build/en/1032/