Conversation
|
Thanks for the proposal. But this kind of extensions adds much more complexity to the library than I want to maintain. The intention was always to have a minimalistic but flexible library. You could use custom drawables to enable font selection on top of cv-plot or you can of course provide a fork with more functionality. As the library is (optionally) header-only, I dont think that a cmake-based solution is a good idea. We could just replace FONT_HERSHEY_SIMPLEX by FONT_HERSHEY_COMPLEX everywhere instead. But in my experiments simplex just looked much better, at least for small fonts. |
Yep, I tried this firstly but it looks like a bit overhead because in this case I need to replace most part of cv-plot classes.
I proposed compile-time cmake switch due to header-only option, I can propose runtime switch if you prefer this way.
Yep, I agree with you! So I decided to stayed simplex as default font.
If there is no other way of course I'll use this way, but why not to try to propose changes to upstream :) |
|
Thanks again, I hope you understand that I decided to keep it simple. Why not also adding font size, color, alignment, padding, latex,...? That's not where I want to go. |
I added another font for unicode support, not for another visual effects |
Proposal for a small switch for internal Hershey fonts, i.e. it's switch between ascii (default) and unicode fonts for opencv/4.*.
Possible solution for #32.