shacklettbp / madrona

MIT License
321 stars 31 forks source link

Implemented Context::iterateQuery in both CPU and GPU backends #15

Closed ZanderMajercik closed 1 year ago

ZanderMajercik commented 1 year ago

Two specific things to highlight:

  1. In SortArchetypeNodeBase::resizeTable, the numDynamicInvocations was previously being set to the numRows in the talble pre-resize, which could cause overflow if the table had shrunk.
  2. The implementation pattern of StateManager::iterateQuery closes matches that of StateManager::iterateArchetypesRaw. The difference is that iterateArchetypes raw does not consider a world ID. I decided to keep them separate for now, but we may want to merge them to simplify the code.