Skip to content

Commit

Permalink
dependency fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
philouail committed Jun 5, 2024
1 parent 8800b36 commit 6f177fd
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 10 deletions.
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
Package: xcms
Version: 4.3.1
Version: 4.3.2
Title: LC-MS and GC-MS Data Analysis
Description: Framework for processing and visualization of chromatographically
separated and single-spectra mass spectral data. Imports from AIA/ANDI NetCDF,
Expand Down
8 changes: 6 additions & 2 deletions NAMESPACE
Original file line number Diff line number Diff line change
Expand Up @@ -559,7 +559,7 @@ importMethodsFrom("Spectra", "uniqueMsLevels")
importMethodsFrom("Spectra", "selectSpectraVariables")
importMethodsFrom("Spectra", "setBackend")
importMethodsFrom("Spectra", "dataOrigin")
importMethodsFrom("Spectra", "dataStorage")
importMethodsFrom("Spectra", "dataStorage", "dataStorage<-")
importMethodsFrom("Spectra", "spectraVariables")
importMethodsFrom("Spectra", "rtime<-")
importMethodsFrom("Spectra", "ionCount")
Expand All @@ -568,7 +568,8 @@ importMethodsFrom("Spectra", "$")
importMethodsFrom("Spectra", "uniqueMsLevels")
importMethodsFrom("Spectra", "backendBpparam")
importFrom("Spectra", "MsBackendMemory", "dataStorageBasePath",
"dataStorageBasePath<-")
"dataStorageBasePath<-", "processingChunkSize",
"dropNaSpectraVariables", "MsBackendMzR", "filterEmptySpectra")

## MsExperiment things
importClassesFrom("MsExperiment", "MsExperiment")
Expand All @@ -595,10 +596,13 @@ exportMethods("fileNames")
## saving/loading xcms objects things
importFrom("jsonlite", "serializeJSON", "write_json", "unserializeJSON",
"read_json")
importFrom("stats", "setNames")
importFrom("MsCoreUtils", "between")
export("RDataParam")
export("PlainTextParam")
exportMethods("storeResults", "loadResults")


## filtering features things
importFrom("MetaboCoreUtils", "rowRsd", "rowDratio", "rowPercentMissing",
"rowBlank", "mclosest")
Expand Down
4 changes: 4 additions & 0 deletions R/PlainTextParam.R
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,10 @@
#'
#' @importFrom MsCoreUtils common_path
#'
#' @importFrom Spectra processingChunkSize dropNaSpectraVariables Spectra MsBackendMzR
#'
#' @importFrom stats setNames
#'
#' @examples
#' ## Load test data set of class `MsExperiment`
#' library(MsExperiment)
Expand Down
2 changes: 2 additions & 0 deletions R/XcmsExperiment-plotting.R
Original file line number Diff line number Diff line change
Expand Up @@ -437,6 +437,8 @@ setMethod(
#'
#' @importFrom grDevices n2mfrow
#'
#' @importFrom Spectra filterEmptySpectra
#'
#' @export
#'
#' @author Johannes Rainer
Expand Down
8 changes: 1 addition & 7 deletions tests/testthat/test_PlainTextParam.R
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,6 @@ test_that("storeResults,loadResults,PlainTextParam,MsBackendMzR works", {
b <- dropNaSpectraVariables(b) #the function does this to be robust, is it a problem ? i should mention it in the doc
expect_equal(b@spectraData, b2@spectraData)
expect_equal(peaksVariables(b), peaksVariables(b2)) # true even without forcing the slot
expect_equal(mz(b), mz(b2))
expect_equal(intensity(b), intensity(b2))
})

test_that("storeResults,loadResults,PlainTextParam,Spectra works", {
Expand All @@ -41,8 +39,6 @@ test_that("storeResults,loadResults,PlainTextParam,Spectra works", {
expect_equal(s@processing, s2@processing)
expect_equal(processingChunkSize(s), processingChunkSize(s2))
expect_equal(s@backend@spectraData, s2@backend@spectraData)
expect_equal(mz(s), mz(s2))
expect_equal(intensity(s), intensity(s2))
expect_equal(rtime(s), rtime(s2))
expect_no_error(filterRt(s2, c(3000, 3500)))
})
Expand All @@ -68,8 +64,6 @@ test_that("storeResults,loadResults,PlainTextParam,MsExperiment works", {
b <- spectra(load_mse)
expect_equal(a@processingQueue[[1L]]@ARGS, b@processingQueue[[1L]]@ARGS)
expect_equal(rtime(a), rtime(b))
expect_equal(intensity(a), intensity(b))
expect_equal(mz(a), mz(b))
expect_no_error(filterRt(load_mse, c(3000, 3500)))
})

Expand Down Expand Up @@ -100,7 +94,7 @@ test_that("storeResults,loadResults,PlainTextParam,XcmsExperiment works", {
expect_equal(adjustedRtime(xmse_full), adjustedRtime(load_xmse))
expect_no_error(filterRt(load_xmse, c(3000, 3500)))
## not sure how to check for the processHistory slot
##
## still not sure how to create UT for spectraPath
})


0 comments on commit 6f177fd

Please sign in to comment.