genicam / harvesters

Image Acquisition Library for GenICam-based Machine Vision System
Apache License 2.0
513 stars 88 forks source link

1.4.1 is very slow #374

Closed HitoshiUchida closed 1 year ago

HitoshiUchida commented 1 year ago

I changed 1.4.0 to 1.4.1 with python 3.8.14 on Linux min 21.0. GenTL is mvIMPACT_Acquire v2.47.0

The image getting process became very slow with PySide6. When back to 1.4.0, there no problems of speed.

kazunarikudo commented 1 year ago

@HitoshiUchida Hi, thank you for the report. Could you tell me the difference in numbers so that I can how much it is, please? Regards, Kazunari.

kazunarikudo commented 1 year ago

@HitoshiUchida I need to ask you one more favor: What would happen if your application does not involve PySide? I would like to ask you to try running an image acquisition script without PySide.

HitoshiUchida commented 1 year ago

Dear Kazunari

Sorry this is not a direct answer to your question, but in my opinion, there is probably some interference with the timer in Pyside. I am not sure why, because Dialog, which should be terminating normally, is not terminating, and I need to force it to terminate.

I am attaching my code.

Regards H. Uchida

2022年10月3日(月) 14:37 Kazunari @.***>:

By the way, why did you know that I have released 1.4.1? Do you frequently visit here or PyPI?

— Reply to this email directly, view it on GitHub https://github.com/genicam/harvesters/issues/374#issuecomment-1264959345, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMKLRA7ZDPPCPU2IHDJ2F4LWBJWITANCNFSM6AAAAAAQ3HQ2JI . You are receiving this because you were mentioned.Message ID: @.***>

GeniCam_Cam.py.tar.gz

jcormier commented 1 year ago

In the meantime, Is it possible to get a version released with #374, without this problematic PySide change?

jcormier commented 1 year ago

Whoops I meant #369

HitoshiUchida commented 1 year ago

Ok, I will try.

2023年1月21日(土) 10:13 Kazunari @.***>:

@HitoshiUchida https://github.com/HitoshiUchida Hi Hitoshi, I need to ask you for a favor. Could you try with the latest master (1f4fedd https://github.com/genicam/harvesters/commit/1f4feddaca8eba3f2d2741a238f18697899c61b0) and check if it still slows down your application, please?

— Reply to this email directly, view it on GitHub https://github.com/genicam/harvesters/issues/374#issuecomment-1399113607, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMKLRAYU7USD2WCMHO2CWVTWTMZ4RANCNFSM6AAAAAAQ3HQ2JI . You are receiving this because you were mentioned.Message ID: @.***>

kazunarikudo commented 1 year ago

@HitoshiUchida Hi, excuse me, you don’t need to check that for now; it was a bit early to ask you. I’ll ask you again once the prototype turned available. Thank you!

HitoshiUchida commented 1 year ago

Sure, no problem.

2023年1月22日(日) 8:01 Kazunari @.***>:

@HitoshiUchida https://github.com/HitoshiUchida Hi, excuse me, you don’t need to check that for now; it was a bit early to ask you. I’ll ask you again once the prototype turned available. Thank you!

— Reply to this email directly, view it on GitHub https://github.com/genicam/harvesters/issues/374#issuecomment-1399349668, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMKLRA4KQ7WRG7E6AU7OF2DWTRTD5ANCNFSM6AAAAAAQ3HQ2JI . You are receiving this because you were mentioned.Message ID: @.***>

HitoshiUchida commented 1 year ago

Hi, Kazunari. I had been tested on ver 1.4.2 for my previous reported codes. There are no troubles. Thank you for a Great Job !.