source is not respected while revisiting dependencies

go version go1.8.3 linux/amd64
dep - v0.3.0-22-g11758a7
TL;DR;
grouped write of manifest, lock and vendor: error while writing out vendor tree: error while exporting k8s.io/client-go: /tmp/dep324558830/vendor/k8s.io/client-go/.github/PULL_REQUEST_TEMPLATE.md already exists, no checkout

$ dep ensure -v
Root project is "github.com/tnozicka/openshift-acme"
 14 transitively valid internal packages
 22 external packages imported from 13 projects
(0)   ✓ select (root)
(1)	? attempt golang.org/x/crypto with 1 pkgs; 1 versions to try
(1)	    try golang.org/x/crypto@master
(1)	✓ select golang.org/x/crypto@master w/1 pkgs
(2)	? attempt k8s.io/api with 1 pkgs; 1 versions to try
(2)	    try k8s.io/api@master
(2)	✓ select k8s.io/api@master w/1 pkgs
(3)	? attempt github.com/ugorji/go with 1 pkgs; 1 versions to try
(3)	    try github.com/ugorji/go@master
(3)	✓ select github.com/ugorji/go@master w/1 pkgs
(4)	? attempt github.com/onsi/gomega with 1 pkgs; 6 versions to try
(4)	    try<EMAIL_ADDRESS>(4)	✓ select<EMAIL_ADDRESS>w/12 pkgs
(5)	? attempt gopkg.in/yaml.v2 with 1 pkgs; 1 versions to try
(5)	    try gopkg.in/yaml.v2@v2
(5)	✓ select gopkg.in/yaml.v2@v2 w/1 pkgs
(6)	? attempt k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes) with 1 pkgs; 3 versions to try
(6)	    try k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes)@5ada62a9f5a4f36199e94d64aa1698386935526a
(6)	✓ select k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes)@5ada62a9f5a4f36199e94d64aa1698386935526a w/355 pkgs
(7)	? revisit golang.org/x/crypto to add 1 pkgs
(7)	  ✓ include 4 more pkgs from golang.org/x/crypto@master
(7)	? revisit github.com/onsi/gomega to add 2 pkgs
(8)	  ✓ include 4 more pkgs from<EMAIL_ADDRESS>(7)	? attempt github.com/evanphx/json-patch with 1 pkgs; 1 versions to try
(9)	    try github.com/evanphx/json-patch@master
(9)	✓ select github.com/evanphx/json-patch@master w/1 pkgs
(8)	? attempt github.com/armon/circbuf with 1 pkgs; 1 versions to try
(10)      try github.com/armon/circbuf@master
(10)  ✓ select github.com/armon/circbuf@master w/1 pkgs
(9)	? attempt github.com/exponent-io/jsonpath with 1 pkgs; 1 versions to try
(11)      try github.com/exponent-io/jsonpath@master
(11)  ✓ select github.com/exponent-io/jsonpath@master w/1 pkgs
(10)  ? attempt golang.org/x/text with 2 pkgs; 1 versions to try
(12)      try golang.org/x/text@master
(12)  ✓ select golang.org/x/text@master w/9 pkgs
(11)  ? attempt github.com/golang/glog with 1 pkgs; 1 versions to try
(13)      try github.com/golang/glog@master
(13)  ✓ select github.com/golang/glog@master w/1 pkgs
(12)  ? attempt golang.org/x/exp (from https://github.com/golang/exp) with 1 pkgs; 2 versions to try
(14)      try golang.org/x/exp (from https://github.com/golang/exp)@292a51b8d262487dab23a588950e8052d63d9113
(14)  ✓ select golang.org/x/exp (from https://github.com/golang/exp)@292a51b8d262487dab23a588950e8052d63d9113 w/1 pkgs
(13)  ? attempt github.com/golang/groupcache with 1 pkgs; 1 versions to try
(15)      try github.com/golang/groupcache@master
(15)  ✓ select github.com/golang/groupcache@master w/1 pkgs
(14)  ? attempt k8s.io/apimachinery with 30 pkgs; 3 versions to try
(16)      try k8s.io/apimachinery@master
(16)  ✓ select k8s.io/apimachinery@master w/51 pkgs
(15)  ? attempt github.com/elazarl/goproxy with 1 pkgs; 2 versions to try
(17)      try<EMAIL_ADDRESS>(17)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(16)  ? attempt vbom.ml/util with 1 pkgs; 1 versions to try
(18)      try vbom.ml/util@master
(18)  ✓ select vbom.ml/util@master w/1 pkgs
(17)  ? revisit k8s.io/apimachinery to add 4 pkgs
(19)    ✓ include 26 more pkgs from k8s.io/apimachinery@master
(17)  ? attempt golang.org/x/oauth2 with 2 pkgs; 1 versions to try
(20)      try golang.org/x/oauth2@master
(20)  ✓ select golang.org/x/oauth2@master w/5 pkgs
(18)  ? attempt github.com/mitchellh/go-wordwrap with 1 pkgs; 1 versions to try
(21)      try github.com/mitchellh/go-wordwrap@master
(21)  ✓ select github.com/mitchellh/go-wordwrap@master w/1 pkgs
(19)  ? attempt github.com/docker/spdystream with 1 pkgs; 1 versions to try
(22)      try github.com/docker/spdystream@master
(22)  ✓ select github.com/docker/spdystream@master w/2 pkgs
(20)  ? attempt github.com/google/gofuzz with 1 pkgs; 1 versions to try
(23)      try github.com/google/gofuzz@master
(23)  ✓ select github.com/google/gofuzz@master w/1 pkgs
(21)  ? attempt github.com/prometheus/common with 2 pkgs; 2 versions to try
(24)      try github.com/prometheus/common@master
(24)  ✓ select github.com/prometheus/common@master w/3 pkgs
(22)  ? attempt github.com/golang/protobuf with 1 pkgs; 1 versions to try
(25)      try github.com/golang/protobuf@master
(25)  ✓ select github.com/golang/protobuf@master w/1 pkgs
(23)  ? attempt github.com/onsi/ginkgo with 1 pkgs; 15 versions to try
(26)      try<EMAIL_ADDRESS>(26)  ✓ select<EMAIL_ADDRESS>w/18 pkgs
(24)  ? attempt github.com/ghodss/yaml with 1 pkgs; 2 versions to try
(27)      try<EMAIL_ADDRESS>(27)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(25)  ? attempt github.com/containernetworking/cni with 2 pkgs; 26 versions to try
(28)      try<EMAIL_ADDRESS>(28)  ✓ select<EMAIL_ADDRESS>w/6 pkgs
(26)  ? attempt gopkg.in/inf.v0 with 1 pkgs; 1 versions to try
(29)      try<EMAIL_ADDRESS>(29)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(27)  ? attempt google.golang.org/appengine with 2 pkgs; 2 versions to try
(30)      try<EMAIL_ADDRESS>(30)  ✓ select<EMAIL_ADDRESS>w/10 pkgs
(28)  ? attempt github.com/davecgh/go-spew with 1 pkgs; 3 versions to try
(31)      try<EMAIL_ADDRESS>(31)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(29)  ? attempt github.com/godbus/dbus with 1 pkgs; 7 versions to try
(32)      try<EMAIL_ADDRESS>(32)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(30)  ? attempt golang.org/x/net with 1 pkgs; 5 versions to try
(33)      try golang.org/x/net@master
(33)  ✓ select golang.org/x/net@master w/1 pkgs
(31)  ? attempt github.com/kr/pty with 1 pkgs; 4 versions to try
(34)      try<EMAIL_ADDRESS>(34)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(32)  ? attempt github.com/spf13/viper with 1 pkgs; 2 versions to try
(35)      try github.com/spf13/viper@master
(35)  ✓ select github.com/spf13/viper@master w/1 pkgs
(33)  ? attempt github.com/mitchellh/mapstructure with 1 pkgs; 1 versions to try
(36)      try github.com/mitchellh/mapstructure@master
(36)  ✓ select github.com/mitchellh/mapstructure@master w/1 pkgs
(34)  ? attempt github.com/docker/go-connections with 1 pkgs; 8 versions to try
(37)      try<EMAIL_ADDRESS>(37)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(35)  ? attempt github.com/influxdata/influxdb with 1 pkgs; 344 versions to try
(38)      try<EMAIL_ADDRESS>(38)  ✓ select<EMAIL_ADDRESS>w/3 pkgs
(36)  ? attempt github.com/openshift/origin with 2 pkgs; 92 versions to try
(39)      try<EMAIL_ADDRESS>(39)  ✓ select<EMAIL_ADDRESS>w/17 pkgs
(37)  ? revisit k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes) to add 5 pkgs
(40)    ✓ include 15 more pkgs from k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes)@5ada62a9f5a4f36199e94d64aa1698386935526a
(37)  ? attempt github.com/dgrijalva/jwt-go with 1 pkgs; 16 versions to try
(41)      try<EMAIL_ADDRESS>(41)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(38)  ? revisit k8s.io/apimachinery to add 2 pkgs
(42)    ✓ include 31 more pkgs from k8s.io/apimachinery@master
(38)  ? revisit golang.org/x/net to add 1 pkgs
(43)    ✓ include 3 more pkgs from golang.org/x/net@master
(38)  ? revisit golang.org/x/text to add 3 pkgs
(44)    ✓ include 17 more pkgs from golang.org/x/text@master
(38)  ? attempt github.com/go-openapi/spec with 1 pkgs; 2 versions to try
(45)      try github.com/go-openapi/spec@master
(45)  ✓ select github.com/go-openapi/spec@master w/1 pkgs
(39)  ? attempt gopkg.in/gcfg.v1 with 1 pkgs; 4 versions to try
(46)      try<EMAIL_ADDRESS>(46)  ✓ select<EMAIL_ADDRESS>w/4 pkgs
(40)  ? revisit golang.org/x/net to add 1 pkgs
(47)    ✓ include 2 more pkgs from golang.org/x/net@master
(40)  ? attempt k8s.io/client-go (from https://github.com/kubernetes/client-go) with 5 pkgs; 16 versions to try
(48)      try k8s.io/client-go (from<EMAIL_ADDRESS>(48)  ✓ select k8s.io/client-go (from<EMAIL_ADDRESS>w/80 pkgs
(41)  ? revisit k8s.io/apimachinery to add 2 pkgs
(49)    ✓ include 9 more pkgs from k8s.io/apimachinery@master
(41)  ? attempt github.com/prometheus/client_model with 1 pkgs; 3 versions to try
(50)      try github.com/prometheus/client_model@master
(50)  ✓ select github.com/prometheus/client_model@master w/1 pkgs
(42)  ? attempt gopkg.in/warnings.v0 with 1 pkgs; 2 versions to try
(51)      try<EMAIL_ADDRESS>(51)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(43)  ? attempt github.com/hashicorp/hcl with 1 pkgs; 5 versions to try
(52)      try github.com/hashicorp/hcl@master
(52)  ✓ select github.com/hashicorp/hcl@master w/9 pkgs
(44)  ? attempt github.com/spf13/pflag with 1 pkgs; 3 versions to try
(53)      try<EMAIL_ADDRESS>(53)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(45)  ? revisit golang.org/x/net to add 1 pkgs
(54)    ✓ include 4 more pkgs from golang.org/x/net@master
(45)  ? revisit golang.org/x/text to add 2 pkgs
(55)    ✓ include 9 more pkgs from golang.org/x/text@master
(45)  ? attempt github.com/docker/engine-api with 6 pkgs; 17 versions to try
(56)      try<EMAIL_ADDRESS>(56)  ✓ select<EMAIL_ADDRESS>w/14 pkgs
(46)  ? revisit github.com/docker/go-connections to add 2 pkgs
(57)    ✓ include 2 more pkgs from<EMAIL_ADDRESS>(46)  ? revisit golang.org/x/net to add 1 pkgs
(58)    ✓ include 1 more pkgs from golang.org/x/net@master
(46)  ? revisit golang.org/x/net to add 1 pkgs
(59)    ✓ include 1 more pkgs from golang.org/x/net@master
(46)  ? attempt github.com/go-openapi/jsonreference with 1 pkgs; 1 versions to try
(60)      try github.com/go-openapi/jsonreference@master
(60)  ✓ select github.com/go-openapi/jsonreference@master w/1 pkgs
(47)  ? attempt github.com/vishvananda/netlink with 1 pkgs; 5 versions to try
(61)      try github.com/vishvananda/netlink@master
(61)  ✓ select github.com/vishvananda/netlink@master w/2 pkgs
(48)  ? attempt github.com/blang/semver with 1 pkgs; 21 versions to try
(62)      try<EMAIL_ADDRESS>(62)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(49)  ? attempt github.com/howeyc/gopass with 1 pkgs; 1 versions to try
(63)      try github.com/howeyc/gopass@master
(63)  ✓ select github.com/howeyc/gopass@master w/1 pkgs
(50)  ? revisit golang.org/x/crypto to add 1 pkgs
(64)    ✓ include 1 more pkgs from golang.org/x/crypto@master
(50)  ? attempt github.com/fsouza/go-dockerclient with 1 pkgs; 2 versions to try
(65)      try github.com/fsouza/go-dockerclient@master
(65)  ✓ select github.com/fsouza/go-dockerclient@master w/1 pkgs
(51)  ? attempt github.com/vishvananda/netns with 1 pkgs; 1 versions to try
(66)      try github.com/vishvananda/netns@master
(66)  ✓ select github.com/vishvananda/netns@master w/1 pkgs
(52)  ? revisit k8s.io/client-go to add 15 pkgs
(67)    ✓ include 29 more pkgs from<EMAIL_ADDRESS>(52)  ? revisit k8s.io/apimachinery to add 1 pkgs
(68)    ✓ include 1 more pkgs from k8s.io/apimachinery@master
(52)  ? attempt github.com/spf13/jwalterweatherman with 1 pkgs; 2 versions to try
(69)      try github.com/spf13/jwalterweatherman@master
(69)  ✓ select github.com/spf13/jwalterweatherman@master w/1 pkgs
(53)  ? attempt github.com/pelletier/go-toml with 1 pkgs; 14 versions to try
(70)      try<EMAIL_ADDRESS>(70)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(54)  ? attempt google.golang.org/api with 4 pkgs; 2 versions to try
(71)      try google.golang.org/api@master
(71)  ✓ select google.golang.org/api@master w/6 pkgs
(55)  ? attempt golang.org/x/sys with 1 pkgs; 1 versions to try
(72)      try golang.org/x/sys@master
(72)  ✓ select golang.org/x/sys@master w/1 pkgs
(56)  ? attempt github.com/juju/ratelimit with 1 pkgs; 1 versions to try
(73)      try github.com/juju/ratelimit@master
(73)  ✓ select github.com/juju/ratelimit@master w/1 pkgs
(57)  ? revisit k8s.io/apimachinery to add 5 pkgs
(74)    ✓ include 9 more pkgs from k8s.io/apimachinery@master
(57)  ? revisit github.com/onsi/ginkgo to add 2 pkgs
(75)    ✓ include 6 more pkgs from<EMAIL_ADDRESS>(57)  ? attempt github.com/appc/spec with 2 pkgs; 28 versions to try
(76)      try<EMAIL_ADDRESS>(76)  ✓ select<EMAIL_ADDRESS>w/4 pkgs
(58)  ? attempt github.com/emicklei/go-restful with 2 pkgs; 16 versions to try
(77)      try<EMAIL_ADDRESS>(78)  ✗   github.com/emicklei/go-restful at v2.2.1 has problem subpkg(s):
(78)    	github.com/emicklei/go-restful/swagger is missing;  required by:
(78)    		k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes) at 5ada62a9f5a4f36199e94d64aa1698386935526a
(78)    		k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes) at 5ada62a9f5a4f36199e94d64aa1698386935526a
(78)    		k8s.io/client-go (from https://github.com/kubernetes/client-go) at v3.0.0
(78)    		k8s.io/client-go at v3.0.0
(77)      try<EMAIL_ADDRESS>(78)  ✗   github.com/emicklei/go-restful at 2.2.0 has problem subpkg(s):
(78)    	github.com/emicklei/go-restful/swagger is missing;  required by:
(78)    		k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes) at 5ada62a9f5a4f36199e94d64aa1698386935526a
(78)    		k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes) at 5ada62a9f5a4f36199e94d64aa1698386935526a
(78)    		k8s.io/client-go (from https://github.com/kubernetes/client-go) at v3.0.0
(78)    		k8s.io/client-go at v3.0.0
(77)      try<EMAIL_ADDRESS>(78)  ✗   github.com/emicklei/go-restful at 2.1.0 has problem subpkg(s):
(78)    	github.com/emicklei/go-restful/swagger is missing;  required by:
(78)    		k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes) at 5ada62a9f5a4f36199e94d64aa1698386935526a
(78)    		k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes) at 5ada62a9f5a4f36199e94d64aa1698386935526a
(78)    		k8s.io/client-go (from https://github.com/kubernetes/client-go) at v3.0.0
(78)    		k8s.io/client-go at v3.0.0
(77)      try<EMAIL_ADDRESS>(78)  ✗   github.com/emicklei/go-restful at 2.0.0 has problem subpkg(s):
(78)    	github.com/emicklei/go-restful/swagger is missing;  required by:
(78)    		k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes) at 5ada62a9f5a4f36199e94d64aa1698386935526a
(78)    		k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes) at 5ada62a9f5a4f36199e94d64aa1698386935526a
(78)    		k8s.io/client-go (from https://github.com/kubernetes/client-go) at v3.0.0
(78)    		k8s.io/client-go at v3.0.0
(77)      try<EMAIL_ADDRESS>(77)  ✓ select<EMAIL_ADDRESS>w/3 pkgs
(59)  ? attempt github.com/coreos/go-oidc with 3 pkgs; 3 versions to try
(78)      try github.com/coreos/go-oidc@master
(78)  ✓ select github.com/coreos/go-oidc@master w/5 pkgs
(60)  ? attempt github.com/jonboulle/clockwork with 1 pkgs; 2 versions to try
(79)      try<EMAIL_ADDRESS>(79)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(61)  ? attempt github.com/docker/go-units with 1 pkgs; 5 versions to try
(80)      try<EMAIL_ADDRESS>(80)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(62)  ? attempt github.com/go-openapi/swag with 1 pkgs; 1 versions to try
(81)      try github.com/go-openapi/swag@master
(81)  ✓ select github.com/go-openapi/swag@master w/1 pkgs
(63)  ? attempt github.com/go-playground/log with 2 pkgs; 17 versions to try
(82)      try<EMAIL_ADDRESS>(82)  ✓ select<EMAIL_ADDRESS>w/2 pkgs
(64)  ? attempt github.com/go-playground/ansi with 1 pkgs; 1 versions to try
(83)      try github.com/go-playground/ansi@master
(83)  ✓ select github.com/go-playground/ansi@master w/1 pkgs
(65)  ? attempt github.com/coreos/go-semver with 1 pkgs; 3 versions to try
(84)      try<EMAIL_ADDRESS>(84)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(66)  ? attempt cloud.google.com/go with 1 pkgs; 15 versions to try
(85)      try<EMAIL_ADDRESS>(85)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(67)  ? attempt github.com/google/cadvisor with 10 pkgs; 73 versions to try
(86)      try<EMAIL_ADDRESS>(86)  ✓ select<EMAIL_ADDRESS>w/40 pkgs
(68)  ? revisit golang.org/x/sys to add 1 pkgs
(87)    ✓ include 1 more pkgs from golang.org/x/sys@master
(68)  ? attempt github.com/abbot/go-http-auth with 1 pkgs; 4 versions to try
(88)      try<EMAIL_ADDRESS>(88)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(69)  ? revisit golang.org/x/crypto to add 1 pkgs
(89)    ✓ include 2 more pkgs from golang.org/x/crypto@master
(69)  ? attempt github.com/opencontainers/runc with 6 pkgs; 20 versions to try
(90)      try<EMAIL_ADDRESS>(90)  ✓ select<EMAIL_ADDRESS>w/7 pkgs
(70)  ? attempt go4.org with 1 pkgs; 2 versions to try
(91)      try go4.org@master
(91)  ✓ select go4.org@master w/1 pkgs
(71)  ? attempt github.com/PuerkitoBio/purell with 1 pkgs; 8 versions to try
(92)      try<EMAIL_ADDRESS>(92)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(72)  ? revisit golang.org/x/text to add 1 pkgs
(93)    ✓ include 6 more pkgs from golang.org/x/text@master
(72)  ? revisit golang.org/x/net to add 1 pkgs
(94)    ✓ include 1 more pkgs from golang.org/x/net@master
(72)  ? attempt github.com/mistifyio/go-zfs with 1 pkgs; 10 versions to try
(95)      try<EMAIL_ADDRESS>(95)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(73)  ? attempt github.com/PuerkitoBio/urlesc with 1 pkgs; 1 versions to try
(96)      try github.com/PuerkitoBio/urlesc@master
(96)  ✓ select github.com/PuerkitoBio/urlesc@master w/1 pkgs
(74)  ? attempt github.com/go-openapi/jsonpointer with 1 pkgs; 1 versions to try
(97)      try github.com/go-openapi/jsonpointer@master
(97)  ✓ select github.com/go-openapi/jsonpointer@master w/1 pkgs
(75)  ? attempt github.com/docker/distribution with 2 pkgs; 62 versions to try
(98)      try<EMAIL_ADDRESS>(98)  ✓ select<EMAIL_ADDRESS>w/8 pkgs
(76)  ? attempt github.com/spf13/cobra with 1 pkgs; 2 versions to try
(99)      try github.com/spf13/cobra@master
(99)  ✓ select github.com/spf13/cobra@master w/1 pkgs
(77)  ? revisit github.com/openshift/origin to add 2 pkgs
(100)   ✓ include 136 more pkgs from<EMAIL_ADDRESS>(77)  ? revisit k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes) to add 54 pkgs
(101)   ✓ include 219 more pkgs from k8s.io/kubernetes (from github.com/tnozicka/openshift-kubernetes)@5ada62a9f5a4f36199e94d64aa1698386935526a
(77)  ? revisit k8s.io/apimachinery to add 2 pkgs
(102)   ✓ include 30 more pkgs from k8s.io/apimachinery@master
(77)  ? attempt google.golang.org/grpc with 2 pkgs; 19 versions to try
(103)     try<EMAIL_ADDRESS>(103) ✓ select<EMAIL_ADDRESS>w/14 pkgs
(78)  ? revisit golang.org/x/net to add 2 pkgs
(104)   ✓ include 4 more pkgs from golang.org/x/net@master
(78)  ? revisit github.com/onsi/ginkgo to add 2 pkgs
(105)   ✓ include 4 more pkgs from<EMAIL_ADDRESS>(78)  ? revisit k8s.io/client-go to add 1 pkgs
(106)   ✓ include 17 more pkgs from<EMAIL_ADDRESS>(78)  ? attempt github.com/vjeantet/ldapserver with 1 pkgs; 3 versions to try
(107)     try<EMAIL_ADDRESS>(107) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(79)  ? attempt github.com/magiconair/properties with 1 pkgs; 28 versions to try
(108)     try<EMAIL_ADDRESS>(108) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(80)  ? revisit github.com/opencontainers/runc to add 1 pkgs
(109)   ✓ include 1 more pkgs from<EMAIL_ADDRESS>(80)  ? attempt github.com/MakeNowJust/heredoc with 1 pkgs; 1 versions to try
(110)     try github.com/MakeNowJust/heredoc@master
(110) ✓ select github.com/MakeNowJust/heredoc@master w/1 pkgs
(81)  ? revisit k8s.io/client-go to add 2 pkgs
(111)   ✓ include 44 more pkgs from<EMAIL_ADDRESS>(81)  ? revisit github.com/docker/distribution to add 2 pkgs
(112)   ✓ include 2 more pkgs from<EMAIL_ADDRESS>(81)  ? attempt github.com/spf13/afero with 1 pkgs; 1 versions to try
(113)     try github.com/spf13/afero@master
(113) ✓ select github.com/spf13/afero@master w/2 pkgs
(82)  ? attempt github.com/pelletier/go-buffruneio with 1 pkgs; 3 versions to try
(114)     try<EMAIL_ADDRESS>(114) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(83)  ? attempt google.golang.org/genproto with 1 pkgs; 3 versions to try
(115)     try google.golang.org/genproto@master
(115) ✓ select google.golang.org/genproto@master w/1 pkgs
(84)  ? revisit github.com/golang/protobuf to add 1 pkgs
(116)   ✓ include 2 more pkgs from github.com/golang/protobuf@master
(84)  ? attempt github.com/inconshreveable/mousetrap with 1 pkgs; 2 versions to try
(117)     try<EMAIL_ADDRESS>(117) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(85)  ? attempt github.com/vjeantet/asn1-ber with 1 pkgs; 1 versions to try
(118)     try github.com/vjeantet/asn1-ber@master
(118) ✓ select github.com/vjeantet/asn1-ber@master w/1 pkgs
(86)  ? attempt github.com/spf13/cast with 1 pkgs; 3 versions to try
(119)     try<EMAIL_ADDRESS>(119) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(87)  ? attempt github.com/prometheus/client_golang with 1 pkgs; 15 versions to try
(120)     try<EMAIL_ADDRESS>(120) ✓ select<EMAIL_ADDRESS>w/2 pkgs
(88)  ? attempt github.com/hashicorp/golang-lru with 1 pkgs; 2 versions to try
(121)     try github.com/hashicorp/golang-lru@master
(121) ✓ select github.com/hashicorp/golang-lru@master w/2 pkgs
(89)  ? attempt github.com/prometheus/procfs with 1 pkgs; 2 versions to try
(122)     try github.com/prometheus/procfs@master
(122) ✓ select github.com/prometheus/procfs@master w/2 pkgs
(90)  ? revisit github.com/docker/distribution to add 7 pkgs
(123)   ✓ include 13 more pkgs from<EMAIL_ADDRESS>(90)  ? attempt github.com/coreos/etcd with 2 pkgs; 120 versions to try
(124)     try<EMAIL_ADDRESS>(124) ✓ select<EMAIL_ADDRESS>w/10 pkgs
(91)  ? revisit google.golang.org/grpc to add 3 pkgs
(125)   ✓ include 3 more pkgs from<EMAIL_ADDRESS>(91)  ? attempt k8s.io/apiserver with 16 pkgs; 2 versions to try
(126)     try k8s.io/apiserver@master
(126) ✓ select k8s.io/apiserver@master w/31 pkgs
(92)  ? revisit k8s.io/apimachinery to add 2 pkgs
(127)   ✓ include 22 more pkgs from k8s.io/apimachinery@master
(92)  ? attempt github.com/gogo/protobuf with 1 pkgs; 15 versions to try
(128)     try<EMAIL_ADDRESS>(128) ✓ select<EMAIL_ADDRESS>w/3 pkgs
(93)  ? attempt github.com/fsnotify/fsnotify with 1 pkgs; 32 versions to try
(129)     try<EMAIL_ADDRESS>(129) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(94)  ? attempt github.com/coreos/go-systemd with 2 pkgs; 17 versions to try
(130)     try github.com/coreos/go-systemd@v15
(130) ✓ select github.com/coreos/go-systemd@v15 w/2 pkgs
(95)  ? attempt github.com/docker/libtrust with 1 pkgs; 4 versions to try
(131)     try github.com/docker/libtrust@master
(131) ✓ select github.com/docker/libtrust@master w/1 pkgs
(96)  ? attempt github.com/matttproud/golang_protobuf_extensions with 1 pkgs; 6 versions to try
(132)     try<EMAIL_ADDRESS>(132) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(97)  ? attempt github.com/pborman/uuid with 1 pkgs; 5 versions to try
(133)     try<EMAIL_ADDRESS>(133) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(98)  ? revisit github.com/prometheus/client_golang to add 1 pkgs
(134)   ✓ include 1 more pkgs from<EMAIL_ADDRESS>(98)  ? attempt github.com/apcera/gssapi with 1 pkgs; 7 versions to try
(135)     try github.com/apcera/gssapi@master
(135) ✓ select github.com/apcera/gssapi@master w/1 pkgs
(99)  ? attempt github.com/russross/blackfriday with 1 pkgs; 11 versions to try
(136)     try<EMAIL_ADDRESS>(136) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(100) ? attempt github.com/coreos/rkt with 1 pkgs; 66 versions to try
(137)     try<EMAIL_ADDRESS>(137) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(101) ? attempt github.com/mailru/easyjson with 2 pkgs; 2 versions to try
(138)     try github.com/mailru/easyjson@master
(138) ✓ select github.com/mailru/easyjson@master w/3 pkgs
(102) ? attempt github.com/gonum/graph with 5 pkgs; 14 versions to try
(139)     try github.com/gonum/graph@bde6d0fbd9dec5a997e906611fe0364001364c41
(139) ✓ select github.com/gonum/graph@bde6d0fbd9dec5a997e906611fe0364001364c41 w/7 pkgs
(103) ? revisit github.com/gogo/protobuf to add 2 pkgs
(140)   ✓ include 2 more pkgs from<EMAIL_ADDRESS>(103) ? attempt github.com/stretchr/testify with 1 pkgs; 9 versions to try
(141)     try<EMAIL_ADDRESS>(141) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(104) ? revisit github.com/coreos/etcd to add 3 pkgs
(142)   ✓ include 4 more pkgs from<EMAIL_ADDRESS>(104) ? revisit k8s.io/apiserver to add 5 pkgs
(143)   ✓ include 11 more pkgs from k8s.io/apiserver@master
(104) ? revisit github.com/coreos/etcd to add 5 pkgs
(144)   ✓ include 67 more pkgs from<EMAIL_ADDRESS>(104) ? revisit google.golang.org/grpc to add 2 pkgs
(145)   ✓ include 3 more pkgs from<EMAIL_ADDRESS>(104) ? attempt github.com/docker/docker with 1 pkgs; 206 versions to try
(146)     try<EMAIL_ADDRESS>(146) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(105) ? attempt github.com/openshift/source-to-image with 2 pkgs; 28 versions to try
(147)     try<EMAIL_ADDRESS>(147) ✓ select<EMAIL_ADDRESS>w/5 pkgs
(106) ? attempt github.com/aws/aws-sdk-go with 11 pkgs; 249 versions to try
(148)     try<EMAIL_ADDRESS>(148) ✓ select<EMAIL_ADDRESS>w/30 pkgs
(107) ? attempt github.com/pmezard/go-difflib with 1 pkgs; 3 versions to try
(149)     try<EMAIL_ADDRESS>(149) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(108) ? revisit k8s.io/apiserver to add 3 pkgs
(150)   ✓ include 15 more pkgs from k8s.io/apiserver@master
(108) ? revisit github.com/coreos/go-systemd to add 1 pkgs
(151)   ✓ include 1 more pkgs from github.com/coreos/go-systemd@v15
(108) ? attempt github.com/Sirupsen/logrus with 1 pkgs; 45 versions to try
(152)     try<EMAIL_ADDRESS>(152) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(109) ? attempt github.com/shurcooL/sanitized_anchor_name with 1 pkgs; 1 versions to try
(153)     try github.com/shurcooL/sanitized_anchor_name@master
(153) ✓ select github.com/shurcooL/sanitized_anchor_name@master w/1 pkgs
(110) ? attempt github.com/google/btree with 1 pkgs; 1 versions to try
(154)     try github.com/google/btree@master
(154) ✓ select github.com/google/btree@master w/1 pkgs
(111) ? revisit github.com/opencontainers/runc to add 1 pkgs
(155)   ✓ include 16 more pkgs from<EMAIL_ADDRESS>(111) ? revisit github.com/vishvananda/netlink to add 1 pkgs
(156)   ✓ include 1 more pkgs from github.com/vishvananda/netlink@master
(111) ? attempt github.com/syndtr/gocapability with 1 pkgs; 1 versions to try
(157)     try github.com/syndtr/gocapability@master
(157) ✓ select github.com/syndtr/gocapability@master w/1 pkgs
(112) ? attempt github.com/Microsoft/go-winio with 1 pkgs; 34 versions to try
(158)     try<EMAIL_ADDRESS>(158) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(113) ? attempt github.com/coreos/pkg with 1 pkgs; 6 versions to try
(159)     try github.com/coreos/pkg@v3
(159) ✓ select github.com/coreos/pkg@v3 w/1 pkgs
(114) ? revisit github.com/coreos/go-systemd to add 1 pkgs
(160)   ✓ include 1 more pkgs from github.com/coreos/go-systemd@v15
(114) ? attempt github.com/seccomp/libseccomp-golang with 1 pkgs; 2 versions to try
(161)     try<EMAIL_ADDRESS>(161) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(115) ? revisit github.com/coreos/pkg to add 1 pkgs
(162)   ✓ include 1 more pkgs from github.com/coreos/pkg@v3
(115) ? attempt github.com/gorilla/mux with 1 pkgs; 8 versions to try
(163)     try<EMAIL_ADDRESS>(163) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(116) ? attempt github.com/rkt/rkt with 1 pkgs; 66 versions to try
(164)     try<EMAIL_ADDRESS>(164) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(117) ? revisit github.com/docker/docker to add 4 pkgs
(165)   ✓ include 5 more pkgs from<EMAIL_ADDRESS>(117) ? attempt github.com/xiang90/probing with 1 pkgs; 2 versions to try
(166)     try<EMAIL_ADDRESS>(166) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(118) ? attempt golang.org/x/time with 1 pkgs; 1 versions to try
(167)     try golang.org/x/time@master
(167) ✓ select golang.org/x/time@master w/1 pkgs
(119) ? attempt github.com/grpc-ecosystem/go-grpc-prometheus with 1 pkgs; 8 versions to try
(168)     try<EMAIL_ADDRESS>(168) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(120) ? revisit github.com/docker/docker to add 5 pkgs
(169)   ✓ include 20 more pkgs from<EMAIL_ADDRESS>(120) ? attempt github.com/vmware/govmomi with 9 pkgs; 28 versions to try
(170)     try<EMAIL_ADDRESS>(170) ✓ select<EMAIL_ADDRESS>w/15 pkgs
(121) ? attempt github.com/gorilla/context with 1 pkgs; 2 versions to try
(171)     try<EMAIL_ADDRESS>(171) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(122) ? attempt github.com/jmespath/go-jmespath with 1 pkgs; 15 versions to try
(172)     try<EMAIL_ADDRESS>(172) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(123) ? attempt github.com/Azure/go-ansiterm with 2 pkgs; 2 versions to try
(173)     try github.com/Azure/go-ansiterm@master
(173) ✓ select github.com/Azure/go-ansiterm@master w/2 pkgs
(124) ? attempt github.com/petar/GoLLRB with 1 pkgs; 2 versions to try
(174)     try github.com/petar/GoLLRB@master
(174) ✓ select github.com/petar/GoLLRB@master w/1 pkgs
(125) ? revisit github.com/coreos/pkg to add 2 pkgs
(175)   ✓ include 3 more pkgs from github.com/coreos/pkg@v3
(125) ? revisit github.com/docker/docker to add 1 pkgs
(176)   ✓ include 3 more pkgs from<EMAIL_ADDRESS>(125) ? attempt github.com/imdario/mergo with 1 pkgs; 10 versions to try
(177)     try<EMAIL_ADDRESS>(177) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(126) ? attempt github.com/boltdb/bolt with 1 pkgs; 8 versions to try
(178)     try<EMAIL_ADDRESS>(178) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(127) ? attempt github.com/beorn7/perks with 1 pkgs; 5 versions to try
(179)     try github.com/beorn7/perks@master
(179) ✓ select github.com/beorn7/perks@master w/1 pkgs
(128) ? attempt github.com/go-ini/ini with 1 pkgs; 41 versions to try
(180)     try<EMAIL_ADDRESS>(180) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(129) ? attempt github.com/gonum/matrix with 1 pkgs; 21 versions to try
(181)     try github.com/gonum/matrix@master
(181) ✓ select github.com/gonum/matrix@master w/2 pkgs
(130) ? revisit github.com/opencontainers/runc to add 1 pkgs
(182)   ✓ include 1 more pkgs from<EMAIL_ADDRESS>(130) ? attempt github.com/gonum/internal with 1 pkgs; 3 versions to try
(183)     try github.com/gonum/internal@master
(183) ✓ select github.com/gonum/internal@master w/1 pkgs
(131) ? attempt github.com/gonum/blas with 2 pkgs; 8 versions to try
(184)     try github.com/gonum/blas@master
(184) ✓ select github.com/gonum/blas@master w/4 pkgs
(132) ? revisit github.com/gonum/internal to add 1 pkgs
(185)   ✓ include 1 more pkgs from github.com/gonum/internal@master
(132) ? attempt github.com/sirupsen/logrus with 1 pkgs; 45 versions to try
(186)     try<EMAIL_ADDRESS>(186) ✓ select<EMAIL_ADDRESS>w/1 pkgs
(133) ? attempt github.com/gonum/lapack with 2 pkgs; 12 versions to try
(187)     try github.com/gonum/lapack@master
(187) ✓ select github.com/gonum/lapack@master w/3 pkgs
(134) ? attempt github.com/gonum/floats with 1 pkgs; 4 versions to try
(188)     try github.com/gonum/floats@master
(188) ✓ select github.com/gonum/floats@master w/1 pkgs
  ✓ found solution with 1252 packages from 135 projects

Solver wall times by segment:
     b-list-versions: 1m43.100346677s
         b-list-pkgs:   20.559791942s
              b-gmal:   15.668104775s
     b-source-exists:    4.181673963s
  b-deduce-proj-root:    2.911799548s
    b-rev-present-in:    1.898973127s
             satisfy:    1.136503081s
         select-atom:    1.127347551s
            new-atom:     12.834534ms
            add-atom:        5.7427ms
         select-root:      2.018068ms
               other:      1.457349ms

  TOTAL: 2m30.606593315s

grouped write of manifest, lock and vendor: error while writing out vendor tree: error while exporting k8s.io/client-go: /tmp/dep324558830/vendor/k8s.io/client-go/.github/PULL_REQUEST_TEMPLATE.md already exists, no checkout
/tmp/dep324558830/vendor/k8s.io/client-go/.travis.yml already exists, no checkout
/tmp/dep324558830/vendor/k8s.io/client-go/CHANGELOG.md already exists, no checkout
/tmp/dep324558830/vendor/k8s.io/client-go/Godeps/Godeps.json already exists, no checkout
/tmp/dep324558830/vendor/k8s.io/client-go/Godeps/Readme already exists, no checkout
...a lot more...

relevant part of Godeps.toml
 [[constraint]]
   name = "k8s.io/client-go"
   source = "https://github.com/kubernetes/client-go"
   version = "=3.0.0"

could you replicate that with a -no-vendor (which should complete successfully), then paste in the contents of Gopkg.lock?

@sdboyer unfortunately after some time also including editing the source I have somehow managed to get around it.
I guess feel free to close it if that's enough data to debug.

my best guess is that you actually ran into #969, which is now fixed, but the version you're running predates the fix. i'll close this one up - please let me know if it reoccurs under the new version!


Can't Build GPU XGBoost with CMake for R in Windows

Hi Folks
First time caller here, having a hard time getting xgboost with gpu support working for R.
It falls over when I run this line:
cmake .. -G"Visual Studio 14 2015 Win64" -DUSE_CUDA=ON -DR_LIB=ON
with the error:
-- The C compiler identification is unknown
-- The CXX compiler identification is unknown
CMake Error at CMakeLists.txt:2 (project):
No CMAKE_C_COMPILER could be found.
Full Details below:
Environment info
Operating System: Windows 10
Compiler: Visual C++ 2015
Package used (python/R/jvm/C++): R
xgboost version used: Latest
If installing from source, please provide
The commit hash (git rev-parse HEAD)
a1b48afa41f888a256ff2645c4153d3d796bfb9c
Logs will be helpful (If logs are large, please upload as attachment).
Compiling the C compiler identification source file "CMakeCCompilerId.c" failed.
Compiler:
Build flags:
Id flags:
The output was:
1
Microsoft (R) Build Engine version 14.0.25420.1
Copyright (C) Microsoft Corporation. All rights reserved.
Build started 3/9/2018 11:03:08 PM.
Project "C:\Users\jordi\Downloads\xgboost\build\CMakeFiles\3.11.0-rc2\CompilerIdC\CompilerIdC.vcxproj" on node 1 (default targets).
PrepareForBuild:
Creating directory "Debug".
Creating directory "Debug\CompilerIdC.tlog".
InitializeBuildStatus:
Creating "Debug\CompilerIdC.tlog\unsuccessfulbuild" because "AlwaysCreate" was specified.
ClCompile:
C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\bin\x86_amd64\CL.exe /c /nologo /W0 /WX- /Od /D _MBCS /Gm- /EHsc /RTC1 /MDd /GS /fp:precise /Zc:wchar_t /Zc:forScope /Zc:inline /Fo"Debug\" /Fd"Debug\vc140.pdb" /Gd /TC /errorReport:queue CMakeCCompilerId.c
CMakeCCompilerId.c
Link:
C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\bin\x86_amd64\link.exe /ERRORREPORT:QUEUE /OUT:".\CompilerIdC.exe" /INCREMENTAL:NO /NOLOGO kernel32.lib user32.lib gdi32.lib winspool.lib comdlg32.lib advapi32.lib shell32.lib ole32.lib oleaut32.lib uuid.lib odbc32.lib odbccp32.lib /MANIFEST /MANIFESTUAC:"level='asInvoker' uiAccess='false'" /manifest:embed /PDB:".\CompilerIdC.pdb" /SUBSYSTEM:CONSOLE /TLBID:1 /DYNAMICBASE /NXCOMPAT /IMPLIB:".\CompilerIdC.lib" /MACHINE:X64 Debug\CMakeCCompilerId.obj
LINK : fatal error LNK1158: cannot run 'rc.exe' [C:\Users\jordi\Downloads\xgboost\build\CMakeFiles\3.11.0-rc2\CompilerIdC\CompilerIdC.vcxproj]
Done Building Project "C:\Users\jordi\Downloads\xgboost\build\CMakeFiles\3.11.0-rc2\CompilerIdC\CompilerIdC.vcxproj" (default targets) -- FAILED.
Build FAILED.
"C:\Users\jordi\Downloads\xgboost\build\CMakeFiles\3.11.0-rc2\CompilerIdC\CompilerIdC.vcxproj" (default target) (1) ->
(Link target) ->
LINK : fatal error LNK1158: cannot run 'rc.exe' [C:\Users\jordi\Downloads\xgboost\build\CMakeFiles\3.11.0-rc2\CompilerIdC\CompilerIdC.vcxproj]
0 Warning(s)
1 Error(s)

Time Elapsed 00:00:00.74
The R sessionInfo()
R version 3.4.3 (2017-11-30)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows >= 8 x64 (build 9200)
The command to install xgboost if you are not installing from source
git clone --recursive https://github.com/dmlc/xgboostcd xgboost
git submodule init
git submodule update
cd xgboost
Steps to reproduce
mkdir build
cd build
cmake .. -G"Visual Studio 14 2015 Win64" -DUSE_CUDA=ON -DR_LIB=ON
this gives the error:
-- The C compiler identification is unknown
-- The CXX compiler identification is unknown
CMake Error at CMakeLists.txt:2 (project):
No CMAKE_C_COMPILER could be found.
CMake Error at CMakeLists.txt:2 (project):
No CMAKE_CXX_COMPILER could be found.
-- Configuring incomplete, errors occurred!
See also "C:/Users/jordi/Downloads/xgboost/build/CMakeFiles/CMakeOutput.log".
See also "C:/Users/jordi/Downloads/xgboost/build/CMakeFiles/CMakeError.log".
What have you tried?
Everything I could find on here and Stackoverflow.
Thanks in advance for any and all feedback, I'm a noob here.

Are you using VS2015 x64 Native Tools command prompt when running cmake?

Hi Khotilov
I’m running that from the windows prompt, pulled up by CMD.
Should I be using the VS2015 x64 Native Tools command prompt? Apologies, I’ve never heard of that before. How would I invoke it?

I'm now running everything in the native tools prompt VS2015 x64.
And when executing:
install.packages('.', repos = NULL, type="source")
I get:
...
cp -r ../../include .
cp -r ../../amalgamation .
c:/Rtools/mingw_32/bin/g++  -std=gnu++11 -I"C:/PROGRA~1/R/R-34~1.3/include" -DNDEBUG -I.//include -I.//dmlc-core/include -I.//rabit/include -I./ -DXGBOOST_STRICT_R_MODE=1 -DDMLC_LOG_BEFORE_THROW=0 -DDMLC_ENABLE_STD_THREAD=0 -DDMLC_DISABLE_STDIN=1 -DDMLC_LOG_CUSTOMIZE=1 -DXGBOOST_CUSTOMIZE_LOGGER=1 -DRABIT_CUSTOMIZE_MSG_ -DRABIT_STRICT_CXX98_ -DDMLC_CXX11_THREAD_LOCAL=0      -fopenmp -pthread   -O2 -Wall  -mtune=generic -c xgboost_R.cc -o xgboost_R.o
c:/Rtools/mingw_32/bin/g++: not found
make: *** [xgboost_R.o] Error 127
...
Am working through this.

OK, solved the g++ not found issue by reinstalling Rtools to the folder where it seems to have been expected.
Still working through it.

Thanks for the help so far guys, but I'm back where I started. Running:
cmake .. -G"Visual Studio 14 2015 Win64" -DUSE_CUDA=ON -DR_LIB=ON
In either the windows prompt or the VS2015 Native Tools Command Prompt both still give the exact same error:
-- The C compiler identification is unknown
-- The CXX compiler identification is unknown
CMake Error at CMakeLists.txt:2 (project):
No CMAKE_C_COMPILER could be found.
CMake Error at CMakeLists.txt:2 (project):
No CMAKE_CXX_COMPILER could be found.

What does running cl from the VS2015 Native Tools Command Prompt show?
This SO might be relevant: https://stackoverflow.com/questions/45228238/qt-5-8-msvc-2015-compile-error
I would just uninstall everything related to Visual Studio and install it again afresh.

Running CL in the VS2015 native prompt gives this:
Microsoft (R) C/C++ Optimizing Compiler Version 19.00.24215.1 for x64
Copyright (C) Microsoft Corporation.  All rights reserved.
usage: cl [ option... ] filename... [ /link linkoption... ]

Could you also run where rc in the same terminal? rc.exe is a part of windows SDK, and based on the error, it might be not correctly installed.

OK so I ran: where rc and got:
INFO: Could not find files for the given pattern(s).
Sounds like we are zooming in on the issue !

Now running:
cmake .. -G"Visual Studio 14 2015 Win64" -DUSE_CUDA=ON -DR_LIB=ON
Returns:
-- The C compiler identification is MSVC 19.0.24215.1
-- The CXX compiler identification is MSVC 19.0.24215.1
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/x86_amd64/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/x86_amd64/cl.exe -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/x86_amd64/cl.exe
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/x86_amd64/cl.exe -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found OpenMP_C: -openmp (found version "2.0")
-- Found OpenMP_CXX: -openmp (found version "2.0")
-- Found OpenMP: TRUE (found version "2.0")
-- Found PythonInterp: C:/Program Files (x86)/Microsoft Visual Studio/Shared/Python36_64/python.exe (found version "3.6.3")
-- Found CUDA: C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v8.0 (found suitable version "8.0", minimum required is "8.0")
-- LIBR_HOME [/registry]
-- LIBR_EXECUTABLE [/registry/bin/x64/R.exe]
-- LIBR_INCLUDE_DIRS [/registry/include]
-- LIBR_LIB_DIR [/registry/bin/x64]
-- LIBR_CORE_LIBRARY []
CMake Error at cmake/modules/FindLibR.cmake:44 (message):
LIBR_LIB_DIR was not set!
Call Stack (most recent call first):
cmake/modules/FindLibR.cmake:154 (create_rlib_for_msvc)
CMakeLists.txt:165 (find_package)
-- Configuring incomplete, errors occurred!
See also "C:/Users/jordi/Downloads/xgboost/build/CMakeFiles/CMakeOutput.log".
See also "C:/Users/jordi/Downloads/xgboost/build/CMakeFiles/CMakeError.log".

Khotilov, I found your response to this issue in another thread:
https://github.com/dmlc/xgboost/issues/3081
The root cause seems to be that the published 3.4.3 R version has issues handling spaces in the argument of -e. Either removing the space or using a different R version, e.g., 3.4.3patched, makes it work right:
But I'm not entirely sure what "removing the space" means 'coz I don't know which space it refers to.  I would prefer to do that than mess with my R install at the moment. Any tips ?

I'm getting the same LIBR_LIB_DIR was not set! error with R3.4.2
CMakeOutput.log

Did you try adding LIBR_EXECUTABLE as a parameter to cmake?
-DLIBR_EXECUTABLE="C:/Program Files/R/R-3.4.3/bin/x64/R.exe"

Thanks Khotilov, that seems to have helped.  Running:
cmake .. -G"Visual Studio 14 2015 Win64" -DUSE_CUDA=ON -DR_LIB=ON
Gives:
-DLIBR_EXECUTABLE="C:/Program Files/R/R-3.4.3/bin/x64/R.exe"
-- Found OpenMP_C: -openmp
-- Found OpenMP_CXX: -openmp
-- Found OpenMP: TRUE
-- LIBR_HOME [C:/Program Files/R/R-3.4.3]
-- LIBR_EXECUTABLE [C:/Program Files/R/R-3.4.3/bin/x64/R.exe]
-- LIBR_INCLUDE_DIRS [C:/Program Files/R/R-3.4.3/include]
-- LIBR_LIB_DIR [C:/Program Files/R/R-3.4.3/bin/x64]
-- LIBR_CORE_LIBRARY []

[C:/Program Files/R/R-3.4.3/bin/x64/R.dll] Found PE+ image
-- Found LibR: C:/Program Files/R/R-3.4.3
-- Found R: C:/Program Files/R/R-3.4.3/bin/x64/R.exe
-- LIBR_CORE_LIBRARY C:/Users/jordi/Downloads/xgboost/build/R.lib
-- Configuring done
-- Generating done
-- Build files have been written to: C:/Users/jordi/Downloads/xgboost/build

Now I must confess that I have no idea how to load the R package.  Tried a few things, and this seemed to come the closest:

Copied the contents of R-package to a folder called new xgboost.
Zipped that new xgboost folder.
Then used RStudio to install that new xgboost.zip, and it appears to install correctly. At least, it gave no error.

But when I type library(xgboost) it says:
Error in library(xgboost) : ‘xgboost’ is not a valid installed package
Thanks for all your help and apologies for being a n00b.  Hopefully this thread helps somebody else !

Closing for the lack of activity in last 90 days. Future Q&A's and discussions should be re-directed to https://discuss.xgboost.ai/.


Create unit tests for PhEDExInjectorPoller

Such that we can catch issues like this one https://github.com/dmwm/WMCore/pull/8625

PhEDEx activity is ramping down and it will get deprecated in the coming months, closing it.


JavaScriptTypeRecovery: Replace [Require|JavaScriptCallLinker]Pass

This PR further overrides and improves upon existing capability in XTypeHintCallLinker and XTypeRecovery to fully replace the work being done under RequirePass and JavaScriptCallLinkerPass
Additionally, require imports that are relative are now resolved and a test verifying this has been added
: is now the path separator of JavaScriptTypeRecovery to remain consistent with the frontend

The failing test is strange
const foo = (x, y) => {
  sink(x);
};
foo(1, 2);

Identifier(foo) = MethodRef(Test0.js::program:anonymous) but Test0.js::program:anonymous does not exist, it seems to have been replaced by Test0.js::program:foo. So my type recovery points foo to Test0.js::program:anonymous. Will need to make a workaround here.

@max-leuthaeuser More on above, seems like MethodRef.methodFullName = Test0.js::program:anonymous but MethodRef.referencedMethod.fullName = Test0.js::program:foo so not sure if that is an inconsistency worth fixing since I can just use the latter.

Pretty sure that's from @fabsx00 https://github.com/joernio/joern/blob/master/joern-cli/frontends/jssrc2cpg/src/main/scala/io/joern/jssrc2cpg/passes/ConstClosurePass.scala

@max-leuthaeuser Ah I see, thanks.


Allow agents to re-stake

User story:

As a "quick" fix, after slashing a sequencer, we could check:
if Sequencer stake < min stake and (min_stake - sequencer stake) < balance
transfer from balance -> stake
That would top them up immediately after slashing, as long as they have a balance thats high enough

Solved by commit d68f040094


Sporadically Hanging Interactive API CI Test

The pytorch kvasir unet CI test has been triggering a CI timeout sporadically in the middle of training:
https://github.com/intel/openfl/runs/7525106079?check_suite_focus=true
https://github.com/intel/openfl/runs/7524916214?check_suite_focus=true
https://github.com/intel/openfl/runs/7510403486?check_suite_focus=true

Closing this issue, as the Interactive API is being deprecated in favor of Workflow API.


Enemies & Flower system

Related Issues

#135
#134
#130

Summary of Changes

Made enemy script which handles movement and attack
Gave enemy damageable script
Made flower counter scriptable object which has flower count and can increase and decrease it

Additional Remarks

Enemies have been tested with fake towers
Ignore the commits for the grid those are being improved in #147

Pull Criteria

[ ] Code commits have been reviewed
[ ] Changes have been made if requested

Changed getcomponent to try get component
Made use of vector 2 distance
Added missing docstrings

Fixed feedback from merge request and made enemies using pathfinding. Dependency from griddata scriptable object removed but now needs grid generator. Is there a way to make enemy not depends on this?


Fix hidden grouped-by columns in the Table chart

Closes #188.
This PR:

keeps the hidden state when using a hidden column as a group (as suggested in the issue raised by @AnninaWalker),
allows to hide/unhide key dimensions used as group-by columns,
shows a "Use as group" checkbox for non-key dimensions that are not of a "Measure" type (previously anyway we could use them as grouping variables by drag&drop), but this might be more convenient and consistent with how it's dealt with in case of key dimensions.

@herrstucki previously it was coded to clear the "isHidden" prop, so let me know if you think it should be the way the PR proposes – but it feels more natural to keep the state IMO, as anyway I had to hide a column first and then use it as a group (the other way I see it could be to always clear the "isFiltered" prop when using a key dimension to group the data by and disable the ability to "Hide and filter column").

Soo to summarize, I think this PR is good but grouped key dimensions should not use isFiltered but isHidden instead (or something like that) … and in the transition from grouped - ungrouped we need to convert between these two states

Do you think it would be possible to add tests here ? The logic seems quite complex and it would be nice to have a spec somewhere, at least for us to know in 6 months what is supposed to happen. IMHO as a general rule, we should not have a fix commit without a corresponding spec test.

I agree that we should have some tests there; will try to add them! Thanks for raising this 👍


Update electron and electron builder to latest versions

Electron v2.0.3 was released a few days ago which fixes a number of minor bugs which are likely to affect us. There is also one fix which is marked as a security fix
See https://github.com/electron/electron/releases/tag/v2.0.3
Also, update electron-builder which has had quite a few nice updates since the version we had installed.
Have tested with npm run dev, npm start, and via a package built with npm run package and everything appears to work as expected.

utACK cea699b


bpf: Correctly use revalidate_data_pull() in do_decrypt()

The IPv6 path in do_decrypt() was already correctly using
revalidate_data_pull(). The IPv4 path was node. If not enough headers
were pull'ed, the call would fail which resulted in ESP packets not
being detected correctly and thus not decrypted due to lacking the
packet mark.
This seems to be a regression introduced with the refactoring commit
9ed106a017c5. Only 1.9 is affected.
Fixes: 9ed106a017c5 ("cilium: create lib for encryption")
Suggested-by: John Fastabend<EMAIL_ADDRESS>

test-me-please

test-me-please

retest-gke

retest-gke


CONFIG_NO_OPTIMIZATIONS=y MPU fault on Zephyr 2.6

Describe the bug
Hello. I've been trying to move a small project built around Zephyr 2.5 to v2.6 and kept getting one issue which I couldn't figure out -  Whenever I've set CONFIG_NO_OPTIMIZATIONS=y (for debugging) my application fails. When I set CONFIG_NO_OPTIMIZATIONS=n the code works. I'm using nrf52832DK as the target for this test.
Eventually setting my app aside I went to work with https://github.com/zephyrproject-rtos/zephyr/tree/zephyr-v2.6.0/samples/bluetooth/scan_adv and can also repeat a crash with CONFIG_NO_OPTIMIZATIONS=y.
The only solution I've found so far https://github.com/zephyrproject-rtos/zephyr/issues/19340#issuecomment-723876554 doesn't work in my case. The error is the following:
00> *** Booting Zephyr OS build zephyr-v2.6.0  ***
00>
00> Starting Scanner/Advertiser Demo
00>
00> [00:00:00.334,350] <err> os: ***** MPU FAULT *****
00> [00:00:00.334,381] <err> os:   Stacking error (context area might be not valid)
00> [00:00:00.334,381] <err> os:   Data Access Violation
00> [00:00:00.334,411] <err> os:   MMFAR Address: 0x2000283c
00> [00:00:00.334,411] <err> os: r0/a1:  0x20000888  r1/a2:  0x00000001  r2/a3:  0x20000888
00> [00:00:00.334,442] <err> os: r3/a4:  0x20000888 r12/ip:  0x20002830 r14/lr:  0x200003d8
00> [00:00:00.334,442] <err> os:  xpsr:  0x20000200
00> [00:00:00.334,472] <err> os: Faulting instruction address (r15/pc): 0x20000620
00> [00:00:00.334,472] <err> os: >>> ZEPHYR FATAL ERROR 2: Stack overflow on CPU 0
00> [00:00:00.334,503] <err> os: Current thread: 0x20000668 (unknown)
00> [00:00:01.117,126] <err> os: Halting system

Surprisingly the scan_adv from v2.6 and v2.5 fails the same way with CONFIG_NO_OPTIMIZATIONS=y though my application worked well with  Zephyr 2.5 and CONFIG_NO_OPTIMIZATIONS=y - Not sure how and why.
To Reproduce
Steps to reproduce the behavior:

Go to https://github.com/zephyrproject-rtos/zephyr/tree/zephyr-v2.6.0/samples/bluetooth/scan_adv
Add these to prj.conf

CONFIG_IDLE_STACK_SIZE=340  # from https://github.com/zephyrproject-rtos/zephyr/issues/19340#issuecomment-723876554
CONFIG_NO_OPTIMIZATIONS=y  # Toggle this for MPU fault vs ok
CONFIG_DEBUG=y
CONFIG_CONSOLE=y
CONFIG_USE_SEGGER_RTT=y
CONFIG_RTT_CONSOLE=y

west build --pristine -b nrf52dk_nrf52832 . && west flash
Get the error on RTT
Set CONFIG_NO_OPTIMIZATIONS=n
It will work ok

Expected behavior
Expected that the build would be bigger but it will work.
Impact
Nobody will get hurt or anything but it would be really cool to be able to debug non optimized code. Had big plans on working with Zephyr and already invested some time.
Environment (please complete the following information):

Linux
Zephyr 2.5.0, 2.6.0

You are running into a stack overflow, because your code needs more stack when it is not optimized.

Thank you @alexanderwachter. Indeed setting CONFIG_BT_RX_STACK_SIZE=2048 helps though in case of my app I'm now getting another issue.
00> [00:00:01.350,830] <err> os: ***** USAGE FAULT *****
00> [00:00:01.350,860] <err> os:   Attempt to execute undefined instruction
00> [00:00:01.350,891] <err> os: r0/a1:  0x00000001  r1/a2:  0x0000000d  r2/a3:  0x0000000d

Since it's a different problem I'll close this issue and try to figure it out.


AttributeError: module 'gluoncvth' has no attribute 'models'

When i run the following code
"
import gluoncvth as gcv
model = gcv.models.resnet50(pretrained=True)
"
I got -- AttributeError: module 'gluoncvth' has no attribute 'models'
And i am confused

When i check .../site-packages/gluoncvth, i only found a init.py file but nothing else,
and the init.py file is also empty

That might be importing from the root folder. Try run your code in a separate folder.

Having the same issue here.
When I run
pip install gluoncv-torch

it says "Successfully installed gluoncv-torch-0.0.0"
and I went to check the /dist-packages/gluoncvth/, it's empty there.
and the same
import gluoncvth
model = gluoncvth.models.get_.....()

I'm getting
AttributeError: module 'gluoncvth' has no attribute 'models'

An expedient:

Run

git clone https://github.com/StacyYang/gluoncv-torch.git
cd gluoncv-torch
python setup.py install

In the .py file right before the line of

import gluoncvth

add
import sys
sys.path.insert(0,'<path to the gluoncv-torch folder created from git>')

This will add the local gluoncv-torch from git to PATH

Another option:
Use gluoncv.model_zoo instead


Loading Weights

Hi!
I want to know if someone wants to use their own weights how can it be done?

Hi! Great question! Check out the Tokenizer init function in utils.mojo to see how the weight values for the tokenizer are being read from the tokenizer.bin file. The same process can be done to other structs in the UNet and VAE

I have provided a "next steps" section in the README to explain the steps that need to be taken in order to load weights - open a pull request if you would be interested in implementing these steps - that would be an amazing side project!


Lazy call

I am trying to replicate the results and I am unable to so because it is always throwing the issue of CUDA  unavailability but I am accessing the DGX A100 supercomputer and still unable to run the commands. When I am trying to check the files in saved models directory to resolve the issue of calling more gpus , they are ".pth" files which cannot be accessed in any way. The model when accessed with right torch version is calling lazy call functions.

FYI,
The .pth files are the weight models to run the coco and flickr30 data


feat(angular): Expose Ionic Animations via the AnimationController

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] Build (npm run build) was run locally and any changes were pushed
[x] Lint (npm run lint) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

This exposes Ionic Animations in a more "Angular-y" fashion via the AnimationController. I will update the Animation Docs to show its usage.
Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

Once this merges we need to update the following PRs:
https://github.com/ionic-team/ionic/pull/19771
https://github.com/ionic-team/ionic-docs/pull/976


How do we build from source?

Hi, are there any instructions to build from source? Also, how did you get your hands on the source? Well done!

We are extracting the source code from APK and you can see it here
Also here is some binaries and dex files extracted with JADX

Awesome work! I'm looking forward to some instructions, thanks!

is this kind of code legal? Can the owner report to GooglePlay to remove it?

Nice iniciative! Blynk is the best and easier App for this kind of development. Thank you guys!

any way we can convert it to run off gradle and updated android studio


Investigate on how to securly configue oauth client credentials for production evnviorment

Currently we are loading oauth-client credentials from csv. The test credentials used for dev/testing are publicly available. We need to find a way to securely handle this for production environment. Maybe mounting csv file as volume would help in docker compose? and add some environment variables to load values for MP front-end.
Any suggestions are welcome. @dennyverbeeck

Cannot find a proper solution yet. Given the time-line, i propose to build MP from source on production and change credentials before building. Few other changes of MP on production are also currently inevitable for a secured deployment based on current deployment strategy.

I agree with the comments in the PR you linked to. For now we keep it in the CSV file, and we'll look into dynamic registration (#18) as soon as possible.


bug: bun create t3-app@latest CLI freezes on tailwind step

Provide environment information
System:
OS: Windows 11 10.0.22621
CPU: (32) x64 13th Gen Intel(R) Core(TM) i9-13900K
Memory: 49.26 GB / 63.77 GB
Binaries:
Node: 20.9.0 - C:\Program Files\nodejs\node.EXE
npm: 10.1.0 - C:\Program Files\nodejs\npm.CMD
pnpm: 8.7.6 - ~\AppData\Roaming\npm\pnpm.CMD
Describe the bug
bun create t3-app@latest CLI freezes on tailwind step
Reproduction repo
na
To reproduce
bun create t3-app@latest
enter
enter
Additional information
No response

can't reproduce. could you provide a more precise set of steps to reproduce this on another system?

bun on windows is still beta so we can't assure things will work there. most likely not an issue caused by create-t3-app

I got the same problem, do anybody here know how to solve this on your wsl?


Spec violation in connectionFromArraySlice

Hi,
It looks like there is a spec violation in connectionFromArraySlice implementation.  Specifically, spec of the Relay pagination algorithm (4.3 / ApplyCursorsToEdges / Step 2.2) indicates that elements should be removed from the slice only if afterEdge exists (if afterEdge does not exist, allEdges are returned) [1].  Here is a counter example:
connectionFromArraySlice(
  [1, 2, 3, 4, 5, 6],
  { first: 6, after: 'YXJyYXljb25uZWN0aW9uOjEw' },
  { sliceStart: 0, arrayLength: 6 } )

This returns empty edges array. Argument after in this case is base64 encoded "arrayconnection:10" string.  Clearly, edge corresponding to that cursor doesn't exist, so connectionFromArraySlice should return all six nodes in the edges array as per the spec.
There is no symmetrical issue with last / before combination, because endOffset is set to min of beforeOffset and array length.
Just wanted to confirm that my read of the spec makes sense to you all before sending a pull request with the fix.
Thanks,
Nemanja
[1] Spec: https://facebook.github.io/relay/graphql/connections.htm#ApplyCursorsToEdges()

Yeah, we should probably treat that cursor the same way we would a malformed one. Good catch.

There is no symmetrical issue with last / before combination, because endOffset is set to min of beforeOffset and array length.

Hm, I'd expect that
connectionFromArraySlice(
[1, 2, 3, 4, 5, 6],
{ last: 6, before: base64('arrayconnection:-5') },
{ sliceStart: 0, arrayLength: 6 } )
would have similar issues? It would see -5 as the offset and return nothing?

yup, spot on, negative offsets bug out as well.  I'll pull together a patch for both along with a couple of tests and send them in.  Thanks

Any progress on this?


Rebuild HPACK encoder table as C++

The penultimate change in this series... the next one is to actually convert the compression code to C++.

@nicolasnoble

Debugging the hpacksize bug

Thanks for doing this!!


Runner.AddTarget will no longer throw NullReferenceException

Prevent Runner.AddTarget from throwing NullReferenceException
If an operation does not exist in the TfGraph, Runner.AddInput can cause a NullReferenceException.
Check if the operation exists before attempting to get the outputs and throw ArgumentOutOfRangeException if it does not exist.

Let me know if you'd prefer another exception type or handling this a different way.

Thank you for your contribution!


App Crash when using ASWebAuthenticationSession in iOS 14.5

Description:
File: ASWebAuthenticationURLHandler.swift
My app is crash when I am using the
ASWebAuthenticationSession(
            url: url,
            callbackURLScheme: callbackUrlScheme,
            completionHandler: { callback, error in
                if let error = error {
                    let msg = error.localizedDescription.addingPercentEncoding(withAllowedCharacters: .urlHostAllowed)
                    let errorDomain = (error as NSError).domain
                    let errorCode = (error as NSError).code
                    let urlString = "\(self.callbackUrlScheme):?error=\(msg ?? "UNKNOWN")&error_domain=\(errorDomain)&error_code=\(errorCode)"
                    let url = URL(string: urlString)!
                    #if !OAUTH_APP_EXTENSIONS
                    UIApplication.shared.open(url, options: [:], completionHandler: nil)
                    #endif
                } else if let successURL = callback {
                    #if !OAUTH_APP_EXTENSIONS
                    UIApplication.shared.open(successURL, options: [:], completionHandler: nil)
                    #endif
                }
        })

Then Crash detail is
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'The provided scheme is not valid.
A scheme should not include special characters such as ":" or "/".'

if I replace callbackUrlScheme   with callbackUrlScheme .addingPercentEncoding(withAllowedCharacters: .urlHostAllowed) it's working fine.
So please let me know how to add this lines into your package and use into my project.
Or should I have to create the new branch for this is send you the PR for merge into this?
OAuth Provider? (Twitter, Github, ..):
OAuth Version:

[ ] Version 1
[x] Version 2

OS (Please fill the version) :

[x] iOS : 14.5
[ ] OSX :
[ ] TVOS :
[ ] WatchOS  :

Installation method:

[ ] Carthage
[ ] CocoaPods
[x] Swift Package Manager
[ ] Manually

Library version:

[x] head
[x] v2.1.0
[ ] v2.0.0
[ ] v1.4.1
[ ] other: (Please fill in the version you are using.)

Xcode version:

[x] 12.5 (Swift 5.0)

[ ] 11.4 (Swift 5.2)

[ ] 11.x (Swift 5.1)

[ ] 10.x (Swift 5.0)

[ ] other: (Please fill in the version you are using.)

[ ] objective c

you could show code with link ...
https://github.com/OAuthSwift/OAuthSwift/blob/dbb74fa8717843017fc54b3584f716ccd8ba3e7c/Sources/Handler/ASWebAuthenticationURLHandler.swift#L38
first yes ! forecast must be forbidden in OAuthSwift code
I see some edit about that recently, maybe it depend on swift version

@phimage Yes forecast must be forbidden,
but my point is different That app crashing while creating the object of ASWebAuthenticationSession because in this we are sending callbackURLScheme normally.
But we required to send like this callbackUrlScheme.addingPercentEncoding(withAllowedCharacters: .urlHostAllowed) I have tested this with my by cloning your package and made this changes

your value of callbackURLScheme is ?
because you pass it , so you could encode it if very necessary (or not is based to ASWebAuthenticationSession is different)
if you put  ":" or "/" in a scheme it's your error maybe?

Yep, it worked for us too in 14.4, but check your logs for the error. What changed in 14.5 is that Apple is now crashing instead of just logging an error.


chore(main): release 0.2.0

:robot: I have created a release beep boop
0.2.0 (2023-08-25)
Features

some feat (766f781)

Bug Fixes

some comment 1 (9c03d06)
some comment 2 (c1a941c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/reemus-dev/gitnr/releases/tag/v0.2.0 :sunflower:


Data Update 2020

Is this data set being kept up to date? It looks like the latest update was in 2015, but the data here was updated in 2020. I can see some differences, for example around Midway:

vs

I'm happy to submit a PR with the updated data, just wanted to check first in case I was missing something. Thanks!

@geneorama ?

@levyj might know more about the last update, as he maintains the portal.

Hi @gavinr, thanks for the note! We are taking a look at the role of the data in this repo and its relationship to the Data Portal. We will look into it!


is-array? throws exception on nil in clj

A minor annoyance that causes an downstream issue:
(d/conn-from-datoms (d/datoms @*conn :eavt) schema) will fail on an empty database.
This is the recommended way to update a schema from DataScript documentation.
This in turn can be easily worked around by only using conn-from-datoms for non-nil.
However type checks typically shouldn't throw exceptions, so submitting a patch.

Thank you! Just pushed DataScript 1.2.1

Thank you :)


Update Odometry using the limelight

This one can be difficult as the vision target is in the middle and we circle around it.
Need to use the horizontal offset and the distance calculation to do small corrections/updates to the current pose
Should also look at utilizing the cable bump for doing more precise updates (we know the vector when this pitch/roll happens).


2021.6.x api gateway

Closing this one as it's not yet ready for PR, should be either marked as draft or finished. Will create a new one on 2021.9.x @zmija @MarioBur

Reopened to draft


Escape column names used in MySQL select statements

We've had issues with columns named "rank", "group" or "from" when selecting from MySQL, so we've escaped the column names.
Sorry if not the best method to add the backticks, I'm not fluent in Python.

Thank you very much!


Enforce key stored in hardware

A parameter on Android to enforce the creation of the key to be only made in TEE would be great. And crash if this is not possible. This would certify the flutter_secure_storage plugin to hardest audit

Hi, i am writing a new implementation for android which could make this possible. Please see #769 for the progress.


Add ClusterError.

Context / purpose
Closes #244
Description
Add ClusterError to display the cause of the error in an easy-to-understand manner.
How was this PR tested?
Deliberately generated an error by running the following command and checked the message.
cargo run -- serve --host=<IP_ADDRESS> --index-uri=file:///home/minoru/tmp/quickwit/indices/hdfslogs-idx --port=80

Checklist

[x] tested locally
[x] included documentation

% cargo run -- serve --host=<IP_ADDRESS> --index-uri=file:///home/minoru/tmp/quickwit/indices/hdfslogs-idx --port=80
    Finished dev [unoptimized + debuginfo] target(s) in 0.13s
     Running `target/debug/quickwit serve --host=<IP_ADDRESS> '--index-uri=file:///home/minoru/tmp/quickwit/indices/hdfslogs-idx' --port=80`
Jul 15 17:40:00.673  INFO quickwit_cluster::cluster: Read existing host key. host_key=57aa5a51-56ef-4d76-9880-16e749788a8c host_key_path="/tmp/quickwit-host-key-<IP_ADDRESS>-80"
Jul 15 17:40:00.673  INFO quickwit_cluster::cluster: Create new cluster. host_key=57aa5a51-56ef-4d76-9880-16e749788a8c listen_addr=<IP_ADDRESS>:81
Command failed: Failed to create cluster. Cause `Artillery :: I/O error occurred: Permission denied (os error 13)`

@fulmicoton
Updated the branch. Please take another look.


feat: add crd resource visitor util

What type of PR is this?
/kind feature

What this PR does / why we need it:
add crd resource visitor util
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

-7 of 34 (79.41%)  changed or added relevant lines in 1 file are covered.
10 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.08%) to 76.558%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/cmd/util/resource/visitor.go
27
34
79.41%

Files with Coverage Reduction
New Missed Lines
%

pkg/engine/operation/parser/parser.go
3
79.27%

pkg/cmd/destroy/options.go
7
81.86%

Totals

Change from base Build<PHONE_NUMBER>:
-0.08%

Covered Lines:
8818

Relevant Lines:
11518

💛  - Coveralls


Target values summary tables

We need to create some summary tables of the values in the targeted column with the various separators removed:
"businesses and schools" -> ["businesses", "schools"]
This should be recorded for each who_code.

@todowede - I tend to put this type of work - which does not have anything to do with the automated cleaning routing - in the src/manual folder.
We should open a different private repo in the lshtm-gis organisation for other analyses so this may be an opportunity for you to create a new repo in the organisation if you would like.


Resistance on Battery Sensor too high

After implementing the code on the battery monitor, I realized that I was using two 1M resistors which exceed the internal impedance of the ESP32.  have switched them out for 100K and will retest.  If this works then will change the schematic

This also seems to be working when reducing the resistance, though at the same time having the pin wrong wasn't helping.  Will update this to V2.2


Create a new pull request by comparing

Pull Request Test Coverage Report for Build 541013790

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 94.589%

Totals

Change from base Build 535451590:
-0.03%

Covered Lines:
1897

Relevant Lines:
1960

💛  - Coveralls


Create dataset loader for khPOS

Dataloader name: khpos/khpos.py
DataCatalogue: http://seacrowd.github.io/seacrowd-catalogue/card.html?khpos

Dataset
khpos

Description
khPOS is a POS-tagging dataset for the Khmer language. The data was collected from websites, student and voter lists. It was automatically tokenized with a CRF model and then manually tagged for POS and checked for errors.

Subsets
-

Languages
khm

Tasks
POS Tagging

License
Creative Commons Attribution Non Commercial Share Alike 4.0 (cc-by-nc-sa-4.0)

Homepage
https://github.com/ye-kyaw-thu/khPOS/tree/master

HF URL
https://huggingface.co/datasets/seanghay/khPOS

Paper URL
https://github.com/ye-kyaw-thu/khPOS/blob/master/khpos.pdf

#self-assign


Typo in doc string? bn_dh.c

bn_dh.c refers to RFC 2312:
https://github.com/openssl/openssl/blob/7b2d995ed79f0653defd682e5097ed937f4cabbc/crypto/bn/bn_dh.c#L32
but I can't find anything about DH primes in RFC 2312, so I'm guessing it's a typo. Any idea what is the correct RFC reference?

Looks like RFC 2412 (OAKLEY groups)


updated gemfile and removed test

I'm going to rebase this and merge it so the conflicts don't keep coming up. I will continue working on it.

Thanks @askl56. I'm not writing any new tests too.

This'll be the last PR which (I think) we'll be merging before we merge to master.

I'll do the rebase now.

:smile:


applyUpdate is fired after entering the page

After deploy I'm opening my webapp in new card and onUpdated is fired. It's killing look and feeling, coz user recently refreshed webapp and offline-plugin tells him to do it again
webpack:
    plugins: [
      new OfflinePlugin({
        autoUpdate: 1000 * 30,
        ServiceWorker: {
          events: true
        }
      })
]

OfflinePluginRuntime.install({
  onUpdateReady(): void {
    OfflinePluginRuntime.applyUpdate();
  },

  onUpdated(): void {
    window.dispatchEvent(new CustomEvent(refreshRequiredEventName));
  }
});

Just don't call applyUpdate(); in there if you don't want to.

Thank you for reply,
Alright, so you're suggesting to control it somehow (detecting "fresh" enter on webapp) and if true then skip applyUpdate().
Could you tell me if I'll do it and script will skip this applyUpdate() then after releasing newer version (and still staying on old page when script has skipped first applyUpdate()) then it will be detected in runtime?

Alright, so you're suggesting to control it somehow (detecting "fresh" enter on webapp) and if true then skip applyUpdate().

I'm not suggesting anything. If you want to do it -- do it.

Could you tell me if I'll do it and script will skip this first applyUpdate() then after releasing newer version (and still staying on old page when script has skipped first applyUpdate()) then it will be detected in onUpdateReady?

onUpdateReady  is called on every update. When you visit a website and there is an update -- it's called. When an autoUpdate check happens and there is an update -- that method is called.
What you do in it is your choice.


how would you run it on windows

I type in the python3 code/final_face_slider_tool.py but it doesn't recognize python3 as a command

This isn't related to this repo.
When you install Python 3, add it to your PATH (google how this works or let the installer do that for you)

Mine just crashes while still in the console.


chore: release 1.2.204

:robot: I have created a release *beep* *boop*
1.2.204 (2023-02-28)
Bug Fixes

kube-run: command stderr was redirected to stdin (4d038d4)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/werf/werf/releases/tag/v1.2.204 :sunflower:


Unable to obtain client_ secret for application

Preflight Checklist

[X] I could not find a solution in the existing issues, docs, nor discussions
[ ] I have joined the ZITADEL chat

Describe your problem
After creating the zitadel organization using Terraform, I can get the client_id of the application by defining outputs.tf, but cannot get client_ secret.

Describe your ideal solution
Please tell me how to obtain it.
Version
v1.0.0-alpha.18
Additional Context
no more

The ZITADEL API only returns the client secret when it is created. After that, there is no way to read the secret from the ZITADEL API anymore.
If you want to have the client secret in your Terraform state so you can reference it, either create a new app using Terraform or import the resource by providing the secret.
Please reopen the issue if this doesn't help.


Hide bigint implementation details

With BigDigit exposed in the interface, we're somewhat limited in what kind of changes we can make to the implementation.  Even just fn new(digits: Vec<BigDigit>) strongly implies that this storage will be directly used by the new BigUint.

Even just fn new(digits: Vec) strongly implies that this storage will be directly used by the new BigUint.

I think that might be implied, but I don't think it is big deal if that isn't the case. You could leave BigDigit defined as it is currently and then use another name, e.g. Limb for the internal type. Then the public functions that take BigDigit values would just be [deprecated] conversion functions.
Note that Limb is a better name anyway, IMO. See this for the origin of the term Limb: https://gmplib.org/manual/Nomenclature-and-Types.html

Yeah, that's true.  I think we even discussed on another issue that we could use careful transmutes and byte swaps to still reuse that storage, even when Limb is a different size.  So yes, we could change the implementation without actually affecting the API, then the only breaking change is to eventually remove the vestigial parts.

pub enum ParseBigIntError is another pain point.  The way it tries to extend ParseIntError makes it clumsy to actually create errors, and we can't change this without breaking.  ParseRatioError is a better example how it should be, where the enum is hidden as a private field in a struct.

It occurred to me recently that serialization is actually a breaking change too, because changes to type internals may alter the serialized format.  That's especially true when they're derived, like we're doing for rustc_serialize, but we should be careful even with our manual serde implementation.

This issue was moved to rust-num/num-bigint#3


fix: use helper function to satisfy dependent project type checks

Description
This updates the type definitions for dependent project usage to match the json, and adds a helper function to get past typescript issues with dynamic keys.
An alternate solution would be to just use any as seen here: https://github.com/openedx/paragon/pull/2940
Deploy Preview
Include a direct link to your changes in this PR's deploy preview here (e.g., a specific component page).
Merge Checklist

[ ] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[ ] Does your change adhere to the documented style conventions?
[ ] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[ ] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[ ] Were your changes tested in the example app?
[ ] Is there adequate test coverage for your changes?
[ ] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

I've been digging into it as well and I think I found what's causing all of the problems. We try to import a JSON file which has no types defined for it, so TypeScript tries to infer these types during import to help us and does it poorly. When it reads the file and specifically these lines
...
"usages": {
  "Alert": [
    {
      "filePath": "openassessment/xblock/static/js/src/lms/components/WaitingStepContent.jsx",
      "line": 40,
      "column": 6,
      "index": 1251
    },
    {
      "filePath": "openassessment/xblock/static/js/src/lms/containers/WaitingStepDetailsContainer.jsx",
      "line": 68,
      "column": 12,
      "index": 2171
    }
  ],
  "Button": [
    {
      "filePath": "openassessment/xblock/static/js/src/lms/components/WaitingStepList.jsx",
      "line": 15,
      "column": 4,
      "index": 445
    }
...

it assumes that usages is an object with literal keys Alert, Button and so on which does not correspond to our defined types, you can see this from this error message that is printed after running npm run type-check (see how the type definition includes literal keys here, instead of generic [key: string] type or something similar)
Type '{ Alert: { filePath: string; line: number; column: number; index: number; }[]; Button: { filePath: string; line: number; column: number; index: number; }[]; DataTable: { filePath: string; line: number; column: number; index: number; }[]; ... 318 more ...; Fieldset?: undefined; }' is not assignable to type '{ [key: string]: IUsage[]; }'.

My guess is that wasn't an issue before https://github.com/openedx/paragon/commit/e6967cf63ed1b15dc0c0d5f236b867743afe73a7 because the first usages object that TypeScript encounter was an empty one, so it probably inferred any type for keys or something like that...
Anyway, I think a better solution would be to somehow disable type inference for the JSON file or enforce our types here, but I haven't found a clean way to do that...
One thing that works for me and seems like an OK solution is updating first lines in getDependentProjectsUsages.tsx file as follows
// @ts-ignore
import dependentProjectsAnalysis from '../../../dependent-usage.json'; // eslint-disable-line
import getGithubProjectUrl from './getGithubProjectUrl';
import { IDependentProjectsUsages, IDependentUsage, IUsage } from '../types/types';

const { projectUsages } = dependentProjectsAnalysis;
const dependentProjectsUsages = projectUsages as unknown as IDependentProjectsUsages[];

which does the following:

projectUsages as unknown - clears the types inferred by TypeScript
as IDependentProjectsUsages[] - casts projectUsages object to be of desired type

this works for me locally.
@brian-smith-tcril I'm going to approve this PR as I really don't have any strong preference for either of the solutions. I'm not a fan of TypeScript, even just simply switching type to any like you did in your other PR works for me 🙂.

closing in favor of https://github.com/openedx/paragon/pull/2951


API ref: "If empty, this Model is used."

The API reference of the following modules mentions that the listener object for some of the attach*-methods will be a model if the parameter oListener was empty:

sap.ui.core.Core
sap.ui.core.routing.Router
sap.ui.core.routing.Route

oListener?	Object on which to call the given function. If empty, this Model is used.

which makes sense for model based modules but not for the above mentioned ones.

Hello @boghyon  ,
Thank you for sharing this finding. I've created a internal incident<PHONE_NUMBER>. You can check its status on GitHub.
Regards,
Gergana

Changes are available as of OpenUI5 1.50 - https://openui5nightly.hana.ondemand.com/#/api


Can multiple mounts occur on one node?

For example, on a server where multiple virtual hosts need to mount different paths in the same S3 bucket? In this scenario, the messaging aspect wouldn't need to be used, either, since there is a 1 to 1 relationship between virtual hosts and mounted directories.

The answer is yes. :)

One follow-up on this topic:
I am mounting 2 different folders in the same bucket to 2 different mount points:
bucket1/folder1 => /var/www/html/wp-content/uploads
bucket1/folder2 => /var/www/protected-files
I'm doing this from multiple web servers to the same bucket (to sync my wp uploads across my various clustered web servers).
In this instance, can I use the same topic, queue, and cachedir, or should I have different ones for each mount point?
Thanks,
Dave


Update polkadot.json to replace Mile/Bravo with Mile/Charlie

Replaced Mile/Bravo polkadot node stash address with Mile/Charlie stash address.

@ironoa Verified with participant that they requested the change


Display files using qlpreviewcontroller

I’m struggling with this issue maybe it applies for turbo navigator too. In hey.com it seems they display files using a qlpreviewcontroller instead of a safariviewcontroller, do you intend to provide this feature in turbo navigator ?

Can you provide a gist or even better a repo outlining the issues you’re having? I’ve never worked with that class before so would need to do some research before being able to help.

Specifically this piece of code:

https://github.com/hotwired/turbo-ios/blob/c476bac66f260adbfe930ed9a151e7637973ff99/Demo/SceneController.swift#L128-L136
In hey, they seem to handle it as a regular turbo navigation and display the file in a qlpreviewcontroller.

https://github.com/joemasilotti/TurboNavigator/assets/2651240/d3a6e299-35c5-4e78-8e9f-5dee73972012

I tried to do something like this, but without success...
import Foundation
import QuickLook

class PreviewController: QLPreviewController, QLPreviewControllerDataSource {
    private var url: URL!

    convenience init(url: URL) {
        self.init()
        self.url = url
        self.dataSource = self
    }

    func numberOfPreviewItems(in controller: QLPreviewController) -> Int {
        return 1
    }

    func previewController(_ controller: QLPreviewController, previewItemAt index: Int) -> QLPreviewItem {
        return url as QLPreviewItem
    }
}

it can't load remote urls

I think that might be your answer - download the file first then use QLPreviewController to display it. I can't say for certain but I would guess that is what HEY is doing.

@lazaronixon, did that work for you? If so, I'd like to close this out. Thanks!


preact ignores/rewrites  dimensions?

I have a React-managed page with some 175+ images, and after switching from react to preact-compat (using the webpack alias solution) all my images come out as  in the DOM, despite having defined dimensions in the JSX source code (e.g.  etc) - is there any reason why it does this? Because that seems like a pretty crazy bug.

STR:
test.jsx
var React = require("react");
var ReactDOM = require("react-dom");

var Test = React.createClass({
  render() {
  	return ;
  }
});

ReactDOM.render(<Test/>, document.querySelector("#app"));

index.html
<!doctype html>

		<meta charset="utf-8">
		<title>test</title>

The following box should have an img tag with width and height 100px

It doesn't, preact or preact-compat are stripping the JSX props for some reason.

webpack config
var webpack = require('webpack');

// Bundle output
var output = {
  path: __dirname,
  filename: 'app.js',
};

// Necessary webpack loaders for converting our content:
var webpackLoaders = [
  'babel-loader',
];

var resolve = {
  alias: {
    react: "preact-compat",
    "react-dom": "preact-compat"
  }
};

// And the final config that webpack will read in.
module.exports = {
  entry:  "./test.jsx",
  output: output,
  resolve: resolve,
  module: {
    loaders: [
      {
        test: /.jsx?$/,
        loaders: webpackLoaders
      }
    ]
  }
};

package.json
{
  "name": "preact",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "webpack"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "devDependencies": {
    "babel-core": "^6.24.0",
    "babel-loader": "^6.4.1",
    "babel-preset-env": "^1.3.2",
    "babel-preset-es2015": "^6.24.0",
    "babel-preset-react": "^6.23.0",
    "preact": "^3.0.0",
    "preact-compat": "^3.0.0",
    "webpack": "^2.3.2"
  },
  "babel": {
    "presets": [
      "es2015",
      "react"
    ]
  }
}

Did a quick test with "real" preact rather than preact-compat, but the same thing happens. Loading the compiled app in the browser shows `"

@Pomax just don't use pixels there. What happens there is Preact doing img.width = '400px', but width and height of the images are number properties, so conversion to number happens. Number('400px') is NaN, so browser makes it 0.
I don't think Preact itself should fix anything here, but could make sense to fix this in preact-compat. @developit What do you think?

Except I really need pixels, because that's how the web works: I have 183 images in a hugely long article that all need to have predefined width/height dimensions so the page doesn't constantly resize when an image loads in, and thus reflows, and completely ruins user experience.
These properties are not numbers, they are strings. They've always been strings, because they have to contain a unit indicator as per the HTML spec. Even if you leave the unit off, that's still a string, so changing the parsing rules here seems by far the better solution.

Okay, step-by-step.
Everything is HTML is a string, that's true. But in JS and DOM it is not. Try running typeof (new Image()).width in console, it will show 'number'.

These properties are not numbers, they are strings.

Properties are number and attributes are strings. Preact creates DOM elements and assigns values to their props (if prop exists in element + few exceptions).

They've always been strings, because they have to contain a unit indicator as per the HTML spec.

Unit indicators in such HTML attributes are artifacts from older versions of HTML, HTML5 however defined only pixels: https://html.spec.whatwg.org/multipage/embedded-content.html#attr-dim-width. You absolutely can use 400px value in pure HTML, this is how browsers keep compatibility.

Even if you leave the unit off, that's still a string, so changing the parsing rules here seems by far the better solution.

You live the unit type off, that's still string, but valid number representation so it could be converted to a number by the browser when assigned to Image's width property.

This feels very much like a case where React does the right thing, Preact does the wrong thing, and this is a bug, not a feature.

It might be a bug per expectations, but that's certainly isn't a bug per how Preact works. After all, you don't pick Preact because it's 100% compatible with HTML and performs all the checks as React does, but it's exactly the opposite why you choose it.
As per preact-compat, I agree, it may need to support this features. That's why I suggest to open an issue about it there.

I think you replied to whatever github sent you as notification email, I rewrote my reply after initial post to make more sense.

@Pomax Unfortunately I'm not going to rewrite my answer.

That is unfortunate indeed, because my rewrite already agrees with many of the points you make - it's more about "JSX doesn't really care, if a property has some value, rewriting it is more work than copying it over and leaving it to the user to have specified a legal value". Especially for apps that need to work within the context of a larger page.

There is no special meaning for width/height of images in Preact. It's passed as is to DOM property, as you suggest. Browser converts that value to number though and Number() was just an example to make it clear what happens.

Then maybe I don't understand something here, because if I write HTML with width="200em", even if it is interprets it as pixels, it will preserve those 200em in the actual DOM node. It doesn't rewrite it. So someone's doing rewriting, and it looks very much like it's not the browser?

and leaving it to the user to have specified a legal value

Right, please specify the right value, which is . I don't really know how to explain this to you in another way.
Again, attributes and properties are different things. What you see in DevTools are tags with attributes. Actual HTMLImageElement DOM object's width property is of type number.

@Pomax why not just use a style attr?

@leeoniya that's too easy :-) Also @Pomax says he needs pixels, so why not just omit px postfix? 😉

@leeoniya that incurs a reflow after DOM parse, which with 183 images over a very long article getting janky.
I already changed my code to omit the px but that doesn't mean I don't want to understand why this happens, as it breaks expectation without any kind of console warning (either during build, or in the browser). Silently rewriting data generally needs a very good reason, and I don't see it mentioned in the docs or reported during build/at runtime.
If nothing is doing an actual Number() cast, then what exactly is causing this to turn a string into the number zero? (Both preact-compat and preact do this on the same JSX input, where as react doesn't, and that is two-eyebrow-raisingly weird)

dimension attrs must not have units [1]. so i'm not sure what all the fuss is about.
[1] https://www.w3.org/TR/html5/embedded-content-0.html#dimension-attributes

@leeoniya that incurs a reflow after DOM parse, which with 183 images over a very long article getting janky.

DOM isn't being parsed, that's true only for HTML markup. Do you do Server Side Rendering?

If nothing is doing an actual Number() cast, then what exactly is causing this to turn a string into the number zero?

That's done by your browser.

(Both preact-compat and preact do this on the same JSX input, where as react doesn't, and that is two-eyebrow-raisingly weird)

React obviously has a special condition to what where it assigns an attribute instead of a property in that case.

@leeoniya I explained that already. The Topic Starter seems to not just get the different between DOM/HTML, attributes/properties and how JSX works.
I'm really running out of patience because we are figuring out the same thing again and again here, mostly explaining basics about the DOM and HTML.  I actually understand all those differences and clearly aren't explaining myself in a way that works for you, so: thank you for the advice to remove the px unit, preact rather than React has shaved 100kb off the deployed app so far while I'm trying to optimize this thing for people on a bad internet connection, or extremely expensive bandwidth, and preact is helping a lot in addressing their needs.
Apologies for taking up your time.

@Pomax

has shaved 100kb off the deployed app so far while I'm trying to optimize this thing for people on a bad internet connection, or extremely expensive bandwidth

meanwhile...

I have a React-managed page with some 175+ images

and

using Preact rather than React has shaved 100kb off the deployed app so far

you saved like 1-3 images worth of bandwidth. sounds like you're optimizing in the wrong places. not that i advocate using slow/bloated React
/hijacking

@leeoniya executing and parsing additional 100kb of JavaScript is kind of a big deal too. Yeah, bandwidth for images is much bigger, but they don't even start to load while your JS is being parsed/executed. So that's very important too.
P.S. I often find people using "we load huge images anyway" as an excuse to have MBs of JS.

yeah that 100kb is actually purely in the JS footprint, not the images themselves (those make up an out-of-app 500KB in gzipped .svg resources. And the insane number of http requests that incurs can also do with an optimization, but bundling them into the app itself, when the images can change independent of it, seemed a bad idea)

@NekR i know, i have a lib faster than preact in this space [1], though i would never promote using js server-side to generate html, even though domvm has very fast SSR/isomorphism (~50x React's).
i detest jsx and mountains of build dependencies that React & friends promote.
@Pomax

Of course the insane number of http requests those images incur

HTTP2, hopefully everywhere soon. Until then (and even afterwards), just use scroll position to load them on demand.
[1] https://github.com/leeoniya/domvm

@leeoniya Yeah, greatly looking forward to HTTP2 making it into browers and servers universally, a lot of sites will benefit from that.

@Pomax - It sounds like you're still not sure what's going on. I'll try to see if I can explain:
Internally, what happens when Preact renders the  in your JSX is roughly this:

The JSX code above is turned by Babel into h('img', { src: 'invalid', width: '400px', height: '400px' })
The result of the h(...) function call is a data structure that looks like this:

{
  nodeName: 'img',
  attributes: {
    src: 'invalid',
    width: '400px',
    height: '400px',
  },
  children: [],
}

Internally, Preact then takes the data structure from step (2) (vdomElement below) and creates an actual DOM element:

var node = document.createElement(vdomElement.nodeName);
// ...
for (var key in vdomElement.attributes) {
  // Does `node` have a property matching the prop name?
  if (key in node) {
    // Yes, set the property.
    node[key] = vdomElement.attributes[key];
  } else {
    // No, use `setAttribute`
    node.setAttribute(key, vdomElement.attributes[key]);
  }
}
// ...
parentElement.appendChild(node);

In the case when vdomElement.nodeName is img, node is an HTMLImageElement, which does have properties called width and height so what ends up getting executed is:

var node = document.createElement('img');
node.src = 'invalid';
node.width = '400px';
node.height = '400px';

Try the code from step (4) in your browser console and then print out the values of node.width and node.height.
Now on the other hand, try this:
// This is what happens when the browser parses the HTML string ''
var node = document.createElement('img');
node.setAttribute('width', '400px');
node.setAttribute('height', '400px');
console.log(node.width, node.height);

You'll see different results.
Does this make things clearer?

I'll add that you've now encountered the downside of Preact compared to React. Whereas React has a ton of checks and debugging messages in the codebase to warn you when you do something like this, Preact has very few.

There is an existing related issue for the difference between React and Preact here: https://github.com/developit/preact/issues/492

@robertknight it does, thanks.

Sorry for missing this discussion - thanks @robertknight for pointing to the previous issue.  Preact can't afford to bundle large whitelists/mappings for DOM edge cases, so right now it uses an in check to determine if a property should be used, otherwise falling back to attributes.  That behaviour handles a lot of cases, but is imperfect when DOM properties are mismatched to attributes of the same name.  Technically what Preact is doing is correct here, but it does differ from React and cause a bit of confusion - JSX properties look like attributes, so when they get used to set DOM properties that can be a little odd (even though it's the most common case).
FWIW, there's a PR in that, if landed, would fix this: https://github.com/developit/preact/pull/511


[scala mode] Add enum

Enums are available in Dotty and soon will be available in Scala.

Thanks, merged.


Update typescript

Updated Node along with other outdated npm dependences https://github.com/ministryofjustice/create-and-vary-a-licence/pull/756. So, closing this PR.


Turned into a package + made more Pythonic

Changed the code to be more pythonic, using classes and raising self made exceptions, for a cleaner and better ordered code, will continue to be adding features to this PR, as i really liked what you did with this idea and I love Ai Dungeon!

Now it is on the aur

Wow, between the 3 of us we've got a real 24/7 operation running :astonished:.
I had the thought of doing a proper python package, but you actually did the thing!
A few remarks:

Please resolve the conflicts. I added the possibility to have credentials as a login config + made the UTF8 splash logo 80 cols. They should be easy to integrate.
The PR name focuses of form ("we did that") over function ("we implemented that"). I would suggest something in the line of: "Turned into a package + made more"
Let's not embark too many stuff in a single branch. I see that your branch name is create-user-if-not-configured. Let's us please first finish and merge the packaging side of thing and then create another PR. Otherwise we'll ultimately fail to maintain a stable main repo.

Ill add the changes you made today, as soon as ill get to my pc, and then i think it willbe ok to merge

Finished with the changes for now, I think its ready for a merge, would be glad if you can review!

fixed it

Could you also mark the conflicts as resolved.
This will facilitate merging.
I'll merge this evening or tomorrow morning (CEST).
I've already created a release on the current branch to signify the following drastic change (of the code structure).

https://github.com/idangur/ai-dungeon-cli/pull/4

all conflicts resolved, thanks orteblan!

A bug fix w/ login + updated doc + various minor refactoring.
https://github.com/idangur/ai-dungeon-cli/pull/5

Ta-dam!
Thanks fro the good work @idangur and @otreblan.

Some comments about the last commits.
global term vars
Fine by me. Their not really part of the game logic. So it's better if they're separate (global or made into a class if they start to go up in number).
Furthermore, I thought already of maybe doing a systematic lookup of terminal width, to handle the fact that people tend to resize their term emulator windows.
handling of /quit
Also, maybe you didn't see my last comment in https://github.com/idangur/ai-dungeon-cli/pull/5.
I proposed raising a QuitSession instead of relying on stop_session.
What do you think about it?

You can raise QuitSession, if you think it fits accordingly, just remember to change the while loop to while true, even though I think it's a bit of bad practice (exiting a loop using an error)

it's a bit of bad practice (exiting a loop using an error)

Yes indeed, ideally exceptions should be caught as early as possible.
But we're already allowing to exit the loop with the KeyboardInterrupt and ConnectionError exceptions.
But I guess that's OK given the low complexity of the code (as of now).
@idangur, are you ok with this reasoning?