-
Notifications
You must be signed in to change notification settings - Fork 49
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
Release 0.5.1 #153
Merged
Merged
Release 0.5.1 #153
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
Adding exceptions around image solving
Update README.md
and support setting by both index or value.
Set gphoto2 faster Closes #108
Use panoptes installation of solve-field (astrometry).
* Catch error from python and show appropriate message
* Reversing the change to matplotlib which should be in a separate issue
Capture missing solve-field error
travis.yml file before running the tests.
Great work toward an install script, thanks @jamessynge ! * wip - Starting on creating some setup scripts. Need to coordinate with Wilfred who is doing a similar exercise with the Raspberry Pi. * wip - Install cfitsio. * wip - Added download, build and install of astrometry.net, but it fails so far because fitsio headers can't be found. Also added download of index files. * wip - Fix creation of astrometry/data directory. * wip - Added functions for each type of dependenciy to be installed and command-line flags to disable each of the various types of dependencies from beig installed. Still need to remove excess crap at end of file, and to resolve a problem running pytest. * Installation script for PANOPTES. Ready for review AND testing (perhaps in a docker container). * Move install script and input files into POCS/scripts/install. * Fix path to requirements.txt. Add libcfitsio-bin to apt packages. * And fix the requirements.txt path for travis too. * Move requirements.txt back to root of repo. Emit message reminding the user that PATH needs to be updated in current shell. * Write paths to .profile instead of .bashrc. Create a conda environment for all the python packages, etc. Install python packages before building astrometry.net, as it uses numpy. * Modify data.py to install both sets of index files. Create panoptes-env in conda. * Fix path in travis.yml. * Remove trailing blank line. * Fetch more of the 4100 series indices, needed for some tests that use images that have narrow fields, unlike regular Panoptes images.
* Fix call to Popen (cmd is supposed to be a list of args, so shouldn't join options into a single string). Add some verbose output. * Remove unnecessary uses of format where just passing strings will do.
…136) * Update documentation with more details about how to setup a computer for Panoptes. * Fix capitalization of PANOPTES. Add missing instructions.
Looks good, thanks @jamessynge ! * Add PanFixedDome, base class for Dome controller classes. Converted the existing bisque dome to use that as its base class. Converted existing bisque/test_dome.py to use slightly altered API. Added a simplistic dome simulator and test. * Fix return value from Simulator.close (dome_simulator.py). Fix formatting errors. * Tag functions that shouldn't counted for code coverage. Increase coverage of dome simulator test. * Add support for loading a dome specified in a config file. * Add 'dome' to the list of 'all' simulators. * Fix whitespace/comments for codestyle. * Rename PanFixedDome to AbstractDome. * Remove duplicate call to get_root_logger, and remove unneeded comment.
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.
The first real release! This moves what is currently in
develop
into the master branch.