Potential bug in recursive holes in polytrees

[nsluhrs]

Hello I should preface this with the fact that I am using https://github.com/drlukeparry/pyclipr, rather than calling this directly However since it is fairly up to date and mostly seems to work I suspect this is likely a problem with clipper2 itself. Additionally this problem does not occur with clipper 1. I am using gdstk (which uses clipper1 under the hood) to generate visuals
This seems similar to #576 but I think my test data may be easier to work with.
my test data is as follows
{8,8, -8,8, -8,0, -6,0, -6,6, 6,6, 6,0, 4,0, 4,4, -4,4, -4,0, -2,0, -2,2, 2,2, 2,-2, -2,-2, -2,0, -4,0, -4,-4, 4,-4, 4,0, 6,0, 6,-6, -6,-6, -6,0, -8,0, -8,-8, 8,-8}
Since my code is in python I will write pseudo code

clpr.addPath(test_path, Subject, True) # closed path with subject  type
PolyTree solution;
clpr.Exectue(Union, solution, EvenOdd);

for reference the actual core python isn't much longer but the other functions I wrote to track this down are a bit longer

pc = pyclipr.Clipper()
pc.addPath(test_shape, pyclipr.Subject)
out = pc.execute2(pyclipr.Union, pyclipr.EvenOdd)
pc.cleanUp()

I found both cases where the 'seams' in the shape were horizontal to be broken to some degree.
All cases I have tested are 90 degree rotations of the base test data provided above.
the first broken case :

This one is closer to being correct but still doesn't get the tree structure I would expect. note that the junction lines in both cases are horizontal.

If the junctions are vertical it works fine.

The labels in the upper left corner of each shape are their index in the tree where each number in the list is the index in the Children array from the polytree. The holes:# is just a mapping of the is_hole property

[AngusJohnson]

Using the latest Repository version ...

	Paths64 subject, solution;
	PolyTree64 tree;
	subject.push_back(MakePath({ 8,8, -8,8, -8,0, -6,0, -6,6, 6,6, 6,0, 4,0, 4,4, -4,4, -4,0, -2,0, 
		-2,2, 2,2, 2,-2, -2,-2, -2,0, -4,0, -4,-4, 4,-4, 4,0, 6,0, 6,-6, -6,-6, -6,0, -8,0, -8,-8, 8,-8 }));
	Clipper64 c;
	c.AddSubject(subject);
	c.Execute(ClipType::Union, FillRule::EvenOdd, tree);
	std::cout << tree << std::endl;
	solution = PolyTreeToPaths64(tree);

	SvgWriter svg;
	std::string filename = "test.svg";
	SvgAddSubject(svg, subject, FillRule::EvenOdd);  // pale blue lines
	SvgAddSolution(svg, solution, FillRule::EvenOdd, false); // black lines and light green fill
	SvgSaveToFile(svg, filename, 400, 400);
	System(filename);

Polytree with 1 polygon.
  +- Polygon (0) contains 1 hole.
    +- Hole (0) contains 1 nested polygon.
      +- Polygon (0) contains 1 hole.