CoreDescriptors and CoreSchedules do not use a transparent hasher for keys

[seadanda]

Both use Twox256 instead of e.g. Twox64Concat, obfuscating the value of the keys.

from a conversation here

Suggest using either Twox64Concat or Blake2_128Concat for the keys

[bkchr]

If you don't need to iterate the keys, there is no need to use a reconstructable hash.

[seadanda]

I think about it the other way round: why use one that isn't reconstructable without reason?
In this case I get that we don't need to iterate them, but I think UIs and anybody reading the state would find it useful to have readable keys.

Happy either way, especially since this is already live and would be a breaking change to change it. I could easily be convinced it's not worth the change, but it's definitely a minor inconvenience.

[eskimor]

Good catch @seadanda ! Will be fixed as part of #1312 . ( I am changing the storage there anyway)