Improved data validation - #81
Open
STFCLovell wants to merge 17 commits into
Open
Conversation
… existing method of fixed tool ID's and changing some of the window properties to fit the new textbox
…om the variable list similarly to the name otherwise import wouldn't work.
… is between 200 and 299 inclusive with an output to the UI console.
…tement to call home repeatability test. Further tests needed for different homing routines and set positions
… bit more sense. Homing test routine finished.
…. Changed default XL80 Path to reflect it's path from an installer
…ing then back to true when test finishes
… when the test is run
…ch provides inherent data validation, this should prevent crashes from invalid data being entered on the axis ID
…the list of elements that get disabled when a test is running
… now software launches with previously populated videos and resets to 0 when reset settings used
…being used for simple moves to keep them consistent with the modern addressing of these values in NcAxis Tests
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Added validation to axis number, set point, and velocity fields by making them have consistent behaviour with rest of the text fields using their UiVal and Val properties instead of converting raw text to the correct data type.