Very slow network running time

Hi,
I trained the R-50 baseline by using the config file of configs/resnet/resnet50_b32x8_imagenet.py in 8 V100s. I did not modify any settings but encountered a problem. The running time of my network is very very slow. The training log is as follows:
2021-08-14 19:49:53,357 - mmcv - INFO - Reducer buckets have been rebuilt in this iteration.
2021-08-14 19:53:03,573 - mmcls - INFO - Epoch [1][100/5005]	lr: 1.000e-01, eta: 20 days, 21:37:27, time: 3.609, data_time: 2.011, memory: 3087, loss: 6.8857
2021-08-14 19:56:47,925 - mmcls - INFO - Epoch [1][200/5005]	lr: 1.000e-01, eta: 16 days, 22:39:18, time: 2.244, data_time: 0.192, memory: 3087, loss: 6.5704
2021-08-14 19:59:46,847 - mmcls - INFO - Epoch [1][300/5005]	lr: 1.000e-01, eta: 14 days, 17:54:58, time: 1.789, data_time: 0.048, memory: 3087, loss: 6.4248
2021-08-14 20:03:17,095 - mmcls - INFO - Epoch [1][400/5005]	lr: 1.000e-01, eta: 14 days, 2:24:04, time: 2.102, data_time: 0.007, memory: 3087, loss: 6.2950
2021-08-14 20:06:22,623 - mmcls - INFO - Epoch [1][500/5005]	lr: 1.000e-01, eta: 13 days, 10:12:07, time: 1.855, data_time: 0.001, memory: 3087, loss: 6.1981
2021-08-14 20:09:47,656 - mmcls - INFO - Epoch [1][600/5005]	lr: 1.000e-01, eta: 13 days, 3:53:59, time: 2.050, data_time: 0.001, memory: 3087, loss: 6.0953
2021-08-14 20:12:50,762 - mmcls - INFO - Epoch [1][700/5005]	lr: 1.000e-01, eta: 12 days, 19:01:58, time: 1.831, data_time: 0.235, memory: 3087, loss: 6.0226
2021-08-14 20:16:15,095 - mmcls - INFO - Epoch [1][800/5005]	lr: 1.000e-01, eta: 12 days, 16:03:11, time: 2.043, data_time: 0.008, memory: 3087, loss: 5.9530
2021-08-14 20:19:47,246 - mmcls - INFO - Epoch [1][900/5005]	lr: 1.000e-01, eta: 12 days, 14:55:42, time: 2.122, data_time: 0.011, memory: 3087, loss: 5.8875
2021-08-14 20:23:07,881 - mmcls - INFO - Epoch [1][1000/5005]	lr: 1.000e-01, eta: 12 days, 12:25:08, time: 2.006, data_time: 0.001, memory: 3087, loss: 5.8032
2021-08-14 20:26:36,350 - mmcls - INFO - Epoch [1][1100/5005]	lr: 1.000e-01, eta: 12 days, 11:20:37, time: 2.085, data_time: 0.004, memory: 3087, loss: 5.7613

And my environments is as follows:
sys.platform: linux
Python: 3.8.8 (default, Apr 13 2021, 19:58:26) [GCC 7.3.0]
CUDA available: True
GPU 0,1,2,3,4,5,6,7: Tesla V100-SXM2-32GB
CUDA_HOME: /usr/local/cuda-10.2
NVCC: Cuda compilation tools, release 10.2, V10.2.89
GCC: gcc (GCC) 7.3.1 20180303 (Red Hat 7.3.1-5)
PyTorch: 1.9.0+cu102
PyTorch compiling details: PyTorch built with:
  - GCC 7.3
  - C++ Version: 201402
  - Intel(R) Math Kernel Library Version 2020.0.0 Product Build 20191122 for Intel(R) 64 architecture applications
  - Intel(R) MKL-DNN v2.1.2 (Git Hash 98be7e8afa711dc9b66c8ff3504129cb82013cdb)
  - OpenMP 201511 (a.k.a. OpenMP 4.5)
  - NNPACK is enabled
  - CPU capability usage: AVX2
  - CUDA Runtime 10.2
  - NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_70,code=sm_70
  - CuDNN 7.6.5
  - Magma 2.5.2
  - Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=10.2, CUDNN_VERSION=7.6.5, CXX_COMPILER=/opt/rh/devtoolset-7/root/usr/bin/c++, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_KINETO -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=1.9.0, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON,

TorchVision: 0.10.0+cu102
OpenCV: 4.5.3
MMCV: 1.3.11
MMCV Compiler: n/a
MMCV CUDA Compiler: n/a
MMClassification: 0.14.0+

Looking forward to your reply.

It looks like you don't have mmcv-full installed， you can try to install mmcv-full refer mmcv doc .

Hello, I tried it but nothing works.

Can you try to run some examples of other repo such as mmdet to see if it is an environmental problem? We will test whether this problem can be reproduced as soon as possible on pytorch1.9.

I will try mmdetection as soon as possible. There is no problem for Detectron2  in the environment as described above.

I tried in my torch1.9 environment as below..(sorry that there is no cuda10.2 in our environment)
sys.platform: linux
Python: 3.8.11 (default, Aug  3 2021, 15:09:35) [GCC 7.5.0]
CUDA available: True
GPU 0,1,2,3,4,5,6,7: Tesla V100-SXM2-32GB
CUDA_HOME: /mnt/lustre/share/cuda-11.1
NVCC: Build cuda_11.1.TC455_06.29069683_0
GCC: gcc (GCC) 5.4.0
PyTorch: 1.9.0
PyTorch compiling details: PyTorch built with:
  - GCC 7.3
  - C++ Version: 201402
  - Intel(R) oneAPI Math Kernel Library Version 2021.3-Product Build 20210617 for Intel(R) 64 architecture applications
  - Intel(R) MKL-DNN v2.1.2 (Git Hash 98be7e8afa711dc9b66c8ff3504129cb82013cdb)
  - OpenMP 201511 (a.k.a. OpenMP 4.5)
  - NNPACK is enabled
  - CPU capability usage: AVX2
  - CUDA Runtime 11.1
  - NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86;-gencode;arch=compute_37,code=compute_37
  - CuDNN 8.0.5
  - Magma 2.5.2
  - Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=11.1, CUDNN_VERSION=8.0.5, CXX_COMPILER=/opt/rh/devtoolset-7/root/usr/bin/c++, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_KINETO -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=1.9.0, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON,

TorchVision: 0.10.0
OpenCV: 4.5.3
MMCV: 1.3.11
MMCV Compiler: GCC 7.3
MMCV CUDA Compiler: 11.1
MMClassification: 0.14.0+c0a4916

And get result:
2021-08-17 16:49:31,759 - mmcls - INFO - workflow: [('train', 1)], max: 100 epochs
2021-08-17 17:00:17,073 - mmcls - INFO - Epoch [1][100/5005]    lr: 1.000e-01, eta: 37 days, 5:46:17, time: 6.430, data_time: 5.762, memory: 3086, loss: 6.8864
2021-08-17 17:00:36,627 - mmcls - INFO - Epoch [1][200/5005]    lr: 1.000e-01, eta: 19 days, 4:23:45, time: 0.196, data_time: 0.008, memory: 3086, loss: 6.5649
2021-08-17 17:00:52,606 - mmcls - INFO - Epoch [1][300/5005]    lr: 1.000e-01, eta: 13 days, 2:12:15, time: 0.158, data_time: 0.023, memory: 3086, loss: 6.4290
2021-08-17 17:01:08,931 - mmcls - INFO - Epoch [1][400/5005]    lr: 1.000e-01, eta: 10 days, 1:19:52, time: 0.165, data_time: 0.005, memory: 3086, loss: 6.3161
2021-08-17 17:01:26,525 - mmcls - INFO - Epoch [1][500/5005]    lr: 1.000e-01, eta: 8 days, 5:54:13, time: 0.176, data_time: 0.001, memory: 3086, loss: 6.2153
2021-08-17 17:01:40,786 - mmcls - INFO - Epoch [1][600/5005]    lr: 1.000e-01, eta: 7 days, 0:10:29, time: 0.142, data_time: 0.002, memory: 3086, loss: 6.1281
2021-08-17 17:01:57,870 - mmcls - INFO - Epoch [1][700/5005]    lr: 1.000e-01, eta: 6 days, 3:30:53, time: 0.171, data_time: 0.005, memory: 3086, loss: 6.0590
2021-08-17 17:02:15,169 - mmcls - INFO - Epoch [1][800/5005]    lr: 1.000e-01, eta: 5 days, 12:03:23, time: 0.173, data_time: 0.005, memory: 3086, loss: 5.9902
2021-08-17 17:02:30,306 - mmcls - INFO - Epoch [1][900/5005]    lr: 1.000e-01, eta: 4 days, 23:41:38, time: 0.151, data_time: 0.004, memory: 3086, loss: 5.9157
2021-08-17 17:02:44,487 - mmcls - INFO - Epoch [1][1000/5005]   lr: 1.000e-01, eta: 4 days, 13:40:16, time: 0.142, data_time: 0.007, memory: 3086, loss: 5.8322
2021-08-17 17:03:09,668 - mmcls - INFO - Epoch [1][1100/5005]   lr: 1.000e-01, eta: 4 days, 6:51:04, time: 0.251, data_time: 0.020, memory: 3086, loss: 5.7813
2021-08-17 17:03:38,101 - mmcls - INFO - Epoch [1][1200/5005]   lr: 1.000e-01, eta: 4 days, 1:33:04, time: 0.285, data_time: 0.011, memory: 3086, loss: 5.7112
2021-08-17 17:03:51,988 - mmcls - INFO - Epoch [1][1300/5005]   lr: 1.000e-01, eta: 3 days, 19:30:51, time: 0.139, data_time: 0.007, memory: 3086, loss: 5.6603
2021-08-17 17:04:06,286 - mmcls - INFO - Epoch [1][1400/5005]   lr: 1.000e-01, eta: 3 days, 14:22:23, time: 0.143, data_time: 0.005, memory: 3086, loss: 5.6014
2021-08-17 17:04:24,257 - mmcls - INFO - Epoch [1][1500/5005]   lr: 1.000e-01, eta: 3 days, 10:15:21, time: 0.179, data_time: 0.008, memory: 3086, loss: 5.5582
2021-08-17 17:04:41,806 - mmcls - INFO - Epoch [1][1600/5005]   lr: 1.000e-01, eta: 3 days, 6:37:11, time: 0.176, data_time: 0.019, memory: 3086, loss: 5.5149
2021-08-17 17:04:56,351 - mmcls - INFO - Epoch [1][1700/5005]   lr: 1.000e-01, eta: 3 days, 3:10:15, time: 0.146, data_time: 0.007, memory: 3086, loss: 5.4500
2021-08-17 17:05:11,732 - mmcls - INFO - Epoch [1][1800/5005]   lr: 1.000e-01, eta: 3 days, 0:09:42, time: 0.153, data_time: 0.002, memory: 3086, loss: 5.4266
2021-08-17 17:05:26,809 - mmcls - INFO - Epoch [1][1900/5005]   lr: 1.000e-01, eta: 2 days, 21:26:56, time: 0.151, data_time: 0.004, memory: 3086, loss: 5.3389
2021-08-17 17:05:43,155 - mmcls - INFO - Epoch [1][2000/5005]   lr: 1.000e-01, eta: 2 days, 19:05:42, time: 0.163, data_time: 0.001, memory: 3086, loss: 5.3111
2021-08-17 17:05:57,585 - mmcls - INFO - Epoch [1][2100/5005]   lr: 1.000e-01, eta: 2 days, 16:50:19, time: 0.144, data_time: 0.003, memory: 3086, loss: 5.2689
2021-08-17 17:06:13,340 - mmcls - INFO - Epoch [1][2200/5005]   lr: 1.000e-01, eta: 2 days, 14:52:11, time: 0.157, data_time: 0.007, memory: 3086, loss: 5.2007
2021-08-17 17:06:30,818 - mmcls - INFO - Epoch [1][2300/5005]   lr: 1.000e-01, eta: 2 days, 13:10:31, time: 0.175, data_time: 0.013, memory: 3086, loss: 5.1628
2021-08-17 17:06:46,992 - mmcls - INFO - Epoch [1][2400/5005]   lr: 1.000e-01, eta: 2 days, 11:32:51, time: 0.162, data_time: 0.004, memory: 3086, loss: 5.1121K
2021-08-17 17:07:07,667 - mmcls - INFO - Epoch [1][2500/5005]   lr: 1.000e-01, eta: 2 days, 10:17:50, time: 0.207, data_time: 0.021, memory: 3086, loss: 5.0653
2021-08-17 17:07:29,649 - mmcls - INFO - Epoch [1][2600/5005]   lr: 1.000e-01, eta: 2 days, 9:12:52, time: 0.220, data_time: 0.019, memory: 3086, loss: 5.0102
2021-08-17 17:07:47,239 - mmcls - INFO - Epoch [1][2700/5005]   lr: 1.000e-01, eta: 2 days, 7:59:04, time: 0.176, data_time: 0.004, memory: 3086, loss: 4.9926
2021-08-17 17:08:14,123 - mmcls - INFO - Epoch [1][2800/5005]   lr: 1.000e-01, eta: 2 days, 7:18:07, time: 0.269, data_time: 0.004, memory: 3086, loss: 4.9213
2021-08-17 17:08:30,246 - mmcls - INFO - Epoch [1][2900/5005]   lr: 1.000e-01, eta: 2 days, 6:09:10, time: 0.161, data_time: 0.002, memory: 3086, loss: 4.9166
2021-08-17 17:08:45,986 - mmcls - INFO - Epoch [1][3000/5005]   lr: 1.000e-01, eta: 2 days, 5:03:44, time: 0.157, data_time: 0.003, memory: 3086, loss: 4.8623
2021-08-17 17:09:02,818 - mmcls - INFO - Epoch [1][3100/5005]   lr: 1.000e-01, eta: 2 days, 4:05:25, time: 0.168, data_time: 0.010, memory: 3086, loss: 4.8188
2021-08-17 17:09:30,778 - mmcls - INFO - Epoch [1][3200/5005]   lr: 1.000e-01, eta: 2 days, 3:39:13, time: 0.278, data_time: 0.013, memory: 3086, loss: 4.7785
2021-08-17 17:09:48,281 - mmcls - INFO - Epoch [1][3300/5005]   lr: 1.000e-01, eta: 2 days, 2:48:58, time: 0.176, data_time: 0.011, memory: 3086, loss: 4.7594
2021-08-17 17:10:09,989 - mmcls - INFO - Epoch [1][3400/5005]   lr: 1.000e-01, eta: 2 days, 2:11:37, time: 0.217, data_time: 0.005, memory: 3086, loss: 4.6983
2021-08-17 17:10:25,380 - mmcls - INFO - Epoch [1][3500/5005]   lr: 1.000e-01, eta: 2 days, 1:21:22, time: 0.154, data_time: 0.014, memory: 3086, loss: 4.6751
2021-08-17 17:10:48,638 - mmcls - INFO - Epoch [1][3600/5005]   lr: 1.000e-01, eta: 2 days, 0:52:03, time: 0.233, data_time: 0.013, memory: 3086, loss: 4.6381
2021-08-17 17:11:07,351 - mmcls - INFO - Epoch [1][3700/5005]   lr: 1.000e-01, eta: 2 days, 0:13:53, time: 0.186, data_time: 0.018, memory: 3086, loss: 4.6175
2021-08-17 17:11:22,554 - mmcls - INFO - Epoch [1][3800/5005]   lr: 1.000e-01, eta: 1 day, 23:30:32, time: 0.153, data_time: 0.008, memory: 3086, loss: 4.5763
2021-08-17 17:11:41,784 - mmcls - INFO - Epoch [1][3900/5005]   lr: 1.000e-01, eta: 1 day, 22:57:38, time: 0.192, data_time: 0.006, memory: 3086, loss: 4.5557
2021-08-17 17:12:03,091 - mmcls - INFO - Epoch [1][4000/5005]   lr: 1.000e-01, eta: 1 day, 22:30:45, time: 0.213, data_time: 0.033, memory: 3086, loss: 4.5332
2021-08-17 17:12:22,068 - mmcls - INFO - Epoch [1][4100/5005]   lr: 1.000e-01, eta: 1 day, 22:00:26, time: 0.190, data_time: 0.031, memory: 3086, loss: 4.4936
2021-08-17 17:12:42,215 - mmcls - INFO - Epoch [1][4200/5005]   lr: 1.000e-01, eta: 1 day, 21:33:30, time: 0.200, data_time: 0.002, memory: 3086, loss: 4.4634
2021-08-17 17:12:59,081 - mmcls - INFO - Epoch [1][4300/5005]   lr: 1.000e-01, eta: 1 day, 21:02:08, time: 0.170, data_time: 0.026, memory: 3086, loss: 4.4181
2021-08-17 17:13:13,871 - mmcls - INFO - Epoch [1][4400/5005]   lr: 1.000e-01, eta: 1 day, 20:27:58, time: 0.148, data_time: 0.011, memory: 3086, loss: 4.4128
2021-08-17 17:13:30,385 - mmcls - INFO - Epoch [1][4500/5005]   lr: 1.000e-01, eta: 1 day, 19:58:20, time: 0.164, data_time: 0.011, memory: 3086, loss: 4.3713

Has this question been solved?

No, I still have a very slow training speed. I do not know what happened to my environment.

No, I still have a very slow training speed. I do not know what happened to my environment.

Have you tried with MMDet?
This will be helpful for us to confirm if this is an mmcls issue or mmcv issue.

My problem is solved. It is related to the unmatch between the torch and the cuda version. Thanks for the misclassification team.


Modifying the XP box manually should reflect in the log

I'm not sure how practical this is, but it sure would be nice :0


[FLINK-20351] Correct the logging of the location of a task failure on the JM

What is the purpose of the change
This commit logs the TaskManagerLocation.toString instead of the LogicalSlot.toString
when logging an Execution failure.
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Thanks for the review @rmetzger. Merging this PR now.


Inline SQL Script doesn't work on 2017 Hosted Agent

We're using the Azure SQL Database Deployment task's "Inline SQL Script" in a few build definitions.
https://github.com/Microsoft/vsts-tasks/blob/master/Tasks/SqlAzureDacpacDeployment/README.md
This worked on the "Hosted" queue, but fails on the "Hosted 2017" queue:
==============================================================================
Task         : Azure SQL Database Deployment
Description  : Deploy Azure SQL DB using DACPAC or run scripts using SQLCMD
Version      : 1.1.23
Author       : Microsoft Corporation
Help         : [More Information](https://aka.ms/sqlazuredeployreadme)
==============================================================================
tempFile= d:\a\_temp\tmp6EE1.tmp
Invoke-Sqlcmd -ServerInstance (snip) -Inputfile "d:\a\_temp\tmp6EE1.tmp" -QueryTimeout 600 -ConnectionTimeout 120
SQL Powershell Module is not installed on your agent machine. Please follow steps given below to execute this task
1. Install PowershellTools & SharedManagementObjects(dependency), from https://www.microsoft.com/en-us/download/details.aspx?id=52676 (2016)
2. Restart agent machine after installing tools to register Module path updates
3. Run Import-Module SQLPS on your agent Powershell prompt. (This step is not required on Powershell 3.0 enabled machines)
The term 'Invoke-Sqlcmd' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
******************************************************************************

@jrr
On Hosted VS 2017 pool SQL client tools are missing.
Invoke-SqlCmd cmdlet is not included as part of Windows Powershell, but instead it is of sqlps(SQL Server PowerShell Host). So we need to install client tool for it in VS 17 hosted agent.
I will suggest you to try with Hosted agent pool till issue will be fixed.
Thanks,
Roshan Kumar

Any updates about that?
I tried the workaround of using the Hosted agent queue, but the version of AzurePS is too old and missing cmdlet updates I need.

@ggirard07 while we wait for this, we're using sqlcmd.exe directly from a Command Line task. It can be found on the 2017 hosted agents in C:\Program Files\Microsoft SQL Server\110\Tools\Binn; you can add it to $PATH with a Batch Script task (with Modify Environment).
Gross, huh? I hope we can remove this hack soon!

@jrr thanks for the tip.
Is there any reason why you don't directly reference the the sqlcmd.exe instead of adding it to the path?

My solution was to split my release (or build) into 2 agent phases as a workaround.

@Izzmo - that shouldn't be necessary correct?  Are you splitting into phases because you're adding to the path?  If your script just declares the path and references it directly as @ggirard07 mentioned, wouldn't that be the simplest workaround?

@bryanmacfarlane Sorry I posted this in haste. Let me explain a little better.
My situation is that I needed to have a VS2017 agent and call out to Azure to create a Azure SQL server. Since the CLI doesn't support cross-subscription database copies, I need to run a inline SQL command to do the copy. With the 2017 agent, it doesn't have the SQL tools installed apparently. BUT, with the regular Hosted agents, it is installed but the Azure CLI tools are not.. ugh.
So, I have a two-phase setup where I'm running my tasks on VS 2017 and then a second phase on a regular Hosted agent for the SQL task.

2 phases releases wasn't  possible for me as cross phases variables are not supported it, except with yaml.
I was able to reference sqlcmd.exe directly by using the path provided by @jrr.
My script is executed properly and firewall seems to not be an issue finally...

The required SQL powershell modules have been installed in Hosted VS2017 agents.

Please reopen in case you still face this issue.

do you have the agent version including it?

Hosted pools have the latest versions only. You can check it by going to the agent capabilities. I think it is 2.126.0.

We are stilling having this issue with Hosted Agent  2.131.0 .  We can't use this:
Import-Module SQLPS

@tank104  can you try using SqlServer module instead?
I verified that SQLPS module is available on Hosted VS2017 agent versioned 2.131.0.
I believe, in your case, the error was not thrown with the line 'Import-Module SQLPS'. It might be a command that you are using after this.
Apparently, there is a known issue with SQLPS module - Refer
Anyway, SQLPS module is not updated anymore. SqlServer module can be used instead. Refer.
You can use this module on Hosted VS2017 agent using below script.
Write-Host "installing modules: start"
Install-Module -Name SqlServer -Force -AllowClobber
Write-Host "installing modules: end"

Write-Host "getting modules list: start"
Get-Module SqlServer -ListAvailable
Write-Host "getting modules list: end"

Write-Host "listing commands: start"
Import-Module -Name SqlServer
Get-Command -Module SqlServer
Write-Host "listing commands: end"

Thanks I will try that shortly and get back to you.

That worked thanks - although the "Install-Module -Name SqlServer -Force -AllowClobber" takes quite a while.  Will that have to run each time?

@tank104 I run into this from time to time (seems like every time I restart my PC).
After you install it once, it goes pretty fast, even if it says the module isn't available.

@tank104 Yes, you have to run it each time if you are using Hosted Agent Pool. In Hosted Agent Pool, for each run, the agents are freshly spin up from base image.

@Izzmo You seems to use private agent. On private agents, you need not run this each time.


Feature Request - force dark/light mode

It would be great if dark or light mode could be enforced somehow independently of the system preferences.
Transparency or not, having a project with a lot of images from dot or mscgen simply looks better in light mode.

I've figured out that it is enough to just add the class light-mode to the html tag in the header template:
<html xmlns="http://www.w3.org/1999/xhtml" class="light-mode">

This disables the dark mode permanently without the need to modify doxygen-awesome.css.
It seems like I already accidentally implemented the requested feature, without knowing. 🙈
Disclaimer: This only works if you don't use the dark-mode toggle extension.

Wahaha that's great. Feature by accident. Thank you! :)

I've added documentation about how to force light-/dark-mode here
If you run into problems with this approach, pls let me know!

More recent firefox version (at least 107.0) seem to break that feature for whatever reason. :(
Here is a comparison with Chromium 107.0.5304.110


intel realsense viewer: cannot connect to the device

librealsense
2.35.2 RELEASE

OS
Linux

Please provide a description of the problem
error message: failed to claim usb interface: 0, error: RS2_USB_STATUS_BUSY
this is with the Upsquared 2 board, D435i camera
librealsense version: 2.35.2
20/10 22:58:33,682 ERROR [140653509986048] (handle-libusb.h:95) failed to claim usb interface: 0, error: RS2_USB_STATUS_BUSY
20/10 22:58:33,683 WARNING [140653535164160] (rs.cpp:282) null pointer passed for argument "device"
20/10 22:58:33,821 INFO [140653944342400] (rs.cpp:2398) Framebuffer size changed to 1853 x 1025
20/10 22:58:33,824 INFO [140653944342400] (rs.cpp:2398) Window size changed to 1853 x 1025
20/10 22:58:33,824 INFO [140653944342400] (rs.cpp:2398) Scale Factor is now 1
20/10 22:58:58,750 INFO [140653944342400] (environment.cpp:85) Found 3 unreachable streams, 3 extrinsics deleted
20/10 22:58:58,750 ERROR [140653518378752] (handle-libusb.h:95) failed to claim usb interface: 0, error: RS2_USB_STATUS_BUSY
20/10 22:58:58,752 WARNING [140653944342400] (rs.cpp:282) null pointer passed for argument "device"

Hi @abc2  Do you have any other RealSense programs already running when the RealSense Viewer is launched?

Hi @abc2  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


Add ExCoveralls code coverage tool

The build is failing because the repo is not in Coveralls yet.

@hidnasio can you rebase with master?

I've added the repo to coveralls.io

https://coveralls.io/github/san650/pretty_print_formatter

Can you add the badge to the readme?
[![Coverage Status](https://coveralls.io/repos/github/san650/pretty_print_formatter/badge.svg?branch=master)](https://coveralls.io/github/san650/pretty_print_formatter?branch=master)


When (in which version) do you provide the fix for #issue 11451?

When (in which version) do you provide the fix for #issue 11451?
...we can´t find it in 18.0.

The fix will be shipped with 18.1.


feat(protocol): need one real ZKP every N blocks

To reduce the prover cost of testnets, a new config realProofSkipSize is introduced, assuming it's 10, then we only need real ZKP every 10 blocks (block 10, 20, ....). For other blocks, the oracle provers will be considered as the real ZKP and will allow those blocks to be verified.
@davidtaikocha this may affect the prover code.

client matching PR:
https://github.com/taikoxyz/taiko-client/pull/198


TCGA lung cancer datasets

Could you please provide more details on generating TCGA datasets?

Hi @wk5475, I have the same question. I found some description in the supplementary materials of this paper:

TCGA Lung Cancer. Lung Adenocarcinoma (LUAD) and Lung Squamous Cell Carcinoma (LUSC) are two sub-type of cancers in the TCGA lung cancer dataset, with 534 LUAD and 512 LUSC slides, respectively. There are only slide-level labels available for this dataset. Compared to CAMELYON-16, tumor regions in tumor slides are signifi- cantly larger in this dataset.

LUAD and LUSC can be downloaded from the TCGA website I guess (however I am also not sure which files do they correspond to). The problem is the result in TCGA paper seems quite different from another work https://arxiv.org/pdf/2301.08125.pdf (This may be due to the different dataset split tho).


Request to Add a License to the Code Repository

First, thank you for sharing your work with the community!
I kindly ask if you could consider adding an open-source license to your repository for enabling others to reference your work.
Thank you for considering our request!

Thank you for your attention to my code. I have added the license.txt file. Thank you!

Great to hear, thank you for your prompt response and for adding the licence!


Formatting errors

There are apostrophe characters all over this page and in the code sample that break the formatting.

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

ID: 16e7478f-bfd9-607d-c5fa-2e9af28302ab
Version Independent ID: d8e1b97b-301d-9353-b279-a72354ffa8f2
Content: Global Discovery Service Sample (C#) (Common Data Service) - Power Apps
Content Source: powerapps-docs/developer/common-data-service/webapi/samples/global-discovery-service-csharp.md
Service: powerapps
GitHub Login: @JimDaly
Microsoft Alias: jdaly

@josiahvinson The page seems to be good now.


Add Source Execution Support (source or .) for UNIX side.

Description
Please provide a short description of what feature you're looking forward to
add into the existing repository below. Please include the story behind your
idea as well to give a better visualization of your idea.

Apparently, the Source Execution Support (source or .) is missing. The script can be used to stage a UNIX terminal which can be highly useful for all.
Expected Behavior
Please specify the expected behavior of your requested feature. Some great and
helpful pointers are your expected interface (e.g. command patterns, simple
sketches of the user interface, etc).

The script has an example of Sourcing Execution Support so that customer can manipulate it without needing to refer a manual.
Current Absent Behavior
Please specify the current behavior (e.g. workaround, blockage, etc).

Sourcing Execution Support is not demonstrated.
Attachment
Please drag and drop the necessary data files (e.g. screenshot, logs, etc)
below.

Not possible actually (or the existing methods are not 100% foolproof). Hence, will not implement because it can introduce ambiguity for higher level implementations.


Typescript version

Hello,
I would like to use your library (so useful) but I'm using Typescript instead of Javascript.
Is it possible to create a d.ts file (or another way) in order to use your library with Typescript ?
Thank you in advance for your answer.

Hi, if you send a PR for TypeScript support I'm happy to review and accept it. It should be relatively easy to write one.


Tests and support structures for necessary for new components in BFE

Unit and integration test structures for BFF including fixures and directory structure.  Could follow Mirador example and mirror structure determined by
Blocked by #93

@jermnelson I think this is done, yes?

Yes, closing now.


Backend search for snippets doesn't work

Did the following things:

Create a snippet template with following tag on title

xml
<tag name="sulu.search.field" role="title" />

Update CR database

app/console sulu:webspaces:init

Create a new Snippet "Google" in Backend and publish it.

Search on the general backend search for "Google" have no result for the snippets

I cannot reproduce that:

Seems to be an issue in an old version, fixed in the meantime, closing this issue.


Usage sub TS migration

Over the cap testing did not seem to work. There was an error in the logs and no record of the overage recorded for the shop

Code is good, just some additional nitpicky type handling stuff

This should be fixed btw


Show password at forms

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
Hello,
I have a problem with displaying a password in Forms. If you write some password to
<input type="password" class="form-control" id="exampleInputPassword1">
and than you click on Eye the password is displayed. This is ok. But when you lost focus (if you click somewhere else) and than you click back to input field the eye is not displayed anymore. What can you do is to clear password and start to write a password again.
Reduced test cases
You can try steps above on this page.
bootstrap forms
video:
https://github.com/twbs/bootstrap/assets/460694/84fa9b4b-da7a-432e-a203-352f051f0e82
What operating system(s) are you seeing the problem on?
Windows
What browser(s) are you seeing the problem on?
Microsoft Edge, Firefox, Samsung Internet
What version of Bootstrap are you using?
v5.2.3, v4

Thanks for reporting an issue @Predrag.
Unfortunately, it comes from Microsoft Edge itself. If you go to https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/password#try_it for example, you're going to see the same behavior (and Bootstrap is not used there).

@Predrag - I made a little library for Bootstrap which fixes that issue and adds the view/hide password toggle to all browsers: https://github.com/coliff/bootstrap-show-password-toggle

Thanks for reporting an issue @Predrag. Unfortunately, it comes from Microsoft Edge itself. If you go to https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/password#try_it for example, you're going to see the same behavior (and Bootstrap is not used there).

I see, than we can close this issue

@Predrag - I made a little library for Bootstrap which fixes that issue and adds the view/hide password toggle to all browsers: https://github.com/coliff/bootstrap-show-password-toggle

Thank you. I will use your library.


registerPortAttributesProvider({portRange:[5000,5000]}... doesn't match port 5000

Testing #183547
Based on the docs, it's hard to tell whether the range includes the lower and upper number (is the range inclusive of the bounds?). Also, is the array of two numbers a lower and upper range? I feel like I made that assumption myself. It'd be great to specify that in the docs.
Repro steps:

Create a basic extension that runs a server on command:

	let disposable = vscode.commands.registerCommand('testtest.helloWorld', () => {
		// The code you place here will be executed every time your command is executed
		// Display a message box to the user
		vscode.window.showInformationMessage('Hello World from test-ext!');

		const server = createServer((request: IncomingMessage, response: ServerResponse) => {
		response.end('Hello world!');
		});

		server.listen(port);

	});

and create a ports attribute provider.
2. Set portRange selector to be [5000,5000]
3. When port 5000 is auto-forwarded, it doesn't respect the providePortAttributes provider :bug:

Since other ranges in the API are [inclusive, exclusive] we will keep it consistent here. I'm adding a doc comment and a shorthand for having just one port:
		/**
		 * Specifying a port range will cause your provider to only be called for ports within the range.
		 * The start is inclusive and the end is exclusive.
		 */
		portRange?: [number, number] | number;

@andreamah do you set the port range to [5000,5000] or to just 5000?

For this, just 5000.

Yeah, I'm still seeing this. I'm running in an alpine devcontainer:
// This method is called when your extension is activated
// Your extension is activated the very first time the command is executed
export function activate(context: vscode.ExtensionContext) {
	const port = 5000;

	// Use the console to output diagnostic information (console.log) and errors (console.error)
	// This line of code will only be executed once when your extension is activated
	console.log('Congratulations, your extension "testtest" is now active!');

	// The command has been defined in the package.json file
	// Now provide the implementation of the command with registerCommand
	// The commandId parameter must match the command field in package.json

	vscode.workspace.registerPortAttributesProvider({portRange:port}, {providePortAttributes(
		port: number,
		pid: number | undefined,
		commandLine: string | undefined,
		token: vscode.CancellationToken
	): vscode.ProviderResult<vscode.PortAttributes> {
			return new vscode.PortAttributes(
				vscode.PortAutoForwardAction.Notify
			);
	}});

	let disposable = vscode.commands.registerCommand('testtest.helloWorld', () => {
		// The code you place here will be executed every time your command is executed

		const server = createServer((request: IncomingMessage, response: ServerResponse) => {
		response.end('Hello world!');
		});

		server.listen(port);

	});

	context.subscriptions.push(disposable);
}

Did I register correctly?

Let's debug this together next week.

Debugged offline. I had remote.portsAttributes set with port 5000 to silent.


Set lock_file for WORKSPACE-style maven_install repository

Without this defined, we'll continue to use coursier for dependency resolution and won't use Bazel's own downloader (and therefore will bypass any downloader.config).
bzlmod
This attribute was already defined in our MODULE.bazel configuration, and was missed here.

Closing this for now, as this needs some more thought.


Error Install

Your requirements could not be resolved to an installable set of packages.
Problem 1
- Root composer.json requires rzakhanov/translation * -> satisfiable by rzakhanov/translation[v1.0.0].
- rzakhanov/translation v1.0.0 requires laravel/framework ^8.0 -> found laravel/framework[v8.0.0, ..., v8.83.27] but it conflicts with your root composer.json require (^10.10).
Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.
You can also try re-running composer require with an explicit version constraint, e.g. "composer require rzakhanov/translation:*" to figure out if any version is installable, or "composer require rzakhanov/translation:^2.1" if you know which you need.
Installation failed, reverting ./composer.json and ./composer.lock to their original content.

Package requirements fixed and minimized.

Try again please.

it is not possible to try it out since version 1.0.0 has remained, the error has not disappeared

same issue


Cluster monitoring cannot be fully restored once disabled

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
enable monitoring at the cluster level and in the Default project on the local cluster
define notifier
enable notifier on one cluster level alert group
disable monitoring
delete apps in both System and Default projects that are installed when monitoring were enabled
enable monitoring at the cluster and Default project
Observation 1 below
delete and re-add notifier to a cluster alert group
Observation 2 below
Result:
Observation 1: the notifier definitions were preserved, but the alertmanager pod is not present.
Observation 2: alertmanager pod is now present.  however, the metric alert for the System project does not show graph when edited.  the metric alerts for the cluster would display "No metric graph data." under their Expression when edited.  The metric alert for the Defult project has graph when edited.
Other details that may be helpful:
Have looked at issues #19209 and #23889
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
v2.3.5
Installation option (single install/HA):
HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Rancher hosting cluster, RKE provisioned
Machine type (cloud/VM/metal) and specifications (CPU/memory):
VMWare instances
Kubernetes version (use kubectl version):

Client
Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.2", GitCommit:"59603c6e503c87169aea6106f57b9f242f64df89", GitTreeState:"clean", BuildDate:"2020-01-18T23:30:10Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.2", GitCommit:"59603c6e503c87169aea6106f57b9f242f64df89", GitTreeState:"clean", BuildDate:"2020-01-18T23:22:30Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"}

Server 1
Client Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.10+1.0.10.el7", GitCommit:"6e82748cdfbb6e84df1b464c489f3d27740a13cd", GitTreeState:"clean", BuildDate:"2020-01-29T23:55:47Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.2", GitCommit:"59603c6e503c87169aea6106f57b9f242f64df89", GitTreeState:"clean", BuildDate:"2020-01-18T23:22:30Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"}

server 2
Client Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.10+1.0.10.el7", GitCommit:"6e82748cdfbb6e84df1b464c489f3d27740a13cd", GitTreeState:"clean", BuildDate:"2020-01-29T23:55:47Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.2", GitCommit:"59603c6e503c87169aea6106f57b9f242f64df89", GitTreeState:"clean", BuildDate:"2020-01-18T23:22:30Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"}

Server 3
Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.8+1.0.2.el7", GitCommit:"a3f0ef943b59de56e1fd6088780e6b2c08486540", GitTreeState:"clean", BuildDate:"2019-10-18T17:19:03Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.2", GitCommit:"59603c6e503c87169aea6106f57b9f242f64df89", GitTreeState:"clean", BuildDate:"2020-01-18T23:22:30Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Server 1
Client:
 Version:           18.09.1-ol
 API version:       1.39
 Go version:        go1.10.8
 Git commit:        e32a1bd
 Built:             Thu Jun  6 14:55:42 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          18.09.1-ol
  API version:      1.39 (minimum version 1.12)
  Go version:       go1.10.8
  Git commit:       e32a1bd
  Built:            Thu Jun  6 14:47:41 2019
  OS/Arch:          linux/amd64
  Experimental:     false
  Default Registry: docker.io

Server 2
Client:
 Version:           18.09.1-ol
 API version:       1.39
 Go version:        go1.10.8
 Git commit:        e32a1bd
 Built:             Thu Jun  6 14:55:42 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          18.09.1-ol
  API version:      1.39 (minimum version 1.12)
  Go version:       go1.10.8
  Git commit:       e32a1bd
  Built:            Thu Jun  6 14:47:41 2019
  OS/Arch:          linux/amd64
  Experimental:     false
  Default Registry: docker.io

Server 3
Client: Docker Engine - Community
 Version:           19.03.1-ol
 API version:       1.40
 Go version:        go1.12.5
 Git commit:        ead9442
 Built:             Wed Sep 11 06:40:28 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.1-ol
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.12.5
  Git commit:       ead9442
  Built:            Wed Sep 11 06:38:43 2019
  OS/Arch:          linux/amd64
  Experimental:     false
  Default Registry: docker.io
 containerd:
  Version:          1.2.1
  GitCommit:
 runc:
  Version:          1.0.0-rc5+dev
  GitCommit:        4bb1fe4ace1a32d3676bb98f5d3b6a4e32bf6c58
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

Just upgraded RKE to v1.1.2, hosting k8s cluster to v1.17.6, Rachner to v2.4.4, monitoring on the hosting k8s cluster and its Default project to 0.1.0.  The monitoring api on the hosting cluster took quite a few minutes to become ready.  All the grafana links now point to a common grafana Home dashboard where individual dashboards can be selected.
On the other hand, a Racher launched k8s have the grafana links open individual dashboards.
In the context of Observation 2 above, the metrics alert in the System project would now show Expression is disabled. Go and enable monitoring when edited.  Clicking on the "enable monitoring" link would send us to the local System's Tools><Monitoring page, which presents two messages:
This is the system project which has all Kubernetes and Rancher system namespaces. Changes made to resources in the system project may harm the cluster.
If you want to collect custom metrics for pods in System Project, please enable cluster level monitoring directly.
The cluster level monitoring is already enabled.
In addition, the local etcd metrics alerts would not display a graph (no without "No metric graph data") when edited.  The first 2 alerts for node would display graph while the 3rd one (Node disk is running full within...) would not.   Rancher launched cluster would display the graphs when editing Metrics alerts.


next line

[excuse my Eng]
"\r\n" escape sequence is not working as expected.
For example. Telematic API. Messaging and Notifications:
The message string:
"2017-10-21 15:26\r\nDriver: Slawa, 56 y.o. (RU)"
HM-LOG contains two records:
15:27:30 - Android Meizu - CapabilityManager - Incoming telematics message for Messaging - ADcADkhlYWx0aGVkZXMgT05FAC0yMDE3LTEwLTIxIDE1OjI2DQpEcml2ZXI6IFNsYXdhLCA1NiB5Lm8uIChSVSk=.
15:27:30 - Android Meizu - Messaging - 0x00 0x37 0x01 - Received message: "2017-10-21 15:26
Driver: Slawa, 56 y.o. (RU)", Sender handle: Healthedes ONE
But the message on head-unit in one line:
MESSAGE RECEIVED FROM: HEALTHEDES ONE
2017-10-21 15:26 Driver: Slawa, 56 y.o. (RU)
Would be nice to have "next-line" feature for long structured messages.
Slava.
PS: The same behaviour with notifications.

This was resolved with the release of Car Emulator 3.0


ETCD metrics for clusters in Rancher

Is your feature request related to a problem? Please describe.
Extra steps required to scrape etcd metrics for k8s clusters  in Rancher (static endpoints).
Feature Description
We have been able to scrape the control plane components metrics, including etcd by having something like this.
    config:
      etcd:
        enabled: true
        staticEndpoint:
          url: https://localhost:2379
          insecureSkipVerify: true
          auth:
            type: mTLS
            mtls:
              secretName: nri-bundle-etcd-cert
              secretNamespace: newrelic
      scheduler:
        enabled: true
        staticEndpoint:
          url: http://localhost:10251
          insecureSkipVerify: true
          auth:
            type: bearer
      controllerManager:
        enabled: true
        staticEndpoint:
          url: http://localhost:10252
          insecureSkipVerify: true
          auth:
            type: bearer
      apiServer:
        enabled: true
        staticEndpoint:
          url: https://localhost:6443
          insecureSkipVerify: true
          auth:
            type: bearer

What I am looking for is to simplify the process to create the secret with the cert and key for etcd to work. Currently we manually create the secret, but we have several clusters which would require too many manual steps to get this done.
I would like to know if there is any alternative, like mounting a host path (/etc/kubernetes/ssl/) to avoid manual steps and make the installation process as simple as possible.
Describe Alternatives
Manually create and deploy the secret for every k8s cluster you have.
Priority
Really Want
Thank you very much!!!

We appreciate you taking the time to bring this to our attention. I add this to our backlog.

Moreover, I checked and as a workaround in our test-cluster there is also the secret etcd-tls in the kube-system namespace with a working certificate.
If you need to you can point to any secret in the cluster https://github.com/newrelic/nri-kubernetes/blob/e1cbd0007575f47296afe88534fc924c5e9bdd60/charts/newrelic-infrastructure/values.yaml#L287

Thanks @paologallinaharbur , sadly I think this is not consistent, some clusters do not create the secret.
As a workaround we ended up running a pod that will get the cert and key files from the etcd node and create the secret. This is based on rancher pushprox that runs an init container to do something similar https://github.com/rancher/charts/blob/dev-v2.7/charts/rancher-pushprox/100.0.1/templates/pushprox-clients.yaml#L86


Input bug

Add a comment
Comment
How to replicate:
1: use following code
while True:
suunta = input()
askeleet = int(input())
for i in range(askeleet):
pupu.liiku(suunta)
2: Enter input, for example "oikea" and then "3".
3: Reset.
4: run code again.
You will notice that input field didn't dissappear on reset, and that now if you enter the input, error will occur.

I looked into this a bit.
Bug seems to happen when you reset while input window is open.


[spaceship] Fixed deprecated call using Faraday

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
I wanted to debug why I can't upload videos to AppStoreConnect using Spaceship, while doing that I noticed a warning about a deprecated Faraday call for setting the proxy. This fixes #14007
Description
I replaced the now deprecated proxy(x) call with proxy = x. That seemed like an easy enough task as my first PR ;)

Wheeee 🎆 💥 🚅 🚀  👍


Tensorflow can't handle my shape forms

So I tried on many env to run PoolNet project with the same data used to have the above results.
I have all the files (filtered.geojson etc... plus the image from GDBX) so usually I am using the same polygons.
When I come to the point to start my VG16 without any data I have some warnings but ...still it works.
After that point ! Everything crash with this particular message error that you can see in the first file.
In the beginning, I thought the main reason is the Keras lib which has changed or even Tenserflow.
That being said...When I tried to downgrade my Keras version Tensorflow will not follow especially with python3.5.
ResultFile.txt
Anyway, with all the changes made.... I still have the same problem.
If the code had made it with good results.....Obviously, the problem is on my side so I made a file with all my lib in my env with the version of each one of it.
LibEnvLocal.txt
Can anyone help me with that, please ?!
Thank you

Hello friend,
I am trying to experiment with datasets provided, but I am quite confused as I cannot find original geojson file (as they called), can you help me to short it out?
I think dataset provided is a little bit messy.


README.md - Getting Started

In the README.md, section Getting started, the user is advised:
After cloning this repo and moving into the repo top directory, first deploy to a local SQLite database and then start the service up:
cds deploy --to sqlite
cds run

May I suggest adding npm install to the bash commands, so the required packages are installed?

Thanks, nice addition 👍


The download icon disappeared.

Until 2 weeks ago, the extension was working perfectly fine, but the download icon suddenly disappeared. I tried reinstalling the extension, but nothing changed. Can you fix this issue?

Thanks for flagging this. It looks like the changes to the ChatGPT web interface are causing the issue. I've updated the extension.
Please follow these steps:

Completely remove the extension:

Go to chrome://extensions/
Find our extension and click "Remove"
Restart your browser

Reinstall the extension:

Visit our GitHub page
Follow the installation instructions in the README file

After installation, refresh the ChatGPT page and check if the download icon appears

Important Note: Using a VPN or other browser extensions may cause unexpected issues. To accurately diagnose the problem, we recommend testing our extension in a clean browser profile (with other extensions disabled and VPN turned off).


:sparkles: enhance dataset.read_table(...) method with type normalisation

Motivation
Historically, we've been using function dataset["my_table"] to access table from a dataset. Recently, a new helper method dataset.read_table(reset_index: bool = False) has been added that lets us read the table with reset index which is significantly faster for large dimensional datasets.
Concept
We could add more functionality to read_table and make it de facto standard to read tables. These could be:

Retype all columns to "standard" types (e.g. uint8 -> int64, Float16 -> float64) and categorical to string type
underscore column names etc. (see .format method)

Should already be in snake case by the time we're talking tables

We think type standardising would remove some common footguns for data folk:

categorical → string
Float* -> Float64
Int* -> Int64


Add settings for Prometheus

Would be cool to roll this in! :)
Will give this commit a shot when I get a chance in my test environment.


INSTA_WORKSPACE_ROOT should also work with assert_snapshot! at runtime, for manual test binaries

I am evaluating insta with a project that uses Buck2 (and also a very happy user using it with Cargo as part of Jujutsu). When using a project with Buck2, Cargo is not used, and so tests for example are run "manually" by the build system itself after compilation with --cfg=test
Currently, I'm using inline snapshots, with a test like this:
#[cfg(test)]
mod tests {
    fn split_words(s: &str) -> Vec<&str> {
        s.split_whitespace().collect()
    }

    #[test]
    fn test_split_words() {
        let words = split_words("goodbye from the other side");
        insta::assert_snapshot!(format!("{:?}", words), @"");
    }
}

By default, when I compile this with buck2 build, I get an error like so:
error: environment variable `CARGO_MANIFEST_DIR` not defined at compile time
   --> src/qq-cli/commands/script.rs:148:9
    |
148 |         insta::assert_snapshot!(format!("{:?}", words), @"");
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Buck2 executes all tests from the root of the project, so luckily I can set CARGO_MANIFEST_DIR=. during the compilation of these tests which solves the problem. But it's a bit ugly I think.
While using Code Search to browse the source code to figure out how the macro worked, it looks like INSTA_WORKSPACE_ROOT also has a similar function. It would at least be more explanatory when I write INSTA_WORKSPACE_ROOT=. in the source code why it exists. However, it seems like it only works when you use cargo insta.
Alternatively, some other more general form or utility for assert_snapshot!() to work in the presence of non-Cargo systems would be nice.

Oh, apparently I need both CARGO_MANIFEST_DIR and INSTA_WORKSPACE_RULE for my tests to work. So my Buck2 rule for building insta tests looks like this (roughly):
rust_test(
    name = 'qq-cli-tests',
    srcs = glob(['**/*.rs']),
    deps = COMMON_DEPS,
    env = {
        'CARGO_MANIFEST_DIR': '.',
        'INSTA_WORKSPACE_ROOT': '.',
    }
)

Thanks for the issue.
If we checked INSTA_WORKSPACE_ROOT before CARGO_MANIFEST_DIR, would that solve everything here? I haven't worked with rust without cargo so I don't have much context on the standard approach for figuring things out such as the root of the workspace...

Should be fixed by #614, feel free to test on that branch


Fix permissions on license files

They come from upstream world-writable for some reason and end up directly copied into the end package--which then installs them as world-writable. This flags on some security checks.
Fixes #45 .

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@dopplershift The tests failed:

chmod: cannot access `COPYING*': No such file or directory

Missed the cd build above in the script. This should do it.


Custom regex implementation

Java's inbuilt regex doesn't support all current regex features.

Allow Builder pattern and direct string input

Allow use of an alternative to \ to avoid having to use double backslashes.

Additional ideas


T-Pose/No Animation Bug

My character will start sliding around with little to no animation in third person. Updated to the most recent version a day or two ago. Doesn't happen all the time but it's fairly common.

Was playing uncapped and I didn't realize MBO would break over a certain FPS. Capping my FPS fixed this. Disregard.


Add PlaceholderAPI support to meta (prefix/suffix)

Description
I feel like there should be a way to add placeholders from PlaceholderAPI into the prefixes of the groups.
I just want to be able to do this so it's easy for me and I've seen some people ask for this before, hopefully this gains support and I hope you have a good day!
Proposed Behaviour
Like if I were to do /lp group farmer meta setprefix 1 "%oraxen_farmer_tag%", it would use the placeholder and it would return something like this:

Note: This uses a texture pack that allows me to edit characters (Oraxen specifically).
Also, if I were to add something like %server_tps% that would return the 1, 5, and 15 tps times like it should if PlaceholderAPI was supported in prefix, anything really.
Extra Details
No response

Placeholder support inside the prefix/suffix must be handled by the plugin fetching them (tab/chat/scoreboard etc plugins), LuckPerms performs no stylistic rendering on them, it will provide the prefix/suffix "as-is" exactly as stored, the same ideas outlined in here for style format also apply for placeholder resolution or any other kind of resolving.


The Windows GetVersionEx API is deprecated. Replace with Version Helper functions

The Win32 GetVersionEx function has been deprecated. It has been replaced with a series of Version Helper functions.
Note that these functions are implemented as inline functions in a header file. They'll need to be exported from the VM.

Please note OSVERSIONINFO class is now deprecated and should not be used. See SystemMetrics for examples of what to do instead.


Check for existence of common.app_tree before attempting to remove widget's reference

When launching wxGlade from the command line, it is possible that the app tree gets "cleaned up" twice.
This patch fixes a problem on my own runs.
I must confess am not 100% sure about the cause of this problem, but this makes wxGlade work for me.
Please let me know if there is any other information I can provide, if this solution is not acceptable or incomplete.

Thanks.
It's not cleared, I think, but not built at all when not having a GUI.
I would guess that removing would not even be required, but the optimization is probably not worth it....


After using quic, the transmission slows down significantly

Without quic，caddy takes 200-300ms to download 3.4MB.
After using quic,it takes 982ms-1.3s to download 3.4MB.

This is an issue for https://github.com/mholt/caddy


Repeated toggling of user role doesn't update

I tried to toggle user status a few time via /user/toggle-user-role/
It seems to toggle the first time, but then stops working the 2nd time onwards.

Might be the bug origin of #173.
Check if you are you updating the local (i.e cookie's) userId after toggling?

Resolved in #169


Will there be a square method?

Currently there is only squareRoot which is 1/2. Will there be a square method in the future so I can pass in 1/x where x is an Integer?
Right now exponentiatedBy only accepts Integers so I cannot pass (1/5) or 0.2 which is also the same as the square of 1/5.
Another alternative is using vanilla Math.pow(x, (1/5)).
Thanks!

Are you thinking of an nthroot method? So the square root would be x.nthroot(2) and the cube root would be x.nthroot(3) etc.
I have considered it before, but nothing came of it, and I am unlikely to add it here in the forseeable future. There may be an implementation of it somewhere in the issues or pull requests for this library or decimal.js.
The decimal.js power operation accepts exponents that are not integers.


Merge development branch into master

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Coverage decreased (-0.3%) to 60.2% when pulling 7e58cf05a00301893c228a3ca83c7db97a275580 on development into a150f852fb544c98d8203e4f3bc7bf50146b034f on master.


Adds sh action

Addresses #619

Hello @JayjeetAtGithub! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file cli/popper/gha.py:

Line 549:14: W291 trailing whitespace
Line 561:1: W293 blank line contains whitespace

@ivotron You may have a look.

@ivotron Please review !

@ivotron Please check ! Thanks

thanks @JayjeetAtGithub . Do you think you can add the changes to the docs in this PR as well? What I mentioned above.

@ivotron Updated the docs.

thanks a lot @JayjeetAtGithub !


Make all Elastic Stack products have same version

Use one variable, elastic_stack:version to ensure that all ES products (e.g. Kibana + Elasticsearch) have the same version.

:+1:


Does not support Sidekiq 4.2?

Didnt see it in the supported versions

We did not tried it with Sidekiq 4.2 yet. There are no hard dependency - so it's possible to install with Sidekiq 4.2 if you want. I'm gonna add tests against Sidekiq 4.2 this week. Thanks for reminder.

Just ran tests suit against sidekiq 4.2 - no failures. So it's safe to use with sidekiq 4.2 :D


Add bound variable numbering

This adds the numbering of bound variables. It introduces some mutable state in the PrettyPrinter (which I also changed into a trait).
Not sure if this is the best way to do it since the numbering does not always work as expected (e.g. see val exp14 in  test("Virtualized Constructs") where x_0 is followed by y_4 (instead of y_1).
@LPTK if you have a better idea of how to handle this, let me know :)

Hah, mutation effects are tricky!
Note how some methods like byname take their argument... by name (def byName(arg: => Rep)), so if you execute the resulting offset => string function several times you'll execute that argument several times.
The while construct is encoded as a call to squid.lib.While which takes by-name arguments, and you have duplicated work here and here (among others). So your counter is going to increment too many times. Making these values lazy fixes the problem. Evaluating the by-name arguments before closing over them with offset => will also work.
In general, since you chose to encode the current state of indentation by making Rep a function, avoiding mutable state, wouldn't it make sense and be cleaner to also add the current variable id as a parameter and return of that function, making it Rep = (Int,Int) => (String,Int)? Granted, that would complexify the code, unless you make Rep a monad and use for comprehensions :^)

Cleanup of old PRs ;)


Include Dockerfile for automated image uploads

PR Summary:
Related to mbuild PR #728, this provides a foyer Docker image based on
alpine linux, with all dependencies installed.
The 8888 port is exposed for jupyter notebook support.
PR Checklist

[ ] Includes appropriate unit test(s)
[ ] Appropriate docstring(s) are added/updated
[ ] Code is (approximately) PEP8 compliant
[x] Issue(s) raised/addressed?
closes #333  when @umesh-timalsina adds the azp files to this pr.

@umesh-timalsina feel free to add your files here if you want. Or we can do that in a separate pr.

Apparently, you cannot connect to a service when the PR is from a fork's branch (makes sense). I tested it, the Docker build happened, but the image was never pushed because of failure to login (https://dev.azure.com/mosdef/mosdef/_build/results?buildId=1332&view=logs&j=82b92b65-e756-5bc9-971a-f18964e0bd4b&t=a9e5d888-3cca-5200-6568-6e35b94289a9). However, this completes the PR and once merged, a docker image foyer:latest is pushed to https://hub.docker.com/repository/docker/mosdef/foyer on every commit to master.


Example code for MDDialog is missing a class

The example code for MDDialog seems to be missing a definition of CheckBoxRightWidget (contained in dialog.py, Line 342), or CheckBoxLeftWidget (contained in https://kivymd.readthedocs.io/en/latest/components/dialog/index.html ).

@michaelu123 This was fixed a long time ago in the master branch.


Clone HadoopConf to avoid cross usage of tags while parsing the xml

This code is to fix the issue 581.
Added unit tests and also verified manually by using below code
import scala.collection.JavaConverters._
import scala.collection.mutable
val jobGroudId_ages = mutable.Set[Long]()

val threads_ages = (1001 to 1010).map { i =>
  new Thread {
    override def run() {
      sc.setJobGroup(s"$i", s"$i")
      val df = spark.read.option("rowTag", "person").format("xml").load("file:/Users/XXXX/spark-xml/src/test/resources/ages.xml")
      if(df.schema.fields.isEmpty) {
        println(s"found repro for the ages run $i **********************")
        jobGroudId_ages.add(i)
      }
    }
  }
}

import scala.collection.JavaConverters._
import scala.collection.mutable
val jobGroudId_books = mutable.Set[Long]()

val threads = (1 to 10).map { i =>
  new Thread {
    override def run() {
      sc.setJobGroup(s"$i", s"$i")
      val df = spark.read.option("rowTag", "book").format("xml").load("file:/Users/XXXX/spark-xml/src/test/resources/books.xml")
      if(df.schema.fields.isEmpty) {
        println(s"found repro for the book run $i **********************")
        jobGroudId_books.add(i)
      }
    }
  }
}

threads_ages.foreach(_.start())
threads.foreach(_.start())
threads_ages.foreach(_.join())
threads.foreach(_.join())
println(s" jobGroudId_books is ${jobGroudId_books.size} ")
println(s" jobGroudId_ages is ${jobGroudId_ages.size} ")

Before fix

After fix

cc @srowen FYI if you find some time to take a look 🙏

cc @srowen FYI if you find some time to take a look 🙏

@srowen just out of curiosity, when do we roughly plan to have the next release?

not super critical but I think it's good to have one ... could we make a release maybe? I will take a look too around next week if you couldn't find to take a look 👍

OK I can do it tomorrow I think

Done, 0.15.0 is released with this change

Thanks!!!!


Uncaught Error: Graph container element not found.

morris.min.js?compile=false:6

anyone know why using a separate js file with the Morris chart info you get the 'graph container element not found'?

I'll check into this, the plugin might be outdated or something.

It is an issue with the way that it is loading the divs and jscript.
Check this
https://github.com/morrisjs/morris.js/issues/137

#39 removed Morris JS and I'm using Chart.js now.

I have error js vendors.bundle.js:78353 Uncaught Error:

Uncaught Error: Graph container element not found

http://stackoverflow.com/questions/18176755/morris-js-graph-container-element-not-found


Namespace collision with System.Reactive.Linq

Is your feature request related to a problem? Please describe.

I was trying to create a Count observable from a SourceCache which did not provide any results. The problem was that I had the namespace System.Reactive.Linq, but not DynamicData.Aggregation which caused the wrong extension method to be picked.
Describe the solution you'd like

I added a DynamicData.Aggregation using statement.
Describe alternatives you've considered

Describe suggestions on how to achieve the feature

It would be nice to include a warning note about using the correct namespace in the aggregation section of the README.md.
Additional context

Happy for a PR to be made for addition to the readme documentation. It'd be a nice easy one to do.

new to reactiveui/dynamicdata, this one caught me out also doing canExecute checks. easy to miss if you have already imported System.Reactive.Linq as it compiles fine, but doesn't work.


Logging metadata improvements

(#83)

Log the job ID rather than the whole struct when a job is dispatched
Log the queue name when a job is dequeued
Move queue name into metadata for worker starting notice

These changes are now available in 1.3.2


stanek.js: log automatically starts up despite having manually closed

Minor issue, but the log of stanek.js automatically starts up despite having manually closed, since the information it provides isn't always useful, I think it should be made so that its log doesn't start up automatically.

Run with --no-tail-logs to manage what gets tailed yourself
On Sun, Nov 24, 2024, 7:01 a.m. mikeymikey64 @.***>
wrote:

Minor issue, but the log of stanek.js automatically starts up despite
having manually closed, since the information it provides isn't always
useful, I think it should be made so that its log doesn't start up
automatically.
—
Reply to this email directly, view it on GitHub
https://github.com/alainbryden/bitburner-scripts/issues/444, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AARN33MBLOODYAUXQVOWFFL2CGWXDAVCNFSM6AAAAABSMDKYLKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGY4DONJWHA4TSMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

You already have two options for dealing with this:

Run autopilot.js with --no-tail-logs to manage what gets tailed yourself (affects all scripts)
Create a file "stanek.js.config.txt" and permanently set the --no-tail argument as an override to never tail this script again. The contents of the txt file would look like this: { 'no-tail': true }

The reason Stanek keeps its tail window open is because while it is running, daemon.js is crippled (home RAM is reserved entirely for charging stanek) so it needs to be made obvious (and easily killable) for anyone who is otherwise left wondering why daemon.js isn't running any batching scripts on home.


Headings do not align with the content

  Line   Column   Type                Message                                                                       Rule ID

     1        1   warning   Definition for rule 'react/no-unused-vars' was not found                      react/no-unused-vars
    15        5   error     Variables within the same declaration block should be sorted alphabetically   sort-vars
    17        5   error     Variables within the same declaration block should be sorted alphabetically   sort-vars
    18        5   error     Variables within the same declaration block should be sorted alphabetically   sort-vars
    35        9   error     Variables within the same declaration block should be sorted alphabetically   sort-vars
    35        9   error     Variable 'appendPaths' should not be initialized on declaration               init-declarations

The code:
table.setHeading([`Line`, `Column`, `Type`, `Message`, `Rule ID`]);
table.setHeadingAlignLeft();
table.addRow([
    message.line || 0,
    message.column || 0,
    messageType,
    message.message.replace(/\.$/, ``),
    chalk.gray(message.ruleId || ``)
]);

I'd expect headings to align with the beginning of the text in each column.

The issue is a lot clearer when borders are added:
.--------------------------------------------------------------------------------------------------------------------------------------------------.
| Line | Column | Type              | Message                                                                     | Rule ID                        |
|------|--------|-------------------|-----------------------------------------------------------------------------|--------------------------------|
|    1 |      1 | warning | Definition for rule 'react/no-unused-vars' was not found                    | react/no-unused-vars |
|   15 |      5 | error   | Variables within the same declaration block should be sorted alphabetically | sort-vars            |
|   17 |      5 | error   | Variables within the same declaration block should be sorted alphabetically | sort-vars            |
|   18 |      5 | error   | Variables within the same declaration block should be sorted alphabetically | sort-vars            |
|   35 |      9 | error   | Variables within the same declaration block should be sorted alphabetically | sort-vars            |

@gajus Hrm.. will have to look into this one.

It appears that it is colours that cause width discrepancy.

Specifically, I am using https://www.npmjs.com/package/chalk.

@gajus I know you've re-implemented this library, but did you ever find a solution for dealing with color encoded strings?

https://www.npmjs.com/package/string-length

Wow, where has this module been all my life? Thank you @gajus !!

Was about to post this issue. Any fix?

I did a quick modification of ascii-table to meet my needs.

I've enclosed the code for the example above (main.js.txt) and the updated ascii-table.js (ascii-table.js.txt)
main.js.txt
ascii-table.js.txt


Write some unit tests

This would be very helpful for the index.js file that has all the logic.
Preferably with Jest and produces a coverage report.

I'll get Jest running with a coverage report, so anyone can start testing. Then, I'll see if I can test the index.js file you indicated.

Thanks @camdenbrown ! I merged your pr #8 . Now we just need some tests from anybody who wants to give it a shot :100:


Query in a run criteria system doesn't work

Bevy version
Bevy main 38feddb87850424df3a0b08bae8dc32c57004798
Operating system & version
Windows 10
What you did
struct Foo;

fn number_of_entities_system(mut commands: Commands, query: Query<&Foo>) {
    println!("num entities from dummy: {}", query.iter().count());
    commands.spawn_bundle((Foo,));
}

fn number_of_entities_critiera(query: Query<&Foo>) -> ShouldRun {
    println!("num entities from criteria: {}", query.iter().count());
    ShouldRun::Yes
}

App::build()
    .add_plugins(MinimalPlugins)
    .add_system(
        number_of_entities_system
            .system()
            .with_run_criteria(number_of_entities_critiera.system()),
    )
    .run();

What you expected to happen
Both systems print the same number of entities
What actually happened
Run criteria reports zero entities, system reports correct number.
Additional information
Console log on my machine:
num entities from dummy: 1695
num entities from criteria: 0
num entities from dummy: 1696
num entities from criteria: 0
num entities from dummy: 1697
num entities from criteria: 0
num entities from dummy: 1698

I'm guessing this is probably caused by SystemStage not calling System::new_archetype() on its run criteria systems.

Pulling in @Ratysz.

Sounds plausible. I'm not sure when I'll have the time to take a proper look.


Storing max pooling indices in MXNet

I find the MShadow side implementation of max pooling used the == logic to find the pooling indices. Is it possible to explicitly store the indices?

is there any reason why the other policy is preferred?

I come to this question when I try to implement the ROIPooling layer used in Fast and Faster RCNN (https://github.com/rbgirshick/caffe-fast-rcnn/blob/fast-rcnn/src/caffe/layers/roi_pooling_layer.cu). The backward operation of ROIPooling may be easier to implement if we know the argmax indices. Also, when we pool over a large region, e.g global pooling, the current way of getting the pooled indices may not be efficient.

You can store the indexes as an second output, but remember to make it invisible.
Please refer to BatchNorm layer for example.

Thanks very much, I'll try on that.


flake8 does not check labcontrol directory

https://github.com/biocore/LabControl/blob/1427ad163682ff11b1086e0e3b1010ba1a2166ee/.travis.yml#L100
This line has labman instead of labcontrol and flake8 does not throw non-zero exit codes if it is passed directories/files that do not exists. So this should be changed to labcontrol so the python code is actually linted.

This is a good catch, thanks!


Add support for Nsis version 3.08.

Don't really know what I am doing.
Need Nsis version 3.08 for added support for !uninstfinalize
Please help.

Plan for Commit ?

Committing 🤞

We will look into it this week. This will most likely be in a version 3 of the task to ensure we don't break existing pipelines..

We will look into it this week. This will most likely be in a version 3 of the task to ensure we don't break existing pipelines..

Super cool, if u need help testing i am glad to lend a hand :)

@JacobIpsen I've merged and released the new version 3.0.0 of the task (ps: don't forget to change version 3 on your yaml pipeline) .
I don't have the path issue you mentioned, but I do have something else that doesn't seem to work well. I'll investigate further tomorrow. In the meantime give it a spin and let me know what you think.

Ok the arguments are passed ok now ( This is related to the issue #18 )


Disable the vscode SQL notebook controller since it breaks query editor.

Fixes #22044
When I added a listener for query completion in the SQL kernel, it broke the rest of the editors listening on the MSSQL provider. Disabling this controller for now.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.02%) to 41.955%

Files with Coverage Reduction
New Missed Lines
%

extensions/sql-database-projects/src/common/httpClient.ts
1
66.18%

extensions/sql-database-projects/src/dialogs/publishDatabaseDialog.ts
2
52.88%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
28747

Relevant Lines:
63627

💛  - Coveralls


Implement converters as classes

Instead of having the to_chatml and from_chatml methods as part of different modules, we should probably define a converter class with those as abstract methods and have specific implementations.
I don't feel very strongly about this, hence opening an issue instead of a PR to solicit more feedback.

https://github.com/deployradiant/pychatml/pull/17 resolves this issue.


jenkins worker is running out of space

Jenkins worker was running out of space because docker images were not
pruned. I decided to put all space reclaiming into a single script that
is run before each e2e stage. So nix GC is no longer run as a service
on worker nodes. The advantage is that we can synchronize GC with running
jobs that way.

bors merge


nativescript-imagepicker is not supported for android

I received this message from the CLI upon installation of the plugin. I tried to use it in an app, and also tried using a demo project but this didn't work either. Any ideas why this is happening? This is the only plugin I've had an issue with.

I reinstalled the plugin and it seemed to work, even though this error was still displayed. The only issue I have now is #8 which is to do with fileUri being invalid.


Refactor/bytereader

Closes #54
Create ByteReader class for better slicing

General comment:

Lets make the folder structure consistent, meaning that it would be better if the folders Interfaces and Utils are uppercase as-well
We can remove the comment text in the Codec interface stating:

!!!IMPORTANT!!! Interfaces are not yet supported in AssemblyScript.
 * The interface will be enforced only by the Editor and not the Compiler!

AS supports interfaces now and we already upgraded to that version of AS (its just a leftover)

We will need inline docs for the newly exported BytesReader

I think that the example project should have new examples for the BytesReader as-well

It does have tho?


Tiles are not loading

Whatever i do the tiles won't load, i had this problem earlier and somehow fixed it on one server by fiddling with symlinks and stuff but still i don't know what is the cause of this. Now i want to merge my maps again into one ui and i have this problem again. Every php file is accessible also im using the dynmap_world.json instead of mysql. In dynmap i have my tilespath set to another disk and webpath to my website path in var/www i have a folder for each server in there and i keep the update wepath in dynmap on because sometimes when it's off the files in standalone folder won't update either. But dynmap is working as normal and in live atlas i can only see the players when they join, no tiles.

PS: in console there's only this one error
index.a82d2095.js:20 Uncaught (in promise) Error: Request returned invalid json
at me.fetchJSON (index.a82d2095.js:20:7057)
at async Proxy.getMarkerSets (index.a82d2095.js:20:33545)
at async Proxy.populateWorld (index.a82d2095.js:20:34563)


feature Support lots of scalar function and fix some bug

Proposed changes

function interfaces that can search the matched signature, say ComputeSignature. It's equal to the Function.CompareMode.

IdenticalSignature: equal to Function.CompareMode.IS_IDENTICAL
NullOrIdenticalSignature: equal to Function.CompareMode.IS_INDISTINGUISHABLE
ImplicitlyCastableSignature: equal to Function.CompareMode.IS_SUPERTYPE_OF
ExplicitlyCastableSignature: equal to Function.CompareMode.IS_NONSTRICT_SUPERTYPE_OF

generate lots of scalar functions
bug-fix: disassemble avg function compute wrong result because the wrong input type, the AggregateParam.inputTypesBeforeDissemble is use to save the origin input type and pass to backend to find the correct global aggregate function.
bug-fix: subquery with OneRowRelation will crash because wrong nullable property

Note: currently no more unit test/regression test for the scalar functions, I will add the test until migrate aggregate functions for unified processing.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[x] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 38.42 seconds
load time: 558 seconds
storage size:<PHONE_NUMBER>2 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221028104344_clickbench_pr_35329.html

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 38.49 seconds
load time: 563 seconds
storage size:<PHONE_NUMBER>3 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221028111426_clickbench_pr_35355.html

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 40.29 seconds
load time: 564 seconds
storage size:<PHONE_NUMBER>8 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221028120820_clickbench_pr_35396.html


How to create query engine with webhook response?

Hi,
I want to create a query engine using the json from webhook response from llama parse. is this possible? it's missing some attributes like embedding, etc.
    index = VectorStoreIndex.from_documents(document_objects)

    # create a query engine for the index
    query_engine = index.as_query_engine()

@AyushParikh you would have to take the resulting text, and embed them. Thats nearly what your code sample is doing
from llama_index.core import Document

# take the text from the webhook
documents = [Document(text=text, ...), ...]

index = VectorStoreIndex.from_documents(documents)

# create a query engine for the index
query_engine = index.as_query_engine()


shadowsocks-libev: fix shared libraries

Motivation for this change
Fix errors about shared libraries. Seems caused by #63185
ss-local: error while loading shared libraries: libipset.so: cannot open shared object file: No such file or directory

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
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 nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)

34693840 (33.09 MiB) -> 34882760 (33.27 MiB)

[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Thanks a lot! I accidentally removed that line.


shorten font property

Shortens the font styling from font-size, font-weight, font-family to font.
Also, I've removed the font fallback declaration because the backup font-family will be inherited from a global tag.
E.g.
body {font-family: helvetica, Arial, sans-serif}.

/* If 'roboto' can't be loaded.
The class will inherit the font-family from the body to become */
.draggable-header-view {font-family: helvetica, Arial, sans-serif}.

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

[ ] Bugfix
[ ] Feature
[ ] Code style update
[x] Refactor
[ ] Build-related changes
[ ] Other, please describe:

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

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[ ] New/updated tests are included

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:

because the backup font-family will be inherited from a global tag.

It's not how CSS works. The computed font-family will be Roboto no matter it's available or not. Further fallback is provided by the browsers.
Honestly I think current code is fine enough.

I omitted 'inherit'.
font: 300 1.8em Arial, inherit;
Check it ✅

inherit is a CSS keyword, not <family-name>s or <generic-family>s which are valid inside a font-family property value. It can only be used as the whole value of a property. The modified version still won't work as you expected, it's even not a valid value...

The font-family is inherited from ancestor element because you are making the whole font property invalid, not because Roboto isn't available.

confirmed.


NULL pointer checks

Not 100% sure these are necessary.

I am closing this one since we switched to using a wrapper for malloc/calloc. Anyways I'd like to thank you for your contribution!


Got undefined is not an object while hiding the toolbar

When set
<Toolbar hidden={false}/>

I got

I suspect it's because hide() was called during show() or vice-versa.

Getting the same error.
@kirill-konshin did you get a fix for this?

The above PR should fix it.

Why this is still not fixed after half a year?

PING


Fixed consistency in language strings

Description
Referencing #6341
Updated strings in Language/en to improve consistency.
Checklist:

[X] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[X] Conforms to style guide

Will work on the failed tests once I understand how to run the database tests locally...

I think this is not a bug fix, so it is better to go to 4.3 branch.

Will work on the failed tests once I understand how to run the database tests locally...

The database tests work by default, because the default config is to use SQLite3.

@tangix The failures all appear to be expected language string changes. If I were you I would pull up the Actions log for one of the tests (I.e. https://github.com/codeigniter4/CodeIgniter4/runs/7966690208?check_suite_focus=true) and use that to track down the strings that need changing.
Thanks for taking this on! Seeing that we previously had strings like this makes me even more grateful:

Invalid "background" color: "Background".

@kenjis I maintain that this is not a breaking change. We made the decision a while ago (#4068) that the content of language strings is not covered. If developers are relying on language string output they should be checking it against the function call, not the content.

@MGatner I did not know it.
It is unreasonable to require users to read and remember all past changelogs.
We need the docs about backwards compatibility.

Agreed. We had a contributor comment to that effect, not sure why it was ignored: https://github.com/codeigniter4/CodeIgniter4/pull/4068#pullrequestreview-562123708

I think this is not a bug fix, so it is better to go to 4.3 branch. See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/workflow.md#if-you-sent-to-the-wrong-branch

@kenjis
Let me know if I should create a new PR for the 4.3 branch.

@MGatner I sent  #6418

@tangix As MGatner says this is not a breaking change. So we can merge into develop.
But I'm not sure which branch (develop or 4.3) is better.

@kenjis OK, I’ll sit on the sideline watching you sort it out.
Let me know what needs to be done if anything.

@MGatner This is not a breaking change, but the changes are many.
These changes may affect the translations.
So I think it is better to go to 4.3 branch.
What do you think?

Oh yes that's absolutely fine! I was referring to the breaking change label, not the branch. This should be a pretty easy one to switch over. Hopefully 4.3 is coming out soon anyways

@tangix Could you create a new PR for the 4.3 branch?

@tangix One more thing.
Could you add a short description in changelog Changes section:
https://github.com/codeigniter4/CodeIgniter4/blob/4.3/user_guide_src/source/changelogs/v4.3.0.rst#changes
Something like this is sufficient:

Updated strings in Language/en to improve consistency.

Hmmmm, not a git-ninja like you guys but I tried to follow the instructions.
Problem is, I cannot see the 4.3 branch in my fork of the repo, even after syncing. Is there a way to solve this before I give up and nuke my fork from orbit?

$ git remote add upstream git@github.com:codeigniter4/CodeIgniter4.git
$ git fetch upstream
$ git switch 4.3
$ git push origin 4.3

$ git remote add upstream git@github.com:codeigniter4/CodeIgniter4.git
$ git fetch upstream
$ git switch 4.3
$ git push origin 4.3

OK.... thanks for all the help!
I think all looks good but I leave that for you to go through and verify.

Can you rebase to resolve the conflict?
See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/workflow.md#updating-your-branch

Obviously you ran merge not rebase, because you added a merge commit.
rebase never adds a commit.
But no problem. All you need to do is still git rebase.
$ git fetch upstream
$ git rebase upstream/4.3

...Resolve conflict...
$ git add -u
$ git rebase --continue

Rebase is complete.
And force push:
$ git push --force-with-lease origin lang-consistency-fixes

@kenjis I really appreciate your efforts but seems like I am a complete idiot when it comes to using github (we are using git on a daily bases for multiple projects but rebase is something I have never needed to do).
Don't know if it's GitKraken or something else causing these issues.
As said before - I am perfectly fine re-forking and doing all again. This might save us both a lot of time and efforts.
msa@verdandi CodeIgniter4 % git fetch upstream
msa@verdandi CodeIgniter4 % git rebase upstream/4.3
Auto-merging user_guide_src/source/changelogs/v4.3.0.rst
CONFLICT (content): Merge conflict in user_guide_src/source/changelogs/v4.3.0.rst
error: could not apply 7c9bf6613... Added info to changelog
Resolve all conflicts manually, mark them as resolved with
"git add/rm <conflicted_files>", then run "git rebase --continue".
You can instead skip this commit: run "git rebase --skip".
To abort and get back to the state before "git rebase", run "git rebase --abort".
Could not apply 7c9bf6613... Added info to changelog
msa@verdandi CodeIgniter4 % git rebase --continue
[detached HEAD d8e098298] Added info to changelog
 1 file changed, 1 insertion(+)
dropping 9a7039387d6a67bc2173eb3028d3195502165607 Fixed additional tests that failed. -- patch contents already upstream
dropping 8f3cd24c10d090e932eef4181232a2d27189369b Updated failed tests due to language strings changed. -- patch contents already upstream
dropping 5be13c5d8f160c21d00b6ce7eda66a8e644e1ea5 Merged from old branch -- patch contents already upstream
Auto-merging user_guide_src/source/changelogs/v4.3.0.rst
CONFLICT (content): Merge conflict in user_guide_src/source/changelogs/v4.3.0.rst
error: could not apply c63c591b1... Added info to changelog
Resolve all conflicts manually, mark them as resolved with
"git add/rm <conflicted_files>", then run "git rebase --continue".
You can instead skip this commit: run "git rebase --skip".
To abort and get back to the state before "git rebase", run "git rebase --abort".
Could not apply c63c591b1... Added info to changelog
msa@verdandi CodeIgniter4 % git add -u
msa@verdandi CodeIgniter4 % git rebase --continue
[detached HEAD 63f4dcf74] Added info to changelog
 1 file changed, 3 insertions(+)
Successfully rebased and updated refs/heads/lang-consistency-fixes.
msa@verdandi CodeIgniter4 % git push
To https://github.com/tangix/CodeIgniter4.git
 ! [rejected]            lang-consistency-fixes -> lang-consistency-fixes (non-fast-forward)
error: failed to push some refs to 'https://github.com/tangix/CodeIgniter4.git'
hint: Updates were rejected because the tip of your current branch is behind
hint: its remote counterpart. Integrate the remote changes (e.g.
hint: 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.
msa@verdandi CodeIgniter4 % git pull
hint: Pulling without specifying how to reconcile divergent branches is
hint: discouraged. You can squelch this message by running one of the following
hint: commands sometime before your next pull:
hint:
hint:   git config pull.rebase false  # merge (the default strategy)
hint:   git config pull.rebase true   # rebase
hint:   git config pull.ff only       # fast-forward only
hint:
hint: You can replace "git config" with "git config --global" to set a default
hint: preference for all repositories. You can also pass --rebase, --no-rebase,
hint: or --ff-only on the command line to override the configured default per
hint: invocation.
Auto-merging user_guide_src/source/changelogs/v4.3.0.rst
CONFLICT (content): Merge conflict in user_guide_src/source/changelogs/v4.3.0.rst
Automatic merge failed; fix conflicts and then commit the result.

No problem. If you don't use rebase, you will be in trouble with open source contributions like this.
Because the upstream development is always progressing, and conflicts may happen anywhere.
And it seems you are getting close to finish the rebase.
rebase modifies previous commits. So it is a bit dangerous operation, but your PR branch is yours.
If you break it completely, no harm in the upstream repository.
After running git rebase upstream/4.3, you need to resolve the conflict.
After resolving the conflict, you will run git add -u and git rebase --continue.
It seems you did them.
rebase modifies previous commits, so normal git push won't work.
You need to force push: git push --force-with-lease origin lang-consistency-fixes
And I recommend you don't use git pull. It is harmful, and is not needed here.


Allow thresholds to be empty

The default values for the thresholds are empty strings in the GUI and not zeros. Zeros have a different meaning, they enforce having 0 failed tests, while an empty string means "do not check this threshold".

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.

Oh, sorry that we got that wrong. I'm fine with changing the signatures to Integer to allow empty (null) values. But we can't change the defaults because that would break compatibility for people relying on the zeros. Can you change the defaults back?

Yes it will break the backward compat, but in this case this is a good thing. Let me try to convince you :) The contribution guide states that "Every option should have the same defaults as the UI", which is not the case here. I personally spent quite some time trying to figure out why the plugin was behaving so strangely in a generated job. I think it might be better to endure the pain now and make people change their code than to continue to confuse the newcomers, of which there will be much more :)
Let me also say that I understand the importance of not breaking the existing code, but I see that the quality of the product and the codebase still takes first priority here (judging by the deprecation policy for example), which is why I ask you to consider breaking the compatibility in this instance.
Still, I am not opposed to leaving the defaults in place (we're building the Job DSL from our own fork anyway), but if you think that we absolutely should not change them, maybe we could do something clever and think of a way to deprecate the current defaults and introduce a way to switch to the new ones, allowing people to use them?

I will not merge an incompatible change. The docs clearly state that the default value is zero. It's a bit unfortunate in this case that this does not match the plugin's default, but potentially breaking user code is even worse. Restore the default values or I will close the PR.

I still believe that changing the defaults would be better for the product, but it is your call :) I've changed them back. Thanks for reviewing the pull request!

Please open a JIRA issue for changing the defaults. There was a similar issue about changing defaults for the Git plugin (JENKINS-33482). If we find a viable way to warn users about changing defaults (e.g. a warning in the log, failing the seed job, etc), we can change the defaults.

Sure, no problem: JENKINS-33554


Generator makes consts generated from a swagger enum as internal to a package

Problem statement
I have moved from go-swagger 0.7.2 to go-swagger 0.7.4. My implementation relied on accessing swagger enumeration as constant values from the generated models package. The current implementation restricts the scope of these constansts to a package which breaks my implementation.
Swagger specification
Cluster:
type: object
description: The cluster layout.
required:
- links
- data
properties:
links:
type: object
required:
- self
properties:
self:
$ref: "#/definitions/Link"
data:
type: object
required:
- status
properties:
noOfNodes:
type: integer
format: int32
description: Number of nodes the cluster has
scheduler:
type: string
description: The scheduler that needs to be used by the job scheduler.
status:
type: string
default: "scheduled"
description: The status of the cluster. It will be one of scheduled, building, up, deleting, exited or error.
enum:
- scheduled
- building
- up
- deleting
- exited
- error
clusterError:
type: string
description: Error associated with building or tearingdown the cluster.
This would generate
Steps to reproduce

Generate go from the swagger description
The models package will contain a cluster.go file with a cluster struct and a Status property of type string
The cluster.go file will also contain the following const values.
const (
clusterDataStatusScheduled string = "scheduled"
clusterDataStatusBuilding  string = "building"
clusterDataStatusUp        string = "up"
clusterDataStatusDeleting  string = "deleting"
clusterDataStatusExited    string = "exited"
clusterDataStatusError     string = "error"
)

Thank you for looking at the problem.

I can see the same issue.
From my perspective it's not a bug then it's a serious breaking change.

Thank you Ivan.


Pin napalm-base driver

See details in the parent issue napalm-automation/napalm-base#107

Fixed here https://github.com/napalm-automation/napalm-ibm/pull/10