search

JulianKemmerer / PipelineC

A C-like hardware description language (HDL) adding high level synthesis(HLS)-like automatic pipelining as a language construct/compiler feature.
https://github.com/JulianKemmerer/PipelineC/wiki
GNU General Public License v3.0
606 stars 50 forks source link

cleaned up utilities.py #155

Closed ghost closed 1 year ago

ghost commented 1 year ago

Added a gitignore, commented failing smoke test (TODO). Utilities.py change was pretty simple.

All imports from os were in os.path. By importing those three things directly we keep the namespace uncluttered. Looking at invocations of REPO_ABS_DIR, this isn't in the hot path. Caching seems unnecessary lacking profiling data to the contrary.

ghost commented 1 year ago

Also fixed imports (everything wants to refer back to src right now, there's probably a way to not do that but right now IDK what it is) which allows the tests to all work. Added a test for edaplay.

JulianKemmerer commented 1 year ago

Hmm it doesnt seen to see the src with modules in it - I cloned your repo and got?

  File "./src/pipelinec", line 12, in <module>
    from src import C_TO_LOGIC,OPEN_TOOLS,SIM,SYN
ModuleNotFoundError: No module named 'src'
ghost commented 1 year ago

Okay seems we've got some OS funsies? I'll try to figure it out tomorrow

ghost commented 1 year ago

Making pytest work this way made everything else not work, obviously not a good way to do things.