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

add timestamp to filename of .sdriq file #170

Closed
beta-tester opened this issue May 7, 2018 · 5 comments
Closed

add timestamp to filename of .sdriq file #170

beta-tester opened this issue May 7, 2018 · 5 comments
Assignees
Milestone

Comments

@beta-tester
Copy link
Contributor

beta-tester commented May 7, 2018

it would be great, when each recording to file would get a unique file name each time you press record.
it happend to me very often that i override an existing recorded sdrif file by accident, because each time sdrangel is using the same file name.

maybe there is a reason for that current behavior, but there is also a reason to do not override files or better create unique files each time you make a recording.
so maybe make it optionional to add a timestamp to the filename.

@beta-tester beta-tester changed the title add timestamp to filename of recording sdriq file add timestamp to filename of .sdriq file May 7, 2018
@f4exb
Copy link
Owner

f4exb commented May 7, 2018

So far the file record feature is very crude with not even the choice of the file name.

@f4exb f4exb added this to the v3.14.6 milestone May 9, 2018
@f4exb f4exb self-assigned this May 9, 2018
@beta-tester
Copy link
Contributor Author

beta-tester commented May 10, 2018

thank you very much for the changed file name - this helps alot.

but guess the ':' (collon) in the filename (yyyy-MM-dd HH:mm:ss) will be an issue for windows file systems. a collon is not allowed - is only allowed for drive letters (like C:\...)

maybe use "yyyy-MM-dd HHmmss" instead.

@f4exb
Copy link
Owner

f4exb commented May 11, 2018

This is why I like Windows so much 😄 The format yyyy-MM-ddTHH:mm:ss is the ISO format for a date/time stamp. OK to remove the colons ...

@beta-tester
Copy link
Contributor Author

beta-tester commented May 11, 2018

oops...
i thought i could help a little bit with doing some easy stuff instead of wasting your time...
by doing pullrequest ...
but i was too slow.
keep it or dump it... i am happy with any choice.

@f4exb
Copy link
Owner

f4exb commented May 11, 2018

kept!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants