WongKinYiu / YOLO

An MIT rewrite of YOLOv9
MIT License
292 stars 19 forks source link

Error training command of README.md #24

Closed chilin0525 closed 3 weeks ago

chilin0525 commented 1 month ago

Hi,

I found the example command of README.md is not correct and got following error when execute to train model.Look like you change train confg (b5fa3f1) but forget to update example command of README.

Reproduce and error:

$ python lazy.py task=train task.batch_size=8 model=v9-c
Could not override 'task.batch_size'.
To append to your config use +task.batch_size=8
Key 'batch_size' is not in struct
    full_key: task.batch_size
    object_type=dict

Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.

And I will create a PR to fix it.

henrytsui000 commented 1 month ago

Hi,

The README in the next version has been updated, but because CI will be re-run every time I push the main branch, I will update it together later~

$python yolo/lazy.py task.data.batch_size=8

best regards, Hao-Tang Tsui

chilin0525 commented 1 month ago

Got it.

chilin0525 commented 1 month ago

BTW, I think to create issue template just like PR template will help user to search and manage issue better. 😃

henrytsui000 commented 3 weeks ago

Hi,

I have added the templates for issues in commits bce644c0306518260774dc9c93220c44c57b7494 and fab25903d9a5be5e3ab8450ea12bd807a306f509. Thank you for your suggestion!

Best regards, Hao-Tang Tsui