-
Notifications
You must be signed in to change notification settings - Fork 66
Fairmat 2024: proposal on optical spectroscopy #1425
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
Fairmat 2024: proposal on optical spectroscopy #1425
Conversation
* Add depends for data field * Use `@reference` * use reference attribute in NXmpes and NXmpes_arpes * change text style * fix duplicated in reference attribute --------- Co-authored-by: Lukas Pielsticker <50139597+lukaspie@users.noreply.github.com>
Roll back changes to NXmpes/instrument/pressure_gauge
fix 2theta and use NXobject instead of collection.
Add radius to NXenergydispersion
* Including scattering vector components perpendicular and parallel to the surface of the sample those are mainly used for RSM diffraction technique. --------- Co-authored-by: Florian Dobener <florian.dobener@physik.hu-berlin.de>
Replacement for closed #270
Replacement for #267
…cision of concept image and how this is connected to tilings
Thanks, I removed it |
As discussed in the Telco, 1. and 2. were addressed here. The changes to the RST document were reverted and will instead come with #1428 |
|
@PeterC-DLS is there anything blocking us from merging this PR? As far as I can tell, all comments have been addressed and CI/CD is passing. |
|
Some more community feedback FAIRmat-NFDI#390 on how to better structure instances of NXsource |
@PeterC-DLS just wanted to remind you of this comment. I would like to merge this PR if possible. There is one more comment from @mkuehbach about |
|
@PeterC-DLS, unresolved conversations are a blocker for this PR. I went through all of these and checked for all if the edits on the classes and appdefs addressed the comments. In those cases where they clearly did, i.e. suggestions for reformulations, fixing of typos, and opening of enums, I resolved. A few comments remain that should be addressed on the next telco @phyy-nx so that we can update this branch with the latest state of main and merge. There was a suggestion for less than a handful of individual fixes that should be addressed with a small follow-up proposal, one that will be substantially shorter than this PR. |
mkuehbach
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
|
Thanks @PeterC-DLS @phyy-nx @mkuehbach! Merging now. |
The aim of this PR is to add a more detailed description of optical spectroscopy experiments, with a specific focus on Raman spectroscopy and ellipsometry.
The starting point are the
NXellipsometryandNXoptapplication definitions that have been in the contributed definitions for > 1.5 years now.NXellipsometrywas a specialization ofNXopt, which from the start was intended to extend possibly in the future several other optical spectroscopy methods, such as photoluminescence, transmission or reflection spectroscopy, as there are various commonalities between those techniques.Now,
NXopthas been renamed toNXoptical_spectroscopyand has been enhanced quite a bit to be more inclusive for all kinds of optical spectroscopy experiments.NXellipsometryis now inheriting fromNXoptical_spectroscopy. In addition,NXramanwas added, which is also a specialization ofNXoptical_spectroscopyand meant to describe Raman spectroscopy data.Summary of changes to
NXoptical_spectroscopy(previouslyNXopt):start_time,end_time,experiment_identifier,experiment_description,experiment_type,experiment_sub_type)NXcoordinate_system(_set). Will enable description of sample, laboratory, beam, sample stage reference frames and more. This is based on Fairmat 2024: use NXcoordinate_system together with NXtransformations #1415.NXoptical_spectroscopy/NXinstrument):device_information(NXfabrication),software_TYPE(NXprogram)angle_reference_frame- to describe either beam or sample-centered reference framesNXbeam, with respect to used terms and properties within optical spectroscopy setupsNXsampleinNXoptical_spectroscopy, to be more in line with theNXsamplebase class. Also a sample history and sample environment and other sample metadataTo describe optical spectroscopy experiments, several base classes are used (these are either completely new or have been in contributed before:
NXlens_opt: description of a lens or objective in an optical setupNXwaveplate: half or quater wave plateNXbeam_device:@device. The fieldprevious_deviceclearly defines a list of (possibly multiple) devices, from which a photon beam came, as it arrived at@device.NXtransformations).NXbeam_transfer_matrix_table, properties of the beam along all beam paths can be calculated.NXbeam_transfer_matrix_tabledefines specific concepts and a large matrix with submatricies, for the respective models to describe the parameters of interest.Due to the large extensions and changes in
NXoptical_spectroscopy, the respectiveNXramandefinition is quite short. It contains some general experiment metadata and a description of the scattering configuration.