[Bug] - N8AO works incorrectly with adaptiveDpr
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- react, three.js, typescript
- Lĩnh vực
- frontend, performance, web-dev
Hướng nghiên cứu
Bắt đầu với hành vi của N8AOPostPass được mô tả trong issue và tái hiện trường hợp này bằng cách thay đổi DPR của renderer từ 2 thành 1 trong các setup react-postprocessing và three được liệt kê. Theo dõi cách các giá trị kích thước của nó được cập nhật; hoàn tất khi adaptiveDpr không còn tạo ra các phản xạ không mong muốn sau quá trình chuyển đổi DPR.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Hey everyone! I seems found a bug in N8AO related to AdaptiveDpr
We found a strange effect in our application after adding N8AO to EffectComposer - sometimes it started to add weird 'reflections' to the materials, and we couldn't understand where it's coming from
This is how it looks like: On the surface of the model you can see "reflection" from grid, which should't be present (but it's not only about grid but any objects can appear as reflection).
After debugging for a while I've noticed that this happens on my machine when DPR value is changed. If we (or system) change dpr from 2 to 1, N8AO started to add this 'reflections'. As I can see class has size values which are half of the initial renderer size (if I understand correctly)
If I manually call setSize(width, height) method on N8AOPostPass instance and upscale this values back to dpr=2 values this bug disappears. So it's either should always keep initial values or there is some error in calculation after updating render size
Steps to reproduce
Lest say screen size is 1000x500 with dpr=2. initial values of renderer is 2000x1000px, dpr=2, we set dpr to 1, values are updated to 1000x500, bug appears. If I set values manually by calling setSize() on N8AOPostPass instance to setSize(2x1000, 2x500) it starts working correctly again
versions
"@react-three/drei": "^9.97.5",
"@react-three/fiber": "^8.15.16",
"@react-three/postprocessing": "^2.16.2",
"postprocessing": "^6.35.3",
"three": "^0.161.0",
Can anyone help with this? I can help with fix if you can explain a bit more of how it works. Thanks in advance
- Ngôn ngữ chính
- TypeScript
- Star
- 1.4k
- Fork
- 134
- Merge trung bình
- 4 phút
- Pull request đã merge (30 ngày)
- 4
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của pmndrs/react-postprocessing
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 42/100
pmndrs/react-postprocessing#343 · 1 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
pmndrs/react-postprocessing#286 ·
-
enhancement
pmndrs/react-postprocessing#253 · 4 reaction · 1 người được giao ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
pmndrs/react-postprocessing#240 · 1 bình luận · 1 reaction ·
-
Integrate realism-effects Đang mởenhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 20/100
pmndrs/react-postprocessing#183 ·
Tất cả issue của pmndrs/react-postprocessing
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
mksglu/context-mode#1200 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
jaegertracing/jaeger-ui#4506 ·
-
area:desktop area:ui bug platform:macos
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
anthropics/claude-code#96687 ·
-
good first issue
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 95/100
AOSSIE-Org/DebateAI#582 · 2 bình luận ·