Skip to content
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 atom probe microscopy (APM) #1422

Open
wants to merge 1,156 commits into
base: main
Choose a base branch
from

Conversation

lukaspie
Copy link
Contributor

@lukaspie lukaspie commented Sep 24, 2024

Refactored version for NXapm including feedback from the atom probe microscopy/tomography community
Refactoring of NXimage, NXspectrum_set base classes and examples for their usage
More complete representation of instrument components
Examples for documenting method-specific results with some contextualization for processing APM data

This PR depends heavily on others that make changes to existing base classes or introduce base classes that are to be used outside of APM as well. Thus, it will be much smaller once the discussion on these PRs is done and this particular branch here has been rebased:
#1407
#1408
#1410
#1412
#1413
#1414
#1415
#1419
#1420 
#1486

There are however several new base classes that are currently only used within NXapm that are introduced here:

  • NXapm_charge_state_analysis
  • NXapm_hit_finding
  • NXapm_msr
  • NXapm_ranging
  • NXapm_reconstruction
  • NXapm_sim
  • NXapm_volt_and_bowl
  • NXchemical_composition
  • NXcs_filter_boolean_mask
  • NXevent_data_apm
  • NXevent_data_apm_set
  • NXpulser_apm
  • NXreflectron

For a full list of dependant changes, see #1466

atomprobe-tc and others added 30 commits July 4, 2024 15:36
…or_circuits

Cleaned the too many base classes for circuits and cleaned base classes for describing a computer
Make geometries and sourceTYPE recommended in NXxps
…NXms with NXmicrostructure to avoid confusing the symbol convention with mass spectrometry as the abbreviation NXms may suggest
…con and eventually make this NXmicrostructure
@lukaspie lukaspie force-pushed the fairmat-2024-apm branch 3 times, most recently from 690f2d2 to 0f7ac38 Compare September 24, 2024 08:42
@lukaspie lukaspie marked this pull request as ready for review September 24, 2024 09:12
@sanbrock sanbrock mentioned this pull request Sep 29, 2024
@lukaspie lukaspie linked an issue Sep 29, 2024 that may be closed by this pull request
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.

NXapm
7 participants