P. L. Lim

icon indicating a website link https://www.linkedin.com/in/pllim/

icon company Space Telescope Science Institute icon location Baltimore, MD, USA icon location My views and opinions are my own and do not represent my employer's.

Results 2472 comments of P. L. Lim

TST: fix incorrect settings in testing dataframe support

cc @TheSkyentist

TST: fix incorrect settings in testing dataframe support

On Astropy Slack, @astrofrog said he would proceed to release v7.2.0 without this patch because this is not really user facing, so it does not block release. FYI.

TST: fix incorrect settings in testing dataframe support

Ah, the cc wasn't a blame. It is just so the PR author is aware of this follow-up PR. I apologize if it came across otherwise.

Implementation of a `loc` keyword argument (similar to that of `matplotlib`'s label setters) for the `wcsaxes` `AxisLabels`

This broke tests on CircleCI -- https://app.circleci.com/pipelines/github/astropy/astropy/27874/workflows/d8a010b1-9007-4238-bbd9-293174743def/jobs/139262

DOC: Plotting using text_coord vs text very confusing

What was confusing for *me* was `ax.text_coord` made it sound like it is from `matplotlib` but it wasn't but `ax.text` is from `matplotlib`. And unclear when one should use `text_coord`...

DOC: Plotting using text_coord vs text very confusing

> Would that satisfy your need Yes, I think so. Thanks!

Use version again when PyCQA/docformatter 1.7.5 or later is released

Ooops thanks for the correction. I updated the original post.

Use version again when PyCQA/docformatter 1.7.5 or later is released

I don't think this is relevant anymore.

wcslib 8.5 released

Looks like it has @astrofrog patch. Is that in reference to another astropy issue/PR? Thanks!

DEP: redefine `dev[_all]` extras as PEP 735 Dependency Groups

> are there any extras that are not controversial to deprecate or remove ? There were all added for a reason, so probably not, at least not without a lot...