Skip to content
This repository has been archived by the owner on Dec 11, 2022. It is now read-only.

how to parse logged .json files? #159

Open
zach-nervana opened this issue Dec 13, 2018 · 3 comments
Open

how to parse logged .json files? #159

zach-nervana opened this issue Dec 13, 2018 · 3 comments
Labels
priority/p3 enhancements not currently in focus or low impact bugs
Projects

Comments

@zach-nervana
Copy link
Contributor

The json files stored in the experiments directory are not valid json objects. Is there anything preventing us from using valid json? Is there a method available for loading this custom file type?

@zach-nervana
Copy link
Contributor Author

Alternatively:

  • this log could be stored in multiple files
  • we could switch to yaml which can be generated incrementally and validly while only appending to the existing file. With json, this is not possible which is what gets us into this weirdness using that serialization type.
@gal-leibovich
Copy link
Contributor

I don't see any reason not to move to YAML.
Usually, we would have just looked at the files manually, not using any tool, in order to compare a set of hyper-parameters between two or more experiments. This was quite inefficient, but didn't bother us too much as it wasn't something being done too often.

I like the idea of storing in multiple files, as it will allow storing hyper-parameters of different agents (e.g. in different hierarchies) and/or different constructs in different files and will thus allow for better readability. I think we just need to watch out, for not splitting to too many different files, as this has the potential to also become cumbersome.

@scttl scttl added this to To do in Coach Dev Jan 10, 2019
@zach-nervana
Copy link
Contributor Author

A few notes after looking into this:

@galnov galnov added the priority/p3 enhancements not currently in focus or low impact bugs label Jan 16, 2019
@scttl scttl moved this from Requires Grooming to Groomed but Not Started in Coach Dev Jan 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
priority/p3 enhancements not currently in focus or low impact bugs
3 participants