Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Feature 4 diffuse radiation and elevation #10

Merged
merged 40 commits into from
Jun 3, 2024
Merged

Feature 4 diffuse radiation and elevation #10

merged 40 commits into from
Jun 3, 2024

Conversation

FlorianK13
Copy link
Member

@FlorianK13 FlorianK13 commented May 13, 2024 •
edited
Loading

This will converted to "ready for review" when development is finished.

FlorianK13 added 30 commits April 22, 2024 10:52
@FlorianK13
First steps to add sun source from url #4
ed2cab5
@FlorianK13
Add elevation rasters to scene attributes #5
167d41b
@FlorianK13
Create function to calculate elevationAngles #5
9d8d690 
Based on a 2D grid which will be given as GeoTIFFs on a running website.
@FlorianK13
Receive elevation raster in shading simulation #5
f9f2eca
@FlorianK13
Include shading from elevation to raytracing function #5
854281e
@FlorianK13
Add separation of spherical and cartesian coordinates #5
99785c3
@FlorianK13
Delete unused variable #5
554d809
@FlorianK13
Repair existing methods #4
42b2f03
@FlorianK13
Add test for fetching data from API #4
d16a4f0
@FlorianK13
Add raytracing to case where irradianceURL is given #4
9ffeaa5
@FlorianK13
Change 2d array to list of sphericalPoints #5
d6e3d85 
By doing so, the strange dependency on grid indeces got removed. Now
the list of points has the information of precise locations.
@FlorianK13
Change type of elevation to list of points #5
3ac9980
@FlorianK13
Correct numDirections parameter #5
67e8488
@FlorianK13
Correct function description #5
6f1a9a4
@FlorianK13
Merge branch 'feature-5-elevation' into feature-4-diffuse-radiation-f…
f453f19 
…rom-file
@FlorianK13
Update data type of fetched irradiance #4
7da35ce
@FlorianK13
Change to new irradiance API format #4
d2f95d3
@FlorianK13
Delete outdated comment #4
339c809
@FlorianK13
Move type definitions to utils #4
17b7104
@FlorianK13
Move type definition to utils #4
7ba184c
@FlorianK13
Convert to unified SpherialPoint type #4
d7aa394
@FlorianK13
Return sun vectors in unified Point types #4
f12eaf0
@FlorianK13
Convert even more arrays to the point types #4
423ba05
@FlorianK13
Add skeleton function for elevation shading #4
38869f4
@FlorianK13
Add diffuse rad parameter to WebGL function #4
bac77de
@FlorianK13
Merge branch 'main' into feature-4-diffuse-radiation-from-file
3b8e538
@FlorianK13
Repair errors from merge #4
1dea7fb
@FlorianK13
Stop running tests on draft PRs
2e25ba0
@FlorianK13
Delete unused console.log #4
4d12be7
@FlorianK13
Delete link to module #4
d011dd2 
This was necessary to make the minimalApp repo work
@FlorianK13 FlorianK13 marked this pull request as ready for review May 27, 2024 12:51
@FlorianK13 FlorianK13 requested a review from khdlr May 27, 2024 12:51
@FlorianK13 FlorianK13 merged commit afe3b0b into main Jun 3, 2024
2 checks passed
@FlorianK13 FlorianK13 deleted the feature-4-diffuse-radiation-from-file branch June 3, 2024 06:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@khdlr khdlr Awaiting requested review from khdlr

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@FlorianK13