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

Docker into develop 02 #951

Merged
merged 238 commits into from
Jun 1, 2020
Merged

Docker into develop 02 #951

merged 238 commits into from
Jun 1, 2020

Conversation

wtgee
Copy link
Member

@wtgee wtgee commented Feb 15, 2020

Merging the docker branch into develop.

This PR is effectively v0.7.0

Related Issue

Replaces #950
Closes #927
Closes #934
Closes #948
Closes #531
Closes #137
Closes #734
Closes #960
Closes #828
Closes #952
Closes #897
Closes #809
Closes #970
Related to #115

Relies on panoptes/panoptes-utils#189

Types of changes

  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

wtgee added 19 commits October 13, 2019 16:56
This will merge all the docker components into a new branch in the main repo. As part of milestone 0.7 these items will be cleaned up as a step towards making the docker version the default (i.e. `develop`) version for milestone 0.8.

This is being merged now because there are too many changes within one PR and it will be better to have new PRs that branch off this `docker` branch. This will also allow for installs on new units from the `panoptes` user rather than my personal branches.
Keeping `docker` in line with `develop`.
Adding local user id
…assigned list. Note that this should ideally be happening by the `__del__` of the `AbstractSDCCamera` but is not getting called.
* Explicitly cast free_space boolean

Camera
* Pass the config_port to the subcomponents
* Allow for a default string representation (needed to help debug object before it has fully initialized)

Debug logging
…en fully initialized.

* Logging statements for debug (will be removed).
* Updating all the POCSTIME to be more current.
* Lots of debugging for now (I'm about to switch to desktop and want to keep these in)
* Update POCSTIMEs and targets.
* Make the used observation a fixture.
@wtgee wtgee mentioned this pull request Feb 16, 2020
8 tasks
@codecov
Copy link

codecov bot commented Feb 16, 2020

Codecov Report

Merging #951 into develop will decrease coverage by 14.71%.
The diff coverage is 91.09%.

Impacted file tree graph

@@             Coverage Diff              @@
##           develop     #951       +/-   ##
============================================
- Coverage    83.07%   68.36%   -14.72%     
============================================
  Files           80       95       +15     
  Lines         6353     9500     +3147     
  Branches       838      873       +35     
============================================
+ Hits          5278     6495     +1217     
- Misses         855     2841     +1986     
+ Partials       220      164       -56     
Impacted Files Coverage Δ
src/panoptes/__init__.py 100.00% <ø> (ø)
src/panoptes/peas/remote_sensors.py 85.00% <ø> (ø)
src/panoptes/peas/sensors.py 48.14% <ø> (ø)
src/panoptes/peas/tests/test_boards.py 85.00% <ø> (ø)
src/panoptes/peas/tests/test_sensors.py 100.00% <ø> (ø)
src/panoptes/pocs/__init__.py 71.42% <ø> (ø)
src/panoptes/pocs/base.py 96.87% <ø> (ø)
src/panoptes/pocs/camera/__init__.py 77.96% <ø> (ø)
src/panoptes/pocs/camera/camera.py 81.02% <ø> (ø)
src/panoptes/pocs/camera/canon_gphoto2.py 33.33% <ø> (ø)
... and 178 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 57a9938...c21c107. Read the comment docs.

@wtgee wtgee mentioned this pull request Feb 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment