Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: Token VBL simplification #1435

Merged
merged 23 commits into from
Mar 30, 2020

Conversation

JamzTheMan
Copy link
Member

@JamzTheMan JamzTheMan commented Mar 30, 2020

Various highlights from commits...

  • VBL_Functions google code cleanup
  • setTokenVBL "shape": "auto" updated for auto-generation options
    sample: [h: options = json.set("{}", "shape", "auto", "sensitivity", 10, "inverse", 0, "level", 2, "method", "Douglas Peucker", "r", 0, "g", 0, "b", 0, "a", 255)]
  • Update tooltips for Token VBL controls
  • Performance increase upwards of 50x
  • doubleclick on preview window resets preview scaling
  • Fix tokenPropertiesDialog.xml to use generic label vs RSyntax beans
    so they are not needed when using forms designer which just removes
    the component if you didn't have forms designer setup properly.
  • Added threading in a few places for performance and for proper UI updates
  • Added Color Picker; you are no longer restricted to just transparency but can now pick a color. Although a popular color like Black may be overzealous.
  • Lots of bug fixes and improvements
  • Updated UI to final preview
  • Tested with very large maps (10,000 x 10,000 pixels) takes ~30-40 seconds but UI no longer locks up and dialog can be closed. Most objects take <1 second.

Closes #1218 - Optimize Token VBL Generation


This change is Reviewable

JamzTheMan and others added 22 commits February 6, 2020 20:31
…plification-revert-revert

Feature token vbl simplification revert revert
Signed-off-by: JamzTheMan <[email protected]>
Signed-off-by: JamzTheMan <[email protected]>
Signed-off-by: JamzTheMan <[email protected]>
Signed-off-by: JamzTheMan <[email protected]>
 * No long requires 'valid geometry' which allows for vbl with 'holes'

#needsTesting

Signed-off-by: JamzTheMan <[email protected]>
 * No long requires 'valid geometry' which allows for vbl with 'holes'
 * Fix tokenPropertiesDialog.xml to use generic label vs RSyntax beans
   so they are not needed when using forms designer which just removes
   the component if you didn't have forms designer setup properly.

#needsTesting

Signed-off-by: JamzTheMan <[email protected]>
 * No long requires 'valid geometry' which allows for vbl with 'holes'
 * Fix tokenPropertiesDialog.xml to use generic label vs RSyntax beans
   so they are not needed when using forms designer which just removes
   the component if you didn't have forms designer setup properly.

#needsTesting

Signed-off-by: JamzTheMan <[email protected]>
 * Performance increase upwards of 50x
 * doubleclick on preview window resets preview scaling
 * No long requires 'valid geometry' which allows for vbl with 'holes'
 * Fix tokenPropertiesDialog.xml to use generic label vs RSyntax beans
   so they are not needed when using forms designer which just removes
   the component if you didn't have forms designer setup properly.

#needsTesting

Signed-off-by: JamzTheMan <[email protected]>
…plification

Feature token vbl simplification
 * Performance increase upwards of 50x
 * doubleclick on preview window resets preview scaling
 * No long requires 'valid geometry' which allows for vbl with 'holes'
 * Fix tokenPropertiesDialog.xml to use generic label vs RSyntax beans
   so they are not needed when using forms designer which just removes
   the component if you didn't have forms designer setup properly.
 * Added threading in a few places for performance and for proper UI updates
 * Added Color Picker; you are no longer restricted to just transparency but can now pick a color. Although a popular color like Black may be overzealous.
 * Lots of bug fixes and improvements
 * Updated UI to final preview
 * Tested with very large maps (10,000 x 10,000 pixels) takes ~30-40 seconds but UI no longer locks up and dialog can be closed. Most objects take <1 second.

Signed-off-by: JamzTheMan <[email protected]>
 * Performance increase upwards of 50x
 * doubleclick on preview window resets preview scaling
 * No long requires 'valid geometry' which allows for vbl with 'holes'
 * Fix tokenPropertiesDialog.xml to use generic label vs RSyntax beans
   so they are not needed when using forms designer which just removes
   the component if you didn't have forms designer setup properly.
 * Added threading in a few places for performance and for proper UI updates
 * Added Color Picker; you are no longer restricted to just transparency but can now pick a color. Although a popular color like Black may be overzealous.
 * Lots of bug fixes and improvements
 * Updated UI to final preview
 * Tested with very large maps (10,000 x 10,000 pixels) takes ~30-40 seconds but UI no longer locks up and dialog can be closed. Most objects take <1 second.

Signed-off-by: JamzTheMan <[email protected]>
 * Performance increase upwards of 50x
 * doubleclick on preview window resets preview scaling
 * No long requires 'valid geometry' which allows for vbl with 'holes'
 * Fix tokenPropertiesDialog.xml to use generic label vs RSyntax beans
   so they are not needed when using forms designer which just removes
   the component if you didn't have forms designer setup properly.
 * Added threading in a few places for performance and for proper UI updates
 * Added Color Picker; you are no longer restricted to just transparency but can now pick a color. Although a popular color like Black may be overzealous.
 * Lots of bug fixes and improvements
 * Updated UI to final preview
 * Tested with very large maps (10,000 x 10,000 pixels) takes ~30-40 seconds but UI no longer locks up and dialog can be closed. Most objects take <1 second.

Signed-off-by: JamzTheMan <[email protected]>
Signed-off-by: JamzTheMan <[email protected]>
 * VBL_Functions google code cleanup
 * setTokenVBL "shape": "auto" updated for auto-generation options
   sample: `[h: options = json.set("{}", "shape", "auto", "sensitivity", 10, "inverse", 0, "level", 2, "method", "Douglas Peucker", "r", 0, "g", 0, "b", 0, "a", 255)]`

Signed-off-by: JamzTheMan <[email protected]>
Signed-off-by: JamzTheMan <[email protected]>
@JamzTheMan JamzTheMan added feature Adding functionality that adds value macro changes This issue adds or changes macro functions. Extra work is required (testing, wiki, code editor) M Medium impact. T-shirt style story pointing/scope labels Mar 30, 2020
@JamzTheMan JamzTheMan requested a review from Phergus March 30, 2020 21:53
@JamzTheMan JamzTheMan self-assigned this Mar 30, 2020
Signed-off-by: JamzTheMan <[email protected]>
Copy link
Contributor

@Phergus Phergus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 10 of 10 files at r1, 1 of 1 files at r2.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@Phergus Phergus merged commit 2c2cc4a into RPTools:develop Mar 30, 2020
@JamzTheMan JamzTheMan deleted the feature-token-vbl-simplification branch June 20, 2020 04:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adding functionality that adds value M Medium impact. T-shirt style story pointing/scope macro changes This issue adds or changes macro functions. Extra work is required (testing, wiki, code editor)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants