remove extraneous operations when not logging

[colinsheppard]

for PEVI-nolog.nlogo, currently only the lines that use "log-data" are being commented out. as found by profiling, it's critical that we also comment out all commands that are only relevant to logging.

for example, in "arrive" the lines

let #is-scheduled true
if item current-itin-row itin-change-flag = 1 [ set #is-scheduled false ]

are not needed because #is-scheduled is only used to log data. there are numerous more examples of this in arrive and throughout the whole code base.

instead of maintaining a separate copy of PEVI.nlogo and PEVI-nolog.nlogo, please do the following:

• add a flag to the end of every line in PEVI.nlogo that should be commented out when logging is not needed, e.g.:

  let #is-scheduled true ;;; LOG if item current-itin-row itin-change-flag = 1 [ set #is-scheduled false ] ;;; LOG

• write a script (Colin can do this) that takes PEVI.nlogo as input and spits out PEVI-nolog.nlogo as output with all of the appropriate lines commented out.

[colinsheppard]

d2128434889d18b9d5636dca4085b0ff8b1599e6

[arharris]

Currently, the original PEVI uses calc-time-until-end-charge-with-logging in procedure charge-time-event-scheduler; with logging statements removed, this creates an error. Currently fixing this by manually deleting the "with-logging" portion, going to try to work an automatic fix into nolog.R

[colinsheppard]

Andy, make sure you regularly pull changes from the repo, I already fixed the calc-time-until-end-charge-with-logging, see 5844619cee17636395b4093d9534eacaa2ae688c