ageller / Firefly

A WebGL interactive particle viewer
GNU Affero General Public License v3.0
66 stars 13 forks source link

velocities should be handled independently of fields #56

Closed agurvich closed 2 years ago

agurvich commented 3 years ago

the field name "Velocities" is special in that it's the only one that is treated as a vector. We should separate this out in the same way we do coordinates rather than have users pass it as a field with a specific name (same as with SPH radius).

agurvich commented 3 years ago

better to have scalar field values and then a single vector quantity you can load in (typically velocity but users may wish something else). Would be nice to expose a hook to change "velocity vectors" to "electric field vectors" or whatever they may want to visualize.

agurvich commented 2 years ago

Implemented in kaitai_io