Not sure why RAM8 implementation does not work

[davidtaylor23]

Hi, I am a bit unsure why my RAM8 implementation does not work:

MOD NOTE: Code removed since issue is resolved.

Any help would be greatly appreciated!

[WBahn]

I assume you say that it doesn't work because it failed the tests?

Look at the test it failed. First look at the contents of the compare file and satisfy yourself that they are correct for the input conditions used in that test.

Then, for the input conditions used in that test, walk through your logic and see if you understand why you get the results in your output file.

Then ask why you get your results and not the correct results.

[davidtaylor23]

I am not sure whether it is that it fails a test or there is some syntax error as it will not actually load onto the simulator in this form for some reason, so I am unsure where exactly the problem is.

[WBahn]

What error message does it give you when you try to load it into the simulator?

[davidtaylor23]

No error message, it just does not load at all i.e the hardware simulator remains blank. For the other projects, it has worked, but usually this not loading thing happens when there is some syntax error in the hdl but I can't seem to find one here in my implementation.  

[WBahn]

It usually tells you what line number the problem occurred on, though often the message is very cryptic.

Post your entire file so that I can see what the problem is. I can't work very will with partial contents on a problem like this.

[WBahn]

One issue I see right away is this:

Not(in=address[0], out=notaddress);
DMux4Way(in=notaddress, sel[0]=address[1], sel[1]=address[2], a=a, b=b, c=c, d=d);
DMux4Way(in=address, sel[0]=address[1], sel[1]=address[2], a=e, b=f,c=g,d=h);

The signal 'address' is clearly a multibit signal, but 'notaddress' is not.

Yet in one DMux4Way part you tie 'address' to 'in' and in the other you tie 'notaddress' to 'in'.

At least one of those has to be wrong, since 'in' for that part is either a multibit signal or it's not.

[davidtaylor23]

Ah I can't believe I did not notice that! It is now working :) Thank you for your help

[WBahn]

Glad to hear it.

I'm going to delete your logic from your first post since the issue is resolved.