Skip to content

fix: clearer logs in build-annotations ΔΔG step (follow-up to #95)#96

Merged
St3451 merged 1 commit into
masterfrom
perf/skip-ddg-input-copy
May 26, 2026
Merged

fix: clearer logs in build-annotations ΔΔG step (follow-up to #95)#96
St3451 merged 1 commit into
masterfrom
perf/skip-ddg-input-copy

Conversation

@St3451
Copy link
Copy Markdown
Collaborator

@St3451 St3451 commented May 26, 2026

Log-polish follow-up to #95 that was left out of that merge. Three small log changes that came out of the #95 review:

  • Replace the misleading top-level "Completed!" (which fired after just setting a variable in the --ddg_dir case) with branch-specific messages: "Using user-supplied ΔΔG inputs from <path>" or "Download completed!".
  • Drop the duplicate "Parsing completed!" line in build_annotations.py that fired ~1 ms after parse_ddg_rasp's own "Parsing of DDG completed!".
  • Add the trailing newline at the end of scripts/plotting/stability_change.py.

Copilot AI review requested due to automatic review settings May 26, 2026 21:53
@St3451 St3451 merged commit 9e86cfe into master May 26, 2026
1 check failed
@St3451 St3451 review requested due to automatic review settings May 26, 2026 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant