Skip to content

Add __repr__ to BoutOptions#134

Merged
ZedThree merged 2 commits intomasterfrom
boutoptions-repr
Jun 27, 2025
Merged

Add __repr__ to BoutOptions#134
ZedThree merged 2 commits intomasterfrom
boutoptions-repr

Conversation

@ZedThree
Copy link
Copy Markdown
Member

(Includes #133)

I was getting annoyed that it was missing one. Returning the dict form
is the most sensible I think, at least it's the easiest to make sense
of in the terminal

ZedThree added 2 commits June 26, 2025 09:32
Not the best, because `eval(repr(f)) != f`, but more useful than
nothing
Copy link
Copy Markdown
Contributor

@bendudson bendudson left a comment

Choose a reason for hiding this comment

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

Thanks @ZedThree !

@ZedThree ZedThree merged commit 7164a89 into master Jun 27, 2025
9 checks passed
@ZedThree ZedThree deleted the boutoptions-repr branch June 27, 2025 08:25
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