Inscopix importer - Githubissues

kushalkolar / MESmerize

Platform for Calcium Imaging analysis. DEPRECATED.

GNU General Public License v3.0

61 stars 14 forks source link

Inscopix importer #72

Closed kushalkolar closed 3 years ago

kushalkolar commented 3 years ago

Basic functionality works. Uses a SharedMemory buffer to load the .isxd file in an external subprocess. The main process can then access the SharedMemory buffer and copies it into a normal numpy array which is loaded into a ViewerWorkEnv.

The .isxd file has to be loaded in an external subprocess because importing the isx library also imports Qt stuff in the inscopix libs which is unavoidable. The Qt imports causes the Qt GUI to freeze and become unresponsive as soon as Mesmerize is launched.

Memory usage is quite high since the SharedMemory array needs to be copied. Might be possible to make this more efficient.

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging f4b63f0d7c99904ae5b32744244f82605f207b4e into 283d01266e0dcb60ad3c4b0b405fc04373bd74cc - view on LGTM.com

new alerts:

1 for Unused local variable
1 for 'import *' may pollute namespace

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging f654ffb4b5f7ab468a653960eb8d9b723b0b1339 into c1a13fb635de5323e27bcc7b670455ac7b017ca1 - view on LGTM.com

new alerts:

1 for Unused local variable
1 for 'import *' may pollute namespace

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging 86b95b1d00696ffe7ecd16275d749694cb6a9b0f into c1a13fb635de5323e27bcc7b670455ac7b017ca1 - view on LGTM.com

new alerts:

1 for Unused local variable
1 for 'import *' may pollute namespace

kushalkolar commented 3 years ago

merging, high RAM usage for now will optimize later