-
Notifications
You must be signed in to change notification settings - Fork 6
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.
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
error in foraging.data_to_change_detection_core #563
Comments
@nickponvert I know... no I am not ... I'm on feature/ production |
Ok I'm going to try and reproduce it. |
I am getting this error for more sessions now: |
@nickponvert Same error with two experiments (from session_id: 851428829) that had issues with motion correction, and just got fixed by LIMS people.... |
@jeromelecoq @nickponvert I am in a python environment, after I check out a new git branch, for the codes to take effect, do I need to restart my python session? I am just surprised that why I used to get a platform_info error on feature/production ... and a type error on fix/issue533 .... but I am not back to feature branch , however I am getting the same error as in fix branch .... there is something, I am up to date with new git changes (did git pull very recently) ... I cant remember if the platform error was before I did git pull or after .... is there a chance that feature codes are updated to what was in fix branch, and hence I am getting a similar type of error now on feature branch? or is it that somehow I am not running feature branch code, even though I am in it (based on git branch start indication).... |
Yes, I usually restart my kernel when I update a package. |
@farznaj that fix branch hasn't yet been merged into production analysis, so that shouldn't be the issue. Would you copy/paste the code you are running to get the error? It would be helpful for me to debug the other issue in the fix branch too |
I found this that might be helpful: class ListHandler(logging.Handler):
|
thanks @jeromelecoq |
@nickponvert update: after restarting the kernel, the newly fixed sessions (by lims folks) are working... the only failed sessions are the ones that I originally sent you about the platform error: 845444695846652517 |
@farznaj since the feature/production_analysis branch got merged into master (master should now be the default branch for all analysis) could we re-check this issue to see if it is still a problem? |
@jeromelecoq
session_id: 855711263
experiment: 856123117
(mesoscope session ; The workflow for the experiment is QC).
I get the error below when running the convert code. Any thoughts?
File "", line 2, in
ophys_data = convert_level_1_to_level_2(experiment_id, cache_dir) #, plot_roi_validation=False);
File "/home/farzaneh/Documents/analysis_codes/visual_behavior_analysis/visual_behavior/ophys/io/convert_level_1_to_level_2.py", line 976, in convert_level_1_to_level_2
core_data = get_core_data(pkl, timestamps_stimulus)
File "/home/farzaneh/Documents/analysis_codes/visual_behavior_analysis/visual_behavior/ophys/io/convert_level_1_to_level_2.py", line 415, in get_core_data
core_data = foraging.data_to_change_detection_core(pkl, time=timestamps_stimulus)
File "/home/farzaneh/Documents/analysis_codes/visual_behavior_analysis/visual_behavior/translator/foraging/init.py", line 40, in data_to_change_detection_core
handler = ListHandler(log_messages)
File "/home/farzaneh/Documents/analysis_codes/visual_behavior_analysis/visual_behavior/utilities.py", line 174, in init
super(ListHandler, self).init()
TypeError: super(type, obj): obj must be an instance or subtype of type
The text was updated successfully, but these errors were encountered: