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

Adding a few more targets (good for beginning of night for PAN006) #387

Merged
merged 1 commit into from
Jan 18, 2018

Conversation

wtgee
Copy link
Member

@wtgee wtgee commented Jan 18, 2018

No harm in adding these to the default target list.

@jamessynge HD 23630 is in the Pleiades, which might make a nice pretty picture. Should be good at start of evening. You could bump the priority in order to ensure it is selected or just let it go and see what happens.

@wtgee wtgee requested a review from jamessynge January 18, 2018 21:56
@codecov
Copy link

codecov bot commented Jan 18, 2018

Codecov Report

Merging #387 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #387   +/-   ##
========================================
  Coverage    69.05%   69.05%           
========================================
  Files           58       58           
  Lines         4640     4640           
  Branches       641      641           
========================================
  Hits          3204     3204           
  Misses        1264     1264           
  Partials       172      172

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 70fda07...14aebdf. Read the comment docs.

@jamessynge
Copy link
Contributor

Thanks.

@jamessynge jamessynge merged commit e6d5299 into panoptes:develop Jan 18, 2018
@wtgee wtgee deleted the new-targets branch August 20, 2022 00:54
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