Support XacmlJsonUtils to load schema files from alternative location

Schema files are not loaded from alternative location
The loading of a custom request schema in JSON cannot be achieved, because the class XacmlJsonUtils seem to only load the schema files from classpath.
Code snippets
In https://github.com/authzforce/xacml-json-model/blob/develop/src/main/java/org/ow2/authzforce/xacml/json/model/XacmlJsonUtils.java lines 67, 68, 69 the schema file names are hardcoded:
static
    {
        REQUEST_SCHEMA = loadSchema("Request.schema.json");
        RESPONSE_SCHEMA = loadSchema("Response.schema.json");
        POLICY_SCHEMA = loadSchema("Policy.schema.json");
    }

In https://github.com/authzforce/xacml-json-model/blob/develop/src/main/java/org/ow2/authzforce/xacml/json/model/XacmlJsonUtils.java line 40, the schema files are loaded from classpath (JAR):
private static Schema loadSchema(String schemaFilenameRelativeToThisClass)
    {
        try (InputStream inputStream = XacmlJsonUtils.class.getResourceAsStream(schemaFilenameRelativeToThisClass))
        {
            final JSONObject rawSchema = new JSONObject(new JSONTokener(inputStream));
            return SchemaLoader.builder().schemaJson(rawSchema).schemaClient(CLASSPATH_AWARE_SCHEMA_CLIENT).resolutionScope("classpath://org/ow2/authzforce/xacml/json/model/").build().load().build();
        } catch (final IOException e)
        {
            throw new RuntimeException(e);
        }
    }

This behavior cannot "simply" be overwritten, as the class XacmlJsonUtils is final:
public final class XacmlJsonUtils

Enhancement request
Please support loading of request schema as configured with jndi_xacmlJsonSchemaRelativePath

If you mention jndi_xacmlJsonSchemaRelativePath, this applies to AuthzForce Server only. This XacmlJsonUtils is only for loading the default XACML/JSON schema. It is not used for loading a custom schema (the method is private anyway), another class is used for loading a custom schema by AuthzForce Server (JsonRiCxfJaxrsProvider), as you can see here:
https://github.com/authzforce/server/blob/master/webapp/src/main/webapp/WEB-INF/beans.xml#L146


If ads fail to load RecyclerViewAdapter hides data

I discovered that in my application where custom data appears but after the add failed to load (Due to error 0) then my data is removed.
It can be easily reproduced in sample application, just enter an invalid ad id in MainActivity_RecyclerView_Express and no data appears.

Thx, we'll check it!

@lujop fixed. Will be released to maven v1.3


URLSearchParams produces incorrect querystring

Description
The URLSearchParams object produces an incorrect querystring when you construct it with "A record of string keys and string values"
MDN docs: https://developer.mozilla.org/en-US/docs/Web/API/URLSearchParams/URLSearchParams
const test = new URLSearchParams([
    ['key1', '23'],
    ['key2', '25'],
    ['key3', '44']
])
console.log(test.toString())
console.log(decodeURIComponent(test.toString()));

Output in browser:
key1=23&key2=25&key3=44
key1=23&key2=25&key3=44

Output in React native (Android & IOS)
0=key1%2C23&1=key2%2C25&2=key3%2C44
0=key1,23&1=key2,25&2=key3,44

React Native Version
0.71.7
Output of npx react-native info
System:
OS: Windows 10 10.0.19045
CPU: (4) x64 Intel(R) Core(TM) i5-4440 CPU @ 3.10GHz
Memory: 4.53 GB / 15.94 GB
Binaries:
Node: 19.8.1 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.17 - ~\AppData\Roaming\npm\yarn.CMD
npm: 9.5.1 - C:\Program Files\nodejs\npm.CMD
Watchman: Not Found
SDKs:
Android SDK: Not Found
Windows SDK: Not Found
IDEs:
Android Studio: Not Found
Visual Studio: Not Found
Languages:
Java: Not Found
npmPackages:
@react-native-community/cli: Not Found
react: 18.2.0 => 18.2.0
react-native: 0.71.7 => 0.71.7
react-native-windows: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
Create a react native project:
npx create-expo-app test-app
Place the folowing snippet at the top of your App() function in App.js
  const data = [
    ['key1', '23'],
    ['key2', '25'],
    ['key3', '44']
  ]
  const test = new URLSearchParams(data)
  console.log(test.toString());

Run the project
npm run start
Scan the QR code and check your console
Snack, code example, screenshot, or link to a repository
Snack example
https://snack.expo.dev/oqdkk1hN-
Note that the issue does not occur on the web variant

The issue is the constructor for URLSearchParams only supports objects:
https://github.com/facebook/react-native/blob/27b631e5b07a056d4bced19fb24537c156d2f8fa/packages/react-native/Libraries/Blob/URL.js#L60
If passed an array the constructor will iterate it's keys which are 0, 1, 2, etc and use them as the URL parameter names.
The react-native implementation is a lot simpler than the one from node, for example.
https://github.com/nodejs/node/blob/f16f41c5b3cf2345abc7bd4d5e951a43479ce4a7/lib/internal/url.js#L332
Probably the team has not had time to devote to it yet. For now I'm patching URLSearchParams with my own very simple implementation
class MyURLSearchParams extends URLSearchParams {
    constructor(params: any) {
        super(params)
        if (Array.isArray(params))  {
            // @ts-ignore
            this._searchParams = params;
        }
}
global.URLSearchParams = MyURLSearchParams;


Helpful links 2

Description of the change
This pull request adds the ability for admins to add "Helpful Links" in the Knowledge Center. These helpful links will be displayed on the Dashboard for users to give them access to Facility Admin recommended links and resources. When a user clicks on a helpful link on the dashboard that activity is tracked in the backend table open_content_activities.

Related issues: Closes #526

Screenshot(s)

Additional context
The Admin dictates the order in which the Helpful links are returned globally, so if an admin for facility 2, sets sort order by created_at DESC, all of facility 2's helpful links will be returned in created_at DESC order. If an admin from facility 1 changes the sort order, then helpful links will be returned in a different order for that facility than the other facility, etc. Currently we are not pulling and or adding favicons to the helpful links, but this is something Carolina is currently working on and we are hoping to have figured out, but wanted to get the MVP done and PR in ASAP. Resource management has been cleaned up in both Frontend and Backend

Fixed what was recommended.

Thank you both for helping with my git snafu again! Really appreciate it, especially with my head feeling like it's going to explode. 🤧


Delay for a duration

Is there a way to replicate delay_for behavior
<EMAIL_ADDRESS>

I would like this as well.


Constants directory is considered built-in

All imports from constants directory are thought of as being built-in by this plugin.
I am using typescript and use absolute paths.
E.g.
import Routes from 'constants/routes.constants'

With the default config I get this error:
`constants/routes.constants` import should occur before import of `react-router-dom`         import/order

From doing little research, I've found that it is likely coming from this lib and this line here:
https://github.com/inspect-js/is-core-module/blob/a4da7a6abf7568e2aa4fd98e69452179f1850963/core.json#L17

This is a correct error. constants is a node core module.
Also, if it was from a directory, it would be ./constants. Bare specifiers come from node_modules (or node core).


TypeNameHandling = Auto fails to deserialize message

Is this a bug report?
Yes
Can you also reproduce the problem with the lastest version?
Reproducible with latest stable MassTransit 3.5.7
Environment

Operating system: Windows 10
Visual Studio version: 2017
Dotnet version: 4.7

Steps to Reproduce

(Write your steps here:)

Clone https://github.com/nkot/typenamehandlingauto-fails
Run it by using Visual Studio
Observe error in console

Expected Behavior
MassTransit is able to deserialize message out of the box
Actual Behavior
Message is not deserialized. Console is spammed with error:
MOVE loopback://localhost/test 8f5a0000-3ede-dc4a-31fd-08d5155e736c loopback://localhost/test_error Fault: Type specified in JSON 'MassTransit.Serialization.JsonMessageEnvelope, MassTransit, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b8e0e9f2f1e657fa' is not compatible with 'GreenPipes.DynamicInternal.MassTransit.Serialization.MessageEnvelope, MassTransit.SerializationGreenPipes.DynamicInternale259301d179d416dbde48e08153bbd64, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. Path '$type', line 2, position 71.
Reproducible Demo
https://github.com/nkot/typenamehandlingauto-fails/blob/master/DeserializationFails/Program.cs

This is a known issue and will not be fixed. TypeNameHandling should not be modified.

Also, duplicate: https://github.com/MassTransit/MassTransit/issues/195

Dropping an example workaround code (custom JsonConverter) that should fix the issue.
https://github.com/nkot/typenamehandlingauto-fails/blob/master/DeserializationWorkaround/Program.cs

@nkot Thanks for the sample code! That approach worked for me.
@phatboyg Am I to understand from https://github.com/MassTransit/MassTransit/issues/195#issuecomment-22528538 that if I were using interfaces exclusively, this scenario would work out of the box? Here's a sketch of the message types I'm working with:
interface IResource { ... }
class SomeResource : IResource { ... }
class AnotherResource : IResource { ... }
class CreateResourceCommand<TResource> where TResource : IResource {
    TResource Resource { get; set; }
}
class DeleteResourceCommand<TResource> where TResource : IResource {
    TResource Resource { get; set; }
}

If your message contracts are all interfaces, there are no types involved because MT creates the backing implementation for the message contracts (interfaces).

To anyone stumbling on this ticket in the future. The sample posted by @nkot above works with a small modification in IsMassTransitOrSystemType as mentioned here: https://github.com/MassTransit/MassTransit/discussions/3314


ConstructorParameters cannot get parameters from an abstract superclass

Using ConstructorParameters<typeof Superclass> allows your constructor to share the same arguments as your superclass without copy and pasting them, but it doesn't work for concrete implementations of abstract classes because the superclass isn't newable.
https://github.com/microsoft/TypeScript/issues/23911 would provide an alternative way to avoid the copy and paste.
TypeScript Version:  3.5.0-dev.20190504

Search Terms: ConstructorParameters, abstract, newable
Code
abstract class Foo {
	constructor(public a: number, public b: string) {
	}

	abstract doSomething(): void;
}

class Bar extends Foo {
	constructor(...args: ConstructorParameters<typeof Foo>) {
		super(...args);

		// some other code here
	}

	doSomething() {
	}
}

Expected behavior: Bar compiles with no error.
Actual behavior:

error TS2344: Type 'typeof Foo' does not satisfy the constraint 'new (...args: any) => any'.
Cannot assign an abstract constructor type to a non-abstract constructor type.

constructor(...args: ConstructorParameters<typeof Foo>) {

Playground Link: https://www.typescriptlang.org/play/#src=abstract class Foo {
	constructor(public a%3A number%2C public b%3A string) {
	}

	abstract doSomething()%3A void%3B
}

class Bar extends Foo {
	constructor(...args%3A ConstructorParameters<typeof Foo>) {
		super(...args)%3B

		%2F%2F some other code here
	}

	doSomething() {
	}
}
Related Issues: https://github.com/microsoft/TypeScript/issues/30991, https://github.com/microsoft/TypeScript/issues/23911

It's intentional that abstract classes don't present as having construct signatures (for obvious reasons); the limitation here is preferable to having some new relation for infer in conditional types that would be different only in allowing that through.

@RyanCavanaugh That may be preferable if it's an either-or situation, but if it's possible to fundamentally change the way abstract classes expose their constructors, that could be invaluable for situations that @aterisetri is describing in his example code (which is the same reason I'm here).
i.e. Where you just want to pass constructor parameters through the prototype chain without redefining their properties every time.
Here's another example. If the below code ran properly, we could add a new argument to the Animal  constructor (say, animalAge: number) and we dould not have to touch the constructors for Mammal, Monotreme, Platypus, or any of what count be countless other classes inheriting animal.
abstract class Animal {
  constructor(public animalName: string) { }
}

abstract class Mammal extends Animal {
  constructor(public numberOfNipples: number, ...animalArgs: ConstructorParameters<typeof Animal>) {
    super(...animalArgs);
  }
}

abstract class Montotreme extends Mammal {
  constructor(public eggLayingCapacity: number, ...mammalArgs: ConstructorParameters<typeof Mammal>) {
    super(...mammalArgs);
  }
}

class Platypus extends Mammal {
  constructor(public venomSeverity: number, ...monotremeArgs: ConstructorParameters<typeof Monotreme>) {
    super(...monotremeArgs);
  }
}

FYI @MaximSagan and everyone else who comes here looking for a solution, I found this workaround on reddit
abstract class Test {
    constructor(a: string, b: number) {}
}

type AbstractConstructorHelper<T> = (new (...args: any) => { [x: string]: any; }) & T;
type AbstractContructorParameters<T> = ConstructorParameters<AbstractConstructorHelper<T>>;

// Params resolved to [string, number]
type Params = AbstractContructorParameters<typeof Test>;

This doesn't work for object destructured assignment:
type AbstractConstructorParameters<T> = ConstructorParameters<(new (...args: any) => any) & T>;

abstract class Test {
    constructor ({
        a,
        b
    }: {
        a: string,
        b: string
    }) {
        console.log(a);
    }
}

class Test2 extends Test {
    constructor (
        {
            c: string,
            ...rest
        }: {
            c: string;
            rest: AbstractConstructorParameters<typeof Test>
        }
    ) {
        super(...rest);
    }

}

The reason being is that the AbstractConstructorParameters and ConstructorParameters resolves to an array. But the example above is trying to spread the first parameter into the first parameter of the subclass... not sure how to deal with it.

ConstructorParameters now uses the new abstract new syntax and works as expected in 4.3.


[A11y] Ensure pages reflow without loss of content or functionality when zoomed

We updated Visualforce Pages to not truncate text, and not require horizontal scrolling, when zoomed in.

Critical Changes
Changes
Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

QE testing for this PR and subsequent WI is now complete.  Based on the WI Description, ACs, PR comments, Test Charter feedback, and regression testing, tests have passed.
For further WI and testing details, see:
https://gus.lightning.force.com/lightning/r/ADM_Work__c/a07B0000008UEbTIAW/view
https://salesforce.quip.com/Vx6HAtHRxbZa
PR has been successfully merged and corresponding WI status has been set to 'Closed'.


Fix key pair derivation: do it the same as smcli does

It fixes #1237
Now Smapp derives key pairs in the same way as smcli in this PR: https://github.com/spacemeshos/smcli/pull/37
Also, it removes the dependency on a custom ed25519.wasm.
How to test

Clone smcli/derivation
Build smcli (make build)
Run Smapp, create the wallet and copy-paste mnemonics
Read wallet file by smcli: smcli wallet read -f ~/path/to/wallet.json
Then create wallet using same mnemonics with smcli: smcli wallet create, paste mnemonics, and type password
Read wallet file: smcli wallet read -f ~/.spacemesh/newly-created-wallet.json
Public key of Main account of Smapp-created wallet should be equal to Child Key 0

Important notes
There are a couple of moments needed which need to be checked:

The secret key of the derived key pair is now 64 bytes in length, whereas the second 32 bytes is the public key.
Not sure it will work correctly for signing/verifying on the go-spacemesh side, can't check it now, has zero balance on both networks.
By default smcli generates mnemonics using 24 words, while Smapp uses only 12 words. As a consequence now Smapp can't import the wallet created by smcli (but it works in another direction).
Seems this is a candidate for the next improvement, so I just want to know what is our plan: (a) always use 24 words, (b) support both.

(a) always use 24 words, (b) support both.

Let's refer to the BIP-39 spec here. It supports mnemonics between 12 and 24 words, including those that are 12, 15, 18, 21, and 24 words in length. See the table in this section. Each maps to a different amount of entropy.
You can see the same thing in the implementations - here's the golang implementation.
Ideally we'd give the user the choice when they generate a new account/keypair/mnemonic. I don't think it's critical that we support all of [12, 15, 18, 21, 24], but it would be nice to offer 12- or 24- word mnemonics. This is not urgent and not critical, since "professional" users will anyway use smcli to generate their genesis accounts. I'm totally okay if Smapp only supports generating 12-word mnemonics for now.
However, it's important that Smapp be able to read wallet files with any valid mnemonic, i.e., any mnemonic for which bip39.validateMnemonic returns true.

automatically close #1136


Feature Extraction using YOLO-NAS

💡 Your Question
Is it possible to do feature extraction using YOLO-NAS. If yes what are the steps in doing that?
Versions
--2024-03-14 06:31:49--  https://raw.githubusercontent.com/pytorch/pytorch/main/torch/utils/collect_env.py
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 22068 (22K) [text/plain]
Saving to: 'collect_env.py'
collect_env.py      100%[===================>]  21.55K  --.-KB/s    in 0.001s
2024-03-14 06:31:50 (14.6 MB/s) - 'collect_env.py' saved [22068/22068]
Collecting environment information...
PyTorch version: 2.1.2
Is debug build: False
CUDA used to build PyTorch: 12.1
ROCM used to build PyTorch: N/A
OS: Ubuntu 20.04.6 LTS (x86_64)
GCC version: (Ubuntu 9.4.0-1ubuntu1~20.04.2) 9.4.0
Clang version: Could not collect
CMake version: version 3.16.3
Libc version: glibc-2.31
Python version: 3.10.13 | packaged by conda-forge | (main, Dec 23 2023, 15:36:39) [GCC 12.3.0] (64-bit runtime)
Python platform: Linux-5.15.133+-x86_64-with-glibc2.31
Is CUDA available: True
CUDA runtime version: 12.1.105
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration:
GPU 0: Tesla T4
GPU 1: Tesla T4
Nvidia driver version: 535.129.03
cuDNN version: Probably one of the following:
/usr/lib/x86_64-linux-gnu/libcudnn.so.8.9.0
/usr/lib/x86_64-linux-gnu/libcudnn_adv_infer.so.8.9.0
/usr/lib/x86_64-linux-gnu/libcudnn_adv_train.so.8.9.0
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_infer.so.8.9.0
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_train.so.8.9.0
/usr/lib/x86_64-linux-gnu/libcudnn_ops_infer.so.8.9.0
/usr/lib/x86_64-linux-gnu/libcudnn_ops_train.so.8.9.0
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
CPU:
Architecture:                       x86_64
CPU op-mode(s):                     32-bit, 64-bit
Byte Order:                         Little Endian
Address sizes:                      46 bits physical, 48 bits virtual
CPU(s):                             4
On-line CPU(s) list:                0-3
Thread(s) per core:                 2
Core(s) per socket:                 2
Socket(s):                          1
NUMA node(s):                       1
Vendor ID:                          GenuineIntel
CPU family:                         6
Model:                              85
Model name:                         Intel(R) Xeon(R) CPU @ 2.00GHz
Stepping:                           3
CPU MHz:                            2000.194
BogoMIPS:                           4000.38
Hypervisor vendor:                  KVM
Virtualization type:                full
L1d cache:                          64 KiB
L1i cache:                          64 KiB
L2 cache:                           2 MiB
L3 cache:                           38.5 MiB
NUMA node0 CPU(s):                  0-3
Vulnerability Gather data sampling: Not affected
Vulnerability Itlb multihit:        Not affected
Vulnerability L1tf:                 Mitigation; PTE Inversion
Vulnerability Mds:                  Mitigation; Clear CPU buffers; SMT Host state unknown
Vulnerability Meltdown:             Mitigation; PTI
Vulnerability Mmio stale data:      Vulnerable: Clear CPU buffers attempted, no microcode; SMT Host state unknown
Vulnerability Retbleed:             Mitigation; IBRS
Vulnerability Spec rstack overflow: Not affected
Vulnerability Spec store bypass:    Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:           Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:           Mitigation; IBRS, IBPB conditional, STIBP conditional, RSB filling, PBRSB-eIBRS Not affected
Vulnerability Srbds:                Not affected
Vulnerability Tsx async abort:      Mitigation; Clear CPU buffers; SMT Host state unknown
Flags:                              fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ss ht syscall nx pdpe1gb rdtscp lm constant_tsc rep_good nopl xtopology nonstop_tsc cpuid tsc_known_freq pni pclmulqdq ssse3 fma cx16 pcid sse4_1 sse4_2 x2apic movbe popcnt aes xsave avx f16c rdrand hypervisor lahf_lm abm 3dnowprefetch invpcid_single pti ssbd ibrs ibpb stibp fsgsbase tsc_adjust bmi1 hle avx2 smep bmi2 erms invpcid rtm mpx avx512f avx512dq rdseed adx smap clflushopt clwb avx512cd avx512bw avx512vl xsaveopt xsavec xgetbv1 xsaves arat md_clear arch_capabilities
Versions of relevant libraries:
[pip3] flake8==7.0.0
[pip3] msgpack-numpy==0.4.8
[pip3] mypy-extensions==1.0.0
[pip3] numpy==1.23.0
[pip3] onnx==1.13.0
[pip3] onnx-simplifier==0.4.36
[pip3] onnxruntime==1.13.1
[pip3] pytorch-ignite==0.4.13
[pip3] pytorch-lightning==2.2.0.post0
[pip3] torch==2.1.2
[pip3] torchaudio==2.1.2
[pip3] torchdata==0.7.1
[pip3] torchinfo==1.8.0
[pip3] torchmetrics==0.8.0
[pip3] torchtext==0.16.2
[pip3] torchvision==0.16.2
[conda] magma-cuda121             2.6.1                         1    pytorch
[conda] mkl                       2023.1.0         h213fc3f_46344
[conda] msgpack-numpy             0.4.8                    pypi_0    pypi
[conda] numpy                     1.23.0                   pypi_0    pypi
[conda] pytorch-ignite            0.4.13                   pypi_0    pypi
[conda] pytorch-lightning         2.2.0.post0              pypi_0    pypi
[conda] torch                     2.1.2                    pypi_0    pypi
[conda] torchaudio                2.1.2                    pypi_0    pypi
[conda] torchdata                 0.7.1                    pypi_0    pypi
[conda] torchinfo                 1.8.0                    pypi_0    pypi
[conda] torchmetrics              0.8.0                    pypi_0    pypi
[conda] torchtext                 0.16.2                   pypi_0    pypi
[conda] torchvision               0.16.2                   pypi_0    pypi

Nope, it is not supported out of the box. You can however load the model and manually replace heads with identity layers to obtain fester maps from the neck output.

How to call or replace the layers manually?

@LakshmySanthosh Here's an example:
from super_gradients.training import models
from super_gradients.common.object_names import Models
from torch.nn import Identity
import torch
model = models.get(Models.YOLO_NAS_S, pretrained_weights="coco")
model.heads = Identity()
model.eval()
x = torch.randn((5,3,640,640))
features = model(x)

You can additionally set the neck to Identity id you want to extract the features straight from the backbone.


Support slice testing of zuul components

Supports #2419 - Slice testing of Zuul filter and properties. If this looks good I can send another PR to update the documentation.

Codecov Report

Merging #2437 into master will increase coverage by 3.96%.
The diff coverage is 93.75%.

@@            Coverage Diff             @@
##           master    #2437      +/-   ##
==========================================
+ Coverage   68.43%   72.39%   +3.96%
==========================================
  Files         234      197      -37
  Lines        8218     6659    -1559
  Branches     1020      877     -143
==========================================
- Hits         5624     4821     -803
+ Misses       2210     1512     -698
+ Partials      384      326      -58

Impacted Files
Coverage Δ

...etflix/zuul/slice/ZuulTestContextBootstrapper.java
100% <100%> (ø)

...loud/netflix/zuul/slice/ZuulTypeExcludeFilter.java
91.66% <91.66%> (ø)

...ework/cloud/netflix/sidecar/SidecarProperties.java

...etflix/turbine/stream/HystrixStreamAggregator.java

...ka/server/event/EurekaInstanceRegisteredEvent.java

...tflix/eureka/server/EurekaDashboardProperties.java

...eureka/server/EurekaServerMarkerConfiguration.java

...d/netflix/eureka/server/EurekaServerBootstrap.java

...s/spectator/SpectatorMetricsAutoConfiguration.java

...rk/cloud/netflix/sidecar/SidecarConfiguration.java

... and 33 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 89441ac...e123f03. Read the comment docs.

Note that Discovery based Ribbon ServerList is disabled for the slice test, this way there is no dependency on needing any registry for these slice tests and the target zuul endpoint needs to be explicitly specified for tests using listOfServers - serviceId.ribbon.listOfServers=localhost:${wiremock.server.port}. If we absolutely need to support Discovery for Ribbon an option could be to hook up SimpleDiscoveryClient based ServerList and use it instead(such an auto-configuration does not exist but can be added if desired)

Is there some kind advantage to doing it this way as opposed to using spring cloud contract?

Yes, good point @ryanjbaxter.  S-C-Contract  would assume that we have contract's defined for all downstream services that Zuul is fronting, which likely may not be available + typically since Zuul is used for hiding legacy services typically those may be say SOAP based services.
The approach in the PR is to leave it to the user during test to define a stub behavior for the downstream service (using @AutoConfigureWireMock) and then test purely using MockMvc

OK maybe I misunderstood the use case.  This is not for testing our Zuul implementation in SC Netflix its for testing apps using Zuul, right?

Yes, that is correct @ryanjbaxter - I faced an issue at a customer location where we added Zuul proxy to an existing application and this application uses some shared libraries with its own set of auto-configurations. Now to test the Zuul specific configuration we could exclude all the auto-configurations that the customer has OR test it by activating only Zuul specific auto-configurations.
These specific auto-configurations which worked for at the customer location are now part of this slice test

Makes sense, could we get some documentation then on how its used?

Oh yes, sounds good @ryanjbaxter, will update the PR with documentation on how to use this

Added in documentation now @ryanjbaxter


Add NeuronInstance as a class to NPO

We say in the text and slides that we have 3 classes of neurons:  CUTs, EBMs and Instances, but there is no instance class that I can find in the ontology at this time.  Please add if necessary or tell me where I can find it.

There is no explicit class for a NeuronInstance because their behavior is essentially the same as a NeuronEBM. There could also be owl:NamedIndividual neurons that represent instance data, but those have not been implemented yet.

Why does that matter? I think we need it anyway or the ontology won’t work properly.  They are basically instances because we assert them to be.  I want it just as a placeholder to refer to in the  text.  Otherwise, please provide a few sentences explaining how we will handle instances.

On Jan 7, 2020, at 10:39 AM, Tom Gillespie<EMAIL_ADDRESS>wrote:
There is no explicit class for a NeuronInstance because their behavior is essentially the same as a NeuronEBM. There could also be owl:NamedIndividual neurons that represent instance data, but those have not been implemented yet.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/SciCrunch/NIF-Ontology/issues/193?email_source=notifications&email_token=ABSGOPKW6SYJQARD3ORDP5TQ4TD7FA5CNFSM4KD5INM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIJ3DBA#issuecomment-571715972, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABSGOPM26Z6H7QRCP3HOO43Q4TD7FANCNFSM4KD5INMQ.

owl:NamedIndividuals can be instantiated at any time using the existing classes, we just haven't done it. The bagged representations for the EBMs are indistinguishable from instances, and it is a bit confusing because NeuronInstance really means NeuronInstanceLiftedToTypeRepresentation because the ontology representation maps to multiple individuals (this happens in Allen). I would be fine dropping the reference to NeuronInstance altogether since all the Allen neurons are EBMs.

But if this is supposed to be able to be used in a laboratory, then individual cells will have to be mapped.  We consider them to be qualitatively different that the EBMs because not all instances will be instances of EBMs.  So I don’t see why it is wrong to just create a class that makes it clear that some instances do not belong to an EBM.

On Jan 7, 2020, at 12:20 PM, Tom Gillespie<EMAIL_ADDRESS>wrote:
owl:NamedIndividuals can be instantiated at any time using the existing classes, we just haven't done it. The bagged representations for the EBMs are indistinguishable from instances, and it is a bit confusing because NeuronInstance really means NeuronInstanceLiftedToTypeRepresentation because the ontology representation maps to multiple individuals (this happens in Allen). I would be fine dropping the reference to NeuronInstance altogether since all the Allen neurons are EBMs.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/SciCrunch/NIF-Ontology/issues/193?email_source=notifications&email_token=ABSGOPOPDT2VUVBLZHIVCI3Q4TPXDA5CNFSM4KD5INM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIKENOY#issuecomment-571754171, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABSGOPMI5WVUJ4C5Q3H4MDLQ4TPXDANCNFSM4KD5INMQ.


RuntimeDriver and PostgresDriver destroy

Hello,
While testing my app in Jest I encountered error signaling that Pool was not destroyed successfully.
Digging down, I found that the problem is that RuntimeDriver doesn't run PostgresDrivers destroy as expected.
My code looks something like this:
// database.ts
const pool = new Pool({ connectionString: '...' });
const dialect = new PostgresDialect({ pool });
const getDatabase = (meta?: Metadata) =>
  new Kysely({ dialect, log: meta && getLogger(meta) });

// test.spec.ts
it('test', () => {
  const database = getDatabase();
  await database.insertInto(...).execute();
  // ...
});

// test-environment.ts
export const teardown = () => {
  const database = getDatabase();
  await database.destroy();
}

The thing is I use only one Postgres pool instance but many Kysely instances (each of them have binded logger with query-specific metadata). Whenever RuntimeDrivers destroy method is called, it checks if current instance was initialized and the teardown's instance is not (though the test one was).
A possible solution for that would be to pass the responsibilty to decide whether destroy should be called to the wrapped driver of the RuntimeDriver.

You shouldn't share a dialect instance with multiple Kysely instances.

You shouldn't share a dialect/pool instance with multiple Kysely instances.

Is there an option to keep logging (or other options - say plugins) distinct while benefiting from having a pg pool?

You can either create new dialect and pool for each Kysely instance or override the dialect driver's init and destroy methods.

Why can't I use a shared pool while creating dialect / Kysely instance on every run?
Can you give an example of how overriding dialect driver's init / destroy should help?


Icons in which-key do not show

As you can see in the image below, the icons do not show!

But when I switch to the different colour scheme, the icons show as normal

I found a solution! Simply replace the icons in your which-key configuration with supported icons in your font.


Publish releases to GitHub Container Registry

Feature Request
I'd love if you were able to publish this action to GitHub Marketplace. Published actions provide significant time savings for workflows since the container doesn't have to be built from scratch on every run.
In our workflows, the "publish unit tests result" action we use takes around 3 seconds to pull from the marketplace, whereas this action takes around 37 seconds to build from scratch.
Expected Behaviour
N/A
Additional Context
GitHub Documentation:
https://docs.github.com/en/actions/creating-actions/publishing-actions-in-github-marketplace
Another Project That Makes Use Of This:
https://github.com/EnricoMi/publish-unit-test-result-action
Linked To
N/A

@Casey-Kiewit I've just published v1.0.2 which uses a pre-built container on GHCR like the other action. If you update to the latest version you should see an improvement in execution time.


Update profile path and operator samples

Few leftover updates for profile path and operator samples.

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
488503bf78a2eaaebc716c0e7259ac74487ccd80
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
466f022a3ebc0c9f2158e76dec2665725bcc0086
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
6c91d93ae0e9722b6bac8c7713c8c4178fabf7f7
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Error when no TTY (Errno::ENOTTY: Inappropriate ioctl for device)

I am hitting an error in my specs as there is no TTY on the build server. I have come up with two ways to address it and would like your input on which you think is best (if either).
Option 1: Update console.rb
def get_char(options)
  return input.getc unless input.tty?
  mode.raw(options[:raw]) do
    mode.echo(options[:echo]) { input.getc }
  end
end

Option 2: Override read_keypress in TestPrompt
def read_keypress(options = {})
  super(options.merge({ echo: true, raw: false }))
end

I am leaning towards Option 1. Let me know and I can submit a PR.
Thanks.

Hi Matt, thanks for using the library.
Option 1 - I see the intention behind it, it will work when reading from file or other IO object and also allow piping stuff to command etc.... Though I'm not sure about the implementation itself. Option 2 feels rather 'dirty' to me and not very maintainable. Currently I have changes in master which will in effect make majority of prompts work in raw mode. This is to do with providing line editing functionality for prompts such as #ask or #multiline. I more inclined to go with Option 1 but would probably put the checks for presence of console down to mode level. However, the tty? check is pretty unreliable on windows console emulators, for example in cygwin it returns false. Do you have any experience in Windows domain to tackle this problem? Anyways, if you fancy submitting PR I will review.

It's a pretty great library, so thanks for sharing it.
No, I don't have experience working with ruby on a Windows machine, just Linux and Mac. So are you thinking of maybe checking if the input responds to raw/noecho in the mode and if not fall back to yield?
Also, I was curious about this in console.rb
def initialize(input)
  @input = input
  @mode  = Mode.new
  @keys  = Codes.keys
  @escape_codes = [[ESC.ord], CSI.bytes.to_a]
end

Should the @mode = Mode.new be @mode = Mode.new(input)?

This is really good catch. This makes things ever more interesting as at this point input.tty? would return nil in test environment when stringio is used. This in turn wouldn't be great as many tests depend on the mode being raw. Not sure how to solve this conundrum.

Thanks a lot for pointing out all of these problems, they are fixed in master. Would you mind taking tty-prompt for a spin?

Thanks for the update. I am going to give master a spin this morning. I'll let you know how it goes.

Master works. My specs are now green. Want me to close this issue? Also, do you know if/when you might be cutting the next release. Thanks again.

Great. Naaa let's keep this issue open for now until I release new version - this will act as a reminder. I don't want to subscribe to a strict deadline - it is open source after all and I want to get this right :smile: However, I'm planning to release shortly after I've implemented https://github.com/piotrmurach/tty-prompt/issues/40 and possibly https://github.com/piotrmurach/tty-prompt/issues/50 as I'm really close getting it done. All the ground work has been done but I need to implement few bits and ensure all works fine on Windows as well. Can you wait? In the meantime if possible you could point your Gemfile at master sha?

Yep, totally understand. I can point to the master sha for now while I am developing. Was only curious as I don't believe it is possible to point to a github repo in a gemspec.

I've just released v0.12.0 with the fix and many changes. Enjoy!


Failed prop type: Required prop monitorState.profileAction was not specified in ProfilerMonitor.

I followed the docs as well as I could, but when I try to render the <DevTools /> component into my app, I get this error and stack trace:
warning.js:44Warning: Failed prop type: Required prop `monitorState.profileAction` was not specified in `ProfilerMonitor`.
    in ProfilerMonitor (created by Connect(ProfilerMonitor))
    in Connect(ProfilerMonitor) (created by DevTools)
    in DevTools (created by App)
    in div (created by App)
    in App
    in Provider

Here are the relevant files:
// App.jsx
import React from 'react';

// eslint-disable-next-line no-unused-vars
import globalStyle from '../stylesheets/globals.scss';

import ReportsSummaryPage from './ReportsSummaryPage';
import DevTools from './DevTools';

const App = () => (
  <div>
    <ReportsSummaryPage />
    <DevTools />
  </div>
);

export default App;

// DevToools.jsx
import React from 'react';
import { createDevTools } from 'redux-devtools';
import { ProfilerMonitor } from 'redux-devtools-profiler-monitor';

export default createDevTools(
  <ProfilerMonitor />
);

Here's the store configuration I pass to <Provider /> that renders <App />:
import { compose, createStore, applyMiddleware } from 'redux';
import thunkMiddleware from 'redux-thunk';
import { profileActionMiddleware } from 'redux-devtools-profiler-monitor';

import rootReducer from '../reducers';

const createStoreWithMiddleware = compose(
  applyMiddleware(thunkMiddleware, profileActionMiddleware),
  window.devToolsExtension ? window.devToolsExtension() : f => f
)(createStore);

const configureStore = (initialState) => {
  const store = createStoreWithMiddleware(rootReducer, initialState);

  if (module.hot) {
    // NOTE Enable Webpack hot module replacement for reducers
    module.hot.accept('../reducers', () => {
      // eslint-disable-next-line global-require
      const nextReducer = require('../reducers/index').default;
      store.replaceReducer(nextReducer);
    });
  }

  return store;
};

export default configureStore;

What did I do wrong?

Just in case, here's my package.json:
{
  "name": "benchpress-reports-client",
  "version": "0.0.1",
  "description": "",
  "engines": {
    "node": "6.1.0"
  },
  "main": "./lib/index.js",
  "scripts": {
    "start": "webpack-dev-server --config webpack.config.babel.js",
    "test": "karma start",
    "build": "npm run clean && npm run client",
    "clean": "rm -rf ./build",
    "client": "webpack -p --config webpack.production.config.babel.js",
    "server": "webpack --config webpack.server.config.babel.js"
  },
  "author": "",
  "repository": {
    "type": "git",
    "url": "https://github.com/APX/benchpress-client"
  },
  "license": "ISC",
  "dependencies": {
    "babel-polyfill": "^6.13.0",
    "express": "^4.13.4",
    "fecha": "^2.2.0",
    "isomorphic-fetch": "^2.2.1",
    "lodash": "^4.13.1",
    "normalize.css": "^4.1.1",
    "react": "^15.3.0",
    "react-addons-shallow-compare": "^15.3.0",
    "react-dom": "^15.0.2",
    "react-redux": "^4.4.5",
    "react-virtualized": "^7.17.1",
    "redux": "^3.5.2",
    "redux-devtools": "^3.3.1",
    "redux-devtools-profiler-monitor": "^0.4.0",
    "redux-thunk": "^2.1.0",
    "reselect": "^2.5.3",
    "string-humanize": "^1.0.0",
    "titleize": "^1.0.0"
  },
  "devDependencies": {
    "babel-core": "^6.8.0",
    "babel-loader": "^6.2.4",
    "babel-plugin-transform-es2015-block-scoping": "^6.10.1",
    "babel-plugin-transform-object-rest-spread": "^6.8.0",
    "babel-preset-es2015": "^6.6.0",
    "babel-preset-react": "^6.5.0",
    "css-loader": "^0.23.1",
    "enzyme": "^2.3.0",
    "eslint": "^2.10.2",
    "eslint-config-airbnb": "^9.0.1",
    "eslint-loader": "^1.3.0",
    "eslint-plugin-import": "^1.8.0",
    "eslint-plugin-jsx-a11y": "^1.2.2",
    "eslint-plugin-react": "^5.1.1",
    "expect": "^1.20.2",
    "extract-text-webpack-plugin": "^1.0.1",
    "fetch-mock": "^5.1.1",
    "isparta-loader": "^2.0.0",
    "karma": "^0.13.22",
    "karma-chrome-launcher": "^1.0.1",
    "karma-cli": "^1.0.0",
    "karma-coverage": "^1.1.1",
    "karma-mocha": "^1.0.1",
    "karma-mocha-reporter": "^2.1.0",
    "karma-sourcemap-loader": "^0.3.7",
    "karma-webpack": "^1.7.0",
    "mocha": "^2.5.3",
    "node-sass": "^3.7.0",
    "null-loader": "^0.1.1",
    "react-addons-test-utils": "^15.1.0",
    "react-hot-loader": "^1.3.0",
    "redux-mock-store": "^1.1.4",
    "sass-loader": "^3.2.0",
    "style-loader": "^0.13.1",
    "webpack": "^1.13.0",
    "webpack-dev-server": "^1.14.1",
    "webpack-node-externals": "^1.2.0",
    "webpack-s3-plugin": "^0.9.0"
  }
}

Oh, I didn't realize that it was actually rendering at the bottom of my page. This looks like an issue of my .eslintrc being stricter than yours. Feel free to close, but it seems like it might be worth fixing?

I'll take a look. We should be defining all propTypes correctly.

@thenickcox Sorry, just getting around to looking into this and I'm not getting the same warning you are. When I debug, I see that props.monitorState.profileAction = "", which is valid. I'm not sure why you're getting an undefined value. Does the monitor work correctly when you type in an action name?

Closing due to inactivity. Re-open if this can be verified.


CT mark is not realized correctly

Describe the bug
CT mark defined in pkg/agent/openflow/field.go like NotServiceCTMark = binding.NewOneBitZeroCTMark(4) is used in a flow as the following:
ConntrackCommitTable.ofTable.BuildFlow(priorityNormal).
		Cookie(cookieID).
		MatchProtocol(ipProtocol).
		MatchCTStateNew(true).
		MatchCTStateTrk(true).
		MatchCTMark(NotServiceCTMark).
		Action().CT(true, ConntrackCommitTable.GetNext(), f.ctZones[ipProtocol], f.ctZoneSrcField).
		MoveToCtMarkField(PktSourceField, ConnSourceCTMarkField).
		CTDone().
		Done(),

The flow should to be like the following:
cookie=0xf010000000000, table=28, priority=200,ct_mark=0x0/0x10, ct_state=+new+trk,ip actions=ct(commit,table=29,zone=65520,exec(move:NXM_NX_REG0[0..3]->NXM_NX_CT_MARK[0..3]))

But it is:
cookie=0xf010000000000, table=28, priority=200,ct_state=+new+trk,ip actions=ct(commit,table=29,zone=65520,exec(move:NXM_NX_REG0[0..3]->NXM_NX_CT_MARK[0..3]))

As a result, the CT mark is not realized in the flow.

Hi Hongliang, this is introduced by an issue in the dependent library ofnet. I have made a fix and it is merged now (https://github.com/antrea-io/ofnet/pull/30). Please update go.mod with ofnet version 0.5.7+, and the fix would be involved.


Show what's updated

Already up-to-date.

Already up-to-date.

Updated successfully

that's what drone printed out while updating my project, it would be nice if drone could also print what library is "Updated successfully" and what is "Already up-to-date".

You're right, this is the https://github.com/cesarferreira/drone-hive pulling the latest changes :)
I'll put a message or hide it :) thanks for pointing it out


Frame bleed

There are two navigation patterns that result in a frame transition turning into a client transition.  This isn't appropriate.  There's no guarantee that a client transition is viable.
This change set addresses these two navigation patterns:

frameback in => refresh => back-button
frameback in => back-button => refresh => forward-button

Note that client-transition => frameback in => back => back => refresh => forward => forward and variants were previously handled.  It was only the refesh on the outer frame (immediately prior to frameback) followed immediately by navigation forward that was broken from outside.  A refresh from any depth inside was broken.
Frameback is fun! :feelsgood:

I think the changes make sense to me.
Also, I think if you pull from master, you'll get more than 1 space for tabs in your diffs :)


Props to translate Copy, Print, More, Cancel and Import in Share

How can I localize the dialog text for Copy, Print, More, Cancel and Import for React native screens based on user language?
http://facebook.github.io/react-native/releases/0.43/docs/share.html

Looks I'm being forced to use https://github.com/AlexanderZaytsev/react-native-i18n
Just installed and now RN will use user language set on the device, anyway, I think this should be part of the core.


Fix full and thumbnail url from 2017-03-10 FA update

As of the recent update to Furaffinity image full view and thumbnail URLs can not be retrieved the same way as before. They are now available in the data-fullview-src and data-preview-src attributes of the submission img tag.
I have no experience at all with Ruby so chances are there are better ways to solve this, but this patch seems to work for me.

Thanks for the fix!


Support sending config via multiple Sysex to limited device types.

The 16nLC should use a 'multiple message' configuration send.
This should be enabled by the work done in #3.

Added support for shorter messagse in fc96a6d74e3bd2d3ea7fe27b1e7bec0e4cb9ffd6.


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
23 seconds
clone-repository

✅ Succeeded
2 minutes
build-container

✅ Succeeded
20 seconds
deprecated-base-image-check

✅ Succeeded
22 seconds
inspect-image

✅ Succeeded
39 seconds
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
18 seconds
sbom-json-check

✅ Succeeded
11 seconds
label-check

✅ Succeeded
10 seconds
optional-label-check

✅ Succeeded
7 seconds
show-summary


206 release

Codecov Report

:exclamation: No coverage uploaded for pull request base (2.0.6-prepare@42ba593). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             2.0.6-prepare   #10767   +/-   ##
================================================
  Coverage                 ?   20.89%
  Complexity               ?     1608
================================================
  Files                    ?      540
  Lines                    ?    22912
  Branches                 ?     2732
================================================
  Hits                     ?     4788
  Misses                   ?    17631
  Partials                 ?      493

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 42ba593...ef87857. Read the comment docs.


Updated text and added guidelines

Need to fix errors

@Delphine - The linter is complaining because you are using non breaking spaces instead of regular spaces.

Closing this in favor of #251


adding widevine CDM support for qutebrowser

Issue description
I have a package, widevine-cdm, mostly taken from the Chromium plugin:
https://gist.github.com/73880669d67a02fcb64e447f8c077fc9
And an overlay for qutebrowser:
{ pkgs ? import <nixpkgs> {} }:

with pkgs;

rec {
  qutebrowser = pkgs.qutebrowser.overrideAttrs (super: {
    propagatedBuildInputs = super.propagatedBuildInputs ++ [ widevine-cdm ];
    postFixup = ''
      ${super.postFixup}
      wrapProgram $out/bin/qutebrowser --add-flags "--set qt.args [ppapi-widevine-path=${widevine-cdm}/usr/lib/qt/plugins/ppapi/libwidevinecdmadapter.so]"
    '';
  });
}

But trying to watch Netflix gets me this error:
[1:1:0910/214639.789895:ERROR:ppapi_thread.cc(317)] Failed to load Pepper module from /nix/store/9mqb31wvfx7wqargrh245h33gp3myir0-widevine-cdm-<IP_ADDRESS>/usr/lib/qt/plugins/ppapi/libwidevinecdmadapter.so (error: /nix/store/9mqb31wvfx7wqargrh245h33gp3myir0-widevine-cdm-<IP_ADDRESS>/usr/lib/qt/plugins/ppapi/libwidevinecdmadapter.so: cannot open shared object file: Operation not permitted)

Technical details

system: "x86_64-linux"
host os: Linux 4.14.69, NixOS, 19.03.git.571c6c6 (Koi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.1
channels(sean): "nixos-19.03pre151837.ca2ba44cab4"
channels(root): "nixos-19.03pre151837.ca2ba44cab4"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

I'd also really like widevice (e.g. Netflix and stuff) support for qutebrowser. Can anyone help with this?

relevant prior art to possibly borrow from: https://aur.archlinux.org/packages/qt5-webengine-widevine

@colemickens, I've already done most of the work re: prior art. See the gist I linked in my OP.
I submitted the issue hoping that someone can tell us why we're getting "operation not permitted".

@ma9e yeah, I did notice the gist last night. I spent a couple of hours trying to get your widevine-cdm to work with Falkon, but couldn't really figure it out...

@ma9e Just to make this easily reproducible, could you post your overlay with the widevine and qutebrowser packages?

Has there been any progress on this front? It looks like Qutebrowser still doesn't have support for drm content like Netflix and Spotify.

Hey.
Has anyone find solution to this problem? Still can't figure out how to overlay qutebrowser to play spotify.

Sorry everyone, didn't realize this issue was open but I have a working widevine package defined here:
https://github.com/nrdxp/devos/blob/nrd/pkgs/applications/networking/browsers/widevine/default.nix
Meant to upstream it at some point but time got away from me. One can use a nix snippet like the following in their qutebrowser config to make use of it:
c.qt.args.append('widevine-path=${pkgs.widevine-cdm}/lib/libwidevinecdm.so')

FYI, I have opened a PR that basically imitates what was done on chromium with the flag enableWideVine : https://github.com/NixOS/nixpkgs/pull/194479.


Kafka ingest avro stuck

Steps to reproduce the behavior (Required)

CREATE TABLE xapi.xapi_logs(atDATETIME COMMENT "时间",log_event_type INT COMMENT "事件类型",request_id STRING NOT NULL) DEFAULT "" COMMENT "请求 ID" DUPLICATE KEY(at, log_event_type) PARTITION BY RANGE(at)() DISTRIBUTED BY HASH(request_id)
CREATE ROUTINE LOAD xapi.xapi_logs_load_job ON xapi_logs
PROPERTIES
(
"format" = "avro"
)
FROM KAFKA
(
"kafka_broker_list" = "kafka-kafka-brokers.kafka.svc.cluster.local:9092",
"confluent.schema.registry.url" = "http://schema-registry.kafka.svc.cluster.local:8081",
"kafka_topic" = "xapi-logs",
"kafka_partitions" = "0,1,2",
"property.kafka_default_offsets" = "OFFSET_END"
);

Expected behavior (Required)
Ingest normal
Real behavior (Required)
Stuck after running 1-2 hours with data ingested, but then stuck with Running and without any error
View fe system?path=//routine_loads/xapi_logs_load_job/103226, previous task aborted because of 'confluent_schema_registry_url' not set shown up

StarRocks version (Required)

3.0.3

maybe fixed already? https://github.com/StarRocks/starrocks/pull/26815

maybe not, after
alter ROUTINE LOAD for xapi_logs_load_job FROM kafka ("confluent.schema.registry.url" = "http://schema-registry.kafka.svc.cluster.local:8081/");
I found confluent.schema.registry.url appear in job config

I will dig deep to find why this happens.

Anybody actually use routine load? 😂

I think this should fix it https://github.com/StarRocks/starrocks/pull/29316


Fix Dialog background scroll

Closes #3906

Sets the root overflow to 'hidden' to prevent background scrolling while the Dialog is open.
Scrolling within the Dialog still works as expected.
Tested on Chrome/Windows, may require additional testing on a Mac due to potential differences in scroll behavior, eg. shifts in positioning when the scrollbar appears/disappears.

May require additional testing on a Mac due to potential differences in scroll behavior, eg. shifts in positioning when the scrollbar appears/disappears.

this change should probably be built as a prop in FloatingComponent rather than in dialog, since it could be reused in other implementations like drawer


figure out type of changerecord.quads

Should it be a DOMQuad? There were some concerns in Blink about shipping DOMMatrix, would the same apply to DOMQuad?

All the current APIs exposed to the web expose rects. We should expose quads eventually, but even then we should expose both for consistency with the rest of the platform. So for v1, we'll just expose boundingClientRect.


Hello! There are lots of errors in the code, please recorrect the README.txt, so that can run the result

@MikeS96 @sachaMorin I have tried following the readme to reproduce the results from the paper, but it seems that the readme has a few issues when trying running the code

Hello, can you share the errors? Are you trying to train components or only deploying the checkpoints?

@sachaMorin I am trying to reproduce the results in the paper, mainly the point navigation.
I have posted my initial problems in https://github.com/montrealrobotics/one4all/issues/4


Series.day

i think this pr is still pending and waiting to be merged. Let me know if I have to do any correction.


feat: create bn254 scalar field chip

Motivation

We want to compute this function in SP1, poseidon hash over bn254 scalar, as it would be computed in a Halo2 circuit. This is for a DA integration with a zkrollup built with Halo2 KZG.
Solution
It appears the bn254 precompiles in SP1 are for the base field, which makes sense to help verify proofs and compute pairings. However, iiuc this is not sufficient to perform computations as a halo2 circuit would, since the circuit inputs are elements of the scalar field.
Here I am developing a bn254 scalar field precompile, I expect it to be the same as Fp precompile but with a different modulus.
PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Breaking changes

bn254 fr is grumpkin fp.  this pr is useful for you  https://github.com/scroll-tech/sp1/pull/17/files


Add latest CosmosDB RP API Version 2024-09-01-preview

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.CosmosDB

@archerzz @ArthurMa1978 Can you please review and merge?

@archerzz @ArthurMa1978 Can you please review and merge?

Generally good. Left some comments.

@archerzz Resolved all comments. Please take a look again and merge if it looks good.

@archerzz Resolved all comments. Please take a look again and merge if it looks good.

@carjackson-msft There is one enum to rename: SupportAction. Also please resolve the conflicts. Thanks.

@archerzz Resolved and merge with master.

@archerzz Resolved and merge with master.

@carjackson-msft the pipeline failed, please check the build logs. looks like you need to regenerate the codes.

Generated code is not up to date.
You may need to rebase on the latest main,
run 'eng\scripts\Update-Snippets.ps1' if you modified sample snippets or other *.md files (https://github.com/Azure/azure-sdk-for-net/blob/main/CONTRIBUTING.md#updating-sample-snippets),
run 'eng\scripts\Export-API.ps1' if you changed public APIs (https://github.com/Azure/azure-sdk-for-net/blob/main/CONTRIBUTING.md#public-api-additions).
run 'dotnet build /t:GenerateCode' to update the generated code and samples.
To reproduce this error locally, run 'eng\scripts\CodeChecks.ps1 -ServiceDirectory cosmosdb'.

@carjackson-msft The test recordings are out-of-date. You need to re-record some tests. Please check the error logs: https://dev.azure.com/azure-sdk/public/_build/results?buildId=4256133&view=logs&jobId=48e16184-ebea-53b1-0660-f349545896e8&j=48e16184-ebea-53b1-0660-f349545896e8&t=648d79f2-01b5-5b89-6c0b-97898f6bef3a


Add ViewPager to slide the images

Can i work this under jwoc?

@SB2318 , If you need any help dm me!! I have implemented this but I am not fully satisfied with the working. May be, we both can help each other.


In test plan, how can I re-order the test cases

After creating a manual test case, I would like to move current test case up to somewhere. How do I do it? After reviewing all test cases, I feel some scattered test cases must stay together. How can I move them together?

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

ID: 9cb42105-a17b-39f2-49a8-712ae7f5e503
Version Independent ID: 91426199-6db1-19ac-8e55-9fc7747323f6
Content: Navigate Test Plans - Azure Test Plans
Content Source: docs/test/navigate-test-plans.md
Service: azure-devops-test-plans
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@VincentXu2021 -- Vincent, thank you for your question.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@KathrynEE -- Kathryn, please look into this issue.

@VincentXu2021 - You can drag and drop test cases into the order you want as shown in the following image:


Fix false positives of RepeatedExample with its examples

RepeatedExample cop incorrectly labels different examples as
repeated since the argument passed to its is ignored, ex.:
its(:x) { is_expected.to be_present }
its(:y) { is_expected.to be_present }
Fixed by adjusting the key used for grouping if its is used.
cc @backus

Great fix. Thank you @redross. Will merge when build is green :smile:

Looks like you've got a few style violations @redross. Could you actually add a changelog entry too?

will update

ping @backus

Thanks again @redross!

Thanks for the quick response and review :+1:

Thanks for the quick response and review 👍

Happy to do so. Feel free to open more PRs if any other improvements stand out to you :smile:


upgrade auth version

fix #3605

Failure looks flaky. Is there a way to rerun the AppVeyor build?

A later build succeeded but this pull request wasn't updated somehow. Should be OK to merge.


Add openlayers

Add a simple map using open layers

@Saela I pushed some changes:

The map is now only shown once, by adding a return value in the useEffect() hook (since in development, React.StrictMode forces components to render twice, and OL initializes two maps which do not get removed once hooked)
I switched from useRef to useId because I think that we only need a unique ID, and not the whole ref thing. If I am wrong, please undo it :)
I found a nice blogpost that uses a contextwrapper so we can have the subcomponents of the mapwrapper nicely done as individual components, while allowing them to access the mapobject. As long as we're not manipulating state, that should be a fine approach.

@Saela The current state works 🥳 Some things I did, and which you can maybe expand on for the create/edit functionality:

I changed the coordinate system and added the Swisstopo WMS - because both Swisstopo as well as the OpenLayers WMS are provided in EPSG:3857, setting the view projection to EPSG:4326 led to a somewhat ugly distortion. We'll have to check whether we can get them in EPSG:4326. As of now, we're transforming our GeoJSON objects on the fly from 4326 to 3857 and have undistorted background maps 🪂 However, this could be more difficult in the edit/create view. If it does not work, we'll have to use the proj4 library to transform the coordinates on the fly. I can show you this if need be (boy, how am I excited for working with geodate, and at the same time already triggered because of these projection issues 😆 )
I added a function that generates our markers and a corresponding react component. This could certainly be improved to be more generic, because we're going to have more geometry types in the futures. However, it serves as a proof of concept and can also be used once we're adding the geotagged photos.

@Tugark We can now set markers and also get the coordinates back to save them to the database. This works for both the edit and create view. I also fixed the display of the map.
Create View: After setting two markers I remove the draw interaction. The user has then to move the already set markers to the position he wants if he made a mistake.
I re-used your marker layer but split it into creating the layer and adding the features. Please check if this is ok for you. :)


TFHE haming distance

writing a TFHE implem of the hamming distance algorithm for 256 bytes

Payment request #2544DA has been processed and payment is complete. See you around on OnlyDust.


Put dotfiles paths all on separate lines?

If you split up the stuff in https://github.com/Aviator45003/easy-dotfiles/blob/04fe42ed52e27a5c17c53d068325f21f3eae6144/easy_setup.sh#L85 onto separate lines, you'll get better diffs.

That would mean changing the way the array is declared, unless I can find a workaround.

False. Can just spread out to newlines.


Bluetooth: hci_raw: Fix number of TX buffers

The numbers should have been the sum of HCI commands count and ACL TX
buffers but instead the buffer size was used which makes the pool holds
substatially more buffer than necessary.
Signed-off-by: Luiz Augusto von Dentz<EMAIL_ADDRESS>

The above change makes host allocate more than a controller ACL size of 27 byte payloads.
We can decide to fix it in subsequent PR.

It looks like this could be fixed by splitting the hci_tx_pool into a hci_cmd_tx_pool and a hci_acl_tx_pool. The order of sending should be maintained by the tx_queue
@Vudentz Is there any reason that we cannot split the hci_tx_pool? Or maybe under certain configurations for the USB H4 mode?
@cvinayak In any case I think that should be a different PR.

The above change makes host allocate more than a controller ACL size of 27 byte payloads.
We can decide to fix it in subsequent PR.

It looks like this could be fixed by splitting the hci_tx_pool into a hci_cmd_tx_pool and a hci_acl_tx_pool. The order of sending should be maintained by the tx_queue
@Vudentz Is there any reason that we cannot split the hci_tx_pool? Or maybe under certain configurations for the USB H4 mode?
@cvinayak In any case I think that should be a different PR.

There is no specific reason not to have it split, I thought it would be simpler to implement using the same pool bit given their difference in size that end up adding extra bytes.

@cvinayak @joerchan Done, there should now use dedicated pools for ACL and CMD with their respective sizes and buffer count.

@Vudentz Thanks. LGTM

@Vudentz There appears to be naming conflicts with EDTT HCI test (edtt_ble_test_app/hci_test_app/src/main.c).

@Vudentz There appears to be naming conflicts with EDTT HCI test (edtt_ble_test_app/hci_test_app/src/main.c).

Actually since EDTT is using hci_raw it should be managing its own buffers pool either so I guess it is just a matter of converting that to use bt_buf_get_tx.

@Vudentz There appears to be naming conflicts with EDTT HCI test (edtt_ble_test_app/hci_test_app/src/main.c).

Actually since EDTT is using hci_raw it should be managing its own buffers pool either so I guess it is just a matter of converting that to use bt_buf_get_tx.

Done, in fact we could replace the other pools as well and just use bt_buf_get_rx but there seems to be some timestamping going on, which is actually pretty weird since the likes o monitors, etc, wont be able to decode this properly and they already do their own time stamping.

Done, in fact we could replace the other pools as well and just use bt_buf_get_rx but there seems to be some timestamping going on, which is actually pretty weird since the likes o monitors, etc, wont be able to decode this properly and they already do their own time stamping.

The timestamps that are added by the EDTT HCI app are used in some EDTT tests. The packets go to the Python side of EDTT and don't pass through a host, so there is no need to conform to a specific format.

@cvinayak your "request changes" is blocking merging this. Could you take another look?

Done, in fact we could replace the other pools as well and just use bt_buf_get_rx but there seems to be some timestamping going on, which is actually pretty weird since the likes o monitors, etc, wont be able to decode this properly and they already do their own time stamping.

The timestamps that are added by the EDTT HCI app are used in some EDTT tests. The packets go to the Python side of EDTT and don't pass through a host, so there is no need to conform to a specific format.

Interesting to know, that said I suppose you could timestamp on the Python side instead, since that anyway is using uptime instead of a proper wall timer, anyway this can be changed later if is deemed necessary.

@jhedberg  looks like we have all the approvals necessary to merge this one.


Add variable PKCS11 label support to using_mbedtls_pkcs11

The current "using_mbedtls_pkcs11.c" implementation requires using the device key and device certificate stored under the labels "pkcs11configLABEL_DEVICE_PRIVATE_KEY_FOR_TLS" and "pkcs11configLABEL_DEVICE_CERTIFICATE_FOR_TLS".
This PR updates the NetworkCredentials to include fields for pClientCertLabel and pPrivateKeyLabel, so other labels can be used with PKCS11. This matches the behavior seen in the CSDK.
This PR also updates the "pkcs11_mqtt_mutual_auth_demo" to set the newly-added NetworkCredentials fields.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

As a general note, using_mbedtls_pkcs11.c and iot_tls.c need a fair amount of work to comply with best practices.
For now, you could set the unimplemented function pointers in the pxCtx->privKeyInfo struct to NULL to make it clear that other operations have not been implemented. In particular, the following functions could read/write to the context while assuming the wrong type:

get_bitlen
can_do
verify_func
verify_rs_func (if MBEDTLS_ECP_RESTARTABLE )
sign_rs_func (if MBEDTLS_ECP_RESTARTABLE)
decrypt_func
encrypt_func
check_pair_func
ctx_alloc_func
ctx_free_func
rs_alloc_func (if MBEDTLS_ECP_RESTARTABLE)
rs_free_func (if MBEDTLS_ECP_RESTARTABLE)
debug_func


Modified genotype network such that generate_genotype_network skips the read_sequences function.

This was extraneous, and I think non-pythonic. Watching Raymond
Hettinger’s PyCon video, Pythonic code keeps the main business logic
expressed in the code, with setup/teardown/other stuff kept hidden.
Hence the rationale for this change.
Also made nodes = sequence, while accession = node metadata. Need to write test for this, don't merge just yet.

Okay, I think this one is ready for merging. The major feature change, which is changing from accession metadata being stored as the node, to being stored as part of the metadata, has been made, and the tests have been made that ensure that the end product is correct.


Foreman Export Upstart should not duplicate ports

Fixes #574. Better late than never.

Those failures look pretty suspicious to me.

I'm running CI again. Thanks for tha patch!

Thanks!


Bump to KUDO 0.11.1

also update the stale URL in a way which will not go stale ever

Investigating these failures via https://github.com/mesosphere/kudo-cassandra-operator/pull/70

superseded by #78


touch support in game for surface devices

Is your feature request related to a problem?  Is the launcher not usable with touch controls? You also mentioned the game itself which I can't do anything about.

Hi lacgacyGamerHD
Please help me to fix PojavLauncher Zink Version
My Phone it not work it exit the game


HDDS-1713. ReplicationManager fail to find proper node topology based…

… on Datanode details from heartbeat. Contributed by Xiaoyu Yao.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
42
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
495
trunk passed

+1
compile
252
trunk passed

+1
checkstyle
69
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
935
branch has no errors when building and testing our client artifacts.

+1
javadoc
155
trunk passed

0
spotbugs
316
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
511
trunk passed

_ Patch Compile Tests _

+1
mvninstall
441
the patch passed

+1
compile
259
the patch passed

+1
javac
259
the patch passed

+1
checkstyle
74
the patch passed

+1
mvnsite
0
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
728
patch has no errors when building and testing our client artifacts.

+1
javadoc
155
the patch passed

+1
findbugs
536
the patch passed

_ Other Tests _

+1
unit
270
hadoop-hdds in the patch passed.

-1
unit
1351
hadoop-ozone in the patch failed.

+1
asflicense
46
The patch does not generate ASF License warnings.

6479

Reason
Tests

Failed junit tests
hadoop.ozone.client.rpc.TestBlockOutputStream

hadoop.ozone.client.rpc.TestBCSID

hadoop.ozone.client.rpc.TestOzoneClientRetriesOnException

hadoop.ozone.scm.node.TestQueryNode

hadoop.ozone.client.rpc.TestOzoneRpcClientWithRatis

hadoop.ozone.container.common.statemachine.commandhandler.TestCloseContainerByPipeline

hadoop.ozone.client.rpc.TestOzoneAtRestEncryption

hadoop.ozone.container.TestContainerReplication

hadoop.ozone.client.rpc.TestCloseContainerHandlingByClient

hadoop.ozone.client.rpc.TestSecureOzoneRpcClient

hadoop.ozone.client.rpc.TestCommitWatcher

hadoop.ozone.client.rpc.TestOzoneRpcClient

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1008/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1008

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 1c1ecfa50501 4.4.0-141-generic #167~14.04.1-Ubuntu SMP Mon Dec 10 13:20:24 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / b28ddb2

Default Java
1.8.0_212

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1008/1/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1008/1/testReport/

Max. process+thread count
5147 (vs. ulimit of 5500)

modules
C: hadoop-hdds/server-scm U: hadoop-hdds/server-scm

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1008/1/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

/retest

New patch to support datanode uuid -> ip/hostname -> network path mapping. Also resolve the heartbeat issue.
https://github.com/apache/hadoop/pull/1112

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
39
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
508
trunk passed

+1
compile
259
trunk passed

+1
checkstyle
64
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
823
branch has no errors when building and testing our client artifacts.

+1
javadoc
152
trunk passed

0
spotbugs
332
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
528
trunk passed

_ Patch Compile Tests _

+1
mvninstall
452
the patch passed

+1
compile
261
the patch passed

+1
javac
261
the patch passed

+1
checkstyle
71
the patch passed

+1
mvnsite
0
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
637
patch has no errors when building and testing our client artifacts.

+1
javadoc
161
the patch passed

+1
findbugs
566
the patch passed

_ Other Tests _

+1
unit
287
hadoop-hdds in the patch passed.

-1
unit
1571
hadoop-ozone in the patch failed.

+1
asflicense
43
The patch does not generate ASF License warnings.

6561

Reason
Tests

Failed junit tests
hadoop.ozone.container.ozoneimpl.TestSecureOzoneContainer

hadoop.ozone.client.rpc.TestOzoneRpcClient

hadoop.ozone.client.rpc.TestOzoneClientRetriesOnException

hadoop.ozone.scm.node.TestQueryNode

hadoop.ozone.client.rpc.TestOzoneAtRestEncryption

hadoop.ozone.client.rpc.TestFailureHandlingByClient

hadoop.ozone.container.common.statemachine.commandhandler.TestCloseContainerByPipeline

hadoop.ozone.client.rpc.TestSecureOzoneRpcClient

hadoop.ozone.client.rpc.TestBlockOutputStream

hadoop.ozone.client.rpc.TestBlockOutputStreamWithFailures

hadoop.ozone.client.rpc.TestCommitWatcher

hadoop.ozone.client.rpc.TestMultiBlockWritesWithDnFailures

hadoop.ozone.client.rpc.TestKeyInputStream

hadoop.ozone.client.rpc.TestOzoneRpcClientWithRatis

hadoop.ozone.web.client.TestKeysRatis

hadoop.hdds.scm.pipeline.TestRatisPipelineCreateAndDestory

hadoop.ozone.client.rpc.TestCloseContainerHandlingByClient

hadoop.ozone.container.server.TestSecureContainerServer

Subsystem
Report/Notes

Docker
Client=18.09.8 Server=18.09.8 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1008/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1008

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 3786908b30e1 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 4e66cb9

Default Java
1.8.0_212

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1008/2/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1008/2/testReport/

Max. process+thread count
4839 (vs. ulimit of 5500)

modules
C: hadoop-hdds/server-scm U: hadoop-hdds/server-scm

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1008/2/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

The JIRA has been taken over with a different PR.


[feature request] during undo/redo scroll to the area being modified

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Is your feature request related to a problem? Please describe.
When performing undo/redo draw.io doesn't re-position the diagram and may apply changes to the currently invisible area, which makes it quite difficult to realize what's going on and when to stop.
Describe the solution you'd like
I'd like draw.io to scroll the viewport so that results of undo/redo action are always visible. Ideally, viewpoint should be saved as part of each undo event and restored accordingly.
Describe alternatives you've considered
scrolling back and forth manually
Additional context

Change in 24.4.14.


Raise NotImplementedError when user sends private transaction to Goerli endpoint

Raise NotImplementedError when user sends private transaction to Goerli endpoint.
cfr https://github.com/flashbots/web3-flashbots/issues/54#issuecomment-1133946629
I think this will be more helpful than just letting the endpoint reply with a 400 client error.

looks good, thanks for the PR @Barabazs !


Will spring cloud security supports reactive feature and netty container?

Hi team,
As you know, spring cloud security is implemented with spring mvc and embedded tomcat is used. Is any plan to support some new features of spring boot2 & spring5?  especially for the webFlux and netty instead of tomcat.
Thanks

org.springframework.boot:spring-boot-starter-tomcat:2.0.1.RELEASE (selected by rule)
org.springframework.boot:spring-boot-starter-web:2.0.1.RELEASE
org.springframework.cloud:spring-cloud-security:2.0.0.M3
org.springframework.cloud:spring-cloud-starter-security:2.0.0.M3
org.springframework.cloud:spring-cloud-starter-oauth2:2.0.0.M3
compile

I found that spring cloud gateway can't work with oauth for gateway uses netty and oauth uses tomcat.
Do we have any plan to support  reactive feature and netty container?  Thank you!


seriouspython.run fails with strange python exception

When running the python script with SeriousPython.run, flutter fails with the error:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
NameError: name 'null' is not defined

more details
Running on linux.
python files are organized this way:
app/
  app.zip
  src/
    __init__.py
    main.py

command as used in dart:
...
await SeriousPython.run("app/app.zip",
    appFileName: "main.py",
);
...

The asset app/app.zip was declared in pubspec.yaml:
(...)
flutter:
  uses-material-design: true
  assets:
    - app/app.zip

verbose output of the error:
[  +32 ms] flutter: Start unpacking app archive
[  +18 ms] flutter: Finished unpacking application archive.
[  +11 ms] modulePaths joined string:
/tmp/test_python-1.0.0-1/app:/tmp/test_python-1.0.0-1/app/__pypackages__:/home/
hfechner/Projects/fluttergettingstarted/test_python/build/linux/x64/debug/bundl
e/python3.11:/home/hfechner/Projects/fluttergettingstarted/test_python/build/li
nux/x64/debug/bundle/python3.11/site-packages
[        ] exe_dir:
/home/hfechner/Projects/fluttergettingstarted/test_python/build/linux/x64/debug
/bundle
[        ] app_dir: /tmp/test_python-1.0.0-1/app
[        ] sync: false
[        ] Traceback (most recent call last):
[        ]   File "<string>", line 1, in <module>
[        ] NameError: name 'null' is not defined

The python stuff could be located in a temporary directory:
ls /tmp/test_python-1.0.0-1/app/
>> __init__.pyc  main.pyc

It seems that despite script being some kind of null, serious-python tries to load a script in line 158:
https://github.com/flet-dev/serious-python/blob/1b7cdbd6252a897d1a326cf3d4cb250e1690cdbd/src/serious_python_linux/linux/serious_python_linux_plugin.cc#L156-L160

i'm getting this with the run_example and flask_example on linux.

@WhiteGobo I tried a few things like setting appFileName and sync: true in main.dart of the run_example. I think the error occurs from fl_value_to_string(script). I don't think the value of script was ever a nullptr when I was changing things.

Yeah another fix qould be to replace:
https://github.com/flet-dev/serious-python/blob/e44dff11516019060d4e37e7a4ab900750c0243c/src/serious_python_linux/lib/serious_python_linux.dart#L30-L37
with this:
   Map<String, dynamic> arguments = {
      'exePath': Platform.resolvedExecutable,
      'appPath': appPath,
      'modulePaths': modulePaths,
      'environmentVariables': environmentVariables,
      'sync': sync
    };
    if (script != null) {arguments['script'] = script;}

But it did seem like a clear problem coming from flutter and not from serious-python. But i didnt had time/forgot later to open up an issue there.

Yes that will work and is more direct. The problem seems to be that the type String? is encoded to 'null' when null. A brief search on the internet didn't find more complaints about this, but they're surly out there.


Modify change area notebook to use util functions

Modifying the change area estimation notebook to use the same(ish) util functions created for the (single year) area estimation notebook
This requires the following steps:

[ ] Adapt existing notebook to use util functions
[ ] Add code for computing area estimates from confusion matrix (was not in notebook previously)
[ ] Test by comparing change area estimates for Tigray computed using this notebook and prior analysis (outside repo)
[ ] Test by computing change area estimates for new ROI (e.g., Bure or Jimma)

Is this ready for review @hannah-rae? First time I see these draft PRs 🤔

@hannah-rae should this be closed or on the path to merging?
FYI on colab pip install shapely==1.8.5 is required to avoid an cartopy import error

@adebowaledaniel Could you clarify what's happened here?

Hannah opened a PR to use some of the new util functions
You updated some of the util functions
Is this correct?

Yes, @ivanzvonkov, It updated the following:

the load_ne function to dissolve sub-region (Admin level 2) geometries into one,
m^2 to Ha conversion,
and the unittest for area_util.

Some tasks still need to be completed for this PR; I plan to review them in the coming week.

[ ]  Add code for computing area estimates from confusion matrix (was not in notebook previously)
[ ]  Test by comparing change area estimates for Tigray computed using this notebook and prior analysis (outside repo)
[ ]  Test by computing change area estimates for new ROI (e.g., Bure or Jimma)

Since the recent changes are needed, especially for single-year area estimation, I will implement them in #212 and merge them.

Close w/ merge of #314


Inconsistent failures from bootstrapper test

The bootstrapper test in test/bootstrapper.js sometimes fails for no apparent reason:
38 passing (2s)
1 failing
1) generator-cake:bootstrapper when enabling download creates the bootstrappers:
    AssertionError: build.ps1, no such file or directory
    + expected - actual
    -false
    +true

    at node_modules\yeoman-assert\index.js:64:12
    at Array.forEach (native)
    at Function.assert.file (node_modules\yeoman-assert\index.js:62:26)
    at Context.created (test\common.js:26:14)

I can't see any sign of why this is happening, but is reliable-ish on Travis and VSTS and can be triggered locally by running repeatedly:
npm test && npm test && npm test && npm test && npm test

Seems like the change to got in 33de659 seems to have fixed it, so likely a problem inside yeoman-remote


Cannot find module 'node:path'

Hi! Thank you very much for publishing this addon, I was about to build it myself as n8n is just such an incredible useful thing :)
But sadly the addon dies during startup.
HA Info:
Home Assistant 2022.12.1
Supervisor 2022.11.2
Operating System 9.3
Frontend 20221208.0 - latest
Current hass n8n addon version: 0.0.4
    Error: Cannot find module 'node:path'
    Require stack:
    - /usr/local/lib/node_modules/n8n/node_modules/n8n-core/dist/DirectoryLoad
    er.js
    - /usr/local/lib/node_modules/n8n/node_modules/n8n-core/dist/index.js
    - /usr/local/lib/node_modules/n8n/dist/commands/start.js
    -<EMAIL_ADDRESS>    -<EMAIL_ADDRESS>    -<EMAIL_ADDRESS>    -
<EMAIL_ADDRESS>    -<EMAIL_ADDRESS>    - /usr/local/lib/node_modules/n8n/bin/n8n
    Code: MODULE_NOT_FOUND

That's all of information I found.
Btw, why do you build the n8n image yourself? There is an official image already: n8nio/n8n

Hey,
Thank you for raising an issue, I'll check it out this week.

Btw, why do you build the n8n image yourself? There is an official image already: n8nio/n8n

Probably because when I started using this addon, this image simply did not exist :)
Thank you for pointing this out, I will probably use it and avoid your home assistant to deal with this time and resource consuming build

Can you please check again with the new version ?

yep, looks good. thank you!


Where is the obo file

I used to be able to access the obo file for obi from http://purl.obolibrary.org/obo/obi/obi.obo, it seems to be gone. Is it still accessible?

OBI has always been developed in OWL format, and only OWL format will give you all the logical content in OBI. However the OBO conversion used to work and be done automatically by the Berkeley group. ROBOT should also be able to convert OBI to OBO format, but I'm having some trouble with that, related to https://github.com/ontodev/robot/issues/64.
I'll try to get this sorted out.

I naively tried using Protege to make an OBO version - it failed with many errors similar to this:
ERROR OWLAPIOwl2Obo    MASKING ERROR «the axiom is not translated : OBI_0000705 SubClassOf (BFO_0000055 some
(REO_0000171 and (RO_0000052 some BFO_0000040))) and (OBI_0000293 some BFO_0000040)

My ultimate aim is to load OBI into Chado - I would normally take this route from an OBO file...
cat OBI.obo | go2fmt.pl -p obo_text -w xml - | go-apply-xslt oboxml_to_chadoxml - > OBI.chado.xml
Any suggestions gratefully received.

OBI was not designed with OBO format in mind, so these masking errors have to be ignored.
I'm not sure what Protégé does, but the new 1.1.0 release of ROBOT has a --check false option that will ignore these errors and complete the conversion. This should work:
robot convert --input obi.owl --output obi.obo --check false

For a while now we've been generating a new obi.obo with each release: http://purl.obolibrary.org/obo/obi.obo


Piped (YouTube privacy-friendly frontend) doesn't work

https://github.com/mtxadmin/ublock/blob/242fb4b294a61e756afdfabee10c39508d960d6f/filters/_removeparams#L743
This line prevents Piped from working. So I propose $removeparam=source,domain=~youtube.com|~kavin.rocks. Ideally, we should also add the address of the community instances.

Yes! Thank you for pointing out the issue.

As you can see, earlier I saw this bug on Youtube, it seems, now it's time for youtube-related projects :-)
Made a list of exclusions

It works! :-)
For instance: https://piped.kavin.rocks/watch?v=BYVZh5kqaFg


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_833bbdm_1a/clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_833bbdm_1a/clone/recipe/meta.yaml:32: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_833bbdm_1a/clone/recipe/meta.yaml:36: incorrect_license: The recipe's about/license key is not an SPDX compliant license or license exception, reference https://spdx.org/licenses/
ERROR: /tmp/abs_833bbdm_1a/clone/recipe/meta.yaml:44: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_b9_v32wbl1/clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_b9_v32wbl1/clone/recipe/meta.yaml:32: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_b9_v32wbl1/clone/recipe/meta.yaml:36: incorrect_license: The recipe's about/license key is not an SPDX compliant license or license exception, reference https://spdx.org/licenses/
ERROR: /tmp/abs_b9_v32wbl1/clone/recipe/meta.yaml:44: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found


Allow reading some XDMF time-series à la FEniCS

Currently the XDMF time-series interface accepts the following XML format, with one Collection grid for time-series and one Uniform grid for mesh
<Xdmf Version="3.0">
  <Domain>
    <Grid Name="TimeSeries_meshio" GridType="Collection" CollectionType="Temporal">
      <Grid>
        <ns0:include xmlns:ns0="http://www.w3.org/2003/XInclude" xpointer="xpointer(//Grid[@Name="mesh"]/*[self::Topology or self::Geometry])"/>
        <Time Value="1"/>
        <Attribute>...</Attribute>
      </Grid>
      <Grid>
        <ns1:include xmlns:ns1="http://www.w3.org/2003/XInclude" xpointer="xpointer(//Grid[@Name="mesh"]/*[self::Topology or self::Geometry])"/>
        <Time Value="2"/>
        <Attribute>...</Attribute>
      </Grid>
    </Grid>
    <Grid Name="mesh" GridType="Uniform">
      <Geometry>...</Geometry>
      <Topology>...</Topology>
    </Grid>
  </Domain>
</Xdmf>

With xdmf_file.parameters["functions_share_mesh"] = True, FEniCS writes only one Collection grid. The mesh is

either defined for each time step (a Uniform grid per time-step)

<Xdmf Version="3.0">
  <Domain>
    <Grid CollectionType="Temporal" GridType="Collection" Name="TimeSeries">
      <Grid GridType="Uniform" Name="Results">
        <Time Value="1.0329"/>
        <Geometry>...</Geometry>
        <Topology>...</Topology>
        <Attribute>...</Attribute>
      </Grid>
      <Grid GridType="Uniform" Name="Results">
        <Time Value="1.0429"/>
        <Geometry>...</Geometry>
        <Topology>...</Topology>
        <Attribute>...</Attribute>
      </Grid>
    </Grid>
  </Domain>
</Xdmf>

either defined at the 1st time step (only one Uniform grid in this collection), when one uses the rewrite_function_mesh = False option. See also #461.

<?xml version="1.0"?>
<!DOCTYPE Xdmf SYSTEM "Xdmf.dtd" []>
<Xdmf Version="3.0" xmlns:xi="http://www.w3.org/2001/XInclude">
  <Domain>
    <Grid Name="TimeSeries" GridType="Collection" CollectionType="Temporal">
      <Grid Name="mesh" GridType="Uniform">
        <Topology>...</Topology>
        <Geometry>...</Geometry>
        <Time Value="0" />
        <Attribute>...</Attribute>
      </Grid>
      <Grid>
        <xi:include xpointer="xpointer(//Grid[@Name="TimeSeries"]/Grid[1]/*[self::Topology or self::Geometry])" />
        <Time Value="1" />
        <Attribute>...</Attribute>
      </Grid>
    </Grid>
  </Domain>
</Xdmf>

This PR allows reading these two formats.
By default, we have xdmf_file.parameters["functions_share_mesh"] = False, this means if several functions are exported per time-step, then we have several Collection grids present in the XML file corresponding to those functions. Allowing this kind of time-series will be tackled by future PR's.

Cool.


Adapt adding new referee pattern to adding 2 referees

See documentation in https://github.com/DFE-Digital/bat-design-history/pull/53

This is great.
My only suggestion is to use the red alert at the top above the h1 rather than the exclamation mark text as you have it. I think it's way hard to spot.

@adamsilver Agree, I've updated the code and the screenshots in the design history

Merge!


Simplify Vmdb::PermissionsStore and make it pluggable

This PR simplifies the permissions system by removing the null object pattern and a lot of overhead with configuration.  Instead this moves more toward convention over configuration via pluggability; that is, if a plugin brings a config/permissions.yml it will be honored.
For now, the code also allows core to have a config/permissions.yml, but we could remove that.

[x] Depends on https://github.com/ManageIQ/manageiq-ui-classic/pull/6955 since the UI code depended on the configure and initialize! methods which are removed here.

@chessbyte Please review.

Jansa backport details:
$ git log -1
commit d2e29a0b9827720d1d7af00c0170640a7f009b4c
Author: Oleg Barenboim<EMAIL_ADDRESS>Date:   Tue Apr 7 14:42:27 2020 -0400

    Merge pull request #20053 from Fryguy/pluggable_permissions

    Simplify Vmdb::PermissionsStore and make it pluggable

    (cherry picked from commit 8693547dfcc18257bf1022f93525bdcbee7c81cd)


Resurrect reporting to codecov

setuptools version
main
Python version
N/A
OS
N/A
Additional environment information
No response
Description
I've been doing some security audit after the recent codecov supply chain attack and discovered that there's been no coverage updates on the service since Dec 20.
I tracked it down to this change: https://github.com/pypa/setuptools/pull/2486.
Funny enough, I've left a number of comments reviewing that but still missed this issue ;)
It used to be integrated in Travis CI and AppVeyor

https://github.com/pypa/setuptools/pull/2486/files?file-filters[]=.ini&file-filters[]=.toml&file-filters[]=.yml#diff-6ac3f79fc25d95cd1e3d51da53a4b21b939437392578a35ae8cd6d5366ca5485L60
https://github.com/pypa/setuptools/pull/2486/files?file-filters[]=.ini&file-filters[]=.toml&file-filters[]=.yml#diff-92ab9a36df5d8e9f7076f2fdec59492d1ac2d9cf27ea046767a7fc4d542ef3dcL43

and the toxenv is still in place but its invocation never survives the migration to GHA and so this needs to be corrected.
Fixing this should be pretty straightforward — just integrate https://github.com/codecov/codecov-action.
Expected behavior
Coverage to be actually reported to codecov.
How to Reproduce
Look at https://app.codecov.io/gh/pypa/setuptools/commits — the last known report was 4 month ago.
Output
N/A
Code of Conduct

[X] I agree to follow the PSF Code of Conduct

Hi,
I would like to work on this.
Thanks

Sure, feel free to send a PR.

hi @webknjaz ,
I have drafted a PR (#2693). I have not worked with codecov before (not very confident about it either) which is why I marked the PR as draft.
Let me know if the change looks correct or otherwise.
Thanks

I may have missed few things, checking the build failure and looking into it.
Thanks