Spelt / ZXing.Delphi

ZXing Barcode Scanning object Pascal Library for Delphi VCL and Delphi Firemonkey
Apache License 2.0
471 stars 206 forks source link

Camera not actually released #86

Closed iwan-c-sugeng closed 5 years ago

iwan-c-sugeng commented 5 years ago

Hi, i've using this library on Delphi Rio 10.3.2, i found out that when i set the camera active to false, it doesn't really release the camera handle, when i try to open another app that use the camera, the app complain that the camera is in use.

After comparing the FMX.Media.Android from EvB code and from Latest Delphi Rio Code, found out that in your code the camera released on the Destroy event, while on the Delphi Rio Code, it is in the StopCapture method. I've tried it and it is working

Step to reproduce:

note:

Contact me if you want to see my update, so you can merge it to your code if not a problem, i'm seeing some new routine in Rio so i don't know how to handle backward compatibility

Spelt commented 5 years ago

Hi

Dmx, yes show me the update or attach a Git patch. Would be great!

Thanks Edward

iwan-c-sugeng commented 5 years ago

Hi,

I've attach my patch, thanks

FMX.Media.Android.zip

Spelt commented 5 years ago

Hi Iwan

In the DoStartCapture you have this code:

CapturePollingTimer := TTimer.Create(nil); CapturePollingTimer.Interval := FCaptureTimerInterval; CapturePollingTimer.OnTimer := OnCaptureTimer;

You should remove this because CapturePollingTimer is already created in the OnCreate and freed in the onDestroy. Otherwise this will cause problems.

Your patch solved the problem but it causes another problem. The app crashes when I switch the camera to On to Off to On.

Any ideas?

Op di 20 aug. 2019 om 05:01 schreef dmx78 notifications@github.com:

Hi,

I've attach my patch, thanks

FMX.Media.Android.zip https://github.com/Spelt/ZXing.Delphi/files/3518488/FMX.Media.Android.zip

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ADAS3VPVFXCWAXMJWZ5AH3LQFNM73A5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4U4ZLA#issuecomment-522833068, or mute the thread https://github.com/notifications/unsubscribe-auth/ADAS3VPQBNCPMFIETR6U6KTQFNM73ANCNFSM4ILLRN4A .

iwan-c-sugeng commented 5 years ago

Hi,

I've checked my code, the create CapturePollingTimer code from DoStartCapture, is originally from the original Embarcadero Delphi Rio Code, and i forget to remove it, while on the OnCreate event, it is from your code which not remarked as added.

I'm attaching my test project, and it work well with the On Off On Off .. the camera. And even when the apps move to background, it is correctly calling the stopcapture method and stop the camera, the problem is that it is not yet handled correctly to set switch to correct state, but it can be added easily

This project have my custom class to wrap you AdvanceTestApp demo Project7.zip

Spelt commented 5 years ago

Thanks,

It looks like somehow when the camera is dynamically created problems are gone.

I changed and simplified the Advanced demo and uploaded it to gitHub. I created a new small version because of a memleak fix past week.

BUT: Because some files of the camera speed tweak used are closed source I do make a new recommendation: use a commercial camera lib.WinSoft has a decent one and you get the source + support.

Many thanks!

Sincerely, Edward

Op di 20 aug. 2019 om 17:05 schreef dmx78 notifications@github.com:

Hi,

I've checked my code, the create CapturePollingTimer code from DoStartCapture, is originally from the original Embarcadero Delphi Rio Code, and i forget to remove it, while on the OnCreate event, it is from your code which not remarked as added.

I'm attaching my test project, and it work well with the On Off On Off .. the camera. And even when the apps move to background, it is correctly calling the stopcapture method and stop the camera, the problem is that it is not yet handled correctly to set switch to correct state, but it can be added easily

This project have my custom class to wrap you AdvanceTestApp demo Project7.zip https://github.com/Spelt/ZXing.Delphi/files/3521297/Project7.zip

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ADAS3VJUXBIX6LF3NDLBIKDQFQB4ZA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4WTUTY#issuecomment-523057743, or mute the thread https://github.com/notifications/unsubscribe-auth/ADAS3VPFQY5OYB64S62UYT3QFQB4ZANCNFSM4ILLRN4A .

iwan-c-sugeng commented 5 years ago

Hi,

Hope you still can read this closed issue, don't know how it work and i don't want to mess up your issue just for this message. I've just wondering, is it possible to submit your code to GetIt manager, your Barcode Scanner library is wonderful, easy to use and fast. I'm asking about the licensing issue with the camera speed up code. If possible, i encourage you to submit the code to Embarcadero

Thank you

On Wed, Aug 21, 2019 at 10:18 PM E Spelt notifications@github.com wrote:

Thanks,

It looks like somehow when the camera is dynamically created problems are gone.

I changed and simplified the Advanced demo and uploaded it to gitHub. I created a new small version because of a memleak fix past week.

BUT: Because some files of the camera speed tweak used are closed source I do make a new recommendation: use a commercial camera lib.WinSoft has a decent one and you get the source + support.

Many thanks!

Sincerely, Edward

Op di 20 aug. 2019 om 17:05 schreef dmx78 notifications@github.com:

Hi,

I've checked my code, the create CapturePollingTimer code from DoStartCapture, is originally from the original Embarcadero Delphi Rio Code, and i forget to remove it, while on the OnCreate event, it is from your code which not remarked as added.

I'm attaching my test project, and it work well with the On Off On Off .. the camera. And even when the apps move to background, it is correctly calling the stopcapture method and stop the camera, the problem is that it is not yet handled correctly to set switch to correct state, but it can be added easily

This project have my custom class to wrap you AdvanceTestApp demo Project7.zip https://github.com/Spelt/ZXing.Delphi/files/3521297/Project7.zip

— You are receiving this because you commented. Reply to this email directly, view it on GitHub < https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ADAS3VJUXBIX6LF3NDLBIKDQFQB4ZA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4WTUTY#issuecomment-523057743 , or mute the thread < https://github.com/notifications/unsubscribe-auth/ADAS3VPFQY5OYB64S62UYT3QFQB4ZANCNFSM4ILLRN4A

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ALMWIXU57BGWDZFKSUDFU2DQFVMDFA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD42AVKY#issuecomment-523504299, or mute the thread https://github.com/notifications/unsubscribe-auth/ALMWIXXAMRZRLLAV5NRYOFDQFVMDFANCNFSM4ILLRN4A .

Spelt commented 5 years ago

Hi

There are no plans to create a component of the zxing lib.

About the camera. Its actually not mine tweak but Eric van Bilsen. Anyway embarcadero is aware of the tweak because its already reported in quality central.

I do recommend another camera component because in the used tweak there is a dependency lib which Eric has the source only.
Embarcadero should rework the camera component but they cant do everything.

Thanks for the compliment.

Op 23 aug. 2019 om 18:06 heeft dmx78 notifications@github.com het volgende geschreven:

Hi,

Hope you still can read this closed issue, don't know how it work and i don't want to mess up your issue just for this message. I've just wondering, is it possible to submit your code to GetIt manager, your Barcode Scanner library is wonderful, easy to use and fast. I'm asking about the licensing issue with the camera speed up code. If possible, i encourage you to submit the code to Embarcadero

Thank you

On Wed, Aug 21, 2019 at 10:18 PM E Spelt notifications@github.com wrote:

Thanks,

It looks like somehow when the camera is dynamically created problems are gone.

I changed and simplified the Advanced demo and uploaded it to gitHub. I created a new small version because of a memleak fix past week.

BUT: Because some files of the camera speed tweak used are closed source I do make a new recommendation: use a commercial camera lib.WinSoft has a decent one and you get the source + support.

Many thanks!

Sincerely, Edward

Op di 20 aug. 2019 om 17:05 schreef dmx78 notifications@github.com:

Hi,

I've checked my code, the create CapturePollingTimer code from DoStartCapture, is originally from the original Embarcadero Delphi Rio Code, and i forget to remove it, while on the OnCreate event, it is from your code which not remarked as added.

I'm attaching my test project, and it work well with the On Off On Off .. the camera. And even when the apps move to background, it is correctly calling the stopcapture method and stop the camera, the problem is that it is not yet handled correctly to set switch to correct state, but it can be added easily

This project have my custom class to wrap you AdvanceTestApp demo Project7.zip https://github.com/Spelt/ZXing.Delphi/files/3521297/Project7.zip

— You are receiving this because you commented. Reply to this email directly, view it on GitHub < https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ADAS3VJUXBIX6LF3NDLBIKDQFQB4ZA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4WTUTY#issuecomment-523057743 , or mute the thread < https://github.com/notifications/unsubscribe-auth/ADAS3VPFQY5OYB64S62UYT3QFQB4ZANCNFSM4ILLRN4A

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ALMWIXU57BGWDZFKSUDFU2DQFVMDFA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD42AVKY#issuecomment-523504299, or mute the thread https://github.com/notifications/unsubscribe-auth/ALMWIXXAMRZRLLAV5NRYOFDQFVMDFANCNFSM4ILLRN4A .

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or mute the thread.

ettiennedup commented 4 years ago

hi

i wonder if you can assist. we bought the component from winsoft and they are not replying to an error . it seems as though there is a bug in the component when using android 9.0. 7.0 and 8.0 works fine. the app just closes down

Spelt commented 4 years ago

I cant help you debugging someone else its software. What I can do is ask also.

Its probably a change in an api which needs updating. Mobile is a fast pacing world.

Op ma 21 okt. 2019 om 03:35 schreef ettiennedup notifications@github.com:

hi

i wonder if you can assist. we bought the component from winsoft and they are not replying to an error . it seems as though there is a bug in the component when using android 9.0. 7.0 and 8.0 works fine. the app just closes down

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ADAS3VOS4FFMU2JQLLWV6NLQPUBMJA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBYZYYQ#issuecomment-544316514, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADAS3VIH7QXMW3T52KH65U3QPUBMJANCNFSM4ILLRN4A .

ettiennedup commented 4 years ago

Thanks

I also think it is that but I cant debug . Bummer and I paid so much for the software

Regards Ettienne du Plessis Cell: 0781307154

Tel : 0837269376

http://www.programmic.co.za/ http://www.programmic.co.za

http://www.docsecure.co.za http://www.docsecure.co.za/

From: E Spelt notifications@github.com Sent: 21 October 2019 11:11 To: Spelt/ZXing.Delphi ZXing.Delphi@noreply.github.com Cc: ettiennedup ettienne@programmic.co.za; Comment comment@noreply.github.com Subject: Re: [Spelt/ZXing.Delphi] Camera not actually released (#86)

I cant help you debugging someone else its software. What I can do is ask also.

Its probably a change in an api which needs updating. Mobile is a fast pacing world.

Op ma 21 okt. 2019 om 03:35 schreef ettiennedup notifications@github.com:

hi

i wonder if you can assist. we bought the component from winsoft and they are not replying to an error . it seems as though there is a bug in the component when using android 9.0. 7.0 and 8.0 works fine. the app just closes down

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ADAS3VOS4FFMU2JQLLWV6NLQPUBMJA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBYZYYQ#issuecomment-544316514, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADAS3VIH7QXMW3T52KH65U3QPUBMJANCNFSM4ILLRN4A .

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ACDOJXLN2JFCU2KIKBF5TALQPVW2DA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBZT5FA#issuecomment-544423572 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ACDOJXMSIKY4NQ7LRUD3Q33QPVW2DANCNFSM4ILLRN4A .

Spelt commented 4 years ago

Dont you have the source?

Op 21 okt. 2019 om 11:15 heeft ettiennedup notifications@github.com het volgende geschreven:

Thanks

I also think it is that but I cant debug . Bummer and I paid so much for the software

Regards Ettienne du Plessis Cell: 0781307154

Tel : 0837269376

http://www.programmic.co.za/ http://www.programmic.co.za

http://www.docsecure.co.za http://www.docsecure.co.za/

From: E Spelt notifications@github.com Sent: 21 October 2019 11:11 To: Spelt/ZXing.Delphi ZXing.Delphi@noreply.github.com Cc: ettiennedup ettienne@programmic.co.za; Comment comment@noreply.github.com Subject: Re: [Spelt/ZXing.Delphi] Camera not actually released (#86)

I cant help you debugging someone else its software. What I can do is ask also.

Its probably a change in an api which needs updating. Mobile is a fast pacing world.

Op ma 21 okt. 2019 om 03:35 schreef ettiennedup notifications@github.com:

hi

i wonder if you can assist. we bought the component from winsoft and they are not replying to an error . it seems as though there is a bug in the component when using android 9.0. 7.0 and 8.0 works fine. the app just closes down

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ADAS3VOS4FFMU2JQLLWV6NLQPUBMJA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBYZYYQ#issuecomment-544316514, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADAS3VIH7QXMW3T52KH65U3QPUBMJANCNFSM4ILLRN4A .

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ACDOJXLN2JFCU2KIKBF5TALQPVW2DA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBZT5FA#issuecomment-544423572 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ACDOJXMSIKY4NQ7LRUD3Q33QPVW2DANCNFSM4ILLRN4A .

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.

iwan-c-sugeng commented 4 years ago

Winsoft have a goox support, i talk to him also, very helpful, maybe he is busy

On Mon, Oct 21, 2019, 23:04 E Spelt notifications@github.com wrote:

Dont you have the source?

Op 21 okt. 2019 om 11:15 heeft ettiennedup notifications@github.com het volgende geschreven:

Thanks

I also think it is that but I cant debug . Bummer and I paid so much for the software

Regards Ettienne du Plessis Cell: 0781307154

Tel : 0837269376

http://www.programmic.co.za/ http://www.programmic.co.za

http://www.docsecure.co.za http://www.docsecure.co.za/

From: E Spelt notifications@github.com Sent: 21 October 2019 11:11 To: Spelt/ZXing.Delphi ZXing.Delphi@noreply.github.com Cc: ettiennedup ettienne@programmic.co.za; Comment < comment@noreply.github.com> Subject: Re: [Spelt/ZXing.Delphi] Camera not actually released (#86)

I cant help you debugging someone else its software. What I can do is ask also.

Its probably a change in an api which needs updating. Mobile is a fast pacing world.

Op ma 21 okt. 2019 om 03:35 schreef ettiennedup < notifications@github.com>:

hi

i wonder if you can assist. we bought the component from winsoft and they are not replying to an error . it seems as though there is a bug in the component when using android 9.0. 7.0 and 8.0 works fine. the app just closes down

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub < https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ADAS3VOS4FFMU2JQLLWV6NLQPUBMJA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBYZYYQ#issuecomment-544316514 , or unsubscribe < https://github.com/notifications/unsubscribe-auth/ADAS3VIH7QXMW3T52KH65U3QPUBMJANCNFSM4ILLRN4A

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub < https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ACDOJXLN2JFCU2KIKBF5TALQPVW2DA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBZT5FA#issuecomment-544423572> , or unsubscribe < https://github.com/notifications/unsubscribe-auth/ACDOJXMSIKY4NQ7LRUD3Q33QPVW2DANCNFSM4ILLRN4A> .

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ALMWIXW2FPI3RLI5SKLB7FLQPXHIFA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB23J2Q#issuecomment-544584938, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALMWIXQOT2YH27ODAPP5D3TQPXHIFANCNFSM4ILLRN4A .

ettiennedup commented 4 years ago

Hi

Well he was responding to me until I asked this question last week . I am ready to go to production on this and my customer is not happy as I cant resolve the issue. I think it might be because of android version 9 .

And I don’t have a device I can debug on.

So yes I am also not happy . Eric could at least have returned my mail , but this is not your problem.

I will have to find an alternative if this is the level of support I am going to get

Thanks anyways

Regards Ettienne du Plessis Cell: 0781307154

Tel : 0837269376

http://www.programmic.co.za/ http://www.programmic.co.za

http://www.docsecure.co.za http://www.docsecure.co.za/

From: dmx78 notifications@github.com Sent: 22 October 2019 07:48 To: Spelt/ZXing.Delphi ZXing.Delphi@noreply.github.com Cc: ettiennedup ettienne@programmic.co.za; Comment comment@noreply.github.com Subject: Re: [Spelt/ZXing.Delphi] Camera not actually released (#86)

Winsoft have a goox support, i talk to him also, very helpful, maybe he is busy

On Mon, Oct 21, 2019, 23:04 E Spelt notifications@github.com wrote:

Dont you have the source?

Op 21 okt. 2019 om 11:15 heeft ettiennedup notifications@github.com het volgende geschreven:

Thanks

I also think it is that but I cant debug . Bummer and I paid so much for the software

Regards Ettienne du Plessis Cell: 0781307154

Tel : 0837269376

http://www.programmic.co.za/ http://www.programmic.co.za

http://www.docsecure.co.za http://www.docsecure.co.za/

From: E Spelt notifications@github.com Sent: 21 October 2019 11:11 To: Spelt/ZXing.Delphi ZXing.Delphi@noreply.github.com Cc: ettiennedup ettienne@programmic.co.za; Comment < comment@noreply.github.com> Subject: Re: [Spelt/ZXing.Delphi] Camera not actually released (#86)

I cant help you debugging someone else its software. What I can do is ask also.

Its probably a change in an api which needs updating. Mobile is a fast pacing world.

Op ma 21 okt. 2019 om 03:35 schreef ettiennedup < notifications@github.com>:

hi

i wonder if you can assist. we bought the component from winsoft and they are not replying to an error . it seems as though there is a bug in the component when using android 9.0. 7.0 and 8.0 works fine. the app just closes down

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub < https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ADAS3VOS4FFMU2JQLLWV6NLQPUBMJA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBYZYYQ#issuecomment-544316514 , or unsubscribe < https://github.com/notifications/unsubscribe-auth/ADAS3VIH7QXMW3T52KH65U3QPUBMJANCNFSM4ILLRN4A

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub < https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ACDOJXLN2JFCU2KIKBF5TALQPVW2DA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBZT5FA#issuecomment-544423572> , or unsubscribe < https://github.com/notifications/unsubscribe-auth/ACDOJXMSIKY4NQ7LRUD3Q33QPVW2DANCNFSM4ILLRN4A> .

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ALMWIXW2FPI3RLI5SKLB7FLQPXHIFA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB23J2Q#issuecomment-544584938, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALMWIXQOT2YH27ODAPP5D3TQPXHIFANCNFSM4ILLRN4A .

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Spelt/ZXing.Delphi/issues/86?email_source=notifications&email_token=ACDOJXK5MYCN4K6MIELXOVDQP2HZBA5CNFSM4ILLRN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB4THTA#issuecomment-544814028 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ACDOJXMKAQUAUMGNCNGQM43QP2HZBANCNFSM4ILLRN4A .