Issue897

Title code tests: update tested planner configurations
Priority wish Status resolved
Superseder Nosy List cedric, jendrik, malte
Assigned To jendrik Keywords
Optional summary
Part of issue888.

Created on 2019-03-01.15:12:26 by jendrik, last changed by cedric.

Summary
Part of issue888.
Messages
msg8580 (view) Author: jendrik Date: 2019-03-02.10:32:18
Thanks! I don't think another review is necessary, we can easily add/remove
configs later.
msg8579 (view) Author: malte Date: 2019-03-02.05:56:24
Looks good to me, but I only checked superficially. If this is a problem,
perhaps ask someone else to have a look, too.
msg8577 (view) Author: jendrik Date: 2019-03-02.00:19:43
I made a pull request at
https://bitbucket.org/jendrikseipp/downward/pull-requests/123 .

I added the configurations that we tested for issue213 (32-bit vs. 64-bit) and
removed the portfolios, which are tested by the driver tests already. Adding LP
configs is a bit more complicated since we don't have an LP solver on many
buildbots and on Bitbucket Pipelines, so I added the configs, but didn't include
them in any tests for now. We should discuss how to test them in issue888.
msg8564 (view) Author: jendrik Date: 2019-03-01.15:12:26
As a simple sanity check, we currently test on the Buildbot and on Bitbucket
pipelines that a set of planner configurations is able to solve some simple
planning tasks. In this issue we'd like to update the set of tested planner
configurations.
History
Date User Action Args
2019-03-04 07:37:13cedricsetnosy: + cedric
2019-03-02 10:32:18jendriksetstatus: reviewing -> resolved
messages: + msg8580
2019-03-02 05:56:24maltesetmessages: + msg8579
2019-03-02 00:19:43jendriksetstatus: in-progress -> reviewing
messages: + msg8577
2019-03-01 15:12:26jendrikcreate