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

DM-20121 Make GenericCamera configurable #1

Merged
merged 2 commits into from
Jul 17, 2019
Merged

Conversation

tribeiro
Copy link
Member

Makes GenericCamera configurable CSC and upgrade it to use SalObj 4.

  • Fix PEP8 violations.
  • Reorganize camera drivers into a separate submodule.
  • Reorganize liveview into a separate submodule.
  • Add unit tests for CSC.
    • Check/validate state transitions
    • Check/validate configuration schema
    • Check takeImage command
    • Check start/stop liveView
  • Add unit tests for Exposure class
  • Add unit tests for liveview server/client
  • Add unit tests for simulatorcamera

Fix PEP8 violations.
Reorganize camera drivers into a separate submodule.
Reorganize liveview into a separate submodule.
Add unit tests for CSC.
	- Check/validate state transitions
	- Check/validate configuration schema
	- Check takeImage command
	- Check start/stop liveView
Add unit tests for Exposure class
Add unit tests for liveview server/client
Add unit tests for simulatorcamera
@tribeiro tribeiro force-pushed the tickets/DM-20121 branch 2 times, most recently from a15d00c to 5e7c73f Compare July 2, 2019 21:54
@jbuffill jbuffill self-assigned this Jul 17, 2019
@jbuffill
Copy link

Good to go

@tribeiro tribeiro merged commit 8087cef into master Jul 17, 2019
@jbuffill jbuffill self-requested a review July 17, 2019 21:13
@jbuffill jbuffill assigned tribeiro and unassigned jbuffill Jul 17, 2019
@wvreeven wvreeven deleted the tickets/DM-20121 branch January 12, 2021 13:16
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