deepstop
commented
9 years ago - the error seemed to be located in LENGTH != length
- now the original source changed which causes merge conflicts.
- if you want i can add back the changes against the latest version
This change affects reading a Pgf file (via readPgf): Using the function readPgf(file, indices = NULL) did not read any elements and just returned the header. This behavior caused trouble, since you cant get the full content of the file without knowing the indices in advance. Also it looks like other packages assume the body to be read, e.g. in package "pdInfoBuilder" (parsePgfClf >> probesetIdxToTripletIdx) (most likely the problem for this report: https://stat.ethz.ch/pipermail/bioconductor/2013-May/052768.html) Plus a change for getting the right indices (imagine an index val of 1, the i++ will be 1 for the first loop, 1-1 != 1). additional suggestion: If you want the possibility to just read the header of the file, you should introduce an argument readBody=TRUE to readPgf(), (as described in the according man page), OR to speed up, add a break for indices[0]==0, which currently just loops over the data.
Signed-off-by: Grischa Toedt