SCM

SCM Repository

[matrix] Diff of /pkg/TODO
ViewVC logotype

Diff of /pkg/TODO

Parent Directory Parent Directory | Revision Log Revision Log | View Patch Patch

revision 2222, Fri Jul 18 10:00:25 2008 UTC revision 2223, Fri Jul 18 23:04:48 2008 UTC
# Line 189  Line 189 
189                   from class "integer" to "numeric" (i.e. double prec),                   from class "integer" to "numeric" (i.e. double prec),
190    since we want to be able to coerce large sparse matrices to sparse    since we want to be able to coerce large sparse matrices to sparse
191    vectors, where length maybe considerably larger than 2^32.    vectors, where length maybe considerably larger than 2^32.
192    
193    - src/Csparce.c etc:  grep 'AS_.*sparse_diagU2N' src/*.?
194      reveals
195         o AS_CHM_SP(Csparse_diagU2N(.)) : as_cholmod_sparse(..)  \ src/chm_common.h
196         o AS_CHM_TR(Tsparse_diagU2N(.)) : as_cholmod_triplet(..) /
197         o AS_CSP   (Csparse_diagU2N(.)) : Matrix_as_cs(..)       > src/cs_utils.h
198    
199      all of which are "inefficient", because several "S4 <-> Cholmod" or
200      "S4 <-> CS" coercions happen inside.
201    
202      Should make the "inner" Csparse_diagU2N()  part of the
203        as_cholmod_sparse() or Matrix_as_cs() functions.
204        Drawback: extraneous check even for all AS_*(x) calls where *sparse_diagU2N()
205        is unneeded because x is known *not* to be triangular.
206    
207      To evade the drawback, do this by adding a new boolean argument
208      'check_Udiag' to the C functions.
209       --> This is an API change however, notably since we export as_cholmod_sparse
210          ( in inst/include/Matrix.h )
211    
212    [[ have done the as_cholmod_* ones, but not yet the Matrix_as_cs() one ]]

Legend:
Removed from v.2222  
changed lines
  Added in v.2223

R-Forge@R-project.org
ViewVC Help
Powered by ViewVC 1.0.0  
Thanks to:
Vienna University of Economics and Business University of Wisconsin - Madison Powered By FusionForge