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.
This PR aims to help with working on bigger floor images and make the plotting more "flexible" for different formats
Problem
Currently some of the parameters for plotting are hardcoded and overfitted to the floor plan inside
examples/
directory. This results in inproper handing of bigger image formats withwhm benchmark
Given below is a example of floor plan image with resolution:
(2560, 831)
. This is perfectly fine example as even with simple PyGUI vertical images can be easily moved around the screen.As we can see GUI "cuts" image to thumbnail with fixed size.
Other problem is hardcoded plotting parameters for
whm plot
.There are several problems to solve:
Proposed Solution
thumbnail
cutting of given images and setting hardcodedmaxsize
to1200 x 850
Obtained Results
Following section shows obtained results after applying changes described above
whm benchmark
resultsImage below show that after removing
maxsize
and removingthumbnail
we can work with full size images:whm plot
resultsFollowing is a combination of obtained results with comparison to "baseline" (current) result
a) examples/floor_plan_white.png
(651, 792)
Result before changes:
Result after changes:
b) flat_example.png
(2560, 831)
Result before changes:
Result after changes:
c) large_example.png
(2490, 2420)
Result before changes:
Result after changes:
Future work
A few ideas which would make working on bigger resolutions easier with
whm bechmark
:whm benchmark
withscale_x=...
andscale_y=...
or justscale=...
paramsimage_x = click_x
) point on generated view, but saves realx
asclick_x * scale_x
The second approach is probably not that hard to implement, is easy to work with, and allows working without changing how GUI works.
There is no need to work on full-resolution images during the benchmark phase (as pixel-wise-accuracy is not needed) but it would be nice to see generated plots on full resolution of the floor plan.