[tor-bugs] #26998 [Metrics/Website]: Make suggested changes to per-graph CSV files and deploy them after August 15

[Tor Bug Tracker & Wiki]

#26998: Make suggested changes to per-graph CSV files and deploy them after August
15
-----------------------------+-----------------------------
 Reporter:  karsten          |          Owner:  karsten
     Type:  enhancement      |         Status:  merge_ready
 Priority:  Medium           |      Milestone:
Component:  Metrics/Website  |        Version:
 Severity:  Normal           |     Resolution:
 Keywords:                   |  Actual Points:
Parent ID:                   |         Points:
 Reviewer:                   |        Sponsor:
-----------------------------+-----------------------------

Comment (by karsten):

 Okay, great! I'm merging this now.

 Here's one smaller change that I'm also making, which is based on a bug
 report in an unrelated ticket:

 {{{
 diff --git a/src/main/R/rserver/graphs.R b/src/main/R/rserver/graphs.R
 index 2502c0e9..66a1414e 100644
 --- a/src/main/R/rserver/graphs.R
 +++ b/src/main/R/rserver/graphs.R
 @@ -895,7 +895,8 @@ plot_userstats <- function(start_p, end_p, node_p,
 variable_p, value_p,
      events_p, path_p) {
    load(paste(rdata_dir, "clients-", node_p, ".RData", sep = ""))
    c <- data
 -  u <- c[c$date >= start_p & c$date <= end_p, ]
 +  u <- c[c$date >= start_p & c$date <= end_p, c("date", "country",
 "transport",
 +      "version", "lower", "upper", "clients")]
    u <- rbind(u, data.frame(date = start_p,
        country = ifelse(variable_p == "country" & value_p != "all",
 value_p, ""),
        transport = ifelse(variable_p == "transport", value_p, ""),
 }}}

 I'm going to squash that change into 895a111 when rebasing to master.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/26998#comment:4>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online