stocnet / rsiena

An R package for Simulation Investigation for Empirical Network Analysis
http://www.stats.ox.ac.uk/~snijders/siena/
GNU General Public License v3.0
101 stars 25 forks source link

tcl-tk should be optional #1

Closed TomSnijders closed 4 years ago

TomSnijders commented 4 years ago

In the first ever R-CMD-check of RSiena on GitHub, there was a failure for the Mac. I think this relates to the message I received from Brian Ripley May 4th:

<<<<<<<<<<<<<>>>>>>>>>>>>>>>>> 'Writing R Extensions' has warned for years that Tktable is no longer widely available (it was last updated in 2008 and has been removed from e.g. Fedora). It is no longer distributed with R 4.0.0 on Windows and is intended to be removed on macOS (that intention was originally for R 4.0.0 but Tcl/Tk has not yet been updated there). Your choices would appear to be

I plan to take tcltk out of "Imports", probably move it to "suggests", and use require(tcltk) or some such as a condition for opening the Siena gui during operation of siena07.

jhollway commented 4 years ago

How critical is a Tktable based gui nowadays? We could instead offer a more verbose console reporting system for teaching purposes and for those who want to see RSiena "in action"? Or are there other dependencies?

perblock commented 4 years ago

I sometimes use the gui to speed up estimation for rough guesstimates (using end phase 2) and convergence (restarting after subphase 2). So I think the gui helps

TomSnijders commented 4 years ago

I find the gui essential for my own use. I use it to monitor what happens in case of difficult convergence or divergence. Perhaps I can write something more about this in the manual. For the solution of this issue, my idea is to use if require(tcltk) and then force batch mode if tcltk is not available. When I’m back from holidays, late July, this is one of the first things on my list.

Cheers, Tom

================================================= Tom A.B. Snijders Professor of Statistics and Methodology Department of Sociology, University of Groningen Emeritus Fellow, Nuffield College, University of Oxford Associate Member, Dept. of Statistics, University of Oxford

From: perblock notifications@github.com Sent: 02 July 2020 16:43 To: snlab-nl/rsiena rsiena@noreply.github.com Cc: Tom Snijders tom.snijders@nuffield.ox.ac.uk; Assign assign@noreply.github.com Subject: Re: [snlab-nl/rsiena] tcl-tk should be optional (#1)

I sometimes use the gui to speed up estimation for rough guesstimates (using end phase 2) and convergence (restarting after subphase 2). So I think the gui helps

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHubhttps://github.com/snlab-nl/rsiena/issues/1#issuecomment-653047495, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHQLN7C65OVWMWBOBBVLPXDRZSMGHANCNFSM4N3M5Y5Q.

jhollway commented 4 years ago

Both uses make sense.

jhollway commented 4 years ago

@auzaheta, @stadtfeldethz mentioned you had been looking into this issue?

auzaheta commented 4 years ago

Not about tcltk. It's a change in the configure file regarding MPI, OpenMP, and C++ in the different operative systems (especially Mac). I'll open a new issue with the specification.

jhollway commented 4 years ago

Great, yes please raise this as another issue