Burhan

Results 173 comments of Burhan

@Jpfonseca and @ambitious-octopus a few notes in general on these additions/changes. Expand the section at the bottom of this comment to see a preview of the changes. - Let's make...

Also, I can't speak to the functionality of this guide and have assumed it to be correct.

@Jpfonseca you never have to ask permission to open a PR 👍 however I will only be able to provide support from the docs formatting perspective, as I have only...

## Local Slow test results Checked slow tests that were failing in the CI after making the proposed change. test_export_openvino_matrix[obb-False-True-False-2] ```sh pytest -l --tb=short tests/test_exports.py::test_export_openvino_matrix[obb-False-True-False-2] --slow ================================= test session starts...

@lakshanthad I was actually thinking it could be good to keep both results (add a tab for the new version). What do you think tho? Would it be worth showing...

Checking back to see if any maintainers are available to review 🙌

@mohammadjadallah thanks for opening a PR! Just a quick note, instead of importing `os` you can refactor your PR to use the `pathlib.Path` object which is already used. This is...

- [ ] refine head (optimize) & R&D @Laughing-q - [ ] once matured, merge into `main` - [ ] test HUB custom metrics / logging? @ambitious-octopus + @sergiuwaxmann

@caopulan the comment was removed by me, I think it was an accidental post (perhaps to the wrong PR). I know he's super busy with all kinds of things lately,...

@mayorblock have you investigated the idea of using the `ckpt_path` attribute instead? This would mean that a new argument wouldn't be required. Currently the `model.model.pt_path` attribute is used, but it...