Skip to content

Conversation

romnovi
Copy link

@romnovi romnovi commented Sep 10, 2024

DOCUMENT_ID is actually node.node_id now.
It's difficult to track documents which are found by retrieval stage.
I suggest adding ref_doc_id in the attributes.

@romnovi romnovi requested a review from a team as a code owner September 10, 2024 20:47
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Sep 10, 2024
Copy link

vercel bot commented Sep 10, 2024

@mystdeim is attempting to deploy a commit to the Arize Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

github-actions bot commented Sep 10, 2024

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@romnovi
Copy link
Author

romnovi commented Sep 10, 2024

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Sep 10, 2024
@RogerHYang RogerHYang self-requested a review September 11, 2024 03:34
@RogerHYang
Copy link
Contributor

OK I now understand the issue. Thank you for bringing this to our attention.

Let me discuss it with the team first, and follow up with you afterwards.

@RogerHYang RogerHYang self-assigned this Sep 12, 2024
@mikeldking mikeldking requested review from nate-mar and removed request for RogerHYang March 21, 2025 18:19
@mikeldking mikeldking removed this from phoenix Jun 9, 2025
codefromthecrypt pushed a commit to codefromthecrypt/openinference that referenced this pull request Sep 15, 2025
**Description**

we should trigger a reconciler after bsps are observed by the secret
controller.

**Related Issues/PRs (if applicable)**

Fixes Arize-ai#1025 

Signed-off-by: carlory <[email protected]>
@mikeldking
Copy link
Collaborator

Apologies from us that this never made it in. I think we never fully resolved this. Is it still something of interest?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants