Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable/disable logging at runtime using C api #356

Open
jithunnair-amd opened this issue Jul 22, 2020 · 5 comments
Open

Enable/disable logging at runtime using C api #356

jithunnair-amd opened this issue Jul 22, 2020 · 5 comments
Assignees

Comments

@jithunnair-amd
Copy link

Hi,
Can we explore whether we can add the ability for a user to enable/disable MIOpen logging at runtime using a C api? The current environment-variable-based approach works well for relatively short testcases, but for a long-running testcase, this can result in a lot of logging output. If we wish to focus only on a section of the workload, it'd be quite useful to be able to turn on/off logging at runtime using a C api.

@atamazov
Copy link
Contributor

Variant (B) from #336 (comment) can be used to implement this.

@jithunnair-amd
Copy link
Author

@atamazov Since I am recently looking into a potential MIOpen issue, how far did this get, if at all? It looks like #336 is Done.

@atamazov
Copy link
Contributor

atamazov commented Feb 4, 2021

In the same place, as no priority is set for this yet. Nevertheless #336 is a good step towards the goal ;) Please indicate how important is this feature for you, thanks.

/cc @daniellowell

@asroy asroy assigned asroy and unassigned asroy Feb 8, 2021
@ppanchad-amd
Copy link

@jithunnair-amd Please advise if this still an issue on ROCm 6.0.2? Thanks!

@atamazov
Copy link
Contributor

@ppanchad-amd

Can we explore whether we can add the ability for a user to enable/disable MIOpen logging at runtime using a C api?

This is done.

If we wish to focus only on a section of the workload, it'd be quite useful to be able to turn on/off logging at runtime using a C api.

We do not know if this feature is really required.

@jithunnair-amd Please:

...indicate how important is this feature for you, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants