Added Wi-Fi Programming Support #55
Merged
Conversation
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
…bc20cc). Untested. Also added chromestore/ assets folder through Git LFS (Large File Storage).
…stake that prevented Git LFS from managing the chromestore folder contents.
…y Git LFS and is readable by all Git users.
…me a download failure occurs. Resets to initial delay for each new download attempt the user initiates.
…ors on some Chromebooks. Cleaned up potential errors from clearing the deferred promise timer without invalidating the timer value after. Increased the mblDeliveryTime [Rx hardware to OS slack time] value from 250 ms to 500 ms (for testing).
…est effectiveness on all devices.
…tResetDelay to 1 ms to see effect on problem machine.
…problem on customer's computer.
… Modules settings tab.
…led depending on connection status. Also changed the button text to 'waiting...' when waiting to connect. Removed time stamp from html-based log.
…lds to ports object in preparation for combining wx_modules into it.
…lso removed potential bug where portPattern could have appeared anywhere within the port name; now it must appear at the start as intended.
…ved serial's addPort() through findPort() functions and declarations to new port.js file. Adjusted index to load files in required order and added some declarations to index.js so they are accessible by port and serial (and wx soon). Retested with serial.
…Cleaned up comments.
…. This solves the problem from previous implementation where a string MAC address would be seen as a path value. Set life value.
…s fabricated name when accessing or comparing port's path.
…record add or update. This saves processing time.
… support both wired and wireless. Verified wired still works correctly.
…any) so that listener can filter out other tcp conversations.
…g: pSocket isn't updated in propComm when send() creates it.
…ecause it's updated later in ports[]. Changed to store port in propComm instead and fixed scope bug near start of hearFromProp(). Works as intended now.
…use Uint8Array view instead of conversion on ArrayBuffer.
…rop set a slice of the Body into the propComm.mblRespBuf for further comparison against what's expected.
…nts, hearFromProp() to have an additional stage for WX Wi-Fi Module responses, and changeBaudrate() to tap into the hearFromProp() promise chain before returning.
…uctured send() to create a pSocket when needed. Next needs send() to be a promise so that execution doesn't continue before transmission with it's wireless.
… use the HTTP-command service or the Telnet (pass thru) service. Bumped up userDeliveryTime for network use. Extended port object to hold phSocket and ptSocket (one for each of HTTP-command or Telnet services). Still working on setPropCommTimer() and closePort() to close socket if timeout.
…/or forget the HTTP-command or Telnet socket.
…ngeBaudrate outside of the start of protocol moment. As it turns out, it wouldn't have mattered (it was set properly anyway) but this is for good measure in potential future uses.
… prevent closing a null port.
…ime by 3x. Both of these because errors happened on occasion presumably due to network congestion.
…ascript-obfuscator tool causes bug in socket code within serial.js. Updated to v0.18.1 (on dev box; not included in repository) and updated MakeRelease to use javascript-obfuscator's new CLI options.
…ay still be a slight problem; this may be the source of strange behavior at the moment it tries to changeBaudrate near end of wireless protocol.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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 is a hefty PR with many changes meant to add Wi-Fi support.
This PR brings Wi-Fi Programming Support plus a number of bug fixes and enhancements along the way that were either necessary or easy to solve. Also includes a fix to a bug that seemed to occur remotely (only) on an instructor's computer.