Nabil Freij comments

Results 802 comments of


                                            Nabil Freij

Add support for Map to load from a generator

> Is there anything else to work on regarding this pr? No, it needs reviews from the other mainainters before a merge.

Add support for Map to load from a generator

Thanks for the PR @Jett-Code, sorry it took so long on our side.

Allow `clip_interval` and `vmin/vmax` be passed to MapSequence plotting

@ryuusama09, there is an open pull request for this issue, #7253 I would suggest looking at another issue.

Allow `clip_interval` and `vmin/vmax` be passed to MapSequence plotting

Probably

Introduced area-based contours

Thank you for the PR @Paras20222. I will do a quick review now.

Introduced area-based contours

Hi @Paras20222, will you have some time soon to address the outstanding comments?

Introduced area-based contours

> Hello, i have made few changes to the gist. `import numpy as np import matplotlib.pyplot as plt Modifying the gist is not the most useful, we can't make suggestions...

Introduced area-based contours

Hi @Paras20222, sorry for the late response. I am going to close this PR for the time being. If you want to take this forward, I would suggest reopening the...

Add Warning for rsun Mismatch in reproject_to() Method

Thanks for the PR @Mubin17, can you add changelog and a unit test? If you need help with these, we can provide that.

Change `Fido.fetch()` to raise an error if one or more files failed to download

Do we want to only handle this for examples or in general?