search

scipp / esspolarization

Polarization data reduction for the European Spallation Source
https://scipp.github.io/esspolarization/
BSD 3-Clause "New" or "Revised" License
0 stars 1 forks source link

added adapted zoom.ipynb for secondrun data to test the workflow #71

Open astellhorn opened 1 month ago

astellhorn commented 1 month ago

Hi Simon, we are at ISIS at the moment and testing the reduction script on new data that I got from the instrument scientist here. Somehow, it does not compile anymore block [9] with the error message: TypeError: Problem setting "LoadMonitors" in Load-v1: No registered converter was able to produce a C++ rvalue of type std::__cxx11::basic_string<char, std::char_traits, std::allocator > from this Python object of type bool

I have added the new data under zoom_reduction_data/Secondrun. Could you please have a look why it cannot read it out anymore? With the data if the "Firstrun" (the data we used before) it still works.

Thanks a lot!

SimonHeybrock commented 1 month ago

I have added the new data under zoom_reduction_data/Secondrun. Could you please have a look why it cannot read it out anymore? With the data if the "Firstrun" (the data we used before) it still works.

You have uploaded a PDF. As large data should anyway not be added in Git, can you share it on NextCloud?

astellhorn commented 1 month ago

Interesting, then it only uploaded the pdf, but not the data (both was in the folder that I have copied in). Is it ok I will send only the used data in slack for you to upload?

SimonHeybrock commented 1 month ago

I presume this is not intended for merging (since this notebook ends up in the public documentation). If it is, please remove the attached large files, and remove comments from debugging that are found in the notebook.

As there are conflicts from other changes on main, it is probably easier to start again from the notebook version in main and just move the necessary changes there.