Crashes when wsl.exe is selected as shell

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Windows 10 Version 1709 (OS Build 16288.1)
Hyper.app version:  1.4.4
Link of a Gist with the contents of your .hyper.js: https://gist.github.com/iamdevlinph/11a3497287df4aa58038cbdeb310b015
The issue is reproducible in vanilla Hyper.app: If by vanilla, no plugins, then Yes

Issue

Hyperterm crashes when shell: 'C:\\Windows\\System32\\wsl.exe', is set in the config. I noticed this after I updated to 1.4.4. After restarting the app, it just crashes.
Windows also updated before hand but Hyper was not crashing that time.

I will try the fix mentioned in https://github.com/zeit/hyper/issues/2200 It's about powershell so I didn't bother with it much.
Really sleepy right now so tomorrow i will try it

Sadly, the fix doesn't work or I'm doing it wrong.
Anyways, I just changed the shell to bash.exe and manually switch to zsh. It's just once a day, so not really that of a hassle. But still annoying.

Why would you want to use wsl.exe over bash.exe?

So that I can set my default shell to zsh? bash.exe reads a different file than the wsl.exe one.

Closing.
1.4.6 release fixed the problem

@iamdevlinph oh interesting, how do you do that then?

I just do a chsh -c zsh (not entirely sure if this is the correct syntax but it's just like what you normally do when setting default shell in linux). wsl.exe reads the file that is generated by that command while bash.exe doesn't.

Oh I see, wsl.exe is new to the Fall Creators update, thanks!

Ohhh. I just realized you were asking "how I got it". I thought you meant "how I got the default shell" lol.
Yes. It's in the Fall Creators update. Right now, it's only available for the insiders.

https://medium.com/@finnzeit/set-and-use-zsh-as-default-shell-in-wsl-on-windows-10-the-right-way-4f30ed9592dc
For those interested

Do note that Hyper still crashes when you want to create a new tab

This still isn't working for me. Any fix?

@itsjoel works here, try
shell: 'C:\\Windows\\System32\\wsl.exe',
shellArgs: [],

Thanks! It seems that it doesn't work if you have anything in shellArgs, even "". It's working now.

I'm not sure what happened but this still doesn't work. It doesn't work with git-bash in Windows, either.

Same as @methodbox. Not working for me also

Guys I actually fixed this; there’s a few gists out there people have posted that work.
Just google “hyper git-bash config” and reference their settings against yours.
In my case I missed removing —login and for powershell it was something similar.
I was able to get both git-bash and PS working, but as the theming is still pretty trash and their docs about the same I gave it up.

Sorry, but git-bash is not WSL, nor recommended to use bash.exe on newer versions.

@leodutra
I know this but the OP just wants to use zsh which doesn’t require the baggage of WSL to work - it only requires Git-Bash in Windows.
One might ask WTF you’re using Hyper in Windows for if you’re intention is to actually use WSL? Why not use a GUI inside WSL and install Hyper there?
If, however, you’re goal is the same as the OP, Git-Bash is more than sufficient.

Well, I stated that and started trying by myself, on my Windows boot.
I moved to WSL2, using the Windows insider version and this solved:

Same as @methodbox. Not working for me also
Current settings
shell: 'C:\\Windows\\System32\\wsl.exe',
shellArgs: [],

For anyone with the same need... I did these steps, more or less:
Installing Windows Subsystem for Linux 2, Hyper, ZSH, Node.js and VSCode extensions
Requirements

Windows 10 Build 18917+ (only as Windows Insider 09/2019)

Steps
1. Enable WSL2
Run in PowerShell, as admin (elevated):
Enable-WindowsOptionalFeature -Online -FeatureName Microsoft-Windows-Subsystem-Linux
Enable-WindowsOptionalFeature -Online -FeatureName VirtualMachinePlatform

2. Install a Linux distro for WSL2

open Microsoft Store on Windows
search for Linux
install Ubuntu 18.04 (recommended)

3. Install Chocolatey
Run in CMD, as admin:
@"%SystemRoot%\System32\WindowsPowerShell\v1.0\powershell.exe" -NoProfile -InputFormat None -ExecutionPolicy Bypass -Command "iex ((New-Object System.Net.WebClient).DownloadString('https://chocolatey.org/install.ps1'))" && SET "PATH=%PATH%;%ALLUSERSPROFILE%\chocolatey\bin"

4. Install Hyper and VSCode using Chocolatey
Run in CMD, as admin:
choco install hyper vscode -y

5. Set Hyper to use WSL

open Hyper;
Edit > Preferences
change lines:

    shell: 'C:\\Windows\\System32\\wsl.exe',
    shellArgs: [],

save preferences;
restart Hyper.

6. Install ZSH and Oh My Zsh
Run in Hyper:
sudo apt install zsh -y
sh -c "$(curl -fsSL https://raw.githubusercontent.com/robbyrussell/oh-my-zsh/master/tools/install.sh)"

7. Install NVM using ZSH
Run in Hyper:
curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.35.0/install.sh | bash
echo "export NVM_DIR=~/.nvm\n[ -s \"$NVM_DIR/nvm.sh\" ] && . \"$NVM_DIR/nvm.sh\"" >> ~/.zshrc
source ~/.zshrc

8. Install Node (latest) using NVM
Run in Hyper:
nvm install node
nvm alias default node

9. Install Yarn
Run in Hyper:
curl -sS https://dl.yarnpkg.com/debian/pubkey.gpg | sudo apt-key add -
echo "deb https://dl.yarnpkg.com/debian/ stable main" | sudo tee /etc/apt/sources.list.d/yarn.list
sudo apt-get update && sudo apt-get install --no-install-recommends yarn

10. Install Nodemon, PM2, ESLint, Pug
Run in Hyper:
npm i -g nodemon pm2 eslint pug

11. Install neofetch
Run in Hyper:
nvm install neofetch
echo 'neofetch' >> ~/.zshrc

12. Install VSCode extensions
Run in Hyper:
code --install-extension amandeepmittal.pug
code --install-extension bierner.markdown-preview-github-styles
code --install-extension DavidAnson.vscode-markdownlint
code --install-extension dbaeumer.vscode-eslint
code --install-extension eamodio.gitlens
code --install-extension formulahendry.code-runner
code --install-extension hollowtree.vue-snippets
code --install-extension jianbingfang.dupchecker
code --install-extension ms-vscode.cpptools
code --install-extension ms-vscode.csharp
code --install-extension ms-vscode.vscode-typescript-tslint-plugin
code --install-extension msjsdiag.debugger-for-chrome
code --install-extension naumovs.color-highlight
code --install-extension octref.vetur
code --install-extension pkief.material-icon-theme
code --install-extension ritwickdey.LiveServer
code --install-extension rust-lang.rust
code --install-extension sysoev.language-stylus
code --install-extension WallabyJs.quokka-vscode
code --install-extension yzane.markdown-pdf
code --install-extension yzhang.markdown-all-in-one

13. Upgrade Ubuntu
sudo apt upgrade -y

14. Exit Hyper
Run in Hyper:
exit

@leodutra
I know this but the OP just wants to use zsh which doesn’t require the baggage of WSL to work - it only requires Git-Bash in Windows.
One might ask WTF you’re using Hyper in Windows for if you’re intention is to actually use WSL? Why not use a GUI inside WSL and install Hyper there?
If, however, you’re goal is the same as the OP, Git-Bash is more than sufficient.

BTW, I think Hyper is the best option of a terminal for Windows, for all of those who support well made open source software.

I just followed the steps in @leodutra comment and once I change the shell to wsl, it doesn't load anything inside Hyper, just a pink cursor is shown but no prompt is shown and I can't type.
This is what I see:

This is the config file:

// Future versions of Hyper may add additional config options,
// which will not automatically be merged into this file.
// See https://hyper.is#cfg for all currently supported options.

module.exports = {
  config: {
    // choose either `'stable'` for receiving highly polished,
    // or `'canary'` for less polished but more frequent updates
    updateChannel: 'stable',

    // default font size in pixels for all tabs
    fontSize: 12,

    // font family with optional fallbacks
    fontFamily: 'Menlo, "DejaVu Sans Mono", Consolas, "Lucida Console", monospace',

    // default font weight: 'normal' or 'bold'
    fontWeight: 'normal',

    // font weight for bold characters: 'normal' or 'bold'
    fontWeightBold: 'bold',

    // line height as a relative unit
    lineHeight: 1,

    // letter spacing as a relative unit
    letterSpacing: 0,

    // terminal cursor background color and opacity (hex, rgb, hsl, hsv, hwb or cmyk)
    cursorColor: 'rgba(248,28,229,0.8)',

    // terminal text color under BLOCK cursor
    cursorAccentColor: '#000',

    // `'BEAM'` for |, `'UNDERLINE'` for _, `'BLOCK'` for █
    cursorShape: 'BLOCK',

    // set to `true` (without backticks and without quotes) for blinking cursor
    cursorBlink: false,

    // color of the text
    foregroundColor: '#fff',

    // terminal background color
    // opacity is only supported on macOS
    backgroundColor: '#000',

    // terminal selection color
    selectionColor: 'rgba(248,28,229,0.3)',

    // border color (window, tabs)
    borderColor: '#333',

    // custom CSS to embed in the main window
    css: '',

    // custom CSS to embed in the terminal window
    termCSS: '',

    // if you're using a Linux setup which show native menus, set to false
    // default: `true` on Linux, `true` on Windows, ignored on macOS
    showHamburgerMenu: '',

    // set to `false` (without backticks and without quotes) if you want to hide the minimize, maximize and close buttons
    // additionally, set to `'left'` if you want them on the left, like in Ubuntu
    // default: `true` (without backticks and without quotes) on Windows and Linux, ignored on macOS
    showWindowControls: '',

    // custom padding (CSS format, i.e.: `top right bottom left`)
    padding: '12px 14px',

    // the full list. if you're going to provide the full color palette,
    // including the 6 x 6 color cubes and the grayscale map, just provide
    // an array here instead of a color map object
    colors: {
      black: '#000000',
      red: '#C51E14',
      green: '#1DC121',
      yellow: '#C7C329',
      blue: '#0A2FC4',
      magenta: '#C839C5',
      cyan: '#20C5C6',
      white: '#C7C7C7',
      lightBlack: '#686868',
      lightRed: '#FD6F6B',
      lightGreen: '#67F86F',
      lightYellow: '#FFFA72',
      lightBlue: '#6A76FB',
      lightMagenta: '#FD7CFC',
      lightCyan: '#68FDFE',
      lightWhite: '#FFFFFF',
    },

    // the shell to run when spawning a new session (i.e. /usr/local/bin/fish)
    // if left empty, your system's login shell will be used by default
    //
    // Windows
    // - Make sure to use a full path if the binary name doesn't work
    // - Remove `--login` in shellArgs
    //
    // Bash on Windows
    // - Example: `C:\\Windows\\System32\\bash.exe`
    //
    // PowerShell on Windows
    // - Example: `C:\\WINDOWS\\System32\\WindowsPowerShell\\v1.0\\powershell.exe`
    shell: 'C:\\Windows\\System32\\wsl.exe',

    // for setting shell arguments (i.e. for using interactive shellArgs: `['-i']`)
    // by default `['--login']` will be used
    shellArgs: [],

    // for environment variables
    env: {},

    // set to `false` for no bell
    bell: 'SOUND',

    // if `true` (without backticks and without quotes), selected text will automatically be copied to the clipboard
    copyOnSelect: false,

    // if `true` (without backticks and without quotes), hyper will be set as the default protocol client for SSH
    defaultSSHApp: true,

    // if `true` (without backticks and without quotes), on right click selected text will be copied or pasted if no
    // selection is present (`true` by default on Windows and disables the context menu feature)
    quickEdit: false,

    // choose either `'vertical'`, if you want the column mode when Option key is hold during selection (Default)
    // or `'force'`, if you want to force selection regardless of whether the terminal is in mouse events mode
    // (inside tmux or vim with mouse mode enabled for example).
    macOptionSelectionMode: 'vertical',

    // URL to custom bell
    // bellSoundURL: 'http://example.com/bell.mp3',

    // Whether to use the WebGL renderer. Set it to false to use canvas-based
    // rendering (slower, but supports transparent backgrounds)
    webGLRenderer: true,

    // for advanced config flags please refer to https://hyper.is/#cfg
  },

  // a list of plugins to fetch and install from npm
  // format: [@org/]project[#version]
  // examples:
  //   `hyperpower`
  //   `@company/project`
  //   `project#1.0.1`
  plugins: [],

  // in development, you can create a directory under
  // `~/.hyper_plugins/local/` and include it here
  // to load it and avoid it being `npm install`ed
  localPlugins: [],

  keymaps: {
    // Example
    // 'window:devtools': 'cmd+alt+o',
  },
};

My windows version is: 1909 (18363.1256).
When I add the --login arg, then it shows an error:
Invalid command line option: --login
Copyright (c) Microsoft Corporation. All rights reserved

So i guess somehow WSL is working but the default shell cannot be shown? Any help would be appreciated.

I just followed the steps in @leodutra comment and once I change the shell to wsl, it doesn't load anything inside Hyper, just a pink cursor is shown but no prompt is shown and I can't type.
This is what I see:

This is the config file:

// Future versions of Hyper may add additional config options,
// which will not automatically be merged into this file.
// See https://hyper.is#cfg for all currently supported options.

module.exports = {
  config: {
    // choose either `'stable'` for receiving highly polished,
    // or `'canary'` for less polished but more frequent updates
    updateChannel: 'stable',

    // default font size in pixels for all tabs
    fontSize: 12,

    // font family with optional fallbacks
    fontFamily: 'Menlo, "DejaVu Sans Mono", Consolas, "Lucida Console", monospace',

    // default font weight: 'normal' or 'bold'
    fontWeight: 'normal',

    // font weight for bold characters: 'normal' or 'bold'
    fontWeightBold: 'bold',

    // line height as a relative unit
    lineHeight: 1,

    // letter spacing as a relative unit
    letterSpacing: 0,

    // terminal cursor background color and opacity (hex, rgb, hsl, hsv, hwb or cmyk)
    cursorColor: 'rgba(248,28,229,0.8)',

    // terminal text color under BLOCK cursor
    cursorAccentColor: '#000',

    // `'BEAM'` for |, `'UNDERLINE'` for _, `'BLOCK'` for █
    cursorShape: 'BLOCK',

    // set to `true` (without backticks and without quotes) for blinking cursor
    cursorBlink: false,

    // color of the text
    foregroundColor: '#fff',

    // terminal background color
    // opacity is only supported on macOS
    backgroundColor: '#000',

    // terminal selection color
    selectionColor: 'rgba(248,28,229,0.3)',

    // border color (window, tabs)
    borderColor: '#333',

    // custom CSS to embed in the main window
    css: '',

    // custom CSS to embed in the terminal window
    termCSS: '',

    // if you're using a Linux setup which show native menus, set to false
    // default: `true` on Linux, `true` on Windows, ignored on macOS
    showHamburgerMenu: '',

    // set to `false` (without backticks and without quotes) if you want to hide the minimize, maximize and close buttons
    // additionally, set to `'left'` if you want them on the left, like in Ubuntu
    // default: `true` (without backticks and without quotes) on Windows and Linux, ignored on macOS
    showWindowControls: '',

    // custom padding (CSS format, i.e.: `top right bottom left`)
    padding: '12px 14px',

    // the full list. if you're going to provide the full color palette,
    // including the 6 x 6 color cubes and the grayscale map, just provide
    // an array here instead of a color map object
    colors: {
      black: '#000000',
      red: '#C51E14',
      green: '#1DC121',
      yellow: '#C7C329',
      blue: '#0A2FC4',
      magenta: '#C839C5',
      cyan: '#20C5C6',
      white: '#C7C7C7',
      lightBlack: '#686868',
      lightRed: '#FD6F6B',
      lightGreen: '#67F86F',
      lightYellow: '#FFFA72',
      lightBlue: '#6A76FB',
      lightMagenta: '#FD7CFC',
      lightCyan: '#68FDFE',
      lightWhite: '#FFFFFF',
    },

    // the shell to run when spawning a new session (i.e. /usr/local/bin/fish)
    // if left empty, your system's login shell will be used by default
    //
    // Windows
    // - Make sure to use a full path if the binary name doesn't work
    // - Remove `--login` in shellArgs
    //
    // Bash on Windows
    // - Example: `C:\\Windows\\System32\\bash.exe`
    //
    // PowerShell on Windows
    // - Example: `C:\\WINDOWS\\System32\\WindowsPowerShell\\v1.0\\powershell.exe`
    shell: 'C:\\Windows\\System32\\wsl.exe',

    // for setting shell arguments (i.e. for using interactive shellArgs: `['-i']`)
    // by default `['--login']` will be used
    shellArgs: [],

    // for environment variables
    env: {},

    // set to `false` for no bell
    bell: 'SOUND',

    // if `true` (without backticks and without quotes), selected text will automatically be copied to the clipboard
    copyOnSelect: false,

    // if `true` (without backticks and without quotes), hyper will be set as the default protocol client for SSH
    defaultSSHApp: true,

    // if `true` (without backticks and without quotes), on right click selected text will be copied or pasted if no
    // selection is present (`true` by default on Windows and disables the context menu feature)
    quickEdit: false,

    // choose either `'vertical'`, if you want the column mode when Option key is hold during selection (Default)
    // or `'force'`, if you want to force selection regardless of whether the terminal is in mouse events mode
    // (inside tmux or vim with mouse mode enabled for example).
    macOptionSelectionMode: 'vertical',

    // URL to custom bell
    // bellSoundURL: 'http://example.com/bell.mp3',

    // Whether to use the WebGL renderer. Set it to false to use canvas-based
    // rendering (slower, but supports transparent backgrounds)
    webGLRenderer: true,

    // for advanced config flags please refer to https://hyper.is/#cfg
  },

  // a list of plugins to fetch and install from npm
  // format: [@org/]project[#version]
  // examples:
  //   `hyperpower`
  //   `@company/project`
  //   `project#1.0.1`
  plugins: [],

  // in development, you can create a directory under
  // `~/.hyper_plugins/local/` and include it here
  // to load it and avoid it being `npm install`ed
  localPlugins: [],

  keymaps: {
    // Example
    // 'window:devtools': 'cmd+alt+o',
  },
};

My windows version is: 1909 (18363.1256).
When I add the --login arg, then it shows an error:
Invalid command line option: --login
Copyright (c) Microsoft Corporation. All rights reserved

So i guess somehow WSL is working but the default shell cannot be shown? Any help would be appreciated.


Modified bucket_table() to compute WoE consistent with category_encoder

#87
Hard-coded WoE calculation in bucket_table to be consistent with the category_encoder  (v2.5.1) implementation of WoE.
A direct call on the WOEEncoder object might be preferable, but seems inconvenient given the currrent code setup.

Hey @dlaprins thanks for the PR. Can you write an unit test to test you change ?


Audit log entry requirements for OBO tokens

We need requirements for how audit logs are impacted by OBO tokens

[Triage] @davidlago Can you follow up with the product team to see if we can get requirements?

Closing, to be added/tackled on https://github.com/opensearch-project/security/issues/3098 as its description/exit criteria.


Running time for each job

Is there a way to track the running time for each job? Like save it in job.time

Implement as a plugin?

Implemented at https://github.com/pwwang/pyppl_jobtime


Large files load indefinitely

Thank you for your feedback.
Current behaviour
Tried to test program using this 8GB scan: https://datadryad.org/stash/downloads/file_stream/223921 , let the page sit for ~40m while i was working on review, remained hung on 'uploading'

If this is a bug, please explain how to reproduce the problem

Which tools did you use in which sequence? - loading
Does the error concern the online display/ functionality or the downloaded result?  - display/functionality
What is the type and resolution of your data? (Can you provide a link to your data?) - https://datadryad.org/stash/downloads/file_stream/223921
Do not include sensitive information.
Upload the browser error message if you are familiar with web developer tools. - no message in console

Expected behaviour
would love to have a progress bar or some indication that the loading is still happening, or else an error message indicating that it failed!
Version information (for bug reports)

Browser + version: Firefox 81.0.1
Your OS + version: macOS 10.15.6

Thank you for the issue report @sneakers-the-rat!
We improved the feedback given when loading a file. We now display a message indicating that an uploading is going on. In the case of the file you were trying, it failed because it's extremely large for MRI standards. In case of error, we now display a message indicating to either try again or to report a bug. We also check for file size, and provide an indication in case the file is >200MB (which is rarely the case, most brain MRI files are <100MB).


[BEAM-1871] Remove another depedendency by moving TestCredential

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @kennknowles @tgroh

LGTM, great

Please follow up with changing the directory, too.

Yeah, was moving stuff out first, and then just doing package renames second

Changes Unknown when pulling 192c6b6a21820a92da0b9c560c7cd368588bd79b on lukecwik:thin_sdk_core into ** on apache:master**.


Detected 0 radio channels. To operate PX4, you need at least 5 channels

I have a pixhawk 3 PRO and have not been able to do RC radio calibration. I get this message "Detected 0 radio channels. To operate PX4, you need at least 5 channels" wondering if this issue has been solved or how it can be fixed? it looks like some kind of HW problem?. Any help will be appreciated thanks.

Hi @cesarengineer As per the reporting issue instructions best to discuss on the forums first. If it looks like a bug, you'll be directed to post an issue here.
It is distracting/time wasting to post here - effectively spam.
In addition, if this was to be posted here, you would need to provide a lot more information about your setup - you don't even say what receiver you are using or how it is connected up and powered. It isn't clear what you have tried already - do you know if the receiver and transmitter have been bound together?
I am closing this for now.


ShortcutCheatSheet.png for Godot Editor + Document Shortcuts?

I don't know of all the shortcuts and the information doesn't seem to be easily available. Perhaps an image showing Godot Editor UI + all the shortcuts listed and placed in docs somewhere would help out.
What do you think?

I did egrep on all the source files and manually went trough all output and produced this list. It may be wrong and also some shortcuts repeat  because each component set has it's own list.
line edit:
Cut = CMD + X
Copy = CMD + C
Paste = CMD + V
Select All = CMD + A
Undo = CMD + Z

text edit:
Cut =  CMD + X
Copy = CMD + C
Paste = CMD + V
Select All = CMD + A
Undo = CMD + Z

animation_editor:
Goto Next Step = CMD + Right
Goto Prev Step = CMD + Left

Open Scene = CMD + O
Save Scene = CMD + S
Save Scene As.. = Shift + CMD + S
Close Scene = Ctrl + W
Close Goto Prev. Scene = Shift + CMD + P
Quick Open Scene = Shift + CMD + O
Quick Open Script.. = ALT + CMD + O
Quick Search File.. = Alt + CMD + P
Undo = CMD + Z
Redo = CMD + Shift + Z
Run Script = Shift + CMD + R
Quit to Project List = Shift + Alt + Q
Quit to Poject List = Shift + Ctrl + Q
Quit = CMD + Q

Play custom scene = Shift + F5

Select Mode = Q
Move = Alt+Drag
Change Pivot = V
Drag Pivot = Shift + V
Depth List = Alt + Right Click

Expand to Parent = CMD + P
Make Bones = Shift + B
Center Selection = F
Frame Selection = CMD + F
Insert Key = Insert
Insert Key (Existing Tracks) = CMD + Insert
Clear Pose = Shift + K

Select Control Points = Shift + Drag
Add Point = CMD + Left Click
Delete Point = CMD + Right Click

Save = CMD + S
Save All = CMD + Shift + S
History Pev = Ctrl + Alt + Left
History Next = Ctrl + Alt + Right
Close = CMD + W
Undo = CMD = Z
Redo = CMD + Y
Cut = CMD + X
Copy = CMD + C
Paste = CMD + V
Select All = CMD + A
Move Up = Alt + Up
Move Down = Alt + Down
Indent Left = Alt + Left
Indent Right = Alt + Right
Toggle Comment =  CMD + K
Clone Down = CMD + B
Complete Symbol = Ctrl + Space
Complete Symbol = CMD + Space
Trimp Training Whitespace = Ctrl + Alt + T
Auto Indent = CMD + I
Find.. = CMD + F
Find Next.. = F3
Replace.. = CMD + R
Goto Function.. = Shift + CMD + F
Goto Line = CMD + L
Find.. = F
Find Next = F3
Toggle BReakpoint = F9
Step Over = F10
Step Into = F11
Close = CMD + W
Move Left = CMD + Left
Contextual = Shift + F1
Undo = CMD + Z
Redo = CMD + Shift + Z
Cut = CMD + X
Copy = CMD + C
Paste = CMD + V
Select All = A
Find.. = F
Find Next = F3
Replce = R
Goto Line = G
Pick Tile = CTRL
Select = CMD + B
Duplicate Selection = CMD + D
Erase Selection = Delete
New Scene Root = A
Add Child Node = A
Move Up = Up
Move Down = Down
Duplicate = D
Delete Node(s) = Delete
Add/Create a New Node = CMD + A

I opened a issue about this some time ago: godotengine/godot-docs#75

@vnen  I decided to organize the shortcuts  by UI component
Here is a cheat cheat I made quickly . May have issues.
Untitleddocument.pdf

hey guys since i made some tentative shortcut documentation myself i will close this

seems ALT+Left/Right don't work anymore on the 29.09.2016 build for Indent.

Use TAB and SHIFT+TAB for indent/dedent. Alt+← and Alt+→ are now the back/forward commands.

you can see all shortcuts now in the editor preferences . There's no need for a cheat sheet

Really? because, not knowing that I made all my way up to here to find out that information. That should be in the documentation page I think.

what does meta mean?

@nandbolt META = system key, depending on the system. for e.g. on windows it's the windows key, on macos it's the command key, on some linux distros it's known as the "super" key

For the record, default key mappings are now documented here: https://docs.godotengine.org/en/latest/tutorials/editor/default_key_mapping.html


How can I specify the rope scaling type when starting the API server?

I noticed that version 0.6.3 already supports yarn, but there doesn’t seem to be a parameter to specify the rope scaling type when starting with lmdeploy serve api_server.

we haven't released the api for setting rope scaling type yet.
It will be our honor if you'd like to pull an request to support this feature.


Use dry-types HEAD

This is required since rom-rb/rom-sql@4746e546dee81895e9c88c6fad44f047316730ff increased the minimum required dry-types version for rom-sql to one that hasn't been released yet.

Thanks :)


add shadcn table

This change is

I would have appreciated knowing this was coming. My PR already has this addition. I haven't looked at the details to know if it is the same as mine or conflicts in some way. Is there a need to add this in a separate PR? (Is some other task waiting on this?)

@tombogle I am sorry, this was not supposed to put any preasure on anybody. As it looks like we are basing different things on the table right now, I thought it would be helpful to have it in already, so we can fiddle around with solutions idependently. E.g. I wanted to use it to try different options of showing the warnings you made us aware of.

I had not seen all the discussion in Discord when I wrote that.


Switching to HTTPS does not work

It always asks to override SSL verification even if I do it.

In web browser, https works just fine.

There you have a handshake exception. The app cannot establish a secure connection with your server. Are you using a self signed certificate?

If you want I can give you an APK with some debugging traces to check what can be happening.

Yes, I am using a self-signed certificate signed by my own CA, the CA is however is present in the Android store and I have no problem in browsing through the web browser.
You can provide me the app with debug traces enabled, I can give it a try.

https://github.com/JGeek00/adguard-home-manager/releases/download/2.15.0-beta.4_(124)/app-release.apk
When you try to connect you will get an unknown error, but I will receive the detailed error on my error tracker.

My app store just updated the app with your new release 2.15.1 and it is working fine with HTTPS connections 👍 .
I have no issues now even without overriding SSL certificate validation.
I guess you already figured the issue even before I could try your beta version 🤪.

I really didn't changed anything on the login 🤷🏻
But great that now it works.


[Bug] Fail to create ingress due to the deprecation of the ingress.class annotation

Why are these changes needed?
This issue is not totally same as #441, and thus I opened a new issue. Users cannot create ingress with the instructions in https://ray-project.github.io/kuberay/guidance/ingress/. The operator also reports the error message:
2022-10-18T00:34:38.187Z	ERROR	controllers.RayCluster	Ingress create error!	{"Ingress.Error": "Ingress.extensions \"raycluster-ingress-head-ingress\" is invalid: annotations.kubernetes.io/ingress.class: Invalid value: \"nginx\": can not be set when the class field is also set", "error": "Ingress.extensions \"raycluster-ingress-head-ingress\" is invalid: annotations.kubernetes.io/ingress.class: Invalid value: \"nginx\": can not be set when the class field is also set"}
github.com/ray-project/kuberay/ray-operator/controllers/ray.(*RayClusterReconciler).reconcileIngress
	/workspace/controllers/ray/raycluster_controller.go:256
github.com/ray-project/kuberay/ray-operator/controllers/ray.(*RayClusterReconciler).rayClusterReconcile
	/workspace/controllers/ray/raycluster_controller.go:192
github.com/ray-project/kuberay/ray-operator/controllers/ray.(*RayClusterReconciler).Reconcile
	/workspace/controllers/ray/raycluster_controller.go:97
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile
	/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.1/pkg/internal/controller/controller.go:114
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
	/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.1/pkg/internal/controller/controller.go:311
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
	/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.1/pkg/internal/controller/controller.go:266
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
	/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.1/pkg/internal/controller/controller.go:227

As shown in the following code snippet, the error message is reported by the function ValidateIngressCreate (Link) in Kubernetes. When both annotations.kubernetes.io/ingress.class and spec.IngressClassName are set at the same time, the validation function will report the error "can not be set when the class field is also set". This PR only sets spec.IngressClassName and avoids setting annotations.kubernetes.io/ingress.class.
if annotationIsSet && ingress.Spec.IngressClassName != nil {
   annotationPath := field.NewPath("annotations").Child(annotationIngressClass)
   allErrs = append(allErrs, field.Invalid(annotationPath, annotationVal, "can not be set when the class field is also set"))
}

In doc, it indicates that annotations.kubernetes.io/ingress.class was deprecated in Kubernetes 1.18, and spec.ingressClassName is a replacement for this annotation.
Related issue number
Closes #645
#441

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[x] Unit tests
[x] Manual tests
[ ] This PR is not tested :(

cc @rokiyer @brucez-anyscale

technically, we don't like to support version < 1.19 since ingress api has been updated. We internally have similar issue

technically, we don't like to support version < 1.19 since ingress api has been updated. We internally have similar issue but make some customization. https://opensource.googleblog.com/2020/09/kubernetes-ingress-goes-ga.html
BTW, in public cloud, I don't think there're still 1.18 clusters?

Thank @Jeffwan for this information!
I am a little confused about your comments. This PR does not support Kubernetes < 1.19. Can you explain more about your concerns? Thank you!

Sorry for unclear comments. I mean the user should not use the deprecated annotation in earlier versions and this validation logic looks good to me. I was wondering why user still give the annotation on > 1.19 clusters? probably forget to clean them up after upgrade

Sorry for unclear comments. I mean the user should not use the deprecated annotation in earlier versions and this validation logic looks good to me. I was wondering why user still give the annotation on > 1.19 clusters? probably forget to clean them up after upgrade

Thank you for your explanation! In my opinion, some users followed the KubeRay doc to add the ingress.class annotation in their YAML files. To keep backward compatibility, I decided to use the annotation to set the value of spec.IngressClassName.
Honestly, I really agreed with you that the logic is weird. Do you have any thoughts on the following two methods?
(1) Keep backward compatibility with weird logic (deprecated annotation) => current logic
(2) Break backward compatibility with rational logic (Maybe we can have a new field IngressClassName in RayCluster CRD)

cc @rokiyer @brucez-anyscale

@kevin85421 Thanks for fast bugfix. I have tried the code changes and ingress can be created.
However, when I visit http://{domain}/cluster_name, "404 Not Found" shows up.
I have to make some changes in ingress before I can visit ray dashboard. Changes:

ingress path: change /cluster_name to /cluster_name(/|$)(.*)
ingress annotation: add new annotation, nginx.ingress.kubernetes.io/rewrite-target: /$2

(1) Keep backward compatibility with weird logic (deprecated annotation) => current logic
(2) Break backward compatibility with rational logic (Maybe we can have a new field IngressClassName in RayCluster CRD)

As @rokiyer said, user needs to give additional annotations to make Ingress work. for short term, I think current PR should be ok. for 2nd option, I suggest we use a specific annotation instead a new field to present the class name.

(1) Keep backward compatibility with weird logic (deprecated annotation) => current logic
(2) Break backward compatibility with rational logic (Maybe we can have a new field IngressClassName in RayCluster CRD)

As @rokiyer said, user needs to give additional annotations to make Ingress work. for short term, I think current PR should be ok. for 2nd option, I suggest we use a specific annotation instead a new field to present the class name.

Maybe I can open an issue to track the ingress support in KubeRay. We can discuss more details about ingress in next community sync meeting:
(1) All annotations in RayCluster will be passed to Ingress
(2) Is the exposed interface general enough for users to configure their custom ingress?
(3) Do we need to keep the backward compatibility?

TBH, I don't think it makes sense for the operator to have been built-in ingress support; details of ingress set-up are sensitive to cloud provider context and user requirements.
It is straightforward enough have users kubectl apply
ingress
---
raycluster

Some discussion of how to set up an ingress in the docs would be great.
My opinions:

(2) Is the exposed interface general enough for users to configure their custom ingress?

We don't need to prioritize this. Given that ingress support is already there, we should get things to a state where basic use-cases are covered. Then we can document the limitations of the built-in support and explain how to set up an ingress yourself.

(3) Do we need to keep the backward compatibility?

I also don't think we should stress about this -- just don't introduce any config structure incompatibilities (don't delete fields).
Either new field or a custom annotation seems fine to me. A custom annotation might be better initially since we're feeling pretty iffy about the ingress support in general at the moment. The only important thing is to clearly document the solution.

Thank @DmitriGekhtman for your reply!
I will write the documents about:
(1) Limitations of KubeRay's built-in ingress support: I still need to figure out the limitation. Currently, I just understand Nginx, and I will start to work on #637 to study the support of AWS LoadBalancer.
(2) Tell users how to set up ingress by themselves.
By the way, do we need to add "custom annotation" support in this PR? Thank you!

This PR is fine as is, I think.

By the way, do we need to add "custom annotation" support in this PR? Thank you!

This one looks good. We can have a separate issue/PR for further improvement.


release: v3.14.2

[v3.14.2] (Apr 18, 2024)
Features

Added outputFormat to the image compression options<SendbirdProvider
  ...
  imageCompression={{
    outputFormat: 'preserve' | 'png' | 'jpeg',
  }}
>
</SendbirdProvider>

Fixes

Removed image section in the OGMessageItemBody if there is no og image
Fixed that safely opens URL to prevent XSS
Fixed that copying URI-list issue in the iOS device/Safari
Fixed that channel badge count is not updated on iOS Webview

/bot create ticket

[Creating Ticket] 🔖 Done https://sendbird.atlassian.net/browse/SDKRLSD-1238


basic: fix more unused-but-set variables

Removed the num_printed variable and replaced asserts on status with error-checking

@bashbaug  can this be merged since it has two approvals. It is breaking release builds right now for us.

Ben, can we merge this, I'll push a follow up change.


addon-installer POC

#14
This patch proposes an API type and small library to install addons via exec'ing kubectl.
The API type addons.config.k8s.io/AddonInstallerConfiguration is used to list addons as
kustomize packages.
The intent is that installers like kops, kubeadm, eksctl, and etc. can make use of this library and API type.
Included is a commandline tool that reads this type from a configration file to apply these addons
to the currently configured cluster.
Care has been taken to support flag overrides of config values as well as signal passing.
Open work:

plumb kubectl extraArgs into the type/config ?
demonstrate unpacking a kustomize addon using docker and/or skopeo+umoci in a separate patch

usage
bin/installer --config demo/dupes.yaml --dry-run
bin/installer --config demo/v1alpha1.yaml

development
# fetch deps + regenerate all API's
make

# build just the binary from existing files
make only-build

This code has been adapted from https://github.com/luxas/sample-config and re-licensed to APL2 with kind permission from @luxas.

/assign

Additional questions:

how kubeadm can pass the kubeconfig for the cluster that we are creating
when dry running, kubeadm currently prints all the resources that will be generated, including also resources for addons. How can we do something similar with the installer?

:)
(***) == needs implementation

Are we expecting the user download addons/patch in advance?
Not normally. This is handled internally in kubectl by kustomize.

How do we think to address the air-gapped scenario?
kustomize supports git repos and local directories for transport.
The user can clone the official repos and copy them to the local filesystem and modify the AddonInstallerConfiguration to use them without a runtime network dependency.
We hope to add an OCI/docker image transport to kustomize which would allow mirroring similar the control-plane images (***).

How kubeadm can pass configurations to the addon-installer
(e.g. the expected kube-proxy version, the kube-proxy component config)
kubeadm can have a default AddonInstallerConfigration.
This can be overridden by adding an AddonInstallerConfiguration to the list of objects in the kubeadm config.
If a user wants to change what's installed, they can omit packages from the addons list or provide their own versions. kustomize allows you to patch and extend the addons.

How are upgrades are expected to work?
kubectl apply -k should upgrade the addon additively.
More complex upgrades can be handled by packaging the addon as an operator /w RBAC objects.
I have not looked at using the --prune kubectl flag, but an operator can handle deletes.

How can kubeadm control the kubeconfig?
This should probably be added to the installer Runtime struct or AddonInstallerConfiguration API (***).

@stealthybox what is trying to understand is: what is the value-added the addon installers/the addon project WRT to kustomize? how can the addon installers/the addon project simplify the addon lifecycle when used as a library by kubeadm?

@fabriziopandini

The CoreDNS, kube-dns, kube-proxy implementations are no longer hard-coded into the kubeadm binary.
These kustomize addons can be shared between multiple installers. (kops + eksctl have been following)
Users can configure kubeadm to omit certain networking components. (ex: https://kube-router.io cannot deploy /w kube-proxy)
Users can fork and extend the networking components. (can supply their own Corefile)
Users can configure their CNI provider to be installed /w kubeadm in a ComponentConfig/GitOps manner. (Calico, Weavenet, etc)
Opening up this config enables kubeadm to execute differing opinions of cluster bootstrap.
Core addons are no-longer a requirement -- just a default

/assign @timothysc
/assign @neolit123

gdoc: Brainstorming on integration between kubeadm and addons project

@stealthybox FYI there are new requirements around any APIs being added under the k8s.io domain, see https://github.com/kubernetes/enhancements/pull/1111
I think there will be less friction about iterating and experimenting with an add-ons API if it is developed under the x-k8s.io domain

I think there will be less friction about iterating and experimenting with an add-ons API if it is developed under the x-k8s.io domain

Thanks for the heads-up @jwforres -- it looks like that's the trend with certain clusterapi providers as well.
I don't think we intend for this API to be served be the APIServer with a CRD at the moment.
It's easy to start with this domain though :+1:

As discussed in the kubeadm planning session for 1.17, as soon as we get a KEP defining all the details/impacts on the users, we are going to start executing for integrating this with kubeadm.
Rif this document for the first brainstorming on use cases to be covered

The KEP is now up at https://github.com/kubernetes/enhancements/pull/1308

Can we go ahead and merge this PR after #28 is fixed and then iterate as necessary?

I've implemented the library-internal KUBECONFIG feature through a filePath.
Hopefully this is an acceptable interface for now.
The config API is now scoped under x-k8s.io -- thanks for pointing this out @jwforres. (Thanks @johnsonj for opening #28)
The library now has a map per-addon requiring a name and kustomizeRef/manifestRef.
It supports installing a single manifest from a URL or File as well as recursive directories via kubectl.
Lastly, I've added a delete mechanism; this is generally useful, and will be necessary for pruning addons based off of an "oldConfig".
I'd like to get this merged and work on these in followups:

prune logic
config validation tests
inline patches

Great work, Leigh!
Big :heavy_plus_sign: :one:  on landing this and filing followup issues to resolved later.

We discussed this in the Cluster Addons meeting yesterday and noted a couple of things which can be fixed in the next revision. I'll make sure to file issues for this. (Can we add a [kind/installer] label? I'll check.)
/lgtm

Can we add a [kind/installer] label? I'll check.
Nevermind... we already have 105 labels to choose from. I think I'm good.

Thanks @stealthybox for your work on this! This is a great foundation for the Cluster Addons project.


Add jquery to the dependency list?

Jquery isn't listed as a dependency, but it doesn't seem to function properly without it. So to save confusion it may be best to add jquery as a dependency in the bower.json and in the readme :). Great little module!

I am in the process of fixing some issues and upgrading the some features so I will add this to the list.  I will also test this with out having jquery installed to make sure that it works with out it.
Thanks

Added to the list and will be in the next release.  Thanks


A bug in the documentation page

This page has |8\rangle instead of |8>
Can I work on this?
Thank you

There is also a grammatical mistake of "od" in the third sentence. It should be "of" instead of "od"

There is also a grammatical mistake of "od" in the third sentence. It should be "of" instead of "od"

Even It was unnoticed for me.

Sure @mitul3737 you can fix that

@orsa-classiq , @mitul3737 - is this closed? Anyone working on it? Thanks


Have argoproj.io.ArgoCD to support resource customization in favor of per resource ConfigMap key

Is your feature request related to a problem? Please describe.
According to the ArgoCD CRD, it's currently using resourceCustomizations to define resource customization, and this will ultimately be applied to the argocd-cm ConfigMap using resource.customizations key. However, according to this post:

In the upcoming release, the resource.customizations key has been deprecated in favor of a separate ConfigMap key per resource. This allows to decompose the Argo CD configuration and makes it more convenient to use with config management tools such as Kustomize.

Describe the solution you'd like
With that, can we have argoproj.io.ArgoCD to support resource customization in favor of per resource ConfigMap key?
For those who are new to resource customization, it would be confusing to them since there are two formats. And from the latest official Argo CD documents, it has already been transformed to use the new format.

Any news on this?
Seems like an important departure from the ArgoCD configuration model if they ever deprecate the (now) old format.

@iam-veeramalla, is it still open to fix? I wd like to do it with pleasure .


Update README.md

Fixed a few typos
Added a "is maintained" badge
Added tiny explanations about the author! (just a link to your GItHub profile and contributors page)
Added and updated explanations about using offline, on smartphone, installing as a PWA etc

I added

Your data is secure, and this website does not use any cookie no_good_man cookie.
to be compliant with European laws about cookies...

In fact I use one cookie to remember the theme choosed :)


Failover configuration using PS and required permissions

Hi,
would be great if you could add the PS script showing how to add the failover configuration and information on what permissions are need on partner server in order to setup the failover partnership

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

ID: 1a614930-18ea-fffb-62df-77ce38bf1393
Version Independent ID: da1c1ecc-30f8-22f8-4ead-189574f823c7
Content: Deploy DHCP Using Windows PowerShell
Content Source: WindowsServerDocs/networking/technologies/dhcp/dhcp-deploy-wps.md
Product: windows-server
Technology: networking
GitHub Login: @eross-msft
Microsoft Alias: lizross

Looks like we need to enlist some assistance from the PowerShell Docs team.

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


fix safari cannot play ts file problem.

Bug:
MacOS safari can't play TS file.
steps to reproduce

clone this project https://github.com/xqq/mpegts.js, and build the project, (check the README).
npm install; npm install -g webpack-cli; npm run build;
run the webpack serve: npm run serve;
access page http://localhost:8080/arib.html in safari;
find/generate a test.ts file with audio codec as opus;
e.g. by ffmpeg: ffmpeg -i [file].[aac|opus] -acodec opus test.ts
start a static web server in the folder of test.ts,
e.g. by this python script: https://gist.github.com/mkows/cd2122f427ea722bf41aa169ef762001
python server-cors.py 9921
in page http://localhost:8080/arib.html, input Stream URL as: http://<IP_ADDRESS>:9921/test.ts, deselect isLive & withCredentials & liveSync checkbox. then click Load button.
the opus audio can not play in safari.

Root Cause
the Safari web browser don't support MIME audio/mp4;codecs=opus, but support audio/mp4;codecs=Opus.
In side safari web dev tools, go to Console tab. Input MediaSource.isTypeSupported('audio/mp4;codecs=opus') and MediaSource.isTypeSupported('audio/mp4;codecs=Opus') to verify.

Thank you!


Adding cerebral region segmentation function

Problem description:
In your MOOSE paper (DOI: 10.2967/jnumed.122.264063), it introduced that a model was trained to segment 83 brain subregions from PET data combined with T1-weighted MR images. However in this repository the only PET segmentation model provided is for tumors in FDG PET, and there is no sign of that cerebral region segmentation model.
Desired behavior:
The aforementioned model for segmenting 83 brain subregions is present.
Alternatives:
Or probably this model is already uploaded. Please point me to where it could be found.
Thank you very much for providing such a powerful image processing tool!

Hi @Samnonstop, many thanks for the message. It is true that moosev1, had the 83 brain segmentations from PET. We removed it in moosev2, due to less usage.
Anyway, we are happy to help you. Would you be kind enough to tell us if your input is just FDG PET brains?

Sure, thank you so much for the response. I have FDG PET/MR images as input. The intention was to segment the PET images using MOOSE and then transform the atlas to the MRI volumes.

Great, so it's just brain PET/MR right?

Yes. The FOV only contains the patient brain.

Hi @Samnonstop fantastic. We will include the moosev1 model in moosev2. But this will take a couple of days - would that be OK? We need to train it.
@mprires would you be kind enough to take the OG moosev1 training data and train it with no_mirroring trainer, since the brain has left and right sides?

Hi @Samnonstop fantastic. We will include the moosev1 model in moosev2. But this will take a couple of days - would that be OK? We need to train it.
@mprires would you be kind enough to take the OG moosev1 training data and train it with no_mirroring trainer, since the brain has left and right sides?

Thank you so much! Please take your time, since this is not an urgent need and you are volunteering to help out. Looking forward to trying the brain region segmentation tool.

Hi @Samnonstop I will get the training started today and will update you when it's available

Really appreciate your help and time.

Hi @Samnonstop training done, now testing 🙌🏽

Hi @Samnonstop your model is ready to use!

Hi @Samnonstop your model is ready to use!

Thank you so much for your kind help! You guys are so efficient. By the way, how could I download the new model?

Hi @Samnonstop your model is ready to use!

Thank you so much for your kind help! You guys are so efficient. By the way, how could I download the new model?

No worries. I just replaced the old code files with your updated ones and the clin_pt_fdg_brain_v1 model is now downloading. Will close this issue once it is tested on my pc.

Hi @Samnonstop your model is ready to use!

Thank you so much for your kind help! You guys are so efficient. By the way, how could I download the new model?

No worries. I just replaced the old code files with your updated ones and the clin_pt_fdg_brain_v1 model is now downloading. Will close this issue once it is tested on my pc.

The new model successfully segmented PET brain images in my environment and the results look promising. Thank you a lot for your great support as well as providing such a powerful segmentation tool! I'm closing this issue now.

Hi @Samnonstop many thanks for your kind words. Happy it worked out well. Leave us a star if you find MOOSE useful!


fix: allow file names and directory names to contain brackets

Summary
fix #8056

Checklist

[x] Tests updated (or not required).
[ ] Documentation updated (or not required).

Thank you !


Adding a --prompt flag

This will prompt the user for the ldap password vs requiring them to put it on the cli, potentially exposing it to cli logging

@edepree @emperorcow

@ahhh I apologize for the slow response. Thanks for the pull request, I appreciate the contribution!


Add raster from service

Description
PR to add a new Kotlin sample "Add raster from service" in Layers category. This sample loads a raster from a service, then applies a function to it.
Links and Data
Sample Epic: runtime/kotlin/issues/1941
What To Review

Review the code to make sure it is easy to follow like other samples on Android
README.md and README.metadata.json files

Thanks @prupani-7, added in the feedback.

Good catch @prupani-7, renamed the sample and applied feedback. 👍


iOS 12.2 screen with contacts opens and closes immediately

After upgrading to iOS 12.2 on some devices contacts crashes (Screen opens and closes immediately). Using the latest version of contacts. What can be the problem?

this is commonly a problem with how you installed the module. Please doublecheck the instructions. If that doesn't work try downgrading the iOS version and see if that fixes it.

@morenoh149 The library is set up correctly. This problem occurs in rare case only on iOS 12.2. Can it be somehow connected with lib core?

anything is possible. This sounds hard to debug though. Anything you can share to help debug it I'd appreciate!

@morenoh149 The issue was solved by upgrading from RN 0.57.7 to 0.59.4. Thanks for your reply!


Knex Playground

knex-playground integrated into Knex's official docs

Desktop
Mobile

[!WARNING]
I currently need help on making light mode work with the editor

Any code feedback is also welcome, I have almost zero knowledge with Vue

I've fixed the PR to support dark mode + light mode

coverage: 92.843%. remained the same
when pulling 98f760e4897fef90765373f143ed6e668a9e0bdf on dgadelha:master
into 9659a20753e22939ffb547e3b10d5bfa6a3fd941 on knex:master.


have problem when resume model

Hi, thank you excellent work. i want to run llava-v1-80k with lora. The model can be trained normally.
however, when i resume the checkpoints from work_dir (work_dir/checkpoint-xx), i meeting some error.
I found that my ssh to cluster was killed.

Also, i found that Memory usage exceeds upper limit

So, how to solve this problem, how to correctly resume model.
thank you very much.

Hi, may I ask if there is any update over the checkpoint resuming issue?

Hi, may I ask if there is any update over the checkpoint resuming issue?

i find that it is the cup memory exceed the limitation. I solve this problem by only using 6 gpu (CUDA_VISIBLE_DEVICE=0,1,2,3,4,5) in a 8gpu machine.


Is Blazor.ServerSideStyled obsolete?

I updated a project today and this line stopped compiling. Is it no longer needed?
@(await Html.RenderComponentAsync<BlazorStyled.ServerSideStyled>(RenderMode.ServerPrerendered))
Thanks, I didn't see a new features for 2.0/2.1 versions, do you have that posted anywhere?
Thanks,
Corby

Yes it is :-)
You can safely delete that line. BlazorStyled only needs  services.AddBlazorStyled(); from 2.0 and up.

That’s correct no longer needed.
I am on mobile right so hard to list here but if you check the releases link on the main github page and go to 2.0.0 you should see it. The main new thing is a whole new underpinning of the project so you no longer need that tag in the head.
Another thing is kind of minor but helpful, when you compose classes it will compose class names instead of the styles, and also allows you to put rules in the Styled being composed where before you couldn’t.

I took it out and it compiled. Good to know, easier is always better out of the box.
Thanks

I am going to add a section to the doc sites with the release notes over the weekend, so hopefully going forward this will be mitigated.
Closing this issue since I think everyone is up and working now.


Document possible cause of memory growth

If you create a producer with delivery reports but don't check the reports memory grows.
>>> with topic.get_producer(delivery_reports=True) as producer:
...     count = 0
...     while True:
...         count += 1
...         producer.produce('test msg', partition_key='{}'.format(count))

This has had me stumped looking for a memory leak for a couple of days and although is obvious to me now it could be mentioned in the readme and possibly save someone else the time in the future.

Thanks @jmwri, I'll add a note.


Fix a problem with missing types for HotTable and HotColumn.

Context
Because the addUnsafePrefixes function returned a class with no type defined, all of the types for HotTable and HotColumn were missing.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)

Related issue(s):

#144

Closing in favor of #146


Consider chunking the string in Rope::from(String)

Currently, when we create a new Rope from a String, we put the whole string in one Leaf node. That means that indexing-based operations will initially have roughly the same time overhead as indexing a String, and we won't see performance improvements until the Rope has been appended or inserted into enough to turn it into a nice tree structure. We might want to consider having the Rope::from<String> implementation split the string into chunks so that we can reap Rope performance benefits immediately.

One way to do this that springs initially to mind is to create an unbalanced branch as the root node, rather than a leaf, and then call .rebalance() on it. If we were to, say, put the whole string in the left child of a branch and then rebalance that node immediately, we'd end up with a nice, balanced tree using our already existing .rebalance() method.
Of course, this may not be the best way to do this, but it was the obvious way, to me.

@cjm00, what do you think?


os/exec: context becomes done, but Cmd.Run() still blocks for a long time

What version of Go are you using (go version)?

$ go version
go version go1.19.2 linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/ubuntu/.cache/go-build"
GOENV="/home/ubuntu/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/ubuntu/.gvm/pkgsets/go1.19.2/global/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/ubuntu/.gvm/pkgsets/go1.19.2/global"
GOPRIVATE=""
GOPROXY="https://goproxy.woa.com,direct"
GOROOT="/home/ubuntu/.gvm/gos/go1.19.2"
GOSUMDB="sum.woa.com+643d7a06+Ac5f5VOC4N8NUXdmhbm8pZSXIWfhek5JSmWdWrq7pLX4"
GOTMPDIR=""
GOTOOLDIR="/home/ubuntu/.gvm/gos/go1.19.2/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.19.2"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build3184007767=/tmp/go-build -gno-record-gcc-switches"

What did you do?

golang-code: https://go.dev/play/p/YZ7rxFZihNv
main.sh:
#!/bin/bash
sleep 10000

please chmod +x ./main.sh

What did you expect to see?
It is expected that after one second, cmd.Run() exits and returns the error that the process is Killed.
What did you see instead?
cmd.Run() blocks forever and cannot return.

Duplicate of #23019

#50436 proposes a solution to this that does not break backward compatibility.


Bug | The Dates for Opt out should start post the dates as entered in application for reschedule

During application filing for reschedule the litigant is asked the date post which they are available.
During Opt out the list of dates suggested should be post that date.

Hi @prasanna2400  @shambhavigoel1002  Please add the development tasks and estimated effort details to resolve this ticket

Hi @rajeshcherukumalli  Please test in Dev environment

Hi @Ramu-kandimalla , this issue is fixed in UAT environement, please find the reference
https://jam.dev/c/f7fad115-f53d-4975-b681-aef73dae9c18

Hi Team this issue is fixed


Added AssertInstanceOf overloads with out parameter

This adds Assert.IsInstanceOfType overloads with an out parameter as discussed in #2702
fixes #2702

@microsoft-github-policy-service agree


Many requests required to get all feedback for student

Hello all.
In order to get all feedback for a particular student, I need to retreive every single assessment.
I have a school of 400 students, and we're currently at 9000 assessments that have a due date in 2023.
9000 api calls to get the marks for a student??? It seems like this would be a common use case for the api.
I could cut down my calls by a factor of 20 if I could get assessments per student/learner

Hey @xkem
If you're just after assessment feedback, you might be able to use the /learning/evidenceFeed/user/{id} and filter by filterNames => [feedback].


1.0.0

I think it is time to merge the EMMO-python 1.0.0 into master even though we haven't released EMMO v1.0.0 yet. The reason is that master is very old and outdated. We could release it as version 1.0.0-alpha-1.

We can leave master in this miserable state. I merge now...


Backport connection handling to Rails 2

Hello, I've backported connection handling for ActiveRecord.
I was getting Bad file descriptor if ActiveRecord::Base.connection was used in plugins or initializers before DJ was started. I've fixed it by clearing ActiveRecord connections in before_fork hook. Branch v2.1 already contains fix, so I'm backporting this trivial fix it for legacy apps.

@bryckbost ping :)

I know I was the one who originally urged the creation of the branch but I don't want to maintain 2.0 anymore :-\


Provide labels at issue templates?

How would this feature be useful?

Currently, when a user creates an issue, it doesn't has labels. Maybe we can alleviate the maintainer's work by automating the labels addition.
Describe the solution you'd like

Add labels to the issue templates: bug at the bug template and enhancement at the feature request, maybe?
Describe alternatives you've considered

If we keep as is, it's ok for me too :+1:

I think this could be done (plus some more nice improvements) with the new GitHub issue yaml forms https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-issue-forms.
I've got them set up on most of my projects and they work great! See some examples here. I also have it set up with release drafter and an auto-labeller which actually thinking about it I'm not 100% sure I need anymore if the issue template sets labels

I think this could be done (plus some more nice improvements) with the new GitHub issue yaml forms

I think a switch to these new templates is a great idea too!


4.1 contains breaking changes

Moving from 4.0 to 4.1 caused a number of issues related to the switch from Popsicle to Axios. Some of these have been fixed, but there's no documentation related to the upgrade path. For example, the getJson function is now not available, which could've been invoked by client code that used the agent.makeApiCall function.
It also seems as if thrown errors do not include an error object, which makes it really hard to troubleshoot what may have failed when performing a request. For example, trying to use the /payment endpoints is failing after the upgrade, but I can't inspect the error because apparently the parameter is now not included.
Overall, it seems as if 4.1 should've been 5.0 since it includes breaking changes that requires changes to the client codebase.
Is it possible to add some migration documentation between 4.0 and 4.1?

Hello @amadeogallardo , thank you for reaching out.
Migration documentation Moving from 4.0 to 4.1:

Upgraded from Popsicle to Axios lib

JSON.stringify(authResponse) ==> should now been changed to  JSON.stringify(authResponse.json)

Axios is not a drop-in replacement for Popsicle. Here are a few of the API differences:
Request config:
Axios uses data instead of body.
Axios.get() will override the the method option.
There is no transport option.
Response schema:
Headers are turned into an object with lower cased names.
Response data is atomically parsed and can be accessed with res.data.
The response can't be stringified using JSON.stringify() because of res.request.

Breaking Change
Given the transport issue, this Migration does introduce a breaking change.
e.g. Users that receive a PDF will have to change their code from this:
oauthClient.makeApiCall({
url: ${url}v3/company/${companyID}/invoice/${invoiceNumber}/pdf?minorversion=59,
headers:{'Content-Type': 'application/pdf','Accept':'application/pdf'},
transport: popsicle.createTransport({type: 'buffer'})
})
To this:
oauthClient.makeApiCall({
url: ${url}v3/company/${companyID}/invoice/${invoiceNumber}/pdf?minorversion=59,
headers:{'Content-Type': 'application/pdf','Accept':'application/pdf'},
responseType: 'arraybuffer'
})
@amadeogallardo May I request you to validate the list of breaking changes that requires changes to the client codebase?

getJson function is now not available, while making the makeApiCall function.
thrown errors do not include an error object. For example, trying to use the /payment endpoints is failing after the upgrade, but this error can't be inspected  because apparently the parameter is now not included.

@amadeogallardo also, a new release branch has been created from the master:
https://github.com/intuit/oauth-jsclient/tree/release-4.1.2
Please feel free to raise a PR over there to include the migration documentation. Thanks

Has 4.1.2 been released? It doesn't show up under https://www.npmjs.com/package/intuit-oauth?activeTab=versions

Here's one of the things that I struggled with: getting the error message/code. Before, it was under this path:
authResponse.json.Fault.Error

Now it's under this path:
response.data.Fault.Error

Hi @amadeogallardo , Release 4.1.2 has been pushed out, please checkout at https://github.com/intuit/oauth-jsclient/releases
closing this issue. please feel free to reopen this if you still see any issues. Thanks


Module Not Found

Everytimre I import (import * as faceapi from "face-api.js") in my react project it gives throws me error like this.
ailed to parse source map from 'D:\Thesis\facial-recognition-v1\facial-recognition\node_modules\face-api.js\src\xception\extractParamsFromWeigthMap.ts' file: Error: ENOENT: no such file or directory, open 'D:\Thesis\facial-recognition-v1\facial-recognition\node_modules\face-api.js\src\xception\extractParamsFromWeigthMap.ts'
@ ./node_modules/face-api.js/build/es6/xception/TinyXception.js 9:0-74 99:11-37
@ ./node_modules/face-api.js/build/es6/ageGenderNet/AgeGenderNet.js 5:0-56 19:33-45
@ ./node_modules/face-api.js/build/es6/ageGenderNet/index.js 1:0-31 1:0-31
@ ./node_modules/face-api.js/build/es6/index.js 5:0-37 5:0-37
@ ./src/App.js 8:0-39
@ ./src/index.js 7:0-24 11:33-36
I dont know what to do please help me

You are using this for browser, this is node API not e6 javascript

@miladganjali not sure I follow, does it mean we should not use this library in the browser or should we ignore those warnings?

@keogh What I did is downgrade my web pack to 4. something and it works fine. Now I am attempting to use it to my server instead of puting it into client side (react js). I'm also confused on how to use it because it has an browser option and node option but I guess I will be using the API in the server side.

@kyahAndoy I see what you mean, and yes, after I finished my prototype I thought it was better to have all this in the server side rather than on the browser
Thank you

seems the "node-modules/face-api.js/src" folder is missing all together from the NPM package "face-api.js" would be nice to include it in this package? oterwise needs to copy manually the src folder


Script to Enable/Disable DNSCrypt on OSX

Is there a command line equivalent to the "Enable DNSCrypt" and "Disable DNSCrypt" MenuBar item?
I'd like to write a script to disable DNSCrypt when using Tunnelblick to establish a VPN connection using OpenVPN (and enable DNSCrypt when Tunnelblick disconnects).  The general idea is posted in the topic "Force DNS Setting While DNSCrypt is Enabled?" on the Tunnelblick forum:
https://groups.google.com/forum/#!forum/tunnelblick-discuss
A suggested approach is clear the DNS service of the active device using something like this:
networksetup -setdnsservers $networkService ""
Another suggested approach is:
https://support.opendns.com/entries/25477860
Can you recommend a way to script enabling/disabling DNSCrypt on OSX?
Thank you.

nello

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/35545339-script-to-enable-disable-dnscrypt-on-osx?utm_campaign=plugin&utm_content=tracker%2F417785&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F417785&utm_medium=issues&utm_source=github).

Looks like this is more of a question about the dnscrypt OSX Client, but might know the answer anyway :)
In order to enable it, create a file (it can be empty - the content is ignored) named /Library/Application Support/DNSCrypt/control/dnscrypt.
In order to disable it, delete the file.
If you look at the OSX Client scripts in /Library/PreferencePanes/DNSCrypt.prefPane/Contents/Resources/usr/scripts, this is exactly what the switch-to-dnscrypt.sh and switch-to-dhcp.sh scripts do.

Yes, you are correct: the question is about the OSX client.
Thank you for your suggestion.


Fix indent

Coverage decreased (-6.4%) to 93.257% when pulling f3abb562eb781b07abf186ec6c32c641539f2a3c on kachick:fix-indent into ea98e05853a1dcbe0a5d615ba6ee498852b0ee38 on mongomapper:master.

Thank you for your contributions! Merged in 284d976e1001fa844c209dacfb9c67cbf2ff1911


Autocomplete does not appear if I write SQL using table synonyms with Oracle 12 database

System information:

Operating system (distribution) and version: Windows  10 Pro 20H2
DBeaver version

Additional extensions: Office integration

Connection specification:

Database name and version

Driver name: Oracle (JDBC)
Do you use tunnels or proxies (SSH, SOCKS, etc)? NetScaler Access Gateway

Describe the problem you're observing:
On SQL Editor:

If I use the table name directly on "FROM" clause, SQL Autosuggestion appears as expected
If I use the table synonym on "FROM" clause, the SQL Autosuggestion does not appear anymore

Steps to reproduce, if exist:

Create a sample table:
CREATE TABLE "BIDTMS"."BID_CALENDAR1"
(
"CALENDAR_DATE" NUMBER(14,0),
"CALENDAR_KEY" DATE
)SEGMENT CREATION IMMEDIATE
PCTFREE 10 PCTUSED 40 INITRANS 1 MAXTRANS 255
NOCOMPRESS LOGGING
STORAGE(INITIAL 65536 NEXT 1048576 MINEXTENTS 1 MAXEXTENTS<PHONE_NUMBER>
PCTINCREASE 0 FREELISTS 1 FREELIST GROUPS 1
BUFFER_POOL DEFAULT FLASH_CACHE DEFAULT CELL_FLASH_CACHE DEFAULT)
TABLESPACE "BIDTMSDAT" ;

Create the Synonym:

On SQL editor:
SELECT f.[hit ctrl + space => SQL suggestion appears] FROM table_name;
SELECT f.[hit ctrl + space => Nothing happens] FROM synonym_name;

thanks for the report

Verified

I tried to build Dbeaver from devel branch, in order to check if it works.
Unfortunately, the problem is still there, exactly the same behavior.

Hello @MatteoR23
As I can see on your images, you use schema identifier with synonym name.
do you use another default schema in SQL Editor?
It works for me without schema identifier for default schema:

and for another schema, if global search is enabled

Effectively, I try to access from another schema.
Anyway, even if I check that setting, the problem is not resolved.
Here some screenshots from my devel version of DbEaver (I cloned the git repo yesterday after 6 PM CEST tz):

@Matvey16 could you please reopen this bug? The problem is not resolved.

Verified

Hello @MatteoR23
In scope with this ticket https://github.com/dbeaver/dbeaver/issues/11327 we added new connection setting for Oracle.
Enabling this setting will help with solving the problem of auto-completion with fully qualified names.


camelCase CSS properties

strokeWidth, fillOpacity, fontSize are camelCased.
backgroundOpacity, while technically not a css property, follows the convention, as well as the various leg_xxxXxxx
closes #6

thank you very much @Fil. It's great


Detailed day 3 schedule

Day 3

08:50 - 09:00 Soft start and icebreaker question
09:00 - 09:15 Recap Git, any Hackmd questions to highlight
09:15 - 09:30 Concepts around collaboration (Sabry leads)

Explain terms. Pull, push, clone, fork. Focus on pull and not fetch.
Focus more on Clone and less on Generating from templates and importing

09:30 - 11:10 Centralized workflow + a break (Radovan leads)

9:30 - 9:45 : Explain concepts.
9:45 - 9:55 break
9:55 - 10:00 Inform clearly what is expected outcome.
10:00 - 10:40 Excercise
10:40 - 11:00 Instructors go through the exercise
11:00 - 11:10 - break

11:10 - 12:10 Distributed version control and forking workflow + one breake (Sabry leads)

11:10 - 11:25 - Concepts and what are exercise outcomes
11:25 - 11:45 - Exercise
11:45 - 12:00 - Instructors go through excercises
12:00 - 12:10 - break

12:10 - 12:30 - How to contribute changes to somebody else’s project and Q&A (Radovan leads)

First exercise: 30 mins instead of 40
Give more time for second exercise, especially updating forks (this should be part of exercise)


Make card list centered

Currently the list is not centered, as you can see here. Just fix this :)

I'll try this one

Fixed in #26


Lots of Lag when running Zoom (OK otherwise) - USB CABLE

On Windows 10 with the latest updates (same for zoom app) and scrcpy 2.4.
In zoom it can be many seconds behind.  It should detect this (obviously not zoom specific) and drop frames to keep up.  I expect what is shown in the app to be as close to realtime as possible.
Command Line:
..\scrcpy.exe --verbosity debug --show-touches --window-title "[ANDROID PHONE] - STARTED: 16:55:01.17 on Tue 19/03/2024" --disable-screensaver --stay-awake --prefer-text --push-target "/sdcard/Download/" --window-x -1918 --window-y 35 --window-x 5 --window-y 60

SOME DEBUG:
Dependencies (compiled / linked):
 - SDL: 2.28.5 / 2.28.5
 - libavcodec: 60.31.102 / 60.31.102
 - libavformat: 60.16.100 / 60.16.100
 - libavutil: 58.29.100 / 58.29.100
 - libusb: - / 1.0.27
...
[server] INFO: Device: [samsung] samsung SM-S918B (Android 14)
...
[server] DEBUG: Using audio encoder: 'c2.android.opus.encoder'
[server] DEBUG: Using video encoder: 'c2.qti.avc.encoder'
[server] DEBUG: Display: using SurfaceControl API
INFO: Renderer: direct3d
DEBUG: Trilinear filtering disabled (not an OpenGL renderer)

I can't see any command line switch for video frames.  How many we collect per second or any to do with dropping, just buffering which I don't think would make any difference.

When you run zoom on your computer, there is lag in scrcpy?

In zoom it can be many seconds behind. It should detect this (obviously not zoom specific)

:thinking:
Is it over USB? What is the result with --print-fps?

Yes it is USB, here are some example fps:
INFO: 69 fps
INFO: 137 fps
INFO: 139 fps
DEBUG: [Audio] Buffer underflow, inserting silence: 107 samples
INFO: 147 fps
INFO: 96 fps
INFO: 112 fps
INFO: 81 fps
INFO: 130 fps
INFO: 97 fps
INFO: 87 fps
INFO: 117 fps
INFO: 92 fps
INFO: 95 fps
INFO: 59 fps
INFO: 70 fps
INFO: 90 fps
INFO: 155 fps
INFO: 150 fps
INFO: 70 fps

That was in zoom with an empty meeting (apart from me) and sharing my screen with my phone idle. I'm guessing it actually happens when more people connect and so might be related to network traffic...
I will try this again during my next meeting.
Q. Audio underflow?  there probably wasn't any audio so is the underflow a problem?  I don't seem to get that message when playing a youtube video, but it looks like it does skip frames...
INFO: 90 fps
INFO: 89 fps (+12 frames skipped)
INFO: 105 fps (+1 frames skipped)
INFO: 111 fps
INFO: 113 fps

Everything looks correct in the logs. Probably an issue related to sharing the graphical resources on your system.

Audio underflow? there probably wasn't any audio so is the underflow a problem?

No, that's expected sometimes. If you don't want to forward audio, use --no-audio.

My shared screen contains scrcpy, when I was in an actual meeting the issue occurred again.  This time I noticed that while scrcpy was lagging so was my phone (just not as much until the phone actually hung).  The  --print-fps output didn't show much difference from the above.
Then someone in my meeting told me that they couldn't see my screen and eventually that Zoom was indicating that I had low bandwidth (even though it wasn't telling me that!)...  I now think that this is a zoom issue, where it wants to send every frame and somehow holds up updates and presumably hangs scrcpy and even my phone itself...  I stopped transmitting my webcam video and it improved a lot.


external-workloads: Run cilium-cli inside a container

Update externalworklads.yaml to run cilium-cli inside a container instead of using cilium-cli-test-job-chart. Update docker run command to specify --user flag so that install-external-workload.sh gets created with correct permissions.
Ref: #2623
Ref: #2627
Ref: cilium/design-cfps#9

✅ success: https://github.com/cilium/cilium-cli/actions/runs/9671104454/job/26681073692?pr=2630
marking it ready for review


Constantly getting "Supermaven is running" notification

I constantly get a notification that says "Supermaven Pro is running".

Usually I get it when I try to use Supermaven after I leave the editor for a few minutes. And sometimes when I go to a new file or a new position in the same file. But I can't reliably reproduce it.

Fixed by https://github.com/supermaven-inc/supermaven-nvim/pull/21


Feature/z-card

Feature - z-card

Priority

[ ] 1 - Highest
[ ] 2 - High
[x] 3 - Medium
[ ] 4 - Low
[ ] 5 - Lowest
[ ] 6 - Not a Priority

Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Feature (non-breaking change which adds functionality)
[x] Component (add a Component as approved dy Design System)
[ ] Docs (add documentation)
[ ] Chore (changes that adds small enhancement)
[ ] Breaking (fix or feature that would cause existing functionality to not work as expected)

Abstract
https://app.abstract.com/projects/fd370780-e659-11e8-99dc-0d08537c5fde/branches/e696391e-bd14-4345-a98c-78343d192340/commits/5078c674ea888e961e09ac28741f206cb8506111/files/9f586aa0-bd5b-4c12-9541-aed2e98a7a9c/layers/61FBCDE5-1C49-442B-A262-EA3E73FA352F?collectionId=12a021f0-42e6-4e22-b64a-df001c3d6917&collectionLayerId=9986ef95-6efb-4e31-ae1f-c037d0f16b9d&mode=build&selected=root-45C828A6-52D2-4722-8891-AEEAA2C6E021
Screenshots

Component and Fix approval flow to Master branch
A Pull Request to be merged must have two approvals, one from a member of the product team who developed it and another from a member of the dst dev team other than the team representative.

There are some things missing in my opinion:

More tests to cover the different component variants
Examples of the different component variants in the src/index.html file
It would be interesting , if there is time to implement an "onCardClicked" event e2e test


Training logs of SST+CenterHead

Hello. Thank you for your great work! I would appreciate it if you can provide the training logs of the CenterHead version of your model.

Sorry for late reply, here is the training log
sst_center_full24e.log
Let me know if you have any further questions.

Thank you!!


Search and replace through all files in a Spyder project

Search and replace exists for the file open in the editor pane, but it would be nice to be able to search and replace throughout all files in the current Spyder project.

Is there anyone working on that feature? I think the main problem is that Spyder does not know the concept of a project folder.

It is a really important feature that is very very basic for an IDE...


API:getTargetAtClientPoint() doesn't work until the editor to be created after a while

i need some help!!!!here is the case:
1.i call getTargetAtClientPoint immediately after  i create the editor ,but the API seems to output a wrong column
2.if i call the api after some seconds,it would works fine

It is possible the editor is not yet painted on the DOM. you can force a paint via editor.render()


ios 无法还原包

[!] 'AliAuthSDK' uses the unencrypted 'http' protocol to transfer the Pod. Please be sure you're in a safe network with only trusted hosts. Otherwise, please reach out to the library author to notify them of this security issue.
注释 #use_frameworks!  后，
其他flutter 插件无法还原
例如fluwx

你好，你的注释以后可以解决问题吗？


secrets folder not exported

Describe your bug here.
the 0.5.2 have the new secrets but in the export file it not appears and when i write 'shadow','bb','river' or 'shubs' on the tittle screen it not happend anything,what happed?
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

it's not supposed to

on Project.xml

 on Project.xml

Thx


fix(google): repeat #539 for JSON

Saving Google JSON keys requires an authenticated user identity, like OAuth.
Also, because we save secret vars twice, make sure their values are not changed after the first save.
Refs: ENG-1470

plz (before merge) check whether we could ensure that finalize is receiving secret vars as key:value and not as key: uuid ref to secret storage


Headings hierarchy h2 & h4

At the moment the heading h2 and the heading h4 are both bold. This shouldn't be the case.

Bitte abklären, ob alle mager oder fett, allenfalls Thomas Suremann, BK, konsultieren.

The size of H2, H3, H4 and "Normaler Lauftext" is in general very similar and as Frontenduser you don't see realy the difference. In the new attachement which I upload today you see how small is the diference. So the size of H2 and H3 should be adapted, that it can distinguished well from H1 and normaler Lauftext" and also between H2 and H3. Also H4 should be distinguished, but it ist used much less then H2 and H3 and so it's less important to make a difference. I think also it would be more logic if all are in bild or not in bold.

SizeH1-H4.docx


Clean up DataflowPipeline[Debug]Options

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

getUpdate should be isUpdate, to meet standard JavaBeans style.
Remove deprecated update property from DataflowPipelineDebugOptions

R: @davorbonaci

LGTM.
R: @lcwik for a second look on getX vs. isX with PipelineOptions.

LGTM
is and get are the same for boolean properties as per javabeans


demo is not working

demo app is running but tabbar buttons are not functional and views are white!

the problem is in JDNavigationBar class
layer.insertSublayer(shadowLayer, below: self.topItem?.titleView?.layer)
is the problem

Yes updated the code ... you can check now


Fix Server-Side Request Forgery vulnerability

📊 Metadata *
systeminformation package is vulnerable to Server-side request forgery. It allows attackers to abuse of @ to make requests to a different domain or possibility to applications that are not publicly exposed through http://effectrenan.com@<IP_ADDRESS>:8080.
Bounty URL: https://www.huntr.dev/bounties/4-npm-systeminformation
⚙ Description *
Sanitization of @.
💻 Technical Description *
The fix adds @ in sanitizeShellString used in inetChecksite and inetLatency functions.
🐛 Proof of Concept (PoC) *
const si = require('systeminformation');
si.inetChecksite("https://effectrenan.com@google.com").then(a => { console.log(a) });

🔥 Proof of Fix (PoF) *
As a request to google.com returns a redirect to www.google.com, it is possible to check the status code. Thus, if it is 301, the redirect happened and it shows that the server made a request to google.com instead of effectrenan.com.
const si = require('systeminformation');
si.inetChecksite("https://effectrenan.com@google.com").then(a => {
    if(a.status == 301)
        console.log("It is vulnerable!");
    else
        console.log("It is not vulnerable!");

    console.log(a);
});

👍 User Acceptance Testing (UAT)
const si = require('systeminformation');
si.inetChecksite("https://effectrenan.com@google.com").then(a => console.log(a));
si.inetChecksite("https://effectrenan.com").then(a => console.log(a));

🔗 Relates to...
Provide the URL of the PR for the disclosure that this fix relates to.

👋 Hello, @sebhildebrandt - @EffectRenan has opened a PR to us with a fix for a potential vulnerability in your repository. To view the vulnerability, please refer to the bounty URL in the first comment, above.
Ultimately, you get to decide if the fix is 👍 or 👎. If you are happy with the fix, please write a new comment (@huntr-helper - LGTM) and we will open a PR to your repository with the fix. All remaining PRs for this vulnerability will be automatically closed.
If you have any questions or need support, come and join us on our community Discord!
@sebhildebrandt & @EffectRenan - thank you for your efforts in securing the world’s open source code! 🎉

🔨 Want more security researchers protecting your repository?
Stick our badge on your README.md, and let security researchers know that they can win bounties protecting your repositories. Sounds cool, huh? 😎
Copy this small code snippet and insert it into your README.md:
[![huntr](https://cdn.huntr.dev/huntr_security_badge.svg)](https://huntr.dev)
👇 👇 👇

Hello @sebhildebrandt,
If the fix is not enough, I would like to suggest you accept this and make changes later (if possible). It helps me in the rewarding process in the platform. Thanks!

@EffectRenan not sure, if this is considered to be a vulnerability. Having a look at your example and If a user can provide an URL, that he can anyway try both URLs separately so https://effectrenan.com as well as google.com can be called, same with your second example ... so just sanitizing the URL this way will not really help. What do you think?

@sebhildebrandt effectrenan.com and google.com is just an example to show the fix working. The real scenario is about a local application that is not publicly exposed.
Instead of the application has a sanitizer for requests from "http://<IP_ADDRESS>", this vulnerability allows bypass it through "@". Therefore, this is necessary to avoid the applications need to know how the function inetChecksite is implemented.
Reference: https://portswigger.net/web-security/ssrf

@EffectRenan yes, I understand this, but I can pass http//:<IP_ADDRESS> directly to inetChecksite() which works quite fine currently - and sometimes this is wanted. So I do not see how your fix prevents from calling the local resource.

@sebhildebrandt I agree that sometimes this is wanted. The reference shows some bypasses, but most of them are not applied in the context of this package.
The point is: if an application needs to block the access of some URL (<IP_ADDRESS>, localhost, ...), it will only need a blacklist. And the character @ doesn't make sense to be accepted in inetChecksite().

@EffectRenan thank you for your explanation and thoughts. Always appreciated. I will then merge it but not consider this to be a security issue for now. Can you push the pull request to my repo?
Once again, I highly appreciate your effort to make this a more secure package!! Thank you.

@sebhildebrandt Thank you for your attention and fast response! I really appreciate this and it makes me invest more time to make this package more secure.

@huntr-helper - LGTM

@huntr-helper - LGTM

@EffectRenan I merged the request. Just one more thing: can you make sure, that next time we will only have the necessary changes in the pull request? I had changes In /lib/poc.js in the pull request. This file is not part of the repo. As fas as I see this is your "proof of concept" file. As this makes sense, I have to think about how to add an entry point for POCs to my repo ... but currently there is no file like that ;-)
Thank you once again.

@sebhildebrandt Ohh I'm so sorry, this wasn't supposed to happen. I will make sure the next time.
Thank you!

@EffectRenan ;-) no problem ...


Support for namespaces config when starting child workflows and activities.

If the application works on a specific namespace the child workflows and activities will go back to 'default'

When calling \Temporal\Client\WorkflowClient::class it doesn't consider namespaces


Create config system for changing message strings

Need to able to support self-referencing other keys in the same config file inside strings as well as Mustache templating (optional).
Currently testing out HOCON style for self-referencing for suitability

working in branch https://github.com/Eluinhost/UHC/tree/features/message-editing
moving to full release and hope to release by sunday

Moving to https://github.com/Eluinhost/UHC/pull/52