-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Prevent system sleep during IO operations that touch (read or write) to/from the device #2457
Comments
A start, seems like a tractable problem to get these translations, though they may not be perfectly idiomatic:
|
No. Having had ample time to consider it I don't see penalizing all users with an annoying prompt that is exceedingly unlikely to apply to them, just so that a few users who should know to disable sleep when an operation is clearly going to take a long time can avoid a few clicks in going to settings and disabling sleep, as something I want to add to Rufus. Plus, the number of requests I have received in 4 years for such a feature is exactly... 2, which doesn't strike me as high demand. → Rejected. Please go through the Windows sleep setting when an operation is likely to take a long time, That's what everybody does. |
Did you read? No prompt. A check box, which defaults to current behavior, which people can enable to have the sane behavior.
Because you close comments so nobody can just comment on the issue! You really do not think that has a cooling effect?
People should not change windows power settings to use a program. It is ridiculous to expect that. I would write better code and handle things better in a proper PR, but here is an example of how it is done in pseudo code ish impl:
|
Not gonna alter the UI either for a feature that very few people will use either.
People can open new issues or e-mail. The closing comment on an issue is literally " Please open a new issue if you think you have a related problem or query." which you did, and which everyone can do. My e-mail is also plastered all over the place.
And I could be cruel and let you waste a lot of time trying to figure out how to alter the Rufus UI to add a new element, because the hard part is not the code you provided. At any rate, considering how few people are going to use this, this would be better as a cheat mode, as I certainly do not see the need for a UI element that exceedingly few people will use. But. again, seeing how few requests I have had for this, and how non blocking it is, considering that anybody can go to their sleep setting, I'm not even considering adding it as a cheat mode. |
Have you met humans? Path of least resistance or bust, 99.9% of the time. You making it high resistance filters out 99.9% of the people who would have said something about it. Does not mean those people did not exist. Similarly, not having the feature means 99.9% of people deal with it, but it does not mean they would not like or appreciate it - again, if you built the feature and analyzed it (not suggesting that, but if you did), I am confident you would have a lot of people using it. Also, it would not be cruel to let me waste time trying to add a new UI element if you would accept the PR, it would only be cruel if you would definitely not accept it. Unless you are adding UI elements via hand-coded assembly, I am sure I could add a UI element to Rufus in my sleep. In any event, I hope you will at least consider what I wrote, before you dismiss it. |
Then bump Add translations to all the files like this, one for each case where
etc.... a bunch of other changes, but nothing I could not do pretty quickly, I think |
Well, I sure wish I could add or even simply alter an existing Rufus UI elements in my sleep, because dd8314b this is what it took to simply replace one. But hey, if you have time to waste, by all mean create a PR... |
I don't understand the addition of |
I guess I should not have said "I do not understand" - I should have just said, it seems like that was not required to be in that PR, it was an ancillary change to bump the limit for the future, because 349 was getting close to the limit? |
I will reject the PR since I disagree with your assumption that people who cared about this feature chose not to voice it. I just haven't seen enough of a demand for it in 4 years, and going on "Oh but there is a demand, it just wasn't voiced to you" from a party that has an obvious vested interest simply won't cut it. And again, the settings to disable sleep is just a couple clicks away, so it's not like not having this feature in Rufus is a showstopper for anyone. |
The OS setting is not intended to be (ab)used that way. I do not want my computer to not sleep for a week after I start a Rufus USB write operation and go on vacation. I want it to not sleep until that operation is done, and then sleep for the remainder of the week. Your opinion here is disregarding quality in this particular instance. Also, you said only two people cared in all those years, but there was already a thumbs up on my post from someone I have never met within less than an hour (?) of me posting it? So that is a clear indication that you are wrong that there is virtually no demand and you would hear about it if there was demand. You are right that I have a bias due to my vested interest and as such, I may overestimate how much interest, but you definitely also underestimate it. Anyway, it is your project, so what can I do? Fork it, I guess. But your loss, I am darned good at this kind of thing, and you just kind of burned my care and affection for your project :/ |
I do still thank you for making Rufus, good software that I am grateful exists. And, I even understand not wanting to do the work of reviewing a PR - it's all work, and reviewing code is not all that fun. But just a bit disappointed that your response was to reject this on the premise of "not a useful feature" rather than "I do not really have the bandwidth to PR" - maybe those are the same thing in the end, but I think expressing that you are too busy would hit a lot better than that there is no value. |
I'm not too busy. From where I stand as the developer of the application I do not see adding an extra UI entry from a feature that I assure you will be of next to no use to 99.99% of people using Rufus, as something that will actually be beneficial. My decision is based on more than 10 years of experience sieving through requests and comments from users (and no I don't wait for people to contact me -- I'm very much looking at social media and user forums as well) to derive features that I estimate will benefit more than a handful of users. And I believe the relative popularity of Rufus derives in part from the fact that I am very conscious about feature creep and make a very concerted effort to review features from a standpoint of the majority of users, rather than the minority that screams loudest. All this to say that I do not see value for the vast majority of users in your proposal period. You are free to disagree with this as much as you want, but as long as I am the designer of the application, you're just going to have to accept it, or fork your own version. |
This is a reasonable idea so count this reply as a +1 to requests. I'll go further - having used Rufus in the past,, I can't think of a scenario where suppressing sleep isn't desirable. I think the "increased UI clutter" argument has some merit here, so I would be in favour of just making this a default behaviour. Apps and games do this already and everyone is happy. |
Absolutely no one does that. Like CookiePLMonster said there is no scenario where you want to keep the OS behavior of going to sleep while Rufus is copying data. This is simply a bug on the software and should be fixed. I can understand not wanting to do the PR yourself or reviewing any code related to this. You're the repo owner and ultimately decide what ends up in the main branch, but not recognizing that this is an actual bug is just bizarre. |
https://old.reddit.com/r/Windows10/comments/m1fzyx/its_2021_why_does_windows_sleephibernate_when/ Maybe there is also a bug in Windows File Explorer then. And that is what I am referring to when I say "That's what everybody does". I believe that at least a couple of people commenting on that thread (and others) are disproving your "Absolutely no one does that." statement. I also have to wonder, since a lot of people here are bound to argue that it is a mere overlook of Microsoft software, if Microsoft perhaps have data that made them keep with the conscious decision of not disabling sleep during long operations (though of course, one could also argue that you wouldn't see the same usage number in long file copy operations using File Explorer and long bad blocks check operations using Rufus, in which case, what might make sense from an environmental standpoint on one hand, and might actually be guiding a conscious decision by Microsoft, might not translate to other software). If you want to go deeper, part of my concern is also that people are bound to run bad blocks check on very large disk, over ill suited USB 2.0 connections, and leave it run for days (in the same way I see tons of people try to use Windows To Go over drives that are completely ill suited for the task), where they really should use something like SMART or something else then. Plus, I have another concern that, if we are running a bad blocks check, i.e. with a device that is suspected to behave in an erratic manner, things may go as bad as causing a Rufus crash (because I've seen bad I/O cascading into application crashes), in which case I'd rather not leave the system in a power state that's different than how it was before they used Rufus. So, I am hearing the arguments. But I still don't see it as black and white as it is being presented by some people here. |
If a cop runs a red light, that doesn't change a judge's opinion on whether to uphold a ticket when you or I run a red light. Windows Shell and Windows File explorer have some obvious bugs and clearly wrong behavior. The answer is usually not because it's the right thing to do. The answer is usually a combination of legacy and other, often worse reasons (sometimes as bad as "if we even just recompile X, or change it by one byte, Y will break, so we are stuck supporting X forever.") https://twitter.com/SheriefFYI/status/1778502664151539894?t=ND_vLWOBx5eEbebQdW9_0w&s=19 |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue if you think you have a related problem or query. |
Hi, thanks for making Rufus, which is great, very useful software!
You closed this (#1519 (comment)) and this (#2160) as won't do, and several others have also complained about sleep interrupting their operations (e.g. #1874). You said you might change your mind if you got a lot of requests for it; but, these issues are locked, so there is no obvious way to request it without opening a new issue. Seems like you may not get many requests, if it is hard to request? :-)
This is a +1 from me for the request (I just ran into this issue).
I would suggest that you do not limit it to badblocks, but rather add a generic check box for whether to prevent sleep during the process that is interacting with the device, rather than an "are you sure?" dialog.
"Prevent system sleep during operations?" seems like the way to go. I assumed Rufus would work this way by default.
This is a couple of lines of code, I can write a PR if you would like. Translation is harder, but I am guessing there should be a way now to translate to 37 languages at once via LLM?
Would love to contribute and think this feature is a big win for Rufus to support.
The text was updated successfully, but these errors were encountered: