Skip to content

feat: ingestion configs for EMPIAR-12049; backend validation fixes#633

Merged
daniel-ji merged 8 commits intomainfrom
daniel-ji/empiar-12049
Mar 26, 2026
Merged

feat: ingestion configs for EMPIAR-12049; backend validation fixes#633
daniel-ji merged 8 commits intomainfrom
daniel-ji/empiar-12049

Conversation

@daniel-ji
Copy link
Copy Markdown
Contributor

@daniel-ji daniel-ji commented Mar 26, 2026

  • Adds ingestion config file (10477) for EMPIAR-12049
  • prevents divide by error for tiltseries staging validation test "Frames: tiltseries pixel spacing is an integer multiple of the frame pixel spacing"
  • b"MAP" is now allowed for a MRC header

@daniel-ji daniel-ji changed the title feat: ingestion configs for EMPIAR-12049 feat: ingestion configs for EMPIAR-12049; backend validation fixes Mar 26, 2026
Copy link
Copy Markdown
Contributor

@uermel uermel left a comment

Choose a reason for hiding this comment

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

Minor comment about reconstruction method.

a K2 Summit detector, Volta phase plate, and energy filter. 12 tomograms were collected at centrosomes in
interphase, prophase, metaphase, anaphase, and telophase stages. Comprehensive annotations of centrioles,
microtubules, ribosomes, membranes, and pericentriolar material are provided. Tilt series were aligned using
patch tracking in IMOD, and tomograms were reconstructed using weighted back projection in Warp.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do they actually say weighted back projection in the paper? As far as I was aware Warp is using some kind of fourier inversion method for reconstruction?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sorry, this is incorrect - fixed now. Thank you.

@daniel-ji daniel-ji marked this pull request as ready for review March 26, 2026 01:44
@daniel-ji daniel-ji force-pushed the daniel-ji/empiar-12049 branch from 56e63bd to 231d5dc Compare March 26, 2026 03:56
@daniel-ji daniel-ji merged commit 21f5f82 into main Mar 26, 2026
10 checks passed
@daniel-ji daniel-ji deleted the daniel-ji/empiar-12049 branch March 26, 2026 20:09
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.

2 participants