esheldon / sdssidl

Automatically exported from code.google.com/p/sdssidl
1 stars 0 forks source link

Cosmology volume element #2

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I'm not sure if it's intended or not but the volume and volume element in the 
cosmology idl class are per steradian at the moment.

Original issue reported on code.google.com by chrisw...@gmail.com on 15 Nov 2010 at 11:48

GoogleCodeExporter commented 9 years ago
Thank you, this should be documented.  Would your expectation different?  Would 
you have expected the result to be times 4pi, and expect the user to divide out 
to get their area?

Perhaps there should be an area= keyword?

Original comment by erin.sheldon@gmail.com on 16 Nov 2010 at 2:31

GoogleCodeExporter commented 9 years ago
I think just adding 'per steradian' to the documentation would be good. 
Personally as it was written at the moment I was expecting it to be over the 
whole sphere. 

Original comment by chrisw...@gmail.com on 16 Nov 2010 at 3:45

GoogleCodeExporter commented 9 years ago
I've added that to the docs in the trunk.

Original comment by erin.sheldon@gmail.com on 16 Nov 2010 at 4:20