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

fix/unstructured #93

Merged
merged 9 commits into from
May 28, 2024
Merged

fix/unstructured #93

merged 9 commits into from
May 28, 2024

Conversation

nRamstedt
Copy link
Member

Temporary fix for running application on elastix with unstructured. The final docker image size has increased after inculding unstructured package and are too big to run in CI/CD pipelines etc making this fix temporary.

@nRamstedt nRamstedt requested a review from dannil76 May 22, 2024 10:05
@zilaei
Copy link
Member

zilaei commented May 23, 2024

@nRamstedt Fixed one of the test for parsing PDF files to also include new lines.
This test will fail with the unstructured version set for this branch but passes with version in main.

@nRamstedt
Copy link
Member Author

revert your commit since it broke test. PR sabotage !!111 :) @zilaei

@nRamstedt nRamstedt merged commit 7fb7bec into master May 28, 2024
2 checks passed
@nRamstedt nRamstedt deleted the fix/unstructured branch May 28, 2024 21:35
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.

2 participants