Skip to content

Allow multiple FIP channels in plot axis of session scroller#101

Merged
alexpiet merged 3 commits into
mainfrom
fip_plot_updates
Apr 14, 2026
Merged

Allow multiple FIP channels in plot axis of session scroller#101
alexpiet merged 3 commits into
mainfrom
fip_plot_updates

Conversation

@alexpiet

@alexpiet alexpiet commented Apr 10, 2026

Copy link
Copy Markdown
Collaborator

Allows plotting multiple FIP channels in the same axis on the session scroller:

fip = [
    ["G_0","Iso_0"],
    ["G_0_dff-bright","Iso_0_dff-bright"],
    ["G_0_dff-bright_mc-iso-IRLS","Iso_0_dff-bright_mc-iso-IRLS"],

]
plot_session_scroller(nwb,fip=fip)

@ZhixiaoSu

ZhixiaoSu commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

Can you specify what is needed to test this function? (e.g. preprocessing for nwb)
And can you also specify what is expected behavior?

This is what I did and the output:

 from aind_dynamic_foraging_data_utils.nwb_utils import load_nwb_from_filename
  import aind_dynamic_foraging_data_utils.nwb_utils as nwb_utils
  from aind_dynamic_foraging_basic_analysis.plot.plot_session_scroller import plot_session_scroller
  %matplotlib widget
  
  nwb_file = '/root/capsule/data/behavior_778077_2025-05-16_09-07-04_processed_2026-03-07_01-16-28/nwb/behavior_778077_2025-05-16_09-07-04.nwb'
  nwb = load_nwb_from_filename(nwb_file)
  
  fip = [
      ["G_0","Iso_0"],
      ["G_0_dff-bright","Iso_0_dff-bright"],
  ]
  
  nwb.df_fip = nwb_utils.create_df_fip(nwb)
  
  plot_session_scroller(nwb, fip=fip)
image

@ZhixiaoSu ZhixiaoSu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Passed test using this example session: behavior_778077_2025-05-16_09-07-04_processed_2026-03-07_01-16-28

@alexpiet alexpiet merged commit 3ab266a into main Apr 14, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants