You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Downloading chain files from the UCSC server is unnecessary and time-consuming, especially when dealing with a large amount of data requiring base position transformation. Allowing the option to load previously downloaded chain files would significantly enhance speed (as it only needs to be downloaded once) and reduce the load on the UCSC server.
The text was updated successfully, but these errors were encountered:
laleoarrow
changed the title
Add local chain file option for MungeSumstats::liftover
Add local chain file option for MungeSumstats::liftover # enhancement
Apr 23, 2024
Hey! Thanks for this and the PR - I will implement these changes with a few modifications so it is more generally useable.I'll also add some unit tests. Just to note though, get_chain_file() already has functionality to not download files more than once so it actually isn't that much of an issue but I agree having the option to use local versions is worthwhile.
Secondly, I'll propagate the parameter to format_sumstats() function as this will be the function most people use in the package. Going to close the PR and make these changes locally.
I'll add you as a contributor to the package README. Also worth you having a look at the changes I made (v 1.11.10), there was a a bit extra required from your PR to get it to work and worth being aware if you contribute again! Thanks!
Downloading chain files from the UCSC server is unnecessary and time-consuming, especially when dealing with a large amount of data requiring base position transformation. Allowing the option to load previously downloaded chain files would significantly enhance speed (as it only needs to be downloaded once) and reduce the load on the UCSC server.
The text was updated successfully, but these errors were encountered: