I had a check built into read_sumstats to see if the path was a remote URL. But now that this func can take in data.tables directly, will need to adjust this to avoid an error.
Actually what's the use case here? read_sumstats only reads in the data to R so essentially could just not call this function if a dataframe/datatable is passed. can you give an example of where the issue would happen so I can correct?
I've just added a check to read_sumstats anyway that just returns the summary statistics if they were already in memory as a dataframe/data.table object just to be safe. Again, I'll push this change today:
> sumstats <- data.table::fread(system.file("extdata","eduAttainOkbay.txt",
+ package="MungeSumstats"))
> sumstats <- MungeSumstats::read_sumstats(path = sumstats)
Summary statistics passed as R object.
I had a check built into
read_sumstats
to see if the path was a remote URL. But now that this func can take in data.tables directly, will need to adjust this to avoid an error.Session Info
Ah that makes sense, I only adjusted the format_sumstats function, I'll update now
Actually what's the use case here? read_sumstats only reads in the data to R so essentially could just not call this function if a dataframe/datatable is passed. can you give an example of where the issue would happen so I can correct?
I've just added a check to read_sumstats anyway that just returns the summary statistics if they were already in memory as a dataframe/data.table object just to be safe. Again, I'll push this change today: