Value always interpreted as string in YAML data when using key other than Str() for MapPattern

[dgg5503]

After pulling strictyaml 1.0.2, I can see that the exception is no longer thrown. However, I'm running into a strange issue when viewing the parsed YAML data dictionary values. Here's an example:

>>> from strictyaml import *
>>> a = {10.25: 23, 20.33: 76}
>>> schema = MapPattern(Float(), Int())
>>> yaml = as_document(a, schema)
>>> yaml.data
OrderedDict([(10.25, '23'), (20.33, '76')])

Notice that the keys are correctly interpreted as floats, however, the values are strings instead of the expected ints.

For contrast, here is what happens if I make the keys strings:

>>> from strictyaml import *
>>> a = {'10.25': 23, '20.33': 76}
>>> schema = MapPattern(Str(), Int())
>>> yaml = as_document(a, schema)
>>> yaml.data
OrderedDict([('10.25', 23), ('20.33', 76)])

Now the values are interpreted correctly as ints.

Environment:

• Python 3.7.3
• strictyaml 1.0.2
• ruamel.yaml 0.15.97

[crdoconnor]

Ouch, that's embarrassing. Thanks for reporting I'll try to roll out a fix asap.

…

On Fri, 28 Jun 2019, 04:35 Douglas, ***@***.***> wrote: After pulling strictyaml 1.0.2, I can see that the exception is no longer thrown. However, I'm running into a strange issue when viewing the parsed YAML data dictionary values. Here's an example: >>> from strictyaml import * >>> a = {10.25: 23, 20.33: 76} >>> schema = MapPattern(Float(), Int()) >>> yaml = as_document(a, schema) >>> yaml.data OrderedDict([(10.25, '23'), (20.33, '76')]) Notice that the keys are correctly interpreted as floats, however, the values are strings instead of the expected ints. For contrast, here is what happens if I make the keys strings: >>> from strictyaml import * >>> a = {'10.25': 23, '20.33': 76} >>> schema = MapPattern(Str(), Int()) >>> yaml = as_document(a, schema) >>> yaml.data OrderedDict([('10.25', 23), ('20.33', 76)]) Now the values are interpreted correctly as ints. Environment: - Python 3.7.3 - strictyaml 1.0.2 - ruamel.yaml 0.15.97 — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#64?email_source=notifications&email_token=ABOJKNO4LQJHOLMQUHGM7XDP4UCA5A5CNFSM4H37CUK2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G4EZTKQ>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABOJKNI33R54SRX7IULOX73P4UCA5ANCNFSM4H37CUKQ> .

[smsearcy]

I'm seeing similar behavior with MapPattern(Int(), CommaSeparated(Decimal())), the values are being returned as strings. It seems this is likely the same issue?