Casting a firefox window from a link cases a blur border to appear

Regression?
Yes
System Info and Version
 System/Version info
Hyprland, built from branch  at commit 9e781040d9067c2711ec2e9f5b47b76ef70762b3  (props: bump version to 0.41.1).
Date: Thu Jun 13 09:54:06 2024
Tag: v0.41.1, commits: 4818

flags: (if any)

System Information:
System name: Linux
Node name: archlinux
Release: 6.9.0-1-mainline
Version: #1 SMP PREEMPT_DYNAMIC Thu, 20 Jun 2024 01:02:00 +0000

GPU information:
08:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Navi 24 [Radeon RX 6400/6500 XT/6500M] [1002:743f] (rev c7) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

======Config-Start======

Config File: /home/mparagames/.config/hypr/hyprland.conf: Read Succeeded
# plugins
#plugin = /home/mparagames/hyprplugins/hyprbars.so
#plugin = /usr/lib/libhycov.so

plugin {
    hyprbars {
        bar_color = rgba(00000066)
            bar_height = 0
            bar_text_size = 0
            bar_part_of_window = false
            extra_area = 12
            buttons {
                button_size = 0
            }
     }
    hycov {
        overview_gappo = 60 # gaps width from screen edge
            overview_gappi = 24 # gaps width from clients
            hotarea_size = 0 # hotarea size in bottom left,10x10
            enable_hotarea = 1 # enable mouse cursor hotarea
            swipe_fingers = 4 # finger number of gesture,move any directory
            move_focus_distance = 100 # distance for movefocus,only can use 3 finger to move
            enable_gesture = 0 # enable gesture
            disable_workspace_change = 0 # disable workspace change when in overview mode
            disable_spawn = 0 # disable bind exec when in overview mode
            auto_exit = 1 # enable auto exit when no client in overview
            auto_fullscreen = 0 # auto make active window maximize after exit overview
            only_active_workspace = 0 # only overview the active workspace
            only_active_monitor = 0 # only overview the active monitor
            enable_alt_release_exit = 1 # alt swith mode,see readme for detail
            alt_toggle_auto_next = 0 # auto focus next window when toggle overview in alt swith mode
            click_in_cursor = 1 # when click to jump,the target window is find by cursor, not the current foucus window.
 }

}

# Please note not all available settings / options are set here.
# For a full list, see the wiki

# rules

windowrulev2 = stayfocused, class:^(steam_app.*)$
windowrulev2 = bordersize 0, class:^(steam_app.*)$ # set no border if it is a steam game
windowrulev2 = rounding 0, class:^(steam_app.*)$
windowrulev2 = float, class:(mpv)
windowrulev2 = maximize, class:(mpv)
windowrulev2 = fullscreen, class:(Waydroid)
windowrulev2 = bordercolor rgba(1155ffee) rgba(905affee) 45deg, fullscreen:1
windowrulev2 = forceinput, class:^(steam_app.*)$
windowrulev2 = windowdance,class:^(steam_app.*)$
windowrulev2 = idleinhibit always,class:^(steam_app.*)$
windowrulev2 = float,class:blueman
windowrulev2 = move 69% 5%,class: blueman
windowrulev2 = size 30% 45%,class: blueman
windowrulev2 = pin, class:blueman
windowrulev2 = float,class:nm-connection
windowrulev2 = pin, class: nm-connection
windowrulev2 = float,class:org.kde.polkit-kde-authentication-agent
windowrulev2 = pin,class:org.kde.polkit-kde-authentication-agent
windowrulev2 = float,class:^(org.kde.dolphin),title:^(Copying)(.*)
windowrulev2 = float,class:^(org.kde.dolphin),title:^(Compressing)(.*)
windowrulev2 = float,class:^(org.kde.dolphin),title:^(Moving)(.*)
windowrulev2 = float,class:^(org.kde.dolphin),title:^(Deleting)(.*)
windowrulev2 = move 69% 5%,class: nm-connection
windowrulev2 = size 30% 45%,class: nm-connection
windowrulev2 = pin,class: nm-connection
windowrulev2 = float,class:pavucontrol
windowrulev2 = pin, class:pavucontrol
windowrulev2 = move 69% 5%,class: pavucontrol
windowrulev2 = size 30% 45%,class: pavucontrol
#windowrulev2 = size 40% 30%,class: org.kde.konsole
#windowrulev2 = maxsize 30% 40%,class:org.kde.konsole
windowrulev2 = float,class:orage
windowrulev2 = move 35% 5%,class: orage
windowrulev2 = size 30% 200,class: orage
windowrulev2 = float, class:file-roller
windowrulev2 = size 30% 45%,class: file-roller
windowrulev2 = center, class: file-roller
windowrulev2 = float, class: transmission
windowrulev2 = pin, class: Rofi
windowrulev2 = float, class: Rofi
windowrulev2 = bordersize 0, class: Rofi

# windowrulev2 = noshadow, fullscreen:1
# windowrulev2 = rounding 0, fullscreen:1
# windowrulev2 = gapsout 0, fullscreen:1

                                                                                layerrule = ignorezero, gtk-layer-shell
                                                                                layerrule = ignorezero, waybar
                                                                                layerrule = blur, swaync-notification-window
                                                                                layerrule = ignorezero, swaync-notification-window
                                                                                layerrule = blur, swaync-control-center
                                                                                layerrule = ignorezero, swaync-control-center
                                                                                layerrule=blur,gtk-layer-shell
                                                                                layerrule=blur,waybar
                                                                                layerrule = blur,rofi

# See https://wiki.hyprland.org/Configuring/Monitors/

                                                                                monitor=,preferred,auto,1

# See https://wiki.hyprland.org/Configuring/Keywords/ for more

# Execute your favorite apps at launch

                                                                                exec-once = waybar
# exec-once = wpaperd
                                                                                exec-once = wlsunset -l -12.9704 -L -38.5124 -t 4500 -T 6600
                                                                                exec-once = swaync
                                                                                exec-once = hyprctl setcursor "LyraP-cursors" 24
                                                                                exec-once = wl-clipboard-history -t
                                                                                exec-once = wl-paste --watch cliphist store
# exec-once = nwg-dock-hyprland -p left -ml 10 -l top -hd 0 -d -ico "/home/mparagames/Pictures/SFW/Misc/Strawberry_gray.png" -c "rofi -show drun"
                                                                                exec-once = /usr/lib/polkit-kde-authentication-agent-1
                                                                                exec-once = swww init && /home/mparagames/swww_randomize.sh "/home/mparagames/Pictures/SFW/Wallpapers" || /home/mparagames/swww_randomize.sh "/home/mparagames/Pictures/SFW/Wallpapers"
                                                                                exec-once  = nm-applet --indicator && disown
                                                                                exec-once = sudo /home/mparagames/overclock.sh
                                                                                exec-once = /bin/bash /home/mparagames/scripts/checkactive.sh
                                                                                exec-once = /home/mparagames/scripts/autosuspend.sh
exec-once = env XDG_MENU_PREFIX=arch- kbuildsycoca6

# Source a file (multi-file configs)
# source = ~/.config/hypr/myColors.conf

# Some default env vars.
                                                                                env = XCURSOR_SIZE,24

# For all categories, see https://wiki.hyprland.org/Configuring/Variables/
                                                                                input {
                                                                                kb_layout = br, us
                                                                                kb_variant =
                                                                                kb_model =
                                                                                kb_options =
                                                                                kb_rules =

                                                                                follow_mouse = 1

                                                                                touchpad {
                                                                                natural_scroll = no
                                                                                }
                                                                                accel_profile = adaptative
sensitivity = 0.4 # -1.0 - 1.0, 0 means no modification.
                                                                                }

general {
# See https://wiki.hyprland.org/Configuring/Variables/ for more

    gaps_in = 5
        gaps_out = 10
        border_size = 2
        col.active_border = rgba(33bbffee) rgba(ac80f4ee) 45deg
        col.inactive_border = rgba(595959aa)

        layout = dwindle
        resize_on_border = true
}

decoration {
# See https://wiki.hyprland.org/Configuring/Variables/ for more

    rounding = 10

        blur {
            enabled = true
                size = 2
                passes = 3
                brightness = 0.75
                popups = true
                popups_ignorealpha = 0.26
        }

    drop_shadow = yes
        shadow_offset = 0 0
        shadow_range = 20
        shadow_render_power = 3
        col.shadow = rgba(1a1a1add)
}

animations {
    enabled = yes

# Some default animations, see https://wiki.hyprland.org/Configuring/Animations/ for more

        bezier = myBezier, 0.05, 0.8, 0.1, 1.05
        bezier = easeOut, 0.23, 1, 0.32, 1.05
        bezier = easeOutCubic, 0.215, 0.61, 0.355, 1.05
        bezier = easeIn, 0.42, 0, 1, 1.05
        bezier = easeInOut, 0.42, 0, 0.58, 1.05
        bezier = linear, 1, 1, 1, 1

        animation = windows, 1, 1, myBezier, popin 100%
        animation = windowsIn, 1, 3, easeOut, popin 0%
        animation = windowsOut, 1, 5, easeOut, popin 50%
        animation = windowsMove, 1, 7, myBezier
        animation = border, 1, 10, easeOut
        animation = borderangle, 1, 8, easeOut
        animation = fadeIn, 1, 2, easeIn
        animation = fadeOut, 1, 7, easeOut
        animation = workspaces, 1, 5, easeOutCubic
}

dwindle {
# See https://wiki.hyprland.org/Configuring/Dwindle-Layout/ for more
    pseudotile = yes # master switch for pseudotiling. Enabling is bound to mainMod + P in the keybinds section below
    preserve_split = yes # you probably want this
}

misc {

    focus_on_activate = true
        animate_manual_resizes=true

}

master {
# See https://wiki.hyprland.org/Configuring/Master-Layout/ for more
    new_is_master = true
}

gestures {
# See https://wiki.hyprland.org/Configuring/Variables/ for more
    workspace_swipe = off
}

# Example per-device config
# See https://wiki.hyprland.org/Configuring/Keywords/#executing for more
#device:epic-mouse-v1 {
#           sensitivity = -0.5
#       }

# Example windowrule v1
# windowrule = float, ^(kitty)$
# Example windowrule v2
# windowrulev2 = float,class:^(kitty)$,title:^(kitty)$
# See https://wiki.hyprland.org/Configuring/Window-Rules/ for more

#pass keybinds

bind = ,F1,pass,^(com\.obsproject\.Studio)$
bind = ,F3,pass,^(com\.obsproject\.Studio)$
bind = ,PAUSE,pass,^(discord)$

# See https://wiki.hyprland.org/Configuring/Keywords/ for more
$mainMod = SUPER

# bind = $mainMod,tab,hycov:toggleoverview
bind = R_CTRL, semicolon, exec, wtype /
bind = R_ALT, semicolon, exec, wtype /
bind = R_CTRL, period, exec, wtype '|'
bind = R_CTRL, comma, exec, wtype '\'
bind = , mouse:276, exec, wtype -M ctrl c -m ctrl
bind = , mouse:275, exec, wtype -M ctrl v -m ctrl

bind = $mainMod, R, exec, rofi -show drun
bind = $mainMod, W, exec, rofi -show window
bind = , PRINT, exec, bash ~/.config/hypr/screenshot.sh area
bind = R_CTRL, PRINT, exec, bash ~/.config/hypr/screenshot.sh output

# Example binds, see https://wiki.hyprland.org/Configuring/Binds/ for more
bind = $mainMod, T, exec, konsole
bind = $mainMod, D, exec, dolphin
bind = ALT, F4, killactive,
     bind = $mainMod, E, exec, dolphin
     bind = $mainMod, V, togglefloating,
     bind = , mouse:274, togglefloating,
# bind = $mainMod, R, exec, wofi --show drun
     bind = $mainMod, P, pseudo, # dwindle
     bind = $mainMod, J, togglesplit, # dwindle

# Move focus with mainMod + arrow keys
     bind = $mainMod, left, movefocus, l
     bind = $mainMod, right, movefocus, r
     bind = $mainMod, up, movefocus, u
     bind = $mainMod, down, movefocus, d
     bind = $mainMod, A, fullscreen, 1
     bind = $mainMod, F, fullscreen, 0
     bind = $mainMod, Tab, cyclenext,
     bind = $mainMod, Tab, bringactivetotop,

# Switch workspaces with mainMod + [0-9]
     bind = $mainMod, 1, workspace, 1
     bind = $mainMod, 2, workspace, 2
     bind = $mainMod, 3, workspace, 3
     bind = $mainMod, 4, workspace, 4
     bind = $mainMod, 5, workspace, 5
     bind = $mainMod, 6, workspace, 6
     bind = $mainMod, 7, workspace, 7
     bind = $mainMod, 8, workspace, 8
     bind = $mainMod, 9, workspace, 9
     bind = $mainMod, 0, workspace, 10

# Move active window to a workspace with mainMod + SHIFT + [0-9]
     bind = $mainMod SHIFT, 1, movetoworkspace, 1
     bind = $mainMod SHIFT, 2, movetoworkspace, 2
     bind = $mainMod SHIFT, 3, movetoworkspace, 3
     bind = $mainMod SHIFT, 4, movetoworkspace, 4
     bind = $mainMod SHIFT, 5, movetoworkspace, 5
     bind = $mainMod SHIFT, 6, movetoworkspace, 6
     bind = $mainMod SHIFT, 7, movetoworkspace, 7
     bind = $mainMod SHIFT, 8, movetoworkspace, 8
     bind = $mainMod SHIFT, 9, movetoworkspace, 9
     bind = $mainMod SHIFT, 0, movetoworkspace, 10

# Scroll through existing workspaces with alt + tal
     bind = ALT, Tab, workspace, e+1
     bind = $mainMod, mouse_up, workspace, e-1

# Move/resize windows with mainMod + LMB/RMB and dragging

     bindm = $mainMod, mouse:272, movewindow
     bindm = $mainMod, mouse:273, resizewindow

======Config-End========

Description
When opening an image from Discord's "Open in Browser" link using a Browser like Firefox, a blur border will appear.
Used to work normally in 0.36 (without showing this border), but no longer works in 0.41. Haven't tested in any versions in between.
How to reproduce

Install Firefox;

Install Discord;

Open an image on Discord and click on "Open in browser", or open any hyperlink from Discord

Firefox has to be your default browser and should be closed before you open the hyperlink.

Crash reports, logs, images, videos
How it looks like:

Are you using custom userChrome.css in Firefox?

Are you using custom userChrome.css in Firefox?

Yes, because there was a bug in firefox where it would overlap default window buttons with custom gtk theme buittons, and using userChrome.css was the only way to prevent this back then. I haven't checked yet if they fixed this problem.
You can see more about this issue and how userChrome.css modifications solve this in here:
https://new.reddit.com/r/FirefoxCSS/comments/1c6cm7e/new_firefox_update_broke_window_control_buttons/
It was the only modification i did

Is your discord running with native wayland backend?

I have a similar issue with xdg-file-picker for example, im on mobile right now, but can elaborate in detail on this later, including logs, Screenshots etc.
One thing i handle differently in xdg-windows is a windowrule Like this (from hyprland.nix config in my home-manager)
windowrulev2 = [
[...]
"float,class:^(xdg-desktop-portal-gtk)"
];
Its not such a big issue for me since the window is fully functional, only the border seems a little off, just cosmetical ^^


suggested template adjustments

A number of suggestions

remove "enable telemetry" from the default example, it is already set by default.
add the required AVM interfaces for things like private endpoints, roles, locks etc.

This also includes some the previous suggestion regarding making the location an optional param.
I note that the line endings in the template are CRLF, when I run make docs & make fmt this makes for a messy commit, maybe it should be LF in the template tho?
more as a discussion point - I'm finding I'm repeating the same work each time i start... 🙂

I've taken the liberty of running make fmt && make docs in WSL2 with the last commit, that changes the line encoding to LF.  Would be nice to have this as a precommit that is more cross platform?

I've taken the liberty of running make fmt && make docs in WSL2 with the last commit, that changes the line encoding to LF. Would be nice to have this as a precommit that is more cross platform?

Thanks @kewalaka
I only use WSL or MacOS - I'll look to add a .gitattributes for the Windows folk
# Set the default behavior, in case people don't have core.autocrlf set.
* text=lf


FIX: Renaming the read PV that was shadowing the read method.

Attn. @tacaswell !

Ha. Good catch.
I briefly considered, "Maybe we should ensure, at the metaclass level, that no component names can shadow a required part of the BlueskyInterface." I'm leaning against that... my bias is toward allowing users to shoot themselves in the foot if they want to... thoughts?

Current coverage is 84.43% (diff: 100%)

Merging #360 into master will not change coverage

@@             master       #360   diff @@
==========================================
  Files            30         30
  Lines          4554       4554
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           3845       3845
  Misses          709        709
  Partials          0          0

Powered by Codecov. Last update a681e57...f77008b

Merging as is for the bug fix, but let discussion continue on my last point.

@danielballan we... er, you, already do that for Device, but not BlueskyInterface: https://github.com/NSLS-II/ophyd/blob/master/ophyd/device.py#L330
Might be worthwhile to consider extending that... I get confused when I shoot myself in the foot.

@danielballan I agree with @klauer... This case was simple but there can be some time wasted on debug of an issue that could be solved with a nice error message!
@klauer Welcome back! 😜

Yeah, fair. I should have gone with my first impulse. Even I had to look up the bluesky interface in the docs to remember it all. See #361.


RT Camera may work incorrectly if two RT Screens are connected to it.

It seems that, if RT Camera is connected to two monitors, of which only one is enabled,
one monitor can break another depending on the clientside :Think order.
I can't right now check if this issue actually exists and fix it, but it seems to exist, according to the code.
I've personally seen a bug when monitor breaks, but I'm not sure if it related to this issue.
https://github.com/wiremod/wire/blob/4038ae0a5cd2335d6af52d6baafb6fbc70d74e21/lua/entities/gmod_wire_rt_screen.lua#L209-L226
To fix this, cameras clientside should store set of entities (for future compatibility, not necessary screens) that are observing it and :AddObserver(ent) and :RemoveObserver(ent) functions.

:AddObserver should call :SetIsObserved(true) (always, as after :AddObserver called, there will be >=1 observers).
:RemoveObserver should call :SetIsObserved(false), if it removed last observer.

Just checked - yeah, one monitor may disable another if in correct order


fix: #3040 - add path to Invalid model definition error

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Need to adjust swarbar's opacity

I am using the following to make swaybar disappear and only appear when I press the $mod key:
sway config:
bar {
    swaybar_command waybar
    position top
    hidden_state hide
    mode hide
    modifier Mod4
}

swaybar config:
    "ipc": true,

The only problem I have with it is the bar's opacity. I can still see what's behind it. I don't like that. I need a way to adjust swaybar's opacity.

You can easily set waybar's opacity in your style.css, please read the documentation on how you configure and style the app, if you haven't already. You may want to check and set the background-color and opacity setting on window#waybar specifically.

Thank you!  I got it to work.
Just a comment -
Fleshing out "man waybar-styles"
And being more explicit about what  mode indicators and workspace buttons are in "man 5 sway-bar" would be real helpful.
Thanks!

Fleshing out "man waybar-styles"
And being more explicit about what mode indicators and workspace buttons are in "man 5 sway-bar" would be real helpful.

It's a good comment. Pretty sure I can speak on @Alexays' behalf and welcome you to make a PR with the improvements you see fit. :slightly_smiling_face:

Given my lack of knowledge on this topic and the time I already spend on open-source projects (https://github.com/blakemcbride), it will not be possible for me to make these changes.  But thanks for all of the help!  I really, really love sway and swaybar! 🙂


investigate Killarney campgrounds

La Cloche Silhouette trail should have a bunch of campgrounds in its northeast quadrant, but seems to have none

I did a quick spot check against https://www.timandvictoria.com/blog/2019-11-11-hike-la-cloche-silhouette-trail/, and enabled camp_pitch POIs -- I think they're legitimately absent from OSM, not a bug on my end, so closing.


High gas while minting from this dapp

High gas while minting from this dapp, I tried to mint from etherscan it works perfectly.
any help please? i tried to change the gas limit via config.js, it didn't update it on metamask...

Are you using it for Ethereum Smart contract Buddy

@mirsgit  Are you using it for Ethereum Smart contract Buddy

im using it for an Ethereum Smart contract, when the contract changed to one with a whitelist it used the gas but returned estimation errors - "out of gas for the contract".
after returning to the shorter simple contract - without the whitelist list of accounts - the dapp worked again. but still how i can help the process to be "smarter" or avoid issues with the gas when using this dapp for presale minting with a Ethereum Smart contract?

@mirsgit Are you using it for Ethereum Smart contract Buddy


Need of Contributions.MD file

Need of Contributions.MD file to setup this project step-by-step.

duplication of https://github.com/DavidGolodetsky/GeekScore-2/issues/31


Experiment with light-client in docusaurus

Using light-client with docusaurus triggers a number of errors. While it might be doable to tweak webpack configuration to overcome those, it looks worth looking at tackling those individually to improve end-user experience.
What can be done

bls (crypto in blst-native); should not be used, might require conditional exports
blst (crypto, ls, path in script); same as previous
bls-eth-wasm (crypto); fixed by https://github.com/ChainSafe/bls/pull/159
eventsource (http, https) https://github.com/ChainSafe/lodestar/issues/6542

Errors
Module not found: Error: Can't resolve 'crypto' in '/Users/julien/Documents/Projects/jeluard/lodestar/docs/node_modules/bls-eth-wasm/src'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "crypto": require.resolve("crypto-browserify") }'
        - install 'crypto-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "crypto": false }

--------------------------

Module not found: Error: Can't resolve 'https' in '/Users/julien/Documents/Projects/jeluard/lodestar/docs/node_modules/eventsource/lib'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "https": require.resolve("https-browserify") }'
        - install 'https-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "https": false }

--------------------------

Module not found: Error: Can't resolve 'http' in '/Users/julien/Documents/Projects/jeluard/lodestar/docs/node_modules/eventsource/lib'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "http": require.resolve("stream-http") }'
        - install 'stream-http'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "http": false }

--------------------------

Module not found: Error: Can't resolve 'crypto' in '/Users/julien/Documents/Projects/jeluard/lodestar/node_modules/@chainsafe/blst/dist'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "crypto": require.resolve("crypto-browserify") }'
        - install 'crypto-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "crypto": false }

--------------------------

Module not found: Error: Can't resolve 'fs' in '/Users/julien/Documents/Projects/jeluard/lodestar/node_modules/@chainsafe/blst/dist/scripts'

--------------------------

Module not found: Error: Can't resolve 'path' in '/Users/julien/Documents/Projects/jeluard/lodestar/node_modules/@chainsafe/blst/dist/scripts'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "path": require.resolve("path-browserify") }'
        - install 'path-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "path": false }

--------------------------

Module not found: Error: Can't resolve 'crypto' in '/Users/julien/Documents/Projects/jeluard/lodestar/docs/node_modules/@chainsafe/bls/lib/blst-native'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "crypto": require.resolve("crypto-browserify") }'
        - install 'crypto-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "crypto": false }

Progress can be tracked here.
Currently blocked by https://github.com/facebook/docusaurus/issues/10010


Reorganize docs

Add prerequisites
Add UI section

Describe your changes :

See #2294

Type of change :

[ ] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation

Frontend Preview (Screenshots) :
For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have performed a self-review of my own.
[x] I have tagged my reviewers below.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] All new and existing tests passed.

Reviewers

@sureshms @harshach @amiorin @pmbrull @Sachin-chaurasiya @shahsank3t @shannonbradshaw

@mithmatt lets keep the UX style guide at the top level. We want to get the attention from the community


Fixed the bi-directional back-button- flow from feed to onbaording

What does this pull request aim to achieve? 💯
**It helps makes the bi-directional flow of the app consistent
Please be concise and link any related issue(s):
https://github.com/FightPandemics/FightPandemics/issues/599
🚨Before submitting this pull request🚨:
Please do NOT submit this PR if you have not done the following:

I have checked that no one else is working on similar changes.
I have read the latest README and understand the development workflow.
The name of this branch is: feature/<branch_name> (need to have cloned the repo not forked).
This branch is rebased/merged with the latest master.
There are no merge conflicts.
There are no console warnings and errors.
On the right hand side of this PR, I have linked the appropriate issue(s) under "Issues" and added a project progress under "Projects".

i don't know, why that, i didn't mak any changes n there at all, i didn't touch it,,

yeah @Gtstride if you run npm install it will generate a lock file, and in that case you shouldn't add it to git since it was made on your local machine
we only want to use the lock files generated from inside the Docker container when building, that's why we have a separate script to install dependencies inside docker


Support only minutes and seconds

Hello everyone.
I was wondering if you guys have plans to support minutes and seconds mode. It's not on material design docs but it could be useful.
For instance, check this example:
https://material-ui-pickers.dev/demo/timepicker

For now we only plan to support described features from the material design specs or use cases we need ourself. So if we come across the use case of seconds we will maybe make this or provide more access to the internal components like Analog Clock so user could more easily build their own picker

Exposing the internal components sounds good. Maybe eventually you could add a section in the README explaining how to create custom pickers.


set collection owner for collections created by CLI

Add owner to collection objects that are created with the CLI.

Fixes #100


fixed a bug and improved testing around NuGet version compare

I found the NuGet version spec too verbose and hard to read, so I tried switching to a table-driven approach. I find this to be much more readable and maintainable for this particular test.
Also while implementing this, I found another bug in the comparisons!
The bug was due to zip returning a shorter array in this case [1].zip([1, 2]). I thought it would pad out with nils as in this case [1, 2].zip([1]) but it does not.
Let me know any Ruby tips to make the style better or if you spot a test that I might have missed.

@Nishnha Great suggestion, added a few with revision numbers.


Contextual Onboarding - Skip additional trackers dialog

Task/Issue URL: https://app.asana.com/0/414709148257752/1208041208498959/f
Description:
This PR addresses:

Don't show dialog for major tracker or owned by major tracker when blocked trackers dialog has been shown previously.
Removed duplicate Wand-16 image used for Suprise Me button.
Fixed missing scheme delimters for suggested websites.

Steps to test this PR:
To test the scenarios you can add the hardcoded value  return VariantIOS(name: "mb", weight: 1, isIncluded: VariantIOS.When.always, features: [.newOnboardingIntro]) in DefaultVariantManager at line 144.
SCENARIO 1 - Site is Major Tracker

Run the new onboarding flow and wait for the first dialogue to appear on screen.
Tap on “Let’s do it”.
Tap on “Skip".
Go Back to DuckDuckGo App.
The New Tab Page should appear on the screen with a new Dax dialogue suggesting DDG searches.
Tap on one of the searches. E.g. “local weather”.
A Dax dialogue should appear informing the user about the DDG search.
Tap on the "Got it!” button.
The Dax dialogue should update and suggest to the user a list of websites to try.
Tap on one of the websites. E.g. “ebay.com”.
The Dax dialogue should now inform the user of the trackers that have been blocked. The Privacy Dashboard icon should be highlighted.
Navigate to www.facebook.com
EXPECTED RESULT: The dax dialog should dismiss and should not show the Major Tracker dialog

SCENARIO 2 - Site is Owned by Major Tracker

Run the new onboarding flow and wait for the first dialogue to appear on screen.
Tap on “Let’s do it”.
Tap on “Skip".
Go Back to DuckDuckGo App.
The New Tab Page should appear on the screen with a new Dax dialogue suggesting DDG searches.
Tap on one of the searches. E.g. “local weather”.
A Dax dialogue should appear informing the user about the DDG search.
Tap on the "Got it!” button.
The Dax dialogue should update and suggest to the user a list of websites to try.
Tap on one of the websites. E.g. “ebay.com”.
The Dax dialogue should now inform the user of the trackers that have been blocked. The Privacy Dashboard icon should be highlighted.
Navigate to whatsapp.com
EXPECTED RESULT: The dax dialog should dismiss and should not show the Owned by Major Tracker dialog

SMOKE TEST CONTEXTUAL ONBOARDING
Run Scenario 1 to 6 defined in Testing
Definition of Done (Internal Only):

[x] Does this PR satisfy our Definition of Done?

Copy Testing:

[ ] Use of correct apostrophes in new copy, ie ’ rather than '

Orientation Testing:

[ ] Portrait
[ ] Landscape

Device Testing:

[ ] iPhone SE (1st Gen)
[ ] iPhone 8
[ ] iPhone X
[ ] iPhone 14 Pro
[ ] iPad

OS Testing:

[ ] iOS 15
[ ] iOS 16
[ ] iOS 17

Theme Testing:

[ ] Light theme
[ ] Dark theme

Internal references:
Software Engineering Expectations
Technical Design Template

LGTM! just one question after it shows me the tracker dialog if I load another site it keeps showing me the fire pulsing animation…. is it expected?

Yes this is an existing behaviour

LGTM! just one question after it shows me the tracker dialog if I load another site it keeps showing me the fire pulsing animation…. is it expected?

If we don’t tap on "Got It” and visit another website the fire button should pulse. Then if we navigate again it should dismiss. I checked and work as expected but waiting for you to clarify “keeps showing".


Update vendored deps (except minio-go)

What vendoring tool is restic using? Is there a timeline on getting it ported to dep?

restic uses gb and gb vendor. Due to the different project layout used with gb, it may not so easy to switch to dep. I haven't had a look yet, is dep already done/stable?

It was announced at gophercon the other day that it is production ready and stable.
Merging it into the core toolchain has yet to happen though.

Ah, thanks for the info. I haven't seen that announcement yet.

Codecov Report

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

@@            Coverage Diff             @@
##           master    #1108      +/-   ##
==========================================
+ Coverage   52.89%   52.92%   +0.02%
==========================================
  Files         118      118
  Lines       11318    11318
==========================================
+ Hits         5987     5990       +3
+ Misses       4616     4614       -2
+ Partials      715      714       -1

Impacted Files
Coverage Δ

src/restic/backend/sftp/sftp.go
60.82% <0%> (+0.95%)
:arrow_up:

Continue to review full report at Codecov.

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


TypeHinting for public functions

Like mentioned in #75 I would prefer type hinting as solution.
Since TypeHinting doesn't allow default parameter values other than null, we have to default them manually.

If I am not mistaken, reverting this to 12645bc should pass on php5.6 as well.

I reverted this and switched back to the ternary version. But Travis CI doesn't seem to like this very much. I ran the test locally with success.
Honestly I am puzzled by the errors raised, especially on the 5.6 environment claiming to expect a Yasum\string.

Your type hint uses 'string' in the current namespace 'Yasumi', which doesn't exist of course. Also 'string' as a type was only introduced as of version 7 :( See: http://php.net/manual/en/functions.arguments.php#functions.arguments.type-declaration

Urg... didn't realized that type hinting first came for not scalar types. Sorry.
I don't know how important php5 support ist for you / your users. If you consider it valuable we can make the check "by hand" inside the functions and manually throwing exceptions.

I wanted to draft a php5 version
https://github.com/InSitu-Software/yasumi/tree/feature/typehinted5
but now all tests fail with
Yasumi\tests\Ukraine\SecondInternationalWorkersDayTest::testSecondInternationalWorkersDay with data set #9 ('1989', DateTime Object (...))
InvalidArgumentException: integer expected, string given

😅

I'll create a branch for the PHP7 development. It's something I've been planning but haven't gotten around to. Maybe better to make all your changes there. In the near future I'd like to end PHP5 support and continue to use PHP7.

PHP7 branch created :). Please go ahead and use that for any PHP7 related enhancements.

I've updated the travis configuration so it should work now.


Managed Auto Scaling Group resources do not receive all specified tags

Description
Using the example provided for EKS Managed Node Groups the actual Auto Scaling Groups which are created do not receive all tags which are defined in either the local.tags block on line 45 or the additional_tags block(s), for example on line 173
Other resources, such as instances and security groups do receive the relevant tags.
Versions

Module version [Required]:

Terraform version:

Terraform v1.3.2
on linux_amd64

Provider version(s):

+ provider registry.terraform.io/gavinbunney/kubectl v1.14.0
+ provider registry.terraform.io/hashicorp/aws v4.34.0
+ provider registry.terraform.io/hashicorp/cloudinit v2.2.0
+ provider registry.terraform.io/hashicorp/helm v2.7.1
+ provider registry.terraform.io/hashicorp/kubernetes v2.14.0
+ provider registry.terraform.io/hashicorp/local v2.2.3
+ provider registry.terraform.io/hashicorp/null v3.1.1
+ provider registry.terraform.io/hashicorp/time v0.9.0
+ provider registry.terraform.io/hashicorp/tls v3.4.0
+ provider registry.terraform.io/terraform-aws-modules/http v2.4.1

Reproduction Code [Required]
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/main/examples/node-groups/managed-node-groups/main.tf
Steps to reproduce the behavior:
Run the above example
Check ASG tags via AWS console or cli
Expected behaviour
AWS autoscaling groups receive tags as defined by
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/main/examples/node-groups/managed-node-groups/main.tf#L45
Actual behaviour
Auto Scaling Groups receive only standard EKS tags

Terminal Output Screenshot(s)

Additional context

Tags are only propagated when a custom launch template is used https://github.com/aws/containers-roadmap/issues/1541


Make Map.getScaleZoom aware of CRS

Right now getScaleZoom is using log2, whereas getZoomScale uses crs.scale().
It looks like we should edit getScaleZoom to use CRSs also. And make unit tests.
This looks like an easy task, so if there's any non-core-developer who wants to give it a shot, I'd love to see it happen :-)

I think this is a duplicate of #2990, but possibly more clearly stated :)

Yes, let's close as duplicate.


ScrollView.scrollTo method not working

🐛 Bug Report
Hi all,
I'm experiencing issues with the method "scrollTo" on the ScrollView component.
I applied the same approach on a FlatList component (using the scrollToIndex method) and it works, while any other try on scrollView fails.
Expected Behavior
Using react-navigation, I expect the list scrolls to the first item every time I return on the screen from another screen. Trying it, the component doesn't scroll to the first item.
Code Example
 class MyClass extends React.PureComponent<Props> {
   private MyRef =  React.createRef<ScrollView>();

    private scrollToTop = () => {
     this.MyRef.current &&
         this.MyRef.current.scrollTo({x: 0, y: 0, animated: false });
    }

    public render() {
       return (
          <NavigationEvents onWillFocus={this.scrollToTop} />
           <ScrollView ref={this.ServiceListRef}>
                  <ListItem>... </ListItem>
                  <ListItem>... </ListItem>
            </ScrollView>
        )};
 }

Environment
React Native Environment Info:
npmPackages:
react: 16.8.3
react-native: 0.59.3

Can you confirm the scrollToTop is being fired and that the ref exists when you do call it.
My guess is the scrollToTop is being fired before the element is initially rendered and so the dom element isn't available to do anything. One way to check is to wrap the scrollTo call in a setTimeout of 500ms and seeing if it works correctly then. If it does it mean you need to change your code logic rather than it being a bug.

@JKCooper2 thank you for your answer. Actually, I found that the problem was related to the fact that my ScrollView was placed as child of a Content tag and the scroll behaviour I experienced was related to it instead of my scrollView.
Thank you and Regards!

@valentinacala scrollTo working for you in Android?

Hi @bhavinwebdesk  , as mentioned, I solved the issue and it is worked as expected

hmm.. I am running into this after I upgraded to v0.59. 🤔

@Monte9  I am also. Any idea on how you resolved it?

I went from 0.57 to 0.60.4 and my previously working scrollTo-code stopped working.
I can confirm that my scroll function is being called with correct parameters, but nothing happens.
I also tried to set scrollEnabled={true} which correctly let me scroll manually via touch, but scrollTo still doesn't work.

@msageryd  I have been trying to have this issue looked at with no help. I created a StackOverFlowPost here https://stackoverflow.com/questions/57385037/react-natives-animated-scrollview-is-not-allowing-me-to-programmatically-scroll. I even went as far as to offer a bounty to no avail. I think this ticket needs to be reopened again.

It seems like something has changed with the timing for mounting stuff (between RN versions). I will go through my code again (in a couple of days), as I suspect that I'm using some of the soon-to-be deprecated lifecycle functions. The fact that they are now considered "unsafe" to use might be due to some timing issues (wild guess).
I got my code to work by simply putting the scrollTo in a 100ms timeout. This indicates that the scrollview is not quite mounted at the first try. I'm actually fine with the timeout for my use case, but I want to solve this anyway because I don't like hacky solutions.

@msageryd  thank you for this. It will help me finally upgrade. If you want, you can respond my to question on StackOverflow, so I can give you some bounty points.

I couldn't get this to work even with adding a timeout. Did you find a way to get this to work, @Louis345 ?

@sftgf  I cannot get it to work even with using a timeout. There is clearly an issue here.  I created a snack here that illustrates the problem. https://snack.expo.io/@louis345/scrollto-not-working. Changing the SDK version on the bottom right the same code works as intended.  I would like to reopen the issue. @sftgf  here is a StackOverflow post I created as well https://stackoverflow.com/questions/57385037/react-natives-animated-scrollview-is-not-allowing-me-to-programmatically-scroll

I also saw this issue after upgrading to 0.60.3
My code worked perfectly, now it doesn't. I made a few check like the existence of ref,...
It's definitely a bug.

@dangkhoa2708  I opened a new ticket here
https://github.com/facebook/react-native/issues/26291

I've gone through all of ScrollView component carefully. Here is my solution to scrollTo() issue.
In order to programmatically scroll, set scrollOverflowEnabled prop to true.
<ScrollView ref={scroller => { this.scroller = scroller; }} scrollToOverflowEnabled={true}>
  {this.props.children}
</ScrollView>

Then it works!!!

I've gone through all of ScrollView component carefully. Here is my solution to scrollTo() issue.
In order to programmatically scroll, set scrollOverflowEnabled prop to true.
<ScrollView ref={scroller => { this.scroller = scroller; }} scrollToOverflowEnabled={true}>
  {this.props.children}
</ScrollView>

Then it works!!!

This does not work for me

@prog1dev just work for iOS

I found a solution that works on iOS & Android!
https://stackoverflow.com/a/59294415/9883426
TL;DR I had to wrap it in a setTimeout with a 1-second delay. The UX is exactly the same as before so even though it's a hack it looks fine to the user.
setTimeout(() => {
  this.scrollView.scrollTo({ x: DEVICE_WIDTH * current_index, y: 0, animated: false });
}, 1)

This is likely caused by a mounting issue due to the deprecated componentWillMount lifecycle method in React which wasn't properly updated in ScrollView. Hope this helps!

@Monte9 Work for me ! Thanks.


[FEATURE] find all node_modules directories in Documents and stop syncing

This can already be done with find command, but might be worth documenting or adding a short cut to nocloud:
download https://github.com/alvaropinot/nocloud
//unzip
cd ~/Downloads/nocloud-master
//install it globally by running:
$ install nocloud.sh /usr/local/bin/nocloud
// move to iCloud Documents folder
cd ~/Documents
// find all node_modules and execute nocloud
find . -name "node_modules" -type d -prune -exec nocloud '{}' \;

Hey 👋 @pdkn,
Thanks for the suggestion 😊! I remember thinking about something similar, it is true that at least it will be cool to add it to the docs.
What do you think it could be the option/flag or the expected way of running nocloud if we add this feature?
I'll happily accept any PR's if you are up to that 😄


⚠ Bofrost.lu has degraded performance

In 990bb3d, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1272 ms

Resolved: Bofrost.lu performance has improved in bf10af4 after 31 minutes.


STM32 - kconfigs to determine if peripheral is available

Is your enhancement proposal related to a problem? Please describe.
While looking into implementing a counter driver for the STM32 hardware timers (see issue https://github.com/zephyrproject-rtos/zephyr/issues/35816), I was looking at the Nordic NRF driver and found it has kconfigs that can be checked such as:
config HAS_HW_NRF_TIMER0
	bool

config HAS_HW_NRF_TIMER1
	bool

config HAS_HW_NRF_TIMER2
	bool

config HAS_HW_NRF_TIMER3
	bool

config HAS_HW_NRF_TIMER4
	bool

This would be great for other platforms, in this case STM32. Each SOC series and variant could then enable the correct set of peripherals/timers. Currently, I don't think there is an easy way for to determine which timers are valid per soc in use. For my prototype implementation, I'll just hard code which timers are available to make sure the code works.
Describe the solution you'd like
a kconfig setup similar to this https://github.com/zephyrproject-rtos/zephyr/blob/main/soc/arm/nordic_nrf/Kconfig.peripherals
Describe alternatives you've considered
Hard coding each driver module to determine using SOC name but this would be hard to update for new series and variants

I forgot to mention, I tried looking for something in soc/arm/st_stm32 but I couldn't see anything there, hence why I assume there isn't a comparative method

@stoberblog We typically use dts (dts/arm/st/xx/*.dtsi) to describe which peripherals are available on which SoCs.
Can you explain why this is not sufficient ?


Fix minor typo

Child task is executed before the parent if sched_child_runs_first is set to 1, but budget branch sets it to 0.

Dracu not merging, fix rypo not emportand.


Connect Segment / Kinesis firehose to Elastic cluster

Part of #4149
Sequel to #4242
Connect newly setup Kinesis Firehose that is fed by Segment to Elastic cluster.

Done in https://github.com/wellcometrust/search-logger


Add Magic areas

#16 and #32

@DigiLive should I move this code to the area card instead?


Fixing grigori's PR

Make sure these boxes are checked before submitting this pull request~!~ Thank you!!

[] The test of the method you have fixed is passing
[] You have written a new skeleton method for someone else to work on!

Hey @grigori-gru !! I'm not sure what happened with your former pull request, but I was seeing some weirdness surrounding a duplicated commit history. I closed https://github.com/BlakeGuilloud/ganon/pull/58 and reopened it from your fork, and it appears to have resolved itself.. Not sure why, but hey! It works now!
Gonna go ahead and merge this puppy in! Thanks for the contribution man!!


OCPBUGS-33727: Fix Gateway API Feature Gate

Update the feature gate framework to reflect that Gateway API is not yet a TechPreview feature. It is only a DevPreview feature.

/jira refresh

Quite a few cluster creation errors in e2e-aws-ovn-hypershift.  For example:

=== NAME  TestUpgradeControlPlane/ValidateHostedCluster
util.go:340:
failed waiting for hostedcluster image rollout: status.version.history[0].state is "Partial", but we want "Completed"
Unexpected error:
<wait.errInterrupted>:
timed out waiting for the condition
{
cause: <*errors.errorString | 0xc00080e900>{
s: "timed out waiting for the condition",
},
}
occurred

/test e2e-aws-ovn-hypershift

From a NE perspective, this seems reasonable, it shouldn't be in the TechPreviewNoUpgrade feature set.
/lgtm

/lgtm
/label acknowledge-critical-fixes-only
As far as I'm aware, this shouldn't break anything. There are no API changes behind the gate so no operator should start crashing on us. I think it's important we have the gates in the right order prior to release, so, lets merge this.

XREF: This corrects https://github.com/openshift/api/pull/1452


sudo apt-get install -y ros-humble-isaac-ros-apriltag

sudo apt-get install -y ros-humble-isaac-ros-apriltag
[sudo] password for sim:
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
E: Unable to locate package ros-humble-isaac-ros-apriltag

Could you please confirm the following:

Are you running this command inside the Isaac ROS Dev Docker container? The Dockerfile we provide as part of Isaac ROS Common automatically adds the Isaac ROS apt repository to the sources list, which is a prerequisite to being able to download the Debian packages.
Are you using the latest version of Isaac ROS? Isaac ROS Debian packages were initially made available with the Isaac ROS 2.1 release
Does your device have a stable connection to the internet? To test if your device is able to fetch the latest sources, please run:

sudo apt-get update


feat: vmodel helper for two way binding

Proposal for having a vmodel function to explicitly wrap vmodel value - @nekosaur, @KaelWD, @nandi95 and cexbrayat
const wrapper = mount(component, {
  props: {
    modelValue: vmodel(1, jest.fn()) // now modelValue will be reactive
  }
})

vmodel
vmodel is a new function which will return a new vmodel object
export interface VModel<T> {
  value: Ref<UnwrapRef<T>>
  onChange?(v: T): void
}

Test:
  it('should return the updated props on 2 way binding', async () => {
    const component = defineComponent({
      template: '<button @click="increment"></button>',
      props: {
        modelValue: {
          type: Number,
          required: true
        }
      },
      emits: ['update:modelValue'],
      setup(props, ctx) {
        return {
          increment: () => ctx.emit('update:modelValue', props.modelValue + 1)
        }
      }
    })

    // @ts-expect-error TODO fix the type
    const wrapper = mount(component, {
      props: {
        modelValue: vmodel(1)
        // Not needed
        // 'onUpdate:modelValue': async (modelValue: number) =>
        //   wrapper.setProps({ modelValue })
      }
    })

    expect(wrapper.props('modelValue')).toBe(1)
    await wrapper.trigger('click')
    expect(wrapper.props('modelValue')).toBe(2)
  })

Other options
Implementation with a new vmodel option : https://github.com/vuejs/vue-test-utils-next/pull/525
Note
Types are not working with this, is this API is accepted I can fix that.

This to me seems more complicated then having an option vModel on the mount method.
Is there a advantage to this I might have missed?

Is there a advantage to this I might have missed?

Arguable (since @lmiller1990 does not like the extra option 😄 ) cleaner API, since you need to import vmodel method to have this functionality, also this should cover some more edge cases:

Passing a ref should provide a two way binding to that ref, making the test possibly "cleaner"
Opt-in approach without breaking or changing the current API.
You can pass a computed and prevent property changes based on the computed.set
simple callback to test if the update is being called without checking for the emitted. vmodel(1, jest.fn())

I think both are fine - either way we add another option (be it a method or mounting option).
I think vModel seems a (bit) cleaner, but if this covers more edge cases and solves the problem in a cleaner way, it might be better.
Maybe we go ahead with another RC while we think about this a bit more.
The advantage of a vModel mounting option is it's more consistent with the existing Test Utils API.

The advantage of a vModel mounting option is it's more consistent with the existing Test Utils API. Do you have a strong opinion @pikax or just doing some exploration?

I don't have a strong opinion, either way is good.

I think vModel seems a (bit) cleaner, but if this covers more edge cases and solves the problem in a cleaner way, it might be better.

This is pretty much a matter of preference, the only edge case this might cover and the vModel does not (at least atm) is the vmodel(1, jest.fn()), which could be added to the API

This is still on the table, but not pre 2.0.0. I'm going to close it. We can revisit this once we hit a 2.0.0 release and Vue 3 is the main version on npm.


Median Filter in point2dep

Hi, I'm using asp 2.7.0. I created point clouds. I tried to generate DEM with median filter, but different values of window size and threshold have no effect on the result.
point2dem results/run-PC.tif --median-filter-params 11  40 -o stereo/image \
--nodata-value -10000 -n

The point of this switch --median-filter-params switch is to remove large
outliers, like spikes, using a median filter (points that are too different
from the median will be wiped). Unless your DEM has those spikes it won't
be affected.
If you would like the points in the cloud that end up forming the grid
heights to not be processed using a weighted average when binning, but to
simply pick the median in each bin, you should use --filter median.
On Sat, Sep 12, 2020 at 11:04 AM MiladGhorbaniG<EMAIL_ADDRESS>wrote:

Hi, I'm using asp 2.7.0. I created point clouds. I tried to generate DEM
with median filter, but different values of window size and threshold have
no effect on the result.
point2dem results/run-PC.tif --median-filter-params 11  40 -o stereo/image
--nodata-value -10000 -n
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NeoGeographyToolkit/StereoPipeline/issues/318, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKDU3H5VJAL7K6YCOOZL5LSFOZ3BANCNFSM4RJZKRHQ
.

Thanks for the fast answer at the weekend. I will test that.


command 'csscomb.execute' not found

Environment

VSCode Version: 1.22.2
OS Version: MacOS High Sierra 10.13.4

Actual behavior
Plugin does format code
Expected behavior
Plugin doesn't format code. VSCode shows error command 'csscomb.execute' not found.
Steps to reproduce

Install plugin vscode-csscomb.
Run formatting.

Config
    "csscomb.preset": "yandex",
    "csscomb.formatOnSave": true,

Hello, @frux,
Works fine for me with VS Code v1.23.0.
You can provide more information about your editor settings, output from Developer Tools after broken command and list of installed extensions.

In developer tools console I see
notificationsAlerts.ts:39 command 'csscomb.execute' not found
e.onDidNotificationChange @ notificationsAlerts.ts:39
(anonymous) @ notificationsAlerts.ts:28
e.fire @ event.ts:140
e.notify @ notifications.ts:127
e.error @ notificationService.ts:61
t.onError @ commandsHandler.ts:326
(anonymous) @ commandsHandler.ts:311
done @ winjs.base.js:1393
(anonymous) @ commandsHandler.ts:307
done @ winjs.base.js:1539
g @ winjs.base.js:1195
enter @ winjs.base.js:867
_run @ winjs.base.js:1083
_completed @ winjs.base.js:1051

List of installed plugins:
[
    "auto-rename-tag",
    "babelrc",
    "bracket-pair-colorizer",
    "code-settings-sync",
    "EditorConfig",
    "gitflow",
    "gitignore",
    "gitlens",
    "material-icon-theme",
    "npm-intellisense",
    "path-autocomplete",
    "postcss",
    "stylelint",
    "touchbar-command-palette",
    "tslint",
    "vscode-csscomb",
    "vscode-docker",
    "vscode-eslint",
    "vscode-jumpy",
    "vscode-open",
    "vscode-snazzy-operator",
    "vscode-stylefmt",
    "vscode-theme-superonedark",
    "vscode-todo-highlight"
]

It looks like the problem is in the environment. Unfortunately, I can't help you here.


Add reconnection backoff to built-in bridge

Proposed changes
This adds some backoff logic to the MQTT bridge reconnection based on the backoff crate.  It also fixes a minor issue with the tedge_config macro I encountered in the process of making the changes.
Still TODO before merging:

[ ] Ensure it is safe not to poll the rumqttc event loop after a connection error (I believe it is, but if there is an active connection, not polling the event loop will cause issues)
[ ] Deal with the exposed configurations for the timeout parameters, either by exposing the configuration to enable the built-in bridge, or by hiding the new configurations

Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue

#2899

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[x] I ran cargo fmt as mentioned in CODING_GUIDELINES
[x] I used cargo clippy as mentioned in CODING_GUIDELINES
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

@jarhodes314 Is there any insight provided to the user what the current backoff period is (e.g. a log entry)? This would be helpful to debug things when checking when the next connection attempt will be made.

There isn't but I'll add it


Fixes guildMemberUpdate not triggering on nick change

data.roles exists, but may be an empty array.
Example of nick change:
{ user:    { username: 'Clyde',      id: '1234',      discriminator: '1234',      avatar: 'abc' },   roles: [],   nick: 'grtrhtrz',   guild_id: '1234' }

I think this may have side effects. Need @hydrabolt to confirm.

If discord gives us an empty array, it means that a member has actually lost all their roles. It looks like your PR would prevent that?

Looks to me like nickname updates are triggering anyway, so I'm closing this PR.

You're right, yeah. They're not triggering on master, but on indev. It's fixed by 48f84c6.


May you add an example for U-Net?

May you add an example for U-Net architecture?
Thanks!

I have added example made with new code format as well. You can also look an example in old format from commit before Dec 20, 2018. Note that I did not make it in Ushape because I needed it in this particular form. You can play around with code to fit your requirements.

@HarisIqbal88 Awesome, thanks!

@HarisIqbal88
Happy New Year!

@TixiaoShan
Happy New Year!

I am having difficulty building UNet using python. The layers do not stack up well Can you provide a python example?


TAJO-1979: Usage of some TajoShellCommand is omitted

Write omitted usage of TajoShellCommand.

If you add some unit test cases for this patch, it would be more better.

@blrunner Thanks for your review. I found another bug that getUsage() has no usage in whole TAJO, so I have no idea about making unit test cases. TajoShellCommand::printHelp() needs getUsage() but It's omitted. After resolve this issue in another ticket, I add unit test cases relate of this PR using shell command like TestTajoCli::testTimeZoneSessionVars1().

I know you've finished the issue which is involved with this PR. Could you add an unit test case?

@blrunner Oh, thanks for your remind. I'll add unit soon.

+1
LGTM!


The SOURCE() / REF() check is incorrectly looking in text strings for the value "FROM"

Describe the bug
I have a SQL statement which has "FROM" in a text string, but which is not related to an actual database object.  For example
, case when text_column = 'some long string FROM something else' then 'value' else 'something else' end as remap
To Reproduce
Steps to reproduce the behavior:
look at the SQL statement above
Expected behavior
Expect the parser to not look within text strings
Version:
v0.1.0
Additional context
Add any other context about the problem here.

I'm having this issue as well. Here is another example
   select
        DATE_AT
      , LOCATION_ID
      , LOCATION_NAME
      , PROVIDER_ID
      , PROVIDER_NAME
      , TOTAL_IN_OFFICE
      , ALL_MED_REVENUE
        -- Calculate the start of the biweekly period
      , dateadd(day, 13 - (extract(week from DATE_AT) % 2) * 7, date_trunc('week', DATE_AT)) as biweekly_end_at
    from {{ ref('fct_dvm_utilization_metrics') }}
                                        )

The extract(week from DATE_AT) is triggering this inspection.


Transparent wrappers should support zero sized fields

As pre https://doc.rust-lang.org/1.26.2/unstable-book/language-features/repr-transparent.html#enter-reprtransparent they are legal in rust. The current code asserts that there is exactly one field, which matches.

We should support zero sized fields
Longer term question: do we want to support a cast_to_structurally_equivilent_type operation which would work as long as the non-zero-sized fields all lined up?

#822 is related

I believe this was closed by accident.

Double check, but I think I added that in to #834 https://github.com/model-checking/kani/pull/834/files#diff-27e8dd8345085ee28ce32bd8cbf016b726b392167d5e14485b8a303a7f103739R640

Ok. Let me add a test of it and close this again.

Is this fixed and closable then?


Email rate limiting

There are various actions users can take that will cause the server to send mail. Often, the target address is user-controlled.
There needs to be some mechanism to ensure that hostile users can't send thousands of requests and flood people with mail.

Password resets are rate limited.
Email verifications are not (yet).

All possible sources of email are rate limited now.


Add Zone default ttl

Zone.ttl could reflect the default TTL value for all records created. This could use provider-specific functionality or instead lookup the SOAData.ttl and apply it.
This was attempted in #351, but not practical due to lack of support.

route53 - unsupported as rrsets require an explicit ttl param
ultradns - unsupported as records require an explicit ttl param
google cloud dns - unsupported as rrsets require an explicit ttl param
dynect - supported on zone create, and updates are via SOA.ttl
clouddns - supported on zone create, update, default ttl is not shown in list, so you need to GET the specific zone.
designate - supported on zone create, update and list

Using the SOAData.ttl in lieu of provider support would make this feature expensive to implement, as that would imply we need to lookup or cache the SOA for any RRSet update that set the ttl to null.

One lighter weight means to do zone default ttl could be to just expose a way to tell denominator what ttl to use when one isn't set in the rrset. This would eliminate a network call dependency and also the non-portable dilemma.

In DynECT SOA.ttl controls the default ttl of new records

for ultradns you can set the default TTL per record type in the SOAP API, see: saveAccountPreferences

@jdamick so, in principle, I'd be able to eliminate the TTL attribute from the below requests, then?
createResourceRecord
addDirectionalPoolRecord
updateDirectionalPoolRecord
addRecordToRRPool
updateRecordOfRRPool

I don't think we will be able to easily make a default zone ttl portable. However, we shouldn't throw the baby out with the bathwater.
Let's make it possible to display the zone (SOA) ttl conceding it may not imply anything else.
We should go back and correct rrset.ttl docs. They say it falls back to the default zone ttl which isn't entirely true. As we found in this thread, a default could be from the account, zone, or even somewhere else (like hard coding :))

sounds good, i need to check the new REST api to see where that stands as well.

Cleared out comments after reworking title and description


changed linkcheck to match the skeleton

Removed changes made to the keepalive workflow during the initial merge

Close #251

@coretl I'm unsure as to why I made those changes in the initial CI adoption... I was probably too aggressive in my adjustments to the skeleton CI and added some unnecessary features.

There is still one difference from https://github.com/bluesky/python3-pip-skeleton/blob/main/.github/workflows/linkcheck.yml, could you copy it again please?


Blazor App 2 seperate  MediatorDependencyInjectionExtensions (AddMediator) in same namespace, but different assemblies.

In a Blazor App, you have the server side and client side applications where the server side references the client side.
This means, that if Mediator is used in the client-side, it will generate an MediatorDependencyInjectionExtensions class in Microsoft.Extensions.DependencyInjection.
It will then also generate an MediatorDependencyInjectionExtensions in the same namespace of the server assembly because it references the client assembly.
Which means it's not possible to use Mediator both on client & server projects? Or am I missing a configuration setting that can be set to change the generation namespace?

Have you reproduced this behavior?
Is there a repo with minimal reproduce?

https://github.com/SGStino/Mediator.BlazorApp-Test
It's far from complete, but the problem already surfaces when you run it, and HTTP GET https://localhost:7070/test/something?request=12345
it returns the response from the client handler instead of the server handler.

Oh, i forgot to add the source generator to the server side project. There was no server side AddMediator.
When there is one, it does exactly as i expected:
Error (active)	CS0121	The call is ambiguous between the following methods or properties: 'Microsoft.Extensions.DependencyInjection.MediatorDependencyInjectionExtensions.AddMediator(Microsoft.Extensions.DependencyInjection.IServiceCollection, System.Action<Mediator.MediatorOptions>)'


Textbox fill - mixed behaviors - blue or white?

Please verify the version of auro-input you have installed
2.13.2
Please describe the bug
There are two different background colors of the auro-input box to view:
Background is WHITE

Go to https://booking-site.ecomm-test-aks.alaskaair.com/order/guests
Click into 'Legal first name'
Type 'TestOne'
Click into 'Middle name'
Type 'TestOne'
Click into 'Legal last name'
Type 'TestOne'
Click on page white space to unfocus previous input
View those three input background colors to be white

Background is BLUE

Go to https://booking-site.ecomm-test-aks.alaskaair.com/order/guests
Tab until 'Legal first name' is in focus
Use arrow keys to highlight a previously entered value, such as 'TestOne'
Press Enter
Tab to focus 'Middle name'
Use arrow keys to highlight a previously entered value, such as 'TestOne'
Press Enter
Tab to focus 'Legal last name'
Use arrow keys to highlight a previously entered value, such as 'TestOne'
Press enter
View those three input background colors to be blue

QUESTION: Why are we seeing two different background colors depending on the method used to enter the same text?

Reproducing the error on the docsite
None
Expected behavior
Background color of auro-input fields would be the same color when valid, regardless of what method the user used to enter the text.
method examples: 1. clicking into input and typing new text, 2. tabbing into input, using arrow keys to navigate to a previously entered value, pressing enter to select
What browsers are you seeing the problem on?
Chrome
Additional context

it didn't seem to matter whether the input was required or not required, either way the questionable behavior is found
if this depends on the browser having previously used values saved, and that causes difficulty, please reach out to me and I can show a screen share demo
currently unable to repro on Auro docsite, perhaps because there's not a set of inputs together within one form (they are stand alone examples of different use cases)

This color is controlled by the browser, and not something we see value in changing within Auro.


Debian 11 Static VM - fdisk

Background
I'm using PKB to run workloads on a static GCP VM and on GCP VMs provisioned by PKB.
Both the static and provisioned VMs are using Debian 11.
Issue
There seems to be an issue with fdisk not being installed on Debian 11. PKB accounts for this in perfkitbenchmarker/linux_virtual_machine.py with a workaround, but this fix is not extended to perfkitbenchmarker/static_virtual_machine.py
Provisioned VMs

Specifying os_type: debian11 in my config works just fine, PKB works as expected: spins up/tears down the VM and runs the workloads successfully

Static VMs

If I don't specify os_type in my config, PKB defaults to ubuntu1804 and then fails on the usage of fdisk (see details below)

2022-07-27 18:29:09,139 6f3d18c4 MainThread coremark(1/1) ERROR    Exception running benchmark
Traceback (most recent call last):
  File "/perfkitbenchmarker/pkb.py", line 1299, in RunBenchmarkTask
    RunBenchmark(spec_for_run, collector)
  File "/perfkitbenchmarker/pkb.py", line 1078, in RunBenchmark
    DoProvisionPhase(spec, detailed_timer)
  File "/perfkitbenchmarker/pkb.py", line 755, in DoProvisionPhase
    spec.Provision()
  File "/perfkitbenchmarker/benchmark_spec.py", line 718, in Provision
    vm_util.RunThreaded(self.PrepareVmAfterBoot, self.vms)
  File "/perfkitbenchmarker/background_tasks.py", line 647, in RunThreaded
    return RunParallelThreads(target_arg_tuples,
  File "/perfkitbenchmarker/background_tasks.py", line 582, in RunParallelThreads
    return _RunParallelTasks(
  File "/perfkitbenchmarker/background_tasks.py", line 556, in _RunParallelTasks
    raise parallel_exception_class(
perfkitbenchmarker.errors.VmUtil.ThreadException: The following exceptions occurred during parallel execution:
Exception occurred while calling PrepareVmAfterBoot(<IP_ADDRESS>):
Traceback (most recent call last):
  File "/perfkitbenchmarker/background_tasks.py", line 252, in Run
    self.return_value = self.target(*self.args, **self.kwargs)
  File "/perfkitbenchmarker/benchmark_spec.py", line 977, in PrepareVmAfterBoot
    vm.PrepareVMEnvironment()
  File "/perfkitbenchmarker/linux_virtual_machine.py", line 471, in PrepareVMEnvironment
    self.RecordAdditionalMetadata()
  File "/perfkitbenchmarker/linux_virtual_machine.py", line 783, in RecordAdditionalMetadata
    self.os_metadata.update(self.partition_table)
  File "/perfkitbenchmarker/linux_virtual_machine.py", line 740, in partition_table
    partition_tables = self.RemoteCommand(cmd)[0]
  File "/perfkitbenchmarker/linux_virtual_machine.py", line 965, in RemoteCommand
    return self.RemoteCommandWithReturnCode(*args, **kwargs)[:2]
  File "/perfkitbenchmarker/linux_virtual_machine.py", line 981, in RemoteCommandWithReturnCode
    return self.RemoteHostCommandWithReturnCode(*args, **kwargs)
  File "/perfkitbenchmarker/linux_virtual_machine.py", line 1053, in RemoteHostCommandWithReturnCode
    raise errors.VirtualMachine.RemoteCommandError(error_text)
perfkitbenchmarker.errors.VirtualMachine.RemoteCommandError: Got non-zero return code (1) executing sudo fdisk -l
Full command: ssh -A -p 22 <user>@<ip_address> -2 -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no -o IdentitiesOnly=yes -o PreferredAuthentications=publickey -o PasswordAuthentication=no -o ConnectTimeout=5 -o GSSAPIAuthentication=no -o ServerAliveInterval=30 -o ServerAliveCountMax=10 -i <ssh_key_path> -o ControlPath="/tmp/perfkitbenchmarker/runs/6f3d18c4/ssh/%h" -o ControlMaster=auto -o ControlPersist=30m sudo fdisk -l
STDOUT: STDERR: sudo: fdisk: command not found

Specifying os_type: debian11 in my config raises a SubClassNotFoundError (details below)

2022-07-27 18:11:02,810 20ddbd71 MainThread coremark(1/1) ERROR    Exception running benchmark
Traceback (most recent call last):
  File "/perfkitbenchmarker/pkb.py", line 1299, in RunBenchmarkTask
    RunBenchmark(spec_for_run, collector)
  File "/perfkitbenchmarker/pkb.py", line 1078, in RunBenchmark
    DoProvisionPhase(spec, detailed_timer)
  File "/perfkitbenchmarker/pkb.py", line 734, in DoProvisionPhase
    spec.ConstructVirtualMachines()
  File "/perfkitbenchmarker/benchmark_spec.py", line 541, in ConstructVirtualMachines
    vms = self.ConstructVirtualMachineGroup(group_name, group_spec)
  File "/perfkitbenchmarker/benchmark_spec.py", line 438, in ConstructVirtualMachineGroup
    static_vm_class = static_vm.GetStaticVmClass(vm_spec.os_type)
  File "/perfkitbenchmarker/static_virtual_machine.py", line 314, in GetStaticVmClass
    return resource.GetResourceClass(virtual_machine.BaseVirtualMachine,
  File "/perfkitbenchmarker/resource.py", line 49, in GetResourceClass
    raise errors.Resource.SubclassNotFoundError(
perfkitbenchmarker.errors.Resource.SubclassNotFoundError: No BaseVirtualMachine subclass defined with the attributes: {'CLOUD': 'Static', 'OS_TYPE': 'debian11'}

Proposed Solution
To workaround this issue I can sudo apt install fdisk on the static VM.
However, I think a more permanent fix in the codebase to support the debian11 mixin for static virtual machines would be better for anyone else who may encounter this issue.

Supporting a debian11 mixin for static VMs seems like a good solution. Feel free to send us a PR.

Sounds good, I'll get working on that, thank you for the response.

Fixed in #3761


fix AUR packaging

Thanks for the tool!
I saw the AUR package is out of date compared to this version and I could not install this one without making some changes to the PKGBUILD so here they are.

Indeed I know. But the maintainer of the package has not been available for quite some time. I have contacted him to become the maintainer but I have not yet had an answer.
But thanks for correcting the package ! I don't have much experience in the Arch community yet, so there is still a lot I don't know.


Compatiblity with IsDisabled()

It's possible, that Buienradar is currently not compatible with IsDisabled() from the official FHEM API Description.

Ok, it is not!


Add IF(), IFNULL(), NULLIF()

Those are useful:
https://dev.mysql.com/doc/refman/8.0/en/control-flow-functions.html#function_ifnull
https://dev.mysql.com/doc/refman/8.0/en/control-flow-functions.html#function_if
https://dev.mysql.com/doc/refman/8.0/en/control-flow-functions.html#function_nullif

Will work on this now

There might be an issue of determining the Type() of NullIfand If functions.
The case is when you know the result you can return it but in that case, we don't know the result type before the function evaluates.
For example in NullIf:
We need to know if Left and Right are equal so we return sql.Null otherwise we return Left.Type()
In If:
We need to know if Left evaluates to true so we return Left.Type() otherwise we return Right.Type()
What is the recommended approach?

You can check how the type is chosen on MySQL documentation: https://dev.mysql.com/doc/refman/8.0/en/control-flow-functions.html#function_if
But basically, using IF(expr1,expr2,expr3) as an example:

If expr2 or expr3 produce a string, the result is a string.
If expr2 and expr3 are both strings, the result is case-sensitive if either string is case sensitive.
If expr2 or expr3 produce a floating-point value, the result is a floating-point value.
If expr2 or expr3 produce an integer, the result is an integer.

Thank you. The question remains for the NullIf is that how we can check if Left and Right are equal as I can see that the Type() gets called before the function evaluates. I mean that I will have to know the result of the Eval beforehand or return a type that will resolve to either sql.Null or Left.Type()
For example if I just return Left.Type() then if the following call:
select nullif(2,2);

will return 0 instead of NULL

The type selection will be done as I described before, the only difference is that the result will be null. Null is not a type itself. I will open an issue to remove it from types list.


Babel is outdated

juanma@le-toucan:/home/proyectos/Pokemon-Showdown-Bot$ npm install
npm WARN deprecated<EMAIL_ADDRESS>Babel 5 is no longer being maintained. Upgrade to Babel 6.

This project was officially abandoned 6 months ago and is no longer maintained, as noted in the readme Feel free to fork it and fix the issue on your fork, but issues or pull requests are no longer dealt with or accepted.


Ionic 2 Badges space issue (RTL)

Ionic version:  (check one with "x")
[ ] 1.x
[ x] 2.x
I'm submitting a ...  (check one with "x")
[x ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Wrong padding/space when using ion-badges in RTL layout Ionic 2 apps:

Expected behavior:
Correct the padding/spacing in RTL layouts, to work as excepted:

Steps to reproduce:
Using  in RTL Ionic 2 apps
Related code:
<ion-item style="direction:rtl">
  <ion-icon name="checkmark" item-left></ion-icon>
  TOTAL
  <ion-badge item-right>1</ion-badge>
</ion-item>

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.5
Ionic CLI Version: 2.1.18
Ionic App Lib Version: 2.1.9
Ionic App Scripts Version: 1.0.0
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v6.9.2
Xcode version: Not installed

Hello, thanks for using Ionic! We really appreciate this feedback on RTL support. This is not currently high priority for us at the moment, but we are tracking it in #5035. Because of this I am going to close this issue as a duplicate of that issue and ask that you put your feedback on that issue. Please know that by me closing this issue I am not saying that your feedback is not important, or that RTL support is not important, but I am simply closing this issue as we like to keep our issues as organized as possible so things do not get lost in the weeds (which is better said than done haha).


Deal with increasing S3 cost

Now that we have a rapidly growing dataset on AWS (~100GB/week, likely more) we should look a bit more into how we can reduce our S3 bill. We are already at 3$/day - assuming that this will only grow it's better to look into this sooner rather than later.
We should think again about how frequently this data will need to be accessed and what kind of delay is acceptable to us. We should also probably think about whether we can live with loosing some of the data - if so, we should consider S3 IA One AZ.
TBD.

Also see issue #4

Ah, cool, didn't see that! Let's try to go for reducing cost without loosing data first, and then later go back to #4, I think.

For now we set up a lifecycle rule that moves all files to S3 Infrequent Access after 30 days. Let's keep an eye on it and see how it behaves.


input/output error on all Keybase folders

I've reprovisioned this device 3 times now with no success. I also deleted ~/.local/share/keybase for good measure between provisions.
my log id: b36a698858d5fc1c3a99911c

We talked privately. @taohansen is running NixOS distributions, which seems to still have Keybase version of 2.6.0 in their packages, which is pretty old and likely not allowed to contact our servers anymore.

The versions we currently have for keybase on master are

keybase-client - 2.13.1
kbfs - 2.11.0
keybase-gui - 2.13.1

It's possible that @taohansen is probably using the 18.09 release and we have

keybase-client - 2.10.1
kbfs - 2.10.1
keybase-gui - 2.11.0

I think you should probably update your machine so you get these newer versions.
However in NixOS this has been a running issue for some people https://github.com/NixOS/nixpkgs/issues/50999

It was an old version that was the issue. I upgraded to 2.10.1 and now have access.


How did you train the MPT-7b-chat model?

Hi everyone, thanks a lot for the great library!
Could someone please explain a bit more on how the chat model has been trained? More specifically, I am interested in how the input/output data has been preprocessed.
For example, the instruct model is using:
prompt = "Below is an instruction that describes a task. Write a response that appropriately completes the request. ### Instruction: what is kangen water? ### Response: "
response = "Kangen water is alkaline ionized water ...".

which follows the expected format of the llmfoundry-SFT dataloader where the model is trained to predict only the response part and inputs are formatted as: Below is an instruction that describes a task. Write a response that appropriately completes the request. ### Instruction: <some_text>. ### Response: <some_text>"

Is the same format used for finetuning of the chat model? From the blog post, I see that the chat model is trained on a mixture of instruction (e.g. Alpaca) and chat datasets (e.g. HH_RLHF). This confuses me a bit, because I am not sure how they are formatted and used together. Is HH_RLHF formatted in the instruct-format: Below is an instruction that describes a task. Write a response that appropriately completes the request. ### Instruction: <some_text>. ### Response: <some_text> where Human part is used as Instruction and Assistant is used as Response, or Alpaca is reformatted to the chat format used in hf_chat.py script: You are a helpful assistant. User: <some_text>. Assistant: <model_write_here> ?
How do you handle multi-turn conversations from HH_RLHF dataset during the finetuning stage? Do you just train on the last answer given the entire conversation history as the prompt or something else?

Thanks for your time and answers!

What I learned is that alpaca and other datasets are converted to ChatML syntax.

We actually use neither of the formats you mention. We use ChatML syntax, described here and implemented in code in this file.
Each turn of a multi-turn conversation results in 1 training sample. Only the response tokens are loss-generating, so it only learns from a given response once.

So there's no reinforcement learning?

There was no RL involved in training mpt-7b-chat or mpt-30b-chat

@samhavens , could you please elaborate?
You wrote

Each turn of a multi-turn conversation results in 1 training sample. Only the response tokens are loss-generating, so it only learns from a given response once.
So I am wondering what exactly you mean.

Given the following conversation:
User:
My name is John.
What is 1+1?
Assistant:
Hi John, 1+1=2
User:
What is my name?
Assistant:
Your name is John
Would we get the following two training datas/samples:
1-1)
User:
My name is John.
What is 1+1?
Assistant:
Hi John, 1+1=2
1-2)
User:
What is my name?
Assistant:
Your name is John
Or would we get the following two training datas/samples:
2-1)
User:
My name is John.
What is 1+1?
Assistant:
Hi John, 1+1=2
2-2)
User:
My name is John.
What is 1+1?
Assistant:
Hi John, 1+1=2
User:
What is my name?
Assistant:
Your name is John
Because from what you wrote it seems like it would be according to the first scenario, but in that case the assistant wouldn't know the user's name is John.
Can you please clarify or explain to me what I am missing?

@TamirHCL The second is what I meant, but I see how what I said was misleading. I just meant there was a 1-to-1 relationship, not that each turn ended up in only one sample.

@samhavens thank you so much for clarifying!
On a separate topic, can I ask how come the storywriter model was trained with max_seq_len set to 65K?
Was it because of memory limitations with 80GB GPUs?

@TamirHCL We needed some max_seq_len, and at that point, the longest LLM context that anyone had access to was 8k, so we wanted to go well beyond that

@samhavens Follow-up questions regarding long prompts.

Is there a rule of thumb for how much much VRAM I need to train MPT-7B with difference sequence lengths?
How many 80GB GPUs are needed to train with max_seq_len set to 80K and 100K?

Is there some recommended value to set "PYTORCH_CUDA_ALLOC_CONF=max_split_size_mb" to when training with max_seq_len set to 60K-100K?


SymbolId is not unique when nested types have the same name

Disclaimer: I'm encountering this problem with Wyam, but I believe applies to Statiq as well.
For example, in the following code, both nested classes will be assigned the symbol id Factory, despite them living in the same namespace.
public class Namespace.Foo
{
    public class Factory {}
}
public class Namespace.Bar
{
    public class Factory {}
}

I think this could be fixed fairly easily in ISymbolExtensions.GetId by checking ISymbol.ContainingType - if it's null, use the original behavior, otherwise
use GetFullName instead of ISymbol.MetadataName.
i.e. Replace this:
https://github.com/statiqdev/Statiq.Framework/blob/c80ebe82a034f66ccc270d6c0c94570c868e08af/src/extensions/Statiq.CodeAnalysis/ISymbolExtensions.cs#L25
with this:
char[] id = symbol.ContainingType == null ?
            symbol.MetadataName.ToCharArray() :
            GetFullName(symbol).ToCharArray();

I'd submit a PR, but I'm not sure how easy it would be to set up an environment I can test this in Statiq.

Thinking about it, just calling GetId recursively might work better.
char[] id = symbol.MetadataName.ToCharArray();
if (symbol.ContainingType != null)
{
    id = symbol.ContainingSymbol.GetId() + "." + id;
}

Good catch - we definitely want unique IDs in case they're used as keys in a lookup or something. I'll classify this as a bug and look at fixing in the AnalyzeCSharp module.

I was able to repro this in a unit test and fix with a recursive GetId() approach very similar to what you suggested. Thanks again for the great bug report!


Updated to use the TabBar

Description
Updated the tab bar
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

Change Status

[x] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

Hey Vici can you check this out?


[styled]styled wraped component, modal disableRestoreFocus property doesn't work.

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
sorry, my english is poor. :(
styled wrapped Button. open modal, then press ESC, wrapped Button component doesn't have focus status.
Expected behavior 🤔
wrapped component should have focus status.
Steps to reproduce 🕹

click styled button
modal opend
press ESC
styled button doesn't have focus status

https://codesandbox.io/s/basicmodal-material-demo-forked-1c65i?file=/demo.tsx
Context 🔦
just a demo
Your environment 🌎

Tech
Version

@mui/material
5.0.4

React
17.0.2

Browser
Chrome/95.0.4638.69 Safari/537.36

OS
Linux 5.4 Ubuntu 20.04 LTS (Focal Fossa)

Thanks for the report. Check this codesandbox: https://codesandbox.io/s/basicmodal-material-demo-forked-9lw8g
I replaced your Button component with TextField so that it is easier to check whether or not focus state is restored after modal is hidden. As you can see, focus state is successfully restored.

This issue should be caused by renderer, move the StyledButton outside of the functional component, it will be ok. So this may not be a bug.  😕

https://stackoverflow.com/questions/57855233/prevent-component-re-render-with-styled-components

@ylhb You are right. Check this out: https://codesandbox.io/s/basicmodal-material-demo-forked-ts3xb


[Meta] Enable multi-hop migration of data through snapshot-restore

This meta issue tracks feature work that enables customers to migrate their data from a cluster running on a legacy version to the latest version of OpenSearch, through the snapshot-restore mechanism.

[x] Enable searchable snapshot support for older versions (ES 6.0) - https://github.com/opensearch-project/OpenSearch/issues/5451
[ ] ~Expand the above functionality to support even older versions (say ES 5.0)~
[ ] ~Investigate re-indexing of old snapshots on the updated cluster~

This method could also be used for upgrading clusters in-place, but would result in downtime

While snapshot restore is great option for lift-and-shift migrations, it would be exciting to see this functionality expanded to in-place upgrades as well as it provides zero downtime option for version migrations. Could we also track that as part of Multi-version meta issue?

Related - https://github.com/opensearch-project/OpenSearch/issues/5443

Ownership of the extended compatibility of searchable snapshots has been moved to the core opensearch repo - https://github.com/opensearch-project/OpenSearch/issues/5948
We will revisit reindexing of snapshots at a later time. Closing the issue.


Add Exception throwing to Android SDK when an error is returned from FFI

Currently, if we receive an error from the FFI inside the Android SDK, we simply print the error and continue. We should throw an exception instead.
This ticket involves - creating set of Mentat specific Exception classes and throwing them when an error is received from Mentat over the FFI.

So I definitely agree we should be exposing Mentat errors as Exception instances to Java, but I'd like to pump the brakes a little bit on the "set of Mentat specific Exception classes".  The errors coming out of Mentat are fluctuating wildly -- for example, 1/2 of the transactor errors are just strings -- so let's not tie ourselves to any one representation too soon.  Could we do the simplest thing first -- a MentatException with a string description -- and then figure out what value we're really trying to provide before we encode a class hierarchy?

Something I've used before and think might be valuable here: some errors are recoverable and some are not, and some are external and some caused by your code.
An error during a query might be:

From SQLite (external) because your disk is full (non-recoverable)
From SQLite because Mentat produced a malformed SQL query (non-recoverable)
From Mentat because your query is malformed

At the very least, then, I think it's worth making parse errors — EDN, query, transact — separate and visible to the developer. The rest can be bucketed as "DB error".

The three error types we have on iOS right now are:
public enum QueryError: Error {
    case invalidKeyword(message: String)
    case executionFailed(message: String)
}

public struct MentatError: Error {
    let message: String
}

public enum PointerError: Error {
    case pointerConsumed
}

Adding TransactError and ParseError to this set is easy enough I think. This is the set I was going to replicate in the Java library.


Enable 'style' rule configuration for eslint-plugin-jest

Using the style configuration over the recommended config enables three more opinionated rules:

prefer-to-be – Suggest using toBe() for primitive literals
prefer-to-contain – Suggest using toContain()
prefer-to-have-length - Suggest using toHaveLength()

These rules are all automagically fixable using --fix.
Raised for discussion as to whether these would be useful in addition to the recommended rules enabled in #4842.

Given all these rules are automatically fixable, I think it's pretty safe to merge and have consistent matchers in the code base. The linting pre-commit (or your editor if it lints on save) will put the right matcher when necessary, as well, so the burden should be quite light if you're used to writing the disallowed version.


Don't serialize expressions of they don't exist.

fixes: ENTESB-13705

@tplevko - can you test this out and approve if its okay?

Let's merge this and in case reopen it again.


container ID either in path OR hostname

@scottx611x : I think you have a branch where you figured this out? I'd like to keep the option for path based dispatch (some users might not be able to set up a dns wildcard domain) ... but your stuff is what we'll need going forward. Just need to figure out what this configuration would look like.

Fixed by #92 and #93


Series.apply not callable when applying tolist()

Describe the bug
A clear and concise description of what the bug is.
Series().apply().tolist() errors with
error: "Series[Any]" not callable  [operator]

after latest release.
To Reproduce

Provide a minimal runnable pandas example that is not properly checked by the stubs.
Indicate which type checker you are using (mypy or  pyright).
Show the error message received from that type checker while checking your example.

s = pd.Series([1, 2, 3]).apply(lambda x: x).tolist()

error: "Series[Any]" not callable  [operator]

Please complete the following information:

OS: [e.g. Windows, Linux, MacOS] MacOS
OS Version [e.g. 22] 12.5.1
python version 3.10
version of type checker mypy 0.960
version of installed pandas-stubs [v<IP_ADDRESS>906]

Additional context
Add any other context about the problem here.

We probably never had annotations for tolist() but it "worked" because __getattr__ was defined on NDFrame with an un-annotated return type (tolist was probably inferred as Any/unknown).

That's not the whole story.  We do have tolist() and to_list() defined for Series.
The issue here is that we have typed apply() to return DataFrame | Series, and tolist() doesn't work for a DataFrame.
The solution is to do
s = cast(pd.Series, pd.Series([1, 2, 3]).apply(lambda x: x)).tolist()

Having said that, we can probably better type apply(), by specifying the type of Callable, because docs say if the Callable returns a Series, then the result is DataFrame.  But for that to work with this example, you'd have to change lambda x: x to be typed, or create a function def myfun(x: Scalar) -> Scalar:  return x
So I'm going to update the title of this issue accordingly.

Same here:
import pandas as pd
def func1(df: pd.DataFrame) -> pd.Series:
      return df.url.apply(get_depth)

def get_depth(url: str) -> int:
      return url.strip("/").count("/") - 2

~  pds ❯ mypy test.py
test.py:4: error: Incompatible return value type (got "Union[Series[Any], DataFrame]", expected "Series[Any]")
Found 1 error in 1 file (checked 1 source file)


fix crc

修复CRC兼容性专有域版本V730(1.0.9)的Bug；

Coverage remained the same at 90.366% when pulling fac1fd69e1bb49a015775cf96f94f532d00298fe on fix-crc into 6c70079c0ab8bbc0a7d3fc32b9ea16d6d6e1cdfb on master.


Plug for Mac OSX 10.9.5

I'm running Mavericks. I used to have Plug for Mac, it's great. I tried to reinstall it recently with brew cask and the cask is only version 2.0.9. Is there any way to get the older version for Mac OSX 10.9.5. I'm not going to be upgrading my OS anytime soon.

You can introduce a conditional. See https://github.com/caskroom/homebrew-cask/blob/4372f55460706a24cdfcbd71062560a2dc4691b6/Casks/calibre.rb#L2L10 for an example.

Thanks. I'll try to submit a PR.

Closing in favour of the PR.


bug:

Describe the bug
No titles visible in the course content sidebar
To Reproduce
Steps to reproduce the behavior:

Clicked on the show content button to open the sidebar
Can't see any titles

Expected behavior

Should be seeing the titles of the videos

Screenshots or GIFs

Info (please complete the following information):

Browser - chrome
Version - 131.0.6778.140 (64-bit)

Can i take up this issue?

Can i take up this issue?
This bug has been resolved, updated built is not deployed yet.
Hey @AlistairM13, please close it in favour of #1632

Fixed in https://github.com/code100x/cms/pull/1632


Awesome Work

Your app is totally awesome. But, please add a way to unsubscribe from multiple sources.

Not planned right now, pull requests are welcome.