Skip to content
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

Jump to bottom

CSAE Bot for search across the notebooks #807

Merged
merged 9 commits into from
Feb 4, 2025
Merged

CSAE Bot for search across the notebooks #807

merged 9 commits into from
Feb 4, 2025

Conversation

chetan-hirapara
Copy link
Collaborator

CSAE Bot for search across the notebooks.

@chetan-hirapara chetan-hirapara added the enhancement Adding graphs and improving output label Jan 17, 2025
@chetan-hirapara chetan-hirapara self-assigned this Jan 17, 2025
@chetan-hirapara
Copy link
Collaborator Author

@DallasBowden - please review this PR

@chetan-hirapara chetan-hirapara mentioned this pull request Jan 22, 2025
Open
@chetan-hirapara
Copy link
Collaborator Author

@JohnThumaTD @DallasBowden - Please review this PR

@JohnThumaTD - Added low code UI but due to technical limitations, removed hyperlink from source documents section.

DallasBowden
DallasBowden previously approved these changes Feb 4, 2025
View reviewed changes
Copy link
Collaborator

@DallasBowden DallasBowden left a comment

Choose a reason for hiding this comment

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

I pushed the most of the functions definitions into another .py file. We should try to alter the prompting so it doesn't bring back releasenotes.ipynb for every query.

DallasBowden
DallasBowden previously approved these changes Feb 4, 2025
View reviewed changes
Copy link
Collaborator

@DallasBowden DallasBowden left a comment

Choose a reason for hiding this comment

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

Removed old .yaml file and added 2 new one.

@DallasBowden
Update .ExperienceBot.yaml
18d3117 
Updated cell number for ai key.
@DallasBowden
Update .ExperienceBot.yaml
7bb7ee0 
Updated .yaml file.
@DallasBowden
Update .ExperienceBot.yaml
2b74a44 
There has to be a space between - and type.
DallasBowden
DallasBowden approved these changes Feb 4, 2025
View reviewed changes
Copy link
Collaborator

@DallasBowden DallasBowden left a comment

Choose a reason for hiding this comment

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

Approved after .yaml file updated for successful playwright test.

@DallasBowden DallasBowden merged commit dc18af3 into main Feb 4, 2025
1 check passed
@DallasBowden DallasBowden deleted the csae_bot3 branch February 4, 2025 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DallasBowden DallasBowden DallasBowden approved these changes

@DougEbel DougEbel Awaiting requested review from DougEbel DougEbel is a code owner

Assignees

@chetan-hirapara chetan-hirapara

Labels
enhancement Adding graphs and improving output
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chetan-hirapara @DallasBowden