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
Running the app as regular user outputs wrong read speeds #103
Comments
That's why a special warning is displayed at the top right. |
The warning does not inform you that the readings will be incorrect, nor it doesn't specify that root access is required to perform a proper benchmark. If the application doesn't work on Flatpak and Appimage and there is no desire to fix issues why builds of this application are available on those platforms? |
It does. Whether it is right or wrong depends on the device and the mounting arguments. |
Right at the beginning. |
That's not true, limited functionality doesn't mean or even imply that the application may produce incorrect results. |
I guess there's no point in arguing like that, so I will just leave my final opinion. Running KdiskMark in usermode as a Flatpak and Appimage is flawed and it's known to produce incorrect values, hence unless there's a will to fix the issues, it should be forbidden to run without root and removed from platforms where it doesn't work correctly. Any kind of warning, even a bigger one is not enough. One of the core principles of such tools is reliability and without strong focus on quality of produced results I personally can’t recommend this application to anyone. |
I say it again: everything works as expected. Whether or not to recommend it is your decision. Flatpak and AppImage packages are successfully used by many people. |
How am I supposed to know when the caching issue occurs or even exists? The warning is not self-explanatory at all. If I wasn't techy enough, didn't have a comparison benchmark and trusted this program I might have trusted the result.
It's your decision, but not realising this as an issue and knowingly showing users incorrect values isn't nice and heavily undermines applications general reliability. It's like you don't want users to run apps from the internet as sudo, but realistically if a normal user just wants to have a functional application and proper results it must run it as administrator. |
Ok, I will expand on the warning text. |
Thank you for understanding. That is a very good idea. I'd think about ways of actually fixing this issue or if it's unfixable just making sure there's no way a user can get an incorrect values. Even if the user skips all the warnings there should be a system in place that prevents incorrect values from ever appearing. Ultimately though, imo. it's unacceptable to have an app that half works dependingly on where and how you turn it on. Good Luck :) |
Description:
Flatpak & Appimage (NO ROOT ACCESS)
Appimage (RUNNING AS ROOT) - Correct results (Compared against other benchmarks of this pendrive)
Command:
sudo env DISPLAY=$DISPLAY XAUTHORITY=$XAUTHORITY ./KDiskMark-3.1.2-fio-3.32-x86_64.AppImage
The text was updated successfully, but these errors were encountered: