Re: [pgAdmin][RM-6916]: Allow disabling GPU hardware acceleration

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Cc: Nikhil Mohite <nikhil(dot)mohite(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin][RM-6916]: Allow disabling GPU hardware acceleration
Date: 2022-01-18 08:51:13
Message-ID: CA+OCxoziCCqzF7ag3taMoZB+QCxS8xgH+riX5gMtcM9nsZSN_g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi

On Tue, Jan 18, 2022 at 6:20 AM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:

> Hi Dave/Nikhil
>
> On Thu, Jan 13, 2022 at 1:31 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>>
>>
>> On Thu, 13 Jan 2022 at 07:38, Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)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(dot)mohite(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Hackers,
>>>>
>>>> Please find the attached patch for RM-6916:
>>>> <https://redmine.postgresql.org/issues/6916> Allow disabling GPU
>>>> hardware acceleration
>>>>
>>>>
>>>> --
>>>> *Thanks & Regards,*
>>>> *Nikhil Mohite*
>>>> *Senior Software Engineer.*
>>>> *EDB Postgres* <https://www.enterprisedb.com/>
>>>> *Mob.No: +91-7798364578.*
>>>>
>>>
>>>
>>> --
>>> *Thanks & Regards*
>>> *Akshay Joshi*
>>> *pgAdmin Hacker | Principal Software Architect*
>>> *EDB Postgres <http://edbpostgres.com>*
>>>
>>> *Mobile: +91 976-788-8246*
>>>
>> --
>> --
>> Dave Page
>> https://pgsnake.blogspot.com
>>
>> EDB Postgres
>> https://www.enterprisedb.com
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
> *pgAdmin Hacker | Principal Software Architect*
> *EDB Postgres <http://edbpostgres.com>*
>
> *Mobile: +91 976-788-8246*
>

--
Dave Page
Blog: https://pgsnake.blogspot.com
Twitter: @pgsnake

EDB: https://www.enterprisedb.com

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2022-01-18 09:21:36 pgAdmin 4 commit: Fixed feature test cases for maintenance dialog.
Previous Message Akshay Joshi 2022-01-18 06:20:39 Re: [pgAdmin][RM-6916]: Allow disabling GPU hardware acceleration