Skip to content

VectorHand encoding -- and some encoding progress#990

Merged
Amarcolina merged 5 commits intodevelopfrom
feature-vectorhand-encoding
Apr 6, 2018
Merged

VectorHand encoding -- and some encoding progress#990
Amarcolina merged 5 commits intodevelopfrom
feature-vectorhand-encoding

Conversation

@vabrador
Copy link
Copy Markdown
Contributor

@vabrador vabrador commented Apr 5, 2018

Early stages for a Networking module that are being pulled in to support some dependencies in the Hierarchy Recorder.

I am not a huge fan of the random Encoding interfaces, they have not been fully fleshed out.

  • VectorHand should have functioning I/O
  • We should probably figure out whether we need the interfaces at all, I can just remove them because they probably are overkill right now? The idea is they can provide the common link for different types of hand encodings in a future NetworkedProvider.
  • The files are placed in Core/Scripts/Encodings because they probably are going to be dependencies for various modules including Networking and the Hierarchy Recorder.

@vabrador vabrador changed the title VectorHand encoding -- and some encoding prograss VectorHand encoding -- and some encoding progress Apr 5, 2018
@Amarcolina Amarcolina self-requested a review April 5, 2018 23:41
@vabrador
Copy link
Copy Markdown
Contributor Author

vabrador commented Apr 6, 2018

Heyy what do you know, ReadBytes doesn't actually work. Fixin' soon. Don't.. merge.

image

@Amarcolina Amarcolina merged commit 6450d70 into develop Apr 6, 2018
@Amarcolina Amarcolina deleted the feature-vectorhand-encoding branch April 6, 2018 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants