Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

astellhorn
Copy link
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
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
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
Copy link
Member

SimonHeybrock commented Aug 9, 2024

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