8000 update-north-markus by mkuehbach · Pull Request #141 · FAIRmat-NFDI/pynxtools · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

update-north-markus #141

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

Merged
merged 20 commits into from
Aug 31, 2023
Merged

update-north-markus #141

merged 20 commits into from
Aug 31, 2023

Conversation

mkuehbach
Copy link
Collaborator

Merge features developed within June (partially during sprint 15) for creating deployable apm-specific OASES in Leoben and Sydney to master so that these can appear in a new nomad oasis deployment

markus.kuehbach and others added 13 commits June 2, 2023 12:23
…m example in update-north-markus nomad FAIR
…loyment for atom probe groups like Leoben and others
…e and functionality, apm: started refactoring loading of ELN data to use em_nion concept of mapping tables instead of custom, hardcoded many section-specific functions, apm: Added support for decoupling metadata from ELN and a deployment-specific configuration file, whereby users of specific OASIS do no longer need to expose all metadata in the ELN template but can instead use the deployment-specific configuration file to inject e.g. not changing metadata like versions, or specific details about instruments directly from the configuration yaml file, thereby the amount of retyping in ELNs can be reduced, an additional potential benefit of this design is that ELNs may appear now less complex for end users which was a point raised by Lorenz Romaner from Leoben
…pport for dual-laser setup of LEAP6000 instrument, initial code styling, linting round, next steps i) implement reading of composition from ELN, ii) test
…and tested the apm dataconverter, modified standalone ipynb, apm reader is fully functional again, next step, check that em_nion reader remains functional after the refactoring of the ELN utility functions and use the same mapping refactoring for em_spctrscpy and em_om reader
…us-north 5ff040d to integrated latest changes of Leoben oasis for atom probe and start the summer break implementation to integrate community feedback for EBSD, EDS, user feedback from Annette Trunschkes group at the FHI"
…s are also affected as many fields in base classes were removed in during the code camp, please inspect @domna, updated dev-requirements and, fixed linting
… Ref_nexus_test, this commit might therefore need to be reverted
@coveralls
Copy link
coveralls commented Jul 20, 2023

Pull Request Test Coverage Report for Build 6039864127

  • 64 of 280 (22.86%) changed or added relevant lines in 14 files are covered.
  • 3 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.1%) to 51.075%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pynxtools/dataconverter/readers/em_om/utils/image_transform.py 0 3 0.0%
pynxtools/dataconverter/readers/apm/reader.py 7 11 63.64%
pynxtools/dataconverter/readers/apm/utils/apm_load_deployment_specifics.py 8 24 33.33%
pynxtools/dataconverter/readers/apm/utils/apm_define_io_cases.py 2 22 9.09%
pynxtools/dataconverter/readers/apm/utils/apm_load_generic_eln.py 14 99 14.14%
pynxtools/dataconverter/readers/apm/utils/apm_parse_composition_table.py 11 99 11.11%
Files with Coverage Reduction New Missed Lines %
pynxtools/dataconverter/readers/mpes/reader.py 1 83.33%
pynxtools/dataconverter/readers/em_om/utils/image_transform.py 2 15.0%
Totals Coverage Status
Change from base Build 5962795360: 0.1%
Covered Lines: 5987
Relevant Lines: 11722

💛 - Coveralls

@domna domna requested review from domna and removed request for domna August 16, 2023 09:21
@sanbrock sanbrock self-requested a review August 16, 2023 12:58
Copy link
Collaborator
@sanbrock sanbrock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but please answer the questions

@mkuehbach mkuehbach merged commit de1cc04 into master Aug 31, 2023
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.

3 participants
0