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

Potential Errors: analysis_file name #961

Closed
samuelbray32 opened this issue May 8, 2024 · 1 comment · Fixed by #970 or #973
Closed

Potential Errors: analysis_file name #961

samuelbray32 opened this issue May 8, 2024 · 1 comment · Fixed by #970 or #973
Labels
question Further information is requested

Comments

@samuelbray32
Copy link
Collaborator

samuelbray32 commented May 8, 2024

Related to #960

PR #937 altered when the key analysis_file_name is added in some make functions. I haven't tested, but here are a few places that might affect ability to fetch needed data from upstream tables. Recording here to avoid redundant debugging:

  • PositionDLCOrient.make()
  • DLCSmoothInterp.make()
  • DLCPosV1.make()
  • TrodesPosV1().make()
@samuelbray32 samuelbray32 added the question Further information is requested label May 8, 2024
@CBroz1
Copy link
Member

CBroz1 commented May 8, 2024

For problematic cases, the moved create can be moved back and replaced with the following to set the start time without creating the file

AnalysisNwbfile()._creation_times["pre_create_time"] = time()

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
2 participants