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

Feature Request: Incorporate tilt angle offset to alignment import #29

Open
bbarad opened this issue Sep 16, 2021 · 0 comments
Open

Feature Request: Incorporate tilt angle offset to alignment import #29

bbarad opened this issue Sep 16, 2021 · 0 comments

Comments

@bbarad
Copy link

bbarad commented Sep 16, 2021

Reasoning: I am doing warp on tilt series collected on lamellae collected dose symmetrically with a pretilt of -11º (due to the milling angle, -11 is the "flat" path through the sample). Because of the relatively large pretilt, I do all my processing in imod with a +11º tilt angle offset in order to get better tilt series alignments - the effect is surprisingly significant for getting better quality output. However, when I import the aligned tilt series into warp, the tilt angle offset is lost and the resulting tomogram quality is relatively poor. Beyond the quality issue, I need a much thicker tomogram to be able to fit the lamella, and that empty space seems to make denoising perform much worse. I expect this to be a consistent issue for users doing tomography on FIB-processed samples, so I suspect its more than just a selfish feature request!

As far as I can tell, the math for this happens here: 

SortedAngle[i].TiltAngle += float.Parse(Parts[3], CultureInfo.InvariantCulture);
. It reads the deltlt column (which does not have the tilt angle offset incorporated), and applies it to the original angles. I've been able to get flat and high quality tomograms out of warp by writing a script to add 11 to each deltlt angle, but that feels like a very hacky solution that could be done better. 

Could that line instead read the tilt angle (instead of deltlt) and just write it in instead of adding on? The tilt angle appears to incorporate the tilt angle offset, but has precision only to the 10s digit. Alternatively, perhaps the Angle offset could be read from one of the numerous logs that it is written to (just not taSolutions!).

I'd be happy to put together a pull request if you think that one of these strategies is appropriate.

@bbarad bbarad changed the title Feature Request Incorporate tilt angle offset to alignment import Feature Request: Incorporate tilt angle offset to alignment import Sep 16, 2021
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

No branches or pull requests

1 participant