Jacob Brown

Results 55 comments of Jacob Brown

Thanks for posting this. This definitely needs to be fixed. A quick fix for now should could be to uniquify the netlist before flattening. ``` from spydrnet.uniquify import uniquify uniquify(netlist)...

Looking at it a little closer, it seems that you must run uniquify before flattening. So I don't think this is a bug. The documentation should be improved to indicate...

I'm not super familiar with the namespace manager. However, line 107 seems like it's checking to make sure that parent is not None and that it's in self.namespaces.

So it looks like it's an issue with upper-case and lower-case. The namespace manager does not take case into account when checking for duplicate names. So F1_reg_TMR_0_Q_VOTER and f1_reg_TMR_0_Q_VOTER are...

It's also interesting that your generated netlist had the same names just different case. Where did the netlist come from?

> Yeah I eventually figured that out. Parent and Parent seems redundant though As for this, it may seem redundant but I don't think it is. It's checking to make...

That pull request looks like it gives the case sensitive option to the parser but not the composer. We probably need to add to it.

See the case_insensitive_naming branch for some work on this.

Note that this is related to issue #116

Note: the case_insensitive_naming branch contains code to tell the EDIF parser to be case sensitive for naming. I merged this code into next release but then removed it, so github...