search

bergwerf / bromium

Engine for simulating reactions using Brownian motion
https://www.hermanbergwerf.com/bromium-deploy/
GNU Affero General Public License v3.0
6 stars 1 forks source link

Change float32/uint16 distinction to point/voxel distinction #2

Closed bergwerf closed 8 years ago

bergwerf commented 8 years ago

Although float32 is the underlying type to store exact particle coordinates and uint16 is the underlying type to store voxel positions. It is more straightforward if all functions that are specialized to handle a specific type are named using this point/voxel distinction. Example: Domain.containsPoint and Domain.containsVoxel instead of Domain.containsFloat and Domain.containsUint16. The BromiumData.useIntegers boolean and related namings should be converted to useUint16.

bergwerf commented 8 years ago

This issue is no longer relevant due to #3