numer() and denom() are poor, almost unused, have trivial more handy equivalences. They shall be removed
@sgougeon)
Reported by Samuel GOUGEON (denom_numer_removal_list.zip (6.11 KB, application/octet-stream)
denom() and numer() occurrences lists in GIT 6.0.1 2018-01-04
BUG DESCRIPTION:
----------------
denom():
-------
It is used only
- twice in the Scilab code (once in kpure.sci, %r_simp.sci and %_filter.sci)
- once in a single demo (poly.intro)
- 5 times in all help examples
Other usages are in tests. The full list of occurrences is attached, to drive the removal.
Since the recursive extraction is available in Scilab 6, the trivial equivalence .den
can even be used without intermediate variable for expressions like
--> ([%s %s^2]/(1-%s) - 1/%s).den
ans =
2 2
s -s s -s
numer():
-------
It is used only
- 7 times in the Scilab code (kpure (1), %r_simp (1), %_filter (1), nyquistfrequencybounds (4))
- none in any demo
- 9 times in all help examples
Other usages are in tests. The full list of occurrences is attached as well.
In Scilab 6, the trivial equivalence .num works even with the recursive extraction,
without intermediate variable, for expressions like
--> ([%s %s^2]/(1-%s) - 1/%s).num
ans =
2 3
-1 +s +s -1 +s +s
Both functions look even more useless in Scilab 6 than before.
We propose to remove them from Scilab 6.
If some great ATOMS modules need some update after these removals, please notice me which ones.
I can contribute to maintain them against this evolution.