Changes to composite mass modeling

[williyamshoe]

Changes include:

• Bug fixes
• QoL changes
  • ability to specify kappa_s and r_scale_angle distributions instead of rho0 and r_scale
  • ability to bypass MGE if provided light profile is already "MULTI_GAUSSIAN"
  • providing amp_2_luminosity parameter in KinConstraintsComposite to help intuitive converting amplitudes to correct units
• Changed population level m2l to log_m2l
• Functionality to input M/L distribution, and so to not model population-level M/L
  • see KinConstraintsCompositeM2l subclass

[codecov-commenter]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (b331dbc) 99.35% compared to head (5a2a42a) 99.33%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #26      +/-   ##
==========================================
- Coverage   99.35%   99.33%   -0.03%     
==========================================
  Files          44       44              
  Lines        2182     2262      +80     
==========================================
+ Hits         2168     2247      +79     
- Misses         14       15       +1     

Files	Coverage Δ
hierarc/Likelihood/hierarchy_likelihood.py	98.25% <100.00%> (+0.15%)	⬆️
hierarc/Likelihood/parameter_scaling.py	100.00% <100.00%> (ø)
hierarc/LensPosterior/kin_constraints_composite.py	99.30% <98.70%> (-0.70%)	⬇️

[ajshajib]

@williyamshoe, the formatting check is failing. You can run these commands locally and then push a commit to resolve this:

black .
docformatter -r ./* --black --in-place

You can pip install black and docformatter if needed.

[ajshajib]

@williyamshoe, thanks a lot for making the changes! Are you done implementing them, or is it still in progress?

[williyamshoe]

@williyamshoe, thanks a lot for making the changes! Are you done implementing them, or is it still in progress?

Yup, I'm done implementing the changes!

[ajshajib]

@williyamshoe, looks good to me, except for I requested a docstring to be added. I also changed the method name to _check_arrays() to make it explicit that it's a private method.

[ajshajib]

@williyamshoe, oops, I missed to check the coverage report before merging! Line 535 in kin_constraints_composite.py is missing coverage. If possible, can you make a quick PR to cover that line?