-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Brainstorm potential Fire CLI configurations #98
Comments
Possible entry points for configs:
|
Just came here to say that I would very much appreciate if this was possible:
I would suggest going with single dash, as that seems to be customarily used in many *nix tools. |
How about adding type hints to the functions/classes passed to fire? So I could have import fire
def use_gpu(gpu: bool = True):
pass
fire.Fire(use_gpu) and then |
#231 suggests being able to disable the printed output for use cases where the developer wants to do something with the returned component from Fire. |
Hi, I would like to vote for the feature Allow passing arguments via stdin The syntax could be yaml or json. Yaml is more flexible, for example it automatically parses date and has syntax for using custom data type. Json is faster. The usecases could be chaining output of one to input of other, but using all power of unix with tools for cli json editing like I suggest two ways of implementation
https://github.com/DaniloZZZ/mongocat/blob/master/mongocat/mongocat.py#L32 In the case of fire we can have our method of interest instead of
(maybe I should add an issue for this feature?) |
I think there are actually two different features:
|
Came here to ask about My use case is: Where but it could also be called like this:
|
I'm starting a list of things people might want to configure in their Fire CLIs
By no means do we plan on supporting all of these; just good to keep a list.
The primary reason for making this list is so that as we start adding support for these, we can do so in a way that's forward looking. We want any configs that we introduce to look and feel consistent with each other, and not to later need to be renamed or removed for consistency with other configs added at a later date.
The text was updated successfully, but these errors were encountered: