Create example JSON config(s) about weather-tools HOT 6 CLOSED

Hi, I am new to open source and I would like to work on this issue. Please guide me further on what needs to be done.

from weather-tools.

Hey @pranay101 ! More information can be found about what configuration files are used for here

As outlined, Configuration files can be written in *.cfg or *.json. However the examples in the config directory only contains examples of *.cfg files.

It would be useful to have some *.json examples also. I suggest creating json versions of some of the cfg examples. These can then be tested locally to ensure the same behaviour is observed e.g.

weather-dl configs/era5_example_config_local_run.cfg --local-run
should work the same as,
weather-dl configs/era5_example_config_local_run.json--local-run

from weather-tools.

I like the idea of having JSON configs. It's worth noting that the JSON representation will not be as full featured as the ConfigParser. See: https://github.com/google/weather-tools/blob/main/weather_dl/download_pipeline/parsers.py#L72

I think we can still create JSON example configs; however, these will have to be more verbose for the time being.

from weather-tools.

xref: #48

from weather-tools.

Hey @pranay101 ! More information can be found about what configuration files are used for here

As outlined, Configuration files can be written in *.cfg or *.json. However the examples in the config directory only contains examples of *.cfg files.

It would be useful to have some *.json examples also. I suggest creating json versions of some of the cfg examples. These can then be tested locally to ensure the same behaviour is observed e.g.

weather-dl configs/era5_example_config_local_run.cfg --local-run should work the same as, weather-dl configs/era5_example_config_local_run.json--local-run

how should I structure the project then?. creating separate folders for .CFG and .JSON inside config. or should I place json file along with the cfg files.

from weather-tools.

I'm a fan of flat file structures. So, json next to cfg sounds good to me.

from weather-tools.