Viacheslav Kukushkin

Results 51 comments of Viacheslav Kukushkin

@sarthakpati @szmazurek I catch the bug with @szmazurek 's model config. The issue is when `ignore_label_validation` is given in the model config, metrics for this specific label is not evaluated,...

😁What a crutch @sarthakpati overrode branch's commits history, making myself as commit author (sorry, @szmazurek ), so failed check should be fixed now. But isn't it strange Szymon's CLA agreement...

Sorry as I am not proficient enough neither in the latest pytorch changes, nor in how GaNDLF uses distributed training. Do we have any tests for multi-gpu training? Cannot find...

Current state: # MLCubes list ![image](https://github.com/user-attachments/assets/676069b3-d84d-4729-ad46-a87af46bf7de) # Datasets list ![image](https://github.com/user-attachments/assets/68bfb2f7-4237-4eb8-af85-dc70d3dccbaf) # Benchmark list ![image](https://github.com/user-attachments/assets/3a321ec3-0880-4a7d-8d9f-4a9ca47c4b23) # MLCube detailed page ![image](https://github.com/user-attachments/assets/54063a1b-7882-41b3-b778-22bdb7bf64e0) # Benchmark detailed page ![image](https://github.com/user-attachments/assets/2f852c15-9b04-4374-9c56-f6eafbea0813) # Dataset detailed page ![image](https://github.com/user-attachments/assets/5988955d-5e60-44be-b8c8-bf1ab0cc6e9d)

@sarthakpati I agree it would be nice to mention logging in extension guide. However, the whole PR is created in the way that we hide all configuration from the developer...

@sarthakpati No, this part was never touched in new API branch. To tell truth I don't get quite well even after reading parent PR what exactly is the issue about?...

Are you talking about just a coverage of this line in `deploy.py`, or also about real check if container built with such an entrypoint would run properly?

Hi @benmalef ! Thanks for the code. The first 4 points look exactly as expected, I personally like the elegancy of the configuration & usage :) Concerning tqdm, your proposal...

concerning tqdm: we can use the following wrapper to handle the redirect context automatically: ```python def custom_tqdm(it: Iterable, *args, **kwargs): with logging_redirect_tqdm(): for x in tqdm(it, *args, **kwargs): yield x...

Hi folks! I was going just to comment how to fix the issue, but it seems it's easier to show the fix on the code itself. https://github.com/benmalef/GaNDLF/pull/6 So, the root...