mtlb_sparse() no longer exists

Reported by Samuel GOUGEON (@sgougeon)

BUG DESCRIPTION:
----------------
From the example given in help mtlb_sparse:

--> X=sparse(rand(2,2)); Y=mtlb_sparse(X);

Undefined variable: mtlb_sparse


ERROR LOG:
----------
Undefined variable: mtlb_sparse


HOW TO REPRODUCE THE BUG:
-------------------------
X = sparse(rand(2,2));
Y = mtlb_sparse(X);

added 6.0.0 Bugzilla Medium Sparse labels

By Samuel GOUGEON (@sgougeon)

I was not reporting that mtlb_sparse() should be restored. I posted this report while i was updating the typeof help page, and comparing with the "type" page, i saw the "Matlab sparse" type.

So, i am just telling that if mtlb_sparse() is removed, its documentation should also be removed, and the "type" page shall be updated.

Thanks

By Aashay Singhal

What about some files in which this function is used still? Like:- http://gitweb.scilab.org/?p=scilab.git;a=blob;f=scilab/modules/compatibility_functions/macros/%25msp_t.sci , http://gitweb.scilab.org/?p=scilab.git;a=blob;f=scilab/modules/compatibility_functions/macros/%25msp_e.sci , http://gitweb.scilab.org/?p=scilab.git;a=blob;f=scilab/modules/compatibility_functions/macros/%25msp_abs.sci Should i remove it from these files also and commit?

Thanks

By Samuel GOUGEON (@sgougeon)

The gateway file SCI/sparse/gateway/sparse_gateway.xml in which mtlb_sparse() was declared up to Scilab 5.5.2 no longer exists.

Other primitives of this module have been rewritten in C++ (there were in C and Fortran), but not mtlb_sparse().

mtlb_sparse() is linked to the Matlab sparse type()==7. Is this abandoned? => to be asked on devs@​​

By Federico Miyara

If the deprecation of this function is intentional and there is no other way to generate a variable of type 7 (Matlab sparse matrix) then this type should be removed.

By Samuel GOUGEON (@sgougeon)

@​​Federico: Definitely agreed. I put this point in private discussions with the core team one year ago.

Your thread on users@​​ about this: http://mailinglists.scilab.org/Scilab-users-mtlb-sparse-tp4039998.html

By Samuel GOUGEON (@sgougeon)

Documentation removal proposed for Scilab 6.1.1 in review @​​ https://gitlab.com/scilab/legacy_codereview/-/wikis/changes/21562

By Samuel GOUGEON (@sgougeon)

Documentation actually removed from Scilab 6.1.1 by df954d31

closed

marked this issue as related to #16285 (closed)

assigned to @sgougeon

mentioned in issue #16285 (closed)