Closed camiloramirezgo closed 1 year ago
When we make the release, we will need to remove the sys.path.append("..")
line and the autoreloads. We will also need to update the paths as output_directory = '../example/results'
I spect the behaviour to be that the user will be able to just download the example notebook, install onstove with conda and run the notebook directly being in the same directory. So the ../example
won't be really required.
I want to "hide" the data download steps into a function that gets called with the country as input only. We can put this function on a utils.py
file in the example and import it. I think this will make the notebook cleaner and will avoid confusion among the participants about those download steps.
Thing done:
mask_all
and aling_all
methodsDataProcessor
creation to the data processing step, and left the definition of the output_directory
to the begining of the notebook
Fixed
Test it for bugs and make sure that the explaining text is correct