remodnav/paper
2
0
Fork 0
Code Issues 1 Pull requests Releases Packages Activity Actions

Revision round 1 #10

Merged
adswa merged 1 commit from revision_1 into master 2025-08-19 08:44:09 +00:00
Conversation 3 Commits 1 Files changed 1 +1 -1
adswa commented 2020-01-15 13:27:01 +00:00 (Migrated from github.com)
Copy link

This branch contains all updates that have been made in response to the review we got a while ago. Here are some of the larger changes:

  • computations of Cohens Kappa, adds a new Table 3
  • plotting a subsegment of eye gaze samples for both studyforrest datasets, adds two elements to Figure 8
  • highlighting the "reproducible research object" and open source nature of the algorithm and manuscript
  • adding eye movement definitions on oculomotor, computational, and functional dimension (based on Hessels et al., 2018), exchange "detection" with "classification"
  • rework the pseudecode overview

The rest are mostly smallish rewordings to comply to reviewer comments.

Please do not merge until this has been reviewed by everyone involved. Prior or in conjunction to this PR, #8 updates the software requirements, as this PR introduces sklearn as an additional dependency.

This branch contains all updates that have been made in response to the review we got a while ago. Here are some of the larger changes: - computations of Cohens Kappa, adds a new Table 3 - plotting a subsegment of eye gaze samples for both studyforrest datasets, adds two elements to Figure 8 - highlighting the "reproducible research object" and open source nature of the algorithm and manuscript - adding eye movement definitions on oculomotor, computational, and functional dimension (based on Hessels et al., 2018), exchange "detection" with "classification" - rework the pseudecode overview The rest are mostly smallish rewordings to comply to reviewer comments. Please do not merge until this has been reviewed by everyone involved. Prior or in conjunction to this PR, #8 updates the software requirements, as this PR introduces sklearn as an additional dependency.
adswa (Migrated from github.com) reviewed 2020-01-16 08:47:04 +00:00
main.tex
adswa (Migrated from github.com) commented 2020-01-16 08:47:03 +00:00
Copy link
The computational definitions of the different eye movements
```suggestion The computational definitions of the different eye movements ```
adswa (Migrated from github.com) reviewed 2020-01-16 08:51:13 +00:00
main.tex
adswa (Migrated from github.com) commented 2020-01-16 08:51:12 +00:00
Copy link
  The algorithm reports saccades within major saccade window (ISAC),
```suggestion The algorithm reports saccades within major saccade window (ISAC), ```
adswa (Migrated from github.com) reviewed 2020-01-16 08:54:12 +00:00
main.tex
adswa (Migrated from github.com) commented 2020-01-16 08:54:12 +00:00
Copy link
  The algorithm reports saccades within major saccade windows (ISAC),
```suggestion The algorithm reports saccades within major saccade windows (ISAC), ```
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something isn't working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature or request

  
good first issue

Good for newcomers

  
help wanted

Extra attention is needed

  
invalid

This doesn't seem right

  
question

Further information is requested

  
wontfix

This will not be worked on

No labels
bug
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
remodnav/paper!10
No description provided.