Skip to content

fix: fix dataframe max recursion depth#1464

Closed
JoanFM wants to merge 2 commits intomainfrom
fix-dataframe-max-recursion-depth
Closed

fix: fix dataframe max recursion depth#1464
JoanFM wants to merge 2 commits intomainfrom
fix-dataframe-max-recursion-depth

Conversation

@JoanFM
Copy link
Copy Markdown
Member

@JoanFM JoanFM commented Apr 27, 2023

No description provided.

@JoanFM JoanFM linked an issue Apr 27, 2023 that may be closed by this pull request
@JoanFM JoanFM force-pushed the fix-dataframe-max-recursion-depth branch 2 times, most recently from 350ccee to efc70c3 Compare April 27, 2023 08:13
@JoanFM JoanFM requested a review from samsja April 27, 2023 08:14
@JoanFM
Copy link
Copy Markdown
Member Author

JoanFM commented Apr 27, 2023

@samsja actually my question is why ID is a subclass of BaseDoc?

@samsja
Copy link
Copy Markdown
Member

samsja commented Apr 27, 2023

@samsja actually my question is why ID is a subclass of BaseDoc?

it is not

@samsja
Copy link
Copy Markdown
Member

samsja commented Apr 27, 2023

ID is a predefined type like NdArray, TorchTensor, ImageUrl, etc. Therefore they are not BaseDoc nor PydanticModel. They just define validator for pydantic to make it work

@JoanFM
Copy link
Copy Markdown
Member Author

JoanFM commented Apr 27, 2023

@samsja check the PR and the bug it solves, the ID field type enters in this infinite loop

Signed-off-by: Joan Fontanals Martinez <[email protected]>
@JoanFM JoanFM force-pushed the fix-dataframe-max-recursion-depth branch from efc70c3 to b68a641 Compare April 27, 2023 09:24
@github-actions
Copy link
Copy Markdown

📝 Docs are deployed on https://ft-fix-dataframe-max-recursion-depth--jina-docs.netlify.app 🎉

@JoanFM
Copy link
Copy Markdown
Member Author

JoanFM commented Apr 27, 2023

We wil handle this in #1467

@JoanFM JoanFM closed this Apr 27, 2023
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.

BUg: DocList.to_dataframe

2 participants