Skip to content

Commit

Permalink
Added print warnings when getGRcoverageFromBw() fails to download cov…
Browse files Browse the repository at this point in the history
…erage data.
  • Loading branch information
jmw86069 committed Nov 2, 2024
1 parent e8816c7 commit a1de394
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 2 deletions.
5 changes: 5 additions & 0 deletions NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,12 @@
## changes

* Moved functions into separate files for better code review.
* `getGRcoverageFromBw()`

* When it fails to download a bigwig file, it prints the error
which is not always informative. Now it also prints the warning,
which can tell whether the URL was available, or another error
was caused by this step.

# splicejam 0.0.81.900

Expand Down
3 changes: 3 additions & 0 deletions R/jambio-gr-coverage.R
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,10 @@ getGRcoverageFromBw <- function
bwUrl,
"', returning NULL.");
jamba::printDebug(warnText);
# print the error
print(e);
# print any associated warnings
print(warnings());
warning(warnText);
NULL;
});
Expand Down
2 changes: 1 addition & 1 deletion man/combineGRcoverage.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion man/getGRcoverageFromBw.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit a1de394

Please sign in to comment.