Skip to content

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

Open
astellhorn wants to merge 2 commits into
mainfrom
Test_reduction_seconddata
Open

added adapted zoom.ipynb for secondrun data to test the workflow#71
astellhorn wants to merge 2 commits into
mainfrom
Test_reduction_seconddata

Conversation

@astellhorn

Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Member

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

Copy link
Copy Markdown
Collaborator Author

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

SimonHeybrock commented Aug 9, 2024

Copy link
Copy Markdown
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants