karllark commented 2 years ago

A couple of recent PRs changed the names of the blackbody (BB) components, fixed part of the BB setup, and put in a quick fix on the limits of the gas line fwhm values (see PR #176 and #177).

This PR updates the tests so that they pass. The M101 fits looks similar and fine.

codecov-commenter commented 2 years ago

Codecov Report

Merging #209 (fe66940) into master (7090b71) will increase coverage by 0.89%. The diff coverage is 52.08%.

@@            Coverage Diff             @@
##           master     #209      +/-   ##
==========================================
+ Coverage   51.48%   52.38%   +0.89%     
==========================================
  Files           8        8              
  Lines         538      567      +29     
==========================================
+ Hits          277      297      +20     
- Misses        261      270       +9

Impacted Files	Coverage Δ
pahfit/scripts/run_pahfit.py	`16.66% <0.00%> (ø)`
pahfit/base.py	`55.18% <50.00%> (+1.02%)`	:arrow_up:
pahfit/PAHFIT_Spitzer_Exgal.py	`96.22% <100.00%> (ø)`
pahfit/component_models.py	`90.38% <100.00%> (+0.80%)`	:arrow_up:

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 53ca3c2...fe66940. Read the comment docs.

karllark commented 2 years ago

Codacy error can be ignored.

PAHFIT / pahfit

Fix CI tests for recent PRs #209

Codecov Report