Skip to content

[nit] Minor formatting error fix#2

Open
gao-yuan-hangzhou wants to merge 1 commit into
thinking-machines-lab:mainfrom
gao-yuan-hangzhou:gaoyuan/minor-f-string-fix
Open

[nit] Minor formatting error fix#2
gao-yuan-hangzhou wants to merge 1 commit into
thinking-machines-lab:mainfrom
gao-yuan-hangzhou:gaoyuan/minor-f-string-fix

Conversation

@gao-yuan-hangzhou
Copy link
Copy Markdown

Use single quote in f-string...

Thanks Jeremy for the amazing post. I enjoyed reading it and playing with the code.

@Jah-yee
Copy link
Copy Markdown

Jah-yee commented Apr 15, 2026

Good day!

I noticed this formatting issue. Here's the fix for lines 184 and 187 in src/main.py:

# Before:
print(f"Saving results to {os.path.join("results", filename)}")

# After:
print(f"Saving results to {os.path.join('results', filename)}")

The issue is using double quotes inside f-strings which should use single quotes.

Thank you for your work on this amazing project!

Warmly,
RoomWithOutRoof

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.

2 participants