WizzWriters / spelldraw-app

Free and open-source whiteboard.
Other
0 stars 0 forks source link

Release ShapeWizard 2.0.0 :mage_man: #34

Closed ventus550 closed 1 year ago

ventus550 commented 1 year ago

Requires newest classifier and bugfixed regressors. Regressor's application interface needs to be updated since the base model now returns structured tensor. From this point on models will arrive frozen, without most of their training package and will contain metadata.

ventus550 commented 1 year ago

@MusicFreak456 If it's alright with you I will keep the [0,1] outputs for the data and training simplicity. Also, it appears to be a common theme for this kind of normalization across many other models out in the wild.

ventus550 commented 1 year ago

image

Ellipses are indeed way more difficult of a regression task than rectangles. :1st_place_medal:

ventus550 commented 1 year ago

image

There is not much difference between triangles an rectangles.

MusicFreak456 commented 1 year ago

@MusicFreak456 If it's alright with you I will keep the [0,1] outputs for the data and training simplicity. Also, it appears to be a common theme for this kind of normalization across many other models out in the wild.

I can live with it. It probably won't cause a lot of computational overhead to multiply it in postprocessing (since there is only a few points returned).

This code can be refactored with adaptation to 2.0.0v: https://github.com/MusicFreak456/interactive-whiteboard-app/blob/bd481e05ad31a04fd45b5556ce7133c9d3250c01/src/services/magic/ShapeWizard.ts#L32 I don't think that this is good a place to rescale the points and also hardcoded 70 has to be removed.

ventus550 commented 1 year ago

Solving #40 required before merging.

ventus550 commented 1 year ago

Some weight loss news

image