[rkward-devel] Provide a UI for CRAN mirrors

[Prasenjit Kapat]

Hi,

On Thu, Nov 26, 2009 at 3:06 PM, Thomas Friedrichsmeier
<thomas.friedrichsmeier at ruhr-uni-bochum.de> wrote:
> Hi,
>
> On Thursday 26 November 2009, Prasenjit Kapat wrote:
>> (2) If the urls are stored, then
>> we'll have to do an indexOf(...) search for the hostname combobox,
>> everytime the settings dialog is loaded.
>
> not to worry. The main rule of thumb is: If each call of the code is a direct
> result of a user action, then wasting a couple thousand CPU cycles is nothing.
> Performance is often an issue for something that may run in a loop of some
> sort, but rarely for anything else.

OK, using indexOf() the earlier bug is gone. But things are still hard coded!

>> (2) I like the idea to give the user more readable choices, like the
>> ones in the combobox (Country, State, City, Hostname), rather than the
>> "choices" offered using the Tcl/Tk menu box.
>> (3) Also, being able to store the choice is a plus point, I think.
>
> Both are good points. But perhaps both are solvable without hardcoding
> anything:
>
> 1) In the settings, perhaps we could offer a "pick from list" button, which
> quite simply calls chooseCRANmirror() in R. Hence we'd have a dropdown choice,
> and still be able to save the result.
> 2) We could replace chooseCRANmirror() to offer a nicer menu, with all the
> additional info from getCRANmirrors().
>
> Would that cover all wishes?

Of course, that would be ideal!

-- 
Prasenjit