puhep / pudb

Purdue CMS FPix Database
0 stars 0 forks source link

Fractional information #178

Closed merkelp closed 8 years ago

merkelp commented 8 years ago

Hello,

could we change this plot http://www.physics.purdue.edu/cmsfpix/Submission_p/graphing/testbybatch_grapher.php such that all bars are the same total size (=100%) and change the blue/red/yellow/green) parts to fractions of the total for that batch? I think it would be more informative when comparing batches with each other. They are always more or less the same size anyway, and when they are not we know it.

Thanks, Petra

drberry85 commented 8 years ago

Petra,

I would vote that we just have a second plot which has the percent yield plotted as a function of batch and not the absolute numbers.

Doug

On Wed, May 4, 2016 at 7:21 AM, merkelp notifications@github.com wrote:

Hello,

could we change this plot http://www.physics.purdue.edu/cmsfpix/Submission_p/graphing/testbybatch_grapher.php such that all bars are the same total size (=100%) and change the blue/red/yellow/green) parts to fractions of the total for that batch? I think it would be more informative when comparing batches with each other. They are always more or less the same size anyway, and when they are not we know it.

Thanks, Petra

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/puhep/pudb/issues/178

gneeser commented 8 years ago

Petra, How does this look?

http://www.physics.purdue.edu/cmsfpix/Submission_t/graphing/testbybatch_fractional_grapher.php

Are there any changes you would like me to make? I can try experimenting with making adjustments for readability as well if you'd like.

-Greg

merkelp commented 8 years ago

Hi Greg,

looks great, thanks!

Petra

On 5/27/16 2:21 PM, gneeser wrote:

Petra, How does this look?

http://www.physics.purdue.edu/cmsfpix/Submission_t/graphing/testbybatch_fractional_grapher.php

Are there any changes you would like me to make? I can try experimenting with making adjustments for readability as well if you'd like.

-Greg

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/puhep/pudb/issues/178#issuecomment-222232325, or mute the thread https://github.com/notifications/unsubscribe/AFCNQMZ7qYoRyJ5_wwDafEck0vOh1I0Yks5qF0RDgaJpZM4IXFtO.

gneeser commented 8 years ago

My pleasure!

drberry85 commented 8 years ago

Greg,

I'm being nit-picky, but could you change the plot maximum to 120 so the legend doesn't coverup the data?

Doug

On Fri, May 27, 2016 at 2:33 PM, gneeser notifications@github.com wrote:

Closed #178 https://github.com/puhep/pudb/issues/178.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/puhep/pudb/issues/178#event-674866718, or mute the thread https://github.com/notifications/unsubscribe/AElNLFGLfqZKkX_wOuexpmeZDtPdD0vtks5qF0chgaJpZM4IXFtO .