Exponential profile - problem with units

[ssim]

The exponential density profile doesn't work - problem with units on the rho_0 value.

Also, a question of names: shouldn't we call this a power-law density profile?

We could also have an exponential one (i.e. rho propto exp(-v / v_0) ).

Putting these in is easy and would be useful - simple profiles that work for exponential and power-law cases are things that may be useful for many people to play with.

[mklauser]

On 02/14/2014 01:59 PM, ssim wrote:

The exponential density profile doesn't work - problem with units on
the rho_0 value.

Also, a question of names: shouldn't we call this a power-law density
profile?

We could also have an exponential one (i.e. rho propto exp(-v / v_0) ).

Putting these in is easy and would be useful - simple profiles that
work for exponential and power-law cases are things that may be useful
for many people to play with.

—
Reply to this email directly or view it on GitHub
#82.

Hi,
Ad 1 The exponential density profile was never adapted to the current
version of TARDIS and hence was never tested. I guess we should fix this
function.
Ad 2 Yes, renaming the function would be reasonable .
Ad 3 Adding a new "real" exponential density profile is easy and can be
done in a few minutes (~30min).

I will take care of this problems asap ---probably on Monday.

MiChi

[ssim]

This seems to all work now. Closing.