search

HazyResearch / safari

Convolutions for Sequence Modeling
Apache License 2.0
864 stars 71 forks source link

Make repo a python package #19

Open tbenst opened 1 year ago

tbenst commented 1 year ago

Right now, it's difficult to install or manage safari with pip This PR allows for simple install by doing pip install -r requirements.txt && pip install -e .

I've only tested from safari.models.sequence.hyena import HyenaOperator and using that as a layer, but I've tried to ensure that the relevant yaml and py files were updated.

It may be possible to do this without the src -> safari rename, but I couldn't figure it out (https://github.com/pypa/hatch/issues/870).

DanFu09 commented 1 year ago

Thanks for this PR! We'll take a look and see if it makes sense to merge in.