search

VLSIDA / OpenRAM

An open-source static random access memory (SRAM) compiler.
http://www.openram.org
BSD 3-Clause "New" or "Revised" License
781 stars 198 forks source link

regression test fail: subunit file missing #115

Closed bvhoof closed 2 years ago

bvhoof commented 3 years ago

Hi,

I pulled commit da3a1003019e31fb8cf8089bb568b093486a23c2 now, I cannot run the regression tests any more. it seems the subunit.py file is not included.

Best regards, Bob.

bvhoof commented 3 years ago

It seems to have come from commit 19e99d1. Then, you added the parallel regression testing.

mguthaus commented 3 years ago

It requires an additional Python dependency. All the dependencies are listed in a requirements.txt now as well so you can install with pip.

mguthaus commented 3 years ago

Note, I need to update the README about these requirements.

bvhoof commented 3 years ago

Oh, I was not aware that this was a python package. I also needed to install scikit-learn. I run your tool in an anaconda environment. Do you want me to put an environment yaml in the main git repo? This might make things easier for other people to use.

Best, Bob.

mguthaus commented 3 years ago

Sure, that would be great. I personally don't use anaconda but I know that some may.

-- Matthew Guthaus (he/him) Associate Dean of Graduate Studies Professor, Computer Science & Engineering Baskin School of Engineering University of California Santa Cruz https://www.soe.ucsc.edu/people/mrg

On Wed, Mar 10, 2021, 05:50 bvhoof @.***> wrote:

Oh, I was not aware that this was a python package. I also needed to install scikit-learn. I run your tool in an anaconda environment. Do you want me to put an environment yaml in the main git repo? This might make things easier for other people to use.

Best, Bob.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/VLSIDA/OpenRAM/issues/115#issuecomment-795438744, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC67SL2FQUCFOUHZDPDLUNLTC52J7ANCNFSM4Y5OWJKQ .

mguthaus commented 2 years ago

I created a requirements.txt for the Python requirements. This can be used with pip install. 5a0c75d573dfa7cab187b86f910c26c4e2f59618