Display sensors -> cortex opens MRI viewer in addition to cortex view

Bugs
,
1

The 29-Apr-2024 update causes an MRI viewer figure to be spawned inappropriately when the user executes display sensors -> cortex.

Only the cortex view figure should be spawned.

Screen Shot 2024-05-02 at 8.25.25 AM
Screen Shot 2024-05-02 at 8.25.25 AM1920×1625 196 KB

3

Hi @aarongeller,
Thank you for writing. This is not an inappropriate behavior. This was added recently for two reasons:

  1. If anyone wants to edit the contacts to fine tune and rectify the contacts while doing any surface display (and not just 3D MRI) through Display sensors option they will have the MRI Viewer on the side to be able to do that by default.
  2. Also, having the MRI Viewer open on the side aids towards better analysis. If not required you can always just close it.
4

First, this is a clear change in behavior which has been stable for more than 2 years (likely longer). In my opinion it isn't appropriate in general for a single action to open multiple figures.

In my case, this change breaks scripted behavior for my surgical workflow because the dimensions of the resulting figures are different.

If you insist on keeping this change, can you provide a hook to view_channels_3d to allow me to suppress the extra figure?

5

Thank you @aarongeller for your feedback, it is because of this that Brainstorm has become the powerful and useful too that it is.

As you may have seen, in the last months we have been developing newer features oriented to facilitate and improve the localization of iEEG electrodes. Some of these update include the revamping of the ECOG and SEEG tutorials.

We will discuss in the team to better integrate the new iEEG features without disrupting previous workflows.

6

thanks!

7

Hello @aarongeller, there has been a recent update on the different ways to interact with the iEEG implantation and how these sensors are visualized. With these changes, we do integrate new features, while preserving the behaviour that was already present.

Please update your Brainstorm instance. Now, the different calls to visualize the iEEG do only what is indicated. Thus the behaviour that your initially reported (displaying iEEG sensors only in the cortex) was rolled-back to what is used to be before the April2024 update.

Apologies for the inconvenience.

FYI @chinmay.chinara

1 Like
8

Thank you so much! I have confirmed things work again for my workflow.

See you in LA!

Aaron

1 Like