Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1 pynormaliz doctest failing in src/sage/geometry/polyhedron/backend_normaliz.py #27782

Closed
seblabbe opened this issue May 6, 2019 · 10 comments
Closed

Comments

@seblabbe
Copy link
Contributor

seblabbe commented May 6, 2019

With SageMath version 8.8.beta4, Release Date: 2019-05-04,

sage -t --long src/sage/geometry/polyhedron/backend_normaliz.py

gives

Using --optional=4ti2,bliss,build,cbc,ccache,cmake,cryptominisat,dochtml,dot2tex,e_antic,glucose,latte_int,lidia,lrslib,memlimit,mpir,ninja_build,normaliz,notedown,pandoc_attributes,pycosat,pynormaliz,python2,qhull,rst2ipynb,sage,topcom
Doctesting 1 file.
sage -t --long src/sage/geometry/polyhedron/backend_normaliz.py
**********************************************************************
File "src/sage/geometry/polyhedron/backend_normaliz.py", line 186, in sage.geometry.polyhedron.backend_normaliz.Polyhedron_normaliz._nmz_result
Failed example:
    p._nmz_result(p._normaliz_cone, 'EquivariantXyzzyModuleSeries')  # optional - pynormaliz
Expected:
    Traceback (most recent call last):
    ...
    error: Some error in the normaliz input data detected: Unknown ConeProperty...
Got:
    <BLANKLINE>
    Traceback (most recent call last):
      File "/home/slabbe/GitBox/sage/local/lib/python2.7/site-packages/sage/doctest/forker.py", line 671, in _run
        self.compile_and_execute(example, compiler, test.globs)
      File "/home/slabbe/GitBox/sage/local/lib/python2.7/site-packages/sage/doctest/forker.py", line 1095, in compile_and_execute
        exec(compiled, globs)
      File "<doctest sage.geometry.polyhedron.backend_normaliz.Polyhedron_normaliz._nmz_result[1]>", line 1, in <module>
        p._nmz_result(p._normaliz_cone, 'EquivariantXyzzyModuleSeries')  # optional - pynormaliz
      File "/home/slabbe/GitBox/sage/local/lib/python2.7/site-packages/sage/geometry/polyhedron/backend_normaliz.py", line 192, in _nmz_result
        return PyNormaliz.NmzResult(normaliz_cone, property)
    NormalizError: Some error in the normaliz input data detected: Unknown ConeProperty string "EquivariantXyzzyModuleSeries"
**********************************************************************
1 item had failures:
   1 of   3 in sage.geometry.polyhedron.backend_normaliz.Polyhedron_normaliz._nmz_result
    [136 tests, 1 failure, 0.52 s]
----------------------------------------------------------------------
sage -t --long src/sage/geometry/polyhedron/backend_normaliz.py  # 1 doctest failed
----------------------------------------------------------------------

CC: @jplab

Component: geometry

Author: Matthias Koeppe

Branch/Commit: 9b3a5ba

Reviewer: Vincent Klein

Issue created by migration from https://trac.sagemath.org/ticket/27782

@seblabbe seblabbe added this to the sage-8.8 milestone May 6, 2019
@seblabbe
Copy link
Contributor Author

seblabbe commented May 6, 2019

Author: jipilab

@mkoeppe
Copy link
Contributor

mkoeppe commented May 7, 2019

Changed author from jipilab to none

@mkoeppe
Copy link
Contributor

mkoeppe commented May 7, 2019

@mkoeppe
Copy link
Contributor

mkoeppe commented May 7, 2019

New commits:

9b3a5baFix error message in doctest for new pynormaliz version

@mkoeppe
Copy link
Contributor

mkoeppe commented May 7, 2019

Author: Matthias Koeppe

@mkoeppe
Copy link
Contributor

mkoeppe commented May 7, 2019

Commit: 9b3a5ba

@vinklein
Copy link
Mannequin

vinklein mannequin commented May 7, 2019

Reviewer: Vincent Klein

@vinklein
Copy link
Mannequin

vinklein mannequin commented May 7, 2019

comment:5

Looks good to me.

@mkoeppe
Copy link
Contributor

mkoeppe commented May 8, 2019

comment:6

Thank you!

@vbraun
Copy link
Member

vbraun commented May 12, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants