Merge pull request #245 from c-bata/remove-statik

Remove `statik` dependency

Author: c-bata

.github/workflows/go-examples.yml

@@ -33,5 +33,4 @@ jobs:
         ./bin/sobol
         ./bin/cmaes_blackhole
         ./bin/simple_tpe
-        ./bin/concurrency
         ./bin/trialnotify

Makefile

@@ -11,7 +11,7 @@ GODOC ?= godoc
 
 .DEFAULT_GOAL := help
 
-SOURCES := $(shell find . -name "*.go" | grep -v -e "sobol/direction_numbers.go" -e "dashboard/statik/statik.go")
+SOURCES := $(shell find . -name "*.go" | grep -v -e "sobol/direction_numbers.go")
 
 .PHONY: fmt
 fmt: $(SOURCES) ## Formatting source codes.
@@ -49,13 +49,6 @@ build: ## Build example command lines.
 	$(GO) build -o ./bin/goptuna -ldflags "$(LDFLAGS)" cmd/main.go
 	./_examples/build.sh
 
-.PHONY: build-dashboard
-build-dashboard: ## Build dashboard and bundle it.
-	docker build -t c-bata/goptuna-dashboard ./dashboard
-	docker run -it --rm \
-		-v `PWD`/dashboard/statik:/usr/src/statik \
-		c-bata/goptuna-dashboard
-
 .PHONY: help
 help: ## Show help text
 	@echo "Commands:"

_examples/build.sh

@@ -10,11 +10,11 @@ set -ex
 
 go build -o ${BIN_DIR}/cmaes ${DIR}/cmaes/main.go
 go build -o ${BIN_DIR}/cmaes_blackhole ${DIR}/cmaes/blackhole/main.go
-go build -o ${BIN_DIR}/concurrency ${DIR}/concurrency/main.go
 go build -o ${BIN_DIR}/enqueue_trial ${DIR}/enqueue_trial/main.go
 go build -o ${BIN_DIR}/trialnotify ${DIR}/trialnotify/main.go
 go build -o ${BIN_DIR}/signalhandling ${DIR}/signalhandling/main.go
 go build -o ${BIN_DIR}/simple_rdb ${DIR}/simple_rdb/main.go
 go build -o ${BIN_DIR}/simple_tpe ${DIR}/simple_tpe/main.go
 go build -o ${BIN_DIR}/sobol ${DIR}/sobol/main.go
-go build -o ${BIN_DIR}/gorgonia_iris ${DIR}/gorgonia_iris/main.go
+#go build -o ${BIN_DIR}/concurrency ${DIR}/concurrency/main.go
+#go build -o ${BIN_DIR}/gorgonia_iris ${DIR}/gorgonia_iris/main.go

dashboard/.node-version

@@ -2,11 +2,7 @@
 
 ## Running dashboard
 
-Pre-built JavaScript files are embedded using:
-
-* `go:embed` for Go1.16 or later.
-* [rakyll/statik](https://github.com/rakyll/statik) for Go1.15 or older.
-
+Pre-built JavaScript files are embedded using `go:embed`.
 So it's ok you just run `goptuna dashboard` command like this:
 
 ```
@@ -43,19 +39,9 @@ Use "goptuna [command] --help" for more information about a command.
 
 ## How to compile TypeScript files
 
-### Compiling TypeScript files and embedding to Go using Docker
-
-You just run to `make build-dashboard` to compile TypeScript files and embedding to Go.
-
-```
-$ docker build -t c-bata/goptuna-dashboard ./dashboard
-$ docker run -it --rm -v `PWD`/dashboard/statik:/usr/src/statik c-bata/goptuna-dashboard
-```
-
-
 ### Compiling TypeScript files manually
 
-Node.js v14.14.0 is required to compile TypeScript files.
+Node.js v20 is required to compile TypeScript files.
 
 ```
 $ npm install
@@ -80,13 +66,6 @@ $ npm run build:prd
 
 </details>
 
-### Embeddeing to Go using rakyll/statik
-
-```
-$ statik -src=./public -include=bundle.js,bundle.js.LICENSE.txt
-```
-
-
 ## Running Dashboard server for reloading
 
 Please pass `-tags=develop` the custom build tag to return JS files inside local directory (not embedded files).

dashboard/Dockerfile

@@ -1,4 +1,4 @@
-//go:build !develop && go1.16
+//go:build !develop
 
 package dashboard
 

dashboard/README.md

@@ -6,7 +6,6 @@ require (
 	github.com/google/uuid v1.3.1
 	github.com/gorilla/mux v1.8.0
 	github.com/jinzhu/gorm v1.9.16
-	github.com/rakyll/statik v0.1.7
 	github.com/spf13/cobra v1.7.0
 	gonum.org/v1/gonum v0.14.0
 	gorm.io/driver/mysql v1.0.3