-
Notifications
You must be signed in to change notification settings - Fork 9
Feature/402 feature silent framework #407
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.
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
Changes from all commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
e65da73
Supress solver output in SegmentedCalculation in favor of a progress …
FBumann 05180a8
Add logging.info() about solving
FBumann aeb5f05
Merge remote-tracking branch 'origin/main' into feature/402-feature-s…
FBumann 5d9c672
Merge main intop feature/402-feature-silent-framework
FBumann 03b1202
Merge main intop feature/402-feature-silent-framework
FBumann fc42bc2
Add extra log_to_console option to solvers.py
FBumann d3bcdc2
Add extra log_to_console option to solvers.py
FBumann 7931580
Add extra log_to_console option config.py
FBumann 168ec61
Add to tests
FBumann 20602f9
Use default from console to say if logging to console (gurobipy still…
FBumann 95b9217
Add rounding duration of solve
FBumann 677f534
Use contextmanager to entirely supress output in SegmentedCalculation
FBumann 767d8ec
Improve suppress_output()
FBumann faf4267
More options in config.py
FBumann 69ffb13
Update CHANGELOG.md
FBumann 2fbbd3a
Use new Config options in examples
FBumann 209cdfd
Sett plotting backend in CI directly, overwriting all configs
FBumann f529d9b
Fixed tqdm progress bar to respect CONFIG.silent()
FBumann 3ea3881
Replaced print() with framework logger (examples/05_Two-stage-optimiz…
FBumann 284e3a5
Added comprehensive tests for suppress_output()
FBumann 8f613bc
Remove unused import
FBumann 2bd25bc
Ensure progress bar cleanup on exceptions.
FBumann 6d6f15e
Add test
FBumann 3ad25a0
Split method in SegmentedCalculation for better distinction if show o…
FBumann 691d95c
USe config show in exmaples
FBumann 8a504ef
USe config show in results.plot_network()
FBumann 59b125a
Improve readabailty of code
FBumann f3f54c9
Typo
FBumann File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Guard
CONFIG.exploring()against missing PlotlyLine 11 now calls
fx.CONFIG.exploring(), which immediately runsCONFIG.browser_plotting()(see flixopt/config.py lines 476-491). That helper unconditionally importsplotly.io; if Plotly is not installed—which was previously fine for this quick-start example—the import raises and the script aborts before doing anything. This regressions affects every entrypoint that now usesCONFIG.exploring(). Please make the helper tolerant of missing Plotly (e.g., catch theImportErrorand fall back todefault_show=False) before switching the examples to it.Apply this change in
flixopt/config.py:This keeps the new configuration API but avoids breaking environments without Plotly. Based on relevant snippet.
📝 Committable suggestion
🤖 Prompt for AI Agents