I think it would be better.

[darrenli6]

https://github.com/rodrigo-brito/ninjabot/blob/181d22019248c1174674b222dcde4801659df677/model/model.go#L62-#L65

I think it would be better.

start := size - positions

if start <= 0 {
    return df
}

[darrenli6]

ninjabot/model/model_test.go

Lines 10 to 22 in 181d220

	func TestCandle_ToSlice(t *testing.T) {
	candle := Candle{
	Pair: "BTCUSDT",
	Time: time.Date(2021, 1, 1, 0, 0, 0, 0, time.UTC),
	Open: 10000.1,
	Close: 10000.1,
	Low: 10000.1,
	High: 10000.1,
	Volume: 10000.1,
	Complete: true,
	}
	require.Equal(t, []string{"1609459200", "10000.1", "10000.1", "10000.1", "10000.1", "10000.1"}, candle.ToSlice(1))
	}

I think it makes more sense to do that.

func TestCandle_ToSlice(t *testing.T) {
	candle := Candle{
		Pair:     "BTCUSDT",
		Time:     time.Date(2021, 1, 1, 0, 0, 0, 0, time.UTC),
		Open:     10000.12,
		Close:    10000.12,
		Low:      10000.14,
		High:     10000.14,
		Volume:   10000.19,
		Complete: true,
	}
	require.Equal(t, []string{"1609459200", "10000.1", "10000.1", "10000.1", "10000.1", "10000.2"}, candle.ToSlice(1))
}

[rodrigo-brito]

Hi @darrenli6, thanks for the suggestions!
Feel free to contribute and welcome to open PRs with the suggestions!