BEAST-Fitting / megabeast

Hierarchical Bayesian Model for Ensembles of Dust Extinguished Stellar Populations
http://megabeast.readthedocs.io/
2 stars 11 forks source link

Major refactor for full MegaBEAST model #55

Closed karllark closed 3 years ago

karllark commented 3 years ago

Major revision of megabeast to support a full ensemble model (stellar + dust). Focus of this PR is to get a fully functioning megabeast that works for fitting a single set of beast results. A latter PR will work on running a megabeast fit for a grid of spatial pixels (previously the only option).

codecov-commenter commented 3 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@6ca165e). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #55   +/-   ##
=========================================
  Coverage          ?   21.42%           
=========================================
  Files             ?        7           
  Lines             ?      308           
  Branches          ?        0           
=========================================
  Hits              ?       66           
  Misses            ?      242           
  Partials          ?        0           

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 6ca165e...20eb7e8. Read the comment docs.

codecov-io commented 3 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@6ca165e). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #55   +/-   ##
=========================================
  Coverage          ?   21.36%           
=========================================
  Files             ?       10           
  Lines             ?      426           
  Branches          ?        0           
=========================================
  Hits              ?       91           
  Misses            ?      335           
  Partials          ?        0           

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 6ca165e...fc9387e. Read the comment docs.

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging fc9387ee0cc4038bfec58d266ac07895b26946ce into 6ca165ee554fac42023dc9ad2c728971b776a51a - view on LGTM.com

new alerts:

karllark commented 3 years ago

Codacy Here is an overview of what got changed by this pull request:


Issues
======
- Added 9

Complexity increasing per file
==============================
- megabeast/plotting/plot_graphic_model.py  6
- megabeast/tests/test_models.py  3
- megabeast/singlepop_dust_model.py  9
- megabeast/mbsettings.py  13

Clones added
============
- megabeast/singlepop_dust_model.py  1
- megabeast/megabeast_fit.py  1

See the complete overview on Codacy

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging 633a309ea8be0a47e5b80a878d5763f3bfd60a78 into 8187e6a504e24a9e28710187465f75d7c8060a4d - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging 31e6b26752166c82c7f85a5ad610dbf58afbc716 into 8187e6a504e24a9e28710187465f75d7c8060a4d - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 3 alerts when merging ebcf5bb35b7922cc8f0457e25fa698d744ba367f into 8187e6a504e24a9e28710187465f75d7c8060a4d - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging b8907966abe0e986a4ed34d3b50dea0e7b63e5b8 into 8187e6a504e24a9e28710187465f75d7c8060a4d - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 803708aa229bec747e48a1fee13f60af83c5316e into 8187e6a504e24a9e28710187465f75d7c8060a4d - view on LGTM.com

new alerts:

karllark commented 3 years ago

Tests failing as there are no tests. Heh.

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert and fixes 1 when merging 8b4dc3476fc7aa5766efa48463f634e455dace89 into 8187e6a504e24a9e28710187465f75d7c8060a4d - view on LGTM.com

new alerts:

fixed alerts: