ds4dm / ecole

Extensible Combinatorial Optimization Learning Environments
https://www.ecole.ai
BSD 3-Clause "New" or "Revised" License
319 stars 68 forks source link

Inconsistent default for pseudo_candidates #238

Closed gasse closed 2 years ago

gasse commented 3 years ago

Describe the bug

The flag pseudo_candidates is set to False by default in some places, for example in ecole.environment.Branching and ecole.observation.Khalil2016, while it is set to True by default in some other places, for example in ecole.observation.StrongBranchingScores. This can be confusing and lead to unexpected results / behaviors when someone instantiates everything with the default values.

Setting

Expected behavior

pseudo_candidates should always be set to False by default, everywhere.

Additional context

See the discussion here.