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

Added Bayer filter support for color cameras #15

Merged
merged 2 commits into from
Nov 5, 2019
Merged

Conversation

karlrees
Copy link
Contributor

@karlrees karlrees commented Nov 3, 2019

  • Added switch to specify whether camera has a Bayer filter or not.
  • Enable Bayer-related text fields when Bayer filter selected.
  • Update minor version number.

@karlrees
Copy link
Contributor Author

karlrees commented Nov 3, 2019

By the way, I was originally hard-coding the driver to automatically recognize color models (those passing the IsColor()) test as having a Bayer filter. I think this would be more user-friendly for new users, as it avoids the need for the user to learn about and enable the switch. However, I decided I know too little about the product line and use cases to make such a change at this point, so a switch would be the less disruptive approach.

@knro
Copy link
Collaborator

knro commented Nov 3, 2019

Very useful change, thank you! Can you also update the debian changelog as well?

@karlrees
Copy link
Contributor Author

karlrees commented Nov 4, 2019

Okay, not entirely sure about the format / conventions of the debian changelog, but hopefully that'll do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants