feat: add control to skip initiative dialog using modifier keys

What?
This PR adds a control that is defaulted to false that can be configured per client to enable the alt/shift processing of initiative rolls via the combat tracker.
Feedback
I'm a BE dev by trade, so JS is outside my comfort zone, so I am ok with (gentle) feedback on the code.
TODO
The one missing piece to this PR is that I think the modifiers should be configurable. Right now it's hard-coded, but if someone can point me to documentation or code that explains how to create a new keybinding entry and use it within the system, that would be great.
Discussion
I realize this may be a point of contention, given this comment by @arbron
The rationale here is that having a dialog box pop up every time, at least for me and for many others, is disruptive to the workflow. Players can easily be trained on using alt/shift if they find it annoying, and can toggle the feature on. Additionally, modules like midi-qol automate a lot of things precisely to be able to avoid having lots of pop-ups for people to click for things to happen. The current implementation removed the ability for the advantageMode to be calculated and set by a module. I'm not sure this PR fixes that; maybe we should have an event that can be hooked for this purpose.
I can probably make this a module if we're absolutely set on not having this feature, but I still would argue strongly at the very least for a hook so _onCombatantControl() doesn't need to be wrapped by a module.

Gonna close this PR for now. I will maybe just make a module for this, as it really is a narrow thing. And I couldn't really come up with a good answer for

I don't see why we need a setting to control this considering we don't have it for any other rolls (e.g. ability checks, attack rolls).

from @kaelad02.


React Sunstone to support VMRC/Guacamole SSH/RDP/VNC in a federation

Description
Normally when a connection is made via VMRC, the connection is:
  graph TD;
      FireEdge_Reverse_Proxy-->ESX;

In the case of federations, there must be an additional proxy to avoid having to expose the different ESX.
  graph TD;
      FireEdge1_Reverse_Proxy-->FireEdge2_Reverse_Proxy;
      FireEdge2_Reverse_Proxy-->ESX;

Interface Changes
FireEdge , maybe Sunstone

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

@jloboescalona2 thanks to redirect me to this issue and close the duplicated one I opened. So, is not yet implemented this functionality. Please, note that the documentation should be more clear on this.


Can't execute lams

Hello,
I created a Dockerfile with the following:
FROM alpine
RUN apk update
RUN apk add --update nodejs nodejs-npm git openssh-client vim-nox

WORKDIR /root/
RUN git clone git@github.com:my_repo/my_looker.git
RUN ls -l /root/my_looker

WORKDIR /root/
RUN git clone https://github.com/looker-open-source/look-at-me-sideways.git
RUN ls -l /root/look-at-me-sideways
RUN ls -l /root

WORKDIR /root/look-at-me-sideways
RUN npm install

WORKDIR /root/my_looker
RUN node /root/look-at-me-sideways

Then i run the docker and:
~ # pwd
/root
~ # ls -l
total 8
drwxr-xr-x    1 root     root          4096 Sep  6 16:01 look-at-me-sideways
drwxr-xr-x    4 root     root          4096 Sep  6 16:00 my_looker
~ #

All looks ok, but then:
~ # cd my_looker/
~/my_looker # node ../look-at-me-sideways/
~/my_looker #

But it's not doing anything...
Help is greatly appreciated!

Since I see the command is like node <dir> rather than using npm run, I suspect this is related from the change of the command-line entry point from index.js to cli.js. I'll take a look at the docker config to see where this needs to be updated
cc: @jax

Thanks for replying!
Actually it's like that because the README states:
cd <directory where you keep your projects>
git clone https://github.com/looker-open-source/look-at-me-sideways.git
cd look-at-me-sideways
npm install
cd ..
git clone <your-lookml-repo-url>
cd <your-lookml-repo>
node ../look-at-me-sideways

I noticed that and updated the readme to refer to cli.js. Does using the new js path make it work for you?

Hello,
Yes! It's working now!
Many thanks!
David


Add watchOS simulator support

Now (before changes from this PR) iOS jre_emul static library is still built with armv7k architecture. This slice should be a part of a separate watchOS jre_emul library that should also contain slice for watchsimulator sdk. Just like it has been done for tvOS.

Thanks for your contribution!


[<Onnx] onnx.pb.h not created as part of build

Operating system
Windows
Compiler
Visual Studio 2022 Commuity
Steps to reproduce the behavior
vcpkg install onnx --recurse

In a program, include the file

#include <onnx/onnx_pb.h>

Failure logs
Compile
Build started at 2:08 PM...
1>------ Build started: Project: test, Configuration: Debug x64 ------
1>main.cpp
1>c:\vcpkg\installed\x64-windows\include\onnx\onnx_pb.h(53,10): error C1083: Cannot open include file: 'onnx/onnx.pb.h': No such file or directory

Note that there are two '.' (periods) in the file name.
Additional context
No response

@DougRogers Please test whether PR https://github.com/microsoft/vcpkg/pull/37469 solves your issue.

I did
git pull
vcpkg upgrade
vcpkg x-add-version --all

The file was not created.

The file was not created.

What file was not created?

You would need to fetch and checkout that PR's branch before rebuilding the affected packages.
You don't need x-add-version,

@jimwang118 onnx.pb.h

@jimwang118 onnx.pb.h

Are you looking for onnx_pd.h?

@jimwang118 onnx.pb.h

Are you looking for onnx_pd.h?

See first post, end of the error mesage...

See first post, end of the error mesage...

Upstream has removed this header file from the code. https://github.com/onnx/onnx/blob/0bb2775098cb72c5619451f37cc0c8f9f335893f/docs/OnnxReleases.md#:~:text=p PYPI_PASSWORD.-,Source Distribution,-Make sure all

Upstream says that "none of the auto-generated header files"  shall be added to the "Source Distribution".
This has no relevance for the installed header files after the build. They are expected to be generated during the build.
(They also don't add the DLL to the Source Distribution...)
(The same pattern exists in other ports. I just dealt with cld3.)


AttributeError: 'CLLocationManager' object has no attribute 'authorizationStatus'

macOS Catalina
10.15.17
Mid 2012
Followed all the steps.
I've allowed python for location services. whats wrong?

Hi there, please ensure that you are running the latest version of macOS.

Hi there, there is no more update available on this Machine.


Wrapping an iterator doesn't work

This simple pattern doesn't work properly on 0.2.3
use kdam::prelude::*;
for _ in tqdm!(iter) {}

The timer does update on every iteration, but the progress bar doesn't tick up, it stays at zero with inf ETA and ?it/s.
This works as it should though:
use kdam::prelude::*;
let mut pb = tqdm!(total = vec.len());
for _ in vec.iter() {
    pb.update(1);
}

Tangential note: I suggest that .refresh() should be automatically called as soon as the tqdm is created. Otherwise, nothing is displayed until the first call to .update().

This bug is fixed and released to crates.io as v0.2.4
@oersted For patterns like tqdm!(iter) .refresh method is called only once when the first value of iterable is used. If you want continuous output from kdam then use force_refresh = true inside the macro. Also, the timer is started when bar is created. For other patterns like tqdm! macro .refresh method is not called.

Awesome! That was fast, thanks.


Accord steers by angle

https://comma.slack.com/messages/D6RC8F681/convo/C6R4QAVH8-1540360847.000100/
@Gernby is testing limits

See my comment here: https://github.com/energee/openpilot/pull/27


Working  & tested Package

The current ros 2 package on the master branch does not work. Here's a version where most of the features have been tested in real conditions with an M1200d oculus (ENSTA Bretagne has an M750d oculus sonar, but that should work too).
Don't hesitate to contact me if you have any questions.

Many thanks, I cannot check it myself but I propose to accept as is


feat(extraction): accept "arguments" in addition to "command" in compdb

some tools generate "arguments" instead of "command",
both are valid according to https://clang.llvm.org/docs/JSONCompilationDatabase.html

/gcbrun

/gcbrun

rebased from master

It looks like this causes go test -mod=readonly ./kythe/... to fail:
--- FAIL: TestExtractCompilationsEndToEnd (0.00s)
    --- FAIL: TestExtractCompilationsEndToEnd/command (0.00s)
        compdb_test.go:62: Unable to change working directory: chdir /workspace/kythe/go/extractors/config/runextractor/compdb/kythe/go/extractors/config/runextractor/compdb/testdata: no such file or directory
    --- FAIL: TestExtractCompilationsEndToEnd/arguments (0.00s)
        compdb_test.go:62: Unable to change working directory: chdir /workspace/kythe/go/extractors/config/runextractor/compdb/kythe/go/extractors/config/runextractor/compdb/testdata: no such file or directory
FAIL

Looks like when tested by go test testPath and extractorPath becomes invalid as go runs the test in the package directory.
If we fix the paths, it'd still depend on cxx_executor being available somewhere.
Maybe we can continue to skip it in go test based on the presence of TEST_TMPDIR or other conditions?

Looks like when tested by go test testPath and extractorPath becomes invalid as go runs the test in the package directory.
If we fix the paths, it'd still depend on cxx_executor being available somewhere.
Maybe we can continue to skip it in go test based on the presence of TEST_TMPDIR or other conditions?

Skipping based on TEST_TMPDIR works for me.

/gcbrun


Add AI.INFO command to Python and C++ clients

The AI.INFO command has been added to the Python and C++ clients.  The commands allow users to retrieve statistics on scripts and models (metadata and execution information).  The API functions follow the same format as other "information" commands:  users are required to give the address of the database nodes where the command is to be executed.  Like the other "information" commands, the Python client supports multiple addresses.  The results are returned in Python as a list of dictionaries.

Codecov Report

Merging #197 (d40e3e3) into develop (bda7a61) will increase coverage by 0.43%.
The diff coverage is 81.08%.

@@             Coverage Diff             @@
##           develop     #197      +/-   ##
===========================================
+ Coverage    80.84%   81.27%   +0.43%
===========================================
  Files           50       50
  Lines         2730     2767      +37
===========================================
+ Hits          2207     2249      +42
+ Misses         523      518       -5

Impacted Files
Coverage Δ

include/client.h
100.00% <ø> (ø)

include/redisserver.h
100.00% <ø> (ø)

src/cpp/client.cpp
87.90% <75.00%> (-0.44%)
:arrow_down:

src/cpp/redis.cpp
75.93% <83.33%> (+0.38%)
:arrow_up:

src/python/module/smartredis/client.py
74.71% <100.00%> (+0.49%)
:arrow_up:

src/cpp/commandreply.cpp
75.66% <0.00%> (+5.30%)
:arrow_up:


[Run][Plugin Manager] UI: Reconsider spacing on ux

We have now   implemented a plugin management for PT Run into the settings ux. (PR #9872;   Issue #5273)
Enhancement
On some places of the new plugin manager ui the spacing between controls is a bit big.
Like here:

As @niels9001 answered in the pr discussion, we probably do not use the correct spacing as on the other pages.

This is a remaining work item form PR #9872.

Great news!  This was resolved in the newly released 0.33 version of PowerToys!  Head to https://aka.ms/installpowertoys to grab latest.


Cannot run @mediapipe/holistic in the web worker

Please make sure that this is a solution issue.
System information (Please provide as much relevant information as possible)

Have I written custom code (as opposed to using a stock example script provided in Mediapipe): yes
OS Platform and Distribution : Window 10
[MediaPipe version] @mediapipe/holistic0.5.1635989137
Bazel version: n/a
Solution (e.g. FaceMesh, Pose, Holistic): Holistic
Programming Language and version ( e.g. C++, Python, Java): javascript

Describe the expected behavior:
I would like to run @mediapipe/holistic in a to avoid performance issues with other code on the main thread.
Standalone code you may have used to try to get what you need :
Worker
If there is a problem, provide a reproducible test case that is the bare minimum necessary to generate the problem. If possible, please share a link to Colab/repo link /any notebook:
web_worker.js code :

error message :

If I don't use the web worker, it works well without errors.
Other info / Complete Logs :
Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached:

Duplicate of #2506, also related #2574
Big +1 to this though.

Duplicate of

if I don't run it in a web worker, it runs well without errors, but the web page stops for a while.
so I should run it in a web worker.
If I initialize holistic when running in a web worker, an error occurs when I load wasm, how I you fix it?
I think I set the locateFile path well.

Hi @byy01 ,
Did you resolve the issue as per https://github.com/google/mediapipe/issues/3406#issuecomment-1150807432  or still are you facing any challenges.

Hi @byy01 ,
Could you please share code changes to investigate on this issue.


ES6 Class: comments break detection.

Consider:
https://jsfiddle.net/Xotic750/vgark47f/
class/*kkk*/ My1 {}
class//kkk
	My2 {}

console.log(isCallable(My1)); // true
console.log(isCallable(My2)); // true

A possible solution, strip any comments from the string before testing.
https://jsfiddle.net/Xotic750/2q7x5gh3/
'use strict';

var STRIP_COMMENTS = /((\/\/.*$)|(\/\*[\s\S]*?\*\/))/mg;
var constructorRegex = /\s*class\s/;
var fnToStr = Function.prototype.toString;
var isNonES6ClassFn = function isNonES6ClassFn(value) {
	try {
		return !constructorRegex.test(fnToStr.call(value).replace(STRIP_COMMENTS, ' '));
	} catch (e) {
		return false; // not a function
	}
};

var tryFunctionObject = function tryFunctionObject(value) {
	try {
		if (constructorRegex.test(fnToStr.call(value).replace(STRIP_COMMENTS, ' '))) { return false; }
		return true;
	} catch (e) {
		return false;
	}
};
var toStr = Object.prototype.toString;
var fnClass = '[object Function]';
var genClass = '[object GeneratorFunction]';
var hasToStringTag = typeof Symbol === 'function' && typeof Symbol.toStringTag === 'symbol';

function isCallable(value) {
	if (!value) { return false; }
	if (typeof value !== 'function' && typeof value !== 'object') { return false; }
	if (hasToStringTag) { return tryFunctionObject(value); }
	if (!isNonES6ClassFn(value)) { return false; }
	var strClass = toStr.call(value);
	return strClass === fnClass || strClass === genClass;
}

class/*kkk*/ My1 {}
class//kkk
	My2 {}

console.log(isCallable(My1), isCallable(My2));
console.log(isCallable(Object), isCallable(function () {}));

Good catch but man is that some horrible comment placement :-)

Actually, in which engine did you check this? Function.toString.call(class/*kkk*/ My1 {}) in Safari gives me function My1() { } which means that Safari can't do "class" detection, for example.

I know the chances of such poor comments are low, but they are possible. :) I tested it on V8 engines, Google Chrome and Chromium.

Firefox also can't detect it, fwiw.

Thanks for the info, and for the fix.

Just thought I'd leave you a little info. I've been looking a little further to see if there is anything by which to identify a class by in non-V8 environments.
In general it seems that:
Object.getOwnPropertyDescriptor(My1, 'prototype')

Return false for everything, whereas
Object.getOwnPropertyDescriptor(function () {}, 'prototype')

Returns true for writable.
On FireFox, there is a toSource method, and if the class is defined with a constructor like:
class My1 {
  constructor () {}
}

It appears to wrap the string in parenthesis  (function My1 () {})
Neither are reliable methods, but I just though I'd mention what I have found so far.

Thanks. Hopefully someday soon Function#toString will be specified more tightly so that class constructs are recognizable as such, but there will sadly always be browsers where it's impossible to distinguish.


Option to remove native video controls from live stream

[REQUIRED] Describe the feature you'd like:
Configuration parameter to disable native video controls that come from the 'controls' attribute of the video tag.
[OPTIONAL] Describe alternatives you've considered:
I'm unaware if there's a workaround to accomplish this right now.
[OPTIONAL] Additional context:
I don't want to see the default video controls while hovering over the live feed, but from what I'm able to tell, there's no option to disable it.

The option is there in the card.
Live > Live Controls > Built-in video controls
live:
  controls:
    builtin: false


Cert taking much longer after recent updates

New Issue Checklist

[x ] Updated fastlane to the latest version
[x ] I read the Contribution Guidelines
[x ] I read docs.fastlane.tools
[x ] I searched for existing GitHub issues

Issue Description
when running cert to create new certificates, it used to take a few minutes and now it takes between 10-20 minutes
our keychain does have a couple thousand entries, but as I said it would not take this long before, any ideas or similar experience?
Command executed
cert(development: true, keychain_path: "xx/keychain-db")

Complete output when running fastlane, including the stack trace and command used
[11:55:57]: Starting login with user ''
[11:55:58]: Successfully logged in
security: SecKeychainSearchCopyNext: The specified item could not be found in the keychain.
[11:56:22]: Certificate XXXXXX (Development) can't be found on your local computer
(IN A LOOP)
[12:02:01]: Couldn't find an existing certificate... creating a new one
[12:02:10]: Using keychain password from keychain item fastlane_keychain_login2 in /xx/keychain-db
[✔] Setting key partition list... (this can take a minute if there are a lot of keys installed)
[12:04:45]: Using keychain password from keychain item fastlane_keychain_login2 in /xx/login2.keychain-db
[✔] Setting key partition list... (this can take a minute if there are a lot of keys installed)
[12:07:18]: Successfully generated XXXXXX which was imported to the local machine.
security: SecKeychainSearchCopyNext: The specified item could not be found in the keychain.
[12:07:43]: Verifying the certificate is properly installed locally...
[12:07:43]: Successfully installed certificate XXXXX
[12:07:43]: Use signing certificate ‘XXXX’ from now on!
+------+----------------------------+-------------+
|                fastlane summary                 |
+------+----------------------------+-------------+
| Step | Action                     | Time (in s) |
+------+----------------------------+-------------+
| 1    | Verifying fastlane version | 0           |
| 2    | default_platform           | 0           |
| 3    | xcversion                  | 0           |
| 4    | cert                       | 706         |
+------+----------------------------+-------------+
Environment

 [REPLACE THIS WITH YOUR INFORMATION]

@nesfahani Thanks for reporting this! I’ve been seeing this too and its on my list of things to look at 😊


infisical: set official infisical team as maintainers

Description of changes
@akhilmhdh from the Infisical team accepted to maintain the package officially.
I'm removing myself and the other maintainer.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Yes please. We are happy to take it over.
Thank you @jgoux for maintaining this till now.

Have you asked @ivanmoreau?

Have you asked @ivanmoreau?

I didn't, Ivan wrote the initial script and I reworked it to automate the updates and accepted all the PRs since then.
We can wait for his feedback. 👍

@akhilmhdh Just a small reminder: the maintainer list is usually not exclusive, i.e. if someone has made meaningful contributions to a package, we cannot prevent them from adding their name to the maintainer list.

Hey @Aleksanaa We don't mind that but if possible some kinda of comment regarding whose the main stakeholder / primary contact point something of those sort would be helpful.
Like we don't want our customers to ping a person who has no direct relation. Not a biggie but still a concern, that's all


Column coerce not working for pandera.String type when handling Null values from pd.read_csv()

If DataFrameSchema columns are specified with "str" type and pandera pa.String type, as well as Nullable=True and coerce = True, and the dataset has all Nulls in those columns, then the "str" column does not fail the validation check and the pa.String column does fail the validation check.
This occurs if the pandas dataframe is loaded from a csv file. The behaviour is the same (for 'str' and pa.String, if the DataFrame is created from pd.DataFrame()
It appears that coerce=True is not applied when the dtype is pa.String, in situations where the pandas Dataframe was created from a CSV File.

[X] I have checked that this issue has not already been reported.
[X] I have confirmed this bug exists on the latest version of pandas.
[ ] (optional) I have confirmed this bug exists on the master branch of pandas.

Code Sample,
For code example, I have a section of code where the pandas dataframe is created by read_csv() and another section (commented out) where the pandas dataframe is created by a call to pandas DataFrame(). The data in both cases are the same.

import pandas as pd
import pandera as pa

def test_pandera_from_csv1_with_nulls():
    filename = "panderatest1.csv"

    df = pd.read_csv(filename)
    assert len(df.index) == 3, f"Expecting 3 rows of data in {filename}"
    print(df.info())

    # df = pd.DataFrame({
    # "colA": ["A", "B", "C"],
    # "colB": [None, None, None],
    # "colC": [None, None, None]
    # })

    schema = pa.DataFrameSchema({
        "colA": pa.Column(str),
        "colB": pa.Column(str, nullable=True, coerce=True),
        "colC": pa.Column(pa.String, nullable=True, coerce=True),
        })

    try:
        schema.validate(df, lazy=True)
    except pa.errors.SchemaErrors as err:
        print(err.failure_cases.head())
        print(err.data.head())
        assert False, "Some dataframe validation errors"

Expected behavior
A clear and concise description of what you expected to happen.
The expected behaviour is that there are no schema validation errors.
But for the pa.String case, there is a validation error:
schema_context   column  check                                      failure_cases n_failure_cases
Column                colC       pandas_dtype('object')           [float64]                              1
colC which has pa.String type specified gets error reported.
colB which has str type specified does not get error reported.
It seems like colB is doing coerce, but colC does not.
Desktop (please complete the following information):

OS: Windows 10
Pandas version 1.1.5
pandera version 0.6.0

Screenshots
N/A
Additional context
The CSV file source is part of a work flow that is out of my control, so need to keep CSV aspect.
It could be that this is more about how pandas is treating CSV files, but it seems that if pandera can validate with "str" type then it shoudl be able to validate with pa.String type.

Thanks for finding this bug @TerrySnow1963 ! #366 should correctly handle this case now. Will be releasing this along with some documentation improvements in v0.6.1 in the new year!
In the mean time, a workaround for this would be to use the "string", pa.STRING, or pd.StringDtype() pandas-native string data type, it should be able to handle the case of all nulls.

fixed by #366


Kong's log system does not allow you to set fields with dots

Is there an existing issue for this?

[X] I have searched the existing issues

Kong version ($ kong version)
<IP_ADDRESS>
Current Behavior
When I set a custom field to the file-log plugin containing dots, it automatically unpacks the name into nested fields. For example, when I try to set Google's magic trace field logging.googleapis.com/trace, it unpacks as:
{
  logging:
    googleapis:
      com/trace: "my-trace"
}

which does not integrate with google cloud at all.
Expected Behavior
When I set field with dots, I expect it to keep dots
{
  logging.googleapis.com/trace: "my-trace"
}

Steps To Reproduce

Configure file-log setting a custom field containing dots

plugins:
- name: file-log
  config:
    path: "/tmp/file.log"
    custom_fields_by_lua:
      "logging.googleapis.com/trace": "return kong.request.get_header('x-cloud-trace-context')"

Hit any exposed endpoint
See in /tmp/file.log that this field was broken into a nested document

Anything else?
This seems to be the default behavior judging by these lines, but I'd expect to be able to integrate with my cloud provider.
Additional info:

Deployed with public helm chart
Tag: <IP_ADDRESS>-ubuntu

Thanks for reporting this. We can locate the logic handling the dots here.
Maybe we could introduce an escape for the dot. We will discuss this.

Create internal ticket KAG-2601

This should be easy to fix: for example, we could use % to escape . ("%." for . and "%%" for %). But this would be a breaking change. This could fit more in the next major version. I'm trying to confirm this.
Before we have such improvement, maybe we could workaround with a custom plugin.

Why use % rather than something more standard such as \ ?

Why use % rather than something more standard such as \ ?

% is just used to illustrate. To make a choice I will pick a character that is less likely to occur in the name.

The issue is sort of conditionally fixed. That is, when you send a JSON request it works (after this PR #11091), but when you send a form data request it doesn't work because the variable name in the form data contains the . symbol, which is recognized by Kong as nested keys, and then normalize dotted keys in objects which will cause schema violation.

@ms2008 will this fix be a either/or situation in which either all dots in, say custom_fields_by_lia, will be escaped or all of them will break into a nested table?

@catbro666 What about your thoughts?

@ms2008 This should be a separate issue from https://github.com/Kong/kong/pull/11091 . That one is about validation failure when the key contains dots, while this one is because the dot character has a special meaning in the keys of custom_fields_by_lua but there are no ways to escape it.


Mockoon DOWN

🚨 Service Alert: Check the details below
Monitor Details

Monitor Name: Mockoon
Incident Status: TRIGGERED
Severity: critical
Monitor Status: DOWN
Monitor Health Checks: 1
Monitor Failure Threshold: 1

Actions

View Monitor

[start_datetime:1732956060]

The incident has been auto resolved
Total downtime: 1 minutes


New member in members.md

Please resolve merge conflict.

Sorry, pushed wrong guy first... could you please resolve once again?


Changed version to v1

Change API version from V0x1 to V1x0

@syncrou Changed accordingly, please review again


[GIS App] Symbology not working in Firefox

Reported by @Lizabrazil and @dtarb
https://www.hydroshare.org/resource/b7822782896143ca8712395f6814c44b/
Symbology works in Chrome but not in Firefox

This issue was moved to CUAHSI-APPS/tethysapp-hydroshare_gis#1


Rename client -> worker in rate controllers

There is an ongoing rename of clients to workers within Caliper.
There is currently lingering references to client within the rate controllers; this should be modified for:

variables
properties passed to the controller
docs for the rate controllers

I guess this will be resolved by #943 soon


Add layout features to GROBID model

Hi @kermitt2
Something you are already well aware of but I thought it's good to have an issue to record the discussion around it. I am not sure whether you already experimented with adding layout features.
I've started doing it and implemented something here: https://github.com/elifesciences/sciencebeam-trainer-delft/pull/16
Maybe you'll find some of it useful. (I don't want to flood you with too many PRs)

Thank you! Yes it's something we definitively want to add, layout features should bring some improvements and make the models competitive with the current CRF models that are using them (and maybe even better).
Just had a quick look at your implementation, great work I think! We can ignore the lexical features like prefix/suffix (8 first), but also more beyond 9, like shadow number and so on (which have many values) because we already have a character input channel in every architectures. Just one-hot encoding of layout features as you are doing is probably enough I think (note: there is a dense_to_one_hot() function in preprocess.py already used for the case features of one model).
The "gazetteers" features could help, but they typically don't help NER models, so it might also the case here.
We could imagine a first pass in the reader to see the number of values for each feature and use that to select which one will be used for one-hot encoding and concatenation.
Thanks a lot for the great contribution! (and sorry not be very reactive currently)

Finally got around to do some end-to-end evaluation.
Not yet using GROBID's evaluation (the first attempt failed and it doesn't quite fit into my workflow yet).
I haven't done it on the full PMC sample 1943 dataset but rather a random sample of 390.

On our author submitted dataset (not trained on yet) this looks about:

(Since 0.5.5 it's failing to convert a number of those manuscripts which I will need to investigate and are just ignored rather than counted negatively)

Implementation question, do we want to add a parameter in command line that says --use_features  or --ignore-features?
I noticed that @de-code implemented a long list of command-line parameters, which might be hard to navigate, but useful to make quick scripting. What's your opinion?

We probably want yes :)
Likely a --ignore-features I guess, because when features are available I think they will likely improve the results for many grobid models (because they capture layout information), so should be the default.

Thanks!
I'm also wondering whether we should pass the features to the CRF layer in some way, explicitly?

No the CRF layer just acts as activation function before the output, so to compute the probability distributions of the possible labels from the last neuron layer. It's the role of the previous layers to "digest" these additional input features.

I see.
now I'm deciding whether a feature is taken, only when the cardinality of it's possible values is below the feature max length (12).
@de-code implemented an additional parameter that allow the user to select explicitly which features to include. I think that would make the approach more resilient, for example, avoiding low variability (potentially) useless (according the use) features to be included.

I think all the features with cardinality more than 12 are useless because they are all character-based patterns (prefix, suffix, word shape, ...) and the DL architectures have a character input channel already specifically dedicated to this. So these features would very likely be redundant and could actually rather degrade the training (via usual overfitting problems).

OK, indeed.
Since @de-code already implemented something working, I would just integrate it as a list (including ranges). If not specified, the system will try automatic selection as I've implemented now.

Just some random thoughts on the feature indices:

I wasn't sure how fixed the features are across the models, including GROBID submodules. Could they potentially provide different features?
In general it probably doesn't make much sense to provide the first 8 features or so to DeLFT as it should be the responsibility of the model to create those on-the fly. But it's probably just easier to keep them the same as what is currently used for Wapiti.
Keeping the feature indices internally has the advantage that they can be stored in the model config for visibility and being able to load an existing model with those indicies even after changing the default
I personally like being able to do as much hyper parameter "tuning" (hacking) via the command line as possible

Just some random thoughts on the feature indices:
* I wasn't sure how fixed the features are across the models, including GROBID submodules. Could they potentially provide different features?

Yes, it's up to the model designer / design

* In general it probably doesn't make much sense to provide the first 8 features or so to DeLFT as it should be the responsibility of the model to create those on-the fly. But it's probably just easier to keep them the same as what is currently used for Wapiti.

Yes, indeed.

* Keeping the feature indices internally has the advantage that they can be stored in the model config for visibility and being able to load an existing model with those indicies even after changing the default

Very good point.

* I personally like being able to do as much hyper parameter "tuning" (hacking) via the command line as possible

I see, for this I'm not trying to change the current approach at the moment

I personally like being able to do as much hyper parameter "tuning" (hacking) via the command line as possible

There are too many different hyper parameters for each model, often several per layer, plus plenty of possible training parameters, I think it's not manageable with command line. What is often done in libraries supporting several architectures is to have dedicated config files, one for each architecture with the different hyper parameters (a bit like the current config file associated to each produced model).

The question is maybe how much these models make sense outside Grobid. In my original intent, DeLFT was not supposed to provide its own controls over the Grobid models: Only Grobid, with delft interfaced via JEP, trains, evals and runs models because only Grobid can generate the training data with features and the data to be labeled with features (because features and tokens are usually derived from the PDF). So ideally the grobidTagger.py file was not supposed to stay or just as a way to debug models.
However I had a problem to train in DeLFT from Grobid, because the python training process and its “stdout” output often stuck when interfaced with JEP and never ends. I didn't find enough time to solve the problem and I added a training method just calling grobidTagger.py train with an external process. It's working of course but it's more a hack, it should also normally use JEP.
Honestly I don't feel very good building too much stuff to manage grobid training data and models in DeLFT, because it is natural to drive that from Grobid and it would be redundant, painful to maintain. At least having exactly the same input files for both Wapiti and DeLFT is a must I think to keep things minimally simple and transparent.
I don't know if I am very clear with my original design idea, but of course if grobid models as such, independently from Grobid (maybe the date or person parsing models), are useful, this effort could be justified.

Training via GROBID has the advantage that is familiar to someone having trained Wapiti before.
But for me personally, it doesn't work very well.
The main one being that it requires a full GROBID setup and me trying to run the training on a separate machine on-demand.
I do not own a GPU but I borrow it from the cloud for a short period. There are tools to do that from a Python code base but having to also have the GROBID setup (which isn't just a library or CLI call) would be a significant road block. And running the training from DeLFT directly, I can run training in parallel.
There is also no reason why a machine learning expert shouldn't be able to just improve the model via the Python code / DeLFT.
As for CLI parameters vs config file:
Google for example offers Hyperparamer tuning (which I haven't used), but as I understand it, it would also pass parameters to the CLI.
A config file is certainly better than having to make code changes. I would think of config files as something more persistent. For example we generate a config file as part of the model or the GROBID configuration. A config file could describe the default arguments while command line arguments could allow overriding the default. The command line parameter could be scoped and generated based the shared "tuning parameters" available via a config file.
The models themselves could probably considered to be be more generic . But maybe it makes sense for the CLI to be grobid specific until we have other use-cases?

There is also no reason why a machine learning expert shouldn't be able to just improve the model via the Python code / DeLFT.

Ok indeed, good to have the possibility to train in DeLFT/python, I agree!
So the only remaining problem is that DeLFT alone cannot generated the training/eval files with the features (the .train and .test generated by Grobid). We could think about a mechanism to generate thoses files in Grobid and place them automatically in the data/ directory of DeLFT, so that it is easy then to switch to Python for training/tuning/evaluating/etc.

We could add a dropwizard command in the grobid service to handle the integration with delft, such as generate the data, and, if needed other stuff

I would love if we could easily generate the training data. Even more so if we could parallelise it (e.g. via a cluster). A service would work well for that.


Takeoff ✈

I'm just starting to go through the repo and wrap my head around how things work.
I got the following issue:
wait  - compiling / (client and server)...
error - ../../node_modules/@personaelabs/spartan-ecdsa/build/helpers/utils.js:39:0
Module not found: Can't resolve 'fs'

Steps to reproduce my bug:

clone repo
Run yarn install
cd packages/test_client && npm run dev

I just cloned fresh and ran yarn && cd packages/test_client && yarn dev and things worked for me
Which version of node? I'm on 16.14.0 so might need to update

cd packages/test_client

Super weird, I still get that error on your exact Node version and fresh clone. To humor me, could you try cloning from a fresh repo and see what happens?
Also re: the specific error, looks like it's coming from here. I'm wondering how this worked in the past for y'all. I see that the code that uses the fs module is explicitly not used on web. But perhaps it's still giving an import error... weird that you don't have that problem though.

@cha0sg0d
Can you change next.config.js to this and try again?
/** @type {import('next').NextConfig} */
const nextConfig = {
  reactStrictMode: true,
  webpack: config => {
    config.resolve.fallback = { fs: false };
    config.experiments = { asyncWebAssembly: true };

    return config;
  },
}

module.exports = nextConfig

This got it working for me in the next.js.config
/** @type {import('next').NextConfig} */
const nextConfig = {
  reactStrictMode: true,
  webpack: (config) => {
    config.resolve.fallback = { fs: false, readline: false };
    config.experiments = { asyncWebAssembly: true };

    return config;
  },
};

module.exports = nextConfig;

Branch with the changes are here: https://github.com/emergenceland/nym/tree/cha0s/svelte

Continuing to document some basic behavior.
I successfully connected a wallet, made a nym, and entered text to post. Then, when I confirmed the signing of the post, I got the following error:

When I address the bug, which is coming from @ethereumjs ... fromRpcSig(), I get a new error in the same function:

Closed by #22


Fix batch_id type

batch_id is a string according to Sendgrid documentation, but in index.d.ts file batch_id is considered a number.

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

Hello @testica,Thanks again for the PR! We want to show our appreciation by sending                                              you some swag. Could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Is target version, 'ES2015' still experimental?

Built from TypeScript master.
Run node .\built\local\tsc.js -h

feel free to update the comment.


Feature selection

Any particular reason for using the cirrus and panchromatic bands?
No magnetics available?

@RichardScottOZ good point. It doesn't make any sense using these bands. We'll remove these layers soon. Thanks for point that out!
We do have magnetics available, but we're just using analytic signal

If your modelling suggests they add value, it would be surprising, anyway!
Analytic signal makes sense, sure.

We've just removed both cirrus and pan bands, as well as renamed notebooks to more meaningful names, as you suggested. Check PR #28 if you'd like to 😃.


[folding] Fold all nodes matching current line/highlighted text

Feature:
As VSCode supports code-folding based on indentation/nodes, it would be quite helpful to be able to fold/unfold all blocks with a matching text 'head' in a document via command/keyboard shortcut.
There's requests for being able to fold comments, this should handle comments/multiple lines with the same starting character.
Nice to haves:

Be able to manually specify a text string to match (this could be another icon in the search box? IE - find, replace, collapse)
Allow the user to specify string(s) in settings to automatically fold when a document is opened. Probably needs to be per-document type.

My specific use case:
I work with Ansible and makes frequent use of inline Vault-encoded secrets, I often have large chunks of text in my code that are gibberish and I dont need to directly edit. IE:
	-name : "my_web_cert"
     data: !vault |
		$ANSIBLEVAULT;1.1:AES256
<PHONE_NUMBER>2436495734589632895763298756238576234987652384756239854623987568395642
		[repeat for many lines]
		45i35928375483975

It would be very handy to be able to automatically collapse all nodes that start with `data: !vault|". Better yet if that happened by default anytime I opened a file in that project.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


The API file appears to be corrupted. You may need to reinstall Raycast.

Extension
https://www.raycast.com/thomas/visual-studio-code
Description
Error:
The API file appears to be corrupted. You may need to reinstall Raycast.

Steps To Reproduce
The API file appears to be corrupted. You may need to reinstall Raycast.
Current Behaviour
The API file appears to be corrupted. You may need to reinstall Raycast.
Expected Behaviour
The API file appears to be corrupted. You may need to reinstall Raycast.

What version of Raycast and macOS are you using?

@eliUtopia looks like a problem with Raycast itself: https://github.com/raycast/extensions/issues/8856#issuecomment-1768976612
1.60.1 should resolve it


Environment markers do not carry out to transitive dependencies

As in title.
Environment Versions
Ubuntu 14.04, pip-tools 1.9.0, pip 9.0.1, python 2.7 + 3.4
Steps to replicate
Imagine some package 'foo` that requires:
enum34; python_version < '3.4'

Require foo in requirements.txt:
foo

With python 2.7, when you run pip-compile requirements.txt...
Expected result
#
# This file is autogenerated by pip-compile
# To update, run:
#
#    pip-compile --output-file requirements.txt requirements.in
#
enum34==1.1.6; python_version < '3.4'           # via foo
foo==1.0.0

Actual result
#
# This file is autogenerated by pip-compile
# To update, run:
#
#    pip-compile --output-file requirements.txt requirements.in
#
enum34==1.1.6           # via foo
foo==1.0.0

So environment marker is not carried on.
If you run with python3.4+, whole enum34 is omitted.

It's not actually only the environment markers that will affect the dependencies. Also different distribution packages for the same requirement might have different dependencies.
An example of this is provided in issue #558: django-auth-ldap==1.2.15 has three different packages:

django_auth_ldap-1.2.15-py2-none-any.whl requiring django and python-ldap>=2.0;
django_auth_ldap-1.2.15-py3-none-any.wh requiring django and pyldap; and
django-auth-ldap-1.2.15.tar.gz requiring django and either python-ldap>=2.0 or pyldap depending on which python version is used to run the setup.py.

In ideal world everyone would use environment markers, but in reality there are many distributions in PyPI which have different dependencies for the different packages of a release.

Carrying over the environment marker would't be easy, because that's handled directly by pip and we simply get the resulting dependencies after their evaluation.
Even if we'd manage to do this in a maintainable way, we'd have to compute the dependencies for each condition branching from those markers for the output to be exact.
So unfortunately, the generated requirements.txt are not guaranteed to be "environment independent", and IMHO, trying to achieve this would be a really hard task that would require a lot of testing.

I feel the problem. I am new to pip-tools, and I thought https://github.com/jazzband/pip-tools/pull/460 was supposed to make builds environment-agnostic. But it works on main level only then, so the implementation is like half-way there?

#460 was really only about keeping explicit environment markers from the requirements.in, and not being environment-agnostic (you can see the discussion the PR). It's only a feature to deal with simple well known cases with no subdependencies where it is safe to simply keep the environment markers.
As an example, you could add enum34; python_version < '3.4' to your requirements.in, as you know it will be one of your dependencies in your example, and it's risk-free as enum34 has no dependencies, so the resulting requirements.txt should not vary. This gives you an environment-agnostic requirements.txt for this simple case, which could not be done before that PR.

I was the author of the other change (#460). And that's true, it was a tactical improvement not intended to automatically address all possible issues. I think that other change may still help with this issue if you can identify and add the "problem" requirements to your requirements.in file along with an appropriate environment marker.
In essence, you would be working around the lack of a complete solution by manually doing what the tool does not do -- marking the problematic transitive dependencies. I'm curious to know if this helps in your situation. As others have mentioned, the potential true solution is more complex, and I don't believe anyone has worked out a detailed design for how that might work and whether it'd be useful to people, i.e. would the cure end up being worse than the disease. :-)

Manually investigating / marking the transitive dependencies with their environment markers would be too cumbersome. It's just not really a workaround, imo.
Anyway, I have no urgent need for what this ticket is about - I just expected transitive dependencies to be also tackled, but they weren't so raised a ticket to be enlightened :).
I tend to develop on the same OS and major version of python (e.g. 3.5.x) as what my servers are running. So locking not only the versions but also losing the environment markers is not a problem. I would assume this applies to most developers out there?
Utility libraries/SDKs need to be compatible with multiple environments, and cannot lock versions, so there's no use for pip-tools in such projects anyway.
Of course, locking development / test dependencies in library/SDK projects can also be useful. But the lock files pip-tools produces are not "safe" to use if developers work in different OS / python versions. Is the recommended way to tackle that by developing on similar environments?.

I'm thinking you only do this for the problematic ones. No problems, no work to do. :-)
IMHO, developers should work on Vagrant or Docker so the environments are similar. That's just me speaking as a developer; I'm a one-time contributor to pip-tools so I can't speak for the project.
Open source can be a challenge. My employer doesn't mind me contributing, but they don't encourage me to contribute in areas beyond those useful to my own work.
Whenever this topic (complicated dependencies) comes up, there are lots of suggestions, but it never quite seems like anybody really has to have it. And with the complexities involved, it could easily be several days or even a few weeks of work to design, code, and test. Realistically, I don't think it's going to happen until it does become a hard requirement for someone.

I have no urgent need for what this ticket is about

Actually, a (pretty standard) use case where this is needed: When you support multiple python versions for a library, and run tests under tox (or jenkins), then you cannot simply have something like this in tox.ini:
commands = \
    pip install -r requirements.txt

As requirements.txt contains the compiled dependencies against specific platform/environment, and probably doesn't work under something else (e.g. 2.7 vs 3.6).

@tuukkamustonen What prevents you to output frozen requirements per environment and use that as an input to the pip install (with the tox matrix reference) (did not test) ?:
commands = pip install -r requirements_{env}.txt

It's just that producing/maintaining those files would be somewhat cumbersome. I would need to switch between virtualenvs and build the files for each env by hand.
Some sort of tooling would be needed to make that sensible. There was a suggestion in https://github.com/jazzband/pip-tools/issues/635#issuecomment-374421771.
Thought, maybe a simple for loop could provide a good start, something like:
for ENV in 27, 34, 35, 36
    pyenv activate project${ENV}
    pip-tools compile -o requirements.txt requirements.in
end

Actually, a (pretty standard) use case where this is needed: When you support multiple python versions for a library, and run tests under tox (or jenkins), then you cannot simply have something like this in tox.ini:
commands = \
    pip install -r requirements.txt

As requirements.txt contains the compiled dependencies against specific platform/environment, and probably doesn't work under something else (e.g. 2.7 vs 3.6).

Now that 5.0.0 is out, using pip install -r requirements.txt is achievable, if still a bit cumbersome, as incompatibly marked requirements will be safely ignored; see my comment here.

Hey :wave: I encountered this exact problem, and I've written a little friendly alternative to pip-compile that successfully carries out transitive dependencies: see pip-compile-cross-platform.


Correcting Confirm Prompt and language

ConfirmPrompt has a problem with Language and doesn't recognize the standard for the code of the language. So I put to lower the standard to work with the data from the activity. Like that, the activity send us 'fr-FR' and ConfirmPrompt need to recognize 'fr-fr'. So, I take the 'fr-FR' from the activity and put it to lower to be recognize by the ConfirmPrompt
My contact information:
Nathan Pire
<EMAIL_ADDRESS>

This will be addressed in this PR


Update istioctl experimental analyze to correctly handle default namespace

kubectl will inject a default namespace if one isn't included in the yaml being applied. When running istioctl experimental analyze, we should mimic that behavior so we can correctly simulate the impact of applying a yaml file when doing analysis.
Note that not all resources are namespace scoped, so we need some configuration to specify which resource definitions are OK to inject on.
If we're running connected to a kube cluster, we can default to whatever the default namespace for the current kubeconfig / context is. Otherwise, or if we simple want to override the behavior, we can specify a default namespace as a command-line option.
[X] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[X] User Experience
[ ] Developer Infrastructure

Fixed in https://github.com/istio/istio/pull/17064


refactor(LinearAlgeba/BilinearMap): Weaken CommSemiring to SMulCommClass

The results in the CommSemiring section of LinearAlgeba/BilinearMap don't require the semirings to be commutative. It is sufficient that scalar multiplication in the modules is commutative.

Build failing due to:
Error: No space left on device : '/home/lean/actions-runner/_work/_actions/dcarbone/install-jq-action/v1.0.1'

Not something I think I can fix?

That's a thing that happens from time to time, the best you can do is ping maintainers on Zulip so that we can disable and fix the faulty runner. It's the case now so I've restarted the build for you.

bors d+

What's the motivation for this change? Are there any interesting actions of non-commuting elements whose actions commute? I think in various places we avoided this generalization because there was no clear use case, and we expected a performance impact.

@eric-wieser I can explain what I'm trying to do, and you can tell me if I'm going about it in the right way.
When I defined Jordan algebras, I left out quadratic Jordan algebras which are more general as they work without restriction on the characteristic.
Now I've got a bit more used to working with Mathlib, I thought I'd see if I could give the quadratic definition, but that requires quadratic maps. Currently, I think Mathlib only has QuadraticForm, but defined over a non-commutative ring.
I started working on quadratic forms over commutative rings, but it was quickly apparent that it's almost identical to the quadratic forms theory, and we would end up with two files with almost exactly the same results in them. I started thinking about whether it might be possible to develop the theory of quadratic maps over commutative rings and quadratic forms over non-commutative rings simultaneously, and I came up with:
structure QuadraticForm (R : Type u) (M : Type v) (N : Type w) [Semiring R] [AddCommMonoid M]
    [Module R M] [Module Rᵐᵒᵖ M] [AddCommMonoid N] [Module R N] [Module Rᵐᵒᵖ N] [SMulCommClass Rᵐᵒᵖ R N] where
  toFun : M → N
  toFun_smul : ∀ (a : R) (x : M), toFun (a • x) = (a * a) • toFun x
  toFun_smulr : ∀ (a : Rᵐᵒᵖ) (x : M), toFun (a • x) = (a * a) • toFun x
  exists_companion' : ∃ B : M →ₗ[R] M →ₗ[Rᵐᵒᵖ] N, ∀ x y, toFun (x + y) = toFun x + toFun y + B x y

The idea here being that [SMulCommClass Rᵐᵒᵖ R R]  will hold for an associative non-commutative ring as R is a left-module over R and a right-module over Rᵐᵒᵖ.
This seemed to be working, until I got to the SMul section where I needed to define scalar multiplication on QuadraticForm R M N which requires instance : SMul S (M →ₗ[R] M →ₗ[Rᵐᵒᵖ] N) for the bilinear map.
You can see my work-in-progress branch here: https://github.com/leanprover-community/mathlib4/compare/master...mans0954/quadratic-maps
So I guess the options are:

Merge this PR (if the performance is okay)
Develop quadratic maps over commutative rings and quadratic forms over non-commutative rings separately
Limit quadratic forms in Mathlib to only commutative rings
Only have (linear) Jordan algebras in Mathlib?

I'd envisaged that the correct statement was
toFun (a • x) = a • (op a • toFun x)

which corresponds to Q(ax) = aQ(x)a; does that sound right to you?

Note that to replace the existing quadratic forms with your definition, we are first going to need #7152.

Note that to replace the existing quadratic forms with your definition, we are first going to need #7152.

I'd envisaged that the correct statement was
toFun (a • x) = a • (op a • toFun x)

which corresponds to Q(ax) = aQ(x)a; does that sound right to you?

That looks plausible - I think the bilinear map is still B : M →ₗ[R] M →ₗ[Rᵐᵒᵖ] N and we will still need to show that a • B is also M →ₗ[R] M →ₗ[Rᵐᵒᵖ] N?

Yes, I agree with the bilinear map form.
However, I don't think r • B is well-defined if r : R and R is not commutative. Better to define s • B for when the actions of R and S commute, which gives us the special case r • B for when R is commutative.

!bench

I'd envisaged that the correct statement was
toFun (a • x) = a • (op a • toFun x)

which corresponds to Q(ax) = aQ(x)a; does that sound right to you?

@eric-wieser  Do you also need:
toFun (op a • x) = op a • (a • toFun x)

i.e. Q(xa) = aQ(x)a e.g. to get B(x,ya) = B(x,y)a?

No, that feels like a bad assumption to me as it means $$x \mapsto x^2$$ doesn't count as a quadratic form over a non-commutative ring

No, that feels like a bad assumption to me as it means x↦x2 doesn't count as a quadratic form over a non-commutative ring.
Edit: hmm, that seems troublesome anyway...

@eric-wieser I don't see that $$x \mapsto x^2$$  satisfies Q(ax) = aQ(x)a either?

You're right, and that was what my edit was intended to convey.
I think the important question here is my earlier message:

Do you have an example in mind (a choice of R and M) where R is non-commutative but its action on M is commutative?

I think the important question here is my earlier message:

Do you have an example in mind (a choice of R and M) where R is non-commutative but its action on M is commutative?

Yes, sorry, I got a bit lost in all of this earlier and had my wires crossed. I agree now that I don't need this PR to get a scalar multiplication on instance : SMul S (QuadraticForm R M N) - I just need the appropriate SMulCommClass as hypothesis.
However, the next place I get stuck is in composition of a quadratic map with a linear map. Specifically LinearMap.compl₁₂ doesn't work because it expects the rings to be commutative (https://github.com/leanprover-community/mathlib4/pull/7569/files#diff-4b8a995fd51c4cb0a5d0ff6413fa1565f87f697d1b552327cfed2c98a4f62f49R609). What I was hoping for from this PR was to have a version of LinearMap.compl₁₂ that works with SMulCommClass instead of commutative rings.
I've opened a draft PR #7569 for the quadratic maps in case it's easier to discuss this there?

Specifically LinearMap.compl₁₂ doesn't work because it expects the rings to be commutative

Well worse, doesn't it require the rings to be the same?
Note that in the short term you can use ((f ∘ₗ g).flip ∘ₗ h).flip); but it would be great to generalize LinearMap.compl₁₂ to make it work here.
I think we should close this PR; I don't think there is anything worth merging here, and the conversation is still useful even if this PR is closed.


Implement iterators support for Collection and Record

https://developer.mozilla.org/ru/docs/Web/JavaScript/Reference/Global_Objects/Symbol/iterator
Record must iterate over all attributes except undefined, Collection must iterate over its elements.
Needed to support for-of in modern JS.

This is very welcome! Then we can do async looping with await like
for (const model of collection) {
  await model.doStuff();
}

Instead of forEach everywhere


[Bug]-[25]:Copying a Chart widget shows an error for missing JS function.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
I am copying and pasting a chart from one app to another (which came from a sample app in my case). When I do that, I am getting the following error:

This error stays persistent and does not affect the app in anyway. This seems to be a bug that is not breaking in anyway. It is just a nuisance.
Steps To Reproduce

Go to the sample app for Fusion charts.
Copy a chart from it
Paste it into your own app.

Public Sample App
No response
Version
Appsmith v1.8.5-SNAPSHOT
 Front conversations

@muezz it's likely that your chart has a binding to utils.transformPieChartData which is not valid inside the new app and thus you are seeing this

@Nikhil-Nandagopal I know that that is the likely reason but there is no binding for utils.transformPieChartData in my chart widget's properties.

Stats

Stat
Values

Reach
25

Effort (months)
0.5

This should be fixed in our new E-Chart


[Feature Request:] Hold traffic inspection when

Contact Details
<EMAIL_ADDRESS>Is your feature request related to a problem? Please describe.
Good day!
As far as I understood, traffic inspection is going to continue even I close the UI, so kubeshark continues to consume the resources. It would be great if somehow we're able to hold packet inspection in k8s and start it by demand.
thank you so much for your time,
P.
Original Thread
No response
Describe the solution you'd like to see
Start/stop button
Provide additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Coincidentally we just made this change and it will be like that in the next release.

Woo! Looks like The Universe responds to me :)
Thanks a lot, @mertyildiran, appreciate it!


New version: Bedrock-OSS.regolith version 1.3.0

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.
Resolve #161704
Microsoft Reviewers: Open in CodeFlow

Hi @Nusiq, 👋🏼
could you please add following contents to your first comment? This can link this PR to the issue and automatically close issue as completed when this PR is merged. This can also prevent other contributors from wasting their time creating duplicate PRs. ❤
Resolve #161704

/azp run

Hi @Nusiq, 👋🏼
could you please add following contents to your first comment? This can link this PR to the issue and automatically close issue as completed when this PR is merged. This can also prevent other contributors from wasting their time creating duplicate PRs. ❤
Resolve #161704

Done

2024-07-06 19:24:05.223 [FAIL] Installer failed security check. Url: https://github.com/Bedrock-OSS/regolith/releases/download/1.3.0/regolith-1.3.0.msi Result: 0x80004005

@wingetbot run

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-07-08 07:34:52.095 [FAIL] Installer failed security check. Url: https://github.com/Bedrock-OSS/regolith/releases/download/1.3.0/regolith-1.3.0.msi Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
(Automated response - build 886.)

/azp run

https://github.com/microsoft/winget-pkgs/pull/162185
Azure-Pipeline-Passed Moderator-Approved New-Manifest Validation-Completed
#162185 opened 20 hours ago by wingetbot
• Approved
 4
https://github.com/microsoft/winget-pkgs/pull/162183
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#162183 opened 20 hours ago by wingetbot
• Approved
 6
https://github.com/microsoft/winget-pkgs/pull/162167
Azure-Pipeline-Passed Manifest-Metadata-Consistency Moderator-Approved New-Manifest Validation-Completed
#162167 opened yesterday by AClark-WHONET
• Approved
7 tasks done
 5
https://github.com/microsoft/winget-pkgs/pull/162164
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#162164 opened yesterday by wingetbot
• Approved
 6
https://github.com/microsoft/winget-pkgs/pull/162160
Azure-Pipeline-Passed Moderator-Approved Retry-1 Validation-Completed
#162160 opened yesterday by wingetbot
• Approved
 10
https://github.com/microsoft/winget-pkgs/pull/162141
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#162141 opened yesterday by edwardskrod
• Approved
7 tasks
 6
https://github.com/microsoft/winget-pkgs/pull/162112
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#162112 opened yesterday by dotnet-winget-bot
• Approved
7 tasks
@stephengillie
 6
https://github.com/microsoft/winget-pkgs/pull/162105
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#162105 opened yesterday by dotnet-winget-bot
• Approved
7 tasks
 8
https://github.com/microsoft/winget-pkgs/pull/162102
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#162102 opened yesterday by dotnet-winget-bot
• Approved
7 tasks
@stephengillie
 6
https://github.com/microsoft/winget-pkgs/pull/162100
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#162100 opened yesterday by dylanratcliffe
• Approved
7 tasks
@stephengillie
 6
https://github.com/microsoft/winget-pkgs/pull/162067
Azure-Pipeline-Passed Moderator-Approved Retry-1 Validation-Completed
#162067 opened yesterday by oo-cmatuschka
• Approved
 13
https://github.com/microsoft/winget-pkgs/pull/162024
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#162024 opened 2 days ago by zengqiu
• Approved
6 of 7 tasks
 11
https://github.com/microsoft/winget-pkgs/pull/162013
Azure-Pipeline-Passed Moderator-Approved New-Manifest Possible-Duplicate Retry-1 Validation-Completed
#162013 opened 2 days ago by wingetbot
• Approved
 14
https://github.com/microsoft/winget-pkgs/pull/161995
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#161995 opened 2 days ago by wingetbot
• Approved
 11
https://github.com/microsoft/winget-pkgs/pull/161990
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#161990 opened 2 days ago by SpecterShell
• Approved
 15
https://github.com/microsoft/winget-pkgs/pull/161983
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#161983 opened 2 days ago by SpecterShell
• Approved
 17
https://github.com/microsoft/winget-pkgs/pull/161962
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#161962 opened 2 days ago by daiyam
• Approved
 10
https://github.com/microsoft/winget-pkgs/pull/161940
Azure-Pipeline-Passed Manifest-Metadata-Consistency Moderator-Approved New-Manifest Retry-1 Validation-Completed
#161940 opened 2 days ago by SpecterShell
• Approved
5 of 7 tasks
 17
https://github.com/microsoft/winget-pkgs/pull/161936
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#161936 opened 2 days ago by SpecterShell
• Approved
 13
https://github.com/microsoft/winget-pkgs/pull/161913
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#161913 opened 3 days ago by SpecterShell
• Approved
 13
https://github.com/microsoft/winget-pkgs/pull/161889
Azure-Pipeline-Passed Manifest-Metadata-Consistency Moderator-Approved New-Manifest Retry-1 Validation-Completed
#161889 opened 3 days ago by aisgbnok
• Approved
7 tasks done
 1
 21
https://github.com/microsoft/winget-pkgs/pull/161700
Azure-Pipeline-Passed Moderator-Approved New-Manifest Retry-1 Validation-Completed
#161700 opened 5 days ago by Nusiq
• Approved
7 tasks done


Allow custom components in jsx-no-target-blank

What is this?
The following PR allows for customising additional components used jsx-no-target-blank.
Closes https://github.com/yannickcr/eslint-plugin-react/issues/1075
Why do this
It's reasonable common for projects to implement <Link> or <Hyperlink> components and use them literally instead of <a> elements.
How does it work
From the docs
components
An array specifying the names of additional components that should also be validated.
Each array element can either be a string with the component name or object specifying the name and url attribute:
When using an object like so:
{
  "name": "Link",
  "linkAttribute": "to"
}

The following patterns are considered errors:
var Hello = <Link target='_blank' to="http://example.com/"></Link>
var Hello = <Link target='_blank' to={ dynamicLink }></Link>

The following patterns are not considered errors:
var Hello = <Link target='_blank' rel='noopener noreferrer' to="http://example.com"></Link>
var Hello = <Link target='_blank' to="relative/path/in/the/host"></Link>
var Hello = <Link target='_blank' to="/absolute/path/in/the/host"></Link>
var Hello = <Link />

When you use a string such as 'Link', it's equivalent to:
{
  "name": "Link",
  "linkAttribute": "href"
}

@ljharb I'm not too familiar with all the specific rules. A quick search for 'a' only yielded this rule, so I'm not sure what settings might look like given the additionally complexity of the href link attribute option.

I think the current schema for the components array would work fine as a plugin-level setting; we can enhance it in the future as needed.


Document Creating Multiple Attachments Using A Web Browser

Summary
The documentation shows that it is possible to load multiple attachments in a single HTTP PUT, the Content-Type being multipart/related. Web browser methods XMLHTTPRequest and Fetch do not natively produce HTTP requests with this Content-Type.
Desired Behaviour
Publish JavaScript illustrating how to PUT multiple attachments in a single request to CouchDB.
NOTE: I tried to proxy a request crafted with FormData() through Node.js and simply rewrite the Content-Type (keeping the boundary) but this fails with a 500 error.

BTW, Project Fauxton does not support (illustrate) multi file uploads. In other words, there does not seem to be a working example available.


(PUP-6993) Fix acceptance tests for Japanese windows

For the full suite to pass, this also requires the fixes from:

BKR-1008 - PR at https://github.com/puppetlabs/beaker/pull/1293
BKR-1011 - PR at https://github.com/puppetlabs/beaker/pull/1294

CLA signed by all contributors.

I validated this against a custom Beaker branch containing the fixes mentioned above:
Japanese Windows 2012R2 results
CLI invocation:
BEAKER_VERSION=git://github.com/Iristyle/beaker#ticket/master/BKR-1011-fix-Host-group_get-function-on-non-English-Windows GEM_SOURCE=http://rubygems.delivery.puppetlabs.net/ bundle exec rake ci:test:aio BEAKER_HOSTS=hosts.yaml SHA='1.8.1' SERVER_VERSION='2.7.1' TEST=./tests/resource/user/should_modify.rb,./tests/resource/user/should_create.rb,./tests/resource/group/should_create.rb,./tests/windows/eventlog.rb,./tests/modules/install/with_version.rb,./tests/modules/install/basic_install.rb,./tests/parser_functions/puppet_lookup_cmd.rb

With a custom hosts.yaml that uses the Japanese Windows 2012R2 template:

Results:
- Host Configuration Summary -

              - Test Case Summary for suite 'tests' -
       Total Suite Time: 276.63 seconds
      Average Test Time: 39.52 seconds
              Attempted: 7
                 Passed: 7
                 Failed: 0
                Errored: 0
                Skipped: 0
                Pending: 0
                  Total: 7

Standard Windows 2012R2 validation
TEST_TARGET=windows2012r2-64a BEAKER_VERSION=git://github.com/Iristyle/beaker#ticket/master/BKR-1011-fix-Host-group_get-function-on-non-English-Windows GEM_SOURCE=http://rubygems.delivery.puppetlabs.net/ bundle exec rake ci:test:aio SHA='1.8.1' SERVER_VERSION='2.7.1' TEST=./tests/resource/user,./tests/resource/group,./tests/windows/eventlog.rb,./tests/modules,./tests/parser_functions/puppet_lookup_cmd.rb

👍


Update Floating Panel usage

Description
This PR updates usage of the Floating Panel across three samples in response to changes in Toolkit #404
How To Test
Test the 3 changed samples on Catalyst, iPhone and iPad.

@yo1995 will this need a 2nd reviewer?

I'll add Zach to take a look

Updated to Nimesh's proposed fix, ready for re-review


Mis-Redirection from Xamarin.Android page

Currently, there is a link on Deep Dive page which is related to Cross-platform applications, however the current scope of document is Xamarin.Android. Hence, this link would deviate a user or mislead from Xamarin.Android to cross-platform apps halfway through.
Kindly investigate on the same.

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

ID: f4b7d0a1-1c45-0ef1-c11e-491e85d32991
Version Independent ID: c7acc4e2-78a4-98f0-2e64-5e395c974c81
Content: Hello, Android Multiscreen: Deep Dive - Xamarin
Content Source: docs/android/get-started/hello-android-multiscreen/hello-android-multiscreen-deepdive.md
Product: xamarin
GitHub Login: @conceptdev
Microsoft Alias: crdun

Hi @mehul19851, thanks for your feedback. A large number of Xamarin developers are interesting in cross-platform development, either with Android and iOS, or sharing existing .NET code with their Android apps.
The link you are referring to is at the end of the document, and is prefaced with an explanation of what is in the link: "Next, you'll learn to build cross-platform applications with Xamarin".
The very first item is then Understanding the Xamarin Mobile Platform, which we think can be useful for all Xamarin developers.
I hope that helps explain why the link is provided.


 U: .

 U: .

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

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

_ master Compile Tests _

+1 :green_heart:
mvninstall
2m 50s
master passed

+0 :ok:
refguide
2m 44s
branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

+1 :green_heart:
spotless
0m 42s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 35s
the patch passed

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

+0 :ok:
refguide
2m 13s
patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

+1 :green_heart:
spotless
0m 40s
patch has no errors when running spotless:check.

_ Other Tests _

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

14m 39s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense refguide spotless

uname
Linux d8e7972712f4 5.4.0-144-generic #161-Ubuntu SMP Fri Feb 3 14:49:04 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / ddc67527f3

refguide
https://nightlies.apache.org/hbase/HBase-PreCommit-GitHub-PR/PR-5290/1/yetus-general-check/output/branch-site/book.html

refguide
https://nightlies.apache.org/hbase/HBase-PreCommit-GitHub-PR/PR-5290/1/yetus-general-check/output/patch-site/book.html

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

modules
C: . U: .

 U: .

 U: .

Console output
https://ci-hbase.apache.org/job/hbase-kustomize-github-pr/job/PR-5290/1/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

_ master Compile Tests _

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

+0 :ok:
refguide
3m 37s
branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

+1 :green_heart:
spotless
0m 45s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

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

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

+0 :ok:
refguide
2m 49s
patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

+1 :green_heart:
spotless
0m 51s
patch has no errors when running spotless:check.

_ Other Tests _

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

18m 50s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/hbase-kustomize-github-pr/job/PR-5290/1/artifact/yetus-general-check/output/Dockerfile

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

Optional Tests
dupname asflicense refguide spotless

uname
Linux 242d6e851396 5.4.0-144-generic #161-Ubuntu SMP Fri Feb 3 14:49:04 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 68da890dca

refguide
https://nightlies.apache.org/hbase/hbase-kustomize-github-pr/PR-5290/1/yetus-general-check/output/branch-site/book.html

refguide
https://nightlies.apache.org/hbase/hbase-kustomize-github-pr/PR-5290/1/yetus-general-check/output/patch-site/book.html

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

modules
C: . U: .

Console output
https://ci-hbase.apache.org/job/hbase-kustomize-github-pr/job/PR-5290/1/console

versions
git=2.34.1 maven=3.8.6

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

This message was automatically generated.

Oh, seems you have already requested a jira account. I have approved the request. Please finish the registration and open an issue.
Thanks!

Ok, finished. @Apache9 👀


[UI] Setup process - success circle cut off

Issue description
The "success circles" in the setup process seem to be cut off on the sides.
Wallet version
v0.3.1
System specification
macOS 11.3 Beta
Errors

This is as designed.


Interface as out Foo in method of another interface leads to slang internal error.

I've tried to make a small repro case, but failed.
To reproduce, check out https://gitlab-master.nvidia.com/tdavidovic/Falcor/-/commits/bug/tdavidovic/internalpathtracer_out_interface compile, run Scene2Test.exe.
You will get this error:
(Error) Failed to link program:
Rendering/Materials/StandardMaterial.slang RenderPasses/InternalPathTracer/TracePass.cs.slang (main)

D:\Projects\Falcor_2\Source\Internal\RenderPasses\InternalPathTracer\InternalPathTracer.slang(358): note 99999: an internal error threw an exception while working on code near this location
    struct GeneratePathClosestHitQuery : IClosestHitQuery
                                         ^~~~~~~~~~~~~~~~
(0): error 99999: Slang compilation aborted due to an exception of class Slang::InternalError: unimplemented: assignment

We have a workaround (return the interface from the return statement instead), but it still seems like a bug worth fixing.

(0): error 99999: Slang compilation aborted due to an exception of class Slang::InternalError: unimplemented: assignment

It must be from here.
I guess something might be wrong with an assignment.
We will need a repro case.

 So, that's the branch up there.


BeforeTestRun not run in .NET462 up to .NET481 in multitarget test project

Reqnroll Version
2.0.0
Which test runner are you using?
xUnit
Test Runner Version Number
2.8.1
.NET Implementation
.NET Framework 4.6.2 or later
Test Execution Method
ReSharper Test Runner
Content of reqnroll.json configuration file
{
"$schema": "https://schemas.reqnroll.net/reqnroll-config-latest.json",
"bindingCulture": {
"language": "en-us"
},
"language": {
"feature": "en-us"
}
}
Issue Description
After upgrade Reqnroll from version 1.0.1 to 2.0.0 method marked with BeforeTestRun attribute is run only on .net6.0 .net7.0 .net8.0 in multitarget integration test project - in .net 4.6.2 till 4.8.1 it is not executed at all which cause all tests dependant on that initialization to fail
Steps to Reproduce

please clone repo https://github.com/AdaskoTheBeAsT/WkHtmlToX
run all integration tests
modify file located in .\test\integ\Directory.Build.props (in repo under this link https://github.com/AdaskoTheBeAsT/WkHtmlToX/blob/master/test/integ/Directory.Build.props)
with content

<PackageReference Include="Reqnroll" Version="2.0.0" />
<PackageReference Include="Reqnroll.Tools.MsBuild.Generation" Version="2.0.0" />
<PackageReference Include="Reqnroll.xUnit" Version="2.0.0" />

rebuild
run all integ tests
only tests for .net 6.0 till .net 8.0 will run successfully

Link to Repro Project
https://github.com/AdaskoTheBeAsT/WkHtmlToX

Reqnroll registers itself as an xUnit test framework:
[assembly: global::Xunit.TestFramework("Reqnroll.xUnit.ReqnrollPlugin.XunitTestFrameworkWithAssemblyFixture", "Reqnroll.xUnit.ReqnrollPlugin")]

But when xUnit tries to load the type (Assembly.GetType) it fails (returns null) and the registered Reqnroll test framework is not used (and so the BeforeTestRun is not called).
Note: Trying to load the type with Assembly.GetTypes() throws a ReflectionTypeLoadException with a LoaderException stating that xunit.execution.dotnet can't be loaded.
This is probably related to the change to netstandard2.0 and an xUnit restriction described in the following discussion https://github.com/xunit/xunit/discussions/2576. 😢


[GLUTEN-4652][VL] Fix min_by/max_by result mismatch

What changes were proposed in this pull request?
Fix min_by/max_by result mismatch. Take max_by for example, we need to keep intermediate result row like <null, 11> which will be compared with another result like <5, 8> and assure final result is <null, 11>.
How was this patch tested?
New UT added

@rui-mo Can you help review this PR? Thanks.

@rui-mo your comments are addressed. Please help review again. Thanks.

I found the result still not right after applying this patch.
Run SQL:
set spark.sql.leafNodeDefaultParallelism=2;
select min_by(a, b), max_by(a, b) from values (5, 6), (null, 11), (null, 5) test(a, b);

Expected result is
NULL    NULL

Actual result is
5      5

@yma11 @rui-mo I created a follow up PR #5711. Can you help to review it?

Also cc @ulysses-you


How to port coroutine-style code & event loops to CIB ?

Hi I have just watched the seminar on the cib.hpp library. I like how we can modularize hardware functions as services, and bind them with external commands, without runtime overhead.
Following #30 , how do I refactor a FreeRTOS-style code from the following LED blinker example to cib.hpp? Here, there are two co-routines performing their own initializations, and then enters their own event loops iterating at 1000ms and 200ms correspondingly. I am assuming a single-thread application compiled by either avr-gcc or stm32-aarch64-gcc.
volatile bool is_led_on = false;

void loopLED() noexcept {
   setupLED();
   bool led_blink_state = true;
   for(;;) {
         if(is_led_on) {
              digitalWrite(led_blink_state);
              led_blink_state = ! led_blink_state;
         } else {
              digitalWrite(false);
         }

         using std::chrono_literal;
         Rtos.resume_after(1000ms);
   }
   teardownLED();
}

void loopButton() noexcept {
    setupButton();
    for (;;) {
        if (button1.is_pressed()) {
            is_led_on = ! is_led_on;
        }
        using std::chrono_literal;
        Rtos.resume_after(200ms);
    }
    teardownButton();
}

In particular, how do I refactor the above setup -> event loop -> teardown code to utilize CIB's compile time init() flow builder? How do I bind the button press event to the LED service? I hope to watch and learn how it is done in the CIB style.

Note to self: cib is a composition library for "gluing" MCU drivers together. It works with co-routines e.g. FreeRTOS, and does not replace it. The above coroutine code can be integrated with CIB by:
--- before.cpp	2024-01-20 13:31:38.323275734 -0800
+++ after.cpp	2024-01-20 13:32:55.599034010 -0800
@@ -1,16 +1,12 @@
 void loopLED() noexcept {
-   setupLED();
-   bool led_blink_state = true;
+   nexus.service<SetupLED>();
    for(;;) {
-         if(is_led_on) {
-              digitalWrite(led_blink_state);
-              led_blink_state = ! led_blink_state;
-         } else {
-              digitalWrite(false);
-         }
+	 // internal state "led_blink_state" handled by device library,
+	 // e.g. by Boot::SML
+	 nexus.service<BlinkLED>();

          using std::chrono_literal;
          Rtos.resume_after(1000ms);
    }
-   teardownLED();
+   nexus.service<TeardownLED>();
 }

Although it begs the question how the advanced message passing interfaces of FreeRTOS (e.g. send messages among services, semaphores, instances of services) can be integrated with CIB.
References:

https://github.com/intel/compile-time-init-build/discussions/103#discussioncomment-3461046
https://github.com/intel/compile-time-init-build/discussions/237


config.TLS object format?

Can someone share an example of what the config.TLS object should look like?
I'm looking at the etcd client code and I don't see anything like that.
Am I missing something here? I'm trying to pass my ca-cert, client-cert and client-key files to the client

ended up create a new Transport to export the TLS object


Update example.rst

The example code had missing n_clf variable definition. Added it as number of classifiers.
There was a mismatched parenthesis in the voting example. Removed it to make the example work
I have tried to copy the modified code into a fresh notebook and it works seamlessly
No tests or modules are added for this change
This should be considered as a documentation improvement only. (No code changes)

All Submissions Basics:

[ X] Have you followed the guidelines in our Contributing document?
[ X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[ X] Have you checked all Issues to tie the PR to a specific one?

All Submissions Cores:

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?

[ ] Have you written new tests for your core changes, as applicable?

[ ] Have you successfully ran tests with your changes locally?

[ ] Does your submission pass tests, including CircleCI, Travis CI, and AppVeyor?

[ ] Does your submission have appropriate code coverage? The cutoff threshold is 95% by Coversall.

New Model Submissions:

[ ] Have you created a .py in ~/pyod/models/?
[ ] Have you created a _example.py in ~/examples/?
[ ] Have you created a test_.py in ~/pyod/test/?
[ ] Have you lint your code locally prior to submission?

Pull Request Test Coverage Report for Build 2005

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.614%

Totals

Change from base Build 2003:
0.0%

Covered Lines:
5014

Relevant Lines:
5244

💛  - Coveralls


web4 library stable

Get the web4 library stable, no UI yet, but all APIs implemented.
This is a milestone and tracks a number of other issues

[ ] KeyBase Controller / encrypted storage
[ ] KeyRing ed25519 simple
[ ] KeyRing ed25519 hd
[ ] Tx Registry (per chain)
[ ] BCPDemo buildTx function
[ ] Tendermint-trust client
[ ] BCP client
[ ] web4 private API
[ ] web4 public API
[ ] ts-repl bindings for developer cli (use both public/private)
[ ] TESTS!

@ethanfrey , just curious: may I ask why you don't use the https://github.com/iov-one/iov-core/milestone/1 for tracking?

Cuz I made than a long time ago and forgot about it....

We have a project board to track cross-project work, but I don't know if it is public

The main dev is on vacation for a bit, when he comes up, I'll work with him to reboot with a public task board. There was not so much outside interest until now, but you are right, it is nice visibility.


Vim9: confusing error when executing custom buffer-local command not installed yet

Describe the bug
In Vim9 script, executing a custom buffer-local command, which has not been installed yet, might give a confusing error message.
To Reproduce
Run this shell command:
vim -Nu NONE -S <(cat <<'EOF'
    vim9script
    au FileType vim Foo3bar
    set rtp^=/tmp/.vim
    mkdir('/tmp/.vim/ftplugin', 'p')
    writefile(['com -buffer Foo3bar echo "Foo3bar"'], '/tmp/.vim/ftplugin/vim.vim')
    com Foo echo 'Foo'
    filetype plugin on
    set ft=vim
EOF
)

E1144 is raised:
E1144: Command is not followed by white space: Foo3bar

This is expected (not really, but anyway it makes sense).
The message tells us that we've forgotten to separate the name of an Ex command from its argument(s).  But it does not tell us which command name.
Expected behavior
The message tells us what is the name of the command which must be separated from its argument(s):
E1144: Command ":Foo" is not followed by white space: Foo3bar
               ^----^

Environment

Vim version: 8.2 Included patches: 1-2808
OS: Ubuntu 16.04.7 LTS
Terminal: xterm(367)

Additional context
The current message is confusing if we have a lot of custom Ex commands, and we've forgotten that Foo was one of them.  In that case, we might lose time trying to figure out what's wrong in the definition of :Foo3bar, while actually the issue is that :Foo3bar has not been installed yet, and Vim wrongly thinks that we want to execute :Foo followed by the argument 3bar.

I found this issue while I was trying to make Vim automatically execute a plugin's command (:Vim9asmFocus) in a special type of buffers (vim9asm):
au FileType vim9asm Vim9asmFocus

A confusing error was raised:
Error detected while processing function vim9asm#disassemble[31]..FileType Autocommands for "vim9asm":
E1144: Command is not followed by white space: Vim9asmFocus

I thought that something was wrong in the definition of :Vim9asmFocus.  In reality, the command was fine, but it was not installed yet.  So, Vim thought that I was trying to execute the :Vim command (another custom command) with the argument 9asmFocus.

If we have a :Foo command, and Vim finds out that :Foo3bar/:Foo_bar is not a valid Ex command, it gives back 3bar/_bar (which is then handled like an argument):
vim9script
com -nargs=* Foo eval 0
Foo3bar

E1144: Command is not followed by white space: Foo3bar

vim9script
com -nargs=* Foo eval 0
Foo_bar

E1144: Command is not followed by white space: Foo_bar

But, if we have a :Foo command, and Vim finds out that :Foobar is not a valid command, it does not give back bar:
vim9script
com -nargs=* Foo eval 0
Foobar

E492: Not an editor command: Foobar

This seems inconsistent.

I found this issue while I was trying to make Vim automatically execute a plugin's command (:Vim9asmFocus) in a special type of buffers (vim9asm):

To be more clear, it was not really a simple autocmd.  It was a command calling an autoload function which executed buffer-local commands which only existed in some special buffers.
The command created those special buffers, and executed the local commands inside them.  The code was fine, but an error was still raised at compile time, because Vim didn't recognize the names of the commands (they would only be installed later, at runtime):
com MyCommand autoload#func()
def autoload#func()
    ...
    BufferLocalcommand
    ...
enddef

The solution in those cases is to suppress the compilation with :execute:
com MyCommand autoload#func()
def autoload#func()
    ...
    execute 'BufferLocalcommand'
    ...
enddef

But the solution would have been easier to find if the error message had been more clear, by also including the name of the command which Vim thought was being executed; and not just the full command line where it's not always obvious what's the name of the command and what are the arguments.


fix: backport knowledge-agents 24.08 changes to versioned folder.

Description
Companion PR to https://github.com/eclipse-tractusx/eclipse-tractusx.github.io/pull/1006 in order to sync the versioned folder of the knowledge agents kit 24.08 with its main version.

Pre-review checks
DEPs are outdated already in the main branch. Did not do any changes to the modules.
Please ensure to do as many of the following checks as possible, before asking for committer review:

[ ] DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
[x] Copyright and license header are present on all affected files

I updated the DEPENDENCIES file → checkbox can be checked

I also created an IP Ticket because of the number of changes in this PR

@jSchuetz88 IP Ticket is still under review, what about the committers?

LGTM!

Thx, still waiting for the IP review

LGTM!

Thx, still waiting for the IP review

@jSchuetz88 IP issues was approved!!

@jSchuetz88 can you please have a look? This PR is older and updates an already released KIT version. THX


Update troubleshooting symlinks configuration

config.resolve.symlinks is not a function
What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[x] Docs
[ ] Underlying tools
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

It is. Please note it's in chainWebpack not configureWebpack.


Add create or update project command

This PR adds a command that allow users to create or update projects as needed.

LGTM @chaws :+1:


How do I disable the tab names while they are not active?

I'm sorry for maybe asking a silly question, I want the only icon be shown on tabs (while it is not active) like it is previewed on readme file. Any suggestions to make it?

Nvm, just realized it is pinned windows, sorry for bothering, new to mozilla


More detailed Sublime Text install instructions

People might typically jump directly to their editor and wonder where the part about installing rust-analyzer is – at least I did. I added a link to the relevant section for ST.
Make ST instructions more detailed and user friendly (especially beginners), include troubleshooting tips.
Minor grammar improvements throughout.

Thanks!
bors r+

bors r-
bors d=lnicola
:)

I think that's all for now? Thanks!
bors r+

Yes, I'm happy to leave it like this. Thanks for your comments! :)


Debug adding backstroke links to queue when they haven't changed

Currently, a lot of Backstroke links are added to the queue even though they don't actually have an update that they need to have made to them. I need to debug this.

Figured this out. https://github.com/backstrokeapp/server/releases/tag/v2.0.1


Some remarks about your license texts

Just some remarks.
Your plugin should have a readme, license and a history tab ;) see: https://kookma.github.io/TW-Gatha/#%24%3A%2Fplugins%2Fkookma%2Fgatha
Your license full text is included in every plugin, which from my point of view is a bit of bloat. ... BUT that's only my opinion, that I wanted to let you know. ... It's perfectly fine, if you ignore it!
I'm using something like this: https://wikilabs.github.io/editions/bundler/#%24%3A%2Fplugins%2Fwikilabs%2Fbundler%2Flicense, because some of my plugins are small and the full license text would need more space then the plugin itself.

Hi Mario,

Your plugin should have a readme, license and a history tab
Many thanks, It is strange I have forget this!

I'm using something like this: https://wikilabs.github.io/editions/bundler/#%24%3A%2Fplugins%2Fwikilabs%2Fbundler%2Flicense, because some of my plugins are small and the full license text would need more space then the plugin itself.

Good idea! Thank you


Add --add-opens options required for Gradle

In order to work around illegal accesses in Gradle.
Workaround for https://github.com/gradle/gradle/issues/15538
This should fix the JDK16 build.
Signed-off-by: Yoann Rodière<EMAIL_ADDRESS>

Merged https://github.com/hibernate/hibernate-reactive/pull/477
Thanks!


tripDur value short by one epoch length

Trip duration values seem to be short one epoch length. For example, hbGPS identified a trip in my GPS data marking 12 rows of 30s epoch data (6 minutes) to be one trip, but tripDur value was 330 rather than expected 360.

Thank you for reporting this, I have not had time yet to look into it. If you are familiar with github and package development then I would welcome active efforts to help improve the repository with pull requests.