-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SEVIRI Wildfire Notebook #12
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
View / edit / reply to this conversation on ReviewNB acocac commented on 2021-11-04T09:06:18Z For the header:
Also, for the latest version of the EnvAI book, the new
Dataset originator/creatorMOD021KM
Firms
Dataset authorsMOD021KM
Firms
Dataset documentation (here just provide me the relevant links e.g. technical report or paper) |
View / edit / reply to this conversation on ReviewNB acocac commented on 2021-11-04T09:06:19Z It seems there is a missing package,
!pip -q install pyspectral !pip -q install 'satpy==0.26.0' !pip -q install gdown !pip -q install pyorbital |
View / edit / reply to this conversation on ReviewNB acocac commented on 2021-11-04T09:06:20Z After the import, the latest version of the EnvAI Book includes a new section of setting the project folder structure. This structure facilitates to organise the input/output files a given directory. In your case you only have to add the following cells:
## Set project structurenotebook_folder = '../sensors/wildfires-sensors-seviri' if not os.path.exists(notebook_folder): os.makedirs(notebook_folder) |
View / edit / reply to this conversation on ReviewNB acocac commented on 2021-11-04T09:06:20Z For the EnvAI book, the notebook won't render with the lines above as it requires to indicate the credentials while the notebook is rendered. To avoid it, I'm holding a copy of the filename inside the notebook project folder and use the conditional to skip the downloading process. I would also suggest to use
I suggest to change the above cells to:
filename = 'MSG4-SEVI-MSG15-0100-NA-20200901104243.230000000Z-NA.nat' |
View / edit / reply to this conversation on ReviewNB acocac commented on 2021-11-04T09:06:21Z Have you preprocessed the pixel data? from which source? Maybe, it would be better to upload the sample fire pixel data to a zenodo repository linked to this notebook. We did something similar with the IceNet demo. I preprocessed certain sample data using the IceNet codebase and then uploaded to a Zenodo repository. |
View / edit / reply to this conversation on ReviewNB acocac commented on 2021-11-04T09:06:21Z Line #1. path = Path(tempfile.mkdtemp()) remove |
View / edit / reply to this conversation on ReviewNB acocac commented on 2021-11-04T09:06:22Z Line #2. catalog_file = path / 'catalog.yaml'
We can now save the catalog into the notebook project folder, please replace the line by:
# set catalogue location |
View / edit / reply to this conversation on ReviewNB acocac commented on 2021-11-04T09:06:23Z wowwww so nice earth! |
View / edit / reply to this conversation on ReviewNB acocac commented on 2021-11-04T09:06:23Z ok, now I got the point of the distortions (: |
View / edit / reply to this conversation on ReviewNB acocac commented on 2021-11-04T09:06:24Z I might be good to point key papers or website explaining further details of the FRP concept e.g. Wooster et al. or another research group. |
View / edit / reply to this conversation on ReviewNB acocac commented on 2021-11-04T09:06:25Z Nice tip about |
View / edit / reply to this conversation on ReviewNB acocac commented on 2021-11-04T09:06:25Z It's worth to mention you create/define a simple intake driver to read |
@samueljackson92 Thanks for the contribution, very nice work! I added some suggestions to make the notebook compatible with the latest version of the EnvAI book. Find below some general comments:
|
Hi @acocac. Thanks for the comments. I think I agree with all your comments. I'll find some time to make the suggested changes. I agree Zenodo is probably the best option for stability. I also think it might be worth spinning out the intake driver to a separate repo/library. |
Hi @samueljackson92, thanks for the reply, no rush. I'll find a way to keep a Zenodo repo linked to a research group, in this case the Environmental AI book. I'll keep you informed about my progress here. |
@acocac Updated! The only thing I have not changed is the requirement for Google Drive. For now, I wait for the Zenodo solution or similar. |
@samueljackson92, the notebook is ready to go so we can release it after a test in Binder tomorrow morning.
FYI, I pushed some changes to fully integrate the notebook with the new template describing further metadata, fix some minor typos and replace fetching the fire sample data from gdrive by a zenodo repository hosted by the Environmental Data Science community. I've found this route could lead us to follow FAIR principles to share/curate future sample data for the Environmental AI book.
Finally, as part of the transparency and credits, it would be great if we can add some key metadata to the Zenodo repository i.e. further details of the original source and preprocessing steps would be very helpful e.g. how did you subset the dates? In addition, can you also share your ORCID? |
Long overdue: a PR for contributing a SEVIRI data demonstrator notebook to the Environmental AI book. This notebook:
satpy
to load, visualise, and regrid SEVIRI level 1.5 data.@acocac let me know if there is anything I need to adjust or add.