dskinner / material

material design for gomobile [DISCONTINUED]
BSD 2-Clause "Simplified" License
88 stars 7 forks source link

dependency on a function in snd #12

Closed joeblew99 closed 7 years ago

joeblew99 commented 7 years ago

Am wanting to play around, but currently cant run Material due to a func in snd changing. i am pretty sure on this because i deleted the snd and simplex folder before go getting.

x-MacBook-Pro:bin apple$ go get dasa.cc/material
# dasa.cc/material
../src/dasa.cc/material/anim.go:59: anim.Sig.SampleUnit undefined (type snd.Discrete has no field or method SampleUnit)
../src/dasa.cc/material/anim.go:87: interp.Sig.SampleUnit undefined (type snd.Discrete has no field or method SampleUnit)

x-MacBook-Pro:bin apple$ go version
go version go1.7.4 darwin/amd64

x-MacBook-Pro:bin apple$ go env
GOARCH="amd64"
GOBIN="/Users/apple/workspace/go/bin"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/apple/workspace/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/wp/ff6sz9qs6g71jnm12nj2kbyw0000gp/T/go-build161440261=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
joeblew99 commented 7 years ago

fix was:

dt := float32(interp.Sig.Interpolate(t))

dskinner commented 7 years ago

Thanks. Interpolate may actually be more appropriate here, but for now I've updated it to call Sig.Index (renamed from SampleUnit) to maintain previous behavior.

joeblew99 commented 7 years ago

cheers :) cool work.

Have you tried running an of this on windows 10 ?

On Mon, Jan 30, 2017 at 5:01 PM Daniel Skinner notifications@github.com wrote:

Thanks. Interpolate may actually be more appropriate here, but for now I've updated it to call Sig.Index (renamed from SampleUnit) to maintain previous behavior.

— You are receiving this because you authored the thread.

Reply to this email directly, view it on GitHub https://github.com/dskinner/material/issues/12#issuecomment-276084613, or mute the thread https://github.com/notifications/unsubscribe-auth/ALcacw8QI_wy8vQ3nlAQD48dCzXYFrJsks5rXfs0gaJpZM4LxeRc .

dskinner commented 7 years ago

I haven't, though I'm under the impression it should work now.

joeblew99 commented 7 years ago

yes it works for me.

thanks. i think this is ready to be closed