[WIP] Add stacindex.org catalog browsing interface by gaolinai8888 · Pull Request #1062 · geojupyter/jupytergis

gaolinai8888 · 2026-01-10T09:07:29Z

Description

Important

This PR is part of an intern project, and will not be merged in to main -- there's still some work to do that didn't fit in the time bounds of the internship before we can merge a generic stac catalog interface into the main branch. We'll merge it in to geojupyter-dse-interns branch for now!

This PR implements the catalog browsing and collection selection workflow for the STAC catalog. It enables users to recursively browse collections and subcollections, view selected collections, and display items within the Results tab using the existing results interface.

What this PR adds:

Retains support for selecting and browsing the Stac catalog using the current UI
Adds recursive collection browsing, allowing deeply nested subcollections
Enables selecting a collection, displaying the selected collection and subcollections, and displaying items from the selected collection in the Results tab

Checklist

PR has a descriptive title and content.
PR description contains references to any issues the PR resolves, e.g. Resolves #XXX.
PR has one of the labels: documentation, bug, enhancement, feature, maintenance
Checks are passing.
Failing lint checks can be resolved with:
- pre-commit run --all-files
- jlpm run lint
If you wish to be cited for your contribution, CITATION.cff contains an author entry for yourself

📚 Documentation preview: https://jupytergis--1062.org.readthedocs.build/en/1062/
💡 JupyterLite preview: https://jupytergis--1062.org.readthedocs.build/en/1062/lite

github-actions · 2026-01-10T09:07:37Z

👈 Launch a Binder on branch gaolinai8888/jupytergis/stac-browser-updated

github-actions · 2026-01-10T09:20:58Z

Integration tests report: appsharing.space

mfisher87

It's really cool to see this in action and play with the new functionality! Thanks for your work.

I posted in our intern slack channel about how these comments are organized and guidance for addressing them!

mfisher87 · 2026-01-13T19:37:35Z

tsconfigbase.json

    "lib": ["ES2019", "WebWorker", "DOM"]
-  },
-  "esModuleInterop": true,
-  "resolveJsonModule": true


Question: Why was this configuration removed? Is removing it necessary for this PR to work?

We don't recall removing it, and we don't think removing it is necessary for this PR to work.

We don't recall removing it, and removing it is not necessary for this PR to work.

OK! In that case, please revert this change :)

packages/base/src/stacBrowser/components/CollectionBrowser.tsx

mfisher87 · 2026-01-13T21:28:16Z

packages/base/src/stacBrowser/components/StacPanel.tsx

+        <TabsList className="w-full grid grid-cols-2 mb-2 px-2 mt-2">
+          <TabsTrigger value="filters" className="text-xs">
+            {isGenericMode ? 'Browser' : 'Filters'}
+          </TabsTrigger>
+          <TabsTrigger value="results" className="text-xs">
+            Results ({activeTotalResults})
+          </TabsTrigger>
+        </TabsList>
+        <TabsContent
+          value="filters"
+          className="flex-1 overflow-hidden flex flex-col h-full px-2"
+        >


Quality: I think we can restore the original styles, classes, and other attributes like defaultValue on these components. I think the removal of the original classes is why the selected tab doesn't have an underline like it does in the currently-released version of JupyterGIS!

Nice! I want to look on my end to see if the underlining of the selected tab is fixed before marking this resolved.

packages/base/src/stacBrowser/components/StacPanel.tsx

packages/base/src/stacBrowser/hooks/useStacIndex.ts

for more information, see https://pre-commit.ci

gaolinai8888 · 2026-01-15T06:52:25Z

@mfisher87 Hey Matt, we are done addressing each comment.

nakul-py

Here are some Ready to commit changes to fix linting issues.

packages/base/src/stacBrowser/components/CollectionBrowser.tsx

packages/base/src/stacBrowser/components/StacPanel.tsx

packages/base/src/stacBrowser/hooks/useGenericStacSearch.ts

mfisher87 · 2026-01-16T19:16:49Z

Great job with the comments so far. I've left responses for some, and marked the rest resolved. After addressing the responses above, we still need to get all the checks to green ✔️ !

As @nakul-py noted the only failing job is the lint job. Linting ensures that our codebase uses consistent patterns and formatting, and the process is mostly automatic. Please either use the GitHub interface to accept and commit the suggestions from Nakul's review, and/or use the jlpm run lint command to run linting locally. It will update some files automatically, so you'll need to commit those changes and push them to the PR branch!

Co-authored-by: Nakul Verma <nakulverma.py@gmail.com>

nakul-py · 2026-01-18T05:16:49Z

Triggering CI to run Checks again.

nakul-py · 2026-01-18T05:21:36Z

@gaolinai8888 As Matt suggests earlier you can also fix the import order lint issues by using the jlpm run lint command to run linting locally. After this i think all checks should be green 🤔.

packages/base/src/stacBrowser/components/CollectionBrowser.tsx

packages/base/src/stacBrowser/hooks/useStacIndex.ts

gaolinai8888 · 2026-01-15T06:43:22Z

tsconfigbase.json

    "lib": ["ES2019", "WebWorker", "DOM"]
-  },
-  "esModuleInterop": true,
-  "resolveJsonModule": true


We don't recall removing it, and we don't think removing it is necessary for this PR to work.

packages/base/src/stacBrowser/components/CollectionBrowser.tsx

packages/base/src/stacBrowser/components/StacPanel.tsx

gaolinai8888 · 2026-01-18T06:14:44Z

@mfisher87 Hey Matt, we successfully completed the lint check and addressed the other comments too.

mfisher87 · 2026-01-22T17:40:30Z

Thanks! There's still one outstanding comment: https://github.com/geojupyter/jupytergis/pull/1062/files#r2696801641

gaolinai8888 and others added 6 commits November 19, 2025 22:52

Initial commit

191affd

Fetch data

a0e9412

Added an input to show selected catalog

f67f0fa

Merge branch 'main' into stac-browser

3d7ab27

Add layers using STAC browser

9f18197

Updated stac browser tab

1f9a8f6

geojupyter-pr-triage bot added this to JupyterGIS PR triage Jan 10, 2026

nakul-py added the enhancement New feature or request label Jan 10, 2026

mfisher87 changed the title ~~Updated Stac Browser Tab~~ Add stacindex.org catalog browsing interface Jan 10, 2026

mfisher87 changed the base branch from main to geojupyter-dse-interns January 10, 2026 17:01

mfisher87 requested changes Jan 13, 2026

View reviewed changes

gaolinai8888 and others added 14 commits January 15, 2026 05:14

Fix line 26 in CollectionBrowser

6df2870

Fix 47 ~ 62 in CollectionBrowser

2129e3a

Inline the conditionals for isCollection

c72ca04

[pre-commit.ci] auto fixes from pre-commit.com hooks

01d43ff

for more information, see https://pre-commit.ci

Remove tailwind classnames

e4d2330

Remove comments

9c3b602

Change to handleOpenSelectCatalogDialog

bb76dc4

Change to activeSearch

a5d5639

[pre-commit.ci] auto fixes from pre-commit.com hooks

3d88082

for more information, see https://pre-commit.ci

Remove unused div

56d2536

Restore Tab Styles

4b56dc5

[pre-commit.ci] auto fixes from pre-commit.com hooks

7700913

for more information, see https://pre-commit.ci

Remove tailwind styles in StacPanel

2d1bdc0

Inline the function in useStacIndex

7024f63

nakul-py requested a review from mfisher87 January 15, 2026 10:08

nakul-py suggested changes Jan 15, 2026

View reviewed changes

gaolinai8888 and others added 10 commits January 17, 2026 20:45

Fix line 83 in CollectionBrowser

891bfda

Co-authored-by: Nakul Verma <nakulverma.py@gmail.com>

Fix imports in CollectionBrowser

7f0d19d

Co-authored-by: Nakul Verma <nakulverma.py@gmail.com>

Fix line 104 in CollectionBrowser

28c132c

Co-authored-by: Nakul Verma <nakulverma.py@gmail.com>

Fix imports in StacPanel

6d36961

Co-authored-by: Nakul Verma <nakulverma.py@gmail.com>

Fix line 58 in StacPanel

db06a80

Co-authored-by: Nakul Verma <nakulverma.py@gmail.com>

Fix line 201 in StacPanel

42cf34e

Co-authored-by: Nakul Verma <nakulverma.py@gmail.com>

Fix imports in useGenericStacSearch

de7120c

Co-authored-by: Nakul Verma <nakulverma.py@gmail.com>

Fix line 38 in useGenericStacSearch

a6b0ce0

Co-authored-by: Nakul Verma <nakulverma.py@gmail.com>

Fix line 115 in useGenericStacSearch

6db26ef

Co-authored-by: Nakul Verma <nakulverma.py@gmail.com>

Fix line 48 in CollectionBrowser

5275854

Co-authored-by: Nakul Verma <nakulverma.py@gmail.com>

nakul-py closed this Jan 18, 2026

github-project-automation bot moved this to Done in JupyterGIS PR triage Jan 18, 2026

nakul-py reopened this Jan 18, 2026

gaolinai8888 commented Jan 18, 2026

View reviewed changes

gaolinai8888 and others added 5 commits January 17, 2026 21:26

Add back IJupyterGISModel import

2b5a4d6

Change to inline function in CollectionBrowser

7d9ab15

Change to inline function in useStacIndex

89f00ca

Fix lint errors

f20cb30

Fix lint errors2

532822d

mfisher87 changed the title ~~Add stacindex.org catalog browsing interface~~ [WIP] Add stacindex.org catalog browsing interface Jan 22, 2026

Revert tsconfigbase back

0cfea7d

mfisher87 merged commit 5e14ad7 into geojupyter:geojupyter-dse-interns Jan 23, 2026
13 of 14 checks passed

geojupyter-pr-triage bot removed this from JupyterGIS PR triage Jan 23, 2026

Conversation

gaolinai8888 commented Jan 10, 2026 • edited by mfisher87 Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Checklist

Uh oh!

github-actions bot commented Jan 10, 2026

Uh oh!

github-actions bot commented Jan 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mfisher87 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

gaolinai8888 commented Jan 15, 2026

Uh oh!

nakul-py left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mfisher87 commented Jan 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nakul-py commented Jan 18, 2026

Uh oh!

nakul-py commented Jan 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

gaolinai8888 commented Jan 18, 2026

Uh oh!

mfisher87 commented Jan 22, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

gaolinai8888 commented Jan 10, 2026 •

edited by mfisher87

Loading

github-actions bot commented Jan 10, 2026 •

edited

Loading

mfisher87 commented Jan 16, 2026 •

edited

Loading

nakul-py commented Jan 18, 2026 •

edited

Loading