Skip to content

Conversation

@whoisj
Copy link
Contributor

@whoisj whoisj commented Oct 27, 2025

This change breaks the monolithic src/libtorch.cc into multiple files, with a modern separation of classes into separate header and code files.

No actual code changes were made besides the separation of the code into multiple files.

@whoisj whoisj requested review from kthui and yinggeh October 27, 2025 21:01
@whoisj
Copy link
Contributor Author

whoisj commented Oct 27, 2025

pre-commit fails on a file I have not changed 😤

@yinggeh
Copy link
Contributor

yinggeh commented Oct 28, 2025

pre-commit fails on a file I have not changed 😤

I found that, too. Pre-commit is now using Python 3.14.

@whoisj whoisj force-pushed the jwyman/refactor-files branch from 2e66c48 to 015d383 Compare October 30, 2025 21:47
This change breaks the monolithic src/libtorch.cc into multiple files,
with a modern separation of classes into separate header and code files.
@whoisj whoisj force-pushed the jwyman/refactor-files branch from 015d383 to 642f74e Compare October 30, 2025 21:51
@@ -0,0 +1,106 @@
// Copyright 2019-2025, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prefer string_utils.hh or simply utils.hh

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated per request.

Renamed 'string_utilities.*' to 'string_utils.*' as requested.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants