Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🔎 Refactor main analysis loop #54

Open
kalaschnik opened this issue Jan 15, 2022 · 0 comments
Open

🔎 Refactor main analysis loop #54

kalaschnik opened this issue Jan 15, 2022 · 0 comments
Labels
refactor Bad code, performance, et al.

Comments

@kalaschnik
Copy link
Member

Think about changing the main analysis loop. Instead of:

  • Iterating over trial ranges
    • Iterating over a single trial
      • iterate over all fixation indexes in that trial
        • iterate over all hit_names in current fixation index

Swap the last to loops, that may be beneficial for looking frequencies and first look, which don't need to iterate of single fixations

@kalaschnik kalaschnik added the refactor Bad code, performance, et al. label Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Bad code, performance, et al.
Projects
None yet
Development

No branches or pull requests

1 participant