Skip to content
This repository has been archived by the owner on Oct 30, 2024. It is now read-only.

[BUG] Spatial frequencies assertion error #76

Open
lauraporta opened this issue Jul 24, 2023 · 2 comments
Open

[BUG] Spatial frequencies assertion error #76

lauraporta opened this issue Jul 24, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@lauraporta
Copy link
Member

Describe the bug
PhtonData throws an exception when running the sanity checks. Row 590.

To Reproduce
Dataset: CX_79_2_hL_RSPd_monitor_right

Expected behaviour
Found spatial frequencies should be identical to those stored in the config file and no exception should be raised

Log file
Log file name: slurm.enc1-node1.3785436.out

                       ERROR    2023-07-24 13:07:19 PM - ERROR -       app.py:45
                                MainProcess app.py:45 -                         
                                Traceback (most recent call last):              
                                  File                                          
                                "/nfs/nhome/live/lporta/source/rsp-vis          
                                ion/rsp_vision/console_application/app          
                                .py", line 42, in inner_function                
                                    func(*args, **kwargs)                       
                                  File                                          
                                "/nfs/nhome/live/lporta/source/rsp-vis          
                                ion/rsp_vision/console_application/app          
                                .py", line 148, in analysis_pipeline            
                                    photon_data = PhotonData(data,              
                                PhotonType.TWO_PHOTON, config)                  
                                                  ^^^^^^^^^^^^^^^^^^^^          
                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^                     
                                  File                                          
                                "/nfs/nhome/live/lporta/source/rsp-vis          
                                ion/rsp_vision/objects/photon_data.py"          
                                , line 112, in __init__                         
                                    self.set_post_data_extraction_vari          
                                ables()                                         
                                  File                                          
                                "/nfs/nhome/live/lporta/source/rsp-vis          
                                ion/rsp_vision/objects/photon_data.py"          
                                , line 590, in                                  
                                set_post_data_extraction_variables              
                                    assert np.all(                              
                                AssertionError         

Computer used (please complete the following information):

  • OS: Linux on hpc
  • Version: on branch shell-script, latest commit: 583a118a5525f9e51da3c6bba06ee5690ece0cf1
  • Hardware specs: RAM requested 150G per cpu, cpu as assigned

Additional context
Sbatch settings:

#SBATCH --array=0-2
#SBATCH -p cpu # partition (queue)
#SBATCH --mem-per-cpu=150G
#SBATCH -t 00-02:00 # time (D-HH:MM)
@lauraporta lauraporta added the bug Something isn't working label Jul 24, 2023
@chrytsi
Copy link

chrytsi commented Feb 14, 2024

Hello,
Checking on my lab notes I found that the wrong screen settings were applied in this experiment so that the spatial frequencies
of the SF/TF protocol were not the desired ones.
We had initially replaced the spatial frequency values with the (subsequently determined) spatial frequencies but I think we should just remove this experiment altogether.

@chrytsi
Copy link

chrytsi commented Feb 14, 2024

Copying part of Lee's exception correction script here that contains the actual spatial frequencies:

switch animal_id
    case 'CX_79_2'
        
        % mapping between actual SFs and desired SFs
        a = [0.2182    0.3200;
            0.1058    0.1600;
            0.0529    0.0800;
            0.0262    0.0400;
            0.0124    0.0200];

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants