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.

Sign up for GitHub

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

Jump to bottom

Add stylus/probe visualization #808

Closed
wants to merge 18 commits into from
Closed

Add stylus/probe visualization #808

wants to merge 18 commits into from

Commits on Jul 18, 2024

  1. Aligned stylus.stl with VTK axes 

    Stylus visualization is working with Aalto Polaris
TODO: add a stylus calibration tab to GUI
    Tolonen Luka committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    a15d66f View commit details
    Browse the repository at this point in the history

  2. added r_change to navigation code, now add GUI to record r_change

    Tolonen Luka committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    f3d9fa4 View commit details
    Browse the repository at this point in the history

  3. StylusPage records coord_raw for Navigation

    Tolonen Luka committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    d80ff28 View commit details
    Browse the repository at this point in the history

  4. Clean up corregistrate_probe

    Tolonen Luka committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    9dc508a View commit details
    Browse the repository at this point in the history

  5. using _ to discard the returned variable resulted in a wxPython error… 

    …! Use __ instead
    Tolonen Luka committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    c16ec2c View commit details
    Browse the repository at this point in the history

  6. add illustration of how to align the stylus with head

    Tolonen Luka committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    16c5fcf View commit details
    Browse the repository at this point in the history

  7. Reduce vertical gaps between GUI components. This has been tested on … 

    …both Windows and Ubuntu 22
    Tolonen Luka committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    d13b58f View commit details
    Browse the repository at this point in the history

  8. save stylus orientation (r_change) to config.json

    Tolonen Luka committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    d46f785 View commit details
    Browse the repository at this point in the history

  9. Handle the case where manual stylus calibration is skipped. Print out… 

    … a warning in this case, but allow navigation to work as normal
    Tolonen Luka committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    c3e2312 View commit details
    Browse the repository at this point in the history

  10. Rename r_change to r_stylus (this is the rotation matrix to orient th… 

    …e stylus in vtk)
    Tolonen Luka committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    23d95f1 View commit details
    Browse the repository at this point in the history

  11. Cleaning up

    Tolonen Luka committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    ce4ed57 View commit details
    Browse the repository at this point in the history

  12. RUFF format

    Tolonen Luka committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    c589203 View commit details
    Browse the repository at this point in the history

  13. Completely replace corregistrate_dynamic by corregistrate_probe. Thes… 

    …e functions were identical, except for r_img in corregistrate_probe... This rotation does not change ICP behavior since ICP only affects the translation
    Tolonen Luka committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    4c34cf5 View commit details
    Browse the repository at this point in the history

  14. more RUFF formatting

    Tolonen Luka committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    a6a3f18 View commit details
    Browse the repository at this point in the history

Commits on Jul 22, 2024

  1. Refactor 'Track coil' button behavior: this determines (on the fly) w… 

    …hether to follow the coil or probe with marker. Join the two Corregistrate threads into one. Edit CoilVisualizer and GUI to work with this change.

Please excuse some stray comments and var-definitions (eg. n_coils)... I will clean these up in future multicoil PR.
    Tolonen Luka committed Jul 22, 2024
    Configuration menu
    Copy the full SHA
    76defe0 View commit details
    Browse the repository at this point in the history

  2. Simplify how r_stylus is saved to file: move r_stylus to 'navigation'… 

    … object and remove bogus 'navigation-stylus' object
    Tolonen Luka committed Jul 22, 2024
    Configuration menu
    Copy the full SHA
    7b212f7 View commit details
    Browse the repository at this point in the history

Commits on Jul 23, 2024

  1. Remove behavior where probe is automatically shown/hidden when naviga… 

    …tion is started/stopped. Automatically show coil/probe when Track coil is pressed/unpressed
    Tolonen Luka committed Jul 23, 2024
    Configuration menu
    Copy the full SHA
    62ed00e View commit details
    Browse the repository at this point in the history

Commits on Jul 25, 2024

  1. Now works with tracker coordinate system where stylus points in x-axi… 

    …s as opposed to y-axis. Also, removed unnecessary disabling of 'Track coil' button before navigation.
    Tolonen Luka committed Jul 25, 2024
    Configuration menu
    Copy the full SHA
    c036972 View commit details
    Browse the repository at this point in the history