GEOS-DEV / LvArray

Portable HPC Containers (C++)
BSD 3-Clause "New" or "Revised" License
47 stars 10 forks source link

Add shallow pretty printers and tests #314

Closed TotoGaz closed 7 months ago

TotoGaz commented 7 months ago

I needed LvArray pretty printers that would match the STL pretty printers pattern. This version of the pretty printers do not show any implementation details and focus on the sole data. As such I've simplifying the existing pretty printers a bit.

I've added unit tests, but I did not activate those unit tests through any CI/CD process (I think that this is another subject).

TotoGaz commented 7 months ago

Any extra debugging features are great! I trust you that they work since I've never written these before.

Thanks you for the review 😉 I hope everything's OK on your side!

One limitation that both my pretty printers and the old ones have is that they are not able to deal with permutations... It's not an algorithmic issue, but moving back and forth from python to gdb brings some pure computer science difficulties I could not overcome. I've added a warning message... 🤷