Burhan
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...