Skip to content

S2S Client Field Selector Widgets#114

Merged
andresfchamorro merged 17 commits intomainfrom
feature/multi-field-selector-ui-demo
May 30, 2025
Merged

S2S Client Field Selector Widgets#114
andresfchamorro merged 17 commits intomainfrom
feature/multi-field-selector-ui-demo

Conversation

@Gabe-Levin
Copy link
Copy Markdown
Collaborator

@Gabe-Levin Gabe-Levin commented Mar 12, 2025

What I Changed:

  • Created a Cross Section Data Field Selector widget
  • Created a Time Series Data Field Selector widget
  • Integrated both widgets into the python client
  • Updated the ui demo notebook to show an example of both widgets

How to test:

  • cd to space2stats_client directory
  • run pip install -e . ( this should download version 1.2.1)
  • run the demo ui notebook: docs/user-docs/space2stats_api_ui_demo.ipynb
  • run tests using pytest from: space2stats_client/tests

Next Steps:

  • Get and incorporate feedback on current UI
  • Potentially add the AOI map selector widget to the python client. see docs/user-docs/space2stats_api_demo_urban_flood_risk.ipynb

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@Gabe-Levin Gabe-Levin temporarily deployed to Space2Stats API Dev March 12, 2025 10:39 — with GitHub Actions Inactive
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 12, 2025

PR Deployment Details:
🚀 PR deployed to https://hzkzbqrseg.execute-api.us-east-1.amazonaws.com/

@Gabe-Levin Gabe-Levin force-pushed the feature/multi-field-selector-ui-demo branch from 95d1dd9 to f42fe56 Compare April 30, 2025 11:04
@Gabe-Levin Gabe-Levin temporarily deployed to Space2Stats API Dev April 30, 2025 11:33 — with GitHub Actions Inactive
@Gabe-Levin Gabe-Levin changed the title WIP: Multi Field Selector UI Demo WIP: S2S Client Field Selector Widgets May 1, 2025
@Gabe-Levin Gabe-Levin temporarily deployed to Space2Stats API Dev May 1, 2025 11:08 — with GitHub Actions Inactive
@Gabe-Levin Gabe-Levin temporarily deployed to Space2Stats API Dev May 9, 2025 12:39 — with GitHub Actions Inactive
@Gabe-Levin Gabe-Levin force-pushed the feature/multi-field-selector-ui-demo branch from c23d923 to bbecbd1 Compare May 9, 2025 12:47
@Gabe-Levin Gabe-Levin temporarily deployed to Space2Stats API Dev May 9, 2025 12:51 — with GitHub Actions Inactive
@andresfchamorro
Copy link
Copy Markdown
Collaborator

Finally got around to test this, works on my windows with visual studio code! will do a few more tests with jupyter lab

@andresfchamorro
Copy link
Copy Markdown
Collaborator

image
@Gabe-Levin why do the flood variables show up as uncategorized? Shouldn't it pull the item name?

@Gabe-Levin Gabe-Levin temporarily deployed to Space2Stats API Dev May 23, 2025 12:20 — with GitHub Actions Inactive
@Gabe-Levin
Copy link
Copy Markdown
Collaborator Author

@andresfchamorro good catch. I meant to resolve this but it got lost in the fold. I have fixed the bug. Essentially the grouping function for the categories was skipping over STAC Items with a Theme = 'climate', rather than the STAC Item name which I had originally intended. In the end, I told it to skip over the stac items that have a startdate field.

I also filtered out the 'ogc_fid' field up-stream, so it is no longer returned when calling the fields endpoint. This removes the 'Categorized' Section completely.

@Gabe-Levin Gabe-Levin changed the title WIP: S2S Client Field Selector Widgets S2S Client Field Selector Widgets May 23, 2025
@Gabe-Levin Gabe-Levin temporarily deployed to Space2Stats API Dev May 23, 2025 13:13 — with GitHub Actions Inactive
@Gabe-Levin Gabe-Levin temporarily deployed to Space2Stats API Dev May 23, 2025 13:25 — with GitHub Actions Inactive
@andresfchamorro
Copy link
Copy Markdown
Collaborator

Now I'm seeing some other uncategorized variables, probably from old tables we were testing?

image

@andresfchamorro andresfchamorro temporarily deployed to Space2Stats API Dev May 28, 2025 18:56 — with GitHub Actions Inactive
@andresfchamorro andresfchamorro self-requested a review May 30, 2025 17:03
@andresfchamorro andresfchamorro merged commit ae4cb28 into main May 30, 2025
6 checks passed
@andresfchamorro andresfchamorro temporarily deployed to Space2Stats API Dev May 30, 2025 17:04 — with GitHub Actions Inactive
@andresfchamorro andresfchamorro deleted the feature/multi-field-selector-ui-demo branch May 30, 2025 17:04
@andresfchamorro andresfchamorro temporarily deployed to Space2Stats API Dev May 30, 2025 17:08 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants