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

Using decorators makes the profile confusing with a lot of circular dependencies. #7267

Open
anmyachev opened this issue May 14, 2024 · 0 comments
Labels
Internals Internal modin functionality P1 Important tasks that we should complete soon

Comments

@anmyachev
Copy link
Collaborator

For example:
image

The situation can be improved with the help of the following workaround proposed here:

traceable_name  = f"wrapped_fn_{fn.__name__}"
wrapped_fn.__name__ = traceable_name ### PATCH - better profiling resolution
wrapped_fn.__code__ = wrapped_fn.__code__.replace(co_name=traceable_name)

With these changes (applied for run_and_log function) the graph will look something like this:
image

@anmyachev anmyachev added P1 Important tasks that we should complete soon Internals Internal modin functionality labels May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Internals Internal modin functionality P1 Important tasks that we should complete soon
Projects
None yet
Development

No branches or pull requests

1 participant