Open just-nilux opened 1 year ago
Essentially, sell events could be labelled as buy events. This should be a big deal, but the behaviour is not significantly different.
I am pushing a current version (to the main branch). Lots of changes though, so will take a while...
Cheers,
Phil
On Fri, May 19, 2023 at 12:27 PM nilux @.***> wrote:
@nateemma https://github.com/nateemma Could you give some more details about the issue you see with label generation and training in NNTC? Or if you push your changes to a dev branch I'll give it some tests as well, I have a few server available right now to do that...
Anyway would be nice to get some details I'll have a look at as well.
— Reply to this email directly, view it on GitHub https://github.com/nateemma/strategies/issues/29, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABD4X57B545MS5TPB3BSQGLXG7CSJANCNFSM6AAAAAAYIFAREU . You are receiving this because you were mentioned.Message ID: @.***>
Fantastic direction where this is going. Thank you for the commit! Yes I already noticed the class imbalance and that many buys get labelled as sells instead... even with various guards in predict_buysell()
@nateemma Could you give some more details about the issue you see with label generation and training in NNTC? Or if you push your changes to a dev branch I'll give it some tests as well, I have a few server available right now to do that...
Anyway would be nice to get some details I'll have a look at as well.