Becksteinlab / scipy_proceedings

Tools used to generate the SciPy conference proceedings
Other
0 stars 2 forks source link

Double check the results for scheduler plugin #14

Closed mkhoshle closed 7 years ago

mkhoshle commented 7 years ago

@iparask Can you please double check the results from scheduler throughput (Based on this comment: I am somewhat surprised to see the distributed scheduler manage 40,000 tasks per second)? Also, we need figures at final size (~2"x 2"), with font size ~10pt so that when we make panels text does not appear to be small (Look at #9).

iparask commented 7 years ago

Do you want each figure 2" by 2" or the three figure panel?

mkhoshle commented 7 years ago

We want a three figure panel. But we need to make sure that the font will be large enough. In our previous figures font was not large enough. You can make a panel of three 2" by 2" figures.

iparask commented 7 years ago

Yes I can.

iparask commented 7 years ago

Hello, I committed a figure in the panel folder named daskThroughputPanel.pdf. Can you please check and tell me if it is according to the style?

mkhoshle commented 7 years ago

For some reason when I add the figure to paper it does not compile. When I scale it to 50% it works but still does not look good. With 10% it compiles but it is too small. Did you create this using Inkscape? Also you forgot to add A, B, C for each sub-figure!

iparask commented 7 years ago

It does not compile? I created using matplotlib. I will add A,B and C and use Inkscape to create the panel. Let me update it

iparask commented 7 years ago

Any better? It actually does not compile with a 40% scale. I am trying to see what is the problem.

iparask commented 7 years ago

Hello Mahzad,

please check this notebook and especially the two figures. Tell me how you want to proceed.

Giannis

mkhoshle commented 7 years ago

Hi, Thank you! Because the page numbers are already more than 8 (9 with references). I suggest you include the data provided with scheduler plugin only. For details you can point out to the additional data and reference the notebook or wiki in the paper. Please make sure you change xlabel, ylabel, xtick and ytick labelsize. Because the ones you added previously were still small and we had to modify it in Inkscape.

iparask commented 7 years ago

Perfect! What fontsize did you use?

mkhoshle commented 7 years ago

I used 18!

iparask commented 7 years ago

I made a merge error and I reverted it. A pdf file with the new panel is in the repo. Please check if the revert is correct and I will commit the correct changes.

I am sorry for the mistake

orbeckst commented 7 years ago

Looks good!

orbeckst commented 7 years ago

I used your PDF in ec946ff, please check that this looks ok (pdf attached). paper.pdf

orbeckst commented 7 years ago

@iparask how many repeats did you run for the scheduler experiments on stampede?

iparask commented 7 years ago

@orbeckst, I did 10 repeats per configuration. It looks great!

orbeckst commented 7 years ago

Thanks, added to paper.

-- Oliver Beckstein email: orbeckst@gmail.com

Am Jul 6, 2017 um 08:57 schrieb Ioannis Paraskevakos notifications@github.com:

@orbeckst 10 per configuration

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.