Well when I went to retest the PDP8e before pushing to it github I decided to re-run the diagnostics. Much to my surprise two failed! The first was the adder test, the second was the extended memory. It turns out that both of these diagnostics test to see which pages are valid and which are not! So even though I told the program it had two fields, it checked to see if there were more! It needs full fields to run these two diagnostics!
So another modification, parameters.v contains two lines that define the memory size. ram.v uses it to define the implementation. So we now have a choice of 8K or 10K. If one is going to run the diagnostics one must us the 8K version.