[BUG] HugeCTR backend   crashed when I make pressure test with 100 concurrents，and the coredump file not generated.

HugeCTR backend   crashed when I make pressure test with 100 concurrents . and the coredump file not generated.
I1215 03:27:54.113361 33740 hugectr.cc:1438] model h3_hugectr, instance h3_hugectr, executing 1 requests
I1215 03:27:54.113399 33740 hugectr.cc:1512] request 0: id = "1", correlation_id = 0, input_count = 3, requested_output_count = 1
I1215 03:27:54.113373 33740 http_server.cc:2727] HTTP request: 2 /v2/models/h3_hugectr/infer
I1215 03:27:54.113420 33740 hugectr.cc:1610]    input CATCOLUMN: datatype = INT64, shape = [1,576], byte_size = 4608, buffer_count = 1
I1215 03:27:54.113434 33740 model_repository_manager.cc:615] GetInferenceBackend() 'h3_hugectr' version -1
I1215 03:27:54.113466 33740 model_repository_manager.cc:615] GetInferenceBackend() 'h3_hugectr' version -1
I1215 03:27:54.113455 33740 hugectr.cc:1626]    input ROWINDEX: datatype = INT32, shape = [1,577], byte_size = 2308, buffer_count = 1
I1215 03:27:54.113478 33740 hugectr.cc:1640]    input DES: datatype = FP32, shape = [1,704], byte_size = 2816, buffer_count = 1
I1215 03:27:54.113484 33740 hugectr.cc:1665]    requested_output OUTPUT0
I1215 03:27:54.113495 33740 infer_response.cc:165] add response output: output: OUTPUT0, type: FP32, shape: [32]
I1215 03:27:54.113493 33740 infer_request.cc:524] prepared: [0x0x7f1f0800d230] request id: 1, model: h3_hugectr, requested version: -1, actual version: 1, flags: 0x0, correlation id: 0, batch size: 1, priority: 0, timeout (us): 0
original inputs:
[0x0x7f1f0800aa88] input: ROWINDEX, type: INT32, original shape: [1,577], batch + shape: [1,577], shape: [577]
[0x0x7f1f0800edc8] input: CATCOLUMN, type: INT64, original shape: [1,576], batch + shape: [1,576], shape: [576]
[0x0x7f1f0800b7d8] input: DES, type: FP32, original shape: [1,704], batch + shape: [1,704], shape: [704]
override inputs:
inputs:
[0x0x7f1f0800b7d8] input: DES, type: FP32, original shape: [1,704], batch + shape: [1,704], shape: [704]
[0x0x7f1f0800edc8] input: CATCOLUMN, type: INT64, original shape: [1,576], batch + shape: [1,576], shape: [576]
[0x0x7f1f0800aa88] input: ROWINDEX, type: INT32, original shape: [1,577], batch + shape: [1,577], shape: [577]
original requested outputs:
OUTPUT0
requested outputs:
OUTPUT0

I1215 03:27:54.113504 33740 http_server.cc:1051] HTTP: unable to provide 'OUTPUT0' in GPU, will use CPU
I1215 03:27:54.113543 33740 http_server.cc:1071] HTTP using buffer for: 'OUTPUT0', size: 128, addr: 0x7f1f44bca620
I1215 03:27:54.113581 33740 hugectr.cc:1438] model h3_hugectr, instance h3_hugectr, executing 1 requests
I1215 03:27:54.113599 33740 hugectr.cc:1512] request 0: id = "1", correlation_id = 0, input_count = 3, requested_output_count = 1
I1215 03:27:54.113615 33740 hugectr.cc:1610]    input CATCOLUMN: datatype = INT64, shape = [1,576], byte_size = 4608, buffer_count = 1
I1215 03:27:54.113629 33740 hugectr.cc:1626]    input ROWINDEX: datatype = INT32, shape = [1,577], byte_size = 2308, buffer_count = 1
I1215 03:27:54.113640 33740 hugectr.cc:1640]    input DES: datatype = FP32, shape = [1,704], byte_size = 2816, buffer_count = 1
I1215 03:27:54.113646 33740 hugectr.cc:1665]    requested_output OUTPUT0
I1215 03:27:54.113651 33740 infer_response.cc:165] add response output: output: OUTPUT0, type: FP32, shape: [32]
I1215 03:27:54.113663 33740 http_server.cc:1051] HTTP: unable to provide 'OUTPUT0' in GPU, will use CPU
I1215 03:27:54.113674 33740 http_server.cc:1071] HTTP using buffer for: 'OUTPUT0', size: 128, addr: 0x7f1fdcbc3b60
I1215 03:27:54.115386 33740 hugectr.cc:1815] *****Processing request on device***** 0 for model h3_hugectr
I1215 03:27:54.120122 33740 hugectr.cc:1815] *****Processing request on device***** 0 for model h3_hugectr
terminate called after throwing an instance of 'std::system_error'
  what():  Resource temporarily unavailable
Signal (6) received.
terminate called recursively
Signal (6) received.
 0# 0x0000555E676798A9 in tritonserver
 1# 0x00007F22576FF210 in /usr/lib/x86_64-linux-gnu/libc.so.6
 2# gsignal in /usr/lib/x86_64-linux-gnu/libc.so.6
 3# abort in /usr/lib/x86_64-linux-gnu/libc.so.6
 4# 0x00007F2257AB5911 in /usr/lib/x86_64-linux-gnu/libstdc++.so.6
 5# 0x00007F2257AC138C in /usr/lib/x86_64-linux-gnu/libstdc++.so.6
 6# 0x00007F2257AC13F7 in /usr/lib/x86_64-linux-gnu/libstdc++.so.6
 7# 0x00007F2257AC16A9 in /usr/lib/x86_64-linux-gnu/libstdc++.so.6
 8# std::__throw_system_error(int) in /usr/lib/x86_64-linux-gnu/libstdc++.so.6
 9# 0x00007F2257AEE0BD in /usr/lib/x86_64-linux-gnu/libstdc++.so.6
10# HugeCTR::embedding_cache<long long>::look_up(void const*, std::vector<unsigned long, std::allocator<unsigned long> > const&, float*, HugeCTR::MemoryBlock*, std::vector<CUstream_st*, std::allocator<CUstream_st*> > const&, float) in /usr/local/hugectr/lib/libhugectr_inference.so
11# HugeCTR::InferenceSession::predict(float*, void*, int*, float*, int) in /usr/local/hugectr/lib/libhugectr_inference.so
12# 0x00007F22383F404A in /opt/tritonserver/backends/hugectr/libtriton_hugectr.so
13# TRITONBACKEND_ModelInstanceExecute in /opt/tritonserver/backends/hugectr/libtriton_hugectr.so
14# 0x00007F225828B83A in /opt/tritonserver/bin/../lib/libtritonserver.so
15# 0x00007F225828C04D in /opt/tritonserver/bin/../lib/libtritonserver.so
16# 0x00007F2258140801 in /opt/tritonserver/bin/../lib/libtritonserver.so
17# 0x00007F2258285DC7 in /opt/tritonserver/bin/../lib/libtritonserver.so
18# 0x00007F2257AEDDE4 in /usr/lib/x86_64-linux-gnu/libstdc++.so.6
19# 0x00007F2257F6B609 in /usr/lib/x86_64-linux-gnu/libpthread.so.0
20# clone in /usr/lib/x86_64-linux-gnu/libc.so.6

Aborted (core dumped)
root@13609cdec7a0:/opt/tritonserver# ll
total 3612
drwxr-xr-x 1 root          root             4096 Nov 29 08:07  ./
drwxr-xr-x 1 root          root             4096 Oct 21 18:44  ../
-rw-r--r-- 1 root          root              641 Nov  4 19:20 '=1.21.6'
-rw-rw-r-- 1 triton-server triton-server    1485 Oct 21 17:20  LICENSE
-rw-rw-r-- 1 triton-server triton-server 3012640 Oct 21 17:20  NVIDIA_Deep_Learning_Container_License.pdf
-rw-rw-r-- 1 triton-server triton-server       7 Oct 21 17:20  TRITON_VERSION
drwxr-xr-x 1 triton-server triton-server    4096 Nov  4 19:33  backends/
drwxr-xr-x 2 triton-server triton-server    4096 Oct 21 18:45  bin/
drwxr-xr-x 1 triton-server triton-server    4096 Oct 21 18:45  include/
drwxr-xr-x 2 triton-server triton-server    4096 Oct 21 18:45  lib/
-rw------- 1 root          root           616982 Dec 10 02:29  nohup.out
-rwxrwxr-x 1 triton-server triton-server    4090 Oct 21 17:20  nvidia_entrypoint.sh*
drwxr-xr-x 3 triton-server triton-server    4096 Oct 21 18:48  repoagents/
drwxr-xr-x 2 triton-server triton-server    4096 Oct 21 18:45  third-party-src/
root@13609cdec7a0:/opt/tritonserver# ll /opt/tritonserver/backends/hugectr/
total 400
drwxr-xr-x 2 root root   4096 Nov  4 19:33 ./
drwxr-xr-x 3 root root   4096 Nov  4 19:33 ../
-rw-r--r-- 1 root root 398904 Nov  4 19:33 libtriton_hugectr.so

my start command：
 tritonserver --model-repository=/data/gux/h3/h3_hugectr/ --backend-config=hugectr,ps=/data/gux/h3/h3_hugectr/ps.json --load-model h3_hugectr --model-control-mode=explicit --log-verbose 1

#pressure test  command by siege

$ siege -H "Content-Type:application/json" "http://<IP_ADDRESS>:8088/xxxx/v1/api  POST < /home/t3mgr/data.json" -b -c 100  -t 5M

I1215 03:44:22.466890 66071 hugectr.cc:748] The model configuration:
{
"name": "h3_hugectr",
"platform": "",
"backend": "hugectr",
"version_policy": {
"latest": {
"num_versions": 1
}
},
"max_batch_size": 4096,
"input": [
{
"name": "DES",
"data_type": "TYPE_FP32",
"format": "FORMAT_NONE",
"dims": [
-1
],
"is_shape_tensor": false,
"allow_ragged_batch": false
},
{
"name": "CATCOLUMN",
"data_type": "TYPE_INT64",
"format": "FORMAT_NONE",
"dims": [
-1
],
"is_shape_tensor": false,
"allow_ragged_batch": false
},
{
"name": "ROWINDEX",
"data_type": "TYPE_INT32",
"format": "FORMAT_NONE",
"dims": [
-1
],
"is_shape_tensor": false,
"allow_ragged_batch": false
}
],
"output": [
{
"name": "OUTPUT0",
"data_type": "TYPE_FP32",
"dims": [
-1
],
"label_filename": "",
"is_shape_tensor": false
}
],
"batch_input": [],
"batch_output": [],
"optimization": {
"priority": "PRIORITY_DEFAULT",
"input_pinned_memory": {
"enable": true
},
"output_pinned_memory": {
"enable": true
},
"gather_kernel_buffer_threshold": 0,
"eager_batching": false
},
"instance_group": [
{
"name": "h3_hugectr_0",
"kind": "KIND_GPU",
"count": 3,
"gpus": [
0
],
"secondary_devices": [],
"profile": [],
"passive": false,
"host_policy": ""
}
],
"default_model_filename": "",
"cc_model_filenames": {},
"metric_tags": {},
"parameters": {
"cat_feature_num": {
"string_value": "18"
},
"config": {
"string_value": "/data/gux/h3/1/dcn.json"
},
"label_dim": {
"string_value": "1"
},
"max_nnz": {
"string_value": "1"
},
"embedding_vector_size": {
"string_value": "128"
},
"gpucacheper": {
"string_value": "0.5"
},
"des_feature_num": {
"string_value": "22"
},
"gpucache": {
"string_value": "true"
},
"embeddingkey_long_type": {
"string_value": "true"
},
"slots": {
"string_value": "18"
}
},
"model_warmup": []
}

Since the issue you encountered cannot be reproduced, we recommend that you use the performance testing tool provided by Triton , which includes a concurrent testing mechanism.
We also provide use cases in the dlrm inference notebook(5. Get Inference benchmark by Triton Performance Tool)for your reference .

按照/hugectr_backend/samples/dlrm/HugeCTR_DLRM_Inference.ipynb 使用perf_analyzer压测， triton crashed
压测命令
perf_analyzer -m dlrm -u localhost:8100 --input-data 1.json --shape CATCOLUMN:26 --shape DES:13 --shape ROWINDEX:27   --concurrency-range 1:4
启动模型服务
root@13609cdec7a0:/opt/tritonserver# tritonserver --model-repository=/data/dlrm_train/dlrm_infer/model/ --load-model=dlrm --backend-config=hugectr,ps=/data/dlrm_train/dlrm_infer/model/ps.json  --backend-directory=/usr/local/hugectr/backends  --model-control-mode=explicit    --grpc-port 8101  --http-port 8100    --metrics-port 8102
错误日志
[HUGECTR][02:16:08][INFO][RANK0]: Parameter server lookup of 0 / 0 embeddings took 155 us.
I1220 02:16:08.129594 539261 hugectr.cc:1832] Prediction execution time is 0 ms
I1220 02:16:08.129628 539261 hugectr.cc:1438] model dlrm, instance dlrm, executing 1 requests
I1220 02:16:08.129638 539261 hugectr.cc:1512] request 0: id = "", correlation_id = 0, input_count = 3, requested_output_count = 1
I1220 02:16:08.129646 539261 hugectr.cc:1610]   input CATCOLUMN: datatype = INT64, shape = [1,26], byte_size = 208, buffer_count = 1
I1220 02:16:08.129652 539261 hugectr.cc:1626]   input ROWINDEX: datatype = INT32, shape = [1,27], byte_size = 108, buffer_count = 1
I1220 02:16:08.129663 539261 hugectr.cc:1640]   input DES: datatype = FP32, shape = [1,13], byte_size = 52, buffer_count = 1
I1220 02:16:08.129672 539261 hugectr.cc:1665]   requested_output OUTPUT0
I1220 02:16:08.129713 539261 hugectr.cc:1815] Processing request on device 0 for model dlrm
[HUGECTR][02:16:08][INFO][RANK0]: Looking up 0 embeddings (each with 128 values)...
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory backend. Table: dlrm#0. Fetched 0 / 0 values.
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory: 0 hits, 0 missing!
[HUGECTR][02:16:08][INFO][RANK0]: Parameter server lookup of 0 / 0 embeddings took 139 us.
I1220 02:16:08.130077 539261 hugectr.cc:1823] *Processing request completed!
I1220 02:16:08.130116 539261 hugectr.cc:1832] Prediction execution time is 0 ms
I1220 02:16:08.130153 539261 hugectr.cc:1438] model dlrm, instance dlrm, executing 1 requests
I1220 02:16:08.130164 539261 hugectr.cc:1512] request 0: id = "", correlation_id = 0, input_count = 3, requested_output_count = 1
I1220 02:16:08.130177 539261 hugectr.cc:1610]   input CATCOLUMN: datatype = INT64, shape = [1,26], byte_size = 208, buffer_count = 1
I1220 02:16:08.130192 539261 hugectr.cc:1626]   input ROWINDEX: datatype = INT32, shape = [1,27], byte_size = 108, buffer_count = 1
I1220 02:16:08.130201 539261 hugectr.cc:1640]   input DES: datatype = FP32, shape = [1,13], byte_size = 52, buffer_count = 1
I1220 02:16:08.130212 539261 hugectr.cc:1665]   requested_output OUTPUT0
I1220 02:16:08.130255 539261 hugectr.cc:1815] Processing request on device 0 for model dlrm
[HUGECTR][02:16:08][INFO][RANK0]: Looking up 0 embeddings (each with 128 values)...
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory backend. Table: dlrm#0. Fetched 0 / 0 values.
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory: 0 hits, 0 missing!
[HUGECTR][02:16:08][INFO][RANK0]: Parameter server lookup of 0 / 0 embeddings took 140 us.
I1220 02:16:08.130619 539261 hugectr.cc:1823] *Processing request completed!
I1220 02:16:08.130661 539261 hugectr.cc:1832] Prediction execution time is 0 ms
I1220 02:16:08.130691 539261 hugectr.cc:1438] model dlrm, instance dlrm, executing 1 requests
I1220 02:16:08.130701 539261 hugectr.cc:1512] request 0: id = "", correlation_id = 0, input_count = 3, requested_output_count = 1
I1220 02:16:08.130716 539261 hugectr.cc:1610]   input CATCOLUMN: datatype = INT64, shape = [1,26], byte_size = 208, buffer_count = 1
I1220 02:16:08.130724 539261 hugectr.cc:1626]   input ROWINDEX: datatype = INT32, shape = [1,27], byte_size = 108, buffer_count = 1
I1220 02:16:08.130732 539261 hugectr.cc:1640]   input DES: datatype = FP32, shape = [1,13], byte_size = 52, buffer_count = 1
I1220 02:16:08.130740 539261 hugectr.cc:1665]   requested_output OUTPUT0
I1220 02:16:08.130779 539261 hugectr.cc:1815] Processing request on device 0 for model dlrm
[HUGECTR][02:16:08][INFO][RANK0]: Looking up 0 embeddings (each with 128 values)...
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory backend. Table: dlrm#0. Fetched 0 / 0 values.
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory: 0 hits, 0 missing!
I1220 02:16:08.131132 539261 hugectr.cc:1823] *Processing request completed![HUGECTR][02:16:08][INFO][RANK0]: Parameter server lookup of 0 / 0 embeddings took 151 us.
I1220 02:16:08.131191 539261 hugectr.cc:1832] Prediction execution time is 0 ms
I1220 02:16:08.131238 539261 hugectr.cc:1438] model dlrm, instance dlrm, executing 1 requests
I1220 02:16:08.131250 539261 hugectr.cc:1512] request 0: id = "", correlation_id = 0, input_count = 3, requested_output_count = 1
I1220 02:16:08.131260 539261 hugectr.cc:1610]   input CATCOLUMN: datatype = INT64, shape = [1,26], byte_size = 208, buffer_count = 1
I1220 02:16:08.131269 539261 hugectr.cc:1626]   input ROWINDEX: datatype = INT32, shape = [1,27], byte_size = 108, buffer_count = 1
I1220 02:16:08.131277 539261 hugectr.cc:1640]   input DES: datatype = FP32, shape = [1,13], byte_size = 52, buffer_count = 1
I1220 02:16:08.131287 539261 hugectr.cc:1665]   requested_output OUTPUT0
I1220 02:16:08.131328 539261 hugectr.cc:1815] Processing request on device 0 for model dlrm
[HUGECTR][02:16:08][INFO][RANK0]: Looking up 0 embeddings (each with 128 values)...
I1220 02:16:08.131681 539261 hugectr.cc:1823] *Processing request completed!
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory backend. Table: dlrm#0. Fetched 0 / 0 values.
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory: 0 hits, 0 missing!
[HUGECTR][02:16:08][INFO][RANK0]: Parameter server lookup of 0 / 0 embeddings took 170 us.
I1220 02:16:08.131716 539261 hugectr.cc:1832] Prediction execution time is 0 ms
I1220 02:16:08.131747 539261 hugectr.cc:1438] model dlrm, instance dlrm, executing 1 requests
I1220 02:16:08.131757 539261 hugectr.cc:1512] request 0: id = "", correlation_id = 0, input_count = 3, requested_output_count = 1
I1220 02:16:08.131767 539261 hugectr.cc:1610]   input CATCOLUMN: datatype = INT64, shape = [1,26], byte_size = 208, buffer_count = 1
I1220 02:16:08.131773 539261 hugectr.cc:1626]   input ROWINDEX: datatype = INT32, shape = [1,27], byte_size = 108, buffer_count = 1
I1220 02:16:08.131778 539261 hugectr.cc:1640]   input DES: datatype = FP32, shape = [1,13], byte_size = 52, buffer_count = 1
I1220 02:16:08.131786 539261 hugectr.cc:1665]   requested_output OUTPUT0
I1220 02:16:08.131836 539261 hugectr.cc:1815] Processing request on device 0 for model dlrm
[HUGECTR][02:16:08][INFO][RANK0]: Looking up 0 embeddings (each with 128 values)...
I1220 02:16:08.132182 539261 hugectr.cc:1823] *Processing request completed!
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory backend. Table: dlrm#0. Fetched 0 / 0 values.
I1220 02:16:08.132213 539261 hugectr.cc:1832] Prediction execution time is 0 ms
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory: 0 hits, 0 missing!
[HUGECTR][02:16:08][INFO][RANK0]: Parameter server lookup of 0 / 0 embeddings took 184 us.
I1220 02:16:08.132245 539261 hugectr.cc:1438] model dlrm, instance dlrm, executing 1 requests
I1220 02:16:08.132256 539261 hugectr.cc:1512] request 0: id = "", correlation_id = 0, input_count = 3, requested_output_count = 1
I1220 02:16:08.132266 539261 hugectr.cc:1610]   input CATCOLUMN: datatype = INT64, shape = [1,26], byte_size = 208, buffer_count = 1
I1220 02:16:08.132273 539261 hugectr.cc:1626]   input ROWINDEX: datatype = INT32, shape = [1,27], byte_size = 108, buffer_count = 1
I1220 02:16:08.132278 539261 hugectr.cc:1640]   input DES: datatype = FP32, shape = [1,13], byte_size = 52, buffer_count = 1
I1220 02:16:08.132282 539261 hugectr.cc:1665]   requested_output OUTPUT0
I1220 02:16:08.132310 539261 hugectr.cc:1815] Processing request on device 0 for model dlrm
[HUGECTR][02:16:08][INFO][RANK0]: Looking up 0 embeddings (each with 128 values)...
I1220 02:16:08.132655 539261 hugectr.cc:1823] *Processing request completed!
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory backend. Table: dlrm#0. Fetched 0 / 0 values.
I1220 02:16:08.132684 539261 hugectr.cc:1832] Prediction execution time is 0 ms[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory: 0 hits, 0 missing!
[HUGECTR][02:16:08][INFO][RANK0]: Parameter server lookup of 0 / 0 embeddings took 199 us.
I1220 02:16:08.132756 539261 hugectr.cc:1438] model dlrm, instance dlrm, executing 1 requests
I1220 02:16:08.132767 539261 hugectr.cc:1512] request 0: id = "", correlation_id = 0, input_count = 3, requested_output_count = 1
I1220 02:16:08.132779 539261 hugectr.cc:1610]   input CATCOLUMN: datatype = INT64, shape = [1,26], byte_size = 208, buffer_count = 1
I1220 02:16:08.132808 539261 hugectr.cc:1626]   input ROWINDEX: datatype = INT32, shape = [1,27], byte_size = 108, buffer_count = 1
I1220 02:16:08.132817 539261 hugectr.cc:1640]   input DES: datatype = FP32, shape = [1,13], byte_size = 52, buffer_count = 1
I1220 02:16:08.132822 539261 hugectr.cc:1665]   requested_output OUTPUT0
I1220 02:16:08.132859 539261 hugectr.cc:1815] Processing request on device 0 for model dlrm
[HUGECTR][02:16:08][INFO][RANK0]: Looking up 0 embeddings (each with 128 values)...
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory backend. Table: dlrm#0. Fetched 0 / 0 values.
I1220 02:16:08.133222 539261 hugectr.cc:1823] *Processing request completed![HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory: 0 hits, 0 missing!
[HUGECTR][02:16:08][INFO][RANK0]: Parameter server lookup of 0 / 0 embeddings took 161 us.
I1220 02:16:08.133269 539261 hugectr.cc:1832] Prediction execution time is 0 ms
I1220 02:16:08.133317 539261 hugectr.cc:1438] model dlrm, instance dlrm, executing 1 requests
I1220 02:16:08.133330 539261 hugectr.cc:1512] request 0: id = "", correlation_id = 0, input_count = 3, requested_output_count = 1
I1220 02:16:08.133345 539261 hugectr.cc:1610]   input CATCOLUMN: datatype = INT64, shape = [1,26], byte_size = 208, buffer_count = 1
I1220 02:16:08.133354 539261 hugectr.cc:1626]   input ROWINDEX: datatype = INT32, shape = [1,27], byte_size = 108, buffer_count = 1
I1220 02:16:08.133364 539261 hugectr.cc:1640]   input DES: datatype = FP32, shape = [1,13], byte_size = 52, buffer_count = 1
I1220 02:16:08.133377 539261 hugectr.cc:1665]   requested_output OUTPUT0
I1220 02:16:08.133421 539261 hugectr.cc:1815] Processing request on device 0 for model dlrm
[HUGECTR][02:16:08][INFO][RANK0]: Looking up 0 embeddings (each with 128 values)...
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory backend. Table: dlrm#0. Fetched 0 / 0 values.
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory: 0 hits, 0 missing!
I1220 02:16:08.133783 539261 hugectr.cc:1823] *Processing request completed![HUGECTR][02:16:08][INFO][RANK0]: Parameter server lookup of 0 / 0 embeddings took 150 us.
I1220 02:16:08.133856 539261 hugectr.cc:1832] Prediction execution time is 0 ms
I1220 02:16:08.133894 539261 hugectr.cc:1438] model dlrm, instance dlrm, executing 1 requests
I1220 02:16:08.133903 539261 hugectr.cc:1512] request 0: id = "", correlation_id = 0, input_count = 3, requested_output_count = 1
I1220 02:16:08.133917 539261 hugectr.cc:1610]   input CATCOLUMN: datatype = INT64, shape = [1,26], byte_size = 208, buffer_count = 1
I1220 02:16:08.133925 539261 hugectr.cc:1626]   input ROWINDEX: datatype = INT32, shape = [1,27], byte_size = 108, buffer_count = 1
I1220 02:16:08.133931 539261 hugectr.cc:1640]   input DES: datatype = FP32, shape = [1,13], byte_size = 52, buffer_count = 1
I1220 02:16:08.133937 539261 hugectr.cc:1665]   requested_output OUTPUT0
I1220 02:16:08.133976 539261 hugectr.cc:1815] Processing request on device 0 for model dlrm
[HUGECTR][02:16:08][INFO][RANK0]: Looking up 0 embeddings (each with 128 values)...
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory backend. Table: dlrm#0. Fetched 0 / 0 values.
[HUGECTR][02:16:08][INFO][RANK0]: ParallelLocalMemory: 0 hits, 0 missing!
I1220 02:16:08.134336 539261 hugectr.cc:1823] *Processing request completed![HUGECTR][02:16:08][INFO][RANK0]: Parameter server lookup of 0 / 0 embeddings took 144 us.
I1220 02:16:08.134390 539261 hugectr.cc:1832] Prediction execution time is 0 ms
I1220 02:16:08.134424 539261 hugectr.cc:1438] model dlrm, instance dlrm, executing 1 requests
I1220 02:16:08.134434 539261 hugectr.cc:1512] request 0: id = "", correlation_id = 0, input_count = 3, requested_output_count = 1
I1220 02:16:08.134444 539261 hugectr.cc:1610]   input CATCOLUMN: datatype = INT64, shape = [1,26], byte_size = 208, buffer_count = 1
I1220 02:16:08.134451 539261 hugectr.cc:1626]   input ROWINDEX: datatype = INT32, shape = [1,27], byte_size = 108, buffer_count = 1
I1220 02:16:08.134455 539261 hugectr.cc:1640]   input DES: datatype = FP32, shape = [1,13], byte_size = 52, buffer_count = 1
I1220 02:16:08.134458 539261 hugectr.cc:1665]   requested_output OUTPUT0
I1220 02:16:08.134488 539261 hugectr.cc:1815] Processing request on device 0 for model dlrm
terminate called after throwing an instance of 'std::system_error'
what():  Resource temporarily unavailable
Signal (6) received.
0# 0x000056510D01E8A9 in tritonserver
1# 0x00007F263065A210 in /usr/lib/x86_64-linux-gnu/libc.so.6
2# gsignal in /usr/lib/x86_64-linux-gnu/libc.so.6
3# abort in /usr/lib/x86_64-linux-gnu/libc.so.6
4# 0x00007F2630A10911 in /usr/lib/x86_64-linux-gnu/libstdc++.so.6
5# 0x00007F2630A1C38C in /usr/lib/x86_64-linux-gnu/libstdc++.so.6
6# 0x00007F2630A1C3F7 in /usr/lib/x86_64-linux-gnu/libstdc++.so.6
7# 0x00007F2630A1C6A9 in /usr/lib/x86_64-linux-gnu/libstdc++.so.6
8# std::__throw_system_error(int) in /usr/lib/x86_64-linux-gnu/libstdc++.so.6
9# 0x00007F2630A490BD in /usr/lib/x86_64-linux-gnu/libstdc++.so.6
10# HugeCTR::embedding_cache::look_up(void const*, std::vector<unsigned long, std::allocator > const&, float*, HugeCTR::MemoryBlock*, std::vector<CUstream_st*, std::allocator<CUstream_st*> > const&, float) in /usr/local/hugectr/lib/libhugectr_inference.so
11# HugeCTR::InferenceSession::predict(float*, void*, int*, float*, int) in /usr/local/hugectr/lib/libhugectr_inference.so
12# 0x00007F261805B04A in /usr/local/hugectr/backends/hugectr/libtriton_hugectr.so
13# TRITONBACKEND_ModelInstanceExecute in /usr/local/hugectr/backends/hugectr/libtriton_hugectr.so
14# 0x00007F26311E683A in /opt/tritonserver/bin/../lib/libtritonserver.so
15# 0x00007F26311E704D in /opt/tritonserver/bin/../lib/libtritonserver.so
16# 0x00007F263109B801 in /opt/tritonserver/bin/../lib/libtritonserver.so
17# 0x00007F26311E0DC7 in /opt/tritonserver/bin/../lib/libtritonserver.so
18# 0x00007F2630A48DE4 in /usr/lib/x86_64-linux-gnu/libstdc++.so.6
19# 0x00007F2630EC6609 in /usr/lib/x86_64-linux-gnu/libpthread.so.0
20# clone in /usr/lib/x86_64-linux-gnu/libc.so.6
Aborted (core dumped)
但没有生成core文件
root@13609cdec7a0:/opt/tritonserver# ll
total 423640
drwxr-xr-x 1 root          root               4096 Nov 29 08:07  ./
drwxr-xr-x 1 root          root               4096 Oct 21 18:44  ../
-rw-r--r-- 1 root          root                641 Nov  4 19:20 '=1.21.6'
-rw-rw-r-- 1 triton-server triton-server      1485 Oct 21 17:20  LICENSE
-rw-rw-r-- 1 triton-server triton-server   3012640 Oct 21 17:20  NVIDIA_Deep_Learning_Container_License.pdf
-rw-rw-r-- 1 triton-server triton-server         7 Oct 21 17:20  TRITON_VERSION
drwxr-xr-x 1 triton-server triton-server      4096 Nov  4 19:33  backends/
drwxr-xr-x 2 triton-server triton-server      4096 Oct 21 18:45  bin/
drwxr-xr-x 1 triton-server triton-server      4096 Oct 21 18:45  include/
drwxr-xr-x 2 triton-server triton-server      4096 Oct 21 18:45  lib/
-rw------- 1 root          root          430725428 Dec 16 06:00  nohup.out
-rwxrwxr-x 1 triton-server triton-server      4090 Oct 21 17:20  nvidia_entrypoint.sh*
drwxr-xr-x 3 triton-server triton-server      4096 Oct 21 18:48  repoagents/
drwxr-xr-x 2 triton-server triton-server      4096 Oct 21 18:45  third-party-src/

Server for some reason returned an empty response. Can you share server side logs with --log-verbose=1? and share more detailed log when launch tritonserver.

This issue will be fixed  in the Feb 2022 release.

Fixed in V3.4


chore: reintroduce option to access window APIs on command fn

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

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

[ ] Yes. Issue #___
[ ] No

The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
[ ] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

since hooks::InvokeMessage has a window inside of it, isn't it already passed? message.window() should give a clone of the window

the command takes each payload key as argument, not the entire message object.


[Tabs] Migrate Tabs to emotion

[x] I have followed (at least) the PR section of the contributing guide.

One chunk of #24405

#24405

@material-ui/core => 100% migrated 🙏

@material-ui/core => 100% migrated 🙏

Looks like we both waited to see this too long now 😄


website generating strange anchor tags

https://pulsar.incubator.apache.org/docs/latest/getting-started/ConceptsAndArchitecture/#Topics-kxiw2
The website documentation is adding a random sequence to the end of the anchor tag which makes it difficult and fragile to link to this documentation from external sources.

This has been fixed with the new website.


JSONLD examples with working scripts.

Contains:

script comparing last commits on nested files
concatenation of all graphs

LD repository:

individual_ld.jsonfiles as identified by their id.
Frame.json file to outline format and to be used as a template for framing.
Schema.json
Version.json (for all the files)
Graph.json to group all entries, and define their origin.

To do:

convert back into identical files as in main repository
network/dependancy graph for all files in both MIP and CV tables using linked IDs

@durack1 I am happy for this to finally be over!
I have created an output MIP_organisations.json which follows our file structure and combines a simplified consortium and institution list.
Is there anything else needed before merging?

Couple of nits, it looks like the wrong institution is listed here
https://github.com/PCMDI/mip-cmor-tables/blob/e22640c2399f367840eb970b70658b3b0f248644/JSONLD/scripts/output/MIP_organisations.json#L43C1-L48C7
"Beijing Biocytogen (China)" -> "Beijing Climate Center" https://ror.org/03kb2jw75

Did we want to ascii (0-127 char) wash these, they will not likely play nice with HTML
https://github.com/PCMDI/mip-cmor-tables/blob/jsonld/JSONLD/scripts/output/MIP_organisations.json#L85-L90
"Centre National d'Études Spatiales" -> "Centre National dEtudes Spatiales"

https://github.com/PCMDI/mip-cmor-tables/blob/jsonld/JSONLD/scripts/output/MIP_organisations.json#L133-L138
"Freie Universität Berlin" -> "Freie Universitat Berlin",

https://github.com/PCMDI/mip-cmor-tables/blob/jsonld/JSONLD/scripts/output/MIP_organisations.json#L265-L270
Also NCAS "name": "Uks",

https://github.com/PCMDI/mip-cmor-tables/blob/jsonld/JSONLD/scripts/output/MIP_organisations.json#L307-L312
"name": "Lawrence Livermore National Laboratory", -> "name": "Program for Climate Model Diagnosis and Intercomparison",
There is already an LLNL defined, with same ROR

Couple of nits, it looks like the wrong institution is listed here
https://github.com/PCMDI/mip-cmor-tables/blob/e22640c2399f367840eb970b70658b3b0f248644/JSONLD/scripts/output/MIP_organisations.json#L43C1-L48C7
"Beijing Biocytogen (China)" -> "Beijing Climate Center" https://ror.org/03kb2jw75

I would like to fix this using a acton one the two have been merged. We should delete the file, and flag an issue.

https://github.com/PCMDI/mip-cmor-tables/blob/jsonld/JSONLD/scripts/output/MIP_organisations.json#L331-L336
"name": "Remote Sensing Systems (United States)", -> "name": "Remote Sensing Systems"

Hoping that we could flag issues on individual files.

Hoping that we could flag issues on individual files not as part of a pull request for something else.

Sure, maybe we can catch these afterward - with me tagged to do it - as a way of figuring out how specific little issues can be done by someone, not you.. I see 4 discrete issues, each with a specific institution, with each that could be captured as a single issue for me to process

Clarified the source JSON files for user interaction.
migrated JSONLD-ification to processing and checks scripts.
Added more metadata to JSONLD
removed it for simple view located in https://github.com/PCMDI/mip-cmor-tables/tree/jsonld/JSONLD/scripts/output folder

@wolfiex just found another nit, I think I couldn't see this before as GitHub only loads the first 3000 files, and this must've been 3001 or more.
The question, in the file name solaris-heppa.json vs the file contents ..consortia/solaris_heppa - so the underscore vs the hyphen. No idea if this is an issue or not, but seemed to be an inconsistency
https://github.com/PCMDI/mip-cmor-tables/blob/1343761411e9ae21b309c4789c57a70a223c43f8/JSONLD/organisations/consortia/solaris-heppa.json#L1-L8

Looks like the JSONLD/miptables/variables/ subdirectories both contain duplicate *.bak files

Latest commits added to re-generate the main repository files - see commits above. These are identical in the diff, just with different 'version metadata' .
The running of these scripts still needs to be incorporated into the action, but happy to merge in the meantime.
@durack1 @matthew-mizielinski

Resolved changes suggested by @durack1

I'm happy to press that big green button if you want me to! This is great, almost there..


Change contrast of initial open ai popup

Imo there is not enough contrast for good readability. Check this tool for good contrast ratio.

Ah, forgot about dark mode. Yes, accessibility facts like contrast are not up for debate.

Ah lol now it makes sense. Was wondering why it looks like this, if it was a design decision or bug.

Improved contrast in both light and dark mode (60e411ce)


[ncnn] Add new port

[ncnn] Add new port

Provide ncnn implementation

Convert CONTROL to vcpkg.json.

Update versions/baseline.json.

Address reviewer comments by using vcpkg_from_github.

Implement recommended changes from MR review.

Update x-add-version.

ncnn uses a proprietary license is BSD

Update versions file.

format portfile.cmake

x-add-version

apply suggestion

x-add-version

Co-authored-by<EMAIL_ADDRESS>

@holylong, Thanks for your pr, Looks the ci tested failed,
this is error log, please taeke a look:
/debug/share should not exist. Please reorganize any important files, then use
    file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/debug/share")
The following EXEs were found in /bin or /debug/bin. EXEs are not valid distribution targets.

    D:/packages/ncnn_x86-windows/bin/darknet2ncnn.exe
    D:/packages/ncnn_x86-windows/bin/mxnet2ncnn.exe
    D:/packages/ncnn_x86-windows/bin/ncnn2int8.exe
    D:/packages/ncnn_x86-windows/bin/ncnn2mem.exe
    D:/packages/ncnn_x86-windows/bin/ncnn2table.exe
    D:/packages/ncnn_x86-windows/bin/ncnnmerge.exe
    D:/packages/ncnn_x86-windows/bin/ncnnoptimize.exe

The following EXEs were found in /bin or /debug/bin. EXEs are not valid distribution targets.

    D:/packages/ncnn_x86-windows/debug/bin/darknet2ncnn.exe
    D:/packages/ncnn_x86-windows/debug/bin/mxnet2ncnn.exe
    D:/packages/ncnn_x86-windows/debug/bin/ncnn2int8.exe
    D:/packages/ncnn_x86-windows/debug/bin/ncnn2mem.exe
    D:/packages/ncnn_x86-windows/debug/bin/ncnn2table.exe
    D:/packages/ncnn_x86-windows/debug/bin/ncnnmerge.exe
    D:/packages/ncnn_x86-windows/debug/bin/ncnnoptimize.exe

Please don't git push this branch, I'm fixing bugs

ok, I add some ncnn cmake options to avoid exe generation

This is error log, please take a look:
C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Tools\MSVC\14.32.31326\include\emmintrin.h(20): fatal error C1189: #error:  This header is specific to X86, X64 and ARM64EC targets
[11/158] C:\PROGRA~1\MICROS~1\2022\ENTERP~1\VC\Tools\MSVC\1432~1.313\bin\Hostx64\arm\cl.exe   /TP -DNOMINMAX -D_CRT_SECURE_NO_DEPRECATE -D_SCL_SECURE_NO_WARNINGS -ID:\buildtrees\ncnn\src\20220420-dbbb03d16a.clean\src\layer\x86 -ID:\buildtrees\ncnn\src\20220420-dbbb03d16a.clean\src -ID:\buildtrees\ncnn\arm-uwp-dbg\src -ID:\buildtrees\ncnn\src\20220420-dbbb03d16a.clean\src\layer /DWIN32 /D_WINDOWS /D_UNICODE /DUNICODE /DWINAPI_FAMILY=WINAPI_FAMILY_APP /D__WRL_NO_DEFAULT_LIB__ /nologo /Z7 /MP /GS /Gd /Gm- /W3 /WX- /Zc:wchar_t /Zc:inline /Zc:forScope /fp:precise /Oy- /EHsc  /utf-8  /D_DEBUG /Od /RTC1 /MDd  -MDd -DNCNN_STATIC_DEFINE -fno-rtti /arch:SSE2 /D__SSE2__ -openmp /showIncludes /Fosrc\CMakeFiles\ncnn.dir\layer\x86\bias_x86.cpp.obj /Fdsrc\CMakeFiles\ncnn.dir\ncnn.pdb /FS -c D:\buildtrees\ncnn\src\20220420-dbbb03d16a.clean\src\layer\x86\bias_x86.cpp
FAILED: src/CMakeFiles/ncnn.dir/layer/x86/bias_x86.cpp.obj
C:\PROGRA~1\MICROS~1\2022\ENTERP~1\VC\Tools\MSVC\1432~1.313\bin\Hostx64\arm\cl.exe   /TP -DNOMINMAX -D_CRT_SECURE_NO_DEPRECATE -D_SCL_SECURE_NO_WARNINGS -ID:\buildtrees\ncnn\src\20220420-dbbb03d16a.clean\src\layer\x86 -ID:\buildtrees\ncnn\src\20220420-dbbb03d16a.clean\src -ID:\buildtrees\ncnn\arm-uwp-dbg\src -ID:\buildtrees\ncnn\src\20220420-dbbb03d16a.clean\src\layer /DWIN32 /D_WINDOWS /D_UNICODE /DUNICODE /DWINAPI_FAMILY=WINAPI_FAMILY_APP /D__WRL_NO_DEFAULT_LIB__ /nologo /Z7 /MP /GS /Gd /Gm- /W3 /WX- /Zc:wchar_t /Zc:inline /Zc:forScope /fp:precise /Oy- /EHsc  /utf-8  /D_DEBUG /Od /RTC1 /MDd  -MDd -DNCNN_STATIC_DEFINE -fno-rtti /arch:SSE2 /D__SSE2__ -openmp /showIncludes /Fosrc\CMakeFiles\ncnn.dir\layer\x86\bias_x86.cpp.obj /Fdsrc\CMakeFiles\ncnn.dir\ncnn.pdb /FS -c D:\buildtrees\ncnn\src\20220420-dbbb03d16a.clean\src\layer\x86\bias_x86.cpp
cl : Command line warning D9002 : ignoring unknown option '-fno-rtti'
cl : Command line warning D9002 : ignoring unknown option '/arch:SSE2'
C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Tools\MSVC\14.32.31326\include\emmintrin.h(20): fatal error C1189: #error:  This header is specific to X86, X64 and ARM64EC targets
[12/158] C:\PROGRA~1\MICROS~1\2022\ENTERP~1\VC\Tools\MSVC\1432~1.313\bin\Hostx64\arm\cl.exe   /TP -DNOMINMAX -D_CRT_SECURE_NO_DEPRECATE -D_SCL_SECURE_NO_WARNINGS -ID:\buildtrees\ncnn\src\20220420-dbbb03d16a.clean\src\layer\x86 -ID:\buildtrees\ncnn\src\20220420-dbbb03d16a.clean\src -ID:\buildtrees\ncnn\arm-uwp-dbg\src -ID:\buildtrees\ncnn\src\20220420-dbbb03d16a.clean\src\layer /DWIN32 /D_WINDOWS /D_UNICODE /DUNICODE /DWINAPI_FAMILY=WINAPI_FAMILY_APP /D__WRL_NO_DEFAULT_LIB__ /nologo /Z7 /MP /GS /Gd /Gm- /W3 /WX- /Zc:wchar_t /Zc:inline /Zc:forScope /fp:precise /Oy- /EHsc  /utf-8  /D_DEBUG /Od /RTC1 /MDd  -MDd -DNCNN_STATIC_DEFINE -fno-rtti /arch:SSE2 /D__SSE2__ -openmp /showIncludes /Fosrc\CMakeFiles\ncnn.dir\mat.cpp.obj /Fdsrc\CMakeFiles\ncnn.dir\ncnn.pdb /FS -c D:\buildtrees\ncnn\src\20220420-dbbb03d16a.clean\src\mat.cpp
FAILED: src/CMakeFiles/ncnn.dir/mat.cpp.obj

You can modify it normally

I'm not have toolchain to compile for arm platform
error: in triplet arm64-windows: Unable to find a valid toolchain for requested target architecture arm64.
The selected Visual Studio instance is at: D:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise
The available toolchain combinations are: x86, amd64, x86_amd64, amd64_x86
See https://github.com/microsoft/vcpkg/blob/master/docs/users/triplets.md#VCPKG_VISUAL_STUDIO_PATH for more information.
note: updating vcpkg by rerunning bootstrap-vcpkg may resolve this failure.

I change vcpkg.json like mnn not sopport mobile and arm
"supports": "!(arm | uwp | ios | android)",

I change vcpkg.json like mnn not sopport mobile and arm
"supports": "!(arm | uwp | ios | android)",

ncnn support these platforms natively?

I change vcpkg.json like mnn not sopport mobile and arm
"supports": "!(arm | uwp | ios | android)",

ncnn support these platforms natively?
ncnn is a inference library like mnn, The method of compiling the arm version with visual studio not supported now.

But I add cmake options to avoid generate exes,there has no exes to clean or copy.
---- Replied Message ----
| From | @.> |
| Date | 06/18/2022 06:02 |
| To | @.> |
| Cc |<EMAIL_ADDRESS>|
| Subject | Re: [microsoft/vcpkg] [ncnn] Add new port (PR #25276) |
@Thomas1664 commented on this pull request.
In ports/ncnn/portfile.cmake:

+file(GLOB EXES "${CURRENT_PACKAGES_DIR}/bin/*.exe")
+if(EXES)

file(COPY ${EXES} DESTINATION "${CURRENT_PACKAGES_DIR}/tools/shapelib")
file(COPY ${EXES} DESTINATION "${CURRENT_PACKAGES_DIR}/tools/shapelib")
file(REMOVE ${EXES})
+endif()

+file(GLOB DEBUG_EXES "${CURRENT_PACKAGES_DIR}/debug/bin/*.exe")
+if(DEBUG_EXES)

file(REMOVE ${DEBUG_EXES})
+endif()

@holylong<tool_names> was meant as a placeholder for the names of the executables. Please replace it with their file names (without file extension. Furthermore, I recommend to read the docs about vcpkg_copy_tools().
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

But I add cmake options to avoid generate exes,there has no exes to clean or copy.
---- Replied Message ---- | From | @.> | | Date | 06/18/2022 06:02 | | To | @.> | | Cc |<EMAIL_ADDRESS>| | Subject | Re: [microsoft/vcpkg] [ncnn] Add new port (PR #25276) | @Thomas1664 commented on this pull request. In ports/ncnn/portfile.cmake:
+file(GLOB EXES "${CURRENT_PACKAGES_DIR}/bin/.exe")
+if(EXES) + file(COPY ${EXES} DESTINATION "${CURRENT_PACKAGES_DIR}/tools/shapelib") + file(COPY ${EXES} DESTINATION "${CURRENT_PACKAGES_DIR}/tools/shapelib") + file(REMOVE ${EXES}) +endif() + +file(GLOB DEBUG_EXES "${CURRENT_PACKAGES_DIR}/debug/bin/.exe") +if(DEBUG_EXES) + file(REMOVE ${DEBUG_EXES}) +endif() @holylong<tool_names> was meant as a placeholder for the names of the executables. Please replace it with their file names (without file extension. Furthermore, I recommend to read the docs about vcpkg_copy_tools(). — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

But I add cmake options to avoid generate exes,there has no exes to clean or copy. so function vcpkg_copy_tools() is it a little superfluous

Ping @holylong for response

Duplicate of #26557


fix: Fix the bug that DeviceItem component style attribute does not t…

…ake effect
Owner

[x] Nickname(Tuya):  南树
[x] Intra-group review（组内审阅人员）：OS一组

Description
修复 DeviceItem 组件 style 属性 背景色不生效
Package
Please select the relevant package.

[ ] tuya-panel-animation-sdk
[ ] tuya-panel-api
[ ] tuya-panel-electrician-sdk
[ ] tuya-panel-ipc-sdk
[ ] tuya-panel-lamp-sdk
[ ] tuya-panel-lock-sdk
[ ] tuya-panel-robot-sdk
[ ] tuya-panel-outdoor-sdk
[ ] tuya-panel-remote-sdk

Type of change
Please select the relevant option.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Self CheckList
Please check all options.

[x] Intra-group review
[x] ESLint lint
[x] Example Run
[x] Jest Run
[x] No Grammer Error
[x] No Spell Error
[x] No Merge Conflict

Codecov Report

Merging #196 (d5ca601) into main (984197d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #196   +/-   ##
=======================================
  Coverage   63.12%   63.12%
=======================================
  Files         368      368
  Lines       14096    14098    +2
  Branches     3265     3265
=======================================
+ Hits         8898     8900    +2
  Misses       5174     5174
  Partials       24       24

Impacted Files
Coverage Δ

...el-remote-sdk/src/components/device-item/index.tsx
98.11% <100.00%> (+0.07%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


[Feature Request] API

Is your feature request related to a problem? Please describe.
I'd like to manage and view the library and the downloader in automation and through other frontends
Describe the solution you'd like
A Restful API with CR(U?)D for the library management and for the downloader
Describe a small use-case for this feature request
It would be nice to integrate the library into dashboards and library management systems like Readarr (Lidarr for for printed media).
Additional context
This would somewhat force the MVC architecture which might prove useful for unit testing and code sanity later as well.

"APIs for library management" is quite broad. Can you give a few examples of the operations that the APIs should perform? We have a set of (undocumented) APIs already so it might not be too hard to do.

So I'm thinking of Mango as the downloader, viewer and library system right now. It would be nice for a system like Readarr (see Sonarr) to be able to see which publications are present, and be able to trigger the download when new releases are available.
Other than that, it would be nice to have an API to be able to integrate this into other manga reader applications, so that you can browse your library and download comics from there to your device.
Also if I could make myself a cronjob to check the mango API, ge anything downloaded between now and the last time a crawl was done, and then make it do whatever I want (like convert to epub and add to calibre-web), it would be great.
I could also make myself an application that would independently of mango check my RSS and have mango download the files for me through the API

Starting from v0.17.0, the API documentation will be available at /api. At this moment the endpoints do not comply with the RESTful conventions, so I will be making some big changes soon.
I am closing this now, and please feel free to open a new issue if you want to request a new endpoint.


Included additional install directions for Windows

Included Npcap install instructions in order to build and use the project on Windows.

Thanks for the contribution!
I wonder if there's a simpler/more automated way to make it work.


Support Tailwind CSS v4

Tailwind v4 moves configuration into CSS instead of tailwind.config.js. https://tailwindcss.com/blog/tailwindcss-v4-alpha
For example in v4 to specify a custom font, you'd just do this and then you can use .font-display within your HTML. (example from their blog post)
/* main.css */
@import "tailwindcss";

@theme {
  --font-family-display: "Satoshi", "sans-serif";
}

Workaround
Currently the workaround to use Tailwind CSS v4 with Nuxt Fonts is simply to specify using font-family in the CSS so it's loaded in.
@import "tailwindcss";

@theme {
  --font-family-marker: "Permanent Marker", cursive;
}

.nuxt-fonts-permanent-marker {
  font-family: "Permanent Marker", cursive;
}

This is already possible. v4 uses CSS variables to determine font families under the hood, so all you need is to enable experimental.processCSSVariables.
A demo here: https://codesandbox.io/p/github/qwerzl/fonts/test/tailwind-v4


New Biome Request

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
The mod "Ultra Amplified Dimension" adds a very cool swamp like biome:

Overall that mods world generation is not good to use but that biome looks really great and it would be fantastic if it could be integrated as a "normal" biome into a useful world generation such like the one this mod provides.
What that biome makes unique is, that the water surface is on different y-levels.
Just like natural hanging gardens:

I generally only add biomes to fill in gaps in the climate model that TerraForged uses. It's otherwise intended for other mods to provide more specific biomes

ok


Incompatibility with connectivity plugin because of 'Reachability dependency/file' (iOS platform)

Steps to reproduce
in pubspec.yaml decalre poth flutter_pusher and connectivity plugins.
What did you expect to happen?
iOS build will fail. (Xcode build will somehow pass, but app won't be installed)
What actually happened? Submit stack traces or anything that you think would help.
The problem is some file/dependency named Reachability.
Another pusher plugin suffers the same problem: https://github.com/HomeXLabs/pusher-websocket-flutter/issues/11
...

Any workaround or solution is welcomed. So far I didn't find a way to deal with this.

Any workaround or solution is welcomed. So far I didn't find a way to deal with this.

Yeah this is a common issue, the connectivity package is incompatible with because of the Reachability library for. The maintainer of the Reachability library needs to update their library to resolve this issue.
The workaround I found was to just use data_connection_checker instead of connectivity, they both work the same way

Yeah this is a common issue, the connectivity package is incompatible with because of the Reachability library for. The maintainer of the Reachability library needs to update their library to resolve this issue.
The workaround I found was to just use data_connection_checker instead of connectivity, they both work the same way

Could you please explain how to implement your workaround ?

Could you please explain how to implement your workaround ?

The package is pretty straightforward, simply use data_connection_checker package instead of connectivity package. They both work the same however data_connection_checker is compatible with this package and other pusher clients

Any solution for this issue ?

This is PITA! Have anyone found better ways to solve this?


Esp32 support ?

Is the esp32 supported ?

Sorry, ESP32 doesn't have a USB port and doesn't have microPython's ability to save execution.Maybe support can be implemented on ESP32S2-Arduino.


Insert Checkin Data Into Google Sheets Database

This may be a very small task.

I did some testing on this... it seems as if this may actually just be DONE. If you check out the spreadsheet, you will see that the console app worked.


Can't install a font

Am I doing something wrong here?:
~|⇒ brew tap caskroom/fonts
Warning: Already tapped!
~|⇒ brew update
Already up-to-date.
~|⇒ brew cask install font-charis-sil
Error: No available Cask for font-charis-sil
Error: nothing to install

I think this cask exists, but maybe by a different name?

I am quite sorry for the late reply.
I am unable to reproduce, but the fact that font-charis-sil is not found suggests that the issue may stem from your local tap. Could you try untapping and re-tapping caskroom/fonts?

FWIW, I'm having a similar issue. I tried untapping and re-tapping but still failed.

If this is still an issue, please open a new one, and follow the procedure.


Re-loading the assembly re-creates the types

Describe the bug
Reloading the assembly creates a new set of types which are treated as different. This is different from the case where the framework loading is used.
To Reproduce
Steps to reproduce the behavior:

Using this version of the library '0.3.0'
Add breakpoint inside ``
Run tests for SomeAssembly.Facade.IntegrationTests project from
https://github.com/godrose/netcore-load/tree/master

Expected behavior
MultipleLoadTests is green
Actual behavior
MultipleLoadTests is red
Additional context
.NET Core 3.0, VS 2019, Test Explorer

This is the intended behavior of creating multiple loaders or using the .Reload() method. If you want types to be the same, reuse the same instance of PluginLoader and don't call .Reload() on it.

@natemcmaster I don't think I invoke this method...

creating multiple loaders or using the .Reload() method

And what about the other one?

@natemcmaster I do create multiple loaders but here I'm just following the example. Maybe I got it all wrong...
//for each dll
var loader = PluginLoader.CreateFromAssemblyFile(
                    pluginDll,
                    sharedTypes: new [] { typeof(IPlugin) });
                loaders.Add(loader);

Which example are you looking at? I can update samples and docs if they are unclear.
Otherwise, I don't think there is any action item here. It appears to me that PluginLoader is working the way it's intended, i.e. each instance of PluginLoader has a unique, isolated execution context.


RuleHandler.js will soon be missing from the latest template in altinn studio

Description of the bug
RuleHandler.js will soon be missing from the latest template in altinn studio.
I think this should be pre-emptively fixed by changing app-frontend-react to not crash when {org}/{app}/api/rulehandler/{id} responds with 404.
The nuget packages in app-lib-dotnet should probably also return 204 No Content instead of 404 Not Found so that it doesn't trash the browser error logs.

I agree. This should be an easy fix, and we're getting real close to the next release anyway. Adding this to the current sprint, and letting @bjosttveit and @Magnusrm know so they can chase me to it. :wink:

I’ll make a PR for the status code change in backend. Missing this file is not an error anymore.

Turns out it's not as critical as we first thought. The file exists in the template, but was gone when an app was copied from altinn2convert.
Link to slack discussion:

https://altinn.slack.com/archives/C02EJ9HKQA3/p1674818338937379

Seems to me like the only thing required here is to make the backend reply with 204 No Content instead of 404 Not Found when RuleHandler.js is missing. Same solution seems to be used with RuleConfiguration.json that currently isn't in the template. (though it returns 200 Ok with an empty response). Not sure what status code makes most sense, but in https://github.com/Altinn/app-lib-dotnet/pull/174 I decided to go with 204 No Content for both files.

With https://github.com/Altinn/app-lib-dotnet/pull/174 merged (and soon released in the Altinn.App.Api nuget package), the only task left on this one is for @Altinn/team-altinnstudio to decide when to actually remove the file. Maybe this issue should be moved to altinn-studio?

Right. I believe removing the file as a default is an issue that should be handled by the apps and studio teams in unison (it's an apps thing, but needs studio support before we remove it). Sadly, expressions still don't support calculations, and there is not a 100% overlap in supported functionality, so we have no pressing plans to remove these files from our template AFAIK.
Btw, the template application is located in this repo: https://github.com/Altinn/app-template-dotnet


Added announcement route to API

Added announcement get route in server.ts
Added configs/announcement.json file that announcements are read from (use configs/announcement.example.json for sample formatting)
Added announcements interface in types.ts

Looks good to me!


book: Clean up the C++ section some more

Trying to set expectations for what we can do with C++ a little better.
r? @emilio

@bors-servo r=emilio
Thanks for the review -- good call on linking that issue :)

:pushpin: Commit c3d9136 has been approved by emilio

:hourglass: Testing commit c3d91364b3be16f29610ff2a3660ccae8ebca21d with merge e78394959f40193a4bf9e5718dd4f779cb57437a...

:sunny: Test successful - status-travis
Approved by: emilio
Pushing e78394959f40193a4bf9e5718dd4f779cb57437a to master...


Implement CupertinoMenuAnchor and CupertinoMenuItem

This PR implements CupertinoMenuAnchor, CupertinoMenuItem, CupertinoLargeMenuDivider using MenuAnchor as a base.
Resolves https://github.com/flutter/flutter/issues/60298, https://github.com/notDmDrl/pull_down_button/issues/26, https://github.com/flutter/flutter/pull/137936
Tests modeled after MenuAnchor have been added to cover the API surface... perhaps too many tests. I split CupertinoMenuItem tests from CupertinoMenuAnchor tests to improve performance.
Four examples have been added with tests

Simple example demonstrating CupertinoMenuItem styling
Menu button example based on material/menu_anchor.0.dart
Context menu example based on simplified material/menu_anchor.1.dart
Nested example that replicates the iOS history stack. This one is more complex and may need to be removed.

Before implementing tests for the modifications made to MenuAnchor (a MenuAnchor.withOverlayBuilder constructor was added), I wanted to get some feedback. I initially made MenuAnchorState public and made the overlayBuilder a protected method, but found that adding a separate constructor required fewer changes and felt simpler overall. The tradeoff is that users would need to know how to properly set up semantics, focus, and actions if they were to use this constructor.
Also, I'm curious whether it would be valuable for me to fullfill this PR: https://github.com/flutter/flutter/issues/135025. I attempted to build a generic Panel widget that would encompass functionality and animations without providing structure, but I decided to stick to a wrapper until I heard feedback.
Some features that were going to be included in a future PR made it into this PR because they were either necessary to fulfill a core functionality of the menu (_PanRegion, which is modeled after _TapRegion, and _CupertinoMenuDivider), or they were so simple that it'd be more work to leave them out (CupertinoLargeMenuDivider). In the case of the former, all classes are private, fully documented, and have been tested in the context of their use in the menu anchor. With regard to CupertinoLargeMenuDivider, it has tests, documentation, and a public API.
A few loose ends:

LocalizedShortcutLabeler was made public rather than implementing a Cupertino version, seeing as it appears to have already accounted for iOS and MacOS and has public documentation/tests
Should accelerators be included?
Should the ability to customize the menu surface be removed?
To account for features like a sticky header, the menu's default scrollable is a CustomScrollView that is shrink wrapped. This introduces a performance penalty by default, although shrink wrapping can be disabled. Is there a better alternative?

Last, CupertinoMenuItem, CupertinoMenuAnchor, MenuItemButtons, and MenuAnchor all interop and can be swapped out pretty easily.
https://github.com/flutter/flutter/assets/59215665/535295d3-e7dd-46ad-8682-a3cba79d2db9
Pre-launch Checklist

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

Missing MenuAnchor tests

[x] All existing and new tests are passing.

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

@Piinks @MitchellGoodwin Sorry in advance if you guys already saw this, but I forgot to tag you both

Hello @davidhicks980, sorry for the delay on this one. It looks like this change is failing some checks. Could you take a look at those and fix the issues? Thanks.

@goderbauer Sure thing! I'll take a look later this week.

Okay, assuming I correctly fixed the last four linux warnings, this should be good for a prelim look!
There is a lot of weirdness in this PR, particularly with regard to color blending. I tried to explain most of it, but please let me know if anything doesn't make sense.
Also, as I've already mentioned, the _PanRegion and related classes were added to match native behavior, but it may make sense to split this feature off into a separate file. I wanted to get feedback on it before doing so, as the implementation I've written is primitive (e.g. there is no way to exclude gestures started from one _PanRegion from all of the other _PanRegions). I'd also be curious to hear whether we should incorporate menu text theming into the CupertinoTheme class... but I was afraid to add too much complexity with this commit.

Absolutely incredible.  Need this yesterday :) :) 👍

why is this a draft, didn't all the checks pass?

any updates on this RP?

any update ?

Whoops, I overlooked these comments. Because there is a circular dependency on the Material library, we decided to split the material MenuAnchor into a RawMenuAnchor base that lives in the Widgets library. In order to do that, I wanted to refactor the focus behavior to make it simpler to split. That PR is here: https://github.com/flutter/flutter/pull/150950. I've also improved the CupertinoPopupSurface (see https://github.com/flutter/flutter/pull/151430) to be used with this menu so it doesn't have to rely on a bespoke surface implementation. Once underlying issues are worked out, we'll commit a RawMenuAnchor, which can then be used by this menu. So for right now, I suggest using https://github.com/notDmDrl/pull_down_button for a full-featured, easily-customizable menu (recommended), or you can just copy the code from this PR and import it into your project.

Looking forward to subsequent updates

Please try and keep comments asking for status updates and any other comments not directly related to the implementation on the original issue. The PRs themselves should be mostly technical discussion to make it easier to review and possible investigations later after this is merged.

(An update from PR triage): There is work outside of this PR ongoing to unblock it.

(triage): This is still blocked on https://github.com/flutter/flutter/pull/158255.


example collage not working for me.

I'm trying to use the example collage, but it just isn't working. I'm getting an error, I'm guessing on the sources.
this is my code: https://pastebin.com/WdfwSmY8
I'm using some local files on my server, which are the last 2 source images.
the error I'm getting is:
[15-Nov-2017 17:24:33 UTC] PHP Fatal error:  Uncaught Filestack\FilestackException: [400]: conv20 provider error: Error fetching file: 401 (idx 0)
thrown in /home/stagmm/public_html/tools/filestack/filestack/mixins/CommonMixin.php on line 314

hi @johncarlson21
A 401 errors means there is an authorization error.  Looks like your sources http://staging2.magicmurals.com/tools/filestack/tests/testfiles/Ultimo_TopNav041816.jpg and
http://staging2.magicmurals.com/tools/filestack/tests/testfiles/Ultimo_ColorScheme-041816L.jpg
requires an auth username and password.
In order to use collage your sources have to be accessible by the code.

Yeah.. I first had the auth turned on, but then I disabled it and it wouldn't work.
I'm going to give it a shot on another server and see how it goes.
Thanks.

Sure, please let us know how it goes.
p.s. Also make the the destination folder exists and your code has permission to write to it.  The created collage file will be downloaded to that destination when you make this call:
$collage_filelink = $client->collage($sources, 800, 600);

$destination = __DIR__ . '/tests/testfiles/collage-test.png';
$result = $collage_filelink->download($destination);

If you don't want it downloaded, you should be able to just grab the URL of the file generated on the Filestack server by calling the url() function on the result Filelink object.
$collage_filelink = $client->collage($sources, 800, 600);
$new_collage_image_url = $collage_filelink->url();

Well, I got it to work on another server, but I was getting a 400 error,
which referred to security.
I didn't have security turned on, so I turned it on and then used the
security and it worked.
On Thu, Nov 16, 2017 at 10:47 AM, Huey Ly<EMAIL_ADDRESS>wrote:

Sure, please let us know how it goes.
p.s. Also make the the destination folder exists and your code has
permission to write to it. The created collage file will be downloaded to
that destination when you make this call:
$collage_filelink = $client->collage($sources, 800, 600);
$destination = DIR . '/tests/testfiles/collage-test.png';
$result = $collage_filelink->download($destination);
If you don't want it downloaded, you should be able to just grab the URL
of the file generated on the Filestack server by calling the url() function
on the result Filelink object.
$collage_filelink = $client->collage($sources, 800, 600);
$new_collage_image_url = $collage_filelink->url();
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/filestack/filestack-php/issues/31#issuecomment-344984567,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABlJat4_qxEmxB5WlW3eq9TUSoPZehgks5s3Gc_gaJpZM4QfTdG
.

👍
Thanks for the updates.  Feel free to hit me up with questions at anytime if you run into another issues.


feat(serverless): send bottlecap failover metric

What does this PR do?
Sends a non-billable metric specifying the failover reason on the Datadog Extension.
Motivation
Get business insights on failover reason for Bottlecap.
datadog-lambda-extension#292
Also 5004
Additional Notes
Possible Drawbacks / Trade-offs

We check every extension string to see if it is what we are looking for, it should be fast tho.

Describe how to test/QA your changes

Unit tests
Manual failover test

/merge

/merge


feat: Add stripIgnoredCharacters option (visitor-plugin-common) - continued

Description
Adds a new stripIgnoredCharacters: boolean option to client-side-base-visitor that removes redundant characters (like line-breaks, whitespaces, etc.) from the query strings.
In the linked issue, I originally proposed doing .replace(/\s+/g, ' ').trim() on the strings, but that turned out to be an unreliable approach as it would fail to cover a number of cases:

It doesn't remove extraneous whitespaces that are not consecutive, so like query ( $var : Int! ) (should become query ($var: Int!)) or more commonly query { foo bar } (should become query{foo bar}).
If you have a string literal inside the query, that contains more than one consecutive space or line breaks, it removes those but it obviously shouldn't.

This all means a GraphQL-specific minifier is actually needed to do this reliably,
Happily I found out there's a built-in function in graphql-js for this very task! stripIgnoredCharacters — see https://github.com/graphql/graphql-js/issues/1523, which does precisely this.
Related # (issue)
Related to #5616 and continuation of #8322
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Screenshots/Sandbox (if appropriate/relevant):
Adding links to sandbox or providing screenshots can help us understand more about this PR and take action on it as appropriate
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] I've added a test in ts-documents.spec.ts, but I'm not really sure this is the best location. I looked at how other options are tested but it seems there are barely any tests for the client visitor.

Test Environment:

OS: Mac M1
@graphql-codegen/...: latest
NodeJS: v16.18

Checklist:

[x] I have followed the CONTRIBUTING doc and the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] 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
[x] Any dependent changes have been merged and published in downstream modules

Further comments
This is the continuation of PR https://github.com/dotansimha/graphql-code-generator/pull/8322 all credits go to @aradalvand

Hi @JanStevens,
Thank you for continuing @aradalvand's work here.
Please know that while adding a new option is a tempting solution, we aim to solve this issue https://github.com/dotansimha/graphql-code-generator/issues/8296 at a higher level (as part of the v3 roadmap, by rewriting codegen's core).
Considering our existing plan to address the issues linked to the amount of generated code, we don't plan to add this feature to visitor-plugin-common.
Also, we don't judge this issue as blocking since:

we recommend not committing generated files (especially types)
we now provide a new preset (client-preset) that generates only types instead of hooks or SDKs, reducing the bundle's impact
as stated above, we plan to rewrite the core only to emit used types

Hi @charlypoly,
Thanks for the swift answer, v3 sounds great! Concerning committing generated files: It's a bit hard to have a working typescript setup in various stages of CI / local development. Especially if those types are changing a lot, it could introduce typescript errors when performing production deploys so we rather have them committed.
Down the line I think that using the stripIgnoredCharacters is still an important feature to include as it greatly reduce the size of the generated.
I'll take a look at the client-preset
Regards

@JanStevens This is not about the types though, it's about the strings, strings aren't wiped out in the compliation step like types are (obviously) and so I don't see how your bullet points apply to this PR.

@JanStevens This is not about the types though, it's about the strings, strings aren't wiped out in the compliation step like types are (obviously) and so I don't see how your bullet points apply to this PR. Correct me if I'm wrong.

My reply was not concerning this PR as it can be closed, but rather at the comment from @charlypoly regarding V3 ;)

My bad, I meant to mention @charlypoly... Made a mistake. I don't think this PR should've been closed. We need this option anyway.

No worries, by all means reopen it 😉

I can't, you're the author 😁
I think you should keep it open unless @charlypoly decides otherwise. In this comment of mine I was trying to point out, in response to @charlypoly, that I'm not sure how his points were relevant to this particular feature, this PR/feature is concerned with the strings, and not the types; unless I've misunderstood something here.
Why shouldn't this PR be merged now, @charlypoly?

@charlypoly Would love for you to clarify.

Sure!
We now recommend using the client-preset for most GraphQL client (ex: front-end) use-cases because this new preset provides the smoothest developer experience and lightest code generator (for now, but will be improved in the future with a rewrite of the core of codegen).
While the client-preset still generates document nodes as follows:
const documents = {
    "\n  query allFilmsWithVariablesQuery($first: Int!) {\n    allFilms(first: $first) {\n      edges {\n        node {\n          ...FilmItem\n        }\n      }\n    }\n  }\n": types.AllFilmsWithVariablesQueryDocument,
    "\n  fragment FilmItem on Film {\n    id\n    title\n    releaseDate\n    producers\n  }\n": types.FilmItemFragmentDoc,
};

it heavily relies on TypeScript function overloads on the provided graphql() helper, meant to be used to replace graphql-tag:
const documents = {
    "\n  query allFilmsWithVariablesQuery($first: Int!) {\n    allFilms(first: $first) {\n      edges {\n        node {\n          ...FilmItem\n        }\n      }\n    }\n  }\n": types.AllFilmsWithVariablesQueryDocument,
    "\n  fragment FilmItem on Film {\n    id\n    title\n    releaseDate\n    producers\n  }\n": types.FilmItemFragmentDoc,
};

export function graphql(source: "\n  query allFilmsWithVariablesQuery($first: Int!) {\n    allFilms(first: $first) {\n      edges {\n        node {\n          ...FilmItem\n        }\n      }\n    }\n  }\n"): (typeof documents)["\n  query allFilmsWithVariablesQuery($first: Int!) {\n    allFilms(first: $first) {\n      edges {\n        node {\n          ...FilmItem\n        }\n      }\n    }\n  }\n"];
export function graphql(source: "\n  fragment FilmItem on Film {\n    id\n    title\n    releaseDate\n    producers\n  }\n"): (typeof documents)["\n  fragment FilmItem on Film {\n    id\n    title\n    releaseDate\n    producers\n  }\n"];

export function graphql(source: string): unknown;
export function graphql(source: string) {
  return (documents as any)[source] ?? {};
}

As you might guess, stripping all ignored characters will break the TS function overloading here.
While this PR still applies to existing plugins, we are right now focusing our efforts on bringing new features to the v3 roadmap and fixing blockers in existing versions.

You could still use stripIgnoredCharacters in both documents and graphql to make it condenser 🤔
I'm looking forward for the new approach, right now I can't really grasp how to use this with for example graphql-request but I guess as v3 comes a long more documentation / migration will become clear

You could still use stripIgnoredCharacters in both documents and graphql to make it condenser 🤔

Yes but then your GraphQL Operations would need to be written in a condensed way at the source too.

I'm looking forward for the new approach, right now I can't really grasp how to use this with for example graphql-request but I guess as v3 comes a long more documentation / migration will become clear

You will find a complete example here: https://github.com/dotansimha/graphql-code-generator/tree/master/examples/front-end/react/graphql-request and new documentation here: https://www.the-guild.dev/graphql/codegen/docs/guides/react-vue

You will find a complete example here: https://github.com/dotansimha/graphql-code-generator/tree/master/examples/front-end/react/graphql-request and new documentation here: https://www.the-guild.dev/graphql/codegen/docs/guides/react-vue

Cool, thanks for the docs. We currently have a setup with .graphql files and fragments as .graphql files (ex: event.fragments.graphql). For larger projects this makes it easier to have reusable (and discoverable) fragments / queries. Would that be supported in v3? I don't necessarily see how that would work 🤔.

I can't wrap my head around this supposed v3 client-preset thing, that's object whose keys are document strings, and whose values are... what?!

Cool, thanks for the docs. We currently have a setup with .graphql files and fragments as .graphql files (ex: event.fragments.graphql). For larger projects this makes it easier to have reusable (and discoverable) fragments / queries. Would that be supported in v3? I don't necessarily see how that would work 🤔.

The client-preset only support inline graphql tag based definitions, using the exposed graphql() helper.

I can't wrap my head around this supposed v3 client-preset thing, the document constant there is an object whose keys are document strings, and whose values are... what?!
Makes no sense.

The documents is mapping string literals (your GraphQL definitions) to their typing equivalent (TypedDocumentNode types)
In short, we use the literal GraphQL definition strings to build function overloads on graphql() that will return the proper typing.

v3 has been released and this aspect of the library hasn't changed at all, therefore I think this PR is still relevant.
@JanStevens Could you please consider reopening this?

@charlypoly Could you please respond?


"AD is watching" makes AD action holder

Describe the issue
Putting a doc into "AD is watching" state makes the AD the action holder. That should IMO not happen, because watching does not imply any action being needed by the AD.
Example: https://datatracker.ietf.org/doc/draft-halpern-gendispatch-antitrust/
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

I am becoming more convinced that AD is watching is not an IESG state and that what it is being used to achieve should be provided a different way.

It's IMO redundant now since we have the ability to tag I-Ds.

Is this worth fixing or should we do away with the state?
If it's to be fixed, preventing the AD from becoming an action holder is easy but the straightforward change will clear out the action holders. It would not be difficult to have it leave the action holders alone but it'd be a new code path.

Lets ask the IESG if we can do away with the state and make no other changes.
cc: @evyncke , @wkumari

@evyncke, @wkumari; nudge

For me, let's simple remove this state, it is rarely used and AD can easily tag I-D to appear in the watch list.

We will do away with the state - editing the issue title to reflect that.


Remove ^ from workspace peerDependencies

Closes https://github.com/iTwin/itwinjs-core/issues/7373
At the moment we have a requirement for consumers to use same version of all the packages delivered from this repos. As a result we should define that in package.json files and specify workspace dependencies using strict version. This should help package managers to resolve all packages coming from itwinjs-core to the same version.

@wgoehrig plz review


Create a Device User

Hi Freddy,
is it somehow possible to create a Device User in a container? Entering the container and trying this did not work:
New-NAVServerUser -LicenseType DeviceOnly -UserName "MDE01" -State Enabled -Password (ConvertTo-SecureString -String "********" -AsPlainText -Force) -ServerInstance *** -Force

Which is odd because the licensing guide clearly says that this should be supported. I rather suspect we need a canny way around the actual portal to create users just as new-navcontaineruser does?

new-navcontaineruser also just uses New-NavServerUser inside the container.
Which docker image are you using and where do you find the information that this should be possible?
Also, does it work if you are using onprem images?

The information is in the Dynamics 365 Business Central Licensing Guide, page 5 and 6: https://go.microsoft.com/fwlink/?LinkId=871590&clcid=0x409
Image used (last night): mcr.microsoft.com/businesscentral/sandbox:de-ltsc2019
But anyway, as new-navcontaineruser uses the standard New-NavServerUser this is most likely unrelated to navcontainerhelper and I will have to got through regular support, I guess.

Did you try the onprem images?
the sandbox images thinks they are saas tenants and that might obstruct something,

Will test with an on-prem image tomorrow. Strange stuff... I am betting this will take a looong support call to sort out :-)

This works fine with onprem containers, as assumed this is somehow blocked because the container thinks it is an online tenant.
If you need a device user, you might be able to create it in code, like I do with the test users here. This of course won't work in SaaS
https://dev.azure.com/businesscentralapps/_git/CreateTestUsers?path=%2Fapp%2Fapi%2050000%20-%20CreateTestUsers.al

Good Morning!
Thanks for being quicker than me :-) And as always thanks for helping!
I will close this issue because there is nothing wrong with containerhelber obviously. I will try and get clarification through regular BC support what the correct way to obtain device users is supposed to be or if the licensing guide needs updating.


Use named export for all top-level symbols

Avoids a default-on error from PyLance about using private symbols in a py.typed module: https://github.com/microsoft/pyright/blob/main/docs/configuration.md#reportPrivateImportUsage
I'm not up-to-date enough with Python packaging rules to know if this is the right thing to do, but it's certainly an easy and unobtrusive fix. Previously it was cleanest to import from the submodules, but this was not a good solution.

Codecov Report

Merging #8 (cfb4079) into main (fb62a71) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main       #8   +/-   ##
=======================================
  Coverage   97.41%   97.41%
=======================================
  Files           5        5
  Lines         309      310    +1
=======================================
+ Hits          301      302    +1
  Misses          8        8

Files Changed
Coverage Δ

dataclasses_struct/__init__.py
100.00% <100.00%> (ø)

Whoops, updated to latest main.


Tutorial Ready

Update readme to the right port
update the python to retry the connection to redis if a failover occurs
update application manifest to use common port of 8080 instead of 4000

@suhuruli,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


make failed

make in a ubuntu container running on mac failed.
# make
cd ctr && go build -ldflags "-X github.com/docker/containerd.GitCommit= " -o ../bin/ctr
../specs/spec_solaris.go:3:8: cannot find package "github.com/opencontainers/specs/specs-go" in any of:
	/usr/lib/go/src/pkg/github.com/opencontainers/specs/specs-go (from $GOROOT)
	/gowork/src/github.com/docker/containerd/vendor/src/github.com/opencontainers/specs/specs-go (from $GOPATH)
	/gowork/src/github.com/opencontainers/specs/specs-go
make: *** [client] Error 1

It seems that specs-go has been moved to github.com/opencontainers/runtime-spec/ but spec/spec_solaris.go still imports it from github.com/opencontainers/specs/.
# grep spec specs/*
...
specs/spec_solaris.go:import ocs "github.com/opencontainers/specs/specs-go"

This is weird, it shouldn't try to build the solaris specific code on a ubuntu container. Try fixing GOOS with export GOOS=linux before calling make.

Still the same error
root@moby:/gowork/src/github.com/docker/containerd# make
cd ctr && go build -ldflags "-X github.com/docker/containerd.GitCommit= " -o ../bin/ctr
../specs/spec_solaris.go:3:8: cannot find package "github.com/opencontainers/spec/specs-go" in any of:
	/usr/lib/go/src/pkg/github.com/opencontainers/spec/specs-go (from $GOROOT)
	/gowork/src/github.com/docker/containerd/vendor/src/github.com/opencontainers/spec/specs-go (from $GOPATH)
	/gowork/src/github.com/opencontainers/spec/specs-go
make: *** [client] Error 1
root@moby:/gowork/src/github.com/docker/containerd# echo $GOOS
linux
root@moby:/gowork/src/github.com/docker/containerd# uname -a
Linux moby 4.4.15-moby #1 SMP Thu Jul 28 22:03:07 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
root@moby:/gowork/src/github.com/docker/containerd# cat /etc/os-release
NAME="Ubuntu"
VERSION="14.04.4 LTS, Trusty Tahr"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 14.04.4 LTS"
VERSION_ID="14.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"

which version of go do you have?
if I do GOOS=solaris make on my host, it fails similar to yours, if I do GOOS=linux it works as expected.

hi, I have submit a PR to fix this issue, you can get my change to your local and try again.
#298

@mlaventure GOOS=linux make makes no difference, and my go version is 1.2.1.
I'll update go and try again later.
root@moby:/gowork/src/github.com/docker/containerd# GOOS=linux make
cd ctr && go build -ldflags "-X github.com/docker/containerd.GitCommit= " -o ../bin/ctr
../specs/spec_solaris.go:3:8: cannot find package "github.com/opencontainers/spec/specs-go" in any of:
	/usr/lib/go/src/pkg/github.com/opencontainers/spec/specs-go (from $GOROOT)
	/gowork/src/github.com/docker/containerd/vendor/src/github.com/opencontainers/spec/specs-go (from $GOPATH)
	/gowork/src/github.com/opencontainers/spec/specs-go
make: *** [client] Error 1
root@moby:/gowork/src/github.com/docker/containerd# go version
go version go1.2.1 linux/amd64

@moypray I tried your solution but got the same error. Why do you create a symbolic link?
root@moby:/gowork/src/github.com/docker/containerd# ls vendor/src/github.com/docker/ -l
total 2
lrwxrwxrwx 1 root root  40 Aug 17  2016 containerd -> /gowork/src/github.com/docker/containerd
drwxr-xr-x 5 root root 170 May 28 08:47 docker
drwxr-xr-x 4 root root 136 May 28 08:47 go-connections
drwxr-xr-x 4 root root 136 Aug  2 07:06 go-units
root@moby:/gowork/src/github.com/docker/containerd# GOOS=linux make
cd ctr && go build -ldflags "-X github.com/docker/containerd.GitCommit= " -o ../bin/ctr
../vendor/src/github.com/docker/containerd/specs/spec_solaris.go:3:8: cannot find package "github.com/opencontainers/spec/specs-go" in any of:
	/usr/lib/go/src/pkg/github.com/opencontainers/spec/specs-go (from $GOROOT)
	/gowork/src/github.com/docker/containerd/vendor/src/github.com/opencontainers/spec/specs-go (from $GOPATH)
	/gowork/src/github.com/opencontainers/spec/specs-go
make: *** [client] Error 1

@mlaventure Everything works well with go1.7. Thanks a lot.


Duplicate automation rules & automation processes

Home Assistant release with the issue:
0.69.1
Operating environment (Hass.io/Docker/Windows/etc.):
Hass.io
Description of problem:
After reloading the automations in Configuration ->Configuration reloading -> Reload Automations
all automations have a clone ( automation_name + automation_name_2) in configuration -> automations & on the "states" page, and both versions of each automation are running at the same time (actions are doubled). The automations.yaml file is unmodified and it is solved by restarting the server.
(Happened 2 times today, occurs randomly)
edit : happened a 3rd time after editing/saving an automation

I can't reproduce that bug anymore, so I guess it's solved. :-)


MWPW-150557 - Split Marquee CLS issues on consuming sites

This addresses an issue where we are seeing CLS Cumulative Layout Shift issue when using a marquee (split) in milo consuming sites like CC, DC, bacom and news.
However when using the same block in milo share-point there is no CLS.
This is intermittent although maximum time seeing the CLS issue.
Resolves: MWPW-150557
Test URLs:
Milo
Before Test Page: https://main--milo--adobecom.hlx.page/drafts/ruchika/marquee/split-marquee
Performance: https://pagespeed.web.dev/analysis/https-main--milo--adobecom-hlx-page-drafts-ruchika-marquee-split-marquee/jjw544duv1?form_factor=mobile
After Test Page: https://rparrish-marquee-cls--milo--adobecom.hlx.page/drafts/ruchika/marquee/split-marquee
Performance: https://pagespeed.web.dev/analysis/https-rparrish-marquee-cls--milo--adobecom-hlx-page-drafts-ruchika-marquee-split-marquee/fmwg7vdiwh?form_factor=mobile
CC
Test Page: https://main--cc--adobecom.hlx.live/drafts/ruchika/marquee/split-marquee?milolibs=rparrish-marquee-cls
Performance: before after  improved - no CLS
DC
Test Page: https://main--dc--adobecom.hlx.live/drafts/ruchika/marquee/split-marquee?milolibs=rparrish-marquee-cls
Performance: before after  improved - no CLS
Bacom
Test Page: https://main--bacom--adobecom.hlx.live/drafts/ruchika/marquee/split-marquee?milolibs=rparrish-marquee-cls
Performance: before after   - Same
Getting CLS consistently though if using svg icon for icon area
Test Page: https://main--bacom--adobecom.hlx.live/drafts/ruchika/document2?milolibs=rparrish-marquee-cls
Performance: Before After  improved - no CLS
News
Test Page: https://main--news--adobecom.hlx.live/drafts/ruchika/marquee/split-marquee?milolibs=rparrish-marquee-cls
Performance: Before After  improved - no CLS
Regression Testing
https://rparrish-marquee-cls–milo–adobecom.hlx.page/docs/library/blocks/marquee
Checks out

closing for squashed testing branch


Added check for go.mod

Description
Addressed https://github.com/dapr/components-contrib/issues/140
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: https://github.com/dapr/components-contrib/issues/140
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation

@youngbupark thanks for your review, please feel free to merge anytime :100:


Follow debian package convention

@feross regarding #290
Package convention
https://www.debian.org/doc/manuals/debian-faq/ch-pkg_basics.en.html

The Debian binary package file names conform to the following convention: -.deb
Note that foo is supposed to be the package name. As a check, one can learn the package name associated with a particular Debian archive file (.deb file) in one of these ways:
inspect the "Packages" file in the directory where it was stored at a Debian FTP archive site. This file contains a stanza describing each package; the first field in each stanza is the formal package name.
use the command dpkg --info foo_VVV-RRR_AAA.deb (where VVV, RRR and AAA are the version, revision and architecture of the package in question, respectively). This displays, among other things, the package name corresponding to the archive file being unpacked.
The VVV component is the version number specified by the upstream developer. There are no standards in place here, so the version number may have formats as different as "19990513" and "1.3.8pre1".
The RRR component is the Debian revision number, and is specified by the Debian developer (or an individual user if he chooses to build the package himself). This number corresponds to the revision level of the Debian package, thus, a new revision level usually signifies changes in the Debian Makefile (debian/rules), the Debian control file (debian/control), the installation or removal scripts (debian/p*), or in the configuration files used with the package.
The AAA component identifies the processor for which the package was built. This is commonly i386, which refers to chips compatible to Intel's 386 or later versions.

package name MUST be lowercase
https://lintian.debian.org/tags/package-not-lowercase.html
otherwise the installer will complain
https://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-Source

Package names (both source and binary, see Package, Section 5.6.7) must consist only of lower case letters (a-z), digits (0-9), plus (+) and minus (-) signs, and periods (.). They must be at least two characters long and must start with an alphanumeric character.

updated nobin-debian-installer 0.0.6


Cant build on Windows 10

Unfortunately i had no luck building the project on Windows 10 with either VS2017,19,2022 or GCC via MinGW.
What kind of compiler and which OS are tested to be working for this project ?

I'm sorry I initially missed your question. We went through the trouble of trying to compile this for Windows ourselves. The project is updated and should work smoothly for Windows too now. Moreover, we have make Github actions, and have automatic compiled releases. So head over to the Releases to find a build for Windows! :smile:

Feel free to reply below in case of any trouble. I should get notifications from this repo now. Will close, as I assume it's fixed.


[FEATURE] Update to Current Quarkus Platform (2.16.3.Final)

Enhancement Type:
Platform
Is your feature request related to a problem? Please describe.
Platform update for Quarkus and Java
Describe the solution you'd like
Update to current Quarkus platform from 2.4.1.Final to 2.16.3.Final and switch to OpenJDK Version 17
Describe alternatives you've considered
Intension is to update to current releases for platform security fixes and other enhancements.
Additional context

Revised. Updated to Quarkus 3.5


profiles: apparmor: remove unused fields

The template was referring to an apparmor profile that we don't ship, we
can just nix the lines from the profile and remove the fields in the
Data struct.
Fixes #20584

Signed-off-by: Aleksa Sarai<EMAIL_ADDRESS>/cc @icecrime @jfrazelle

So the peer-profile is referring to the apparmor profile not the binary
path, so this is cases where the daemon is running with the /usr/bin/docker
profile it's confusing because depending on the daemon profile being
installed it could mean different things, but it's not technically the
daemon path, since it's reliant on a profile we are not even shipping, I
say we nix the corresponding lines from the docker-default profile entirely
since it's not even applicable without the daemon profile
On Sunday, March 13, 2016, Aleksa Sarai<EMAIL_ADDRESS>wrote:

In order to deal with cases where the docker binary is not in $PATH,
read the full path of the Docker binary from /proc/self/exe (which is
guaranteed to always be an absolute path and always be correct for the
process -- unless the file has been unlinked in which case there's not
much we can do).
Previously the version field for the template was not filled, resulting
in all version-based checks failing. Fix this by actually filling the
field.
Fixes #20584 https://github.com/docker/docker/issues/20584
[image: cute kitty]
https://camo.githubusercontent.com/420e8b4262439d47560d46d6027cdbb0b0e0df24/687474703a2f2f6661726d342e737461746963666c69636b722e636f6d2f333236302f323738393630343439305f303636646431633738365f622e6a7067
Signed-off-by: Aleksa Sarai<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','asarai@suse.de');>
/cc @icecrime https://github.com/icecrime @jfrazelle
https://github.com/jfrazelle
You can view, comment on, or merge this pull request online at:
https://github.com/docker/docker/pull/21176
Commit Summary

profiles: apparmor: set ExecPath to $(readlink /proc/self/exe)
profiles: apparmor: actually fill .Version

File Changes

M profiles/apparmor/apparmor.go
https://github.com/docker/docker/pull/21176/files#diff-0 (6)

Patch Links:

https://github.com/docker/docker/pull/21176.patch
https://github.com/docker/docker/pull/21176.diff

—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/21176.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

@jfrazelle Wait, so both of the version-based {{if}}s are actually not valid (in which case we can nix Version as well as ExecPath and the peer-profile).

Yup
On Sunday, March 13, 2016, Aleksa Sarai<EMAIL_ADDRESS>wrote:

@jfrazelle https://github.com/jfrazelle Wait, so both of the
version-based {{if}}s are actually not valid (in which case we can nix
Version as well as ExecPath and the peer-profile).
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/21176#issuecomment-196160932.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

@jfrazelle There, I've updated this PR.

@jfrazelle There, fixed.

WindowsTP4 failing on https://github.com/docker/docker/issues/21247 again (flaky test)

The win2lin failure is because the server has run out of memory (or it's a containerd issue?).

It's all :green_heart:.
/ping @jfrazelle

LGTM!


Adding Quotes

👨💻 What did you do ?
I've added new quotes to the QuoteVerse project. These quotes cover various categories, including Motivation, Life, and Success. Each quote follows the specified format with the author and category.
Note to reviewers
NA
📷 Screenshots
Not an UI issue
✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title and description of the PR is clear and explains the approach.
[x] I have read the contribution guideline properly and following the criteria.

can you star the repo, as it is the contribution criteria in guideline?


XML encoding issues

When using jamf-migrator to download scripts as trimmed sources (Download-only mode), the resulting downloaded file is not valid.
Steps to reproduce:

create a simple script in Jamf Pro with specific content (the script does nothing and makes no sense - this is just to get the expected result)

#!/bin/zsh

LogFile="/private/tmp/test.log"

exec > >(tee -a "${LogFile}") 2>&1

exit 0

The downloaded trimmed xml source gets:
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<script>
    <name>Test Script</name>
    <category>Settings</category>
    <filename>Test Script</filename>
    <info></info>
    <notes></notes>
    <priority>After</priority>
    <parameters></parameters>
    <os_requirements></os_requirements>
    <script_contents>#!/bin/zsh

LogFile="/private/tmp/test.log"

exec > >(tee -a "${LogFile}") 2>&1

exit 0
</script_contents>
</script>

But instead, this would be correct:
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<script>
    <name>Test Script</name>
    <category>Settings</category>
    <filename>Test Script</filename>
    <info></info>
    <notes></notes>
    <priority>After</priority>
    <parameters></parameters>
    <os_requirements></os_requirements>
    <script_contents>#!/bin/zsh

LogFile="/private/tmp/test.log"

exec > >(tee -a "${LogFile}") 2>&1

exit 0
</script_contents>
</script>

Please note the encodings of the "greater than", "lower than" and ampersand characters. The actual script contents are not correctly encoded, which brakes the surrounding xml and subsequent uploads are going to fail.

Did you find the migrated script was actually incorrect on the destination server?
I have to confess I didn't dig into the code, but my recollection is that the downloaded file displays the script in a format (decoded xml) that makes it easy to simply copy the script and run it.  As the xml is uploaded the script is encoded.

Sorry, I need to be a bit more precise.
The migration works when only using jamf-migrator. Seems to match your description that the XML is encoded before uploading.
We're using a mixed set of jamf-migrator to download e.g. scripts from a dedicated instance and then use a cURL command to upload the xml (this is used as a quick-setup-script for new Jamf Pro instances, but shouldn't matter here).
cURL, or at least the Jamf Pro API, won't accept the downloaded file from jamf-migrator as XML is not correctly encoded.
Does this make sense to you?

Makes perfect sense and I was afraid that's where you were going.
Files are intended to be used with migrator, couple of thoughts.

the app is scriptable, use that to upload instead of cURL
add some code to your process to encode the script before uploading

Suppose it would also be a nice option to have in the app the ability to choose what 'format' to export scripts.

Well, at least it worked in previous versions of jamf-migrator. So I'd assume you changed the code at some point to encode/decode the XML before down- and uploads. If that is the case, an option to select the downloaded format would indeed help a lot. Of course, if this is not too much effort in implementing this.
For the other two options:

the app is scriptable, use that to upload instead of cURL => I'll definitely have a look at this
add some code to your process to encode the script before uploading => are you aware of any built-in tools or 3rd party libraries that I can use to encode my stuff?

Afraid I do not have any existing resources to do the encoding.  Thinking some regex/stylesheets (xslt) would be involved.


esphome: 1.16.0 -> 1.17.0

Motivation for this change
https://github.com/esphome/esphome/releases/tag/v1.17.0
https://github.com/esphome/esphome/releases/tag/v1.16.2
https://github.com/esphome/esphome/releases/tag/v1.16.1
Update licensing, enable test suite.
Not tested against an actual device yet.
[17:56:18][I][app:105]: ESPHome version 1.17.0 compiled on May  4 2021, 17:54:20
[...]
[17:56:31][D][api.connection:617]: Client 'Home Assistant 2021.4.6 (<IP_ADDRESS>)' connected successfully!
[...]
[17:57:05][D][switch:029]: '208393_relay' Toggling ON.
[17:57:06][D][switch:045]: '208393_relay': Sending state ON
[17:57:06][D][light:265]: 'led' Setting:
[17:57:06][D][light:274]:   State: ON
[17:57:06][D][light:304]:   Transition Length: 1.0s
[17:57:07][D][switch:029]: '208393_relay' Toggling OFF.
[17:57:07][D][switch:045]: '208393_relay': Sending state OFF
[17:57:07][D][light:265]: 'led' Setting:
[17:57:07][D][light:274]:   State: OFF
[17:57:07][D][light:304]:   Transition Length: 1.0s

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 121705 run on x86_64-linux 1

1 package built:

esphome

Result of nixpkgs-review pr 121705 run on x86_64-linux 1

I'd prefer more qualitative reviews, especially if I have done more thorough testing in the top post already and the changeset is bigger than a version/sha256 bump.

[root@dione:/etc/nixos/nixpkgs]# uname -m
aarch64

[root@dione:/etc/nixos/nixpkgs]# nix-build -A esphome
error: i686 Linux package set can only be used with the x86 family.
(use '--show-trace' to show detailed location information)

Fails to build on aarch64, investigating.

My log doesn't know colors (could be related to esphome/esphome#1671 which will remove colorlog and add a different coloring approach).

Works for me, will be part of the next release.

And updated to 1.17.1, removed colorlog, as that change was backported.