Skip to content

fix: remove embedding creation from pdf ingestion#83

Merged
sergioramos merged 3 commits into
mainfrom
50-remove-embedding-creation-from-pdf-ingestion
Jun 2, 2023
Merged

fix: remove embedding creation from pdf ingestion#83
sergioramos merged 3 commits into
mainfrom
50-remove-embedding-creation-from-pdf-ingestion

Conversation

@gjreda

@gjreda gjreda commented Jun 2, 2023

Copy link
Copy Markdown
Collaborator

Fixes #50

@gjreda gjreda requested review from cguedes and sehyod June 2, 2023 17:43
@gjreda gjreda marked this pull request as ready for review June 2, 2023 17:43
@gjreda

gjreda commented Jun 2, 2023

Copy link
Copy Markdown
Collaborator Author

@sehyod I think this should resolve the issues you're seeing in #63. I think it'd be good to get CI/CD working properly and then I can worry about adding this back in later, since we do not need it right now.

@gjreda gjreda mentioned this pull request Jun 2, 2023
@sergioramos sergioramos force-pushed the 50-remove-embedding-creation-from-pdf-ingestion branch from ce4b152 to 94c3bcf Compare June 2, 2023 22:29
@sergioramos sergioramos changed the title Remove embedding creation from pdf ingestion (#50) fix: remove embedding creation from pdf ingestion Jun 2, 2023
@sergioramos sergioramos merged commit 8428397 into main Jun 2, 2023
@sergioramos sergioramos deleted the 50-remove-embedding-creation-from-pdf-ingestion branch June 2, 2023 22:35
gjreda added a commit that referenced this pull request Jun 6, 2023
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.

Remove embedding creation from PDF ingestion

2 participants