profile
viewpoint

asimshankar/go-tensorflow 21

Toying around with TensorFlow in Go

asimshankar/java-tensorflow 16

Toying around with the TensorFlow Java API

asimshankar/amarok-remote 5

Remotely control amarok from a web browser like Safari on the iPhone

asimshankar/bn256 2

Experimenting with C-bindings for bn256 pairings (https://godoc.org/golang.org/x/crypto/bn256)

asimshankar/imagej-tensorflow 2

Toying around with TensorFlow image models in an ImageJ plugin

asimshankar/triangles 2

Go+V23+Mobile

asimshankar/visionapi 1

Toying around with the Google and Microsoft vision APIs

asimshankar/edward 0

A probabilistic programming language in TensorFlow. Deep generative models, variational inference.

asimshankar/prometheus-operator 0

Prometheus Operator creates/configures/manages Prometheus clusters atop Kubernetes

delete tag neevaco/slack

delete tag : v0.6.1

delete time in 4 months

created tagneevaco/slack

tagv0.6.3

Slack API in Go

created time in 4 months

created tagneevaco/slack

tagv0.6.1

Slack API in Go

created time in 4 months

delete tag neevaco/slack

delete tag : v0.6.1-neeva

delete time in 4 months

created tagneevaco/slack

tagv0.6.1-neeva

Slack API in Go

created time in 4 months

delete tag neevaco/slack

delete tag : v0.6.1+neeva

delete time in 4 months

delete tag neevaco/slack

delete tag : v0.6.2

delete time in 4 months

delete tag neevaco/slack

delete tag : v0.6.1

delete time in 4 months

delete branch neevaco/slack

delete branch : fix-579

delete time in 4 months

push eventneevaco/slack

Asim Shankar

commit sha b0be807e6b557c43da031a7fe7a53186713a17e4

Do not throw error when unknown block types and block element types are encountered (#1)

view details

push time in 4 months

PR merged neevaco/slack

Reviewers
Patch fix for https://github.com/nlopes/slack/issues/579

Patch fix to v0.6.0 for https://github.com/nlopes/slack/issues/579

It seems there is active discussion on the GitHub issue and we are appreciative of the original fix. The plan is to temporarily commit this change as a workaround and then resync with the upstream repository when the official fix is in.

Steps:

git remote add acaloiaro https://github.com/acaloiaro/slack.git
git fetch acaloiaro
# From https://github.com/acaloiaro/slack/pull/1/commits/1a8dd0bd9c08699f9869f75d945ae142a3a16e78
git cherry-pick 1a8dd0bd9c08699f9869f75d945ae142a3a16e78
+30 -2

0 comment

3 changed files

asimshankar

pr closed time in 4 months

PR opened neevaco/slack

Reviewers
Patch fix for https://github.com/nlopes/slack/issues/579

Patch fix to v0.6.0 for https://github.com/nlopes/slack/issues/579

It seems there is active discussion on the GitHub issue and we are appreciative of the original fix. The plan is to temporarily commit this change as a workaround and then resync with the upstream repository when the official fix is in.

Steps:

git remote add acaloiaro https://github.com/acaloiaro/slack.git
git fetch acaloiaro
# From https://github.com/acaloiaro/slack/pull/1/commits/1a8dd0bd9c08699f9869f75d945ae142a3a16e78
git cherry-pick 1a8dd0bd9c08699f9869f75d945ae142a3a16e78
+30 -2

0 comment

3 changed files

pr created time in 4 months

create barnchneevaco/slack

branch : fix-579

created branch time in 4 months

push eventneevaco/slack

push time in 4 months

create barnchneevaco/slack

branch : neeva

created branch time in 4 months

fork asimshankar/prometheus-operator

Prometheus Operator creates/configures/manages Prometheus clusters atop Kubernetes

fork in 4 months

pull request commentneevaco/dropbox-sdk-go-unofficial

Update to latest spec

One thought - we should perhaps name our branches "version-<neeva>" or something, so we don't diverge/conflict from upstream versions.

aaronstaley

comment created time in 4 months

Pull request review commentneevaco/dropbox-sdk-go-unofficial

Fix cgen to support union types correctly

 const ( 	hostAPI       = "api" 	hostContent   = "content" 	hostNotify    = "notify"-	sdkVersion    = "5.4.0"+	sdkVersion    = "5.0.0"

Why this change? (Seems to be downgrading a few minor versions?)

aaronstaley

comment created time in 5 months

Pull request review commentneevaco/dropbox-sdk-go-unofficial

Fix cgen to support union types correctly

 const ( func (u *PathRoot) UnmarshalJSON(body []byte) error { 	type wrap struct { 		dropbox.Tagged+		// Root : Paths are relative to the authenticating user's root namespace+		// (This results in `PathRootError.invalid_root` if the user's root+		// namespace has changed.).+		Root json.RawMessage `json:"root,omitempty"`

Same comment here, I'm not following the RawMessage usage here.

aaronstaley

comment created time in 5 months

Pull request review commentneevaco/dropbox-sdk-go-unofficial

Fix cgen to support union types correctly

 const ( func (u *LaunchResultBase) UnmarshalJSON(body []byte) error { 	type wrap struct { 		dropbox.Tagged+		// AsyncJobId : This response indicates that the processing is+		// asynchronous. The string is an id that can be used to obtain the+		// status of the asynchronous job.+		AsyncJobId json.RawMessage `json:"async_job_id,omitempty"`

Why make this a json.RawMessage and then unmarshal it again in line 65? Could this be a string and then at that point:

  • Either just set `u.AsyncJobId = w.AsyncJobId=, or
  • Get rid of wrap altogether and Unmarshal directly into u?

I'm probably missing something.

aaronstaley

comment created time in 5 months

more