great-northern-diver / loon

A Toolkit for Interactive Statistical Data Visualization
http://great-northern-diver.github.io/loon/
GNU General Public License v2.0
48 stars 16 forks source link

tcltk dependency for loon #22

Closed chriskuchar closed 5 years ago

chriskuchar commented 5 years ago

Please remove Tcl/Tk from your dependency list of installed packages and resubmit this to CRAN.

Tcl/Tk is now included in Base R and is not downloadable as a package. Because you have it listed as a dependency, people can't download loon now.

The package called rfviz has just been submitted to CRAN and has been accepted. It uses loon as a dependency. As a result, my package automatically tries to install dependencies for loon. Since tcl/tk is an un-needed dependency, users cannot install my package from CRAN. Please fix this as soon as possible.

Thanks, Chris Beckett

waddella commented 5 years ago

Hi @chrisbeckett8,

Congratulations on your new rfviz R package submission. I will have a look at it.

Tcl/Tk has been in base R for a while. loon needs tcltk in the dependencies as tcltk needs to be attached to the search path. This also should not prevent the installation of the rfviz package. For example I can install the package well with R 3.5.1 on a Mac:

screen shot 2018-11-22 at 10 52 48

The reason you might not be able to install your package is because you are missing tcltk which needs some special care on some operating systems, for example:

https://cran.r-project.org/bin/macosx/

you can see my setup instructions for loon here:

http://waddella.github.io/loon/install.html

It is also common practice (and needed) to put base R packages into the dependencies, see:

https://CRAN.R-project.org/package=tcltk2

and

https://CRAN.R-project.org/package=MASS

If my installation instructions on the loon website do not address the mentioned dependency problems then please share a screenshot of an installation that fails.

Best, Adrian

chriskuchar commented 5 years ago

Adrian,

Thanks for the help in resolving my installation error. I ended up having to make sure my Mac had the most recent update, I reinstalled R and Xtools and it worked great. Thanks for helping me see my error in installing loon.

Best, Chris

On Thu, Nov 22, 2018 at 3:07 AM Adrian Waddell notifications@github.com wrote:

Hi @chrisbeckett8 https://github.com/chrisbeckett8,

Congratulations on your new rfviz R package submission. I will have a look at it.

Tcl/Tk has been in base R for a while. loon needs tcltk in the dependencies as tcltk needs to be attached to the search path. This also should not prevent the installation of the rfviz package. For example I can install the package well with R 3.5.1 on a Mac:

[image: screen shot 2018-11-22 at 10 52 48] https://user-images.githubusercontent.com/7786462/48895475-705b8400-ee45-11e8-951b-045df1db8f4e.png

The reason you might not be able to install your package is because you are missing tcltk which needs some special care on some operating systems, for example:

https://cran.r-project.org/bin/macosx/

you can see my setup instructions for loon here:

http://waddella.github.io/loon/install.html

It is also common practice (and needed) to put base R packages into the dependencies, see:

https://CRAN.R-project.org/package=tcltk2

and

https://CRAN.R-project.org/package=MASS

If my installation instructions on the loon website do not address the mentioned dependency problems then please share a screenshot of an installation that fails.

Best, Adrian

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/waddella/loon/issues/22#issuecomment-440978802, or mute the thread https://github.com/notifications/unsubscribe-auth/AXdIAchu3_VtPWx6SLM5RzNVv0BD1aQPks5uxnd3gaJpZM4YoUgN .

chriskuchar commented 5 years ago

Here is the best link for the package documentation: https://chrisbeckett8.github.io/Rfviz

On Thu, Dec 20, 2018 at 5:26 PM Christopher Beckett chrisbeckett8@gmail.com wrote:

Adrian,

Thanks for the help in resolving my installation error. I ended up having to make sure my Mac had the most recent update, I reinstalled R and Xtools and it worked great. Thanks for helping me see my error in installing loon.

Best, Chris

On Thu, Nov 22, 2018 at 3:07 AM Adrian Waddell notifications@github.com wrote:

Hi @chrisbeckett8 https://github.com/chrisbeckett8,

Congratulations on your new rfviz R package submission. I will have a look at it.

Tcl/Tk has been in base R for a while. loon needs tcltk in the dependencies as tcltk needs to be attached to the search path. This also should not prevent the installation of the rfviz package. For example I can install the package well with R 3.5.1 on a Mac:

[image: screen shot 2018-11-22 at 10 52 48] https://user-images.githubusercontent.com/7786462/48895475-705b8400-ee45-11e8-951b-045df1db8f4e.png

The reason you might not be able to install your package is because you are missing tcltk which needs some special care on some operating systems, for example:

https://cran.r-project.org/bin/macosx/

you can see my setup instructions for loon here:

http://waddella.github.io/loon/install.html

It is also common practice (and needed) to put base R packages into the dependencies, see:

https://CRAN.R-project.org/package=tcltk2

and

https://CRAN.R-project.org/package=MASS

If my installation instructions on the loon website do not address the mentioned dependency problems then please share a screenshot of an installation that fails.

Best, Adrian

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/waddella/loon/issues/22#issuecomment-440978802, or mute the thread https://github.com/notifications/unsubscribe-auth/AXdIAchu3_VtPWx6SLM5RzNVv0BD1aQPks5uxnd3gaJpZM4YoUgN .