search

UFO-101 / auto-circuit

A library for efficient patching and automatic circuit discovery.
https://UFO-101.github.io/auto-circuit
18 stars 8 forks source link

AutoCircuit datasets not included with `pip install` #3

Open dtch1997 opened 4 months ago

dtch1997 commented 4 months ago

The demos usually rely on having loaded data from datasets, but this does not appear to be included in the PyPI wheel. To get access to the data, I had to clone the github repo and install from source. If this is intended, then I think the docs should clarify that datasets must be downloaded separately

UFO-101 commented 4 months ago

It is intentional, because it would make it the wheel quite bloated.

Do you think they should be included?

dtch1997 commented 4 months ago

No strong opinion on whether they should be included! But currently I think the docs are a bit misleading, because the examples will not work out of the box after 'pip install auto-circuit'.