You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Question: Why is save_order False by default? #118
I was trying to understand the meaning of save_order better and I see that it can skip some long words when they will not fit whole within max_length. I'm very curious about why this is the default behaviour.
Things have been added over years. Have a look at the code, for the behavior, especially where the params are passed in, we have description of each arg.
This is Dojo, posting a maintainer-authorized follow-up linking this issue to #191.
save_order remains False by default to preserve existing outputs. The behavior is documented rather than silently changing the default.
The default algorithm remains legacy; improved output rules are opt-in. This note does not announce a published release, and no individual PR is being merged by this follow-up. Thank you for the contribution and discussion.
Activity
un33k commentedon Sep 27, 2022
Things have been added over years. Have a look at the code, for the behavior, especially where the params are passed in, we have description of each arg.
jdevera commentedon Sep 27, 2022
I played around with it enough to understand what it does, and it seemed "interesting" to have a feature but I was surprised it was default behaviour.
un33k commentedon Nov 22, 2022
Evolving software sometimes start with things that don't make sense many years later. Minimally we have ability to on/off things.
un33k commentedon Sep 8, 2026
This is Dojo, posting a maintainer-authorized follow-up linking this issue to #191.
save_order remains False by default to preserve existing outputs. The behavior is documented rather than silently changing the default.
The default algorithm remains legacy; improved output rules are opt-in. This note does not announce a published release, and no individual PR is being merged by this follow-up. Thank you for the contribution and discussion.
🚀 Generated with Dojo ⛩️