Jack Kelly

## Detailed Description Each example should include at least one PV system. These PV systems don't have to be in the center of the satellite imagery. This way, we get...

## Detailed Description When we create batches which are _just_ satellite imagery (from the entire extent of the imagery) then `SatelliteDataSource.get_locations` will need to be implemented. ## Possible Implementation Should...

## Detailed Description It used to be necessary to do this to load NetCDF quickly from a cloud storage bucket: ```python with fsspec.open(filename, mode="rb") as file: file_bytes = file.read() with...

## Detailed Description Ideally, each variable should only have a default value defined once in the code. ## Context At the moment, defaults (for filenames and other parameters) are scattered...

Rename `time_resolution_minutes` to `sample_period_minutes` to make the code consistent

1

## Detailed Description `model.py` uses `time_resolution_minutes`. Elsewhere in the code this same concept is called `sample_period_minutes`. ## Context Good to be consistent throughout the code.

## Detailed Description ESO have provided us with a list of datetimes when their national PV forecast was particularly bad. We should modify `nowcasting_dataset` to allow us to manually specify...

Backfill data from 2016 to 2020

1

Let's do this in early 2022 (for WP2). Related to #81

## Detailed Description I'm pretty certain that modern video compression (e.g. AV1) and/or image compression (e.g. AVIF) will result in _much_ smaller file sizes for our image data sources (satellite...

`PVDataSource` is ignoring 52 PV systems from the PassivSystems data... let's manually check if those 52 PV systems really should be removed :slightly_smiling_face:

1

Not urgent ``` 2021-11-17 17:55:06,391 DEBUG processID=1555225 Creating pv DataSource object. | /home/jack/dev/ocf/nowcasting_dataset/nowcasting_dataset/manager.py#L103 2021-11-17 17:55:06,393 DEBUG processID=1555225 Loading PV metadata from /mnt/storage_b/data/ocf/solar_pv_nowcasting/nowcasting_dataset_pipeline/PV/Passiv/ocf_formatted/v0/system_metadata_OCF_ONLY.csv | /home/jack/dev/ocf/nowcasting_dataset/nowcasting_dataset/data_sources/pv/pv_data_source.py#L72 DEBUG:nowcasting_dataset.data_sources.pv.pv_data_source:Loading PV metadata from /mnt/storage_b/data/ocf/solar_pv_nowcasting/nowcasting_dataset_pipeline/PV/Passiv/ocf_formatted/v0/system_metadata_OCF_ONLY.csv 2021-11-17...

After `rioxarray` 1.3 is released, remove `OpenData` from `TopographicDataSource.__post_init__()`

2

Right now, `rioxarray` can't open data directly from cloud storage buckets, so `TopographicDataSource.__post_init__()` downloads files to the VM's local hard disk using `OpenData`. But `rioxarray` v1.3 will enable `rioxarray` to...