Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Adding Azure computer vision tool #458

Closed
wants to merge 2 commits into from

Conversation

ajhofmann
Copy link
Contributor

Azure computer vision tool for OCR and image classification

CV_URL_TMPL = "https://{resource}.cognitiveservices.azure.com/computervision/imageanalysis:analyze"

class AzureCVToolSpec(BaseToolSpec):
"""Wolfram Alpha tool spec."""
Copy link
Collaborator

Choose a reason for hiding this comment

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

docstring

Here's an example usage of the AzureCVToolSpec.

```python
from llama_hub.tools.google_search.base import AzureCVToolSpec
Copy link
Collaborator

Choose a reason for hiding this comment

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

import is wrong

@ajhofmann
Copy link
Contributor Author

Once a make a few more of these Azure tools I'll make a more advanced combined notebook playing around with some image and audio stuff!

@ajhofmann ajhofmann force-pushed the adam/azure branch 2 times, most recently from 75233cd to 3f2fd11 Compare August 10, 2023 20:49
@jerryjliu
Copy link
Collaborator

Once a make a few more of these Azure tools I'll make a more advanced combined notebook playing around with some image and audio stuff!

awesome!

Copy link
Collaborator

@jerryjliu jerryjliu left a comment

Choose a reason for hiding this comment

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

@ajhofmann would you be able to help fix merge conflicts?

@ajhofmann
Copy link
Contributor Author

@ajhofmann would you be able to help fix merge conflicts?

Should be fixed but worried the two Azure PRs are gonna create another one, I'll fix it up again if they do 👌

@ajhofmann
Copy link
Contributor Author

Combined all the Azure tools into one PR for easy of merging 🆗 https://github.com/emptycrown/llama-hub/pull/459/files

@ajhofmann ajhofmann closed this Aug 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants