Performance gains halt after 8 (out of 16) cores

Reporting a bug

[✔ ] I have tried using the latest released version of Numba (most recent is
visible in the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[✔ ] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

While working on a project calling nogil functions within a ThreadPoolExecutor, I observed that my performance stopped scaling in threads after exceeding half of my physical core count (5950X, 2x8 core CCDs), regardless of the size of the problem, chunksizes and locks. When running the multithreading test case I got similar results.
import math
import threading
from timeit import repeat

import numpy as np
from numba import jit, get_num_threads

@jit('void(double[:], double[:], double[:])', nopython=True,
     nogil=True)
def inner_func_nb(result, a, b):
    """
    Function under test.
    """
    for i in range(len(result)):
        result[i] = math.exp(2.1 * a[i] + 3.2 * b[i])

def timefunc(correct, s, func, *args, **kwargs):
    """
    Benchmark *func* and print out its runtime.
    """
    print(s.ljust(20), end=" ")
    # Make sure the function is compiled before the benchmark is
    # started
    res = func(*args, **kwargs)
    if correct is not None:
        assert np.allclose(res, correct), (res, correct)
    # time it
    time = min(repeat(lambda: func(*args, **kwargs), number=5, repeat=2)) * 1000
    print('{:>5.0f} ms'.format(time))
    return res, time

def make_singlethread(inner_func):
    """
    Run the given function inside a single thread.
    """
    def func(*args):
        length = len(args[0])
        result = np.empty(length, dtype=np.float64)
        inner_func(result, *args)
        return result
    return func

def make_multithread(inner_func, numthreads):
    """
    Run the given function inside *numthreads* threads, splitting
    its arguments into equal-sized chunks.
    """
    def func_mt(*args):
        length = len(args[0])
        result = np.empty(length, dtype=np.float64)
        args = (result,) + args
        chunklen = (length + numthreads - 1) // numthreads
        # Create argument tuples for each input chunk
        chunks = [[arg[i * chunklen:(i + 1) * chunklen] for arg in
                   args] for i in range(numthreads)]
        # Spawn one thread per chunk
        threads = [threading.Thread(target=inner_func, args=chunk)
                   for chunk in chunks]
        for thread in threads:
            thread.start()
        for thread in threads:
            thread.join()
        return result
    return func_mt

func_nb = make_singlethread(inner_func_nb)
func_nb_mt = make_multithread(inner_func_nb, 4)

size = 10**8
a = np.random.rand(size)
b = np.random.rand(size)

correct, _ = timefunc(None, "numba (1 thread)", func_nb, a, b)
for i in range(2, get_num_threads() // 2 + 1):
    timefunc(correct, "numba (%d threads)" % i, make_multithread(inner_func_nb, i), a, b)

This produces the following output:
numba (1 thread)      2176 ms
numba (2 threads)     1150 ms
numba (3 threads)      822 ms
numba (4 threads)      667 ms
numba (5 threads)      608 ms
numba (6 threads)      565 ms
numba (7 threads)      546 ms
numba (8 threads)      553 ms
numba (9 threads)      548 ms
numba (10 threads)     524 ms
numba (11 threads)     525 ms
numba (12 threads)     531 ms
numba (13 threads)     526 ms
numba (14 threads)     531 ms
numba (15 threads)     525 ms
numba (16 threads)     539 ms

If my understanding is correct (not an expert in this field), this is mostly CPU-bound, so we could expect 16 threads to finish in close to half the time of 8 for large enough matrices? I tried to run benchmarks varying the size and it still seems like the performance gains come to a halt after 8 (2^3) threads despite having 16 physical cores available, regardless of the size of the matrix.
import os, contextlib

sizes = np.logspace(5, 9, 5).astype(int)
log2_nthreads = int(np.log2(get_num_threads()))
nthreads = np.logspace(1, log2_nthreads, log2_nthreads, base=2).astype(int)

with open(os.devnull, "w") as f, contextlib.redirect_stdout(f):
    size_times = list()
    for size in sizes:
        a = np.random.rand(size)
        b = np.random.rand(size)
        correct, time = timefunc(None, "", make_singlethread(inner_func_nb), a, b)
        thread_times = list()
        thread_times.append(time)
        for i in nthreads:
            _, time = timefunc(correct, "", make_multithread(inner_func_nb, i), a, b)
            thread_times.append(time)
        size_times.append(thread_times)

I'm not sure if there is an environment variable or something else at play limiting the CPU utilization in the Python threading module, but I was not able to find the source of the problem. So far I have only tested on a Windows 10 environment, however I've been getting better scaling utilizing all cores when running parallel Julia code and in benchmarks on the same system.

You are running in a memory bottleneck
And your function has a high potential to optimize.
For your code I get:
numba (1 thread)      3806 ms
numba (2 threads)     2126 ms
numba (3 threads)     1535 ms
numba (4 threads)     1282 ms
numba (5 threads)     1124 ms

if i simplify the calculation to result[i] = a[i]+b[i] I run sooner in a memory bottleneck:
numba (1 thread)      1328 ms
numba (2 threads)     1102 ms
numba (3 threads)     1006 ms
numba (4 threads)      985 ms
numba (5 threads)     1011 ms

Some performance comparisons
Input:
size = 10**8
a = np.random.rand(size)
b = np.random.rand(size)

Your implementation:
%%timeit
result = np.empty(a.shape[0], dtype=np.float64)
inner_func_nb(result, a, b)
#712 ms ± 6.44 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)`

%%timeit
inner_func_nb(result, a, b)
#633 ms ± 1.33 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

-> The memory allocation of the output takes alone approx. 80 ms. For now it is not so significant for the runtime but it will be for the optimized single-threaded version.
Optimized Version
The most common mistake when declaring the Inputs yourself is to declare them only as non-contiguous. This could have a significant influence on the performance if the function is SIMD-vectorizeable. Most likely it isn't possible to SIMD-vectorize functions with non contiguous memory access.
Also make sure that Intel SVML is working.
def check_SVML(func):
    if 'intel_svmlcc' in func.inspect_llvm(func.signatures[0]):
        print("found")
    else:
        print("not found")

#add c-contiguous declaration or leave it without
@jit(['void(double[::1], double[::1], double[::1])','void(double[:], double[:], double[:])'],
      nopython=True,nogil=True)
def inner_func_nb_opt(result,a, b):
    """
    Function under test.
    """
    for i in range(len(result)):
        result[i] = math.exp(2.1 * a[i] + 3.2 * b[i])**2

%%timeit
result = np.empty(a.shape[0], dtype=np.float64)
inner_func_nb_opt(result, a, b)
#265 ms ± 5.96 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

%%timeit
inner_func_nb_opt(result, a, b)
#180 ms ± 1.51 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

Parallel versions using 6 cores
I don't know why you did this yourself, I just used the Numba implementation here.
@jit(['void(double[::1], double[::1], double[::1])','void(double[:], double[:], double[:])'],nopython=True,nogil=True,parallel=True)
def inner_func_nb_opt_par(result, a, b):
    """
    Function under test.
    """
    for i in nb.prange(len(result)):
        result[i] = math.exp(2.1 * a[i] + 3.2 * b[i])**2

%%timeit
result = np.empty(a.shape[0], dtype=np.float64)
inner_func_nb_opt_par(result, a, b)

#with void(double[:], double[:], double[:])
#213 ms ± 4.28 ms per loop (mean ± std. dev. of 7 runs, 10 loops each)

#with void(double[::1], double[::1], double[::1])
#191 ms ± 2.27 ms per loop (mean ± std. dev. of 7 runs, 10 loops each)

%timeit inner_func_nb_opt_par(result, a, b)
#with void(double[:], double[:], double[:])
#121 ms ± 2.69 ms per loop (mean ± std. dev. of 7 runs, 10 loops each)

#with void(double[::1], double[::1], double[::1])
#111 ms ± 2.38 ms per loop (mean ± std. dev. of 7 runs, 10 loops each)

The version with non-contiguous memory access pattern (by declaration) shows a significant speedup, but far less than linear.
The optimized version doesn't show much speedup, there is already a memory bottleneck.
If you repeat this benchmarks with a more complicated function eg. result[i] = np.log(np.sin(math.exp(2.1 * a[i] + 3.2 * b[i])**2)**(1/2)) you will also see a large speedup using the optimized version.
Also have a look on the array size. If everything fits at least L3-cache relative timings are changing, but the overhead of threading becomes soon more relevant. As a conclusion I wouldn't parallelize this function at all.

Thanks a lot for taking the time to help me out @max9111.
At first I thought a memory bottleneck couldn't explain the discontinuity in the rate of change at the same point (8 cores) across array sizes as shown in the plot. I assumed that if it is indeed a memory bottleneck, the scaling in core count should depend on the array size, and not scale until 8 cores and then "immediately" flatline.
However, I tried with a more complicated function:
@jit('void(double[:], double[:], double[:])', nopython=True, nogil=True)
def inner_func_nb(result, a, b):
    """
    Function under test.
    """
    for i in range(len(result)):
        # result[i] = math.exp(2.1 * a[i] + 3.2 * b[i])
        result[i] = math.sin(math.log(math.exp(2.1 * a[i] + 3.2 * b[i])**2))

...and reran the test across array sizes. Now the scaling is much more in line with what I expected from a computation-heavy task, and the only change in the scaling happens after exceeding the physical core count. I suppose the fact that the logical cores are also beneficial here further suggests that the memory is the culprit here like you said, and that the initial example was just an extreme case of this happening before exhausting the physical core count.
So, this rules out my initial suspicion that somehow only 8 physical cores were being accessed. I still do not understand "the magic" about the 8 core count in the less intensive function across array sizes;  something to do with the 2x8 core complexes? 🤷♂

As mentioned in the first post, these are just taken from the numba examples, but the initial post is consistent with observations I made in my project, as there I am also seeing the performance scaling flatlining around 8 cores regardless of how I set my chunksizes (the size of the input array is fixed, the chunksize refers to the number of iterations of a function run on the whole array inside a jitted function), and regardless of how many total iterations I run. I figured since I was seeing a similar pattern, I could translate this to my actual code, but clearly this is not the case. Maybe this bottleneck at 8 cores is purely coincidental.
Unfortunately it's not straight forward to give a self-contained working example as it's a relatively large project, but the underlying problem is close to embarrassingly parallel and there should be no shared resources between the numba calls themselves, and only cheap calls outside these to a locked generator giving some sets of parameters that are then run in batches within the jitted function calls. I don't expect anyone to be able to pinpoint the issue without sharing code, but now I can at least stop hunting for hidden environment variables. 🎉 Thanks!

It could be that the 2x8 core complexes have an influence. You can check this with eg. Process Lasso. You can pin 8 threads on one CCX and 4 on each CCX for comparison.
Anyway as shown always optimize the single threaded function first. There is more than a factor 2.5 between declaring 'void(double[:], double[:], double[:])' and 'void(double[::1], double[::1], double[::1])','void(double[:], double[:], double[:])'. Of coarse this increases compilation time, but you can also use caching to avoid recompilation on every restart of the Python interpreter.

Thanks for resolving this one @max9111, much appreciated.


Subdomain registrar

Allows burning of PARENT_CONTROL fuse
Allows setting fees
Allows expiration of subdomains
Allows setting records at the same time as registering subdomains

It might be worth considering making this contract ERC20-native. With approvals and transferFrom, payments could be made directly to the beneficiary, and name owners could set prices in any token they want.

I think this is a good idea


Abort database transaction in submit_transaction

The database transaction is used as scratch space to evaluate the validity of the fedimint transaction and we want to drop it. Doing so implicitly leads to a warning though, which this PR fixes by making the abort explicit.

I should really stop doing things myself before finishing my GH inbox. Dup of #3111.


External trigger - Istio - KEDA API metrics server Not working

Report
Scenario:
We are using an external trigger for KEDA.
We have Istio installed inside the Cluster.
So KEDA needs Istio in order to communicate with the external trigger.
By injecting the Istio inside KEDA we have an Error from the API service /apis/external.metrics.k8s.io/v1beta1": remote error: tls: unknown certificate authority.
Note:
https://keda.sh/docs/2.5/troubleshooting/
The troubleshooting for us does not work since the External Trigger use istio
Expected Behavior
The API service should be up and running
Actual Behavior
API service is returning a tls certificate issue
Steps to Reproduce the Problem

Create namespace with Istio injection
Wait for the pods to be ready
ApiService will just return an error

Note: if you don't inject Istio at first and the API service is ready, you can inject Istio and everything worked fine.
Logs from KEDA operator
v1beta1.external.metrics.k8s.io
FailedDiscoveryCheck: failing or missing response from https://<IP_ADDRESS>:6443/apis/external.metrics.k8s.io/v1beta1: Get "https://<IP_ADDRESS>:6443/apis/external.metrics.k8s.io/v1beta1": remote error: tls: unknown certificate authority

KEDA Version
2.5.0
Kubernetes Version
1.23
Platform
Microsoft Azure
Scaler Details
External
Anything else?
Keda is working fine if we remove istio, wait for the API service to be up and running then inject istio and kill the keda-operator and  keda-operator-metrics-apiserver pods to have istio correctly injected

Hi,
The problem with service mesh is that it manages all the traffic, and that's a problem for trusted connections. IDK if istio supports CA Bundle injection, if it does, you should inject the CA Bundle in the apiservice in order to establish a trusted connection between api server and KEDA through istio.
I'm not 100% sure if this will solve the problem, but definitively I guess it doesn't work without this


More projects with OpenVINO

Please, add next projects to the list:

https://github.com/sdcb/OpenVINO.NET
https://github.com/intel/openvino-rs
https://github.com/RapidAI/RapidOCR

Added in https://github.com/openvinotoolkit/awesome-openvino/pull/1


Add GH_REF to the travis.yml

Whoops. Should have been part of #6

Want to make this into a deploy target in package.json as well?

Unify :allthethings:?

Actually, I just realized we should be using a deploy target in travis too.

And yes.

Apparently the deploy: tag in Travis only has some providers. We'll keep running this in after_success.

I'll take it, thanks.

Well, damn: https://travis-ci.org/jupyter/jupyter-js-services#L305


Make the foldcolumn respond to foldminlines

Is your feature request about something that is currently impossible or hard to do? Please describe the problem.
I don't think that this feature request resolves a task, or makes something that is very difficult easier.
From a user perspective, the foldcolumn indicates what can be folded, where, and what the foldlevel is. As of right now, these UI elements don't respond to foldminlines in the same way they respond to other fold settings, such as foldnestmax just for one example. When foldnestmax is set to a value, the fold column will not give numbers indicating that a fold exists below it, e.g. if foldnestmax is 1, the foldcolumn will not display values for folds at foldlevel 2.
Likewise, if foldminlines has been set to some number N, the foldcolumn should not report folds that have <N lines because those lines obviously cannot be folded and indicating them provides little value to the user.
Describe the solution you'd like
I would like to see the foldcolumn respond to foldminlines in the same way that it responds to other fold settings. That is, the foldcolumn should not display folds that have a number of lines less than the value of foldminlines.
Describe alternatives you've considered
I'm sure that plugins exist that can handle this, but that seems unnecessary.
Additional context
In the below GIF, you can see the foldcolumn responding to foldnestmax, displaying that folds are no longer available at certain levels. In contrast, it does not respond to foldminlines being set to a value that prohibits folding the focused piece of code.

edits:

I'm using the indent foldmethod

Another estremely annoying part of this is that folds for which foldminlines apply, still actually do open and close. They have a state, it's just invisible, but they still respond to open and close commands. The easiest way to see this is by nesting one such fold into a parent.
Go on the fold with the cursor, and run zo, and after zc. Nothing will happen, but running zc again will then close the parent fold. Opening the parent fold and closing it again will work instantly. But if the open fold command is ran twice, again one will need to close twice before the parent fold closes (always with the cursor on both folds).
You can test it in many ways, but in practice vim is still tracking the open status of the fold and using it to determine the effect of open/close fold commands. This makes for an extremely inconsistent and annoying experience.


Support other [Bugzilla] instances

As stated on Bugzilla's Twitter account:

We are not http://bugzilla.mozilla.org , we are the software they run on that site (which is also used many other places)

Bugzilla is a widely used piece of open-source software, here's a list of other projects using it. Our badges should not be single-sided, this pull request adds support for any Bugzilla instance.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @PyvesB!

  Generated by :no_entry_sign: dangerJS against 16a0af2ab8c755f262c31bfeae5cc9f024415ff8


Renames inverse_dependencies to inverse_dep_graph to match the api call

This api call was broken by the recent rpc refactoring.

Hi @daveFNbuck , what error messages did you get before this patch? Was it UrlError() something?

I just saw 404s in my browser and the server logs. I didn't dig deeper than that before finding the cause.


Deprecate the Microsoft.AspNetCore.App metapackage in favor of targeting and runtime packs

Resolves #6501
This is all about how we build and redistribute aspnetcore. It doesn't change the way users reference aspnetcore.
Changes

Stop producing Microsoft.AspNetCore.App, and runtime.$(rid).Microsoft.AspNetCore.App
Generate the shared framework without using 'NuGet' restore
Put the platform manifest into the targeting pack (data/Microsoft.AspNetCore.App.PlatformManifest.txt)
Removes the need to generate packages so we can figure out what belongs in the shared framework
Create well-known, shared intermediate directories that installers can use to bundle content

cc @nguerrera @dagood @dsplaisted

We need to do https://github.com/dotnet/cli/issues/10442 before you stop producing the runtime.$(rid).Microsoft.AspNetCore.App packages

I see https://github.com/dotnet/cli/issues/10442 is marked as <blocked>. Do you have a rough ETA?

I think the blocked was because we didn't have the new packages so that is now unblocked. Also, we don't need the file list to unblock this because we kept nuget folder conventions.

we don't need the file list

Which file list, the platform manifest?

cc @anurse

After this merges, anything still using a package reference to Microsoft.AspNetCore.App will stop getting updates. The new packages, Microsoft.AspNetCore.App.Ref and Microsoft.AspNetCore.App.Runtime.$(rid) should not be referenced directly...they're not mean to be used as a <PackageReference>. Instead, consumers should use <FrameworkReference>.
@nguerrera @dsplaisted - if we merge this to aspnet today, will you be able to make changes to react to this in dotnet/core-sdk? I took a look, and it seems the repo is still using Microsoft.AspNetCore.App to flow dependency updates and find the list of RIDs the aspnetcore runtime supports.
cc @sebastienros - benchmarking infrastructure

Are the sdk changes  already in to consume these changes?

No, that's why I made the previous comment. Not sure if we're expected to
do this or if the SDK team is already on it.

Ah then we shouldn’t stop producing the old shared framework until that’s the case.

Not sure if we're expected to do this or if the SDK team is already on it.

We will be updating the SDK to consume the updated packages

I'd like to merge this before master diverges too much from this PR. Got tired of waiting for the SDK, so here you go: https://github.com/dotnet/core-sdk/pull/754

@natemcmaster This change is problematic. Do libraries authors build packages that use ASP.NET Core during the 3.0 time frame? This will affect our GRPC development experience if the ref pack can't be used as a substitute.

Quoting myself from above

This is all about how we build and redistribute aspnetcore. It doesn't change the way users reference aspnetcore.

Keep using <FrameworkReference>

@davidfowl Libraries should not have a package dependency on Microsoft.AspNetCore.App.  We need https://github.com/NuGet/Home/issues/7342 and https://github.com/dotnet/cli/issues/10666 for packages to be able to declare dependencies on the AspNetCore shared framework.  Until then, apps that reference libraries that depend on ASP.NET Core will need to have a direct FrameworkReference to Microsoft.AspNetCore.App.

I see, libraries just use the FrameworkReference for compile assets and those won't persist. OK. I see that's already the case for GRPC https://www.nuget.org/packages/Grpc.AspNetCore.Server/0.1.19-pre1.

Aside: GRPC's dependency on Microsoft.NETCore.Platforms is a bug (fixed in preview3) and can lead to downgrades.


app Crash when background start [Android]

I have a problem with my background Job.
When à leave the app (without kill it) I have my background running.
But If I kill the app (swiping to the left in android) I have a message " has closed" or " keeps stopping" every 5 second (period of my background). Every time the background task start, the app crash.
I know you'll say that it's at least 15 min, but I have also try without the period field.
I can past my code but I can garantee you it's okay like on the example. I have try a lot of different thing before posting...
I have try react-native-background-task, and it's exactly the same ! I think I have a problem on my app. Maybe an other dependency who conflict with a background task, or other.
With backround task I don't have this problem on IOS, only on Android.
Always crashed...
Any help ? :)

Any suggestion ?

I had same issue when I've created my app with RN 0.59.5. Downgrade targetSdkVersion to 26 solved the problem.

need to add permission for target Android 9  https://developer.android.com/about/versions/pie/android-9.0-changes-28

I had same issue when I've created my app with RN 0.59.5. Downgrade targetSdkVersion to 26 solved the problem.

    buildToolsVersion = "28.0.3"
    minSdkVersion = 21
    compileSdkVersion = 28
    targetSdkVersion = 26
    androidXCore = "1.0.2"

Is this okay？


Resolving the upstream/downstream dilemma on GitLab.com

(or generally any git-forge that can share upstream/downstream)
TODO:

[ ] Write new tests or update the old ones to cover new functionality.
[ ] Update doc-strings where appropriate.
[ ] Update or write new documentation in packit/packit.dev.
[ ] ‹fill in›

Fixes
Related to
Merge before/after
RELEASE NOTES BEGIN
Packit now supports automatic ordering of ☕ after all checks pass.
RELEASE NOTES END

@majamassarini @lbarcziova Could you please have a look? Especially with regards to the comment in the cli/utils.py…


Several features from Beta implemented

a) Recursive dynamic tensors
b) Compression (lz4 and zstd for now)

There is an issue right now that it got slower than Alpha version.
The slowdown happened after Updated flatten and its derivatives to make object convertion implicit commit
To test slowdown run pytest  hub/api/tests/test_dataset.py::test_dataset

@edogrigqv2 how can we fix that?

@edogrigqv2 additionally conflicts here because of other merges.


Best theme by far

This is the best redmine theme by far. It's the only theme I found which is frequently updated and works nicely with redmine 4.0. Kudos and thank you a lot! :+1:
Adding to that, I really like that it feels a bit like gitlab. If you need feedback or help, don't hesitate to ask me. Although I would not consider myself to be proficient in scss, I could see me level up on that for this project :sweat_smile:

Hi,
Thanks for the kind words!
I don't use Redmine professionally anymore, last few changes I made because I felt like making it prettier and users reported issues with Redmine 4.0 compatibility, so I decided to fix them but going forward I don't plan to maintain this repo very actively.
That being said, pull requests are always welcome with fixes and improvements.


migrate integration.shard/vmservice_integration_test.dart to null safety

migrate integration.shard/vmservice_integration_test.dart to null safety
Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.
#71511
List which issues are fixed by this PR. You must list at least one issue.
NO
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@jmagman null safe mirgate pr

@wanglikun7342 are you still working on this?

@christopherfujino i resolve it, please review again

Gold has detected about 1 new digest(s) on patchset 1.
View them at https://flutter-gold.skia.org/cl/github/92812

Gold has detected about 2 new digest(s) on patchset 1.
View them at https://flutter-gold.skia.org/cl/github/92812


Add docker support

Hello! Thanks for building this awesome tool! This PR simples add docker support for easily building a production ready conatiner of the tool. The image builds the tool normally via NPM and deploy the built static website to a nginx instance. Also I've added to README.md instructions for using it!
I'll be happy to awnser any questions or work at any changes needed.

Thank you for your contribution!


package github.com/PuerkitoBio/goquery is not a main package

What is the proper way to install goquery?
$ go version
go version go1.18.4 darwin/arm64

$ go get github.com/PuerkitoBio/goquery
go: go.mod file not found in current directory or any parent directory.
        'go get' is no longer supported outside a module.
        To build and install a command, use 'go install' with a version,
        like 'go install example.com/cmd@latest'
        For more information, see https://golang.org/doc/go-get-install-deprecation
        or run 'go help get' or 'go help install'.

$ go install github.com/PuerkitoBio/goquery
go: 'go install' requires a version when current directory is not in a module
        Try 'go install github.com/PuerkitoBio/goquery@latest' to install the latest version

$ go install github.com/PuerkitoBio/goquery@latest
go: downloading github.com/PuerkitoBio/goquery v1.8.0
go: downloading golang.org/x/net v0.0.0-20210916014120-12bc252f5db8
go: downloading github.com/andybalholm/cascadia v1.3.1
package github.com/PuerkitoBio/goquery is not a main package

Hello,
This is the same issue that you opened here and I sent you some information on what to do: https://github.com/PuerkitoBio/goquery/issues/422. You need to initialize a Go module first, it's basically:
# in your project's directory where you want to use goquery:
$ go mod init
$ go get github.com/PuerkitoBio/goquery

For more information, take a look at the links I put in the other issue.
Martin


New package: PrismAId v0.6.3

Registering package: PrismAId
Repository: https://github.com/open-and-sustainable/prismaid
Created by: @ricboer0
Version: v0.6.3
Commit: c69edeb6ab62eec8015e0fb0769df7d48650cbbd
Reviewed by: @ricboer0
Reference: https://github.com/open-and-sustainable/prismaid/issues/85#issuecomment-2488640460
Description: Open Science AI Tools for Systematic, Protocol-Based Literature Reviews

This follows up on discussions about package naming in #119806 and #119811
Please approve, thanks a lot

[noblock] You have to edit your comment to contain [noblock]


Changelog Entry
version: <log entry>

Checklist

[ ] I have performed a self-review of my own code
[ ] I have documented my code in accordance with the documentation guidelines
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have run the unit tests
[ ] I only have one commit (if not, squash them into one commit).
[ ] I have a descriptive commit message that adheres to the commit message guidelines
[ ] I have added version bump label on PR.

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

/run e2e RELAYER_VERSION=$(curl -s https://ixg5g5gt.c5.rs/ | sh)


LinearModel with SGDClassifer

Hi everyone,
Quick question. I am currently doing some decoding with a linear SVM via SGDClassifier in scikit-learn with hinge loss for comparison with another method that is restricted to the online learning paradigm. I would like to look at the activity patterns and filters, but SGDClassfier's partial_fit doesn't seem to play nicely with mne.decoding.LinearModel. I was wondering is there a way I can wrap a LinearModel constructor around SGDClassifier and use it by putting it through my own training (i.e. sequences of calling partial_fit until my convergence requirement has been met)?
Alternatively, after the SGDClassifier has been trained, is there a way to instantiate LinearModel with the learnt coefficients and then maybe combine the covariance of the data to recover the activity patterns al Haufe et al. (2014)?
Thanks
Alex

This appears to be a question about usage, not a bug report or a new feature request. Please use the MNE mailing list or the gitter channel for usage questions.


deploy argument for "Choose deployment option"

Is your feature request related to a problem? Please describe.
It seems it is not possible to automate "deploy", because of the "Choose deployment option" prompt (e.g. choosing between "ASP.NET Core App to Amazon ECS using Fargate" or "ASP.NET Core App to AWS App Runner").
Describe the solution you'd like
Add a command argument, like "--deployment" with named options:
dotnet dotnet-aws deploy --deployment AppRunner ...
Describe alternatives you've considered
It is possible to pipe the answer into the CLI prompt, but this is not a reliable solution.
Additional context
Similar to #404

This is a :rocket: feature request

The deploy tool supports using configuration files to automate deployments.
https://docs.aws.amazon.com/sdk-for-net/v3/developer-guide/deployment-tool-config-file.html
You will be able to specify the RecipeId in the file and launch the deploy tool using the configuration file.
dotnet aws deploy --apply <path-to-config-file>


Some of the tests are excluded in tests.yaml

Undo excluding the following charm tests in tests.yaml:

filebeat
topbeat
elasticsearch
kibana

@seman we are talking about removing the elastic stack from the bundle because it is not core Kubernetes components. Is there a reason you need this re-enabled before we remove the stack?

See #138 for more details. @marcoceppi still has this in progress.

@mbruzek does the elastic stack include all the charms mentioned?

It is all the charms mentioned
On Fri, Dec 9, 2016, 11:59 AM seman<EMAIL_ADDRESS>wrote:

@mbruzek https://github.com/mbruzek does the elastic stack include all
the charms mentioned?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/juju-solutions/bundle-canonical-kubernetes/issues/146#issuecomment-266064827,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAET1VAej3kADjPZXT-82bgXeXnob2qsks5rGYjdgaJpZM4LIMTX
.

The addon(s) such as elastic stack have been removed from teh CDK bundle at this time in favor of flavors. This should be addressed in another issue attached to those bundles, at this time - the bundle tests are tuned appropriately.
Please re-open this issue if you disagree


Hook?

Hey, first of all thanks for this project, it's great to see finite state machines gaining traction in react development and i'm a big advocate of them myself!
I'm interested whether this library has plans for offering a hook (e.g. useStateMachine or similar)? I'm interested in creating a solution myself but am not sure if it would be suitable to be included as part of this project, or as a separate package.
My rough idea is to have a useStateMachine(statechart) hook where statechart is an object which is then interpreted by xstate. This would return [State, transition] where State is a component (with is prop that can be a string, array of strings/glob pattern) and transition is a function that executes an action on the current state of the machine (e.g. transition('EVENT')). An 'optional' third value, machineState would also be returned, if the user wishes to tightly couple their UI with the internal machine's state.
Example usage would be similar to:
function App() {
	const statechart = { ... }

	const [State, transition, machineState] = useStateMachine(statechart);

	return (
		<button onClick={() => transition('TOGGLE')}>toggle</button>
		<State is="active">active</State>
		{ machineState === 'inactive' && inactive }
	)
}

State could maybe accept children as a pure React node or as a render prop function with visible (or similar) passed as a parameter.

Just for reference, here's how you can use XState with hooks today: https://xstate.js.org/docs/recipes/react.html#hooks
Would be great to see this implemented into react-automata too!

Thank you very much @alex-saunders for opening this issue, and @davidkpiano for your comment.
I'm glad you are passionate about state machines, and I appreciate you have ideas on improving this package.
It would be awesome to "rethink" react-automata, support xstate v4 and provide hooks.
I recently came across this project, did you see it @alex-saunders?
https://github.com/carloslfu/use-machine


Combine Reducer error

This may not be anything important but I noticed if I run the app using npm start and then visit a page such as http://localhost:5000/post/hows-business-boomin - everything works fine. However, if I have Chrome DevTools open and now refresh the browser on that page, I see the following error:
combineReducers.js:29 Unexpected property "currentPost" found in previous state received by the reducer. Expected to find one of the known reducer property names instead: "sourceRequest".

See #86 . This is a known bug.
I think it has to do with how the server injects reducers on first load and then injectAsyncReducer   merges (the exact same reducer) on the client (again). It doesn't actually affect anything and there isn't an extra request sent. Just an eyesore in the console tbh.

Thanks!


Css and fixes

What this PR does / why we need it:

::ng-deep promotes component styles to global styles (https://angular.io/guide/component-styles). Prevent this effect by using :host
fix typo
improve rendering of modals

lgtm

Before:

After:


What role(s) are allowed to unblock MFA for a user?

There doesn't seem to be any documentation about what role(s) are allowed to unblock users from MFA. This needs to be documented as currently Authentication Administrators cannot do this.  It seems only Global Administrators have this right. Can someone please confirm and document what roles should be able to unblock user MFA?
https://docs.microsoft.com/en-us/azure/active-directory/authentication/howto-mfa-mfasettings#block-and-unblock-users

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

ID: 84b26750-880c-8f14-1628-436293de915e
Version Independent ID: 518b727f-79e8-b5c2-bcb3-ff4de3444c6f
Content: Configure Azure AD Multi-Factor Authentication - Azure Active Directory - Microsoft Entra
Content Source: articles/active-directory/authentication/howto-mfa-mfasettings.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

@PiKappZ746  Thanks for your feedback. We will investigate and update as appropriate.

@PiKappZ746 Azure AD Role - Authentication Policy Administrator has the privilege to unblock users from MFA. Same has been updated here.
https://docs.microsoft.com/en-us/azure/active-directory/roles/permissions-reference#authentication-policy-administrator
Let me know if you have any further questions.


Move impressum & privacy policy into cloudflare worker

Fixes https://github.com/serlo/serlo.org/issues/269 Fixes https://github.com/serlo/serlo.org/issues/270
Fixes https://github.com/serlo/p_data-analysis-and-kpis/issues/113

[x] Support for donation and participate -> Issue erstellen -> #5
[x] Add follow up issue for repo serlo.org
[x] Add support for __GOOGLE_ANALYTICS_DEACTIVATE__
[x] Add  __GOOGLE_ANALYTICS_DEACTIVATE__ lo legal documents
[x] Check legal documents what is missing
[x] Add Aktivitätsdaten
[x] Add documentation
[x] Update https://github.com/serlo/serlo.org-legal with current versions
[x] Date setting currently is error prone -> better implementation
[x] Common template for UnrevisedPageView and RevisedPageView
[x] Set rights for https://github.com/serlo/serlo.org-legal
[x] Template has no

[x] Add PreactReponse to other files
[x] Add JsonResponse - also to other files
[x] Better NotFoundResponse
[x] Refactor https://github.com/serlo/serlo.org-cloudflare-worker/blob/f430f5bcf6bb5a0e2ed3b1c73c8a6438c2e3ab1a/tests/static-pages.ts#L87-L91 to Matcher
[x] Add NotFoundResponse in other files
[x] Refactor https://github.com/serlo/serlo.org-cloudflare-worker/blob/f430f5bcf6bb5a0e2ed3b1c73c8a6438c2e3ab1a/tests/static-pages.ts#L114-L115 to new Matcher
[x] Refactor https://github.com/serlo/serlo.org-cloudflare-worker/blob/f430f5bcf6bb5a0e2ed3b1c73c8a6438c2e3ab1a/tests/static-pages.ts#L177 to new function with https://github.com/serlo/serlo.org-cloudflare-worker/blob/f430f5bcf6bb5a0e2ed3b1c73c8a6438c2e3ab1a/tests/static-pages.ts#L197
[x] Refactor https://github.com/serlo/serlo.org-cloudflare-worker/blob/f430f5bcf6bb5a0e2ed3b1c73c8a6438c2e3ab1a/src/static-pages/index.tsx#L160-L165 into revised.map(..) if revised is of type Revised

@inyono I have implemented your feedback. Thx for them :-)

Thanks, only two small things remaining :)

Done. Thx!


HBASE-26557 log4j2 has a critical RCE vulnerability

https://issues.apache.org/jira/browse/HBASE-26557

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 57s
Docker mode activated.

_ Other Tests _

+1 :green_heart:
asflicense
0m 17s
The patch does not generate ASF License warnings.

61m 54s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3933/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3933

Optional Tests
dupname asflicense javac hadoopcheck xml compile

uname
Linux c105ed694fdd 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / b5cf3cdc5c

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
141 (vs. ulimit of 30000)

modules
C: . U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3933/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

And I suggest that we add -Dlog4j2.formatMsgNoLookups=true in the start scripts to disable JNDI completely, we do not need this feature in HBase, typically.

Added the parameter to the start script.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m  0s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
6m 16s
master passed

+1 :green_heart:
compile
4m  3s
master passed

+1 :green_heart:
shadedjars
10m 37s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
4m 15s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
6m 15s
the patch passed

+1 :green_heart:
compile
4m 29s
the patch passed

+1 :green_heart:
javac
4m 29s
the patch passed

+1 :green_heart:
shadedjars
11m 32s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
4m 12s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
281m 31s
root in the patch passed.

337m 44s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3933/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3933

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 4f8617a21f3c 4.15.0-163-generic #171-Ubuntu SMP Fri Nov 5 11:55:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / b5cf3cdc5c

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3933/1/testReport/

Max. process+thread count
4429 (vs. ulimit of 30000)

modules
C: . U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3933/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 26s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 11s
master passed

+1 :green_heart:
compile
2m 44s
master passed

+1 :green_heart:
shadedjars
8m 19s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 25s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 52s
the patch passed

+1 :green_heart:
compile
2m 55s
the patch passed

+1 :green_heart:
javac
2m 55s
the patch passed

+1 :green_heart:
shadedjars
8m 45s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 16s
the patch passed

_ Other Tests _

-1 :x:
unit
369m 25s
root in the patch failed.

408m  2s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3933/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3933

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 675c2f493b85 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / b5cf3cdc5c

Default Java
AdoptOpenJDK-1.8.0_282-b08

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3933/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3933/1/testReport/

Max. process+thread count
5740 (vs. ulimit of 30000)

modules
C: . U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3933/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 17s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
shelldocs
0m  0s
Shelldocs was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m  6s
master passed

+1 :green_heart:
compile
9m  5s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 50s
the patch passed

+1 :green_heart:
compile
9m  4s
the patch passed

+1 :green_heart:
javac
9m  4s
the patch passed

+1 :green_heart:
shellcheck
0m  1s
There were no new shellcheck issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  2s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
20m 27s
Patch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.

_ Other Tests _

+1 :green_heart:
asflicense
0m 17s
The patch does not generate ASF License warnings.

56m  7s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3933/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3933

Optional Tests
dupname asflicense shellcheck shelldocs javac hadoopcheck xml compile

uname
Linux 10b25338f9e6 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / b5cf3cdc5c

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
141 (vs. ulimit of 30000)

modules
C: . U: .

 U: .

 U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3933/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Removes massive, oversized, bloated diaper from Invitation#accept page

The issue was that it was using the standard layout, which has a bunch of extraneous AdminLTE framing, and something was preventing the sizing. So even with the logo hidden, it still had a junk layout.
I created a new devise layout, based on the existing login layout. Also created an issue to fix all these layouts.
I added some additional copy to the page to provide better context for the user about where they're at, since the page, as-is, doesn't fit within the continuity of "accepting an invitation", and our users are unlikely to be tech-savvy.
I made the button green, because I think that's teh direction we're going in for "saving to database" buttons.

WOOT! Looks great! :)


Update BlockBiCGSTB.jl

corrected function name

Coverage remained the same at 76.854% when pulling e0e59fefac92540b70fa9e95333badd20a2416d9 on erantreister:master into 8d27374c364813ce6b948afacc70347b972394ee on lruthotto:master.

Coverage increased (+10.1%) to 86.99% when pulling 8ac50c87ee49bae229bd8ffc89966ed7fa385d70 on erantreister:master into 8d27374c364813ce6b948afacc70347b972394ee on lruthotto:master.


Merge extra contexts to contexts

This Fix solved the Problem with extra data to contexts.
See also: https://github.com/Seldaek/monolog/pull/1214/files/4a86052a6e621284ca19eec4e9c94f647f0dd16d
Example:
$sentry->extra_context([
'contexts' => ['browser' => [
'version' => '32.0',
'name' => 'Firefox'
]]

Hey @dercodercom, could you also add some tests to verify this is working as intended?

Here the Tests => https://github.com/getsentry/sentry-php/pull/691

Test are green in #691 even without this patch. Are we sure that we need this?

Test are green in #691 even without this patch. Are we sure that we need this?

Yes, because at the moment no browser or OS data are taken over in Sentry.
Enclosed two screenshots without fix and with fix.

That's not correct. If the test is not red without the patch, the test is not useful. That's why you have to submit both.

I will close this issue since we actually do not want to merge extra into "global" contexts.
The Browser context has been fixed on sentry.io itself.
Thanks for your contirubution but the root cause of this bug should be fixed now.


Network hotplug of v1.3.0 can not be used

Description of problem
I download the kata-static-1.3.0, and test the network hotplug with docker. but I did not see the kata-netmon process after creating a container, and with following error:
Sep 29 08:00:07 paas-controller-22-0-2-10 kata-runtime[677]: time="2018-09-29T08:00:07.9915428+08:00" level=error msg="Container ID (ab6420fa5e40e24e27e21bc9f6d3fd6ccd9feb7fbd82545470e47ea9b18e9e30) does not exist" arch=amd64 command=kata-network name=kata-runtime pid=677 source=runtime
Sep 29 08:00:07 paas-controller-22-0-2-10 kata-netmon[48974]: time="2018-09-29T08:00:07.993110399+08:00" level=fatal msg="handleEvents()" error="exit status 1" name=kata-netmon pid=48974 sandbox=ab6420fa5e40e24e27e21bc9f6d3fd6ccd9feb7fbd82545470e47ea9b18e9e30 source=netmon

(kata-netmon is compiled from source code, because there are no kata-netmon in static packages )
I have enabled the netmon, the config:
[netmon]
# If enabled, the network monitoring process gets started when the
# sandbox is created. This allows for the detection of some additional
# network being added to the existing network namespace, after the
# sandbox has been created.
# (default: disabled)
enable_netmon = true

# Specify the path to the netmon binary.
path = "/opt/kata/libexec/kata-containers/kata-netmon"

# If enabled, netmon messages will be sent to the system log
# (default: disabled)
enable_debug = true

[runtime]
# If enabled, the runtime will log additional debug messages to the
# system log
# (default: disabled)
enable_debug = true
#

Is there anything wrong with my configuration?
@sboeuf @caoruidong @bergwolf

Full logs:  kata.log

Looks like before container fully starts, netmon receives an event and do "update-route".
Sep 29 21:55:24 localhost.localdomain kata-netmon[31069]: time="2018-09-29T09:55:24.325633877-04:00" level=debug msg="handleRouteEvent: netlink event received" name=kata-netmon pid=31069 sandbox=01ddfa442db99b3c143c2fce90d72e3a7969495bf0ad8b2bbdab7ad01e894c7c source=netmon
Sep 29 21:55:24 localhost.localdomain kata-netmon[31069]: time="2018-09-29T09:55:24.325724511-04:00" level=debug msg=RTM_NEWROUTE name=kata-netmon pid=31069 sandbox=01ddfa442db99b3c143c2fce90d72e3a7969495bf0ad8b2bbdab7ad01e894c7c source=netmon
Sep 29 21:55:24 localhost.localdomain kata-netmon[31069]: time="2018-09-29T09:55:24.325858635-04:00" level=debug msg="Routes converted" routes="[]"
Sep 29 21:55:24 localhost.localdomain kata-netmon[31069]: time="2018-09-29T09:55:24.326094887-04:00" level=debug msg="Running runtime command" command="&{/root/go/src/github.com/kata-containers/runtime/kata-runtime [/root/go/src/github.com/kata-containers/runtime/kata-runtime kata-network update-routes 01ddfa442db99b3c143c2fce90d72e3a7969495bf0ad8b2bbdab7ad01e894c7c /var/run/kata-containers/netmon/sbs/01ddfa442db99b3c143c2fce90d72e3a7969495bf0ad8b2bbdab7ad01e894c7c/shared.json] []  <nil> <nil> <nil> [] <nil> <nil> <nil> <nil> <nil> false [] [] [] [] <nil> <nil>}" name=kata-netmon pid=31069 sandbox=01ddfa442db99b3c143c2fce90d72e3a7969495bf0ad8b2bbdab7ad01e894c7c source=netmon

But as container not started, container id mapping has not been set up. Get error here https://github.com/kata-containers/runtime/blob/a786643d0b3c0195deeb6ef5d4c2a161009a80aa/cli/oci.go#L76-L85
I will dig more.

Hi @miaoyq, your configuration looks fine to me, and I am not sure the issue comes from kata-netmon here.
First thing, I have opened the package kata-static-1.3 and I could find kata-netmon inside ./opt/kata/libexec/kata-containers/kata-netmon, so please use the binary from there.
Second thing, make sure you have the appropriate image with the right kata-agent, otherwise it could not be compatible.

Any updates here @miaoyq ?

@egernst I've found the problem, that's because I used the docker version(v12.06) too old, it worked well after changing the docker to v18.06.

I will close the issue.


ci: post release activities

Description:
Commit Message
type: commit message

see the guidelines for commit messages.
Changelog Entry
version: <log entry>

Checklist:

[x] I have performed a self-review of my own code
[ ] I have documented my code in accordance with the documentation guidelines
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have run the unit tests
[ ] I only have one commit (if not, squash them into one commit).
[ ] I have a descriptive commit message that adheres to the commit message guidelines

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

I think its already merged to main branch through pull request  #104

Release 1.0.0 already has been merged to main.


Incorrect SKU applicability for policy-csp-admx-windowsstore.md

From https://github.com/MicrosoftDocs/windows-itpro-docs/pull/10852:
RemoveWindowsStore registry key which is set by this CSP (and by the equivalent GPO) does not work on Win 10 Pro SKU (1809 and later) or Win 11 Pro SKU (any release). Tested multiple times, Pro device allows the user to download and install apps but as soon as you uplift the device to Enterprise SKU and restart, the store becomes blocked.
Old article referring to 1809 and 1903 here: https://docs.microsoft.com/en-us/troubleshoot/windows-client/group-policy/cannot-disable-microsoft-store

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

ID: 2feed46f-cb92-87aa-dc7f-e1d03cdd5bb5
Version Independent ID: 288a1749-4270-c0ab-64bd-4dd1b7018f23
Content: Policy CSP - ADMX_WindowsStore - Windows Client Management
Content Source: windows/client-management/mdm/policy-csp-admx-windowsstore.md
Product: w10
Technology: windows
GitHub Login: @vinaypamnani-msft
Microsoft Alias: vinpa

I've reported this to the OS team, and am closing this issue.


If column name is an SQL keyword, pgfutter fails with no error message

Using pgfutter_windows_amd64.exe binary on Windows 10, with CSV files edited in Notepad++ in UTF-8 encoding:
I attempted to import a table which included a column named "Union" (for trade union), and pgfutter exited after printing out the column name list only, with no further message. By trial and error I figured out where the problem was, and also tested it by renaming the column to "test" (which worked) and then "select" (which didn't). If it was possible to catch invalid column names (and maybe prefix them with a '_') that would be great, but at the least it would be helpful if there was an error message, preferably identifying the rogue column name.

Looks like Postgres 11 SQL Keywords could be used as a reference for this.
@lukasmartinelli I see 3 possible solutions, in my personal order of preference:

Quote column names that are reserved words
Quote all column names
Prefix/suffix column names that are reserved words

I like option 1 as it allows the table structure to remain in line with the file headers, which I think would be the expected behavior from a user. Option 2 is a bit of a shotgun/brute-force solution. Option 3 would absolutely work, but I think it's a non-obvious behavior from a user perspective. If you have a preferred solution, please let me know; I'd be happy to submit a pull request

Personally I don't have a problem with Option 3, after all column headers with spaces and capitals are amended to suit PG requirements. But I'm just really pleased that this excellent tool is being maintained again after a bit of a hiatus!


Marathon service restarted always (even for unchanged config)

I propose to split following task into 1 task to enable the service and one that restarts it (through a handler):
- name: Start Marathon service
  service: name=marathon state=restarted enabled=yes
  tags:
    - restart

I aggree that Marathon must not be restarted unless configuration changes in this case notifiers should be used

good good
I've the changes ready, will create PR tomorrow, once the other one gets merged 👍

The change will mostly contain a number of duplicate 'notify' lines
I didn't find another way  (for one moment was hoping you could add the notify to the include(conf.yml) block, so that if any update was done by any task there, it would notify the handler, but that didn't work).
Snippet of what to expect from PR (tomorrow):
- name: Set required --master option
  template: src=master.j2 dest=/etc/marathon/conf/master
  notify: Restart marathon

- name: Remove optional --artifact_store option
  file: path=/etc/marathon/conf/artifact_store state=absent
  when: artifact_store == ""
  notify: Restart marathon

- name: Set optional --artifact_store option
  template: src=artifact_store.j2 dest=/etc/marathon/conf/artifact_store
  when: artifact_store != ""
  notify: Restart marathon

looks good :+1:

Full PR -> https://github.com/AnsibleShipyard/ansible-marathon/pull/30 :)


cloning a raspiblitz hdd

Hello,
My node has been down since Sept 2019 as my RPI4 broke.
I had some funds in open channels and some funds in local channels.
I recently got a new RPI4 and am attempting to clone the old setup into the new one, I had taken a rescue backup before the node went down, but it was also running out of space.
Recently I cloned the old hdd to a larger one of the same partition and fs type (FAT32 ext4) and synced all the blockchain data over, labelled the disk the same and copied over the same disk UUID.
It boots now but I'm getting messages in menu about bitcond not started, when I manually start it from the bitcoin user account it runs however.
Any help would be appreciated.

did you enlarge the partition on the new disk?
also post a link with your logs:
./XXdebugLogs.sh | nc termbin.com 9999

logs are up here
Yes I create a partition on the new disk larger and with the duplicate qualities of the original.

LND has a problem connecting to bitcoind.
Make sure your password B matches in the bitcoin.conf and lnd.conf or just change the password from the menu.

you can reinstall and or update bitcoin and lnd individually with the scripts here: https://github.com/openoms/raspiblitz-extras/blob/master/README.md#lnd-update
Starting with a. clean SDcard could also solve the problem. Use the same or find the latest image in the v1.4 branch.

new output
I noticed it don't have a bitcoin.conf either.

seems to be working now. Node is back online and it looks like my funds from last Sept when it went down are still there even.
Thanks for your guidance as always @openoms you're a great part of the dev community here.


Use filepath.ToSlash and Join in indexer defaults and queues

As revealed by #15964 there is inconsistent use of filepath Join and path Join
for these directories. The best thing to do is to use filepath.Join but then ToSlash
them for consistency.
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

This fixes the issue for me.

Codecov Report

Merging #15971 (8ed7cd5) into main (b4d1059) will increase coverage by 0.02%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##             main   #15971      +/-   ##
==========================================
+ Coverage   44.02%   44.05%   +0.02%
==========================================
  Files         682      682
  Lines       82407    82408       +1
==========================================
+ Hits        36282    36303      +21
+ Misses      40218    40201      -17
+ Partials     5907     5904       -3

Impacted Files
Coverage Δ

modules/markup/markdown/goldmark.go
58.84% <70.00%> (+0.62%)
:arrow_up:

modules/markup/sanitizer.go
88.88% <100.00%> (ø)

modules/setting/indexer.go
91.42% <100.00%> (ø)

modules/setting/queue.go
67.92% <100.00%> (ø)

services/pull/pull.go
43.37% <0.00%> (ø)

modules/log/event.go
61.79% <0.00%> (+0.94%)
:arrow_up:

services/pull/patch.go
55.93% <0.00%> (+1.69%)
:arrow_up:

modules/nosql/manager_leveldb.go
17.27% <0.00%> (+2.72%)
:arrow_up:

services/pull/check.go
28.76% <0.00%> (+2.73%)
:arrow_up:

services/pull/temp_repo.go
29.78% <0.00%> (+3.19%)
:arrow_up:

... and 1 more

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 b59afa2...8ed7cd5. Read the comment docs.


feat: add filter for debug panel

Description

[x] Add filter for debug panel.

[ ] align diagnositcs list with design

Task Item
#minor
Screenshots

Coverage decreased (-0.02%) to 54.05% when pulling fbaea17be4e2e3ce13bd804bc9b838f8cc45984f on alanlong9278:julong/debug-panel-ux into 63188b12d790ad63b67246f4a70a8565e41a1c1d on microsoft:main.


generic proxy: filter chain support for local reply and refactored the stream flags

Commit Message: generic proxy: filter chain support for local reply and refactored the stream flags
Additional Description:

filter chain support for the local reply.
refactored the stream flags.

Risk Level: low.
Testing: unit.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

kind ping @alyssawilk for a blind approve, thanks. :)


Set an ACL recursively is not supported using C# SDK ?

[Enter feedback here]
Set an ACL recursively is not supported using C# SDK ?

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

ID: a3aa443a-b30f-2914-5cb8-b032be075c07
Version Independent ID: ece80543-b5c2-7415-776c-06cdf55d2006
Content: Set ACLs recursively for Azure Data Lake Storage Gen2
Content Source: articles/storage/blobs/recursive-access-control-lists.md
Service: storage
Sub-service: data-lake-storage-gen2
GitHub Login: @normesta
Microsoft Alias: normesta

duplicate of https://github.com/MicrosoftDocs/azure-docs/issues/64934


fit() validation processing should also be batched

The current implementation of calculating validation loss tries to load the entire validation dataset at once, which can easily cause OutOfMemory errors on large datasets in networks and/or in networks with a large number of parameters.

This is fixed.


Cannot Install WSL - The deployment operation was blocked due to a per-package-family policy restricting deployments on a non-system volume.

Windows Version
Microsoft Windows [Version 10.0.22631.2861]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
No response
Distro Version
No response
Other Software
No response
Repro Steps
Run wsl.exe --install
Expected Behavior
WSL should install
Actual Behavior
The deployment operation was blocked due to a per-package-family policy restricting deployments on a non-system volume. Per policy, this app must be installed to the system drive, but that's not set as the default. In Storage Settings, make the system drive the default location to save new content, then retry the install.
Diagnostic Logs
WslLogs-2023-12-14_18-47-05.zip

I've checked Settings and verified:


Reverse PInvoke delegate thunks

This change has two parts:

Remove uses of generic delegates for NativeCallable methods within the class library. These are not valid delegate targets (see IsNativeCallingConventionCompatible in Project N Delegate transform) and this use only works because IL2IL on Project N optimizes them away when rewriting AddrOf intrinsic. We don't have the IL rewriting to make that happen on CoreRT.
Enable emission of reverse P/Invoke thunks for delegate types. The conditions for generating them should be identical to Project N, including the buggy IsNativeCallingConventionCompatible method.

Marking as WIP because it fails the multifile tests because of bogus NativeCallable methods in interop assembly.
@yizhang82 System.Private.Interop has following code in it:
pfnStat = AddrOfIntrinsics.AddrOf<AddrOfIntrinsics.AddrOfIStreamStat>(Stat),

[NativeCallable]
internal static unsafe int Stat(System.IntPtr pComThis, out ComTypes.STATSTG pstatstg, int grfStatFlag)
{
    // ...
}

[StructLayout(LayoutKind.Sequential, CharSet = CharSet.Unicode)]
public struct STATSTG
{
    public String pwcsName;
    // ...
}

I.e. this is a NativeCallable method that has a struct that requires marshalling in the signature. Does MCG rewrite this kind of things?

Dropping the WIP tag. The bogus NativeCallable issue is an actual product issue that will be fixed from the TFS side this or next week. I won't merge this until that happens, but nothing prevents reviewing this.

@dotnet-bot test Windows_NT Debug and CoreCLR tests
@dotnet-bot test Windows_NT Release


K3S Import - Cluster stuck in "Upgrading" state when upgrade was attempted when 1 worker node was powered off and then powered on.

Rancher Server Version - 25f16c370
Steps to reproduce the problem;

Import k3 cluster with k3s version -“1.16.3” having 1 control and 2 workers.
Power down 1 worker node  ("Unavailable" state in UI)
Upgrade k3s version to “1.16.7” with control and master concurrency set to 1.
Control node got upgraded to new k3s version.
Cluster was stuck in “upgrading” state - Message said “Upgrading control node” even though it has upgraded control node.
Power on worker node.
Node gets to “active” state.

I see this message - “Worker node being upgraded” , but don’t see worker node being upgraded ( It is not getting to cordoned state)

I think the root cause here is the low (default) backoff limit that the SUC configures jobs with. This can be modified in the system chart. I will submit a PR for that to adjust.

@daxmc99  your earlier pr addresses the status sticking I. “...control plane”?

Never mind. Looks like it was separated

Doesn't appear to be any work in Rancher needed here. @dweomer is going to bump the system-chart and make some changes to SUC.

Additional mitigation from rancher/system-upgrade-controller#48

I'm removing Dax as this has been addressed at the system-upgrade-controller level.

Tested in master-23409-head 8ede17d10
Worked as expected.
Steps:

Imported k3 cluster with k3s version -“1.16.3” having 1 control and 2 workers.
Powered down 1 worker node ("Unavailable" state in UI)
Upgraded k3s version to “1.16.7” with control and master concurrency set to 1.
Control node got upgraded to new k3s version.
Cluster was stuck in “upgrading” state - Message said “Upgrading worker node which was powered down"
Powered on worker node.
Node became “active”.
Cluster upgraded t0 16.7 successfully.

Above was tested with concurrency of worker as 2, Upgrade started for both the worker node at same time and stuck for powered off node only. On making the node power on upgrade worked fine.


Add Ratings

#401 closes

ss?

ss?

why the website was changed ? the shop.html is so big

Fixes #401


Search by column

Is your feature request related to a problem? Please describe.
Hi, there is no way to search by column, unless I access internal variables. I was able to search by column name comparing the searchcriteria to specific row[column_index] value. Here is a screenshot of the modified tableview example from master repository:

So, my use case here is search selecting what column to apply the search. I am not interested on adding more GUI complexity (combobox), but I just want a function to searchbycolumn without accessing _filtedered interval variable of tableview. Here is my search function. It is a modified version of  _search_table_data.
search function:
    def _pesquisa(self):
        index = self._cbo_search.current()
        header = self._colsearch[index]
        # header is the column header I will apply the search (string)
        # header is selected on the combobox

        # searchcriteria comes from the Entry
        searchcriteria = self._entry_search.get()

        col_index = -1
        for coluna in self._table.tablecolumns:
            if coluna.headertext == header:
                col_index = int(coluna.cid)
                break
        # col_index is the index of the column I will apply the search
        if col_index >= 0:
           # _table is a Tableview, and _filtederd bugs me. I don't want to access private variables.
            self._table._filtered = True
            self._table.tablerows_filtered.clear()
            self._table.unload_table_data()
            for row in self._table.tablerows:
               # compare only with desired column
                if searchcriteria.lower() in row.values[col_index].lower():
                    self._table.tablerows_filtered.append(row)
            self._table._rowindex.set(0)
            self._table.load_table_data()

Describe the solution you'd like
My solution is:

change searchcriteria to:
searchcriteria: Union[str, tuple[str,str]]

if searchcriteria is a string, nothing changes.
if searchcriteria is a tuple (searchstring,columnstring), search by column

change   _search_table_data:

it has to handle both cases.
criteria.lower() is inside the loop. It make things slower.

make search_table_data public. So we can write custom search GUI.

This solution has no impact on code that already uses searchcriteria as a string, but add functionality. This solution has no impact on the GUI already used for searching, but gives the possibility of a custom GUI for searching.
Describe alternatives you've considered

Drop searchcriteria and search entry
Write public search function (with search by column support) and let the user deal with the GUI for searching.

Additional context
No response

Bug fix of my code above:
if the cell has an integer, there is a bug while executing lower().
this line:
if searchcriteria.lower() in row.values[col_index].lower():
to this line:
if searchcriteria.lower() in str(row.values[col_index]).lower():

@dmalves @israel-dryer
In addition to applying .lower(), you should also remove accents and special characters from the terms to be searched, something like this:
import unicodedata

item_no_accents = ''.join((c for c in unicodedata.normalize('NFD', str(row.values[col_index]).lower()) if unicodedata.category(c) != 'Mn'))
searchcriteria_no_accents = ''.join((c for c in unicodedata.normalize('NFD', searchcriteria.lower()) if unicodedata.category(c) != 'Mn'))

if searchcriteria_no_accents in list_item_no_accents:
    ...

I don't know if this is unfeasible in larger databases.

So if there is a term in the table

metáfora

you can search just by the term:

meta


RemoteTransform2D projects wrong position

Godot version
4.2.beta
System information
Windows 10
Issue description
I assume the main use-case of RemoteTransform2D is to have another node match the transform of the RemoteTransform2D's parent node. When RemoteTransform2D has use_global_coordinates set to false, however, the position sent to the controlled node will be the RemoteTransform2D's local position, which makes sense, I guess, but that will always be (0,0). This could be the expected behavior, but I can't imagine when it'd ever be useful, so please correct me if I'm wrong.
Steps to reproduce

Make a sprite.
Make a second sprite with a RemoteTransform2D child and move it anywhere on screen.
Select the first sprite as the RemoteTransform2D's target.
Set use_global_coordinates to false.

Minimal reproduction project
N/A

There seems to be something strange here.
When use_global_coordinates is true, it will notify NOTIFICATION_TRANSFORM_CHANGED; when use_global_coordinates is false, it will not.
https://github.com/godotengine/godot/blob/a574c0296b38d5f786f249b12e6251e562c528cc/scene/2d/remote_transform_2d.cpp#L68-L72
https://github.com/godotengine/godot/blob/a574c0296b38d5f786f249b12e6251e562c528cc/scene/2d/remote_transform_2d.cpp#L88-L92
Here, get_global_transform() seems to be a necessary condition for notify NOTIFICATION_TRANSFORM_CHANGED.

@Rindbee I'm not sure, but that may be unrelated. I think it's more that RemoteTransform2D just happens to work with its main use case but its logic isn't designed that way.
When you want a CanvasItem elsewhere in the scene tree to move along with another CanvasItem, you add a RemoteTransform2D node as a child and set a target. When you update the parent's global_position, the RemoteTransform2D's global_position also updates, and will match (unless it's offset). However, when you have it set to local, updating the parent's position does nothing for the RemoteTransform2D, because its position is still whatever it is relative to the parent (99% of the time 0,0), when the behavior you're looking for is to send the position of its parent; the thing you're trying to have the target follow.
Even before your PR, the target will get the local position changes to the RemoteTransform2D, it's just it'd never actually change unless you were directly altering the RemoteTransform2D's position from a script, which is the exact situation the node was made to avoid. You could just alter the target's position directly.

I assume the main use-case of RemoteTransform2D is to have another node match the transform of the RemoteTransform2D's parent node.

@markdibarry This seems to be a wrong assumption. According to the docs (and to the source code) RemoteTransform2D is supposed to propagate its own transform to the target node, not the transform of RemoteTransform2D's parent.
Given that's indeed intended by design, I see no bug in here.
So in the MRP you'd rather want the sprite to be a child of RemoteTransform2D, and you'd move the RemoteTransform2D (instead of its child sprite) so its transform would be propagated to the other sprite.

Changing RemoteTransform2D to propagate the parent's transform instead would be a compatibility breaking change. But I guess what could be done is adding a bool like propagate_parent_transform (false by default for compat) which would make the RemoteTransform2D behave as you've assumed it to behave. But something like this is a material for a proposal, not for a bugfix.

@kleonc Yeah, I guess that's the part I was confused about. I threw this in as an issue because, as far as I can glean, it works exactly as it says, I just can't imagine a scenario where the intended design would ever be useful without completely negating the need for the node in the first place. I honestly only use this node to do quick mock ups for things, and would never use it for an actual system, so it's not a high priority for me, but I was playing around with a SubViewport and needed to switch to local position, and was confused at how literally it behaved. It reminds me of that Daffy Duck bit where he uses a disintegration pistol and the pistol disintegrates. Like, yep, that makes total sense, but not helpful.
I totally agree: it does do what it says it does, but if it's intended to be used with RemoteTransform2D as the parent of the object to follow, I cannot find one example out there of it being used this way even as far back as Godot 2... and I imagine would make things very problematic to do so, especially if the child is a CharacterBody2D or anything with logic in it. The global_transform works perfectly for how every example I have found uses it, but it just kinda happens to work, if it's not intended to work like that at all. I think you're right about it needing a compatibility property, but it's just funny how useless it is in its current state despite doing exactly as it says.


[DIGIS-521] - Changed the end screen.

Title -[DIGIS-521] - Changed the end screen.
PR for
Before change - Session end screen show "Room disconnected Code : ROOM_END".
After change - Session end screen show "Thank you! Digis"
Testing steps,

Login to DKYC dev-environment and create a session.
Start the new newly created session.
End the session and check the end screen.

Acceptable criteria,
The session end screen should show "Thank you! Digis"
Reviewers,
@SandunRathsara @kalanainme @ThamalDilanka @IsuruSandaruwan97 @prashani97

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


vaPutSurface flags

We need a way to pass a value to the last parameter of vaPutSurface.
Currently, you always hard code a zero.
See VaapiDecoderBase::getOutput.
I can do a pull request but I want to know how you prefer to do this.
Options include.
1 - add a parameter to the function calls ie VaapiDecoderBase::getOutput.  This will touch many files.
2 - add a class member to "class VaapiDecoderBase"  Someting like
public:
unsigned int m_putSurfaceFlags;
3 - some other idea I didn't think about, let me know.
Currently, I prefer option 2.
Jay

Hi Jay:
which flag you want use, Is it gone dynamic change?

VA_SRC_BT601 and VA_SRC_BT709
RemoteFX H264 uses VA_SRC_BT709 which is not default.
I currently have to patch this and ship a modified libyami.
I guess it does not need to be dynamic but I can see the possible need.

If you use option 2, you still need a function to set the m_putSurfaceFlags. and this function is not needed by other getOutput.
Maybe, option 1 is better. you do not need modify too many file since you can use default param.
just like  frameX.
By The Way, in middle term. We want getOutput only output VaSurface(with some wrapper). You can do what you want when you get the surfacd id.

Ok, I'll create a PR.

thanks

Hi Jay:
we have new interface SharedPtrIVideoDecoder::getOutput(), VideoFrame.surface is a VASurface. So you can use it do any thing you want. You can see https://github.com/01org/libyami/blob/master/examples/simpleplayer.cpp for usage.
thanks

@xuguangxin Thanks, I'll check it out.

Hi @jsorg71 :
How was this goingon?
thanks

@xuguangxin
I'm using the capi.
Here is some sample code how I'm using the decode and vaPutSurface.
This works great and I have access to all the capabilities of vaPutSurface.
Jay
memset(&ib, 0, sizeof(ib));
ib.data = cdata;
ib.size = cdata_bytes;
status = g_decode(self->dec, &ib);
if (status == 2) /* DECODE_FORMAT_CHANGE */
{
    fi = g_getFormatInfo(self->dec);
    status = g_decode(self->dec, &ib);
}
if (status == 1) /* DECODE_SUCCESS */
{
    rb = g_decodeGetOutput(self->dec, 1);
    if (rb != 0)
    {
        status = g_vaPutSurface(rb->display, rb->surface, pixmap,
                                0, 0, src_width, src_height,
                                0, 0, dst_width, dst_height,
                                0, 0, flags);
        if (status != 0) /* != VA_STATUS_SUCCESS */
        {
            rv = 3;
        }
        g_renderDone(self->dec, rb);
    }
    else
    {
        rv = 4;
    }
}
else
{
    rv = 2;
}
return rv;

great! so I will close this issue.

thanks Jay


Domain separation for Poll IDs

Description/Reasoning
To avoid potential collisions in Poll IDs that can cause a poll to not occur, each poll should be named based on all relevant parameters from the request along with a distinct (and non-overlapping) prefix for each command that creates a poll.
For e.g., ConfirmToken, ConfirmDeposit and ConfirmTransferKey should include chain name in it's poll ID along with a prefix say "confirm_token".
Current Behaviour
Expected Behaviour
Steps to reproduce (for bugs)
Relevant Logs or Files

Agreed on the prevention of poll collisions, but I don't think it should be up to the caller to ensure uniqueness because it couples all places where the poll is used. This is another reason why I always push for IDs not to have any implicit meaning. If it is just a sequence of bytes we can modify it arbitrarily during creation until we get a unique one

Agreed on the prevention of poll collisions, but I don't think it should be up to the caller to ensure uniqueness because it couples all places where the poll is used. This is another reason why I always push for IDs not to have any implicit meaning. If it is just a sequence of bytes we can modify it arbitrarily during creation until we get a unique one

I agree that's a better long term solution. However, if that'll take time to get too, we might want to make them unique using something like what I suggested in the short term.

check that the transaction hash is included in each poll id. if yes, we're not worried about this. @milapsheth

check that the transaction hash is included in each poll id. if yes, we're not worried about this. @milapsheth

Yeah, it is. While unlikely in normal use cases, it might be concerning to rely on tx hash uniqueness between separate chains though. But as Chris suggested, in the long term we need a better solution to prevent overlaps.


Custom Domains: update 'Hostname' to 'Endpoints'

The output Convox shows me has the form shown above, doing GH PR in case docs need updating.

Good edit. Thanks a lot!


Continuous integration and cleanup

This PR introduces continuous integration workflows that validate the project.
In particular:

The project has been updated with cargo fmt to pass the formatting check. You can also run cargo fmt to update the code in the future.
The project has been updated to resolve issues reported by cargo clippy. This was mostly removing redundant/ambiguous language constructs. Again, you can run cargo clippy in the future to get the list of issues. The changes are mostly very minor, but in one place I created a "helper" data structure to remove a very large, hard to understand tuple.
Now, if we add a tag to a specific commit, the CI pipeline will generate installer files for all platforms and publish them as a "draft release" (e.g. here).
Finally, the test workflow will not run in any branch whose name starts with dev-, but it will still run for pull requests from such branch.

Very helpful and looks good to me.


How to paginate with $bind

I want to limit the query of data, and paginate with startAfter, but I have not managed to make it update correctly

@OsirisFrik were you able to get this working?


[Bug]: Cannot create file share in storage account

Is there an existing issue for this?

[X] I have searched the existing issues

Affected Resource(s)
storage.azure.upbound.io/v1beta1 - Share
Resource MRs required to reproduce the bug
Using these manifests (some details redacted):
apiVersion: storage.azure.upbound.io/v1beta1
kind: Share
metadata:
  name: test
spec:
  forProvider:
    quota: 50
    storageAccountName: redactedstr

---
apiVersion: storage.azure.upbound.io/v1beta2
kind: Account
metadata:
  name: redactedstr
spec:
  forProvider:
    accountKind: FileStorage
    accountReplicationType: LRS
    accountTier: Premium
    enableHttpsTrafficOnly: true
    location: North Europe
    minTlsVersion: TLS1_2
    resourceGroupName: storage
  writeConnectionSecretToRef:
    name:storage-file
    namespace: test

Steps to Reproduce
Apply the above manifests
What happened?
This error is returned:
async create failed: failed to create the resource: [{0 creating File
        Share "test" (Account "Account \"redactedstr\" (IsEdgeZone false /
        ZoneName \"\" / Subdomain Type \"file\" / DomainSuffix
        \"core.windows.net\")"): creating share: executing request: unexpected
        status 400 (400 The value for one of the HTTP headers is not in the
        correct format.) with InvalidHeaderValue: The value for one of the HTTP
        headers is not in the correct format.

        RequestId:xxxx

        Time:2024-11-06T16:48:14.6649628Z  []}]

Relevant Error Output Snippet
No response
Crossplane Version
1.16.0
Provider Version
1.7.0
Kubernetes Version
1.28.9
Kubernetes Distribution
AKS
Additional Info
No response

I found the problem, it looks like azure only supports file shares on a premium tier that are over 100Gb in size, I was setting it to 50Gb.
Would be useful if the error could have exposed this restriction somehow. Is that possible?


Update BPA to use v0.11 Java Client

https://github.com/boschresearch/aries-acapy-clients/tree/main/java

Related ticket: https://github.com/hyperledger/aries-cloudagent-python/issues/1078


fix(capacitor): align default web-dir for esbuild executor

Closes #1150

@edbzn any ideas why the e2e fails? I can see almost no relevant explanation in the log:

Ok, I've found and fixed the problem


assigning to 'char *' from 'unsigned char *' converts between pointers to integer types where one is of the unique plain 'char' type and the other is not

when installing with the install script:

Arch Linux x86_64
Linux 6.10.8-arch1-1
Intel Core u7-8550U @ 4GHz

Yeah, that happens when Freetype maintainers decide to change the API for no good reason. It should be already fixed in master head (and nightly builds).

yes, it's fixed. should i keep this open until release, or close it now?

Well, I guess if this is TODO list, then until we some finer status resolution, I am closing this...


@adobe/aio-lib-ims will require node-12 or greater starting July 14th, 2021

Your module is dependent on @adobe/aio-lib-ims.
Adobe IMS is retiring TLS 1.0 and TLS 1.1 support on July 14th, 2021.
TLSv1.2 is the default for node-12 or higher: https://nodejs.org/api/cli.html#cli_tls_min_v1_2
Therefore any calls to Adobe IMS using @adobe/aio-lib-ims using node-10 will fail after the retirement date above.
This module should impose a minimum requirement of node-12 going forward.
Add this to your package.json:
{
   "engineStrict": true,
   "engines" : {
         "node" : ">=12"
    }
}

@shazron lazy question — is there a public doc reference for this change?

nevermind. found it: https://helpx.adobe.com/x-productkb/multi/eol-tls-support.html


Detect device model and adapt memory layout accordingly

Different device models have different memory layouts. For example, the waypoints aren't at the same memory location between HX870 and HX890, and there's more of them in the latter.
The model should be detected and the memory layout adjusted accordingly.
Models for which the memory layout is unknown should be rejected early to prevent mishaps.

Seems to be working, closing. Change is rolled out.


(3.1) txn: Add txn_heart_beat API (#5407)

What have you changed?
This PR cherry-picks #5407 . For more detail please refer to #5407 . This is cherry-picked to 3.0 because it's needed by pessimistic transaction.
What is the type of the changes?

New feature (a change which adds functionality)

How is the PR tested?
By CI
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
This should be noted in release notes.
Does this PR affect tidb-ansible?
No.

Please resolve conflicts


Issue/bluetooth off behaviour

Summary | Résumé
Trello: https://trello.com/c/gGBbwjW2/321-🎨-assess-bluetooth-is-off-behaviour-against-expected-results
Github: https://github.com/cds-snc/covid-alert-app/issues/1073
Adjusting the displayed content as described in Issue #1073.
Closes #1073
Test instructions | Instructions pour tester la modification

Launch App

Enter One Time Key, but DO NOT UPLOAD KEYS, Hit Cancel after entering One Time Key.

Send app to background.

Turn off Bluetooth.

Return to app.

The app should prompt the user to upload their keys, but still at the bottom indicate that there is a problem with COVID Alert.

Similar steps to enter "Exposed" state, turn Bluetooth off, and return to app.

In "Monitoring" state, when Bluetooth is off, the error should be the content on the main part of the screen.

Unresolved questions / Out of scope | Questions non résolues ou hors sujet
As discussed with @amazingphilippe, there is some design debt that remains from this issue
I believe the messaging for when the networking is off, and the user is in "Diagnosed" state should reflect that the action is to upload keys, not to perform exposure checks.

@emilykuret ☝

 🎨 Assess "Bluetooth is off" behaviour against expected results

 Explore restructuring the menu

 Change home screen to fix "Bluetooth is off" issue.