Integer as dict key

[noklam]

I get this error when using int as dictkey in config, is this being blocked for specific reason? If I remove the assert statement I can run successfully.

I encounter this error when I use facebookresearch/hydra, which depends on omegaconf

from omegaconf import OmegaConf
conf = OmegaConf.create( dict([(1, 2), (3, 4)]))
print(conf.pretty())  

File "C:\ProgramData\Anaconda3\lib\site-packages\omegaconf\dictconfig.py", line 35, in setitem
assert isinstance(key, str)
AssertionError

[omry]

As you have noted, integers are not supported as dictionary keys in OmegaConf.
This is something I will consider for the version after 2.0.0. which already adds a lot of new surface area.
As a temporary solution, do not use int keys (you convert them to string somehow, maybe like:

1_: foo
2_: bar

the keys should now be strings.
Definitely look at 2.0.0 (docs), it adds support for Enum as keys which may be even better for you. don't try 2.0.0 with stable Hydra, they are incompatible (Integration is still work in progress).

[noklam]

@omry Thanks for the quick response! Ok I guess I will try convert it into string for now.

[Jasha10]

I've submitted PR #454 related to this issue.
Before moving further with test coverage, here is a design question:
will we want to support other primitive key types besides int?
Currently (before the above PR), DictConfig keys can be one of
[Any, str, Enum, None]
and DictConfig values can be one of
[Any, str, float, bool, int, None, StructuredConfig].

[omry]

Let's move the discussion to the PR.

[Jasha10]

I think this can be closed now.

[omry]

Yup.