Shape.edges() fails to return majority of edges

[MatthiasJ1]

s = GridLocations(10,10,5,5) * Rectangle(10,10)
s = Curve() + [f.wire() for f in s]

creates the following geometry:

e = s.edges()

returns the following edges:

[gumyr]

The problem here is with s = Curve() + [f.wire() for f in s] as this attempts to create a Wire given all of the inputs are Wires but that doesn't work as a single Wire can't be created from these objects. There needs to be more logic in this to ensure a Wire can actually be created, if not a Compound needs to be created which isn't easy given the structure of the code.

The quick fix is to do: s3 = Compound() + [f.wire() for f in s].

[MatthiasJ1]

show(s) shows all of the expected edges though

[jdegenstein]

What is the output of show_topology?

[MatthiasJ1]

show_topology gives all the edges. show has to be getting them from somewhere so I would assume they are there.

[snoyer]

Most likely a BRepTools_WireExplorer vs TopoDS_Iterator question, see #864

[gumyr]

I should have clarified - show_topology does show the edges but we're kinda in a weird state where this isn't really a good Wire and other wire methods are going to have trouble. As a solution how about the __add__ method of Curve/Compound keeps the object as a Curve/Compound and never attempts to create a Wire? The goal of attempting to create a Wire was to make it easier to use the other "line like" methods but this example shows the danger in that.