search

oasys-elettra-kit / OASYS1-Wiser

The implementation of WISER into OASYS
MIT License
0 stars 1 forks source link

ALPHA - SERIOUS #25

Closed capitanevs closed 3 years ago

capitanevs commented 4 years ago

The chain of responsibility stopped working. We have to investigate that AASAP

immagine

General question: which version of LibWiser does Oasys use? Does it fish the info from GIT repository, or is there an intermediate step, where a "push" from GIT to another space is required? [I guess it is the second]

aljosahafner commented 4 years ago

This is the version: https://pypi.org/project/LibWiser/ It has to be updated manually each time when a new release is made (if there are newer requirements necessary). I don't know how to force OasysWiser to update LibWiser automatically as well. In setup.py (as you can see in the repository), such thing is reflected in line 46.

capitanevs commented 4 years ago

Debug notes:

for Michele: path to LibWiser (for oasys) C:\Users\Mike\Miniconda3\Lib\site-packages

How to get to The Opticalelement

in_object_1.wise_beamline.get_propagation_element(-1).get_optical_element().native_optical_element.Name

Intensity

print(in_object_1.wise_beamline.get_propagation_element(-1).get_optical_element().native_optical_element.PropagationData.Intensity)

capitanevs commented 4 years ago

After Meeting we concluded that

1) We need info from Luca on: "what is the best practice to make OASYS automatically update when LibWiser and WofryWiser are updated?"

2) The command print(in_object_1.wise_wavefront.wise_computation_result.Intensity),

called after the first mirror, returns NOT nan for Aljosa (which is correct), but returns [nan---nan] for Michele (which is weird). The N of Samples is correct.

No clues about that for the moment.

3) Apply button and functionality is removed for the moment. => How about the "Reference Field?" apparently it does not make any sense right now. Let's keep it for the moment, but be prepared to hide it in the next future.

capitanevs commented 3 years ago

Random errors, but Aljosa solved it! B-)