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.

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

StarDICE #148

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

StarDICE #148

wants to merge 23 commits into from

Conversation

jeremyneveu
Copy link
Collaborator

Check differences between stardice and master branches

tsouverin and others added 21 commits September 15, 2023 16:02
@tsouverin
change extension of the source file from xyls to axy
ae44ddc
Merge branch 'stardice' of https://github.com/LSSTDESC/Spectractor in…
e189098 
…to stardice
@tsouverin
add few lines to do the astrometry with the data with masked spectrums
2b95133
@tsouverin
change extension from .xyls to .axy
f812658
@tsouverin
"deal with data with spectrums masked"
9d74ab6
@tsouverin
Merge branch 'stardice' of github.com:LSSTDESC/Spectractor into stardice
69b55cb
@tsouverin
small change of syntax
a5ccff5
@tsouverin
add w in returns of SpectractorRun
e2ace08
Merge branch 'v3.0.3' of https://github.com/LSSTDESC/Spectractor into…
13aa26f 
… stardice
@jeremyneveu
Merge pull request #146 from LSSTDESC/v3.0.3
2ffe6d3 
V3.0.3 for stardice
optimal Ndof with weighted FWHM mean
1f116ae
add image.target_label in load_STARDICE
e451b79
SPECTRACTOR_DECONVOLUTION_PSF2D = False -> True
ab11026 
SPECTRACTOR_DECONVOLUTION_FFM = False -> True
PSF_POLY_ORDER = 2 -> 4
PSF_PIXEL_STEP_TRANSVERSE_FIT = 10 -> 1
Merge branch 'stardice' of https://github.com/LSSTDESC/Spectractor in…
e830e27 
…to stardice
SPECTRACTOR_ATMOSPHERE_SIM = getobsatmo
851a7dd
OBS_NAME = STARDICE -> OHP
a15f1fa 
for getobsatmo
OBS_NAME = STARDICE -> OHP
4a8723a 
for getobsatmo
don't use apply_max_width_to_bounds otherwise y_c params have boundar…
6fbfa78 
…ies (0, 2Ny) in FFM and spectrogramfit when using from_poly_to_profile_params functions : problem if y_c is negative
evaluate PSF FWHM centering the PSF on the pixel grid
0b102a3
cleaning
ae0a9f4
add shift_x
ce84ed5
@coveralls
Copy link

coveralls commented Dec 23, 2023 •
edited
Loading

Pull Request Test Coverage Report for Build 7613481679

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+2.1%) to 89.88%
Totals Coverage Status
Change from base Build 7275666541: 2.1%
Covered Lines: 7238
Relevant Lines: 8053
💛 - Coveralls

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 @tsouverin