Release 0.6.0 #271
Merged
Release 0.6.0 #271
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Pep8 and cleanup
* Get rid of print statement that happens too often * Status line that shows connected sensors (TODO: Show connected port)
Added requirements file
Note: This is not ideal because of the hard-coded ports but the fix will have to wait until we can figure out the port issue in general.
Uses a weakref so that the MongoClient instance is discarded if there are no referrers.
* Explicitly convert equinox to float for FITS header
* This will show a warning but return same file * Haven't figured out a good way to test it yet.
* Adding some docstrings for the config loaded and save
* Removing 2.0 second delay on opening serial connection until we identify why it is there in the first place. * Removing unused modules
) Fix timing of command responses from Astrohaven simulator. Was emitting response too soon, so driver was skipping past it when looking for a response, thus never detected that the desired state had in fact been reached. Added logging to the Astrohaven simulator to help with future debugging. Remove obsolete comment. Add clarifying comments about placement of sleep and drain (discard) operations.
Add a 'dome' attribute to Observatory. Add logger kwarg to create_dome_from_config, enabling caller to provide logger to be used. Remove obsolete comment from observatory.py. Add TODO about non-bisque mounts, which assumes all are serial.
Add notes about root-relative imports.
* Add minimal support for domes into POCS. * Close dome when entering the parking state and when powering down. * Tests TBS.
…optes#249) * Make PanBase hermetic across tests by clearing _config each time.
* Replace PanLogger with custom `LogRecord` class
* Allow logger to use either `%` or `{}` (str.format) style for actual
log messages.
* Log messages show correct file/lineno
* Reorder log.yaml
* Reduced log message format via filter (see example below)
* Add docstring
* Fix logger on messaging
* Remove unused coloredlogs module (will put `grc` info in wiki)
Closes panoptes#253
```
I1228 10:05:01.572 logger.py:112 ****************************** Starting PanLogger ******************************
I1228 10:05:04.897 observatory.py:35 Initializing observatory
I1228 10:05:04.897 observatory.py:38 Setting up location
D1228 10:05:04.897 observatory.py:483 Setting up site details of observatory
```
…noptes#256) Add opening of the dome to the on_enter method of the ready state. Add testing with dome to test_pocs.py and test_observatory.py. Move some testing of PanBase to test_base.py (a new file) from test_pocs.py. Add open_dome and close_dome commands to pocs_shell. Add command history to pocs_shell, stored in ~/.pocs_shell_history. Experiment with adding help to a command in pocs_shell (setup_pocs, in this case).
…panoptes#269) the PANOPTES logger in peas_shell. Add command history to peas_shell.
Codecov Report
@@ Coverage Diff @@
## master #271 +/- ##
==========================================
- Coverage 80.6% 79.02% -1.59%
==========================================
Files 36 46 +10
Lines 2527 3275 +748
Branches 319 426 +107
==========================================
+ Hits 2037 2588 +551
- Misses 386 540 +154
- Partials 104 147 +43
Continue to review full report at Codecov.
|
jamessynge
approved these changes
Dec 30, 2017
Yeah, this is the big ugly merge. In the future they shouldn't be like this. I just followed steps from https://github.com/panoptes/POCS/wiki/Creating-a-New-Release |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updates from
0.5.1.[0.6.0] - 2017-12-30
Changed
Observatoryis now a parameter for a POCS instance 195.Added
Removed