search

gopherdata / gophernotes

The Go kernel for Jupyter notebooks and nteract.
MIT License
3.8k stars 262 forks source link

update Dockerfile.DS to solve series type convertion bugs in kniren/gota #238

Closed DaruiShi closed 2 years ago

DaruiShi commented 2 years ago

As shown in Issue #201 , the bug is in the docker container for data science, when we try to convert some columns' data types, it shows reflect.Value.Convert: value of type reflect.Value cannot be converted to type series.Type.

We identified that the bug is from the go package kniren/gota. But the package go-gota/gota can work well. So I modified the Dockerfile.DS to replace kniren/gota with go-gota/gota.

Also, I updated the Alpine image version and some pre-installed packages and software which solves the cyclic imports error in issue #233 .

cosmos72 commented 2 years ago

Someone with more Docker expertise than me can please review this? Maybe @SpencerPark or @sbinet ?

sbinet commented 2 years ago

ping?

sbinet commented 2 years ago

superceded by https://github.com/gopherdata/gophernotes/pull/242