-
Notifications
You must be signed in to change notification settings - Fork 30
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
Invoice Extraction #84
Conversation
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update. I have some comments here. Also, please add this notebook's path to .github/notebook_lists/vanilla_notebooks.txt so the notebooks will be tested in the build.
Also, please rebase upon the tip of the main branch.
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Also, please add this notebook's path to .github/notebook_lists/vanilla_notebooks.txt so the notebook will be tested in the build. |
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
I have changed the invoice path as mentioned above and added the notebook path to the vanilla_notebooks.txt file. |
Testing Notebooks workflow launched on this PR: View run |
@adampingel I think this is ready to merge. |
|
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
|
Testing Notebooks workflow launched on this PR: View run |
Testing Notebooks workflow launched on this PR: View run |
@SinghSuryaDeep The latest ci test fails with
I am not sure exactly what is going on, but it seems to be a problem with docling. |
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Signed-off-by: Surya Deep Singh <[email protected]>
Hi @bjhargrave and @adampingel |
Testing Notebooks workflow launched on this PR: View run |
Just a gentle reminder regarding the PR. Please let me know if any action or additional information is required from my side. If not, may we proceed with merging? Thanks in advance! |
Thanks @SinghSuryaDeep! |
PR Checklist
Model Interaction
Data
Notebook requirements
!pip install git+https://github.com/ibm-granite-community/utils
in the first code cell in order to makeget_env_var
available to accessing secrets and variables in the recipe.Incoming References
GitHub