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.

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

reflect.Value.Convert: value of type reflect.Value cannot be converted to type series.Type #201

Open
MilesLin opened this issue Apr 11, 2020 · 4 comments

Comments

@MilesLin
Copy link

@MilesLin MilesLin commented Apr 11, 2020

I ran container by docker run -it -p 8888:8888 -v %CD%:/usr/share/notebooks gopherdata/gophernotes:latest-ds.
The code below is that I tried to converting a column into a different type, it got an error.
image

I found that the go version in the container is go1.11.4.

However, I ran the same code on my local, it works.
computer info:

  • Windows10
  • go version: go1.13.4 windows/amd64

Is that any version problem?

@0xckylee
Copy link

@0xckylee 0xckylee commented Apr 14, 2020

If you check the travis.yml file, CI for Go 1.11 version is not there anymore. There's only 1.14, 1.13, 1.12, and it's been updated recently in February. However, The Dockerfile points to building Go 1.11. Maybe try redownloading this source and rewrite part of the Dockerfile to build from 1.13 and start again while this issue is resolved?

language: go

go:
  - 1.14.x
  - 1.13.x
  - 1.12.x
  - master
@oneoneonepig
Copy link

@oneoneonepig oneoneonepig commented Apr 14, 2020

If you check the travis.yml file, CI for Go 1.11 version is not there anymore. There's only 1.14, 1.13, 1.12, and it's been updated recently in February. However, The Dockerfile points to building Go 1.11. Maybe try redownloading this source and rewrite part of the Dockerfile to build from 1.13 and start again while this issue is resolved?

language: go

go:
  - 1.14.x
  - 1.13.x
  - 1.12.x
  - master

You won't have to rewrite the Dockerfile, it pulls the latest go package in the alpine repo

&& apk --update-cache --allow-untrusted \
--repository http://dl-4.alpinelinux.org/alpine/edge/community \
--arch=x86_64 add \
go \

Since it was built an year ago, go 1.11 probably was the latest version at that time.

@0xckylee
Copy link

@0xckylee 0xckylee commented Apr 14, 2020

I digged into Dockerhub, and @oneoneonepig you are right the docker image was last pushed a year ago, while the latest Dockerfile source on this repo added support for 1.11 version. Looks like you'd have to try building this locally instead of using Docker @MilesLin.

@MilesLin
Copy link
Author

@MilesLin MilesLin commented Apr 15, 2020

I think that is the bug from github.com/kniren/gota which is the package built-in in the container.
I created a sample to demonstrate this issue.
image

here is the code.

import(
    "github.com/kniren/gota/dataframe"
    "github.com/kniren/gota/series"
)
type TestSt struct {
	Name string
	Score int `dataframe:"Score,int"`
}
data := []TestSt{}
	data = append(data, TestSt{
		Name: "Miles",
		Score: 21,
	})
df := dataframe.LoadStructs(data)
result := series.New(df.Col("Score"), series.Float, "Score")
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.