Re: [lmi] Strategy for Halloween release

[Greg Chicares]

On 2015-10-09 18:12, Greg Chicares wrote:
[...]
> assert consistent run order across all cells:
>   https://github.com/vadz/lmi/pull/1/files?w=1
>   (useful starting point for ensuring input consistency for group quotes)

Committed 20151010T1930Z, revision 6346.

Correct me if I'm wrong, but I thought parentheses had become unnecessary
in code like
  std::cout << "The number is " << (n + 1) << std::endl;
Other than that, our implementations don't differ materially. You had
hoisted cells.size() out of the loop, and I had thought of hoisting
case_default["RunOrder"], but I guess even a decade-old compiler must do
that sort of optimization routinely, based on your measurements here:
  https://github.com/vadz/lmi/pull/1/
(was that gcc-3.4.5, or some other compiler?).

I created a trivial census with three cells, changing the run order in the
third cell. Formerly, this would have "worked" silently. Now, running it in
the GUI triggers an error message. So does running it at the command line:

/opt/lmi/bin[0]$./lmi_cli_shared --accept --ash_nazg --data_path=/opt/lmi/data 
--file=/opt/lmi/eraseme/run_order.cns
Case-default run order 'Life by life' differs from run order 'Month by month' 
of cell number 3. Make this consistent before running illustrations.
[file /lmi/src/lmi/illustrator.cpp, line 261]