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

[feature] stdout writing #37

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

jamesb93
Copy link
Member

@jamesb93 jamesb93 commented Sep 16, 2022

Adds support for setting the output to stdout.

Interface isn't great and is not very "UNIX"y, however, it works.

For example, you can specify output like this:

fluid-noveltyfeature -source infile.wav -features stdout

This will write the values out with commas inbetween them.


Why?

The major benefit of this format is that you don't need to manage, cleanup and parse an intermediary file if you have the CLI tools embedded elsewhere. My selfish reason is to remove those needs from ReaCoMa. It would also facilitate viewing results more easily in the terminal if you want to prototype there fast.

@jamesb93 jamesb93 requested a review from weefuzzy September 16, 2022 15:53
@weefuzzy
Copy link
Member

That seems relatively painless. Would suggest using -- as the marker for using std out, to be more unixy. And we should think about whether we'd want to enable input from stdin for pipey goodness.

@jamesb93
Copy link
Member Author

That seems relatively painless. Would suggest using -- as the marker for using std out, to be more unixy. And we should think about whether we'd want to enable input from stdin for pipey goodness.

stdin is an idea! I think it would let you do, for example, ffmpeg -> flucomathing. In terms of using -- do you mean instead of stdout? AFAIK, and I am likely to be wrong here, -- signifies "don't read any more args" rather than stdout.

@jamesb93
Copy link
Member Author

That seems relatively painless. Would suggest using -- as the marker for using std out, to be more unixy. And we should think about whether we'd want to enable input from stdin for pipey goodness.

stdin is an idea! I think it would let you do, for example, ffmpeg -> flucomathing. In terms of using -- do you mean instead of stdout? AFAIK, and I am likely to be wrong here, -- signifies "don't read any more args" rather than stdout.

Ah yes, but I see how it makes sense. Like if you want to tell git diff that foo is a path and not a commit with git diff -- foo. So -- kinda means, "no output file".

@weefuzzy weefuzzy changed the base branch from dev to main February 21, 2023 09:05
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.

2 participants