Bug#996336: pygalmesh: autopkgtest regression on i386

[Nico Schlömer]

Yeah, that's a little weird. And it only occurs on i386? Sounds like a
CGAL bug then.

On Wed, Oct 13, 2021 at 3:24 PM Drew Parsons  wrote:
>
> On 2021-10-13 13:42, Nico Schlömer wrote:
> > Or PR upstream. I can merge and release any time.
>
> Thanks Nico. I'll check which tolerances it needs and file a PR.
>
> The error in test_rectangle might need your attention,
> "At index 0 diff: 279 != 276". I guess it's not just tolerances.
>
> Drew

Bug#996336: pygalmesh: autopkgtest regression on i386

[Drew Parsons]

On 2021-10-13 13:42, Nico Schlömer wrote:

Or PR upstream. I can merge and release any time.


Thanks Nico. I'll check which tolerances it needs and file a PR.

The error in test_rectangle might need your attention,
"At index 0 diff: 279 != 276". I guess it's not just tolerances.

Drew

Bug#996336: pygalmesh: autopkgtest regression on i386

[Nico Schlömer]

Or PR upstream. I can merge and release any time.

Cheers,
Nico

On Wed, Oct 13, 2021 at 1:39 PM Drew Parsons  wrote:
>
> Not entirely unexpected.  Upstream reorganized tests and tolerances, so
> I wanted to check if we had a full pass or not.
> Looks like we'll have to reinstate test_relax_tolerance.patch
>
>
> On 2021-10-13 12:03, Adrian Bunk wrote:
> > Source: pygalmesh
> > Version: 0.10.5-1
> > Severity: serious
> >
> > https://ci.debian.net/data/autopkgtest/testing/i386/p/pygalmesh/15928897/log.gz
> >
> > ...
> > 
> > assert abs(max(mesh.points[:, 2]) - 1.0) < 1.1e-3
> >>   assert abs(min(mesh.points[:, 2]) + 0.0) < tol
> > E   assert 0.0010112326126545668 < 0.001
> > E+  where 0.0010112326126545668 = abs((-0.0010112326 + 0.0))
> > E+where -0.0010112326 = min(array([ 0.e+00,
> > 1.e+00,  0.e+00,  1.e+00,\n
> > 0.e+00,  1.e+00,  0...-01,\n4.62765425e-01,
> > 8.98388922e-02,  3.73478711e-01,  4.52255577e-01,\n
> > 9.99695361e-01], dtype=float32))
> >
>

Bug#996336: pygalmesh: autopkgtest regression on i386

[Drew Parsons]

Not entirely unexpected.  Upstream reorganized tests and tolerances, so 
I wanted to check if we had a full pass or not.

Looks like we'll have to reinstate test_relax_tolerance.patch


On 2021-10-13 12:03, Adrian Bunk wrote:

Source: pygalmesh
Version: 0.10.5-1
Severity: serious

https://ci.debian.net/data/autopkgtest/testing/i386/p/pygalmesh/15928897/log.gz

...

assert abs(max(mesh.points[:, 2]) - 1.0) < 1.1e-3

  assert abs(min(mesh.points[:, 2]) + 0.0) < tol

E   assert 0.0010112326126545668 < 0.001
E+  where 0.0010112326126545668 = abs((-0.0010112326 + 0.0))
E+where -0.0010112326 = min(array([ 0.e+00,
1.e+00,  0.e+00,  1.e+00,\n
0.e+00,  1.e+00,  0...-01,\n4.62765425e-01,
8.98388922e-02,  3.73478711e-01,  4.52255577e-01,\n
9.99695361e-01], dtype=float32))

Bug#996336: pygalmesh: autopkgtest regression on i386

[Adrian Bunk]

Source: pygalmesh
Version: 0.10.5-1
Severity: serious

https://ci.debian.net/data/autopkgtest/testing/i386/p/pygalmesh/15928897/log.gz

...
=== FAILURES ===
 test_rectangle 

def test_rectangle():
points = np.array([[0.0, 0.0], [1.0, 0.0], [1.0, 1.0], [0.0, 1.0]])
constraints = [[0, 1], [1, 2], [2, 3], [3, 0]]

mesh = pygalmesh.generate_2d(
points, constraints, max_edge_size=1.0e-1, num_lloyd_steps=10
)

>   assert mesh.points.shape == (276, 2)
E   assert (279, 2) == (276, 2)
E At index 0 diff: 279 != 276
E Use -v to get the full diff

tests/test_2d.py:15: AssertionError
_ test_extrude _

def test_extrude():
p = pygalmesh.Polygon2D([[-0.5, -0.3], [0.5, -0.3], [0.0, 0.5]])
domain = pygalmesh.Extrude(p, [0.0, 0.3, 1.0])
mesh = pygalmesh.generate_mesh(
domain,
max_cell_circumradius=0.1,
max_edge_size_at_feature_edges=0.1,
verbose=False,
)

tol = 1.0e-3
assert abs(max(mesh.points[:, 0]) - 0.5) < tol
assert abs(min(mesh.points[:, 0]) + 0.5) < tol
assert abs(max(mesh.points[:, 1]) - 0.8) < tol
assert abs(min(mesh.points[:, 1]) + 0.3) < tol
# Relax tolerance for debian, see 

assert abs(max(mesh.points[:, 2]) - 1.0) < 1.1e-3
>   assert abs(min(mesh.points[:, 2]) + 0.0) < tol
E   assert 0.0010112326126545668 < 0.001
E+  where 0.0010112326126545668 = abs((-0.0010112326 + 0.0))
E+where -0.0010112326 = min(array([ 0.e+00,  
1.e+00,  0.e+00,  1.e+00,\n0.e+00,  
1.e+00,  0...-01,\n4.62765425e-01,  8.98388922e-02,  
3.73478711e-01,  4.52255577e-01,\n9.99695361e-01], dtype=float32))

tests/test_volume_mesh.py:486: AssertionError
=== short test summary info 
FAILED tests/test_2d.py::test_rectangle - assert (279, 2) == (276, 2)
FAILED tests/test_volume_mesh.py::test_extrude - assert 0.0010112326126545668...
 2 failed, 29 passed in 41.69s =
autopkgtest [21:44:46]: test command1: ---]
autopkgtest [21:44:46]: test command1:  - - - - - - - - - - results - - - - - - 
- - - -
command1 FAIL non-zero exit status 1