Fix: Modified and corrected the pnccd dark notebook.

@ahmedk Hi Karim,

I have corrected pnCCD dark notebook so that it has the correct blocksize (verified by Steffen), correct common mode definition and it sends the correct offsetmap into the database. I also added a bunch of diagnostic tables and graphs. Misha knows about these issues but his name is not listed under the "Assignees" so I had to add you instead. Sorry.

Cheers, Kiana

Looks like temperature and integration time are still hard-coded. Is there a reason for this?

Hi Misha. Yes everything even bias voltage is hard coded for pnCCD. I don't know why this is the case. Perhaps no one had the time to have karabo devices that would read these values automatically. But I have spoken to Steffen and Controls people and they will start working on it as of yesterday. Once they make Karabo devices that can read these values and incorporate the values into DAQ, I will modify the notebook so that these parameters are read automatically. Cheers, Kiana

mentioned in merge request !239 (merged)

added 1 commit

• df30b0b7 - Push to DB to False

Compare with previous version

Notebooks runs and give some reasonable (for the first glance) result (see report PNCCD_dark.pdf )

Without suggested changes offset map is completely meaningless. Therefore I would suggest to consider this MR as a fix. Questions, if further modifications are required may be clarified later.

I modified a flag push to DB to False.

Code refactoring, I guess, was performed bu automatic pep8 formatter.

changed title from modified and corrected the pnccd dark notebook. to Fix: Modified and corrected the pnccd dark notebook.

changed the description

Thanks Misha.

time, h5py, datetime and minuit are not used

I will take these out. Thanks a lot.

@setoodeh

Hi Kiana, I already fixed them. Thank you.

beside the few comments, LGTM.

added 1 commit

• aa414f51 - Push to DB to False

Compare with previous version

@setoodeh

I have a question concerning CM correction algorithm. Do you know the origination of peak at 0?

No, Misha. I talked to you on the phone about this issue. Thank you very much and thank you Karim also for your time and help.

Best, Kiana

added 24 commits

• aa414f51...795bcece - 23 commits from branch master
• 498cc754 - modified and corrected the pnccd dark notebook.

Compare with previous version

added 1 commit

• 1fb20397 - modified and corrected the pnccd dark notebook.

Compare with previous version

merged

mentioned in commit c4d0daf8