Fix remaining instances of ArithmeticError: 0^0 is undefined

[orlitzky]

Some of these were fixed in #10772, but a few remain. Find/grep show,

• ./rings/padics/padic_capped_relative_element.pyx
• ./rings/padics/padic_fixed_mod_element.pyx
• ./rings/padics/padic_ZZ_pX_CR_element.pyx
• ./rings/padics/padic_ZZ_pX_FM_element.pyx

Unless any of these are special (for whatever reason), we should return 1 instead from the appropriate ring.

See also: #13894, #13895, #13897, #13941.

CC: @kcrisman

Component: algebra

Author: Michael Orlitzky

Reviewer: Travis Scrimshaw

Merged: sage-5.8.beta2

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

[orlitzky]

Dependencies: #13940

[orlitzky]

Changed dependencies from #13940 to #13894, #13895, #13897, #13940, #13941

[tscrim]

comment:8

Just a general statement on the documentation in these patches. I'd prefer to see the 0^0 formatted either in latex `0^0` (which would display as 0⁰) or as a code literal ``0^0``.

Thanks,

Travis

[orlitzky]

comment:9

I finally added double backticks to the other patches. While those functions aren't in the reference manual, they should and hopefully someday will be.

[tscrim]

comment:10

Thank you. It suppose to make the docreader (which may only really be me) understand that it is suppose to be code/input.

Also, for the remaining ones in the padics, could you just put them all in one patch (possibly on this ticket)?

Thanks,

Travis

[orlitzky]

comment:11

Attachment: sage-trac_13786.patch.gz

Patch is up. I had split them initially to make it easier for (potentially) multiple reviewers. Thanks for taking a look at all of them.

I've removed ntl_lzz_p.pyx from the list per #13940. This patch should take care of the remaining four.

[orlitzky]

Author: Michael Orlitzky

[tscrim]

comment:13

Looks good to me. Thank you.

Travis

[tscrim]

Reviewer: Travis Scrimshaw

[tscrim]

comment:15

Replying to @orlitzky:

I've removed ntl_lzz_p.pyx from the list per #13940. This patch should take care of the remaining four.

Does this depend on #13940 anymore?

[orlitzky]

Changed dependencies from #13894, #13895, #13897, #13940, #13941 to none

[orlitzky]

comment:16

No, good catch. I've removed all of the dependencies; no need to confuse things. This patch can be merged without any of those. If an archaeologist is curious, he can check the description diff.

[jdemeyer]

Merged: sage-5.8.beta2