Skip to content

[SYNPY-1893] Add search index tutorial - #1447

Merged
thomasyu888 merged 14 commits into
developfrom
SYNPY-1893-search-index-tutorial
Aug 26, 2026
Merged

[SYNPY-1893] Add search index tutorial#1447
thomasyu888 merged 14 commits into
developfrom
SYNPY-1893-search-index-tutorial

Conversation

@thomasyu888

@thomasyu888 thomasyu888 commented Aug 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Problem:

No search index tutorial

Solution:

Added tutorial

Testing:

https://synapsepythonclient--1447.org.readthedocs.build/en/1447/tutorials/python/search/

BryanFauble
BryanFauble reviewed Aug 17, 2026
View reviewed changes
Comment thread docs/tutorials/python/tutorial_scripts/search.py Outdated
@thomasyu888
thomasyu888 marked this pull request as ready for review August 21, 2026 04:23
@thomasyu888
thomasyu888 requested a review from a team as a code owner August 21, 2026 04:23
Copilot AI lite review requested due to automatic review settings August 21, 2026 04:23
thomasyu888
thomasyu888 commented Aug 21, 2026
View reviewed changes
Comment thread docs/tutorials/python/search.md Outdated
Copilot AI reviewed Aug 21, 2026

Copilot AI 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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@thomasyu888
thomasyu888 requested a lite review from Copilot August 21, 2026 04:25
Copilot AI reviewed Aug 21, 2026

Copilot AI 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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

andrewelamb
andrewelamb requested changes Aug 21, 2026
View reviewed changes
Comment thread docs/tutorials/python/tutorial_scripts/search.py Outdated
Comment thread docs/tutorials/python/tutorial_scripts/search.py
Comment thread docs/tutorials/python/tutorial_scripts/search.py Outdated
Comment thread docs/tutorials/python/tutorial_scripts/search.py Outdated
BryanFauble
BryanFauble reviewed Aug 21, 2026
View reviewed changes
Comment thread docs/tutorials/python/tutorial_scripts/search.py
BryanFauble
BryanFauble reviewed Aug 21, 2026
View reviewed changes
Comment thread docs/tutorials/python/tutorial_scripts/search.py Outdated
BryanFauble
BryanFauble reviewed Aug 21, 2026
View reviewed changes
Comment thread docs/tutorials/python/search.md Outdated
BryanFauble
BryanFauble reviewed Aug 21, 2026
View reviewed changes
Comment thread docs/tutorials/python/search.md Outdated
BryanFauble
BryanFauble reviewed Aug 21, 2026
View reviewed changes
Comment thread docs/tutorials/python/search.md Outdated
BryanFauble
BryanFauble approved these changes Aug 21, 2026
View reviewed changes

@BryanFauble BryanFauble left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There are a few suggested changes to improve this, but I'll approve now because it's shaping up well.

Feel free to ignore the build status on this PR since it isn't affecting anything tested.

thomasyu888
thomasyu888 commented Aug 21, 2026
View reviewed changes
Comment thread docs/tutorials/python/search.md
thomasyu888
thomasyu888 commented Aug 21, 2026
View reviewed changes
Comment thread docs/tutorials/python/search.md Outdated
thomasyu888
thomasyu888 commented Aug 21, 2026
View reviewed changes
Comment thread docs/tutorials/python/search.md Outdated
thomasyu888
thomasyu888 commented Aug 21, 2026
View reviewed changes
Comment thread docs/tutorials/python/search.md Outdated
thomasyu888
thomasyu888 commented Aug 21, 2026
View reviewed changes
Comment thread docs/tutorials/python/search.md
@thomasyu888
thomasyu888 merged commit 526d515 into develop Aug 26, 2026
19 of 23 checks passed
@thomasyu888
thomasyu888 deleted the SYNPY-1893-search-index-tutorial branch August 26, 2026 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BryanFauble BryanFauble BryanFauble approved these changes
Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@andrewelamb andrewelamb andrewelamb requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@thomasyu888 @andrewelamb @BryanFauble