error: building boost-vcpkg-helpers:x64-windows failed with: BUILD_FAILED

Operating system
Win
Compiler
MinGW
Steps to reproduce the behavior
vcpkg install boost-iostreams:x64-windows

Failure logs

Additional context
No response

Thanks ,I was able to download it after I opened the web proxy.

I'm glad your problem has been resolved.


cmd/compile: bounds check not eliminated

I am writing code to count how long a common prefix two slices have, and I can't seem to make it bounds-check-free.
At current master, this function does not eliminate the bounds check for y[j], although it does for x[i]:
func f1(x, y []int, i, j int) int {
	i0 := i
	if i >= 0 && i < len(x) && j >= 0 && j < len(y) {
		for i < len(x) && j < len(y) && x[i] == y[j] {
			i++
			j++
		}
	}
	return i - i0
}

If I remove the i < len(x) && and the i++, then the y[j] bounds check is eliminated:
func f2(x, y []int, i, j int) int {
	i0 := i
	if i >= 0 && i < len(x) && j >= 0 && j < len(y) {
		for j < len(y) && x[i] == y[j] {
			j++
		}
	}
	return i - i0
}

If I leave out the i++ but bring back the (unnecessary) i < len(x), the y[j] bounds check returns:
func f3(x, y []int, i, j int) int {
	i0 := i
	if i >= 0 && i < len(x) && j >= 0 && j < len(y) {
		for i < len(x) && j < len(y) && x[i] == y[j] {
			j++
		}
	}
	return i - i0
}

This is a simpler case that also has a bounds check and may share the same root cause:
func g1(x, y []int, i, j int) int {
	i0 := i
	if i >= 0 {
		for ; ; i++ {
			if i >= len(x) {
				break
			}
			if x[i] != 0 {
				break
			}
		}
	}
	return i - i0
}

This equivalent program has no check:
func g2(x, y []int, i, j int) int {
	i0 := i
	if i >= 0 {
		for ; i < len(x) && x[i] != 0; i++ {
		}
	}
	return i - i0
}

This suggests the problem has to do with either the inverted condition or the use of a short-circuit to exit the loop, neither of which should fundamentally change what is being proved.
The same happens counting backward: h0 and h1 have checks (h0 has just one), but h2 is check-free:
func h0(x, y []int, i, j int) int {
	i0 := i
	if i <= len(x) && j <= len(y) {
		for i > 0 && j > 0 && x[i-1] == y[j-1] {
			i--
			j--
		}
	}
	return i0 - i
}

func h1(x, y []int, i, j int) int {
	i0 := i
	if i <= len(x) {
		for ; ; i-- {
			if i <= 0 {
				break
			}
			if x[i-1] != 0 {
				break
			}
		}
	}
	return i - i0
}

func h2(x, y []int, i, j int) int {
	i0 := i
	if i <= len(x) {
		for ; i > 0 && x[i-1] != 0; i-- {
		}
	}
	return i - i0
}

/cc @aclements @randall77 @rasky @brtzsnr (prove authors as best I can tell from git blame)

Also CC @zdjones

Does this work for you?
func f(x, y []int, i, j int) int {
	i0 := i
	for i >= 0 && i < len(x) && j >= 0 && j < len(y) && x[i] == y[j] {
		i++
		j++
	}
	return i - i0
}

Seems to be equivalent, has no bounds checks, generates considerably shorter code. The key piece is the addition of && j >= 0 to the inner loop condition. (Note that this should have no cost, as the compiler should optimize 0 <= j < len(y) into a single unsigned comparison.) Then I removed the outer if statement, as it was almost entirely redundant with the inner loop.

Another example (on ARM): in this function the order of i and j in the loop body statement affects the number of bounds checks.
func reverse(s []int) {
	for i, j := 0, len(s)-1; i < j; i, j = i+1, j-1 {
		s[i], s[j] = s[j], s[i]
	}
}

The natural order, s[i], s[j] = s[j], s[i], produces two checks:
// 0x0028 00040 (a.go:5)	MOVD	(R0)(R2<<3), R5
// 0x002c 00044 (a.go:5)	MOVD	R5, (R0)(R3<<3)
// 0x0030 00048 (a.go:5)	MOVD	R4, (R0)(R2<<3)
// 0x0034 00052 (a.go:4)	ADD	$1, R3, R3
// 0x0038 00056 (a.go:4)	SUB	$1, R2, R2
// 0x003c 00060 (a.go:4)	CMP	R3, R2
// 0x0040 00064 (a.go:4)	BLE	92 ; out of bounds
// 0x0044 00068 (a.go:5)	CMP	R3, R1
// 0x0048 00072 (a.go:5)	BLS	112 ; out of bounds
// 0x004c 00076 (a.go:5)	MOVD	(R0)(R3<<3), R4
// 0x0050 00080 (a.go:5)	CMP	R2, R1
// 0x0054 00084 (a.go:5)	BHI	40

The reversed order, s[j], s[i] = s[i], s[j], produces only one check:
// 0x0028 00040 (a.go:5)	MOVD	(R0)(R3<<3), R4
// 0x002c 00044 (a.go:5)	MOVD	(R0)(R2<<3), R5
// 0x0030 00048 (a.go:5)	MOVD	R4, (R0)(R2<<3)
// 0x0034 00052 (a.go:5)	MOVD	R5, (R0)(R3<<3)
// 0x0038 00056 (a.go:4)	ADD	$1, R3, R3
// 0x003c 00060 (a.go:4)	SUB	$1, R2, R2
// 0x0040 00064 (a.go:4)	CMP	R3, R2
// 0x0044 00068 (a.go:4)	BLE	84 ; out of bounds
// 0x0048 00072 (a.go:5)	CMP	R2, R1
// 0x004c 00076 (a.go:5)	BHI	40 // continue
// 0x0050 00080 (a.go:5)	; out of bounds


Wrong avatar displayed in the "post cards" view

The screenshot below shows on the bottom a post by someone else that I reposted, but it is shown with my avatar on it. This happens with all reposts from Michael Nielsen's account, possibly because he has not set a user name. It also happens with reposts from other people, but not systematically (looks random to me).

Hmm, it might also be because it is your (re)post.

I confirm the element reuse suspicion. When I scroll through a list of posts, it happens that I see the right avatar initially, but when I come back to the same place, it's mine instead.


fix: bump max instances from 5 to 100 as a default

What does this PR do?
Our sample infrastructure we test this on regularly hits the max number of instances when it is set to the current value of 5.  By default, GCP sets the max number of instances to 100 (https://cloud.google.com/functions/docs/configuring/max-instances) so we should just use that to make sure we don't hit the limit.
Testing
Tested using our sample infrastructure and at 50 it easily is able to run.  Setting to 100 should be more than adequate for a more modest deployment.

If there is not a ton of extra data that needs to be processed, the added instances will not be created and there will be no associated cost with them. If there was data that wasn't getting processed before, it will now get processed and there will be an increase proportional to extra processing we require.  However, this should be just enough to process all of the data we need and is not wasted cost (i.e. adding a min number of instances where we pay for them to sit idly and accrue cost).


Using Replicate Loras / Weights ?

Hi,
Is it possible to use weights that are already available on Replicate as a standalone deployment?

I know that uploading weights from HF or CivitAI into the input attribute returns an error.
I've tried inserting the deployment URL of a Flux Lora (example) in the Lora loader node, but that didn't work either..

Maybe I'm missing something, but in any case that would be a great feature to have 😅

Bumping this up !

Yes we also need custom model and lora loading and we have been waiting for weeks for https://www.prompts.ai/
@fofr

https://github.com/[sourceful-official/ComfyUI_LoadLoraModelOnlyWithUrl](https://github.com/sourceful-official/ComfyUI_LoadLoraModelOnlyWithUrl)
maybe you can use a node that download the external lora, lora is usually quite small

[https://github.com/[sourceful-official/ComfyUI_LoadLoraModelOnlyWithUrl](https://github.com/sourceful-official/ComfyUI_LoadLoraModelOnlyWithUrl)](https://github.com/[sourceful-official/ComfyUI_LoadLoraModelOnlyWithUrl](https://github.com/sourceful-official/ComfyUI_LoadLoraModelOnlyWithUrl))
maybe you can use a node that download the external lora, lora is usually quite small

which node?
we are using what fofr deployed on replicate is that node available? please give more info if possible
also we want to be able to download base models as well
i don't think it should take this much
today another client asked me this who is also trying to develop another system
i told that replicate best to scale but this feature that you need still lacking @fofr


ERESOLVE unable to resolve dependency tree

Trying to install with npm install  --save react-stockcharts. I instead get an error:
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^16.0.0" from<EMAIL_ADDRESS>npm ERR! node_modules/react-stockcharts
npm ERR!   react-stockcharts@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

Try npm install react-stockcharts --force

Try npm install react-stockcharts --force


Using commit that updates checksums does not allow new versions

TL;DR
If I use the following github action configuration
      - name: 'Setup Terraform'
        uses: 'abcxyz/secure-setup-terraform@4d361110d3718e05b118be20051ba2855a2cc9e1'
        with:
          terraform_version: '1.7.1`
          protect_lockfile: true
          terraform_wrapper: false

the action will fail with the message
shasum: terraform.sha256: no properly formatted SHA checksum lines found

even though there is a checksum for version 1.7.1 in for commit 4d361110d3718e05b118be20051ba2855a2cc9e1
https://github.com/abcxyz/secure-setup-terraform/commit/4d361110d3718e05b118be20051ba2855a2cc9e1
Expected behavior
No response
Observed behavior
No response
Environment Details
Standard GitHub action environment

Additional information
No response

I am no longer able to replicate this issue. Closing.


syz-manager: add simple email support

Users can specify an email address to reveive notifications when a
bug is discovered for the first time, without setting up a full fledged
dashboard. The supported mailer is mailx.
Signed-off-by: Tim Tianyang Chen<EMAIL_ADDRESS>

Also note bot build failed:
https://travis-ci.org/google/syzkaller/jobs/315623022
Run make format before committing changes.

Thanks for the review Vyukov. I see in Makefile clang-format uses a file for styling. Can you provide .clang-format? I can't find it anywhere for golang and if I use C/llvm style many other files will be changed.
Suggestions applied. PTAL

.clang-format is in top dir:
https://github.com/google/syzkaller/blob/master/.clang-format
"make format" should do the right thing.
But if it does not work, just run "go fmt ./..." (go fmt does not any style files), since you are not changing any C/C++ files anyway.

Oh I thought it was generated by clang-format when I first ran it, because I got the errors like below.  It looks like I have an older version of clang-format than travis. Anyways, go fmt fixed it.
go build "-ldflags=-s -w -X github.com/google/syzkaller/sys.GitRevision=f5c54e6ee3ffe886179d829a3561c817226f77d1+" -o bin/syz-fmt ./tools/syz-fmt
go fmt ./...
clang-format --style=file -i executor/*.cc executor/*.h tools/kcovtrace/*.c
YAML:10:35: error: unknown key 'AllowShortFunctionsOnASingleLine'
AllowShortFunctionsOnASingleLine: false
                                  ^~~~~
...
...
Error reading /home/titichen/gopath/src/github.com/google/syzkaller/.clang-format: Invalid argument
Can't find usable .clang-format, using LLVM style
...

clang-format -version
LLVM (http://llvm.org/):
  LLVM version 3.4.2
  Optimized build.
  Built Dec  7 2015 (09:37:36).
  Default target: x86_64-redhat-linux-gnu
  Host CPU: core-avx2

Looks good. Thanks.


How can I select an icon from dashboard which was looped using *ngFor in protractor? Can anyone help

This is the code:


Fix blaze websocket server example

If I run the websocket server example for blaze from the examples-blaze sub-project
runMain com.example.http4s.blaze.BlazeWebSocketExample
and point the websocket.org echo client at it, it fails during connection with the following error:
[NIO1SocketServerChannelFactory] INFO  o.h.blaze.channel.ServerChannelGroup - Connection to /0:0:0:0:0:0:0:1:59297 accepted at Tue Jun 23 13:29:03 PDT 2015.
[info] 13:29:04.286 [pool-1-thread-1] INFO  org.http4s.blaze.pipeline.Stage - Unknown connection header: 'Upgrade'. Closing connection upon completion.

For control, I spun up a sample Unfiltered websocket server and did the same thing, and it worked as expected.

Works for me! Are you using the latest master? What is the url you're connecting with? ws://localhost:8080/http4s/wsecho and ws://localhost:8080/http4s/ws both work for me. What browser are you using? I've tested with current Chrome and Firefox.

@bryce-anderson It works for me too. I was using the wrong url, missing the http4s part.
Sorry about that.

I am getting the same error but using the right url according to the above?
2021-05-22 18:14:42,930 [ioapp-compute-0] INFO  o.h.b.c.nio1.NIO1SocketServerGroup - Service bound to address /<IP_ADDRESS>:8081
2021-05-22 18:14:42,938 [ioapp-compute-0] INFO  o.h.server.blaze.BlazeServerBuilder -
  _   _   _        _ _
 | |_| |_| |_ _ __| | | ___
 | ' \  _|  _| '_ \_  _(_-<
 |_||_\__|\__| .__/ |_|/__/
             |_|
2021-05-22 18:14:43,035 [ioapp-compute-0] INFO  o.h.server.blaze.BlazeServerBuilder - http4s v0.21.23 on blaze v0.14.16 started at http://<IP_ADDRESS>:8081/
2021-05-22 18:14:52,011 [blaze-selector-0] INFO  o.h.b.c.nio1.NIO1SocketServerGroup - Accepted connection from /<IP_ADDRESS>:50508
2021-05-22 18:14:52,323 [ioapp-compute-1] INFO  o.h.s.blaze.Http1ServerStage$$anon$1 - Unknown connection header: 'Upgrade'. Closing connection upon completion.

URL: ws://localhost:8081/http4s/ws

I can reproduce this.  It's probably a different issue than 2015, but the context is here, so I'll just reopen this one.

Thanks. I've determined that the http4s prefix is definitely wrong for 0.21, but even when the request reaches the route, this is always logged:
2021-05-22 22:12:54,890 [blaze-selector-2] INFO  o.h.b.c.nio1.NIO1SocketServerGroup - Accepted connection from /0:0:0:0:0:0:0:1:59264
2021-05-22 22:12:54,891 [ioapp-compute-5] INFO  o.h.s.blaze.Http1ServerStage$$anon$1 - Unknown connection header: 'Upgrade'. Closing connection upon completion.

:+1: I was just looking at the code and trying it without and it works. Thanks for testing!

I think one of the difficulties I had is that I am testing a client impl at the same time. The moral here is to use a known-good client test tool and debug only one side at a time! 🙄


[Feature Request] Allow model selections

Running the command within the localhost framework, there are several options. Model (tiny, etc) isn't one of them. Could that be added?
Again, great job with this.
Thomas

Models currently need to be selected at the time of initializing docker/gunicorn. Maybe this can be an endpoint option to dynamically change models in the future.


AsyncTaskTarget_TestThrottleOnTaskDelay - Changed to ConcurrentQueue

More fixes to the async-mocks:
https://travis-ci.org/NLog/NLog/builds/519862238?utm_source=github_status&utm_medium=notification

This change is

Codecov Report

Merging #3307 into dev will increase coverage by <1%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             dev   #3307    +/-   ##
======================================
+ Coverage     80%     80%   +<1%
======================================
  Files        356     356
  Lines      28062   28062
  Branches    3734    3734
======================================
+ Hits       22465   22473     +8
+ Misses      4520    4514     -6
+ Partials    1077    1075     -2

smart one 👍


appany/helm-oci-chart-releaser:0.4.0 Helm Push is broken (+ incorrect status) on ubuntu-20.04

Describe the bug

Helm Push is broken: The Helm Package is saving to wrong location or file
Even the package wasn't pushed, the workflow reported success

To Reproduce

The Workflow stopped working without changing the version
Version<EMAIL_ADDRESS>
jobs:
  helm-publish-and-release-charts:
   # if: ${{ github.event.workflow_run.conclusion == 'success' || github.event_name == 'workflow_dispatch' }}
    runs-on: ubuntu-latest
	- name: Push helm-app chart
	        uses<EMAIL_ADDRESS>	        with:
	          repository: some-service
	          registry: ghcr.io/${{ github.repository_owner }}
	          registry_username: __token__
	          registry_password: ${{ secrets.GITHUB_TOKEN }}
	          name: service-app
	          path: k8s/helm-app
	          tag: 1.2.3

From the logs you see that the helm push is not finding the output:
Run echo *** | helm registry login -u __token__ --password-stdin ghcr.io/xxx
Login succeeded
  helm package k8s/helm-app --version 0.0.123
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    HELM_EXPERIMENTAL_OCI: 1
Successfully packaged chart and saved it to:

saved it to:  is literally empty
The following helm push is assuming the previous step was successful:
  helm push service-app-0.0.123.tgz oci://ghcr.io/xxx/yyy
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    HELM_EXPERIMENTAL_OCI: 1
The Kubernetes package manager

To begin working with Helm, run the 'helm init' command:

	$ helm init

This will set up any necessary local configuration.

It will just print the helm help, because the packaged file could not be found

I can't reproduce this on v0.4.0, does your directory k8s/helm-app contains service-app?
  - name: Chart
    uses<EMAIL_ADDRESS>    with:
      name: ${{ ... }}
      repository: ${{ ... }}
      tag: ${{ steps.release.outputs.tag }}
      path: src
      registry: ${{ ... }}
      registry_username: ${{ ... }}
      registry_password: ${{ ... }}

@sergeyshaykhullin thanks for looking into it.
Sorry the bug was on my end, we included the following action, which installed helm version 3.0.0.-alpha1 for some reasons
- name: Install helm
        uses: Azure/setup-helm@v1

However this action still reports success, even if nothing was pushed .. this could be a nice improvement

Yes, introducing errors that would break a workflow when raised can help in finding issues for stabilising the pipelines.


[ES6] ImportDeclaration, create specifiers node even if specifiers is empty

Today tern can support completion for specifier when specifier contains one character. I mean with the demo http://ternjs.net/doc/demo/#ES6
import {L// here Ctrl+Space shows List (it works)} from "./list"

I would like to benefit with tern completion for specifiers when specifier is empty. I mean with the demo http://ternjs.net/doc/demo/#ES6
import {// here Ctrl+Space shows List (doesn't work)} from "./list"

Today tern cannot support that, because specifiers array is empty. So it should be very cool if acorn could generates a specifiers array with a node in order to tern can retrieve where completion is triggered.
For your information, this completion feature is available for TypeScript and it is very helpful when you are using a module that you don't very well. You do Ctrl+Space inside specifiers and it shows you the whole exported functions, variables, etc

We don't need to change Acorn for this. See https://github.com/ternjs/tern/commit/ca8c4ac6f83095b79c3203bcc6482892d231e076

Thanks @marijnh !

I have tested with http://ternjs.net/doc/demo/#ES6, it works great, but it should be cool if completion could filter the existing specifier.
For instance:
import {PI, // Here Ctrl+Space shows only halve and not PI} from "./mymath"


Bug: Text annotation box area is not respected

When you create a text annotation it has a box area limiting the visibility in the slide
The script is not respecting the height, all the text is shown even it goes beyond the box area:
Live/playback:

Exported:

Hi! Nice catch.
This is relatively easy to fix: we can nest the text inside of a new SVG element that has the width and height of the text box.
<svg>
  <!-- https://stackoverflow.com/a/42608084 -->
  <svg x="10" y="10" width="10" height="10">
    <text x="0" y="0">Test test test</text>
  </svg>
</svg>


Add keybindings

Introduce useful keybindings to enable a smooth workflow. E.g. for item creation, deletion etc.

Another great option would be a ⌘ + K interface

Could be handy: https://github.com/timc1/kbar


Determine claim resolution timespan

Description

As a user, I want the system to  tell me about procedural delays regarding my claim

Scope of Work
Acceptance Criteria
Story Points

40

Priority

Low

Risk

High

Clarification needed. What kind of delays are we supposed to warn about?

Where do we obtain data about delays?

This would be a simple value that we would get from government sources or courts: someone at the Lab will ask for it. We can see if it's worth updating it regularly manually, fixing it as such or if we can get a dynamic value (unlikely).
FYI:  for example, the average case in small claims court takes 12-18 months from the moment it is introduced until a decision is rendered. Note that this will DEFINITELY be different in the case of the Régie du Logement + cases that have an urgency component are often expedited through, at least until a temporary decision is rendered on the urgent element of the case.
So all in all, I would tell you not to worry about it.

In other words, no computation required? It is hardcoded? If that is the case then the story points will be significantly less.

Alternative is using AI to identify the moment the case is introduced and decision rendered and computing the value between those two dates. Not sure if it will be accurate or meaningful, but perhaps this can be another avenue.


Fix how we filter active offers using started_at and finished_at fields

Description:
In previous change (where I removed action_status field from database) I introduced a bug in how funciton active filters offers. This pull request aims to fix that.
Proper computed action_status property should fulfill this table:

started_at \ finished_at
past
not set
future

past
finished
ongoing
ongoing

not set
finished
ongoing
ongoing

future
XXXXXXXX
future
future

New imports / dependencies:

N/A

Unit Tests:
All tests that use:

TestOffersCommons from backend/apps/volontulo/tests/views/offers/commons.py as to simply create active offer, we changes finished_at to None
test_offer_list_fields from backend/apps/volontulo/tests/common.py as started_at and finished_at should be able to be also None/null

Codecov Report

Merging #1005 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #1005   +/-   ##
=======================================
  Coverage   88.82%   88.82%
=======================================
  Files         108      108
  Lines        2505     2505
  Branches       28       28
=======================================
  Hits         2225     2225
  Misses        277      277
  Partials        3        3

Impacted Files
Coverage Δ

backend/apps/volontulo/models.py
96.47% <ø> (ø)
:arrow_up:

...ckend/apps/volontulo/tests/views/offers/commons.py
100% <ø> (ø)
:arrow_up:

backend/apps/volontulo/tests/common.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e0da50b...dc48d50. Read the comment docs.


Enable coredns etcd secure mode

 Issue by gyliu513
Wednesday Aug 16, 2017 at 09:41 GMT
Originally opened as https://github.com/kubernetes/kubernetes/issues/50761

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug

/kind feature
What happened:
The kubernetes federation has a provider of coreos, we cannot enable secure mode for etcd, it is better support secure mode for etcd.
[root@cfc-555 federation]# cat coredns-provider.conf
[Global]
etcd-endpoints = http://etcd-cluster.default:2379
zones = example.com.
coredns-endpoints = coredns-coredns:53

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version):
Cloud provider or hardware configuration**:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/sig federation
/cc @kubernetes/sig-federation-feature-requests

 Comment by gyliu513
Wednesday Aug 16, 2017 at 09:45 GMT

/cc @liqlin2015

cc @gyliu513

This is addressed by https://github.com/kubernetes/kubernetes/pull/47049
/close


Register common variables rather than passing them in all the time

Before I did the work for the user-agent (which will require extensionContext to be passed a bunch of places to get the package version), I decided to fix #137. This allows us to use common variables without having to pass them in everywhere.
I implemented this all the way through for the functions extension to make sure it would work with tests, etc. Assuming you approve, I'll start fixing in the other extensions as well.

This is replacing extensionVariables in some of our extensions then?  I'm pro this.

Okay I've sent out all PRs to give a full example using the functions extension: https://github.com/Microsoft/vscode-azurefunctions/pull/434
I would say this leverages the extensionVariables, rather than replaces the extensionVariables. I wanted to keep the ability for each extension/package to have different variables that they use. That's why 'ext' is not exposed from the ui package. You can register the exact same 'ext', but the ui package will only use the 4 variables that it needs from within the package.


Enable DB HA in Production.

DEVOPS PULL REQUEST
Related Issue

Resolves #5337.

Changes Proposed

Enables zone-redundant high availability of the production database.

Additional Information

Ideally, this should eliminate the prevalence of alerts that can kick off when the prod database goes down for maintenance.
It can take up to 120 seconds for a database to fail over, which may not eliminate all of our alerts. Thresholds may have to be tweaked accordingly, if we find this isn't working.
ZoneRedundant HA is in place for this pull request. This grants us additional redundancy, and has been fully tested to work with our setup.

Testing

HA can be verified after-the-fact via Azure. Aside from that, so long as the database isn't deleted, we will be good to go.
This item has been FULLY VERIFIED in pentest.
VERIFY THE TERRAFORM PLAN CHECK BEFORE MERGING.

Checklist for Primary Reviewer
Infrastructure

[ ] Consult the results of the terraform-plan job inside the "Terraform Checks" workflow run for this PR. Confirm that there are no unexpected changes!

Cloud

[ ] Oncall has been notified if this change is going in after-hours
[ ] If there are changes that cannot be tested locally, this has been deployed to our Azure test, dev, or pentest environment for verification

Reiterating from the description: This item has been FULLY VERIFIED in pentest. All testing is complete, and it simply needs approvals at this stage.

Plan lgtm. :rocket:


Add paging and limit returns on large sets to perf doc

[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9599d917-4e43-f1ae-452a-1f11aaa985ed
Version Independent ID: 5cc05b6c-c671-3607-3217-7db73ebc1f28
Content: ASP.NET Core Performance Best Practices
Content Source: aspnetcore/performance/performance-best-practices.md
Product: aspnet-core
Technology: aspnetcore-performance
GitHub Login: @mjrousos
Microsoft Alias: riande

Fixed in #22192


jigasi: init at 1.1-288-gccf44d7 + module

Motivation for this change
Jitsi Gateway to SIP is a server-side application that links allows
regular SIP clients to join Jitsi Meet conferences hosted by Jitsi
Videobridge.
Resolves #136911
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

While fiddling with the module, I noticed this exception occur:
Sep 11 15:08:58 yunix jigasi-start[1859763]: java.lang.UnsatisfiedLinkError: Directory separator should not appear in library name: /tmp/jna--116
0142225/jna500083100422037839.tmp
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at java.base/java.lang.Runtime.loadLibrary0(Runtime.java:827)
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at java.base/java.lang.System.loadLibrary(System.java:1873)
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at org.jitsi.utils.JNIUtils.loadNativeInClassloader(JNIUtils.java:137)
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at org.jitsi.utils.JNIUtils.loadLibrary(JNIUtils.java:86)
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at org.jitsi.utils.JNIUtils.loadLibrary(JNIUtils.java:45)
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at net.java.sip.communicator.impl.sysactivity.SystemActivityNotifications.<clinit>(SystemAct
ivityNotifications.java:125)
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at net.java.sip.communicator.impl.sysactivity.SystemActivityNotificationsServiceImpl.isSuppo
rted(SystemActivityNotificationsServiceImpl.java:606)
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at net.java.sip.communicator.impl.sysactivity.SystemActivityNotificationsServiceImpl.start(S
ystemActivityNotificationsServiceImpl.java:143)
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at net.java.sip.communicator.impl.sysactivity.SysActivityActivator.start(SysActivityActivato
r.java:70)
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at org.jitsi.impl.osgi.framework.BundleImpl.start(BundleImpl.java:307)
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at org.jitsi.impl.osgi.framework.launch.FrameworkImpl.startLevelChanged(FrameworkImpl.java:4
72)
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at org.jitsi.impl.osgi.framework.startlevel.FrameworkStartLevelImpl$Command.run(FrameworkSta
rtLevelImpl.java:137)
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at org.jitsi.impl.osgi.framework.AsyncExecutor.runInThread(AsyncExecutor.java:122)
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at org.jitsi.impl.osgi.framework.AsyncExecutor.access$000(AsyncExecutor.java:28)
Sep 11 15:08:58 yunix jigasi-start[1859763]:         at org.jitsi.impl.osgi.framework.AsyncExecutor$1.run(AsyncExecutor.java:231)

No idea how to fix this yet.

Done! Tested on my own jitsi-meet instance and got calls from jitsi-meet -> phone network and phone-network to jitsi-meet working. This is now ready to be reviewed.

Alright, it seems I have figured out how to bring the dial-in feature to work. I have tested the PR with a self-hosted Asterisk connected to my SIP Trunk. Each participant was able to each each other.
Looking forward to see this tested by someone else to confirm that my packaging is working as intended.

Marking as draft again in order to bring the jigasi package on par with the rest of the jitsi stack on master.

Rebased against master and updated jigasi to the latest release. Haven't tested it yet.

Still not giving up! Tested my changes with success:

code looks good, nixpkgs-review seems happy, and the nixos test for jitsi-meet still works.


Update building.md

looks like the command line flag for version was changed to use --

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "patch-1" git@github.com:vikstrous/distribution.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

hi @vikstrous go accepts -- and - for flag arguments.
Did -version fail for you?

$ ./registry
configuration error: configuration path unspecified
Usage:
  registry <config> [flags]

Flags:
  -h, --help[=false]: help for registry
  -v, --version[=false]: show the version and exit

Yeah, it looks like it's --version now.
On Mon, Oct 5, 2015 at 10:11 AM, Richard Scothern<EMAIL_ADDRESS>wrote:

hi @vikstrous https://github.com/vikstrous go accepts -- and - for flag
arguments.
Did -version fail for you?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/distribution/pull/1067#issuecomment-145601425.

Fixed in #1076


Invalid config - The following components and platforms could not be set up: alexa-media

Describe the bug
Restart HA - .. The following components and platforms could not be set up: alexa-media
To Reproduce
Steps to reproduce the behavior:

Stop HA

Delete alexa_media.******.pickle

Start HA Configurator ..  Click button to configure Alexa Media Player - Begin...

Works

RESTART HA

Invalid config - The following components and platforms could not be set up: alexa-media

System details
arch | armv7l
dev | false
docker | false
hassio | false
os_name | Linux
python_version | 3.7.3
timezone | Europe/Vaduz
version | 0.98.5
virtualenv | true

HACS installed Alexa Media Player version: | v2.1.2
Logs
Sep 14 17:32:54 rb3b hass[12190]: -------------------------------------------------------------------
Sep 14 17:32:54 rb3b hass[12190]: alexa_media
Sep 14 17:32:54 rb3b hass[12190]: Version: 2.1.2
Sep 14 17:32:54 rb3b hass[12190]: This is a custom component
Sep 14 17:32:54 rb3b hass[12190]: If you have any issues with this you need to open an issue here:
Sep 14 17:32:54 rb3b hass[12190]: https://github.com/custom-components/alexa_media_player/issues
Sep 14 17:32:54 rb3b hass[12190]: -------------------------------------------------------------------
Sep 14 17:32:54 rb3b hass[12190]:
Sep 14 17:32:54 rb3b hass[12190]: 2019-09-14 17:32:54 INFO (MainThread) [custom_components.alexa_media] Loaded alexapy==1.1.2
Sep 14 17:32:54 rb3b hass[12190]: 2019-09-14 17:32:54 DEBUG (MainThread) [alexapy.alexalogin] Trying to load pickled cookie from file /home/homeassistant/.homeassistant/alexa_media.....
Sep 14 17:32:56 rb3b hass[12190]: 2019-09-14 17:32:56 DEBUG (MainThread) [alexapy.alexalogin] cookie loaded:....
Sep 14 17:32:58 rb3b hass[12190]: 2019-09-14 17:32:58 DEBUG (MainThread) [alexapy.alexalogin] Trying to load aiohttpCookieJar to session
Sep 14 17:32:58 rb3b hass[12190]: 2019-09-14 17:32:58 DEBUG (MainThread) [alexapy.alexalogin] Updating self._cookies with amazon.de session cookies:
Sep 14 17:32:58 rb3b hass[12190]: amazon.de: session-id=259-9115812-9018616
Sep 14 17:32:58 rb3b hass[12190]: amazon.de: session-id-time=2199193391l
Sep 14 17:32:58 rb3b hass[12190]: amazon.de: ubid-acbde=258-4470430-8796302
....
Sep 14 17:32:58 rb3b hass[12190]: amazon.de: csrf=2092436387
Sep 14 17:32:58 rb3b hass[12190]:
Sep 14 17:32:58 rb3b hass[12190]: 2019-09-14 17:32:58 DEBUG (MainThread) [alexapy.alexalogin] Loaded 9 cookies

HA release 0.97.2 - August 11, 2019 all is working w/o any errors.

Can you provide the rest of the logs please? The logs don't show any problems where you've stopped.

@alandtse
See here:
HA-98-5-home-assistant restart
https://privatebin.at/?eaeaec0a9380f125#4L33cpeZ9Ypj842mifagXHY836oiGkRTKe2uTbrzKSTt
HA-98-5-home-assistant first start
https://privatebin.at/?fbc6ed11036537b2#J2pBcLxQgk1ogC4kSqX8tX4Kz9Sky1JPXZQCT4JNu2o4
HA-97-2-home-assistant - works
https://privatebin.at/?44dcc444168693b1#HhJsvyn6DMhT75Xd3RE42LMJrNcyRRuYN81hyuKZw6QX

Thanks. Ok. That's weird. So on reboot the logs just stop? Does HA load immediately or does it take some period of time before the UI shows up?

@alandtse
No, the logging goes on after a while and I see everyone else.
HA starts normal only that I see the message "Invalid config - The following components and platforms could not be set up: alexa-media" see.
But if I delete **alexa_media. ****. pickle and then reboot HA then it will work to the next restart.
Therefore, I always have to delete this when I restart.

Ok, the logging continues but it doesn't say anything else from alexa_media? Normally it should say something in the logs if it's giving you the "invalid config" message.

@alandtse
No more messages for alexa_media only from others - only the: Invalid config
I have a behavior in the web interface, which comes closer to this.
With this URL it works:
https://alexa.amazon.de/spa/index.html?#cards
Here I can sign in, but there is no content:
https://alexa.amazon.com/spa/index.html?#cards
Could it be that there is a problem with the domains?

I don't think it's a domain problem as that's the normal way it works when you try the webapp with the wrong domain. Sometimes Amazon will redirect you sometimes just the blank page.
Can you please enable advanced debugging for the second restart?  Please run until you see the invalid config error in the UI so I have a bit more in logs.

@alandtse
Log New installation
https://privatebin.at/?54bde0ce945547b0#CvnALZxdPTMBnKr55T6roRCwyq988Gx39THkKHqbjjkK
Log Restart
https://privatebin.at/?d25af102a91a352c#9SmzFZGYQVhbpGJkDRYRfAKbAaR84ZApXubyfqxQFmPN
successfully installed
  homeassistant-0.98.5
  importlib-metadata-0.19
  python-slugify-3.0.3
  pyyaml-5.1.2
  ruamel.yaml-0.15.100
  voluptuous-0.11.7
  voluptuous-serialize-2.2.0

alexa_media:
  accounts:
  - email: !secret alexa_user
    password: !secret alexa_pass
    url: amazon.de
    debug: True
    include_devices:
      - Esszimmer
      - Wohnzimmer
      - Office
      - Fitnessraum
      - Bad
    exclude_devices:
      - "Peter's Alexa Apps"
      - "Peters Alexa Apps"
      - "This Device"

Ahh, you're having an SSL error. According to the aiohttp thread, you need to update to python 3.7.4. Anyway you can try that?

@alandtse
OK, you're right. On HA 0.97.2 I do not see this error message.
Not so good, because upgrading from Python 3.7.3 to Python 3.7.4 for Debian 10 (Buster) on a Rasperry is a challenge and not so easy.
If I switch to Python 3.7.4, then I have to compile Python myself and lose the apt updates.
I still have to think about whether I would rather not stay on the HA version 0.97.2.
Thanks for the support.

This may be tied to the connection errors people have been reporting. I'll continue to look at a solution from the component level.

@alandtse
Thanks 👍

I have just experienced the same problem. It happened just after upgrade to HA 0.99.0 (on HassOS 2.12).
Strange thing is that it fixed itself after couple HA restarts. And I didn't have to delete *.pickle (as @zibous was saying).
After first restart (just after the upgrade) I saw notification about invalid configuration (same as the title of this issue). I have noticed that there is 2.2.0 update for alexa-media integration (I was on 2.1.2) so I've upgraded it as well.
Second restart and notification about "new devices detected" appeared ("invalid configuration" one was gone). Though all my alexa media player entities were unavailable.
Decided to do another restart (just in case - before trying to delete pickle file) and surprisingly everything worked.
I'm wondering what could be the reason here... maybe python was upgraded to 3.7.0 within 0.99.0 HA release? I see a Deprecate Python 3.6.0 note, in HA release notes, but I'm not sure if this means an upgrade or not.

Please try 2.2.1 and let me know if it persists.

I'm wondering what could be the reason here... maybe python was upgraded to 3.7.0 within 0.99.0 HA release? I see a Deprecate Python 3.6.0 note, in HA release notes, but I'm not sure if this means an upgrade or not.
That was a specific version of 3.6. The min version is 3.6.1 now, so it's not related.

@alandtse
Thank you, until now it works with the new version.
I have updated HA 0.97.2 to the latest version.
After the reboot notify.alexa_media works fine again.
arch 	armv7l
dev 	false
docker 	false
hassio 	false
os_name 4.19.66-v7+ armv7l
python_version 	3.7.3
timezone 	Europe/Vaduz
version 	0.99.2
virtualenv 	true
Raspbian GNU/Linux 10 (buster)


Release 2.8.0 for Windows & MacOs

Replacing Python Backend Web Development Course (with Django) with Argument Discussion/Dispute - Sound Effect for editing.

https://github.com/kkamara/kelvinkamara/releases


Adding Windows images for different Windows versions

Request
The recent versions of Grafana agent has started to include a Windows image (#646), but it currently only offers one image with mcr.microsoft.com/windows/nanoserver:1809 as the base image. As Windows containers are only compatible with a host OS with the same base version, this would make Windows version other than 1809 unable to run the Grafana agent's Windows image.
I propose that Grafana agent can build multiple Windows image, each for a different windows version, similar to how golang images build golang:1.20.4-windowsservercore-ltsc2022 and golang:1.20.4-windowsservercore-1809.
Use case
I am trying to run Grafana agent inside a Kubernetes cluster with both Windows and Linux nodes, but ran into compatibility issues on Windows nodes as the Windows 2022 machines cannot run the Grafana agent image which only has 1809 as base image.
Currently I am building my own Grafana agent Windows image using mcr.microsoft.com/windows/nanoserver:ltsc2022 as the base image to work around the issue.

Agree this would be very helpful. @Ccccclong would you mind sharing your Dockerfile please as I am having troubles building my own 2022 image? Thanks!

I am hitting this as well with nanoserver:1809 on Windows Server 2019. In my case Alloy runs, but I cannot mount the C:\var\log hostPath for log collection. Works with ServerCore, not with nanoserver.
I can attempt log collection through the k8s API but EKS can throttle those requests and that is somewhat uncomfortable.


[pyspark] Cleanup the comments

@WeichenXu123 @trivialfis Add a small PR to cleanup some comments and add some log printing
Thx

@trivialfis could you help to review it?


deepin.dde-launcher: init at 4.6.11

Motivation for this change
Add dde-launcher, the launcher of Deepin Desktop Environment.
About packaging deepin for NixOS: #59023
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Tested on a virtual machine.

@GrahamcOfBorg build deepin.dde-launcher

There's a few executables used with QProcess that we should hardcode.

Done.


open/R build fails while running build_openr.sh with default configuration

Issue Description
open/R build fails while running build_openr.sh with default configuration.  It fails in Allocator i.e. Prefix and Range Allocators
Environment
OS version:  ubuntu-16.04
Minimal test code / Steps to reproduce the issue
Below is the step followed for open/R build:
openr/build$ bash ./build_openr.sh | tee output.txt
What's the actual result?
openr/openr/allocators/PrefixAllocator.cpp:451:12:   required from here
/home/infonet/git-openr/openr/openr/allocators/RangeAllocator-inl.h:39:1: error: mangled name for ‘openr::RangeAllocator::RangeAllocator(const string&, const string&, openr::KvStoreClient*, std::function<void(folly::Optional) noexcept>, std::chrono::milliseconds, std::chrono::milliseconds, bool) [with T = unsigned int; std::__cxx11::string = std::__cxx11::basic_string; std::chrono::milliseconds = std::chrono::duration<long int, std::ratio<1, 1000> >]’ will change in C++17 because the exception specification is part of a function type [-Werror=noexcept-type]
RangeAllocator::RangeAllocator(
^~~~~~~~~~~~~~~~~
/openr/openr/allocators/RangeAllocator-inl.h:39:1: error: mangled name for ‘openr::RangeAllocator::RangeAllocator(const string&, const string&, openr::KvStoreClient*, std::function<void(folly::Optional) noexcept>, std::chrono::milliseconds, std::chrono::milliseconds, bool) [with T = unsigned int; std::__cxx11::string = std::__cxx11::basic_string; std::chrono::milliseconds = std::chrono::duration<long int, std::ratio<1, 1000> >]’ will change in C++17 because the exception specification is part of a function type [-Werror=noexcept-type]
/openr/openr/allocators/RangeAllocator-inl.h:39:1: error: mangled name for ‘openr::RangeAllocator::RangeAllocator(const string&, const string&, openr::KvStoreClient*, std::function<void(folly::Optional) noexcept>, std::chrono::milliseconds, std::chrono::milliseconds, bool) [with T = unsigned int]’ will change in C++17 because the exception specification is part of a function type [-Werror=noexcept-type]
In file included from /openr/openr/allocators/PrefixAllocator.h:29:0,
from /home/infonet/git-openr/openr/openr/allocators/PrefixAllocator.cpp:8:
/openr/openr/../openr/kvstore/KvStoreClient.h: In member function ‘void openr::PrefixAllocator::operator()(const openr::PrefixAllocatorModeStatic&)’:
/openr/openr/../openr/kvstore/KvStoreClient.h:194:34: error: mangled name for ‘folly::Optionalopenr::thrift::Value openr::KvStoreClient::subscribeKey(const string&, openr::KvStoreClient::KeyCallback, bool)’ will change in C++17 because the exception specification is part of a function type [-Werror=noexcept-type]
folly::Optionalthrift::Value subscribeKey(std::string const& key,
^~~~~~~~~~~~
cc1plus: all warnings being treated as errors
make[2]: *** [CMakeFiles/openrlib.dir/allocators/PrefixAllocator.cpp.o] Error 1
CMakeFiles/openrlib.dir/build.make:62: recipe for target 'CMakeFiles/openrlib.dir/allocators/PrefixAllocator.cpp.o' failed
CMakeFiles/Makefile2:1458: recipe for target 'CMakeFiles/openrlib.dir/all' failed
make[1]: *** [CMakeFiles/openrlib.dir/all] Error 2
make: *** [all] Error 2
Makefile:138: recipe for target 'all' failed

This happens because of your default compiler settings. Can you pass -Wnoerror=noexcept-type in CMakeLists.txt and you will be able to compile successfully.
Feel free to send pull request for the same if it succeeds for you.

Appreciate your pull request. Closing issue as fix has been https://github.com/facebook/openr/commit/5df6ab0d7624e028229f95004d4b00c178c7ceab


Getting error 'Missing required parameter: client_id'

After the script open the external link (browser) and I try to log into Spotify I get 'Missing required parameter: client_id' and thus cannot continue further.

Yes in the spotify.py file on line 11 & 12 you have to insert your spotify API ids
# Spotify API client_id = 'INSERT CLIENT ID HERE' client_secret = 'INSERT CLIENT SECRET HERE'

I've added a readme with some instructions

More instruction are needed. I was able to resolve this issue but came across another issue which I will open as a separate issue. I believe you should let people know that they have to:

Create/have a Spotify developer account to geta client id and client secret.
Copy the client id and secrete into the python script.
The redirect url on the Spotify developer dashboard should be the same as the one on the script? Which is http://localhost:5000/?
Save and run the script?

i have this issue too i get onto the thing where it says accept and whatever you named the app shows up saying it can change/control and then  you accept and it just says localhost refused to connect so what do i do from here?

From localhost refused to connect you should copy the url in the program


Auto locale

this is some code to detect lang attr in html tag and add related js to page. this feature is required for multilingual sites.

Thanks for the effort here, but this isn't an appropriate addition to this project. Fee free to contribute code examples like this to this repository's wiki, but I cannot accept a core change like this.


Virtual hosting

It doesn't look like there's any way to specify a domain name when running Gig, it'd be nice to be able to do that. And then to have multi-host support, so certain paths can apply only to one host, while other paths do the other. Similar to CompositeApplication in Jetforce.

Gig apps can be combined using ServerGemini function. So you can do something like this:
apps := map[string]*gig.Gig{}

// App A

a := gig.Default()
apps["app-a.example.com"] = a

a.Handle("/", func(c gig.Context) error {
    return c.Gemini("I am App A")
})

// App B

b := gig.Default()
apps["app-b.example.com"] = b

b.Handle("/", func(c gig.Context) error {
    return c.Gemini("I am App B")
})

// Server
g := gig.New()
g.Handle("/*", func(c gig.Context) error {
    app := apps[c.URL().Host]

    if app == nil {
        return gig.ErrNotFound
    } else {
        return app.ServeGemini(c)
    }
})

g.Run("my.crt", "my.key") // must be wildcard SSL certificate

I will add this example to README if it makes sense

This looks like a great example, thanks! And yeah, I think adding it to the README would be a good idea.


Nested property assignment evaluates incorrectly

(Originally issue filed in: https://github.com/svaarala/duktape-wiki/issues/59; see also http://stackoverflow.com/q/33489083/449161)
var NodeType;
(function (NodeType) {
    NodeType[NodeType["none"] = 0] = "none";
    NodeType[NodeType["text"] = 1] = "text";
})(NodeType || (NodeType = {}));

Duktape will resolve NodeType["text"] => 1 but not NodeType[1] => "text". The same code works fine in a browser such as Chrome.

This bug seems to exist in Duktape 1.3.0 but seems to work correctly in master.

Duktape 1.3.0 bytecode for the inner function:
; c0 "none"
; c1 {"type":"number","data":"0000000000000000","_value":0}
; c2 "text"
; c3 {"type":"number","data":"3ff0000000000000","_value":1}
00000 00000040   LDREG      r1, r0
00001 80c00056   PUTPROP    r1, c0, c1
00002 00000080   LDREG      r2, r0
00003 000040c0   LDREG      r3, r1
00004 8000c096   PUTPROP    r2, r3, c0
00005 00000040   LDREG      r1, r0
00006 81c08056   PUTPROP    r1, c2, c3
00007 00000080   LDREG      r2, r0
00008 0000c0c0   LDREG      r3, r3
00009 8100c096   PUTPROP    r2, r3, c2
00010 0000006f   RETURN     0x1, r0          ; have_retval

Bytecode in master:
; c0 "none"
; c1 {"type":"number","data":"0000000000000000","_value":0}
; c2 "text"
; c3 {"type":"number","data":"3ff0000000000000","_value":1}
00000 00000040   LDREG      r1, r0
00001 80c00056   PUTPROP    r1, c0, c1
00002 00000080   LDREG      r2, r0
00003 80404096   PUTPROP    r2, c1, c0
00004 00000040   LDREG      r1, r0
00005 81c08056   PUTPROP    r1, c2, c3
00006 00000080   LDREG      r2, r0
00007 8140c096   PUTPROP    r2, c3, c2
00008 0000002f   RETURN     0x0, r0

Just to ensure this is not working in master because the assignment value is mapped to a constant, also checked that there's no issue with dynamic results:
// Slightly modified
var obj2;
(function (obj2) {
    var t = '';
    var i = 0;
    obj2[obj2[t + "none"] = i + 0] = "none";
    obj2[obj2[t + "text"] = i + 1] = "text";
})(obj2 || (obj2 = {}));

[ 'none', 'text', 0, 1 ].forEach(function (k) {
   print(k, k in obj2, obj2[k]);
});

This works and the bytecode in master for that is:
; c0 ""
; c1 "none"
; c2 {"type":"number","data":"0000000000000000","_value":0}
; c3 "text"
; c4 {"type":"number","data":"3ff0000000000000","_value":1}
00000 00000042   LDCONST    r1, c0
00001 80000083   LDINT      r2, 0
00002 808040da   ADD        r3, r1, c1
00003 00000100   LDREG      r4, r0
00004 8100815a   ADD        r5, r2, c2
00005 0280c116   PUTPROP    r4, r3, r5
00006 00000180   LDREG      r6, r0
00007 80814196   PUTPROP    r6, r5, c1
00008 818040da   ADD        r3, r1, c3
00009 00000100   LDREG      r4, r0
00010 8200815a   ADD        r5, r2, c4
00011 0280c116   PUTPROP    r4, r3, r5
00012 00000180   LDREG      r6, r0
00013 81814196   PUTPROP    r6, r5, c3
00014 0000002f   RETURN     0x0, r0

This was probably fixed in #381. Pull request #428 provides a testcase for this bug, and I added this to Duktape 1.3.1 fix list.

That must be awesome, finding a bug and then realizing you already fixed it by accident a while ago. :)


Should we limit the node PATH env var ?

Description:
Should we limit the node PATH env var in node.mk ?
ifneq ($(CONFIG_OUI_USE_HOST_NODE),)
OUI_NODE_PATH := PATH=/usr/local/bin:/usr/bin:/bin
NODE := $(OUI_NODE_PATH) node
NPM := $(OUI_NODE_PATH) npm
$(eval $(call ColorInfo,Using Node.js from Host))
else

Since I'm using nvm for node install management, node is not installed in system's PATH.

Maybe I can add a menu config to set the path.

This should be set a default PATH env. If there is a Node exists in OpenWrt and left the PATH empty, then oui will use node from OpenWrt.

config OUI_HOST_NODE_PATH
	string "PATH for host node search"
	default "/usr/local/bin:/usr/bin:/bin:$$\(NVM_BIN\)"
	help
	  This will set PATH env for host node search.

Checking Node.js for build oui...
NODE: PATH=/usr/local/bin:/usr/bin:/bin:/home/zjh/.nvm/versions/node/v14.18.3/bin node
Using Node.js from Host
Node.js version: 14.18

This should be set a default PATH env. If there is a Node exists in OpenWrt and left the PATH empty, then oui will use node from OpenWrt.

How about patsubst openwrt's host bin dir from PATH? Since openwrt's host bin dir is fixed as $(TOPDIR)/staging_dir/host/bin,
so $(patsubst %/staging_dir/host/bin:,,$(PATH)) should do the trick.

This should be set a default PATH env. If there is a Node exists in OpenWrt and left the PATH empty, then oui will use node from OpenWrt.

You are right, I think openwrt's host bin dir can be handled by sed.
OUI_NODE_PATH := PATH=$(shell echo $(PATH) | sed -E 's|[^:]{1,}staging_dir/host/bin:||g')

This should be set a default PATH env. If there is a Node exists in OpenWrt and left the PATH empty, then oui will use node from OpenWrt.

You are right, I think openwrt's host bin dir can be handled by sed.
OUI_NODE_PATH := PATH=$(shell echo $(PATH) | sed -E 's|$(TOPDIR)/staging_dir/host/bin:||g')

Below is the PATH after:
PATH=/home/liuchao/sandbox/openwrt/openwrt-22.03.3/staging_dir/toolchain-mipsel_24kc_gcc-11.2.0_musl/bin:/home/liuchao/sandbox/openwrt/openwrt-22.03.3/staging_dir/toolchain-mipsel_24kc_gcc-11.2.0_musl/bin:/home/liuchao/local/bin:...

openwrt's host bin dir is gone.

If thus, there is no need to add CONFIG_OUI_HOST_NODE_PATH.

Since #214 is merged, I think now host node search will satisfy all needs, this issue can be closed.

@zhaojh329 Thanks for your refactor about this issue!
But in 06656f14225689fed42f70bd719313b1a49bf0fb,
ifneq ($(CONFIG_OUI_USE_HOST_NODE),)
NODE_PATH := PATH=$(PATH)
else

oui will use node from OpenWrt.

@zhaojh329 Thanks for your refactor about this issue! But in 06656f1,
ifneq ($(CONFIG_OUI_USE_HOST_NODE),)
NODE_PATH := PATH=$(PATH)
else

oui will use node from OpenWrt.

Oh, I just find out STAGING_DIR_HOSTPKG:=$(TOPDIR)/staging_dir/hostpkg, and openwrt's host node is in hostpkg/bin, so this is not a problem.

@zhaojh329 Thanks for your refactor about this issue! But in 06656f1,
ifneq ($(CONFIG_OUI_USE_HOST_NODE),)
NODE_PATH := PATH=$(PATH)
else

oui will use node from OpenWrt.

Oh, I just find out STAGING_DIR_HOSTPKG:=$(TOPDIR)/staging_dir/hostpkg, and openwrt's host node is in hostpkg/bin, so this is not a problem.

STAGING_DIR_HOSTPKG not included in PATH on this stage, but it's inclued in PATH on Prepare and Compile.

Thanks for your reply!
I think this issue is resolved.


Reset Select component correctly in Firefox/Edge

Fixes https://github.com/yewstack/yew/issues/630
On Firefox/Edge, it's needed to set value using DOM API to reflect a state of dropdown. To hold a reference of select element, I introduced a new field select_ref.
I didn't add dedicated example because it's very small. I thought that increasing redundant examples is not good for the project for maintenance cost. Instead of that, I placed "Reset" button in showcase example. This is not MUST feature, but good demonstration.

@jstarry Thanks for helping me on Gitter. Ready for review.


Auto-enable for backspace and delete keystrokes

Would be very rewarding...

is this what you were talking about @marxian https://github.com/JoelBesada/activate-power-mode/issues/183 ?

@SalomonSmeke It totally is, though it is much better articulated in #183


Patch for ff latest crashing see issue #1521

Workaround for https://github.com/JedWatson/react-select/pull/1556 until it's published


Colours cafe add

Colours cafe added for color picker section

@LisaDziuba when you get a change merge it, Thanks!

hey @oliver-gomes, I will review the tool now.

@oliver-gomes some follow up question:

why have you decided to change * to - in the Markdown?
I'm not sure about https://www.instagram.com/colours.cafe/ as it's an Intagram account not an actual tool. Let's ask community.

I will mention ppl who submitted another colour picker tools and ask their opinion. That should be a fair way.
Thanks again for contribution!

I didn't notice it, think my code editor changed it all at once
I found it really helpful and tool coming soon on their site

Thanks for the review, i will make sure in future the markdown doesn't change

hey! I talked to @valianka and other folks internally, we decided yo add Colour Cafe. I will close your PR and will add it by myself because I will need to fix too much small conflicts 😅
Thanks again for contribution and understanding!

@LisaDziuba Sure! My bad on that, will make sure my prettier plugin is turned off when making future changes 😃


Issue with new node modules. Docker build doesn't work

Build command doesn't work
Some kind of problem with installing node.js modules. When I add new node.js modules , application doesn't see them.

Need accept pull request


FR: Support AWS Secrets Manager JSON Secret Strings

The AWS Secrets Manager support requires a secret for each part of the SNMP credential configuration. This makes the management of the credentials in Secrets Manager a bit cumbersome. It would be ideal to either reference the keys within a JSON SecretString value to fill in the credentials, or set the JSON SecretString as the YAML configuration for default_v3 within the configuration file.
Example JSON SecretString
MY_SM_SECRET_JSON =
{
	"user_name": "username",
	"authentication_protocol": "MD5"
	"authentication_passphrase": "authpass",
	"privacy_protocol": "AES",
	"privacy_passphrase": "privpass"
}

Key Reference Example
default_v3:
	user_name: aws.sm.$MY_SM_SECRET_JSON["user_name"]
	authentication_passphrase: aws.sm.$MY_SM_SECRET_JSON["authentication_passphrase"]

SecretSting JSON as default_v3 value Example
default_v3: aws.sm.$MY_SM_SECRET_JSON

This form should work now:
default_v3: aws.sm.$MY_SM_SECRET_YAML

I didn't want to support JSON as a config because the snmp config file is yaml and it seemed nasty to embed json in yaml.
I tested with
MY_SM_SECRET_YAML =
"user_name": "username"
"authentication_protocol": "MD5"
"authentication_passphrase": "authpass"
"privacy_protocol": "AES"
"privacy_passphrase": "privpass"

Let me know if this works for you

@i3149 I can confirm this is working as it should, thanks for the quick turn around!

@i3149  I may have spoke too soon, when I initially set the aws.sm under global and run discovery, it replaces that line (and all device snmp_v3 fields) with all of the values from the secret. Ideally the reference to aws.sm should stay in place instead of replacing the fields to ensure no secret information is stored on disk.

Oh right! I didn't think this through for the discovery step. I will fix.

Try one more time. default_v3: aws.sm.$MY_SM_SECRET_YAML should not get overwritten on discovery now.

@i3149  I can confirm that nothing is getting overwritten and secrets are not being stored in the configuration files now. This is awesome and one of my last hurdles of getting this into production. Thank you!

Sweet! Closing this one. I wasn't sure what happened to #230 but that should be resolved now too.


kops_1_16: 1.16.0 -> 1.16.1

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@ofBorg built kops_1_16

@ofborg build kops_1_16

Closed in favor of #86112


Added resource limits & requests for mysql deployment (FINERACT-1301)

Description
Added Resource Limits & Requests for MYSQL server deployment. See FINERACT-1301
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[x] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[ ] Create/update unit or integration tests for verifying the changes made.

[x] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[ ] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/api-docs/apiLive.htm with details of any API changes

[x] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@vorburger @xurror Please review :)

@vorburger @xurror Please review :)

LGTM. Great work @BLasan

Thanks :)

LGTM. Great work @BLasan

Thanks :)


Kazi: Changed Navbar to icons over text

Fixes ENG-148

Changed it from text to icons where if you hover over it will show text.

totally random, but just noticed the socials look like it's saying "instagram in github"


Auto resend unfulfilled commands for cluster

Attempting to add autoResendUnfulfilledCommands in cluster mode.  Please test :)

autoResendUnfulfilledCommands option is passed to the Redis instances of the cluster(https://github.com/luin/ioredis/blob/master/lib/cluster.js#L167), so it should work. I tried the following code:
var cluster = new Redis.Cluster([
  { port: 6380 },
  { port: 6381 },
  { port: 6382 }
]);

function loop(err, res) {
  (err || res) && console.log(err, res);
  cluster.blpop('qqq', 0, loop);
}

loop();

When I lpush a item to qqq, it prints it, and when I stop the cluster and restart it, lpush a item to qqq again, it still prints it correctly.

d'oh.  I thought it was autoResend that wasn't working in cluster mode - turns out it's auto-resubscribe.


[Text Analytics] Remove includes of esnext

In before, esnext was needed to be imported because it has definitions for async iterators that @types/node@8 did not have. Now, we support NodeJS v12 and up which includes ES2019 that includes ES2018 that includes the async iterators stuff. Thanks @witemple-msft for pointing that out!
The PR also does the following:

deletes src/dom.d.ts. I am not sure why it exists in the first place but it is not referenced anywhere.
removes use of @azure/core-asynciterator-polyfill. It was needed for older versions of NodeJS that we no longer support.

@witemple-msft yes! see https://github.com/Azure/azure-sdk-for-js/issues/15882


Use PSScriptAnalyzer scan to create issues for problem code

[x] #9
[ ] #10
[ ] #11
[ ] #12

[x] #13

Formatting rules are handled by default. Closing #9

May revisit #13 later but Microsoft's PSScriptAnalyzer action already populates the Security tab in the repo with any problems


CSS Framework only javascript based (MVP)

Prerequisites :

Tiny size 👍
Usable in all plateform 👍
Usable with any framework 👍
Can generate almost CSS properties 👍
Supports responsive 👍
Own grid system based on flexbox fully customisable 👍
Can support pseudo-element 🔢


Form: Solve server-side error catching

For example on registration page when an error occures it's not handled.
Base idea is to use exception message as an "interface" of the error.

Done.


Missing page title in the title bar of the Dockerfile reference

File: engine/reference/builder.md
Refer the browser's title bar when browsing https://docs.docker.com/engine/reference/builder:

The page title should  be visible (probably "Dockerfile reference")

Seems like a simple fix, but I can't even find the file to fix it myself. Could someone point in the right direction, please?


Pointer lock option

From here:

Hey man, I have a request for htmlifier. It would be great if you could have checkbox settings to enable mouse lock (allowing mouse x and y values to go infinitely above their limits while keeping the actual mouse position in the center of the screen). Thanks and keep up the good work

I think this'll be a very simple option that locks the pointer on click; there'll be no way to control when to lock or unlock the pointer (they can always implement their own cursor, I guess)

It's possible that mousemove fires events faster than a forever loop, but meh


Running iotedge without tpm module

Describe the bug
After adding a config.toml with proper hostname and connection string
[provisioning] source = "manual" connection_string = "..."
and applying the config, the command iotedge system logs tells me WARNING:tcti:../tpm2-tss-3.2.2/src/tss2-tcti/tcti-device.c:440:Tss2_Tcti_Device_Init() Failed to open default TCTI device file /dev/tpmrm0: No such file or directory. Thing is that I run this on a Raspberry Pi 4 without TPM module but I cannot find a way to disable tpm.
To Reproduce
Build yocto with meta-iotedge for raspberry pi4, configure config.toml with hostname and connection string, apply config and read iotedge system logs iotedge system logs
Expected behavior
Configuration parameter to disable tpm usage?
Logs
 aziot-tpmd[4224]: WARNING:tcti:../tpm2-tss-3.2.2/src/tss2-tcti/tcti-device.c:440:Tss2_Tcti_Device_Init() Failed to open default TCTI device file /dev/tpmrm0: No such file or directory  aziot-tpmd[4224]: WARNING:tcti:../tpm2-tss-3.2.2/src/tss2-tcti/tcti-device.c:440:Tss2_Tcti_Device_Init() Failed to open default TCTI device file /dev/tpm0: No such file or directory  aziot-tpmd[4224]: ERROR:tcti:../tpm2-tss-3.2.2/src/tss2-tcti/tcti-device.c:445:Tss2_Tcti_Device_Init() Could not open any default TCTI device file  aziot-tpmd[4224]: ERROR:tcti:../tpm2-tss-3.2.2/src/tss2-tcti/tctildr-dl.c:154:tcti_from_file() Could not initialize TCTI file: device  aziot-tpmd[4224]: ERROR:tcti:../tpm2-tss-3.2.2/src/tss2-tcti/tctildr.c:428:Tss2_TctiLdr_Initialize_Ex() Failed to instantiate TCTI  aziot-tpmd[4224]: 2024-01-26T15:43:57Z [ERR!] - service encountered an error  aziot-tpmd[4224]: 2024-01-26T15:43:57Z [ERR!] - caused by: internal error  aziot-tpmd[4224]: 2024-01-26T15:43:57Z [ERR!] - caused by: could not initialize TPM  aziot-tpmd[4224]: 2024-01-26T15:43:57Z [ERR!] - caused by: tcti:IO failure  aziot-tpmd[4224]: 2024-01-26T15:43:57Z [ERR!] -    0: <unknown>  aziot-tpmd[4224]:    1: <unknown>  aziot-tpmd[4224]:    2: <unknown>  aziot-tpmd[4224]:    3: <unknown>  aziot-tpmd[4224]:    4: <unknown>  aziot-tpmd[4224]:    5: <unknown>  aziot-tpmd[4224]:    6: <unknown>  aziot-tpmd[4224]:    7: <unknown>  aziot-tpmd[4224]:    8: <unknown>  aziot-tpmd[4224]:    9: <unknown>  aziot-tpmd[4224]:   10: <unknown>  aziot-tpmd[4224]:   11: __libc_start_call_main  aziot-tpmd[4224]:              at usr/src/debug/glibc/2.35-r0/git/csu/../sysdeps/nptl/libc_start_call_main.h:58:16  aziot-tpmd[4224]:   12: __libc_start_main_impl  aziot-tpmd[4224]:              at usr/src/debug/glibc/2.35-r0/git/csu/libc-start.c:389:3  systemd[1]: aziot-tpmd.service: Main process exited, code=exited, status=1/FAILURE  systemd[1]: aziot-tpmd.service: Failed with result 'exit-code'.
Build System (please complete the following information):

OS: Ubuntu
Version 22.04

Target System (please complete the following information):

OS: yocto kirkstone
Version 4.0.14 (kirkstone)

Additional context
Add any other context about the problem here.

I deactivate the aziot-tpm in my override:
SYSTEMD_SERVICE:${PN}:remove:my-tpm-less-machine = "aziot-tpmd.service"


Disable "Follow Terrain"

I want to be able to disable the "follow terrain" feature of the default cam.
By default the cam will try to keep the distance between the camera and terrain constant if you move it accross a hill with wasd for example. I prefer a mode that just tries to keep the height constant while keeping the camera above ground. So as long as cam.y < groundClearance, don't modify cam.y.
Since I made it into my own little private cam mod, here is the code for the transpiler of CameraController.UpdateTargetPosition.
Too lazy to create a full PR with an option and all, sry ;P
Maybe you'd have to adjust it somewhat to your needs.
private static IEnumerable<CodeInstruction> UpdateTargetPositionTranspiler(IEnumerable<CodeInstruction> codeInstructions)
{
    var sampleRawHeightSmoothWithWaterMethodInfo = typeof(TerrainManager)
        .GetMethod("SampleRawHeightSmoothWithWater", new[] { typeof(Vector3), typeof(bool), typeof(float) });

    var codes = codeInstructions.ToList();

    for (var i = 0; i < codes.Count; i++)
    {
        var code = codes[i];
        if (code.opcode == OpCodes.Callvirt && code.operand == sampleRawHeightSmoothWithWaterMethodInfo)
        {
            codes.Insert(i - 5, new CodeInstruction(OpCodes.Ldc_R4, 0f));
            codes.RemoveRange(i - 4, 6);
            break;
        }
    }

    return codes;
}

All it does is it replaces:
m_targetHeight = Singleton<TerrainManager>.instance.SampleRawHeightSmoothWithWater(m_targetPosition, timeLerp: true, 2f);
with
m_targetHeight = 0;

Actually the behaviour with the transpiler is not as I'd like it to be.
The m_targetHeight should actually follow terrain.
Its just that camera.y should not change if target.y changes, if you know what I mean.

Thanks for the code - I'll take a look.  I'm not sure how things will work out if m_targetHeight follows terrain but the camera doesn't change, could lead to some disorientation - but we'll see!

I actually find the current behaviour distoring, because the camera bobs around like crazy if you move it here and there.
If you disable that, it kinda feels smooth and calming in comparison :)


typesafe-joi blows up TypeScript 3.2+

TypeScript Version:  3.3.0-dev.20181205

Search Terms:
Maximum call stack size exceeded
Code Repo:
https://github.com/hjkcai/typesafe-joi
Reproduction:
mkdir typesafe-joi-issue && cd typesafe-joi-issue
yarn add typescript@next @types/node typesafe-joi joi
echo '{"compilerOptions":{"strict":true}}' > tsconfig.json
echo 'import * as Joi from "typesafe-joi"; console.log(Joi)' > index.ts
npx tsc -p .

Expected behavior:
tsc should work.
I admit that typesafe-joi uses lots of type recursion, but it works fine in TypeScript 3.1. After upgrading to TypeScript 3.2, the error comes.
Actual behavior:
tsc itself encountered 'Maximum call stack size exceeded' (No stack trace printed)

Without npx using command node_modules\.bin\tsc -p . I see the callstack:
RangeError: Maximum call stack size exceeded
    at resolveNameHelper (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:26036:35)
    at resolveName (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:26034:20)
    at resolveEntityName (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:26728:26)
    at resolveTypeReferenceName (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:31885:20)
    at getTypeFromTypeReference (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:32061:30)
    at getTypeFromTypeNode (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:33421:28)
    at getConstraintFromTypeParameter (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:31747:72)
    at fillMissingTypeArguments (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:31337:49)
    at getTypeFromClassOrInterfaceReference (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:31837:78)
    at getTypeReferenceTypeWorker (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:31950:24)

Running node setting a stack-trace-limit to 100000 (to see the full stack trace) with node --stack-trace-limit=100000 node_modules\typescript\bin\tsc -p .: full stack trace (is to large to paste here).
Seems that the recursion starts at call to getTypeFromTypeReference from checkTypeReferenceNode and the repetition is this callstack fragment:
    at getTypeFromTypeNode (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:33422:28)
    at getConstraintFromTypeParameter (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:31748:72)
    at fillMissingTypeArguments (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:31338:49)
    at getTypeFromClassOrInterfaceReference (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:31838:78)
    at getTypeReferenceTypeWorker (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:31951:24)
    at getTypeReferenceType (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:31893:24)
    at getTypeFromTypeReference (Z:\tmp\ts\node_modules\typescript\lib\tsc.js:32063:28)

The issue here is that AbstractSchema specifies itself as a constraint and relies on type argument defaults in that specification:
interface AbstractSchema<Schema extends AbstractSchema = any, Value = any> extends JoiObject { ... }

The use of default type arguments in combination with the recursive constraint causes an infinite loop. An easy fix for now is to specify the type arguments:
interface AbstractSchema<Schema extends AbstractSchema<any, any> = any, Value = any> extends JoiObject { ... }

@weswigham Best I can tell the infinite loop is caused by fillMissingTypeArguments attempting to get the constraint of the type parameter, which leads back to the same type, which leads back to needing to add default type arguments, and down the rabbit hole we go. I think you were last person to work on this so I'm assigning it over to you.

I made local fix, you can try to apply and see if it helps
https://github.com/EvgenyMuryshkin/TypeScript/commit/9f88c2f2ca0fd42dc9e16baf7eae240ffc380def

@EvgenyMuryshkin Doh! You're right, it definitely should be calling getConstraintOfTypeParameter (which checks for circularities) and not getConstraintFromTypeParameter (which is a worker function that doesn't). Thanks for catching this, I will turn it into a PR.

@weswigham @rbuckton @DanielRosenwasser I'm thinking that all of the proposed solutions to this issue paper over the the real culprit: Type parameter defaults should only be allowed to reference previously declared type parameters (i.e. type parameters declared to the left). I alluded to this in my comment here. Such a rule would be analogous to JavaScript's restriction that parameter defaults can only reference parameters to the left, and it would make nonsensical constructs such as the following become errors:
type Foo<T = T> = T[];
type Bar<T = U, U = T> = [T, U];

After all, when a type parameter default comes into play, we know that type arguments for that type parameter and type parameters to the right of it weren't specified, so there is nothing meaningful to reference. The OP issue is caused by us trying to come up with a meaningful default for a forward reference (and our answer effectively amounts to the constraint of the forwardly referenced type parameter), but I'm thinking it should just be an error (and we should just use the error type).
I suppose you could make a weak argument that type parameter defaults in generic functions should be able to reference later type parameters because type inference might produce useful inferred types, but I have yet to see a real world example where that is useful.
I have a branch that implements this restriction and I'm the process of running validating all of Definitely Typed on it. So far nothing seems to be affected.

We are using this project and I had to tweak type definition a bit after 3.2 migration
Seems that there is a circular reference in types
https://github.com/projectstorm/react-diagrams/blob/master/src/BaseEntity.ts

Hey guys, typesafe-joi make typescript crash again in #30794. Please help!


Create Unique Number in JAVA Program

Could you please Approve it at the earliest ?

do star this repo

Could you please share such more repositories where I can make PR ?


I cloned the git, and getting pwsh no logo error

Process "pwsh -NoLogo" in domain "local" didn't exit cleanly
Exited with code 1.
This message is shown because exit_behavior="CloseOnCleanExit"

thank you it was pwsh was not installed I had windows powershell this is awesome thanks!


Avoid duplication of match_words in some cases.

In some cases the filetype is set multiple times and the code that sets b:match_words executes for each instance. This commit add a tsx_match_words existence precondition check, and fix the '%' command issue discussed here https://github.com/neovim/neovim/issues/9187 and https://github.com/chrisbra/matchit/issues/11.
For more information, please checkout the original PR to vim-jsx by @ivangeorgiew . https://github.com/mxw/vim-jsx/pull/177

Thanks! I will give it a look!

Thanks!


chore: remove gemini

We're getting constant reminders about security issues from Gemini (which is no longer maintained), so this PR removes it from the repository.
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] If applicable, have a visual design approval

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] clarity.design website / infrastructure changes
[x] Other... Please describe: Removal of unused code

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

@gnomeontherun : Do we want to merge this?


clarify ssh option

I wasn't sure what the ssh option did, hopefully this clarifies it a little.

lgtm, thanks :slightly_smiling_face:


Replace unmaintained create-release action

Replaces actions/create-release, which is unmaintained, with ncipollo/release-action.
Closes #18453.

Tested this with the one shot release workflow locally in my fork.


Missing white space in HTTP Envoy log format since Istio 1.0.3

Describe the bug
There is a missing whitespace in the hard-coded  log format of Envoy between the request (%REQ(X-ENVOY-ORIGINAL-PATH?:PATH)%) and the protocol (%PROTOCOL%):
	EnvoyHTTPLogFormat = "[%START_TIME%] \"%REQ(:METHOD)% %REQ(X-ENVOY-ORIGINAL-PATH?:PATH)%" +
		"%PROTOCOL%\" %RESPONSE_CODE% %RESPONSE_FLAGS% %BYTES_RECEIVED% %BYTES_SENT% " +

As a result, we'll see logs like this:
[2019-03-13T14:57:14.768Z] "GET /.well-known/apple-app-site-associationHTTP/1.1" 200 - 0 175 48 46 "<IP_ADDRESS>,<IP_ADDRESS>" "swcd (unknown version) CFNetwork/976 Darwin/18.2.0" "9481374-12e5-9053-9146-80bc14a0e45f" "xxx.com" "<IP_ADDRESS>:80" inbound|80||service.default.svc.cluster.local - <IP_ADDRESS>:80 <IP_ADDRESS>:0

Expected behavior
To have a whitespace between these two fields. Instead of
 "GET /.well-known/apple-app-site-associationHTTP/1.1"

should be
 "GET /.well-known/apple-app-site-association HTTP/1.1"

Steps to reproduce the bug
n/a
Version
This has been in Istio since 1.0.3 up till the latest version.
Installation
n/a
Environment
AWS
Cluster state
n/a

I just found that this has been fixed in 1.1.0-snapshot.5.


cloud.gov training for 18F-wide onboarding

Let's discuss what we want this to be like.
@berndverst and I believe that having a recorded training might work well.

Also worth noting that this is not a business-related communication (Skyporter focuses on materials that support our business, as a reminder).
We can leave it in Skyporter, but it's my position that this stay iceboxed for this quarter. As we improve the website so that it successfully communicates what we do (to technical and non-technical audiences), new 18Fers can sufficiently learn about cloud.gov there.

basic training for new 18F folks --> hopefully our website will be able to provide an overview of the product soon enough. Maybe we can rely on those to educate the 18F community more broadly?


booster handle is invalid

For bugs or installation issues, please provide the following information.
The more information you provide, the more easily we will be able to offer
help and advice.
Environment info
Operating System: linux cent os 6+
Compiler:
Package used (python/R/jvm/C++): R
xgboost version used:
If installing from source, please provide

The commit hash (git rev-parse HEAD)
Logs will be helpful (If logs are large, please upload as attachment).

If you are using python package, please provide

The python version and distribution
The command to install xgboost if you are not installing from source

If you are using R package, please provide

The R sessionInfo()

R version 3.3.1 (2016-06-21)
Platform: x86_64-redhat-linux-gnu (64-bit)
Running under: CentOS release 6.8 (Final)
locale:
[1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C               LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8
[5] LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8    LC_PAPER=en_US.UTF-8       LC_NAME=C
[9] LC_ADDRESS=C               LC_TELEPHONE=C             LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
loaded via a namespace (and not attached):
[1] tools_3.3.1
2. The command to install xgboost if you are not installing from source
Steps to reproduce

I saved a xgboost model in a RDS file, and read it to the R environment to use
When I read RDS file, xgboost can correctly get the model, but after I library(xgboost), the xgboost model got a message that booster handle is invalid. Then the predicted probability that I used by this model is different with the correct model produced.
I don't know what happened.

What have you tried?

It is normal that a model saved as R object would have invalid handle after reading it back. R has a primitive mechanism to add hooks that may initialize unserialized objects, but it would be rather awkward for users to deal with that stuff (e.g., see the refhook parameter in ?readRDS). Thus, we restore the handle only during the operations which require it. That should also have no effect on predictions. To look into the different predictions issue that you are seeing, please provide some reproducible example.

I've found that this problem is caused by different xgboost version. I train the model with R 3.2 and xgboost 4.x . I can correctly use my model in 4.x version xgboost, but if I use R 3.3 and library a 6.x version xgboost in the environment, this model will be invalid and it will cause the problem I've mentioned.

I tried to attach my model and test dataset into a zip file,but it doesn't work. How can I get the model to you.

You've mentioned that you had used an older version for the initial model. See if #1938 would help.


android:debuggable="true" In manifest leads to app Rejection

android:debuggable="true" should be removed from the manifest.
As this is a library to be used within other projects android:debuggable="true" should not be set in the manifest. When a release apk is assembled a project using this library the android:debuggable="true" will be merged into said projects manifest resulting in a debuggable release apk which will promptly be rejected by Google.
I can see this was added due to this ticket https://github.com/smartdevicelink/sdl_android/issues/40. If you want to disable logs you can do so using custom BuildConfig flags defined within the library gradle file and referencing them in the code. i.e buildConfigField "boolean", "LOG", "false" and then if (!BuildConfig.LOG) // disable logging

Fixed with hotfix release https://github.com/smartdevicelink/sdl_android/releases/tag/4.3.1