PAHFIT / pahfit

Model Decomposition for Near- to Mid-Infrared Spectroscopy of Astronomical Sources
https://pahfit.readthedocs.io/
18 stars 26 forks source link

Combined PAH feature strengths #162

Closed alexmaragko closed 2 years ago

alexmaragko commented 2 years ago

This PR calculates the combined PAH feature strengths, and should replace #142.

Differences with #142:

An EQW column is created in the output ipac table as placeholder for the next PR (that will replace #143).

codecov-commenter commented 2 years ago

Codecov Report

Merging #162 (9f58f47) into master (04d7b08) will decrease coverage by 0.86%. The diff coverage is 23.07%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #162      +/-   ##
==========================================
- Coverage   55.79%   54.92%   -0.87%     
==========================================
  Files           8        8              
  Lines         466      477      +11     
==========================================
+ Hits          260      262       +2     
- Misses        206      215       +9     
Impacted Files Coverage Δ
pahfit/helpers.py 43.33% <ø> (ø)
pahfit/feature_strengths.py 31.57% <20.00%> (-12.87%) :arrow_down:
pahfit/base.py 56.78% <33.33%> (-0.29%) :arrow_down:

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 04d7b08...9f58f47. Read the comment docs.

alexmaragko commented 2 years ago

Last commit includes minor docstring updates, to address issues #157 and #137, and an update to the output table in fit_spectrum.rst to address #160.

Most of the test fail now because sphinx is not able to generate the rst file.

I have copied the ipac output table to the rst file, but it seems formatting issues appear. I'm not sure what is the quickest way to deal with this in order to avoid formating line by line.

I have also tried the astropy ascii.rst format output which generates the table's body properly when pasted in the rst file but not the header.

alexmaragko commented 2 years ago

Fixed the formatting issue by using the ascii.rst format output. Now the matrix test seems to be failing due to Python 3.6 setup with tox environment on macos.

karllark commented 2 years ago

Will work on a separate PR to update the text dependencies.

karllark commented 2 years ago

Test failures fixed in PR #163. Best solution would be to rebase this PR once that PR is merged. That would allow the full tests to be run. Ask and I can help through the 4-5 commands needed to do a rebase (should be clean and straightforward for this PR).

alexmaragko commented 2 years ago

Rebased the PR and changed PAH_112_C to PAH_113_C.