Closed sashankaryal closed 2 months ago
[!IMPORTANT]
Review Skipped
Review was skipped due to path filters
Files ignored due to path filters (2)
* `e2e-pw/src/oss/specs/3d/fo3d-pcd-stl.spec.ts-snapshots/scene-chromium-darwin.png` is excluded by `!**/*.png`, `!**/*.png` * `e2e-pw/src/oss/specs/3d/fo3d-pcd-stl.spec.ts-snapshots/scene-chromium-linux.png` is excluded by `!**/*.png`, `!**/*.png`You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
The recent updates introduce significant enhancements to the looker-3d
package, including new grid configuration options, updated component styling, and improved state management. Additionally, the PopoutSectionTitle
component's styling was refined, and Docker image building scripts were optimized. These changes collectively improve the user interface, usability, and maintainability of the codebase.
File Path | Change Summary |
---|---|
app/packages/components/src/components/PopoutSectionTitle/PopoutSectionTitle.tsx |
Added flexbox styling for better alignment and spacing in the PopoutSectionTitle component. |
app/packages/looker-3d/src/action-bar/ToggleGridHelper.tsx |
Introduced a GridConfigurator component and updated ToggleGridHelper with new state management and UI elements. |
app/packages/looker-3d/src/action-bar/shared.tsx |
Converted ActionPopOver to use forwardRef for better ref forwarding. |
app/packages/looker-3d/src/action-bar/style.module.css |
Added new CSS classes for grid configuration styling. |
app/packages/looker-3d/src/fo3d/Gizmos.tsx |
Updated grid-related calculations and state management using Recoil atoms. |
app/packages/looker-3d/src/fo3d/point-cloud/Pcd.tsx |
Replaced useMemo with useEffect for point cloud data handling. |
app/packages/looker-3d/src/hooks/use-pcd-material-controls.ts |
Increased the maximum pointSize value from 20 to 50. |
app/packages/looker-3d/src/state.ts |
Added new Recoil atoms for grid settings. |
e2e-pw/scripts/generate-screenshots-docker-image/build-docker-image.sh |
Updated script to exclude additional directories and added a Docker image build step. |
In the realm of code, where changes dance, New features bloom, enhancing chance. With grids and styles, and Docker's might, The future of apps shines ever bright. Flex and atoms, all in line, A rabbit's joy in every sign. π
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Please upload report for BASE (
release/v0.24.0@d1b48f2
). Learn more about missing BASE report.:exclamation: Current head 53a8f82 differs from pull request most recent head 9d8cad7
Please upload reports for the commit 9d8cad7 to get more accurate results.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
What changes are proposed in this pull request?
Add grid configurator
Fix rgb shader not responding to point size change
~Center points geometry~
Use
points.clone()
to do a shallow copy so that point clouds with same URL are treated as different copies in the scene graphHow is this patch tested? If it is not, please explain why.
Manual testing
Summary by CodeRabbit
New Features
GridConfigurator
component for enhanced grid settings customization.Improvements
PopoutSectionTitle
component styling for better alignment and spacing.ActionPopOver
component to useforwardRef
for improved ref forwarding.Gizmos
component with dynamic grid settings.Bug Fixes
useMemo
withuseEffect
inPcd
component for better point cloud data handling.Performance
pointSize
in point cloud material controls from 20 to 50.Chores
screenshot
Docker image.