-
Notifications
You must be signed in to change notification settings - Fork 15
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
probably bug in common #165
Comments
this line you have referenced
is using are you wanting it to be the unix time that gets saved at the end of the batch file when being saved to disk? |
I'm pretty sure it's just time(), you can check the git history of that
line
…On Fri, Mar 10, 2023, 10:36 Caitlin Lewis ***@***.***> wrote:
this line you have referenced
bak = path.with_suffix(path.suffix + f"bak.{time()}")
is using datetime.time() method I think as opposed to unix time from time
module
are you wanting it to be the unix time that gets saved at the end of the
batch file when being saved to disk?
—
Reply to this email directly, view it on GitHub
<#165 (comment)>,
or unsubscribe
<https:/notifications/unsubscribe-auth/ACHXXRHEODMXXIDD66YPUQLW3NC7XANCNFSM6AAAAAAUEN2GWU>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
And then find the commit that it first appeared to check the import
…On Fri, Mar 10, 2023, 10:46 Kushal K ***@***.***> wrote:
I'm pretty sure it's just time(), you can check the git history of that
line
On Fri, Mar 10, 2023, 10:36 Caitlin Lewis ***@***.***>
wrote:
> this line you have referenced
>
> bak = path.with_suffix(path.suffix + f"bak.{time()}")
>
> is using datetime.time() method I think as opposed to unix time from time
> module
>
> are you wanting it to be the unix time that gets saved at the end of the
> batch file when being saved to disk?
>
> —
> Reply to this email directly, view it on GitHub
> <#165 (comment)>,
> or unsubscribe
> <https:/notifications/unsubscribe-auth/ACHXXRHEODMXXIDD66YPUQLW3NC7XANCNFSM6AAAAAAUEN2GWU>
> .
> You are receiving this because you authored the thread.Message ID:
> ***@***.***>
>
|
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
time
is not imported fromtime
https:/nel-lab/mesmerize-core/blob/v0.1.0/mesmerize_core/caiman_extensions/common.py#L152
The text was updated successfully, but these errors were encountered: