Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ROI naming - must not contain a "," #1280

Closed
2 tasks done
jomjol opened this issue Nov 5, 2022 · 3 comments
Closed
2 tasks done

ROI naming - must not contain a "," #1280

jomjol opened this issue Nov 5, 2022 · 3 comments
Labels
enhancement New feature or request

Comments

@jomjol
Copy link
Owner

jomjol commented Nov 5, 2022

The naming of the ROIs must not contain a "," (comma), because this is internally used as column separator.

  • Check naming convention in ROI
  • Information in the documentation
@caco3 caco3 added the enhancement New feature or request label Nov 5, 2022
@caco3
Copy link
Collaborator

caco3 commented Nov 5, 2022

Also check for whitespaces and double quotes

@caco3
Copy link
Collaborator

caco3 commented Nov 6, 2022

#1214 (comment)

@jomjol
Copy link
Owner Author

jomjol commented Nov 6, 2022

Implemented in rolling with f725adf5ba9ced8a76df142da72788f42b62800a

@jomjol jomjol closed this as completed Nov 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants