Skip to content

add TriggerSender #5

Closed
Closed

Description

@steelec

Create TriggerSender class in hardware and pull all the way through to the Trial level as per discussion w/ @theHilikus
skeleton outlined in df91dc31ad12553760eff484f6dfabb954cda9b9

Activity

  1. self-assigned this
    on Aug 27, 2022
  2. steelec commented on Sep 17, 2022

    @steelec
    MemberAuthor

    implemented in https://github.com/neuralabc/spft-java/tree/enh-trigger, awaiting final device testing for merge

  3. steelec commented on Sep 17, 2022

    @steelec
    MemberAuthor

    @ajhelum to confirm:

    • inclusion of / enabling of trigger does not affect data recording
    • inclusion of / enabling of trigger does not affect temporal precision of program (including smoothness of display to monitor)
    • inclusion of / enabling of trigger does not affect CPU usage (which could result in any of the above)
    • arduino trigger device does not affect EMG
    • arduino trigger device does not affect EEG
  4. steelec commented on Sep 17, 2022

    @steelec
    MemberAuthor

    implemented in https://github.com/neuralabc/spft-java/tree/enh-trigger, awaiting final device testing for merge

    note other changes in this branch will include altering the meaning of presentation frequency to refer to # milliseconds for displaying each element of a sequence (from Hz, which was less precise) and likely some intermediate implementation of dynamic time warping ( #6 )

  5. ajhelum commented on May 3, 2023

    @ajhelum

    Arduino trigger device is receiving trigger signals from SPFT for all testing files (yaml) smoothly, in one java session. (no need to open fresh session for running separate files)
    Tested with 1.4 jar (recent as of May 03 2023).
    No trigger delays observed.

  6. steelec commented on May 3, 2023

    @steelec
    MemberAuthor

    thank you for the testing, I think this is ready for merge into main

  7. pinned this issue on May 3, 2023
  8. steelec commented on May 3, 2023

    @steelec
    MemberAuthor

    included here: #8

  9. steelec commented on Aug 25, 2023

    @steelec
    MemberAuthor

    now #8 merged into main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions