NeonGeckoCom / neon-llm-chatgpt

Other

0 stars 6 forks source link

Rename chat_gpt -> chatgpt #15

Closed NeonBohdan closed 4 months ago

NeonBohdan commented 4 months ago

Description

Issues

Other Notes

NeonKirill commented 4 months ago

@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 commented 4 months ago

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 commented 4 months ago

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