-
Notifications
You must be signed in to change notification settings - Fork 48
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
FractalComputeConfig
inherit from BaseSettings
instead of BaseModel
#729
Comments
For now I am using the following patch: hadim@494a78a It works with the below config for example: export QCF_COMPUTE_BASE_FOLDER=/tmp/qcf_compute
export QCF_COMPUTE_CLUSTER=manager_demo_1
export QCF_COMPUTE_LOGLEVEL=INFO
export QCF_COMPUTE_UPDATE_FREQUENCY=30
export QCF_COMPUTE_SERVER_FRACTAL_URI=http://127.0.0.1:7777
export QCF_COMPUTE_SERVER_USERNAME=compute_user
export QCF_COMPUTE_SERVER_PASSWORD=compute_password
export QCF_COMPUTE_SERVER_VERIFY=false
export QCF_COMPUTE_ENVIRONMENTS_USE_MANAGER_ENVIRONMENT=true
export QCF_COMPUTE_ENVIRONMENTS_CONDA='[]'
export QCF_COMPUTE_ENVIRONMENTS_APPTAINER='[]'
export QCF_COMPUTE_EXECUTORS='{"local": {"type": "local", "queue_tags": ["demo"], "cores_per_worker": 16, "memory_per_worker": 16, "max_workers": 1}}' The only thing that is not ideal is |
I had to add another patch to make it works (and the tests to pass): hadim@ee83118 Also somehow I also need to set
It works well for me. |
FractalComputeConfig
inherit from BaseSettings
instead of BaseModel
FractalComputeConfig
inherit from BaseSettings
instead of BaseModel
The main reason would be to allow parsing env variables instead of providing a config YAML file (more convenient in a container-based env for example).
The text was updated successfully, but these errors were encountered: