geoutils icon indicating copy to clipboard operation
geoutils copied to clipboard

Consolidate profiling tests into a manual benchmark GitHub workflow

Open rhugonnet opened this issue 1 month ago • 2 comments

Since #738, the test_dask.py are moved to a "benchmark/" folder, but not used yet, in anticipation for a manually-triggered benchmarking GitHub workflow.

Need to consolidate once https://github.com/GlacioHack/xdem/pull/775 is finalized. That PR is in xDEM, not GeoUtils. @adebardo @belletva @marinebcht How should we address profiling tests in GeoUtils? Should we move the profiler module from xDEM, or copy it? (it's relatively short)

rhugonnet avatar Oct 09 '25 23:10 rhugonnet

I don't really see the point of having it in both Geoutils and xDem, do you ? I think the https://github.com/GlacioHack/xdem/pull/775 is now finalized, I will look about the transfert :)

marinebcht avatar Oct 10 '25 13:10 marinebcht

OK, yes let's have it only in GeoUtils then. Thanks! 😄

rhugonnet avatar Oct 10 '25 19:10 rhugonnet