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.

Sign up for GitHub

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

Jump to bottom

Define csv separator #423

Closed
tormento opened this issue Sep 23, 2024 · 1 comment · Fixed by #429
Closed

Define csv separator #423

tormento opened this issue Sep 23, 2024 · 1 comment · Fixed by #429
Labels
feature
Milestone
0.6.5

Comments

@tormento
Copy link

Not every region uses "," as csv separator.

The solution is to add

sep=,

as first line in the csv file, so that it can be directly opened with Excel in any region.

A proper flag in config file would be useful.
@Breakthrough
Copy link
Owner

Good idea, happy to add this. This probably should be done for timecodes as well at some point.

@Breakthrough Breakthrough added this to the 0.6.5 milestone Sep 24, 2024
Breakthrough added a commit that referenced this issue Sep 30, 2024
@Breakthrough
[cli] Add ability to customize delimiters for CSV output (#423)
74b4747
Breakthrough added a commit that referenced this issue Sep 30, 2024
@Breakthrough
[cli] Add ability to customize delimiters for CSV output (#423)
9be43ba
@Breakthrough Breakthrough mentioned this issue Sep 30, 2024
Merged
3 tasks
@Breakthrough Breakthrough linked a pull request Sep 30, 2024 that will close this issue
[scene_manager] Allow setting delimiters for CSV output #429
Merged
3 tasks
Breakthrough added a commit that referenced this issue Sep 30, 2024
@Breakthrough
[cli] Add ability to customize delimiters for CSV output (#423)
dc77965
Breakthrough added a commit that referenced this issue Sep 30, 2024
@Breakthrough
[cli] Add ability to customize delimiters for CSV output (#423)
7a81de2
Breakthrough added a commit that referenced this issue Oct 19, 2024
@Breakthrough
[cli] Add ability to customize delimiters for CSV output (#423)
8a83222
Breakthrough added a commit that referenced this issue Oct 20, 2024
@Breakthrough
[cli] Add ability to customize delimiters for CSV output (#423)
834db38
Breakthrough added a commit that referenced this issue Nov 13, 2024
@Breakthrough
[cli] Add ability to customize delimiters for CSV output (#423)
482c3ef
Breakthrough added a commit that referenced this issue Nov 20, 2024
@Breakthrough
[scene_manager] Allow setting delimiters for CSV output (#429)
e1d86d9 
* [cli] Add ability to customize delimiters for CSV output (#423)

* [cli] Add tests for CSV delimiters and enforce constraints for delimiter lengths
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
0.6.5
Development

Successfully merging a pull request may close this issue.

[scene_manager] Allow setting delimiters for CSV output Breakthrough/PySceneDetect
2 participants
@Breakthrough @tormento