Closed GoogleCodeExporter closed 9 years ago
Original comment by bestchai
on 18 Apr 2012 at 6:03
Original comment by bestchai
on 18 Apr 2012 at 7:03
Original comment by bestchai
on 18 Apr 2012 at 7:03
[deleted comment]
Completed tests and other various changes for ConstrainedInvMiner. Please
review code at revision bc07083664a5.
Original comment by kevin.a....@gmail.com
on 14 May 2012 at 9:18
Before I review the code, can you merge the two heads in Issue243 branch? That
is, revisions bc07083664a5 and f3c0b1235628.
Original comment by bestchai
on 14 May 2012 at 6:31
I believe I merged the heads correctly. Revision 44d739fe187e.
Original comment by kevin.a....@gmail.com
on 14 May 2012 at 6:52
I've made a variety of smaller changes to your code, but overall it looks good.
One important change that I've made is to have ConstrainedInvMiner no longer
implement ITOInvariantMiner so that we are not as constrained by it. As well,
I've made ConstrainedInvMiner much more state-full.
Merged issue branch into default with revision 4e2d018fe7b4.
Original comment by bestchai
on 14 May 2012 at 10:11
Original comment by bestchai
on 9 Jan 2014 at 4:35
Original issue reported on code.google.com by
kevin.a....@gmail.com
on 18 Apr 2012 at 3:38Attachments: