Skip to content
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

fixed conversion factor and reordered builder #530

Open
wants to merge 237 commits into
base: master
Choose a base branch
from

Conversation

lfrank
Copy link
Collaborator

@lfrank lfrank commented Aug 28, 2020

No description provided.

lfrank and others added 30 commits August 27, 2020 20:02
these lines are duplicates of preceding lines 268, 274 & 290.

they were trying to create keys that already exist, raising errors.
that was missing in the latest update
previously one would get error at write time,
only after contents are built (which takes a while)
* add new attribute preprocessing_path in builders

* copy xml headers to the preprocessing path, if specified

the preprocessing path can be different from raw data path

* pass preprocessing_path to extract_trodes_rec_file

so that preprocessed binaries are saved to preprocessing_path.

this depends on rec_to_binaries update (pending pull request at the
moment)

* pass dates to preprocess selectively

* fix dependency: require rec-to-binaries updated version
no change to existing user interface
- allow separate preprocessing_path
- reorder video files originator in build()
- use a dummy default value for session_start_time
- timestamps are not available in this case; fill with dummy values
- timestamps file should be treated differently
- should add timestamp conversion in future updates
(required method retrieve_real_timestamps)
edeno and others added 30 commits April 28, 2023 12:19
Co-authored-by: Eric Denovellis <[email protected]>
save multiple LED position data as separate spatialSeries
Fix error caused by Controller_Din headstage channels
Use ptp to detect pause time if dio video stamps unavailable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.