Enhance Covalent's ConfigManager
to become more centralized
#1432
venkatBala
started this conversation in
General
Replies: 1 comment
-
@cjao @santoshkumarradha @kessler-frost @wjcunningham7 @AlejandroEsquivel @FyzHsn |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
ConfigManager
class handles the core logic of managing Covalent's configuration (in-memory/on disk). It gets instantiated upon importing the module in any environment can the number of times the configuration gets either overwritten or opened during workflow execution is slightly convoluted. This discussion is aimed towards enhancing theConfigManager
to become a bit more centralized and consistent.Few issues encountered with the current way of doing configuration management
The very first time the dispatch is sent, the error noticed is that it fails and raises a connection error that dispatcher not found at
localhost:48008
. But if one is to re-dispatch the workflow again everything works properly and a user can expect their workflow to have been submitted to their self-hosted covalent server.This behaviour is non-intutive and warrants a thorough dissection of the way covalent is performing configuration updates.
Few ideas for enhancement:
config
CLI command can be augmented by providingset
andget
methods for various configuration options i.e.Beta Was this translation helpful? Give feedback.
All reactions