Script not persisting after cold boot

The script seems to work just fine upon activation, stopping and starting appropriately. However it doesn't auto-start after a cold boot.
If I select one of the play/stop/skip scripts without re-running the main bgm script, it outputs something about there not being a "/tmp/bgm.sock" file or directory.
It works again if I run the main script, but I have to do this manually every time I cold boot the system.

I can confirm this happens for me as well.

Hi. Sounds like the mister startup script didn't get setup right. Can you please find the file from your SD card in the linux folder called user-startup.sh and post the contents here (/media/fat/linux/user-startup.sh if you ssh in)

Mine is actually there but with an underscore at the front of the filename (so presumably inactive), and that's it.
The only thing in it is
`#!/bin/sh
echo "" $1 ""
`

#!/bin/sh
echo "" $1 ""

Thank you. You both have an underscore in front?
I think I can fix this then. I assumed that appending to a file in Python would also create the file if it doesn't exist but apparently not
Try to rename that file without an underscore in front and see if that fixes it
I can put in an update to fix automatically but not til tomorrow probably

Oh and run bgm from the scripts menu after you rename it

This did fix it, although it runs pretty late, like 18 seconds after the main menu appears-- I'm not sure if that's just how it works or not.

It does run quite late unfortunately although 18 seconds sounds a little longer than usual. I can't change it too much though I'm at the mercy of when mister launches that startup script
For reference mine usually kicks in just after the Bluetooth and wifi icons appear in the menu

Yep, my filename is "_user-startup.sh" also.
I've renamed it, re-run the script and it seems to work fine for me. There's a moment where the tv-input handshakes with the Mister at the start of cold-boot, but once I see video, I also hear my _boot.wav track has begun playing.
Thanks :)

For reference, I've just got two and it seems pretty instant ("Solar Sailer" as boot and the "End Titles" theme, both from Tron Legacy :) )

I wonder if it's because I have a ton of mp3s in that folder, maybe if I cull them it'd be faster.

It's certainly possible but I'd also expect you to have a significant delay in between tracks playing too! The script doesn't generate a playlist to run off, it just scans the whole directory every time and picks a random file. I might need to do some experimenting with different amounts of tracks in there
If that is the cause, I can probably put together something clever enough to avoid/hide this delay

There's definitely no delay when skipping tracks. And as you say, my tracks start playing when the wifi and Bluetooth icons appear, but for me that's about 18 seconds on average. I significantly reduced the number of tracks and it didn't make any noticeable difference. It might just be the way my device or card is configured. It's really not an enormous issue either way, now that it's at least working.

Anyway since we figured out the user-startup.sh issue and any other delays don't seem related to the script itself, I'll close this. Thanks for the script!

Fix is in for this now. Thanks for your help


PsfLauncher with more than one command line argument.

Found issue with PsfLauncher on Anaconda (it is a huge app but the issue should be reproducable with any app needing multiple command line arguments in the config.json file). The process never tries to start.
It looks like the code in StartProcessHelper.h for the StartProcess case assumes no more than one argument present (Line 50 when initializing the arguments).  This should be looked at, and while there the shell launch case should be reviewed as well.

Update: The Shell launch case is similarly broken.  Example app is MathMechanxs which uses a shell launch for the help file.

Hey Tim,
PSFLauncher grabs the whole argument string from the configuration and passes it into StartProcessHelper.  All arguments should be passed onto CreateProcess.as the argument "Command line"
Could you send me the MSIX you have tested with to my email and I will take a look at it on Monday?
Also, Line 50 in StartProcessHelper.h points to the InitializeProcThreadAttributeList call.  I looked in both master and develop.  Maybe I looked in the wrong place.

Can you get access to the teams area Vlad has me use? File size issue with email.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Darren Hoehna<EMAIL_ADDRESS>Sent: Saturday, December 14, 2019 1:41:36 PM
To: microsoft/MSIX-PackageSupportFramework<EMAIL_ADDRESS>Cc: TIMOTHY MANGAN<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/MSIX-PackageSupportFramework] PsfLauncher with more than one command line argument. (#108)
Hey Tim,
PSFLauncher grabs the whole argument string from the configuration and passes it into StartProcessHelper. All arguments should be passed onto CreateProcess.as the argument "Command line"
Could you send me the MSIX you have tested with to my email and I will take a look at it on Monday?
Also, Line 50 in StartProcessHelper.h points to the InitializeProcThreadAttributeList call. I looked in both master and develop. Maybe I looked in the wrong place.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/MSIX-PackageSupportFramework/issues/108?email_source=notifications&email_token=AEVBJFOALJ3R6FXCW6NSSXLQYUSGBA5CNFSM4J23NS5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEG4IROY#issuecomment-565741755, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEVBJFNWK5RZ3FOPXVC4B4DQYUSGBANCNFSM4J23NS5A.

Uploaded the two packages to Vlad's shared Teams File area.

Anaconda: Use shortcut "Anaconda Navigator"  (uses CreateProcess)
MathMechanixs: Use shortcut  for MathMechanxs Help file (uses Shell Launch)

Something in the back of my mind says I may have fixed this when I added the shell launch into the code last winter.  Perhaps that was lost during refactoring?

Addressed in PR#132

Addressed in replacement PR#137.


Added ZSH_CACHE_DIR variable for oh-my-zsh

As discussed in https://github.com/zplug/zplug/issues/190 this adds the ZSH_CACHE_DIR variable which is used by a number of oh-my-zsh plugins (like the value of cache-path in completion.zsh)

Please use spaces instead of tabs. Other than that LGTM :+1:


Narrowing inconsistencies

TypeScript Version:
nightly (1.9.0-dev.20160619-1.0)
Code
Three functions that narrow three types to see how narrowing behaves. foo1 and foo2 are identical apart from which type guards they use.

// A discriminated union with three cases
interface A { type: 'A' }
interface B { type: 'B' }
interface C { type: 'C' }

// Custom type guards for A/B/C
function isA(x: A|B|C): x is A { return x.type === 'A'; }
function isB(x: A|B|C): x is B { return x.type === 'B'; }
function isC(x: A|B|C): x is C { return x.type === 'C'; }

// Using if/else with type guard functions
function foo1(x: A|B|C): any {
    x // x is A | B | C
    if (isA(x)) {
        return x; // x is A
    }
    x // x is B | C
    if (isB(x)) {
        return x; // x is B
    }
    x // x is C
    if (isC(x)) {
        return x; // x is C
    }
    x // x is C, shouldn't x be 'never' here?           <===== (1)

    // If x was 'never', #8548 should apply here:
    if (isA(x)) {
        return x; // x is C & A                         <===== (2)
    }
    else {
        x // x is C                                     <===== (3)
    }
}

// Using if/else with discriminated unions
function foo2(x: A|B|C): any {
    x // x is A | B | C
    if (x.type === 'A') {
        return x; // x is A
    }
    x // x is B | C
    if (x.type === 'B') {
        return x; // x is B
    }
    x // x is C
    if (x.type === 'C') {
        return x; // x is C
    }
    x // x is A | B, shouldn't x be 'never' here?       <===== (4)

    // If x was 'never', #8548 should apply here:
    if (x.type === 'A') {
        return x; // x is A                             <===== (5)
    }
    else {
        x // x is B                                     <===== (6)
    }
}

// Using switch/case with discriminated unions
function foo3(x: A|B|C): any {
    x // x is A | B | C
    switch (x.type) {
        case 'A': return x; // x is A
        case 'B': return x; // x is B
        case 'C': return x; // x is C
        default:
            x // x is never                                 <===== (7)
            if (isA(x)) {
                x // x is never, why not A due to #8548?    <===== (8)
            }
    }
}

Expected behavior:

The inferred types at (1)/(2)/(3) should be the same as the inferred types at (4)/(5)/(6)
The inferred type at (1) and at (4) should be never, like it is at (7)
Type of x should be A at (8) due to #8548

Actual behavior:
See the comments in the code.

I wrote this code trying to understand what was going on in #9246 and #9254, and now I think I'm more confused. Firstly, foo1 and foo2 have identical logic, but tsc infers types differently in each. Secondly, I couldn't work out how to invoke the 'type guards as assertions' (#8548) behaviour. I though it should kick in after (1) and (4), where the types have been exhausted and then another type guard appears. But control flow analysis never infers never in either function. We do get never inferred at (7), but then #8548 doesn't seem to apply at (8) where I expected it should.

#8548 works a bit differently from what you assume. It basically says that in either the when true or when false branch of a type guard for some x, the resulting type becomes never only when the type guard produces never for both the current control flow type of x and the declared type of x. In other words, you only get never if it's the result whether we trust the control flow or view the guard as an assertion. However, once we've produced a never, nothing but a manifest type assertion will make it go away.
Let me start with (4). Prior to the x.type === 'C' guard, the control flow type of x is C. Therefore, in the when false branch of that type guard (i.e. at the (4) marker), the control flow type would become never. That causes #8548 to kick in (because we view the type guard as an assertion) and repeat the type guard with the declared type instead. So, the control flow type of x becomes A | B in the when false branch.
I think it is fair to assume you should see the same type at (1). The reason you don't is that (for historical reasons) we don't narrow in the when false branch of a typeof, instanceof, or user defined type guard unless the current type is a union type. We probably should, which would produce a never, which would then cause #8548 to kick in and produce A | B.
Regarding the switch statement, here we view the entire construct as a single type guard, so whether we apply it to the current control flow type or the declared type, we still end up with type never in the default clause. So we let that stand.

Thanks @ahejlsberg, that helps me understand #8548 better. So it is applying in the above example, just not where or how I expected.
I can see the problem #8548 solves by following its link to #8513. However, by applying to the example above, it seems to introduce problems of its own. Here is a simpler example showing what I mean:
type Coin = {type: 'heads', h} | {type: 'tails', t};

function f1(c: Coin) {
    if (c.type === 'heads') {
        return c.h;
    }
    else /* must be 'tails' */ {
        return c.t;
    }
    c; // ERROR: unreachable code detected
}

function f2(c: Coin) {
    if (c.type === 'heads') {
        return c.h;
    }
    else if (c.type === 'tails') {
        return c.t;
    }
    c; // c is 'heads'
}

Assuming that the argument c meets its preconditions (i.e. that it is either 'heads' or 'tails' at runtime, and nothing else), then f1 and f2 do the same thing, with just different implicit/explicit styles. Some people favour one style and some the other.
With #8548 in play, tsc sees f2 differently. It no longer detects unreachable code. Also, the inference that c is 'heads' after the if/else block does not agree with either runtime or intuitive behaviour. What if we had checked for 'tails' first and then 'heads'? Then tsc would infer that c was 'tails' after the if/else. So depending on the order of checks, we may or may not get a compiler warning about a typo in the following version of f2:
function f2b(c: Coin) {
    if (c.type === 'heads') {
        return c.h;
    }
    else if (c.type === 'tails') {
        return c.t;
    }

    c.h; // No compiler error
    c.t; // ERROR: 't' doesn't exist

    // --- If we had checked for 'tails' before 'heads' in the if/else above, we'd get the opposite result: ---
    c.h; // ERROR: 'h' doesn't exist
    c.t; // No compiler error
}

These errors are really artifacts of #8548. They arise from a type inference that does not reflect runtime behaviour, and they are sensitive to code order that has no logical relevance. Personally, I'd rather see the 'unreachable code' error in all the examples above.
I'd like to think that there is some exception that could be implemented around #8548 so that it still applies in situations like #8513, but no longer produces artifacts like those here and in #9246 and #9254.

Suggestion: don't apply #8548 when encountering a type guard whose asserted type exactly matches the control flow type at that point in the code.
Here is a example demonstrating the type inference behaviour from before #8548 was merged:
// compiled with<EMAIL_ADDRESS>let x: number | string;
x // x is number | string
x = 1234;
x // x is number
if (typeof x === "string") {
    return x; // x is nothing           <===== #8513 problem
}
x // x is number
if (typeof x === "number") {
    return x; // x is number
}
x // x is nothing

Here is the behaviour of the same example currently (i.e. with #8548):
// compiled with<EMAIL_ADDRESS>let x: number | string;
x // x is number | string
x = 1234;
x // x is number
if (typeof x === "string") {
    return x; // x is string            <===== fixed #8513
}
x // x is number
if (typeof x === "number") {
    return x; // x is number
}
x // x is string                        <===== new #9246/#9254/#9260 problem

Here is what I think the behaviour would be if #8548 is not applied when a type assertion exactly matches the current control flow type:
// Compiled with hypothetical change to #8548 behaviour
let x: number | string;
x // x is number | string
x = 1234;
x // x is number
if (typeof x === "string") {
    return x; // x is string            <===== fixed #8513 (still)
}
x // x is number
if (typeof x === "number") { //         <===== DON'T apply #8548 here
    return x; // x is number
}
x // x is never                         <===== fixed #9246/#9254/#9260

The last example still fixes #8513, but now also produces the 'expected behaviour' for #9246 and #9254 and #9260.

I see how that might work for this example, but I'm not sure we could formulate a notion of exactly matches for all the different kinds of type guards we support. For example, it's not clear what would exactly match a truthy or falsy check. Also, I'm not really sure it works in all cases. Here's a twist on the motivating example from #8513:
let cond: boolean;
function foo() {
    let x: string | number = 0;
    x;  // number
    while (cond) {
        x;  // number, then string | number
        x = typeof x === "number" ? "abc" : x.slice();
        x;  // string
    }
}

I think you're arguing x should be never in the false branch of the conditional operator in the initial analysis of the loop body (before the flow back is considered). That would cause an error to be reported, which is what we're trying to avoid.
We could potentially solve #8513 by only applying #8548 when we know we're analyzing a loop body with incomplete data. But that would require more closely tracking the distinction between the phases of loop analysis. And it still wouldn't solve the initial example in #8548, i.e. the cases where you really want the type guard to function as an assertion because control flow analysis isn't aware of an assignment that has occurred in a nested function.

@ahejlsberg I don't know the details of the compiler's loop analysis. So speaking purely from my meat-powered type analysis of your loop example, it's unclear to me why the above suggestion would infer never anywhere?
Here is the same code expanded a little:
let cond: boolean;
function foo() {
    let x: string | number = 0;
    x;  // number
    while (cond) {
        x;  // number | string              <===== (1)
        if (typeof x === 'number') {
            x;  // number                   <===== (2)
            x = "abc";
            x;  // string
        }
        else {
            x;  // string                   <===== (3)
            x = x.slice();
            x;  // string
        }
        x;  // string                       <===== (4)
    }
    x;  // number | string
}

In your example at (1) you've commented x;  // number, then string | number, whereas intellisense just shows number | string. Is the 'x, then y' type tracking important to tsc's loop analysis? Because if it's just analysed as number | string at (1), then #8548 would not apply to the type guard, and we'd get number at (2), string at (3), and string at (4), and no never anywhere. I suppose I am missing something?

I'm not sure we could formulate a notion of exactly matches for all the different kinds of type guards we support

I'm not sure why that would even be necessary, at least initially. If 'exactly matches' meant literally 'the same type', it would fix all cases of the 'type exhaustion' pattern where a series if if/thens check for every possible type in a union, much like the way discriminated unions now work with switch/case.
That would resolve #9246, #9254, and this issue. Anything more sophisticated than that would be a further improvement that could be addressed later.

Fixed in #10118.

@ahejlsberg can this please be re-opened? Some of the reported inconsistencies are still there with 2.1.0-dev.20160804 - it seems like the improvements from #10118 affect some parts of the code but not others.
Remaining issues with the original example:

The inferred types at (1)/(2)/(3) should be the same as the inferred types at (4)/(5)/(6), but they are still different. The type of x at (4), (5), and (6) is now never, so #10118 seems to have fixed that. But inference at (1)/(2)/(3) is unchanged, and x at (1) remains inferred as C. Shouldn't it be never, like it is in (4)? The two functions have the same logical control flow, after all.

Remaining issues with the heads | tails example from this comment above:

The f2 function looks better - c is now infered as never after the if/else. But why does f1 report unreachable code on the last line, but f2 does not? If c is inferred as never on the last line of function f2, doesn't that imply unreachable code?

Another example with unreachable code:
In #8652 it says "never is the return type for functions that never return". So the following example, where both functions behave identically, surprises me:
function nonReturningA() { // inferred return type is string|number
    let x = Math.random() > 0.5 ? 'foo' : 42; // x is string|number
    if (typeof x === 'string') {
        throw x; // x is string
        x; // ERROR: unreachable code detected
    }
    else /* x must be a number */ {
        throw x; // x is number
        x; // ERROR: unreachable code detected
    }
    return x; // ERROR: unreachable code detected. Doesn't that imply return type is never?
}

function nonReturningB() { // inferred return type is never
    let x = Math.random() > 0.5 ? 'foo' : 42;
    if (typeof x === 'string') {
        throw x; // x is string
        x; // ERROR: unreachable code detected
    }
    else if (typeof x === 'number') {
        throw x; // x is number
        x; // ERROR: unreachable code detected
    }
    return x; // x is never, and return type is never, implying this is unreachable
              // no error reported here. Why doesn't tsc report this as unreachable code?
}

tsc correctly infers that nonReturningA doesn't return, reporting 'unreachable code' on the last line, so shouldn't the inferred return type be never?

In nonReturningB, tsc infers x is never on the last line, implying that line is unreachable according to the above definition of 'never'. And indeed, the function's inferred return type is never. So shouldn't it report the last line as unreachable code like the other function does?

But inference at (1)/(2)/(3) is unchanged, and x at (1) remains inferred as C. Shouldn't it be never, like it is in (4)? The two functions have the same logical control flow, after all.

Agreed. If !isFoo(x) applied to x: Foo | Bar produces Bar, then !isFoo(x) applied to x: Foo ought to produce never. The current behavior is a holdover from before we had control flow analysis and never types and is unrelated to #10118. I think we need a separate issue for it.

But why does f1 report unreachable code on the last line, but f2 does not? If c is inferred as never on the last line of function f2, doesn't that imply unreachable code?

We only report unreachable code errors based on the structure of the code, i.e. when we are 100% certain the code is unreachable in all circumstances. A never type does indeed imply unreachable code, but only if the assumptions made by the type checker are correct. There are many ways the assumptions could be violated, such as incorrect type assertions or bad parameters passed from unchecked code. You want to be able to write code to guard against that (e.g. assertNever calls), and that wouldn't be possible if we always flagged such code as unreachable.

@ahejlsberg I've submitted issue #10145 for the remaining type guard narrowing issue.
For the unreachable code part, I understand the rationale you've just given, the compiler is being conservative with its inferences. But the different compiler behaviour for nonReturningA and nonReturningB still do seem surprising when put side-by-side, given the meaning of never. Maybe there's some room for future improved 'strict analysis mode' if enough people wanted it.

I was about to post about 'never' in switch and unreachable code in a new issue and found this.
I guess I'll instead take the time to +1 the idea of a strict analysis mode. In cases where TypeScript calls TypeScript, it's somewhat frustrating to make concessions for where the type system 'could have been given the wrong information' (see also my issue #10765 in a similar vein, where all code is TS but we rely upon the user providing valid type information).
In line with A. Hejlsberg's comments, perhaps we could at least /warn/ here? I'll see if it's possible to add a lint to TSLint, for now.

Original issue:
type X = 'a' | 'b';

function sw(x: X): number {
  switch (x) {
    case 'a':
      return 1;
    case 'b':
      return 2;
    default:
      return 'string' // Error: Type 'string' is not assignable to type 'number'.
  }
}

Expected behavior: Error on the line 'default' indicating unreachable code (or better, an over-exhaustive match).
Actual behavior: Error on the returned string line.


tch-rs example mnist works fine, but fails to find libtorch_cuda.so as a dependency?

when doing a fully clean build of myproject, i am getting this error:
target/release/myproject : error while loading shared libraries: libtorch_cuda.so: cannot open shared object file: No such file or directory

but i can build and run the tch-rs mnist example just fine in the same environment.
tch-rs is added to my dependencies via:
"tch" = { git = "https://github.com/LaurentMazare/tch-rs" }

Are you certain your env vars are configured correctly?
These are mine:
export LIBTORCH_USE_PYTORCH=1
export LIBTORCH_BYPASS_VERSION_CHECK=1
export LD_LIBRARY_PATH=/home/ubuntu/anaconda3/lib/python3.10/site-packages/torch/lib/:$LD_LIBRARY_PATH

Indeed you probably want to set your LD_LIBRARY_PATH appropriately, you could also try having the same build script as used in the main crate build.rs: this should set the proper rpath value so that there is no need for LD_LIBRARY_PATH to be set.

my LD_LIBRARY_PATH was missing the trailing /lib (i.e. it was /usr/local/libtorch rather than /usr/local/libtorch/lib). the fact that the mnist example still compiled and ran threw me off! thanks folks.


update support for presto  v338

update support for presto  v338；
because of the conflict of powermock with jdk11, so I delete the powermock invoking in testing code.

LGTM, merged, thanks!   :)


Do you have samples for Android, iOS with .NET MAUI?

I want to create an AI-powered mobile app using .NET MAUI, but I'd like an example of how to use it.

Hi thanks for the suggestion we will create some samples for calling Phi-3 using the Azure Inference API for GitHub Marketplace Models or Azure AI Model Catalog

Thank you!
I would like to know how to run inference with phi-3-mini directly on a mobile device through a .NET MAUI app, if possible.

This is a good idea ！


Problem connecting to Impala

Impala server version: impalad version 3.2.0-cdh6.3.2 RELEASE
Python version: 3.12.3
I keep getting errors when adding an impala connection to DVT. The connection requires no authentication, no ssl and uses binary transportation.
Here are the errors I get with different command settings.
data-validation connections add \
--connection-name impala_conn Impala \
--host xxx.xxx.xxx.xxx
--port 21050

Traceback (most recent call last):
  File "/home/user/google-dvt/venv/lib/python3.12/site-packages/thrift/transport/TSocket.py", line 150, in read
    buff = self.handle.recv(sz)
           ^^^^^^^^^^^^^^^^^^^^
TimeoutError: timed out

data-validation connections add \
--connection-name impala_conn Impala \
--host xxx.xxx.xxx.xxx
--port 21050
--auth-mechanism NOSASL

12/23/2024 07:32:58 AM-INFO: Using database default as default
12/23/2024 07:32:58 AM-INFO: Pinging the impalad
12/23/2024 07:32:58 AM-INFO: Success! Config output written to /home/user/.config/google-pso-data-validator/prd_impala.connection.json
12/23/2024 07:32:58 AM-INFO: Closing active operation
12/23/2024 07:32:58 AM-ERROR: Failed to open transport (tries_left=3)
Traceback (most recent call last):
  File "/home/user/google-dvt/venv/lib/python3.12/site-packages/thrift/transport/TSocket.py", line 90, in isOpen
    self.handle.settimeout(0)
OSError: [Errno 9] Bad file descriptor

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/user/google-dvt/venv/lib/python3.12/site-packages/impala/hiveserver2.py", line 1098, in _execute
    open_transport(transport)
  File "/home/user/google-dvt/venv/lib/python3.12/site-packages/impala/hiveserver2.py", line 1162, in open_transport
    if not transport.isOpen():
           ^^^^^^^^^^^^^^^^^^
  File "/home/user/google-dvt/venv/lib/python3.12/site-packages/thrift/transport/TTransport.py", line 152, in isOpen
    return self.__trans.isOpen()
           ^^^^^^^^^^^^^^^^^^^^^
  File "/home/user/google-dvt/venv/lib/python3.12/site-packages/thrift/transport/TSocket.py", line 98, in isOpen
    self.handle.settimeout(original_timeout)
OSError: [Errno 9] Bad file descriptor

data-validation connections add \
--connection-name impala_conn Impala \
--host xxx.xxx.xxx.xxx
--port 21050
--auth-mechanism NOSASL
--use-ssl False

12/23/2024 07:34:58 AM-WARNING: using legacy validation callback
/home/user/google-dvt/venv/lib/python3.12/site-packages/thrift/transport/TSSLSocket.py:53: DeprecationWarning: ssl.PROTOCOL_TLS is deprecated
  self._context = ssl.SSLContext(ssl_version)
12/23/2024 07:35:43 AM-INFO: Could not connect to ('xxx.xxx.xxx.xxx', 21050)
Traceback (most recent call last):
  File "/home/user/google-dvt/venv/lib/python3.12/site-packages/thrift/transport/TSocket.py", line 137, in open
    handle.connect(sockaddr)
  File "/usr/lib/python3.12/ssl.py", line 1353, in connect
    self._real_connect(addr, False)
  File "/usr/lib/python3.12/ssl.py", line 1344, in _real_connect
    self.do_handshake()
  File "/usr/lib/python3.12/ssl.py", line 1320, in do_handshake
    self._sslobj.do_handshake()
TimeoutError: _ssl.c:983: The handshake operation timed out
12/23/2024 07:35:43 AM-ERROR: Could not connect to any of [('xxx.xxx.xxx.xxx', 21050)]

Tested in Python 3.11.11, same results ...

Hi @CysOrion, are you able to connect directly from Python using the impyla library (i.e. outside of DVT)?

Hi @CysOrion, are you able to connect directly from Python using the impyla library (i.e. outside of DVT)?

Hi @nj1973, I was able to connect and query results.
from impala.dbapi import connect

conn = connect(host = "xxx.xxx.xxx.xxx",  port = 21050)
cursor = conn.cursor()
cursor.execute("SELECT * FROM table LIMIT 5")

print(cursor.description) # printed correct results

Hi @CysOrion, I just wanted to update you that I have been able to reproduce this myself, it took me a while to create a test environment.
The initial TimeoutError: timed out exception is because we have a different default to Impyla, we default auth mechanism to PLAIN instead of NOSASL. So you were correct to add that to your command.
You can see in the subsequent message that the connection file was created successfully:
12/23/2024 07:32:58 AM-INFO: Success! Config output written to /home/user/.config/google-pso-data-validator/prd_impala.connection.json
12/23/2024 07:32:58 AM-INFO: Closing active operation

We only create the file after successfully connecting. The exception is coming from connection cleanup in Ibis/SQLAlchemy.
So I'll continue to try and figure out how to prevent the exception but I think you should be able to continue testing with the connection created just before the exception was thrown.

A note for my reference, the exception is being throw in thrift/transport/TSocket.py here:
        try:
            self.handle.settimeout(0) <<<<<<<
            try:
                peeked_bytes = self.handle.recv(1, socket.MSG_PEEK)
            except (socket.error, OSError) as exc:  # on modern python this is just BlockingIOError
                if exc.errno in (errno.EWOULDBLOCK, errno.EAGAIN):
                    return True
                return False
        finally:
            self.handle.settimeout(original_timeout)

The line is throwing OSError but is outside of the try/except block.
The source of the call is:
  /home/user/professional-services-data-validator/env/lib/python3.12/site-packages/impala/hiveserver2.py(179)__del__()
-> self.close_operation()

Part of clean up as the calling process is being cleaned up.


Issue549redo

Fixes issue #549. Third attempt, after two iterations of breaking stuff not covered in the UTs and extending UTs to cover.

OK - FV tests pass. Please verify last markups, then merge.

Third time's the charm right?

No, no it's not.

@plwhite - there are still error logs in the FV test output.


Bump rules_go to 2383294

This is to pick up fixes that allow rules_go to work on Windows. There
are several required steps to make this work:

Remove fork of go_proto_library.bzl and rework pgv_go_proto_library
to use proto rules from rules_go
Because of 1, bump dependencies in Gopkg.toml to match those in
rules_go
a. github.com/golang/protobuf -> v1.1.0
b. github.com/lyft/protoc-gen-star -> v0.3.7
c. github.com/gogo/protobuf -> v1.0.0
Correct location in Makefile for generated validate.pb.go
Update protos in tests/kitchensink to explicitly set go_package
option. Without this option, the generated *.pb.validate.go files had
empty package names (the regular *.pb.go files were fine)
Rerun "gazelle fix" in the vendor directory to regenerate BUILD.bazel
files.

Signed-off-by: Sam Smith<EMAIL_ADDRESS>

Thanks! @sesmith177 This deprecates https://github.com/lyft/protoc-gen-validate/pull/86.
@akonradi can you review and approve this?

Thanks! Looks good!


PptxGenJS slide.addTable Feature's Rows Property Error

PptxGenJS returns an error when executing slide.addTable. The rows property of type string[] is not assignable to the parameter of type 'tableRow[]'. This error started with the patch for issue #734.

Angular CLI: 9.1.4
Node: 12.16.3
OS: win32 x64

Thanks for reporting this @robertsoaa
The type defs were not good to begin with, hence the rows issue.
Fixed now in master branch.
  let rows = [];
  rows.push(["First", "Second", "Third", "Fourth"]);
  rows.push([{ text: "TODO" }, { text: "optionsChk", options: { fontFace: "Arial" } }]);
  slide1.addTable(rows, { x: 1, y: 1 });
  slide1.addTable([[{ text: "cell 1" }]], { x: 1, y: 1 });


Handle deprecated est_time_step

Since commit 33987abcb04254dc6b195eefe7b47011fa3545f0 ICgen_utils.est_time_step is deprecated but still is used for making binary snapshots.

Actually, for now this should be fine.  It's not ideal, but it works.


Support Skip Layer Guidance (SLG) for Stable Diffusion 3.5 Medium

This closes #2588 .
I noticed SLG doesn't perform well when number of inference steps is low. So, below is a comparison with --num-inference-steps=50.

w/o SLG
w/ SLG

Appearantly the fingers look better. This could be an evidance that supports claimed benefits (improved anatomy). But other aspects of the image changes, too.
The SLG parameter is copied from that in SD3 reference design, which is originally with dpmpp_2m sampler (not Euler sampler in the Candle example here). So, it might be worth to tweak the parameters with more experiments.

Thanks, neat addition!


Add giantswarm.azurecr.io to External Domain allowlist

sort external domain list
add giantswarm.azurecr.io to the domain allowlist

Please remember to

[x] Give the PR a meaningful title -- it will be shown in the release notes
[x] Add (or remove) user questions associated with any updated docs

i had an almost identical PR in progress :sweat_smile:

i'm gonna merge this in @kubasobon because i want to add to the doc :)


dnn-button issue with reversed and type combinations.

When a dnn-button is reversed, certain style types do not function correctly.
I found that reversed when matched with primary or danger type work accordingly, but not with the other types; secondary and tertiary. The buttons become filled with this dark gray color.

May be related with #749 or should at least be tested together...

Appears to be fixed now


allow customizing the worker names and do not override defaults

In order to support using dask-mpi as a backend for multiple workers per job in dask-gateway, I am introducing the capability to set a custom name, to which the rank is appended. This change is needed for keeping track of workers in dask-gateway (PR to be submitted). In addition, I am reverting some of the default overrides for the CLI options, as those for example set the TCP protocol and a scheduler-file, whereas the dask-gateway use case requires being able to specify a scheduler address from the command line.

@jglaser:  I suspect that the

@jglaser:  I suspect that the

@jglaser:  Hey!  I got this to pass.  Do you want me to merge this in now?

Thanks for fixing it , @kmpaul .... and sorry for abandoning the issue. Please feel free to merge.

Now worries.  And thanks, @jglaser!


Update tutorial-suspicious-activity.md

December 15

Learn Build status updates of commit b195f65:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

CloudAppSecurityDocs/tutorial-suspicious-activity.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Bug when using the meter widget (related to how the Pillow package handles image resize)

Desktop (please complete the following information):
ttkbootstrap Version 1.10.1
pillow Version 10.0.1
python Version 3.12.0
OS: Windows 10
IDE: PyCharm Version 2023.2.2
Describe the bug
when using the meter widget with new updated version of the pillow library which ttkbootstrap uses it gives an error suggesting using BICUBIC method instead of CUBIC used by default in ttkbootstrap
this is probably more related to the pillow library but I think it should be mentioned to see if the developer can update the ttkbootstrap  library to support it and fix the error.
Note: the error can be evaded by using an older version of the pillow package like v 9.5.0
To Reproduce
Using the meter widget with a newer version of Pillow module than v 9.5.0 for example v 10.0.1
Expected behavior
the meter widget shall work fine and the app starts up and work fine instead of crashing due to the error mentioned
Screenshots

Additional context
No response

Duplicate of #472, and a bunch of others.

The issue was resolved by the merge of #490. Please close it.


Feature: Update default lower thresholds for AE and SAE Workflows

Feature Details
Default thresholds for AE and SAE should be set to -1 and -2 instead of the current standard values of -2 and -3.
Example Code
simple update to workflows kri001, kri002, Cou001 and Cou002
Possible Implementation

Additional Comments

As I recall, we're only concerned with AEs, not SAEs, because SAEs are so sporadic. But it probably won't hurt either.


npm error not available in the internal and external

pls my npm is not working pls help asap is tell me git not found


Storage improvements

Use Path.Combine instead of '+'  to combine file paths.

Looks good, thank you!

No worries, I can fix 'em later!


Feature Request: variable height

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[x] Mac OS X
[x] Windows
[x] Etc.

Shell

[x] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce
I don't like using fzf in fullscreen because I lose the context, but if I set a fixed height sometimes it's too much (and there's a blank gap that is unused)  and sometimes isn't enough and I need to scroll within a tiny area.
To solve this I'm thinking of a --height auto option that uses as many lines as there are files to show.
If there are more files than available rows, then fzf would start fullscreen and have scroll, otherwise all the entries would fit in screen without any blank space.

This is exactly I wanted 😄 You are awesome!


Report to user when ytt run ran templates not included in output

It's unclear from the result that when a user includes a non-YAML template and the destination is standard out, that its output is not included on purpose.
Let's give them a hint. For example, on stderr, we could say,
There are templates marked for output that are not included in this output.
Non-YAML templates are not rendered to standard output.
If you want to include those results, use the --output-directory flag.

cc: @pivotal-dean

mind providing more detailed example when this is the case?

A user is developing a text template (call it app.ini).
They run ytt with just that template:
$ ytt -f app.ini.txt
$

They expected to see output, but see nothing.
They try again with --debug
$ ytt -f app.ini.txt --debug
-  (private false)
  files:
  - app.ini.txt
  libraries:
    <none>
## file app.ini.txt
### template
src:  tmpl: code: | srccode
????:    1: __ytt_tpl2_set_ctx_type("text") |
????:    2: __ytt_tpl2_start_ctx("text") |
????:    3: __ytt_tpl2_start_node(1) |
   1:    4: __ytt_tpl2_start_node(2) |
   1:    5: __ytt_tpl2_set_node(2) |
   1:    6:  module="core"  |  module="core"
   1:    7: __ytt_tpl2_start_node(3) | [
   1:    8: __ytt_tpl2_set_node(3) | [
   2:    9: __ytt_tpl2_start_node(4) | =- module -
   2:   10: __ytt_tpl2_set_node(4, ( module )) | =- module -
   2:   11: __ytt_tpl2_start_node(5) | ]
  param1 = value

   2:   12: __ytt_tpl2_set_node(5) | ]
  param1 = value

????:   13: __ytt_tpl2_end_ctx() | ### app.ini.txt result
[core]
  param1 = value
### result
total: 439.781µs

and see that the template runs, but not included in the result.

As a Configuration Consumer (particularly someone who hasn't used ytt to render non-YAML files before),
I want to quickly (and in a kind way) learn how to specify where output should go
So that I don't waste time wondering why my non-YAML files didn't seem to render.

Scenario: Library includes just non-YAML files and no output flag is specified
Given the enumerated input files include only non-YAML output files (e.g. app.ini.txt)
When ytt is invoked without any flag indicating location of output
ytt -f app.ini.txt

Then I see a warning explaining that nothing will output without help from a flag.
__
Scenario: Library includes a mix of YAML and non-YAML files and no output flag is specified
Given the enumerated input files includes some YAML files and some non-YAML files (e.g. app.ini.txt and config.yml)
When ytt is invoked without any flag indicating location of output (e.g. --output-directory or --dangerous-emptied-output-directory)
ytt-f app.ini.txt -f config.yml

Then I see the YAML results on Standard In and a warning explaining that nothing will output without help from a flag on Standard Error

see also:

https://github.com/k14s/ytt/blob/develop/docs/outputs.md
https://github.com/k14s/ytt/blob/develop/docs/file-marks.md
https://github.com/k14s/ytt/blob/develop/docs/ytt-text-templating.md


Add a parameter to output or not object coordinates

We have started to output the coordinates in the original PDF of some TEI elements (e.g. figure markers) with bounding boxes. As we will output coordinates for more objects, it would nice to have in the APIs for the full text processing a parameter to choose to output or not these coordinates.

I am wondering if you provide this information for the TEI elements like author, title and ,,,?

Not for the moment, but the area coordinates can be provided for any fields/structures of the document, including those where the text is normalized. Currently coordinates appear for bibliographical references and reference markers which are used in the PDF annotation demos.
We're really open to suggestion on how to specify which coordinates to get in the output, given that it would be very heavy to output too many of them. We can also provide the coordinates as JSON annotations independently from the XML TEI (see web service referenceAnnotations). We're still exploring also how to exploit nicely those annotations for creating dynamic layer on top of a PDF rendered in the browser.

Hi Patrice, what would be involved in adding coordinates for the header, e.g. author and affiliation?

I wonder if this hasn't been implemented yet


fix(productSync): Update categories before categoryOrderHints

Summary
This PR fixes #218
Description
When generating product update actions category actions should be placed before categoryOrderHint actions so we can add product to category and create a categoryOrderHint in one request.
Todo

Tests

[x] Unit
[ ] Integration
[ ] Acceptance

[ ] Documentation

Coverage remained the same at 97.555% when pulling 91e7f9df0ec89b765ccbaeec0c5526e52373832b on 218-fix-category-order-hints-creation-order into aaaf4d95bab4e2d88b0987474e95230c2cf59141 on master.

After a short discussion with @wizzy25 ,
I would rather see that the setCategoryHint is moved out of the actionMapsBase and that you create a separate mapper for this action, with the specified criteria (such as categories should be linked to the product) to fulfil the action
code
// in utils/product.coffee
const actionCategoryOrderHint = {
  action: 'setCategoryOrderHint',
  key: 'categoryOrderHints',
}

actionsMapCategoryOrderHint: () => {
  const action = this.buildBaseAttributesAction(item, diff, old_obj)
  // do your thing then push to the actions list!
}

// in product-sync.coffee
// below 'categories' mapper configuration
allActions.push(
  this._mapActionOrNot(
    ('categoryOrderHint'),
    () => this._utils.actionsMapCategoryOrderHint(diff)
  )
)

Once you take this approach, make sure to remove the setCategoryHint from actionsBaseList, and you can move the categories action configuration back to it original order

@wizzy25 @adnasa - I refactored code according to your comments.. please check it now.

Coverage increased (+0.007%) to 97.562% when pulling 576b0b6f25a79b0b471cee0d6395ae234d743d1a on 218-fix-category-order-hints-creation-order into aaaf4d95bab4e2d88b0987474e95230c2cf59141 on master.

@wizzy25 can you please merge this PR? I am not authorized to do that 😢

@emmenko Could you review and merge this PR? I don't have write access on this repo


Failed to resolve: com.jakewharton.rxbinding4:rxbinding:4.0.0

Hello,
I am trying to add a dependency to library but it shows error
ERROR: Failed to resolve: com.jakewharton.rxbinding4:rxbinding:4.0.0
Is it not released yet?

It is not. You can find all of the available releases in two places:

Our release tab: https://github.com/JakeWharton/RxBinding/releases
Maven central: https://search.maven.org/search?q=a:rxbinding "jakewharton"

It'll be out soon.

You can also use the 4.0.0-SNAPSHOT although CI looks completely dead so presumably it's not publishing.


Hover for type synonyms broken?

In a Yesod app I have
import           Foundation
import           Yesod.Core
import           Text.Blaze

getHomeR :: Handler Html
getHomeR = defaultLayout $ setTitle "asdf"

tmp = toHtml ("asdf" :: String)

Hover over toHtml works just fine. It's located at module Text.Blaze.Html. But hover over Html fails, although it's located in the same file than toHtml.

So, you request the documentation of a symbol, but it could not be fetched, thus, it failed to show you the documentation for a type?


Deploy mirage-www from master branch

Now that the deployment seems to be working, it's probably easiest to deploy from the master branch.
/cc @hannesm

sounds good, I am only not sure about the branches:

master deploys via travis to some xen machine where mirage.io and openmirage.org is hosted (this uses TLS certificates + private keys via a FAT fs)
live deploys via ocurrent to our packet.net machine for mirageos.org

I did not catch the migration plan for (a) above (the deployment of (b) uses Dns_mirage_certify, which uses DNS for let's encrypt certificates -- i.e. mirage.io/openmirage.org DNS would need to be migrated (this can easily be done to ns1&ns2.mirageos.org which are already running as MirageOS unikernels)). Maybe @avsm has a more clear picture in mind.


Remove some lagginess from TextEdit's smooth scrolling

In a nutshell:

If the scroll direction changes, reset the velocity to 0 (instead of adding to it until it reaches 0)
for small scroll amounts (< 1, seen on gesture inputs with a high frequency) improve feedback time by immediately scrolling

I'll do a video on this to show the effects compared to the current master.

The promised videos. It's subtle, but I hope it's visible:
With master:
https://www.dropbox.com/s/rpcf5wk0g9i1hgt/scroll-before.mp4?dl=0
With PR:
https://www.dropbox.com/s/o32csmj1b3fhhqs/scroll-after.mp4?dl=0
Note that this doesn't affect smooth scrolling as long as the direction doesn't change.


Fix Cython compile for v3.0.0 release

Cython v3.0.0 was recently released (https://github.com/cython/cython/releases/tag/3.0.0) and is used in newly built docker images. This causes a compilation issue since 3.0.0 expects function definitions to be explicitly declared with the noexcept annotation. This change should be backwards compatible to v0.29.31. For more details see the discussion here: https://github.com/scipy/scipy/issues/17234#issuecomment-1493423202.

Windows and MacOS builds are failing as they use cython==0.29.28 which is not compatible with the new noexcept keyword. This functionality was added in 0.29.31 (https://github.com/cython/cython/blob/master/CHANGES.rst#02931-2022-07-27). We could either upgrade the Windows and MacOS builds to use a newer version of cython (I'm not too sure yet how to get later versions in conda), or we can constrain the version in main CI to <= 0.29.36.

My guess would be that because the Conda build uses 3.7 of Python, it isn't receiving newer builds of Cython.
https://github.com/apache/tvm/blob/e2d65111616dfa95797c0dd7e082e4050b71701d/conda/build-environment.yaml#L28
That may be quite concerning given 3.7 just passed end of life (https://devguide.python.org/versions/).

We can remove the 3.7 here from tehe build env

Let's get this PR merged instead!

After updating the python version to 3.8, the same issues mentioned in https://github.com/apache/tvm/pull/15346#issuecomment-1640409277 seem to persist

https://github.com/apache/tvm/pull/15469


Is it possible to add a limit create permission?

I would like to add a limitation on the amount of "create" to a permission on a role. The user-case is like bronze, silver and gold subscriptions. These roles should be able to create a specific model but in a limited amount. Is that possible?

What immediately comes to mind is using a Permissions' Criteria to achieve this:
{
  model: 'Item',
  action: 'create',
  criteria: [ { where: { count: { '<': 5 } } ]
}

Nice! :+1:


remove bottom button height

Fix #88.
Allows padding to be added to the bottom button.

Thanks! I've just added this in a larger commit 1c3d6bbced0ce089f2bdfc4533be5e3f9a375929


Connection resiliency 2

Coverage remained the same at 55.412% when pulling 8d3bc7b6ae56f409659a227e3df1f8466c81fb42 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage remained the same at 55.412% when pulling 67b6468462a8caf53916ab8e95e3058cbff1f6b9 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage remained the same at 55.412% when pulling 132b4e0a455e96eb29b426d865afd8514bbac3af on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage remained the same at 55.412% when pulling f2974fdc9e4c86a639c6a39572ebbc95d729dce8 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage remained the same at 55.412% when pulling ed2c04ee94526022fb55172cc49a4d205c9ee0ed on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage remained the same at 55.412% when pulling 56b543861bee63021860160555ff5aeaa4c1f5eb on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage remained the same at 55.412% when pulling bac38618572d4f2a7a09f98152468e4cf9e5e547 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage remained the same at 55.412% when pulling 1b59ae6eafccfea4990353c878a3b6c6a4ce13b9 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage remained the same at 55.412% when pulling ba4deda27c4f4424f2e5cde4a680714fdba14f39 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage remained the same at 55.412% when pulling d75304638dbaddfc3778c7f832567ed5c5645f05 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage remained the same at 55.412% when pulling 8c466bb8914136419b2b2840f0ceadf6bc102fca on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage increased (+0.02%) to 55.429% when pulling 5eaae8181416157c28e65391a54d836f134761b7 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage decreased (-26.5%) to 28.919% when pulling 5eaae8181416157c28e65391a54d836f134761b7 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage increased (+0.02%) to 55.429% when pulling 5b192c1bc8babcf5e83b5afdc9426b5ac57001d9 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage increased (+0.02%) to 55.429% when pulling 611d8656855c841ec626502a291822fa278c9fc2 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage increased (+0.02%) to 55.429% when pulling 5f8ac9ac001694812ad975850a97b9b88edff7a2 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage increased (+0.02%) to 55.429% when pulling bf7049bbf6217a9d2d052f20cd6be894d0d9e5a3 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage increased (+0.02%) to 55.429% when pulling bf7049bbf6217a9d2d052f20cd6be894d0d9e5a3 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage increased (+0.02%) to 55.429% when pulling bf7049bbf6217a9d2d052f20cd6be894d0d9e5a3 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage increased (+0.02%) to 55.429% when pulling 1b016204d7255cb9713eaaccf768c2fee7edf372 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage increased (+0.02%) to 55.429% when pulling 2f6ba8deedd503567b6b9a7ccc0d3d9c3b675cef on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage increased (+0.02%) to 55.429% when pulling 6eb273a9e71fdd926454f03dc27f3e2807bb67f6 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage increased (+0.02%) to 55.429% when pulling 6eb273a9e71fdd926454f03dc27f3e2807bb67f6 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage increased (+0.02%) to 55.429% when pulling 85983b100e73772ce5827e8f32994c59958d0c8a on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Coverage decreased (-25.8%) to 29.578% when pulling b8267db0ba359f8577441bc2231d4dec3046d362 on david-puglielli:connection_resiliency_2 into bd8d1e1bf48f0cd3f4dd2e1513c73ec4bf07b347 on Microsoft:dev.

Closing because we have a newer PR for connres.


Merging function "repeat"

For changelog. Remove if this is non-significant change.
Category (leave one):

Other

Short description (up to few sentences):
Continuation of #6999.

A query like
SELECT repeat('',<PHONE_NUMBER>)
opens the possibility for DoS.


feat(apigateway): allow customizing response for non-matched requests

Issue #, if available:
Description of changes:
Allow custom HTTP response for requests which would normally result in 404 messages.  Supports a static Response object, or a callable function that returns one.  Callable function takes the method and path as parameters.  All of the Response can be overridden, including the status code.
Examples:
not_found_body = open("404.html", "r").read()
not_found_response = Response(status_code=404, content_type=content_types.TEXT_HTML, body=not_found_body)
app = ApiGatewayResolver(custom_not_found=not_found_response)

def not_found_body(method: str, path: str):
    return Response(
        status_code=404,
        content_type=content_types.TEXT_PLAIN,
        body=f"Sorry, {path} not found!"
    )

app = ApiGatewayResolver(custom_not_found=not_found_body)

Checklist

[x] Meet tenets criteria
[x] Update tests
[ ] Update docs
[x] PR title follows conventional commit semantics

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #727 (cb30ebe) into develop (610f569) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #727   +/-   ##
========================================
  Coverage    99.93%   99.93%
========================================
  Files          116      116
  Lines         4910     4916    +6
  Branches       271      273    +2
========================================
+ Hits          4907     4913    +6
  Misses           1        1
  Partials         2        2

Impacted Files
Coverage Δ

aws_lambda_powertools/event_handler/api_gateway.py
100.00% <100.00%> (ø)

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 610f569...cb30ebe. Read the comment docs.

Closing this as we have launched the approach of using not_found as well as any possible Python exception: https://awslabs.github.io/aws-lambda-powertools-python/latest/core/event_handler/api_gateway/#handling-not-found-routes


Add tags to outgoing emails for tracking

https://github.com/opencollective/opencollective/issues/902

Coverage increased (+0.08%) to 74.94% when pulling cf85a1037fd8c5831d72eeec390c1011ff386c3f on mailgun-tag into c969488b60700f4366a1b59390527552615adcb6 on master.


There is way to group files in android

Currently, if I have too many files it is almost impossible to search for files because it gives me an untimely list of files, there is some way of grouping them or appearing by folder.
Thanks for the plugin

I don't think so. You can ask this in native library repo here: https://github.com/fishwjy/MultiType-FilePicker


Enable "Save as"

Hi there,
With the latests update, "save as" feature has been lost.
Dowloaded documents go straigt away to Donwload folder.
Thank you!

yes. i have the same problem, plz get it back
Thank in advance :)

All sorted, wonderful!


feat(api): variable jobs backoff limits

The number of retries before considering a K8s job as failed (see this) can now be explicitly set through the following 2 environment variables for executor and filer jobs, respectively:

TESK_API_TASKMASTER_ENVIRONMENT_EXECUTOR_BACKOFF_LIMIT
TESK_API_TASKMASTER_ENVIRONMENT_FILER_BACKOFF_LIMIT

Codecov Report

Merging #22 into master will decrease coverage by 0.27%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
- Coverage   27.55%   27.28%   -0.28%
==========================================
  Files           8        8
  Lines         606      612       +6
==========================================
  Hits          167      167
- Misses        439      445       +6

Impacted Files
Coverage Δ

src/tesk_core/filer_class.py
0.00% <0.00%> (ø)

src/tesk_core/taskmaster.py
0.00% <0.00%> (ø)

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 acebd70...c9ec8b4. Read the comment docs.

Thanks for contributing.
Unfortunately it did not want to work e2e.
Issue 1: While passing env vars to taskmaster, api strips TESK_API_TASKMASTER_ENVIRONMENT_ prefix. I forgot about it and actually, that needs to be specified in the docs as well.
Issue 2: K8s unfortunately expects all env vars as Strings. So conversion to int was necessary.


Enable rotation of passwords

See #61 for the original ticket.
While this ticket was about generating and rotating passwords only generation was implemented in the linked PRs.
Implementation of rotation got deferred.
This is what this ticket is for, to capture the remaining tasks from the referenced.

Could be an epionio rotate command taking the same credentials options as epinio install.
Would have to call on the relevant deployments to re-deploy the parts of the system with the credentials.
Sounds a bit like we need an additional method in the Deployment interface, i.e. Rotate(...).

Outdated.


DailyProblrm, Day 30

Faster than 100% in java.

Already file Present so did not merged, I copied your solution and pasted in the file though , So Closing this pull request
https://github.com/Jiganesh/High-On-DSA/blob/main/binarySearch/searchA2DMatrix.java


Add tests

https://pragmaticnotes.hashnode.dev/combining-test-coverage-reports-for-sonarcloud-in-github-actions-workflows


DRILL-7297: Query hangs in planning stage when Error is thrown

Please see DRILL-7297 for problem description.

+1


Run CLI integration tests in separate processes

This is an idea on how the CLI integration tests could be run in separate processes.
I'm using shaded jar from the CLI module, because the distribution from dist/build/ module still has the class loader issue (https://github.com/wildfly-extras/prospero/issues/73), but that could be changed later. However, if dist/build/ was to be be used, it needs to be somehow ensured that it's built ("dist" profile should be activated on -DallTests?).

Looks good 👍
Would it be possible to use prospero script instead of directly calling jar? This way we don't need to read the project version, build the path to cli etc.

"dist" profile should be activated on -DallTests
+1

I updated the PR to use prospero script. It simplifies the execution indeed.

Rebased

Thanks!


Internal error occurred: failed calling webhook

{"level":"error","ts":1661965729.2943187,"logger":"anchor.reconcile","msg":"while updating on apiserver","rid":88,"trigger":"cp/test-jk","error":"Internal error occurred: failed calling webhook \"objects.hnc.x-k8s.io\": Post \"https://hnc-webhook-service.hnc-system.svc:443/validate-objects?timeout=2s\": context deadline exceeded"}
We're reinitializing our cluster from a backup of etc.d and we've run into a concern like shown above.  Is there a good mechanism for troubleshooting this type of concern?

I'm also looking into similar errors at the moment. HNC seems to be working correctly, but we get these errors occasionally in the manager log too.
Running on EKS version 1.0

@Alan01252 interesting, how did you narrow this down to RestartOnSecretRefresh, just the prior log message? If that's the case, then this is harmless if annoying.
@zybiko89 does the problem eventually go away, or is it persistent? Webhooks can fail for an almost unlimited number of reasons so I'm not sure what would cause this, but as long as HNC's running and accessible from the control plane, these kinds of problems should be transient.

The issue was persistent for us but it won't be an issue for us going forward.

@adrianludwin I think the certificate might have been a red herring.  We're still crashing regularly in our preprod environment, but not in our prod.
The concerning thing for me is that this seems to cause the propagated resource to be deleted/rebuilt which is obviously not ideal when we've got things like network policies being propagated.
I'm still trying to ascertain what the difference between the environments is, they should be carbon copy of one another...

@Alan01252 If you have cert-manager installed in your clusters, I can recommend switching to cert-manager for issuing webhook certificates. I've had a bunch of issues with the cert-rotator that HNC uses by default.....

Using cert-manager as a more production-read system sgtm. cert-rotator is
designed as a simple batteries-included feature.
LMK if you find out what the difference is between the environments.
On Thu, Sep 22, 2022 at 3:39 AM Erik Godding Boye @.***>
wrote:

@Alan01252 https://github.com/Alan01252 If you have cert-manager
installed in your clusters, I can recommend switching to cert-manager for
issuing webhook certificates. I've had a bunch of issues with the
cert-rotator that HNC uses by default.....
—
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-sigs/hierarchical-namespaces/issues/224#issuecomment-1254646119,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE43PZG25GLVC5D4DCA23BDV7QEMXANCNFSM6AAAAAAQBUNJNM
.
You are receiving this because you were mentioned.Message ID:
@.***>

@adrianludwin one of our preprod nodes was running out of CPU and memory, which seemed to be impacting this pod more than others because we configure most pods without CPU limits just CPU resources for critical pods, but the yaml here enforces a limit.
Became way more obvious in ha mode


Bigger font size

This PR increases font in chat 12px -> 13px and channel title 14px -> 15px.
I'm still not sure about this. I believe it should be changed in settings, because it so much depends on user screen size. For example, 13px is small for my desktop, but perfect for mobile.
It was also discussed in #841.

I'm still not sure about this. I believe it should be changed in settings, because it so much depends on user screen size.

that's why the css mod textfield is the perfect location for it ^^

that's why the css mod textfield is the perfect location for it

It's inconvenient to type in those rules on every device (especially on mobile). Simple select in setting will be nice.

Shouldn't this PR change font size in way more places in style.css?

Shouldn't this PR change font size in way more places in style.css?

Everything else is fine, even on my big display. We can also increase font in input area, but that's not very noticeable.

What do you think about increasing font to 14px only on big displays? @media (min-width: 1610px) for example.

Am a fan of that, @bews

We can also increase font in input area

increase font to 14px only on big displays

Done.

@bews, are you planning to fix conflicts and rebase this PR on master? :)

@astorije done


"quite" CLI option should be "quiet"

Just a typo.
https://github.com/Gamevy/pixi-packer/blob/master/cli.js#L15

Doh, that does sound like something I could have done... Thanks, will fix!

Released as<EMAIL_ADDRESS>


refactor: continuous builder document registration

Currently, the runtime only sends builder documents during the initialization
of the builder <-> host connection. This means that any functionality that could
rely on a remount of a document after the connection has already been
established will not work, since the registration will not be communicated to
the builder.
This change addresses this by adding a document registration middleware that
communicates any document registrations and unregistrations to the builder. In
addition, the document registration action includes a carve out to prevent
a builder initiated registration from being ping ponged.
https://github.com/user-attachments/assets/d0c7a560-b3a0-4f94-92ae-f4a02fda36bc

Looks good. We could also use the existing messageChannelMiddleware, but you could argue making a new middleware is better since it's easier to understand the function of each middleware.

I think the MessageChannel middleware is an active part of this by making sure the dispatched actions make their way from the host to the builder and vice versa in addition to being sent to the Redux store's reducers.


[Cherry-pick][Branch-3.0][Enhancement] support some binary/date function for trino parser (#26916)

Fixes #issue
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.1
[ ] 3.0
[ ] 2.5
[ ] 2.4

[FE PR Coverage Check]
:heart_eyes: pass : 17 / 17 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/connector/parser/trino/ComplexFunctionCallTransformer.java
11
11
100.00%
[]

:large_blue_circle:
com/starrocks/connector/parser/trino/Trino2SRFunctionCallTransformer.java
5
5
100.00%
[]

:large_blue_circle:
com/starrocks/connector/parser/trino/AstBuilder.java
1
1
100.00%
[]


WEBPACK_IMPORTED_MODULE_0_aws_sdk_global.util.crypto.lib.randomBytes is not a function error when signin is called

I think there was a similar issue here: #224 but I'm not using the react library. These are my dependencies:
"dependencies": {
    "@angular/common": "5.0.3",
    "@angular/compiler": "5.0.3",
    "@angular/compiler-cli": "5.0.3",
    "@angular/core": "5.0.3",
    "@angular/forms": "5.0.3",
    "@angular/http": "5.0.3",
    "@angular/platform-browser": "5.0.3",
    "@angular/platform-browser-dynamic": "5.0.3",
    "@ionic-native/core": "4.5.3",
    "@ionic-native/splash-screen": "4.5.3",
    "@ionic-native/status-bar": "4.5.3",
    "@ionic/storage": "2.1.3",
    "@types/node": "^9.4.4",
    "aws-amplify": "^0.2.4",
    "aws-sdk": "^2.192.0",
    "cordova-android": "7.0.0",
    "cordova-plugin-device": "^1.1.4",
    "cordova-plugin-ionic-webview": "^1.1.16",
    "cordova-plugin-splashscreen": "^4.0.3",
    "cordova-plugin-whitelist": "^1.3.1",
    "cordova-sqlite-storage": "^2.2.0",
    "enum-values": "^1.2.0",
    "ionic-angular": "3.9.2",
    "ionic-plugin-keyboard": "^2.2.1",
    "ionicons": "4.0.0-13",
    "rxjs": "5.5.2",
    "sw-toolbox": "3.6.0",
    "zone.js": "0.8.18"
  },
  "devDependencies": {
    "@ionic/app-scripts": "^3.1.8",
    "lodash": "^4.17.4",
    "typescript": "2.7.1"
  }

The error message:
__WEBPACK_IMPORTED_MODULE_0_aws_sdk_global__.util.crypto.lib.randomBytes is not a function
    at AuthenticationHelper.generateRandomSmallA (AuthenticationHelper.js:90)
    at new AuthenticationHelper (AuthenticationHelper.js:42)
    at CognitoUser.authenticateUser (CognitoUser.js:216)
    at Auth.js:270
    at new t (polyfills.js:3)
    at AuthClass.signIn (Auth.js:269)
    at AuthProvider.webpackJsonp.53.AuthProvider.login (auth.ts:28)
    at LoginPage.webpackJsonp.522.LoginPage.login (login.ts:34)
    at Object.eval [as handleEvent] (LoginPage.html:31)
    at handleEvent (core.js:13255)

The error occurs when the signIn method is called (simplified version of my code)
import Amplify, {Auth} from 'aws-amplify';
import aws_exports from './../../aws-exports';

Amplify.configure(aws_exports);

login(user: User) {
	return Auth.signIn(user.username, user.password);
}

Any ideas how this can be fixed? Thanks

@rayhaanq could you remove that aws-sdk dependency or make it to 2.177.0?

@rayhaanq The issue is the aws-sdk version that you have included:
"aws-sdk": "^2.192.0",

The version of the cognito-identity-js sdk is currently incompatible with that aws-sdk version. we are pushing a fix to this next week that will fix this, but if you are including the aws-sdk on your own as well (as in your dependencies) you will need to pin it at:
"aws-sdk": "2.177.0"

@rayhaanq this issue is fixed in latest aws-amplify and amazon-cognito-identity-js 2.0. If you install the latest aws-amplify you should get the latest cognito as well. Please let us know if you still see the issue.


Adding unit-test for matching:newTaskListID

Bumping coverage of tasklist.go to 100%

unit-test

increase code-coverage

make test

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 018de674-b6b4-4841-b6c5-ec283d95cf1b
Details

0 of 0   changed or added relevant lines in 0 files are covered.
85 unchanged lines in 10 files lost coverage.
Overall coverage increased (+0.05%) to 62.952%

Files with Coverage Reduction
New Missed Lines
%

common/task/weighted_round_robin_task_scheduler.go
2
88.06%

common/util.go
2
91.69%

common/persistence/statsComputer.go
3
96.07%

service/matching/taskListManager.go
3
79.7%

common/persistence/nosql/nosqlplugin/cassandra/tasks.go
4
76.09%

service/history/task/transfer_standby_task_executor.go
4
87.42%

common/persistence/nosql/nosql_task_store.go
5
61.95%

common/persistence/nosql/nosqlplugin/cassandra/gocql/public/client.go
12
20.93%

common/persistence/sql/workflowStateMaps.go
19
81.25%

service/history/execution/mutable_state_task_refresher.go
31
57.28%

Totals

Change from base Build 018dd83e-52b0-4905-a381-bf89981dace4:
0.05%

Covered Lines:
93028

Relevant Lines:
147777

💛  - Coveralls


Prepare to release azure spring boot bom 3.6.1

Prepare to release azure spring boot bom 3.6.1

I'll ignore the pipeline failure caused by java - keyvault - ci.

Hi, @saragluna , please help to review this PR.

@pallavit

Hi, @saragluna .
FYI:
When I do the patch releases, I didn't use the same branch.
In this PR, the target branch(azure-spring-boot_3.6.x) is checked out from the azure-spring-boot-starter-active-directory_3.6.1 tag,
it does not contain the content of azure-spring-boot-starter-keyvault-certificates_3.0.1.
So I added it manually in the latest commit.


Dreamview's SimControl separated into its own module

As of 099f135a464f85b10599040073806fcb6b2d6c25, Apollo's open source simulator is located in dreamview/backend/sim_control. It would be nice if it was in its own module. That way it can be started and stopped without a UI. This feature is useful for process level, real-time integration frameworks.
I know there are a few solutions to running dreamview without a UI in https://github.com/ApolloAuto/apollo/issues/6795. But it seems simpler to decouple the simulation from the visualizer.
Also, the dreamview UI should probably be changed at the same time. Instead of having the SimControl button on the front page, it should be started like the other programs. There can be another config located in modules/dreamview/conf/hmi_modes/simulation.pb.txt.

@Fla3inH0tCheet0s
That's a good suggestion.
The dreamview/backend/sim_control is PerfectControlModel.
Do you know where is the Control-in-loop Simulation?
@natashadsouza Dynamic Model - Control-in-loop Simulation

Hi
@xmyqsh do you find out where is the Control -in-loop Simulation ?
@natashadsouza  the announce of apollo that mention r5.0 have the feature of it ? but it seems that i can find out any instruction of it ?

control-in-loop can be enabled on Apollo Simulation Service http://apollo.auto/platform/simulation.html

@unacao
Got it. Thanks.

So,it is invisible for us?
Can I find anything about the model in this repo?

We provide two dynamic models: Echo and Intelli. Echo is a rule-based well-tuned dynamic model for Lincoln MKZ, and offered by third-party via a license. Due to the license restrictions, we are not able to disclose the details. Intelli is a dynamic model trained by machine learning methods. The corresponding paper will be available on IROS 2019.

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


new transaction is not allowed because there are other threads running in the session

I am getting this error even though I am using asynchronous methods.
Here is the code where I am calling the method. I am getting error when I am trying to save Resource entity.
                    {
                        CultureId = resource.CultureId,
                        ResourceCategoryId = resource.ResourceCategoryId,
                        Key = resource.Key.Trim(),
                        Value = resource.Value.Trim(),
                        CreatedDate = DateTime.UtcNow,
                        LastUpdated = DateTime.UtcNow
                    };

                    await _resourceService.Add(newResource);
                    await _unitOfWork.SaveAsync();

                    var oldResource = new Resource();
                    var resourceAuditLogModel = new AuditLogModel
                    {
                        AuditActionType = AuditActionType.Create,
                        KeyField =  newResource.Id.ToString(),
                        OldObject = oldResource,
                        NewObject = newResource,
                        LoggedUserEmail = loggedUser.Email,
                        ComparisonType = ComparisonType.ObjectCompare
                    };
                    await _auditLogService.Add(resourceAuditLogModel);
                    await _unitOfWork.SaveAsync();

Here is the how I have SaveAsync() method
public async Task SaveAsync()
        {
            try
            {
                _transaction = _baseContext.Database.BeginTransaction();
                await _baseContext.SaveChangesAsync();
                _transaction.Commit();
            }
            catch (Exception ex)
            {
                _transaction.Rollback();
                throw;
            }
            finally
            {
                _transaction.Dispose();
            }
        }

So, what happens is the error doesn't occur on record for first. But It occurs in next time when we save the new record.
Here is the screenshot of the stack trace.
http://prntscr.com/ha8fwx
Further technical details
EF Core version: 2.0
Database Provider: (Microsoft.EntityFrameworkCore.SqlServer)
Operating system:
IDE: (e.g. Visual Studio 2017)

@supernarwen Can you post the full stack trace as well? How is _baseContext being scoped. Is it possible that it is being shared by different threads?

Here is the full stack trace of the error.
at System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
 at System.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
 at System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
 at System.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
 at System.Data.SqlClient.TdsParser.Run(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj)
 at System.Data.SqlClient.TdsParser.TdsExecuteTransactionManagerRequest(Byte[] buffer, TransactionManagerRequestType request, String transactionName, TransactionManagerIsolationLevel isoLevel, Int32 timeout, SqlInternalTransaction transaction, TdsParserStateObject stateObj)
 at System.Data.SqlClient.SqlInternalConnectionTds.ExecuteTransactionYukon(TransactionRequest transactionRequest, String transactionName, IsolationLevel iso, SqlInternalTransaction internalTransaction)
 at System.Data.SqlClient.SqlInternalConnectionTds.ExecuteTransaction(TransactionRequest transactionRequest, String name, IsolationLevel iso, SqlInternalTransaction internalTransaction)
 at System.Data.SqlClient.SqlInternalConnection.BeginSqlTransaction(IsolationLevel iso, String transactionName, Boolean shouldReconnect)
 at System.Data.SqlClient.SqlConnection.BeginTransaction(IsolationLevel iso, String transactionName)
 at System.Data.SqlClient.SqlConnection.BeginDbTransaction(IsolationLevel isolationLevel)
 at System.Data.Common.DbConnection.BeginTransaction(IsolationLevel isolationLevel)
 at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.BeginTransactionWithNoPreconditions(IsolationLevel isolationLevel)
 at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.<BeginTransactionAsync>d__28.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
 at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
 at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
 at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
 at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.<BeginTransactionAsync>d__26.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
 at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
 at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
 at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
 at Lumle.Data.Data.UnitOfWork.<SaveAsync>d__4.MoveNext() in C:\Users\ekbana-niraj\Documents\Visual Studio 2017\Projects\lumle\Infrastructures\Lumle.Data\Data\UnitOfWork.cs:line 41

Here is how I have registered my baseContext
services.AddScoped(typeof(BaseContext));

@supernarwen It's certainly the case that you need to dispose the DbContext properly. Beyond that, the context is not thread safe, nor are the underlying ADO.NET objects like DbConnection. Can you check that you don't have multiple threads accessing these thing concurrently? If after that you're still seeing an issue, then please post a complete code listing or project that reproduces that issue.

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.

Hi,
I am experiencing same issue with EF Core 2.0 (dbProvider:Microsoft.EntityFrameworkCore.SqlServer) , the same code perfectly works using EF 6.2.
I am pretty sure that i don't have multiple threads accessing Context or Connection.
Can you give me some hints?
Regards
Luigi

@luigi68 Please file a new issue including a runnable project/solution or complete code listing that demonstrates the behavior you are seeing.

I have the same problem after upgrading to EF Core 2.1, at least for now in two different places, this code was running with no problems before upgrade from 2.0

Hi.  I had the same issue and fixed the problem as per shaher's solution.  i.e. two separate reads then union the result.


#11: Added verbosity user option

Resolves #11

Pull Request Test Coverage Report for Build 46

3 of 7 (42.86%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-10.9%) to 86.364%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

setuptools_pyecore/command.py
3
7
42.86%

Totals

Change from base Build 39:
-10.9%

Covered Lines:
26

Relevant Lines:
30

💛  - Coveralls

Pull Request Test Coverage Report for Build 46

3 of 7 (42.86%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-10.9%) to 86.364%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

setuptools_pyecore/command.py
3
7
42.86%

Totals

Change from base Build 39:
-10.9%

Covered Lines:
26

Relevant Lines:
30

💛  - Coveralls

Pull Request Test Coverage Report for Build 46

3 of 7 (42.86%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-10.9%) to 86.364%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

setuptools_pyecore/command.py
3
7
42.86%

Totals

Change from base Build 39:
-10.9%

Covered Lines:
26

Relevant Lines:
30

💛  - Coveralls


Mess with id's of headers

Hello.
# Title generates
Title

Well... This is very annoying if the page already have a tag with id="title".
Is it possible not to add those id's ?
For ever or just with a parameter.
Other suggestion: add a prefix to the id (for example "showdown-")
Thanks.

Related to #145 which is related to #82. Should be fixed once merged.

OK, Merci Pascal.


Add a basic functional test for the expected TrustSource report

Please have a look at the individual commit messages for the details.

FYI @gr-markin.


NUnit assertion to fluentassertions migration refactorings. Patterns Assert.AreEqual to Should().Be()

@mu88 @inyutin-maxim thanks for all your pull-requests. I finally merged all of them. Though I had to force-push a bit and add some minor cleanups (just out of convenience and some conflicts).  If everything is good, I will trigger a deployment.

Hey @inyutin-maxim @matkoch
All the tests from today's master are running fine on my Windows machine. From my side, we're ready to merge.

@mu88 @inyutin-maxim thanks for all your pull-requests. I finally merged all of them. Though I had to force-push a bit and add some minor cleanups (just out of convenience and some conflicts).  If everything is good, I will trigger a deployment. If not, just throw it back to me and I will take another look.

Good afternoon colleagues, forgive the late answer, I was on vacation and did not follow github)
I got all the changes from your master branch and checked all the tests on my Windows machine and they pass successfully


Add get_buffered_amount() to WebRTCDataChannel (GDNative)

This is a companion to these PRs:

Godot 4.x: https://github.com/godotengine/godot/pull/50658
Godot 3.x: https://github.com/godotengine/godot/pull/50659

This requires a minor change to godot-headers in order to compile:
diff --git a/net/godot_webrtc.h b/net/godot_webrtc.h
index 15e2df8..d5260cc 100644
--- a/net/godot_webrtc.h
+++ b/net/godot_webrtc.h
@@ -108,6 +108,13 @@ typedef struct {
        void *next; /* For extension? */
 } godot_net_webrtc_data_channel;

+/* Extensions to WebRTCDataChannel */
+typedef struct {
+       int (*get_buffered_amount)(const void *);
+
+       void *next; /* For extension? */
+} godot_net_webrtc_data_channel_ext;
+
 /* Set the default GDNative library */
 godot_error GDAPI godot_net_set_webrtc_library(const godot_net_webrtc_library *);
 /* Binds a WebRTCPeerConnectionGDNative to the provided interface */

As explained on the Godot 4.x PR, this has some backwards compatibility challenges. The godot_net_webrtc_data_channel struct changes, which will cause old version of the GDNative plugin to not work until updated, and also updated versions of the GDNative plugin won't work with older versions of Godot. So, I'm not sure how to handle that...
UPDATE: Backcompat is now handled per suggestion from @Faless!

Here's a PR to godot-headers that should allow this to actually build:
https://github.com/godotengine/godot-headers/pull/94

I think for now you can add that struct in this repository (without updating godot-cpp).
In src/net/WebRTCDataChannelNative.hpp:
#if GODOT_NET_WEBRTC_API_MAJOR == 3 && GODOT_NET_WEBRTC_API_MINOR < 4
extern "C" {
/* Extensions to WebRTCDataChannel */
typedef struct {
	int (*get_buffered_amount)(const void *);

	void *next; /* For extension? */
} godot_net_webrtc_data_channel_ext;
}
#endif

See my comment https://github.com/godotengine/godot/pull/50659#issuecomment-884276131

Alright, I added the conditional definition of the struct.

Woah, sorry, I forgot to merge this it seems :disappointed: .

Thanks!

Thanks!!


Schur and generalized Schue decomposition

This PR closes #5.

Codecov Report

Merging #6 (7bbec16) into main (296453c) will increase coverage by 0.34%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
+ Coverage   98.63%   98.97%   +0.34%
==========================================
  Files           1        1
  Lines          73       98      +25
==========================================
+ Hits           72       97      +25
  Misses          1        1

Impacted Files
Coverage Δ

src/DifferentiableFactorizations.jl
98.97% <100.00%> (+0.34%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[USM] Process event monitor filter only needed events

What does this PR do?
Process monitor will received exec,fork,comm,exit netlink events from the kernel, let's parse and and send only subscribed events to the channel.
Motivation
Avoid useless memory allocation
This include modification of our netlink library (fork from vishvananda
process monitor filter https://github.com/vishvananda/netlink/commit/f11d5ab0583826ab0f40eb30a7cfb8a5bdae61de

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Dashboard


Adding 2 Vim Interactive Games/Tutorials

What does this PR do?
Adding 2 Vim Interactive Games/Tutorial
For resources
The Vim Adventure Games - Learn Vim by playing adventure game, it's really fun game that force player to use Vim commands and shortcut keys to pass the levels.
Vim Genius - This a flashcard style game to help you learn the basics of Vim
Why is this valuable (or not)?
They are popular and fun games that help beginner find the enjoyment while learning Vim Editor
How do we know it's really free?
They are free to play
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

these will fit better into the playgrounds list

these will fit better into the playgrounds list

Thank you, I moved to the playground list instead.

On a whim I decided to try The Vim Adventure Games. After completing(?) level 3 a popup appears that indicates it costs $25 (game-only, or $30 with challenges) for a 6-month license. Not sure if it's playable beyond the popup.

@nclements thanks for checking that. It should be removed.


How to use MSAL.js 2.x (msal-browser) to implement authentication code (with PKCE) flow?

Hello, I am using angular9 and msaljs (V2.2.0), how to implement authentication code (with PKCE) flow, how to configure options?

@fengzhihenxs it is best practice to read the documentation https://github.com/AzureAD/microsoft-authentication-library-for-js and examples.
There you find a link for v2.x. https://github.com/AzureAD/microsoft-authentication-library-for-js/tree/dev/lib/msal-browser
There you also find an Angular example.

@fengzhihenxs In the future, please fill out the full issue template with config, versions, and usage.
Currently our angular library does not support the auth code flow with PKCE (version 2 of our library), and can only be used with version 1 (implicit flow). There's a PR containing the sample for angular with version 2 of the library in #2306.

Hi @jmckennon
I am in the middle of implementing MSAL in our Angular application. I've used 1.0 with implicit flow and have that figured out. But last night on Ignite a saw a session about 2.0.
Just to be clear, would I be better of using msal.js 2.0 directly (maybe inspired by the sample you link above) or should I just go for msal-angular 1.0 for now - and once you release msal-angular 2.0 I should be able to pretty easily switch?
And do you have a ETA/deadline for msal-angular 2.0 - it seems you are working heavily on the samples these days. A week, month, quarter or year?

I'm also interested in this, although I'm using React and not Angular. The README.md says:

Important: MSAL.js 2.0 with Authorization Code Flow is not yet available for B2C tenants (coming soon).

Is that still true? You can't use the library to implement the PKCE flow for B2C customers?

@AnkHansen I would recommend using angular 1.x for now. There may be a few breaking changes from 1.x to 2.x, but we will have a migration guide to make the change as simple as possible. You can expect to see a full<EMAIL_ADDRESS>in the next few months, but no release date as of today.
@karol-majewski It's possible to use B2C in development, but we don't recommend moving into prod with those apps yet. We are still (unfortunately, sorry for the delay) working out a few issues with B2C in our 2.0 library. Additionally, the msal-react library (supported by auth code flow with PKCE/2.0) is in the works for us, you can probably expect to see it early next year. As you mention, there is a react wrapper library but it uses the implicit flow and we also do not directly support/own it.

@karol-majewski Following up on the comment from @jmckennon. We are trying to use Azure AD B2C + Custom Policies + PKCE flow etc. for local, AAD and Google identity providers. Any luck using msal-browser?

@TissonMathew I ended up building my own component around @azure/msal-browser inspired by msal-react-v2.

Thanks @karol-majewski Does it support Azure AD B2C + Custom Policies + MSAL.js 2.0 + Authorization Code flow?
I see another issue posted but its on closed status -- https://github.com/AzureAD/microsoft-authentication-library-for-js/issues/2271

@TissonMathew It definitely works with Azure AD B2C + Authorization Code flow.
The docs at msal-browser say it doesn't support B2C:
Important: MSAL.js 2.0 with Authorization Code Flow is not yet available for B2C tenants (coming soon).

but it's not true — it does work. How do I know? The API call made to https://login.microsoftonline.com contains the response_type=code part, which indicates that the PKCE flow is used. I think the docs are just outdated.
My solution, just like msal-react-v2, depends on<EMAIL_ADDRESS>which in turn uses<EMAIL_ADDRESS>I don't use custom policies, so I can say anything about this part.

@karol-majewski

The docs at msal-browser say it doesn't support B2C:
Important: MSAL.js 2.0 with Authorization Code Flow is not yet available for B2C tenants (coming soon).
but it's not true — it does work. How do I know? The API call made to https://login.microsoftonline.com contains the >response_type=code part, which indicates that the PKCE flow is used. I think the docs are just outdated.

B2C + MSAL.js v2 does work but is not yet considered production ready due to a few outstanding issues on the server, which is why the docs have not been updated yet. You are free to use MSAL.js v2 with B2C in development, we are just recommending you do not deploy your app to production until we announce that B2C is fully supported.
@TissonMathew

Does it support Azure AD B2C + Custom Policies + MSAL.js 2.0 + Authorization Code flow?
Custom policies will be supported at the same time as the built in flows. There is no difference as far as MSAL.js is concerned.

Thank you @tnorling what's the ETA on B2C official support?

@TissonMathew Due to the fact that the work needed is server-side and deployments can be delayed for a number of reasons I'm unfortunately not able to provide an ETA. What I can say is that supporting B2C with Auth Code + PKCE is one of our top priorities and the server team is working to get the necessary fixes rolled out as quickly as they can.

I am able to setup a sample application with MSAL 2.0 + Azure AD B2C + Custom polices, but getting a issue with aquireTokenSlient when login with Azure account
https://github.com/AzureAD/microsoft-authentication-library-for-js/issues/2364

I encountered a different behavior when using B2C that quickly gave the impression that it doesn't fully support the Auth Code Flow with PKCE. But in the end I managed to find out.
I'm using the latest @azure/msal-browser version 2.2.1.
We first used https://login.microsoftonline.com/common/ to test the Auth Code Flow with PKCE. Everything worked fine.
When I switched to our corporate's Azure AD B2C, the login was still fine but I was missing the AccessToken.
I only had the IdToken and RefreshToken, but no AccessToken.
I figured that behind the scenes the library always added the User.Read scope when using https://login.microsoftonline.com/common/, but this was not automatically the case when using our AAD B2C.
When I tried adding this scope also for the B2C, I got the following error:
"AADB2C90117: The scope 'User.Read' provided in the request is not supported.

However, this difference in the scopes got me in the right direction, because I first had to add a custom scope for our exposed API to our app. Only after doing that I got the AccessToken upon login.

This somehow makes sense in the end, but I think this behavior might not be obvious at first when you just want to try out B2C with the Auth Code Flow and PKCE.
Long story short: It doesn't automatically return the AccessToken unless you explicitly request permission to one of your APIs.

Since the open issues on this issue are resolved, we'll be closing this. Support for B2C in msal-browser is our top priority and can be tracked on #2381


JSON indexing adapter

This makes the process of indexing JSON documents in a forward index pluggable, so the forward index doesn't need to store the documents as strings.

Codecov Report

Merging #7876 (2ada9ae) into master (c999a23) will decrease coverage by 50.67%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master    #7876       +/-   ##
=============================================
- Coverage     65.07%   14.40%   -50.68%
+ Complexity     4082       80     -4002
=============================================
  Files          1538     1541        +3
  Lines         79993    80010       +17
  Branches      12035    12034        -1
=============================================
- Hits          52057    11523    -40534
- Misses        24219    67638    +43419
+ Partials       3717      849     -2868

Flag
Coverage Δ

unittests1
?

unittests2
14.40% <0.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


custom album artwork

idea:
Can we create a new env flag that would allow for a custom album artwork.
use case:
If I don't want to use the artwork from youtube but I still wanna upload my own picture
This could be even more useful if I use the same picture for every episode...

I'm thinking of a flag custom_artwork
together with users saving the picture as thumbnail.jpg
behind the scenes the script will just skip downloading the artwork from youtube and then use the custom artwork saved as  thumbnail.jpg
and obviously will be documented in the readme

Is this something that are you interested in working with?

Nice idea, only thing remains is if we can find a better name for the flag custom_artwork.
Currently, we have env variable LOAD_THUMBNAIL.
What if we define a new flag called THUMBNAIL_SOURCE and it will have as value either VIDEO or FILE
And then we can have another flag THUMBNAIL_FILE which can then be configured as thumbnail.jpg.
What do you think?

I agree with your vision