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

check_install() #1

Open
petermeissner opened this issue Aug 20, 2015 · 4 comments
Open

check_install() #1

petermeissner opened this issue Aug 20, 2015 · 4 comments

Comments

@petermeissner
Copy link

Hey, I just wanted to let you know that while check_install() function is mentioned in the manual of blatr (CRAN: version blatr_1.0.1) it is not accessible:

?check_install
# -> works

check_install
## Error: object 'check_install' not found

While some other functions become available when using the Github version (blatr_1.0.1) this is not true for check_install()

@smbache
Copy link
Owner

smbache commented Aug 20, 2015

The idea was that install_blat and check_install should be called with ::: as they are not meant for common use. Could export it though if it makes more sense...

at least it should be mentioned somewhere, to avoid confusion...

@petermeissner
Copy link
Author

Ah-ok, than I simply did not get it. The one than worked for me because I could copy paste the whole thing while the other I had to write myself.

I thought ::: is an R-Inferno thing ... anyways, I think that if user should use it, it should be accessible - but that's just an opinion.

@smbache
Copy link
Owner

smbache commented Aug 20, 2015

Yeah, not sure what I like more. 99% of the time, users shouldn't use it ;-)

Maybe they could be named .check_blat_install and .install_blat instead, and then exported.

@petermeissner
Copy link
Author

Or go for an explicit usage example in the documentation (as example usage?):

blatr:::install_blat(arch = 32) 
blatr:::check_install() 

Than it is a not really to be used functionality but still easy to use if needed.

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