6004x / jade Goto Github PK

Hi and thanks for sharing this software.

I browsed the documentation at https://6004.mit.edu/ and the source tree in search for licensing information.

Is there a license file, or reference to an existing license, that I have missed?
If not, can you please provide details about the conditions for using your code?

Browsing to the python server results in a blank white page containing only:

{heirarchical: true}

jade.html is requesting the unminified version of jquery, but the jade repository only has the minified version.

specifying the minified version on line 5 of jade.html fixes it

<script type="text/javascript" src="jquery-3.2.1.min.js"></script>
                                                 ^^^

Here's a diff drhodes@25c587c that adds a button to the export dialog. It looks at all the modules the user has selected and recursively finds their submodules. Then, all the submodules are checked off in the dialog, ready to be exported.

I've just set up Jade to run locally (localhost:8000/jade.html) and imported my Edx6004 modules successfully. Functionally, everything works as it did with the Edx online version, however, some gates that have been rotated from east/west to north/south orientation disappear and are replaced with circles. I'm using Chrome on a Mac. I've tried Firefox and Safari with the same result.
Here's a screen shot:

Our class is using Jade to design a simple computer and we'd like some way to use an I/O device such as a keyboard or monitor.

Here's the stdout:

git.exe clone -v --progress "[email protected]:eric-wieser/jade.git" jade
Cloning into 'jade'...
remote: Total 4276 (delta 0), reused 0 (delta 0), pack-reused 4273        
Checking connectivity... done.
error: unable to create file node_modules/grunt-contrib-qunit/node_modules/grunt-lib-phantomjs/node_modules/phantomjs/node_modules/request/node_modules/form-data/node_modules/combined-stream/node_modules/delayed-stream/test/integration/test-delayed-http-upload.js (Filename too long)
error: unable to create file node_modules/grunt-contrib-qunit/node_modules/grunt-lib-phantomjs/node_modules/phantomjs/node_modules/request/node_modules/form-data/node_modules/combined-stream/node_modules/delayed-stream/test/integration/test-delayed-stream-auto-pause.js (Filename too long)
error: unable to create file node_modules/grunt-contrib-qunit/node_modules/grunt-lib-phantomjs/node_modules/phantomjs/node_modules/request/node_modules/form-data/node_modules/combined-stream/node_modules/delayed-stream/test/integration/test-delayed-stream-pause.js (Filename too long)
error: unable to create file node_modules/grunt-contrib-qunit/node_modules/grunt-lib-phantomjs/node_modules/phantomjs/node_modules/request/node_modules/form-data/node_modules/combined-stream/node_modules/delayed-stream/test/integration/test-handle-source-errors.js (Filename too long)
fatal: unable to checkout working tree
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry the checkout with 'git checkout -f HEAD'

In chrome on linux,

1. select an any number of gate circuit elements
2. do gate level timing analysis
3. a timing analysis window pops up, close that window
4. move cursor slightly
5. poof, all the selected elements disappear.

A mouse click anywhere on the grid will bring them back.

In firefox the behavior is slightly different. Step 4 isn't required and although the elements disappear their terminal circles remain visible.

When I click "Input Module" and check one of the boxes, nothing happens.

Over on line 239 of plot.js, there's a comment about using keypress events to pan the waveforms. I got to hunting around on this because my trackball doesn't have a scroll wheel, anyways, here's a fix.

There's more info over at this stackoverflow post, including a different workaround that adds the tab index directly to html, but I haven't tested that.

Sometimes resizing the plot window causes the window contents dimensions to go haywire, so, I looked into the css flexbox feature that is documented well in this guide.

Here's a new git branch that changes the layout to use flex for the plot window. So far, so good - and moves some fiddly js window calculations to the CSS renderer. Here's a plot window that uses flexbox.

Here's the diff.

Jade would be more ergonomic for mouse-based interaction if it supported panning by holding any area of the background and dragging it, or zooming in by selecting a rectangular area, holding, and pressing Enter. For touch screens, this input method would still work for panning, but pinching to zoom should be used instead.

nested component selection

There was an idea about decimal literals in signal names. Sometimes we get stuck in a false assumption thinking that 1010'8 means 0b00001010, but is actually silently interpreted as one-thousand-and-ten. A first glance might suggest that all decimal literals should use a 0d prefix, like 0d10 to explicitly indicate the number ten. After some consideration, it's clear that the proposed fix is heavy handed because of the prevalence of signals named like A[31:0].

A[0d31:0] is too noisy, but here's a compromise. Only signal names containing decimal literals with certain patterns of 1 and 0 would fall under scrutiny and be rejected as invalid.

unamiguous lits, like [0, 1, 00, 000, 42, 31...] would be accepted.
[10, 11, 11110, ...] would be rejected

Would this break too many schematics? Thoughts?

Steps to reproduce:

1. Do a git clone repository.
2. Run transient simulation on the RC circuit.
3. Get the error in the screenshot.

I think this occurred sometime between now and last September, as it used to work then.

Thanks for all your good work Chris!
Rob

Are there any dependencies to run server.py? If so, these should be put in a requirements.txt file.

Also, you can just open jade_workbook.html in a browser window without doing anything extra and it just works. This should be reflected in README.md.

This would probably help hasten the debug cycle. Shown below is a result split into two parts: a "got" value located in the top and an "expected" value in the bottom.

It would be great if we could turn Jade into a package for the Atom text editor with the ability to load circuits from anywhere on disk and export circuit diagrams to SVG (possibly animated) or PDF.