Skip to content

Conversation

@bryevdv
Copy link
Contributor

@bryevdv bryevdv commented Nov 9, 2022

This PR adds a new option --cprofile to the driver that enables the Python stdlib cProfile module.

The cProfile module must be loaded with that -m python (legate) command line option. This PR handles things in cmd_module function that normally handles --module, in order to make an explicit exception if --cprofile is used together with --module (i.e. they are incompatible)

The cprofile output is stored at

config.logging.logdir / f"legate_{LEGATE_GLOBAL_RANK_SUBSTITUTION}.cprof"

so that multi-rank profiling is possible.

@bryevdv bryevdv added the category:improvement PR introduces an improvement and will be classified as such in release notes label Nov 9, 2022
@bryevdv bryevdv requested a review from manopapad November 10, 2022 00:28
@bryevdv bryevdv merged commit 75e46db into nv-legate:branch-22.12 Nov 14, 2022
@bryevdv bryevdv deleted the bv/cprofile branch November 14, 2022 22:32
manopapad added a commit that referenced this pull request Mar 5, 2025
* Push CCCL include before Legion

* Set the Legion commit ref to the cuda::std::complex fix

* Add a TODO to reinstate print_dense_array

* Point CI to PR adding support for cuda::std::complex
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:improvement PR introduces an improvement and will be classified as such in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants