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
Currently, plot_pileup() is exported, and depends on get_pileup() to generate it's first argument; yet get_pileup() is not exported. If the intention is for plot_pileup() to be user-facing, then it seems that either get_pileup() should also be exported, or plot_pileup() should take a file as its argument instead of a pileup. My vote would be for the first option, to export get_pileup(), as I personally find it useful.
The text was updated successfully, but these errors were encountered:
mikemc
changed the title
Exportingget_pileup()
Exporting get_pileup()May 14, 2024
Currently,
plot_pileup()
is exported, and depends onget_pileup()
to generate it's first argument; yetget_pileup()
is not exported. If the intention is forplot_pileup()
to be user-facing, then it seems that eitherget_pileup()
should also be exported, orplot_pileup()
should take a file as its argument instead of a pileup. My vote would be for the first option, to exportget_pileup()
, as I personally find it useful.The text was updated successfully, but these errors were encountered: