-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Batch kernels for backward pass of Preprocessing #3
Open
sandeepnmenon
wants to merge
36
commits into
mlsys/forward_preprocess_batch
Choose a base branch
from
mlsys/batched_preprocess
base: mlsys/forward_preprocess_batch
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
c408be1
preprocess batches for backward outline
sandeepnmenon 6444aa7
solve syntax errors in backward
prapti19 8df1b63
Refactor GaussianRasterizationSettings class to handle raster_setting…
sandeepnmenon f59e67c
Merge branch 'mlsys/batched_preprocess' of github.com:TarzanZhao/diff…
sandeepnmenon 529710b
added focal_x and focal_y calculation inside the kernel
sandeepnmenon 3ac1ad3
Refactor rasterization_tests.py to use raster_settings_batch instead …
sandeepnmenon ce314e2
fixed namedtuple setting bug
sandeepnmenon fdd3b4f
Refactor GaussianRasterizationSettings class to handle raster_setting…
sandeepnmenon cdf3bc1
remove focal_x and focal_y calculations
sandeepnmenon 591a5c1
Refactor CUDA rasterizer code to include width and height parameters …
sandeepnmenon 1b54023
Renamed W and H to image_width and image_height parameters in prepr…
sandeepnmenon 9294a07
reverted focal_x and focal_y removal in normal preprocessBackward
sandeepnmenon 877677f
grad_means2D to handle more than 2 dimensions
sandeepnmenon 444e8a5
add tests for backward
prapti19 46b83eb
ruff formatting and gradients for remaining inputs
sandeepnmenon 710b56f
Add pyproject.toml file with ruff line-length set to 120
sandeepnmenon 2ca5ae6
Refactor ruff.toml file to set line-length to 120 and indent-width to 4
sandeepnmenon 7a4b6b4
Refactor compare_tensors function to handle None values in rasterizat…
sandeepnmenon 14889e6
Update ruff.toml file to set line-length to 120
sandeepnmenon 5682d26
Refactor rasterization_backward_tests.py to include gradient checks f…
sandeepnmenon 945e8cf
gradients calculated for all the variables to check and cloning them
sandeepnmenon c84d7cd
converted to pytest testing
sandeepnmenon 6f38446
fixed colon bug and ruff formatiting
sandeepnmenon 7b30782
Add __pycache__/ to .gitignore
sandeepnmenon 13a5559
renamed to *_test.py
sandeepnmenon 5b24881
Update .gitignore to include __pycache__/
sandeepnmenon 9e6f4a9
moved test into tests folder
sandeepnmenon 7be38fa
Add instructions for running tests in README.md
sandeepnmenon 1887e14
Merge branch 'mlsys/forward_preprocess_batch' of github.com:TarzanZha…
sandeepnmenon 307e156
deleted old test file
sandeepnmenon 21ee225
renamed idx to point_idx and view_idx to result_idx in backward
sandeepnmenon 363b4ee
moved from python time to torch record
sandeepnmenon 91d1582
fixed num_points in preprocessForwardBatches
sandeepnmenon ee767da
Refactor test function names for clarity and consistency
sandeepnmenon 2e7f032
fixed but in printing only first 5 non matching indices
sandeepnmenon e8edb86
fixed backward bug of backward kernel not getting executed
sandeepnmenon File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,3 +3,4 @@ diff_gaussian_rasterization.egg-info/ | |
dist/ | ||
diff_gaussian_rasterization/__pycache__/ | ||
*so | ||
__pycache__/ |
This file contains 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
This file contains 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should make sure your two pull request will not conflict with each other when we merge both of them. Maybe we could do this together in tomorrow meeting.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They are separate PRs. We'll resolve merge conflict after the forward pass PR is merged in.