Prevent balanced strategy from modifiying orders when nothing has changed about kelp HOT 5 CLOSED

@Reidmcc this is a good idea.
The change should probably be made in balancedLevelProvider.go

balancedLevelProvider can save (in memory) it's asset balances and the levels it returns from the GetLevels call. Anytime we call GetLevels it can first check if the asset balances have changed from what it has saved in memory:

• if it has changed then it can recompute new levels
• otherwise it can return the levels it has saved in memory.

This will be a pretty quick and simple code change.

The suggested approach will not consider amount tolerances since any balance change will trigger levels to be recomputed. Adjusting this to handle amount tolerances can be worked on as a separate issue which is a relatively small improvement over the suggested approach.

from kelp.

That makes sense. Do you have the code change planned out or shall I take a crack at it?

from kelp.

@Reidmcc The approach described above is what I had been thinking about for a while but never got to making the change. Would be great if you could take a crack at it!

from kelp.

Ok, will do.

from kelp.

@Reidmcc I've uploaded the fill tracking and handling logic, you can take a look at how that works for the mirror strategy as an example: https://github.com/interstellar/kelp/blob/master/plugins/mirrorStrategy.go#L312

from kelp.