keflavich / imf

Simple tools to work with the Initial Mass Function
MIT License
44 stars 17 forks source link

Chabrier distribution is ignoring mmin #21

Closed hcferguson closed 2 years ago

hcferguson commented 4 years ago

I was puzzled that make_cluster with mmin set to 2.0 was returning a list that included lower masses. Looking at the code, it seems Chabrier is using the LogNormal distribution, which ignores mmin and mmax.

keflavich commented 4 years ago

Good catch. It should be using a truncated lognormal instead.

keflavich commented 2 years ago

This was fixed in #26