CEMeNT-PSAAP / MCDC

MC/DC: Monte Carlo Dynamic Code
https://mcdc.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
20 stars 20 forks source link

merging `harmonize-integration` with `main` #166

Open jpmorgan98 opened 6 months ago

jpmorgan98 commented 6 months ago

The situation

harmonize-integration which really was more of the of the GPU develop branch had fallen significantly behind main. @braxtoncuneo started the merge and successfully got all kernels compiled, operable, and running with Harmonize. Unfortunately when those kernels exit the data is not able to be moved back from the GPU to the CPU. This is due to some issue with the data layout of the mcdc global variable.

Other factors

The np.ndarray() structure is clearly at issue not just for this issue but also for #158 where it requires every entry to be of the same size. Finding an alternative data structure is probably the long term solution here but is very annoying. Trouble shooting the merge to be able to get all data working then making on the fly checks is maybe a less overhead issue but still fraught