search

Farama-Foundation / SuperSuit

A collection of wrappers for Gymnasium and PettingZoo environments (being merged into gymnasium.wrappers and pettingzoo.wrappers
Other
452 stars 57 forks source link

Changed openCV requirements #127

Closed RichardDev01 closed 2 years ago

RichardDev01 commented 2 years ago

We changed the requirements of opencv to be more flexible. Is it okey if this request is pushed to the main branche?

jkterry1 commented 2 years ago

If I merge this, it'll install the newest version of opencv by default, which most people can't easily install

RichardDev01 commented 2 years ago

yes, indeed, by default the newest version would be installed then. I don't know which kind of problem most people would run in to but with this change the option to install older version is still on the table if people get installing issues.

jkterry1 commented 2 years ago

So we're going to have a release that gets rid of opencv all together in favor of something faster in a month or so. Can you just install supersuit with --no-deps and specify your own version of opencv until then?