Joe Crop

Results 37 comments of Joe Crop

Hi @DanChianucci, I finally got around to this. I don't think the problem is the `include` inside a package. When I rename the file to be `transaction_class.sv` (removed the `h`...

Thanks @DanChianucci, I am able to reproduce this now. There are quite a few places that are missing allowance of includes. I'll try and capture as many as I can...

Thanks for pointing this out. I have confirmed the bug and found a fix. The parameter detection is off. I'll merge the fix shortly.

Can you give me examples about what you are proposing?

I agree that the result from the instantiator is definitely wrong when there is an include in the definition. But are you sure that copying that include will work? I...

Seems similar to #165 with regard to the remote SSH issue.

@AndreMerendeira can you attach the file please? How many indexed objects does your workspace have?

This change is not innocuous and is breaking tests. We'll have to wait for the downstream libraries that rely on this to fix it first.