Alexander Grund

Results 844 comments of Alexander Grund

I found a small reproducer: ``` import jax import gc for i in range(1000): print(jax.jit(lambda x, y: x * y)(1,i)) jax.clear_backends() jax.clear_caches() gc.collect() ``` On my machine (208 cores) this...

> IIUC from your latest comment, it seems like something in our C++ / extension code is breaking reference counting. Does that sound right? Not necessarily. I experimented with breakpoints...

> If you exclude `clear_backends_test`, is there still a problem running the test suite? Yes it still fails. This particular test was just one I investigated in more detail after...

It looks like `flatc -c -o horovod/common/wire horovod/common/wire/message.fbs` does indeed update that one file correctly. However there is also `horovod/tensorflow/custom_call_config_generated.h` requiring an update but autogenerating that is a bit harder....

Can you split off the MDAnalysis EC into an extra PR? It is also needed in e.g. #19398

Just for reference: These are the regexes I use: ``` # Lines marked with LCov or coverity exclusion comments '.*// LCOV_EXCL_LINE' '.*// coverity\[dead_error_line\]' # Lines containing only braces '\s*[{}]*\s*' #...

> Singularity flags such as `pem-path` are also linked to environment variables, and there are valid workflows where someone would want to avoid specifying the pem-path each time... but also...

> Could you perhaps write out, long form, how you envisage this working? E.g. if I do, or do not, set the `SINGULARITY_ENCRYPTION_PEM_PATH` environment variable... what flags are necessary /...

Thanks for you analysis > This would be a behaviour change for the `--pem-path` case, and it does seem relatively _subtle_. I worry that it won't be obvious that supplying...

> I might have misunderstood your question, here, but isn't [this](https://docs.sylabs.io/guides/latest/user-guide/signNverify.html#signing-your-own-containers) the mechanism in question...? Indeed this is what I was looking for. Especially the [`ecl.toml`](https://docs.sylabs.io/guides/3.11/admin-guide/configfiles.html#ecl-toml) looks useful in allowing...