[GenABEL-dev] [Genabel-commits] r658 - in pkg/VariABEL: R src

Yurii Aulchenko yurii.aulchenko at gmail.com
Fri Feb 18 17:02:25 CET 2011


A minor comment:

> -output_column_names <- c(output_column_names, "main_snp_effect")
> -output_column_names <- c(output_column_names, "main_snp_effect_se")
> +output_column_names <- c(output_column_names, "snp_eff_dispertion")
> +output_column_names <- c(output_column_names, "snp_se_dispertion")

Not sure it is great idea to call that 'dispersion' in terms of
readability of the code. In English, the first connotation is optic
dispersion. Russian (and I think German) "dispersion" is variance --
is the thing you call dispersion a variance or square root of that?

Yurii


More information about the genabel-devel mailing list