Skip to content

improved box visualizations with supervision#172

Open
SkalskiP wants to merge 5 commits intoPeterande:masterfrom
SkalskiP:master
Open

improved box visualizations with supervision#172
SkalskiP wants to merge 5 commits intoPeterande:masterfrom
SkalskiP:master

Conversation

@SkalskiP
Copy link

@SkalskiP SkalskiP commented Feb 20, 2025

Hi everyone 👋🏻

Great job! While working with the model, I noticed that the visualizations are quite simple. I decided to open a PR adding visualizations using supervision annotators.

BEFORE:

download (1)

AFTER:

download

@ArgoHA
Copy link
Contributor

ArgoHA commented Feb 22, 2025

Isn't it too much to import whole Supervision just to make visualizations nicer? Can't we just add 1 function and not import Supervision?

@SkalskiP
Copy link
Author

I'm supervision maintainer so I'm not objective, but it's rather lightweight package. OpenCV and Pillow are two heaviest dependencies it have, and you already have them with transformers.

I'd also want to open a PR allowing to run inference in more SDK way. Not via script but import the model.

I plan to create D-FINE fine-tuning tutorial here.

@ArgoHA
Copy link
Contributor

ArgoHA commented Feb 22, 2025

I also work on some features right now (gradient accumulation during training; inference class; other small things). Maybe we can sync to not do the same thing 2 times (about inference)

@developer0hye
Copy link
Contributor

@SkalskiP @ArgoHA I created hf demo for d-fine with supervision!

@Peterande Peterande force-pushed the master branch 2 times, most recently from c3f5fce to 75c6230 Compare April 6, 2025 18:15
@anyaflya
Copy link

Thanks for the implementation! The annotations are clearer this way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants