Re: [pgAdmin][RM-6916]: Allow disabling GPU hardware acceleration - Mailing list pgadmin-hackers

From Aditya Toshniwal
Subject Re: [pgAdmin][RM-6916]: Allow disabling GPU hardware acceleration
Date
Msg-id CAM9w-_mRMRuKKi0DAWAq7yFiP=hN=kNt-ojJ4iZsbJ0qCiRopg@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin][RM-6916]: Allow disabling GPU hardware acceleration  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin][RM-6916]: Allow disabling GPU hardware acceleration  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
Hi,

The only ways to pass flags are using cli or putting it in package.json.
If we want to keep it configurable then we need to spawn in from the main process and pass the flags.

On Tue, Jan 18, 2022 at 2:21 PM Dave Page <dpage@pgadmin.org> wrote:
Hi

On Tue, Jan 18, 2022 at 6:20 AM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Dave/Nikhil

On Thu, Jan 13, 2022 at 1:31 PM Dave Page <dpage@pgadmin.org> wrote:


On Thu, 13 Jan 2022 at 07:38, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Thanks, the patch applied.

Should we really hard code this? I’m not sure the GPU would actually be used much, but I really don’t know for sure. It might be, for example, for videos on our website.

I wonder if this should be a runtime config option (if possible), or a command line option.

    We can make it configurable by performing the following changes:
  • Add a switch control in the configure dialog labeled 'Disable GPU hardware acceleration'. By default its value is False.
  • Will create a config option in "runtime_config.json" like other options port, timeout, etc...
  • When the user would like to disable the GPU, they will update the switch control and it needs a restart of pgAdmin anyway.
  • We will update the package.json file like "chromium-args": "--disable-popup-blocking --disable-gpu", and restart the pgAdmin.
     If the above seems OK then @Nikhil you can start working on the above changes.

Hmm, no, we can't do that. First, it would invalidate the appbundle signature on macOS, and secondly, on most OS's it would require root privileges to modify an installed file.

Is there no way of setting it from code at startup?
 



On Thu, Jan 13, 2022 at 11:53 AM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:
Hi Hackers,

Please find the attached patch for RM-6916:  Allow disabling GPU hardware acceleration


--
Thanks & Regards,
Nikhil Mohite
Senior Software Engineer.
Mob.No: +91-7798364578.


--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Principal Software Architect
EDB Postgres
Mobile: +91 976-788-8246

--


--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Principal Software Architect
EDB Postgres
Mobile: +91 976-788-8246



--


--
Thanks,
Aditya Toshniwal
pgAdmin Hacker | Software Architect | edbpostgres.com
"Don't Complain about Heat, Plant a TREE"

pgadmin-hackers by date:

Previous
From: Aditya Toshniwal
Date:
Subject: [pgAdmin][patch] Form Input Select enhancements
Next
From: Aditya Toshniwal
Date:
Subject: Re: [pgAdmin][patch] Form Input Select enhancements