-
Notifications
You must be signed in to change notification settings - Fork 14
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
Saf zero subspaces iterator #37
base: master
Are you sure you want to change the base?
Conversation
videlec
commented
Nov 2, 2021
•
edited
Loading
edited
- implement a more generic iterator over isotropic subspaces (with a filter option for positive vector containmnent)
- make the iterator smarter by eliminating early candidates that do not contain positive vectors. For example, the entries just after a pivot might be forced to be positive (if the entries above are <= 0)
- presence of positive vectors can be tested via linear optimization
@saraedum Strange warning on my machine (conda is up to date)
|
EDIT: this error is normal, there is a zero length. Even worse
From the debugger I don't see anything wrong with the input
|
The warning you mention in #37 (comment) looks like a problem in cppyy to me @wlav. The problematic snippet is nothing we wrote it seems so I guess it's cppyy's way to copy a struct around.
Could you verify that your version of cppyy is current, i.e., 2.1.0? If so, we should probably try to create a smaller reproducer and report this upstream. |
Apparently not up to date
|
The difference between 1.x and 2.x are rather big: different version of Clang. Let me know if the error persists with the newer version and I'll take a look. As-is, based on my limited reading/understanding, my first guess would have been this: as it has the same error message. However, that report says that that was precisely fixed with 1.9.3 ... |
@saraedum I am a bit unhappy with
|
@saraedum I am a bit puzzled by the doctest error on sage 9.5
|
This will be fixed by conda-forge/conda-forge-repodata-patches-feedstock#414 |
Is there something to do against the failing doctest in
|
This needs cppyy=3 to make the warning go away. Anyway, I'd just set the doctest to |