Replies: 2 comments
-
Hi @cvanderaa
The proposed
|
Beta Was this translation helpful? Give feedback.
-
Hello @LiNk-NY, Thank your prompt and clear answer. This will help us guide our discussion for |
Beta Was this translation helpful? Give feedback.
-
Hi MAE team!
tldr: what were your reasons for having
getWithColData(mae, 1)
andmae[[1]]
as separate functions?Our
QFeatures
packages heavily relies onMultiAssayExperiment
.We are rethinking our interface for extracting assays from our objects (see rformassspectrometry/QFeatures#212). We realized that user and developer are often not aware of the
getWithColData(mae, 1)
, but instead use the more naturally feelingmae[[1]]
syntax which doesn't manage colData. Conversely, I find myself usinggetWithColData()
most of the time because colData management is much better. See here for an example:In option 1. we only get the colData within the
SummarizedExperiment
object, wheter 2. provides additional information from thecolData
in theMultiAssayExperiment
object.I personnally see no reason to prefer behavior in 1. over 2. Therefore, we were thinking reimplementing 1. to rely on
getWithColData()
and provide as much colData as available. Before doing so, I prefer to hear your opinion and to have your thoughts whether this could be a good or bad idea.(ping @lgatto)
Beta Was this translation helpful? Give feedback.
All reactions