You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
From Pete: NXsolid_geometry is the work of others. The most substantial contributions in this extremely brief NXDL have the signature of mkuehbach and atomprobe-tc. I can't speak for this NXDL.
NXxpcs was put forward by me. It has languished for many years after a very active development by ALS, APS, & BNL. I doubt any of these facilities will use it as consensus was never reached. I'm not aware of a single champion who should be the designated maintainer. It is a candidate for removal or overhaul by others who wish to define a standard for XPCS. Like other newer techniques, XPCS has multiple paths for analysis and each of these has a set of expectations that are not easily generalized into a common NXDL.
For now, I'd recommend both of these NXDL files remain in contributed. NXxpcs would benefit from a new champion.
When the email bounced I was unable to find a Marie Yao that worked at ORNL. Looking at the commits, hers were the only commits relevant to the meaning of the definition - the rest are standard NIAC housekeeping. An AI search through GitHub repos, publications, etc. found no references to this contributed definition anywhere, so it appears to have been specific to the SNS and never widely (if at all) adopted.
Jul 13: ORNL contacted and asked for a contact at the lab. Heard back from Peterson, Peter <petersonpf@ornl.gov>: they long ago settled on NXevent_data instead of using either of these two (NXsnshisto, NXsnsevent), so he said to remove them.
@lukaspie: NXapm_paraprobe_* definitions describe APM analysis in the open-source paraprobe-toolbox; fairly complete and probably ready for standardization, though the community is small so it's more on NIAC to decide if it wants to support this going forward.
@lukaspie: useful but not really in use right now; implicitly introduces math notation (see Math support in NeXus #711) which has historically been seen as problematic. Suggests keeping as-is for future discussion, or removing - these are the least-used FAIRmat definitions.
@mkuehbach: good as an idea but not completed, and controversial as mentioned.
I contacted Sandor as these are suspected to be FAIRmat contributions. His response:
"Yes, these are our FAIRmat contributions. Carola and Ron worked on them together with Chris (in CC). We have used them in modelling optical spectroscopy setups. They have been used in demo applications, but have not yet been taken up by the community and this is why we have not pushed them further for standardisation.
Note that during the standardisation of NXoptical_spectroscopy some of the base classes used also here (e.g. NXcomponent) have been altered, so the related demo application may also need further rework before these base classes could be used properly by the community again. I guess quite some further effort is needed before these contributions will be standardised. In the meantime, they were expected to stay as contributed, so the community knows about them and can use them or adjust any new contributions similar to these.
Note that I have put Lukas in CC, too, because he is going to coordinate future activities in these domains."
Not the paraprobe-toolbox - this is the config for the separate CompositionSpace tool (A. Saxena et al., github.com/eisenforschung/CompositionSpace) for atom probe data. It was introduced/edited in the same FAIRmat PR #1427 as the NXapm_paraprobe_* batch, but neither lukaspie's nor mkuehbach's feedback explicitly covers it. The same small-APM-community reasoning likely applies, but should be confirmed with them directly rather than assumed.
@lukaspie: contributed by FAIRmat, but far from finished - propose to keep as-is.
@mkuehbach: not meaty/cross-covering enough; would benefit from staying in contributed unless worked on further. Not a fan of a PanAlytical-specific definition - would prefer a single unified XRD definition (spot/1d/2d) worked on harder. Wasn't a priority in FAIRmat phase 1.
@mkuehbach: NXmicrostructure* should remain in contributed generally, but flags NXmicrostructure_ipf/NXmicrostructure_odf specifically as already used as base classes within FAIRmat with thousands of examples.
@lukaspie: FAIRmat has an existing PR to move the SPM app defs/base classes out of contributed: #1632. Worked with several scientific groups to check data compatibility - seems fairly complete. Also preparing a large community SPM dataset for publication on NOMAD.
SPM is a large community though, so there could be more feedback from sub-techniques not currently covered - asking NIAC members what they think of this contribution.
Python script that generated the original table ...
Contributions investigated in this campaign:
🔵 Leave as is
From Pete: NXsolid_geometry is the work of others. The most substantial contributions in this extremely brief NXDL have the signature of mkuehbach and atomprobe-tc. I can't speak for this NXDL.
NXxpcs was put forward by me. It has languished for many years after a very active development by ALS, APS, & BNL. I doubt any of these facilities will use it as consensus was never reached. I'm not aware of a single champion who should be the designated maintainer. It is a candidate for removal or overhaul by others who wish to define a standard for XPCS. Like other newer techniques, XPCS has multiple paths for analysis and each of these has a set of expectations that are not easily generalized into a common NXDL.
For now, I'd recommend both of these NXDL files remain in contributed. NXxpcs would benefit from a new champion.
🔴 Remove
When the email bounced I was unable to find a Marie Yao that worked at ORNL. Looking at the commits, hers were the only commits relevant to the meaning of the definition - the rest are standard NIAC housekeeping. An AI search through GitHub repos, publications, etc. found no references to this contributed definition anywhere, so it appears to have been specific to the SNS and never widely (if at all) adopted.
Jul 13: ORNL contacted and asked for a contact at the lab. Heard back from Peterson, Peter <petersonpf@ornl.gov>: they long ago settled on NXevent_data instead of using either of these two (NXsnshisto, NXsnsevent), so he said to remove them.
🔴 Remove
see NXsnshisto above - same rationale.🟡 Still Waiting
Jul 27: sent email to ISIS group leader Adrian Hillier.
Jul 28: received email from Freddie - he needs to ask around and get back to me.
🟡 Still Waiting
see NXspin_rotator above🟡 Still Waiting
see NXspin_rotator above🟡 Still Waiting
see NXspin_rotator above🟡 Still Waiting
see NXspin_rotator above🟡 Still Waiting
see NXspin_rotator above🟡 Pending
Waiting on response from @keeble.🟢 Promote #1648
🟢 Promote #1648
see NXcsg above (identical feedback covers both).🟢 Promote #1646
@daurer is the primary contact; actively used at DLS I08-1.🟡 Pending
Waiting on response from @PeterC-DLS.🔵 Leave as is
see NXsolid_geometry above🟢 Promote
🟢 Promote
see NXapm_paraprobe_ranger_config above🔵 Leave as is / possible removal
🟡 Pending follow-up
Will write to Markus.🔵 Leave as is
🔵 Leave as is
see NXmicrostructure_feature above🔵 Leave as is / possible removal
see NXdispersion_single_parameter above🔵 Leave as is / possible removal
see NXdispersion_single_parameter above🔵 Leave as is / possible removal
see NXdispersion_single_parameter above🔵 Leave as is / possible removal
see NXdispersion_single_parameter above🔵 Leave as is / possible removal
see NXdispersion_single_parameter aboveContacted
sandor.brockhauser@cbs.mpg.de;lukas.pielsticker@physik.hu-berlin.de; csturm@physik.uni-leipzig.de🔵 Leave as is
I contacted Sandor as these are suspected to be FAIRmat contributions. His response:
"Yes, these are our FAIRmat contributions. Carola and Ron worked on them together with Chris (in CC). We have used them in modelling optical spectroscopy setups. They have been used in demo applications, but have not yet been taken up by the community and this is why we have not pushed them further for standardisation.
Note that during the standardisation of NXoptical_spectroscopy some of the base classes used also here (e.g. NXcomponent) have been altered, so the related demo application may also need further rework before these base classes could be used properly by the community again. I guess quite some further effort is needed before these contributions will be standardised. In the meantime, they were expected to stay as contributed, so the community knows about them and can use them or adjust any new contributions similar to these.
Note that I have put Lukas in CC, too, because he is going to coordinate future activities in these domains."
Contacted
sandor.brockhauser@cbs.mpg.de;lukas.pielsticker@physik.hu-berlin.de; csturm@physik.uni-leipzig.de🔵 Leave as is
see NXoptical_fiber aboveContacted
sandor.brockhauser@cbs.mpg.de;lukas.pielsticker@physik.hu-berlin.de; csturm@physik.uni-leipzig.de🔵 Leave as is
see NXoptical_fiber above🟢 Promote
see NXapm_paraprobe_ranger_config above🟢 Promote
see NXapm_paraprobe_ranger_config above🟡 Needs explicit follow-up
Not the paraprobe-toolbox - this is the config for the separate CompositionSpace tool (A. Saxena et al., github.com/eisenforschung/CompositionSpace) for atom probe data. It was introduced/edited in the same FAIRmat PR #1427 as the NXapm_paraprobe_* batch, but neither lukaspie's nor mkuehbach's feedback explicitly covers it. The same small-APM-community reasoning likely applies, but should be confirmed with them directly rather than assumed.
🟢 Promote
see NXapm_paraprobe_ranger_config above🔵 Leave as is
see NXmicrostructure_feature above🟢 Promote
see NXapm_paraprobe_ranger_config above🟢 Promote
see NXapm_paraprobe_ranger_config above🟡 Needs explicit follow-up
see NXapm_compositionspace_config above🟢 Promote
see NXapm_paraprobe_ranger_config above🟢 Promote
see NXapm_paraprobe_ranger_config above🟢 Promote
see NXapm_paraprobe_ranger_config above🟢 Promote
see NXapm_paraprobe_ranger_config above🟢 Promote
see NXapm_paraprobe_ranger_config above🟢 Promote
see NXapm_paraprobe_ranger_config above🔵 Leave as is
🟢 Promote
see NXapm_paraprobe_ranger_config above🟢 Promote
see NXapm_paraprobe_ranger_config above🟢 Promote
see NXapm_paraprobe_ranger_config above🔵 Leave as is (possible future promotion)
🟢 Promote
see NXapm_paraprobe_ranger_config above🟢 Promote
see NXapm_paraprobe_ranger_config above🟢 Promote
see NXapm_paraprobe_ranger_config above🟢 Promote
see NXapm_paraprobe_ranger_config above🟢 Promote
see NXapm_paraprobe_ranger_config above🔵 Leave as is
see NXmicrostructure_feature above🔵 Leave as is
see NXmicrostructure_feature above🔵 Leave as is
see NXmicrostructure_feature above🟢 Promote
see NXapm_paraprobe_ranger_config above🔵 Leave as is
see NXmicrostructure_feature above🔵 Leave as is (possible future promotion)
see NXmicrostructure_odf above🔵 Leave as is
see NXxrd above🔵 Leave as is
see NXmicrostructure_feature above🟡 Pending NIAC discussion
@lukaspie: FAIRmat has an existing PR to move the SPM app defs/base classes out of contributed: #1632. Worked with several scientific groups to check data compatibility - seems fairly complete. Also preparing a large community SPM dataset for publication on NOMAD.
SPM is a large community though, so there could be more feedback from sub-techniques not currently covered - asking NIAC members what they think of this contribution.
🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🟡 Pending NIAC discussion
see NXstm above🔵 Leave as is
Jul 27: sent email to ISIS group leader Adrian Hillier.
Jul 28: received email from Freddie - he only recently added this definition, so it should stay where it is for the time being.