Skip to content

Bench #79

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.

Sign up for GitHub

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

Jump to bottom
Open
jeremyneveu wants to merge 38 commits into
base: master
Choose a base branch
from
Open

Bench #79

jeremyneveu wants to merge 38 commits into from

Conversation

@jeremyneveu
Copy link
Collaborator

@jeremyneveu jeremyneveu commented Mar 5, 2021

No description provided.

@coveralls
Copy link

coveralls commented Mar 5, 2021 •
edited
Loading

Pull Request Test Coverage Report for Build 1477

  • 71 of 81 (87.65%) changed or added relevant lines in 11 files are covered.
  • 8 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.2%) to 90.921%
Changes Missing Coverage Covered Lines Changed/Added Lines %
spectractor/simulation/throughput.py 27 28 96.43%
spectractor/extractor/targets.py 12 14 85.71%
spectractor/extractor/images.py 6 9 66.67%
spectractor/extractor/extractor.py 3 7 42.86%
Files with Coverage Reduction New Missed Lines %
spectractor/extractor/chromaticpsf.py 8 90.22%
Totals Coverage Status
Change from base Build 1441: -0.2%
Covered Lines: 7210
Relevant Lines: 7930
💛 - Coveralls

jeremyneveu added 22 commits March 9, 2021 21:21
@jeremyneveu
Merge branch 'master' of https://github.com/LSSTDESC/Spectractor into…
5db3191 
… auxtel
@jeremyneveu
update values for holo4_003
c4babc3
@jeremyneveu
if no guess position is given, apply medfilt2d to the image and find …
b69c124 
…the birghtest pixel
@jeremyneveu
only use logbooks if CTIO
8e29db5
@jeremyneveu
modified load_AUXTEL_image for recent data set
e42bbb5
@jeremyneveu
flip RA orientations
0edf152
@jeremyneveu
nansum for projections
adfc3e9
@jeremyneveu
use HASTART in hourangle units
49d6411
@jeremyneveu
normalize ref spectrum with max in lambda window
cb4017a
@jeremyneveu
clean-up
bb6a3c3
@jeremyneveu
increase medfilt2d kernel size to detect brightest star
62de124
@jeremyneveu
add auxtel filters
61da7ed
@jeremyneveu
divide by 100
c60d8bc
@jeremyneveu
Merge branch 'auxtel' of https://github.com/LSSTDESC/Spectractor into…
e5d68e0 
… auxtel
@jeremyneveu
restrain parameters.LAMBDA_MAX and parameters.LAMBDA_MIN range automa…
196d47e 
…tically if a filter is given
@jeremyneveu
use config path to load provided config files
7b3b849
@jeremyneveu
doctest for load_config
4b808b9
@jeremyneveu
clean-up
4628b78
@jeremyneveu
debug image.filter default None to ""
9146a35
@jeremyneveu
debug image.filter default None to ""
162c589
@jeremyneveu
completely change the throughput gestion: delete Throughput class; re…
985b389 
…name filter attribute to filter_label; add reset_lambda_range() method
@jeremyneveu
change image.filter into image.filter_label
80a84ba
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jeremyneveu @coveralls