I have renamed a few bits and added information to the documentation. Abi, you’ve done an absolutely amazing job in annotating the script @abigailterkuile
Are planning to change the script that it can process several sum stats or shall it just run one file after the other? This is important because at few places we still discussing the option of adding loops
Are we using the median of the effect size to double check it? @johanzvrskovec
Are we currently checking if SE is ln(SE) or SE? @johanzvrskovec
The OR filter has the following message: +++JZ: TODO - adapt to all kinds of effect types; do we want to extend the OR filter to other effect sizes? @johanzvrskovec
N filter +++ATK: do we want to remove SNPs that have a large N as set here? Or just SNPs with small N? +++CH: Why are we doing this? What is the rationale? Can we add more explanation?
keep_indels +++CH: There is something wrong with this chunk. It doesn't do what it is supposed to do @johanzvrskovec
Z_check instead of Z? +++CH: Have we generated Z_check somewhere before? @opain
For MAF it says we need to import the reference file? @opain
I was wondering if I was working on the wrong script or if we had read in the reference panel for the harmonisation?! @opain