Skip to content

Conversation

@avinash2692
Copy link
Contributor

No description provided.

@avinash2692 avinash2692 requested a review from nrfulton August 19, 2025 22:45
Comment on lines +56 to +70
Install with pip:

```bash
pip install mellea
```

> [!NOTE]
> `mellea` comes with some additional packages as defined in our `pyproject.toml`. I you would like to install all the extra optional dependencies, please run the following commands:
>
> ```bash
> uv pip install mellea[hf] # for Huggingface extras and Alora capabilities.
> uv pip install mellea[watsonx] # for watsonx backend
> uv pip install mellea[docling] # for docling
> uv pip install mellea[all] # for all the optional dependencies
> ```
Copy link
Contributor

Choose a reason for hiding this comment

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

This implies a mixture of uv pip and pip. Does that work as expected?

Should we just push people to uv pip.

<img src="https://github.com/generative-computing/mellea/raw/main/docs/GetStarted_py.png" style="max-width:800px">

Install with pip:
Install with [uv](https://docs.astral.sh/uv/getting-started/installation/):
Copy link
Contributor

Choose a reason for hiding this comment

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

Instead of "building it up" in a tutorial fashion, should the first command just give you something that would work out of the box for all the examples in the tutorial and only then delve into the details?

"docling>=2.45.0",
]

all = ["mellea[watsonx,docling,hf]"]
Copy link
Contributor

Choose a reason for hiding this comment

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

sweet thanks :)

@nrfulton nrfulton merged commit b1807ae into main Aug 20, 2025
3 checks passed
@nrfulton nrfulton deleted the avi/readme-update branch August 20, 2025 14:39
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.

3 participants