-
Notifications
You must be signed in to change notification settings - Fork 34
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
No curation ephys AND kilosort triggering #54
Closed
Closed
Changes from 1 commit
Commits
Show all changes
136 commits
Select commit
Hold shift + click to select a range
f740aef
added NWB export
042cc46
copied `ephys` to `ephys-no-curation`, added `recording_duration`, make
a39a9b1
remove `Curation`
590310e
make `clustering_output_dir` user-input optional, auto infer
adfad95
minor bug fix
b7c0845
improve kilosort loading routine - add `validate()` method
de84ce0
helper for `ProbeInsertion` - `auto_generate_entries(session_key)`
4397dd7
no-curation, store processed data in user-specified `processed_data_d…
5420ae0
using `find_full_path` for session dir as well - improve robustness
70e93b3
Update ephys_no_curation.py
a7e7554
bugfix
0d56456
automate ClusteringTask insertion
047bfa1
triggering Kilosort with ecephys_spike_sorting package
60105da
remove `_required_packages_paths`
a9326ea
minor stylistic improvements
149ef3f
Apply suggestions from code review
9f98195
typo fix
a24bd1a
Merge branch 'no-curation' of https://github.com/ttngu207/element-arr…
ba28637
handles a weird windows/unix path incompatibility (even with pathlib)
5cdc994
fix missing `clustering_method`
6c646bb
flag to create spike sorting output dir
dd01fd2
Update kilosort_trigger.py
f699ce7
minor tweak/improvements in kilosort triggering
6bf0eb1
bugfix in triggering ecephys_spike_sorting
6788180
handle cases where `fileTimeSecs` is not available
d63561f
bugfix
375e437
missing generate module json
c593baf
change default `noise_template_use_rf` to False
69c5e51
minor bugfix
a6cae12
add kilosort 2.5 as default content
a0ea9f7
make variable naming consistent
acdab12
include `clustering_method` into the calculation of `param_set_hash`
38c5be6
updating `kilosort_repository` depending on which KSVer to be used
28c4452
bugfix
94aade7
bugfix, convert path to string
ae966aa
log the folder creation
b9f4e92
add debugging lines
044c389
bugfix - no dir created
0f0c212
Update kilosort.py
67341d8
new version 0.1.0b1
c2ee64f
improve clusteringtask and waveform ingestion routine
dc7ddd9
minor cleanup
408532c
stage tracking and resumable kilosort run
55bec01
minor bugfix
f34e95d
bugfix logging for kilosort triggering
19b704b
duration for each module run
beaf765
check `ap.bin` file validity before triggering kilosort (based on fil…
74e3ead
Apply suggestions from code review
e8ffe17
stylistic improvements, addressing code review comments
d07f830
handle NWB export with multiple curated clusterings from one session
19616ef
Update nwb.py
09c1e60
minor bugfix
3ebdf23
Merge pull request #2 from ttngu207/nwb-export
b25f065
add nwb export to `no-curation` ephys
49ca0be
first prototype for running the ecephys_pipeline with OpenEphys
199a2ba
bugfix for running kilosort for Open Ephys data
58b5984
specify additional recording-info as part of the `params`
ddc3b94
add `neuropixels UHD` probe type
df599fb
triggering kilosort analysis for open-ephys
819ff19
first prototype for pykilosort
81d99c8
use_C_waves=False for OpenEphys
96931a4
using fork of pyopenephys
b6f8f99
minor bugfix in running pykilosort
4250220
update openephys loader
af2b18b
fix package requirement formatting error
e325a30
handles format differences between npx1 vs 3A
db3027b
supporting `neuropixels UHD` in `ephys.EphysRecording`
f55a6a7
bugfix in electrode sites design for Neuropixels UHD probe
d105419
search recording channels for Open Ephys based on channel names
0d16e7e
bugfix openephys loader
c213325
bugfix - extract recording datetime (instead of using experiment date…
adffe34
bugfix, timedelta as seconds
43d6614
handles extraction of `connected` channels for NP_PROBE format in Ope…
da10c66
clusters extraction - check `cluster_group.tsv` and `cluster_KSLabel.…
02069c9
bugfix in triggering pykilosort
142459d
handle missing `sample_rate` from pykilosort params.py
1c39568
for pykilosort's probe, provide both Nchan and NchanTOT
2022e91
openephys loader - handles signalchain and processor as single elemen…
02c4b67
handles multi-probe for older OE version
8dff08b
NWB export fix, specifying which ephys module
6107e8e
version bump
dae36d1
fix NWB export - null `waveforms` - back to version 0.1.0b1
8657d58
version bump
aa47c8a
include probe as part of the electrode_group name for uniqueness
31bba8c
bugfix in assigning unit electrode indices
ttngu207 f9a4754
Merge pull request #3 from ttngu207/nwb-export
f2881ca
all three ephys files up to date
A-Baji c56cd18
applied requested changes
A-Baji 9dd6b42
Merge pull request #4 from A-Baji/no-curation
4951b39
DEPRECATING NWB EXPORT
b4832ea
Merge branch 'main' of https://github.com/datajoint/element-array-eph…
d66368c
minor bugfix
779b2fb
name tweak
f4052cc
Apply suggestions from code review
03cab02
rename `sess_dir` -> `session_dir`
cb1a041
Update requirements.txt
7320f9f
Update requirements.txt
f0b3d4a
Update requirements.txt
bdcfa46
handle older open ephys format for single probe
83649f5
remove unintended prototyping work
c2f8aea
delete nwb export - rename `ephys` module -> `ephys_acute`
e0a9a4f
Update __init__.py
769de13
bugfix
39c8579
configurable `paramset_idx` for auto ClusteringTask generation
85c7c8b
Update openephys.py
11a12ba
update openephys loader - handle new open ephys format
cd5fe70
handles new probe naming in latest Open Ephys format
af864d7
Apply suggestions from code review
47babf3
added assertion - safeguard against failed loading of continuous.dat
7fecff1
Merge branch 'no-curation' of https://github.com/ttngu207/element-arr…
ddd4095
Merge remote-tracking branch 'upstream/run_kilosort' into no-curation
a578d85
Update requirements.txt
f4b917d
`kilosort2` also as part of the `contents` for ClusteringMethod
f11e016
bugfix for LFP electrode mapping
07604e2
update open ephys loader to handle "STREAM" in latest format
4e367d7
added loading of electrode location for new openephys format
67039ac
more robust loading of openephys format
9a59e57
improve logic for running kilosort modules in a resumable fashion
b97566e
code cleanup, minor bugfix
3f1ee37
bugfix, match new implementation for openephys
0c77826
improve kilosort calls, handle spaces in paths
b71b459
remove space escaping character
a3c5c2f
improve error message
fd331bd
Merge branch 'no-curation' of https://github.com/ttngu207/element-arr…
b2bd0ee
implement data compression using `mtscomp` for openephys and spikeglx…
97f3d21
garbage collect openephys objects
d8aea04
garbage collect openephys objects
f9f18d0
Update kilosort_triggering.py
5905392
bugfix in running catgt
987231b
enable catgt run
aed42ca
bugfix
fbdbe24
bugfix
35da39b
`missing_ok` arg only available in python 3.8+
c33d1b0
bugfix - path search for catgt output
f60ba3d
improve error handling
71d87ae
Update kilosort_triggering.py
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.