byuccl / RapidSmith2

RapidSmith2 - the Vivado successor to RapidSmith. Released Jan 4, 2017.
Other
41 stars 13 forks source link

Site::instanceX and instanceY variables may not be loaded #325

Closed trharoldsen closed 6 years ago

trharoldsen commented 6 years ago

Has anyone verified that the instanceX and instanceY variables are deserialized correctly. Since changing the setName method to not parse the XY coordinates from the name, we may not be resetting them when the structure is deserialized.

If not, adding the instanceX/Y variables to the serialized object class at the bottom of the site class along with the following code should fix it.

if (instanceX != null || instancyY != null || !site.parseCoordinatesFromName(name)) { site.instanceX = (instanceX != null) ? instanceX : -1; site.instanceY = (instanceX != null) ? instanceX : -1; }

trharoldsen commented 6 years ago

Fixed in #328