Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document and export get_out_of_bound_index #53

Open
rpolicastro opened this issue Jan 19, 2021 · 2 comments
Open

Document and export get_out_of_bound_index #53

rpolicastro opened this issue Jan 19, 2021 · 2 comments

Comments

@rpolicastro
Copy link

rpolicastro commented Jan 19, 2021

Hello,

The internal function get_out_of_bound_index is handy for getting the index of out-of-bounds ranges. I think people would find common use for it if it was documented and exported.

Cheers!

@hpages
Copy link
Contributor

hpages commented Jan 22, 2021

Hi @rpolicastro ,

What's your use case? My approach is to export and document this kind of low-level helper function only if there's a need for it.

Thanks,
H.

@rpolicastro
Copy link
Author

Hello @hpages,

My use was prompted by an old answer to a bioconductor post you made here.

I have ranges of variable width I want to extend, and then remove any that result in out-of-bounds ranges. There are of course ways to do so without this function, but the function was both simple and convenient.

I ended up copy/pasting the code for it into my own repo, but of course it would be better if the function itself was exported to avoid code duplication.

Cheers,
Bob

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants