Giter Site home page Giter Site logo

Comments (11)

JonMagon avatar JonMagon commented on June 16, 2024

That's why a special warning is displayed at the top right.

from kdiskmark.

raspin0 avatar raspin0 commented on June 16, 2024

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?

from kdiskmark.

JonMagon avatar JonMagon commented on June 16, 2024

It does. Whether it is right or wrong depends on the device and the mounting arguments.

from kdiskmark.

raspin0 avatar raspin0 commented on June 16, 2024

obraz
Where does it say that the application may produce incorrect results? I don't think the app should launch at all if there are such known problems.

from kdiskmark.

JonMagon avatar JonMagon commented on June 16, 2024

Right at the beginning.

from kdiskmark.

raspin0 avatar raspin0 commented on June 16, 2024

That's not true, limited functionality doesn't mean or even imply that the application may produce incorrect results.

from kdiskmark.

raspin0 avatar raspin0 commented on June 16, 2024

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.

from kdiskmark.

JonMagon avatar JonMagon commented on June 16, 2024

I say it again: everything works as expected.
If the caching distorts the result, the user needs to run the portable app as a super user to be able to bypass it.
Running the GUI as root user is not the best practice, but the only possible one in this case.
Moreover, it is usually the internal devices that are tested. Or right configured external ones.

Whether or not to recommend it is your decision. Flatpak and AppImage packages are successfully used by many people.
I'm not going to limit the ways of delivery, or to force the user to run an app from the Internet as sudo.
The warning in the upper right corner is sufficient.

from kdiskmark.

raspin0 avatar raspin0 commented on June 16, 2024

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.

Whether or not to recommend it is your decision. Flatpak and AppImage packages are successfully used by many people.
I'm not going to limit the ways of delivery, or to force the user to run an app from the Internet as sudo.
The warning in the upper right corner is sufficient.

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.

from kdiskmark.

JonMagon avatar JonMagon commented on June 16, 2024

Ok, I will expand on the warning text.

from kdiskmark.

raspin0 avatar raspin0 commented on June 16, 2024

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 :)

from kdiskmark.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    πŸ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. πŸ“ŠπŸ“ˆπŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❀️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.