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.

Sign up for GitHub

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

Jump to bottom

Signal pdu #4

Closed
wants to merge 10 commits into from
Closed

Signal pdu #4

wants to merge 10 commits into from

Commits on Sep 2, 2016

  1. Creating initial constructor for TransmitterPdu

    Sean Idzenga committed Sep 2, 2016
    Configuration menu
    Copy the full SHA
    a247402 View commit details
    Browse the repository at this point in the history

  2. reverse erroneous addition of duplicate transmitterpdu

    Sean Idzenga committed Sep 2, 2016
    Configuration menu
    Copy the full SHA
    630faea View commit details
    Browse the repository at this point in the history

Commits on Sep 6, 2016

  1. By adding the node-int64 library to the project we are able to work w… 

    …ith 64-bit integers (no arithmetic, unfortunately)

Completed InputStream.prototype.readLong()
    Sean Idzenga committed Sep 6, 2016
    Configuration menu
    Copy the full SHA
    d1bef9a View commit details
    Browse the repository at this point in the history

Commits on Sep 7, 2016

  1. Settled on 'long' npm package because of its ease of use and conversi… 

    …on to and from a buffer. readLong() and writeLong() implemented now.
    Sean Idzenga committed Sep 7, 2016
    Configuration menu
    Copy the full SHA
    52746bf View commit details
    Browse the repository at this point in the history

  2. Added decision tree entry for Signal PDUs

    Sean Idzenga committed Sep 7, 2016
    Configuration menu
    Copy the full SHA
    84fe9dc View commit details
    Browse the repository at this point in the history

Commits on Sep 8, 2016

  1. Fix for encode and decode binary dis in signalpdu - uses sample rate … 

    …to determine how many oneByteChunks are to be added to this.data
    Sean Idzenga committed Sep 8, 2016
    Configuration menu
    Copy the full SHA
    5d33c35 View commit details
    Browse the repository at this point in the history

Commits on Sep 12, 2016

  1. Replace all (n)ByteChunk classes with a singular Chunk class - this c… 

    …lass takes a single argument chunkSize to determine how many bytes long it should be, ie 2 produces a "TwoByteChunk" - usage doesn't change much:

var foobar = new Chunk(4);
foobar.initFromBinaryDIS(inputStream);
this.data.push(foobar.otherParameters);

Refactoring already complete.
    Sean Idzenga committed Sep 12, 2016
    Configuration menu
    Copy the full SHA
    1425d63 View commit details
    Browse the repository at this point in the history

  2. Corrected erroneous loop condition.

    Sean Idzenga committed Sep 12, 2016
    Configuration menu
    Copy the full SHA
    bc23697 View commit details
    Browse the repository at this point in the history

  3. "otherParameters" is an awkward name for the byte array and is mislea… 

    …ding, the byte arrays are not parameters. Changed to simply read "data".
    Sean Idzenga committed Sep 12, 2016
    Configuration menu
    Copy the full SHA
    c6c1b45 View commit details
    Browse the repository at this point in the history

Commits on Sep 19, 2016

  1. Refactor OneByteChunk() 

    Missed three instances of OneByteChunk() corrected to use dis.Chunk(1)
    Sean Idzenga committed Sep 19, 2016
    Configuration menu
    Copy the full SHA
    6213d31 View commit details
    Browse the repository at this point in the history