Fernando Pérez-García
Fernando Pérez-García
Hi, @wahabk. Thanks for the PR. Apologies for the delay. I will take a look at all this as soon as possible.
Closing this as the review was ignored. Thanks anyway for your contribution. Feel free to reopen if needed.
@corcra @Shruthi42 @ozan-oktay @qianchu Could you please share your thoughts?
Superseded by - https://github.com/microsoft/hi-ml/pull/928
Things should work after this is merged: - https://github.com/microsoft/hi-ml/pull/928/
> the docker shm size is not propagated to the child runs, so they get 2Gb (default) and immediately go out of memory (dataloader killed error) FYI: https://github.com/microsoft/hi-ml/pull/880#issuecomment-2108464251
IPython 8 seems to require Python 3.8, but `hi-ml-multimodal` requires 3.7. This can probably be merged after - #770
@dependabot close
Shall we review now or wait until it's not a draft anymore?
Can you please take a look at the failing tests and the merge conflicts?