sage-devel
https://groups.google.com/d/forum/sage-devel
This email list is for discussion of Sage (<a href="http://www.sagemath.org">www.sagemath.org</a>) development issues. This list usually has heavy traffic.enRe: [sage-devel] Re: Unify keywords certificate and certify
https://groups.google.com/d/msg/sage-devel/b__PEZIjKmg/51jF2hrNBQAJ
Can we keep certify() for functions that return True/False, or possibly return None if ok and raise an error otherwise (as in an assert statement ); and only use certificate() for functions that return some sort of certificate? There are primality certificates, for example. On 30 Jul 2016https://groups.google.com/d/topic/sage-devel/b__PEZIjKmg
John CremonaSat, 30 Jul 2016 09:48:29 UTCRe: [sage-devel] Re: Unhandled case in EllipticCurve_from_cubic
https://groups.google.com/d/msg/sage-devel/ywH31HkRo7g/e6JThdjMBQAJ
I think that an error should be raised if the projective point constructor is called with (0,0,0). On 29 Jul 2016 23:54, "Robin Houston" <robin....@gmail.com> wrote: I think there’s another case that isn’t handled correctly, exemplified by: R.<x,y,z> = QQ[] T = EllipticCurve_from_cubic(y^2*z -https://groups.google.com/d/topic/sage-devel/ywH31HkRo7g
John CremonaSat, 30 Jul 2016 09:43:44 UTCRe: [sagemath-admins] Re: (CRITICAL) sage-trac google group concern
https://groups.google.com/d/msg/sage-devel/mlMdVUXzRIE/GILeTofIBQAJ
sage-trac was also cc'd to a bunch of seemingly random users, as well as a web mirror... Anyhow, now it's OK to change trac passwords, even though the apparent primary problem, that all the emails trac sends get cc'd to the googlegroup, is still not helped with. Dimahttps://groups.google.com/d/topic/sage-devel/mlMdVUXzRIE
Dima PasechnikSat, 30 Jul 2016 08:24:37 UTCRe: [sage-devel] Re: Unify keywords certificate and certify
https://groups.google.com/d/msg/sage-devel/b__PEZIjKmg/uYC93FzBBQAJ
On Fri, 29 Jul 2016, Dima Pasechnik wrote: > IIRC, there is at least one place where such a parameter is named "cert" > :-) Didn't found any. > And one can have names like "proof", "validate", etc :-) True, but "proof" is used for another meaning. -- Jori Mäntysalohttps://groups.google.com/d/topic/sage-devel/b__PEZIjKmg
Jori MäntysaloSat, 30 Jul 2016 06:13:18 UTCRe: [sage-devel] Re: Unify keywords certificate and certify
https://groups.google.com/d/msg/sage-devel/b__PEZIjKmg/Iiqa0nvABQAJ
On Fri, 29 Jul 2016, Volker Braun wrote: > Usually its better to have separate methods, e.g. n.factor() instead of > n.is_prime(certificate=True) Techically 100=25*4 is one possible certificate to show that 100 is not a prime, whereas 100=2*2*5*5 is the factorization. As a better example thinkhttps://groups.google.com/d/topic/sage-devel/b__PEZIjKmg
Jori MäntysaloSat, 30 Jul 2016 05:57:12 UTCRe: [sagemath-admins] Re: (CRITICAL) sage-trac google group concern
https://groups.google.com/d/msg/sage-devel/mlMdVUXzRIE/8lGVBf67BQAJ
On Friday, July 29, 2016, Karl-Dieter Crisman <kcri...@gmail.com> wrote: > > > > This is indeed very bad (the relevant message is [1]). And not the only >> one [2]. Please do not change any password on trac since this is fixed!! >> >> > Yikes! > On the plus side, it's not universal since > "Youhttps://groups.google.com/d/topic/sage-devel/mlMdVUXzRIE
WilliamSat, 30 Jul 2016 04:34:53 UTCRe: [sage-trac-account] Re: (CRITICAL) sage-trac google group concern
https://groups.google.com/d/msg/sage-devel/mlMdVUXzRIE/5aIjPSi6BQAJ
Yikes! On the plus side, it's not universal since "You must be a member of this group to view and participate in it." though this is a trivial thing to avoid, I guess.https://groups.google.com/d/topic/sage-devel/mlMdVUXzRIE
kcrismanSat, 30 Jul 2016 04:01:16 UTCRe: Unhandled case in EllipticCurve_from_cubic
https://groups.google.com/d/msg/sage-devel/ywH31HkRo7g/SsHL5mmpBQAJ
I think there’s another case that isn’t handled correctly, exemplified by: R.<x,y,z> = QQ[] T = EllipticCurve_from_cubic(y^2*z - x^3 - z^3, (0,1,0)) A simple change that fixes this problem is to make the projective_point() method not blow up when given (0,0,0), e.g.: --- a/src/sage/schemes/ellhttps://groups.google.com/d/topic/sage-devel/ywH31HkRo7g
Robin HoustonFri, 29 Jul 2016 22:54:26 UTCRe: Unify keywords certificate and certify
https://groups.google.com/d/msg/sage-devel/b__PEZIjKmg/eJ6N0aWfBQAJ
Really its a variable (keyword argument), so verb/noun naming should match whether it hold a function or an object. And we certainly don't pass something that is callable. Really both verb and noun are a bit awkward, its a dummy flag to have polymorphic return values. Which isn't a trulyhttps://groups.google.com/d/topic/sage-devel/b__PEZIjKmg
Volker BraunFri, 29 Jul 2016 19:55:28 UTCRe: Unify keywords certificate and certify
https://groups.google.com/d/msg/sage-devel/b__PEZIjKmg/EHO7EXadBQAJ
IIRC, there is at least one place where such a parameter is named "cert" :-) And one can have names like "proof", "validate", etc :-) > > -leif > > >https://groups.google.com/d/topic/sage-devel/b__PEZIjKmg
Dima PasechnikFri, 29 Jul 2016 19:15:24 UTCRe: Unify keywords certificate and certify
https://groups.google.com/d/msg/sage-devel/b__PEZIjKmg/xRiQXsKaBQAJ
Even worse then, since it is meant to be the (imperative form of the) /verb/. (Out of context, I'd assume the function takes an object of some Certificate class, not a Boolean.) > Also I think it is the imperative form, though truth be told I've never > heard anybody use "certificate" as ahttps://groups.google.com/d/topic/sage-devel/b__PEZIjKmg
leifFri, 29 Jul 2016 18:25:53 UTCRe: (CRITICAL) sage-trac google group concern
https://groups.google.com/d/msg/sage-devel/mlMdVUXzRIE/SKGR_qiZBQAJ
This is indeed very bad (the relevant message is [1]). And not the only one [2]. Please do not change any password on trac since this is fixed!! [1] https://groups.google.com/forum/#!topic/sage-trac/xT0JWnw0LkU [2] https://groups.google.com/forum/#!searchin/sage-trac/password|sort:datehttps://groups.google.com/d/topic/sage-devel/mlMdVUXzRIE
vdelecroixFri, 29 Jul 2016 18:05:45 UTCsage-trac google group concern
https://groups.google.com/d/msg/sage-devel/mlMdVUXzRIE/UFv2ZcuYBQAJ
I just saw a password-reset message posted to sage-trac, and the message included the new password. This can't be a good idea, can it? -- Johnhttps://groups.google.com/d/topic/sage-devel/mlMdVUXzRIE
John H PalmieriFri, 29 Jul 2016 17:49:53 UTCRe: Unify keywords certificate and certify
https://groups.google.com/d/msg/sage-devel/b__PEZIjKmg/B6kY2NuWBQAJ
While technically true, "certificate" is not commonly used as a verb. I don't think anybody would misguess noun vs verb here. Also I think it is the imperative form, though truth be told I've never heard anybody use "certificate" as a verb..https://groups.google.com/d/topic/sage-devel/b__PEZIjKmg
Volker BraunFri, 29 Jul 2016 17:14:25 UTCRe: Unify keywords certificate and certify
https://groups.google.com/d/msg/sage-devel/b__PEZIjKmg/o-pWY_uVBQAJ
Well, while "certificate" is ambiguous (as it can be a verb as well as a noun), "certify" is not, and for boolean parameters/keywords, we usually take the imperative from. So I'd prefer the latter. (There are of course dozens of instances where one could misguess the expected type because ofhttps://groups.google.com/d/topic/sage-devel/b__PEZIjKmg
leifFri, 29 Jul 2016 16:58:20 UTC