Skip to content
Snippets Groups Projects

CPAlgorithms: better default settings for OverlapRemoval

Merged Baptiste Ravina requested to merge ravinab/athena:automatic_overlapremoval_settings into main

The Overlap Removal (OR) config block has a boolean option addToAllSelections. Users can have a hard time figuring out the proper settings of the config block and this option in particular. In our notes on handling multiple object selections, we show how to set up multiple instances of OR: in those cases, one certainly doesn't want to add both OR selections to all objects! On the other hand, with only one OR setup, one would like to not have to remember to add passesOR_%SYS% everywhere...

In this MR, I change the addToAllSelections default value to None, which will make it be autoconfigured depending on how many OR algorithms have been registered with the ConfigAccumulator. This solves a lot of worries user-side. Experts can still set that option to True/False to obtain whatever configurations they like.

The addPreselection option is also changed to None, and will be True if there are multiple selections and False otherwise. Setting it to True ensures that the newly created selection picks up the OR decision + all the preselections.

Edited by Baptiste Ravina

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading