chore: mark imageErr as unhealthy

In the case of an ImagePullError we do not mark the monoVertex pod status as unhealthy
{
    "data": {
        "resourceHealthStatus": "healthy",
        "dataHealthStatus": "unknown",
        "resourceHealthMessage": "mono vertex \"simple-mono-vertex\" is healthy",
        "dataHealthMessage": "MonoVertex data flow is in an unknown state",
        "resourceHealthCode": "M1",
        "dataHealthCode": "D4"
    },
    "errMsg": null
}

  - containerID: docker://2b1994e75d7bc05b6f5c25d3a580c9d0f0d2e7f75c4fe33b483b5d1462262dfc
    image: quay.io/numaio/numaflow-java/simple-sink:stable
    imageID: docker-pullable://quay.io/numaio/numaflow-java/simple-sink@sha256:abef6a0e609266190d1077ae3b711c7fe7f4f35cb6ae6131421793c6b9efbe08
    lastState:
      terminated:
        containerID: docker://4eb708f0610330fb3c7c64d3861c2ca6b3893975f7be58a8e708c2dbb38843e4
        exitCode: 143
        finishedAt: "2024-08-26T06:18:02Z"
        reason: Error
        startedAt: "2024-08-26T06:11:03Z"
    name: udsink
    ready: true
    restartCount: 1
    started: true
    state:
      running:
        startedAt: "2024-08-26T06:18:02Z"
  - image: source-simple-source:234
    imageID: ""
    lastState: {}
    name: udsource
    ready: false
    restartCount: 0
    started: false
    state:
      waiting:
        message: Back-off pulling image "source-simple-source:234"
        reason: ImagePullBackOff

status:
  conditions:
  - lastTransitionTime: "2024-08-26T06:10:27Z"
    message: All vertex pods are healthy
    reason: Running
    status: "True"
    type: PodsHealthy

After making the fix
status:
  conditions:
  - lastTransitionTime: "2024-08-26T06:48:48Z"
    message: Successful
    reason: Successful
    status: "True"
    type: DaemonHealthy
  - lastTransitionTime: "2024-08-26T06:43:47Z"
    message: Successful
    reason: Successful
    status: "True"
    type: Deployed
  - lastTransitionTime: "2024-08-26T07:17:46Z"
    message: Pod simple-mono-vertex-mv-0-yrvjj is unhealthy
    reason: PodImagePullBackOff
    status: "False"
    type: PodsHealthy
  lastScaledAt: "2024-08-26T06:43:47Z"
  lastUpdated: "2024-08-26T07:17:46Z"
  message: 'Degraded: Pod simple-mono-vertex-mv-0-yrvjj is unhealthy'
  observedGeneration: 1
  phase: Running
  reason: ChildResourceUnhealthy
  replicas: 1
  selector: app.kubernetes.io/component=mono-vertex,numaflow.numaproj.io/mono-vertex-name=simple-mono-vertex

  {
    "data": {
        "resourceHealthStatus": "unhealthy",
        "dataHealthStatus": "unknown",
        "resourceHealthMessage": "mono vertex \"simple-mono-vertex\" is unhealthy: Degraded: Pod simple-mono-vertex-mv-0-yrvjj is unhealthy:ChildResourceUnhealthy",
        "dataHealthMessage": "MonoVertex data flow is in an unknown state",
        "resourceHealthCode": "M2",
        "dataHealthCode": "D4"
    },
    "errMsg": null
}

@vigith Will check that and update if that is easier to maintain


@enum values not documented when the enumeration is exported

Documenting the values of a bare @enum works as expected. But if I export the enumeration, its values are not documented anymore.
Input code
// The values of the following enumeration are documented.

/**
 * Provides the list of tokens supported by the parser.
 * @enum {string}
 */
const Token = {

  /** The coverage data of a branch. */
  branchesFound: 'BRF',

  /** The number of branches found. */
  branchesHit: 'BRH',

  /** The number of branches hit. */
  branchData: 'BRDA'
};

// The values of the following enumeration are NOT documented.

/**
 * Provides the list of tokens supported by the parser.
 * @enum {string}
 */
export const Token = {

  /** The coverage data of a branch. */
  branchesFound: 'BRF',

  /** The number of branches found. */
  branchesHit: 'BRH',

  /** The number of branches hit. */
  branchData: 'BRDA'
};

JSDoc configuration
{
  "opts": {
    "destination": "doc/api",
    "readme": "README.md",
    "recurse": true
  },
  "plugins": ["plugins/markdown"],
  "source": {"include": ["lib"]},
  "sourceType": "module",
  "tags": {
    "allowUnknownTags": false,
    "dictionaries": ["jsdoc"]
  },
  "templates": {
    "default": {"includeDate": false}
  }
}

JSDoc debug output
DEBUG: JSDoc 3.6.2 (Sun, 12 May 2019 16:47:31 GMT)
DEBUG: Environment info: {"env":{"conf":{"plugins":["plugins/markdown"],"recurseDepth":10,"source":{"includePattern":".+\\.js(doc|x)?$","excludePattern":"","include":["lib"]},"sourceType":"module","tags":{"allowUnknownTags":false,"dictionaries":["jsdoc"]},"templates":{"monospaceLinks":false,"cleverLinks":false,"default":{"includeDate":false}},"opts":{"destination":"doc/api","readme":"README.md","recurse":true}},"opts":{"_":[],"configure":"etc/jsdoc.json","debug":true,"destination":"doc/api","readme":"README.md","recurse":true,"encoding":"utf8"}}}
DEBUG: Parsing source files: ["/repo/js/lcov.js/lib/branch.js","/repo/js/lcov.js/lib/function.js","/repo/js/lcov.js/lib/index.js","/repo/js/lcov.js/lib/line.js","/repo/js/lcov.js/lib/record.js","/repo/js/lcov.js/lib/report.js","/repo/js/lcov.js/lib/token.js"]

Expected behavior
The values of an exported @enum should appear in the final output.
Current behavior
The values of the exported @enum don't appear in the final output.
Your environment

Software
Version

JSDoc
3.6.2

Node.js
12.4.0

npm
6.9.0

Operating system
Windows 10 and Ubuntu 19.04

My annual summer cleaning of open issues 😄 Closing this one because it's still not resolved after one year... and I don't use JSDoc anymore, so this is no longer a problem for me.


[SOLVED] Network Module always shows "Not Connected"

It always shows "disconnected" even when it is connected and working.
The interface is right and i can't find any other issue.
I'm using an USB Wi-Fi adapter, and it should work normally, but this doesn't happens.
WM: i3-gaps
OS: Arch Linux
[module/wireless-network]
type = internal/network
interface = wlp0s18f2u2
interval = 1.0
ping-interval = 10
unknown-as-up = true

format-connected = <label-connected>
label-connected =  %essid%   %signal%    %downspeed%    %upspeed%
label-disconnected =   Disconnected
label-disconnected-foreground = #66

ramp-signal-0 = 
ramp-signal-1 = 
ramp-signal-2 = 
ramp-signal-3 = 
ramp-signal-4 = 

output of polybar -vvv:
polybar 3.2.0-70-g9725f8fb

Features: +alsa +curl +i3 +mpd +network(libnl) +pulseaudio +xkeyboard

X extensions: +randr (+monitors) -render -damage -sync -composite +xkb +xrm +xcursor

Build type: RelWithDebInfo
Compiler: /usr/bin/c++
Compiler flags: -march=x86-64 -mtune=generic -O2 -pipe -fstack-protector-strong -fno-plt -Wall -Wextra -Werror -Wno-noexcept-type -pedantic -pedantic-errors
Linker flags: -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now

output of polybar -l info:
* Loading config: /home/zsucrilhos/.config/polybar/config
* Created ipc channel at: /tmp/polybar_mqueue.11144
* Loaded monitor DVI-D-0 (1368x768+0+0)
* Found matching X resource "color0" (value=#0a141c)
* Found matching X resource "color7" (value=#d0e1e5)
* Bar geometry: 1368x39+0+0
* Found matching X resource "color1" (value=#6C909E)
* Found matching X resource "color0" (value=#0a141c)
* Found matching X resource "color1" (value=#6C909E)
* Found matching X resource "color0" (value=#0a141c)
* Found matching X resource "color3" (value=#8DADB4)
* Found matching X resource "color1" (value=#6C909E)
* Found matching X resource "color7" (value=#d0e1e5)
* Starting application
* Starting module/scratchpad
* Starting module/i3
* Starting module/xwindow
* module/scratchpad: Invoking shell command: "python ~/_my-scripts/i3-scratchpad-indicator.py"
* Starting module/filesystem
* Starting module/wireless-network
* Entering event loop (thread-id=1)
* Eventqueue worker (thread-id=6)
* Configured DPI = 96x96
* Loaded font "Iosevka Medium:pixelsize=10;1:antialias=true" (name=Iosevka, offset=0, file=/usr/share/fonts/TTF/iosevka-medium.ttf)
* Loaded font "Hack Nerd Font:pixelsize=10;1:antialias=true" (name=Hack Nerd Font, offset=0, file=/usr/share/fonts/TTF/Hack Regular Nerd Font Complete.ttf)
* Loaded font "Fixedsys Excelsior:pixelsize=10:size=8:antialias=true" (name=Fixedsys Excelsior, offset=1, file=/usr/share/fonts/TTF/FSEX302-alt.ttf)
* Loaded font "Unifont:fontformat=truetype:size=8:antialias=true" (name=Unifont, offset=0, file=/usr/share/fonts/TTF/Unifont.ttf)
* Loaded font "Wuncon Siji:pixelsize=10;1:antialias=true" (name=Wuncon Siji, offset=0, file=/usr/share/fonts/misc/siji.pcf)
* Loaded font "Font Awesome 5 Brands:pixelsize=10;1:antialias=true" (name=Font Awesome 5 Brands, offset=0, file=/usr/share/fonts/TTF/fa-brands-400.ttf)
* Loaded font "Font Awesome 5 Free:pixelsize=10;1:antialias=true" (name=Font Awesome 5 Free, offset=0, file=/usr/share/fonts/TTF/fa-regular-400.ttf)
* Loaded font "MaterialIcons-Regular:pixelsize=10;1:antialias=true" (name=Material Icons, offset=0, file=/usr/share/fonts/TTF/MaterialIcons-Regular.ttf)
* Bar window: 0x3200002
* Disabling tray manager (reason: missing `tray-position`)
* module/scratchpad: Rebuilding cache
* module/i3: Rebuilding cache
* module/xwindow: Rebuilding cache
* module/filesystem: Rebuilding cache
* module/wireless-network: Rebuilding cache
* Redrawing bar window
* Redrawing bar window
* Redrawing bar window
* module/scratchpad: Rebuilding cache
* Redrawing bar window
* Redrawing bar window
* Redrawing bar window
* module/scratchpad: Rebuilding cache
* Redrawing bar window
* module/xwindow: Rebuilding cache
* Redrawing bar window
* module/wireless-network: Rebuilding cache
warn: Termination signal received, shutting down...
* module/scratchpad: Stopping
* Deconstruction of module/scratchpad took 0 ms.
* module/i3: Disconnecting from socket
* module/i3: Stopping
warn: module/i3: Attempting to reconnect socket (reason: Unexpected EOF while reading header)
* module/i3: Reconnecting socket succeeded
* Deconstruction of module/i3 took 0 ms.
* module/xwindow: Stopping
* Deconstruction of module/xwindow took 0 ms.
* module/filesystem: Stopping
* Deconstruction of module/filesystem took 0 ms.
* module/wireless-network: Stopping
* Deconstruction of module/wireless-network took 0 ms.
* Waiting for spawned processes to end
* Reached end of application...

output of iwconfig:

enp3s0    no wireless extensions.

wlp0s18f2u2  IEEE 802.11bgn  ESSID:"FN_Claudineia"  Nickname:"rtl_wifi"
          Mode:Managed  Frequency:2.417 GHz  Access Point: C8:E7:D8:9A:93:CC
          Bit Rate:150 Mb/s   Sensitivity:0/0
          Retry:off   RTS thr:off   Fragment thr:off
          Power Management:off
          Link Quality=98/100  Signal level=86/100  Noise level=0/100
          Rx invalid nwid:0  Rx invalid crypt:0  Rx invalid frag:0
          Tx excessive retries:0  Invalid misc:0   Missed beacon:0

lo        no wireless extensions.

output of nmcli device wifi:
IN-USE  SSID           MODE   CHAN  RATE       SIGNAL  BARS  SECURITY
*       FN_Claudineia  Infra  2     54 Mbit/s  92      ▂▄▆█  WPA1 WPA2

Hey, I suspect this is the same issue as #457. To confirm, please post the output of ip link. Try setting unknown-as-up = true in the network module

Hello! I tested unknown-as-up = true already, it didn't work. Here is the output of ip link:
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN mode DEFAULT group default qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
2: enp3s0: <NO-CARRIER,BROADCAST,MULTICAST,UP> mtu 1500 qdisc fq_codel state DOWN mode DEFAULT group default qlen 1000
    link/ether 1c:1b:0d:cc:1c:a5 brd ff:ff:ff:ff:ff:ff
3: wlp0s18f2u2: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc fq_codel state UP mode DORMANT group default qlen 1000
    link/ether 08:10:77:41:b7:9c brd ff:ff:ff:ff:ff:ff

Then I'm not sure what exactly is going wrong.
Could your run
$ find -L /sys/class/net/wlp0s18f2u2 -maxdepth 1 -type f -print0 2>/dev/null | xargs -0 -I '{}' sh -c 'echo "{}: $(cat {})"'

and post the output here. This will help us debug where something went wrong.

Then I'm not sure what exactly is going wrong.
Could your run
$ find -L /sys/class/net/wlp0s18f2u2 -maxdepth 1 -type f -print0 2>/dev/null | xargs -0 -I '{}' sh -c 'echo "{}: $(cat {})"'

and post the output here. This will help us debug where something went wrong.

Sure! Here is the output of the command:
/sys/class/net/wlp0s18f2u2/uevent: DEVTYPE=wlan
INTERFACE=wlp0s18f2u2
IFINDEX=3
/sys/class/net/wlp0s18f2u2/carrier_changes: 1
/sys/class/net/wlp0s18f2u2/carrier: 1
/sys/class/net/wlp0s18f2u2/dev_id: 0x0
/sys/class/net/wlp0s18f2u2/carrier_down_count: 0
/sys/class/net/wlp0s18f2u2/proto_down: 0
/sys/class/net/wlp0s18f2u2/address: 08:10:77:41:b7:9c
/sys/class/net/wlp0s18f2u2/operstate: up
cat: /sys/class/net/wlp0s18f2u2/phys_switch_id: Operation not supported
/sys/class/net/wlp0s18f2u2/phys_switch_id:
/sys/class/net/wlp0s18f2u2/link_mode: 1
/sys/class/net/wlp0s18f2u2/dormant: 0
/sys/class/net/wlp0s18f2u2/mtu: 1500
/sys/class/net/wlp0s18f2u2/gro_flush_timeout: 0
/sys/class/net/wlp0s18f2u2/carrier_up_count: 1
/sys/class/net/wlp0s18f2u2/ifalias:
cat: /sys/class/net/wlp0s18f2u2/speed: Invalid argument
/sys/class/net/wlp0s18f2u2/speed:
/sys/class/net/wlp0s18f2u2/netdev_group: 0
cat: /sys/class/net/wlp0s18f2u2/phys_port_name: Operation not supported
/sys/class/net/wlp0s18f2u2/phys_port_name:
/sys/class/net/wlp0s18f2u2/ifindex: 3
/sys/class/net/wlp0s18f2u2/broadcast: ff:ff:ff:ff:ff:ff
/sys/class/net/wlp0s18f2u2/type: 1
/sys/class/net/wlp0s18f2u2/dev_port: 0
cat: /sys/class/net/wlp0s18f2u2/phys_port_id: Operation not supported
/sys/class/net/wlp0s18f2u2/phys_port_id:
/sys/class/net/wlp0s18f2u2/name_assign_type: 4
cat: /sys/class/net/wlp0s18f2u2/duplex: Invalid argument
/sys/class/net/wlp0s18f2u2/duplex:
/sys/class/net/wlp0s18f2u2/addr_assign_type: 3
/sys/class/net/wlp0s18f2u2/addr_len: 6
/sys/class/net/wlp0s18f2u2/tx_queue_len: 1000
/sys/class/net/wlp0s18f2u2/iflink: 3
/sys/class/net/wlp0s18f2u2/flags: 0x1003

Oh, I forgot something. Can you also give me the output of ls -Ahl /sys/class/net/wlp0s18f2u2.
I suspect that polybar can't properly detect the interface as a wireless device.

Yup! Here it is:
lrwxrwxrwx 1 root root 0 Dec 13 14:19 /sys/class/net/wlp0s18f2u2 -> ../../devices/pci0000:00/0000:00:12.2/usb1/1-2/1-2:1.0/net/wlp0s18f2u2

I don't seem to be focused today. I meant to say
ls -Ahl /sys/class/net/wlp0s18f2u2/

(Notice the slash at the end)
This will print the contents of the directory not what it's linked to

Haha, it happens, don't worry :)
 total 0
-r--r--r-- 1 root root 4.0K Dec 13 16:44 addr_assign_type
-r--r--r-- 1 root root 4.0K Dec 13 16:44 address
-r--r--r-- 1 root root 4.0K Dec 13 16:45 addr_len
-r--r--r-- 1 root root 4.0K Dec 13 16:45 broadcast
-rw-r--r-- 1 root root 4.0K Dec 13 16:45 carrier
-r--r--r-- 1 root root 4.0K Dec 13 16:45 carrier_changes
-r--r--r-- 1 root root 4.0K Dec 13 16:45 carrier_down_count
-r--r--r-- 1 root root 4.0K Dec 13 16:45 carrier_up_count
lrwxrwxrwx 1 root root    0 Dec 13 16:44 device -> ../../../1-2:1.0
-r--r--r-- 1 root root 4.0K Dec 13 16:44 dev_id
-r--r--r-- 1 root root 4.0K Dec 13 16:44 dev_port
-r--r--r-- 1 root root 4.0K Dec 13 16:45 dormant
-r--r--r-- 1 root root 4.0K Dec 13 16:45 duplex
-rw-r--r-- 1 root root 4.0K Dec 13 16:45 flags
-rw-r--r-- 1 root root 4.0K Dec 13 16:45 gro_flush_timeout
-rw-r--r-- 1 root root 4.0K Dec 13 16:45 ifalias
-r--r--r-- 1 root root 4.0K Dec 13 16:44 ifindex
-r--r--r-- 1 root root 4.0K Dec 13 16:44 iflink
-r--r--r-- 1 root root 4.0K Dec 13 16:45 link_mode
-rw-r--r-- 1 root root 4.0K Dec 13 16:45 mtu
-r--r--r-- 1 root root 4.0K Dec 13 16:45 name_assign_type
-rw-r--r-- 1 root root 4.0K Dec 13 16:45 netdev_group
-r--r--r-- 1 root root 4.0K Dec 13 16:44 operstate
-r--r--r-- 1 root root 4.0K Dec 13 16:44 phys_port_id
-r--r--r-- 1 root root 4.0K Dec 13 16:44 phys_port_name
-r--r--r-- 1 root root 4.0K Dec 13 16:45 phys_switch_id
drwxr-xr-x 2 root root    0 Dec 13 16:45 power
-rw-r--r-- 1 root root 4.0K Dec 13 16:45 proto_down
drwxr-xr-x 4 root root    0 Dec 13 16:45 queues
-r--r--r-- 1 root root 4.0K Dec 13 16:45 speed
drwxr-xr-x 2 root root    0 Dec 13 16:45 statistics
lrwxrwxrwx 1 root root    0 Dec 13 16:44 subsystem -> ../../../../../../../../class/net
-rw-r--r-- 1 root root 4.0K Dec 13 16:45 tx_queue_len
-r--r--r-- 1 root root 4.0K Dec 13 16:44 type
-rw-r--r-- 1 root root 4.0K Dec 13 16:44 uevent
drwxr-xr-x 2 root root    0 Dec 13 16:44 wireless

Aand here: https://i.imgur.com/EVesNwP.png

Thanks.
So polybar correctly detects that you are using a wireless device, that's not the issue.
No I suspect that your wifi adapter doesn't play well with the nl80211 interface (we recently switched over to that from wireless_tools).
Could you try running iw dev wlp0s18f2u2 info and posting the output of that here as well. iw uses only nl80211, as opposed to iwconfig which, I think, uses the old Wireless-Extension

Hello! I tried running iw dev wlp0s18f2u2 info but it says "zsh: command not found: iw" which is strange because i have wireless_tools installed:
$ pacman -Q wireless_tools
wireless_tools 30.pre9-2

Yeah, seems like your wifi adapter doesn't work with the new library. Is it an older adapter?
You can try to recompile polybar and pass the -DWITH_LIBNL=OFF to cmake, this should force polybar to build with wireless_tools, which should support the adapter.

Ok, i'll try to recompile it when i get home and post here the results. Many thanks for now.

If this is indeed the case, I would be interested what kind of device you have. I thought nl80211 (which is the new interface that we are using) was supported by almost all drivers, it seems though not by yours.

Hello! I got home and recompiled polybar with the -DWITH_LIBNL=OFF flag to cmake. IT WORKED :)
I'd like to thank you for your effort in helping me @patrick96 . Merry Christmas and happy new year.

I'm closing the issue as is is solved.

Awesome! Merry Christmas to you too!
I haven't found much information about that adapter and no release year. I hope there aren't too many adapters out there where nl80211 isn't supported. I was planning on dropping support for wireless_tools in the next major release, but maybe we need to wait on that.

You mentioned that it'd been fixed and that a release was not made yet.
Installing polybar-git from the AUR seems to have fixed the problem. Should I close the issue?


All signs are errors (no warning sign)

I've configured syntastic to check php errors and phpmd syntax warnings. All I see are errors, with the error sign and highlight (even the syntax ones, marked as x). Did I miss anything? I expected the syntax ones to be warnings (⚠).
My syntastic config:
set statusline+=%#warningmsg#
set statusline+=%{SyntasticStatuslineFlag()}
set statusline+=%*

let g:syntastic_always_populate_loc_list = 1
let g:syntastic_auto_loc_list = 0
let g:syntastic_check_on_open = 1
let g:syntastic_check_on_wq = 1
"no annoying code sniffer errors
let g:syntastic_php_checkers = ['php', 'phpmd']
let g:syntastic_error_symbol='✗'
let g:syntastic_warning_symbol='⚠'
let g:syntastic_style_error_symbol = '✗'
let g:syntastic_style_warning_symbol = '⚠'
let g:syntastic_enable_signs = 1

let g:syntastic_mode_map = {
            \ "mode": "passive",
            \ "active_filetypes": ["ruby", "php"],
            \ "passive_filetypes": ["puppet"] }

Syntastic has syntax checkers and style checkers (the distinction is pretty much arbitrary).  Each of them can in principle (depending on the corresponding linter's capabilities) recognize errors and warnings.  So you can have syntax errors and warnings, and style errors and warnings.  You can configure syntastic to show different signs for them using error_symbol, warning_symbol, style_error_symbol, and style_warning_symbol.
Now, php is s syntax checker, and it can produce only errors (the linter doesn't distinguish between errors and warnings).  On the other hand phpmd is a style checker, and it can also produce only errors (the linter doesn't distinguish between errors and warnings).  Since you configured both g:syntastic_error_symbol and g:syntastic_style_error_symbol  to be ✗, you won't be able to see any difference between them.

Thanks a lot


Added options for scrollspy.

Added options :

activeClass : permits to change the class name added.
getActiveElement : permits to change the element where add class.

For example, these options are useful to make works the scrollspy with a navbar. In this case, the active class is added to the li but not to the a element. To do this, the getActiveElement function must return the base.parent() object.

scrollspy has been rewritten


Check Parent then Check Childs

Hello
Please Add an Option When selectable option set true, on Check Parent node checkbox then child of that Checking Automatically And Add an event onCheckNode for Customize Checking Nodes Checkbox Functionality.
regards

Hi Again,
When selectable option set true on node open animation not working.
regards


chore (Order.Ring.Defs): rings comes before order in classes

Some classes like OrderedSemiring extend Semiring, OrderedAddCommMonoid while others like OrderedCommSemiring extend OrderedSemiring, CommSemiring. This makes Lean unhappy and results in users being forced to provide mul_comm fields in the presence of CommSemiring.

!bench


Extremely slow cache on self-hosted from time to time

It usually takes a few seconds to load the cache. But from time to time it takes a very long time. In my opinion, this has something to do with the slow operation of the Github API. But I can't prove it yet.
I would like to have some transparency about what happens when the action returns a response like Received 0 of XXX (0.0%), 0.0 MBs/sec. This will allow us to localize and solve the problem completely, or find a workaround.

I have been facing  with similar situation

Seeing the same behaviour on self-hosted (GCP, europe-west3) and hosted runners. Unfortunately, it's not consistent and debugging the hosts when it's occurring doesn't point to anything on the boxes themselves.

In our case, the severity rose to critical, because it affected the operation of the CD pipeline. In our case, self-hosted runners are running in GCP southamerica-east1.

Experiencing the same issues on GCP, I had to implement a timeout on the whole job because the cache action routinely takes 10 plus minutes for 30MB

Our workaround
    - name: Cache mvn repository
      uses: actions/cache@v2
      timeout-minutes: 5
      continue-on-error: true
      with:
        path: ~/.m2/repository
        key: cache-m2-${{ hashFiles('pom.xml') }}
        restore-keys: |
          cache-m2-

@levonet @akushneruk @SophisticaSean @es Are you still seeing the same behaviour on self-hosted runners? If yes, can one of you please enable debugging and the share log file here. Thank you.

There have been no problems in the last month.

Hey,
We are seeing this happening fairly regularly with our customers at BuildJet for GitHub Actions.
Most of the reports in this thread are from people self-hosting, but this issue doesn't seem to be unique to self-hosting, the hosted runners have the same issue. I don't know whether you have more insights into workflow runs that use native runners, but here is one example from Google Chrome Lighthouse where the actions/cache@v3 step is failing exactly like described in this thread.
As we have a lot of traffic running on our infrastructure, we really notice when bugs happen with commonly used actions, and we'd be more than happy giving providing you with insights into how users are failing. You can reach
me at<EMAIL_ADDRESS>@vsvipul

@adamshiervani The workflow you pointed to ran 18 days ago. Can you share anything which happened in the last week. 1 point of slowness might be an outlier in which case an upgrade/downtime might have happened? If this happens every few days, I'd like to know more about the frequency.

We've been experiencing this intermittently on builds. When it happens, the cache takes 12+ hours to retrieve. It would be nice if it were possible to set a timeout to deal with this.

@vsvipul - we are heavy users of BuildJet and this action and this is probably the most common issues we face.
Here are some runs for you:
https://github.com/WaterfoxCo/Waterfox/runs/7544425566?check_suite_focus=true#step:5:83
https://github.com/WaterfoxCo/Waterfox/runs/7543708504?check_suite_focus=true#step:5:436
We'll get this issue every week, as we push a lot of builds out. No rhyme or reason to when it occurs.

we are heavy users of BuildJet and this action and this is probably the most common issues we face.

We're also users of BuildJet, bless their offerings, and had to disable caching in some of our jobs because of this.

This has consequences in incurring additional runner costs, if not timeout limited.
There is a newer issue opened https://github.com/actions/cache/issues/810, and an environment variable SEGMENT_DOWNLOAD_TIMEOUT_MIN was introduced https://github.com/actions/cache/issues/810#issuecomment-1211766031 which was then later reported as not working https://github.com/actions/cache/issues/810#issuecomment-1258700696.
Just happened on a github-hosted runner, which failed to download 3GB during 1 hour (I was not aware of the issue so didn't have any remediation settings), by slowing down the download more and more, https://github.com/marcindulak/jhalfs-ci/actions/runs/3210575787/jobs/5249037125
...
Received<PHONE_NUMBER> of<PHONE_NUMBER> (55.8%), 0.6 MBs/sec
Warning: Failed to restore: Aborting cache download as the download time exceeded the timeout.
Cache not found for input keys: something

"Re-run failed" executed several minutes after the original failure, proceeded without the cache download slowdown, and finished the cache download in 2 minutes.
...
Received<PHONE_NUMBER> of<PHONE_NUMBER> (100.0%), 103.4 MBs/sec
Cache Size: ~3666 MB<PHONE_NUMBER> B)

i've also had this happen sporadically on github hosted runners and pretty consistently on buildjet runners

We now use
uses: actions/cache@v3
with that timeout suggested in https://github.com/actions/cache/issues/720#issuecomment-1030412967 .
I cannot recall having seen the issue (builds failing due to this timeout) in the last while, perhaps last few months.

We see this with actions/cache@v3 all the time, unless you go and add a short timeout to every cache definition jobs back up waiting forever.

In the cases where this problem occurred for us, it seemed to be the case that restoring from the cache, according to the logs, went 0% to say 60% very quickly and then stalled or trickled the rest of the way.
I never saw one like the one shown in the screenshot https://user-images.githubusercontent.com/862951/206079348-b522b45f-9fde-404c-aa64-0c64cd335dd0.png actually complete. I had the impression if it was going to complete it was going to do it quickly and right away. Once it started reporting that it was going much more slowly, no amount of waiting would have sufficed. Thus we looked at what our restores were taking and set timeouts accordingly. Basically either the restore is finished with the 2min (for example) or it will not finish (it will not take 30min and if it starts taking such a long time, it is one that hangs).

We now use uses: actions/cache@v3 with that timeout suggested in #720 (comment) .

This workaround does not work for us, it doesn't seem to be a supported argument (perhaps because it's in the step of an action, rather than a workflow):
Error: /.github/actions/foo/action.yml (Line: 38, Col: 5): Unexpected value 'timeout-minutes'
Error: GitHub.DistributedTask.ObjectTemplating.TemplateValidationException: The template is not valid. /runner/_work/foo/foo/./.github/actions/foo/action.yml (Line: 38, Col: 5): Unexpected value 'timeout-minutes'

expanding the example:
name: Caching Primes

on: push

jobs:
  build:
    runs-on: ubuntu-latest

    steps:
    - uses: actions/checkout@v3

    - name: Cache Primes
      id: cache-primes
      uses: actions/cache@v3
      timeout-minutes: 10                   # <---------
      with:
        path: prime-numbers
        key: ${{ runner.os }}-primes

    - name: Generate Prime Numbers
      if: steps.cache-primes.outputs.cache-hit != 'true'
      run: /generate-primes.sh -d prime-numbers

    - name: Use Prime Numbers
      run: /primes.sh -d prime-numbers

@definelicht
We have added an environment variable SEGMENT_DOWNLOAD_TIMEOUT_MINS to timeout the download after their segment download crosses a certain amount of time. You can refer the readme for more info.

@adamshiervani The workflow you pointed to ran 18 days ago. Can you share anything which happened in the last week. 1 point of slowness might be an outlier in which case an upgrade/downtime might have happened? If this happens every few days, I'd like to know more about the frequency.

@vsvipul this happens to us pretty often, at least a few times a week. We also use Buildjet. One case where we recently ran into this was the setup-node step at this build, if it helps you debug: https://github.com/stainless-api/stainless/actions/runs/4103099342/jobs/7076935062

👋🏽 @rattrayalex, are these private repos? unfortunately we can't see the logs. Can you please elaborate on the issue you are facing.

Ah ok. Basically the network hangs for many minutes (often long enough to hit our 20min timeout) - same as the other screenshots posted here.
Yesterday this was happening on almost every PR for us.
I'd recommend you reach out to @adamshiervani for more details as they've done some investigations of this at buildjet.

Also noticing this behavior while using buildjet:
https://github.com/FuelLabs/fuel-core/actions/runs/4179573798/jobs/7239680006
Using the rust-cache action sporadically takes several factors of time longer than a clean build. On average this makes self-hosted runners perform worse overall than using basic github runners (even if the actual execution stage is faster).

Same here: https://github.com/mlflow/mlflow/actions/runs/4179427251/jobs/7239362333

I'm now seeing this on github's larger hosted runners as well:
Run actions/setup-node@v3
Found in cache @ /opt/hostedtoolcache/node/16.19.0/x64
Environment details
/usr/local/bin/yarn --version
1.22.19
/usr/local/bin/yarn cache dir
/home/runner/.cache/yarn/v6
Received 243269632 of 424308030 (57.3%), 231.3 MBs/sec
Received 420113726 of 424308030 (99.0%), 199.8 MBs/sec
…
Received 420113726 of 424308030 (99.0%), 0.3 MBs/sec
Received 420113726 of 424308030 (99.0%), 0.3 MBs/sec
Received 420113726 of 424308030 (99.0%), 0.3 MBs/sec
Error: The operation was canceled.

We've had this happen a handful of times recently. This job was https://github.com/stainless-api/stainless/actions/runs/4232471756/jobs/7352268632 if that's useful to the GHA team.

We're seeing it on normal GitHub hosted runners now.

@sammcj , we are aware of this problem in the GitHub hosted runners and are investigating it thoroughly.

@adamshiervani The workflow you pointed to ran 18 days ago. Can you share anything which happened in the last week. 1 point of slowness might be an outlier in which case an upgrade/downtime might have happened? If this happens every few days, I'd like to know more about the frequency.

We currently use BuildJet and see this fairly regularly.

We can't see this run @michael-projectx you should paste the output of the terminal...

@Sytten
The logs are quite long, but I'll try to post the relevant bits here:

To start, spinning up the container usually takes a short period of time, but in this case, over an hour.
Hopefully this is helpful:
github_action.log

Hey everyone,
I'm Adam, co-founder of BuildJet. Our customers often complain about reliability and speed issues like the one reported here. Today, we launched BuildJet Cache, a GitHub Cache alternative thats reliable, fast and free.
It’s fully compatible with the GitHub’s Cache, all you need to do is replace action/cache with buildjet/cache. It works on any runner - be it official, self-hosted, or BuildJet. Whichever runner you choose, BuildJet Cache doubles your free storage, offering a generous 20 GB/repo/week storage space at no cost.
Head over to our launch post for more details: https://buildjet.com/for-github-actions/blog/launch-buildjet-cache

@thinkafterbefore Since you hijacked the post for promotion let me push back a little.
Your cache fork is really not well done, it's just a build on top of the existing cache action with a random closed source https://www.npmjs.com/package/github-actions.cache-buildjet.  Ideally do proper tagging too.
I would be much more supportive of an action that actually allowed people to not be dependant on any one provider similar to actions-cache-s3. Devs of the the github cache don't seem to care about that.

@Sytten, I hear you, and I apologize if my prior message was too promotional, my intention was simply to help people solve a long-standing issue with the actions/cache.
Regarding the github-actions.cache-buildjet package, I totally get your concerns about it being closed source. To ensure full insight, we’ve open sourced it. You can review the repository here.
Lastly, as we wrote in the launch post, if people are interested in using their own object storage, whywaita/actions-cache-s3 is perfect for that. We simply wanted to alleviate that for the users.
Thanks for pushing back and calling out where we need to improve.

@thinkafterbefore can you publish in https://github.com/marketplace?type=&verification=&query=buildjet/cache please? :)

I also started my own solution, forked the actions/cache code and modified it to place a tarball on a local volume instead upload/download all the cache everytime.
You can use it as a drop-in replacement for actions/cache@v3: https://github.com/maxnowack/local-cache

Still an issue

Can we have an ENV variable that disables saving/restoring cache (maybe 2 separate variables) so that we can stop when other actions try to use this action.

Will we?


[Base] Add a task to display Kotlin compiler settings

Fixed with https://github.com/aalmiray/kordamp-gradle-plugins/commit/c41b3afb17298ce69f953c9065b5cfdc404cf005


test: fixing test files

that were pointing to the previous destination of the typechain's types

Good catch!


fix json#any bool value, fixed #3632

Actually, if the underlying value is null maybe a JSON::Any with a value of nil should be returned. For example:
require "json"

p JSON.parse(%q<{"some": null}>)["some"].class # => JSON::Any

That is, there is a value in that position, it's just null.
I have a fix for this (using Hash#fetch(...) { ... } and Array#at(...) { ... }. I can push it, but I wanted to discuss this first.

i think null should be nil

Yeah, I think the semantic difference is very tiny and returning nil is more convenient in the end


[Bug - conflict] The Outliner (plug-in) folded indicator not showing

Hi,
Thanks for a great theme.
Step to reproduce:

Use Outliner plugin.
Fold a header or a bullet point.
(expected behavior: show ... by the end of the parent line)
Show the line is folded with no indicator ...

I understand that you want to maximize the focus in this theme so you hide a lot of elements. And it seems like not needed (as the arrow indicator showing fold or unfold already). However the ... is pretty visual hint for me.
Hope you can check for an option to re-enable it.
I attached a picture showing the quick inspecting of the CSS class for that (in other themes).

By the way, the current folding also carry the child code-block formatting if included.
Unfolded

Folded

You can simply re-enable the folding buttons with the style settings plugins :)

Oh sorry I missed that. Did dig a bit in the Style Settings. My bad. I guess I was looking for something mentioning "..." :D
Thanks for the great theme.
NA

On an updated note: I tried and don't think that the setting is working (did try updated the theme).

ah, you meant the other folder markers 😝
Sorry, I forgot that Obsidian has two types of folder markers (the arrow & the ...). I added a new style setting option to re-enable the ... :)


Add parallel stream analysis

This can help identify possible race issues in a migraphx program. This will run on the gpu tests.

@pfultz2 The analyze_stream is not actually called right?

The analyze_stream is not actually called right?

Its called by the unit test, and also by the gpu tests (here)[https://github.com/ROCmSoftwarePlatform/AMDMIGraphX/pull/629/files#diff-dd8cbdcad1cd67247f99f938ed5db789R134].


Suggestion for key metrics

Hi,
I used this exporter a year or two ago, but revisiting it now I can see a few things missing which I'd consider essential info:

# commits
# contributors

Would you be interested in adding this or accepting a PR?
Alex

I wanted this capability so I went ahead and worked on it for my personal use. I have submmited PR. Hopefully it can be reviewed and merged into master branch.
https://github.com/infinityworks/github-exporter/pull/39


Cannot read property 'LatestVersion' of null

Getting this attempting to install BetterDiscord with fresh Discord install.

Has this been fixed with 0.2.8?

Sorry for the (very) delayed response.
Yep! Seems to be working fine with 0.2.8.


Whitespace padding in object literals, destructuring assignment, and import statements

Enforce whitespace padding in object literals:
{ myField } // good
{ myField: "hello" } // good
{myField} // bad
{myField: "hello"} //bad

I assume the same style should apply to named imports.

Any progress?

+1

+1

an option for the same thing for arrays would be great too. [a, b] vs [ a, b ]

I prefer it without the spaces, so an option for that would also be nice.

+1

It would also be nice if we could enforce spacing before and after the colon like
{ foo : 'bar' } //good { foo: 'bar' } //bad { foo:'bar' } //bad

Any progress on this? I would really like to use this in our projects at Acquia.

OMG, it still open?

BTW tslint-eslint-rules supports in nearly all cases (not in object literal types) with the rule object-curly-spacing. Perhaps this could be ported to tslint?

we use the following to achieve this:
{
    "extends": [
        "tslint-eslint-rules"
    ],
    "rules": {
        "object-curly-spacing": true,
        "array-bracket-spacing": [
            true,
            "never"
        ]
    }
}

Any update on this? :)

@micheleangioni I'm just using eslint plugin for integration tslint with eslint and that solves my problem linked with this issue

I tried out object-curly-spacing from tslint-eslint-rules but it doesn't seem to cover empty objects ({} vs. { }), unless I misconfigured something.
Would be great to have something like this in tslint proper!

I see https://github.com/palantir/tslint/pull/4068 added a check for spaces after open braces, but it doesn't detect more than one space, and doesn't deal with closing braces and empty objects.

There have been very few PRs submitted to address this issue, and it is not a priority for TSLint core (see #4534), so I'm going to close this. If you want to implement this whitespace check, it can happily live in a custom rule set outside this repo.

I was looking for the solution and a colleague suggested it is already part of TSlint.
https://palantir.github.io/tslint/rules/whitespace/

You just need to add below rules in tslint.json file:
    "whitespace": {       "options": [         "check-branch",         "check-decl",         "check-operator",         "check-separator",         "check-type",         "check-typecast",         "check-module",         "check-rest-spread",         "check-preblock",         "check-postbrace"       ]     },
Here         "check-preblock",         "check-postbrace" this rules enforce to add white space after braces

🤖 Beep boop! 👉  TSLint is deprecated 👈  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


How to render TMX How to render TMX in ThreeJs plane?

Hi.
I have studied your project for more than a month.I tried to run the code but it didn't work.

I haven't used three.js in this project in a really long time. Around 6 years ago I did this.
More recently I made a generic webgl renderer for tiled maps: https://github.com/englercj/gl-tiled

Thank you very much.


Added complete color theming support for Git

I've noticed that the recently merged Git feature only has one default color, so I completed it. I've added a default color theme for it that looks like this (ignore the conflicted, I didn't bother actually making a repo for it):

... and with this theme of mine:

... it looks like this, works fine:

Edit: also suppressed ErrorKind::NotFound for Git status since if you don't this could happen when you are in a directory that's been deleted:

TODO

[x] Use cargo fmt
[x] Add necessary tests
[x] Add changelog entry
[x] Update default config/theme in README (if applicable)
[x] Update man page at lsd/doc/lsd.md (if applicable)

Codecov Report

Merging #852 (f40bd21) into master (2fe3fcd) will decrease coverage by 0.19%.
The diff coverage is 20.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master     #852      +/-   ##
==========================================
- Coverage   86.48%   86.30%   -0.19%
==========================================
  Files          49       49
  Lines        4602     4614      +12
==========================================
+ Hits         3980     3982       +2
- Misses        622      632      +10

Impacted Files
Coverage Δ

src/color.rs
47.08% <0.00%> (-2.36%)
:arrow_down:

src/git.rs
87.42% <0.00%> (-0.53%)
:arrow_down:

src/theme/color.rs
79.13% <100.00%> (+0.36%)
:arrow_up:

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

thanks @k4yt3x! nice work!


BasePathDetector::getBasePath returns wrong path with php-fpm

getBasePath returns an empty path when you are using php-fpm with apache.
This patch fixes the problem.
Saludos
--- selective/basepath/src//BasePathDetector.orig.php	2020-12-09 19:19:09.000000000 -0300
+++ selective/basepath/src/BasePathDetector.php	2021-07-12 14:18:49.880003620 -0400
@@ -37,7 +37,7 @@
     public function getBasePath(): string
     {
         // For apache
-        if ($this->phpSapi === 'apache2handler') {
+        if ($this->phpSapi === 'apache2handler' || $this->phpSapi === 'fpm-fcgi') {
             return $this->getBasePathFromApache($this->server);
         }

php-fpm works differently and is currently not supported. Can you create a PR (with tests)?

php-fpm works differently and is currently not supported. Can you create a PR (with tests)?

Done. New proposal with a generic solution, although probably not fully complete. #9

Ok thanks.


Request to be updated on resolution of NextJS app router scroll reset by searchParam issue

I am following the progress on the following issue and pull request and would like to be updated when they are resolved:

Issue: [ref1]
Link: https://github.com/vercel/next.js/issues/49087

PR: [ref2]
Link: https://github.com/vercel/next.js/pull/52498

Please update or close this issue when the aforementioned issue and PR have been officially fixed and merged. Thank you!

Ok I will look over it


should poshtools host echo command in REPL as it does output?

When I run a command/script via the host, the output is redirected to REPL.  This is nice, but I don't see the command that was executed that generated all of that output.  I can't tweak it and try again without going through the host.
This is slightly different than the ISE.  The ISE shows the command but it's not part of command history - this feels like the correct behavior.  I can see the command(s), I can copy paste and now make them part of REPL command history and modify, etc.
Feels like we should follow suit here, but worth discussion for sure.

Do you have an example?  I don't understand how it differs yet :)

In the ISE, run a script that's loaded by pressing F5.  You'll see the command line in REPL echo the command that is run by pressing F5, i.e. the full path of the file.
In poshtools, we don't echo the command though we do echo the output from the host.  So we have a bunch of output over in REPL, but no indication where it came from.
Somewhat related, we also echo the output above the command prompt.  We should probably disable the command prompt during that execution since we don't allow another pipeline when the host is running.

Oh, you mean running it via F5/Editor?  It doesn't put it into the command history [eg, pressing up] in the ISE Repl, but you're right in that it does display the output.  I think we should display something in the REPL Output, but I'm not too keen on the entirety of the script.  Imagine a large script there.  How useful would that be?

yeah F5/Editor - I think we should put the command the was executed... so if the command was:
c:\foo.ps1 -someparam "value"
we put that...
But a host called could also submit a script in-line (i.e. multiple lines and not from a file) - we probably need to be consistent and show that too - i.e. show what created the output (ISE does this for scripts that are not save to disk yet).
It could be monstrous but we should be predictable/consistent

First part wouldn't have any params, given that when we run stuff from the editor, we just run it and push out any commands when we get a prompt from the Host.
We should mirror the ISE though for running from a saved file :)

This is where there's something different though: if the File is unsaved in the ISE, it'll run the whole thing in the REPL on F5.  However, we always force a save.  I think we should continue always forcing a save.
ISE:

For the interactive scenarios (e.g. F5) I think forcing a save is a good idea (consistent & VS behavior).

Tweaked title a bit now that we've figured out we want to do!

Cool - but remember there are 2 scenarios here: 1) from editor and 2) from host


Add MenuPopupChanged (orig. Qiming Zhao)

This is a translation of https://github.com/neovim/neovim/pull/9616, which adds an autocmd for when the popup menu item changes during insert completion (and also when the pum is redrawn due to resizing).  This is useful e.g., for showing information about the selected item in the cmdline or making text edits during selection.  The original author is Qiming Zhao aka. @chemzqm.

The name is very confusing, since MenuPopup is for the right-click popup menu, while this event is for the completion menu, aka PUM.  We do have the CompleteDone event, which comes much closer.  Can we call it CompleteChanged ?

There is definitely confusion in the documentation between the two popup menus (PUM vs right click) but I am not certain this autocmd adds to it.
FYI the specific justification for the name MenuPopupChanged was given by @justinmk as follows:

MenuPopupChanged also sounds very similar to MenuPopup which is autocmd for right click. Do we want to change the autocmd to CompleteChanged or CompleteItemChanged instead?

CompleteFoo as already discussed is not appropriate because this event is not completion-related, it is popup-related. The name MenuPopupChanged is intentional and was explained in #9616 (comment) :
Vim has an old event called MenuPopup (grep EVENT_MENUPOPUP in the Vim source) which used to be GUI-only but is now also used for :Termdebug stuff.
So let's name this new event MenuPopupChanged. Like Vim, we'll eventually have the ability for plugins to show an arbitrary popupmenu, so it's not just for "completion".

This autocmd also reacts to resize events, even when the selected item doesn't change, and provides the position/dimensions of the completion PUM.
What do you think?

The name is very confusing, since MenuPopup is for the right-click popup menu, while this event is for the completion menu, aka PUM

What is the "right-click popup menu" if not a PUM? (actual question, not rhetorical)

What is the "right-click popup menu" if not a PUM?

See :help 'mousemodel'.  If it is set to "popup" (it is the default of Windows), the popup menu will be shown (on gVim).

I mean semantically, not technically.

There is definitely confusion in the documentation between the two
popup menus (PUM vs right click) but I am not certain this autocmd
adds to it.

It does, because it uses the prefix MenuPopup instead of Complete, and
the event is associated with Complete and totally unrelated to
MenuPopup.
Both are popup menus, just like popup menus exist in applications
everywhere.  To distinguish them, one is called the right-click popup
menu, the other the completion popup (or Insert mode completion popup
menu, to be more specific).

FYI the specific justification for the name MenuPopupChanged was given
by @justinmk as follows:

MenuPopupChanged also sounds very similar to MenuPopup which is
autocmd for right click. Do we want to change the autocmd to
CompleteChanged or CompleteItemChanged instead?

CompleteFoo as already discussed is not appropriate because this event is not completion-related, it is popup-related. The name MenuPopupChanged is intentional and was explained in #9616 (comment) :
Vim has an old event called MenuPopup (grep EVENT_MENUPOPUP in the Vim source) which used to be GUI-only but is now also used for :Termdebug stuff.
So let's name this new event MenuPopupChanged. Like Vim, we'll eventually have the ability for plugins to show an arbitrary popupmenu, so it's not just for "completion".

This autocmd also reacts to resize events, even when the selected item
doesn't change, and provides the position/dimensions of the completion
PUM.
What do you think?

This reasoning is bogus.  Autocommand events are not there for generic
notifications, they must be as specific as possible.  In this case what
the plugin writers want is an event when the completion popup menu
changed.  It's definitely not for the right-click popup menu.
The "PUM" as it's called internally, can be used for other things, but I
doubt we want to use the same event for contents not related to
Completion.
--
hundred-and-one symptoms of being an internet addict:
130.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

It matters very little to me what the name is, because all choices so far are suboptimal. MenuPopupChanged was an attempt to read the tea-leaves of Vim design decisions.

This reasoning is bogus.  Autocommand events are not there for generic
notifications, they must be as specific as possible.

It is conceivable that plugin authors will also want MenuPopupChanged information for the "right-click menu", not only the "completion menu". Does that mean there will be MenuPopupChanged and CompletionChanged events?

The "PUM" as it's called internally

It's called PUM externally as well, including for pumvisible(), Pmenu, etc. None of this makes any sense.

It matters very little to me what the name is, because all choices so
far are suboptimal. MenuPopupChanged was an attempt to read the
tea-leaves of Vim design decisions.

I prefer coffee :-)

This reasoning is bogus.  Autocommand events are not there for generic
notifications, they must be as specific as possible.

It is conceivable that plugin authors will also want MenuPopupChanged
information for the "right-click menu", not only the "completion
menu". Does that mean there will be MenuPopupChanged and
CompletionChanged events?

I don't see how the right-click popup menu can change after it's
displayed.  Or why someone would be interested in that change if it were
possible.
For Completion there are lots of plugins trying to do something
complicated, thus adding an event specifically for completion would make
sense.
Most interesting is probably when the selected entry changes, then a
preview of the suggestion could be displayed.  Would be more useful once
the popup window support has been implemented.
--
Birthdays are healthy.  The more you have them, the longer you live.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Updated for recent insexpand.c refactor.  Replaced MenuPopupChanged with CompleteChanged (Note: I imply no opinion on the naming and do not intend to cause tension between vim and neovim).

@brammool  once again, https://github.com/vim/vim/commit/d7f246c68cfb97406bcd4b098a2df2d870b3ef92 does not include attribution to the original author nor the Neovim project. That's not how copyright works.

Attribution has nothing to do with copyright.  Since a pull request is
created, the author has explicitly confirmed agreeing with the copyright
conditions of the project.
Every change can be changed by others, it's close to impossible to
mention all people involved.  I also don't mention what I changed on top
of the pull request.  I do try to mention at least the person who sent
the change.  Github user names don't make this easier.
--
Get a life?  What is the URL where it can be downloaded?
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Since a pull request is created, the author has explicitly confirmed agreeing with the copyright conditions of the project.

I think you mean the licensing conditions of the project.  Since Vim doesn't use a copyright assignment agreement, the author retains copyright.  The assumption is that, unless stated otherwise, they're contributing the code under Vim's license.

I do try to mention at least the person who sent the change. Github user names don't make this easier.

For what it's worth, Andy made sure to include the original author's name in both the subject of this PR and the commit message.


VTK 9.4 compatibility issue

VTK 9.4 is now released and changed its OpenGL loader from glew to glad.
While our master branch is already compatible, it's needed to fix a few issues:

[x] Add VTK 9.4 in the CI
[x] Check if F3D 2.5 is compiling. If not, patch and release a 2.5.1 version

Fixed


cant push the tag

I have yarn and npm set up so they dont use the 'v' prefix on releases and that causes skpm to not be able to push the tag for obvious reasons as it cant find 'v1.0.1'

hum wait, npm version patch return v1.0.1 even if the tag is called 1.0.1?

in .npmrc I have tag-version-prefix= set so my tags are just 1.0.0 etc. example https://github.com/mcansh/blog/releases/tag/2.4.5, notice the tag is just 2.4.5 and not v2.4.5.

for my plugin repo though, I could initiate a local npmrc file which reverses that ¯_(ツ)_/¯

Yes but what I’m doing is getting the output from npm version. There shouldn’t be any issue if non output the right tag. But I guess it doesn’t. Not sure how to solve it, looks like an npm bug

¯_(ツ)_/¯ seems to have solved itself


Base ViT-small & ViT-tiny pretrain model/weight

Hi authors,
I would like to ask about the pretrain model/weight you provided. I'm currently using base ViT model with 16 patches on my project, if I want to use the pretrain model you provided, do you have any suggestion to use which pretrain model/weight?
Thank you for your help

Hey -- if you want to reproduce the results in the main text of the paper, use the weights in the CAML_CLIP folder. If you want to maximize the performance of CAML (see appendix tables), use CAML with the Laion2b weights.
Just make sure the feature encoder is set to CLIP (or laion2b) to match the weights of the transformer encoder.


Document PMF generation choices

Not in scope for this PR, but I think we need to do a better job documenting and explaining our choice to always discretize the distribution ourselves and pass the pmf

Originally posted by @kgostic in https://github.com/CDCgov/cfa-epinow2-pipeline/pull/26#discussion_r1752921040

As in alternative could be to take parameters and discretise within modelling functions here? It would be less information to store to do it that way. But would also need to know the distribution. And also would be less flexible as to other ways to get PMFs (that aren't a particular workflow with a distribution).

Adam, in light of the work in cfa-parameter-estimates what are your thoughts on what should go here vs. there? Have they changed at all?

cfa-parameter-estimates is not public and the path to making it public is not easy. Not a complete answer but just to say if you want documentation to be public it's going to be a while before it can fully be pushed upstream. But yes eventually that's my preference too.

@kgostic -- thoughts?


Add prettier

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

Comparison is base (7d7ba75) 33.97% compared to head (864f592) 33.97%.
Report is 3 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2600   +/-   ##
=======================================
  Coverage   33.97%   33.97%
=======================================
  Files         204      204
  Lines       13083    13083
=======================================
  Hits         4445     4445
  Misses       8285     8285
  Partials      353      353

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

Please resolve conflict


Custom header collection sorting doesn't support date format

Hi,
I'm trying to establish custom sorting of collection of pages in a timetable like this:
order:
    by: header.eventdate
    dir: asc

Unfortunately it doesn't seem to support datetime format that is stored in this custom header (i.e. eventdate: '03-02-2017 22:18').
As a result event from February 3rd comes before January 20th for example. I can resort to using just date for sorting, but date of creation an item is not an event date per se.

I've looked at this and really we have no way to really know what or how to order custom header type sort fields. For this reason, I suggest you change the header.eventdata to use ISO format dates: yyyy-mm-dd.  This way the sort order will be correct with regular natural sort order that is assumed for all such fields.
You can then reformat this on display via the Twig Date filter: http://twig.sensiolabs.org/doc/2.x/filters/date.html

The thing is, though, that the format in the field is defined by this blueprint:
header.eventdate:
                      type: datetime
                      label: Event date

And the actual data is then set through Admin plugin, is there any way to change the format Admin plugin uses?

This is now fixed. The fix involves a new option for the datetime field that allows you to specify manually the format. So in your case you could write in the blueprint:
header.eventdate:
  type: datetime
  label: Event date
  format: 'Y-m-d'

which will force the format to 2017-01-20 for that particular field.


Bump dwn-sdk-js to 0.2.14

Bump dwn-sdk-js to 0.2.14 which now uses @web5/dids.
Using TestDataGenerator to generate a did:key Persona.
Bump dwn-sql-store to 0.2.8

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

Comparison is base (f16e915) 90.63% compared to head (dd0024a) 90.63%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #104   +/-   ##
=======================================
  Coverage   90.63%   90.63%
=======================================
  Files          20       20
  Lines        1752     1752
  Branches      211      211
=======================================
  Hits         1588     1588
  Misses        164      164

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


Forgot password functionality

I checked this, and the process works up to the point when you click on the link on the email.
When the new page opens up, it does not take you anywhere and shows me this:

Should this be working now? or is it not able to due to being in the current testing URL?

Database - Dan TODO

This feature will be for therapists only as reviewers will log into accounts using an email/password combo


fix SSE connection release

If client.ReceivePayload() closed, client will be not released forever.

If client.ReceivePayload() closed, it is correct to return the response status of success or failure?
Now, gRPC returns success but SSE failure

successやろ


Double jit-compilation in loop when passing plain types

Consider the following code snippet:
import jax
import jax.numpy as np

my_fn_calls = 0

@jax.jit
def my_fn(x, y):
    # side effect to detect multiple tracing
    global my_fn_calls
    my_fn_calls += 1
    print("my_fn")

    return x + y

x = 0

for i in range(10):
    x = my_fn(x, i)

assert(my_fn_calls == 1) # fails because my_fn_calls == 2

Here, my_fn is traced twice during the loop. The reason for this is that for the first iteration, x is a python integer which jax transforms into a Traced<ShapedArray(int32[], weak_type=True)> but after the first iteration, it is of type Traced<ShapedArray(int32[])>. Due to the missing weak_type=True, these are not seen as equivalent and tracing is once more performed in the second for-loop iteration, even though types have actually not changed.
While in this example this is somewhat benign, this has happened to us in cases where the compilation my take some minutes, in which case it would be nice to only perform it once.
An obvious fix would be to ensure that no regular types are passed into jax here, but that puts an additional burden on the programmer that we would wish to avoid.
Could someone explain what the purpose of the weak_type parameter is exactly and whether it would be possible for jax to detect these cases and perform compilation only once?

Thanks so much for the crystal-clear explanation!
The weak_type parameter was added in #1709. The PR message has a description, but basically it exists to better follow NumPy-like dtype promotion behavior.
Still thinking about this...

To some extent, this is working as intended: you passed in a different type (a Python float vs a NumPy float of some fixed type) and so we recompiled. If the behavior was the same, we might avoid the recompilation, but the point of the change is that the behavior isn't always the same.

We might consider caching Jaxpr -> XLA computation, if we believe that the same program was generated in both cases.

Somehow along the way this was fixed!


[Overview] Community: Transparency/Documentation/Roadmap/Public perception/Beta

Issue
Title
Status
JIRA
Release

ISSUES: Documentation/Transparency/Roadmap

https://github.com/corona-warn-app/cwa-wishlist/issues/376  related https://github.com/corona-warn-app/cwa-app-ios/issues/1393
Publish technical specification repository
OPEN
EXPOSUREAPP-3428

https://github.com/corona-warn-app/cwa-wishlist/issues/258
Publish Development Roadmap
OPEN
EXPOSUREAPP-3643
.

https://github.com/corona-warn-app/cwa-wishlist/issues/397
Add screenshots for planned versions/features to website
OPEN
EXPOSUREAPP-5837

ISSUES: PUBLIC PERCEPTION

https://github.com/corona-warn-app/cwa-wishlist/issues/352
Discussion about public perception of the app
OPEN
EXPOSUREAPP-4916
.

https://github.com/corona-warn-app/cwa-wishlist/issues/150
Discussion about the app's efficiency
OPEN
EXPOSUREAPP-4916
.

ISSUES: F-Droid / microG / LineageOS / APK

https://github.com/corona-warn-app/cwa-wishlist/issues/57
Publish APK
OPEN
EXPOSUREAPP-2140
.

https://github.com/corona-warn-app/cwa-wishlist/issues/275
Reproducible build
OPEN
EXPOSUREAPP-2891
.

https://github.com/corona-warn-app/cwa-wishlist/issues/192
LineageOS and CWA: how to use?
OPEN
EXPOSUREAPP-2956
.

https://github.com/corona-warn-app/cwa-app-android/issues/1483
F-Droid release and reproducible builds
OPEN
EXPOSUREAPP-3447
.

https://github.com/corona-warn-app/cwa-documentation/issues/478
Link to new Corona Tracing app on F-Droid for non-Google phones
OPEN
/
.

ISSUES: BETA TESTING

https://github.com/corona-warn-app/cwa-documentation/issues/214
Is it really possible to test the apps? How to become whitelisted?
OPEN
EXPOSUREAPP-2963

https://github.com/corona-warn-app/cwa-wishlist/issues/57
Why not publish apks assets of releases on Github?
OPEN
EXPOSUREAPP-2140

https://github.com/corona-warn-app/cwa-wishlist/issues/153
Test a new version as beta in the community
OPEN
EXPOSUREAPP-2840

https://github.com/corona-warn-app/cwa-app-ios/issues/27
Test Concept
CLOSED
/

https://github.com/corona-warn-app/cwa-app-ios/issues/29
Testflight?
CLOSED
/

https://github.com/corona-warn-app/cwa-app-ios/issues/999
European corona gateway: beta testers
CLOSED
/

https://github.com/corona-warn-app/cwa-app-ios/issues/1329
testflight
CLOSED
/

https://github.com/corona-warn-app/cwa-documentation/issues/92
iOS TestFlight?
CLOSED
/

https://github.com/corona-warn-app/cwa-documentation/issues/141
Is it possible to be added as a testflight tester ?
CLOSED
/

https://github.com/corona-warn-app/cwa-documentation/issues/159
Beta in Testflight
CLOSED
/

https://github.com/corona-warn-app/cwa-wishlist/issues/276
iOS Testflight Invites
CLOSED
/

Internal Tracking ID EXPOSUREAPP-5775

Oh, okay I see. Thanks for the clarification. Have a nice evening.


How to get a list of the connected clients?

How to get a list of connected clients?
I want to get list of the connected client in server m earlier i was able to get it using GetClientSessionsStatus now it is not available

In the new version these are GetSessionStatusAsync and GetClientStatusAsync.


Please invite me to the Recode-Hive GitHub Community Organization

Name
Indrajith Bandara
Discord Username (if applicable)
No response
Additional Context
<EMAIL_ADDRESS>

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people


Uncaught TypeError: Cannot set property 'f7View' of undefined

Hello, I know someone has made a similar issue but that issue thread had no helpful information.
I get this in the console of Google Chrome
Uncaught TypeError: Cannot set property 'f7View' of undefined framework7.js:247
My App's JS
// Init App
var myApp = new Framework7({
    modalTitle: 'My Cool App',
    // Enable Material theme
    material: true,
});

// Expose Internal DOM library
var $$ = Dom7;

// Add main view
var mainView = myApp.addView('.view-main', {
});

var view1 = myApp.addView('#view-1');
var view2 = myApp.addView('#view-2');
var view3 = myApp.addView('#view-3');

// Add another view, which is in right panel
var rightView = myApp.addView('.view-right', {
});

If anyone has a solution then please share below

a: it would be helpful to know the line on which the error occurred
b: does the elements exist? looking at this you should have a
,
,
,
 &

a: The console says framework7.js on line 247.
b: I actually fixed the problem with changing views in the tabs but I still get the error in the console.
I actually have
 and
 in one line of code so something like this
. Should I remove view-right from the javascript if I am not using it. I have view-2 implemented but not view-3 yet

If .view-right & #view-3 don't exist you shouldn't be trying to initiate them. Also .view-main is being init'd twice, once as .view-main then again as #view-3.
I recommend you take a look at the documentation and iOS sample (http://framework7.io/kitchen-sink-ios/) for further information on using f7.
Finally if your problem is solved please close this issue because this is not a problem with the framework.

@Neville2312 I don't know if you figured this out, but in my case, it was simply that I was including the Framework7 javascript before the DOM.  I'm using Rails--in my main layout, just by including the javascript at the end (just before the closing body tag), it solved the problem.  I spent about four hours puzzling this out.  sigh.

hey I had the same issue with the
var mainView = myApp.addView('.view-main', {
});
and what solved my problem is that I just added the CSS class view-main in my index.html file at the i


06/25/2018 22:10 : Create Pull Request for branch master -> prTesting1_2018062522

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

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.


Git operations fail due to empty VSCODE_GIT_ASKPASS_EXTRA_ARGS

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

VS Code Version: 1.62.2
OS Version: Debian 12

Steps to Reproduce:

Run env | grep VSCODE_GIT_ASKPASS_EXTRA_ARGS returns VSCODE_GIT_ASKPASS_EXTRA_ARGS=
Move to a git repository and run git fetch returns: remote: HTTP Basic: Access denied
Run export VSCODE_GIT_ASKPASS_EXTRA_ARGS="--ms-enable-electron-run-as-node"
Running git fetch then works.

VSCODE_GIT_ASKPASS_EXTRA_ARGS should be set to --ms-enable-electron-run-as-node.
Output of process.versions in debug console:
ares: "1.16.1"
brotli: "1.0.9"
chrome: "91.0.4472.164"
electron: "13.5.2"
icu: "68.1"
llhttp: "2.1.3"
microsoft-build: "10663358"
modules: "89"
napi: "7"
nghttp2: "1.41.0"
node: "14.16.0"
openssl: "1.1.1"
unicode: "13.0"
uv: "1.40.0"
v8: "9.1.269.39-electron.0"
zlib: "1.2.11"

The same problem is also present in 1.62.1. 1.62.0 is the last working version.

Facing the same issue. Had to revert to 1.62.0 which fixed the issue. I'm using Xubuntu 20.04.3.

@lszomoru I ran into the same issue on version 1.62.2, using Ubuntu 20.04 LTS. Setting the envvar as above fixed it

This issue should be fixed in 1.62.2. Can you please confirm that you are able to reproduce the issue with the latest version?

I can confirm it happens with 1.62.2 and 1.62.1 and all extensions disabled as well.

I have the same issue happens with the latest version 1.62.2

I am unable to repro with snap version of vscode following the steps at https://github.com/microsoft/vscode/issues/137183#issue-1053060480. Can anyone please share a screen capture with exact steps to repro the issue ? Thanks!
Also it would be great if you can start with NODE_DEBUG="child_process" code --verbose and attach the log when the failure happens.

I am using the deb package https://packages.microsoft.com/repos/vscode stable/main amd64 code amd64 1.62.2-1636665017.

code.log

The problem is not limited to the terminal, builtin git operations fail as well.

Thanks for the steps, we were able to narrow down the root cause. I will ping here again once the fix is rolled into our insiders for testing.

Thanks @deepak1556 and @tpambor
Just added "git.terminalAuthentication": false to .vscode/settings.json which allows auth creds to be passed via terminal.
Once the fix is released, "git.terminalAuthentication" in settings.json to true which is the default value or even removed to avoid confusion.

facing the same issue in version 1.62.2 Setting the envvar as above fixed it. thanks for this

The same problem is present in INSIDER build also

Closing for verification.

Fix is now available with latest insiders Version: 1.63.0-insider (d450b04c783dfa6f83dc439017767acccd4e9fac) and will be shipped with recovery release 1.62.3 this week. Please give it a try and sorry for the breakage!

I can confirm that the insider build is working now. Thanks @deepak1556 and others involved!

3. export VSCODE_GIT_ASKPASS_EXTRA_ARGS="--ms-enable-electron-run-as-node"

me devuelve este error
help me
Missing or invalid credentials.
Error: connect ECONNREFUSED /run/user/1000/vscode-git-c0839b2070.sock
at PipeConnectWrap.afterConnect [as oncomplete] (net.js:1146:16) {
errno: -111,
code: 'ECONNREFUSED',
syscall: 'connect',
address: '/run/user/1000/vscode-git-c0839b2070.sock'
}
Missing or invalid credentials.
Error: connect ECONNREFUSED /run/user/1000/vscode-git-c0839b2070.sock
at PipeConnectWrap.afterConnect [as oncomplete] (net.js:1146:16) {
errno: -111,
code: 'ECONNREFUSED',
syscall: 'connect',
address: '/run/user/1000/vscode-git-c0839b2070.sock'
}
remote: No anonymous write access.
fatal: Autenticación falló para

@MichelVarela what exact version number of VS Code are you running?

My issue was closed and was pointed back to this issue, which is supposed to be fixed in 1.62.3.  However this issue is still not fixed for me in 1.62.3, Ubuntu 21.04
On any git command, eg a:  git fetch:
remote: Repository not found.
fatal: Authentication failed for 'https://github.com/some-repo.git/'

My issue was closed and was pointed back to this issue, which is supposed to be fixed in 1.62.3. However this issue is still not fixed for me in 1.62.3, Ubuntu 21.04
On any git command, eg a: git fetch:
remote: Repository not found.
fatal: Authentication failed for 'https://github.com/some-repo.git/'

Does this reply
solve anything for you?

@deepak1556 with WSL, it is still not working, tried  "git.terminalAuthentication": false as well


support for stackexchange

Because I set up a webhook in openuserjs.org, whenever there is a push code operation, the user is reminded to update. To avoid constantly reminding users to make updates, I want to collect more websites and then update the code uniformly. You can submit more websites on here, thanks.


New set_labels_on_build_config() API method (fixes #188)

A naming issue: intuitively, I'd say that these methods would reset the labels to precisely the dict that I pass, but instead they do .update(). Shouldn't they be called update_labels... instead of set_labels...?

Possibly. That would be an API change at this point, as set_labels_on_build() already existed before this PR. @TomasTomecek, what do you think?

I sort of agree with Slavek. Having set and update methods would make total sense. In reactor we use set_annotations.

Right, we use set_annotations_on_build() in atomic-reactor's store_metadata_in_osv3 plugin, but it's actually an update(). Do we want that to be a set, or an update?
I can rename the set_ methods to update_, and introduce new set_ methods, but should I do that in a separate PR to this?

AFAIK there are no annotations on the build object, so even though it's update, in the end the result is the same as in case of set. Not sure if your chain rebuild work adds some annotations (or if openshift has some native ones).

That's true, and there aren't any native ones set on Builds at the moment.
Should I do this renaming work in a separate PR, or extend this one?

I'd say there's not big need to create another PR, just add a commit here.

LGTM


CAS User Interface Enhancements

Improve the following shortcomings of the admin user interfaces:

[X] Allow the configuration screen to edit CAS properties in place. Update/Refresh and notify.
[X] Allow reporting back of the refreshed properties when done manually.
[X] Add new field for signing algorithm to management app
[x] Allow configuration of log levels.
[] Allow successful/failed authn attempts into the stats page as a chart

Working with @mindblender on this one.

Includes: https://github.com/apereo/cas/issues/1976

@mindblender how are we doing on this issue?

Reported back the following issues to @mindblender  with the current management webapp:
Service Type: should be WS Federation Client
OpenID Client type:

extra scopes: OAuth2 Client, SAML2 Service Provider, OpenID Connect Client (Should be removed)
Attribute release tab should be removed/hidden
SAML2 Service Provider:
Attribute Release Return Allowed/Mapped/PatternMatcing disabled?! They seem to be disabled for all types.
Multifactor Authentication Providers: Change into dropdown (with the ability to type). List of options are: https://apereo.github.io/cas/development/installation/Configuring-Multifactor-Authentication.html#supported-providers. Use the ID column (without Custom)
Username attribute: Anonymous: Add option for "Principal Attribute". Text field.
Access Strategy: Required attributes: unable to add/remove attributes. No buttons?

Server work is still expected to be verified.

Closing to move the conversation into the PR directly.


probelm about attention

I have a question about the code with attention,the hidden state is always zeros,and in my think,it should be a encoded hidden state of batch_H with lstm , and in you code  I can't think what function the rnn_cell do?

Hello,
This line is for initial hidden value (0), this means 'not always zero'.
https://github.com/clovaai/deep-text-recognition-benchmark/blob/master/modules/prediction.py#L33
and loop attention_cell in here.
https://github.com/clovaai/deep-text-recognition-benchmark/blob/master/modules/prediction.py#L41
and LSTMCell is in the attention_cell.
https://github.com/clovaai/deep-text-recognition-benchmark/blob/master/modules/prediction.py#L67
Best.


Feat(compiler_base_session): Add trait SessionDiagnosticWithError to support custom error return type.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

issue #350

2. What is the scope of this PR (e.g. component or file name):
compiler_base_session.

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[x] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

Add trait SessionDiagnosticWithError to support custom error return type.

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

12 of 14 (85.71%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 89.675%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

compiler_base/session/src/tests.rs
12
14
85.71%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
1381

Relevant Lines:
1540

💛  - Coveralls


[Bug Report] MessageBox default insert to body, causing MessageBox to disappear when switching routes.

Element UI version
2.3.7
OS/Browsers version
macOS10.13.4 / chrome66.0
Vue version
2.5.17-beta.0
Reproduction Link
https://github.com/ElemeFE/element/blob/dev/packages/message-box/src/main.js#L112
Steps to reproduce
https://github.com/ElemeFE/element/blob/dev/packages/message-box/src/main.js#L112
What is Expected?
能提供 MessageBox 插入位置的选项
What is actually happening?
默认是插入到body上的

You can close it in the beforeDestroy hook: https://jsfiddle.net/emr8s9n1/


fix mypy issues for events

Getting 404 after having paid for an event.

fixed d371dac8
was unrelated to mypy changes though, it was a bug

Codecov Report

Merging #1269 (d371dac) into main (efc1566) will increase coverage by 0.02%.
The diff coverage is 18.96%.

@@            Coverage Diff             @@
##             main    #1269      +/-   ##
==========================================
+ Coverage   48.82%   48.85%   +0.02%
==========================================
  Files         270      270
  Lines       15718    16630     +912
==========================================
+ Hits         7674     8124     +450
- Misses       8044     8506     +462

Impacted Files
Coverage Δ

lnbits/extensions/events/tasks.py
73.33% <0.00%> (-2.67%)
:arrow_down:

lnbits/extensions/events/views_api.py
31.18% <6.25%> (-3.60%)
:arrow_down:

lnbits/extensions/nostrnip5/views_api.py
28.80% <11.11%> (-4.25%)
:arrow_down:

lnbits/extensions/nostrnip5/tasks.py
43.47% <22.22%> (-8.91%)
:arrow_down:

lnbits/extensions/events/views.py
47.50% <100.00%> (-1.29%)
:arrow_down:

lnbits/extensions/nostrnip5/crud.py
28.81% <100.00%> (ø)

lnbits/extensions/nostrnip5/views.py
58.62% <100.00%> (-1.38%)
:arrow_down:

lnbits/db.py
81.20% <0.00%> (-10.07%)
:arrow_down:

lnbits/extensions/lndhub/utils.py
60.00% <0.00%> (-6.67%)
:arrow_down:

lnbits/commands.py
73.61% <0.00%> (-1.39%)
:arrow_down:

... and 31 more

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


how large datasets are handled under the hood

Hi
I want to use multiple large datasets with a mapping style dataloader, where they cannot fit into memory, could you tell me how you handled the datasets under the hood? is this you bring all in memory in case of mapping style ones? or is this some sharding under the hood and you bring in memory when necessary, thanks

This library uses Apache Arrow under the hood to store datasets on disk.
The advantage of Apache Arrow is that it allows to memory map the dataset. This allows to load datasets bigger than memory and with almost no RAM usage. It also offers excellent I/O speed.
For example when you access one element or one batch
from datasets import load_dataset

squad = load_dataset("squad", split="train")
first_element = squad[0]
one_batch = squad[:8]

then only this element/batch is loaded in memory, while the rest of the dataset is memory mapped.

How can we change how much data is loaded to memory with Arrow? I think that I am having some performance issue with it. When Arrow loads the data from disk it does it in multiprocess? It's almost twice slower training with arrow than in memory.

How can we change how much data is loaded to memory with Arrow? I think that I am having some performance issue with it. When Arrow loads the data from disk it does it in multiprocess? It's almost twice slower training with arrow than in memory.

How can we change how much data is loaded to memory with Arrow? I think that I am having some performance issue with it. When Arrow loads the data from disk it does it in multiprocess? It's almost twice slower training with arrow than in memory.

Loading arrow data from disk is done with memory-mapping. This allows to load huge datasets without filling your RAM.
Memory mapping is almost instantaneous and is done within one process.
Then, the speed of querying examples from the dataset is I/O bounded depending on your disk. If it's an SSD then fetching examples from the dataset will be very fast.
But since the I/O speed of an SSD is lower than the one of RAM it's expected to be slower to fetch data from disk than from memory.
Still, if you load the dataset in different processes then it can be faster but there will still be the I/O bottleneck of the disk.

EDIT:
My fault! I had not seen the dataloader_num_workers in TrainingArguments ! Now I can parallelize and go fast! Sorry, and thanks.

Ok let me know if that helps !


Pattern not visible

The pattern does not show up on the screen where we view words and the grid.
For eg :I have selected three words at the top of the grid and the rest at the bottom, but they do not appear on the screen.

Completed


Segmentation fault in simple ebpf program

Running a program based on https://github.com/foniod/redbpf/blob/main/examples/example-probes/src/tcp_lifetime/main.rs with code:
// This program can be executed by
// # cargo run --example tcp-lifetime [interface]
#![no_std]
#![no_main]
use core::mem::{self, MaybeUninit};
use memoffset::offset_of;

use redbpf_probes::socket_filter::prelude::*;

use monitor::usage::{SocketAddr,message};

#[map(link_section = "maps/perf_events")]
static mut perf_events: PerfMap<message> = PerfMap::with_max_entries(1024);
#[map(link_section = "maps/usage")]
static mut usage: HashMap<u32, u32> = HashMap::with_max_entries(4096);

#[map(link_section = "maps/time")]
static mut time: HashMap<u32, u64> = HashMap::with_max_entries(4096);

program!(0xFFFFFFFE, "GPL");
#[socket_filter]
fn measure_tcp_lifetime(skb: SkBuff) -> SkBuffResult {
    let eth_len = mem::size_of::<ethhdr>();
    let eth_proto = skb.load::<__be16>(offset_of!(ethhdr, h_proto))? as u32;
    if eth_proto != ETH_P_IP {
        return Ok(SkBuffAction::Ignore);
    }

    let ip_proto = skb.load::<__u8>(eth_len + offset_of!(iphdr, protocol))? as u32;
    if ip_proto != IPPROTO_TCP {
        return Ok(SkBuffAction::Ignore);
    }

    Ok(SkBuffAction::Ignore)
}
´´´

Gives a segmentation fault, however, used to work about a month ago don't know if it started to happen due to rust updates or redBPF updates.
Thanks in Advance.

@sebastiaoamaro Do you experience Segfault while compiling BPF programs?
Or it happens while running an executable?

@sebastiaoamaro I used to experience segfault while compiling BPF programs if versions mismatch between rust and LLVM.
The latest stable rust depends on LLVM 13. But compiling BPF programs will fail if your system has LLVM 12 libraries.

You can check your rustc uses which version of LLVM by executing this:
$ rustc -vV
rustc 1.55.0 (c8dfcfe04 2021-09-06)
binary: rustc
commit-hash: c8dfcfe046a7680554bf4eb612bad840e7631c4b
commit-date: 2021-09-06
host: x86_64-unknown-linux-gnu
release: 1.55.0
LLVM version: 12.0.1

And you can check the version of LLVM in your system by running this command:
$ llvm-config --version
12.0.1

Hello @sebastiaoamaro
How's it going with the problem?
Did you find some clue?

This is my rustc -vV command result:
rustc 1.56.1 (59eed8a2a 2021-11-01)
binary: rustc
commit-hash: 59eed8a2aac0230a8b53e89d4e99d55912ba6b35
commit-date: 2021-11-01
host: x86_64-unknown-linux-gnu
release: 1.56.1
LLVM version: 13.0.0
´´´

And my llvm-config --version:
13.0.1

I get the error:
error: Can not use LLVM12 with Rust >= 1.56
  --> /home/sebas/.cargo/registry/src/github.com-1ecc6299db9ec823/cargo-bpf-2.1.0/src/llvm.rs:26:9
   |
26 |         compile_error!("Can not use LLVM12 with Rust >= 1.56");

When running cargo install cargo-bpf.
I changed versions using these commands: https://askubuntu.com/a/1310730

I created a new ubuntu (21.10) with only llvm13, and this time I get:
error:
No suitable version of LLVM was found system-wide or pointed
to by LLVM_SYS_120_PREFIX.
   Consider using `llvmenv` to compile an appropriate copy of LLVM, and
   refer to the llvm-sys documentation for more information.

   llvm-sys: https://crates.io/crates/llvm-sys
   llvmenv: https://crates.io/crates/llvmenv

--> /home/sebasamaro/.cargo/registry/src/github.com-1ecc6299db9ec823/llvm-sys-120.2.1/src/lib.rs:486:1
|
486 | / std::compile_error!(concat!(
487 | |     "No suitable version of LLVM was found system-wide or pointed
488 | |        to by LLVM_SYS_",
489 | |     env!("CARGO_PKG_VERSION_MAJOR"),
...   |
496 | |        llvmenv: https://crates.io/crates/llvmenv"
497 | | ));
When running llvm-13-config:
I have 13.0.1
I also tried installing with llvmenv, however their build is not working.

@sebastiaoamaro
There were two options you could choose.

Downgrade rust to 1.55 that depends on LLVM 12 and LLVM 12 installed in system
Use rust 1.56 that depends on LLVM 13, and LLVM 13 installed in system

It seems that you chose option no. 2
In this situation you can pass llvm13 feature, to tell redbpf to use it instead of llvm12.
$ cargo build --no-default-features --features=llvm13

Currently, RedBPF uses LLVM12 as a default LLVM version. But you can change the version by the command above. Or you can specify the feature through Cargo.toml if you use redbpf as a dependency.
Please refer to valid combination of rust and llvm versions for more information.
If you have any problem please let me know.

I understand that step when building bpf programs, but I am still stuck on installing (cargo install cargo-bpf).

If i install running cargo install cargo-bpf --no-default-features --features=llvm13 then I get:
error[E0433]: failed to resolve: use of undeclared crate or module bpf_sys
--> /home/sebasamaro/.cargo/registry/src/github.com-1ecc6299db9ec823/cargo-bpf-2.1.0/src/build_constants.rs:1:5
|
1 | use bpf_sys::headers::prefix_kernel_headers;
|     ^^^^^^^ use of undeclared crate or module bpf_sys
error[E0425]: cannot find function prefix_kernel_headers in this scope
--> /home/sebasamaro/.cargo/registry/src/github.com-1ecc6299db9ec823/cargo-bpf-2.1.0/src/build_constants.rs:64:5
|
64 |     prefix_kernel_headers(&KERNEL_HEADERS).ok_or(())
|     ^^^^^^^^^^^^^^^^^^^^^ not found in this scope
Because some default features need to be installed.

@sebastiaoamaro
Thank you for filing your issue.
You can build cargo-bpf cargo sub-command with LLVM13 like this:
$ cargo install cargo-bpf --no-default-features --features=llvm13,command-line

Please, see cargo-bpf/Cargo.toml to understand why I passed the --features=llvm13,command-line.
Thank you

It makes sense yes, maybe changing the tutorial so that other users don't get the same error would be good.
It also fixed the segmentation fault. Thanks so much for the help.

@sebastiaoamaro
Good idea. It would be nice to state rust and LLVM version issue in the tutorial.
And I didn't realize that installing cargo-bpf may fail because of LLVM version mismatch before you mentioned. So it would be also helpful to mention how to build cargo-bpf properly.
Thank you

@sebastiaoamaro
As you suggested, I wrote about LLVM version issue into the tutorial.
See #217 for more information.
Thanks

@rhdxmr Thank you so much for your comments. I've been trying to install this for a while. Did not find anyway around. I was stuck with this error.
Now it's installed successfully!


CameraRollAndroid is undefined

I'm new to React-native and it appears that this line "const CameraRollAndroid = React.NativeModules.CameraRollAndroid;" is not working,
When I console.log ("CameraRollAndroid is " , CameraRollAndroid);
CameraRollAndroid is undefined
I've done "npm install rn-camera-roll", and "react-native link rn-camera-roll"
My configuration:
react-native --version
react-native-cli: 0.1.10
react-native: 0.21.0
Any help in this matter would be great.
Thanks!

Hi @NTDNguyen123,
First, CameraRoll should be available for Android in react-native 0.19 and above, so I would suggest using that instead :)
If you still want to make this module work, can you check that the CameraRollPackage has been added to your list of packages ?
In your MainActivity, you should see something like
protected List<ReactPackage> getPackages() {
   return Arrays.<ReactPackage>asList(
     new MainReactPackage(),
     new CameraRollPackage()
   );
}

If you don't see it, then add it and also add
import fr.bamlab.rncameraroll.CameraRollPackage;

at the top of the MainActivity file

Hi @NTDNguyen123, I'm closing this since there has been no activity for 15 days.
Feel free to reopen if the problem persists! :)


Attempt to fix master

Oh, apparently this was done in https://github.com/paritytech/substrate/pull/3357 :man_shrugging:


[CPT-1479] Enable select clear button for the qb

CPT-1479
Description
Describe the changes and motivations for the pull request.
How to test

Temploy
Open Query builder story
On the Default story select Also plays rule
Check if enableReset property is working as expected

Screenshots

Before.
After.

Development checks

[x] Add changeset according to guidelines (if needed)
[x] Read CONTRIBUTING.md and Component API principles
[x] Make sure that additions and changes on the design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[x] Annotate all props in component with documentation
[x] Create examples for component
[x] Ensure that deployed demo has expected results and good examples
[x] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[x] Covered with tests (visual tests included)

Breaking change

n/a codemod is created and showcased in the changeset
[x] test alpha package of Picasso in StaffPortal

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-bot run package:alpha-release

@toptal-anvil ping reviewers

The behavior works correctly. Good job. However, do we have a ticket where we will introduce this in the documentation? I need help finding the property listed somewhere.

The behavior works correctly. Good job. However, do we have a ticket where we will introduce this in the documentation? I need help finding the property listed somewhere.

@toptalwadiibasmi The property is provided from Picasso Select and it is listed there,
we were following the same approach for enableResetSearch prop - example for it is stored in Picasso Select
:thinking:
https://picasso.toptal.net/?path=/story/forms-select--select

The behavior works correctly. Good job. However, do we have a ticket where we will introduce this in the documentation? I need help finding the property listed somewhere.

@toptalwadiibasmi The property is provided from Picasso Select and it is listed there, we were following the same approach for enableResetSearch prop - example for it is stored in Picasso Select 🤔 https://picasso.toptal.net/?path=/story/forms-select--select

Okey! I thought we would also have to update the field property documentation, but apparently, it wasn't typed in the documentation in the first place. I will approve as the code looks good and the test passes.

QA tested, and almost everything works as expected 🥸 in exception to the following scenarios:

[ ] Assuming enableReset property is provided for this filter, the filter should have the ability to clear the choices and reset the field to the default state (only the first option is selected)

It does not reset to the first option ❌

[ ] When only the first option is selected, you may not reset (the button is not present or it does nothing)
- The button is still present and I can still clear the option ❌

Also this scenario is not clear enough to me:

[ ] The field is selected regardless of the number of option selected (check with 1 and more than 10)
- When should this field be selected and why?

CC @mkasperowicz

@karlavdelgadof
We synced with Rasit and established that it is not a reset functionality but actually clearing functionality so indeed the implementation did not match scenarios. So that answers to your first 2 questions
When it comes to this

The field is selected regardless of the number of option selected (check with 1 and more than 10)
It's a typo. I was thinking too hard about selecting. 😂 The word should be cleared so The field is **cleared** regardless of the number of option selected (check with 1 and more than 10)


Climate code 1501

Added Broadlink base64 codes for Sanyo SAP-KR124EHEA air conditioner, and likely others too.

Whoops, I forgot to update the CLIMATE.md file. I'll open another pull request, please feel free to close this one.

Άψογος. Ευχαριστώ!


Links in Script Explanation do not match script

The sample script uses New-AzStorageContext and Start-AzStorageBlobCopy, but the Script Explanation section links to New-AzureStorageContext and Start-AzureStorageBlobCopy. It looks like the script was updated, but the doc links weren't.

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

ID: 07bf8ea7-91db-de43-acf1-af16543971fd
Version Independent ID: 22c2d984-0d21-b8f8-905a-dd173c171f86
Content: Export/Copy snapshot as VHD to a storage account in different region - PowerShell Sample
Content Source: articles/virtual-machines/scripts/virtual-machines-windows-powershell-sample-copy-snapshot-to-storage-account.md
Service: virtual-machines-windows
GitHub Login: @ramankumarlive
Microsoft Alias: ramankum

@jpduffield Thank you for your feedback! We will review and update as appropriate.

Thanks for bringing this to our attention. I have created a PR with the update, changes will be live shortly.
We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


7h randomly fails to detect compression type.

This is likely a cause of random reports of download failure.
I was able to reproduce the fail to install message, when this occurs 7h has to be restarted for another attempt to be made.
Extract of 7h log below, the file downloaded fine after restarting 7h and trying again.
19:20:12|INFO|SeventhHeavenUI.ViewModels.MainWindowViewModel|Error installing Spell Textures - The mod is queued for install. 19:20:12|ERROR|SeventhHeavenUI.ViewModels.MainWindowViewModel|System.InvalidOperationException: Cannot determine compressed stream type. Supported Archive Formats: Zip, GZip, Tar, Rar, 7Zip, LZip    at SharpCompress.Archives.ArchiveFactory.Open(Stream stream, ReaderOptions readerOptions)    at Iros._7th.Workshop.Install.InstallModProcedure.ProcessDownload(Object state)

The mod file isn't corrupt? That's my first guess mod archive failed to finish downloading resulting in a corrupt file. Maybe should offer to redownload it? If that isn't it maybe the file isn't being loaded correctly.

I will check if a corrupt file exists if I see it happen again, the error did appear immediately when I seen it.

I will check if a corrupt file exists if I see it happen again, the error did appear immediately when I seen it.

Is there a reliable way to check? does the catalogue store a hash or file size for the download we can use to verify?

It stores the file size but it needs to be manually entered by the catalog maintainer.
I wouldnt be against hash checking.
At the time I was using the latest catalog url, and sadly I cannot remember which mod caused the compression error.

There has been fixes added to fix download problems, so closing this, will reopen if proves to be a problem in the future.