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

Remove the Form Recognizer (Document Intelligence) resources and para… #187

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

JasonHaley
Copy link
Contributor

I see a Azure Document Intelligence resource deployed when provisioning to Azure, but I don't see any usage of it.

In the other RAG demos it is used when processing the PDF file, but it looks like this demo uses the mozilla/pdfjs-dist for that functionality.

Am I missing something?

I can create a PR to remove it (if I am correct).

This issue is for a: (mark with an x)

- [x ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

azd auth login
azd up

Expected/desired behavior

only deploy resources that are used

@JasonHaley
Copy link
Contributor Author

@microsoft-github-policy-service agree

@anfibiacreativa anfibiacreativa requested review from anfibiacreativa and sinedied and removed request for anfibiacreativa February 26, 2024 13:51
@anfibiacreativa anfibiacreativa added the needs-triage Issues to be triaged by maintainers label Feb 26, 2024
Copy link
Contributor

@sinedied sinedied left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! We initially planned to use the form recognizer, but forgot to clean up :)

@sinedied sinedied merged commit 0fec15c into Azure-Samples:main Feb 26, 2024
8 checks passed
@anfibiacreativa anfibiacreativa added backend Backend features or code and removed needs-triage Issues to be triaged by maintainers labels Mar 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Backend features or code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants