-
-
Notifications
You must be signed in to change notification settings - Fork 481
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
polytopes_db_4d converted to new style package #26029
Comments
comment:1
Related discussions:
|
comment:2
The package is listed at: |
comment:3
And it's available from each of the SageMath download mirrors. E.g.
etc. The problems encountered with this huge package might instead have to do with |
comment:4
As I wrote on sage-devel, we could keep supporting this package if it were repackaged as a |
comment:6
The following shows sage can find packages in "huge":
The package cannot be installed with
but
Once this is done and after restarting Sage, one can follow the
|
comment:7
Retargeting some of my tickets (somewhat optimistically for now). |
comment:8
Removing most of the rest of my open tickets out of the 8.7 milestone, which should be closed. |
comment:9
According to Andrey Novoseltev:
As I wrote on sage-devel, perhaps it would make sense to host this data (as well as the 3D polytopes; why not?) on an online database somewhere that can be easily queried (including by the code in Sage which uses this data)? This way it is possible to use as-needed without downloading the entire, very large database (though perhaps with the possibility of pre-downloading/caching all or parts of it as needed). It would be very easy to write a simple web service, and perhaps the data and service could be hosted on sagemath.org. This might also be a good opportunity for some MitM synergy. There is a workshop later this year on mathematical data, and frameworks and services for accessing them, so that might be a good opportunity to work on such a project (I would be happy to help, especially if someone can help me with the math end of things as I have no idea what a reflexive Hodge polytope is much less how it would be represented in a MitM framework): https://opendreamkit.org/2019/08/17/WorkshopOnDataInMathematics/ |
comment:10
A reflexive polytope is a full-dimensional lattice polytope (i.e. the vertices have integer coordinates), containing the origin in the interior, so that its dual |
comment:11
3D polytopes take so little space, that there is no sense at all removing them from Sage in their current form. As for 4D, there are probably very few people who are using them and so there is a lack of effort to do something about them. On the other hand, since there are papers referring to the use of this package in Sage, it would be nice to support repeating those computations in some way, even if installation is "non-standard". |
Author: Dima Pasechnik |
Commit: |
New commits:
|
This comment has been minimized.
This comment has been minimized.
Branch: u/dimpase/packages/refpoly4d |
comment:13
Do we want this to be experimental or optional? I feel like optional would be the better option for this. |
comment:14
I also think it should be "optional" |
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:16
OK, done |
comment:17
Thanks. I am ready to set a positive review. Any objections? |
Reviewer: Travis Scrimshaw |
comment:18
No objection; I haven't tried it though because downloading was taking too long. |
comment:19
It built for me without any problems, so I am setting it to positive. |
Changed branch from u/dimpase/packages/refpoly4d to |
This spkg is mentioned in #14467 and was apparently "huge" (and hence optional). But it is no longer listed under
build/pkgs
nor does it appear to be available on the download mirrors.Here is a branch to convert it to a new style experimental (but, maybe optional?) package.
the tarball is the old *.spkg file, but as it's a tar file, it works
tarball url (see in checksum.ini), but it's the same as one gets
via
./sage -p polytopes_db_4d
(which still works with old style pkfs)A few options:
Done.
CC: @mkoeppe @vbraun
Component: packages: huge
Author: Dima Pasechnik
Branch/Commit:
b49f89a
Reviewer: Travis Scrimshaw
Issue created by migration from https://trac.sagemath.org/ticket/26029
The text was updated successfully, but these errors were encountered: