Remove option to give initial_position from run_inference_algorithm.
Description:
run_inference_algorithm currently uses a try-except clause to allow the user to either provide an initial position or an initial state. This has led to some problems, when the except clause fails to trigger.
More broadly, there doesn't seem to be a good reason to
Solution:
Make run_inference_algorithm only take initial_state, so that the caller of run_inference_algorithm is responsible for providing initial_state from initial_position, rather than deferring this task.
A few important guidelines and requirements before we can merge your PR:
[x] The branch is rebased on the latest main commit;
Remove option to give
initial_positionfromrun_inference_algorithm.Description:
run_inference_algorithmcurrently uses a try-except clause to allow the user to either provide an initial position or an initial state. This has led to some problems, when theexceptclause fails to trigger.More broadly, there doesn't seem to be a good reason to
Solution:
Make
run_inference_algorithmonly takeinitial_state, so that the caller ofrun_inference_algorithmis responsible for providinginitial_statefrominitial_position, rather than deferring this task.A few important guidelines and requirements before we can merge your PR:
maincommit;pre-commitis installed and configured on your machine, and you ran it before opening the PR;