ParisNeo / lollms-webui

Lord of Large Language Models Web User Interface
https://lollms.com
Apache License 2.0
4.27k stars 537 forks source link

Update requirements.txt #463

Closed kdqemre closed 8 months ago

kdqemre commented 9 months ago

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

Checklist:

Please put an x in the boxes that apply. You can also fill these out after creating the PR.

Screenshots

If applicable, add screenshots to help explain your changes.

kdqemre commented 9 months ago

i have added opencv into requirements file which creates loop

ParisNeo commented 9 months ago

I don't want to add opencv as a requirements as people may not use the vision parts of lollms. So it is installed dynamically when the user tryed to use the vision functionnalities.

Thanks for your contribution though.