Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Yolo v8 #16

Open
Tracked by #29
jafioti opened this issue Jan 21, 2024 · 6 comments
Open
Tracked by #29

Yolo v8 #16

jafioti opened this issue Jan 21, 2024 · 6 comments
Labels
enhancement New feature or request

Comments

@jafioti
Copy link
Owner

jafioti commented Jan 21, 2024

No description provided.

@AdrianEddy
Copy link

YOLOv9 is out already, might be a better candidate: https://github.com/WongKinYiu/yolov9/

@jafioti jafioti added the enhancement New feature or request label Mar 1, 2024
@jafioti jafioti mentioned this issue Mar 2, 2024
11 tasks
@AdrianEddy
Copy link

YOLOv10 https://github.com/THU-MIG/yolov10

@jafioti
Copy link
Owner Author

jafioti commented May 27, 2024

Just finished whisper which involved some work on the conv layers. Should be pretty straightforward to do yolo now. With the current rate of change I expect it'll be outdated in 2 weeks

@xnorpx
Copy link
Contributor

xnorpx commented Aug 1, 2024

Trying out the Yolo8 example (to compare speed with Candle) and see if I should switch.

Running into:

error: couldn't read `src\../roboto-mono-stripped.ttf`: The system cannot find the file specified. (os error 2)
   --> src/main.rs:190:26
    |
190 |     let font = Vec::from(include_bytes!("../roboto-mono-stripped.ttf") as &[u8]);

Copy in the fonts it compiles (with some warnings)

Then when running there is a dim missmatch

C:\git\luminal\examples\yolo_v8\target\release>yolo_v8.exe
thread 'main' panicked at C:\git\luminal\src\hl_ops\binary.rs:65:9:
assertion `left == right` failed: Dims must match to multiply tensors.
  left: [16, 27]
 right: [16, 1, 1, 1]
stack backtrace:
   0: std::panicking::begin_panic_handler
             at /rustc/051478957371ee0084a7c0913941d2a8c4757bb9/library\std\src\panicking.rs:652
   1: core::panicking::panic_fmt
             at /rustc/051478957371ee0084a7c0913941d2a8c4757bb9/library\core\src\panicking.rs:72
   2: core::panicking::assert_failed_inner
             at /rustc/051478957371ee0084a7c0913941d2a8c4757bb9/library\core\src\panicking.rs:403
   3: core::panicking::assert_failed
   4: luminal::hl_ops::binary::<impl core::ops::arith::Mul for luminal::graph_tensor::GraphTensor>::mul
   5: yolo_v8::model::ConvBlock::new
   6: yolo_v8::model::Yolo::new
   7: petgraph::graph_impl::stable_graph::StableGraph<N,E,Ty,Ix>::add_node
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

@xnorpx
Copy link
Contributor

xnorpx commented Aug 1, 2024

Btw there is an ongoing Yolo9MIT rewrite here: https://github.com/WongKinYiu/YOLO

@jafioti
Copy link
Owner Author

jafioti commented Aug 2, 2024

Yeah sorry been backed up, the yolo example isn't complete yet. should be done very soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants