Skip to content

Feat/oct25 #16

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.

Sign up for GitHub

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
+94 −251
Open

Feat/oct25 #16

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
060a40b
fix: review edits to section 2
wildintellect Oct 24, 2025
0ae1650
fix: most review edits to section 3
wildintellect Oct 24, 2025
42f1bb0
fix: applied ruff to 03
wildintellect Oct 27, 2025
47a1b18
fix: progress on 04 fixes
wildintellect Oct 27, 2025
4e78bc8
fix: linter fix f string
wildintellect Oct 27, 2025
a94ca2e
feat: revised 05 based on #15
wildintellect Oct 28, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions docs/02-database.ipynb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@
"source": [
Copy link
Contributor

@hrodmn hrodmn Oct 28, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

We should say "... within ~2 degrees of your provided location in the last 60 days"


Reply via ReviewNB

Copy link
Author

Choose a reason for hiding this comment

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

yes, I missed that in the review. Do you want to do that, or would you rather actuall fix the dates in the temporal extent to be that date range?

Copy link
Contributor

Choose a reason for hiding this comment

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

I like the idea of using a recent set of dates, but if you think keeping a fixed range is better that's fine with me.

"# 2. The Database: pgstac\n",
"\n",
"pgstac is a PostgreSQL extension that enables STAC metadata management in a PostgreSQL database.\n",
"**pgstac** is a PostgreSQL extension that enables STAC metadata management in a PostgreSQL database.\n",
"eoAPI is useful to many organizations because the other components are configured to work seamlessly with STAC metadata that is stored in your pgstac database.\n",
"\n",
"**pypgstac** is a Python package for interacting with a pgstac database. You will learn how to use pypgstac to perform the following operations on a pgstac database:\n",
"**pypgstac** is a Python package for interacting with a pgstac database. You will learn how to use pypgstac to perform the following operations on a pgstac database:\n",
"1. Generate STAC collection record\n",
"2. Add the record to the `collections` table with `Loader.load_collections`\n",
"3. Generate STAC item records\n",
Expand All @@ -30,7 +30,7 @@
"id": "jivtk3bczsd",
"metadata": {},
"source": [
"# Workshop Setup\n",
"## Workshop Setup\n",
"\n",
"Run this cell to fetch database credentials. You'll be prompted to enter the workshop access token provided by your instructor.\n",
"\n",
Expand Down Expand Up @@ -117,7 +117,7 @@
"source": [
"## 2.1 Load a collection object\n",
"\n",
"Start by creating a collection that will contain items within ~2 degrees of your provided location between January 1 2025 and April 18 2025."
"We'll start by creating a collection, that will eventually contain items, within ~2 degrees of your provided location between January 1 2025 and April 18 2025."
]
},
{
Expand Down Expand Up @@ -178,7 +178,7 @@
"id": "af1fa92d-97b3-4b70-b7b9-b371a06741a3",
"metadata": {},
"source": [
"The `load_collections` method accepts an iterable of STAC collection dictionaries or a file path to a collection.json file or a .ndjson file with multiple collection records. You already have your `pystac.Collection` in memory in this session so you can just write it to a dictionary and upload it (in a list). \n",
"The `load_collections` method accepts an iterable of STAC collection dictionaries, a file path to a collection.json file, or a [.ndjson](https://en.wikipedia.org/wiki/JSON_streaming#Newline-delimited_JSON) file with multiple collection records. You already have your `pystac.Collection` in memory in this session so you can just write it to a dictionary and upload it (in a list, i.e. iterable object). \n",
"\n",
"The `upsert` method will add your collection to the `collections` table if it does not exist and update any fields with new values if the record does already exist."
]
Expand Down Expand Up @@ -466,7 +466,7 @@
"name": "python",
"nbconvert_exporter": "python",
"pygments_lexer": "ipython3",
"version": "3.12.12"
"version": "3.12.11"
}
},
"nbformat": 4,
Expand Down
Loading