Closed Christopher-R-Perkins closed 4 months ago
The update introduces new type hint classes and refines several methods to enhance the handling of store configurations and server information. These improvements aim to provide more structured and detailed data for configurations and responses within the application.
Files | Change Summary |
---|---|
bundled/tool/type_hints.py |
Added new type hint classes: ZenmlStoreInfo , ZenmlStoreConfig , ZenmlServerInfoResp , and ZenmlGlobalConfigResp . |
bundled/tool/zenml_wrappers.py |
Expanded imports, updated methods for setting store configuration, and improved data structure for server and global configurations. |
In fields of code where data flows,
New types emerge as wisdom grows.
Stores and servers speak so clear,
Configs refined, no need to fear.
A rabbit hops with joy and grace,
For changes made, improving pace.
🌟🚀
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
This pull request contains changes to fix an issue with Pydantic not being compatible with a zenml library call at the moment. Original code was developed when zenml used pydantic 1.x, but currently uses 2.x and causes problems with retrieving server information.
Changes:
json()
from object generated byzenml.config.global_config.GlobalConfiguration().zen_store
andzenml.config.global_config.GlobalConfiguration().get_store_info()
which would cause an error in thegetServerInfo
command (ZenServerWrapper.get_server_info
method). Instead now builds the dict manually for the response instead of converting to json and back to a dict.getGlobalConfig
command (GlobalConfigWrapper.get_global_configuration
), though this is never called in the entire extension, just to be safe by not using the problematic function.Summary by CodeRabbit