Skip to content

Feature: Cyton Impedance Check Headplot #648

Open
0 of 1 issue completed
Open

Description

@retiutut

Is your feature request related to a problem? Please describe.
Takes too long to check impedance on all channels when using Cyton and Cyton+Daisy.

Describe the solution you'd like
A way to test impedance on all Cyton channels at once, and possibly display this on a headplot diagram.

Describe alternatives you've considered
Current solution requires users to click all 16buttons on and off. Could also implement an Expert Mode keyboard shortcut.

Additional context
https://openbci.com/forum/index.php?p=/discussion/2364/feature-request-impedance-check-map#latest

f1hiqcU

Activity

  1. added this to the 5.0.0 milestone on Nov 14, 2019
  2. self-assigned this
    on Nov 14, 2019
  3. removed this from the 5.0.0 milestone on Jan 30, 2020
  4. added this to the 5.1.0 milestone on May 7, 2021
  5. markopetrovic-neurotech commented on Jun 8, 2021

    @markopetrovic-neurotech

    Cant this be done simply by swapping the eeg data used in Headplot with the global impedence data variable?

    headPlot.setIntensityData_byRef(dataProcessing.data_std_uV, is_railed);
    -> headPlot.setIntensityData_byRef(data_elec_imp_ohm, is_railed);

  6. retiutut commented on Jun 8, 2021

    @retiutut
    MemberAuthor

    @markopetrovic-neurotech It's technically correct but not so simple. Thanks for mentioning this here on the thread though!

    If you look at the image above, the issue arises when users choose something other than the default electrode positions. We need to expand the electrode map to something like what's shown above and make it configurable/assignable to channels. This requires some careful UI/UX work that I have been discussing with the OpenBCI team at meetings and contemplating for some time.

    #963

  7. markopetrovic-neurotech commented on Jun 8, 2021

    @markopetrovic-neurotech

    @retiutut Thanks for the timely responses!

    I thought we could load other electrode positions from electrode_position.txt files, or is the issue for mappings with more than 16 electrodes?

    Also I see the position electrode_xy array is being updated when the electrodes are dragged, it seems to be working but I'm not sure if I can tell the difference.

    Thanks again, I'm excited for the upcoming milestone and would love to help where I can.

  8. retiutut commented on Jun 8, 2021

    @retiutut
    MemberAuthor

    Thanks for the timely responses!

    I thought we could load other electrode positions from electrode_position.txt files, or is the issue for mappings with more than 16 electrodes?

    Also I see the position electrode_xy array is being updated when the electrodes are dragged, it seems to be working but I'm not sure if I can tell the difference.

    Thanks again, I'm excited for the upcoming milestone and would love to help where I can.

    Woah... I just learned something new about the GUI! But I think that moving them to an arbitrary XY position is a "hacky" solution in comparison to showing a full 10-20 diagram and clicking the ones we want.

    Just tested this on 5.0.5 and drag/drop in headplot doesn't seem reliable or well-put-together.

    Very interesting.... thanks for sharing again!

  9. markopetrovic-neurotech commented on Jun 8, 2021

    @markopetrovic-neurotech

    Happy to help! Have a few developers have moved on from the project?

    I agree it seems a bit hacky. I wonder if a drop down to select between different text files would be viable.

  10. retiutut commented on Jun 8, 2021

    @retiutut
    MemberAuthor

    Haha happy to help! Have a few developers have moved on from the project?

    I agree it seems a bit hacky. I wonder if a drop down to select between different text files would be viable.

    #966 . I don't really have time to go over how complicated 5.1.0 is, but I will be handling the bulk of the work. Let's keep this thread light, because the more I have to read later will only increase complexity.

  11. modified the milestones: 5.1.0, 5.0.7 on Aug 5, 2021
  12. modified the milestones: 5.0.7, 5.1.0 on Sep 3, 2021
  13. modified the milestones: 5.1.0, 5.1.1 on Apr 1, 2022
  14. modified the milestones: 5.1.1, 5.1.2 on Aug 15, 2022
  15. removed this from the GUI Maintenance FY23 Q4 milestone on Jun 30, 2023
  16. retiutut commented on Apr 25, 2025

    @retiutut
    MemberAuthor

    Cyton Impedance widget now exists. This should be done now that the HeadPlot widget has been removed.

  17. removed their assignment
    on Apr 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions