alan-turing-institute / dymechh

repository for the DyME-CHH project
Other
3 stars 0 forks source link

[WIP] This adds a script which converts .nc files to .tif and also re projects them to BNG #31

Open AoifeHughes opened 1 year ago

AoifeHughes commented 1 year ago

Script needs a little work to make more user-friendly

review-notebook-app[bot] commented 1 year ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

AoifeHughes commented 1 year ago

On my mac this took about 2 seconds per file to convert and project. So, I'm kinda worried I've done something wrong. I'll want to double, triple check everything first. Current plan is to use QGIS to test open the files, or if there's other software people suggest using?

crangelsmith commented 1 year ago

I'm struggling to run the script on my machine. I'll try to fix it and add an environment file. Changing this to WIP for now.

AoifeHughes commented 1 year ago

Hmm strange @crangelsmith, I'll be back on this on Monday, did you get any specific errors or did it just not do anything?

Either way, I'll sort the environment stuff and make sure a minimal example is runnable for this next week.

AoifeHughes commented 1 year ago

I added an environment for conda in: c97f4b3

I run something like this from the root directory of the project.

python scripts/reproject_all.py raw_data output_folder
crangelsmith commented 1 year ago

After a lot of debugging with @AoifeHughes I've finally managed to run this on my laptop and the new environment. Before creating the conda environment I had to:

brew install hdf5 netcdf

Before merging it will be good to have the following:

AoifeHughes commented 1 year ago

Confirming steps to have this run and be tested on Azure

Issues that came up: