Skip to content

tickets/SP-3204-patch - #190

Merged
sgreenstreet merged 2 commits into
mainfrom
tickets/SP-3204-patch
Aug 6, 2026
Merged

tickets/SP-3204-patch#190
sgreenstreet merged 2 commits into
mainfrom
tickets/SP-3204-patch

Conversation

@sgreenstreet

@sgreenstreet sgreenstreet commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Add table joins to SSObject and mpc_orbits table notebooks and update headers for SSSource, current_identifications, and numbered_identifications table notebooks. Also reduced full table queries to retrieve TOP 100 rows.

@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

Comment thread DP2/200_Data_products/201_Catalogs/201_07_SSObject_table.ipynb
@jrob93

jrob93 commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Hey, this all looks good to me. Just an extremely minor comment above about a pandas dataframe but feel free to ignore

@jrob93 jrob93 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.

I marked the comment above as resolved because it is so minor and approving here so that you can merge if it's not worth looking at.

@sgreenstreet
sgreenstreet merged commit b676aa4 into main Aug 6, 2026
2 of 3 checks passed
@sgreenstreet
sgreenstreet deleted the tickets/SP-3204-patch branch August 6, 2026 17:21
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