Pycharm scipy SVD returning error code without message

I am trying to compute the SVD of a large matrix (4799 x 53130) using Pycharm as my IDE. The following returns the exit code -1073741819 (0xC0000005), but no error message is displayed. I have looked up this error code on here and people have found a mixture of things causing it. I have re-installed Pycharm and all dependencies, yet this issue persists. I have tried also using the numpy version, which returns the same error. I'm confident this problem is related to the lapack package, since both use it. Any direction would be greatly appreciated.
import numpy as np
import pandas as pd
from scipy.linalg import svd
df = pd.read_pickle(r'filepath\data.pkl')
df = df.to_numpy()
svd(df)

I tried using df=np.ones((4799,53130)) too in case it was my data that was the problem, and this yields the same issue. It isn't the amount of RAM, my machine has 120GB and the process doesn't go near the limit.

You probably run out of 32bit indexing. Could you try building both using 64bit Lapack known as ilp64? Here is the first hit after a search about mkl and openblas
https://github.com/pytorch/pytorch/issues/51720

You probably run out of 32bit indexing. Could you try building both using 64bit Lapack known as ilp64? Here is the first hit after a search about mkl and openblas
pytorch/pytorch#51720

Hi Ilayn, what do you mean by building both using 64bit? Sorry, this is above my comprehension.

The default underlying lapack library uses 32 bit integers internally for indexing arrays and for large arrays they might overflow. There is also a 64bit integer version of lapack that can be used to build numpy and scipy. That's what is being mentioned in the linked issue.

64bit integer version of lapack

I looked in the linalg folder of scipy, and found the lapack code that is called, which has the following:

It seems to already have the 64bit version?

Running Pearu's example on a machine with 16Gb of memory generates a MemoryError:
$ cat svd_oops.py
import scipy.linalg
import numpy as np
df=np.ones((4799,53130), dtype=np.float64)
breakpoint()
scipy.linalg.svd(df)
br@gonzales:~/sweethome/temp/chol$ python svd_oops.py
> /home/br/sweethome/temp/chol/svd_oops.py(5)<module>()
-> scipy.linalg.svd(df)
(Pdb) c
Traceback (most recent call last):
  File "/home/br/sweethome/temp/chol/svd_oops.py", line 5, in <module>
    scipy.linalg.svd(df)
  File "/home/br/repos/scipy/scipy/build-install/lib/python3.10/site-packages/scipy/linalg/_decomp_svd.py", line 127, in svd
    u, s, v, info = gesXd(a1, compute_uv=compute_uv, lwork=lwork,
numpy.core._exceptions._ArrayMemoryError: Unable to allocate 21.0 GiB for an array with shape (53130, 53130) and data type float64

dropping into pdb shows that lwork is OK, but the the unitary matrices in the decomposition are too large:
(Pdb) lwork
92155197
(Pdb) 53130**2
2822796900
(Pdb) np.iinfo(np.int32).max
2147483647

On a larger-memory machine (not sure how much, definitely more than 128Gb), the snippet above does segfault.
To double-check that that it's indeed U and VT matrices, changing scipy.linalg.svd(df) into scipy.linalg.svd(df, compute_uv=False) does not segfault and returns a reasonable result:
>>> u, s, vt = scipy.linalg.svd(df, compute_uv=False)
>>> (u @ np.diag(s) @ vt - df).max()
1.1737300020797647e-10

Since U and VT matrices are generated internally by f2py, it looks like the issue is somewhere in f2py.

Since U and VT matrices are generated internally by f2py, it looks like the issue is somewhere in f2py.

Spoke too soon. No, f2py is not to blame. The segfault is in lapack. Stepping through build/scipy/linalg/_flapackmodule.c, what segfaults is this line
/*callfortranroutine*/
                (*f2py_func)((compute_uv?(full_matrices?"A":"S"):"N"),&m,&n,a,&m,s,u,&u0,vt,&vt0,work,&lwork,iwork,&info) ;
                /*(*f2py_func)(&m,&n,&minmn,&u0,&u1,&vt0,&vt1,a,&compute_uv,&full_matrices,u,s,vt,work,&lwork,iwork,&info);*/

and the backtrace is
(gdb) bt
#0  0x00007ffff684d77b in dlarf_ ()
   from /home/ev-br/.conda/envs/scipy-dev/lib/python3.11/site-packages/numpy/core/../../../../libcblas.so.3
#1  0x00007ffff6865899 in dorgl2_ ()
   from /home/ev-br/.conda/envs/scipy-dev/lib/python3.11/site-packages/numpy/core/../../../../libcblas.so.3
#2  0x00007ffff6865f8d in dorglq_ ()
   from /home/ev-br/.conda/envs/scipy-dev/lib/python3.11/site-packages/numpy/core/../../../../libcblas.so.3
#3  0x00007ffff67f976a in dgesdd_ ()
   from /home/ev-br/.conda/envs/scipy-dev/lib/python3.11/site-packages/numpy/core/../../../../libcblas.so.3
#4  0x00007ffddc2a894b in f2py_rout__flapack_dgesdd (capi_self=<optimized out>,
    capi_args=<optimized out>, capi_keywds=<optimized out>,
    f2py_func=0x7ffff67f7380 <dgesdd_>) at scipy/linalg/_flapackmodule.c:11139
#5  0x0000555555734323 in _PyObject_MakeTpCall (
    tstate=0x555555ad0558 <_PyRuntime+166328>, callable=0x7ffddc4df4b0,
    args=<optimized out>, nargs=1, keywords=0x7ffddbffcc20)
    at /usr/local/src/conda/python-3.11.8/Objects/call.c:214
#6  0x0000555555741e36 in _PyEval_EvalFrameDefault (tstate=<optimized out>,
    frame=<optimized out>, throwflag=<optimized out>)
    at /usr/local/src/conda/python-3.11.8/Python/ceval.c:4769
#7  0x00005555557f842d in _PyEval_EvalFrame (throwflag=0, frame=0x7ffff7fb0020,
    tstate=0x555555ad0558 <_PyRuntime+166328>)

conclusion: needs ILP64 lapack. Until then, use full_matrices=False. This may push the segfault range a bit.

Rereading the conversation I see that all I said above has already been said by Pearu. Sorry for the noise.

See also https://github.com/numpy/numpy/issues/25564 that suggests that we should catch the integer overflow at the spot when it happens rather than propagating wrong values around until something along the computational path triggers a segfault situation.
To perform computations with the given sizes, using ILP64 lapack is indeed the ultimate solution.

This looks a bit different from the numpy/jax issue: the int overflow here does not seem related to the work array, it's just 2D indexing of the VT array.
Here's a check: using
$ mamba list
# packages in environment at /home/ev-br/.conda/envs/openblas-ilp64:
#
# Name                    Version                   Build  Channel
_libgcc_mutex             0.1                 conda_forge    conda-forge
_openmp_mutex             4.5                       2_gnu    conda-forge
binutils_impl_linux-64    2.40                 hf600244_0    conda-forge
gcc                       13.2.0               hd6cf55c_3    conda-forge
gcc_impl_linux-64         13.2.0               h338b0a0_5    conda-forge
gfortran                  13.2.0               h98b45c4_3    conda-forge
gfortran_impl_linux-64    13.2.0               h76e1118_5    conda-forge
kernel-headers_linux-64   2.6.32              he073ed8_17    conda-forge
ld_impl_linux-64          2.40                 h41732ed_0    conda-forge
libgcc-devel_linux-64     13.2.0             ha9c7c90_105    conda-forge
libgcc-ng                 13.2.0               h807b86a_5    conda-forge
libgfortran-ng            13.2.0               h69a702a_5    conda-forge
libgfortran5              13.2.0               ha4646dd_5    conda-forge
libgomp                   13.2.0               h807b86a_5    conda-forge
libopenblas               0.3.26          pthreads_h413a1c8_0    conda-forge
libopenblas-ilp64         0.3.26          pthreads_h384dd9e_0    conda-forge
libsanitizer              13.2.0               h7e041cc_5    conda-forge
libstdcxx-ng              13.2.0               h7e041cc_5    conda-forge
openblas                  0.3.26          pthreads_h7a3da1a_0    conda-forge
openblas-ilp64            0.3.26          pthreads_h3d04fff_0    conda-forge
sysroot_linux-64          2.12                he073ed8_17    conda-forge

with OpenBLAS from https://github.com/conda-forge/openblas-feedstock, consider
$ cat gesdd.f90
implicit none

character*1 :: jobz = 'A'

integer*8 :: m, n
integer*8 :: lda, ldu, ldvt, lwork
integer*8 :: info
integer*8, allocatable :: iwork(:)
real*8, allocatable :: a(:, :)
real*8, allocatable :: s(:)
real*8, allocatable :: u(:, :)
real*8, allocatable :: vt(:, :)
real*8, allocatable :: work(:)

! m < n only
m = 4799
n = 53130

lda = m
ldu = m
ldvt = n
allocate(a(lda, n), s(n), u(ldu, m), vt(ldvt, n))
allocate(iwork(8*n))

a = 1.d0

! workspace query
allocate(work(1))
lwork = -1

call dgesdd_64(jobz, m, n, a, lda, s, u, ldu, vt, ldvt, work, lwork, iwork, info)

lwork = int(work(1))

print*, n*n
print*, 'info = ', info
print*, 'opt lwork = ', lwork

deallocate(work)
allocate(work(lwork))

call dgesdd_64(jobz, m, n, a, lda, s, u, ldu, vt, ldvt, work, lwork, iwork, info)
print*, 'info = ', info
!print*, s
print*, 'max(sigma) = ', maxval(s)

end

compile it with
$ !gfo
gfortran gesdd.f90 -lopenblas64_
(openblas-ilp64) ev-br@qgpu3:~/temp$ ./a.out
<PHONE_NUMBER>
 info =                     0
 opt lwork =              92155197
 info =                     0
 max(sigma) =    15967.807300941449

So it "just works".
Changing interger*8 to integer*4, removing the _64 suffix from dgesdd call and linking with -lopenblas segfaults:
(openblas-ilp64) ev-br@qgpu3:~/temp$ gfortran gesdd.i4.f90 -lopenblas
(openblas-ilp64) ev-br@qgpu3:~/temp$ ./a.out
 -1472170396
 info =            0
 opt lwork =     92155197

Program received signal SIGSEGV: Segmentation fault - invalid memory reference.

Backtrace for this error:
#0  0x7fbdc76c451f in ???
	at ./signal/../sysdeps/unix/sysv/linux/x86_64/libc_sigaction.c:0
#1  0x7fbdc94dd77b in ???
#2  0x7fbdc94f5898 in ???
#3  0x7fbdc94f5f8c in ???
#4  0x7fbdc9489769 in ???
#5  0x5624109d8261 in ???
#6  0x5624109d843a in ???
#7  0x7fbdc76abd8f in __libc_start_call_main
	at ../sysdeps/nptl/libc_start_call_main.h:58
#8  0x7fbdc76abe3f in __libc_start_main_impl
	at ../csu/libc-start.c:392
#9  0x5624109d7138 in ???
Segmentation fault (core dumped)

where
$ cat gesdd.i4.f90
implicit none

character*1 :: jobz = 'A'

integer*4 :: m, n
integer*4 :: lda, ldu, ldvt, lwork
integer*4 :: info
integer*4, allocatable :: iwork(:)
real*8, allocatable :: a(:, :)
real*8, allocatable :: s(:)
real*8, allocatable :: u(:, :)
real*8, allocatable :: vt(:, :)
real*8, allocatable :: work(:)

! m < n only
m = 4799
n = 53130

lda = m
ldu = m
ldvt = n
allocate(a(lda, n), s(n), u(ldu, m), vt(ldvt, n))
allocate(iwork(8*n))

a = 1.d0

! workspace query
allocate(work(1))
lwork = -1

call dgesdd(jobz, m, n, a, lda, s, u, ldu, vt, ldvt, work, lwork, iwork, info)

lwork = int(work(1))

print*, n*n
print*, 'info = ', info
print*, 'opt lwork = ', lwork

deallocate(work)
allocate(work(lwork))

call dgesdd(jobz, m, n, a, lda, s, u, ldu, vt, ldvt, work, lwork, iwork, info)
print*, 'info = ', info
!print*, s
print*, 'max(sigma) = ', maxval(s)

end

Note that the optimal lwork is the same for 32- and 64-bit builds, and is always smaller than np.iinfo(np.int32).max.
The same segfault occurs when building with netlib blas/lapack (32-bit), so there is no reliable way to catch the overflow really:
$ gfortran gesdd.i4.f90 -lblas -llapack
(scipy-dev-netlib) ev-br@qgpu3:~/temp$ ./a.out
 -1472170396
 info =            0
 opt lwork =     92155197

Program received signal SIGSEGV: Segmentation fault - invalid memory reference.

Backtrace for this error:
#0  0x7f02e260251f in ???
	at ./signal/../sysdeps/unix/sysv/linux/x86_64/libc_sigaction.c:0
#1  0x7f02e2cfe6f0 in ???
#2  0x7f02e2d186c5 in ???
#3  0x7f02e2d1899d in ???
#4  0x7f02e2cab780 in ???
#5  0x55a721d34261 in ???
#6  0x55a721d3443a in ???
#7  0x7f02e25e9d8f in __libc_start_call_main
	at ../sysdeps/nptl/libc_start_call_main.h:58
#8  0x7f02e25e9e3f in __libc_start_main_impl
	at ../csu/libc-start.c:392
#9  0x55a721d33138 in ???
Segmentation fault (core dumped)

So, it looks like the integer overflow happens inside dgesdd function. When looking at its source code, there are several places where n * n is computed and that would obviously overflow for large values of n when its type is declared as integer*4.
To generalize these observations, each lapack function has limits to its inputs and apparently there are no checks implemented to catch events when these limits are exceeded.
We could determine these limits by trial-and-error for each lapack function and add the corresponding checks to f2py signature files in order to fail earlier with a nicer exception message rather than later with a segfault when calling the functions. What do you think?

I principle yes, it'd be nice to detect integer overflows. One simple heurstics could be that m*n > HUGE(int_type) --- this would detect flat indexing overflows. Could f2py grow this sort of automatic check?
More specific checks would indeed need trial-and-error on a per each function basis. How realistic it is given the LAPACK surface area I'm not sure TBH. Also, if ILP64 interface arrives within a reasonable timeframe, np.sqrt(np.iinfo(np.int64).max) is 3_037_000_500, and this ought to be large enough for everybody (tm)? The complexity of most linalg functions is at least cubic in the matrix size, after all.

These sizes are typically not good to be used with LAPACK dense algorithms. Because even the error bounds go out of acceptable bounds. Large scale linalg is not just trying to fit things into memory but also handling the error growth etc. Hence these are extreme cases that we don't have much control and I don't think it should have checks for all these cases. We can also error out on the Python side anyways.

The ILP64 discussion is a distraction I'd say. It's highly unlikely that ILP64 will be the default any time soon. For wheels that's going to be blocked on a unified 32/64-bit build of OpenBLAS not existing (yet). And even once it does, distros will still be building with LP64. The real fix here is to avoid the segfaults with LP64 and give MemoryError or a similarly appropriate exception instead.


lib64-nvidia did not contain libcudart.so

===================================BUG REPORT===================================
Welcome to bitsandbytes. For bug reports, please submit your error trace to: https://github.com/TimDettmers/bitsandbytes/issues
For effortless bug reporting copy-paste your error into this form: https://docs.google.com/forms/d/e/1FAIpQLScPB8emS3Thkp66nvqwmjTEgxp8Y9ufuWTzFyr9kJ5AoI47dQ/viewform?usp=sf_link
/usr/local/lib/python3.7/dist-packages/bitsandbytes/cuda_setup/paths.py:99: UserWarning: /usr/lib64-nvidia did not contain libcudart.so as expected! Searching further paths...
f'{candidate_env_vars["LD_LIBRARY_PATH"]} did not contain '
/usr/local/lib/python3.7/dist-packages/bitsandbytes/cuda_setup/paths.py:21: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('"<IP_ADDRESS>","jupyterArgs"'), PosixPath('["--ip=<IP_ADDRESS>"],"debugAdapterMultiplexerPath"'), PosixPath('"/usr/local/bin/dap_multiplexer","enableLsp"'), PosixPath('6000,"kernelManagerProxyHost"'), PosixPath('true}'), PosixPath('{"kernelManagerProxyPort"')}
"WARNING: The following directories listed in your path were found to "
/usr/local/lib/python3.7/dist-packages/bitsandbytes/cuda_setup/paths.py:21: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('module'), PosixPath('//ipykernel.pylab.backend_inline')}
"WARNING: The following directories listed in your path were found to "
/usr/local/lib/python3.7/dist-packages/bitsandbytes/cuda_setup/paths.py:21: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('/env/python')}
"WARNING: The following directories listed in your path were found to "
CUDA_SETUP: WARNING! libcudart.so not found in any environmental path. Searching /usr/local/cuda/lib64...
CUDA SETUP: CUDA runtime path found: /usr/local/cuda/lib64/libcudart.so
CUDA SETUP: Highest compute capability among GPUs detected: 8.0
CUDA SETUP: Detected CUDA version 111
CUDA SETUP: Loading binary /usr/local/lib/python3.7/dist-packages/bitsandbytes/libbitsandbytes_cuda111.so...
Caching latents: 100% 47/47 [00:03<00:00, 12.56it/s]
Steps:   0% 0/1000 [00:00<?, ?it/s]Traceback (most recent call last):
File "/usr/local/bin/accelerate", line 8, in
sys.exit(main())
File "/usr/local/lib/python3.7/dist-packages/accelerate/commands/accelerate_cli.py", line 43, in main
args.func(args)
File "/usr/local/lib/python3.7/dist-packages/accelerate/commands/launch.py", line 837, in launch_command
simple_launcher(args)
File "/usr/local/lib/python3.7/dist-packages/accelerate/commands/launch.py", line 354, in simple_launcher
raise subprocess.CalledProcessError(returncode=process.returncode, cmd=cmd)
subprocess.CalledProcessError: Command '['/usr/bin/python3', 'train_dreambooth.py', '--pretrained_model_name_or_path=CompVis/stable-diffusion-v1-4', '--use_auth_token', '--instance_data_dir=/content/data/KatieHiltonIn', '--class_data_dir=/content/data/person', '--output_dir=/content/drive/MyDrive/stable_diffusion_weights/KatieHiltonOut', '--with_prior_preservation', '--prior_loss_weight=1.0', '--instance_prompt=KatieHilton', '--class_prompt=person', '--seed=1337', '--resolution=512', '--center_crop', '--train_batch_size=1', '--mixed_precision=fp16', '--use_8bit_adam', '--gradient_accumulation_steps=1', '--learning_rate=5e-6', '--lr_scheduler=constant', '--lr_warmup_steps=0', '--num_class_images=12', '--sample_batch_size=4', '--max_train_steps=1000']' died with <Signals.SIGABRT: 6>.

apt install nvidia-cuda-toolkit solved for me

Thank you this is helpful! I relayed your comment to #46. Please continue the discussion there.

Bitsandbytes was not supported windows before, but my method can support windows.（yuhuang）
1  open folder J:\StableDiffusion\sdwebui，Click the address bar of the folder and enter CMD
or WIN+R, CMD  。enter，cd /d J:\StableDiffusion\sdwebui
2  J:\StableDiffusion\sdwebui\py310\python.exe -m pip uninstall bitsandbytes
3 J:\StableDiffusion\sdwebui\py310\python.exe -m pip uninstall bitsandbytes-windows
4 J:\StableDiffusion\sdwebui\py310\python.exe -m pip install https://github.com/jllllll/bitsandbytes-windows-webui/releases/download/wheels/bitsandbytes-0.41.1-py3-none-win_amd64.whl
Replace your SD venv directory file（python.exe Folder） here（J:\StableDiffusion\sdwebui\py310）


Missing tests for ArrayUtilities.java

I noticed that ArrayUtilities.java located in Ghidra/Framework/Utility/src/main/java/utilities/util/ did not have any tests. This is a beginner friendly issue. This issue's intention is to have unit tests added for ArrayUtilities.java. I also noticed that in one of the functions, there was typo, namely isArrayPrimativeEqual() should have been isArrayPrimitiveEqual.

This class was removed in 2a4a7747e68a9264ad8a6ef5a4c43c17d8f8ae79


OpenAI v1 support

Based on #224, merge that first

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@7379a5a). Click here to learn what that means.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #441   +/-   ##
=======================================
  Coverage        ?   81.80%
=======================================
  Files           ?       44
  Lines           ?     4243
  Branches        ?        0
=======================================
  Hits            ?     3471
  Misses          ?      772
  Partials        ?        0

Flag
Coverage Δ

unittests
81.80% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[bug]: sub menu indication line should be on left side

Is there an existing issue for this?

[X] I have searched the existing issues

Current behavior
When we generate demo test cases data, the UI shows the active submenu-item indicator on the right side which ideally should be on the left side.
Steps to reproduce

Go to the home page of the Keploy UI.
Go to the test lists page from the side menu.
See the behavior in the submenu.

if we put the indicator on the left side it would look better.
Environment
No response
Version
Local

@nehagup can it be assigned to me?


Invite users even if registration is disabled

Feature Description
Request from https://github.com/go-gitea/gitea/pull/20307#issuecomment-1275313104

In the future it'd be nice to have registrations disabled but also allow admins to individually invite users who can then register (since they were explicitly invited).

Sorry, am I correct that "invitation only registration" is not available right now? Or did I miss something?


Device list doesn't load when legacy_api set to false

Issue description
When setting legacy_api : false in Zigbee configuration.yaml and restarting zigbee2mqtt service, the device list doesn't load any more.

Setting legacy_api: true, and restart zigbee2mqtt service, the device list loads again.

Additional information
Zigbee2MQTT version: v.1.17.0 (ConBee2/RaspBee2 0x264a0700)
Python version: 3.7.3
Domoticz version: V2020.2 (build 12896)
Plugin version: (Zigbee) Initialized version 3.0.0

Why do you want legacy api? In >1.17.0 the new api is mandatory, so you can remove the line from configuration.yaml

Hi @Anjerlaan, thanks for reply.
Well, logically I believe setting a parameter (true/false) eg. legacy_api = false means I don't want it.
I found that default setting for the legacy_api is true if not set in configuration.yaml
So by removing the setting in configuration it will become true.
Br
Lennart

Hi @Anjerlaan, thanks for reply.
Well, logically I believe setting a parameter (true/false) eg. legacy_api = false means I don't want it.
I found that default setting for the legacy_api is true if not set in configuration.yaml
So by removing the setting in configuration it will become true.
Br
Lennart

If you remove this line in configuration.yaml, the new API is true...you don't need this line to tell you want to use the new API.
Try it and see if the list is populated

By the wat did you follow the Guise by deleting the plug-in first in the domoticz hardware tab?

Hi, the plugin works great.
After som further checks, issue seems to origin from zigbee2mqtt not reporting devices to plugin when legacy_api = false.
So I should probably ask the question in zigbee2mqtt forum.
Anyway the default value of legacy_api is actually true by default.
if you check /opt/zigbee2mqtt/lib/util/settings.js:

so removing the legacy_api from configuration.yaml is the same as adding the legacy_api = true. ;-)
You can try adding legacy_api = false, restart service and see if it works as expected.
But as mentioned, the plugin works great, so I'm closing this issue :)
Br
Lennart


fix: Withdrawal Confirmation page can't be closed

This pull request fixes the issue
#207

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@b80fe36). Click here to learn what that means.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             develop     #259   +/-   ##
==========================================
  Coverage           ?   65.26%
==========================================
  Files              ?       39
  Lines              ?     6161
  Branches           ?      236
==========================================
  Hits               ?     4021
  Misses             ?     2137
  Partials           ?        3

Continue to review full report at Codecov.

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

Thanks @GitOfJason for participating 👍


console.error only logs in production #tiny

No need to perform these __DEV__ checks ourselves.

          1 Error

:no_entry_sign:
No CHANGELOG added.

          1 Warning

:warning:
The following files do not have tests: lib/components/artist/header.js, lib/components/home/artist_rails/artist_card.js, lib/components/home/artist_rails/artist_rail.js & lib/components/home/artwork_rails/artwork_rail_header.js

  Generated by :no_entry_sign: danger

👍


Tests make live requests?

Running the phpunit tests seems to (attempt to) make live, actual requests over the Internet to ... a postmark test server I guess?
Just wondering if this is entirely necessary. For the purpose of this library, we should be able to assume that Postmark itself its working and just mock the HTTP interaction. Guzzle's mocking middleware is awesome for this.

Yeah, that's a "feature, not a bug." It ensures tests catch any typos or other incompatibilities with the Postmark API.
Under normal circumstances, I would want to mock an I/O, but since this library is the integration point between the two, the tests need to actually run against our live API.

:+1: fair enough


Upgrade Karma to 6.0.3

'npm audix fix' recommends to update Karma to the latest v6.0.3.
The same issue is created at systelab-components. Refer to https://github.com/systelab/systelab-components/issues/414

Covered by #62


[web]: Gallery app fonts inconsistency

On both iphone 11 Pro Max and iPhone X viewing the Gallery app in chrome, the fonts look different than they do in chrome desktop browser specifically for the cupertino alerts.

Platform detection is working now. Tested on latest flutter.gallery.dev , clearing browser cache and reloading on iPhone11 Pro Max:

Correction, works in Safari. Fails on Chrome.


Fix ticks.minor and ticks.major configuration issues

This is a fork from #5961 separating the major/minor tick config issues from the rotation and label layout issues.
Problems

Can't update scale.ticks.* options at runtime because the scale class copies the minor and major ticks options from ticks.* on the initial chart creation, and it doesn’t update them on the following update calls as the values already exist (#4896).
ticks.major.enabled is set to false by default, but ticks.major options are effective. The behavior should be consistent with the ticks.major.enabled setting.

Solutions

Deprecates mergeTicksOptions and adds parseTickFontOptions to parse minor and major tick font options. Omitted options are inherited from ticks at runtime.
Uses minor tick options when ticks.major.enabled is false.

Master: https://jsfiddle.net/nagix/76fLvg8u/

This PR: https://jsfiddle.net/nagix/4n5rd2gz/

Fixes #4896

Regarding mergeTicksOptions, should we remove it completely in v3?

No, this is ready to merge.

Thanks @nagix


Create semver package

This closes #169. I am unsure of what to do with the doctests entrypoint, so I put it in its own file.

@tomschr Should I include the tests in the src directory?

@tomschr Should I include the tests in the src directory?

No, tests shouldn't be part of the semver package IMHO. It's something for developers, not users. Hynek thinks that a test dir shouldn't be included into the package.

I kind of deleted setup.py in favor of setup.cfg because I found that it required hackery to make the importing of the metadata possible. As I went I found it pretty clean, so I ported all the config from setup.py. Let me know if that is problematic 😕

So you still see it even though I resolved the conversation @tomschr

If you solved the issues above, it would be great if you could clean up your commits. :+1: 🙂

@tomschr I think this could be a good opportunity to rename the VersionInfo to Version to address issue #305.

@tlaferriere I researched a bit and found (hopefully) a good solution. Due to changing semver from a package to a module there were some additional changes necessary.
The output looks a bit different now, but I think it's okay. Let me know if you think this should be changed. There are some warning messages that are a bit disturbing:
WARNING: missing attribute mentioned in :members: or __all__: module semver.__about__, attribute __about__
WARNING: missing attribute mentioned in :members: or __all__: module semver.cli, attribute __about__
WARNING: missing attribute mentioned in :members: or __all__: module semver.cli, attribute __author__
WARNING: missing attribute mentioned in :members: or __all__: module semver.cli, attribute __author_email__
WARNING: missing attribute mentioned in :members: or __all__: module semver.cli, attribute __maintainer__
WARNING: missing attribute mentioned in :members: or __all__: module semver.cli, attribute __maintainer_email__
WARNING: missing attribute mentioned in :members: or __all__: module semver.cli, attribute __description__
WARNING: missing attribute mentioned in :members: or __all__: module semver.version, attribute __version__
WARNING: missing attribute mentioned in :members: or __all__: module semver.version, attribute __about__
WARNING: missing attribute mentioned in :members: or __all__: module semver.version, attribute __author__
WARNING: missing attribute mentioned in :members: or __all__: module semver.version, attribute __author_email__
WARNING: missing attribute mentioned in :members: or __all__: module semver.version, attribute __maintainer__
WARNING: missing attribute mentioned in :members: or __all__: module semver.version, attribute __maintainer_email__
WARNING: missing attribute mentioned in :members: or __all__: module semver.version, attribute __description__

Maybe we need to introduce an __all__ variable. That isn't covered (yet) in the above commits.
Additionally, I've split the changelog into entries for semver3 and semver2. I think, this makes it easier.
Hope these changes make sense to you. :smile:

@tlaferriere I've tried a different approach now. I removed all the autosummary stuff and used the sphinx-apidoc command (included in tox.ini and run before make). This gives me a little bit more control. With this approach, I get a "decent" API documentation which looks like this:
    Submodules
        semver.__about__ module
        semver.__main__ module
        semver._deprecated module
        semver.cli module
        semver.version module

As you can see, there are also some private modules (as semver._deprecated) listed. I can suppress that by removing the -P option in `tox.ini (line 73) which would gives this result:
    Submodules
        semver.cli module
        semver.version module

So I'm a bit unsure which would be better. I could also use the first part with private, but exclude semver._deprecated. What do you think?
The above commits contains some improvements of docstrings in modules and functions. Hope I didn't commit anything useless by accident.
If you agree with my suggestions, we could arrange and squash the commits and I can merge it to master.

Since some of the important metadata is imported in the __init__.py file, wouldn't it show up in the docs in there? Anyway, I saw the  py.typed and pep-561 commit, and here I was thinking that just having type hints would basically enable checking for users 🤣. Well I learned something today 😀

Since some of the important metadata is imported in the __init__.py file, wouldn't it show up in the docs in there?

If I remove the -P (for private) flag, they disappear. I've tried to add a .. autodata:: __version__ but that gave me this:
semver package

semver package major release 3.

A Python module for semantic versioning. Simplifies comparing versions.

semver.__version__ Semver version

    str(object=’’) -> str str(bytes_or_buffer[, encoding[, errors]]) -> str

    Create a new string object from the given object. If encoding or errors is specified, then the object must expose a data buffer that will be decoded using the given encoding and error handler. Otherwise, returns the result of object.__str__() (if defined) or repr(object). encoding defaults to sys.getdefaultencoding(). errors defaults to ‘strict’.

which is of course nonsense as it kind of copies the docstring from the str object. :facepalm:
What I can do is to manually add that. Maybe with __version__ and a few others that would be okay. On the other side, with a name autodoc of this extension, I would expect that this could be done automatically. Maybe there is a configuration option. :thinking:

Anyway, I saw the py.typed and pep-561 commit, and here I was thinking that just having type hints would basically enable checking for users.  Well I learned something today

Haha, me too. I hope this makes sense. :wink:

Can't we just create a template that interpolates these variables? It's not like we're going to change the meaning or the usage of these variables very often 😉. They are accessible from the setup.cfg using the attr: keyword. Isn't there an equivalent for Sphinx? And it's not like we need the type info explicitly in the docs for these too badly.

Well, my knowledge about Sphinx and autodoc is limited. I've tried several things, but either I got some warning messages (like the one above) or I didn't get what I wanted. :wink:
The autodoc extension can be configured by a autodoc_default_options variable. However, that is limited.

And it's not like we need the type info explicitly in the docs for these too badly.

True. But it would be nice to have at least __version__ and SEMVER_SPEC_VERSION

I think I found a (somewhat) unorthodox solution. :laughing: The API doc shows now only the public interface, but additionally the semver.__about__ module (with __version__ and SEMVER_SPEC_VERSION).
If you are okay with that, I would tomorrow squash all the commits into some useful history and merge it to master.
Any objections?

An unorthodox solution is still a solution 😉. If it's that bad I guess we can open an issue with Sphinx for this use case. For now I think this should do. It appears we've got good timing for the second dev release 🎉

Hehe, yes.If it turns out that it is bad, we can always publish a new dev release.
I'll go through the commits and try to come up with some decent history. As I heard no "stop" or other objections, I'll merge them to master. :wink:
Thanks for all your contributions and efforts! :+1:

I just realized that it is not possible to rearrange the commits and force-push them into this pull request. I'll open a new branch and merge the commits from there.

Closing this in favor of #306.


Fix fatal error of SteamAPIInitError::from_result_and_message.

Update SDK to 1.58a   has a code error, will cause the program to crash when throwing SteamAPIInitError.
This PR fixed it.

Hello? @Noxime @ceifa


Internationalize an item in AppBar.js

I don't know how to internationalize the 'Se connecter' line 49 of the file AppBar.js
Is anyone able to help?

You can pass props to makeStyles like in this example:
https://v4.mui.com/styles/api/#examples-3

@Tonfa do you want to try with sebastian's explanations ?

Yes thank :)
On Tue, 12 Sept 2023, 14:09 VincentFarcy, @.***> wrote:

@Tonfa https://github.com/Tonfa do you want to try with sebastian's
explanations ?
—
Reply to this email directly, view it on GitHub
https://github.com/chemins-de-la-transition/semapps/issues/615#issuecomment-1715695401,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXQP67HZS4KRLJ2E7GSHULX2BNIJANCNFSM6AAAAAA4QGDDTE
.
You are receiving this because you were mentioned.Message ID:
@.***>

I couldn't do it :(
I need help for this one.
On Tue, 12 Sept 2023 at 15:41, Abdulrahman OTMAN <
@.***> wrote:

Yes thank :)
On Tue, 12 Sept 2023, 14:09 VincentFarcy, @.***>
wrote:

@Tonfa https://github.com/Tonfa do you want to try with sebastian's
explanations ?
—
Reply to this email directly, view it on GitHub
https://github.com/chemins-de-la-transition/semapps/issues/615#issuecomment-1715695401,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXQP67HZS4KRLJ2E7GSHULX2BNIJANCNFSM6AAAAAA4QGDDTE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Tonfa it's done here on i18n branch : https://github.com/chemins-de-la-transition/semapps/commit/053f71623db75073e73d0f58bca66df6882fc750


Question of teleop_twist_keyboard

System Configuration:
Tell us about your system.

OS: Ubuntu 22.04
ROS Version:ROS 2, Humble
Gazebo Version:Gazebo Garden
VRX

I have a question regarding the teleop_twist_keyboard.
I referenced the VRX_DRIVING example, but it only had joystick control.
Is it not possible to use teleop_twist_keyboard with it?
When I ran the code, it seemed like the values were being subscribed to the /cmd_vel topic, but is there no implemented logic to handle this?

We haven't directly added keyboard support to the project yet but it should be possible if you map the keystrokes to topics in a similar way we do it here:
https://github.com/osrf/vrx/blob/main/vrx_gz/config/wamv.yaml


Hugging Face or Replicate demo

Hello. I'm very interested in trying IRSDE but I can't install Linux (no space)
Is there a chance we can expect online demo soon? I will be more than happy to pay for it.

Sorry we don't have the plan to provide an online demo since IRSDE is designed only for research :) However, we also provided a gradio interface which you can use to create the online demo.


MN - 2 Files to be resent

Problem Statement
We received an email in the ReportStream inbox where the State Minnesota says they have not received the test results submitted via SimpleReport. The Facility Name is The Glenn Hopkins - Terraces.
What you need to know
I was able to locate two records:
The SimpleReport record submitted on 9/10/2023 messageid is 800bca95-b4f5-401e-8c50-93100799c573 sent to MN-DOH on 9/11/2023 at 1:40 PM.
The SimpleReport record submitted on 9/18/2023 messageid is 1a38b375-6698-4bc4-b68f-112f85052f35 sent to MN on 9/20/2023 at 1:40 PM
Please let us know if you need any further assistance.
I'll post the email chain in the comments.
Acceptance Criteria
Engineer Assigned
Engineer confirms that file was resent.
Let the customer know when sent.
Confirm receipt by customer/MN.
To Do
A quick note to the assignee of the ticket.
Additional Comments
If applicable, add screenshots, links, etc. to further explain the ticket

Japa, Srujana (MDH<EMAIL_ADDRESS>Report Stream (CDC);Julie Nichols<EMAIL_ADDRESS>Blair, Benjamin (He/Him/His) (MDH<EMAIL_ADDRESS>Thank you, Sharon, for sharing this information. Upon our review, we couldn't find these messages in the batch files received on 9/11 and 9/20. Could you please re-trigger these messages to MDH and notify us once they are sent?
Srujana
Srujana Japa
ELR Onboarding Coordinator | MEDSS Operations Unit
Minnesota Department of Health
From: Report Stream (CDC<EMAIL_ADDRESS>Sent: Friday, October 13, 2023 1:48 PM
To: Japa, Srujana (MDH<EMAIL_ADDRESS>Julie Nichols<EMAIL_ADDRESS>Cc: Blair, Benjamin (He/Him/His) (MDH<EMAIL_ADDRESS>Subject: Re: MDH : Missing COVID Labs Great Lakes MC
Hello,
I was able to locate two records:
The SimpleReport record submitted on 9/10/2023 messageid is 800bca95-b4f5-401e-8c50-93100799c573 sent to MN-DOH on 9/11/2023 at 1:40 PM.
The SimpleReport record submitted on 9/18/2023 messageid is 1a38b375-6698-4bc4-b68f-112f85052f35 sent to MN on 9/20/2023 at 1:40 PM
Please let us know if you need any further assistance.
Best regards,
From: Japa, Srujana (MDH<EMAIL_ADDRESS>Sent: Friday, October 13, 2023 2:13 PM
To: Report Stream (CDC<EMAIL_ADDRESS>Julie Nichols<EMAIL_ADDRESS>Cc: Blair, Benjamin (He/Him/His) (MDH<EMAIL_ADDRESS>Subject: RE: MDH : Missing COVID Labs Great Lakes MC
Adding @Julie Nichols<EMAIL_ADDRESS>from Great Lakes on this email thread.
Julie,
I am Srujana Japa from MDH, I'm connecting you with the Simple Reports staff to address the issue of missing COVID labs that should have been reported to MDH. To assist in the investigation, could you please provide the dates of submission, the CLIA number, or any facility details related to these labs on this thread?
Thank you,
Srujana Japa
ELR Onboarding Coordinator | MEDSS Operations Unit
Minnesota Department of Health
From: Report Stream (CDC<EMAIL_ADDRESS>Sent: Friday, October 13, 2023 11:43 AM
To: Japa, Srujana (MDH<EMAIL_ADDRESS>Cc: Blair, Benjamin (He/Him/His) (MDH<EMAIL_ADDRESS>Subject: Re: MDH : Missing COVID Labs Great Lakes MC
Hi Srujana,
Thank you for contacting ReportStream Support.
We are unable to find any facility information on Great Lakes MC. Please reach out to the contact for dates of submission, CLIA number, or facility details so that we can research further. Please also have them reach out to SimpleReport to confirm success result upload or manual entry.
If you have any further question or concerns please don't hesitate to reach out.
Thank you,
From: Japa, Srujana (MDH<EMAIL_ADDRESS>Sent: Friday, October 13, 2023 10:07 AM
To: Report Stream (CDC<EMAIL_ADDRESS>Cc: Blair, Benjamin (He/Him/His) (MDH<EMAIL_ADDRESS>Subject: MDH : Missing COVID Labs Great Lakes MC
Good Morning Report Stream Team,
Could you please check your logs for COVID submissions from Great Lakes Management (Great Lakes MC), a senior living facility based in Minnesota, and verify if they have indeed submitted any data? As of now, we haven't received any data from them. Great Lakes MC claims that they have submitted data using simple reports.
Thank you,
Srujana Japa
ELR Onboarding Coordinator | MEDSS Operations Unit
Minnesota Department of Health
Office:<PHONE_NUMBER>
Report Stream (CDC)
Japa, Srujana (MDH) Srujana.Japa@state.mn.us;Julie Nichols<EMAIL_ADDRESS>Blair, Benjamin (He/Him/His) (MDH<EMAIL_ADDRESS>Hello,
Some content in this message has been blocked because the sender isn't in your Safe senders list. General
Japa, Srujana (MDH<EMAIL_ADDRESS>Report Stream (CDC);Julie Nichols<EMAIL_ADDRESS>Blair, Benjamin (He/Him/His) (MDH<EMAIL_ADDRESS>Adding @Julie Nichols<EMAIL_ADDRESS>from Great Lakes on this email thread.
Julie,
I am Srujana Japa from MDH, I'm connecting you with the Simple Reports staff to address the issue of missing COVID labs that should have been reported to MDH. To assist in the investigation, could you please provide the dates of submission, the CLIA number, or any facility details related to these labs on this thread?
Thank you,
Srujana Japa
From: Report Stream (CDC<EMAIL_ADDRESS>Sent: Friday, October 13, 2023 11:43 AM
To: Japa, Srujana (MDH<EMAIL_ADDRESS>Cc: Blair, Benjamin (He/Him/His) (MDH<EMAIL_ADDRESS>Subject: Re: MDH : Missing COVID Labs Great Lakes MC
Report Stream (CDC)
Fri 10/13/2023 12:42 PM
Hi Srujana, Thank you for contacting ReportStream Support. We are unable to find any facility information on Great Lakes MC. Please reach out to the contact for dates of submission, CLIA number, or facility details so that we can research further. Please also
Report Stream (CDC)
Japa, Srujana (MDH<EMAIL_ADDRESS>Blair, Benjamin (He/Him/His) (MDH<EMAIL_ADDRESS>Hi Srujana,
Thank you for your email.  We are looking into this and will get back to you as soon as we have an update.
For your reference, please visit the online Support FAQ.
Best regards,
Japa, Srujana (MDH<EMAIL_ADDRESS>Report Stream (CDC)
Blair, Benjamin (He/Him/His) (MDH<EMAIL_ADDRESS>Good Morning Report Stream Team,
Could you please check your logs for COVID submissions from Great Lakes Management (Great Lakes MC), a senior living facility based in Minnesota, and verify if they have indeed submitted any data? As of now, we haven't received any data from them. Great Lakes MC claims that they have submitted data using simple reports.

Done.


[stable/grafana] Support Provisioner Configuration

This PR extends the stable/grafana chart to allow for outside configuration of provisioners.
With Grafana 5 it will be possible to manage datasources and dashboards using provisioners.  Adding a provisioner is done by adding one or more yaml configuration files in the provisioning/datasources and provisioning/dashboards directory. See also: http://docs.grafana.org/administration/provisioning/
For datasources the file can contain a list of datasources that will be added or updated during start-up. If the datasource already exists, Grafana will update it to match the configuration file.
For dashboards the files contain the configuration for provisioning from external resources. It currently only support reading dashboards from file but more provisioners are in the works.
This functionality partly clashes with the jobs provided by this Helm chart. In order to support both Grafana 4/5 in the same chart, I think, it is a sensible approach to allow for both provisioning mechanisms. In the absence of more sophisticated provisioners (that allow the decoupling from Helm and dashboards) it is anyway required to load the dashboard from the filesystem and inject it using the existing mechanism.

/assign @linki

/assign @prydonius

/unassign @linki

Looking forward to this to land. Too bad though that creating a default datasource for e.g. Prometheus deployed by the stable Chart (in a Chart having both Grafana and Prometheus as requirements) still won't be possible using this approach, because its service name/DNS entry depends on the Helm release name 😞

@BugRoger
Looking forward to see this merged. Would you mind to fix the the merge conflict?

Fixed. It was just some other change increasing the version

/assign @prydonius

Can confirm that it works like a charm and grafana 5.0.0 was just released
https://github.com/grafana/grafana/releases

/ok-to-test

/retest

This merge proposal doesn't actually change the default version to 5.0.0 (relevant issue: #3937).

This should bump the version to 5.0.1 and add the AppVersion to the charts.yaml

@BugRoger could you fix the linting issues from the circleci build

@BugRoger are you still working on this, if not I can help

In my opinion is better to bump the minor version and remove at least the support to configure datasources using the Job, it is a hack that is difficult to maintain and not needed anymore. In general the dashboards Job will also not be needed but I think is handy to have the functionality to import remote dashboards.
About bumping the minor version (or major) of the Chart, Grafana is not a project that has Long Term Support releases so keeping a Chart that is compatible with version 4.6.x and another compatible with >= 5 is not a big issue I think, eventually the Chart versions compatible with Grafana <= 4 will not be updated anymore.
I want to use this Chart on cores/kube-prometheus "meta Chart", they maintain what is basically a copy of the stable Chart but adding grafana-watcher that essentially serves the same purpose as the Jobs here, both cases are basically hacks that with Grafana >= 5 are not required.

Fixed the linter nitpicks. Bumped the version. Rebased on master.

Any way to move this forward?

Please update to the newest Grafana 5.0.4 Docker image as the older images don't work with Kubernetes 1.7.14, 1.8.9, or 1.9.4.
See: https://github.com/grafana/grafana-docker/issues/140

Rebased. Updated Image to 5.0.4. Added appVersion to make Linter happy. Tests are green.

@BugRoger, sadly again conflicting files. I think this should be coordinated with someone that can merge, do the fix and immediately apply it. I'm waiting for this PR to be merged to work on the integration in https://github.com/coreos/kube-prometheus

The same issue on 1.9.6-gke.0

It seems i can't fix this. There's a deadlock between the different checks.
The linter requires me to set the version to version in chart.yaml to 0.8.5. This causes a conflict against master. That fails the e2e tests.
I can either resolve the conflict or fix the version.

master is already at 0.8.5.  You need to rebase and bump it to 0.8.6

Ah, rebased and bumped the version again....

@unguiculus if #4713 is merged it would make this PR no longer needed since #4713 is such a large overhaul and simplification.
I have not tested #4713, but the changes themselves are heading in a good direction.

Closing in favor of #4713. Thanks @BugRoger.


KAFKA-3923: Use Metric interface instead of final KafkaMetric in MetricsReporter and its implementations

This is a patch for the previously highlighted issue:
https://issues.apache.org/jira/browse/KAFKA-3923
I've minimised the scope of changes, so that now Metrics class is not affected and generics are introduced on the method level instead of class level.

It's funny that there are both FakeMetricsReporter and MockMetricsReporter - maybe the first one may be dropped.

Hello, @guozhangwang ,
May I ease the process of reviewing for you by any chance?

Any updates?

Sorry for the late reply @stepio , I think the motivation behind this PR makes sense, but the proposed change modifies MetricsReporter interface, which will break anyone implementing this class (i.e. not source compatible).
I'm wondering for making the mocking easier, and I'm assuming by mocking you want to add some bookkeeping logic for the function called, etc, would it be simpler to just make KafkaMetric not final? cc @ijuma @junrao .

Hi @guozhangwang , it's Ok. Better late than never, so thank you anyway.
Your solution is simpler and safer, still allowing easy testing without powermock "hacks". I don't think that there are many implementations of this interface (did not find any by searching on github), but still removing "final" is safer. My approach just improves a bit the architecture - for example, similar generics are already used in Consumer and Producer interfaces.
So you're the maintainers and decision is yours.

@stepio I am actually aware of a couple of MetricsReporter implementations, and some of them comes from confluentinc open source repos :P Personally I agree that your fix is actually more ideal, and that we did make a mistake at the very beginning when writing MetricsReporter. But for retaining backward compatibility we'd probably have to go with the sub-optimal approach.
I have synced with @ijuma about this, and either myself or Ismael will be happy to help review and merge this once you refactored this PR.

I've did the requested updates, although now I'm a bit ashamed to push such a trivial change :)
As a bonus, I've did minor clean-up in interface and added the javadoc, describing approach, which saved me eventually and was not clear for me without external help.
I see some test failures, but I'd rather blame race conditions in tests... What do you think?
P.S.: I've also kept the original "major" approach, but moved it to another branch, just for history:
https://github.com/stepio/kafka/tree/feature/KAFKA-3923

test this please.

test this please.
Do you mean that I should write some specific tests for removal of final? Some tests with extension?

Updated tests... and the main code as well... Sorry, sometimes it's hard to stop :)

Updated tests... and the main code as well... Sorry, sometimes it's hard to stop :)

I've done a kind of "demo" for using assertj:
https://github.com/stepio/kafka/commit/ca1c1d306b7cf961685aad18520711e18bb167d7
Will this suit Kafka project? Or would you prefer staying with pure junit?

I've done a kind of "demo" for using assertj:
https://github.com/stepio/kafka/commit/ca1c1d306b7cf961685aad18520711e18bb167d7
Will this suit Kafka project? Or would you prefer staying with pure junit?

Dear @guozhangwang ,
I've done the necessary changes, please take a look when you have some time

About assertj, seems it does not add more asserting functionalities that junit cannot provide? Is there any specific reasons you want to add a new dependency jar into Kafka?

Is there any specific reasons you want to add a new dependency jar into Kafka?

It's a test-dependency, which incorporates functionality of junit and hamcrest-matchers. As for me, I love the "builder-like" syntax for writing tests - allows applying multiple checks for a single value. Also no mess with "actual" and "expected" values (which one is the first, and which one is a second). As a bonus, it supports Java 8 constructions starting from 3.x.x version.
But it's still just a suggestion.

But it's still just a suggestion.

Personally I'd try to avoid growing dependencies, both runtime and test unless it is necessary. For assertj I found it provides some syntax sugar and more clear argument semantics but not providing any asserting functionalities that junit and harmcrest already provides.

Hi @guozhangwang  and @ijuma
Did more updates, so it's your turn again :)

LGTM overall. Could you respond to @ijuma 's comment?

LGTM overall. Could you respond to @ijuma 's comment?

I was unable to respond here, so I went to "FIles changed" tab of the pull request and replied there:

Unfortunately that answer is not displayed here. @ijuma, is it visible to you in the diff?

Hi @ijuma
Any update? I've replied to your comment, but have not changed anything, as I haven't seen any clear suggestion... So what do you think?

@ijuma
Thanks for the detailed review. Seems that it might be reasonable to reconsider not only the exact changes, but the pull request in general.

About the duplicated code: I know it is subjective, but to me I feel this is worthwhile to make the locking scope clearer. And it adds about 6 lines only.

Closing for cleanup.


Extracting feature importance

Is it possible to extract "feature importance" from an WTTE-RNN model?

In theory, feature importance for neural networks is a tricky question and in general there's no nice tricks/clear winning strategy as with trees that I know about (but I know very little about the subject).
There's some tricks ranging from advanced plotting to checking the derivative wrt to inputs or maybe training with dropout and shutting of certain inputs and evaluating on testset. Maybe this question can get you started.
In practice, with my experiences a more informal analysis can be helpful using wtte-rnn by just eyeballing plots of sequences of predictions and features lined up and (depending on your problem/data) you can sometimes see quite clearly that some data will make an impact on the state of the rnn. This wont help feature selection of course.


adding Page Load Response Time metrics to datadog

@udemy/sre
Previously a PR in main ansible repo, repurposing to work with new external datadog repo.
https://github.com/udemy/ansible/pull/743/

LGTM


Mass edit monitors

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
UI Feature
🔖 Feature description
Select few monitors and change it in one click...
For now i need to click one monitor, edit, change value, save.
Click next, next next.
Just mass settings edit.
heartbeat, check times, notifications rules etc.
✔ Solution
UI with checkbox on monitor and Edit selected button.
❓ Alternatives
No response
📝 Additional Context
No response

Another +1 for this feature.

+1 as well, would love this feature.

@x86dev @edrock200
Please refrain from posting +1 / requests for updates things on issues, as this makes issue-management harder.
Issues are for discussing what needs to be done how by whom.
We use 👍🏻 on issues to prioritise work.
Related PR that implements part of this issue: https://github.com/louislam/uptime-kuma/pull/1886

@streamthing
I think this is a duplicate of https://github.com/louislam/uptime-kuma/issues/455
If you agree, could you please close this Issue, as duplicates only create immortal zombies and are really hard to issue-manage?
If not, what makes this issue unique enough to require an additional issue? (Could this be integrated into the issue linked above?) ^^

@streamthing
We are consolidating duplicate issues a bit to make issue management easier.
I think, we should track this issue in #455 as there Mass editing and bulk editing are essentially the same thing
⇒ I am going to close this as a duplicate.


'Will participants receive coaching?' replicated

There are two of these on the Additional Info page.

Fixed


prevent syncing at each eval point

Fixes #46

Although, might not be so straightforward: time seems to actually increase. Closing for now


App crashes android 9

When starting to inspect with chrome app crashes with error:

Invalid Region.Op - only INTERSECT and DIFFERENCE are allowed

Same here. In my case it happens when I try to inspect layout. When the app tries to highlight the element that is being selected (hovered) in chrome console it crashes. It makes the layout inspector useless.

I believe this is a duplicate of https://github.com/facebook/stetho/issues/607, however this still hasn't been released.

Release is incoming now.  1.5.1 should include the fix.


Formula paste

Hello
is it possible to copy and paste formula like in Excel (For example: if you copy formula from E7=(C7 * D7) and paste it into E4, formula should contains =(C4 * D4) and not =(C7*D7) like now.
Thanks

This has been implemented on the Pro distribution and should be brought to CE eventually.
https://jexcel.net


Fix Robotidy crashing if configuration file does not exist

Fixes #526

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (a8ca849) 97.09% compared to head (bb56ef7) 97.09%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #527   +/-   ##
=======================================
  Coverage   97.09%   97.09%
=======================================
  Files          49       49
  Lines        4790     4792    +2
=======================================
+ Hits         4651     4653    +2
  Misses        139      139

Impacted Files
Coverage Δ

robotidy/config.py
96.86% <100.00%> (+0.02%)
:arrow_up:

robotidy/version.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Introduce babel-addons config.

To:
@udemy/team-f
What:
Also,

Refactor website config to accommodate static/src/.
Refactor basics config to accommodate non-ES6 modules syntax.

Forgive my by-passing review. I am just too eager to finish this today.
Trello:
Part of https://trello.com/c/CGOZCMkS/1706-re-organize-all-eslint-udemy-repos
What did you test:
yarn run test
What dashboards will you be monitoring:
None

 Re-organize all eslint-udemy-* repos.


Problem loading RAPIDS.jl

Looks like a python package management issue, which leaves me a bit lost.
Stacktrace:
 [1] error(s::String)
   @ Base ./error.jl:33
 [2] compilecache(pkg::Base.PkgId, path::String, internal_stderr::Base.TTY, internal_stdout::Base.TTY, ignore_loaded_modules::Bool)
   @ Base ./loading.jl:1385
 [3] compilecache(pkg::Base.PkgId, path::String)
   @ Base ./loading.jl:1329
 [4] _require(pkg::Base.PkgId)
   @ Base ./loading.jl:1043
 [5] require(uuidkey::Base.PkgId)
   @ Base ./loading.jl:936
 [6] require(into::Module, mod::Symbol)
   @ Base ./loading.jl:923

(jl_FSq8U5) pkg> st
      Status `/private/var/folders/4n/gvbmlhdc8xj973001s6vdyw00000gq/T/jl_FSq8U5/Project.toml`
  [add582a8] MLJ v0.18.2
  [2764e59e] RAPIDS v0.1.0 `~/Julia/RAPIDS`

julia> versioninfo()
Julia Version 1.6.5
Commit 9058264a69 (2021-12-19 12:30 UTC)
Platform Info:
  OS: macOS (x86_64-apple-darwin19.6.0)
  CPU: Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-11.0.1 (ORCJIT, skylake)
Environment:
  JULIA_LTS_PATH = /Applications/Julia-1.6.app/Contents/Resources/julia/bin/julia
  JULIA_PATH = /Applications/Julia-1.6.app/Contents/Resources/julia/bin/julia
  JULIA_EGLOT_PATH = /Applications/Julia-1.6.app/Contents/Resources/julia/bin/julia
  JULIA_NUM_THREADS = 5
  JULIA_NIGHTLY_PATH = /Applications/Julia-1.7.app/Contents/Resources/julia/bin/julia

Also, I have PYTHON=/Users/anthony/anaconda2/envs/py37/bin/python which is python 3.7. Maybe that's it. I'll try updating my python installation  and adding those libraries manually. And then get back to you.
  CondaPkg Resolving changes
             + cudatoolkit
             + python
             + rapids

Looks like a python package management issue, which leaves me a bit lost. Is this, perhaps, because I don't have a GPU?

Unfortunately, RAPIDS requires linux, and a Pascal or more recent NVIDIA gpu with at least 6 TFLOPS (1070 or better). I hope that platform support will expand over time.

Also, I have PYTHON=/Users/anthony/anaconda2/envs/py37/bin/python which is python 3.7. Maybe that's it. I'll try updating my python installation and adding those libraries manually. And then get back to you.

I used CondaPkg.jl, which has micromamba instead of conda, and installs a fresh environment in the project directory instead of utilizing other environments. This was for two reasons, the first is that the python environment will hopefully be fully contained to avoid errors when installing the RAPIDS python libraries. This will hopefully make it easier for other developers to use the library. I have personally had a lot of headaches with RAPIDS compatibility and breaking python environments.  The second is that micromamba gives us an order of magnitude quicker installation of all the RAPIDS dependencies.
However, I am not married to this approach. I am open to suggestions/changes to any part of the library.
I have tried it on Google Colab, but getting a compatible GPU seems less and less common as time goes on. Additionally, the factory reset option seems to have been replaced with a disconnect and delete runtime, which then gives you a captcha. Here is a Google Colab link for testing the library (if you get lucky with gpu allocation): https://colab.research.google.com/drive/11Cpsm4vwLJjYcbhjdXJtBrHEnb54hIsL?usp=sharing

Thanks for those helpful and exhaustive comments.  I will wait for my University to re-activate my GPU allocation and go from there.
By the way the MLJ interface looks very good - but I'd like to play around with this some more before compiling my feedback. Thanks for your continued patience.

If I change PYTHON="" then my issue is resolved. Closing, as I think this confirms this is a python package mngmt issue over which RAPIDS.jl has no control.


[Bug] axis label position difference on Safari

Safari：

Chrome：

Reason: Safari's canvas text rendering is slightly lower than the Chrome baseline.
Safari：

Chrome:

There are some similar issues, and the rendering effects of Chrome and Firefox are also different:
https://stackoverflow.com/questions/39379392/placing-text-on-html5-canvas-inconsistent-across-browsers
https://stackoverflow.com/questions/35407614/html5-canvas-textbaseline-top-looks-different-in-firefox-and-chrome
What needs to be done now is to add a buffer to the VRender text measure method to make it compatible with various browsers.

fixed: https://github.com/VisActor/VRender/issues/294


ambex: Upgrade Envoy go-control-plane

So I was spending my Sunday experimenting with Go tooling stuff, and was using ambassador.git as a test-bed, and the weird go-control-plane situation with ambex was causing me headaches, so... here's a PR.
Description
Upgrade the go-control-plane used by ambex.  But instead of importing github.com/envoyproxy/go-control-plane, vendor a modified copy.
This is because:

github.com/envoyproxy/go-control-plane/pkg (hand-written code) needs to make use of github.com/envoyproxy/go-control-plane/envoy (protoc-generated code),
but it's important that we use github.com/datawire/ambassador/pkg/api/envoy instead of github.com/envoyproxy/go-control-plane/envoy

and it turns out that it's not terribly hard to just have make generate vendor a modified copy of go-control-plane.
Related Issues
https://github.com/datawire/ambassador/issues/2146
Testing
go test ./pkg/envoy-control-plane/... && go build ./cmd/ambex
Other
To the reviewer: The interesting stuff is in build-aux-local/generate.mk.  You can probably ignore reading any of the pkg/envoy-control-plane/ file diffs.

Are the KAT tests actually testing Ambex' ability to talk to Envoy, or do I need to do some "disable gold" test?

Do I need to do something special to make sure KAT actually tests ambex's ability to talk to Envoy? (disable gold?).  Or is CI already testing that?

OK, after some shenanigans with GKE giving me junk nodes, I verified that make pytest KAT_RUN_MODE=envoy passes on kubernaut.


Fix compilation failures and tests

Purpose
$subject.

@Bhashinee Please fix review suggestions by @MaryamZi in a separate PR.


Metrics format changed?

I was trying to setup Prometheus metrics exporting but I couldn't make it work with the current config, it seems that the separator between the namespace and app name could have changed?
Eg. I removed all the rules from the config (by default anything that doesn't match gets dropped) and I could see metrics again but I got the following help text:
# HELP spark_streaming_driver_receivers Attribute exposed for management (metrics<name=spark-popular-products.driver.popular-products.StreamingMetrics.streaming.receivers><>Value)

Where spark is the namespace and popular-products is the app name.
Checking the match pattern I could see that it has a / as the separator. I added the rules back and I changed the match pattern to the following and it seems to work again:
  - pattern: metrics<name=(\w+)-(\S+)\.driver\.(\S+)\.StreamingMetrics\.streaming\.(\S+)><>Value

No, it's not filled in.

OK, can you try adding the following and see if the patterns with a / work?
spec:
    sparkConf:
        "spark.metrics.namespace": "<your app namespace>/<your app name>"

I'm afraid that adding "spark.metrics.namespace": "<my apps namespace>/<my apps name>" to my spark application's spec has made no difference for me when using the / separator.

Can confirm, nothing changed.

OK, so looks like / is not a valid separator. We will fix the patterns.

This issue can be closed. As an FYI, the bug was introduced in this commit, where the separator changed from a backslash to a hyphen.

I see config was fixed in ba9258d84355ba21598206e4448b544ee1a30d6b as well.

Fixed in #405.

I updated my spark operator a few days ago and I am having issues getting metrics again. I see the matching pattern has changed again and I have updated the all files to the latest versions but I still can't get metrics from the driver and executor pods. The fix from #405 did work but I think it may have broken again somewhere in  #409 or #421


README should explain how to add to supervision tree

For the life of me I can't figure out what I have to do to get this working in my app. Explicitly calling Stripe.start works great, but in my app's supervisor, what am I supposed to add as the worker? Can you address this in the readme please?

Sure - sorry for the frustration :) it was my first package. You added the dependency and the app as well? Can you show me your mix file?

No problem. Pretty sure it's not the mix file, as directly calling Stripe.start makes everything work. But what I'm wondering is exactly what to do in my supervisor. I tried this:
    children = [
      # Define workers and child supervisors to be supervised
      # worker(TakeMyMoney.Worker, [arg1, arg2, arg3]),
      supervisor(Stripe.Supervisor, [])
    ]

after looking at what Stripe.start calls. But that doesn't seem to work. When I attempt to call anything, it says it's not running:
iex(2)>  {:ok, customers} = Stripe.Customers.list
** (exit) exited in: :gen_server.call(:hackney_manager, {:new_request, #PID<0.151.0>, #Reference<<IP_ADDRESS>>, {:client, :undefined, :hackney_dummy_metrics, :hackney_ssl_transport, 'api.stripe.com', 443, "api.stripe.com", [], nil, nil, nil, true, :hackney_pool, 5000, false, 5, false, 5, nil, nil, nil, :undefined, :start, nil, :normal, false, false, false, :undefined, false, nil, :waiting, nil, 4096, "", [], :undefined, nil, nil, nil, nil, :undefined, nil}}, :infinity)
    ** (EXIT) no process
             (stdlib) gen_server.erl:212: :gen_server.call/3
                      src/hackney_client/hackney_manager.erl:66: :hackney_manager.init_request/1
                      src/hackney_client/hackney_manager.erl:56: :hackney_manager.new_request/1
                      src/hackney_connect/hackney_connect.erl:183: :hackney_connect.socket_from_pool/4
                      src/hackney_connect/hackney_connect.erl:36: :hackney_connect.connect/5
                      src/hackney_client/hackney.erl:321: :hackney.request/5
                      lib/httpoison/base.ex:394: HTTPoison.Base.request/9
    (stripity_stripe) lib/stripe.ex:61: Stripe.make_request/5

My Mix.exs file says
  def application do
    [applications: [:logger, :stripity_stripe],
     mod: {TakeMyMoney, []}]
  end

  defp deps do
    [ {:stripity_stripe, "~> 0.3.0"} ]
  end

I'm probably botching something simple here. Thanks for taking a look.

Try:
  def application do
    [applications: [:logger, :stripe],
     mod: {TakeMyMoney, []}]
  end

take_my_money$ iex -S mix
Erlang/OTP 18 [erts-7.1] [source] [64-bit] [smp:8:8] [async-threads:10] [hipe] [kernel-poll:false] [dtrace]

Compiled lib/take_my_money.ex
Generated take_my_money app

=INFO REPORT==== 1-Dec-2015::13:12:42 ===
    application: logger
    exited: stopped
    type: temporary
** (Mix) Could not start application stripe: could not find application file: stripe.app

Hmmm - yeah "no process" means it hasn't started - asking Johnny Winn for some quick help. Gimme just a minute here...

Hooray for @nurugger07! Thanks Johnny - I was wondering if that was the problem. I'll push again in 3...2...1

@napcs it's not you. The Stripe.start is actually calling HTTPoison.Base.start which ensures that HTTPoison is running.

OK up your dependency to 0.5 and you're good! Thanks Johnny :)

@robconery no problem, any time :)


Adding search bar to website

A search bar would improve the functionality of the website and also improve the user experience. I am a GSSOC 22 contributor. Please assign this issue to me.

/assign


Can't download the zip file :/

Im trying to download the zipfile and it loads and evertyhing but then nohting happens it just doesnt download any tips or help I can get?

The file may get deleted upon download by Windows Defender, try to download after disabling it.


Final update

Merging in the latest commits from vercel.

QA'd the deploy and functionality looks to be 👌

I've added commit with changes since there was an issue that cartId left in cookies after removal last item in Cart. In this case we couldn't add a new item to the Cart afterwards. I attach screenshots before and after.
Before:
https://github.com/bigcommerce/nextjs-commerce/assets/67792608/f824d74a-22b2-40ac-b600-8fe291e5a334
After:
https://github.com/bigcommerce/nextjs-commerce/assets/67792608/334243f3-b085-4ce7-9823-e158c6a774cb


Update http-response-codes.md

Modified this part to clarify: backend response changed to backend response time
504 – Request timeout
HTTP 504 errors are presented if a request is sent to application gateways using V2 SKU, and the backend response time exceeds the time-out value associated to the listener's rule. This value is defined in the HTTP setting.

@greg-lindsay - Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Modifying EC2 instance attributes

Hello,
Firstly, thank you so much for creating this.  It's solved so many problems I was having with AWS controller code testing!
This is more of a question than an issue, so apologies if this is in the wrong place.
I am testing some code that has some validation checking around how long an instance has been running and that's problematic to test here as it's impractical to wait for the 10 minute threshold in the test to pass, especially when the entire test suite takes under a minute!  Are you aware of any way in which I can modify attributes of mock created EC2 instances so that I can push the launchtime back in time a bit please?
Thanks again!

Can you use freezegun and then have it move the time forward?
This won't work if you are using the standalone server mode.

Hi Spulec,
Thanks for the response and also apologies for my delay in responding to you.  This looks like it'll do exactly what I need, so I'll take a look now.
Thank you!


Finder float window too small with vertical splits

Is your feature request related to a problem? Please describe.
When using :Lspsaga finder with 4 windows side by side, the window is too small to be useful:

Describe the solution you'd like
Finder should have an option to utilize the whole screen space rather than be limited by the window width where the command was run.
Like this (from :FzfLua lsp_finder)

Describe alternatives you've considered
:FzfLua lsp_finder :/
Additional context
opening from left most window:

that was quick 😮  thank you!

The right most one isn't affected by the fix though:

I'm assuming the left window ("REFERENCES") needs similar attention.

how about now

a little broken

thanks for report it how about now.

except pic one other works fine i think

increased.

seems better than before

hardcore min is 50 maybe not good in your screen ? for me it works like this


Update setup.py to use new version of pandas and pyarrow

Hey,
Thanks for this.  I will release a new version with these updates.  Any particular reason you wanted the latest versions?


Fix bdist_pex interpreter cache directory

Running bdist_pex results in the creation of a directory literally
named {pex_root}. Within it is a subdirectory named interpreter.
This fixes it by following behavior similar to that found at https://github.com/pantsbuild/pex/blob/master/pex/bin/pex.py#L536.

thanks for the PR @brbsix - I'm planning to cut the 1.1.12 release this week which this should go out in.
@digwanderlust - FYI.


[Platform] Question of Darwin build option

Reproduction steps
I have a question regarding Darwin builds.

When building Darwin App (src/darwin), is there a Build Option used for libCHIP.a?
Is there anything I can do to build the library with minimal library size?

Platform
darwin
Platform Version(s)
No response
Type
Manually tested with SDK
(Optional) If manually tested please explain why this is only manually tested
No response
Anything else?
No response

When building Darwin App (src/darwin), is there a Build Option used for libCHIP.a?

It depends on exactly how you are building it, and exactly what you mean by "Build Option".

Is there anything I can do to build the library with minimal library size?

The thing that would probably help the most here is stripping debug symbols from the built library.
All of this only matters if you're not using the system Matter.framework, of course.

@bzbarsky-apple
Thank you for your response.
I'm looking for a way to reduce the size of the libCHIP.a image when building the Matter.Framework. Using this, we are trying to implement a Matter-supported iOS App.
Please ask if there are any additional places to check.

Additionally, can you tell me how to clear the debug symbol you mentioned?

@joonhaengHeo If you are building your own Matter.framework, presumably see src/darwin/Framework/chip_xcode_build_connector.sh for the builds that build libCHIP.  Make sure you are not building a Debug configuration.  Past that, you could try setting optimize_debug_level="s" in the gn args in that script.
And for removing the debug symbols, strip, but which exact arguments depends on exactly what you are trying to do.

Please re-open if you still have questions here.


Automatic way to update versions (Example 7.2.1 to 7.3.0) ?

Hi @jeremylong,
Can you please share if we have any update through command line using any argument or a command to update the versions for dependency check or we have to go through the entire process of downloading and extracting the zip. I have already followed all the steps mentioned here https://jeremylong.github.io/DependencyCheck/dependency-check-cli/index.html but I dont see the version upgrade happening? Can you please advise if I am missing anything in here ?
I am trying to upgrade the version to upgrade the apache-common-text issues.

Currently, the CLI does not have an automated upgrade process. You would currently need to download the zip and extracting the zip.  If using a mac - we do have a homebrew formula.


ISPN-7131 Random failures in SharedStoreInvalidationDuringRehashTest

https://issues.jboss.org/browse/ISPN-7131

@slaskawi I have completely forgotten that this is still open, thanks for reminding. Updated.


Slides.update() should return a future allowing blocking until it is complete

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[x] feature request
Current behavior:
The documentation for the slides component tells us to call update() when child slides are added or removed. The problem is that the app has no way of knowing when this process has completed.
Expected behavior:
It would be very helpful if update() could return a future such as a Promise that resolves once the update process has completed.
Steps to reproduce:
Create a slides component backed by an ngFor loop over an array. Attempt to slide to a recently added slide.
<ion-slides>
<ion-slide *ngFor="let quote of quotes">{{quote}}</ion-slide>
</ion-slides>

addQuote(): void {
  this.quotes.push('lorem ipsum');
  this.slides.update();
  // this won't work because the update hasn't completed yet
  this.slides.slideTo(this.quotes.length-1);

  // this works but is well below optimal
  setTimeout(() => {
    this.slides.slideTo(this.quotes.length-1);
  }, 200);

  // this is what I would like to do
  this.slides.update().then(() => {
    this.slides.slideTo(this.quotes.length-1);
  });
}

Other information:

Potentially related: #6703.

Hello! Thank you for the feature request. I'm going to move this issue over to our internal list of feature requests for evaluation. We are continually prioritizing all requests that we receive with outstanding issues. We are extremely grateful for your feedback, but it may not always be our next priority. I'll copy the issue back to this repository when we have begun implementation. Thanks!

This issue was moved to ionic-team/ionic-feature-requests#331

Is there any way to keep track of the internal issue? Will you update this issue when it is fixed? Right now I have a timeout setup to wait for the update which is extremely ugly.


Update data transfer methods to use refactored CS3APIs

I guess the list is anyway very short, often with a single element, right?

Yes, for most of the drivers, the response will have two elements, simple and tus. Since we weren't able to configure the tus client last time, we just opt for simple for now. I'll fix that soon. Regarding the list comprehension, the protocol parameter would be unique, so the filtered list will just have a single element. I don't think we can write comprehensions without storing the values in a container, so had to use this.


How to install and configure GeyserMC for your CivilHost Minecraft Server

GeyserMC is a proxy for Minecraft that allows Bedrock edition players to join Java edition servers.

GeyserMC is a proxy, meaning worlds and playerdata cannot exist on GeyserMC. GeyserMC was built to bridge the connection between Bedrock and Minecraft Java Editon. This is to prevent the need for two servers. CivilHost recommends this as it is the easiest way to set up GeyserMC.

What Bedrock clients can connect?

The following client(s) can connect to your Minecraft server:

Minecraft for Windows 10

Minecraft Pocket Editon for IOS and Android devices

Kindle Fire

Fire TV

Installing GeyserMC

The quickest way to install GeyserMC for your CivilHost Minecraft server is to install it as a plugin.

Download the latest version of GeyserMC [here](https://ci.opencollab.dev/job/GeyserMC/job/Geyser/job/master/)

Go to your [control panel](https://control.civilhost.net) and STOP your server

At the top of your screen, navigate to Files

Locate and open the plugins folder

Drag and drop your GeyserMC plugin from your downloads folder

Return to the Console tab and start your server

Test your bedrock connection with the following command “geyser connectiontest IP:PORT”

Your GeyserMC configuration should automatically adjust to your Minecraft server’s IP and Port, if the connection happens to fail, check the config.yml file found in /plugins/GeyserMC/config.yml

Congratulations! Your Bedrock players should now be able to successfully connect to your Minecraft server using GeyserMC!

If you run into any issues with the following setup, please do not hesitate to open a ticket [here](https://civilhost.net/hosting/submitticket.php).

Article written by rooveinnit

https://wiki.geysermc.org/geyser/setup/
explain that the server owner will need to get a second port for geyser and edit the config

Closed due to inactivity.


Get all tracks from a Playlist object using pagination

Given a Playlist object, there is only the Playlist#getTracks() method, which does not seem to offer getting all tracks of the playlist (limit is 100).
The problem that I encountered, is the following:
If api.getPlaylistTracks() (which does offer support for pagination) is used, the tracks appear to be sorted alphabetically.
If you use Playlist#getTracks() having already received a Playlist object, the tracks are in proper order, that is, as they are in the playlist checking in the spotify client.
As I really need them in order, I tried to use the Playlist object approach, but got stuck since there is no pagination available.

Pagination will be implemented in version 2 on which I'm currently working.

I'll close this. Please follow #102 for updates.

Version 2.0.0-RC1 was just released.


attentive learning table

i haved singed
------------------ 原始邮件 ------------------
发件人:                                                                                "google-research/google-research"                                                                                @.>;
发送时间: 2021年3月26日(星期五) 晚上9:19
@.>;
抄送: "な法--言说<EMAIL_ADDRESS>主题: Re: [google-research/google-research] attentive learning table (#655)
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.
What to do if you already signed the CLA
Individual signers
It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.
Corporate signers
Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.
ℹ Googlers: Go here for more info.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


az login with managed identity times out.

This is an autogenerated template. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
:open_mouth:

HTTPConnectionPool(host='<IP_ADDRESS>', port=80): Max retries exceeded with url: /metadata/identity/oauth2/token?resource=https%3A%2F%2Fmanagement.core.windows.net%2F&api-version=2018-02-01&msi_res_id=%2Fsubscriptions%2F55ac4ce5-f395-4dca-90bd-f92edcdd82b7%2Fresourcegroups%2Ftest-project-setup%2Fproviders%2FMicrosoft.ManagedIdentity%2FuserAssignedIdentities%2FHRC-API-Deployer (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x11047ce48>: Failed to establish a new connection: [Errno 60] Operation timed out'))Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/urllib3/connection.py", line 159, in _new_conn
    (self._dns_host, self.port), self.timeout, **extra_kw)
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/urllib3/util/connection.py", line 80, in create_connection
    raise err
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/urllib3/util/connection.py", line 70, in create_connection
    sock.connect(sa)
TimeoutError: [Errno 60] Operation timed out

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/urllib3/connectionpool.py", line 600, in urlopen
    chunked=chunked)
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/urllib3/connectionpool.py", line 354, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/usr/local/opt/python/Frameworks/Python.framework/Versions/3.7/lib/python3.7/http/client.py", line 1229, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/local/opt/python/Frameworks/Python.framework/Versions/3.7/lib/python3.7/http/client.py", line 1275, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/local/opt/python/Frameworks/Python.framework/Versions/3.7/lib/python3.7/http/client.py", line 1224, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/local/opt/python/Frameworks/Python.framework/Versions/3.7/lib/python3.7/http/client.py", line 1016, in _send_output
    self.send(msg)
  File "/usr/local/opt/python/Frameworks/Python.framework/Versions/3.7/lib/python3.7/http/client.py", line 956, in send
    self.connect()
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/urllib3/connection.py", line 181, in connect
    conn = self._new_conn()
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/urllib3/connection.py", line 168, in _new_conn
    self, "Failed to establish a new connection: %s" % e)
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x11047ce48>: Failed to establish a new connection: [Errno 60] Operation timed out

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/requests/adapters.py", line 449, in send
    timeout=timeout
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/urllib3/connectionpool.py", line 638, in urlopen
    _stacktrace=sys.exc_info()[2])
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/urllib3/util/retry.py", line 398, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='<IP_ADDRESS>', port=80): Max retries exceeded with url: /metadata/identity/oauth2/token?resource=https%3A%2F%2Fmanagement.core.windows.net%2F&api-version=2018-02-01&msi_res_id=%2Fsubscriptions%2F55ac4ce5-f395-4dca-90bd-f92edcdd82b7%2Fresourcegroups%2Ftest-project-setup%2Fproviders%2FMicrosoft.ManagedIdentity%2FuserAssignedIdentities%2FHRC-API-Deployer (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x11047ce48>: Failed to establish a new connection: [Errno 60] Operation timed out'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/knack/cli.py", line 206, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/azure/cli/core/commands/__init__.py", line 326, in execute
    raise ex
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/azure/cli/core/commands/__init__.py", line 384, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/azure/cli/core/commands/__init__.py", line 377, in _run_job
    six.reraise(*sys.exc_info())
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/six.py", line 693, in reraise
    raise value
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/azure/cli/core/commands/__init__.py", line 354, in _run_job
    result = cmd_copy(params)
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/azure/cli/core/commands/__init__.py", line 145, in __call__
    return self.handler(*args, **kwargs)
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/azure/cli/core/__init__.py", line 451, in default_command_handler
    return op(**command_args)
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/azure/cli/command_modules/profile/custom.py", line 111, in login
    return profile.find_subscriptions_in_vm_with_msi(username, allow_no_subscriptions)
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/azure/cli/core/_profile.py", line 288, in find_subscriptions_in_vm_with_msi
    msi_creds = MSIAuthentication(resource=resource, msi_res_id=identity_id)
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/msrestazure/azure_active_directory.py", line 689, in __init__
    self.set_token()
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/msrestazure/azure_active_directory.py", line 697, in set_token
    token_entry = self._vm_msi.get_token()
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/msrestazure/azure_active_directory.py", line 746, in get_token
    token_entry = self._retrieve_token_from_imds_with_retry()
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/msrestazure/azure_active_directory.py", line 766, in _retrieve_token_from_imds_with_retry
    result = requests.get(request_uri, params=payload, headers={'Metadata': 'true', 'User-Agent':self._user_agent})
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/requests/api.py", line 75, in get
    return request('get', url, params=params, **kwargs)
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/requests/api.py", line 60, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/requests/sessions.py", line 533, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/requests/sessions.py", line 646, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/Cellar/azure-cli/2.0.62/libexec/lib/python3.7/site-packages/requests/adapters.py", line 516, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='<IP_ADDRESS>', port=80): Max retries exceeded with url: /metadata/identity/oauth2/token?resource=https%3A%2F%2Fmanagement.core.windows.net%2F&api-version=2018-02-01&msi_res_id=%2Fsubscriptions%2F55ac4ce5-f395-4dca-90bd-f92edcdd82b7%2Fresourcegroups%2Ftest-project-setup%2Fproviders%2FMicrosoft.ManagedIdentity%2FuserAssignedIdentities%2FHRC-API-Deployer (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x11047ce48>: Failed to establish a new connection: [Errno 60] Operation timed out'))

To Reproduce:
Steps to reproduce the behavior. Note: Command arguments have been redacted.

Fill in additional info here
Run: az login --identity -u {}

Expected Behavior
A clear and concise description of what you expected to happen.
Environment Summary
Darwin-18.2.0-x86_64-i386-64bit
Python 3.7.3
Shell: bash

azure-cli                         2.0.62

Additional Context
Add any other context about the problem here.

Hi @wordrak,
Thanks a lot for your bug report (and for using our new and improved az feedback mechanism!). Can you tell us a little bit more about what you are trying to accomplish here?
Thanks,
Arun Chandrasekhar
Principal Program Manager
Azure Management Experience

@wordrak How did you install the Azure CLI? Want to see if this is related to #9030.

@achandmsft says:
@wordrak How did you install the Azure CLI? Want to see if this is related to #9030.

Stack trace indicates that the CLI is installed in the Homebrew Cellar.
However, the failure looks pretty different to me. This stack trace makes no mention of SSL failure, which leads me to believe it's not as likely to be about newtwork configuration.

I have the same problem with a Windows VM on Azure. I'm pasting the "Feedback" here, instead of opening a new issue.
Describe the bug
Command Name
az login --identity
Errors:
MSI: Failed to acquire tokens after 12 times
Traceback (most recent call last):
pip-install-dsvrkf2x\knack\knack\cli.py, ln 206, in invoke
cli\core\commands\__init__.py, ln 560, in execute
cli\core\commands\__init__.py, ln 618, in _run_jobs_serially
cli\core\commands\__init__.py, ln 611, in _run_job
Temp\pip-install-dsvrkf2x\six\six.py, ln 693, in reraise
cli\core\commands\__init__.py, ln 588, in _run_job
cli\core\commands\__init__.py, ln 297, in __call__
...
pip-install-dsvrkf2x\msrestazure\msrestazure\azure_active_directory.py, ln 584, in set_token
pip-install-dsvrkf2x\msrestazure\msrestazure\azure_active_directory.py, ln 632, in get_token
pip-install-dsvrkf2x\msrestazure\msrestazure\azure_active_directory.py, ln 674, in _retrieve_token_from_imds_with_retry
TimeoutError: MSI: Failed to acquire tokens after 12 times

To Reproduce:
Run following command in an Azure VM with identity.

az login --identity

Expected Behavior
This command has been working in this VM since yesterday. Today it doesn't.
Environment Summary
Windows-10-10.0.177

Additional Info
When run with --debug flag, output looks like this:
urllib3.connectionpool : Starting new HTTP connection (1): <IP_ADDRESS>:80
urllib3.connectionpool : http://<IP_ADDRESS>:80 "GET /metadata/identity/oauth2/token?resource=https%3A%2F%2Fmanagement.core.windows.net%2F&api-version=2018-02-01 HTTP/1.1" 500 21
msrestazure.azure_active_directory : MSI: Retrieving a token from http://<IP_ADDRESS>/metadata/identity/oauth2/token, with payload {'resource': 'https://management.core.windows.net/', 'api-version': '2018-02-01'}
msrestazure.azure_active_directory : MSI: wait: 0.1s and retry: 1
urllib3.connectionpool : Starting new HTTP connection (1): <IP_ADDRESS>:80
urllib3.connectionpool : http://<IP_ADDRESS>:80 "GET /metadata/identity/oauth2/token?resource=https%3A%2F%2Fmanagement.core.windows.net%2F&api-version=2018-02-01 HTTP/1.1" 500 21
msrestazure.azure_active_directory : MSI: Retrieving a token from http://<IP_ADDRESS>/metadata/identity/oauth2/token, with payload {'resource': 'https://management.core.windows.net/', 'api-version': '2018-02-01'}
msrestazure.azure_active_directory : MSI: wait: 0.1s and retry: 2
...

Same command has following debug output when successful, in another VM with virtually identical configuration:
urllib3.connectionpool : Starting new HTTP connection (1): <IP_ADDRESS>:80
urllib3.connectionpool : http://<IP_ADDRESS>:80 "GET /metadata/identity/oauth2/token?resource=https%3A%2F%2Fmanagement.core.windows.net%2F&api-version=2018-02-01 HTTP/1.1" 200 1604
msrestazure.azure_active_directory : MSI: Retrieving a token from http://<IP_ADDRESS>/metadata/identity/oauth2/token, with payload {'resource': 'https://management.core.windows.net/', 'api-version': '2018-02-01'}
msrestazure.azure_active_directory : MSI: Token retrieved
MSI: token was retrieved. Now trying to initialize local accounts...
msrest.universal_http.requests : Configuring retry: max_retries=4, backoff_factor=0.8, max_backoff=90
msrest.async_paging : Paging async iterator protocol is not available for SubscriptionPaged
msrest.universal_http : Configuring redirects: allow=True, max=30
msrest.universal_http : Configuring request: timeout=100, verify=True, cert=None
msrest.universal_http : Configuring proxies: ''
msrest.universal_http : Evaluate proxies against ENV settings: True
urllib3.connectionpool : Starting new HTTPS connection (1): management.azure.com:443
urllib3.connectionpool : https://management.azure.com:443 "GET /subscriptions?api-version=2016-06-01 HTTP/1.1" 200 347
Event: CommandInvoker.OnTransformResult [<function _resource_group_transform at 0x033B1588>, <function _x509_from_base64_to_hex_transform at 0x033B15D0>]
Event: CommandInvoker.OnFilterResult []
[
  {
      //my account data
  }
]
...

After costing a day of time, stopping and then starting the VM again solved the issue. Mind you that a reboot has not worked; had to Stop and then Start from the Azure web interface (deallocation and reallocation of the VM).

Given that this is solved by a hard stop/start of the VM, I suspect this is an MSI issue, not a CLI issue. I'm going to close this out for now. @achandmsft, mind forwarding this issue on to the appropriate service team?

@marstr it cannot be an MSI issue, because the issue was originally reported on Mac, then on a Windows VM in Azure and I am now having the same issue on Ubuntu VM.