Skip to content

Commit

Permalink
Fixed memory leak bug in EphysNWBData
Browse files Browse the repository at this point in the history
Changed import package from functools to methodtools and updated other files as appropriate.
  • Loading branch information
Katherine Baker authored and sgratiy committed Feb 23, 2021
1 parent 75a3ea7 commit 9a8681f
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 3 deletions.
12 changes: 12 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,18 @@ All notable changes to this project will be documented in this file.
### Added

### Changed
Bug fixes:
- Fixed memory leak in method of `EphysNWBData` caused by `@lru_cache` decorator

## [1.0.2] = 2021-01-06

Changed:
- Add features_state information to the pipeline output json
- Improve performance of loading sweeps from NWB2 files by using LRU cache
- More robust error checking when loading time_series

Bug fixes:
- Fix segment length rounding error in the DAT file converter

## [1.0.2] = 2021-01-06

Expand Down
3 changes: 1 addition & 2 deletions ipfx/dataset/ephys_nwb_data.py
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
from typing import Dict, Tuple, Sequence, Union, Type
import warnings
from functools import lru_cache
from methodtools import lru_cache

import numpy as np
import pandas as pd
from dateutil import parser as dateparser

from io import BytesIO
Expand Down
2 changes: 1 addition & 1 deletion ipfx/version.txt
Original file line number Diff line number Diff line change
@@ -1 +1 @@
1.0.2
1.0.3
1 change: 1 addition & 0 deletions requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ dictdiffer
h5py==2.10.0
marshmallow==3.0.0rc6
matplotlib>=1.4.3
methodtools
numpy>=1.15.4,<1.19.0
pandas>=0.25.1,<=0.25.3
pg8000
Expand Down

0 comments on commit 9a8681f

Please sign in to comment.