Update the mixer.sif container but shows mixer 1.2 not 1.3

comorment / mixer

Singularity wrapper for https://github.com/precimed/mixer

GNU General Public License v3.0

5 stars 4 forks source link

Update the mixer.sif container but shows mixer 1.2 not 1.3 #29

Open showers14 opened 3 months ago

showers14 commented 3 months ago

i Updated the mixer.sif container by singularity pull docker://ghcr.io/comorment/mixer:latest

but at the time of operation, display mixer1.2 not 1.3

INFO: Environment variable SINGULARITY_BIND is set, but APPTAINER_BIND is preferred INFO:root:init(lib_name=/tools/mixer/src/build/lib/libbgmg.so, context_id=0) INFO:root:init_log(PD.fit.rep10.log) INFO:root:log_message(***

mixer.py: Univariate and Bivariate Causal Mixture for GWAS
Version 1.2.0
(c) 2016-2020 Oleksandr Frei, Alexey A. Shadrin, Dominic Holland
Norwegian Centre for Mental Disorders Research / University of Oslo
Center for Multimodal Imaging and Genetics / UCSD
GNU General Public License v3

i don not know how to deal with it,

may i get u help?

thanks

espenhgn commented 3 months ago

Hi @showers14, The version number reported corresponds to that of the current master branch of the MiXeR source code repository (https://github.com/precimed/mixer/blob/f56a44bde5bd3f8bc0d143d89d1b629035b2d675/precimed/mixer/cli.py#L49), not the MiXeR container builds provided via this repository. Sorry for the confusion! These repositories are presently on different versioning tracks.

showers14 commented 3 months ago

I have completed my analysis using mixer.sif, but the test.power.svg graphic is not displaying correctly (it appears to be overlaid by something else on the layers).

If I use python3.sif (which includes mixer) instead of the current mixer.sif, will the results be consistent?

Thank you for your assistance.

BD_and_PD test power

espenhgn commented 3 months ago

Hi, I opened a separate issue (#30) for the missing power curve. I've seen it, but have not investigated further. It appears that power curves are not fitted by default: https://github.com/precimed/mixer/blob/f56a44bde5bd3f8bc0d143d89d1b629035b2d675/precimed/mixer/cli.py#L205

It may also be due to some updates in matplotlib version in these container builds or similar that would better be fixed in the MiXeR codes.

The python3.sif may have contained MiXeR in some earlier iteration, but would now only include the dependencies (numpy, etc.). You can try, but this would require some changes in syntax etc. If you have the capacity, you can otherwise roll back to a previous version of the mixer.sif file (https://github.com/comorment/mixer/commits/main/singularity/mixer.sif) that is still present in the git LFS history, i.e., this one: https://github.com/comorment/mixer/blob/1cd5af7c4653c90e80d5f5bfbc799c226ac59b52/singularity/mixer.sif