Split up pixelization by field type

#2383 Declined
Repository
MatthewTurk
Branch
yt
Repository
yt_analysis
Branch
yt
Author
  1. MattT
Reviewers
Description

We want to pixelize based on the field type as well as the axis. This starts the process, but is restricted to the cartesian coordinate handler for the time being.

Comments (6)

  1. Andrew Myers

    The changes to the way the unstructured mesh pixelizer is called all look great to me. It looks like this causes some issues with field detection, though?

    1. MattT author

      I was going to, I just can't fix it up today or tomorrow. If you want to take a try before Thursday that'd be awesome.

      1. Nathan Goldbaum

        I've got some commits on top of this PR on the mesh-sampling-type bookmark in my fork that I think fix the tests.

        After looking over this code in detail, I have a small bikesheddy issue. Right now I think 'mesh' and 'cell' might be confusing to people, since cells generally live in meshes. What if instead of 'mesh' we had 'element'? I think 'element' more closely maps to finite elements in my mind, which might help avoid confusion in the future.