[Feature][Transformer] Supporting fake data generation in transformer for sensitive data masking options

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
Hi All Following to the short discussion I create this issue.
https://github.com/apache/seatunnel/discussions/7746
So there is an idea and goal to source and sink completely full postgres (or etc) to another postgres (source) with data masking or generation fake data for sensitive attributes. Good to know that there are a lot of fakesource available with random generators but at this moment I don't know is it working in transformer or not. Also some good news that there is dynamic compilation available for some completely custom cases.
What do you think?
Usage Scenario
Some maybe will try to use Transformer in case of masking and fake generation.
The real case is to make data synchronization from prod to test environment with some predefined option by user request

[ ] Support fake data generation in transformer for sensitive data masking options in full DB sync case or partial

Related issues
Supporting fake data generation in transformer
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

How about support join with dimension table (fake source is one type of dimension table)?
I think we can extend this requirement to any source.
eg:
join with jdbc
transform {

    JoinWithSource {
        join_on = "source.id = type_bin.item_id"
        source = [
            Jdbc {
                url = "jdbc:mysql://localhost/test?serverTimezone=GMT%2b8"
                driver = "com.mysql.cj.jdbc.Driver"
                connection_check_timeout_sec = 100
                user = "root"
                password = "123456"
                query = "select * from type_bin"
            }
        ]
    }
}

or join with fake source
transform {
    JoinWithSource {
        join_on = "source.id = fake.c_int"
        source = [
            FakeSource {
                row.num = 5
                schema {
                    fields {
                    c_string = string
                    c_tinyint = tinyint
                    c_smallint = smallint
                    c_int = int
                    c_bigint = bigint
                    c_float = float
                    c_double = double
                    }
                }
            }
        ]
    }
}

Then we can use SQL transform to filter data you want.

Or join with sql transform
env {
  parallelism = 10
  job.mode = "BATCH"
}
source {
    Jdbc {
        url = "jdbc:mysql://localhost/test?serverTimezone=GMT%2b8"
        driver = "com.mysql.cj.jdbc.Driver"
        connection_check_timeout_sec = 100
        user = "root"
        password = "123456"
        table_path = "testdb.table1"
        query = "select * from testdb.table1"
        split.size = 10000
    }
    FakeSource {
                row.num = 5
                schema {
                    fields {
                    c_string = string
                    c_tinyint = tinyint
                    c_smallint = smallint
                    c_int = int
                    c_bigint = bigint
                    c_float = float
                    c_double = double
                    }
                }
            }
}

transform {
    sql {
        query = "select * from table1 join table2 on table1.id = table2.id"
    }
}

sink {
  Console {}
}


Build(deps-dev): Bump ts-loader from 9.2.6 to 9.2.7 (#2968)

Pull request opened by the merge tool on behalf of #2968

An unexpected error occurred while processing this pull request. The request might have been partially processed. Caused by: GitHub API query failed: got errors in response: [{'type': 'UNPROCESSABLE', 'path': ['createPullRequest'], 'locations': [{'line': 2, 'column': 9}], 'message': 'A pull request already exists for inmanta:dependabot/npm_and_yarn/ts-loader-9.2.7.'}]


The native git_repository rule is deprecated.

In Install.md, it told me to install gflags in Bazel by edit WORKSPACE:
git_repository(
    name = "com_github_gflags_gflags",
    commit = "<INSERT COMMIT SHA HERE>",
    remote = "https://github.com/gflags/gflags.git",
)

However, git_respository is deprecated:
no such package '@com_github_gflags_gflags//': The native git_repository rule is deprecated. load("@bazel_tools//tools/build_defs/repo:git.bzl", "git_repository") for a replacement.
Use --incompatible_remove_native_git_repository=false to temporarily continue using the native rule. and referenced by '//external:gflags'

Environment:
Ubuntu 16.04, bazel 0.21.0

Oh I've fixed. Thanks.


image translate animation not working

I have a simple image translate animation. it's fine with the lottie player but dosen't work with pixi-ae.  only the last frame is visible.

@imokya
It has been verified to work with version 5.4.1 of 'lottie-web (bodymovin)', but it does not seem to work with the recent 'lottie-web (bodymovin)' AfterEffects plugin.
we could reproduce it, so we will try to fix it


Training with custom dataset but it stuck in an infinite loop

I try to train the mask2former for own dataset. The annotation format is same with coco format. Just resize the image size to (512, 512) and change normalize parameter.
It does print any error message so i can't find the problem. Have anyone get same problem? Thanks.
There's where the program stuck in an infinite loop.

Can you set some breakpoints to locate where it gets stuck?

same here

same here

I fixed it, turns out the classes list has to be given to the dataset, also make sure the num classes are correct everywhere and defined in model and also in dataset config


Developer experience - all the features in dev mode

What does this PR do
https://github.com/nix-rust/nix/pull/2091 left developer experience in a bad state: as far as tools are concerned by default - the project is broken. cargo test fails with lots of bogus errors, rust-analyzer ignores majority of the code, etc.
This PR tries to address this problem by explicitly enabling all the features for the nix crate in dev mode and making sure they will be added manually to the list by removing explicit --all-features from a few places.
According to cargo docs path-only dev dependencies are allowed and will be ignored when publishing the crate.
With this change in place I can run tests with usual cargo test, rust-analyzer and other tools work as expected.
Checklist:

[x] I have read CONTRIBUTING.md
[x] I have written necessary tests and rustdoc comments
[x] A change log has been added if this PR modifies nix's API

Good point. Will check what's going on with it tomorrow.
On Mon, Sep 23, 2024, 21:29 SteveLauC @.***> wrote:

@.**** commented on this pull request.
In Cargo.toml
https://github.com/nix-rust/nix/pull/2516#discussion_r1772428017:

@@ -77,6 +77,11 @@ parking_lot = "0.12"
rand = "0.8"
tempfile = "3.7.1"
semver = "1.0.7"
+nix = { path = ".", features = ["acct", "aio", "dir", "env", "event", "fanotify",

Yeah, that's right, but that means it only builds or runs the clippy check
for a small set of code, which is not gated with any feature
—
Reply to this email directly, view it on GitHub
https://github.com/nix-rust/nix/pull/2516#discussion_r1772428017, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAQFIZKGZNDFYBHZ6ZP64LZYC57HAVCNFSM6AAAAABOXFIMHWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGMRTHAYTMMZRGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

#2518 is merged.


Test action with PAT

Unit Test Results
1 files  ±0      1 suites  ±0   1s :stopwatch: ±0s
147 tests ±0  141 :heavy_check_mark:  - 6  0 :zzz: ±0  6 :x: +6
Results for commit c34e88e5. ± Comparison against base commit e3088100.

Unit Test Results
1 files  ±0      1 suites  ±0   1s :stopwatch: ±0s
147 tests ±0  141 :heavy_check_mark:  - 6  0 :zzz: ±0  6 :x: +6
Results for commit c34e88e5. ± Comparison against base commit e3088100.


Added check to get NPQ certificate that the qualification Id is associated with the authenticated user

Context
The endpoint that returns a NPQ certificate takes a qualification ID parameter (since a teacher can have multiple). We don’t currently check that the current user owns the qualification specified which could allow a teacher to get a certificate for a qualification that isn’t their own.
Changes proposed in this pull request
Add an additional check to the GetNpqCertificateHandler that the qualification specified belongs to the currently authenticated teacher. If it does not, return null from the handler.
Guidance to review
N/A
Checklist

[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 Ensure the requested qualification belongs to the current user


Cirrus: Minor cleanup of dependencies and docs

:smile: Actually, just docs and task-dependency cleanup then removing a long-disused "optional" task.

/approve

LGTM

Changes LGTM, but the merge bug has bit ya @cevich

Seems I'm my own worst enemy lol.  No worries, I'll need to rebase this again after other PRs of mine merge as well.

/lgtm


dart2js: Type from signature emitted in main unit when superclass in deferred unit

In the following example, SystemMessage is placed in the main unit, but the superclass GeneratedMessage is placed in a 'part' file.
It appears that the problem is that DeferredLoadTask does not walk the signature of the 'StaticUse' of the KLocalFunction to add the types in the signature to the dependencies.
main:
import 'bug1a7m1_lib1.dart' deferred as lib1;
import 'bug1a7m1_lib3.dart' deferred as lib3;

main() async {
  await lib1.loadLibrary();
  lib1.test1();
  await lib3.loadLibrary();
  lib3.test3();
}

bug1a7m1_lib1.dart:
import 'bug1a7m1_lib2.dart';

class SystemMessage extends GeneratedMessage {}

test1() {
  List<SystemMessage> xx = <Null>[];
  new GeneratedMessage();
// This closure has inferred type `int Function(SystemMessage, SystemMessage)`:
  xx.sort((a, b) => a == null ? (b == null ? 0 : 1) : (b == null ? -1 : 0));
}

bug1a7m1_lib2.dart:
class GeneratedMessage {}

bug1a7m1_lib3.dart:
import 'bug1a7m1_lib2.dart';

test3() {
  new GeneratedMessage();
}

A similar problem can be provoked with the bounds of generic function type parameters.

Fix available

Fixed via e1df47b15e798d73bf3049f55320ad7867dc2477


llvm_mode and OpenBSD

When trying to build llvm_mode in OpenBSD, it does not work right out of the box.
clang-local does not search for llvm libs in their installed directories by default (for security purposes).
So when trying to build some test program with afl-clang-fast it fails:
[*] Testing the CC wrapper and instrumentation output...
unset AFL_USE_ASAN AFL_USE_MSAN AFL_INST_RATIO; AFL_QUIET=1 AFL_PATH=.
So, for OpenBSD we have to set the llvm lib path explicitly.
I will investigate later and report back.

Changing
CLANG_LFL    = $(LLVM_CONFIG) --ldflags $(LDFLAGS)
to
CLANG_LFL    = $(LLVM_CONFIG) --ldflags $(LLVM_CONFIG) --libdir/libLLVM.so.0.0 $(LDFLAGS)
avoids the linking error.
But instrumentation does not work yet.

Hmm, it seems clang under OpenBSD 6.5 is restricted and does not support -Xclang options. Clang seems to accept them, but ignores them. There is no opt either :-(. That would mean stock afl would not work in standard llvm_mode. Can anybody using OpenBSD confirm?

Apparently there are two slightly different versions of LLVM on OpenBSD. One from the main tree in /usr/bin and /usr/lib and one from the ports tree in /usr/local/bin and /usr/local/lib. I tried the following changes to the Makefile on my machine:
diff --git a/llvm_mode/Makefile b/llvm_mode/Makefile
index 160a8fe..86aa085 100644
--- a/llvm_mode/Makefile
+++ b/llvm_mode/Makefile
@@ -25,7 +25,12 @@ BIN_PATH     = $(PREFIX)/bin

 VERSION     = $(shell grep '^\#define VERSION ' ../config.h | cut -d '"' -f2)

-LLVM_CONFIG ?= llvm-config
+ifeq "$(shell uname)" "OpenBSD"
+  LLVM_CONFIG ?= $(BIN_PATH)/llvm-config
+else
+  LLVM_CONFIG ?= llvm-config
+endif
+
 LLVMVER  = $(shell $(LLVM_CONFIG) --version)
 LLVM_UNSUPPORTED = $(shell $(LLVM_CONFIG) --version | egrep -q '^[12]|^3\.0|^1[0-9]' && echo 1 || echo 0 )
 LLVM_MAJOR = ($shell $(LLVM_CONFIG) --version | sed 's/\..*//')
@@ -61,7 +66,7 @@ ifeq "$(shell uname)" "Darwin"
 endif

 ifeq "$(shell uname)" "OpenBSD"
-  CLANG_LFL += `$(LLVM_CONFIG) --libdir`/libLLVM.so.0.0
+  CLANG_LFL += `$(LLVM_CONFIG) --libdir`/libLLVM.so
 endif

 # We were using llvm-config --bindir to get the location of clang, but
@@ -69,8 +74,13 @@ endif
 # probably better.

 ifeq "$(origin CC)" "default"
-  CC         = clang
-  CXX        = clang++
+  ifeq "$(shell uname)" "OpenBSD"
+    CC         = $(BIN_PATH)/clang
+    CXX        = $(BIN_PATH)/clang++
+  else
+    CC         = clang
+    CXX        = clang++
+  endif
 endif

 # sanity check.

After this you still get warnings during compilation, but the instrumentation seems to be working:
[*] Testing the CC wrapper and instrumentation output...
unset AFL_USE_ASAN AFL_USE_MSAN AFL_INST_RATIO; AFL_QUIET=1 AFL_PATH=. AFL_CC=/usr/local/bin/clang AFL_LLVM_LAF_SPLIT_SWITCHES=1 AFL_LLVM_LAF_TRANSFORM_COMPARES=1 AFL_LLVM_LAF_SPLIT_COM
PARES=1 ../afl-clang-fast -O3 -funroll-loops -Wall -D_FORTIFY_SOURCE=2 -g -Wno-pointer-sign -DAFL_PATH=\"/usr/local/lib/afl\" -DBIN_PATH=\"/usr/local/bin\" -DVERSION=\"++2.53d\" ../test
-instr.c -o test-instr
clang-7:../split-switches-pass.so: /usr/local/bin/clang-7 : WARNING: symbol(_ZTVNSt3__110__function6__funcIPFvRKN4llvm18PassManagerBuilderERNS2_6legacy15PassManagerBaseEENS_9allocatorIS
A_EES9_EE) size mismatch, relink your program
clang-7:../compare-transform-pass.so: /usr/local/bin/clang-7 : WARNING: symbol(_ZTVNSt3__110__function6__funcIPFvRKN4llvm18PassManagerBuilderERNS2_6legacy15PassManagerBaseEENS_9allocato
rISA_EES9_EE) size mismatch, relink your program
clang-7:../split-compares-pass.so: /usr/local/bin/clang-7 : WARNING: symbol(_ZTVNSt3__110__function6__funcIPFvRKN4llvm18PassManagerBuilderERNS2_6legacy15PassManagerBaseEENS_9allocatorIS
A_EES9_EE) size mismatch, relink your program
clang-7:../afl-llvm-pass.so: /usr/local/bin/clang-7 : WARNING: symbol(_ZTVNSt3__110__function6__funcIPFvRKN4llvm18PassManagerBuilderERNS2_6legacy15PassManagerBaseEENS_9allocatorISA_EES9
_EE) size mismatch, relink your program
Rewriting 1 switch statements
switch: 2 cases 8 bit
Running split-compare-pass 64
Running split-compare-pass 32
Running split-compare-pass 16
../afl-showmap -m none -q -o .test-instr0 ./test-instr < /dev/null
echo 1 | ../afl-showmap -m none -q -o .test-instr1 ./test-instr
[+] All right, the instrumentation seems to be working!

Thanks a lot, I will have a look soon. Two versions of llvm was a bit confusing for me.
What version of OpenBSD are you running?

I am running OpenBSD 6.5 with the llvm-7.0.1p9 port/package installed.
I was confused by the different versions as well. A colleague told me that LLVM on OpenBSD started out as a port. Later they added LLVM to base OpenBSD, but the port still exists. And apparently it has more features enabled and is in part newer than the version in base.

Thanks, I can reproduce it. The linker errors are mysterious, though. But it is certainly a progress.

I adapted your solution and added a check for the required package with commit 3bfd88aa . Thanks.


testrunner: do not rely on shebang when running pip3

Why is this PR needed?
When running pip3 directly, the shebang command can exceed the 128
limit very easily depending on the job name that is being run on CI.
To avoid this limit, call to python3 directly, that will run the
pip3 command, instead of relying on the pip3 venv script, that
contains the shebang that can exceed 128 characters.
More information: https://github.com/pypa/pip/issues/1773
Merge restrictions
(Please do not edit this)
We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:
What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

An alternative option is to run pip as a module (python3 -m pip) instead.

Updated pr_manager as well, and used python -m pip instead on both.

As an aside I find it a little weird to activate the venv, install requirements and deactivate right after that? When we run the script we are in no specific environment, but we installed the requirements in the $VENVDIR environment.
Am I misunderstanding how python does this behind the covers?


Respect heights

with media-item + ignore-crops flag - respect any height values passed in, including responsive heights
This PR should help control rendered crops from our-img directly, instead of taking the original image's height ratio for crops (this is done using a new optional boolean/flag ignore-crops.

with src - respect height values passed in
When using src , the heights were being ignored automatically too. This change will respect any height props passed on <our-img>.

Solves https://github.com/umbraco-community/Our-Umbraco-TagHelpers/issues/65

not sure ignore crops is the correct naming.. or a new ignoreCrops param is requried? isn't it more a case of ..

ImgWidth+ImgHeight is provided, then don't use the original aspect ratio of the image, crop to the new aspect ratio from these explicit dimensions
ImgWidth only, then we should use the original aspect (current code) to calculate height
ImgHeight only, then we should use the original aspect (currently missing) to calculate width

so need to look at if image height (ImgHeight) is provided for these use cases?
It would also apply that if ImgHeight and cropalias supplied then we should get an image based on the cropalias aspect and focal point but adjusted to the ImgHeight, currently it only allows ImgWidth control. For all three supplied cropalias, ImgHeight and ImgWidth that should perhaps give you the overridden aspect ratio but respect the focal point?

Good question! I didn't want to change the existing logic too much (because we don't use crops,doesn't mean this logic doesn't suit others!)
If you think it's acceptable to change the code to always respect the height when provided  - I'm happy to edit it to match that and remove the ignoreCrops flag.

Thanks @warrenbuckley for tagging me in on this. I think this PR covers a few more scenarios which weren't originally anticipated. All looks good but as @mistyn8 pointed out the introduction of a new ignoreCrops may be overkill, especially when if we don't define a cropalias or width/height could be a better indicator to detect this. How are you getting on with the edit @kahlan88? Happy to test the code once ready!

efine a cropalias or width/height could be a better indicator to detect this. How are you getting on with the edit @kahlan88? Happy to test the code once ready!

I did think of doing it in the way you suggested initially, but opted for an option that won't be breaking for people when they update to the new version. As it is, it might change their crops without them realising.
I agree, though, that those params should be respected and ignoring crops implied. I will proceed with:

replace ignoreCrops with ImgWidth and ImgHeight

@AndyBoot, I'm currently unable to work on it, but it's on my list. I'll tag you again when I've done the work.

@AndyBoot, @mistyn8, @warrenbuckley we've made some progress on this in https://github.com/umbraco-community/Our-Umbraco-TagHelpers/pull/69
@stayawakesteve took over my progress and addressed your comments. I just looked at it and happy with the work and it resolves height and aspect ratio issues we were experiencing.
Could you review that please and we can close this PR then? Thanks!


mysqltuner: add basic_passwords.txt

Describe the bug
When running mysqltuner, it says:
-------- Security Recommendations ------------------------------------------------------------------
...
[!!] There is no basic password file list!

So i suggest to add basic_passwords.txt to the package that in included in the projects repo.
https://github.com/major/MySQLTuner-perl/blob/master/basic_passwords.txt
To Reproduce
Steps to reproduce the behavior:

...
...
...

Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.
Notify maintainers
@peterhoeg
Metadata
used with nix run -f channel:nixos-unstable mysqltuner -c mysqltuner
Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

Good catch - feel like doing a PR?

Not now, maybe later. But it's fine if someone else does it.
Should be pretty easy, so good first issue.


Create a review of the example-fn with two generalised functions

First:

Get the env variable with the file to extract the edges (from Dockerfile)
Decide if it is a folder or a file
Then change the code to adapt it and refactor the way to get the files and read the folder or the way to get one single image to produce a set of files as the output of the original images: ie. input: ncg16.jpg -> output ncg16_edges.jpg.

I have finished and I have updated it in GitHub.

It should work with differents scenarios.

Great, let me check in the afternoon.
On Wed, 4 Oct 2023 at 13:18, AlvaroRodriguezGallardo <
@.***> wrote:

It should work with differents scenarios.
—
Reply to this email directly, view it on GitHub
https://github.com/AlvaroRodriguezGallardo/src-brokering/issues/1#issuecomment-1746667462,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVVE22WNAQEEQ5DWCLLNLDX5VAZ3AVCNFSM6AAAAAA5SG6ANGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBWGY3DONBWGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Manuel Parra-Royón
Postdoctoral researcher
Extragalactic Astronomy - IAA CSIC https://iaa.es - Granada, Spain
This message is intended solely for the addressee and may contain
confidential information. If you have received this message in error,
please inform the sender, and immediately and permanently delete the email.
Do not use, copy or disclose the information contained in this message or
in any attachment.


Chatbot not replying message Facebook Messenger

The chatbot starts receiving message and replying well but suddenly it stops. I have debug activated and its output on the console is :

App 27152 output: debug: RECEIVED MESSAGE
App 27152 output: debug: CUSTOM FIND CONVO<PHONE_NUMBER>4354545<PHONE_NUMBER>4354545
App 27152 output: debug: WEBHOOK SUCCESS { recipient_id: '12333333354354545' }

The message sent by the user is being ticked(without the full blue).. and the typing icon is appearing; and the chatbot starts working or replying again to the message the user sent after me restarting it.

where can this problem be coming from ?

This problem is still there I am using Botkit version : "botkit": "^0.6.14", and node.js version v8.11.4.
The chatbot only replies after restarting it.


[1.6.3] FIRST(),LAST() do not return the relevant timestamp with * or regex input

System info: [InfluxDB shell version: 1.6.3, Ubuntu 16.04.5 LTS]
Steps to reproduce:
Using the sample data from the docs:
The following queries return the correct timestamp as expected:
> use NOAA_water_database
Using database NOAA_water_database
> SELECT FIRST(water_level) FROM h2o_feet WHERE location = 'santa_monica'
name: h2o_feet
time                 first
----                 -----
2015-08-18T00:00:00Z 2.064
> SELECT LAST(water_level) FROM h2o_feet WHERE location = 'santa_monica'
name: h2o_feet
time                 last
----                 ----
2015-09-18T21:42:00Z 4.938

but when using * or a regular expression a 1970 timestamp is returned:
> SELECT FIRST(/level/) FROM h2o_feet WHERE location = 'santa_monica'
name: h2o_feet
time                 first_level description first_water_level
----                 ----------------------- -----------------
1970-01-01T00:00:00Z below 3 feet            2.064
> SELECT LAST(/level/) FROM h2o_feet WHERE location = 'santa_monica'
name: h2o_feet
time                 last_level description last_water_level
----                 ---------------------- ----------------
1970-01-01T00:00:00Z between 3 and 6 feet   4.938
> SELECT FIRST(*) FROM h2o_feet WHERE location = 'santa_monica'
name: h2o_feet
time                 first_level description first_water_level
----                 ----------------------- -----------------
1970-01-01T00:00:00Z below 3 feet            2.064
> SELECT LAST(*) FROM h2o_feet WHERE location = 'santa_monica'
name: h2o_feet
time                 last_level description last_water_level
----                 ---------------------- ----------------
1970-01-01T00:00:00Z between 3 and 6 feet   4.938

Additional info:
This appears to be a subcase of issue #5890

Still an issue as of 1.7.9

@jsternberg any thoughts on this issue?

@jsternberg any thoughts on this issue?


NfT BRIDGE

Describe the feature request
THE NFT BRIDGE SUPPOSED TO BE ABLE TO DETECT NFTS ON THE WALLETS WITHOUT FILLING MANUALLY
Spam policy

[X] I verify that this issue is NOT SPAM and understand SPAM issues will be closed and reported to GitHub, resulting in ACCOUNT TERMINATION.

It only detects them on Katla


Various enhancements to dotnet-build

Build AOT toolchain for trunk build as well
Taking constructors into account while disassembling

Sorry this got missed. looking now


Write tutorial on how to use BLE

Write a tutorial that explains how to use the JavaScript BLE API.
Should this be a Markdown document in the repository or a page on the Wiki?

Specifically document how to add support for custom BLE hardware.
Also document how to identify a device using advertisement data.

BLE Guide published here: https://evothings.com/doc/tutorials/how-to-connect-to-ble-devices.html


Build fails on velox_memory_pool.cc

Bug description
I'm trying to build Gluten, and it checks out the latest version of Velox. I'm getting a build error on velox_memory_pool.cc. Evidently this file is looking for MemoryPool when the class name has been changed to MemoryPoolBase. You can see the full compiler error output on this Gluten issue:
https://github.com/oap-project/gluten/issues/593
System information
Latest version of Gluten. This is the build command for Gluten:
mvn clean package -Pbackends-velox -Pspark-3.2 -Pfull-scala-compiler -DskipTests -Dcheckstyle.skip -Dbuild_cpp=ON -Dbuild_velox_backend=ON -Dbuild_velox_from_source=ON -Dbuild_arrow=ON -Dvelox_enable_s3=ON

Relevant logs
No response

Velox does not have a stable API yet so any issues caused by that are generally up to the downstream project to handle. Which Gluten does, as you have received an answer in the linked issue I am closing this one.


Support GeoParquet

Feature request
Support the GeoParquet format
Motivation
GeoParquet (https://geoparquet.org/) is a common format for sharing vectorial geospatial data on the cloud, along with "traditional" data columns.
It would be nice to be able to load this format with datasets, and more generally, in the Datasets Hub (see https://huggingface.co/datasets/joshuasundance/govgis_nov2023-slim-spatial/discussions/1).
Your contribution
I would be happy to help work on a PR (but I don't think I can do one on my own).
Also, we have to define what we want to support:

load all the columns, but get the "geospatial" column in text-only mode for now
or, fully support the spatial features, maybe taking inspiration from (or depending upon) https://geopandas.org/en/stable/index.html (which itself depends on https://fiona.readthedocs.io/en/stable/, which requires a local install of https://gdal.org/)

Thank you, @severo ! I would be more than happy to help in any way I can. I am not familiar with this repo's codebase, but I would be eager to contribute. :)
For the preview in Datasets Hub, I think it makes sense to just display the geospatial column as text. If there were a dataset loader, though, I think it should be able to support the geospatial components. Geopandas is probably the most user-friendly interface for that. I'm not sure if it's currently relevant in the context of geoparquet, but I think the pyogrio driver is faster than fiona.
But the whole gdal dependency thing can be a real pain. If anything, it would need to be an optional dependency. Maybe it would be best if the loader tries importing relevant geospatial libraries, and in the event of an ImportError, falls back to text for the geometry column.
Please let me know if I can be of assistance, and thanks again for creating this Issue. :)

Just hitting into this same issue too showing GeoParquet files in Datasets Viewer. I tried to implement a custom reader for GeoParquet in https://huggingface.co/datasets/weiji14/clay_vector_embeddings/discussions/1, but it seems like HuggingFace has disabled datasets with custom loading scripts from using the dataset viewer according to https://discuss.huggingface.co/t/dataset-repo-requires-arbitrary-python-code-execution/59346 :frowning_face:

I'm thinking now if there's a way to simply map files with GeoParquet extensions (*.gpq, *.geoparquet, etc) to use the Parquet reader. Maybe we could allowlist these geoparquet extensions at https://github.com/huggingface/datasets/blame/0caf91285116ec910f409e82cc6e1f4cff7496e3/src/datasets/packaged_modules/init.py#L30-L51? Having the table columns show up would be a quick win.
Longer term though, it would certainly be nice if the WKB geometry columns could be displayed in a nicer form. Geopandas' read_parquet function is supposedly faster than pyogrio.read_dataframe according to https://github.com/geopandas/geopandas/discussions/2724#discussioncomment-4606048, but there's also pyogrio.raw.read_arrow now that can read into an pyarrow.Table directly.

Update: It looks like renaming the GeoParquet file to have a file extension of *.parquet works (see https://huggingface.co/datasets/weiji14/clay_vector_embeddings). HuggingFace's default parquet reader is able to read the GeoParquet file, though the geometry column is of an unknown type:

I've opened a quick PR at #6508 to let files with a *.geoparquet or *.gpq extension be read the default Parquet reader. Let's see how that goes :smile:

If I'm understanding this correctly, the code below wouldn't be recommended to due to dependency headaches? If that's the case, what solution would there be to see the geometry features for .gpq files in huggingfaceHub?
code for dataset_loader.py
import geopandas as gpd
# ... (other imports remain the same)

class ClayVectorEmbeddings(datasets.ArrowBasedBuilder):
    # ... (other parts of the class remain the same)

    def _info(self):
        # Read the GeoParquet file to get the schema for the 'geometry' feature
        gdf = gpd.read_file("path/to/your/geoparquet/file.gpq")  # Replace with your file path
        geometry_schema = str(gdf.geometry.dtype)

        return datasets.DatasetInfo(
            # This is the description that will appear on the datasets page.
            description="Clay Vector Embeddings in GeoParquet format.",
            # This defines the different columns of the dataset and their types
            features=datasets.Features(
                {
                    "source_url": datasets.Value(dtype="string"),
                    "date": datasets.Value(dtype="date32"),
                    "embeddings": datasets.Value("string"),
                    "geometry": datasets.Value(dtype=geometry_schema),  # Use the schema read by GeoPandas
                    # ... (other features)
                }
            ),
        )

# ... (rest of the script remains the same)

Hi @mehrdad-es, I'm not sure if HuggingFace would be keen to add geopandas to HuggingFace Hub (maybe a question for @severo?). Having a geometry viewer would be an even bigger task, and if you're thinking of a map-viewer, it might involve some redesign of the website UI. Some of my colleagues are working on streamlining GeoParquet visualization from cloud-hosted instances like HuggingFace (see e.g. https://github.com/developmentseed/lonboard/issues/314), and we could definitely come up with something if there's interest.

I've created https://github.com/huggingface/datasets-server/issues/2416 to discuss the possibility of supporting (vectorial) geospatial columns in the dataset viewer, or in the converted parquet files.
At the same time, it would be super interesting to see what is already possible to do with a Hugging Face dataset that hosts geospatial data.

Some of my colleagues are working on streamlining GeoParquet visualization from cloud-hosted instances like HuggingFace (see e.g. https://github.com/developmentseed/lonboard/issues/314), and we could definitely come up with something if there's interest.

It would be awesome to show this inside a Space.


Add errorType in debugger error analytics payload for plugin execution errors

Adds errorType received from action execution response in analytics payload for PLUGIN_EXECUTION errors
Description
Fixes #6320
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Manually
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: feat/plugin-execution-errors

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
54.62 (0)
36.69 (-0.01)
33.05 (0)
55.22 (-0.01)

:red_circle:
app/client/src/utils/autocomplete/TernServer.ts
50.47 (-0.24)
37.72 (-0.88)
36.21 (0)
55.08 (-0.27)

/ok-to-test sha=d39fd27


Links in footnotes run off the page when justify is set to true

Description
As per title, links in footnotes run off the page when justify is set to true.
Reproduction code:
#set par(justify: true)

This is a test#footnote[https://example.com/Loremipsumdolorsitamet,consecteturadipiscingelit,seddoeiusmodtemporincididuntutlaboreetdoloremagnamaliquamquaeratvoluptatem.].

Screenshot of issue:

For anyone needing a temporary workaround:
#show footnote.entry: set par(justify: false)

Thanks for your time!
Reproduction URL
No response
Operating system
Windows
Typst version

[X] I am using the latest version of Typst

Duplicate of https://github.com/typst/typst/issues/1271
(It's not really specific to footnotes)


Dealing with timezones?

I'm using this gem to analyze some S3 logs where the timestamps are UTC. It doesn't look like there is any option available to convert these to a specific timezone before generating the report.
Any thoughts on how to best address this challenge?

I guess the biggest issue here aside from how gross working with timezones can be is that we need to tell the program what the current timezone in the log is before even thinking about trying to convert it to a different specific one. Bleh.

The tricky part is that there's two timezones at play:

The timezone of the timestamps in the logs. In many cases, this includes a timezone but this is not always the case. So we'd need a command line argument for this. I don't like forcing people to set this, so it should have a default value. We could default to UTC if this command line option is not provided, although this could easily be incorrect.

The output timezone. This would require a command line option as well, which will also need a default. We could default to either: input timezone, local timezone (often UTC on servers). Whatever timezone we end up using, it should be included in the report.

THis is quite hairy but could be done. @barttenbrinke any thoughts?

I'm definitely in agreement that this should not be a required CLI parameter. Regarding default behavior, it might make more sense to completely ignore the timezone issue completely and simply report what's in the logfile as it does now in the event that no information is provided by the user at runtime.
The param would have to take two values logtimeZone and reportTimeZone maybe something of the form: ---timezone UTC:EST ?
Thanks for opening up this discussion, I know this is likely to be a whole thing.

How does the S3 timestamp look? Does it have any timezone info? Because the user friendly thing to do would be to 'auto detect' the source timezone (or fallback to UTC if we have no clue) and to output using the local machine's timezone.

The timestamp in my S3 logs is formatted like this:
[05/Feb/2015:14:49:24 +0000]

Regarding the local machine's timezone I think that's a reasonable default provided there's still a command line option override since it's not necessarily true that the majority of a services users live in the same time zone as the log analyzer.

Does the standard library have enough timezone support to do this? Or would we need to introduce a runtime dependency for this?

I'd guess you'd want to include a dependency just to avoid having to implement DST aware transformations and the gross complexity that adds, as I don't think the standard library is capable of that on its own.


Indent files in account

Related to https://github.com/jhipster/jhipster-vuejs/issues/73

If I'd right to do it, I would merge this :)


[stable/minecraft] Added option to use existing persistentVolumeClaim

Signed-off-by: Joseph Stanton<EMAIL_ADDRESS>
Is this a new chart
No.
What this PR does / why we need it:
Updates deployment to allow for pre-existing persistent volume claims to be used to back the minecraft chart.
Which issue this PR fixes
None.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @itzg

/ok-to-test

/lgtm


Enable ssh's forward agent in Vagrant image

This allows reusing the ssh keys in the host.

I just saw there is a mechanism for loading local configurations from Vagrant.local and I am not sure that ssh forwarding will be used by most people so I am not sure if this pull request makes sense anymore


Roll V8, remove unused compat flags arg

V8 roll is pretty minor, primarily adding this to transition to the tagged version
Binary size impact of cleaning up the compat flag arg is less than expected/rather low (~6K for macOS debug), still good to clean up the affected headers a bit

Thank you Sam!


OBC Fan operation

Confirm OBC can control attached fans (i2c)

Fan controllers can successfully control and measure speed of fans.


Implement New CRUD spec for mongo

The mquery syntax is mostly similar to mongodb native driver queries with some sugar added, All of find, delete, update queries are still present, with the exception of a insert query. So with this, MongoDataProvider uses mquery for all operations except 'create'. I also noticed that we don't test the batchRead operation, which I also changed to use mquery, so if possible I would like to write a test for it just to be safe?

Changed base to @craicoverflow branch. This might break things

Overall I do see this being quite nice.and fluent api. We can try it in new crud

Only used in findBy as of a95b07a
Will add other implementations in time,
also tests to be written

I would skip tests for now. What we want is early alpha of crud spec to play with in real project to see possible issues or extensions

I have been using DataSync starter for testing.

Mongo now uses advanced filtering in both findBy and batchLoading

Seems like there is some issue with Input Fields in #1211
There is no note_id input field in this test snapshot, Is that intentional?
I was having issues testing it out with relationships because of this

Yep.  That is a blocker bug for us.
@craicoverflow is off for 3 days so I will check why this is happening

Merging this into @craicoverflow PR in order to organize work


Fix service facade inspector headers

Changes

Fixes: #7583

Verification

As a reviewer, it is possible to check out this change locally by using the following
commands (substituting {PR_ID} with the ID of this pull request):
git fetch origin pull/{PR_ID}/head:name_of_local_branch
git checkout name_of_local_branch

/azp run


Add UK Holiday for Queen Elizabeth II Funeral

@dr-prodigy this holiday is in 6 days (and has only just been announced), can this please be merged ASAP? Thank you.
https://www.gov.uk/government/news/bank-holiday-announced-for-her-majesty-queen-elizabeth-iis-state-funeral-on-monday-19-september

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-7.0e**-05%**) to 99.625%

Totals

Change from base Build<PHONE_NUMBER>:
-7.0e-05%

Covered Lines:
6428

Relevant Lines:
6440

💛  - Coveralls

Hi @violuke thx for your contribution!
I'm currently working to merge this PR.
Unfortunately, due to our standard release workflow, we can't receive it as a direct hot-fix towards main (master) branch (which would not trigger a new PyPi library version anyway).
We are then merging it on beta branch: please refer to here
https://github.com/dr-prodigy/python-holidays#beta-version
for instructions on how to add a temporary dependency to it.
Regarding an official release, at the moment we can't guarantee it in the required time, but we will do our best.
Please let me know if this sorts your need out, thx!

Hey @dr-prodigy,
Thanks for merging the PR. I appreciate the quick turnaround and thank you for the heads up that you can't guarantee the release will occur prior to Monday. We rely on this package for a production payment system so I will have to create a catch directly in our system for this.
Thanks for your continued work on this package, it's super useful!

Thanks also to @JPunter

Of course, thank you very much @JPunter, sorry for not mentioning you before.. KUTGW!

Hi @violuke @JPunter
as you may have noticed, we have just released v.0.16, which includes, along with your PR, all the up-to-now-known holidays regarding Queen Elizabeth mourning across the world: it's been quite a tough work, but we did it :-)
I hope it will be a beneficial for your objectives and needs: I kindly ask you to keep on reporting any inconsistencies you may find, and we will do our best to address them at once.
Best regards
dr-p

Thank you, that's very much appreciated 😄


segmentation fault after orbital command finishes

happens on ladder
likely during removing orbital command from productionQueue

important to note that it doesn't always segmentation fault (though i think most of the other crashes so far have been due to similar reasons)

changing how things get removed from lists seems to fix this issue, since Monte doesn't seem to crash when finishing orbital command.
new issue seems to be with workers dying. they get removed from the list, but while looking at the logs and replays the bot crashes as soon as a worker dies. however, this happens inconsistently:
crashes in 625499, 624833, 625422 after worker removed successfully line
in 625499, it crashes immediately when a worker dies in the replay. it could be something else, but it's likely Monte is trying to reference a dead worker and crashes when doing so
important to note that many workers died in 625128 but it didn't crash then

with versions alpha.14 and alpha.16 it seems to be mostly fixed, so closing the issue.
in the end, the issue didn't seem to be with how things got removed, but moreso accidentally referencing a null pointer. Using gdb on linux helped to find segfaults in the code


Make StyleCop a private asset

The StyleCop nuget should probably be made a private asset so that it doesn't impose any rules as transitive dependency onto other projects. I added some of the full node nuget packages into my own project and got hundreds of useless stylecop warnings.
The following issue and a comment in it explain the problem in more detail: https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/2278#issuecomment-383321896

If anyone happens to come across this, it is possible to suppress the StyleCop warnings with a .editorconfig file.
root = true

[*.cs]
dotnet_diagnostic.SA1009.severity = none
dotnet_diagnostic.SA1011.severity = none
dotnet_diagnostic.SA1012.severity = none
dotnet_diagnostic.SA1013.severity = none
dotnet_diagnostic.SA1028.severity = none
dotnet_diagnostic.SA1101.severity = none
dotnet_diagnostic.SA1116.severity = none
dotnet_diagnostic.SA1117.severity = none
dotnet_diagnostic.SA1118.severity = none
dotnet_diagnostic.SA1191.severity = none
dotnet_diagnostic.SA1200.severity = none
dotnet_diagnostic.SA1201.severity = none
dotnet_diagnostic.SA1202.severity = none
dotnet_diagnostic.SA1208.severity = none
dotnet_diagnostic.SA1210.severity = none
dotnet_diagnostic.SA1203.severity = none
dotnet_diagnostic.SA1303.severity = none
dotnet_diagnostic.SA1309.severity = none
dotnet_diagnostic.SA1505.severity = none
dotnet_diagnostic.SA1508.severity = none
dotnet_diagnostic.SA1512.severity = none
dotnet_diagnostic.SA1515.severity = none
dotnet_diagnostic.SA1516.severity = none
dotnet_diagnostic.SA1600.severity = none
dotnet_diagnostic.SA1623.severity = none
dotnet_diagnostic.SA1633.severity = none
dotnet_diagnostic.SA1649.severity = none


Update description about serviceaccount token controller

This change specifies how to enable token controller in
controller-manager.

This change is

We have GOT to make these flag names clearer in the future.

I already did?

@lavalamp D'oh! Sorry, I didn't see the label change. :(


[Table] Remove our version of TableToolbarSearch when upgrading to Carbon v10.10.x

Summary
To fix a table search default value bug in Carbon 10.9.x we had to duplicate the TableToolbarSearch component including their fix.
The PR for this is here: https://github.com/IBM/carbon-addons-iot-react/pull/953
What to do

Remove our component
Reference Carbon's TableToolbarSearch instead
Change any reference of the prop value to defaultValue

:tada: This issue has been resolved in version 2.84.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Hanoi University of Science and Technology

Update latest domain.
In 2017, they changed the domain from students.hust.edu.vn to sis.hust.edu.vn.

@searSandcrab22 I am sorry, the domain sis.hust.edu.vn is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Upgrade Apache Commons Collections to v3.2.2

Version 3.2.1 has a CVSS 10.0 vulnerability. That's the worst kind of
vulnerability that exists. By merely existing on the classpath, this
library causes the Java serialization parser for the entire JVM process
to go from being a state machine to a turing machine. A turing machine
with an exec() function!
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2015-8103
https://commons.apache.org/proper/commons-collections/security-reports.html
http://foxglovesecurity.com/2015/11/06/what-do-weblogic-websphere-jboss-jenkins-opennms-and-your-application-have-in-common-this-vulnerability/

Very strange.  Even manually editing has challenges with this file.  Will leave as-is for now.


how can I get folderid of folder which name by chinses？

three month ago I can get folder id by url https://memect.quip.com/folder/xxxxxxxxx
but the url has been changed  by server，url is  https://memect.quip.com/folder/z-turnshi-yan-shi
z-turnshi-yan-shi  is not folder id。so how can i get the url id？

Have you solved this problem？

If the folder is shared with the user, you can find the folder id from shared_folder_ids by Get User API.


WIP: move format tooltip logic to chart utils

This should fix the current error by using a different format for the report summary trend and the report summary details.

Codecov Report

Merging #348 into master will decrease coverage by 0.44%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master     #348      +/-   ##
==========================================
- Coverage   89.07%   88.62%   -0.45%
==========================================
  Files         142      142
  Lines        1703     1715      +12
  Branches      221      223       +2
==========================================
+ Hits         1517     1520       +3
- Misses        165      173       +8
- Partials       21       22       +1

Impacted Files
Coverage Δ

src/pages/ocpDashboard/ocpDashboardWidget.tsx
83.33% <ø> (ø)
:arrow_up:

src/pages/awsDashboard/awsDashboardWidget.tsx
88.05% <ø> (ø)
:arrow_up:

src/components/trendChart/trendChart.tsx
77.41% <100%> (ø)
:arrow_up:

src/components/commonChart/chartUtils.ts
72.28% <22.22%> (-6.09%)
:arrow_down:

src/utils/formatValue.ts
86.95% <81.81%> (-8.05%)
:arrow_down:

Continue to review full report at Codecov.

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

@chargio this is how it looks like now.

Do you mind checking that I didn't miss anything :smile:

Looking at the video it seems to be great. I will have another look at it when is live

Merging for Natalie's demo with Red Hat IT


Result set doesn't contain information about connections.

Currently result set for the validation tool doesn't contain connection information. Can we add that information also because that could become useful for customers with multiple projects.

This should also include adding the 'filters' specified to the result handler.


Add javac includes and excludes.

This PR adds support for passing the javac includes and excludes attributes. This is useful for setups that might have two applets in one directory, for example with different applet main classes.

Would you have some great ideas on how to combine sources, sources2, includes and excludes into a meaningful package?


abseil-cpp_202308: 20230802.1 -> 20230802.2

Description of changes
Changelog: https://github.com/abseil/abseil-cpp/releases/tag/20230802.2
cc @andersk

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[x] x86_64-darwin
[x] 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

[x] 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

[x] Fits CONTRIBUTING.md.

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


Audit templates for Issue removal

Issues are deeply embedded into the current ALA website.  To determine what changes need to occur so that we can remove Issues, we need to audit the templates and create a list for:

How many templates need to be modified
Which modifications are cosmetic vs. complicated
Which modifications require decisions vs. simple implementation

Tim audited and placed results in Google Doc.


[webrtc] Add permission for WebRTC audio channel

Permission MODIFY_AUDIO_SETTINGS is also needed for audio channel in WebRTC
BUG=https://crosswalk-project.org/jira/browse/XWALK-6069

@haoxli Thanks so much for bringing this up! I had to scour half the web figuring out why sound wouldn't transmit in my cordova application. You're a lifesaver!

ping @zqzhang

@zqzhang is on vacation
Yes, this issue due to the app-tools restrict the default permission support than make_apk.py.
so Apps need to manually request the permissions in manifest.json


Upgrade to cats-effect 3.0.0 final and fs2 3.0.0 final

Supersedes https://github.com/http4s/http4s/pull/4662.
I ensured everything compiles locally via sbt compile test:compile but haven't run tests.

There are some test failures, but I am mostly worried about:
java.lang.NoSuchMethodError: fs2.Pull.stream(Lscala/Predef$$less$colon$less;)Lfs2/Stream;

:+1: once we green it up.  I have a PR open for jawn-fs2, and am ready to release vault, keypool, and log4cats.  That should line up the whole CE3 ecosystem.

Was about to jump on it. Your jawn-fs2 PR has built fine.

Thanks, no worries, fair play :-) I'm portig a fair bit of code now. CE3 and friends is a shot in the arm.


feat: Clean up / standardize yaml; fix: pass yamllint

Hey there! I really love the work you've all been doing, and since I do a lot of yaml cleanup locally, I figured I could clean up the template to more closely match AP standards, pass yamllint, and generally be a bit more readable. Additionally, since Archipelago is generally for multiworld, I've set jingaling_jiggy: 'true' by default, per its comment.
Cheers! 😊

Thank you for your contribution :)


open file from path not hapenning

VSCode Version:1.23.1
OS Version: MAC OS High sierra v 10.13.4

Steps to Reproduce:
1.create tank.js file , export it
2. create bazooka.js file , import import tank.js file and see you are able to open tank.js

Does this issue occur when all extensions are disabled?: Yes

Please share the file contents. How are you exporting and importing for example?

Code Sample
Header.js
import React, { Component } from 'react';
export class Header extends Component {
  render(){
   return Yo Header
 }
}

export default Header

App.js
import React, { Component } from 'react';
import Header from './Header';
export class App extends Component {
  render(){
    return

  }
}
export default App;

so on cmd + click on './Header';
i am not able to navigate to header.js file
on ubuntu its working nicly

I cannot repo this in VS Code insiders. Can you please test in insiders with all extensions disabled.
Also, does your project have a jsconfig.json?

Please try creating a jsconfig.json at the root of your project with the contents:
{
    "compilerOptions": {
        "target": "ES6"
    },
    "exclude": [
        "node_modules",
        "**/node_modules/*"
    ]
}

1 : no , it dosen't navigate
2 :  also tried by disabling all extensions
3 : also tried {
"compilerOptions": {
"target": "ES6"
},
"include": [
"client//*","server//*"
]
}

its working in jan build 2018 version 1.20

@mjbvz
Closing this issue
I think it my machine related problem .
i checked on other 2 mac machines with exact same mac & vs code version its working fine


RFE: GitOps-style global/multi-cluster configuration policy management

What kind of request is this (question/bug/enhancement/feature request):
Feature Request
Other details that may be helpful:
Customer Input:
We need the ability to comprehensively configure and manage all of our Kubernetes clusters and Rancher masters, using a GitOps style of management. A Rancher master should be able to be linked to a Git repository and branch, and any pushes to that branch should trigger immediate reconfiguration.
The configuration and policy itself should be comprehensive: it should cover relevant host configurations like sysctl settings and Docker configuration; RKE configuration (e.g. cluster.yaml); Kubernetes configs like admission controllers, ClusterRoles, ClusterRoleBindings, and 3rd party deployments like Portworx and Gatekeeper OPA. It must also cover Rancher policies like project membership, default resource reservations/requests on namespaces, cluster membership, and global roles. And finally, it should also cover the Rancher master configuration itself: authentication, access control, logging, auditing, monitoring, etc.
In addition to configuration management, we should be able to set policy: K8s NetworkPolicy, Calico NetworkPolicy, RBAC roles, Gatekeeper OPA constraints, etc -- all from a GitOps model. This would allow us to develop, test, preview, and deploy these policies in an automated fashion.
Basically anything that we need to do to configure that is in the scope of Rancher or Kubernetes, should be configurable via GitOps.
Of course, for this to work, we can't depend on copying and pasting things like cluster.yaml files and then updating dozens or hundreds of files when we need to make global or regional changes to our cluster configurations. We should be able to assign resources (clusters, masters, groups, namespaces, projects, etc.) tags/labels, and then use selectors (just like Kubernetes does) to apply policies on those resources. This model might also be paired with a hierarchy of sorts, to make the user experience a little friendlier. But note that hierarchies (e.g. a "tree view") is insufficient as the sole organizational model for configuration and policy.
As it stands today (version 2.4.x), Rancher's overall approach to management is still very fragmented. There's the "master" configuration and policy, the "cluster" configuration and policy, and then all the stuff that Rancher currently doesn't manage directly, like network/calico policy. As Rancher evolves, I would like to see Rancher approach the whole stack as a uniformly covered arena for applying configuration and policy. The Rancher API is a substantial step toward achieving this, but its reach is limited (for example, it can't reach into Kubernetes itself). And an API alone isn't a configuration management system.

I was researching this functionality too. It seems at present you need to either use the api or create the clusters outside of rancher then import (which is a bummer).
Having the option to have a cluster repo allowing you to spin up clusters on the fly, paired with fleet to "onboard the cluster to a spec" is very powerful for both ephemeral and production environments imo


[Meta] Cascadia Theme

Meta tracking issue for updating our base theme to use the new Cascadia styling.
All colors, fonts, styling, etc should be updated to use new Cascadia theme. Note: this eng work item is to update the existing base theme. Creating a robust theming system is not tracked by this.

We have this meta issue here: https://github.com/opensearch-project/oui/issues/212

Clarification - will this be an overhaul of the base light and dark themes and not new, separate themes?

@BSFishy  that is correct. There have been several threads about a new theme vs changing the base theme. We agreed to revise the base theme, but it is up to you if you want to build a new theming system overall, since the forked one may be have an out of date approach.

Closing this issue in favor of the original meta issue https://github.com/opensearch-project/oui/issues/212 for tracking purposes.


Preview does not match when notification is for a pre-release

When there is a notification for a pre-release, the preview shows the latest release instead of the pre-release. See screenshot below:

Ctrl+b opens this link. It should be this one.

Issue
The original problem is that calling the REST API for notifications does not return the tagName of the release.
  {
    "id": "4675083817",
    "unread": false,
    "reason": "subscribed",
    "updated_at": "2022-10-24T20:10:47Z",
    "last_read_at": "2022-10-24T20:24:57Z",
    "subject": {
      "title": "boonGUI v2.6.5",
      "url": "https://api.github.com/repos/LangLangBart/boonGUI/releases/80834449",
      "latest_comment_url": "https://api.github.com/repos/LangLangBart/boonGUI/releases/80834449",
      "type": "Release"
    },
    "repository": {
...

The script later calls:
gh release view -R home-assistant/core

But this does not consider pre-releases.
gh is able to also show pre-releases, but only when it gets the tag number.
gh release view -R home-assistant/core 2022.11.0b2

To mitigate the problem, an additional GraphQL API query could be performed to get the latest release tag.

GraphQL
Query to get latest release tag
query ($owner: String!, $name: String!) {
  repository(owner: $owner, name: $name) {
    releases(last: 1, orderBy: {direction: ASC, field: CREATED_AT}) {
      nodes {
        tagName
        isPrerelease
      }
    }
  }
}

Result  for home-assistant/core
{
  "data": {
    "repository": {
      "releases": {
        "nodes": [
          {
            "tagName": "2022.11.0b2",
            "isPrerelease": true
          }
        ]
      }
    }
  }
}

TAG=$(gh api graphql --raw-field owner=home-assistant --raw-field name=core --raw-field query=$QUERY --jq '.data.repository.releases.nodes | .[].tagName')
gh release view -R home-assistant/core $TAG
# 2022.11.0b2
# Pre-release • balloob released this about 1 hour ago
...

final thoughts

the issue can be mitigated as seen above, for a better solution we need GitHub to provide a GraphQL API for notifications. The goal would be to retrieve hundreds of notification details with a single request.

According to the official GitHub public roadmap, this does not seem to be planned.

Related threads

Get notification list via GraphQL API - 13/Mar/19
Can't get release neither comment using their id via GraphQL API - 7/Oct/19
Notifications GraphQL API - 19/Mar/22


Questions about making more complex arbitrary A

I have a number of functions that I would like to test which take some data that is based on other data. For example let's say that I wanted to test a function equivalent to String.split() so lets says
split : string -> string -> [string]
split splitToken strToSplit
I could construct a test by doing
jsc.assert(jsc.forall(jsc.pair(jsc.asciistring, jsc.asciistring), function(stringpair ) {
  //various assertions about split(stringpair[0], stringpair[1])
})))

However this isn't a very good test because most often the generated pairs don't have anything in common. In particular we want splitToken to be a substring of strToSplit
A similar situation I am trying to test is generating a random graph of the form
{
  nodes: [string],
  edges: [pair(string, string)] //Where the string values are a in the set of nodes
}

I wrote the following which gives roughly what I want. However it returns an Arb (Arb B) so when I use sample it returns a Arb B rather than a B
function genEdges(x) {
    return jsc.record({
        nodes: jsc.constant(x.nodes),
        edges: jsc.array(jsc.tuple(
            [jsc.elements(x.nodes), jsc.elements(x.nodes)]
        ))
    });
}
var graphbuilder = jsc.record({nodes: jsc.array(jsc.asciistring)}).smap(genEdges);
jsc.sampler(graphbuilder)();

What is the correct way to construct dependent test cases like this?
Thanks

The first example is simple. If you need some structural property, construct your value so it has it:
// btw, j`sc.forall` can generate multiple arguments:
jsc.assert(jsc.forall(jsc.asciistring, [jsc.asciistring], function(splitToken, arrToSplit) {
  var strToSplit = arrToSplit.join(splitToken); // now the split token is in the string by construction!
  //various assertions about split(splitToken, arrToSplit);
})));

Similarly graph could be constructed by:
function toGraph(nodes, edgesIdx) {
  // (nat, nat) -> (nodeid, nodeid)
  var edges = _.chain(edgesIdx)
    .map(function (e) {
      if (e[0] < nodes.length && e[1] < nodes.length) {
        return [nodes[e[0]], nodes[e[1]]];
      }
    })
    .collect()
    .value();

  return {
    nodes: nodes,
    edges: edges,
  };
}

// the idea is to generate edges not directly, but thru indexes:
var graphArb = jsc.pair(jsc.array(jsc.asciistring), jsc.array(jsc.pair(jsc.nat, jsc.nat))).smap(toGraph, fromGraph);

Note: smap needs an inverse too.
There isn't flatmap, maybe there will. There definitely will be map.

Thanks,
So would the inverse function for the graph look something like
function fromGraph(graph) {
  var edge_indices = graph.edges.map(function(edge) {
     return [nodes.indexOf(edge[0]), nodes.indexOf(edge[1])]
  });
  return [graph.nodes, edge_indices];
}

Yes, that function looks about right. I emphasise, it doesn't need to be proper inverse, just that
toGraph(fromGraph(graph) ≡ graph

holds. In this case you don't need to care about the index pairs we drop with .compact().

Awesome,
Last couple question, what is the intended use case for the generator and shrink family of functions? It seems like the arbitraries could generate random data for any other conceivable property that one would want to test. What do they give us that Arbitrary doesn't?

They lay on the lower level. Sometimes you need more control on how shrinking strategies, especially with recursive data.

Interesting, I'll have to think more about that. Thanks again for all your help. It's a very exciting library!


AndroidControllers: possibility to not ignore some buttons

ffe15f0d introduced that AndroidControllers extension ignore all controller buttons that are not considered a Gamepad button. However, some users could be irritated that some buttons on their controller can't be used. With this commit, the developer can switch to the old behaviour and the game will get events for all buttons the controller provides.

Added this to be backwards compatible.


STM32F3DISCOVERY replacement

Hello all,
Given that the board that he book is based on is no longer available, is it worth making an updated version?  I imagine the easy answer is yes but the next question is what the replacement development board should be.  One from the NUCLEO series?  The STM32F3348-DISCO?
Another question is how to manage multiple copies of the book with different boards.  A branch per board?
Note: I am happy to do the work, but it would be good to have alignment before I start.
Best wishes.

I was gonna look into rewriting large parts of this books as per #262. Without actually having one of these discovery boards it seems kinda hard to me to actually figure out whether my rewrite works though so a big :+1: from my side on using a widely available board being used for this. Until we switch or I get my hands on one I will be kind of blocked on this sadly.

If the discovery board is not obtainable, my suggestion would be to target a nucleo board.
I found the nucleo-64 with the stm32f446retx on it to be easily obtainable. The question then becomes finding a good handful of sensors and the sort to go with.

Do you have a source on it being NLA? I bought one off Amazon (US) a few days ago.

I talked to another person on Discord yesterday who also reported not being able to find anything online. And just now i went through the first two pages or duck duck go results for the board and couldnt get anything in stock there either, not even at big companies like Mouser.
So even if it might be in stock somewhere it doesnt seem to me like it is trivial to acquire one, hence Id definitely be in for a change.

Just chiming in here: I tried to get one of these boards (located in Germany) to start the guide and I can't get my hands on any. It's either noted as out-of-stock or delivery date in 4+ weeks.

It has been agreed during the wg meeting that we will be looking into using the micro:bit for the rewrite so stay tuned for that.
Closing as this resolves the original question asked in the issue

@sebpuetz for what it's worth, I was also put off by the 4 week delivery date. ordered it anyhow, thinking I probably wouldn't get to it sooner anyhow, and got it within a few days... :P

Mouser and digi-key have these in stock again.  Perhaps just a supply chain disruption.

I think the discovery board is still available. I just bought two discovery boards from a german electronics vendor and they were delivered in two days. As @David-OConnor pointed out, did someone check the official status (as by STMicro)?
Their website says the STM32F3DISCOVERY is active and in volume production.

FYI, I just ordered direct from STMicro's website and received it in 2 days. It was also at least half the price of other websites.

And, ironically, it looks like micro:bit may be out of stock as of now, at least in the US.

And, ironically, it looks like micro:bit may be out of stock as of now, at least in the US.

v1 or v2? ;)


Update README documentation for the new package.

LGTM

[Current coverage][cc-pull] is 98.47%

Merging [#292][cc-pull] into [development][cc-base-branch] will not change coverage

@@           development       #292   diff @@
=============================================
  Files               34         34
  Lines             2622       2622
  Methods              0          0
  Messages             0          0
  Branches           130        130
=============================================
  Hits              2582       2582
  Misses              35         35
  Partials             5          5

Powered by Codecov. Last updated by [696adfb...0ef7584][cc-compare]
[cc-base-branch]: https://codecov.io/gh/GoogleCloudPlatform/compute-image-packages/branch/development?src=pr
[cc-compare]: https://codecov.io/gh/GoogleCloudPlatform/compute-image-packages/compare/696adfba023c573fdf49d33081dfb253c9d7245e...0ef7584db721d3a2afe64f59f7ddc2d45a5aa642
[cc-pull]: https://codecov.io/gh/GoogleCloudPlatform/compute-image-packages/pull/292?src=pr


RAM suggestions and Swap

Some people have issues compiling or running because of RAM. I added a "suggested ram" thingy to 4GB, and directed users to a tutorial on how to add swap if it was not working with below 4GB of RAM.
Nothing big, just think more documentation would be nice so the "Issues" section doesn't get clogged as much.

Oops, did I end up attempting to merge into master?
I meant to merge into 1.9.
Well, we can worry about this when 1.9 comes :)

Revisit this later, find "sane" numbers maybe


Add Token Range to Endpoints endpoint

Fixes #374

Confirming that I've manually tested this against a real cluster and it seems to behave as expected:
$ curl <IP_ADDRESS>:8080/api/v2/tokens/rangetoendpoint | jq
{
  "token_range_to_endpoints": [
    {
      "tokens": [
        -1070326447217483000,
<PHONE_NUMBER>84027300
      ],
      "endpoints": [
        "<IP_ADDRESS>"
      ]
    },
    {
      "tokens": [
        -3772959478813367000,
        -1070326447217483000
      ],
      "endpoints": [
        "<IP_ADDRESS>"
      ]
    },
    {
      "tokens": [
        -5592763185766151000,
        -3772959478813367000
      ],
      "endpoints": [
        "<IP_ADDRESS>"
      ]
    },
    {
      "tokens": [
        -8583309438055266000,
        -5592763185766151000
      ],
      "endpoints": [
        "<IP_ADDRESS>"
      ]
    },
    {
      "tokens": [
<PHONE_NUMBER>168102400,
<PHONE_NUMBER>656065000
      ],
      "endpoints": [
        "<IP_ADDRESS>"
      ]
    },
    {
      "tokens": [
<PHONE_NUMBER>84027300,
<PHONE_NUMBER>168102400
      ],
      "endpoints": [
        "<IP_ADDRESS>"
      ]
    },
    {
      "tokens": [
<PHONE_NUMBER>656065000,
<PHONE_NUMBER>716966000
      ],
      "endpoints": [
        "<IP_ADDRESS>"
      ]
    },
    {
      "tokens": [
<PHONE_NUMBER>716966000,
        -8583309438055266000
      ],
      "endpoints": [
        "<IP_ADDRESS>"
      ]
    }
  ]
}

The only nit left to address is the name of the test class, I've approved so it can be merged as soon as that is updated.

@emerkle826 do you want to make that final change to the name of the test class and get this merged?


api bot

API bot is an api server that posts messages to redis
can be used for:

notifications from internet - like github hooks
send test messages
get service status

update config is done, can send to discord, closing this general issue off,
can open up specific features as they come up


Placement Width and Height Return is Zero In Dynamic Size Zone

PlacementRequestConfig config = new PlacementRequestConfig.Builder(170188, 273944, 320, 75).build();

AdButler adbutler = new AdButler();
adbutler.requestPlacement(config, new PlacementResponseListener() {
    @Override
    public void success(PlacementResponse placementResponse) {
        List<Placement> placementList = placementResponse.getPlacements();

        if ( !placementList.isEmpty() ){
            final Placement placement = placementList.get(0);

            if ( placement != null ){
                if ( BuildConfig.DEBUG ){
                    Log.e(LOG_TAG, String.format(Locale.US, "Placement Width:(%d) Height:(%d)", placement.getWidth(), placement.getHeight()));
                }
            }
        }
    }
}

#1

Thanks for the reminder, the dimensions returned should be those of the selected ad - we're going to fix that shortly, and I'll update this issue again once complete.

For now - as a work around - I'd recommend adding banners to campaigns in Advertisers, rather than in the zone directly.

We've made a change to the behavior of images in Dynamic Zones. I'd recommend contacting support directly for further help with your account regarding this issue!


tensorflow gredient Tape

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with:
python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Source code / 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. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

@yashthesia ,
Can you please fill the details for all the fields in Standard template ?Thanks!

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

@yashthesia ,
Any update on the issue?Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Update opentelemetry-sdk-testing to 1.37.0

About this PR
📦 Updates io.opentelemetry:opentelemetry-sdk-testing from 1.11.0 to 1.37.0
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.opentelemetry", artifactId = "opentelemetry-sdk-testing" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "io.opentelemetry", artifactId = "opentelemetry-sdk-testing" }
}]

labels: test-library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #663.


47195 Reskin/Uplift daily NM content

Code changes for Reskin/Uplift daily NM Content and Individual file download
Unit Test cases
Functional Test cases
Integration Tests
Note: Implemented PBI 45744 Daily NM individual file download in this feature branch

@ray-sj, @nevillejrbrown & @prasad07b please review the changes you requested asap so that this feature can be completed.


Matplotlib upgrade

Hey @sash-a, thought I'd grab a look at #253.
Changes here update Matplotlib version and then makes changes to pass type checking, so it builds and passes tests.
But ...
The new API now requires that the animation update function returns a list of artists that have been updated, these are objects returned from plotting calls like a plot object, or text box (see the table here). The state/data of these objects can be updated without redrawing the whole thing.
I've done this rubiks cube environment:
# Create the first frame of the plot and return a list of drawn image artists
images = self._draw(ax, states[0])

def make_frame(state: State) -> Sequence[Artist]:
    # Update the data in each image from the state
    for i, image in enumerate(images):
        image.set_data(state.cube[i])
    # Return the updated images
    return images

# Create the animation object.
self._animation = matplotlib.animation.FuncAnimation(
    fig,
    make_frame,
    frames=states,
    interval=interval,
)

Doing this for all environments is easier for some than others given how they've been written, it means passing all the relevant artists to the animate function when the plot is created, and then correctly updating them inside the update function.
I think the animations will still work as is, as they still generally update the axes by reference inside the animation function, it's just less efficient then the new API that just updates the relevant artists.

Wow this is an awesome addition, thanks @zombie-einstein!
Which environments are still remaining?
Also I think the way you've done it with make_frame looks good, but I have no experience with the new API

Which environments are still remaining?

Apart from rubix cube they all need a look at, with varying degrees of complexity. I think the general pattern needs to be something like a draw function that draws the first frame (and also draws the individual frames via the render method, and returns any actors that will be updated during the animation, then use this to initialise the first frame and update actors inside the update-func.
Some also currently have actors that are created dynamically during plotting I've been trying to work out the best pattern for, in some cases it seems it would be more efficient to initialise all the actors ahead of time, and update their states as needed. E.g. for the sudoko environment initialise all the text actors and update the text duting animation (rather than create them on the fly).
So there's a decent chunk of work to refactor all the plots. One option could be merge something like this PR, which will updated the requirement and passes linting/tests, and still create plots (just less efficiently), and then incrementally update the individual viewers in separate PRs to make best use of the new API.


Merge Download Paddle Model, Paddle to Onnx, Onnx to Mlir, Mlir to Bmodel into infer.py

PR types(PR类型)

Description

 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.

Optimized the function logic, changed the function name style add "--auto" into parser and modified README.md, README_CN.md documentation.


Possible dead project?

Hello kmwhite,
I am currently creating a project and would like to use the name artisan on PyPI, seeing as this project hasn't had any updates in the past four years and there isn't anyone downloading the package from PyPI I came to ask if you would perhaps unpublish the module from PyPI so I may use the name? :)
If you decide not to it is okay, but I would be very appreciative if you were able to do this.
Thank you so much,
Seth Larson

Sure. I can unpublish. I didn't get the time I wanted to for working on this, so I am happy to release the name. :) Can I do it later today?

You can do it whenever, thank you so much for taking time out of your schedule for this. :) Kudos to you! Just tweet at me or post here when you do.  Thanks again!!! :tada: :cake: :tada:

Released.


look to mapPartitions for performance increase

After converting XSLT transformations that use pyjxslt to use rdd.mapPartitions (commit), and a relatively substantial performance increase, look to other areas where this could be applied:

validation udfs in record_validation.py
python transformation

move the temp ModuleType to the partition mapping

Field mapping for ES, for index_mapper_handle() that comes from GenericMapper

Anything that is not row specific, can be removed from the UDF and applied at the partition level.

Aside from instantiating index mapper only once per partition, as opposed to each record -- which has been implemented -- not seeing much potential for improvement from mapPartitions in field mapping.

Applied where applicable, some definite performance gains (2-8x, depending).


Support IPv6

Support for ipv6 addresses would be nice.

Slight correction. It is supported... However, it only seems to pinpoint it up to the location of the country.


Change pki validity defaults such that TRC > Issuer > Leaf.

This prevents the occasional CI blow-up when a leaf cert expires
slightly after an issuer cert, etc.

This change is

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

python/lib/crypto/certificate.py, line 70 at r1 (raw file):

    :cvar str enc_alorithm: default algorithm used to encrypt messages.
    """
    AS_VALIDITY_PERIOD = 363 * 24 * 60 * 60

They are only used in the generator, so we might as well move these constants there.

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, 1 unresolved discussion.

python/lib/crypto/certificate.py, line 70 at r1 (raw file):
Previously, shitz wrote…
They are only used in the generator, so we might as well move these constants there.

Done.

Comments from Reviewable

Reviewed 3 of 3 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Exception in FrequencyBasedBootstrapping -> TokenizerStandford

java.lang.NoSuchMethodError: edu.stanford.nlp.process.DocumentPreprocessor.setTokenizerFactory(Ledu/stanford/nlp/process/TokenizerFactory;)V
at io.github.infolis.algorithm.TokenizerStanford.applyPTBTokenizer(TokenizerStanford.java:59)
to reproduce, use example call in wiki
(see https://github.com/infolis/infolis-web/issues/85)

This was a bug in the build dependencies for infolis/infoLink. I think this is fixed now.


DeviceOrientation API for accessing Accelerometer & Gyroscope sensor data

DeviceOrientation API for accessing Accelerometer & Gyroscope sensor data

This issue was imported from Google Moderator
Moderator votes: +6
Issue added by mmiesi on 2012-03-06
To vote this issue up or down, simply include +1 or -1 in your comment.

Covered by http://caniuse.com/#feat=deviceorientation


Subclass documentation #121

This is a proposed fix for issue
Subclasses keep the parent's .documentation? #121
Currently each Entity keeps track of its own exposures in a class object instance variable. A new entity class makes a copy of it's parent exposures then goes on and modifies its own exposures (add or remove exposures).
For the moment, I think we can fix the documentation issue by simply removing the call to superclass.documentation.
In the future we might have to replace the class object instance variable (which is a simple hash at the moment) with something more elaborate (like a chained filter or something) to handle the child - parent relationship better. Ex. what happens if we add some more exposures to the parent entity after we defined the child entity? I think that the child entity will not catch that extra exposure.
Please review this pull request and let me know what you think.

This looks good. Please update CHANGELOG and squash the commits.

Thank you for reminding me.
Done
On Mon, Apr 13, 2015 at 1:29 AM, Daniel Doubrovkine (dB.) @dblockdotorg <
<EMAIL_ADDRESS>wrote:

This looks good. Please update CHANGELOG and squash the commits.
—
Reply to this email directly or view it on GitHub
https://github.com/intridea/grape-entity/pull/122#issuecomment-92060020.

Can you please squash these and correct the super minor thing in CHANGELOG? Thanks.

squashed

Merging, thanks.


Add hover state to TableColumnHeader when 'allowSorting' is true

Closes #5284
Hello, this PR adds support for hover state (data-hovered) in TableColumnHeader when Column is sortable (allowsSorting = true).
I'm not 100% sure whether data-hovered should only be supported when a column is sortable, but judging by the discussions in #5284 and #4411, it seems we want hover state support only when an item (whether a column or a row) is interactive? I'd love to know you guys' thoughts on this.
Thanks!
✅ Pull Request Checklist:

[x] Included link to corresponding React Spectrum GitHub Issue.
[x] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

GET_BUILD


[FLOC-973] Changed GitHub issue links to JIRA

Fixes https://clusterhq.atlassian.net/browse/FLOC-973 along with running the script posted on that issue.
I have replaced all occurrences of https://github.com/ClusterHQ/flocker/issues/ with https://clusterhq.atlassian.net/browse/FLOC-.
Note that
Talk to Us
==========

in CONTRIBUTING.rst remains in tact.

There's one issue link that pre-dates the switch to ClusterHQ organisation:
(link-to-jira-FLOC-973 ? =)[~/.../HybridLogic/flocker]$ ack https://github.com/hybridlogic
flocker/node/functional/test_docker.py
249:        https://github.com/hybridlogic/flocker/issues/120

And also various links to other ClusterHQ projects whose issues are now in JIRA.
(link-to-jira-FLOC-973 ? =)[~/.../HybridLogic/flocker]$ ack -i https://github.com/clusterhq

...

flocker/restapi/_infrastructure.py
73:        # See https://github.com/ClusterHQ/eliot/issues/29

admin/test/test_packaging.py
38:# See https://github.com/ClusterHQ/build.clusterhq.com/issues/32

May as well update those here I suppose and make a note in the parent ticket to close those issues using your script.
Then please merge.

The eliot issue is not in JIRA (@robhaswell says rightly so) and neither is the build.clusterhq.com issue but I've removed that XXX as it has been done (confirmed by @wallrj ). I will create a new subtask of the parent to move issues from our other repositories to JIRA where appropriate.


chore: move 'moreLeanArgs' and 'weakLeanArgs' from 'lean_lib's to 'package'

As far as I can see, these options should just be set for all the libraries in the mathlib package, and it is only historical accident that they were set on a per-library basis.
Ping @tydeu and @digama0 as this came up in our discussion, but I don't think they need to review.

Rolled into #8456


Is there documentation and demo for this extension?

Is there documentation and demo for this extension? Are you considering using a connection pool to manage connections?

@janyyy this extension only enables JSch to be compiled to native. You can look at the integration tests if you're looking for an example on how to use JSch, but I think you'll find more examples in http://www.jcraft.com/jsch/

The current extension documentation lies in https://quarkiverse.github.io/quarkiverse-docs/quarkus-jsch/dev/index.html


Add flag for badging.

JIRA: https://openedx.atlassian.net/browse/MA-1669

PR Cover Letter

[ ] A detailed description of the changes in the body of the PR
[ ] If available, a link to the relevant JIRA ticket in the body of the PR

[PROJ-XXXX](https://openedx.atlassian.net/browse/PROJ-XXXX)

Sandbox

[ ] Build a sandbox for your branch and add a link here

Testing

[ ] i18n
[ ] RTL
[ ] Unit, integration, acceptance tests as appropriate
[ ] Analytics
[ ] Performance
[ ] Database migrations are backwards-compatible

Reviewers
If you've been tagged for review, please check your corresponding box once you've given the :+1:.

[ ] Code review #1
[ ] Code review #2
[ ] Docs review (required for UI strings/error messages)
[ ] UX review
[ ] Accessibility review
[ ] Product review

Documentation considerations

[ ] Are there user visible changes? If so, list who is affected, and include a pointer to the relevant code.
[ ] Should this be mentioned in the release notes? If so, make sure that you included a description.

DevOps assistance
If it's needed, or even if you suspect it might be needed, do any/all these things as early as possible. Don't surprise devops right before merging! For example, touching envs/aws.py or or envs/common.py are dead giveaways that you'll need assistance when pushing your changes out. Things that require devops attention include, but are not limited to:

configuration changes
feature flags
migrations
Ways to get devops attention, pick-and-choose whichever is most appropriate for your specific PR:
[ ] Tag devops on your PR (@edx/devops), either for review (more specific to devops = better) or just a general FYI
[ ] File a support ticket

FYI: Tag anyone who might be interested in this PR here.
Post-review

[ ] Squash commits

@mikekatz please review

Current coverage is 48.80%

Merging #700 into master will decrease coverage by -0.01% as of 0fa8d1b

@@            master    #700   diff @@
======================================
  Files          436     436
  Stmts        28449   28451     +2
  Branches         0       0
  Methods          0       0
======================================
  Hit          13886   13886
  Partial          0       0
- Missed       14563   14565     +2

Review entire Coverage Diff as of 0fa8d1b

Powered by Codecov. Updated on successful CI builds.

:+1:


Source code of the demo

I cannot find the full source code including the form input fields

I found some demo source code here: https://ovvwzkzry9.codesandbox.io/

Just use the ID in the sandbox editor.
https://codesandbox.io/s/ovvwzkzry9


Weird "default" answer from the external API?

In #5308 we saw a weird answer from a newer external API endpoint when you query it in a certain way:
{
  "available": []
}

This response doesn't really seem to have anything to do with that specific endpoint, so I dug around a bit and indeed, you can also get it from other, totally unrelated endpoints, for example
curl -v -L 'https://develop.opencast.org/api/workflows/1.json' \
  -H 'accept: application/v1.0.0+json' \
  -u admin:opencast

Two things to note about that call:

The endpoint doesn't actually end in .json,
and we are querying a version which it is not yet part of.

With both of these, that should also give you the weird response. It seems to be some kind of fallback response? But it's not very useful at that, so I don't think it's intended?

Seems to be somehow related to adding .json to the endpoint AND explicitly specifying an external api version header. The specified external api endpoint does not even get called, so the weird response is coming from somewhere else.


Remove macro definitions of snprintf and vsnprintf

These macros can create problems in projects that include xgboost and use vsnprintf/snprintf.
vsnprintf and snprintf are supported since Visual Studio 14, so we don't need these macros anymore.

Thank god I just have Autogluon for this now! LOL
On Sat, Dec 25, 2021 at 3:51 PM Jiaming Yuan @.***>
wrote:

@.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
https://github.com/dmlc/xgboost/pull/7536#pullrequestreview-840042360,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIVELNDJIC7M5YL4HZS3POLUSZKGVANCNFSM5KX23Z5Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>