Skip to content

Conversation

@Remi-Gau
Copy link
Contributor

No description provided.

@Remi-Gau Remi-Gau linked an issue Oct 14, 2020 that may be closed by this pull request
@Remi-Gau Remi-Gau requested a review from CerenB October 14, 2020 09:11
@Remi-Gau
Copy link
Contributor Author

@CerenB This one is for you. 😉

keyCode = []; %#ok<NASGU>

[~, ~, keyCode] = KbCheck(deviceNumber);
[~, lastTriggerTimeStamp, keyCode] = KbCheck(deviceNumber);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes this is exactly what I'm / was doing! thanks! I didn't update wirh the previous PR so I was using:
[sec, keyCode] = KbPressWait(deviceNumber);

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okidoki so I will merge this now. :-)

@codecov
Copy link

codecov bot commented Oct 14, 2020

Codecov Report

Merging #125 into dev will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev     #125   +/-   ##
=======================================
  Coverage   21.31%   21.31%           
=======================================
  Files          55       55           
  Lines         788      788           
=======================================
  Hits          168      168           
  Misses        620      620           
Flag Coverage Δ
#unittests 21.31% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/waitForTrigger.m 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8ef09d3...b80f8e0. Read the comment docs.

@Remi-Gau Remi-Gau merged commit 3dccdcb into cpp-lln-lab:dev Oct 14, 2020
@Remi-Gau Remi-Gau deleted the remi-wait_for_trigger branch October 14, 2020 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

using waitForTrigger as a timestamp

2 participants