Josh-XT / AGiXT

AGiXT is a dynamic AI Agent Automation Platform that seamlessly orchestrates instruction management and complex task execution across diverse AI providers. Combining adaptive memory, smart features, and a versatile plugin system, AGiXT delivers efficient and comprehensive AI solutions.
https://AGiXT.com
MIT License
2.6k stars 346 forks source link

Create Update-Timezone-AGIXT.sh #1212

Closed AliveDedSec closed 2 months ago

AliveDedSec commented 2 months ago

Create Update-Timezone-AGIXT.sh

Josh-XT commented 2 months ago

Hello! I appreciate the effort, but the docker-compose files do not need modified to accomplish this. You can simply add a line to your .env file with your time zone.

For example:

TZ=America/New_York
AliveDedSec commented 2 months ago

Привет! Я ценю усилия, но docker-composeдля этого файлы не нужно модифицировать. Вы можете просто добавить строку в свой .env файл с вашим часовым поясом.

Например:

TZ=America/New_York

But why can’t this happen automatically then? Do you think everything needs to be done manually? Why do I need to manually edit the files every time? You yourself are a supporter of process automation. Or do you think there is only one time zone, America? I did this in such a way as not to interfere with the main program code, and get the current server's time zone automatically, and set it. If you want a proper solution, you need to add similar functionality to the main AGiXT.sh script, AGiXT.ps1. So that the user does not even think about this issue, and everything happens automatically. I don't understand your stubbornness on this issue, I just don't want to manually edit the files every time I update or reinstall AGIXT. Why do I need these complications as an ordinary user? I made an option to the best of my qualifications, which definitely does not disrupt the operation of AGIXT and is not implemented into its code. If you can do it better and more gracefully, then do it yourself, I will be only glad. You claim to welcome any positive contribution to AGIXT, yet immediately resist adding new features when all you have to do is accept them. What should I do then? I can take the time to build this functionality into the main AGiXT.sh script, but is it worth doing, what if you again say that this is not necessary? Then you can make a code that adds a line with the server time zone to the .env file. But again, I already doubt that you will approve of this, and it turns out that I will waste time and effort in vain.

Josh-XT commented 2 months ago

You would only have to set the environment variable once, but since you know the more proper places and way to set this, I would say that you are correct that adding to AGiXT.sh and/or AGiXT.ps1 would be more correct than a new file if this was to be automated away. For anyone on Linux, this is already automated because Linux sets the env var TZ and the container would inherit it.

AliveDedSec commented 2 months ago

For anyone on Linux, this is already automated because Linux sets the env var TZ and the container would inherit it. No, you're wrong. What you describe does not happen to me. Otherwise, I wouldn’t bother with this and wouldn’t create this script. Manjaro Linux latest version