Jasha Sommer-Simpson

Results 164 comments of Jasha Sommer-Simpson

Feature idea: metadata option to omit field

Hi @bottler, I'd like to get a better feel for how you are using structured configs. How do you translate between `DictConfig` instances and dataclass instances? Are you using `OmegaConf.to_object`?...

Feature idea: metadata option to omit field

Adding a check for `metadata["omegaconf_ignore"]` seems reasonable to me. I'll bring this up with the rest of the OmegaConf dev team next week to see if there is consensus. One...

Follow up on deprecation of `register_resolver()`

See also [this comment on #426](https://github.com/omry/omegaconf/issues/426#issuecomment-1162375875); we need to revise the deprecation schedule outlined in that issue.

fails on `Dict[Any, str]` if passed in a list of strings

In OC 2.1, nested containers do not have full runtime type safety. This means dicts-of-lists were not type-checked deeply. OC 2.2's nested container typing was implemented by PR #890. To...

Would you please consider realizing an comparing funciton like "Differ.compare()"?

Thanks for the feature request, @songyuc. Here is a starting point: ```python from omegaconf import DictConfig, OmegaConf def differ(cfg1: DictConfig, cfg2: DictConfig): keys1 = cfg1.keys() keys2 = cfg2.keys() extras1 =...

Set and Sequence in structured

Hi @bottler, Thanks for the report. For `typing.Sequence`: with v2.1.2 I'm getting `out.ii == [3, 43]` and with v2.2.2 I'm getting a `ConfigTypeError` (as you described) For `typing.Set`: with both...

Set and Sequence in structured

> `inspect.isclass(ref_type) and issubclass(ref_type, Enum)` - to avoid this errror from all manner of things. Great idea -- thanks!

Add a Get_leaves_str() function for printing all the leave values in an OmegaConf object

Cross-linking to related discussion: https://github.com/omry/omegaconf/discussions/961

`get_attr_data` doesn't handle default factory correctly

Thanks for the bug report!

```OmegaConf.to_yaml``` adds unexpected new lines and question mark to a long string

The short answer is "no, OmegaConf does not currently support disabling this behavior". The longer explanation is that OmegaConf uses [`pyyaml`](https://pyyaml.org/) as a backend for loading and dumping yaml. Adding...