Skip to content

Small lsp logging changes #794

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
merged 2 commits into from
Oct 7, 2025
Merged

Small lsp logging changes #794

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1d5858e
small lsp logging format changes
mohammedahmed18 Oct 7, 2025
ef6259d
Merge branch 'main' into lsp/small-logging-format-changes
Saga4 Oct 7, 2025
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
4 changes: 2 additions & 2 deletions codeflash/context/unused_definition_remover.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,7 @@ def _expand_qualified_functions(self) -> set[str]:
# Find class methods and add their containing classes and dunder methods
for qualified_name in list(self.qualified_function_names):
if "." in qualified_name:
class_name, method_name = qualified_name.split(".", 1)
class_name, _method_name = qualified_name.split(".", 1)

# Add the class itself
expanded.add(class_name)
Expand Down Expand Up @@ -511,7 +511,7 @@ def revert_unused_helper_functions(
if not unused_helpers:
return

logger.info(f"Reverting {len(unused_helpers)} unused helper function(s) to original definitions")
logger.debug(f"Reverting {len(unused_helpers)} unused helper function(s) to original definitions")

# Group unused helpers by file path
unused_helpers_by_file = defaultdict(list)
Expand Down
6 changes: 3 additions & 3 deletions codeflash/optimization/function_optimizer.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -304,7 +304,7 @@ def generate_and_instrument_tests(
]

with progress_bar(
f"Generating new tests and optimizations for function {self.function_to_optimize.function_name}",
f"Generating new tests and optimizations for function '{self.function_to_optimize.function_name}'",
transient=True,
revert_to_print=bool(get_pr_number()),
):
Expand Down Expand Up @@ -1657,8 +1657,8 @@ def establish_original_code_baseline(

loop_count = max([int(result.loop_index) for result in benchmarking_results.test_results])
logger.info(
f"h2|⌚ Original code summed runtime measured over {loop_count} loop{'s' if loop_count > 1 else ''}: "
f"{humanize_runtime(total_timing)} per full loop"
f"h3|⌚ Original code summed runtime measured over '{loop_count}' loop{'s' if loop_count > 1 else ''}: "
f"'{humanize_runtime(total_timing)}' per full loop"
)
console.rule()
logger.debug(f"Total original code runtime (ns): {total_timing}")
Expand Down
Loading