Re: [Bioc-devel] TabixFileList() constructor broken in devel

2017-01-05 Thread Robert Castelo
On 05/01/2017 01:37, Martin Morgan wrote: On 12/29/2016 12:25 PM, Vincent Carey wrote: A while ago Martin and I discussed the possibility of idempotence for TabixFile ... this seemed to make sense and so TabixFile function (file, index = paste(file, "tbi", sep = "."), ..., yieldSize = NA_int

Re: [Bioc-devel] TabixFileList() constructor broken in devel

2017-01-04 Thread Martin Morgan
On 12/29/2016 12:25 PM, Vincent Carey wrote: A while ago Martin and I discussed the possibility of idempotence for TabixFile ... this seemed to make sense and so TabixFile function (file, index = paste(file, "tbi", sep = "."), ..., yieldSize = NA_integer_) { if (is(file, "TabixFile"))

Re: [Bioc-devel] TabixFileList() constructor broken in devel

2016-12-29 Thread Vincent Carey
A while ago Martin and I discussed the possibility of idempotence for TabixFile ... this seemed to make sense and so > TabixFile function (file, index = paste(file, "tbi", sep = "."), ..., yieldSize = NA_integer_) { if (is(file, "TabixFile")) return(file) tryCatch({ .

Re: [Bioc-devel] TabixFileList() constructor broken in devel

2016-12-29 Thread Robert Castelo
hi Lori, thanks for the clarification. if this is an intended change then i'd say the documentation needs to be updated since, as i showed in my email below, it currently says that for functions such as 'TabixFileList()' the input 'file' argument should be a 'TabixFile' instance and not a char

Re: [Bioc-devel] TabixFileList() constructor broken in devel

2016-12-28 Thread Shepherd, Lori
Thank you for bringing this to our attention. There were some updates made a few weeks ago to the behavior and input of 'TabixFileList()' . Its current implementation will work with a character vector as input. I will investigate further into why this change was made and if it needs to be addr