Issue #1 new

New field attribute: positivity?

Kacper Kowalik
created an issue

I think we should consider adding new property to "additional field" that would state if it's safe to take a log of all values, namely if field have only positive values. This way we could easily take advantage of //take_log// in //add_field//.

Comments (2)

  1. Sam Skillman

    I'm not sure I'm a fan of adding more atributes like this because it adds on more requirements on the I/O side. I think I'd be a bigger fan of yt presenting a more straightforward handling of linear/log fields. I think there are cases where positivity does not imply that taking the log is the best thing to do.

  2. Log in to comment