Develop#315
Merged
Merged
Conversation
…essed version of an earlier visualization
…-have-nested-window-dressings
…upyter-notebooks' into 307-code-feature-direct-sun-example
… ViewFalseColorImageProcessor to show the image at all
…upyter-notebooks' into 307-code-feature-direct-sun-example
…pixels_transforms() methods, make PerspectiveTransform OpenCSP compliant by adding unit tests and docstrings
…ransforms in CroppingImageProcessor
…_to_pixels_conversion() to PerspectiveTransform
…oordinates_transform
…it tests for ViewCrossSectionImageProcessor, add invert_ylim option to View3d.draw_image()
…or View3dImageProcessor
…to use operable.transform_coordinates() in their visualizations
…' into 307-code-feature-direct-sun-example
…DirectSun example
…on outside image boundaries" issue
…n23/OpenCSP into bbean23-307-code-feature-direct-sun-example
Merged
jehsharp
added a commit
to jehsharp/OpenCSP
that referenced
this pull request
Jan 23, 2026
This reverts commit beaa810.
jehsharp
added a commit
that referenced
this pull request
Jan 23, 2026
Revert "Revert "Develop (#315)""
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
The purpose of this pull request is to get this PR merged onto develop. #314. There were too many conflicts to resolve using Githubs web interface, so I had to do it manually in Git Bash.
Summary of changes
Please summarize the code changes in this pull request.
Implementation notes
Please describe any relevant implementation details for reviewers, e.g. how correctness was verified.
Submission checklist
develop, notmainopencsp/test/test_DocStringsExist.pyare verified to include this change or have been updated accordinglydoc/are verified to include this change or have been updated accordinglyAdditional information
Please provide any additional information here.