Skip to content

t.connect: add a mapset option to list connections across project - #7594

Merged
ninsbl merged 9 commits into
OSGeo:mainfrom
ninsbl:t_connect_mapset
Jun 25, 2026
Merged

ninsbl merged 9 commits into
OSGeo:mainfrom
ninsbl:t_connect_mapset

Conversation

@ninsbl

@ninsbl ninsbl commented Jun 23, 2026

Copy link
Copy Markdown
Member

This PR adds a mapset option to t.connect to report TGIS database connections across mapsets.
The point is to get a fast overview over mapsets that contain TGIS databases. It changes the format of the recently added JSON output to always return an array. Furthermore, the mapset option is only available for reporting the connection(s) not setting it.
Changes are oriented on g.list.

claude assisted with code writing.

@saket0187 This should significantly speed-up listing temporal data for the data catalog. The mapset option may be propagated to t.list to 1: screen for possible TGIS DBs and 2) list content for the relevant mapsets in one t.list call...

@ninsbl ninsbl added this to the 8.6.0 milestone Jun 23, 2026
@ninsbl
ninsbl requested a review from petrasovaa June 23, 2026 09:02
@ninsbl ninsbl added temporal Related to temporal data processing C Related code is in C database Related to database management module labels Jun 23, 2026
@github-actions github-actions Bot added Python Related code is in Python docs markdown Related to markdown, markdown files tests Related to Test Suite labels Jun 23, 2026
Comment thread temporal/t.connect/t.connect.md Resolved
@ninsbl

ninsbl commented Jun 24, 2026

Copy link
Copy Markdown
Member Author

This is also one step towards solving #7344 with Tools instead of API.

@ninsbl
ninsbl requested a review from petrasovaa June 24, 2026 07:28

@petrasovaa petrasovaa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@ninsbl
ninsbl merged commit 4846566 into OSGeo:main Jun 25, 2026
26 checks passed
ninsbl added a commit to ninsbl/grass that referenced this pull request Jul 7, 2026
…Geo#7594)

* document mapset option

* add mapset option

* add tests for mapset option

* add testcase for empty VAR file

* adjust to t.connect JSON output

* adjust to t.connect JSON output

* use text for output

---------

Co-authored-by: Edouard Choinière <27212526+echoix@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C Related code is in C database Related to database management docs markdown Related to markdown, markdown files module Python Related code is in Python temporal Related to temporal data processing tests Related to Test Suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants