7 |
?Math.data.frame and the examples in |
?Math.data.frame and the examples in |
8 |
?SetGeneric |
?SetGeneric |
9 |
|
|
10 |
- check to see if the .onLoad function to require the methods package |
- bCrosstab(): do we really want the diagonal "V:V" crosstabs? |
11 |
(in the AllClass.R file) is needed |
(if so or in any case: add to man/bCrosstab.Rd ) |
12 |
|
|
13 |
- Should the uplo and diag slots continue to be stored as character? |
- src/Metis/ : one of the two Makefiles needs fixing, as changing |
14 |
An alternative is to use a factor as in the enum values for the |
src/Metis/*.c does not lead to recompilation. |
15 |
cblas. |
--DB - it seems both Makefiles need fixing. I think I have the |
16 |
|
src/Metis/Makefile fixed but not src/Makefile |
|
- Organization of the source code files - right now they are organized |
|
|
according to class (e.g. dgeMatrix.R, dgeMatrix.h, dgeMatrix.c). Is |
|
|
there a better way? |
|
17 |
|
|
18 |
- Fix the calculation of the Dim slot for the crossprod method for |
- man/Matrix.Rd : has example with dimnames, but we just drop them! |
19 |
dgCMatrix objects (too tired to do that now). |
MM thinks dimnames should be supported (but then ...) |
20 |
|
|
21 |
|
- data/ : names 'mm' and even more 'y' are ``too short''. |
22 |
|
If we really want to keep them, don't use "LazyData" |
23 |
|
(such that one needs data(*) explicitly); |
24 |
|
But MM would rather want something like ex.mm and ex.y |
25 |
|
|
26 |
|
- spelling style: Should "coersion" be "coercion" ? |
27 |
|
|
28 |
|
- "factors" maybe should move up to "Matrix" |
29 |
|
|
30 |
|
------- |
31 |
|
|
32 |
|
We have a (at least one) basic problem : |
33 |
|
Currently the show() method fail sometime after coercion: |
34 |
|
e.g. 'sy' show()s wrongly, even though it "str()" fine : |
35 |
|
|
36 |
|
(po <- crossprod(Matrix(0:3, 2))) # ok |
37 |
|
(ge <- as(po, "dgeMatrix")) # ok |
38 |
|
(sy <- as(po, "dsyMatrix")) # BAD |
39 |
|
str(sy) # looks fine |
40 |
|
|
41 |
|
or |
42 |
|
example(expand) # -> ex$L and ex$U look bad, however |
43 |
|
as(ex$L, "dgeMatrix") # `works' |
44 |
|
|
45 |
|
{Of course, we don't need a workaround but must understand |
46 |
|
and solve the problem} |