Skip to content

Correctly load range values#224

Merged
JLLeitschuh merged 1 commit intoWPIRoboticsProjects:masterfrom
ThomasJClark:master
Dec 29, 2015
Merged

Correctly load range values#224
JLLeitschuh merged 1 commit intoWPIRoboticsProjects:masterfrom
ThomasJClark:master

Conversation

@ThomasJClark
Copy link
Copy Markdown
Contributor

Previously, the GUI messed up newly loaded sliders so the upper bound
was the default value

Closes #223

@codecov-io
Copy link
Copy Markdown

Current coverage is 48.36%

Merging #224 into master will not affect coverage as of 8300b1f

@@            master    #224   diff @@
======================================
  Files          104     104       
  Stmts         2504    2504       
  Branches       253     253       
  Methods          0       0       
======================================
  Hit           1211    1211       
  Partial         45      45       
  Missed        1248    1248       

Review entire Coverage Diff as of 8300b1f


Uncovered Suggestions

  1. +0.48% via .../MainWindowView.java#84...95
  2. +0.44% via .../ExceptionAlert.java#95...105
  3. +0.40% via ...InputSocketView.java#36...45
  4. See 7 more...

Powered by Codecov. Updated on successful CI builds.

Previously, the GUI messed up newly loaded sliders so the upper bound
was the default value

Closes WPIRoboticsProjects#223
JLLeitschuh added a commit that referenced this pull request Dec 29, 2015
Correctly load range values
@JLLeitschuh JLLeitschuh merged commit 1116646 into WPIRoboticsProjects:master Dec 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants