Zach Burnett

Results 149 comments of Zach Burnett

there's an issue with the package data so I will convert this back to draft for now

> > Resolves [JP-2157](https://jira.stsci.edu/browse/JP-2157) > > How does this fix JP-2157? Seems like this is strictly related to CI tests for PRs. Oh whoops, that was accidental. I'll remove that

note: `miniconda3` (installed from the `*.pkg` distribution) did not work, so I built this with `miniforge3` for Mac OSX AMD64 (installed via the shell script) instead. I think the issue...

Does this warrant a changelog entry so users are aware of the new citation?

It looks like the following warnings are being flagged by RTD before the build proceeds: https://readthedocs.org/projects/jwst-pipeline/builds/18203877/ ``` /home/docs/checkouts/readthedocs.org/user_builds/jwst-pipeline/checkouts/6919/docs/getting_started/faq.rst:13: WARNING: Inline interpreted text or phrase reference start-string without end-string. /home/docs/checkouts/readthedocs.org/user_builds/jwst-pipeline/checkouts/6919/docs/getting_started/faq.rst:32: WARNING:...

@cshanahan1 it is giving a warning about `skip_step_cli`, but I can't seem to find that anchor anywhere. Is it supposed to be in the `faq.rst` file? ``` /home/docs/checkouts/readthedocs.org/user_builds/jwst-pipeline/checkouts/6919/docs/getting_started/faq.rst:128: WARNING: undefined...

I can reproduce this behavior with the following steps using a layer control: 1. Add the `VectorFieldAnim` layer 2. Add a `ScalarField` layer, in this case from an ASCII grid...

I can confirm from my local installation that this pull request fixes the problem with `removeLayer`. It is a single-line fix, and definitely should be merged into master.