deepsparse icon indicating copy to clipboard operation
deepsparse copied to clipboard

Revert Haystack to Auto Install

Open kylesayrs opened this issue 3 years ago • 0 comments

This PR reverts the changes made in https://github.com/neuralmagic/deepsparse/pull/532. It was found that installing Haystack dependencies through extras was not feasible. Haystack requires HF transformers as a dependency. However, having HF transformers breaks the auto install of NM transformers, as there is no good way to dynamically reload modules in python. There is also no good way to install Haystack in extras without installing its dependencies.

Given these blocks, reverting back to dynamical install fixes these issues by controlling which dependencies are need by Haystack through a requirements.txt file. We can then remove HF transformers from this dependency list, thus avoiding the issue of dynamic module reload.

kylesayrs avatar Aug 10 '22 18:08 kylesayrs