| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
tagging @nden and @perrygreenfield for review. |
Sorry, something went wrong.
|
This fixes the error in jwst. Can you add a docstring to get_bounding_box? And rebase and squash. |
Sorry, something went wrong.
Initial fix for bug Extended test to include a bounding box Added regression test for compound models having the same issue Fixed issue for compound models as well Added changelog
|
Failure is unrelated. |
Sorry, something went wrong.
…ith_bounding_box=True`
…380-on-v5.0.x Backport PR #12380 on branch v5.0.x (Bugfix for truncated outputs when using `with_bounding_box=True`)
| Back | FazBrowse Home | New Git URL |
Description
For models with n_outputs=1 sometimes the outputs are truncated to just the first input when using the with_bounding_box=True keyword argument. This was fixed by calling on the slightly more sophisticated logic
present in the get_bounding_box() method for Model instead of a few basic tests on the with_bounding_box keyword argument.
Fixes #12373
Checklist for package maintainer(s)
This checklist is meant to remind the package maintainer(s) who will review this pull request of some common things to look for. This list is not exhaustive.