search

pymtl / pymtl3

Pymtl 3 (Mamba), an open-source Python-based hardware generation, simulation, and verification framework
BSD 3-Clause "New" or "Revised" License
388 stars 46 forks source link

pytest should't be in install_requires in seup.py #246

Open yurivict opened 1 year ago

yurivict commented 1 year ago

pytest is only required for testing, and not for running the module.

ptpan commented 11 months ago

Hi @yurivict,

Sorry for getting back to you so late. I think you are correct that pytest is used for testing and not for running the PyMTL package. However we do have a set of testing methodologies that build on pytest (and hypothesis). There are many use cases of that methodology and we think it makes sense to include pytest and hypothesis by default.

Does having pytest/hypothesis as a dependency complicate your use case? I think so far our experiences has been pretty smooth, and pip install does pretty good jobs handling the installation..

Peitian