Bug in OutputOneOf and OutputEquilateral: incorrect construction of NominalItem

[GoogleCodeExporter]

In Encog 2.5.3 OutputOneOf and OutputEquilateral do not work for me. I tracked 
this down to being because they call the constructor to NominalItem with the 
high and low arguments in the wrong order. I am attaching a patch with the 
change for each of the two classes.

Perhaps some argument checking would help too? NominalItem could throw an 
IllegalArgumentException if the high argument is lower than the low argument.

Original issue reported on code.google.com by justinem...@gmail.com on 24 Apr 2011 at 8:29

Attachments:

• OutputOneOf.java.patch
• OutputEquilateral.java.patch

[GoogleCodeExporter]

Issue fixed on C# as well.  Also standardized to high,low.  Fixed in Encog 3.0.

Original comment by JeffHeat...@gmail.com on 15 Jul 2011 at 1:22

• Changed state: Fixed