Skip to content

Conversation

@ArDiouscuros
Copy link
Contributor

Allow save fetched commands to a file and using wildcards
Keep the same behavior if just one file is selected
Replay command to get commands from file inside of interactive prompt
Related to #871

E.g.
dream> !fetch outputs\selected-imgs*.png

File outputs\selected-imgs\commands.txt with commands created

Edit the commands file adjust the parameters

dream> !replay outputs\selected-imgs\commands.txt
...

Allow save fetched commands to a file Replay command to get commands from file inside of interactive prompt
Related to invoke-ai#871
Copy link
Collaborator

@lstein lstein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a nice feature. Could I ask you to make a few minor modifications?

  1. Extend the syntax of the !fetch command so that you can specify the file to write the prompt commands out to. If the file is not specified, you can default to commands.txt as you do now.
  2. Before each command in commands.txt would it be possible to create a #comment that indicates the name of the file that the following command was extracted from?
  3. Document the extended !fetch and !replay commands in docs/features/CLI.md
  4. If you are brave, add !replay to the autocomplete commands in ldm/dream/readline.py
    Thanks!

-- !fetch takes second optional argument name of the file to save commands to
@ArDiouscuros ArDiouscuros requested a review from lstein October 3, 2022 08:40
@ArDiouscuros
Copy link
Contributor Author

Requested changes was implemented, but on Windows I could not see effects of autocomplete commands

Copy link
Collaborator

@lstein lstein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm approving this, but there are a few conflicts that need to be resolved. I will try to do so today.

lstein added a commit that referenced this pull request Oct 21, 2022
- This merges PR #882

Coauthor: ArDiouscuros
@lstein lstein merged commit 55db9db into invoke-ai:development Oct 21, 2022
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