-w flag enabled by default
See original GitHub issueI just noticed an issue with the CLI arguments, and more particular concerning the -w (write_to_movie) flag.
It appears that when -s (save_last_frame) is False (most of the time, I think), write_to_movie is set True even though -w hasn’t been placed. (that’s why for example in the README just -pl is used instead of -plw).
That is shown at l. 166 of config.py:
"write_to_movie": args.write_to_movie or not args.save_last_frame,
This may cause issues when one wants to get the last frame AND the video.
Should we fix it or let -w enabled by default (even though it’s not very rigorous) ?
Issue Analytics
- State:
- Created 3 years ago
- Comments:14 (14 by maintainers)
Top Results From Across the Web
Runtime Enabled Features
Features that are hidden behind a runtime flag are known as Runtime Enabled Features. It is a requirement of the Blink Launch Process...
Read more >Warning Options (Using the GNU Compiler Collection (GCC))
The warning is emitted only with --coverage enabled. By default, this warning is enabled and is treated as an error. -Wno-coverage-invalid-line-number can be ......
Read more >verbose` flag be enabled by default in magit commit screen?
The magit workflow seems to always be open status buffer and then open split with commit buffer, you always end up with two...
Read more >Enable and disable GitLab features deployed behind feature ...
The feature becomes enabled by default. The feature flag is removed. These features can be enabled and disabled to allow or prevent users...
Read more >What are Chrome flags?
Chrome flags enable the user to activate or deactivate experimental features, whereas the controls available from the chrome://settings page ...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found

How about this:
-wOR nothing-s-ws-d(for “dry run”, or any other available letter)@huguesdevimeux if it is ready for merging, then go ahead. We can always add tests later*.