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

I can't use vision command with same session #6

Open
afr-ux opened this issue Jan 7, 2025 · 4 comments
Open

I can't use vision command with same session #6

afr-ux opened this issue Jan 7, 2025 · 4 comments
Labels
enhancement New feature or request

Comments

@afr-ux
Copy link

afr-ux commented Jan 7, 2025

I am using:
echo "This is an image to describe" | chatgpt-cli vision -f 1.jpg -c afr.cfg

In afr.cfg file, I have:
API_KEY=sk-my-secret-key
TEMPERATURE=0.5
SESSION_FILE=afr.json

I can't use session file with vision command. In afr.json nothing is added when I use vision command.
I have chatgpt-clie v2.1.1 in Windows 10-64 bits.
This tool is wonderfull for describe images for blind people but I would like record session to teach some things in images and chatGPT remember it in other images.
Thanks.

@afr-ux
Copy link
Author

afr-ux commented Jan 14, 2025

I tested v2.1.2 and is same. I can't use record session with vision command in Windows. Please, @duanemay, if I can help you, you can ask me. Thank you a lot.

@duanemay
Copy link
Owner

I guess I inadvertently added the session file as an option to the vision command, but did not plan to support that. The reason being that the encoded image data would be included in those files and they could get rather large. I could add the feature, knowing they will be larger than the original image, probably should not create the files by default or remove this unused option.

PRs are welcome.

@afr-ux
Copy link
Author

afr-ux commented Jan 19, 2025 via email

@duanemay duanemay added the enhancement New feature or request label Jan 21, 2025
@duanemay
Copy link
Owner

The use case you shared makes it clear what you are trying to use this for, and makes sense that the feature is needed for multiple images. I'll look into adding this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants