Nick Murphy

Results 333 comments of Nick Murphy

> I'm trying to figure out if there's some type in Python that includes the usual `Iterable` types like tuples, lists, and sets, but doesn't include strings Hm...I don't know...

Thank you for raising this issue! If `uv` does get incorporated into CI, it would become reasonably straightforward to test against the minimum allowed versions of dependencies (which are being...

This looks like a reasonable step on the way to #2655.

Now that #3060 is done and the plasma calculator is in a standalone repository, `ipykernel`, `ipywidgets`, and `voila` are no longer required dependencies. `ipykernel` and `ipywidgets` are still part of...

This PR is presently ignoring `.github/workflows/scheduled_builds.yml` since this file is missing the `runs` field. This was the full error from `zizmor`: ``` failed to register input: .github/workflows/scheduled_builds.yml Caused by: 0:...

> I think this will want to get put into the package template, rather than here ultimately, but we can work through issues with this before that. Agreed! I had...

I'm wondering whether it would be worth it to split this PR up into a single PR per subpackage...since I generally strongly recommend against PRs that modify ∼100 files! 😅

We did try using `scipy.io.special.read_genx` but got this error: ```Python >>> f = read_genx('/home/namurphy/Downloads/xrt_channels_v0016.geny') --------------------------------------------------------------------------- IndexError Traceback (most recent call last) in ----> 1 f = read_genx('/home/namurphy/Downloads/xrt_channels_v0016.geny') ~/.miniconda/lib/python3.9/site-packages/sunpy/io/special/genx.py in read_genx(filename)...

Thank you — we'll give @hayesla's example a go. The main reason I brought up this issue is due to [this comment](https://github.com/sunpy/sunpy/issues/1927#issuecomment-730027115) in #927 and because SunPy does have `sunpy.io.special.genx.read_genx`....

In case there's anyone in the future who is trying to work with `.geny` files in Python and happens across this issue, the workaround that we decided on using was...