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

Transpose text databases of reflexive polytopes for PALP #22400

Closed
novoselt opened this issue Feb 21, 2017 · 6 comments
Closed

Transpose text databases of reflexive polytopes for PALP #22400

novoselt opened this issue Feb 21, 2017 · 6 comments

Comments

@novoselt
Copy link
Member

We store reflexive polytopes in 2 and 3 dimensions as text files. The current version lists points as columns of matrices, but the default output for PALP is reversed and it gives cleaner code not handle two cases. The main point, however, is that it would be faster to construct reflexive polytopes lists in memory if files were written point by point.

The current code in Sage will accept both old and new versions of the database, while the new one I am (re)writing will work only with the new one.

There is also a comment in SPKG.txt (exactly five years old) that text files will be removed in favour of the also included binary ones. However this didn't happen yet and I think it makes sense to keep small lists in plain form which is so easy to parse directly in Python without relying on any database modules or PALP itself for that matter.

CC: @vbraun

Component: packages: standard

Author: Andrey Novoseltsev

Branch/Commit: f42ea43

Reviewer: Volker Braun

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

@novoselt
Copy link
Member Author

comment:1

Attachment: polytopes_db-20170220.tar.bz2.gz

@novoselt
Copy link
Member Author

@novoselt
Copy link
Member Author

Commit: f42ea43

@novoselt
Copy link
Member Author

New commits:

f42ea43Transpose databases of small reflexive polytopes

@vbraun
Copy link
Member

vbraun commented Aug 11, 2017

Reviewer: Volker Braun

@vbraun
Copy link
Member

vbraun commented Aug 13, 2017

Changed branch from u/novoselt/transpose_lattice_polytopes to f42ea43

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

2 participants