JJguri / bestiapop

Python package to automatically generate gridded climate data to be used as input for crop models
Other
13 stars 6 forks source link

Help complete documentation #31

Open darkquasar opened 4 years ago

darkquasar commented 4 years ago

Hey @JJguri, I've re-structured the documentation in README and index.rst to reflect the new functionalities of the package. There are some sections though that need further touches or information to be expanded. I've flagged these with [yoni to expand] comments in the files.

Please check and action as required, if no further info is needed, then delete the comments. It would be good if you could also make sure there is cohesion in the text.

JJguri commented 4 years ago

@darkquasar I already updated the readme file, I will leave the update of the index file in your hands. Please add a description of the visualization of BestiaPop running, I have added a new section called BestiaPop in action! for this purpose.

darkquasar commented 4 years ago

I will create a gif of BestiaPop running, that always catches the eye.

Regarding the performance tests, it would seem that multiprocessing is always slower than the alternative, which seems kind of odd, and would render multiprocessing kind of useless. Can I ask you to run the tests but instead of testing with 36 lat/lon combinations to increase that to at least 100 combinations and at least 5 years instead of 1. So this 36 lat-lon combinations (36 files) for a single year (36f1y) would be 100 lat-lon combinations (100 files) for 5 years (100f5y). This is because of the way that BestiaPop fetches data from the cloud APIs, multiprocessing should be more effective the more lat/lon combinations you request since each combination is a call to cloud APIs.

darkquasar commented 4 years ago

@JJguri it would be great if you could also have a go at updating index.rst. I can jump in after you've tried to add the required content and help if things don't go well. Here are some pointers:

Some basics

Instead of using # for H1, use= underlining the title. The quantity of = symbols needs to be as long or more than the length of the title being underlined. Example:

This is my Title
================

Instead of using ## for H2, use ~ underlining the title. Instead of using ### for H3, use + underlining the title. For links, instead of doing this [my link](https://mylink.com) you need to do this

`My Link <https://mylink.com>`_ 
JJguri commented 4 years ago

I will create a gif of BestiaPop running, that always catches the eye.

Regarding the performance tests, it would seem that multiprocessing is always slower than the alternative, which seems kind of odd, and would render multiprocessing kind of useless. Can I ask you to run the tests but instead of testing with 36 lat/lon combinations to increase that to at least 100 combinations and at least 5 years instead of 1. So this 36 lat-lon combinations (36 files) for a single year (36f1y) would be 100 lat-lon combinations (100 files) for 5 years (100f5y). This is because of the way that BestiaPop fetches data from the cloud APIs, multiprocessing should be more effective the more lat/lon combinations you request since each combination is a call to cloud APIs.

@darkquasar please have a look at the updated table for performance in the documentation. You were right, the use of -m is recommended when the lat-long combinations are high.