On Sunday, March 24, 2024 at 4:02:25 PM UTC Nils Bruin wrote:
On Sunday 24 March 2024 at 04:41:25 UTC-7 Przemysław Koprowski wrote: Let me just comment on your words "searching the source, this routine isn't actually used elsewhere in sage" (here: https://github.com/sagemath/sage/pull/37657). It is not entirely true, because, as far as I know, the method is_padic_square (in the class of a number field element) internally relies on the quadratic_defect. Thanks! I based my assessment on the search result that github returned for `quadratic_defect`. It only returned two hits for me. Lesson: Don't rely on github search. It's far from exhaustive. indeed, entries in src/sage/rings/number_field/number_field.py are just missing in GitHub search. I filed a "feedback". -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/sage-devel/1c8cefe6-0a14-4a93-91fb-da1c09ab5984n%40googlegroups.com.