biomedepi / seizure_detection_validation

Repository containing the pipeline to evaluate the models submitted in task 2 of the Seizure Detection Challenge - ICASSP23
0 stars 2 forks source link

question about the ''functions.get_events' #1

Open whubaichuan opened 1 year ago

whubaichuan commented 1 year ago

Hi, thanks for your effort in this reduced version. By the way, I see there is no return in _functions.mergeevents. Thus in _functions.getevents, we actually do not merge the events by 0.2*margin. Is that your original logic or ?

biomedepi commented 1 year ago

The function changes the events list without returning it.

whubaichuan commented 1 year ago

cool, these are about mutable and immutable objects in Python3. You are correct.