-
Notifications
You must be signed in to change notification settings - Fork 62
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
lc for multivariates but lead for univariates #508
Comments
On Wed, Oct 16, 2019 at 12:24:04PM -0700, wbhart wrote:
We should unify these names, as it is not helpful for users.
See e.g. issue Nemocas/Nemo.jl#662
Definitely - and please not lc. lc is one of the precious 2-letter
combinations that I would rather reserve for users and not the system.
leading_coefficient would be my favourite, any user is welcome to say
lc = leading_coefficient
but lead_coeff or similar would be fine as well.
…
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
#508
|
You'll have to argue the toss with the Singular people. It's called lc because that's what they wanted. There's a whole series of these, lc, lm, lt, etc. |
On Thu, Oct 17, 2019 at 01:36:22AM -0700, wbhart wrote:
You'll have to argue the toss with the Singular people. It's called lc because that's what they wanted. There's a whole series of these, lc, lm, lt, etc.
I know - and I'll talk to them. IN particular lt "less than" is not a
good abbreviation if you want to be more general
…
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
#508 (comment)
|
Not that I believe that my opinion matters, but I find that names like |
Ok, I agree with the changes here, but what was the response from the Singular people? Do we have permission to make this change? It removes what was essentially a feature request from Singular.jl (they explicitly wanted these names because they are used so often). We need a final decision on this and I'll handle the PR's. |
Note that in Nemo we currently have lead and trail:
so the changes will be more invasive if we go for lead_coeff and trail_coeff. |
By the way, when there is an iterator, one can always do first(coeffs(f)) instead of lead_coeffs(f). But note that univariates have terms in the reverse order to multivariates in a single variable (for lex ordering). |
The Singular people (ie, THE BOSS) decided that leading_coefficient is fine and lc can become history |
We should unify these names, as it is not helpful for users.
See e.g. issue Nemocas/Nemo.jl#662
The text was updated successfully, but these errors were encountered: