Resolved GIL acquisition deadlock #33

dzhoshkun · 2018-10-23T07:53:36Z

Closes #16

BEFORE MERGING:

Consider running the check with a device
Run the demo application
Check this link works

Subject to #35

…ll up disk space)

…g files

…param

…y calling it from within IObserverWrapper

…ctly

…e starting

… 12G

…ipt BM part

…Link device

…VI2PCIe Duo DVI port

…phan DVI2PCIe Duo as well

Merge branch '16-iobservableobserverwrapper-unable-to-acquire-gil' of github.com:gift-surg/GIFT-Grab into 16-iobservableobserverwrapper-unable-to-acquire-gil

…DVI port in checker

…sure BroadcastDaemon thread does not start until it's safe to do so

… broad try-catch block

…revent symlink problems

dzhoshkun added 30 commits October 12, 2018 11:54

Issue #16: using Debug CMake type in build script

125b11e

Issue #16: renamed variable for clarity in run script

20fb19e

Issue #16: setting ulimit to dump cores when running check

7dac7e6

Issue #16: saving all output in a hierarchical session directory

470cfc8

Issue #16: added indentation for readability

5e89c91

Issue #16: can now specify number of reps in check script

bc2a2d9

Issue #16: can now optionally specify root output dir (e.g. not to fi…

435b302

…ll up disk space)

Issue #16: logging frequency of exit codes as well

d69aee7

Issue #16: nicer formatting of exit code frequencies log

dfab9e4

Issue #16: ignoring output folder to avoid an accidental commit of bi…

b74aeb5

…g files

Issue #16: using absolute path expansion if root-dir passed as a CLI …

099903a

…param

Issue #16: intermediately writing exit codes every 10 iterations

a0cc695

Issue #16: using git-describe to tag the output with the version used

2ee8a52

Issue #16: querying for update override after the acquisition of GIL

e319205

Issue #16: checking update method overridden in Python before actuall…

40777b6

…y calling it from within IObserverWrapper

Issue #16: using Pythonic percentage formatting

f47b76e

Issue #16: added a SnapshotSaver class that implements IObserver dire…

ed3a02a

…ctly

Issue #16: inserted snapshot saver into the pipeline

f87c8ff

Issue #16: fixed timed saving of snapshots

9be73e9

Issue #16: renamed buffer => np_buffer as buffer is built-in type

e4c17d0

Issue #16: renamed global buffers for clarity

93a6cc7

Issue #16: commented checker script

782c8a1

Issue #16: moved issue-specific comments to shell script

b6261c5

Issue #16: added synopsis display to shell script

fdfbde5

Issue #16: shell script now shows the current session directory befor…

9108bdd

…e starting

Issue #16: shorter name for Python script

345aa97

Issue #16: shorter name for Bash script

1a88e0b

Issue #16: fixed typos

5db2dcc

Issue #16: reduced snapshot output

37ff47d

Issue #16: fixed calling of Python script from Bash script

67ee662

dzhoshkun added 27 commits October 22, 2018 07:34

Issue #16: renamed global buffers for clarity

5099c9d

Issue #16: commented checker script

58a2afe

Issue #16: moved issue-specific comments to shell script

a78214c

Issue #16: added synopsis display to shell script

1844626

Issue #16: shell script now shows the current session directory befor…

8ef5a46

…e starting

Issue #16: shorter name for Python script

ad14262

Issue #16: shorter name for Bash script

4a32b61

Issue #16: fixed typos

0408f12

Issue #16: reduced snapshot output

66b0c33

Issue #16: fixed calling of Python script from Bash script

6f3982d

Issue #16: added an RTD page linking to the complex pipeline example

f434c03

Issue #16: fixed cross-reference to NumPy example from new RTD page

0bd666a

Issue #16: added link from new RTD page to example source code on GitHub

144a517

Issue #16: made the title of the new RTD page more descriptive

c890e4d

Issue #16: revised the RTD page wording for clarity

0bf6a10

Issue #16: removed line numbers from example on new RTD page

eaa67d4

Issue #16: checker script now supports Blackmagic DeckLink 4K Extreme…

c6d611a

… 12G

Issue #16: revised checker script for more generic variable names

41db15f

Issue #16: added missing variable and removed cropping in checker scr…

65b8724

…ipt BM part

Issue #16: added missing file extension specification when using Deck…

9fa8640

…Link device

Issue #16: multi-threading reliability checker now supports Epiphan D…

64f1e8b

…VI2PCIe Duo DVI port

Issue #16: multi-threading reliability launch script now supports Epi…

30a2ed9

…phan DVI2PCIe Duo as well

After rebasing #16 branch onto master:

6a43bec

Merge branch '16-iobservableobserverwrapper-unable-to-acquire-gil' of github.com:gift-surg/GIFT-Grab into 16-iobservableobserverwrapper-unable-to-acquire-gil

Issue #16: using the SDI port of Epiphan DVI2PCIe Duo instead of the …

7e355c7

…DVI port in checker

Issue #16: using default thread constructor with move semantics to en…

c5f590f

…sure BroadcastDaemon thread does not start until it's safe to do so

Issue #35, #16: surrounded VideoSourceOpenCV colour conversion with a…

8bdc48c

… broad try-catch block

Issue #16: build directory is now relative to calling directory, to p…

0322034

…revent symlink problems

dzhoshkun mentioned this pull request Oct 23, 2018

Consider adding the multi-threading reliability checker to CI #36

Closed

dzhoshkun merged commit f08502e into master Oct 23, 2018

dzhoshkun mentioned this pull request Nov 2, 2018

GIL acquisition deadlock related to frame dimensions #48

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Resolved GIL acquisition deadlock #33

Resolved GIL acquisition deadlock #33

dzhoshkun commented Oct 23, 2018 •

edited

Loading

Resolved GIL acquisition deadlock #33

Resolved GIL acquisition deadlock #33

Conversation

dzhoshkun commented Oct 23, 2018 • edited Loading

dzhoshkun commented Oct 23, 2018 •

edited

Loading