Skip to content

Commit 9ce6b61

Browse files
committed
Correct typo in parameter startsAreBased
1 parent 17ff90e commit 9ce6b61

3 files changed

Lines changed: 25 additions & 25 deletions

File tree

R/prepareInputRegions.R

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@
111111
#' "galGal6","dm6", "ce11", and "sacCer3". Please see also
112112
#' help for [Seqinfo::seqinfo()] for more details.
113113
#'
114-
#' @param starsAreBased Either 0, 1 (Default), or NA. Define if the provided
114+
#' @param startsAreBased Either 0, 1 (Default), or NA. Define if the provided
115115
#' input data is 0 or 1-based. Only, if paramter is NA
116116
#' then GenomicRanges object, tibbles and dataframes are
117117
#' considered 1-based, while data loaded from a
@@ -170,7 +170,7 @@ prepareInputRegions <- function(
170170
data,
171171
outputFormat = "GenomicRanges",
172172
genome = NA,
173-
starsAreBased = 1,
173+
startsAreBased = 1,
174174
showMessages = TRUE
175175
) {
176176
### -----------------------------------------------------------------------###
@@ -281,17 +281,17 @@ prepareInputRegions <- function(
281281
### Check if input is 0 or 1 based
282282
### -----------------------------------------------------------------------###
283283

284-
if(!is.na(starsAreBased) & !is.numeric(starsAreBased)) {
284+
if(!is.na(startsAreBased) & !is.numeric(startsAreBased)) {
285285
cli::cli_abort(c(
286-
"x" = "Argument {.arg starsAreBased} has to be either numeric {.val 0},
286+
"x" = "Argument {.arg startsAreBased} has to be either numeric {.val 0},
287287
numeric {.num 1} or {.val NA}}.",
288-
"i" = "Argument {.arg starsAreBased} is {.val {starsAreBased}}."
288+
"i" = "Argument {.arg startsAreBased} is {.val {startsAreBased}}."
289289
))
290290
}
291291

292-
if(is.na(starsAreBased)) {
292+
if(is.na(startsAreBased)) {
293293
cli::cli_inform(c(
294-
"i" = "Argument {.arg starsAreBased} is {.val {starsAreBased}}."
294+
"i" = "Argument {.arg startsAreBased} is {.val {startsAreBased}}."
295295
))
296296

297297
if (all(required_samplesheet_colnames %in% colnames(data))) {
@@ -312,18 +312,18 @@ prepareInputRegions <- function(
312312

313313
}
314314

315-
} else if (starsAreBased %in% c(0,1)) {
315+
} else if (startsAreBased %in% c(0,1)) {
316316
cli::cli_inform(c(
317-
"i" = "Argument {.arg starsAreBased} is {.val {starsAreBased}}."
317+
"i" = "Argument {.arg startsAreBased} is {.val {startsAreBased}}."
318318
))
319319

320-
if (starsAreBased == 1) {
320+
if (startsAreBased == 1) {
321321
starts.in.df.are.0based <- FALSE
322322
cli::cli_inform(c(
323323
"i" = "Argument {.arg starts.in.df.are.0based} is
324324
{.val {starts.in.df.are.0based}}."
325325
))
326-
} else if (starsAreBased == 0) {
326+
} else if (startsAreBased == 0) {
327327
starts.in.df.are.0based <- TRUE
328328
cli::cli_inform(c(
329329
"i" = "Argument {.arg starts.in.df.are.0based} is
@@ -333,9 +333,9 @@ prepareInputRegions <- function(
333333

334334
} else {
335335
cli::cli_abort(c(
336-
"x" = "Argument {.arg starsAreBased} has to be either {.val 0}, {.val 1}
336+
"x" = "Argument {.arg startsAreBased} has to be either {.val 0}, {.val 1}
337337
or {.val NA}}.",
338-
"i" = "Argument {.arg starsAreBased} is {.val {starsAreBased}}."
338+
"i" = "Argument {.arg startsAreBased} is {.val {startsAreBased}}."
339339
))
340340
}
341341

@@ -375,10 +375,10 @@ prepareInputRegions <- function(
375375
)
376376
} else if (all(required_colnames %in% colnames(data))) {
377377

378-
#if (!is.na(starsAreBased) | starsAreBased == 2) {
378+
#if (!is.na(startsAreBased) | startsAreBased == 2) {
379379
# cli::cli_inform(c(
380380
# "i" = "Provide input {.arg data} is a tibble and the parameter
381-
# {.arg starsAreBased} was set to {.var 0} by user.",
381+
# {.arg startsAreBased} was set to {.var 0} by user.",
382382
# ">" = "Converting input to 1-based."
383383
# ))
384384
# data_prepared <- data |>
@@ -617,10 +617,10 @@ prepareInputRegions <- function(
617617
data_prepared <- data_prepared
618618
} else if (inherits(data, "tbl_df") || inherits(data, "data.frame")) {
619619

620-
if (starsAreBased %in% c(1)) {
620+
if (startsAreBased %in% c(1)) {
621621
data_prepared <- data_prepared
622622

623-
} else if (starsAreBased %in% c(0, NA)) {
623+
} else if (startsAreBased %in% c(0, NA)) {
624624
data_prepared <- data_prepared |>
625625
dplyr::mutate(start = .data$start + 1)
626626
}

man/prepareInputRegions.Rd

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vignettes/peakCombiner.Rmd

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -209,18 +209,18 @@ combined_input |>
209209
prepareInputRegions(
210210
data = combined_input,
211211
outputFormat = "tibble",
212-
starsAreBased = 1,
212+
startsAreBased = 1,
213213
showMessages = FALSE
214214
)
215215
```
216216

217-
If you set the `starsAreBased = 0`, the package assumes the input is 0-based and the value in the column `start` is adjusted.
217+
If you set the `startsAreBased = 0`, the package assumes the input is 0-based and the value in the column `start` is adjusted.
218218

219219
```{r, eval=TRUE}
220220
prepareInputRegions(
221221
data = combined_input,
222222
outputFormat = "tibble",
223-
starsAreBased = 0,
223+
startsAreBased = 0,
224224
showMessages = FALSE
225225
)
226226
```
@@ -268,7 +268,7 @@ sample_sheet
268268
```{r, eval = FALSE}
269269
prepareInputRegions(
270270
data = sample_sheet,
271-
starsAreBased = 0,
271+
startsAreBased = 0,
272272
showMessages = FALSE
273273
)
274274
```
@@ -343,7 +343,7 @@ combined_input |>
343343
prepareInputRegions(
344344
data = combined_input,
345345
outputFormat = "tibble",
346-
starsAreBased = 1,
346+
startsAreBased = 1,
347347
showMessages = FALSE
348348
)
349349
```
@@ -415,7 +415,7 @@ And now we use it as input for `prepareInputRegions`.
415415
prepareInputRegions(
416416
data = syn_data_bed,
417417
outputFormat = "data.frame",
418-
starsAreBased = 0,
418+
startsAreBased = 0,
419419
showMessages = FALSE
420420
)
421421
```

0 commit comments

Comments
 (0)