doctest failure in functions/special.py #11
Comments
|
comment:1
Fixed: The main trick was to use float conversions instead of RR before |
vbraun
pushed a commit
that referenced
this issue
Mar 26, 2023
### 📚 Description Trac branch `u/gh-collares/gap-gc` from #34701, now migrated to GitHub. Currently based atop #35093; will rebase once that is merged. The rest of the description below is copied from #34701: A refactor in #27946 introduced "unprotected" (not surrounded by `GAP_Enter`/`GAP_Leave`) `GAP_ValueGlobalVariable` calls. I believe this might be a GC hazard, because after updating to GAP 4.12.1 I started seeing aarch64 crashes on NixOS infrastructure such as: ``` #0 0x0000fffff79740e8 in wait4 () #1 0x0000fffff5dc6b78 in print_enhanced_backtrace () #2 0x0000fffff5dc8190 in sigdie () #3 0x0000fffff5dcb1c0 in cysigs_signal_handler () #4 0x0000fffff7ffb7cc in __kernel_rt_sigreturn () #5 0x0000ffff99a0bf28 in ConvString () #6 0x0000000000000000 in ?? () #7 0x0000000000000000 in ?? () #8 0x0000000000000000 in ?? () #9 0x0000ffff99989930 in Pr () #10 0x0000ffff9998aa18 in CloseOutput () #11 0x0000ffff99884828 in capture_stdout () at /build/sage- src-9.7/pkgs/sagemath-standard/sage/libs/gap/element.pyx:154 ... ``` I also see cases where `capture_stdout` throws errors such as `sage.libs.gap.util.GAPError: Error, Length: <list> must be a list (not the integer 255)` and then crashes. Both types of errors are fixed by this ticket. Note that I am nesting `GAP_Enter`/`GAP_Leave` calls because I didn't remove the preexisting calls inside `capture_stdout`. That's because I feared removing the innermost calls might create a new footgun (and I believe nested `GAP_Enter`/`GAP_Leave` calls are explicitly supported), but removing them should cause no problem. Removing them might even be preferable for performance reasons, I don't know. Fixes #34701 ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> <!-- If your change requires a documentation PR, please link it appropriately --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> - [x] I have made sure that the title is self-explanatory and the description concisely explains the PR. - [x] I have linked an issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open pull requests that this PR logically depends on --> <!-- - #xyz: short description why this is a dependency - #abc: ... --> - #35093: GAP 4.12.2 upgrade, which touches the same function and should land first. URL: #35114 Reported by: Mauricio Collares Reviewer(s): Dima Pasechnik
mkoeppe
pushed a commit
to jsantillan3/sage
that referenced
this issue
Aug 30, 2023
vbraun
pushed a commit
to vbraun/sage
that referenced
this issue
Oct 28, 2023
* approve_without_body initial * add missing positional arg * nochmal * issue 2 * Introduce SYNC_LABELS_BOT_TOKEN and fix issue 1 * Fix in gh_cmd because of mark_as_ready * Ignore trigger of bot * Ignore actor's reviews on needs review * dismiss_bot_reviews and refrain from removing labels * fix broken call of review_by_actor * add get_review_requests * consistent use of state versus status * add full stops in comments
xuluze
pushed a commit
to xuluze/sage
that referenced
this issue
Dec 12, 2023
tobiasdiez
pushed a commit
to tobiasdiez/sage
that referenced
this issue
Feb 22, 2024
* Add azure badget * Fix azure pipeline for windows
xuluze
pushed a commit
to xuluze/sage
that referenced
this issue
Feb 23, 2024
xuluze
pushed a commit
to xuluze/sage
that referenced
this issue
Jun 10, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
sage -t -optional devel/sage-darcs/sage/functions/special.py*******************************************************\
File "special.py", line 756:
sage: inverse_jacobi("sn",0.47,1/2)
Exception raised:
Traceback (most recent call last):
File "/home/was/sage/local/lib/python2.4/doctest.py", line 1243, in __run
compileflags, 1) in test.globs
File "<doctest main.example13[1]>", line 1, in ?
inverse_jacobi("sn",RealNumber('0.47'),Integer(1)/Integer(2))###line 756:
sage: inverse_jacobi("sn",0.47,1/2)
File "/home/was/sage/local/lib/python2.4/site-packages/sage/functions/special.py", line 785, in inverse_jacob
i
return eval(maxima.eval("inverse_jacobi_sn(%s,%s)"%(RR(x),RR(m))))
File "", line 1
^[[
^
SyntaxError: invalid syntax
File "special.py", line 758:
sage: inverse_jacobi("sn",0.4707504,1/2)
Exception raised:
Component: basic arithmetic
Issue created by migration from https://trac.sagemath.org/ticket/11
The text was updated successfully, but these errors were encountered: