Avoid callbacks

Hi!
In Pixi, we usually bind events to sprites which mean we don't user some global mouse or keyboard event handler.
So usually I pass a context (=model) with mutable fields into sprites and some callbacks to create animations or interact on a global rendering context.
For instance: when I click o a sprite, I create an animation in the global rendering context using the sprite local information (position for instance). I use a callback to do so.
And if clicking on a sprite updates the score, I update the context by modifying some mutable field. Then the context is evaluated every frame in my update loop through a FSM to check for instance that I have some win conditions.
Last but not least, we use the pixi Ticker which gives us a convenient update loop.
So, as a commonly recurring topic among us can we do better? 😉
Thanks!

Not to sure to understand everything yet because I didn't dig into the code but when I was developing a game in JS i was using custom event instead of callbacks.
Thanks to F# we can have strongly typed event and perhaps we can even create some local state machine to handle them as we love them :)
By F#, i am speaking of:

// Describe our event
// Here I am using a DU, we can pass what use any type/structure
type PlayerEvent =
  | MoveTo of float * float
  | TakeDamage of float

// Instantiate the event delegate
let playerEvent = Event<PlayerEvent>()

// Register, our handler/state machine :)
playerEvent.Publish.Add(fun msg ->
  math msg with
  | MoveTo (x, y) -> // do something
  | TakeDamage value -> // do something
)

// Trigger the event
playerEvent.Trigger(TakeDamage 10.)

So we are not using callbacks anymore, or at least not directly because we would still need them to catch pixi event. But where before, we could have used XXX callback we can use one event listener and probably have a better abstraction layer ?

Ah indeed that's what I'm using in Suave. Didn't think about it for pixi. I need to try this.
Thanks @MangelMaxime 👏

What about using Elmish with Pixi? I had a look at Pixi.ticker and if I understand it correctly this is the place to put the view/render function. Besides that, we would have the classic update function to renew the Model (aka context) and when creating a sprite we would pass a dispatch function so the sprites can send messages to Elmish when an event is triggered (same way as React components do). In pseudo-code:
type Model =
  { Sprites = Sprite[] } // Using array for performance

type Msg =
  | Create of sprites: Sprite[]
  | MoveTo of name: string * x: float * y: float
  | Animate of whatever

let createSprite dispatch =
  let sprite = Sprite()
  sprite.onClick(fun ev -> Animate "foo" |> dispatch)
  sprite

let init dispatch =
   [| for i=0 to 10 do yield createSprite dispatch |]
   |> Create |> dispatch

let update model msg =
  match msg with
  | Create sprites ->
     { model with sprites = Array.append model.sprites sprites }, []
  | Animate _ -> failwith "TODO: Animate"
  | _ -> failwith "TODO: etc, etc"

let ticker = PIXI.ticker.Ticker()
ticker.stop()
ticker.add(fun deltaTime ->
  // Here comes the tricky part: we probably need to update the model
  // again using the deltaTime and there're several options for that.
  // We can send a message to update (though we have to make sure it
  // runs synchronously) or we can a specific updatePhysics (or similar) version.
  renderer.render(stage))
ticker.start()

ok. I need to do a few tests. I'll update the sample I'm currently working on. Thanks guys!

Hi @alfonsogarciacaro

Here comes the tricky part: we probably need to update the model
again using the deltaTime and there're several options for that.
We can send a message to update (though we have to make sure it
runs synchronously) or we can a specific updatePhysics (or similar) version.

I'm currently trying to play with the tricky part ;)
How can I dispatch a message  to the update function from the ticker loop?

Ok I have something working with subscriptions. I'll post a sample tomorrow.

BTW, it seems the elmish doc is not exactly up to date. (but that's another subject for another repo 😉 )


[Merged by Bors] - ci: don't quote the branch name twice

We end up with
const response = await github.rest.repos.getContent({
  owner: context.repo.owner,
  repo: context.repo.repo,
  path: 'lean-toolchain',
  ref: '"bump/v4.8.0"'
});

which turns out not to work.

The more proper fix would be to add
result-encoding: string

to the step above, according to https://github.com/actions/github-script?tab=readme-ov-file#result-encoding. But no need to obsess over this.


Rename Error Messages

I've got a The url '/' has no matching route. Consider using VRoute(path: '.*', widget: UnknownPathWidget()) at the bottom of your VRouter routes to catch any wrong route. and was wondering if instead of VRouter shouldn't we have VWidget?

If so I would be happy to make a contribution. :)

Oh you are right, how did I not see that x)
you can make a contribution sure! I'll wait for it this time :smile:

Should I target main or dev?

You did well, thanks again! 😊


[WIP] Clean up LS initialization options, update LS user config, make extraPaths nullable

Removes some old unused initialization options and adds new configuration options to package.json. extraPaths is now allowed to be null (the new default) to indicate that it's unset. The Jedi code has been updated to allow this as well.
The language server now reads extraPaths/searchPaths/etc directly from the user config, so does not need it in the initialization options. The minimum required language server version after this change is 0.4.9 for all of the new config options, but the current LS still supports searchPaths in the initialization options for backwards compatibility.
For #7338

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[x] Title summarizes what is changing
[x] Has a news entry file (remember to thank yourself!)
[x] Appropriate comments and documentation strings in the code
[x] ~Has sufficient logging.~
[x] ~Has telemetry for enhancements.~
[x] Unit tests & system/integration tests are added/updated
[x] ~Test plan is updated as appropriate~
[x] ~package-lock.json has been regenerated by running npm install (if dependencies have changed)~
[x] ~The wiki is updated with any design decisions/details.~

Marking WIP for the moment pending some feedback on one of these options. Please don't merge. 😃

Going to close this for now, pending some designing around overall settings. I'll bring it back later.


Importer button not doing anything

I exported my game and saved it in the right place but when I try to import something... nothing happens.
Foundry vtt ver. 6.5
The forge vtt hosting site

can you provide the file?  I'll take a look and see whats happening

Added a file upload button to fix issues where the file was not in the right place or if the file permissions were off, now works in the forge hosting environment.


rviz_ogre_vendor package doesn't seem to try to find_package(Ogre)

It seems that the assimp vendor package correctly tries to find an installed assimp and uses that if available. However, the Ogre vendor CMakeLists doesn't attempt to find Ogre (if I am reading the CMake correctly).
It would simplify our builds in the RoboStack project quite a bit if we could use the pre-built binaries from conda-forge. (some context here: https://medium.com/robostack/cross-platform-conda-packages-for-ros-fa1974fd1de3).

I don't recall now why we decided to unconditionally vendor OGRE.  @wjwwood do you remember?

I think that we vendored it because the versions below it didn't work for us, the ones in Ubuntu, and later versions had bugs too. I imagine this has been resolved by now, at least partially. But I know that the version used by rviz in ROS 1 and ignition/gazebo didn't work for us.
One reason we didn't have the conditional "find it upstream first" is that the cmake files that are provided by ogre were broken in some way and so we have a lot of custom logic for loading them as imported targets. I don't know if this logic can be used if the libraries are being provided separately. But maybe they can and we just didn't do that because it was extra work that we didn't have time to implement.

Thanks for the reply.
We've got a bunch of patches in the ros-galactic build-scripts repo: https://github.com/RoboStack/ros-galactic/tree/main/patch
I think some are very specific to our use-case, but we could try to generalize them enough to get them into a merge-able format. Would you be interested in that?

PS: rviz2 should work as a conda-package on linux: {conda or mamba} create -n ros2 ros-galactic-rviz2 -c robostack -c conda-forge if you want to try it out :)


Add fixes

display.set_matplotlib_formats('svg') is deprecated since IPython 7.23
modelparams[i,1] = grad

previous code:
from IPython import display
display.set_matplotlib_formats('svg')

modelparams[i,0] = grad

modified code:
from matplotlib_inline.backend_inline import set_matplotlib_formats
set_matplotlib_formats('svg')

modelparams[i,1] = grad

Thanks for catching that indexing bug. It was one I accidentally introduced after it being correct in the video :\
The svg issue is trickier, because the current version works in colab without requiring any additional installations. Visualization is annoyingly IDE-specific in Python and there are never solutions that work for everyone.


net/http/httptest: NewTLSServer doesn't support HTTP/2 requests

What version of Go are you using (go version)?

$ go1.13 version
go version go1.13.1 darwin/amd64
$ go-tip version
go version devel +f6c624a22a Wed Oct 16 15:58:33 2019 +0000 darwin/amd64

Does this issue reproduce with the latest release?
Yes, go1.13.1 and tip are affected.
What operating system and processor architecture are you using (go env)?
go env Output
$ go1.13 env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/rhys/Library/Caches/go-build"
GOENV="/Users/rhys/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GONOPROXY="*"
GONOSUMDB="*"
GOOS="darwin"
GOPATH="/Users/rhys/go"
GOPRIVATE="*"
GOPROXY="direct"
GOROOT="/Users/rhys/go/version/go1.13"
GOSUMDB="off"
GOTMPDIR=""
GOTOOLDIR="/Users/rhys/go/version/go1.13/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/49/zmds5zsn75z1283vtzxyfr5hj7yjq4/T/go-build721053994=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I called httptest.NewTLSServer and used the attached http.Client to make requests to it.
What did you expect to see?
I expected the requests to the local test server to use HTTP/2. (This is important for me at the moment because I'm writing reproducers for bugs I've seen in HTTP/2 client requests.)
What did you see instead?
The requests are sent over HTTP/1.1.
It looks like this is because the httptest package's TLS setup sets a tls.Config on the http.Transport before its first use, which defeats net/http's usual HTTP/2 autoconfiguration.
$  go1.13 test -v ./repro_test.go
=== RUN   TestHTTPTestHTTP2
=== RUN   TestHTTPTestHTTP2/basic
=== RUN   TestHTTPTestHTTP2/advertise-h2
=== RUN   TestHTTPTestHTTP2/transport-hacks
2019/10/16 13:23:20 http: TLS handshake error from <IP_ADDRESS>:64474: remote error: tls: bad certificate
--- FAIL: TestHTTPTestHTTP2 (0.14s)
    --- FAIL: TestHTTPTestHTTP2/basic (0.00s)
        repro_test.go:17: Request is not http/2: "HTTP/1.1"
    --- FAIL: TestHTTPTestHTTP2/advertise-h2 (0.00s)
        repro_test.go:37: Request is not http/2: "HTTP/1.1"
    --- PASS: TestHTTPTestHTTP2/transport-hacks (0.14s)
FAIL
FAIL    command-line-arguments  0.193s
FAIL

$ go-tip test -v ./repro_test.go
=== RUN   TestHTTPTestHTTP2
=== RUN   TestHTTPTestHTTP2/basic
=== RUN   TestHTTPTestHTTP2/advertise-h2
=== RUN   TestHTTPTestHTTP2/transport-hacks
2019/10/16 13:23:55 http: TLS handshake error from <IP_ADDRESS>:64490: remote error: tls: bad certificate
--- FAIL: TestHTTPTestHTTP2 (0.14s)
    --- FAIL: TestHTTPTestHTTP2/basic (0.00s)
        repro_test.go:17: Request is not http/2: "HTTP/1.1"
    --- FAIL: TestHTTPTestHTTP2/advertise-h2 (0.00s)
        repro_test.go:37: Request is not http/2: "HTTP/1.1"
    --- PASS: TestHTTPTestHTTP2/transport-hacks (0.14s)
FAIL
FAIL    command-line-arguments  1.254s
FAIL

package repro

import (
	"crypto/tls"
	"net/http"
	"net/http/httptest"
	"sync/atomic"
	"testing"
)

func TestHTTPTestHTTP2(t *testing.T) {
	t.Run("basic", func(t *testing.T) {
		var calls int64
		s := httptest.NewTLSServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
			atomic.AddInt64(&calls, 1)
			if !r.ProtoAtLeast(2, 0) {
				t.Errorf("Request is not http/2: %q", r.Proto)
			}
		}))
		defer s.Close()

		resp, err := s.Client().Get(s.URL)
		if err != nil {
			t.Fatalf("HTTP GET: %v", err)
		}
		resp.Body.Close()
		if have, want := atomic.LoadInt64(&calls), int64(1); have != want {
			t.Errorf("HTTP handler called %d times, expected %d times", have, want)
		}
	})

	t.Run("advertise-h2", func(t *testing.T) {
		var calls int64
		s := httptest.NewUnstartedServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
			atomic.AddInt64(&calls, 1)
			if !r.ProtoAtLeast(2, 0) {
				t.Errorf("Request is not http/2: %q", r.Proto)
			}
		}))
		s.TLS = &tls.Config{
			NextProtos: []string{"h2"},
		}
		s.StartTLS()
		defer s.Close()

		resp, err := s.Client().Get(s.URL)
		if err != nil {
			t.Fatalf("HTTP GET: %v", err)
		}
		resp.Body.Close()
		if have, want := atomic.LoadInt64(&calls), int64(1); have != want {
			t.Errorf("HTTP handler called %d times, expected %d times", have, want)
		}
	})

	t.Run("transport-hacks", func(t *testing.T) {
		var calls int64
		s := httptest.NewUnstartedServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
			atomic.AddInt64(&calls, 1)
			if !r.ProtoAtLeast(2, 0) {
				t.Errorf("Request is not http/2: %q", r.Proto)
			}
		}))
		s.TLS = &tls.Config{
			NextProtos: []string{"h2"},
		}
		s.StartTLS()
		defer s.Close()

		transport := s.Client().Transport.(*http.Transport)
		clientConfig := transport.TLSClientConfig
		transport.TLSClientConfig = nil

		// make a request to trigger HTTP/2 autoconfiguration
		resp, err := s.Client().Get(s.URL)
		if err == nil {
			t.Errorf(`Expected harmless "certificate signed by unknown authority" error`)
			resp.Body.Close()
		}
		// now allow the client to connect to the ad-hoc test server
		transport.TLSClientConfig.RootCAs = clientConfig.RootCAs

		resp, err = s.Client().Get(s.URL)
		if err != nil {
			t.Fatalf("HTTP GET: %v", err)
		}
		resp.Body.Close()
		if have, want := atomic.LoadInt64(&calls), int64(1); have != want {
			t.Errorf("HTTP handler called %d times, expected %d times", have, want)
		}
	})
}

@rhysh thank you for filing this issue and great to catch you here!
Here is how you can enable HTTP/2 for httptest but also for other servers
--- before_test.go	2019-10-16 13:42:17.000000000 -0700
+++ fixed_test.go	2019-10-16 13:48:43.000000000 -0700
@@ -6,20 +6,34 @@
 	"net/http/httptest"
 	"sync/atomic"
 	"testing"
+
+	"golang.org/x/net/http2"
 )

 func TestHTTPTestHTTP2(t *testing.T) {
 	t.Run("basic", func(t *testing.T) {
 		var calls int64
-		s := httptest.NewTLSServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		s := httptest.NewUnstartedServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 			atomic.AddInt64(&calls, 1)
 			if !r.ProtoAtLeast(2, 0) {
 				t.Errorf("Request is not http/2: %q", r.Proto)
 			}
 		}))
+		if err := http2.ConfigureServer(s.Config, new(http2.Server)); err != nil {
+			t.Fatalf("Failed to configure HTTP/2 server: %v", err)
+		}
+		s.TLS = s.Config.TLSConfig
+		s.StartTLS()
 		defer s.Close()

-		resp, err := s.Client().Get(s.URL)
+		tr := &http.Transport{TLSClientConfig: s.Config.TLSConfig}
+		if err := http2.ConfigureTransport(tr); err != nil {
+			t.Fatalf("Failed to configure http2 transport: %v", err)
+		}
+		tr.TLSClientConfig.InsecureSkipVerify = true
+		client := &http.Client{Transport: tr}
+
+		resp, err := client.Get(s.URL)
 		if err != nil {
 			t.Fatalf("HTTP GET: %v", err)
 		}
@@ -37,13 +51,21 @@
 				t.Errorf("Request is not http/2: %q", r.Proto)
 			}
 		}))
-		s.TLS = &tls.Config{
-			NextProtos: []string{"h2"},
+		if err := http2.ConfigureServer(s.Config, new(http2.Server)); err != nil {
+			t.Fatalf("Failed to configure HTTP/2 server: %v", err)
 		}
+		s.TLS = s.Config.TLSConfig
 		s.StartTLS()
 		defer s.Close()

-		resp, err := s.Client().Get(s.URL)
+		tr := &http.Transport{TLSClientConfig: s.Config.TLSConfig}
+		if err := http2.ConfigureTransport(tr); err != nil {
+			t.Fatalf("Failed to configure http2 transport: %v", err)
+		}
+		tr.TLSClientConfig.InsecureSkipVerify = true
+		client := &http.Client{Transport: tr}
+
+		resp, err := client.Get(s.URL)
 		if err != nil {
 			t.Fatalf("HTTP GET: %v", err)
 		}

and I have encountered this is in the wild which is why I shared this gist https://gist.github.com/odeke-em/cacfe2ea5efc57c5a7a562aba1478378
but anyways the crux of it is that you'll need to:
a) Make an Unstarted server
b) Invoke http2.ConfigureServer
c) do s.TLS = s.Config.TLSConfig and then s.StartTLS
d) For the client: make an http.Transport using s.Config.TLSConfig and obviously insecureSkipVerify
What I think we can do though is for the net/http/httptest package is add a new helper for example NewTLSServerWithHTTP2 which under the hood encompasses all those steps I listed out in a) to d), @bradfitz what do you think?

Or perhaps an option for NewTLSServer that enables HTTP/2 instead of the entirely new function NewTLSServerWithHTTP2

Howa about a new EnableHTTP2 bool on https://golang.org/pkg/net/http/httptest/#Server that then does the setup in Server.Start? (With any necessary changes to the Client method to return a configured client)
And an example, if docs aren't sufficient.
So the flow would be NewUnstartedServer, set the bool to true, and Start.

Perfect, SGTM, thank you @bradfitz! CL coming up shortly with an example too and tests :)

Thanks @odeke-em and @bradfitz . A new boolean field sounds like the right level of complexity for the user. Do we need even that?
Should http/2 be the default for httptest setups just as it is for production net/http clients and servers? When would a user want to opt out of http/2?
httptest has a mechanism for that opt-out already: setting the server's NextProtos to not include "h2". Do we need a new specialized field?

Should http/2 be the default for httptest setups just as it is for production net/http clients and servers? When would a user want to opt out of http/2?

net/http/httptest has been around for ages before HTTP/2 and the norm when testing is to use HTTP/1.1, and as you can see configuring HTTP/2 on these servers is a little tricky. Changing it to always use HTTP/2 might break lots of people's code who've been using this package for years. I think we can perhaps schedule a gradual roll out to invert the behavior, and like you mention, instead make HTTP/1.1 toggleable by the opt-out but after a while of having EnableHTTP2 around.


CB: Fix Sort

addresses: #1654
merge after: #1653

Current coverage is 64.84% (diff: 18.18%)

Merging #1662 into master will decrease coverage by 0.30%

@@             master      #1662   diff @@
==========================================
  Files           341        341
  Lines         16179      16216    +37
  Methods       15903      15940    +37
  Messages          0          0
  Branches        137        137
==========================================
- Hits          10542      10516    -26
- Misses         5637       5700    +63
  Partials          0          0

Powered by Codecov. Last update c4da1d2...ce4522a

@drostron also, this needs a changelog.

@alissapajer Will add with [skip ci] after underway build completes, as to not collapse its status here.

Added and ready. Status was collapsed anyway — passing build here.

@drostron I still see the two integration test marked as skipped.

^^ re the integration tests, I misread the diff :) all is good, will merge!


TRIVIAL: remove reference to deprecated template

_internal/google_news.html was deprecated and removed from Hugo in 0.111. Using newer hugo throws error.

Build succeeded (check pipeline).

https://checklist.intgdc.com/job/compliance/job/gooddata-docs-theme-oss-check/20/ : SUCCESS in 5s


PoC Electron 20 support?

📝 Description
a PR to check if electron 20 would pass the CI. when including the work of #903
❓ Context

Impacted projects: ``
Linked resource(s): ``

✅ Checklist

[ ] Test coverage
[ ] Atomic delivery
[ ] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

ok that quick check is done and nope, not passing the CI :) point added in https://ledgerhq.atlassian.net/wiki/spaces/WALLETCO/pages/3777495234/Upgrade+to+Electron+20 then


Mark Shows as Watched

Mark shows after 90% viewed as watched.

The Boxee Box is dead, this project is dead.


Fix 'Chat disabled due to missing profile public key' error

This was caused by the session object's type being AccountType.MOJANG instead of AccountType.MSA. Because of this, the profile public key failed to generate.

Thank you for this. I'll give it a test with my account soon to check if it's account-specific.
Otherwise, it looks good to merge! 👌

I've been hitting the same issue, and testing with this patch seems to have resolved it for me.


DST support

Is possible to use the date timezone offset to check if the current date is in DST period.
How do that:

Create a date on Jan first;
Create a date on Jul first;
Get the lower offset of the two dates;
Create a new date to get the current date;
If the current date offset is equal the lower one, is DST period.

Why is this useful?
To prevent missing and duplicate executions on DST changes.

There are cities where DST does not exist, so they have equal time zones always. For these cases, the algorithm described would return the existence of a DST where it does not exist.
I believe that algorithm below prevent this.
Date.prototype.stdTimezoneOffset = function() {
    var jan = new Date(this.getFullYear(), 0, 1);
    var jul = new Date(this.getFullYear(), 6, 1);

    return Math.max(jan.getTimezoneOffset(), jul.getTimezoneOffset());
}

Date.prototype.isDst = function() {
    return this.getTimezoneOffset() < this.stdTimezoneOffset();
}


The Image shape doesn't work in some cases

I want to custom the image size like that,  but it doesn't work.
var backgroundImage = new Image();
backgroundImage.src = 'imgs/2.jpg';
backgroundImage.width=200;
backgroundImage.height=200;
createShape(  'Image', {image: backgroundImage,x:0, y:0, scale:1})

According your sample doc, I try to write to add a image tool ,but it doesn't work.

MyTool(lc) {  // take lc as constructor arg
var self = this;
var newImage = new Image()
newImage.src = 'imgs/bear.png';
var firstx,firsty;
return {
usesSimpleAPI: false,  // DO NOT FORGET THIS!!!
name: 'MyTool',
iconName: 'image',
strokeWidth: lc.opts.defaultStrokeWidth,
optionsStyle: 'stroke-width',
didBecomeActive: function(lc) {
  var onPointerDown = function(pt) {

    firstx=pt.x;
    firsty=pt.y;

    self.currentShape = createShape(
      'Image', {image: newImage,x:{firstx}, y:{firstx}, scale:1});

    lc.setShapesInProgress([self.currentShape]);
    lc.repaintLayer('main');
  };

  var onPointerDrag = function(pt) {
    //self.currentShape.width = pt.x - self.currentShape.x;
    //self.currentShape.height = pt.y - self.currentShape.y;
    //lc.setShapesInProgress([self.currentShape]);
    //lc.repaintLayer('main');

    newImage.width=pt.x - firstx;
    newImage.height= pt.y - firsty;

  };

  var onPointerUp = function(pt) {
    newImage.width=pt.x - firstx;
    newImage.height= pt.y - firsty;

    //alert(firstx+"  "+firsty+"  "+newImage.width+"  "+newImage.height);

    lc.setShapesInProgress([self.currentShape]);
    lc.repaintLayer('main');
    lc.saveShape(self.currentShape);

  };

  var onPointerMove = function(pt) {
    console.log("Mouse moved to", pt);
  };

  // lc.on() returns a function that unsubscribes us. capture it.
  self.unsubscribeFuncs = [
    lc.on('lc-pointerdown', onPointerDown),
    lc.on('lc-pointerdrag', onPointerDrag),
    lc.on('lc-pointerup', onPointerUp),
    lc.on('lc-pointermove', onPointerMove)
  ];
},

willBecomeInactive: function(lc) {
  // call all the unsubscribe functions
  self.unsubscribeFuncs.map(function(f) { f() });
}

}
}

I'll try to look at this soon but I'm really busy right now. Without reading your code closely, I will ask: has the image already loaded when you try to use it?

Thanks for your reply.  Yes,  I  try put the preloading  code of "newImage.src = 'imgs/bear.png' "  on initial position to make sure the image has been loaded.


Add fields in Api

Currently listVerifiedDate, scope, contentType is not displayed in the api as fields. Is needed for laser.
Thx.

In which API request(s) you want to get those values?

We need the field in Package/GetRecord calls.

These are package fields, so it is about the OAI package API

Added in #141


[5.1][Stdlib] Special-case major version 9999 as always available.

rdar://problem/43776049

@swift-ci please test and merge

The failure on Linux was due to a package manager issue; it has been resolved by https://github.com/apple/swift-package-manager/pull/2027.

@swift-ci please test linux platform

Build failed
Swift Test Linux Platform
Git Sha - 8fc8e79a8a418f6fe63030d511ce6a28650f75cf

Not sure what happened that time around. Something crashed but I can't tell what or why. Trying again in the hope that it's spurious.

@swift-ci please test linux platform


Incompatibility between alias and maxUsages together in a label definition

Describe the bug
When using both the attribute alias and maxUsages in a label, maxUsages is not effective.
To Reproduce
Try to modify the playground example here as follows:
<View>
  <Labels name="type" toName="txt-1">
    <Label value="Brand" maxUsages="1"/>
    <Label alias="P" value="Product" maxUsages="1" />
  </Labels>
  <Text name="txt-1" value="$text" />
</View>

Then try to annotate twice both the label "Brand" and the label "Product". The first action won't be allowed, while I could select as many times as I wanted the label "Product".
Expected behavior
The attribute maxUsage to be effective also when using the attribute alias.
Screenshots

Environment (please complete the following information):

I used the playground.

Hello Spagliarini,
Thank you for reporting this. I'll create a ticket for this and will let you know.


Plugin doesn't allow 2FA

If you enabled 2FA in your Vereinsflieger user account, you cannot login using this plugin.

Because of that I now developed my own plugin.
For everyone else have look at https://github.com/Diginize/wp-vereinsflieger
The plugin will be published to the wordpress plugin page soon. Then you can also download it from there.


401 - Unauthorized: Access is denied due to invalid credentials.You do not have permission to view this directory or page using the credentials that you supplied

Hello i create my server with container sandox enviroment and hosted locally, my server ask for password change and when i change it i cannot sig in into Business central anymore i already change the password for the first i used but i have the same issue. am i missing something else?
Thanks in advance

@freddydk ?

@adrianordonez have you found a solution for your problem? Can you share it for future reference?

@adrianordonez, can we close this is issue or are you still facing it? Also, can you try to reproduce it on our latest developer previews?

no, i have to create a new sandbox enviroment


[QUESTION] Accessing the 9th row without using WithHeadingRow

Prerequisites

[x] Have read and understood: https://github.com/Maatwebsite/Laravel-Excel/blob/3.1/.github/SUPPORT.md
[x] Checked if your Laravel Excel version is still supported: https://docs.laravel-excel.com/3.1/getting-started/support.html#supported-versions
[x] Checked that your question isn't already asked before.
[x] Filled in the entire issue template

Versions

PHP version: 7.2
Laravel version: 7
Package version: 3.1

Description
Hi,
I am trying to import an Excel file with lots of merged cells in the upper rows. So, I came up implementing WithHeadingRow class so that I can set which row should the importing start (with the use of headingRow()). However, my header rows are also merged cells and I am having trouble with pulling data on those rows.
So, I was thinking using index number on accessing the data from that cell. However, when I try to use $row[0] it starts on the first row.
Is there any way in which I can use $row[0] starting on the 9th row?
Note: Rows 1-8 has lots of merged cells and I want to bypass them and start on 9th row in which records are placed
Your response is highly appreciated. Thank you.
Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.

Could you perhaps show me a bit more what you try to achieve / how your file looks like. I'm not really sure I understand what you want.
Maybe you can try Maatwebsite\Excel\Concerns\WithStartRow instead of WithHeadingRow. That one will skip the first x rows and have $row[0] index keys.

Maybe you can try Maatwebsite\Excel\Concerns\WithStartRow instead of WithHeadingRow. That one will skip the first x rows and have $row[0] index keys.

This is what I've been looking. Thank you so much!


batchInsert with existing transaction

batchInsert automatically starts/commits/rollsback a transaction but it would be useful if it could be used with an existing transaction which then isn't automatically completed.

@alexwhitman Does this not work?
knex.transaction(trx => {
  return trx.batchInsert(/* ... */);
});

I'm not at work to test it now but in theory it should.  However, from looking at the code wouldn't it start a new transaction within the existing?  If so that would seem redundant.

@alexwhitman This has been added to knex 0.11.1, and the documentation has been updated as to how this works.

@wubzz Thanks, just tested and it works fine.
Just a thought though, the example that @rhys-vdw gave was to call batchInsert on the transaction itself but that still starts a nested transaction.  Instead it would have to be called as
trx.batchInsert(data).transacting(trx);

which looks slightly redundant.  Would it be possible to modify so that both
knex.batchInsert(data).transacting(trx);

trx.batchInsert(data);

act in the same way?

The following patch appears to work but I'm not sure if it's the correct way to implement it:
diff --git a/src/util/batchInsert.js b/src/util/batchInsert.js
index 4aaaa86..5594c7f 100644
--- a/src/util/batchInsert.js
+++ b/src/util/batchInsert.js
@@ -19,7 +19,10 @@ export default class BatchInsert {
     this._returning       = void 0;
     this._transaction     = null;
     this._autoTransaction = true;
-
+
+    if (client.transacting) {
+      this.transacting(client);
+    }
   }

   /**

@alexwhitman I can see the convenience in that, and I think your diff makes perfect sense. Feel free to make a PR on that!


Crash in command generate. internal error: entered unreachable code

Hi Prisma Team! The following command just crashed.
Command
generate --schema [redacted]
Versions

Name
Version

Platform
windows

Node
v18.12.0

Prisma CLI
5.1.1

Engine
6a3747c37ff169c90047725a05a6ef02e32ac97e

Error
RustPanic: RuntimeError: panicked at 'internal error: entered unreachable code: No unsupported field should reach that path', query-engine/schema/src/build/input_types/mod.rs:26:40

@JulioJuarezM Is this still a problem for you?
Without more information, we are not able to reproduce this.
We would need your Prisma schema here.

Hey @JulioJuarezM  ,
Did you get a chance to have a look at my collegues comment? It'd be helpful for us to move your issue forward.
Without further information, we'll have to close this issue. Thank you 🙏

We're closing this issue due to inactivity. Please leave a comment below, ideally with a reproduction repository, if you have more actionable feedback about this issue. Thanks!


recipes-connectivity: Port ModemManager update

Until it gets merged in meta-balena PR https://github.com/balena-os/meta-balena/pull/3184
Changelog-entry: recipes-connectivity: Port ModemManager update

@resin-jenkins test this please

@balena-ci I self-certify!


Setup issues

I have a couple of problems I encountered while trying to set everything up, I would like to ask for help:

I am in venv, I intalled the tools from requirements.txt and I surely have numpy (when I tried to run pip install numpy I got
Requirement already satisfied: numpy in /home/emilia/digital_humans/dg/lib/python3.8/site-packages (1.24.2) ). Despite that, when I want to try running tests locally with pytest I get:

ImportError while importing test module '/home/emilia/digital_humans/a3-emilia-szymanska/tests/test01.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
tests/test01.py:5: in
import numpy as np
ImportError: No module named numpy
!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 1 errors during collection !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

Pytest was not in requirements, I installed it manually - shouldn't it be in requirements.txt if we want to run the tests with it?
When running the demo_01_sample_bodies in the second cell I get an error:

RuntimeError: Attempting to deserialize object on a CUDA device but torch.cuda.is_available() is False. If you are running on a CPU-only machine, please use torch.load with map_location=torch.device('cpu') to map your storages to the CPU.
I guess I don't have cuda to run the notebook on it, I can use only CPU, so what should I change in the demo to check if I installed everything properly?

Thanks for the feedback.

This is because the pytest does not use the current python environment. You could optionally try python -m pytest ***.
Yes, you can install pytest manually in order to test it locally.
Yes, the demo scripts use cuda by default. You could set its device to CPU. Please feel free to explore.

Thank you, it helped.
I tried to explore, I set it to CPU and got the aforementioned error, I tried solutions in here and then I get an error of "AttributeError: 'dict' object has no attribute 'seek'. You can only torch.load from a file that is seekable. Please pre-load the data into a buffer like io.BytesIO and try to load from it instead.". I looked for solutions for LISSTCore, but Google doesn't show much for LISST. I think this error is LISST-specific and trying what is proposed by the interpreter is focused on torch. I would be grateful for help in making the demos work on CPU because as I see similar steps are done in the file for task 1, so for CPU-users (I guess many of us) it would mean problems again.

Using a CPU I am also running into problems. In the script files (which we need to generate the test files, right?), there is no way to change the loading behavior of Lisst. Lisst however, by default assumes GPU. If the scripts would use our local LISST installation we could fix this manually. But like this we cannot or am I overseeing something?

Regarding the device error in e.g. demo 1, you could look into shaper.load() or lisst/models/body.py, Line 56, and replace ckpt = torch.load(model_path) to ckpt = torch.load(model_path, , map_location=torch.device('cpu')) as a temporary solution. You could check here for more details. For demo2, you could fix the problem in a similar way.
Thanks for your feedback anyway, I will update the code in the next version :)

Perfect, now it works, thank you! Just to make sure before closing the issue - you mentioned it "as a temporary solution". Should we then keep the changes when running the non-demo code?

It is up to you, because the test scripts only read your result files for grading.

On 3 Apr 2023, at 16:52, emilia-szymanska @.***> wrote:
Perfect, now it works, thank you! Just to make sure before closing the issue - you mentioned it "as a temporary solution". Should we then keep the changes when running the non-demo code?
—
Reply to this email directly, view it on GitHub https://github.com/Digital-Humans-23/a3/issues/2#issuecomment-1494474008, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD5BAGD4JISOEBDELXKHDMDW7LP2TANCNFSM6AAAAAAWPWJC3E.
You are receiving this because you commented.

Okay, thank you for help!


CAMEL-12947:MockEndpoint.expectedHeaderReceived should fail when no e…

…xchange received (#2669)

I wonder if it had already been merged.
Moreover, the related issue seems to have been already resolved.

It's a backport of the fix for 2.23.x. it is fixed on 3 and 2.x and 2.24.x

There was a discussion about this

Merged. Thanks


nudge-to-fine workflow not updating SSTs

The nudging runs are supposed to update coarse-model SSTs with the reference state but the N2F workflow in the sklearn_runfile is missing the actual state update call after it gets the reference SSTs:
https://github.com/VulcanClimateModeling/fv3net/blob/f646892b3c7fc752efadae9d70df94a45e512489/workflows/prognostic_c48_run/runtime/steppers/nudging.py#L89

The wrapper should override whatever the SST is set to in the fortran model; part of me wonders though whether we should verify this is still true since VulcanClimateModeling/fv3gfs-fortran#93 was merged.

My suspicion here was correct.  I have verified this with a test in the Python wrapper.  Similar to the surface radiative fluxes, I think we will need to use a payload + namelist flag approach to setting this from Python.

This notebook illustrates that when completing a nudged run with the namelist flag added in VulcanClimateModeling/fv3gfs-fortran#173, that the SSTs are indeed modified when compared with a case without the namelist flag active.  No changes to fv3net beyond updating submodules will be needed.  Care will be needed to update configs for nudging runs, however; in those we should set the flag gfs_physics_nml.override_sea_surface_temperature to .true..

So what were the differences in the notebook above from if they weren't from the nudging runs setting the SSTs?

It's a little subtle.  If we do not set the SST via the wrapper, but use Sfcprop%tsfco as the sea surface temperature in the fortran, it stays set at its initial condition value.  In the default case, where we use Sfcprop%tsfc as the sea surface temperature in the fortran, it starts to follow the climatology.  So after two days, we can see a difference appearing.

Thanks that makes sense.

#1101 adds the fix required to fv3net to take advantage of this.  Note per discussion with @mcgibbon I have modified the relevant namelist parameter in the fortran model.  It is now called gfs_physics_nml.use_climatological_sst and by default is set to .true..  We must set it to .false. to enable overriding it in nudged runs.


BUG: two failing tests related to ARROW_DEFAULT_MEMORY_POOL in conda builds on unix

Describe the bug, including details regarding any error messages, version, and platform.
#14832 switched on the test suite within the conda-builds, which itself synced with https://github.com/conda-forge/arrow-cpp-feedstock/pull/875
In the process of doing two tests failing with the same error were skipped,
https://github.com/apache/arrow/blob/2c768a1767c0d7b33644910e0e96a949bc6f6f42/dev/tasks/conda-recipes/arrow-cpp/meta.yaml#L380-L381
That error looks like:
FAILED test_memory.py::test_env_var - assert 2 == 1
 +  where 2 = len(['WARNING: Logging before InitGoogleLogging() is written to STDERR', "W20221123 12:16:17.825700 443543 memory_pool.cc:113] Unsupported backend 'nonexistent' specified in ARROW_DEFAULT_MEMORY_POOL (supported backends are 'mimalloc', 'jemalloc', 'system')"])

which might indicate a missing default somewhere?
To fix this, the line quoted above should be removed, and the conda tests still pass.
Component(s)
C++, Continuous Integration, Python

I think that we should improve our test for -DARROW_USE_GLOG=ON build:
diff --git a/python/pyarrow/tests/test_memory.py b/python/pyarrow/tests/test_memory.py
index 092c50de33..77057bef59 100644
--- a/python/pyarrow/tests/test_memory.py
+++ b/python/pyarrow/tests/test_memory.py
@@ -134,8 +134,14 @@ def check_env_var(name, expected, *, expect_warning=False):
         res.check_returncode()  # fail
     errlines = res.stderr.splitlines()
     if expect_warning:
-        assert len(errlines) == 1
-        assert f"Unsupported backend '{name}'" in errlines[0]
+        assert len(errlines) == 1 or len(errlines) == 2
+        if len(errlines) == 1:
+            # ARROW_USE_GLOG=OFF
+            assert f"Unsupported backend '{name}'" in errlines[0]
+        else
+            # ARROW_USE_GLOG=ON
+            assert f"InitGoogleLoggingd()" in errlines[0]
+            assert f"Unsupported backend '{name}'" in errlines[1]
     else:
         assert len(errlines) == 0


First Nations Page - Global Notes for Nation

We want IPE to be able to write/update the notes that ALL EAO staff should know about the nation.
it should appear as rich text editor for IPE and as a label for everyone else

As a (type of user)
I want (goal)
so that (reason/ why/ value)
Acceptance Criteria
Scenario 1:
GIVEN ... (you or your condition)
WHEN ... (what you do)
THEN ... (what you see)
Scenario 2:
xxxxxx
Validation Rules?
(If yes, list here)
Screenshot
(Link above will always be the most up-to-date version.)

EPIC.consult


Incomplete build instruction

How to Buld The Project

typo in the headline
After installing yarn and cloning the repo I get the following error:

yarn install
00h00m00s 0/0: : ERROR: [Errno 2] No such file or directory: 'install'

Also typo in npm isntall instruction:
npm intsall -g @oxheadalpha/tznft

The modular contract tutorial states this instructions which seems to not be working:
yarn add -D @oxheadalpha/fa2-contracts
Usage: yarn [options]

yarn: error: no such option: -D

yarn install
00h00m00s 0/0: : ERROR: [Errno 2] No such file or directory: 'install'

Unable to reproduce. It might be a problem with your installation of `yarn'

 yarn add -D @oxheadalpha/fa2-contracts
 Usage: yarn [options]

yarn: error: no such option: -D

Unable to reproduce. It might be a problem with your installation of `yarn'


Extra row in Functions table GUI

Description

found by @zolutal
Steps to reproduce the bug

Rename function normally
See it added to the function table
Rename it again
There should now be an empty row as the last row in the functions table

Environment
No response
Additional context
No response

Closed in #204


1.1.0 release

since there's added functionality but backwards compatible I think this should be a Semantic Versioning Minor release.

Setting a release date for next Monday in the Milestone

Overview of changes

Hide empty HTML markup

profile social media
<ul> category tags

Profile ACF repeater fields can be now be empty

affiliation
degrees
network
social networks
links

Super Admin access for Theme Options Page
Post show/hide author now ACF field instead of hard-coded
All Blocks now use native anchor functionality instead of ACF field

Legacy anchor names should still work until you update to new field. New field is in right hand menu under "Advanced" dropdown.


moved header and footer code out of mashlib

Initial issue: https://github.com/solid/mashlib/issues/107
This change moves the header and footer code out of mashlib.
Solid-panes is not responsible to create the main page of mashlib which in turn wires in the header and footer elements from solid-ui.
The following PRs need to be approved first (in this order):

https://github.com/solid/solid-ui/pull/461 -> this PR does not influence anything in production
https://github.com/solid/solid-panes/pull/323 -> this PR does not influence anything in production

Remark: no code was lost or deleted, it was refactored.

This feature does not appear at all in the web app (mashlib/dist/browse.html).  I am strongly opposed to pushing commits to mashlib that do not work on mashlib/dist/browse.html.  If they do not work there, they are unlikely to work in any non-server based version of SolidOS.  If we have any commitment to use of SolidOS in other contexts, we can not see keeping the web app working as an enhancement always in the do-later pile.  We don't need to necessarily test in Data Kitchen or other apps and stacks which use SolidOS, but the web app is critical.
In a very few cases there will be situations where the auth flow prevents the web app from operating like the server-based app.  In those situations, we can simply note the problem and move on. A much more likely reason features would work in SolidOS but not the webapp is checking for logged-in status rather than currentUser and that should be fairly easy to fix in the code.  The person who is most able to check for those kinds of issues is the person who wrote the new code.
I am absolutely willing to help debug web app issues, please do ping me with questions or problem areas.  I am NOT willing to be the only member of the team who checks if the web-app works or be the one expected to fix everyone else's code in relation to the web app.  We should discuss in team whether the web app is an integral part of SolidOS (I believe it is), or an appendage that we may or may not keep up to date.

The header and footer didn't work in the web app before this PR so perhaps it is not the responsibility of this PR to fix that.  I'll go ahead and raise a separate issue and approve this PR.

@jeff-zucker you are not the only one to test the web-app. I do it regularly either locally https://.localhost:8443/browse.html or on solid.github.io or an the test-server.
The code restructuring should have no impact on browse.html.

@bourgeoa thanks for doing that testing.

The code restructuring should have no impact on browse.html.

That is true and why I approved this PR. It seems to me that restructuring is a time when it is appropriate to look at the web app and to at least raise an issue if the feature is missing.

@bourgeoa - could we make it standard practice to make browse.html available on the server the way /mashlib.js is?  Or if you don't want it available on the main server for some reason, maybe just on the test server.  It would make it much easier to test things.

@jeff-zucker https://solidcommunity.net:8443/browse.html or https://:8443/browse.html on the test server is running the mashlib's from the test server.
Same on the regular solidcommunity.net server.

@bourgeoa, excellent, thanks!


Accessing structure member's respective buffer region during decoding?

I've got a question. I have a structure which has hash of one of it's members calculated over it's ASN.1 representation. Is there a way to know from what part of the buffer the member was decoded? I'm using DER.

The DER is defined in a special way that guarantees that if you encode it again you'd get a buffer which corresponds to the original buffer exactly. That is called a "canonical" form. I'd recommend you just encoding the part of the stucture using asn_encode_to_new_buffer() and using a hash function over the result.


[Sensor][PYTHON][GFX] FishEyeSensor Bindings

Motivation and Context

Adds Python bindings for FishEye Sensor

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[X] I have completed my CLA (see CONTRIBUTING)
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.

Need to lock in generation of ground truth.

Also, it would be awesome if we can add this new fisheye sensor to the colab tutorial, e.g., in the navigation section.

BTW, wanted to give a heads up to @matsuren about this if you want to help implement any other sensors/shaders. This looks like it will be at least 3X as fast the ObservationTransformer hack. :)

Hi @Skylion007, thank you for the information! It's awesome that it's written in C++ with shaders!

@matsuren If there are any other shaders, you would be interested in contributing like (Omnidirectional Stereo or such), let us know. Would be happy to get it merged.

@Skylion007 Thank you! I will let you know when I have something in my mind. (recently, I'm swamped with other projects, so I hope I can contribute in the future;)


Updated Terraform files to support Terraform >0.13

In Terraform 0.13 and later, provider requirements are declared in a required_providers block. In previous version of Terraform, versions could be declared directly within the provider block.
References:
https://www.terraform.io/docs/language/providers/requirements.html#requiring-providers
https://www.terraform.io/upgrade-guides/0-13.html
What was modified:

Updated ./main.tf to use the required_providers block to declare provider versions.

Updated ./agones-cluster/main.tf to use the required_providers block to declare provider versions. Also referenced an updated helm module (that works with Terraform >0.13), by switching from release-1.12.0 to release-1.13.0 (https://github.com/googleforgames/agones.git//install/terraform/modules/helm3/?ref=release-1.13.0)

Updated ./citadel/main.tf to remove load_config_file, since it is now referenced in a different way (https://registry.terraform.io/providers/hashicorp/kubernetes/latest/docs#file-config). The flag was set to false anyway, so I just commented out this line and replaced with a proper config reference. Also updated requests to include an equal (=) sign in the kubernetes_deployment resources, which is now required. (https://registry.terraform.io/providers/hashicorp/kubernetes/latest/docs/resources/deployment)

Tested with Terraform v1.0.5
on linux_amd64

LGTM, just wanted to double check that @roberthbailey doesn't see anything I might have missed.

Looks like we are good to go! Merging!


Search in PATH when failing to retrieve the current directory

https://github.com/r-darwish/topgrade/issues/818
Reporting due to this bug. The which crate should search the executable in PATH even if the current directory doesn't exist.

fixed in 4.2.3


YARN_VERSION in netlify.toml has security issues & is deprecated.

When I use the YARN_VERSION (1.3.2 which is deprecated) specified in netlify.toml is often get the warning:
[DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
If this is an issue with netlify-cms as well than an issue should be there too.

@Jamamuuga thanks for letting us know, we are working on it!

yarn is now updated to 1.9.4


Test if the database can be created 'on the fly'

Need to specify database_name to a non-existing value and see if it creates during runtime.

Resolves https://github.com/radical-collaboration/GB-SC18/issues/2

As per the comments in the above ticket, we cannot create a db instance on the fly and expect the instance to be up and available. We do however create db collections on the fly.


Dots not created

Hi,
Thank you for the module. it's very useful i have a little question concerning the dots attribute.
I have this configuration with dots="true" but there is no dots showed:

    <slick  class="slider single-item slick-master" init-onload="false" data="dataLoaded" slides-to-show="1" dots="false"  slides-to-scroll="1" class="slider one-time" arrows="true" as-nav-for=".slick-nav" >

    </slick>

 1">

    <slick class="slick-nav" init-onload="false" data="dataLoaded" slides-to-show="2" slides-to-scroll="1" as-nav-for=".slick-master" dots="true" center-mode="true" focus-on-select="true" >

    </slick>

Thanks,

Hi,
Thanks for this plugin.
I am also facing the same issue but only on lenovo tablet. The dots are showing in iPad, samsung tablet and nexus tablet but not in lenovo 7 inches tab.
Appreciate any help.
Thanks,


Php 8 nullsafe operator doesnt work

Template
{{ $entity->executors->first()?->name() }}
Context (Environment)
NAME="Linux Mint"
VERSION="18.3 (Sylvia)"
LARAVEL 8.12
Description
Parse Error : syntax error, unexpected '->' (T_OBJECT_OPERATOR) on line 1 > 1 | executors->first()?->name() ?> | ^

same with https://github.com/shufo/vscode-blade-formatter/issues/61
we need to wait for upstream dependency to be support php 8.

Just released v0.7.4 with null safe operator support. It would be nice if it works properly on your platform.


chore(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/landing-zone-vpc/ibm (source)
module
patch
7.13.2 -> 7.13.3

terraform-ibm-modules/landing-zone/ibm (source)
module
minor
5.3.2 -> 5.4.0

Release Notes

terraform-ibm-modules/terraform-ibm-landing-zone-vpc (terraform-ibm-modules/landing-zone-vpc/ibm)
v7.13.3
Compare Source
Bug Fixes

Resolve the issue where the VPC stack continues to auto-generate address prefixes, despite having subnet address prefixes explicitly passed to the module. (#679) (e3640c6)

terraform-ibm-modules/terraform-ibm-landing-zone (terraform-ibm-modules/landing-zone/ibm)
v5.4.0
Compare Source
Features

remove all code to create a secrets manager instance - use the terraform-ibm-secrets-manager module instead (#660) (67c1bf7)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.

/run pipeline


Update composer.json

Update to use Laravel 10

Did you test it on Laravel 10?

I have little to no unit test so I just manually test it. Will add it soon though.

@josezenem Good day! Please join me on my Discord server if you have issues with this package.
Discord: https://discord.gg/bFpDTgp3


NB to connect to postgres in AWS

NB to connect to postgres in AWS and how to upload pandas DF to postgres DB

looks good

reviewed


Using zero in an integer field of a compound key results in bad insert query

When I run the program below, I receive an SqlException: "The number of columns for each row in a table value constructor must be the same."
Using the SQL profiler I discovered that the underlying query was this invalid query:
exec sp_executesql N'SET NOCOUNT OFF;
INSERT INTO [DayOfWeek] ([EventId])
OUTPUT INSERTED.[DOW]
VALUES (@p0),
(@p1, @p2),
(@p3, @p4);
',N'@p0 int,@p1 int,@p2 int,@p3 int,@p4 int',@p0=111,@p1=1,@p2=111,@p3=3,@p4=111

Test program:
using Microsoft.Data.Entity;
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace EF7InsertReport
{
   public class DayOfWeek
   {
      public virtual int DOW { get; set; }

      public virtual int EventId { get; set; }
   }

   public class MyContext : DbContext
   {
      public DbSet<DayOfWeek> DaysOfWeek { get; set; }

      protected override void OnModelCreating(ModelBuilder modelBuilder)
      {
         base.OnModelCreating(modelBuilder);
         modelBuilder.Entity<DayOfWeek>().Key(dow => new { dow.DOW, dow.EventId });
      }

      protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
      {
         optionsBuilder.UseSqlServer(...);
      }
   }

   class Program
   {
      static void Main(string[] args)
      {
         MyContext db = new MyContext();
         db.Database.EnsureDeleted();
         db.Database.EnsureCreated();

         //db.DaysOfWeek.Add(new DayOfWeek() { DOW = -1, EventId = 111 });
         db.DaysOfWeek.Add(new DayOfWeek() { DOW = 0, EventId = 111 });
         db.DaysOfWeek.Add(new DayOfWeek() { DOW = 1, EventId = 111 });
         db.DaysOfWeek.Add(new DayOfWeek() { DOW = 3, EventId = 111 });
         db.SaveChanges();
      }
   }
}

Hey,
This is because by default key properties will have values generated by the database if they are left set to the CLR default for the data type. Because int defaults to zero, it just looks like you never set a value for the first row, so EF has the database generate a value (presumably 1 since it's the first row being inserted) which then conflicts with the entity that you gave a value to.
We're still thinking about how to improve this scenario, since it seems to be tripping a few folks up. But given that it looks like you want to explicitly provide values rather than having the database generate them, you can just switch value generation off.
            modelBuilder.Entity<DayOfWeek>()
                .Property(d => d.DOW)
                .ForSqlServer()
                .UseNoValueGeneration();

related to https://github.com/aspnet/EntityFramework/issues/1653

Hey, thanks for the fast reply.  I have added the UseNoValueGeneration statement above and that definitely helps, but now I get an exception stating:

The instance of entity type 'EF7InsertReport.DayOfWeek' cannot be tracked because it has an invalid (e.g. null or CLR default) primary key. Either set the key explicitly or consider using an IValueGenerator to generate unique key values.

I assume it is because the field is also part of the primary key.  Is that expected?

@jlddodger yes, this is a known issue, check out the #1653 for a bit more context on this

@jlddodger in EF7 we introduced the idea of a sentinel value that represents null for properties that can not contain null. For integer values this is set to zero, but if you want zero the be a valid key value you can change it:
modelBuilder.Entity<DayOfWeek>()
                .Property(d => d.DOW)
                .Metadata.SentinelValue = -1;

BTW we just had a design meeting on this to simplify the API and provide better exception messages to help folks understand how things work https://github.com/aspnet/EntityFramework/wiki/Design-Meeting-Notes:-July-2,-2015#sql-server-value-generation-scenarios


Is there a fork that allows quicker inference on cpu?

See title.

Right now only a little quicker, but maybe much quicker eventually https://github.com/PABannier/bark.cpp

Right now only a little quicker, but maybe much quicker eventually https://github.com/PABannier/bark.cpp

Cheers man, just what I was looking for.

Right now only a little quicker, but maybe much quicker eventually https://github.com/PABannier/bark.cpp

Cheers man, just what I was looking for.

One more you may want to explore: https://github.com/intel-analytics/BigDL/pull/9016


GLTFStandardPBR material can not be processed in O3DE 22.10

Tilesets currently aren't displaying in O3DE 22.10 due to an issue with the material.
When opening a 22.10 project, the Asset Processor produces a warning that it can not process the GLTFStandardPBR material or any of its variants.
As noted in this issue,

O3DE 22.10 introduces multiple changes, e.g. to the organization of Shader code in the Atom Core gem, that break the build of the Cesium O3DE gem

seems windows have'nt been solved yet ? pity...


Protobuf in blockchain data

The current blockchain implementation serializes and deserializes into crdt by calling scale::encode method on SuperGenius and writing as a base::Buffer. That could make it difficult for the dashboard to get this data.
Consider changing how things are serialized by inserting protobuf structures instead.

Got the block header, hash and data to be serialized/deserialized in protobuf structures, however hash wasn't used on the dashboard, since it's mostly used for internal purposes.


Riak 2.0 Feature: security

The PHP client needs to support security per basho/riak#355 .

[ ] configure local credentials
[ ] return appropriate errors when requests fail
[ ] work correctly when requests succeed

php currently doesn't need "connect to protobuffs over TLS" because this only does support http.

Security is being released within the next few days as part of the rewrite.


Bugfix unquoted refs to schema in FDW code

Tested this on a vm with a fresh provision with username set to common-data and was able to import datasets. Going to merge now so that the client can continue to test in their staging environment.


ci: Update CI workflow to only build what has changed

Overview
Example run: https://app.circleci.com/pipelines/github/Codecademy/client-modules/8343/workflows/33995123-372c-4439-90d7-d6c9c27ce260

Runs jest tests against source code instead of expecting pre-built packages. Requires some setup if we add more packages that require other monorepo packages, but it's pretty simple to set up.
Splits out CI tasks based on whether they can run with only the built packages that have changed, or whether they need all packages to be built. Right now only the deploy tasks require all packages to be built.

What it doesn't do:

All tests are still run, I went down a rabbit hole trying to get only the relevant changed tests to run and it either didn't run enough tests to be valid (jest onlyChanged options), or was too complex to set up (jest projects) (maybe requires a jest upgrade).

Previous Workflow:

New Workflow

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

🚀 Styleguide deploy preview ready!https://5fc80b24a4fdfa00922b4aa7--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5fc80cabeb1a83009c768c96--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5fc8106c57cfa00158716fc3--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5fc8133f56db45014ee33efd--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5fc90cec478dae0baeac9a5a--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5fc91091d97f1a0ef726b46e--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/styleguide@21.0.5-alpha.9a4b92.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/styleguide@21.0.5-alpha.c6267f.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fc91488e379ab152b5703ab--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5fc957fb87cd8b0e7c21c728--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/styleguide@21.0.5-alpha.a53f7d.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/styleguide@21.0.5-alpha.a59e1e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fc95865ef7da00d3133d503--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/styleguide@21.0.5-alpha.247c03.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fc959067e589310314aa2ad--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.ad792c.0@codecademy/gamut-labs@7.0.5-alpha.ad792c.0@codecademy/gamut-styles@7.4.9-alpha.ad792c.0@codecademy/gamut-system@0.1.6-alpha.ad792c.0@codecademy/gamut@19.0.4-alpha.ad792c.0@codecademy/markdown-overrides@0.3.17-alpha.ad792c.0@codecademy/styleguide@21.0.5-alpha.ad792c.0@codecademy/tracking@0.1.3-alpha.ad792c.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.7e31d3.0@codecademy/gamut-labs@7.0.5-alpha.7e31d3.0@codecademy/gamut-styles@7.4.9-alpha.7e31d3.0@codecademy/gamut-system@0.1.6-alpha.7e31d3.0@codecademy/gamut@19.0.4-alpha.7e31d3.0@codecademy/markdown-overrides@0.3.17-alpha.7e31d3.0@codecademy/styleguide@21.0.5-alpha.7e31d3.0@codecademy/tracking@0.1.3-alpha.7e31d3.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.e18aa2.0@codecademy/gamut-labs@7.0.5-alpha.e18aa2.0@codecademy/gamut-styles@7.4.9-alpha.e18aa2.0@codecademy/gamut-system@0.1.6-alpha.e18aa2.0@codecademy/gamut@19.0.4-alpha.e18aa2.0@codecademy/markdown-overrides@0.3.17-alpha.e18aa2.0@codecademy/styleguide@21.0.5-alpha.e18aa2.0@codecademy/tracking@0.1.3-alpha.e18aa2.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fc95e257e58931c024aa05d--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.63789a.0@codecademy/gamut-labs@7.0.5-alpha.63789a.0@codecademy/gamut-styles@7.4.9-alpha.63789a.0@codecademy/gamut-system@0.1.6-alpha.63789a.0@codecademy/gamut@19.0.4-alpha.63789a.0@codecademy/markdown-overrides@0.3.17-alpha.63789a.0@codecademy/styleguide@21.0.5-alpha.63789a.0@codecademy/tracking@0.1.3-alpha.63789a.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fc95fdcadf6bc1a5d96dfbf--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.59dd42.0@codecademy/gamut-labs@7.0.5-alpha.59dd42.0@codecademy/gamut-styles@7.4.9-alpha.59dd42.0@codecademy/gamut-system@0.1.6-alpha.59dd42.0@codecademy/gamut@19.0.4-alpha.59dd42.0@codecademy/markdown-overrides@0.3.17-alpha.59dd42.0@codecademy/styleguide@21.0.5-alpha.59dd42.0@codecademy/tracking@0.1.3-alpha.59dd42.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fc961a27b043518d79bec71--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.4e40ba.0@codecademy/gamut-labs@7.0.5-alpha.4e40ba.0@codecademy/gamut-styles@7.4.9-alpha.4e40ba.0@codecademy/gamut-system@0.1.6-alpha.4e40ba.0@codecademy/gamut@19.0.4-alpha.4e40ba.0@codecademy/markdown-overrides@0.3.17-alpha.4e40ba.0@codecademy/styleguide@21.0.5-alpha.4e40ba.0@codecademy/tracking@0.1.3-alpha.4e40ba.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fc96b086ca15627e224ee02--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.39b85e.0@codecademy/gamut-labs@7.0.5-alpha.39b85e.0@codecademy/gamut-styles@7.4.9-alpha.39b85e.0@codecademy/gamut-system@0.1.6-alpha.39b85e.0@codecademy/gamut@19.0.4-alpha.39b85e.0@codecademy/markdown-overrides@0.3.17-alpha.39b85e.0@codecademy/styleguide@21.0.5-alpha.39b85e.0@codecademy/tracking@0.1.3-alpha.39b85e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fc96c23425c6b25030ad07e--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.500f21.0@codecademy/gamut-labs@7.0.5-alpha.500f21.0@codecademy/gamut-styles@7.4.9-alpha.500f21.0@codecademy/gamut-system@0.1.6-alpha.500f21.0@codecademy/gamut@19.0.4-alpha.500f21.0@codecademy/markdown-overrides@0.3.17-alpha.500f21.0@codecademy/styleguide@21.0.5-alpha.500f21.0@codecademy/tracking@0.1.3-alpha.500f21.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fc96d4887cd8b262621c5a1--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5fc96e3001780e266a1953e3--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.73e281.0@codecademy/gamut-labs@7.0.5-alpha.73e281.0@codecademy/gamut-styles@7.4.9-alpha.73e281.0@codecademy/gamut-system@0.1.6-alpha.73e281.0@codecademy/gamut@19.0.4-alpha.73e281.0@codecademy/markdown-overrides@0.3.17-alpha.73e281.0@codecademy/styleguide@21.0.5-alpha.73e281.0@codecademy/tracking@0.1.3-alpha.73e281.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.aaf6db.0@codecademy/gamut-labs@7.0.5-alpha.aaf6db.0@codecademy/gamut-styles@7.4.9-alpha.aaf6db.0@codecademy/gamut-system@0.1.6-alpha.aaf6db.0@codecademy/gamut@19.0.4-alpha.aaf6db.0@codecademy/markdown-overrides@0.3.17-alpha.aaf6db.0@codecademy/styleguide@21.0.5-alpha.aaf6db.0@codecademy/tracking@0.1.3-alpha.aaf6db.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fc97183aaa7a7009093cdbf--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5fc97347aaa7a700a493cd0e--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5fca54ba747b1d013cc7d6f0--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.080445.0@codecademy/gamut-labs@7.0.5-alpha.080445.0@codecademy/gamut-styles@7.4.9-alpha.080445.0@codecademy/gamut-system@0.1.6-alpha.080445.0@codecademy/gamut@19.0.4-alpha.080445.0@codecademy/markdown-overrides@0.3.17-alpha.080445.0@codecademy/styleguide@21.0.5-alpha.080445.0@codecademy/tracking@0.1.3-alpha.080445.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.a86484.0@codecademy/gamut-labs@7.0.5-alpha.a86484.0@codecademy/gamut-styles@7.4.9-alpha.a86484.0@codecademy/gamut-system@0.1.6-alpha.a86484.0@codecademy/gamut@19.0.4-alpha.a86484.0@codecademy/markdown-overrides@0.3.17-alpha.a86484.0@codecademy/styleguide@21.0.5-alpha.a86484.0@codecademy/tracking@0.1.3-alpha.a86484.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fca55b5d4c15800cb745719--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.f0277e.0@codecademy/gamut-labs@7.0.5-alpha.f0277e.0@codecademy/gamut-styles@7.4.9-alpha.f0277e.0@codecademy/gamut-system@0.1.6-alpha.f0277e.0@codecademy/gamut@19.0.4-alpha.f0277e.0@codecademy/markdown-overrides@0.3.17-alpha.f0277e.0@codecademy/styleguide@21.0.5-alpha.f0277e.0@codecademy/tracking@0.1.3-alpha.f0277e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fca58ff170311008d065920--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5fca5d79bf10e40096698191--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.59e3ad.0@codecademy/gamut-labs@7.0.5-alpha.59e3ad.0@codecademy/gamut-styles@7.4.9-alpha.59e3ad.0@codecademy/gamut-system@0.1.6-alpha.59e3ad.0@codecademy/gamut@19.0.4-alpha.59e3ad.0@codecademy/markdown-overrides@0.3.17-alpha.59e3ad.0@codecademy/styleguide@21.0.5-alpha.59e3ad.0@codecademy/tracking@0.1.3-alpha.59e3ad.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.d56736.0@codecademy/gamut-labs@7.0.5-alpha.d56736.0@codecademy/gamut-styles@7.4.9-alpha.d56736.0@codecademy/gamut-system@0.1.6-alpha.d56736.0@codecademy/gamut@19.0.4-alpha.d56736.0@codecademy/markdown-overrides@0.3.17-alpha.d56736.0@codecademy/styleguide@21.0.5-alpha.d56736.0@codecademy/tracking@0.1.3-alpha.d56736.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fca60f04bd789008fb18ce2--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5fca6b8ceccfcc0d06c70799--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.68e47a.0@codecademy/gamut-labs@7.0.5-alpha.68e47a.0@codecademy/gamut-styles@7.4.9-alpha.68e47a.0@codecademy/gamut-system@0.1.6-alpha.68e47a.0@codecademy/gamut@19.0.4-alpha.68e47a.0@codecademy/markdown-overrides@0.3.17-alpha.68e47a.0@codecademy/styleguide@21.0.5-alpha.68e47a.0@codecademy/tracking@0.1.3-alpha.68e47a.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.8073c6.0@codecademy/gamut-labs@7.0.6-alpha.8073c6.0@codecademy/gamut-styles@7.5.1-alpha.8073c6.0@codecademy/gamut-system@0.1.6-alpha.8073c6.0@codecademy/gamut@19.0.5-alpha.8073c6.0@codecademy/markdown-overrides@0.3.18-alpha.8073c6.0@codecademy/styleguide@21.0.6-alpha.8073c6.0@codecademy/tracking@0.1.3-alpha.8073c6.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fca7080dc10531712d31203--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.0c0ca3.0@codecademy/gamut-labs@7.0.6-alpha.0c0ca3.0@codecademy/gamut-styles@7.5.1-alpha.0c0ca3.0@codecademy/gamut-system@0.1.6-alpha.0c0ca3.0@codecademy/gamut@19.0.5-alpha.0c0ca3.0@codecademy/markdown-overrides@0.3.18-alpha.0c0ca3.0@codecademy/styleguide@21.0.6-alpha.0c0ca3.0@codecademy/tracking@0.1.3-alpha.0c0ca3.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fca7520ee709317167d3acd--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@3.11.2-alpha.b47f52.0@codecademy/gamut-labs@7.0.6-alpha.b47f52.0@codecademy/gamut-styles@7.5.1-alpha.b47f52.0@codecademy/gamut-system@0.1.6-alpha.b47f52.0@codecademy/gamut@19.0.5-alpha.b47f52.0@codecademy/markdown-overrides@0.3.18-alpha.b47f52.0@codecademy/styleguide@21.0.6-alpha.b47f52.0@codecademy/tracking@0.1.3-alpha.b47f52.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fca7a71dff9b000d5005314--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/styleguide@21.0.6-alpha.f3ba90.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fca92391990ca00ddc926ce--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/styleguide@21.0.6-alpha.e4809e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fcaa4461990ca1b49c924ec--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/styleguide@21.0.6-alpha.634b7c.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5fcab130b98fe32c7786851f--gamut-preview.netlify.appDeploy Logs


quit button

quit button doesn't pop out like the other buttons and just moves over a bit

it worked before dillyz messed with how the animations work so that's a him problem

resolved as of 51d538d46cd0d0b684126611c9c7393d9a5bc4d1
(i renamed quit hover to quit hover reanimated, assuming it was under a weird name for the system. i then realized that quit hover reanimated existed under the root folder, which means impostor placed them in the wrong spots himself.)


docs: simplify PR template to use github labels for change type

Motivation
The PR template is heavy-handed, and it causes Github to think that the change type list is a todo list in the PR list view.
Change Type
Choose one of the options below by adding an x now or at any time before submitting for review

[ ] Protocol Feature: changes the design of the hub and the protocol
[ ] Hub Feature: changes the design of the hub, but does not affect the protocol
[ ] Bugfix: solves an issue with the current implementation
[x] Chore: miscellaneous improvement to the system

Change Summary
Ask contributors to use github labels to mark PRs with the change type instead of a checklist.
Merge Checklist
Choose all relevant options below by adding an x now or at any time before submitting for review

[x] The title of this PR adheres to the conventional commits standard
[ ] Add feature, chore, bugfix to repo labels if we're going to merge this

Codecov Report

Merging #78 (e493abe) into main (88cf472) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #78   +/-   ##
=======================================
  Coverage   89.10%   89.10%
=======================================
  Files          10       10
  Lines         817      817
  Branches      199      199
=======================================
  Hits          728      728
  Misses         88       88
  Partials        1        1

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Example of how github processes the checklists


[SPARK-40724][PS] Simplify corr with method inline

What changes were proposed in this pull request?
Simplify corr with method inline
Why are the changes needed?
after inline was introduced into PySpark, we can use it to simplify existing implementations
Does this PR introduce any user-facing change?
No
How was this patch tested?
existing UT

Merged into master, thanks @HyukjinKwon


use version number instead of release in .travis.yml

release will change over time, but your REQUIRE file says this package supports julia 0.3 so it should continue to be tested

Coverage remained the same at 0.0% when pulling bf97684523ca98fea6924eb0ce0a20489580ff47 on tkelman:travisver into f3090fce81c5145f3a56e584683246ac53905380 on Tao-Hu:master.

Coverage remained the same at 0.0% when pulling bf97684523ca98fea6924eb0ce0a20489580ff47 on tkelman:travisver into f3090fce81c5145f3a56e584683246ac53905380 on Tao-Hu:master.

Coverage remained the same at 0.0% when pulling bf97684523ca98fea6924eb0ce0a20489580ff47 on tkelman:travisver into f3090fce81c5145f3a56e584683246ac53905380 on Tao-Hu:master.


Fix trading tutorial shares label

https://github.com/AugurProject/augur/issues/8518

@Mergifyio backport dev


Using '/' as resolve.alias breaks the assets urls

Describe the bug
Vite: 5.4.10
Inside vite.config.ts I'm setting
resolve: {
  alias: {
     '/': srcDir,
  },
},

and inside my React app I'm importing TS file and assets files as
import {Component} from '/my-app/Component'
import AssetUrl from '/assets/my-asset.svg'

TS files work as expected, but assets urls don't. Generated assets urls are correct, but the Vite Dev Server does not serve them.
<img src={AssetUrl}/>
<!-- This shows no image. -->

Removing the resolve.alias and changing the imports to be relative makes the assets urls work again.
In production using the build and vite preview everything works as expected. It's a Vite Dev Server only issue.
Reproduction
https://stackblitz.com/edit/vitejs-vite-wbvawj
Steps to reproduce
npm run init
npm run start

Watch the images not being served by Vite.
System Info
Vite: 5.4.10

System:
    OS: macOS 15.1
    CPU: (10) arm64 Apple M2 Pro
    Memory: 308.89 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 23.1.0 - /opt/homebrew/bin/node
    npm: 10.9.0 - /opt/homebrew/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 130.0.6723.71
    Safari: 18.1

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I don't think Vite can realistically or consistently support / alias. / is already reserved by Vite as root-relative (relative to your project root). It can also be used to access public files, e.g. import logoUrl from '/favicon.png'. It's best to use a more unique alias instead.

From my experience using / as src root is quite common. Not so common as other prefixes like @, but still broadly used. In fact resolution of modules using the / prefix works perfectly with Vite/rollup/webpack.
The problem arises only in the Vite Dev Web Server, which resolves the file but fails in serving it.
This is clearly a bug in the file web server and definitely not a “feature”.
If it is a core idea of Vite project to not support this common case, it should be at least stated clearly in the documentation at the “resolve.alias” section.

@bluwy before going straight to the documentation route, given the broad usage of the / prefix, are you 100% sure it is not possible to implement in the web server a lookup logic that tries to resolve files using the resolve.alias and in case of failure, then fallbacks to the public folder?
It would be nice to have some more opinions and discussions here to better understand the feasibility of the idea.

@sapphi-red

I don't think we can support it either. I guess we won't be able to ensure that it work consistently everywhere and would open up a new can of worms.
Also I would recommend using a different character because aliasing / would make the application codebase difficult to understand since / has kind a special meaning in Vite.

Thank you guys for your time and the support.


record number / handle

I removed handle from the base class. At one point it was being used for record number. I need two things: 1. record number as part of the base class, and 2. a unique id (possibly called handle). The fact that the two were combined is wrong.
At the same time, I noticed that record number was implemented with pParent which is a bad idea because it's a CBaseNode * that needs to be cast. That cast is bad. Try to remove pParent.

Moved to https://github.com/Great-Hill-Corporation/quickBlocks/issues/77


Unable to run grapher step for electricity mix dataset (sources related?)

Step grapher://grapher/energy/2022-08-03/electricity_mix fails. The error starts with:
MySQLdb.DataError: (1406, "Data too long for column 'name' at row 1")

And in the end it has:
sqlalchemy.exc.DataError: (raised as a result of Query-invoked autoflush; consider using a session.no_autoflush block if this flush is occurring prematurely)
(MySQLdb.DataError) (1406, "Data too long for column 'name' at row 1")
[SQL: INSERT INTO sources (description, `createdAt`, `updatedAt`, name, `datasetId`) VALUES (%s, %s, %s, %s, %s)]
[parameters: ('{"link": "https://www.bp.com/en/global/corporate/energy-economics/statistical-review-of-world-energy.html ; https://ember-climate.org/data/data-catal ... (444671 characters truncated) ...  Ember; Ember; Ember; Ember; Ember; Ember; Ember; Ember; Ember; Ember; Ember; Ember; Ember; Ember; Ember; Ember; Ember", "dataPublisherSource": null}', datetime.datetime(2022, 9, 22, 15, 53, 10, 825422), datetime.datetime(2022, 9, 22, 15, 53, 10, 825423), "Our World in Data based on BP Statistical Review of World Energy (2022); Our World in Data based on Ember's Global Electricity Review (2022); Our Wor ... (6493 characters truncated) ... ew (2022); Our World in Data based on Ember's Global Electricity Review (2022); Our World in Data based on Ember's European Electricity Review (2022)", 5678)]
(Background on this error at: https://sqlalche.me/e/14/9h9h)

I haven't looked into it, but there I see the source name repeated many times, so, maybe there's a problem when combining sources, which we have modified recently?

Yeah, maybe we need to be deduplicating already here. It doesn't look like a great end-user experience to have all the double-ups.

@pabloarosado it's a bug, combine_metadata_sources was modifying the input sources. Fix https://github.com/owid/etl/pull/500


ImportError: /data/repos/maskrcnn-benchmark/maskrcnn_benchmark/_C.cpython-37m-x86_64-linux-gnu.so: undefined symbol: _ZN2at18SparseCUDATensorIdEv

❓ Questions and Help
I have installed according to directions from INSTALL.md, running on the latest nightly build of PyTorch 1.0, inside a fresh, new conda environment called maskrcnn_benchmark.
I am getting the error when running the command > python tools/train_net.py

(maskrcnn_benchmark) user_name@server_name:  /data/repos/maskrcnn-benchmark$ python tools/train_net.py
Traceback (most recent call last):
File "train_net.py", line 18, in
from maskrcnn_benchmark.engine.inference import inference
File "/data/repos/maskrcnn-benchmark/maskrcnn_benchmark/engine/inference.py", line 10, in
from maskrcnn_benchmark.data.datasets.evaluation import evaluate
File "/data/repos/maskrcnn-benchmark/maskrcnn_benchmark/data/datasets/evaluation/init.py", line 3, in
from .coco import coco_evaluation
File "/data/repos/maskrcnn-benchmark/maskrcnn_benchmark/data/datasets/evaluation/coco/init.py", line 1, in
from .coco_eval import do_coco_evaluation
File "/data/repos/maskrcnn-benchmark/maskrcnn_benchmark/data/datasets/evaluation/coco/coco_eval.py", line 10, in
from maskrcnn_benchmark.structures.boxlist_ops import boxlist_iou
File "/data/repos/maskrcnn-benchmark/maskrcnn_benchmark/structures/boxlist_ops.py", line 6, in
from maskrcnn_benchmark.layers import nms as _box_nms
File "/data/repos/maskrcnn-benchmark/maskrcnn_benchmark/layers/init.py", line 8, in
from .nms import nms
File "/data/repos/maskrcnn-benchmark/maskrcnn_benchmark/layers/nms.py", line 3, in
from maskrcnn_benchmark import _C
ImportError: /data/repos/maskrcnn-benchmark/maskrcnn_benchmark/_C.cpython-37m-x86_64-linux-gnu.so: undefined symbol: _ZN2at18SparseCUDATensorIdEv

How to fix this?

Hi,
This is a duplicate of #119
You are probably not using PyTorch 1.0, and there is an old PyTorch version laying around in your machine that is conflicting.
Try doing in your interpreter
import torch
print(torch.__version__)

and it will probably show that you are not using 1.0

No, that is incorrect.  We are running this on an AWS AMI image, but everything is contained within a conda virtual environment:
$ source activate maskrcnn_benchmark
$(maskrcnn_benchmark)$ python

import torch
torch.version
'1.0.0.dev20181124'

Same question. The version of pytorch I use is  1.0.0a0+11ef519, it is built from source code.

Hello,
I have an issue to run the training after successful install of the repo.
This is the command line and the generated error message.
Thanks in advance,
Raouf
python -m pointnet2.train.train_cls
Traceback (most recent call last):
File "/home/raouf/workspace/gitprojects/Pointnet2_PyTorch/pointnet2/utils/pointnet2_utils.py", line 20, in
import pointnet2._ext as _ext
ImportError: /home/raouf/workspace/gitprojects/Pointnet2_PyTorch/pointnet2/_ext.cpython-36m-x86_64-linux-gnu.so: undefined symbol: _ZN3c105ErrorC1ENS_14SourceLocationERKSs
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/raouf/anaconda3/lib/python3.6/runpy.py", line 183, in _run_module_as_main
mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
File "/home/raouf/anaconda3/lib/python3.6/runpy.py", line 109, in _get_module_details
import(pkg_name)
File "/home/raouf/workspace/gitprojects/Pointnet2_PyTorch/pointnet2/init.py", line 17, in
from pointnet2 import utils
File "/home/raouf/workspace/gitprojects/Pointnet2_PyTorch/pointnet2/utils/init.py", line 8, in
from . import pointnet2_utils
File "/home/raouf/workspace/gitprojects/Pointnet2_PyTorch/pointnet2/utils/pointnet2_utils.py", line 24, in
"Could not import _ext module.\n"
ImportError: Could not import _ext module.
Please see the setup instructions in the README: https://github.com/erikwijmans/Pointnet2_PyTorch/blob/master/README.rst

I have exactly the same problem as topic while running the single gpu training line. I have removed and installed this for more than 10 times and still it's not working. And this is the result of collect_env.py
Collecting environment information...
PyTorch version: 1.0.1.post2
Is debug build: No
CUDA used to build PyTorch: 10.0.130
OS: Ubuntu 18.04.2 LTS
GCC version: (Ubuntu 7.3.0-27ubuntu1~18.04) 7.3.0
CMake version: version 3.10.2
Python version: 3.7
Is CUDA available: Yes
CUDA runtime version: 10.1.105
GPU models and configuration: GPU 0: TITAN Xp
Nvidia driver version: 415.27
cuDNN version: Could not collect
Versions of relevant libraries:
[pip3] numpy==1.16.2
[pip3] torch==1.0.1.post2
[pip3] torchvision==0.2.2.post3
[conda] blas                      1.0                         mkl
[conda] mkl                       2019.1                      144
[conda] mkl_fft                   1.0.10           py37ha843d7b_0
[conda] mkl_random                1.0.2            py37hd81dba3_0
[conda] pytorch                   1.0.1           py3.7_cuda10.0.130_cudnn7.4.2_2    pytorch
[conda] pytorch-nightly           1.0.0.dev20190316 py3.7_cuda10.0.130_cudnn7.4.2_0    pytorch
[conda] torchvision               0.2.2                      py_3    pytorch
Does anyone have any idea what else can I do?
Thanks in advance

I have a similar problem to the topic
from maskrcnn_benchmark import _C
ImportError: /home/s/maskrcnn-benchmark/maskrcnn_benchmark/_C.cpython-37m-x86_64-linux-gnu.so: undefined symbol: _ZN3c1019UndefinedTensorImpl10_singletonE
The error occurs while running the single gpu training line. I have removed and installed this for more than 10 times and still it's not working. And this is the result of collect_env.py:
Collecting environment information...
PyTorch version: 1.0.1.post2
Is debug build: No
CUDA used to build PyTorch: 10.0.130
OS: Ubuntu 18.04.2 LTS
GCC version: (Ubuntu 7.3.0-27ubuntu1~18.04) 7.3.0
CMake version: version 3.10.2
Python version: 3.7
Is CUDA available: Yes
CUDA runtime version: 10.1.105
GPU models and configuration: GPU 0: TITAN Xp
Nvidia driver version: 415.27
cuDNN version: Could not collect
Versions of relevant libraries:
[pip3] numpy==1.16.2
[pip3] torch==1.0.1.post2
[pip3] torchvision==0.2.2.post3
[conda] blas 1.0 mkl
[conda] mkl 2019.1 144
[conda] mkl_fft 1.0.10 py37ha843d7b_0
[conda] mkl_random 1.0.2 py37hd81dba3_0
[conda] pytorch 1.0.1 py3.7_cuda10.0.130_cudnn7.4.2_2 pytorch
[conda] pytorch-nightly 1.0.0.dev20190316 py3.7_cuda10.0.130_cudnn7.4.2_0 pytorch
[conda] torchvision 0.2.2 py_3 pytorch
Does anyone have any idea what else can I do?
Thanks in advance

this problem result from the version of nvcc, you can run by
conda uninstall pytorch
conda uninstall pytorch-nightly
conda uninstall torch
codna uninstall torchvision
because torchvision 0.3.0 has problem,
then:
conda install -c pytorch pytorch-nightly torchvision cudatoolkit=10.0
pip install torchvision==0.2.2
successfully!!

reinstall with changes the line into:
conda install -c pytorch pytorch-nightly torchvision cudatoolkit=9.0

conda install -c pytorch pytorch-nightly=1.0.0 torchvision=0.2.2 cudatoolkit=9.0

still have the same problem
ImportError: $/maskrcnn/maskrcnn-benchmark/maskrcnn_benchmark/_C.cpython-37m-x86_64-linux-gnu.so: undefined symbol: _ZN6caffe26detail37_typeMetaDataInstance_preallocated_30E

try python 3.6.5

Does the following error mean I need to uninstall cuda10.0 and reinstall cuda 9.0?
/maskrcnn_benchmark/layers/nms.py", line 3, in
from maskrcnn_benchmark import _C
ImportError: libcudart.so.9.0: cannot open shared object file: No such file or directory

this problem result from the version of nvcc, you can run by
conda uninstall pytorch
conda uninstall pytorch-nightly
conda uninstall torch
codna uninstall torchvision
because torchvision 0.3.0 has problem,
then:
conda install -c pytorch pytorch-nightly torchvision cudatoolkit=10.0
pip install torchvision==0.2.2
successfully!!

Thanks @futureisatyourhand, it worked for me :)

PyTorch version: N/A
Is debug build: N/A
CUDA used to build PyTorch: N/A

OS: Arch Linux
GCC version: (GCC) 4.8.5
CMake version: Could not collect
Python version: 3.7
Is CUDA available: N/A
CUDA runtime version: 10.1.243
GPU models and configuration:
GPU 0: GeForce GTX 1080 Ti
GPU 1: GeForce GTX 1080 Ti
Nvidia driver version: 440.31
cuDNN version: /usr/lib/libcudnn.so.7.6.4
Versions of relevant libraries:
[pip3] numpy==1.17.4
[pip3] torch==1.3.1
[pip3] torchvision==0.4.2
[conda] _pytorch_select           0.2                       gpu_0    https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/main
[conda] blas                      1.0                         mkl    https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/free
[conda] mkl                       2019.4                      243    https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/main
[conda] mkl-service               2.3.0            py37he904b0f_0    https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/main
[conda] mkl_fft                   1.0.15           py37ha843d7b_0    https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/main
[conda] mkl_random                1.1.0            py37hd6b4f25_0    https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/main
[conda] pytorch                   1.2.0           cuda100py37h938c94c_0    https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/main
[conda] pytorch-nightly           1.0.0.dev20190328 py3.7_cuda10.0.130_cudnn7.4.2_0    https://mirrors.tuna.tsinghua.edu.cn/anaconda/cloud/pytorch
[conda] torchvision               0.4.0           cuda100py37hecfc37a_0    https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/main

this problem result from the version of nvcc, you can run by
conda uninstall pytorch
conda uninstall pytorch-nightly
conda uninstall torch
codna uninstall torchvision
because torchvision 0.3.0 has problem,
then:
conda install -c pytorch pytorch-nightly torchvision cudatoolkit=10.0
pip install torchvision==0.2.2
successfully!!

worked for me!!!

this problem result from the version of nvcc, you can run by
conda uninstall pytorch
conda uninstall pytorch-nightly
conda uninstall torch
codna uninstall torchvision
because torchvision 0.3.0 has problem,
then:
conda install -c pytorch pytorch-nightly torchvision cudatoolkit=10.0
pip install torchvision==0.2.2
successfully!!

Thanks @futureisatyourhand, it worked for me :)

You're welcome. Many of the PyTorch's libraries are unstable and not very compatible.

this problem result from the version of nvcc, you can run by
conda uninstall pytorch
conda uninstall pytorch-nightly
conda uninstall torch
codna uninstall torchvision
because torchvision 0.3.0 has problem,
then:
conda install -c pytorch pytorch-nightly torchvision cudatoolkit=10.0
pip install torchvision==0.2.2
successfully!!

worked for me!!!

Many of the PyTorch's libraries are unstable and not very compatible.


Reduce requests needed for delete()

This change adds a new getObjectInstance method which creates the Swift object but does not call retrieve(). It can be used for delete() where retrieve() is not required. This saves a HEAD request.
Background:
I observed calls to delete() which threw exceptions because the object to delete does not exist (any more). What I expected in this case was a returned false and no exception. The exception originated from getObject() which fails at the retrieve() call when the object does not exist. My first impulse was to move getObject() inside the try catch block but then I noticed that the retrieve() is not required for delete() in the first place.

Hey @chrisnharvey, sorry for nagging again. If you don't have the time to look after this repository at the moment, feel free to add me as collaborator. I'd be happy to help out with reviewing PRs or issues.

Hi @mzur, so sorry for the delay in responding. This PR got completely lost in my inbox.
I will take a look this evening and merge for you.
I am going to be moving this repository to a new namespace, so I've been thinking about how I'm going to do that without causing too many issues. I'll be happy to add you as a collaborator when I've moved the repo.

@mzur This works great! Thank you.

Thanks! Will you publish a new release as well?

Hey there, any progress with moving this repo to a new namespace or publishing the new release? I'd love to push this version to production to get rid of all the error messages.

@chrisnharvey If you no longer have time to take care of this repo or move it to a new namespace, I'd be happy to maintain a fork from now on, too. Just say the word :wink:


[manager] Add check in manager for multiple init and allocate

Add a check in the manager to avoid multiple initializations of the same
variables and their allocations. This happens when inference/train is called
multiple times on the same model.
This patch adds a check inside the manager which does not re-init or allocate
if it is already initialized/allocated.
See Also #965 #974
Self evaluation:

Build test: [x]Passed [ ]Failed [ ]Skipped
Run test: [x]Passed [ ]Failed [ ]Skipped

Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #975. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

:octocat: cibot: @kparichay, One of the module of prebuild group is failed. If you want to get a hint to fix this issue, please go to https://github.com/nnstreamer/nntrainer/wiki/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/975-202103031923150.40810394287109-0c5f5525f050f0e65f6b85cb930b2015001dca52/.

Merged into a common PR in #976.


Limit the amount of readable data from response body, honor content-length in response headers.

Looks like Content-Length is not set for all the S3 call replies, there is no easier and effective way to verify server sending malicious data right now, since S3 itself doesn't set Content-Length.
The reason is also i think they don't set Content-Length is since server is sending data with Transfer-Encoding set to 'chunked' . Because of this reason they do not have to. Since they guarantee that server is sending in chunks not as one large blobs for client to consume. So i think we can defer the handling of response bodies for now.. Since for chunked transfer-encoding Content-Length will not be set and we cannot effectively limit and verify this.


Add art

As a visitor, I'd like to see some indication of where I am.
AC

favicon added to site
Banner added to site
Logo added to site

Favicon was already added via 05a86784a9ca5f17092c9a55a892c9f8c46a345a

This is now on Master