Skip to content

Conversation

@arjunpatel7
Copy link
Contributor

Describe the purpose of this change. What problem is being solved and why?

Replaced current install pattern with UV for a much faster experience. Added text around how API keys work to see how that affects completion rates.

@arjunpatel7 arjunpatel7 self-assigned this Aug 6, 2025
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@arjunpatel7
Copy link
Contributor Author

The notebook runs fine but the test doesn't properly grab the new uv installs. I'll see if I can change that to make the test work as intended

… but we'll ignore them for now to see the effect on ease of use
@arjunpatel7
Copy link
Contributor Author

Tests are not passing but we'll experiment with this format for now and watch usage.

Copy link
Contributor

@jennapederson jennapederson left a comment

Choose a reason for hiding this comment

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

LGTM

@arjunpatel7 arjunpatel7 merged commit 902ec2d into master Aug 7, 2025
8 of 9 checks passed
@arjunpatel7 arjunpatel7 deleted the arjun-semantic-search-uv-text branch August 7, 2025 22:06
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.

3 participants