Browse code

Add TODOs

Peter Hickey authored on 13/06/2018 00:25:21
Showing 3 changed files

... ...
@@ -172,7 +172,9 @@ BSseq <- function(M = NULL, Cov = NULL, coef = NULL, se.coef = NULL,
172 172
             stop("Cannot collapse when 'coef' or 'se.coef' are present.")
173 173
         }
174 174
         # NOTE: reduce() sorts the output
175
+        # TODO: Clarify above comment.
175 176
         unique_gr <- unique(gr)
177
+        # TODO: type = "equal"?
176 178
         ol <- findOverlaps(unique_gr, gr)
177 179
         gr <- unique(gr)
178 180
         idx <- as.list(ol)
... ...
@@ -1,5 +1,5 @@
1 1
 # TODO: Need to update NAMESPACE
2
-# TODO: Search both forward and reverse strands
2
+# TODO: Search both forward and reverse strands in a single search.
3 3
 # TODO: Default value of seqlevels as missing or NULL? Check what other
4 4
 #       functions with this arg use.
5 5
 findCytosines <- function(BSgenome, context = c("CG", "CA", "CC", "CT"),
... ...
@@ -578,3 +578,4 @@ read.bismark <- function(files,
578 578
 #       .cytosineReport files and don't want this behaviour (i.e. you want to
579 579
 #       retain strand) then you'll need to construct your own 'gr' and pass
580 580
 #       this to the function. Add unit tests for this behaviour.
581
+# TODO: Use verbose = getOption("verbose") as default.