Leandro Hermida

I'm getting the same issue on Snakemake 8.16 with the slurm executor. I couldn't understand why it didn't have enough memory for a job. Seems to affect jobs that are...

> I am bumping this issue since it is still present and is rather severe. It still exists as of 8.16 and I believe the reason some couldn't reproduce the...

Just throwing out an observation, it seems like `mem_mib` always gets set to 954 no matter what `mem_mb` is. 954 is the default setting I think so it's not getting...

For others coming here, the workaround is to not use human readable `mem` values in your config, if you specify `mem` as megabyte numbers then `mem_mib` gets set properly

> I don't know why this is so difficult to fix. I'll take a look at the relevant code. It seems like when human readable `mem` is specified that the...

> OK, small update: > > The memory parsing seems to work technically correctly, but in an unexpected manner. `humanfriendly.parse_size()` has an option to always assume binary that should _probably_...

> @hermidalc, would you be willing to write a unit test if I fix the code? Yes I can write it, I will use https://github.com/snakemake/snakemake/tree/main/tests/test_inferred_resources as a template. Bear with...

> here is my branch: > > https://github.com/BEFH/snakemake/tree/mem_disk_fix > > You could maybe do a PR to that or I can add you as a contributor, @hermidalc I forked your...

> > here is my branch: > > https://github.com/BEFH/snakemake/tree/mem_disk_fix > > You could maybe do a PR to that or I can add you as a contributor, @hermidalc > >...

> > Hey did this effect `disk_mib` in addition to `mem_mib`? > > Everything that effects mem_mb or mem_mib effects `disk_mb` or `disk_mib`. The same functions are at play. >...