@rohany pointed out that, since we started using the Rust-based Legion profiler (and stopped doing the post-processing as part of running the launcher), the message printed out at the end, which is supposed to tell the user how to view the profile, is not actually copy-pasteable:
Profiles have been generated under /home/bryan/work/cunumeric, run legion_prof --view /home/bryan/work/cunumeric/legate_*.prof to view them
Because we don't "install" the legion_prof executable under $CONDA_PREFIX/bin, at least in editable mode, so legion_prof is not on the PATH. The user would need to dig through to find it in the build directory, in this case under /home/bryan/work/legate.core/_skbuild/linux-x86_64-3.10/cmake-build/_deps/legion-build/bin/legion_prof.
@bryevdv and I think our best strategy here is to install a script under $CONDA_PREFIX/bin that points to the legion_prof in the build directory (I assume in non-editable mode we would just copy legion_prof itself), similar to what we do for things like legate, legate-jupyter and legate-create-library (see #669). Then we could keep the simple message, and have it be copy-pasteable.
@jjwilke @trxcllnt thoughts?
@rohany pointed out that, since we started using the Rust-based Legion profiler (and stopped doing the post-processing as part of running the launcher), the message printed out at the end, which is supposed to tell the user how to view the profile, is not actually copy-pasteable:
Because we don't "install" the
legion_profexecutable under$CONDA_PREFIX/bin, at least in editable mode, solegion_profis not on thePATH. The user would need to dig through to find it in the build directory, in this case under/home/bryan/work/legate.core/_skbuild/linux-x86_64-3.10/cmake-build/_deps/legion-build/bin/legion_prof.@bryevdv and I think our best strategy here is to install a script under
$CONDA_PREFIX/binthat points to thelegion_profin the build directory (I assume in non-editable mode we would just copylegion_profitself), similar to what we do for things likelegate,legate-jupyterandlegate-create-library(see #669). Then we could keep the simple message, and have it be copy-pasteable.@jjwilke @trxcllnt thoughts?