You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
_SKIP_VALIDATION=environ.get(_VALIDATION_ENV_FLAG, "false").lower() in {
However, an alternative option that would allow local extensibility is the following:
Extend the schemas to also allow mostly free form config sections for class specifications under particular new namespaces (e.g., the one listed above).
Create your new class under that namespace and pip install it locally or else just adjusts your PYTHONPATH
This would allow local configs and classes to be developed inside the local repo and outside of the main MLOS repo.
One challenge with this is that it would be more challenging to collect new Services and Environments into the main repo unless people submit a PR, but there would now be less incentive to do that.
Not sure I have a good solution to that.
Right now our configs mostly look like this:
Similar for Environments.
It should be possible to also load external classes as well, except that the schema checks reject them.
We have a hack around that using an environment variable currently:
MLOS/mlos_bench/mlos_bench/config/schemas/config_schemas.py
Line 31 in d50f1f4
However, an alternative option that would allow local extensibility is the following:
config
sections forclass
specifications under particular new namespaces (e.g., the one listed above).pip install
it locally or else just adjusts yourPYTHONPATH
This would allow local configs and classes to be developed inside the local repo and outside of the main MLOS repo.
One challenge with this is that it would be more challenging to collect new Services and Environments into the main repo unless people submit a PR, but there would now be less incentive to do that.
Not sure I have a good solution to that.
If we decide to do this:
(especially after Rework documentation generation #869)
The text was updated successfully, but these errors were encountered: