DiamondLightSource / DiamondLightSource/scanpointgenerator

ROIExcluder rename, ROIIncluder

Open
#78 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
6
Forks
6
PR merge metrics
No merged PRs in 30d

Description

ROIExcluder currently *includes* all points within a ROI. Shouldn't it exclude all points within a ROI and include everything not in the ROI?
Should there therefore be a ROIIncluder class which does what a ROIExcluder currently does?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.