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

Rename chat_gpt -> chatgpt #15

Closed
wants to merge 1 commit into from
Closed

Rename chat_gpt -> chatgpt #15

wants to merge 1 commit into from

Conversation

NeonBohdan
Copy link
Collaborator

Description

Issues

Other Notes

@NeonKirill
Copy link
Collaborator

NeonKirill commented Mar 10, 2024

@NeonDaniel we need your review here, as we decided to move away completely from naming convention chat_gpt and use chatgpt in all places, as for now it sometimes have a naming conflicts across the system.

Places to adjust configs are:

  • RMQ ChatGPT user and its LLM vhost permission
  • Potentially some OVOS configs

@NeonBohdan FYI

@NeonDaniel
Copy link
Member

Are there existing references to chatgpt that this refactor is fixing? As noted, this is a breaking change for existing RMQ and container configs, so it would require duplicating some users/configuration to make the transition.

@NeonKirill
Copy link
Collaborator

Are there existing references to chatgpt that this refactor is fixing? As noted, this is a breaking change for existing RMQ and container configs, so it would require duplicating some users/configuration to make the transition.

agree, after some adjustments it seems like we are good to go with existing naming, so @NeonBohdan please close this PR

@NeonBohdan NeonBohdan closed this Mar 21, 2024
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.

3 participants