microsoft / microsoft/OmniParser

Remove unused packages

Open
#85 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Jupyter Notebook
Stars
25.4k
Forks
2.2k
PR merge metrics
No merged PRs in 30d

Description

The following three lines can be removed.

https://github.com/microsoft/OmniParser/blob/a14c4010ebd91aae83a240badadc9e5721aaf0c8/utils.py#L11
https://github.com/microsoft/OmniParser/blob/a14c4010ebd91aae83a240badadc9e5721aaf0c8/requirements.txt#L5
https://github.com/microsoft/OmniParser/blob/a14c4010ebd91aae83a240badadc9e5721aaf0c8/requirements.txt#L8

azure-identity and openai are listed in requirements.txt, but they are not actually used in the code. AzureOpenAI is imported from openai, but it's never used.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with utils.py at the linked import and requirements.txt at the two linked lines. Check that azure-identity and openai, including AzureOpenAI, are unused elsewhere, then remove the three specified lines. Done means the unused imports and package entries are gone without affecting the project’s dependency list.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
build-system
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.