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.

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

GH-45665: [Docs] Add kapa AI bot to the docs #45667

Merged
merged 5 commits into from
Mar 5, 2025

Conversation

thisisnic
Copy link
Member

@thisisnic thisisnic commented Mar 4, 2025

Rationale for this change

Add kapa AI bot to the docs to make it easier to find answers to questions

What changes are included in this PR?

Config for enabling bot

Are these changes tested?

Manually, yep

Are there any user-facing changes?

Yep, adds widget to header in docs pages

@thisisnic
Copy link
Member Author

@github-actions crossbow submit preview-docs

Copy link

github-actions bot commented Mar 4, 2025

Revision: 64873ba

Submitted crossbow builds: ursacomputing/crossbow @ actions-2469749b9e

Task Status
preview-docs GitHub Actions

@assignUser
Copy link
Member

http://crossbow.voltrondata.com/pr_docs/45667/ sadly I only get There was an error in generating the answer. Please refresh the page and try again. probably because it's not coming from the expected url?

@thisisnic
Copy link
Member Author

Exactly that; have enabled there now. The button I created didn't work, but the other one at the bottom right does so I'll experiment with making that a bit nicer!

@thisisnic
Copy link
Member Author

@github-actions crossbow submit preview-docs

Copy link

github-actions bot commented Mar 4, 2025

Revision: b88e50b

Submitted crossbow builds: ursacomputing/crossbow @ actions-a4683479a2

Task Status
preview-docs GitHub Actions

@thisisnic
Copy link
Member Author

http://crossbow.voltrondata.com/pr_docs/45667

@assignUser
Copy link
Member

Works for me now, cool!

@assignUser
Copy link
Member

I put the recent ML question about encryption into it and received an accurate (no hallucination I could detect) answer that even expanded on the answers given on the ML with page_checksum_verification. 🚀 very nice!

@AlenkaF
Copy link
Member

AlenkaF commented Mar 5, 2025

This is great! I like how it lists the sources! I inquired about the struct layout and got a nice answer. Not too long! ;) Asking about the PyCapsule protocol implementation I like that it emphasises the fact that it is currently experimental and can change in the future.

Thank you for working on this @thisisnic!

@AlenkaF
Copy link
Member

AlenkaF commented Mar 5, 2025

Aha, and the button is nice!

@thisisnic thisisnic marked this pull request as ready for review March 5, 2025 11:06
@thisisnic thisisnic requested a review from jonkeane as a code owner March 5, 2025 11:06
@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting committer review Awaiting committer review labels Mar 5, 2025
@thisisnic
Copy link
Member Author

It wasn't working with the R docs, but now appears to be sorted - this is ready for review! Once it's merged and is up on the dev docs, I'll put a note on on the dev mailing list asking folks to test it out and give feedback on the answers to see where we can improve them.

@thisisnic thisisnic merged commit 784aa6f into apache:main Mar 5, 2025
13 checks passed
@thisisnic thisisnic removed the awaiting changes Awaiting changes label Mar 5, 2025
Copy link

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 784aa6f.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 59 possible false positives for unstable benchmarks that are known to sometimes produce them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants