Wed, 25 May 2022 20:36:34 +0300
Fix pick() picking from weird places on the screen with high DPI scaling
glReadPixels reads data from the frame buffer, which contains data after
high DPI scaling, so any reads to that need to take this scaling into account
19 | 1 | /* |
2 | * LDForge: LDraw parts authoring CAD | |
3 | * Copyright (C) 2013 - 2019 Teemu Piippo | |
4 | * | |
5 | * This program is free software: you can redistribute it and/or modify | |
6 | * it under the terms of the GNU General Public License as published by | |
7 | * the Free Software Foundation, either version 3 of the License, or | |
8 | * (at your option) any later version. | |
9 | * | |
10 | * This program is distributed in the hope that it will be useful, | |
11 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | |
12 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |
13 | * GNU General Public License for more details. | |
14 | * | |
15 | * You should have received a copy of the GNU General Public License | |
16 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | |
17 | */ | |
18 | ||
19 | #include "boundingbox.h" | |
20 | ||
189
815fbaae9cb2
cleanup, gl::Compiler changed to gl::ModelShaders
Teemu Piippo <teemu@hecknology.net>
parents:
33
diff
changeset
|
21 | /** |
815fbaae9cb2
cleanup, gl::Compiler changed to gl::ModelShaders
Teemu Piippo <teemu@hecknology.net>
parents:
33
diff
changeset
|
22 | * @brief Resizes @c box so that @c vertex fits inside |
815fbaae9cb2
cleanup, gl::Compiler changed to gl::ModelShaders
Teemu Piippo <teemu@hecknology.net>
parents:
33
diff
changeset
|
23 | * @param box |
815fbaae9cb2
cleanup, gl::Compiler changed to gl::ModelShaders
Teemu Piippo <teemu@hecknology.net>
parents:
33
diff
changeset
|
24 | * @param vertex |
815fbaae9cb2
cleanup, gl::Compiler changed to gl::ModelShaders
Teemu Piippo <teemu@hecknology.net>
parents:
33
diff
changeset
|
25 | */ |
815fbaae9cb2
cleanup, gl::Compiler changed to gl::ModelShaders
Teemu Piippo <teemu@hecknology.net>
parents:
33
diff
changeset
|
26 | void addPointToBox(BoundingBox &box, const glm::vec3 &vertex) |
19 | 27 | { |
196 | 28 | box.minimum.x = std::min(vertex.x, box.minimum.x); |
29 | box.minimum.y = std::min(vertex.y, box.minimum.y); | |
30 | box.minimum.z = std::min(vertex.z, box.minimum.z); | |
31 | box.maximum.x = std::max(vertex.x, box.maximum.x); | |
32 | box.maximum.y = std::max(vertex.y, box.maximum.y); | |
33 | box.maximum.z = std::max(vertex.z, box.maximum.z); | |
19 | 34 | } |
35 | ||
36 | /* | |
37 | * Returns the length of the bounding box on the longest measure. | |
38 | */ | |
33
4c41bfe2ec6e
replaced matrix and vertex classes with glm
Teemu Piippo <teemu@hecknology.net>
parents:
23
diff
changeset
|
39 | float longestMeasure(const BoundingBox& box) |
19 | 40 | { |
22
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
41 | if (box != emptyBoundingBox) |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
42 | { |
33
4c41bfe2ec6e
replaced matrix and vertex classes with glm
Teemu Piippo <teemu@hecknology.net>
parents:
23
diff
changeset
|
43 | const float dx = std::abs(box.minimum.x - box.maximum.x); |
4c41bfe2ec6e
replaced matrix and vertex classes with glm
Teemu Piippo <teemu@hecknology.net>
parents:
23
diff
changeset
|
44 | const float dy = std::abs(box.minimum.y - box.maximum.y); |
4c41bfe2ec6e
replaced matrix and vertex classes with glm
Teemu Piippo <teemu@hecknology.net>
parents:
23
diff
changeset
|
45 | const float dz = std::abs(box.minimum.z - box.maximum.z); |
4c41bfe2ec6e
replaced matrix and vertex classes with glm
Teemu Piippo <teemu@hecknology.net>
parents:
23
diff
changeset
|
46 | const float size = std::max(std::max(dx, dy), dz); |
4c41bfe2ec6e
replaced matrix and vertex classes with glm
Teemu Piippo <teemu@hecknology.net>
parents:
23
diff
changeset
|
47 | return std::max(size / 2.0f, 1.0f); |
22
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
48 | } |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
49 | else |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
50 | { |
33
4c41bfe2ec6e
replaced matrix and vertex classes with glm
Teemu Piippo <teemu@hecknology.net>
parents:
23
diff
changeset
|
51 | return 0.0f; |
22
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
52 | } |
19 | 53 | } |
54 | ||
55 | ||
56 | /* | |
57 | * Yields the center of the bounding box. | |
58 | */ | |
33
4c41bfe2ec6e
replaced matrix and vertex classes with glm
Teemu Piippo <teemu@hecknology.net>
parents:
23
diff
changeset
|
59 | glm::vec3 boxCenter(const BoundingBox& box) |
19 | 60 | { |
22
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
61 | if (box != emptyBoundingBox) |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
62 | { |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
63 | return { |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
64 | (box.minimum.x + box.maximum.x) / 2, |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
65 | (box.minimum.y + box.maximum.y) / 2, |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
66 | (box.minimum.z + box.maximum.z) / 2 |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
67 | }; |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
68 | } |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
69 | else |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
70 | { |
33
4c41bfe2ec6e
replaced matrix and vertex classes with glm
Teemu Piippo <teemu@hecknology.net>
parents:
23
diff
changeset
|
71 | return glm::vec3{0, 0, 0}; |
22
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
72 | } |
19 | 73 | } |
74 | ||
75 | /* | |
76 | * Returns the length of the bounding box's space diagonal. | |
77 | */ | |
33
4c41bfe2ec6e
replaced matrix and vertex classes with glm
Teemu Piippo <teemu@hecknology.net>
parents:
23
diff
changeset
|
78 | float spaceDiagonal(const BoundingBox& box) |
19 | 79 | { |
33
4c41bfe2ec6e
replaced matrix and vertex classes with glm
Teemu Piippo <teemu@hecknology.net>
parents:
23
diff
changeset
|
80 | return glm::distance(box.minimum, box.maximum); |
19 | 81 | } |
22
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
82 | |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
83 | bool operator==(const BoundingBox &box_1, const BoundingBox &box_2) |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
84 | { |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
85 | return box_1.minimum == box_2.minimum and box_1.maximum == box_2.maximum; |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
86 | } |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
87 | |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
88 | bool operator!=(const BoundingBox &box_1, const BoundingBox &box_2) |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
89 | { |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
90 | return not (box_1 == box_2); |
6da867fa5429
commit work on GL rendering
Teemu Piippo <teemu@hecknology.net>
parents:
20
diff
changeset
|
91 | } |