test: save hparams to yaml

[Borda]

What does this PR do?

loading Attrib params from YAML

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

[Borda]

the cases are failing on minimal requirements... @omry any idea?

>       save_hparams_to_yaml(path_yaml, OmegaConf.create(hparams))

tests/models/test_hparams.py:428: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

data = {'batch_size': 32, 'data_root': './any/path/here', 'learning_rate': 0.001, 'nasted': {'any_num': 123, 'anystr': 'abcd'}}
stream = None, Dumper = <class 'yaml.dumper.Dumper'>
kwds = {'allow_unicode': True, 'default_flow_style': False, 'sort_keys': False}

    def dump(data, stream=None, Dumper=Dumper, **kwds):
        """
        Serialize a Python object into a YAML stream.
        If stream is None, return the produced string instead.
        """
>       return dump_all([data], stream, Dumper=Dumper, **kwds)
E       TypeError: dump_all() got an unexpected keyword argument 'sort_keys'

[mergify]

This pull request is now in conflict... :(

[codecov]

Codecov Report

Merging #2198 into master will increase coverage by 0%.
The diff coverage is 91%.

@@          Coverage Diff           @@
##           master   #2198   +/-   ##
======================================
  Coverage      88%     88%           
======================================
  Files          68      68           
  Lines        5249    5248    -1     
======================================
+ Hits         4600    4603    +3     
+ Misses        649     645    -4