ccshan
2018-7-10 14:59:00

Good morning. My default task is to make Section 3 shorter and more readable.


carette
2018-7-10 15:02:31

[Sorry for being offline, travel and conference have interfered]


carette
2018-7-10 15:02:44

It seems that we’re currently almost 2 pages over length?


samth
2018-7-10 15:05:20

yes


carette
2018-7-10 15:09:15

As suggested (quite strong) by Klaus Ostermann, we could get rid of Figure 1. I don’t think it really communicates much. It would be good to resurrect it in a talk.


carette
2018-7-10 15:14:42

I’ll start doing some small space-saving edits. Not in Section 3.


samth
2018-7-10 16:48:29

@ccshan I need you to write something about LDA in section 5, where there’s currently a fixme


samth
2018-7-10 16:48:40

I also took out several of your plots to save space


carette
2018-7-10 16:49:47

The plots really help with the message that we are much faster. We should try hard to keep them, even if we had to reduce their size. And be aggressive about cutting in other places.


samth
2018-7-10 16:50:39

the plots I cut were the update times for NB, and the plots with only Psi on them


samth
2018-7-10 16:51:07

I think the message about both of those benchmarks is still very clear


carette
2018-7-10 16:51:26

Ok!


carette
2018-7-10 17:08:02

If we (selectively) had ‘let … and … in’ in some of the figures in section 5, we could gain quite a bit of space without sacrificing readability. Opinions?


ccshan
2018-7-10 17:09:06

Only if it doesn’t affect understanding what the optimizations do


carette
2018-7-10 17:09:20

Also, do we have any control at all over the space after the caption of a figure? Some pages have ridiculous amounts of space wasted there.


carette
2018-7-10 17:09:29

@ccshan I agree.


carette
2018-7-10 17:09:52

[I have to go off for dinner now, but I’ll be back in a couple of hours]


ccshan
2018-7-10 17:10:38

“Wasted” space is due to widow/orphan avoidance, which has its purpose. Space saving calls for directed efforts.


carette
2018-7-10 17:11:47

As usual with latex, saving space in earlier pages often has a dramatic effect. Section 3 is fairly ripe for that…


samth
2018-7-10 18:02:57

after removing the radar chart, we are exactly 1 page over


samth
2018-7-10 18:46:34

@rjnw I’d like to re-generate some of the plots, but the data doesn’t seem to be checked in. I guess it’s no longer possible for you to do that? Is the 50–10000 Gmm data somewhere?


rjnw
2018-7-10 18:48:05

No, the data files were pretty big so I didn’t check them in.


samth
2018-7-10 18:53:45

ah, ok


samth
2018-7-10 18:53:56

i’m just editing the pdf directly then


carette
2018-7-10 19:14:23

Altogether we seem to have ½ page to go.


ccshan
2018-7-10 19:17:57

9 lines in my local working copy


carette
2018-7-10 19:18:39

Ah, I thought your latest check-in meant you were ‘done’ with section 3 and had moved on to something else.


carette
2018-7-10 19:19:17

Ok, then I won’t be too aggressive with changes to section 1 and 2. But more frequent check-ins might be good.


carette
2018-7-10 19:24:18

I just noticed a FIXME in section 6!


carette
2018-7-10 20:39:45

I’ve now commented it out (the FIXME), so that it has no danger of slipping out in the final version.


samth
2018-7-10 20:47:22

@carette I believe @ccshan needs to fix that FIXME


carette
2018-7-10 20:49:22

Indeed. I just didn’t want to run the risk of it being forgotten.


carette
2018-7-10 20:50:13

I think the paper could go out without the missing introduction paragraph, and it would not be disastrous. With a clear FIXME, it would be.


ccshan
2018-7-10 21:03:06

@rjnw, would you please confirm or deny that in all plots, all Hakaru (LLVM) and JAGS curve markers represent 1 sweep, and all AugurV2 curve markers represent 100 sweeps?


rjnw
2018-7-10 21:24:16

In gmmgibbs both hakaru and jags are every ten, and 1 sweep for nauve bayes and lda


rjnw
2018-7-10 21:25:39

For augur I think it’s every 100, I will verify it when I get back to the hotel.


carette
2018-7-10 22:02:40

Praveen’s name had been misspelled on hotcrp, so I fixed it. And entered my conflicts while I was at it.


rjnw
2018-7-11 01:50:49

lda is every 10, nb every 100


ccshan
2018-7-11 04:11:28

Which system are you talking about?


ccshan
2018-7-11 05:49:52

(Hmm, we can almost fit the PSI plots back in…)