Incorporate changes to workshop materials from old repo #493
Conversation
| @@ -30,20 +30,63 @@ On Linux you can do this using the `-restrict-paths` command line argument, or b | |||
| On Windows and Mac OS this option is in the "Access" tab of the configuration options. | |||
| Globus will only be able to transfer files onto your machine in the specified paths. | |||
|
|
|||
| ### The Globus web app | |||
There was a problem hiding this comment.
Do we actually want to include all this? Also you seem to be missing the images if we do.
| ``` | ||
|
|
||
| would save the file to `~/dkist_data/BKPLX/VISP_2023_10_16T18_21_47_508_00630200_I_BKPLX_L1.fits`. | ||
| would save the file to `~/sunpy/data/BKPLX/VISP_2023_10_16T18_21_47_508_00630200_I_BKPLX_L1.fits`. |
There was a problem hiding this comment.
I think the old path was correct?
| aia.plot(axes=ax) | ||
| ``` | ||
|
|
||
| #### Computing the VISP field of View |
There was a problem hiding this comment.
These examples probably suffer from the issue that they wont handle rotation properly. I think the best thing to do would be to either leave this for now or use sunpy.visualization.drawing.extent from 6.1.
| ## More advanced plotting | ||
| ### Using AIA as a Context Image | ||
|
|
||
| #### Fetching an AIA Image |
There was a problem hiding this comment.
This feels kinda out of place, also it duplicates what I added here: https://github.com/DKISTDC/dkist/pull/492/files#diff-f5104acda48fa6b068e8705f2a8b81af1e7666df7b70dc6b0628889ab9ff1e42R65-R89
| ``` | ||
|
|
||
| ```{code-cell} ipython3 | ||
| res = Fido.search(a.dkist.Dataset("BKPLX")) | ||
| asdf_file = Fido.fetch(res, path="~/dkist_data/{instrument}_{dataset_id}") |
There was a problem hiding this comment.
This path change is wrong, it means it's not the path the sample data is in.
There was a problem hiding this comment.
This file seems to be missing from the index.md file
|
|
||
| ```python | ||
| import dkist | ||
| from dkist.data.sample import VBI_AJQWW |
There was a problem hiding this comment.
We need to be consistent on how we load data and sample data, but this probably isn't the PR to argue it out.
No description provided.