puhep / pudb

Purdue CMS FPix Database
0 stars 0 forks source link

FNAL IV results @17C #119

Closed drberry85 closed 8 years ago

drberry85 commented 8 years ago

Hey Greg,

We need to be able to upload the results of the 17C IV scan ran here at FNAL to the Purdue DB. Also, on the IV by position page we need to have a total plot for the 17C and -20C IV results. The latter is already there, the plot needs to say it's at -20C so we can differentiate it from the 17C results.

Doug

gneeser commented 8 years ago

Doug,

I've made the DB able to handle both the -20C and +17C results. To upload the -20C results, the tag should have the "fnal" as before. For the +17C scan, it should have a of "fnal_17c"

The plots have also been updated accordingly.

This required a lot of little changes, so it's possible I missed something. Let me know if you catch any problems.

-Greg

drberry85 commented 8 years ago

John,

I have no idea how to test if this is working. Would it be simple-ish to make the uploadTest.py script handle 17C and -20C IV results? In theory the 17C IV results should be done before the -20, but sometimes they'll be done at UNL and we won't have access to the data. We may have to make two seperate uploads. One for the 17C results and one for the -20C.

Thoughts? Doug

On Fri, Jan 8, 2016 at 1:39 PM, gneeser notifications@github.com wrote:

Doug,

I've made the DB able to handle both the -20C and +17C results. To upload the -20C results, the tag should have the "fnal" as before. For the +17C scan, it should have a of "fnal_17c"

The plots have also been updated accordingly.

This required a lot of little changes, so it's possible I missed something. Let me know if you catch any problems.

-Greg

— Reply to this email directly or view it on GitHub https://github.com/puhep/pudb/issues/119#issuecomment-170103116.

jstupak commented 8 years ago

The latter option you mentioned is definitely the way to go. We have the shifters run uploadTest for 17 and -20 tests anyway (to move the elCom output to the production area). So I'll just add a command line option that controls whether all the usual results get zipped, or just the IV. Then modify the twiki to instruct shifters to upload the zip file after each test.

gneeser commented 8 years ago

I did run a test using a simple zip that I made. You can see the result on the part summary pages for P-A-test3-test3 and P-A-test2-test2, some of the database testing modules I made a while back. If you turn show preproduction on, you can also see them on the +17C plot on the IV by Position page. They have the same data, so they'll overlap, but they're both there.

jstupak commented 8 years ago

It looks to me like we can close this now

gneeser commented 8 years ago

Will do then. Let me know if you encounter any problems with it later.