When profits are negative, GA selects for POOR fitness about gekkoga

[kerbe6]

Ok, I thought long and hard, and this is the most elegant solution I could come up with:

Store a minFitness similar to how we store a maxFitness:

let minFitness = [1000, 0, 0, 0];
else if (populationProfits[i] < minFitness[0]) 

          minFitness = [populationProfits[i], populationSharpes[i], populationScores[i], i];

If we are dealing with negative profits, offset everything based on the lowest profit to ensure all values are > 0, and recalculate fitnessSum:

 if (minFitness[0] < 0) {
		let offset = 0 - minFitness[0];
		fitnessSum = 0;
		
		for (let i = 0; i < this.populationAmt; i++) {
			populationProfits[i] += offset;
			fitnessSum += populationProfits[i];
		}
	}

Calculate fitness like this, while summing the weights:

selectionProb[j] = populationProfits[j] / fitnessSum;
		sumOfWeights += selectionProb[j];`

When selecting parents to breed, we do this:
let selectedProb = randomExt.float(sumOfWeights, 0);
instead of
let selectedProb = randomExt.float(1, 0);

I also think it's a nice idea to do:

let a=0, b=0;
      while (a == b) {

to ensure we aren't breeding a parent with itself.

[kerbe6]

I'm actually not sure about that formula for calculating fitness on a negative profit. Might want to look into that.

[illion20]

I am facing the same issue, I tried to update the code with your changes but it seems the selection is still random when profits are minus. Has there been a patch? Something in another branch that solves this?

[generalectric]

Nice find @kerbe6. It also needs to be fixed so that even negative profit is improved. As it is.. it will infinite loop until it has profit greater than zero, meaning we never get genes to start from. Currently it isnt a big deal with 10 loose settings and a decent strat... but if you have 20+ settings or multiple candles it starts to get a little tougher to get off the ground. There is no patch, the fix is pretty simple. Someone just needs to PR the fix or I can do it next time Im testing a strat though Im kinda backed up and have alot on the backburner.

[hugokoopmans]

@kerbe6 your adjustments are great suggestions, be aware that now the reported profits are by definition to positive!!! You add an offset to the to make them positive.

[hugokoopmans]

@kerbe6 I do not understand why you do this:

When selecting parents to breed, we do this:

let selectedProb = randomExt.float(sumOfWeights, 0);

instead of

let selectedProb = randomExt.float(1, 0);

The selectProb is already a proportion, a probability should be between 0 and 1 so why the extra step?

[hugokoopmans]

my suggestion is to split populationProfit and populationFitness so we can create a custom fitness function regardless of the actual profits a strat makes...

[mboremski]

could there be (as a first step) a way to keep the best profit even if negative?
If marked is going down and down in a timespan this might be good way to minimize losses...

[mboremski]

Any news here?

[generalectric]

I've visited this a couple times without solving it. I just haven't had the time to figure it out and got my hands full at the moment.

[mboremski]

could there be (as a first step) a way to keep the best profit even if negative?
If marked is going down and down in a timespan this might be good way to minimize losses...

Any news?