-
Notifications
You must be signed in to change notification settings - Fork 827
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The moving_sphere class is mostly redundant with sphere, and lacks UV coordinates #1125
Milestone
Comments
hollasch
added a commit
that referenced
this issue
May 11, 2023
This change deprecates the moving_sphere class, and instead extends the existing sphere class to include this functionality. Note that in TheRestOfYourLife, the PDF functions ignore the moving aspect, and assume a static sphere (which preserves existing behavior). See issue #1125
hollasch
added a commit
that referenced
this issue
Jun 6, 2023
hollasch
added a commit
that referenced
this issue
Jun 6, 2023
hollasch
added a commit
that referenced
this issue
Jun 7, 2023
trevordblack
pushed a commit
that referenced
this issue
Jun 8, 2023
trevordblack
pushed a commit
that referenced
this issue
Jun 25, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It should be a simple matter of adding a second constructor to the
sphere
class to add the ability to be a moving sphere.The text was updated successfully, but these errors were encountered: