Skip to content

"Fixed" tracing bug if module name and current working directory are identically named #137

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.

Sign up for GitHub

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

Merged
Essoz merged 5 commits into from
Jun 23, 2026
Merged

"Fixed" tracing bug if module name and current working directory are identically named #137

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b6b0e5b
fixed module parsing if the module and current directory are named th…
danielryckman Jun 4, 2026
fec6917
temp fixes
danielryckman Jun 8, 2026
173297e
fixed epoch looping and kernel on cpu errors
danielryckman Jun 9, 2026
70aae97
implemented suggestions
danielryckman Jun 15, 2026
a5527dd
Merge branch 'main' into daniel/module-name-bug
danielryckman Jun 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion traincheck/instrumentor/tracer.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -523,7 +523,7 @@ def get_module_path_from_file_path(file_path: str, root_module: str) -> str | No
):
return None
# get the path of the module from the file path
path_after_root_module = file_path.split(f"/{root_module}/")[1].split(".py")[0]
path_after_root_module = file_path.rsplit(f"/{root_module}/", 1)[-1].split(".py")[0]
Comment thread
danielryckman marked this conversation as resolved.
Outdated
module_path = f"{root_module}.{path_after_root_module}".replace("/", ".")
return module_path

Expand Down
Loading