-
Notifications
You must be signed in to change notification settings - Fork 58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bitarray dependency is unfortunate #242
Comments
Maybe Glasgow's |
I'm not going to touch pysim until 0.1, and there's a complete rewrite planned for 0.2 that will remove the dependency on Note that Glasgow depends on both anyway. |
Glasgow only uses |
I already tried to do that during migration to |
Fair enough; but in any case, using Glasgow on PyPy is a lot more niche than using nmigen on PyPy, and it has a lot more dependencies regardless. |
I thought I'd take a look at nmigen (migrating from migen), and I just ran into this issue.
I'm not actually using the simulator, so this dependency is definitely unfortunate. |
@xobs Have you actually installed nMigen's dependencies? Because there is no goal to reduce the amount of dependencies to zero. |
@whitequark I've cloned the repository next to the existing I was hoping |
@xobs - You install |
@xobs A user of nMigen is expected to be familiar with the absolute basics of Python package management. That's like saying "Yosys has dependencies, so I will stick to Vivado for now". I mean, sure, you can do that, but that's not a problem with Yosys or nMigen, it is a problem with your lack of desire to learn the fundamentals of the programming toolkit you are using. |
@mithro I don't install migen, I check it out. I don't like using For example: C:\Users\smcro>python -m pip install git+https://github.com/m-labs/nmigen.git
Collecting git+https://github.com/m-labs/nmigen.git
Cloning https://github.com/m-labs/nmigen.git to c:\users\smcro\appdata\local\temp\pip-y_btvmjq-build
Unknown requires Python '~=3.6' but the running Python is 3.5.2
You are using pip version 9.0.1, however version 19.2.3 is available.
You should consider upgrading via the 'python -m pip install --upgrade pip' command.
C:\Users\smcro> With |
@xobs nMigen requires 3.6 and above, though I'm acutely aware that upgrading Python and pip3 on Windoze can be a painful process. |
@xobs In your example pip correctly tells you that nMigen requires Python 3.6, which is written in its README file:
|
I also have the official 3.7.3 installed: C:\Users\smcro>python --version
Python 3.7.3
C:\Users\smcro>python -m ensurepip --default-pip
D:\Software\ecp5-toolchain-windows-v1.5\bin\python.exe: No module named ensurepip
C:\Users\smcro>python -m pip install git+https://github.com/m-labs/nmigen.git
D:\Software\ecp5-toolchain-windows-v1.5\bin\python.exe: No module named pip
C:\Users\smcro> |
OK, so what you're actually asking for is reliable installation instructions for Windows. I agree that this is something that is currently not provided but should be (especially given that you will also need Yosys to do anything). |
Since 3.3 Python has venv module which allows one to have different application-specific package installs. |
maybe the windows installation issue should be in a separate issue |
Just FYI - I've worked with @xobs a bit to give him some more Python background knowledge and some probably terrible advice ;-) |
@xobs FYI as of today, nMigen master is pure Python. You only need to install |
@xobs I apologize for being dismissive earlier. I now have a better understanding of the reasons someone would want to put packages on Please let me know if these changes work for you. |
@whitequark Thank you for the changes. I will take another look at nmigen. The approach is really exciting, so I'm looking forward to being able to try it out. |
It's currently broken in PyPy and I feel like I've managed to poke it into memory unsafe behaviour from the CPython REPL before. Only pysim uses it so I imagine it could pretty easily be replaced with a combination of Python's bigints and lists.
The text was updated successfully, but these errors were encountered: