relion icon indicating copy to clipboard operation
relion copied to clipboard

CTFFIND5 support

Open huwjenkins opened this issue 1 year ago • 6 comments

even for no new feature support CTFFIIND4 -> CTFFIND5 is not a drop in replacement: it needs an extra no in the script that RELION writes.

I'm sure RELION team have plans to support the new version so opening this as a placeholder.

huwjenkins avatar Mar 02 '24 00:03 huwjenkins

Will CTFFIND4 remain available, or will they replace CTFFIND4 with CTFFIND5?

Will the additional "no" break CTFFIND4 compatibility?

biochem-fan avatar Mar 02 '24 01:03 biochem-fan

Will CTFFIND4 remain available, or will they replace CTFFIND4 with CTFFIND5?

I don't know. AFAICT currently the only source of a CTFFIND5 binary is pulling it out of a cisTEM tarball but that executable can be called from RELION with --ctffind_exe

looking at /src/ctffind_runner.cpp The extra question is "Determine samnple thickness? : no"

so yes I think it probably could break compatibility?

huwjenkins avatar Mar 02 '24 01:03 huwjenkins

If adding another "no" works for both versions, I am happy to add. (If it is the last question, CTFFIND4 won't complain)

Otherwise, this will remain low priority as long as people can get CTFFIND4; we don't need/support CTFFIND5's new features. Of course we can parse CTFFIND's header and determine the version, but it will be another layer of complexity & a point of failure. If someone sends us a pull request, I am happy to merge, but I myself won't implement it.

biochem-fan avatar Mar 02 '24 01:03 biochem-fan

Yes sure. It's pretty clear from the RELION GUI that the support is for CTFFIND4 and that the new version is CTFFIND5. If it becomes clear that CTFFIND5 is easily available outside of cisTEM and SPA users want to use it I'll send a PR but I think this would be a new CTFFIND5 tab with quite a bit of extra code because (is_ctffind4) is in quite a few places in src/ctffind_runner.cpp

Presumably RELION-5 tomo will support the new CTFFIND features?

huwjenkins avatar Mar 02 '24 01:03 huwjenkins

Hey,

just wanted to chime in that if you need any help from our side, whether it is about how to call ctffind (which I know is a bit bothersome) or how to grab the binaries of whatever version, feel free to ping me, email me, or open an issue at our repo

Best,

Johannes

jojoelfe avatar Mar 03 '24 01:03 jojoelfe

I added some code to support CtfFind5 in my relion fork:

P.S. currently its still missing GUI

Poko18 avatar Mar 21 '24 21:03 Poko18