nvidiaProfileInspector icon indicating copy to clipboard operation
nvidiaProfileInspector copied to clipboard

[Enhancement] Backport changes from the Revamped fork

Open Patrxgt opened this issue 10 months ago • 8 comments

It would be nice to have all of the improvements from the Revamped fork made by @xHybred in the original version of NVIDIA Profile Inspector.

Patrxgt avatar Feb 10 '25 01:02 Patrxgt

no, it wouldn't.

not the forced requirement for versions of dot net that aren't preincluded anyway.,

Squall-Leonhart avatar Feb 15 '25 08:02 Squall-Leonhart

Yes, it would.

Installing .NET runtime isn't an issue for most of the people, so your argument is flawed.

Patrxgt avatar Feb 15 '25 17:02 Patrxgt

Yes, it would.

Installing .NET runtime isn't an issue for most of the people, so your argument is flawed.

No, sorry, off you go.

Squall-Leonhart avatar Feb 18 '25 11:02 Squall-Leonhart

No, thanks. I'm staying.

Patrxgt avatar Feb 18 '25 16:02 Patrxgt

its totally fine when there are forks

i dont like dotnet core runtime with exact version match requirement either.

with old .net framework you always just needed the latest version and you were fine with all apps build on lower versions. also .net framework is still preincluded in windows.

the new ahead of time compilation binary output with self contained dotnet is bloating and does not work for windows forms yet. so you need windows app sdk libs anyway. or rely on having the correct runtime version already installed.

lets keep it small and simple, there are no real benefits switching to new dotnet core at least for this project

Orbmu2k avatar Feb 18 '25 18:02 Orbmu2k

Thank you for deciding to stay with the old .NET! I'm one of those who doesn't want to install these new frameworks :)

achtchaern avatar Feb 18 '25 22:02 achtchaern

i dont like dotnet core runtime with exact version match requirement either.

Major rollforward is a thing, now that theres a guarantee the new abi will support the previous, but the link microsoft uses on net core apps to get the required version often breaks and doesn't get thrown anyway if you have say 9.0.2 but the app needs 9.0.4, it will usually silent fail instead.

Squall-Leonhart avatar Feb 19 '25 06:02 Squall-Leonhart

As the author of Revamped, I support this decision, but I also appreciate someone making a request on it!

Hopefully/maybe though some of the improvements can be brought over to the the original NVPI without the new dotnet core dependency.

xHybred avatar Feb 25 '25 06:02 xHybred