Skip to content
Snippets Groups Projects

[JUNGFRAU][CORRECT] Add fragment file and remove precorrection notebook

Merged Karim Ahmed requested to merge feat/fragment_jungfrau into master
1 unresolved thread

Add fragment file and remove precorrection notebook from JUNGFRA correction.

Description

  • remove precorrection notebook.
  • copy write_constants_fragment from !828 (merged)
  • add sequence number for validation against values of reference corrected runs.

How Has This Been Tested?

Relevant Documents (optional)

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist:

Reviewers

@kluyvert @schmidtp

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
618 613 " save_reduced_rois(outp_file, data_corr, mask_corr, local_karabo_da)\n",
619 614 "\n",
620 615 " # Create METDATA datasets\n",
621 " outp_file.create_metadata(like=seq_dc)\n",
616 " outp_file.create_metadata(\n",
617 " like=seq_dc,\n",
618 " sequence=seq_dc.run_metadata()[\"sequenceNumber\"],\n",
  • Author Owner

    I just added this for the tests to succeed. I think I added this every time for the tests to pass but never pushed the change. I should have added this separately but I was lazy to add a line of code in a separate MR.

    Edited by Karim Ahmed
  • Please register or sign in to reply
  • Karim Ahmed added 46 commits

    added 46 commits

    Compare with previous version

  • Karim Ahmed added 4 commits

    added 4 commits

    Compare with previous version

  • Karim Ahmed deleted the feat/jungfrau_calcat_interface branch. This merge request now targets the master branch

    deleted the feat/jungfrau_calcat_interface branch. This merge request now targets the master branch

  • Karim Ahmed added 36 commits

    added 36 commits

    Compare with previous version

  • Karim Ahmed changed milestone to %3.11.0

    changed milestone to %3.11.0

  • Karim Ahmed mentioned in merge request !845 (merged)

    mentioned in merge request !845 (merged)

  • Karim Ahmed added 124 commits

    added 124 commits

    Compare with previous version

  • Author Owner

    Thank you @kluyvert

  • merged

  • Karim Ahmed mentioned in commit 2bba5cc8

    mentioned in commit 2bba5cc8

  • Please register or sign in to reply
    Loading