profile
viewpoint
Justin D. Harris juharris @Microsoft / @Maluuba Eastern Time Zone https://ca.linkedin.com/in/justindharris This is the OFFICIAL GitHub account for Justin D. Harris. I work from Toronto at @microsoft Montreal (formerly @Maluuba).

NiklasRosenstein/myo-python 155

Python bindings for the Myo SDK

juharris/switch-remoteplay 141

NOT AN OFFICIAL NINTENDO PRODUCT - Control your Switch remotely

juharris/moabb 1

Mother of All BCI Benchmarks

juharris/my-bot 1

A bot to replace me.

juharris/textworld-bots 1

Bots to play https://aka.ms/textworld.

juharris/altymeter 0

Monitor cryptocurrency prices.

juharris/awesome-question-answering 0

Resources, datasets, papers on Question Answering

juharris/community 0

A collection of documents and resources relating to initiatives from the Community Development group at MakerDAO.

juharris/examples 0

A set of examples around pytorch in Vision, Text, Reinforcement Learning, etc.

created tagjuharris/switch-remoteplay

tag0.6

NOT AN OFFICIAL NINTENDO PRODUCT - Control your Switch remotely

created time in 5 days

release juharris/switch-remoteplay

0.6

released time in 5 days

delete branch juharris/switch-remoteplay

delete branch : client-clickable

delete time in 5 days

push eventjuharris/switch-remoteplay

Justin D. Harris

commit sha a9b3f767c61030b302ca835ebda9d87f2faf0323

[UI] Clickable/Tappable Button and Sticks (#26) * client/Joystick: Move both joysticks with touch. * client: Support clicking and dragging joysticks with the mouse. * client: Handle passing around controllerState to buttons a little better * ui: Show when two buttons are touched at the same time. * README: Mouse/Touchscreen are supported. * ui: Get most buttons to show on mobile screens. * sendComand: Allow updating state. * client: Only keep one ControllerState. * client: Only run command once per press. * client: Bump to 0.6

view details

push time in 5 days

PR merged juharris/switch-remoteplay

[UI] Clickable/Tappable Button and Sticks

Click/Tap on the buttons or use the joysticks with your mouse or touch screen.

+707 -226

1 comment

21 changed files

juharris

pr closed time in 5 days

push eventjuharris/switch-remoteplay

Justin D. Harris

commit sha ff539ac4ecc2c18bd3b22888476eb5c00ac6d34c

client: Bump to 0.6

view details

push time in 5 days

push eventjuharris/switch-remoteplay

Justin D. Harris

commit sha 768bdacf7ede1ac1f7bb867349a0448b39e66b4d

client: Clarify some commends and warnings.

view details

push time in 5 days

Pull request review commentjuharris/switch-remoteplay

[UI] Clickable/Tappable Button and Sticks

 const buttonNameToStateMember: { [buttonName: string]: string } = { 	right: 'arrowRight', } +/**+ * Updates `controllerState`.+ * @param command A single command that can be represented by one state such as just pressing a button down or just letting a button back up but not tapping a button (e.g. 'x' which involves pushing down then back up). Can use '&' to combine commands.+ * @param controllerState The current state.+ * @param fullCommand The full command that `singleCommand` came from.+ */+function updateState(command: string, controllerState: ControllerState, fullCommand?: string): void {+	if (fullCommand === undefined) {+		fullCommand = command+	}+	for (let singleCommand of command.split('&')) {+		singleCommand = singleCommand.trim()+		const commandParts = singleCommand.split(/\s+/)+		if (commandParts.length < 2) {+			// A button might be tapped. Not really supported but it should not update the state.+			console.warn("Ignoring unrecognized part of command: \"%s\" from \"%s\"", singleCommand, fullCommand)

To make it clear that the command will still get sent, these warnings should be clarified to say that the state will not be updated for that part.

juharris

comment created time in 5 days

Pull request review commentjuharris/switch-remoteplay

[UI] Clickable/Tappable Button and Sticks

+import { createStyles, withStyles } from '@material-ui/core' import React from 'react'-import classes from './Joystick.module.css'+import { SendCommand } from '../../../key-binding/KeyBinding'+import { ControllerState } from '../ControllerState' -const Joystick: React.FunctionComponent<any> = (props: any) => {-	let joystickHolderClassList = classes.joystickHolder-	let joystickClassList = classes.joystick-	const movedThreshold = 0.15-	if (props.pressed) {-		joystickHolderClassList += " " + classes.pressed+const styles = () => createStyles({+	joystickHolder: {+		height: '6rem',+		width: '6rem',+		backgroundColor: '#111',+		borderRadius: '50%',+		margin: '1rem auto',+	},+	joystick: {+		height: '5rem',+		width: '5rem',+		backgroundColor: '#222',+		borderRadius: '50%',+		margin: '1rem auto',+		position: 'relative',+		top: '0.5rem',+	},+	pressed: {+		backgroundColor: '#999', 	}-	if (Math.abs(props.x) > movedThreshold || Math.abs(props.y) > movedThreshold) {-		joystickClassList += " " + classes.pressed+})++class Joystick extends React.Component<{+	name: string,+	sendCommand: SendCommand,+	controllerState: ControllerState,+	x: number, y: number,+	pressed: boolean,+	classes: any,+}> {+	// We can probably use the controllerState to track these but it might not be reliable+	// if another class creates a new state.+	/** Indicates if the stick has been moved since it has been selected. */+	moved = false+	/** Indicates if the stick have been pressed inwards. */

has

juharris

comment created time in 5 days

push eventjuharris/switch-remoteplay

Justin D. Harris

commit sha 09fa05d47c3d21c5cd5f6f8e508bdf8807ade9ab

client: Handle state more nicely with for macro commands with tapping a button.

view details

push time in 6 days

push eventjuharris/switch-remoteplay

Justin D. Harris

commit sha be89585d4b0cf8f8818aabc8c2f3474ad21c893d

client: Only run command once per press.

view details

push time in 6 days

push eventjuharris/switch-remoteplay

Justin D. Harris

commit sha f464cf858130d57a3cfa595584e4f0621da45f6c

client: Only keep one ControllerState.

view details

push time in 6 days

issue closedjuharris/switch-remoteplay

Capture Card in Hosted Client

Is it possible to use the hosted client that you provide with the video from a capture card?

closed time in 6 days

ciberharry97

issue commentjuharris/switch-remoteplay

Capture Card in Hosted Client

Not anymore. The hosted client just supports Mixer which is no longer available. When I try out another service, then it will be offered in a similar way.

ciberharry97

comment created time in 6 days

push eventjuharris/switch-remoteplay

Justin D. Harris

commit sha 4441d9f3e7c975f7f6eedae1611eddc8f52bae82

client/updateState: Handle combined commands.

view details

push time in 6 days

push eventjuharris/switch-remoteplay

Justin D. Harris

commit sha 51e8427e1e0216f6ae23ad3ba44ec604bf15f6af

sendComand: Allow updating state.

view details

push time in 6 days

pull request commentjuharris/switch-remoteplay

[UI] Clickable/Tappable Button and Sticks

I'm mostly happy with this. I'll sleep on it and update the Docker image before merging.

juharris

comment created time in 7 days

push eventjuharris/switch-remoteplay

Justin D. Harris

commit sha 9d02d7359fada64fedd7d4d94f09603d263b7083

ui: Get most buttons to show on mobile screens.

view details

push time in 7 days

push eventjuharris/switch-remoteplay

Justin D. Harris

commit sha a7f5c91b762328df64cbf1ab7d1e75186bda82da

README: Mouse/Touchscreen are supported.

view details

push time in 7 days

PR opened juharris/switch-remoteplay

[UI] Clickable/Tappable Button and Sticks

Click/Tap on the buttons or use the joysticks with your mouse or touch screen.

+515 -187

0 comment

12 changed files

pr created time in 7 days

push eventjuharris/switch-remoteplay

Justin D. Harris

commit sha 12875ddfedbfe1078f9f22d3d0f98f54ce2fbd1e

ui: Show when two buttons are touched at the same time.

view details

push time in 7 days

push eventjuharris/switch-remoteplay

Justin D. Harris

commit sha d4ec6d276e1442008cafe1b8f7081e4af70eb7ce

ui: Show when two buttons are touched at the same time.

view details

push time in 7 days

issue commentOpenMined/PySyft

[torch] Cannot set values at indices in a tensor

Can anyone advise on where to make the change the correct this?

juharris

comment created time in 10 days

push eventjuharris/switch-remoteplay

Justin Harris

commit sha 2cc9d202cb4291b41adaf25d86bf6e8474095c46

ui: Handle passing around controllerState to buttons a little better (WIP)

view details

push time in 12 days

push eventjuharris/switch-remoteplay

dependabot[bot]

commit sha f584fa3fdf8d63fe88e83f64ea11007c07e08c46

build(deps): bump elliptic from 6.5.2 to 6.5.3 in /website-client Bumps [elliptic](https://github.com/indutny/elliptic) from 6.5.2 to 6.5.3. - [Release notes](https://github.com/indutny/elliptic/releases) - [Commits](https://github.com/indutny/elliptic/compare/v6.5.2...v6.5.3) Signed-off-by: dependabot[bot] <support@github.com>

view details

Justin D. Harris

commit sha fcd5c8bbeb224a867fbc7dda4f549218884e4bd7

Merge pull request #25 from juharris/dependabot/npm_and_yarn/website-client/elliptic-6.5.3 build(deps): bump elliptic from 6.5.2 to 6.5.3 in /website-client

view details

push time in 13 days

PR merged juharris/switch-remoteplay

build(deps): bump elliptic from 6.5.2 to 6.5.3 in /website-client dependencies

Bumps elliptic from 6.5.2 to 6.5.3. <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/indutny/elliptic/commit/8647803dc3d90506aa03021737f7b061ba959ae1"><code>8647803</code></a> 6.5.3</li> <li><a href="https://github.com/indutny/elliptic/commit/856fe4d99fe7b6200556e6400b3bf585b1721bec"><code>856fe4d</code></a> signature: prevent malleability and overflows</li> <li>See full diff in <a href="https://github.com/indutny/elliptic/compare/v6.5.2...v6.5.3">compare view</a></li> </ul> </details> <br />

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

</details>

+3 -3

0 comment

1 changed file

dependabot[bot]

pr closed time in 13 days

push eventjuharris/switch-remoteplay

Justin D. Harris

commit sha d768d275fa1ac284d6ba071b662b0ade9f8ec6a6

client: Add TODOs for passing controllerState.

view details

push time in 20 days

push eventjuharris/switch-remoteplay

Justin D. Harris

commit sha 01f9bb1fc053aac1438c7435ba6d64ec83cfd680

client/touch: Update state and enable pressed control sticks inwards.

view details

push time in 20 days

push eventjuharris/PySyft

Justin D. Harris

commit sha dbb70f24cc55a7dca032fb06f1a8662cb15092a9

test: Add RNN Plan example (#3860) * test: Add RNN Plan example (WIP) This doesn't work yet. You can see many of the custom changes that I had to make to methods in PySyft (Search for "# FIX". It seems like the last piece to get this working is to get reshape/flatten/view in `forward` to work. See the FIXME. * test/rnn: Attempt to get a Plan with an RNN working. * test/plan/rnn: Got simple example working. * test/plan/rnn: Convert tabs to spaces. * test/plan/rnn: Lint. * test/plan/rnn: Lint. * test/plan/rnn: 100% test coverage. * Add output comparison & torchscript translate * test/plan/rnn: Torchscript works using frozen embeddings. * Add expected type to Placeholder to generate correct dummy vars * test/plan/rnn: Add clarifications. Take the sequence_length as input. * Placeholder: type -> dtype. * play.py: type -> dtype. * test/plan/rnn: Add clarifications for padding index. * test/plan/rnn: Add `train.add_translation(PlanTranslatorTfjs)` to show that it fails with `KeyError: 'getitem'` * test/plan/rnn: Be lenient around translating the Plan to Tensorflow.js. Co-authored-by: Justin Harris <justindharris@gmail.com> Co-authored-by: Vova Manannikov <vova@promail.spb.ru> Co-authored-by: Justin D. Harris <>

view details

push time in 22 days

delete branch juharris/PySyft

delete branch : test_rnn_plan

delete time in 22 days

startedaarmea/noise

started time in 22 days

delete branch juharris/Threepio

delete branch : pytorch-rsub

delete time in 23 days

delete branch juharris/Threepio

delete branch : py-translate-TranslationMissing

delete time in 23 days

push eventjuharris/Threepio

Justin D. Harris

commit sha 5ac3e67444bcf12f50aff3147475ec6162d92fa7

[python] translate: Throw TranslationMissing exception instead of KeyError (#121) * Fixed: Issue #73 (#108) * # addded documentation * # typos corrected * # added documentation * # improvements made Co-authored-by: Mike Nolan <me@michael-nolan.com> * py/translate: TranslationMissing exception instead of KeyError. * py/translate: Throw custom exception instead of KeyError. * Undo changes to aggregate_crawler_output.py * test_torch_tfjs.py: Use more consistent indenting. * test_torch_tfjs.py: Add newlines. * Threepio: Add check for `from_lang`. Co-authored-by: Ramesht Shukla <36106177+ramesht007@users.noreply.github.com> Co-authored-by: Mike Nolan <me@michael-nolan.com> Co-authored-by: Justin Harris <justindharris@gmail.com>

view details

push time in 23 days

push eventjuharris/PySyft

Héricles Emanuel

commit sha 8a65122db3c09a1bcc213c803be38a86a8e909ef

Grid module architecture refactoring (#3818) * module refactoring * delete http module * move static fl worker to syft.workers * update grid module architecture * update grid references * fix black issues * fix pep8 issues * fix type-checking issues * fix pep8 issues * fix some imports * fix black issues * update notebooks and docs

view details

Vova Manannikov

commit sha e330a38d56f5bb0bf9bc9e7eb4bef26a03615613

Add unit test for MNIST example tfjs translation (#3802) * Add unit test for MNIST example tfjs translation * Fix tfjs translation for mnist training example * Fix missing import * Update threepio dependency * Fix unit test

view details

Mike Nolan

commit sha bfadfac1151ab70b2adaf83ffd309db26c605558

update threepio to 0.2.0 (#3838) Threepio 0.2.0 is identical to 0.0.1

view details

Vova Manannikov

commit sha 1152786a106343e1f82aa3059a9c73abd6077a5c

Update Host Plan notebook to work with updated PyGrid (auth, tfjs plans) (#3839)

view details

Héricles Emanuel

commit sha 8d9e7975d0bf840b8061b7a60ef6385fd9a2fa5a

Update Static FL Notebooks (#3842) * fix speed test and auth methods * try to add avg plan in notebooks * fix black issues * Adding iterative plan setting * Moving the notebooks to a better place * Fix avg plan in Create Plan * Notebooks (#3847) * cleared speed testing * update execute plan notebook to use auth token * black reformat * Adding auth * A few changes * Run notebooks Co-authored-by: Patrick Cason <me@patrickcason.com> Co-authored-by: Vova Manannikov <vova@promail.spb.ru> Co-authored-by: varun khare <varunkhare1234@gmail.com>

view details

Vova Manannikov

commit sha fa8f8c31a8033d784bdc9cdb0c6d085104efce09

Fix wording (#3850)

view details

Andrew Trask

commit sha ed8d0c69dd01a7e1f2c7a3b30823dc348ec45f36

Update setup.py

view details

Patrick Cason

commit sha cc9e27461ab0b4062afa206cd0214ea658ee18ad

Update README.md

view details

Nick Doiron

commit sha f692b992fe14fab7fe5c2006d087a683a856d914

update install instructions in notebooks (#3830)

view details

Patrick Cason

commit sha 9a804235f78d9301649a7b9d01094f29442daa64

Adding bandit plan, initial

view details

Patrick Cason

commit sha 2787fb01d9b9adc876894a96dd46fb5ca4f70887

Cleaning up the notebook for bandit

view details

Patrick Cason

commit sha 0a6c04bc56f682608790f1fe7f54a05b6abd1c96

Bandit plan is finished again

view details

Sachin Kumar

commit sha bc5337507047b325e79cb9b2060abac4bdc08f3f

Namespace dynamic_fl_client route (#3853) * Namespace dynamic_fl_client route * Add vertical bar to url

view details

Théo Ryffel

commit sha e5d9a27784ea8b57efa9fe2d42546829d2bfc721

Improve remote garbage collection so support batched GC (#3805) * Add a parameters to batch GC calls * Add tests * Lint * Fix tests * Update requirements.txt * Respond to review * Fix protobuf serde tests * Add a trash max_capacity * Update requirements.txt

view details

Théo Ryffel

commit sha aaf373985b79030a3676c4021373fa0aab23b1a6

Improve public mul and div for AdditiveSharingTensor (#3835) * Clean and refactor the change * Improve confidentiality in public mul

view details

Vova Manannikov

commit sha 1e1706ea3eca84b1ada2600ec95612c6646d0ec1

Static fl/get model notebook (#3861) * Add notebook to show getting model from pygrid * Add note about latest checkpoint

view details

Théo Ryffel

commit sha 1ee500768073b2fd7f106357fbf7c6b48e575faa

Optimize communication in AvgPool, MaxPool and AdaptivePooling (#3836) * Add optimized versions of Avg, Max, and adaptive pooling * Add tests for Avg and Max Pool * Lint * Fix conv2d to support fixed precision * Fix pooling to support dim < 4 * Clean code * Fix conv and add tests for SNN + FSS * Clean code and add test for maxpool with kernel_size=3

view details

Justin D. Harris

commit sha 96f07097dd473227e9ee25f51c5a80c68bd328fe

autograd: return self for += and -=. (#3872) Co-authored-by: Justin Harris <justindharris@gmail.com>

view details

Patrick Cason

commit sha 1ec66b064651ee5dfadeaae1a8ff90bb689ba508

Namespacing for static FL

view details

Patrick Cason

commit sha 57f7f7394415a4aa9fc90d47767819e7dbaddd42

Fixing dynamic FL client

view details

push time in 23 days

pull request commentOpenMined/PySyft

test: Add RNN Plan example (WIP)

Tests should pass now. An unreleased version of 3p0 with my translations for select and __rsub__ is required to translate the Plan to Tensorflow.js. The example is lenient and will not fail if these translations are missing.

juharris

comment created time in 23 days

push eventjuharris/PySyft

Justin D. Harris

commit sha b860702c05301e7ff3f96cfd19cc48112fcb5ef5

test/plan/rnn: Be lenient around translating the Plan to Tensorflow.js.

view details

push time in 23 days

push eventjuharris/Threepio

Justin D. Harris

commit sha 98f7b4223dc92fd2bab7bcb9ed628ba5282b70aa

py/translate: Map PyTorch tensor __rsub__ to sub for Tensorflow.js. (#122) * py/translate: Map PyTorch tensor __rsub__ to sub for Tensorflow.js. * py/translate: Lint Co-authored-by: Justin Harris <justindharris@gmail.com>

view details

Justin D. Harris

commit sha ace7b8863545cc9f7ec28596f2985ff1d8738588

torch/tfjs: Translate select to gather. (#120) * torch/tfjs: Translate select to gather. * torch/tfjs: Add JS tests for translating select to gather. * Remove extra import in tests. * pytorch.js: Make prettier. * pytorch.js: Add comment. Mainly to hopefully get the tests to run. * pytorch.js: Prettier. * pytorch.js: Prettier. * js: Correct test. Co-authored-by: Justin Harris <justindharris@gmail.com>

view details

push time in 23 days

push eventjuharris/Threepio

Justin Harris

commit sha de45e3b423ee7ca56be29b6a17643d9bc80d7ea0

Threepio: Add check for `from_lang`.

view details

push time in 23 days

pull request commentOpenMined/PySyft

test: Add RNN Plan example (WIP)

Thanks @iamtrask ! The main issue is that translating the Plan to Tensorflow.js doesn't work yet but I'm making PRs to Threepio to get it working.

juharris

comment created time in 23 days

push eventjuharris/Threepio

Justin Harris

commit sha 7ba340d096fd9662b3f053a6c7417ec32c62f9ae

test_torch_tfjs.py: Add newlines.

view details

push time in 23 days

push eventjuharris/Threepio

Justin Harris

commit sha 58b6ea54c7dc3e6a85ed44b4b3635297a3bc8e61

test_torch_tfjs.py: Use more consistent indenting.

view details

Justin Harris

commit sha ba1a599fb9b5bac1ded0c8c18320345d3d189560

Merge branch 'dev' of github.com:OpenMined/Threepio into py-translate-TranslationMissing

view details

Justin Harris

commit sha e4111148f54ec27dc997ebc2f2a97de5b3d65d2a

Merge branch 'py-translate-TranslationMissing' of github.com:juharris/Threepio into py-translate-TranslationMissing

view details

push time in 23 days

delete branch juharris/Threepio

delete branch : torch-select

delete time in 23 days

pull request commentOpenMined/Threepio

[python] translate: Throw TranslationMissing exception instead of KeyError

@Nolski I "rebased"

juharris

comment created time in 23 days

push eventjuharris/Threepio

Justin Harris

commit sha ad3a6039d4651241c42dd4ed88e212be8cfac7de

Undo changes to aggregate_crawler_output.py

view details

push time in 23 days

pull request commentOpenMined/Threepio

torch/tfjs: Translate select to gather.

@Nolski I resolved the conflicts.

juharris

comment created time in 23 days

push eventjuharris/Threepio

Justin D. Harris

commit sha 98f7b4223dc92fd2bab7bcb9ed628ba5282b70aa

py/translate: Map PyTorch tensor __rsub__ to sub for Tensorflow.js. (#122) * py/translate: Map PyTorch tensor __rsub__ to sub for Tensorflow.js. * py/translate: Lint Co-authored-by: Justin Harris <justindharris@gmail.com>

view details

Justin Harris

commit sha 76b0371212761ad6fb16ec6756e1f876060b7299

Merge base

view details

push time in 23 days

push eventjuharris/Threepio

Justin D. Harris

commit sha 98f7b4223dc92fd2bab7bcb9ed628ba5282b70aa

py/translate: Map PyTorch tensor __rsub__ to sub for Tensorflow.js. (#122) * py/translate: Map PyTorch tensor __rsub__ to sub for Tensorflow.js. * py/translate: Lint Co-authored-by: Justin Harris <justindharris@gmail.com>

view details

Justin Harris

commit sha a5daa19c17da0f7dcc2f495cdc3fecef60130ba6

Merge base

view details

push time in 23 days

pull request commentOpenMined/Threepio

[python] translate: Throw TranslationMissing exception instead of KeyError

Those are the changes from master. I branched off of master instead of dev by mistake. I can rebase.

juharris

comment created time in 23 days

push eventjuharris/Threepio

Justin Harris

commit sha 39c04507db29aa70a4b835be2bd787639c9d3849

py/translate: Lint

view details

push time in 24 days

PR opened OpenMined/Threepio

py/translate: Map PyTorch tensor __rsub__ to sub for Tensorflow.js.

Description

This will help with using tanh in a PySyft Plan since TanhBackward has grad_self_ = grad * (1 - self.self_.tanh() ** 2).

Affected Dependencies

How has this been tested?

Tests in PR.

Checklist

+74 -3

0 comment

7 changed files

pr created time in 24 days

create barnchjuharris/Threepio

branch : pytorch-rsub

created branch time in 24 days

PR opened OpenMined/Threepio

[python] translate: Throw TranslationMissing exception instead of KeyError

Description

Throw TranslationMissing exception instead of KeyError

Affected Dependencies

How has this been tested?

Python tests.

Checklist

+143 -13

0 comment

4 changed files

pr created time in 24 days

push eventjuharris/Threepio

Justin Harris

commit sha 11b874232a2110a1a088cc4223f76ccacc077453

py/translate: Throw custom exception instead of KeyError.

view details

push time in 24 days

create barnchjuharris/Threepio

branch : py-translate-TranslationMissing

created branch time in 24 days

push eventjuharris/Threepio

Justin Harris

commit sha 574d20906acfdf60ee42310f212d55a306a0045b

js: Correct test.

view details

push time in 24 days

push eventjuharris/Threepio

Justin Harris

commit sha 8c3475d612c9b380efa3000ae99482b594afb05f

pytorch.js: Prettier.

view details

push time in 24 days

push eventjuharris/Threepio

Justin Harris

commit sha 00126412f940b9048548627d7b767fb62dcee4e1

pytorch.js: Prettier.

view details

push time in 24 days

push eventjuharris/Threepio

Justin Harris

commit sha 18027a49860412069d39b0039d580af706939707

pytorch.js: Add comment. Mainly to hopefully get the tests to run.

view details

push time in 24 days

push eventjuharris/Threepio

Justin Harris

commit sha 5a14a5f6bb454ada663f3421b1f8a1a033bad91c

pytorch.js: Make prettier.

view details

push time in 24 days

push eventjuharris/Threepio

Justin Harris

commit sha 7b7cb9d3b970209b87ce133dc2449e987ae2f686

Remove extra import in tests.

view details

push time in 24 days

PR opened OpenMined/Threepio

torch/tfjs: Translate select to gather.

Description

Translate PyTorch's gather to Tensorflow.js' gather. I am trying to write a Plan in Python with PyTorch that uses tensor.select and I would like it translated so that I can use syft,js. Did I do this right?

How has this been tested?

Added tests.

Checklist

+104 -6

0 comment

7 changed files

pr created time in 24 days

push eventjuharris/Threepio

Justin Harris

commit sha 7789cded380ebd944682589aeaad8c2fa4ad35e7

torch/tfjs: Add JS tests for translating select to gather.

view details

push time in 24 days

create barnchjuharris/Threepio

branch : torch-select

created branch time in 24 days

fork juharris/Threepio

A multi-language library for translating commands between PyTorch, TensorFlow, and TensorFlow.js

fork in 24 days

push eventjuharris/PySyft

Justin Harris

commit sha 62620258e821c297958c8c919ef3bb43e2b04a41

test/plan/rnn: Add `train.add_translation(PlanTranslatorTfjs)` to show that it fails with `KeyError: 'getitem'`

view details

push time in 25 days

push eventjuharris/PySyft

Justin Harris

commit sha 5f4e396ade08359afd6c3a3293eb5ddc2c17860b

test/plan/rnn: Add clarifications for padding index.

view details

push time in 25 days

pull request commentOpenMined/PySyft

test: Add RNN Plan example

@vvmnnnkv I can't tell why those tests fail. Some of those pass on my machine. Others fail but I think it's because I don't have Tensorflow set up.

juharris

comment created time in a month

push eventjuharris/PySyft

Justin Harris

commit sha 47f8a08dcf52417509c444522dbaf0adfcc4b7ca

play.py: type -> dtype.

view details

push time in a month

push eventjuharris/PySyft

Théo Ryffel

commit sha 1ee500768073b2fd7f106357fbf7c6b48e575faa

Optimize communication in AvgPool, MaxPool and AdaptivePooling (#3836) * Add optimized versions of Avg, Max, and adaptive pooling * Add tests for Avg and Max Pool * Lint * Fix conv2d to support fixed precision * Fix pooling to support dim < 4 * Clean code * Fix conv and add tests for SNN + FSS * Clean code and add test for maxpool with kernel_size=3

view details

Justin Harris

commit sha 6c7e986328f61e1935a0391060395533a6d03893

Merge branch 'master' of github.com:OpenMined/PySyft into test_rnn_plan

view details

Justin D. Harris

commit sha 96f07097dd473227e9ee25f51c5a80c68bd328fe

autograd: return self for += and -=. (#3872) Co-authored-by: Justin Harris <justindharris@gmail.com>

view details

Justin Harris

commit sha b16f0f3b97014f74f761e30225b6760511bd931c

Merge branch 'test_rnn_plan' of github.com:juharris/PySyft into test_rnn_plan

view details

Justin Harris

commit sha 0802aff0a63f9ab48b034d5f02d9d1ee729fec7d

Merge branch 'master' of github.com:OpenMined/PySyft into test_rnn_plan

view details

Justin Harris

commit sha baae5610cf1f3c1390906168cf96b881791edadb

test/plan/rnn: Add clarifications. Take the sequence_length as input.

view details

Justin Harris

commit sha 1a7dea7bcfc09beb636aa63d23e588420096ba4b

Placeholder: type -> dtype.

view details

push time in a month

push eventjuharris/PySyft

Vova Manannikov

commit sha a473dedfd60e0e54a289ccb5117587493fd5ca44

Add expected type to Placeholder to generate correct dummy vars

view details

Justin D. Harris

commit sha b679a2cbceb5282ec4786fc848d6c0f3015f6c69

Merge pull request #1 from vvmnnnkv/fix/placeholder-dtype Add expected type to Placeholder to generate correct dummy vars

view details

push time in a month

PR merged juharris/PySyft

Add expected type to Placeholder to generate correct dummy vars
  • Add expected type to Placeholder to generate correct dummy vars
  • Make RNN test use nn.Embedding inside the Plan again
+36 -20

0 comment

3 changed files

vvmnnnkv

pr closed time in a month

Pull request review commentjuharris/PySyft

Add expected type to Placeholder to generate correct dummy vars

 def create_placeholders(args_shape):             mapped_shapes.append(tuple(map(lambda y: 1 if y == -1 else y, shape)))          return [-            syft.framework.hook.create_zeros(shape, requires_grad=False) for shape in mapped_shapes+            syft.framework.hook.create_zeros(+                shape, dtype=args_types[i] if i < len(args_types) else None, requires_grad=False+            )+            for i, shape in enumerate(mapped_shapes)

It would be nice to use zip here. I can make that change after merging.

vvmnnnkv

comment created time in a month

push eventjuharris/switch-remoteplay

Justin Harris

commit sha 30f42b6f366d302fc0e6bce625b5c63a63cd5e4f

client: Support clicking and dragging joysticks with the mouse.

view details

push time in a month

PR opened OpenMined/PySyft

autograd: return self for += and -=.

Description

return self in AutogradTensor's += and -=. Otherwise the tensor would be None and not usable afterwards.

How has this been tested?

Ran test_autograd.py

Checklist

+30 -0

0 comment

2 changed files

pr created time in a month

create barnchjuharris/PySyft

branch : autograd-iadd-isub-return

created branch time in a month

push eventjuharris/PySyft

Justin Harris

commit sha 48953387919389968a9977112e8a9655a08e571a

test/plan/rnn: Torchscript works using frozen embeddings.

view details

push time in a month

push eventjuharris/PySyft

Justin Harris

commit sha 96fb3dc1719e86b14e33af0d361ffa95fb0a4fdf

test/plan/rnn: 100% test coverage.

view details

push time in a month

push eventjuharris/PySyft

Théo Ryffel

commit sha e5d9a27784ea8b57efa9fe2d42546829d2bfc721

Improve remote garbage collection so support batched GC (#3805) * Add a parameters to batch GC calls * Add tests * Lint * Fix tests * Update requirements.txt * Respond to review * Fix protobuf serde tests * Add a trash max_capacity * Update requirements.txt

view details

Théo Ryffel

commit sha aaf373985b79030a3676c4021373fa0aab23b1a6

Improve public mul and div for AdditiveSharingTensor (#3835) * Clean and refactor the change * Improve confidentiality in public mul

view details

Vova Manannikov

commit sha 1e1706ea3eca84b1ada2600ec95612c6646d0ec1

Static fl/get model notebook (#3861) * Add notebook to show getting model from pygrid * Add note about latest checkpoint

view details

Justin Harris

commit sha fb1ed10c16774a005b83aaf2a5b095799a20e639

Merge branch 'master' of github.com:OpenMined/PySyft into test_rnn_plan

view details

push time in a month

push eventjuharris/PySyft

Justin Harris

commit sha 53ce75b2447977b07c6156f5bca9251cbb4ad363

test/plan/rnn: Lint.

view details

push time in a month

push eventjuharris/PySyft

Justin Harris

commit sha 4d56d7bef399ad558afef7f1b06bb5c56e49b84b

test/plan/rnn: Lint.

view details

push time in a month

push eventjuharris/PySyft

Justin Harris

commit sha 53ff8e0c46b63c78465ab82d627fff417a40e855

test/plan/rnn: Convert tabs to spaces.

view details

push time in a month

push eventjuharris/PySyft

Justin Harris

commit sha 606b707891c903bad1c0906513261bb9fda8bed9

test/plan/rnn: Got simple example working.

view details

push time in a month

push eventjuharris/PySyft

Justin Harris

commit sha 3cd533bbde4a83dc302cfd80f9bd39058e828aab

test/rnn: Attempt to get a Plan with an RNN working.

view details

push time in a month

PR opened OpenMined/PySyft

test: Add RNN Plan example (WIP)

Description

I'm trying to create a language model in a Plan. I'm trying to mix the Creating a Plan tutorial and a word based language model from https://github.com/pytorch/examples/tree/master/word_language_model

This doesn't work yet. You can see many of the custom changes that I had to make to methods in PySyft (search for "# FIX"). It seems like the last piece to get this working is to get reshape/flatten/view in forward to work. See the FIXME.

I'll add more details on this later. I'm mostly just submitting this in case anyone can help get it working. @vvmnnnkv

Affected Dependencies

List any dependencies that are required for this change.

How has this been tested?

I ran the file I submitted with pytest.

Checklist

+167 -0

0 comment

1 changed file

pr created time in a month

create barnchjuharris/PySyft

branch : test_rnn_plan

created branch time in a month

issue commentMaluuba/nlg-eval

METEOR issue

To fully "install" Java, make sure that javac and java command are available in your PATH. Try running javac -h and java -h to check.

gaobb

comment created time in a month

issue commentMaluuba/nlg-eval

Correctly Installed But Failed to Run (AttributeError: 'Meteor' object has no attribute 'meteor_p')

It looks like it's having a problem running the Meteor JAR file: FileNotFoundError: [WinError 2] 系统找不到指定的文件。 What happens if you just run where java in your terminal?

ChawDoe

comment created time in a month

fork juharris/examples

A set of examples around pytorch in Vision, Text, Reinforcement Learning, etc.

fork in a month

push eventjuharris/PySyft

Théo Ryffel

commit sha 5c8e7b78eba693ea29e07eac7476aa7517e5dfef

Fix issues in handle_func_command for syft tensors (#3807) * Fix issues in handle_func_command in syft tensors * Fix handle_func_cmd in native

view details

push time in a month

push eventjuharris/PySyft

Justin D. Harris

commit sha 424cbb1d4e84a3cc37b0b869a252f99eb7981995

test/torch/tensors: Add a test for __set_item__ for https://github.com/OpenMined/PySyft/issues/3825

view details

push time in a month

create barnchjuharris/PySyft

branch : torch_set_item

created branch time in a month

fork juharris/PySyft

A library for encrypted, privacy preserving machine learning

https://www.openmined.org/

fork in a month

issue openedOpenMined/PySyft

[torch] Cannot set values at indices in a tensor

Description

Code like:

t = torch.zeros((2, 3), dtype=torch.float)
indices = torch.rand_like(t) > 0.5
t[indices] = 1

is fine when not hooked but fails when Torch is hooked.

How to Reproduce

import syft as sy
import torch


def set_indices():
	t = torch.zeros((2, 3), dtype=torch.float)
	indices = torch.rand_like(t) > 0.5
	t[indices] = 1
	print("Set indices")


# The first call works fine.
set_indices()
hook = sy.TorchHook(torch)
# This call fails.
set_indices()
Set indices
Traceback (most recent call last):
  File "code.py", line 16, in <module>
    set_indices()
  File "code.py", line 8, in set_indices
    t[indices] = 1
  File "[REDACTED]\lib\site-packages\syft\generic\frameworks\hook\hook.py", line 170, in overloaded_native_method
    raise route_method_exception(e, self, args, kwargs)
  File "[REDACTED]\lib\site-packages\syft\generic\frameworks\hook\hook.py", line 166, in overloaded_native_method
    response = method(*args, **kwargs)
IndexError: The shape of the mask [3] at index 0 does not match the shape of the indexed tensor [2, 3] at index 0

Expected Behavior

Should be able to assign a value.

System Information

  • OS: Windows
  • OS Version: 10
  • Language Version: Python 3.7.7
  • Package Manager Version: Anaconda
  • syft==0.2.6 (Installed via Pip)

created time in a month

startedOpenMined/KotlinSyft

started time in a month

issue commentMaluuba/nlg-eval

assertion error

Thanks for trying out our library. Can you give some more details like what the input the was that caused this? Is there anything else helpful above this stacktrace? That message "invalid value encountered..." could be related.

A-Rain

comment created time in a month

issue closedjuharris/switch-remoteplay

how do i figure out what my servers address is to connect

how do i figure out what my servers address is to connect

closed time in 2 months

shinykody

issue commentjuharris/switch-remoteplay

how do i figure out what my servers address is to connect

Hey, thanks for trying out the project. There are a few ways. If you're using a Raspberry Pi then you might be able to use http://raspberrypi. What you want is the local/internal/private IP for the device. There are a few ways listed at https://opensource.com/article/18/5/how-find-ip-address-linux I like ip addr where you might find something like 192.xxx.xxx.xxx where each x is a digit. You'll need to input that with the port on the client website. For example: http://192.168.27.121:5000 since 5000 is the default port for the server.

shinykody

comment created time in 2 months

push eventjuharris/switch-remoteplay

Justin Harris

commit sha 91af1e291f0481672a93bae7ef540de2761b095f

client/Joystick: Move both joysticks with touch.

view details

push time in 2 months

issue commentmicrosoft/0xDeCA10B

Storing models for cheap on Factom

Thanks for your interest in this project! That's a good suggestion. Storing lots of data in Ethereum isn't ideal but I have some concerns with other storage methods. Maybe you already saw my FAQ about it: https://github.com/microsoft/0xDeCA10B#what-about-storing-models-off-chain ? I've looked into Factom a bit and some of those concerns might apply to Factom as well. The information on their site was vague. Here are some questions for my notes or you to answer if you happen to know the answers:

  1. Is it cheap to update data? Factom advertises that data is immutable but if I can efficiently update a model then that might be good to do rather than storing a new model for every update.
  2. We might have to use an oracle to get the model if it is stored externally, this cost might be too high for some models.
  3. Is the data storage persistent? A peer-to-peer network is mentioned and if it is like IPFS, then it's not reliable.
kirit93

comment created time in 2 months

more