Added Support for online Models from Open Router with --online option#2
Added Support for online Models from Open Router with --online option#2ashishpawar517 wants to merge 8 commits into
Conversation
…0.33.0 dependencies
…ith structured prompts
…pping functionality
|
Thank you so much for contributing !! This is a great feature, I will review and change and get back to you either today or tomorrow. |
|
Thank you so much for taking the time to contribute to this project! I really appreciate your effort in adding online model support - this is definitely a feature that would be valuable. However, after reviewing the changes, I have a few concerns about the current approach: Issues with the current implementation:
Alternative approach I'd prefer:Instead of the ptn model set-custom
# Opens a TUI interface prompting user to enter any OpenRouter model name
# Saves it persistently as the selected model
# Future `ptn` commands use this custom model automatically
This would:
- Make custom models persistent (no need to specify each time)
- Provide a clean TUI interface for model input
- Minimize changes to existing core logic
- Follow the existing ptn model command pattern
Would you be interested in reworking the implementation with this approach? I'm
happy to discuss the technical details further.
Again, thank you for the contribution - the core idea is great, I just think we
can make the implementation cleaner! 🙏 |
So you can use any models from OpenRouter to query