The behaviors of commands nice and renice are different from those of Linux

This bug-tracker is monitored by developers and other technical types.  We like detail!  So please use this form and tell us, concisely but precisely, what's up.  Please fill out ALL THE FIELDS!
If you have a feature request, please post to the UserVoice.
Important: When reporting BSODs or security issues, DO NOT attach memory dumps, logs, or traces to Github issues. Instead, send dumps/traces to<EMAIL_ADDRESS>referencing the GitHub bug number. Ideally, please configure your machine to capture minidumps, repro the issue, and send the minidump from "C:\Windows\minidump".

Your Windows build number:  (Type ver at a Windows Command Prompt)
17025

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)
First, try these commands in Bash or Dash to examine the behavior of nice.

nice -n 0 echo
nice -n 1 echo
nice -n -1 echo
sudo nice -n 1 echo
sudo nice -n -1 echo

Second, open 2 Bash or Dash shells using, for example, tmux.  Run top in one of them by:
Shell # 0:
top

Specify the PID of the top and execute these command to examine renice in the other:
Shell # 1:
renice -n -1 -p $THE_PID_OF_TOP
renice -n -2 -p $THE_PID_OF_TOP
renice -n -1 -p $THE_PID_OF_TOP
renice -n 0 -p $THE_PID_OF_TOP
renice -n 1 -p $THE_PID_OF_TOP
renice -n 0 -p $THE_PID_OF_TOP
sudo renice -n -1 -p $THE_PID_OF_TOP

By these operations in WSL, I got the following results:
$ nice -n 0 echo
nice: cannot set niceness: Permission denied
$ nice -n 1 echo
nice: cannot set niceness: Permission denied
$ nice -n -1 echo
nice: cannot set niceness: Permission denied
$ sudo nice -n 1 echo
$ sudo nice -n -1 echo

$ renice -n -1 -p 73
73 (process ID) old priority 0, new priority -1
$ renice -n -2 -p 73
73 (process ID) old priority -1, new priority -2
$ renice -n 0 -p 73
73 (process ID) old priority -2, new priority 0
$ renice -n 1 -p 73
73 (process ID) old priority 0, new priority 1
$ renice -n 0 -p 73
73 (process ID) old priority 1, new priority 0
$ sudo renice -n -1 -p 73
73 (process ID) old priority 0, new priority -1

The final nice value of top that is shown in it was 1.
73 user      21   1  111860   1964   1396 R   0.0  0.0   0:00.18 top

What's wrong / what should be happening instead:

The Linux's behavior:
$ nice -n 0 echo
$ nice -n 1 echo
$ nice -n -1 echo
nice: cannot set niceness: Permission denied
$ sudo nice -n 1 echo
$ sudo nice -n -1 echo

$ renice -n -1 -p 31854
renice: failed to set priority for 31854 (process ID): Permission denied
$ renice -n -2 -p 31854
renice: failed to set priority for 31854 (process ID): Permission denied
$ renice -n 0 -p 31854
31854 (process ID) old priority 0, new priority 0
$  renice -n 1 -p 31854
31854 (process ID) old priority 0, new priority 1
$ renice -n 0 -p 31854
renice: failed to set priority for 31854 (process ID): Permission denied
$ sudo renice -n -1 -p 31854
31854 (process ID) old priority 1, new priority -1

The final nice value of top that is shown in it was -1.
31854 user     19  -1   47396   4032   3348 R   0.3  0.0   0:02.24 top

In short:

Running nice with non-negative values must not require the privilege.
Running renice with lesser values must require the privilege.
The nice values of processes after reniced to negative values are wrong.

This issue prevents us from using https://github.com/zplug/zplug because it uses nice.  I get the following error messages when I enable zplug and run zsh:
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted
__load__:47: nice(5) failed: operation not permitted

Strace of the failing command, if applicable:  (If some_command is failing, then run strace -o some_command.strace -f some_command some_args, and link the contents of some_command.strace in a gist here)

LANG=C strace -o nice.strace -f nice -n 0 echo
cat nice.strace

5755  execve("/usr/bin/nice", ["nice", "-n", "0", "echo"], [/* 15 vars */]) = 0
5755  brk(NULL)                         = 0x1eaa000
5755  access("/etc/ld.so.nohwcap", F_OK) = -1 ENOENT (No such file or directory)
5755  mmap(NULL, 8192, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7fe5260c0000
5755  access("/etc/ld.so.preload", R_OK) = -1 ENOENT (No such file or directory)
5755  open("/etc/ld.so.cache", O_RDONLY|O_CLOEXEC) = 3
5755  fstat(3, {st_mode=S_IFREG|0644, st_size=47586, ...}) = 0
5755  mmap(NULL, 47586, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7fe5260b4000
5755  close(3)                          = 0
5755  access("/etc/ld.so.nohwcap", F_OK) = -1 ENOENT (No such file or directory)
5755  open("/lib/x86_64-linux-gnu/libc.so.6", O_RDONLY|O_CLOEXEC) = 3
5755  read(3, "\177ELF\2\1\1\3\0\0\0\0\0\0\0\0\3\0>\0\1\0\0\0P\t\2\0\0\0\0\0"..., 832) = 832
5755  fstat(3, {st_mode=S_IFREG|0755, st_size=1868984, ...}) = 0
5755  mmap(NULL, 3971488, PROT_READ|PROT_EXEC, MAP_PRIVATE|MAP_DENYWRITE, 3, 0) = 0x7fe525a30000
5755  mprotect(0x7fe525bf0000, 2097152, PROT_NONE) = 0
5755  mmap(0x7fe525df0000, 24576, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_DENYWRITE, 3, 0x1c0000) = 0x7fe525df0000
5755  mmap(0x7fe525df6000, 14752, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_ANONYMOUS, -1, 0) = 0x7fe525df6000
5755  close(3)                          = 0
5755  mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7fe5260b0000
5755  mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7fe5260a0000
5755  mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7fe526090000
5755  arch_prctl(ARCH_SET_FS, 0x7fe5260a0700) = 0
5755  mprotect(0x7fe525df0000, 16384, PROT_READ) = 0
5755  mprotect(0x607000, 4096, PROT_READ) = 0
5755  mprotect(0x7fe526025000, 4096, PROT_READ) = 0
5755  munmap(0x7fe5260b4000, 47586)     = 0
5755  brk(NULL)                         = 0x1eaa000
5755  brk(0x1ecb000)                    = 0x1ecb000
5755  getpriority(PRIO_PROCESS, 0)      = 20
5755  setpriority(PRIO_PROCESS, 0, 0)   = -1 EACCES (Permission denied)
5755  write(2, "nice: ", 6)             = 6
5755  write(2, "cannot set niceness", 19) = 19
5755  write(2, ": Permission denied", 19) = 19
5755  write(2, "\n", 1)                 = 1
5755  execve("/usr/local/sbin/echo", ["echo"], [/* 15 vars */]) = -1 ENOENT (No such file or directory)
5755  execve("/usr/local/bin/echo", ["echo"], [/* 15 vars */]) = -1 ENOENT (No such file or directory)
5755  execve("/usr/sbin/echo", ["echo"], [/* 15 vars */]) = -1 ENOENT (No such file or directory)
5755  execve("/usr/bin/echo", ["echo"], [/* 15 vars */]) = -1 ENOENT (No such file or directory)
5755  execve("/sbin/echo", ["echo"], [/* 15 vars */]) = -1 ENOENT (No such file or directory)
5755  execve("/bin/echo", ["echo"], [/* 15 vars */]) = 0
5755  brk(NULL)                         = 0xb10000
5755  access("/etc/ld.so.nohwcap", F_OK) = -1 ENOENT (No such file or directory)
5755  mmap(NULL, 8192, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f228ad10000
5755  access("/etc/ld.so.preload", R_OK) = -1 ENOENT (No such file or directory)
5755  open("/etc/ld.so.cache", O_RDONLY|O_CLOEXEC) = 3
5755  fstat(3, {st_mode=S_IFREG|0644, st_size=47586, ...}) = 0
5755  mmap(NULL, 47586, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f228ad04000
5755  close(3)                          = 0
5755  access("/etc/ld.so.nohwcap", F_OK) = -1 ENOENT (No such file or directory)
5755  open("/lib/x86_64-linux-gnu/libc.so.6", O_RDONLY|O_CLOEXEC) = 3
5755  read(3, "\177ELF\2\1\1\3\0\0\0\0\0\0\0\0\3\0>\0\1\0\0\0P\t\2\0\0\0\0\0"..., 832) = 832
5755  fstat(3, {st_mode=S_IFREG|0755, st_size=1868984, ...}) = 0
5755  mmap(NULL, 3971488, PROT_READ|PROT_EXEC, MAP_PRIVATE|MAP_DENYWRITE, 3, 0) = 0x7f228a630000
5755  mprotect(0x7f228a7f0000, 2097152, PROT_NONE) = 0
5755  mmap(0x7f228a9f0000, 24576, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_DENYWRITE, 3, 0x1c0000) = 0x7f228a9f0000
5755  mmap(0x7f228a9f6000, 14752, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_ANONYMOUS, -1, 0) = 0x7f228a9f6000
5755  close(3)                          = 0
5755  mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f228ad00000
5755  mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f228acf0000
5755  mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f228ace0000
5755  arch_prctl(ARCH_SET_FS, 0x7f228acf0700) = 0
5755  mprotect(0x7f228a9f0000, 16384, PROT_READ) = 0
5755  mprotect(0x606000, 4096, PROT_READ) = 0
5755  mprotect(0x7f228ac25000, 4096, PROT_READ) = 0
5755  munmap(0x7f228ad04000, 47586)     = 0
5755  brk(NULL)                         = 0xb10000
5755  brk(0xb31000)                     = 0xb31000
5755  fstat(1, {st_mode=S_IFCHR|0620, st_rdev=makedev(136, 1), ...}) = 0
5755  write(1, "\n", 1)                 = 1
5755  close(1)                          = 0
5755  close(2)                          = 0
5755  exit_group(0)                     = ?
5755  +++ exited with 0 +++

See our contributing instructions for assistance.

5755  setpriority(PRIO_PROCESS, 0, 0)   = -1 EACCES (Permission denied)

#1838

#1838 does not refer to renice.  The difference from nice is the 2nd argument of setpriority and getpriority.
LANG=C.UTF-8 strace -o renice.strace -f renice -n -1 -p 78
cat renice.strace

81    execve("/usr/bin/renice", ["renice", "-n", "-1", "-p", "78"], [/* 19 vars */]) = 0
81    brk(NULL)                         = 0x1591000
81    access("/etc/ld.so.nohwcap", F_OK) = -1 ENOENT (No such file or directory)
81    mmap(NULL, 8192, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f6004200000
81    access("/etc/ld.so.preload", R_OK) = -1 ENOENT (No such file or directory)
81    open("/etc/ld.so.cache", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=47586, ...}) = 0
81    mmap(NULL, 47586, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f60041f4000
81    close(3)                          = 0
81    access("/etc/ld.so.nohwcap", F_OK) = -1 ENOENT (No such file or directory)
81    open("/lib/x86_64-linux-gnu/libc.so.6", O_RDONLY|O_CLOEXEC) = 3
81    read(3, "\177ELF\2\1\1\3\0\0\0\0\0\0\0\0\3\0>\0\1\0\0\0P\t\2\0\0\0\0\0"..., 832) = 832
81    fstat(3, {st_mode=S_IFREG|0755, st_size=1868984, ...}) = 0
81    mmap(NULL, 3971488, PROT_READ|PROT_EXEC, MAP_PRIVATE|MAP_DENYWRITE, 3, 0) = 0x7f6003a30000
81    mprotect(0x7f6003bf0000, 2097152, PROT_NONE) = 0
81    mmap(0x7f6003df0000, 24576, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_DENYWRITE, 3, 0x1c0000) = 0x7f6003df0000
81    mmap(0x7f6003df6000, 14752, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_ANONYMOUS, -1, 0) = 0x7f6003df6000
81    close(3)                          = 0
81    mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f60041f0000
81    mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f60041e0000
81    mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f60041d0000
81    arch_prctl(ARCH_SET_FS, 0x7f60041e0700) = 0
81    mprotect(0x7f6003df0000, 16384, PROT_READ) = 0
81    mprotect(0x601000, 4096, PROT_READ) = 0
81    mprotect(0x7f6004025000, 4096, PROT_READ) = 0
81    munmap(0x7f60041f4000, 47586)     = 0
81    brk(NULL)                         = 0x1591000
81    brk(0x15b2000)                    = 0x15b2000
81    open("/usr/lib/locale/locale-archive", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=2704400, ...}) = 0
81    mmap(NULL, 2704400, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f600379b000
81    close(3)                          = 0
81    open("/usr/share/locale/locale.alias", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=2995, ...}) = 0
81    read(3, "# Locale name alias data base.\n#"..., 4096) = 2995
81    read(3, "", 4096)                 = 0
81    close(3)                          = 0
81    open("/usr/lib/locale/C.UTF-8/LC_IDENTIFICATION", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=168, ...}) = 0
81    mmap(NULL, 168, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f6004204000
81    close(3)                          = 0
81    open("/usr/lib/x86_64-linux-gnu/gconv/gconv-modules.cache", O_RDONLY) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=26258, ...}) = 0
81    mmap(NULL, 26258, PROT_READ, MAP_SHARED, 3, 0) = 0x7f60041f9000
81    close(3)                          = 0
81    open("/usr/lib/locale/C.UTF-8/LC_MEASUREMENT", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=23, ...}) = 0
81    mmap(NULL, 23, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f6004203000
81    close(3)                          = 0
81    open("/usr/lib/locale/C.UTF-8/LC_TELEPHONE", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=47, ...}) = 0
81    mmap(NULL, 47, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f6004202000
81    close(3)                          = 0
81    open("/usr/lib/locale/C.UTF-8/LC_ADDRESS", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=131, ...}) = 0
81    mmap(NULL, 131, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f60041f8000
81    close(3)                          = 0
81    open("/usr/lib/locale/C.UTF-8/LC_NAME", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=62, ...}) = 0
81    mmap(NULL, 62, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f60041f7000
81    close(3)                          = 0
81    open("/usr/lib/locale/C.UTF-8/LC_PAPER", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=34, ...}) = 0
81    mmap(NULL, 34, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f60041f6000
81    close(3)                          = 0
81    open("/usr/lib/locale/C.UTF-8/LC_MESSAGES", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
81    close(3)                          = 0
81    open("/usr/lib/locale/C.UTF-8/LC_MESSAGES/SYS_LC_MESSAGES", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=48, ...}) = 0
81    mmap(NULL, 48, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f60041f5000
81    close(3)                          = 0
81    open("/usr/lib/locale/C.UTF-8/LC_MONETARY", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=270, ...}) = 0
81    mmap(NULL, 270, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f60041f4000
81    close(3)                          = 0
81    open("/usr/lib/locale/C.UTF-8/LC_COLLATE", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=1514422, ...}) = 0
81    mmap(NULL, 1514422, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f600405e000
81    close(3)                          = 0
81    open("/usr/lib/locale/C.UTF-8/LC_TIME", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=2470, ...}) = 0
81    mmap(NULL, 2470, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f60041f3000
81    close(3)                          = 0
81    open("/usr/lib/locale/C.UTF-8/LC_NUMERIC", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=50, ...}) = 0
81    mmap(NULL, 50, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f60041f2000
81    close(3)                          = 0
81    open("/usr/lib/locale/C.UTF-8/LC_CTYPE", O_RDONLY|O_CLOEXEC) = 3
81    fstat(3, {st_mode=S_IFREG|0644, st_size=158652, ...}) = 0
81    mmap(NULL, 158652, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f6004037000
81    close(3)                          = 0
81    getpriority(PRIO_PROCESS, 78)     = 20
81    setpriority(PRIO_PROCESS, 78, -1) = 0
81    getpriority(PRIO_PROCESS, 78)     = 21
81    open("/usr/share/locale/C.UTF-8/LC_MESSAGES/util-linux.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
81    open("/usr/share/locale/C.utf8/LC_MESSAGES/util-linux.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
81    open("/usr/share/locale/C/LC_MESSAGES/util-linux.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
81    open("/usr/share/locale-langpack/C.UTF-8/LC_MESSAGES/util-linux.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
81    open("/usr/share/locale-langpack/C.utf8/LC_MESSAGES/util-linux.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
81    open("/usr/share/locale-langpack/C/LC_MESSAGES/util-linux.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
81    fstat(1, {st_mode=S_IFCHR|0620, st_rdev=makedev(136, 1), ...}) = 0
81    write(1, "78 (process ID) old priority 0, "..., 48) = 48
81    close(1)                          = 0
81    close(2)                          = 0
81    exit_group(0)                     = ?
81    +++ exited with 0 +++


Fix source links in docs

Hi, thanks for creating this package, it's looking very promising!
This PR fixes the source links in the generated docs, so that they now reference the tagged version in github.

haha thank you for fixing the copypasta fail!!

docs update has been pushed to https://hexdocs.pm/mavis/Type.html


SABATH does not need to depened on FlatDict from Cloudmesh

PR #11  introduced FlatDict class implemented in the common set of utilities available in Cloudmesh: cloudmesh/cloudmesh-common/cloudmesh/common//FlatDict.py. The usage suggest to be equivalent for what is offered by standard str.Template to replace braced identifiers such as {python}. Replacing FlatDict with str.Template will keep the main SABATH code base self-contained without external dependences.

Here's my output log:
$ python bin/sabath || python3 bin/sabathx || sh bin/sabath

zsh: command not found: python
Traceback (most recent call last):
  File "sabath/bin/sabath", line 16, in <module>
    import sabath.py2or3
  File "sabath/sabath/py2or3.py", line 22, in <module>
    import sabath.argvparse
  File "sabath/sabath/argvparse.py", line 15, in <module>
    from .commands import dispatch
  File "sabath/sabath/commands.py", line 18, in <module>
    from .executors import ContextExecutor
  File "sabath/sabath/executors.py", line 11, in <module>
    from cloudmesh.common import FlatDict
ModuleNotFoundError: No module named 'cloudmesh'


haddock-api build fails on macOS sierra

Hi,
For some reason, installation of haddock-api fails on my machine running macOS Sierra 10.12.1. I just made a clean installation of Haskell environment using Homebrew:
$ brew install ghc
$ brew install cabal-install

And then, when trying to install haddock I ran into a problem with haddock-api build. This persists even when I attempt to install only haddock api using
$ cabal install haddock-api

With output of a few warnings and three Duplicate instance declarations errors:
cabal: Entering directory '/var/folders/w3/whgjdx991_b_sd6jvsrmk7xr0000gn/T/cabal-tmp-44525/haddock-api-2.17.3'
Configuring haddock-api-2.17.3...
Building haddock-api-2.17.3...
Preprocessing library haddock-api-2.17.3...
[ 1 of 38] Compiling Paths_haddock_api ( dist/build/autogen/Paths_haddock_api.hs, dist/build/Paths_haddock_api.o )
[ 2 of 38] Compiling Haddock.Version  ( src/Haddock/Version.hs, dist/build/Haddock/Version.o )
[ 3 of 38] Compiling Haddock.Syb      ( src/Haddock/Syb.hs, dist/build/Haddock/Syb.o )
[ 4 of 38] Compiling Haddock.Parser   ( src/Haddock/Parser.hs, dist/build/Haddock/Parser.o )
[ 5 of 38] Compiling Haddock.GhcUtils ( src/Haddock/GhcUtils.hs, dist/build/Haddock/GhcUtils.o )

src/Haddock/GhcUtils.hs:20:1: warning: [-Wunused-imports]
    The import of ‘Data.Function’ is redundant
      except perhaps to import instances from ‘Data.Function’
    To import instances alone, use: import Data.Function()

src/Haddock/GhcUtils.hs:27:1: warning: [-Wunused-imports]
    The import of ‘RdrName’ is redundant
      except perhaps to import instances from ‘RdrName’
    To import instances alone, use: import RdrName()

src/Haddock/GhcUtils.hs:28:1: warning: [-Wunused-imports]
    The import of ‘GhcMonad’ is redundant
      except perhaps to import instances from ‘GhcMonad’
    To import instances alone, use: import GhcMonad()

src/Haddock/GhcUtils.hs:30:1: warning: [-Wunused-imports]
    The import of ‘UniqFM’ is redundant
      except perhaps to import instances from ‘UniqFM’
    To import instances alone, use: import UniqFM()
[ 6 of 38] Compiling Haddock.Backends.Xhtml.Types ( src/Haddock/Backends/Xhtml/Types.hs, dist/build/Haddock/Backends/Xhtml/Types.o )
[ 7 of 38] Compiling Haddock.Backends.Hyperlinker.Types ( src/Haddock/Backends/Hyperlinker/Types.hs, dist/build/Haddock/Backends/Hyperlinker/Types.o )
[ 8 of 38] Compiling Haddock.Types    ( src/Haddock/Types.hs, dist/build/Haddock/Types.o )

src/Haddock/Types.hs:451:10: error:
    Duplicate instance declarations:
      instance NFData Name -- Defined at src/Haddock/Types.hs:451:10
      instance NFData Name -- Defined in ‘Name’

src/Haddock/Types.hs:452:10: error:
    Duplicate instance declarations:
      instance NFData OccName -- Defined at src/Haddock/Types.hs:452:10
      instance NFData OccName -- Defined in ‘OccName’

src/Haddock/Types.hs:453:10: error:
    Duplicate instance declarations:
      instance NFData ModuleName
        -- Defined at src/Haddock/Types.hs:453:10
      instance NFData ModuleName -- Defined in ‘Module’
cabal: Leaving directory '/var/folders/w3/whgjdx991_b_sd6jvsrmk7xr0000gn/T/cabal-tmp-44525/haddock-api-2.17.3'

I'm not sure whether this is a problem of my machine and if there is a way to debug this? Has anyone run into similar issue? Any help would be appreciated.

Same error in here. I tried to use all the versions available and none of them work.

Also facing the same issue here. Not sure if anyone has found a workaround yet?

The problem is that the homebrew installation is using a patched unofficial release of GHC and haddock hasn't been updated yet for this new release.
@ilovezfs

GHC 8.0.2_rc1 has the same problem. https://github.com/haskell/haddock/commit/08aa47916d6bb5a0f65d4da1021e0700b30b4b3b fixed it in master.

The brew Sierra version is nearly identical right now to 8.0.2-rc1. I recommend you apply that patch @trofi found.

Having the same problem on windows and I do have ghc 8.0.2. If the 8.0.2_rc1 has fixed the problem then I would love to apply it in my environment. Does it mean I need to download the entire source and compile myself or is there a patching command that can upgrade my ghc environment to the suggested version?

In the mean time a new haddock is released. Closing this. Feel free to reopen if you think this hasn't been addressed.


Unit testing: uncommented accepts tests and added another one

Added:

types unit test for checking that it returns the first option when there's no header

Fixes:

uncommented accept.types() tests

Codecov Report

Merging #97 into master will increase coverage by 0.68%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #97      +/-   ##
==========================================
+ Coverage   60.19%   60.88%   +0.68%
==========================================
  Files          44       44
  Lines        1613     1613
  Branches      416      416
==========================================
+ Hits          971      982      +11
+ Misses        640      629      -11
  Partials        2        2

Impacted Files
Coverage Δ

packages/accepts/src/index.ts
75.92% <0.00%> (+20.37%)
:arrow_up:

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 248faf5...085909e. Read the comment docs.

LGTM


⚠ Bofrost.es has degraded performance

In f7514ee, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1390 ms

Resolved: Bofrost.es performance has improved in be59826 after 6 minutes.


Blend between Leap & Mecanim animation

If the issue is Done, can we close it?

Closing because the issue was marked Done.


[Draft] Add Cluster Name To Visualiser

Signed-off-by: Pranav Singh<EMAIL_ADDRESS>Description
This PR adds initial work for the cluster name to be visible on the visualizer
This PR fixes #
Notes for Reviewers
Please ignore the comments, I've added them for debugging. I'll remove them after completing this (and supporting) PRs.
Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #5348 (1462de3) into master (9880a6d) will decrease coverage by 4.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5348      +/-   ##
==========================================
- Coverage   17.35%   13.16%   -4.20%
==========================================
  Files         107      107
  Lines        9864     9882      +18
==========================================
- Hits         1712     1301     -411
- Misses       7731     8304     +573
+ Partials      421      277     -144

Flag
Coverage Δ

gointegrationtests
?

unittests
13.16% <ø> (+0.12%)
:arrow_up:

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

Impacted Files
Coverage Δ

handlers/k8sconfig_handler.go
0.00% <ø> (ø)

mesheryctl/internal/cli/root/system/update.go
1.05% <0.00%> (-58.95%)
:arrow_down:

mesheryctl/internal/cli/root/system/start.go
1.88% <0.00%> (-55.19%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
1.14% <0.00%> (-55.18%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
1.92% <0.00%> (-48.08%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
2.15% <0.00%> (-44.61%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
3.26% <0.00%> (-36.96%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.03% <0.00%> (-23.03%)
:arrow_down:

mesheryctl/internal/cli/root/system/dashboard.go
1.70% <0.00%> (-9.41%)
:arrow_down:

mesheryctl/pkg/utils/healthcheck.go
8.82% <0.00%> (-4.42%)
:arrow_down:

... and 1 more

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 3413774...1462de3. Read the comment docs.

Removing draft label as its ready for review


Change the cluster api management method. #2262

Change the cluster api management method.

Current coverage is 39.86% (diff: 28.44%)

Merging #2315 into master will decrease coverage by 0.01%

@@             master      #2315   diff @@
==========================================
  Files          1614       1613     -1
  Lines         55903      55943    +40
  Methods       10988      10992     +4
  Messages          0          0
  Branches       7717       7724     +7
==========================================
+ Hits          22296      22304     +8
- Misses        33607      33639    +32
  Partials          0          0

Powered by Codecov. Last update f3de4c4...7287777


See how much time is spent reading and how much waiting

Normally, it seems we spend

60 ms on work
30 ms on reading the load cell (mostly blocking until data is ready)
Giving a 90 ms / 11 Hz loop, which sounds okay

When data is ready before i call read() on the load cell, I get the data in 0-2 ms.


Main process should use less CPU

Right now the main process does a busy wait.  It should instead make the listener have a timeout for accepting new connections or reading new data I guess.  Anyway it should use less CPU.

More context from an email by @dpino:

When running lwaftr run --reconfigurable (this starts the lwaftr with a leader and a follower), the leader is run in busywait mode, so it takes 100% of the core it is assigned to.
The leader doesn't need to take control full-control of a core, since the ratio of incoming commands won't be that high to make this worth. It could even be desirable to pin the leader to a kernel-managed core.

Same problem is present when running the lwaftr bench --reconfigurable command, fixed in #738.


[#152157743] general fixes for Android

NOTE:

missing getStats, for now.

we'll re-open this later, after we do a bit more android work..


update docs, staging deployment branch in .travis.yml

This PR updates the Readme and Contributing docs, and corrects the staging deployment branch to main.

This change is

schema_annotationrow.md, line 1 at r1 (raw file):
Previously, stephen-soltesz (Stephen Soltesz) wrote…
Should these files be added outside the _includes directory?

No, they should be inside it. Will fix.

CONTRIBUTING.md, line 135 at r3 (raw file):
Previously, stephen-soltesz (Stephen Soltesz) wrote…
This link does not render as intended.

done

CONTRIBUTING.md, line 154 at r3 (raw file):
Previously, stephen-soltesz (Stephen Soltesz) wrote…
The schema/descriptions are in the etl repo, not etl-schema (for better or worse).

changed


feat: dynamic windows provider

:x: Build TheIdServer 6.1.1-dinamyc-windows-provider1-0048 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/3a00f10c20 by @)

:white_check_mark: Build TheIdServer 6.1.1-dinamyc-windows-provider1-0050 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/31c53442ad by @)

:white_check_mark: Build TheIdServer 6.1.1-dinamyc-windows-provider1-0051 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/db2fbe424f by @)

:x: Build TheIdServer 6.1.1-dinamyc-windows-provider1-0052 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/480dd4761b by @)

:x: Build TheIdServer 6.1.1-dinamyc-windows-provider1-0053 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/2494614320 by @)

:x: Build TheIdServer 6.1.1-dinamyc-windows-provider1-0054 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/3285e1a109 by @)

:white_check_mark: Build TheIdServer 6.1.1-dinamyc-windows-provider.1+54.build.2502 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/3285e1a109 by @)


Checkbox preview to markdown problem

Hello! Sorry but I'm facing issues again with the checkbox functionality. The "ticking" works in preview mode but does not change the markdown value like this GIF, and there is no longer a strikethrough like before.  Any edits to checkboxes inside the admonition on preview mode disappears when exiting the note.

Hmm, odd. It was working, right?

Yeah I just noticed now, even on a fresh vault with no other plugins. The markdown was working fine before the title collapse issue was fixed.

Okay, bad news. Looks like the latest Obsidian update broke this, and it seems to no longer be possible for me to both:

Update the checkbox value in the raw markdown
Not collapse a collapsible Admonition

6.2.2 restores the checkbox clicking


Compilation error messages in the plus-loader css file

I've installed using bower. It worked well, but errors are showing up in body:before {}
It is coming from the following file
bower_components/css-spinners/css/spinner/plus-loader.css

Reference:
https://github.com/jlong/css-spinners/blob/master/css/spinner/plus-loader.css


Badges should be hashbangs

When clicking on a badge link - it takes us to the top of the page.
Let's switch them to hashbangs (#!) like every other link on the gh-pages branch.

@RachelRVasquez PR submitted.

@fuhton Merged.


403 Forbidden from import

I have a django web app deployed as an Azure App Service. I have also been using the azure-storage SDK for python to upload and download images from Azure Blob Storage. When I run it locally, the app runs fine and I can successfully push and pull images. However, when I deploy the app, I get this error:
2017-02-26T22:15:16
Microsoft.WindowsAzure.Storage.StorageException: The remote server returned an error: (403) Forbidden. ---> System.Net.WebException: The remote server returned an error: (403) Forbidden.
at System.Net.HttpWebRequest.GetResponse()
at Microsoft.WindowsAzure.Storage.Core.Executor.Executor.ExecuteSync[T](RESTCommand1 cmd, IRetryPolicy policy, OperationContext operationContext)    --- End of inner exception stack trace ---    at Microsoft.WindowsAzure.Storage.Core.Executor.Executor.ExecuteSync[T](RESTCommand1 cmd, IRetryPolicy policy, OperationContext operationContext)
at Microsoft.WindowsAzure.Storage.Blob.CloudBlockBlob.DownloadBlockList(BlockListingFilter blockListingFilter, AccessCondition accessCondition, BlobRequestOptions options, OperationContext operationContext)
at Microsoft.WindowsAzure.WebSites.Diagnostics.AzureBlobTraceListener.AppendStreamToBlob(Stream stream)
at Microsoft.WindowsAzure.WebSites.Diagnostics.AzureBlobTraceListener.ConsumeBuffer()
Request Information
RequestID:25101a94-0001-00f5-3a7d-90b249000000
RequestDate:Sun, 26 Feb 2017 22:15:16 GMT
StatusMessage:Server failed to authenticate the request. Make sure the value of Authorization header is formed correctly including the signature.
ErrorCode:AuthenticationFailed
I've pinpointed it to import statements. If I just include from azure.storage.blob import BlockBlobService, I get this error. I don't create a BlockBlobService or call anything else yet.
I have tried adding an account SAS from my Azure Storage account and setting it as a DIAGNOSTICS_AZUREBLOBCONTAINERSASURL setting, but that doesn't work either.
I'm not sure what's causing this error?

Hi @cqw1 ,
What version of the Python Storage SDK are you using?

I've pinpointed it to import statements. If I just include from azure.storage.blob import BlockBlobService, I get this error. I don't create a BlockBlobService or call anything else yet.

Can you check if this occurs when importing any of the other Blob types?
Few thoughts -- there is a known issue with deployment  on Azure WebApps introduced in version 0.33.0 and onwards due to the cryptography dependency and outdated pip on WebApps. See #219.
For quick confirmation, use version 0.32.0 of the library in your requirements.txt and deploy/see if this problem still persists. You'll want to log into your WebApps kudu console to make sure that version 0.32.0 is in fact installed and being used. If that is the case, see the issue reference above for resolution.
I have the latest version deployed and working fine right now, but I did have a similar issue which I was able to fix with the above steps. Let me know if that helps.

I was using 0.34.0, but after setting azure-storage==0.32.0 in my requirements.txt it works now!
Thanks for all the help!


[cherry-pick] Add support for ca bundle injection.

A new image has been built to help with testing out this PR: quay.io/opendatahub/data-science-pipelines-operator:pr-526
An OCP cluster where you are logged in as cluster admin is required.
To use this image run the following:
cd $(mktemp -d)
git clone git@github.com:opendatahub-io/data-science-pipelines-operator.git
cd data-science-pipelines-operator/
git fetch origin pull/526/head
git checkout -b pullrequest ef5f1f7e8ad6cb6bda148d54d8b9199067145d75
make deploy IMG="quay.io/opendatahub/data-science-pipelines-operator:pr-526"

More instructions here on how to deploy and test a Data Science Pipelines Application.


HTTP::ConnectionError: failed to connect: Connection reset by peer - SSL_connect

I am trying to upload a video file (in development mode), I often get this error while uploading video file
HTTP::ConnectionError: failed to connect: Connection reset by peer - SSL_connect

My assumption is this error won't occur in production when I have SSL.

I'm seeing a lot of #<HTTP::ConnectionError: error reading from socket: Connection reset by peer> so maybe it's deeper?


fix: fix offset reset key in taos-ws

Codecov Report

Merging #254 (4aa640d) into main (bbf76be) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #254   +/-   ##
=======================================
  Coverage   60.14%   60.14%
=======================================
  Files         102      102
  Lines       25312    25312
=======================================
  Hits        15225    15225
  Misses      10087    10087

Files
Coverage Δ

taos-ws/src/consumer/messages.rs
96.92% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Automation: persist command surfaces and detect changes between commits.

And compare the dump between build to detect the command surface changes.

Related to, but not a dupe of, #733

Cleaning u old issues, reopen if this is still active.

We have numerous other ways of detecting changes to commands. Closing this.


supports for concurrent development on different branches

This plugin is great - almost dose everything that we needed.
We have a requirement for concurrent development (probably pretty common) - we have two teams - one team work on bug fixing for v1.0, the other team work on new feature for v2.0.
Let's say two teams use the two branches b1 and b2 for these two release.
Ideally, on branch b1 , the versions go like v1.0.0, ..., v1.0.5, ..., v1.2.0,  etc
while on branch b2, the versions go like v.2.0.0, ..., v.2.3.5,..., v2.5.0, etc
My understanding is that since this plugin relies on tags only (NOT version value inside gradle file), it will be difficult to for these two branches to evolve on two different series of tags.
What is the correct setup/configuration in this situation?
Any thoughts? Thanks
-cl

In your case, the existing functionality should work fine. It does rely on tags, but specifically on tags that are reachable from the branch you're on. This should allow you to pick up the correct version on the b1 and b2 branhes. I'd suggest giving it a try and seeing if it meets your needs. Let me know if it turns out not to.

tried and no luck. I think the problem is that tags are reachable from  any branch.
so if tag t1 is created on branch b1, after switching to branch b2
git tag will show t1 on branch b2
I found the reference: http://stackoverflow.com/questions/14613540/git-tag-in-branches
... A tag is a pointer to a commit, and commits exist independently of branches.
It is important to understand that tags have no direct relationship with branches - they only ever identify a commit...
the concurrent development is a must per our requirement, any suggestions for us to move forward with this? I compared this plugin  with the other four that you mentioned. This one is definitely better in all aspects except the multiple branch developments.

It's true that tags aren't tied to the branch they were created on, but it's not true that they are reachable from any branch.
If you're literally calling git tag you will see all branches in the repo. A more accurate representation of how gradle-git finds relevant (i.e. reachable) tags is git tag --merged. This will show all tags that can be found by walking back through the ancestry of the current HEAD.
So to test your use case:

git checkout b1 and git tag --merged will show tags that will be used to infer the version when you work on branch b1
git checkout b2 and git tag --merged will show tags that will be used to infer the version when you work on branch b2

That's not to say that the current functionality is a panacea. There are many use cases where gradle-git isn't smart enough to determine the version a user would intend. That's where #115 comes in,  but that's a complex problem I haven't solved yet.

Thanks for the quick response. For git tag  the --merged option was introduced at git version 2.7.x (introduced Sep, 2015). Our git version is at 2.6.4. I ran the test that you described and it did work with git 2.8.x.
Anyway to make it work for 2.6.x? Thanks so much.

gradle-gituses JGit, a pure Java implementation of Git, to interact with repositories. The version you have installed for your CLI git commands is completely independent. You shouldn't have any issues still being on 2.6.4 for the CLI.


Sb/expose name

addressing #7 and an assumed structure that was wrong in ArgusIO

I believe this one is ready for merge (please look carefully, we did a few manual adjustments) ... I will create a new one for the nframe adjustments


Error while installing OpenNI2 (chacked on two PCs)

Hello,
I'm trying to install, but there is an error in file scripts/cmake/vcpkg_execute_required_process.cmake:56, err msg:
Command failed: msbuild;C:/vcpkg/buildtrees/openni2/x64-windows/2.2-beta2-a8c7829707/OpenNI.sln;/p:Configuration=Release;/t:Rebuild;/p:Platform=x64;/p:PlatformToolset=v141;/p:VCPkgLocalAppDataDisabled=true;/p:UseIntelMKL=No;/p:WindowsTargetPlatformVersion=10.0.17134.0;/m
Working Directory: C:/vcpkg/buildtrees/openni2/x64-windows-rel
See logs for more information:
C:\vcpkg\buildtrees\openni2\build-x64-windows-rel-out.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_build_msbuild.cmake:119 (vcpkg_execute_required_process)
ports/openni2/portfile.cmake:46 (vcpkg_build_msbuild)
scripts/ports.cmake:71 (include)
Error: Building package openni2:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: openni2:x64-windows
Vcpkg version: 0.0.113-nohash
I know that similar issue was solved here #1937 but solution didn't help me.

Victor

See logs for more information:
C:\vcpkg\buildtrees\openni2\build-x64-windows-rel-out.log

build-x64-windows-rel-out.log
looks like there is a problem with NuiApi.h

'NuiApi.h': No such file or directory
https://github.com/Microsoft/vcpkg/issues/1927#issuecomment-334685776

I followed instructions and i have the same issue.
after changing file replace_environment_variable.patch and deleting  folder C:\vcpkg\buildtrees\openni2 there is problem with
Applying patch C:/vcpkg/ports/openni2/replace_environment_variable.patch
Applying patch failed. This is expected if this patch was previously applied.
All other patchs were applied correcty.

Thanks for posting this issue. And I cannot reproduce it on the latest vcpkg. So close it first.
Please open a new issue if this is still a problem for you.


ConfigEditor: Update MaxBytesBilled type

Fix type error from incorrectly set type maxBytesBilled -> MaxBytesBilled. As we already used uppercased MaxBytesBilled, I updated type.
Now:

Before:

how is this not showing up in CI?

how is this not showing up in CI?

I can have a look and try to fix this, so it does not happen in the future


RTLD_GLOBAL not defined

   _       _ _(_)_     |  A fresh approach to technical computing
  (_)     | (_) (_)    |  Documentation: http://docs.julialang.org
   _ _   _| |_  __ _   |  Type "help()" for help.
  | | | | | | |/ _` |  |
  | | |_| | | | (_| |  |  Version 0.4.0-dev+4998 (2015-05-26 02:12 UTC)
 _/ |\__'_|_|_|\__'_|  |  Commit 116238c (1 day old master)
|__/                   |  x86_64-apple-darwin13.4.0

julia> using OpenCV
libopencv_core.3.0.0 is not found in pkg-config
libopencv_highgui.3.0.0 is not found in pkg-config
libopencv_imgproc.3.0.0 is not found in pkg-config
libopencv_video.3.0.0 is not found in pkg-config
libopencv_videoio.3.0.0 is not found in pkg-config
libopencv_calib3d.3.0.0 is not found in pkg-config
libopencv_features2d.3.0.0 is not found in pkg-config
libopencv_flann.3.0.0 is not found in pkg-config
libopencv_imgcodecs.3.0.0 is not found in pkg-config
libopencv_ml.3.0.0 is not found in pkg-config
libopencv_objdetect.3.0.0 is not found in pkg-config
libopencv_photo.3.0.0 is not found in pkg-config
libopencv_shape.3.0.0 is not found in pkg-config
libopencv_stitching.3.0.0 is not found in pkg-config
libopencv_superres.3.0.0 is not found in pkg-config
libopencv_videostab.3.0.0 is not found in pkg-config
libopencv_viz.3.0.0 is not found in pkg-config
Warning: redefining constant cvlibdir
Warning: redefining constant cvheaderdir
ERROR: LoadError: UndefVarError: RTLD_GLOBAL not defined
while loading /Users/arnaud/.julia/v0.4/OpenCV/src/OpenCV.jl, in expression starting on line 71

I may add that I am NOT using the master branch of the Cxx package, I switched to the bug branch as advised by Keno. Also, I did not compile openCV, I was hoping to use the shared library shipping with openCV.jl.

I will have a look soon at this since I could get past the RTLD_GLOBAL (I was travelling).

I went through the julia/Cxx/OpenCV/OpenCV.jl build+install sequence today and am seeing a similar problem:
julia> using OpenCV
libopencv_viz.3.0.0 is not found in pkg-config
ERROR: LoadError: UndefVarError: RTLD_GLOBAL not defined
while loading /Users/adare/.julia/v0.4/OpenCV/src/OpenCV.jl, in expression starting on line 71

The libopencv_viz issue may be completely unrelated (but I'm not sure, so I won't open a separate issue). I'm noticing just now that the viz module was listed as "Unavailable" when I ran CMake for my OpenCV installation. Maybe viz depends on VTK, and it doesn't look like I have that installed. If OpenCV.jl will not load without libopencv_viz, would it be worth mentioning these prerequisites in the installation directions?

That is correct, viz requires VTK flag switched on.  See e.g.,  [this link] (http://answers.opencv.org/question/32502/opencv-249-viz-module-not-there/). Regarding, the dependencies for loading OpenCV.jl, this is unfortunately affected by changes made in Julia base or Cxx. I am not sure we can have full protection from these changes.  I am hoping that once v0.4 is released, it will be easier to maintain OpenCV.jl as a stable package. But I agree we could have more clearly stated all the requirements for OpenCV.jl and if you want to add these yourself, I welcome suggestions in a PR. Thank you.

Thanks for the info. I'd be happy to write a PR once I sort it out. In the meantime, I'll cope with the warning that libopencv_viz is not being loaded and get back to the topic of this issue: if I qualify a few names with Libdl, I get past this error. Specifically,

RTLD_GLOBAL --> Libdl.RTLD_GLOBAL and dlopen_e(...) --> Libdl.dlopen_e(...) in OpenCV.jl
dlopen(...) --> Libdl.dlopen(...) in deps/Qt_support.jl

Now I'm having Qt problems, so on to that....

Just an update: after installing Qt 5.4 and changing the hard-coded paths in Qt_support.jl to that version, OpenCV.jl seems to work. I can run all the tests, goat video and all. When I first load OpenCV, I do see this:
Warning: requiring "OpenCV" did not define a corresponding module.

But I'm not sure what to make of it, since my preliminary trials seem to indicate things are working.

Great. That message is related to Cxx.jl not being packaged as a module.


fix paths to virtualenvs in jenkinsfiles

No longer relevant, we have all the changes we want on master for this work.


Pivot Table digital data display issue

Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Hello, I met an issue about Pivot Table digital data display. If the integer number of digits reaches 9 digits, pivot table will cut the last 2 interger digit to 0.
Superset version
0.28.1
Expected results

Actual results

Steps to reproduce
Just add data into Pivot table.

Data display format can be set in pivot options, and in this case, it may be necessary to keep the integer format

Data display format can be set in pivot options, and in this case, it may be necessary to keep the integer format

Thanks for your suggestions. But It dosen't work. And I've fix this problem already. First, in viz.py, it render query data into html by pandas's to_html() function. This function will change float numbers into 'x.xxxxxxe+07’ style by default, it omits the last two digits if there is 9 digits, and then return html string to borrowser.  And at the mean time, Js also repsent a long number as 'x.xxxxxxe+07’ style, I also need to  re-format this number.
To fix it, I add an parameter 'float_format' to to_html() function at viz.py like this:

And change src/visualizations/PivotTable/PivotTable.js like this:


pull request for e2e test at 01/23/2017 14:26:16

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


介紹

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Close because the contributor didn't response to the change request.


fix: apply widget syntax in WYSIWYG replaceSelection API and remove widget syntax in getMarkdown() API

Please check if the PR fulfills these requirements

[x] It's the right issue type on the title
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] It does not introduce a breaking change or has a description of the breaking change

Description

applied widget syntax to WYSIWYG replaceSelection API when matched with widget text rules
removed widget syntax($$widget0 ... $$) on calling getMarkdown() API

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

이견 없습니다. 고생하셨어요~


Improper name mapping

@fxn i am contributing in Rails since last year. I checked my name in contributors list but i think it's not proper listed and also reflecting wrong spelling (surname). May be it's releated to one (me @gaurav2728) only but can be happen with others so i created issue
See different scrren shot below for more details

.
Correct Name is "Gaurav Sharma"

Showing Wrong Name here. ("Gaurav Sharam")
Also commits are not included with my previous rails commit.

As name is different that's why it's showing single commit.

This information was get from the git commit.
commit 434c768a7dbf46f7d127d63b825f6eec511b882a
Author: Gaurav Sharam<EMAIL_ADDRESS>Commit: Gaurav Sharam<EMAIL_ADDRESS>
Maybe your configuration was wrong when you did the commit.

@rafaelfranca ok.... Name can be change/update for this commit.

Don't worry, the app is ready to fix situations like this. I'm on a trip but will update de credit soon.

Could add the correction and deploy from a train (we live in awesome times!), you'll see production is :+1:  now.

@rafaelfranca  @fxn thanks :yellow_heart:  Sounds good app is ready/able to fix that kind of situations. I'll try to do more contributions. Can you tell me some RoR chat room/form where i can/should join and chat or whats new going on so i can keep in touch.

@Gaurav2728 There's the rubyonrails-core mailing list, though the vast majority of work/discussions happen in GitHub PRs or issues. Also there's a weekly newsletter called This Week in Rails with highlights of what is new in the repo and the project to stay current.
Thanks for your interest.


Range Problem

I have NodeMCU ESP32S V1.1 Module (ESP WROOM32) and the advertising range is less than 40cm's how can I fix that?
iOS Version: 18.1

Hi buiscuitaep,
I do not have that exact model, so I'm not sure I can be of too much help. It seems like the max power level for your chip might be  ESP_PWR_LVL_N12 .
Can you change this line: https://github.com/ckcr4lyf/EvilAppleJuice-ESP32/blob/28d6be64f0417d51cad37c4b67ac01619f8fae0f/src/main.cpp#L19 to ESP_PWR_LVL_N12 , and see if that helps?

Hi buiscuitaep,
I do not have that exact model, so I'm not sure I can be of too much help. It seems like the max power level for your chip might be  ESP_PWR_LVL_N12 .
Can you change this line: https://github.com/ckcr4lyf/EvilAppleJuice-ESP32/blob/28d6be64f0417d51cad37c4b67ac01619f8fae0f/src/main.cpp#L19 to ESP_PWR_LVL_N12 , and see if that helps?

Now it isnt working, isnt P9 best value?

I'm not too sure about your specific device.
I can recommend purchasing a ESP32C3, which is what I've tested with and performs well in my experience.

Ok Ty, Also when I select long range devices (short like appletv) nothing happens why do you know?

>  select long range devices (short like appletv)
Do you mean AppleTV is an example of the device where "nothing happens", or AppleTV is a "short device"?


initial commit of results populate support

Powered by $lookup :D

Fails

:no_entry_sign:

The changelog has not been updated. Please update the changelog.

:no_entry_sign:

The version was not updated. Please update the version.

Warnings

:warning:

The README has not been updated. Please update the README.

Messages

:book:

We were able to automatically fix some formatting issues in this PR for you!

  Generated by :no_entry_sign: dangerJS


huge performance problems in 4-window vimdiff

Reproduction Steps
0. configuration:
require "nvim-web-devicons".setup {
  -- globally enable default icons (default to false)
  -- will get overriden by `get_icons` option
  default = true
}

run the following, or set it as your git mergetool for conflict resolution:

vim -d $LOCAL $BASE $REMOTE $MERGED

where $LOCAL $BASE $REMOTE $MERGED are 4 files.
2. Move cursor to bottom pane
3. use j/k to move up and down. It's super-laggy.
Note
I tried to take a performance log but it was not actually useful. I think I just lucked out to figure out it was this plugin as there was something in it about vim-devicons, but did not quite justify the large lag. And problem is still there when I remove vim-devicons plugin and leave just nvim-web-devicons.

nvm, it's something else 😕


remove trailing slash when setting genie url in python client

For a GenieJob, when setting the Genie URL to use via .genie_url(), if there is a trailing slash remove it so constructing URLs will not contain extra slashes.

Coverage remained the same at 88.706% when pulling 17e40de66b762b477bfd5fd022c0285633bcc80f on irontable:master into a3b5830dad7be39b1bd9cfa43bfdbb047e04c315 on Netflix:master.


Randomize User Agents

This article suggests randomized user agent requests as another layer of concealment.: http://searchnewscentral.com/20110928186/General-SEO/how-to-scrape-search-engines-without-pissing-them-off.html
I don't know if you can randomize location data too?
Any way to load a custom list of proxy IPs? HMA maintains an extensive list: http://proxylist.hidemyass.com/. Could hook into services like Luminati, too: https://luminati.io/

How much does this cost to run on AWS?

Thanks for you feedback and questions !
You have a lot of questions :)
User Agent
Scrapoxy randomizes the user agent. When you start an instance, Scrapoxy sets a random UA for the life of the instance (=IP address in fact). Scrapoxy overwrites the useragent of your request by the useragent of the instance.
I test multiple strategy and associate an useragent with an IP address is the best one.
Load a custom list of proxies
Not in the dev scope. Why ?
If you want to use a static list of proxies, it's easier to implement a roundrobin dispatcher in your scrapers, than using Scrapoxy.
(And public proxies are not reliable...)
Could I hook Scrapoxy into luminati.io ?
Luminati is a competitor, based on a P2P network (Hola).
No hook.
How much does this cost to run on AWS ?
It depends on the volume of downloaded data.
My configuration has 3 instances (t1.micro). I run the instances 3 hours/day
My monthly bill is 6,25€ (without VAT).

I didn't understand:
"I don't know if you can randomize location data too?"

Thanks. I was referring to geographic data. IE, mark the request as coming from Germany, France, China, etc.

I'm planning to create a map view with all instances. And to trace requests live :)
Is that what your asking ?

Well I guess geolocation is dependent on the IP address. Some websites block visitors from certain countries. So it would be helpful to specify geographic regions which scraproxy is allowed to use. And if 3 proxies are created, they should not be from the same region.

Right :)
We could assert that geographic region == datacenter region.
So if you can configure multiple simultaneous regions from AWS, it's could works.
I will look at it.
Thanks,
Fabien.


Resolve image name with library prefix only when it's from DockerHub

Hello crashvb,
I've encountered another interesting issue: At my company we're working with a lot of Artifactory-based registries and this uncovers a "bug" in the image resolution logic. The current logic is to always prefix images that don't contain "namespace" with library. However, Artifactory recommends to their users to move the "namespace" part of the repository as sub-domain part of the URL. In other words, usually one would expect to work with Artifactory as every other registry like so:
some-artifactory.com/library/python, but Artifactory restructures this into library.some-artifactory.com/python (the namespace will never be called library, but it's good for illustration purposes). Obviously, if DRCA parses that reference, it will modify it to library.some-artifactory.com/library/python which would return 404.
It was my impression that if image doesn't have repository "namespace" it should be assigned to library repository. Apparently, DRCA thinks the same. But it turns out that's not the case -- there is nothing in the Docker Registry spec about that and surprisingly docker pull command copes just fine with library.some-artifactory.com/python.
So, I've spend some hours looking through Go code of containerd and registry-handling code in Docker to see what's going on and found these:
containerd: https://github.com/containerd/containerd/blob/49a945b26b9d7bd485e49041b74aab3aab97c15a/reference/docker/reference.go#L666
Docker: https://github.com/distribution/distribution/blob/main/reference/normalize.go#L87
(They are the same)
So, it turns out that library should be only prefixed when working with DockerHub. Tomorrow I'll create a pull request with potential fix. Can you think of potential problems with that change?

@crashvb, did you have time to review the PR.

@skrech If I'm following, am I correct in understanding that the library namespace is limited to only dockerhub (i.e. docker-library?
@kyatite heads up.

@skrech, @kyatite,
I took a much closer look at the problem and pushed out a fix (v0.2.10) that should work if ImageName.DEFAULT_NAMESPACE is set to "", either directly or using the environment variable. This should have the expected behavior while still maintaining the position of defaulting official images to DockerHub.
I stopped short of bumping a major version and changing the DEFAULT_NAMESPACE to "" (with a note in the readme),  because I think that would need more explicit testing downstream.
Side Note: I tested with both default=library,override="" and default="",override="library" and both of them seem to work; although it did expose some test data corruption that should be fixed now.


feat: serve AsyncAPI JSON Schema definitions

Description
This PR let's our website to serve all AsyncAPI JSON Schema definition files from it.
All AsyncAPI JSON Schema definition files are being served within the /definitions/<file> path. The content is literally being served from GH, in particular from https://github.com/asyncapi/spec-json-schemas/tree/master/schemas.
This is possible thanks to the following:

A Netlify Rewrite rule located in the netlify.toml file, which acts as proxy for all requests to the /definitions/<file> path, serving the content from GH without having an HTTP redirect.
A Netlify Edge Function that modifies the Content-Type header of the rewrite response to become application/schema+json. This lets tooling, such as Hyperjump, to fetch the schemas directly from their URL.

In order to see what the final result is, just grab the Netlify build preview link and run:
curl -v <preview-url>/definitions/2.3.0.json

Btw, this Netlify edge function could be used also for pushing metrics on the hit count, solving asyncapi/website#780.
Related issue(s)
Iteration over https://github.com/asyncapi/website/pull/502
cc @jonaslagoni @derberg @fmvilas @magicmatatjahu

Ok, this is embarrasing. For some reason the Content-Type header is not being properly set. I think Netlify didn't detected the function. I will need an admin to see if it appears as Edge Function in the site dashboard.
I'm sure this works because I did a test days ago here: https://deploy-preview-680--asyncapi-website.netlify.app/definitions/2.3.0.json (and you can see the header is right).

Interesting approach 🤔

I understand that we can have an edge function that on each request can do anything, like API call to some service, right?
You do know it is experimental? Also have a look at -> https://github.community/t/raw-githubusercontent-com-rate-limit/142444. For me, it is actually ok because I have a totally opposite view on the High Availability argument you made here. Just want to make sure you are aware

I understand that we can have an edge function that on each request can do anything, like API call to some service, right?

Yes, that's right.

You do know it is experimental?

Yes I know, but right now it is a public beta which has been passed through the private beta stage.

For me, it is actually ok because I have a totally opposite view on the High Availability argument you made https://github.com/asyncapi/server-api/issues/61#issuecomment-1098383443.

I still believe Netlify availability is pretty reliable and superior to what we can support ourselves with the people who are involved. Including using beta features such as Edge Functions.

Also have a look at -> https://github.community/t/raw-githubusercontent-com-rate-limit/142444

You made a good point here. The authentication token is going to be needed to have those 5k req/hour.
I could not make proper testing around how github counts a cached VS non-cached request in terms of rate limit because GH API /rate_limit endpoint doesn't shows such stats for raw.githubusercontent.
However, there is something to consider here and it's the fact Netlify should cache each request for "some time":

If the API supports standard HTTP caching mechanisms like ETags or Last-Modified headers, the responses will even get cached by our CDN nodes.
Source: https://docs.netlify.com/routing/redirects/rewrites-proxies/#proxy-to-another-service

If we run the following:
curl -I -H "Authorization: $MY_TOKEN" https://raw.githubusercontent.com/asyncapi/spec/v2.4.0/examples/websocket-gemini.yml

HTTP/2 200
cache-control: max-age=300
content-security-policy: default-src 'none'; style-src 'unsafe-inline'; sandbox
content-type: text/plain; charset=utf-8
etag: "755fbfddca88cfd66072e57e21f9a33f3fabbe6ae24ce7c0249d83ab8d3d7788"
strict-transport-security: max-age=31536000
x-content-type-options: nosniff
x-frame-options: deny
x-xss-protection: 1; mode=block
x-github-request-id: F288:9F7F:6918C6:6DD3EC:626FDD1E
accept-ranges: bytes
date: Mon, 02 May 2022 13:31:10 GMT
via: 1.1 varnish
x-served-by: cache-mad22067-MAD
x-cache: MISS
x-cache-hits: 0
x-timer: S1651498270.022946,VS0,VE205
vary: Authorization,Accept-Encoding,Origin
access-control-allow-origin: *
x-fastly-request-id: 5327fa939e275e7ddbd14bc99a8fb310421282a5
expires: Mon, 02 May 2022 13:36:10 GMT
source-age: 0
content-length: 9100

We can see the content should be cached for 5 mins (300 seconds): cache-control: max-age=300. And considering that Vary header value is Authorization,Accept-Encoding,Origin, we can also do the following:

Force always a value in Accept-Encoding header so all requests go with the same one.
Remove the Origin request header, so GH always serves the same cached content, no matter the Origin header.

That would give us a really big number of requests allowed (cached or non cached), all pending to be tested and confirmed.  In the really worst scenario, we could switch to what @jonaslagoni suggested in https://github.com/asyncapi/website/pull/502 so we serve the files right from Netlify instead of GH, but keeping the Edge Function in there for the Headers change + metrics collection.

I added the following:

New /schema-store path to be used only by schemas hosted at SchemaStore.org.
Send metrics to NewRelic (More info at https://github.com/asyncapi/server-api/issues/61#issuecomment-1116286643).

Some env vars will be needed to be set at Netlify level:

GITHUB_TOKEN: Preferably, a token from a user is not being used in any automation yet.
NR_ACCOUNT:  the New Relic account id. Can be found on New Relic's website, path: /admin-portal/api-keys.
NR_API_KEY: The New Relic API KEY: A License type API Key that can be created on New Relic's website, path: /admin-portal/api-keys

As a side note, the URL to NewRelic insights collector should be changed in case AsyncAPI New Relic Account is not hosted in EU to https://insights-collector.newrelic.com/v1/accounts/YOUR_ACCOUNT_ID/events.
Here a screenshot of how our dashboard could look in New Relic:

I think we need some page under https://deploy-preview-680--asyncapi-website.netlify.app/schema-store that explains the concept and lists all available schemas
also to go further lets make sure others have nothing against new relic and hosting from GitHub Raw atm -> https://github.com/asyncapi/server-api/issues/61#issuecomment-1124815189

I think we need some page under https://deploy-preview-680--asyncapi-website.netlify.app/schema-store that explains the concept and lists all available schemas

Who could help on that part?

Who could help on that part?

I don't think it has to be a beautiful new view on AsyncAPI Website. It could even be a redirect to spec JSON schema repo to a markdown file that explains it

It seems people agreed on this finally!

YES!!! we just need to setup official account on New Relic using<EMAIL_ADDRESS>Can you do it? somehow reconfigure what you have already? or something like that? Basically<EMAIL_ADDRESS>should be the owner and others invites as team members with proper admin/moderator rights

I don't think it has to be a beautiful new view on AsyncAPI Website. It could even be a redirect to spec JSON schema repo to a markdown file that explains it

Maybe something like this? https://github.com/asyncapi/spec-json-schemas/pull/229

@derberg redirects has been added to the schemas README from https://github.com/asyncapi/spec-json-schemas/pull/229, which will be located at https://github.com/asyncapi/spec-json-schemas/tree/master/schemas

This is ready for review. Tested locally through netlify dev.

@derberg I checked the retention New Relic provides to the free and standard tier:

Source: https://newrelic.com/pricing
As I was sending custom events, I decided to change it to send metrics instead, so we can take profit of the 13 months of retention. That means the code changed a bit so you will need to review it again (sorry for that).
Also, the NR_ACCOUNT  env var is no longer needed with this new approach.

/rtm

/rtm

/help

@alequetzalli I need your review again after all suggestions were applied. 🙏 thank you!

❤ Once it is merged we need to open PR to schemastore

Changes are pushed to my fork but waiting for this PR to be merged (approval by @alequetzalli is required) to be turned into a PR. https://github.com/SchemaStore/schemastore/compare/master...smoya:feat/asyncapiSchemaStore?expand=1

/rtm


Fixes #97

According to the specification for JSON-RPC over HTTP the request and response Content-Type should be one of the following:
"application/json-rpc" - preferred
"application/json"
"application/jsonrequest"

@DMEvans I don't have a windows machine to test this on myself. Do you want to give this a spin?


Vertical panning in Chrome mobile scrolls the page

Expected Behavior
Touching the image and dragging it will only reposition the image
Actual Behavior
The image gets slightly repositioned but the browser page scrolls as well, making accurate positioning very tough and frustrating.
Steps to Reproduce the Problem

Open a page with a croppie on Chrome mobile browser (tried on Android)
Zoom in a little, then try to reposition the image vertically.

Example Link

Link: https://foliotek.github.io/Croppie/

Specifications

Browser: Chrome mobile browser
Version: 72

This behaviour is also seen on DuckDuckGo browser but not on Firefox mobile. It works like a charm on Firefox, unfortunately more people use Chrome mobile.

I think Google just fixed this

Try updating Chrome

Thanks for the update! Yes, it seems to be fixed in Chrome 73. Updating WebView also improved the behaviour in DuckDuckGo - the address bar still moves but I guess that is up to DDG to fix.


Generalize to other storage services

This repo is plain awesome. One suggestion I have is to allow a user to define the storage account in use (and remove depenency on s3 or aws). Many people don't have aws accounts and are working with other cloud providers or on-premises.
Is the dependency on the aws Python package and s3 connectivity really mandatory for using this tool? Should it should be some kind of extension?
EDIT: I see that this is actually taken care of in #162. Cool! What's keeping this PR from being merged?

Yep, this is planned in #162.
That PR needs working since the tests aren't passing yet, particularly on Windows (see #205, which is going in to that branch if you want a head start there).
Also, there aren't tests yet for each of the particular providers which need manual testing then mocks to make sure those functions work across providers and OSes.
Would love contributions there!

Closing this issue since those items are covered elsewhere.


Code guidelines

We have to choose some "code guidelines".
C# has some very strong code guidelines made by Microsoft that most projects adhere to.

coding conventions
naming guidelines
type design guidelines
member design guidelines
designing for extensibility
design guidelines for exceptions
usage guidelines
common design patterns

I propose we adopt them too.
Can everyone take a look at them and respond with a consent or disagreement?
\cc @bluebeel @WilliamHdd @JonathanPetit @momodonatien

I agree too
Le 27 févr. 2017 18:39, "Mathieu David"<EMAIL_ADDRESS>a écrit :

We have to choose some "code guidelines".
C# has some very strong code guidelines made by Microsoft that most
projects adhere to.

coding conventions
https://msdn.microsoft.com/en-us/library/ff926074.aspx
naming guidelines
https://msdn.microsoft.com/en-us/library/ms229002(v=vs.110).aspx
type design guidelines
https://msdn.microsoft.com/en-us/library/ms229036(v=vs.110).aspx
member design guidelines
https://msdn.microsoft.com/en-us/library/ms229059(v=vs.110).aspx
designing for extensibility
https://msdn.microsoft.com/en-us/library/ms229028(v=vs.110).aspx
design guidelines for exceptions
https://msdn.microsoft.com/en-us/library/ms229014(v=vs.110).aspx
usage guidelines
https://msdn.microsoft.com/en-us/library/ms229035(v=vs.110).aspx
common design patterns
https://msdn.microsoft.com/en-us/library/ms229046(v=vs.110).aspx

I propose we adopt them too.
Can everyone take a look at them and respond with a consent or
disagreement?
\cc @bluebeel https://github.com/bluebeel @WilliamHdd
https://github.com/WilliamHdd @JonathanPetit
https://github.com/JonathanPetit @momodonatien
https://github.com/momodonatien
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/azerupi/ecam-kitbox/issues/28, or mute the thread
https://github.com/notifications/unsubscribe-auth/AYZV9GVBKH9SMhRiIOwly6681IYYUqU_ks5rgwpHgaJpZM4MNaDO
.


Utility for concurrently running commands on nodes in HA

:nut_and_bolt: Description: What code changed, and why?
In Automate HA, currently commands run in consecutive manner on frontend and backend nodes.So a Utility is created to run commands concurrently on nodes.
Flags for Nodes:-

Arg/ Flag
Duplicate
Shorthand

--frontend
--fe
-f

--automate
--a2
-a

--chef_server
--cs
-c

--postgresql
--pg
-p

--opensearch
--os
-o

:chains: Related Resources
Jira Ticket :- https://chefio.atlassian.net/browse/CHEF-1015
:+1: Definition of Done

Created a common Utility to run commands concurrently on nodes like automate,chef-server,etc.
Made changes for chef-automate config show and chef-automate config patch commands for using this utility

Scenarios Tested:-

config show for all nodes
config show with output file for all nodes
config patch for all nodes
config show and patch for managed services

:athletic_shoe: How to Build and Test the Change

Checkout this Branch
Rebuild automate-cli
Install cli on running ha deployment(hab pkg install)
Run config show or patch commands from bastion with proper flags

:white_check_mark: Checklist
All PRs must tick these:

[x] I have read the CONTRIBUTING document.
[x] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[ ] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[ ] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[ ] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[ ] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[ ] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[x] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable
Demo videos :-
Demo_Aws_Deployment.mov
Demo_OnPrem_Deployment.mov
Screenshots:-
Automate-Ha AWS deployment :-

Automate-Ha OnPrem deployment :-

Error for Managed services:-

when we run chef-automate config show --a2 --result-json new.json the command result is not in json
new.json file contains something like {"command":"chef-automate config show --a2 --result-json new.json","status":"OK","error_code":0,"error_description":"","error_cause":"","error_stack_trace":"","error_recovery":"","error_type":""}


Added "accepting_volunteers" checkbox to projects

Started a PR for #107
Changelog

Added accepting_volunteers boolean
Added checkbox to new/edit forms. Also added JS to hide relevant fields if you arent actually needing volunteers.
Same fields get hidden on show page if you uncheck volunteers.
Added query string filter for project#index. Will expose in the view at some later point.
Wrote tests for all of the above and added some missing specs

Think this is good for review. cc @sradu @jamiew

@caseypugh Can you please add a snapshot with PRs if possible? It would be hugely appreciated 🙏


Should iCloud files that are not yet available locally be downloaded?

When an iCloud folder is opened in the Explorer, files that are not available locally are displayed as follows:
.foo.txt.icloud

Should they be displayed as foo.txt instead and downloaded from iCloud when they are opened?

Indeed, this isn't being handled correctly in the moment - we should show the proper name and download on demand
This resource might be helpful for us in implementing: https://apple.stackexchange.com/questions/328329/is-there-a-way-via-the-command-line-to-cause-icloud-files-to-download
Related to #2707

Indeed, this isn't being handled correctly in the moment - we should show the proper name and download on demand
This resource might be helpful for us in implementing: https://apple.stackexchange.com/questions/328329/is-there-a-way-via-the-command-line-to-cause-icloud-files-to-download
Related to #2707


platformio-core: expose unwrapped platformio as platformio-core

PlatformIO is a pure python package that is in turn a package
manager. In a pure NixOS environment, this means that any downloaded
binary packages will not run. To make Platform IO usable, there's a
chrootenv wrapper. However, in a mixed environment like other linux or
darwin, the pure python version will work, and in the case of darwin
only the pure version will work, since the chrootenv wrapper is not
supported.
To handle the above use cases we have:

platformio -- unwrapped on darwin, wrapped on linux. Should always
provide a functional platformio.

platformio-core -- always unwrapped (like "bintools-unwrapped") for
when the wrapper is explicitly not required. For
example, on other linux where the chrootenv is
not supported.

Description of changes
I want to expose platformio-core for use on darwin, but I don't mind too much how it's exposed. This change seems reasonable to me.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@thefloweringash I was just about to make a PR to expose platformio but found yours.
Everything looks reasonable to me. Could you rebase and fix the conflicts so I can merge?

@thefloweringash I was just about to make a PR to expose platformio but found yours.
Everything looks reasonable to me. Could you rebase and fix the conflicts so I can merge?

I pushed a rebased version. I think the new version is simpler, but I'd appreciate a sanity check. Tested only on aarch64-darwin.

Tested on x86_64-linux and works fine.


Does sru use more memory then lstm?

Just replaced LSTMs with SRUs and got OOM error.
Wonder if its me or intended.

Dublicate, i guess
https://github.com/asappresearch/sru/issues/69


Upgrade breaking changes crates

Why
Some crates upgrade include breaking changes:

sqlite
http

What
We need to upgrade these crates to their latest version and handle the breaking changes
How

[x] Upgrade sqlite crate
[x] Upgrade http crate

We have postponed the upgrade of the http crate as it crated conflicts with the re-exposed version used by warp.


Directory renamed (loot_table)

Thanks for updating the mod to 1.21! I absolutely love this mod. I assume that you could be already aware of this, but just in case:
There is an error in the data folder. The loot_tables directory got renamed to loot_table in 1.21. In addition, looting_enchant & random_chance_with_looting was renamed in the functions field within the loot_table to enchanted_count_increase & random_chance_with_enchanted_bonus, respectively. For example:
Old file for armored_spider:
armored_spider.json
Revised file:
armored_spider.json
Old file for frozen_zombie:
frozen_zombie.json
Revised file:
frozen_zombie.json
I might've missed other things. Anyways, keep up the great work 😼

Fixed. Thanks for reporting, I completely forgot about loot tables.


GDScript 2.0: Type testing (is) not working with typed arrays

Godot version
v4.0.dev.custom_build.3bebbcacd
System information
manjaro linux
Issue description
print([1,2] is Array[int])

yields parse error:
  built-in:5 - Parse Error: Builtin type cannot be used as a name on its own.
  built-in:5 - Parse Error: Identifier "int" not declared in the current scope.
  modules/gdscript/gdscript.cpp:855 - Method/function failed. Returning: ERR_PARSE_ERROR

Steps to reproduce
Run the above
Minimal reproduction project
No response

Seems still reproducible as of 4.0 RC 1.


windows install screenpipe didnt work

Hi,bro. I can't get it normally

@mtylovesqs i cannot help if you don't provide any information:

what did you do/try
what is the error (share logs)
what is your hardware/software details

bro，I installed the PC locally, but it doesn't work, it's not in green status, it's always in red. Because my skills are not very strong, I have no way to start.
I tried to obtain the new PC software by sharing on social media, but I haven't received it yet. My attempts are currently at a standstill.
---- Replied Message ----
| From | Louis @.> |
| Date | 10/23/2024 23:54 |
| To | @.> |
| Cc |<EMAIL_ADDRESS>|
| Subject | Re: [mediar-ai/screenpipe] windows install screenpipe didnt work (Issue #573) |
@mtylovesqs i cannot help if you don't provide any information:
what did you do/try
what is the error (share logs)
what is your hardware/software details
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Skoda unsupported connector footnote; note J533 alternative isn't available on all cars

cc:  @jyoung8607 @vanillagorillaa
see https://discord.com/channels/469524606043160576/534359517836607488/999917791262031912

You will need to add the footnote programmatically instead, see here where the others were defined: https://github.com/commaai/openpilot/blob/master/selfdrive/car/volkswagen/values.py#L99

You will need to add the footnote programmatically instead, see here where the others were defined: https://github.com/commaai/openpilot/blob/master/selfdrive/car/volkswagen/values.py#L99

Thank you-- you mean that CARS.md is generated.
By the way, I noticed that multiple footnotes is not rendered correctly (only the first footnote is referenced).  E.g.:
VWCarInfo("Volkswagen Passat 2015-22", footnotes=[Footnote.VW_HARNESS, Footnote.PASSAT, Footnote.VW_VARIANT], harness=Harness.j533),

renders as
Volkswagen	Passat 2015-22⁷

(At the moment I don't have an openpilot dev environment with proper dependencies to run selfdrive/car/docs.py.)

@sshane thank you for running docs.py, though I think I need to rebase to master first to get the multiple-footnote fix, otherwise this PR would overwrite the generated code

rebased-- this PR will need another run of docs.py before merge

Footnote.VW_HARNESS is for MY2021/2022 and beyond vehicles where the required harness changes due to introduction of the MFK 3.0 camera. It's not brand specific. I've been adding the footnote references as MY2022 support comes up. It's Volkswagen-heavy right now because I started a sweep recently (#25123) to extend supported model-years. It's research-intensive and I haven't gotten to the other brands yet.
You're correct to note some vehicles have popped up that don't have the traditional J533 connector anymore. The affected cars appear to be MY2022 and forward MQB-A0 with a new combined BCM/CAN gateway, but that is based on a small sample size. I'm not exactly sure what to say about that in the footnote, and have really been hoping @vanillagorillaa's work on a new MFK 3.0 connector pays off soon.

Footnote.VW_HARNESS is for MY2021/2022 and beyond vehicles where the required harness changes due to introduction of the MFK 3.0 camera. It's not brand specific. I've been adding the footnote references as MY2022 support comes up.

Do you mean this footnote shouldn't be added to all Skoda models?
Since MFK 3.0 is confirmed in one model, it seems highly likely to be adopted by other models.  The footnote is and asking users to verify the connector before purchasing a harness.  Is there harm in conservatively marking all models of the brand?

Do you mean this footnote shouldn't be added to all Skoda models?

I mean it should be added as we bring MY2021/2022 onboard, or update the docs with research showing we can. Technically your PR #25271 would have been the time to add it to Karoq, but I didn't want to hold it up for revision given that you had this PR open on similar grounds.

I mean it should be added as we bring MY2021/2022 onboard, or update the docs with research showing we can. Technically your PR #25271 would have been the time to add it to Karoq, but I didn't want to hold it up for revision given that you had this PR open on similar grounds.

@jyoung8607 will you OK this PR if it the footnote is on Karoq only?

You're correct to note some vehicles have popped up that don't have the traditional J533 connector anymore. The affected cars appear to be MY2022 and forward MQB-A0 with a new combined BCM/CAN gateway, but that is based on a small sample size. I'm not exactly sure what to say about that in the footnote

I'd really like to disclose something in the footnote about the possibility of no J533 (however small).  It's not a great user experience to be promised a workaround and find out after purchasing the harness that you don't have the port.

Is this PR ready to merge?

It's close enough. I wish I had a better definition of which newer cars have combined BCM and CAN gateway hardware (I think it's the MQB-A0 compacts) but this is better than no note at all.

Is this PR ready to merge?

yes, please run docs.py and merge


[HOLD] Restore options

refs #6122

restore original options after delete
this is a fix for one use case, long term we should aim to leave
options untouched and execute special queries with temporary data

Is this good to merge now that we're post-release?
For reference #6122 and some cleanup in #6088 are dependent on this fix for accessing users/1/?include=count.posts

LGTM :+1:


fixes for StringDType

This brings over some functionality from asciidtype. I also include some asciidtype changes here to keep stringdtype and asciidtype more uniform.

Makes StringScalar a str subclass. This makes it possible to use the functions in np.char to manipulate StringDType data

Adds partition and rpartition wrappers to make sure the result of those functions has uniform types.

Adds a missing incref to common_instance. Also simplifies it a bit since StringDType isn't parametric.
Can create a StringDType with an optional size argument, which gets ignored. This makes the API more uniform with numpy's other string dtypes. In particular, this way of creating a string dtype is now used in np.char, so this also allows StringDType to work with functions in np.char. Currently we just ignore the size but in principle we could use it as a size hint? I'm not sure if that's actually useful.

Honestly, now that I see arbitrary sized strings passing length seems weird.  For most functions, the long-term thing would be a ufunc replacement.  For others, I suppose we have no good story, we could hide something away like arr.dtype.array_funcs.char.mod.
Anyway, don't want to derail trials here, just seems like something better is needed if we really want full support for something like np.char.

Agreed that passing a size is weird. I'd like to come back to that, I agree that it might be better to just expose all the functionality in np.char as ufuncs. That just seemed like a bigger project than I wanted to take on right now.


Added warning that module names exist in URL space

Added a warning that module names exist in URL space, and the bundle, buildStatic and trace argument is a module name so should be / separated. In reference to #814.

Sure, thanks!


The tool of markdown maybe have a bug.

I sent a PR for fixing a document style bug:  fix markdown syntax by diguage · Pull Request #16452 · kubernetes/website
I review the render result, and I found the tool of Markdown maybe have a bug.
Before I fix the error:
    On Fedora, edit `/etc/kubernetes/kubelet` to include this line:

    ```
    KUBELET_ARGS="--cluster-dns=<IP_ADDRESS> --cluster-domain=kube.local --manifest-url=<manifest-url>`
    ```

Now, restart the kubelet. On Fedora, you would run:

    ```shell
    # Run this command on the node where the kubelet is running
    systemctl restart kubelet
    ```

https://kubernetes.io/docs/tasks/configure-pod-container/static-pod/

The four spaces shoud be render as the source code block. So the source code blocks in the four spacke block do not be render as the source code.
The blockquote tag should be used in the case.
After:
> On Fedora, edit `/etc/kubernetes/kubelet` to include this line:
>
> ```
> KUBELET_ARGS="--cluster-dns=<IP_ADDRESS> --cluster-domain=kube.local --manifest-url=<manifest-url>`
> ```
>
> Now, restart the kubelet. On Fedora, you would run:
>
> ```shell
> # Run this command on the node where the kubelet is running
> systemctl restart kubelet
> ```

https://deploy-preview-16452--kubernetes-io-master-staging.netlify.com/docs/tasks/configure-pod-container/static-pod/

But the blockquote tag does not be render in the right way.
Right render
But it should render like as following (Preview on VS Code):

The way I see it, the rendering via Hugo is canonical. Visual Studio Code's preview is a hint at how the source might get rendered.
Would you agree with
/triage support
@diguage?

@sftim

Visual Studio Code's preview is a hint at how the source might get rendered.

Sorry, I don't agress with the statement.
The > tags are a blockquote tag of Markdown.
There are two guides for Markdown which explains the syntax.

Basic Syntax | Markdown Guide
Mastering Markdown · GitHub Guides

/remove-triage support
Sounds like there's more to discuss.

Hi @diguage - I think Hugo is rendering the Markdown correctly.
Repeating part of the comment I left on your PR #16452:
Thanks for all the detail! Using blockquote '>' syntax: markdownguide.org's blockquotes with other elements section says "Blockquotes can contain other Markdown formatted elements. Not all elements can be used — you’ll need to experiment to see which ones work."    So nesting code block syntax inside of blockquote syntax doesn't work.

For everyone reading this Issue: regarding the 4 spaces (or 1 tab) at the beginning of L135-139: if you want some text to show up exactly as you write it, without Markdown doing anything to it, indent every line by at least 4 spaces (or 1 tab).
Rendered:

Code:

Looking at the source code file, the 4 spaces (or 1 tab) at the beginning of each line 135-139 is a mistake.  Those spaces should be removed - those lines should not be a blockquote.
If you install Hugo, you can test your changes locally. Hugo uses the BlackFriday Markdown processor.

/priority awaiting-more-evidence

I used to use the GitHub Format Markdown.
Markdown has too many dialects. There are many differences amount the dialects.
Let me learn the BlackFriday Markdown processor. 😆

@diguage are you OK to close this issue?


feat: rolling update for Pipeline Vertex

RollingUpdate for Pipeline.
apiVersion: numaflow.numaproj.io/v1alpha1
kind: Pipeline
metadata:
  name: simple-pipeline
spec:
  vertices:
    - name: my-vertex
      updateStrategy:
        rollingUpdate:
          maxUnavailable: 25%
        type: RollingUpdate

Hey. When reading Relase notes I have to dig deep to understand new features:
Changelog -> PR -> Linked issue -> Changed files -> Updated Markdown -> Open the updated webdocs.
Would be so nice if you could link to the updated web docs directly in the release notes :-)
For example rolling updates -> https://numaflow.numaproj.io/user-guide/reference/configuration/update-strategy/#update-strategy


Inline Message Builder / Embed Builder

I don't know if I'm using the correct implementation, but when I use this code the embeds always come out as being empty and gives the error message can't be built without content.
Dependencies are the exact same versions as the ones listed on the READ_ME
I am also getting this error within my editor when trying to use the code sample in the READ_ME:
Error Image
The code I tried
`
val message = Message {
embeds += Embed {
    title = "Hespori Reaction Roles"
    description = """
        React below to select which Hespori Seeds you'd like to be notified about.

        You'll be assigned a Discord Role for each reaction, and be tagged whenever your selected seeds are planted

        :one: = Attas (Bonus XP)
        :two: = Kronos (x2 Raids)
        :three: = Iasor (x2 Drops)
        :four: = Golpar (x2 Crystal Keys, Clues, etc.)
        :five: = Noxifer (x2 Slayer points)
        :six: = Buchu (x2 Boss Points)
        :seven: = Celastrus (x2 Brimstone keys)
        :eight: = Consecration (+5 PC Points)
        :nine: = Kelda (x2 Larran's keys)
    """.trimIndent()
    }

}`

It looks like you accidentally imported things unrelated to the builders. SendDefault.components is not something you want here.

Thank you for the prompt reply.
Oddly enough I am not able to see embeds or components within the InlineMessage context
Error

You have to use the latest master commit.

Sorry for the trouble, after Invalidating the caches then refreshing my Gradle project the context was received.
Thank you again, Happy Holidays. Hope you have a great day :D


hotfix: refactor man.dvc.org redirect again [WIP]

This time reduced code as much as possible, reintroduced ternary operator...
For #768

@jorgeorpinel merge this?


No subfolder 'emhass' in /share/ when saving outside of the default-folder

When opting to save files into /share/ there's no emhass subfolder (as indicated by the explanation)
The description below it does mention such a folder:
"data_path (Default default), selecting /data/ will store data in an enclosed folder, only assessable by the addon itself. Select /share/emhass to store data in the mounted share folder, accessible outside of the addon"
Adapting the yaml to /share/emhass is not accepted.
A few other add-ons saving data there all save it in a subfolder with their own name.

Hi, yes the files are just saved directly into the share folder.
We probably just need to update that description

I've just modified this message.
Closing this issue


Proposal for URL link correction

[フィードバックをこちらに入力してください]
In the Japanese page, the link URL of the "contribution guide" is "https://docs.microsoft.com/en-us/contribute/", but maybe it would be better for "https://docs.microsoft.com/ja-jp/contribute/"?
(I think that the link to ja-jp page is better than the link to en-us page.)
Probably the same phenomenon is occurring in pages of other languages.
It is a problem that the value of the loc variable property "disclaimer.machineTranslatedOnly" in the source of "https://docs.microsoft.com/_themes/docs.theme/master/ja-jp/_themes/global/js/4cf8db1.global.min.js" is "<a href="https://docs.microsoft.com/en-us/contribute/">".
How about changing the value of "disclaimer.machineTranslatedOnly" depending on the language?

ドキュメントの詳細
⚠ このセクションを編集しないでください。 docs.microsoft.com で必須です ➟ GitHub の問題のリンク。

ID: c34dbafa-df3c-a5a5-32d5-2ce152893534
Version Independent ID: cebdd9a6-45d7-6306-373c-7d31121f9e16
Content: PIN がパスワードより安全な理由 (Windows 10)
Content Source: windows/security/identity-protection/hello-for-business/hello-why-pin-is-better-than-password.md
Product: w10
GitHub Login: @DaniHalfin
Microsoft Alias: daniha

@samanthaw @MollyMalone
cc: @AndreaBarr

I believe this one should be closed for now.
Localized versions currently do not support contributions, therefore, the localized contribution guide isn't relevant. We'll see if we can remove it to avoid confusion.

@mypil

@AndreaBarr - This issue can be closed. Thank you.

Thank you.


Prepare for k8s 1.25

Towards https://github.com/giantswarm/giantswarm/issues/29470
What does this PR do?
Prepares the app to be deployed to gazelle / operations which is K8s 1.25 with PSS enforced.
In detail:

Add security spec for PSS to deployments
Add a switch to only render PSP resources if .Values.global.podSecurityStandards.enforced is not true
Set namespace based on helm .Release.Namespace
Change a registry to gsoci
Remove an image pull policy (Always)

Ran into this problem logged by the staticjscms pod:
/app/netlify-cms-github-oauth-provider/login_script.js:5
  throw new Error('process.env.ORIGIN MUST be comma separated list

The deployment had the ORIGINS env variable defined, but not ORIGIN, so I added it.

https://github.com/vencax/netlify-cms-github-oauth-provider/blob/master/login_script.js#L5
Turns out the error message is mistyped and the env variable must be ORIGINS.


Discussion:  Should PROCTIME() return the Timestamptz type by default？

when we use PROCTIME() functions when creating table/source, we must display and set the Timestamptz type：
dev=> CREATE TABLE proctime(v1 int, v2 AS PROCTIME());
ERROR:  QueryError: sql parser error: Expected ',' or ')' after column definition, found: PROCTIME at line:1, column:45
Near "proctime(v1 int, v2 AS"

dev=> CREATE TABLE proctime(v1 int, v2 Timestamptz AS PROCTIME());
CREATE_TABLE

It seems that flink does not need to set the type, whether we can and should be removed, otherwise we should write it clearly in the document.
https://nightlies.apache.org/flink/flink-docs-master/docs/dev/table/concepts/time_attributes/#processing-time

Not related with PROCTIME(), but our (& PG's) "generated columns" feature requires specifying type.


Need documentation in using the model

Hi,
I would like to use this model for extracting word vectors for Telugu text.
I am looking for code snippets on how to use the model for extracting word vectors. Can you please help with this? Or if there is any documentation available on using the model in Hugging Transformers, please share the same.
Thank you in advance

You can try something like this.
The snippet is taken from the discussion here and the colab seems to work as expected for indicbert model.
Please reopen this if you face any other issues.

Hey!
I found the post from Hugging Face forum yesterday and tried the same. Yep, it worked.
Thank you for sharing the information.
Best Regards
Kranti