Add ability to extend the set of Git pre-commit hooks

This is very useful for forks of Godot with custom needs.

@Faless suggested on chat to add misc/hooks/pre-commit-custom-* to .gitignore.
In theory it shouldn't be needed as the custom hooks have to be placed in .git/hooks/ and so they don't need to be in misc/hooks/. But I personally tend to make my .git/hooks a symlink to misc/hooks so that hooks update automatically, so for my use case I'd actually want to put custom hooks in misc/hooks/.

I'm quite a noob in what comes to Git hooks. How does Git pick additional hooks normally without modifying the list of known pre-commit hooks in the main pre-commit script?

I'm quite a noob in what comes to Git hooks. How does Git pick additional hooks normally without modifying the list of known pre-commit hooks in the main pre-commit script?

What I do is symlink the whole misc/hooks/ folder to .git/hooks/. So if a new pre-commit hook is added upstream, and included in misc/hooks/pre-commit, then I get it automatically via the symlink.

Oh, I wasn't getting it right. It's clear now.

Thanks!

Cherry-picked for 3.6.

Cherry-picked for 3.5.1.


feat(firebase): added support to firebase

Description of change
Add support to Firebase Functions by creating a Handler.

Pull-Request Checklist

[x] Code is up-to-date with the main branch
[x] npm run lint passes with this change
[x] npm run test passes with this change
[x] This pull request links relevant issues as Fixes #0000
[x] There are new or updated unit tests validating the change
[x] Added documentation inside www/docs/main folder.
[x] Included new files inside index.doc.ts.
[x] The new commits follow conventions outlined in the conventional commit spec

Codecov Report

Merging #34 (2f4dbcf) into main (efe4894) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #34   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           38        39    +1
  Lines          960       966    +6
  Branches       227       227
=========================================
+ Hits           960       966    +6

Impacted Files
Coverage Δ

src/handlers/firebase/http-firebase.handler.ts
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Fix heading levels and spelling

Always place a subsection under a section for a proper layout and to avoid warnings.

The changes are added manually in #12. the difference between subsection and section is not done. See the comments in file for clarification.


Microchip Xec watchdog driver

Adding watchdog driver for MEC1501 SoC

@franciscomunoz can you take another look?

Why is an alternate register structure required?  If the HAL does not have defines are for all the bits in each register please let me know and we will do an update.

Since @tbursztyka is on vacation, I am getting this through review now. New PR is at #18984 .

https://github.com/zephyrproject-rtos/zephyr/pull/18984 has merged. Closing this...


⚠ Bofrost.at has degraded performance

In 83ed90a, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1847 ms

Resolved: Bofrost.at performance has improved in a430194.


Standard Library: INI, YML/YAML, XML

This really should be 3 issues instead of 1. Also worthwhile to consider TOML instead of INI due to being pretty similar but actually having a formal spec.

INI is too common of a format. There is going to be versions of it we can't support because of the lack of specification — there's some really cursed ones out there — but I'm fine with TOML alongside it.
This is fine as one issue too, it's all solutions solving one major problem: the lack of data format parsers.

If you're fine with not being able to support "all versions of it", then I think you'd be fine with adding TOML instead of INI. We could document it as "TOML/INI" so users will know what's up.

Upon further investigation, it seems TOML is not a superset of INI as I initially thought. Of course, we'd still have the issue of many INI dialects existing in the wild, but I hope some configurable options can address that.


add utility methods for asserting tags

Issue #, if available: https://github.com/aws-controllers-k8s/community/issues/1368
Description of changes:

This change adds some methods to bring consistency on tag assertions in various controller e2e tests
This change also adds useful methods for asserting ACK system tags

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


.tmux.conf being ignored

Hi hi,
so far I've been using your awesome config for several month on my mac. But I can't use it on fedora, it seems to be ignored and if I try to manually load it from within a tmux (tmux source-file ~/.tmux.conf) window I get the following feedback:
'cut -c3- ~/.tmux.conf | sh -s _apply_configuration' returned 2
hope yoou got a hint...

Hello @mircsicz 👋
Can you please launch tmux and then from a page run
$ cut -c3- ~/.tmux.conf | sh -sx _apply_configuration

And paste the output here?

@gpakosz thx for taking the time
`
mircsicz@M720q:~$ cut -c3- ~/.tmux.conf | sh -sx _apply_configuration

:
set -e
unset GREP_OPTIONS
export LC_NUMERIC=C
LC_NUMERIC=C
sed -E s///
printf ''
++ uname -s
_uname_s=Linux
++ tmux -V
++ awk '{gsub(/[^0-9.]/, "", $2); print ($2+0) * 100}'
_tmux_version=320
command -v pkill
sh: Zeile 750: !\s+: event not found
sh: Zeile 756: !\bssh\b: event not found
sh: Zeile 761: Syntaxfehler beim unerwarteten Symbol »(«
`

So far I have no idea.

Can you please make sure you're running the most recent commit of `.tmux.conf?
Can you please make sure you didn't modify .tmux.conf?
Can you please tell me which version of Fedora you're running, and which shell?

So far I have no idea.

Can you please make sure you're running the most recent commit of `.tmux.conf?

mircsicz@M720q:~/.tmux$ git pull
remote: Enumerating objects: 21, done.
remote: Counting objects: 100% (17/17), done.
remote: Compressing objects: 100% (7/7), done.
remote: Total 21 (delta 12), reused 10 (delta 10), pack-reused 4
Entpacke Objekte: 100% (21/21), 22.04 KiB | 1.47 MiB/s, fertig.
Von https://github.com/gpakosz/.tmux
   9a2387c..18a111f  master     -> origin/master
Aktualisiere 9a2387c..18a111f
Fast-forward
 .tmux.conf       | 121 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-----------------------------------------
 .tmux.conf.local |  25 +++++++++++++++++++------
 2 files changed, 99 insertions(+), 47 deletions(-)
mircsicz@M720q:~/.tmux$ cd -
/home/mircsicz
mircsicz@M720q:~$ cut -c3- ~/.tmux.conf | sh -sx _apply_configuration
+ :
+ set -e
+ unset GREP_OPTIONS
+ export LC_NUMERIC=C
+ LC_NUMERIC=C
+ sed -E s///
+ printf ''
++ uname -s
+ _uname_s=Linux
++ tmux -V
++ awk '{gsub(/[^0-9.]/, "", $2); print ($2+0) * 100}'
+ _tmux_version=320
+ command -v pkill
sh: Zeile 774: !\s+: event not found
sh: Zeile 781: !\bssh\b: event not found
sh: Zeile 785: Syntaxfehler beim unerwarteten Symbol »(«

Can you please make sure you didn't modify .tmux.conf?

see my "git pull"

Can you please tell me which version of Fedora you're running, and which shell?

mircsicz@M720q ~ $  cat /etc/redhat-release
Fedora release 36 (Thirty Six)
mircsicz@M720q ~ $  bash --version
GNU bash, Version 5.1.16(1)-release (x86_64-redhat-linux-gnu)

Which version of perl do you have?

mircsicz@M720q ~ $  perl -v
This is perl 5, version 34, subversion 1 (v5.34.1) built for x86_64-linux-thread-multi

I can't reproduce your issue so far
$ vagrant init generic/fedora36
$ vagrant up
$ vagrant ssh
$ cd
$ sudo dnf -y install tmux
$ git clone https://github.com/gpakosz/.tmux.git
$ ln -s .tmux/.tmux.conf
$ tmux

I've two physical machines and one vm all quite fresh setup, all with same config'S deployed by ansible where it hits:
Screenshot

Can you share that ansible conf?

Sure, just installing vagrant to reproduce your test... Gi'me a sec

There u go:
---
- name: Copy dotfiles and/or configs to the host(s)
  hosts: "{{machines}}"
  gather_facts: false
  become: true
  vars:
    tmpldir: ~/ansible/templates/mircsicz/

  tasks:
    - set_fact:
      remote_user: "{{remoteuser}}"
    - name: Add nopasswd entry for {{remoteuser}} to sudoers.d
      ansible.builtin.template:
        src: '{{ tmpldir }}/sudoers.d-mircsicz.j2'
        dest: '/etc/sudoers.d/{{remoteuser}}'
        validate: /usr/sbin/visudo -cf %s
      tags:
        - sudo

    - name: Copy dotfiles
      ansible.builtin.copy:
        src: "{{ item.src }}"
        dest: "{{ item.dst }}"
        owner: "{{ remoteuser }}"
        group: "{{ remoteuser }}"
      with_items:
        - { src: '{{ tmpldir }}.tmux', dst: '/home/{{ remoteuser }}/' }
        - { src: '{{ tmpldir }}.tmuxinator', dst: '/home/{{ remoteuser }}/' }
        - { src: '{{ tmpldir }}.vimrc', dst: '/home/{{ remoteuser }}/' }
       - { src: '{{ tmpldir }}.bash_aliases', dst: '/home/{{ remoteuser }}/' }
        - { src: '{{ tmpldir }}.bash_profile', dst: '/home/{{ remoteuser }}/' }
        - { src: '{{ tmpldir }}.profile', dst: '/home/{{ remoteuser }}/' }
        - { src: '{{ tmpldir }}.config', dst: '/home/{{ remoteuser }}/' }
      tags:
        - files
    - name: Create symlinks
      ansible.builtin.file:
        src: "{{ item.src }}"
        dest: "{{ item.dst }}"
        state: link
      with_items:
        - { src: '/home/{{ remoteuser }}/.tmux/.tmux.conf', dst: '/home/{{ remoteuser }}/.tmux.conf' }
        - { src: '/home/{{ remoteuser }}/.tmux/.tmux.conf.local', dst: '/home/{{ remoteuser }}/.tmux.conf.local' }
      tags:
        - files

- name: Install basic Apps
  ansible.builtin.import_playbook: play_user-apps.yml
  tags:
    - apps

which is run like this:
ansible-playbook -i hosts_my_inv.yml myplay.yml --extra-vars "machines=myhostgroup remoteuser=mircsicz --ask-become-pass --tags=files,apps -vv

Do you have set +H or histchars=! in .bash_profile or /etc/profile?

Doesn't look like:
mircsicz@fedora ~ $  grep 'set -H|histchars=!' .bash_profile
mircsicz@fedora ~ $  grep 'set -H|histchars=!' .profile
mircsicz@fedora ~ $  grep 'histchars=!' .profile
mircsicz@fedora ~ $  grep 'histchars=!' .bash_profile
mircsicz@fedora ~ $  grep 'set -H' .profile
mircsicz@fedora ~ $  grep 'set -H' .bash_profile
mircsicz@fedora ~ $  grep 'hist' .bashrc
# don't put duplicate lines or lines starting with space in the history.
# append to the history file, don't overwrite it
shopt -s histappend

Or
set -o history
set -o histexpand

Can you please share the content of your .bash_profile, .profile and .bashrc?

Or
set -o history
set -o histexpand

none of those:
mircsicz@fedora ~ $  grep 'hist' .*
.bashrc:# don't put duplicate lines or lines starting with space in the history.
.bashrc:# append to the history file, don't overwrite it
.bashrc:shopt -s histappend
.bashrc:# for setting history length see HISTSIZE and HISTFILESIZE in bash(1)

Can you please share the content of your .bash_profile, .profile and .bashrc?

.profile
echo 'EXECUTING ~/.profile'

# if running bash
if [ -n "$BASH_VERSION" ]; then
    # include .bashrc if it exists
    if [ -f "$HOME/.bashrc" ]; then
        . "$HOME/.bashrc"
    fi
fi

# set PATH so it includes user's private bin if it exists
if [[ -L "$HOME/bin" || -d "$HOME/bin" ]] ; then
        echo 'Adding ~/bin to PATH'
        PATH="$HOME/bin:$PATH"
else
        echo '~/bin does not exist'
fi

.bash_profile:
export SHELLOPTS

# if running bash
if [ -n "$BASH_VERSION" ]; then
    # include .bashrc if it exists
    if [ -f "$HOME/.bashrc" ]; then
        . "$HOME/.bashrc"
    fi
fi

and finally .bashrc:
# If not running interactively, don't do anything
case $- in
    *i*) ;;
      *) return;;
esac

echo 'EXECUTING ~/.bashrc'

HISTCONTROL=ignoreboth

# append to the history file, don't overwrite it
shopt -s histappend

HISTSIZE=1000
HISTFILESIZE=2000

shopt -s checkwinsize

#
# Exports
#
export EDITOR="vim"

#
# OS Detection
#
case "$OSTYPE" in
  solaris*) echo "SOLARIS" ;;
  darwin*)  echo "Welcome to bash on MacOS"; export CLICOLOR=YES; export LS_OPTIONS=''; export LSCOLORS="Gxfxcxdxbxegedabagacad";;
  linux*)   echo "Welcome to bash on LINUX"; export CLICOLOR=YES; export LS_OPTIONS='--color=auto -h';;
  bsd*)     echo "Welcome to bash on BSD"; export CLICOLOR=YES; export LSCOLORS="Gxfxcxdxbxegedabagacad"; export LS_OPTIONS='--color=auto -h';;
  freebsd*) echo "Welcome to bash on FreeBSD"; export CLICOLOR=YES; export LSCOLORS="Gxfxcxdxbxegedabagacad"; export LS_OPTIONS='--color=auto -h';;
  msys*)    echo "WINDOWS" ;;
  *)        echo "unknown: $OSTYPE" ;;
esac

#
# SSH Autocompletion
#
if [ $OSTYPE == darwin* ]; then
	if [ -f $(brew --prefix)/etc/bash_completion ]; then
		. $(brew --prefix)/etc/bash_completion
	fi
fi

# set variable identifying the chroot you work in (used in the prompt below)
if [ -z "${debian_chroot:-}" ] && [ -r /etc/debian_chroot ]; then
    debian_chroot=$(cat /etc/debian_chroot)
fi

# set a fancy prompt (non-color, unless we know we "want" color)
case "$TERM" in
    xterm-color|*-256color) color_prompt=yes;;
esac

if [ -n "$force_color_prompt" ]; then
    if [ -x /usr/bin/tput ] && tput setaf 1 >&/dev/null; then
	# We have color support; assume it's compliant with Ecma-48
	# (ISO/IEC-6429). (Lack of such support is extremely rare, and such
	# a case would tend to support setf rather than setaf.)
	color_prompt=yes
    else
	color_prompt=
    fi
fi

if [ "$color_prompt" = yes ]; then
	PS1='${debian_chroot:+($debian_chroot)}\[\033[38;5;209m\]\u\[$(tput sgr0)\]\[\033[38;5;3m\]@\[$(tput sgr0)\]\[\033[38;5;33m\]\h\[$(tput sgr0)\]\[\033[38;5;15m\] \[$(tput sgr0)\]\[\033[38;5;3m\]\w\[$(tput sgr0)\]\[\033[38;5;15m\] \[$(tput sgr0)\]\[\033[38;5;163m\]\\$\[$(tput sgr0)\]\[\033[38;5;15m\] \[$(tput sgr0)\] '
else
    PS1='${debian_chroot:+($debian_chroot)}\u@\h:\w\$ '
fi
unset color_prompt force_color_prompt

# If this is an xterm set the title to user@host:dir
case "$TERM" in
xterm*|rxvt*)
    PS1="\[\e]0;${debian_chroot:+($debian_chroot)}\u@\h: \w\a\]$PS1"
    ;;
*)
    ;;
esac

# enable color support of ls and also add handy aliases
if [ -x /usr/bin/dircolors ]; then
    test -r ~/.dircolors && eval "$(dircolors -b ~/.dircolors)" || eval "$(dircolors -b)"
    alias ls='ls --color=auto'
fi

if [ -f ~/.bash_aliases ]; then
    . ~/.bash_aliases
fi

# enable programmable completion features (you don't need to enable
# this, if it's already enabled in /etc/bash.bashrc and /etc/profile
# sources /etc/bash.bashrc).
if ! shopt -oq posix; then
  if [ -f /usr/share/bash-completion/bash_completion ]; then
    . /usr/share/bash-completion/bash_completion
  elif [ -f /etc/bash_completion ]; then
    . /etc/bash_completion
  fi
fi

The problem is caused by export SHELLOPTS

Arggghhh

Can you please try the gh-582 branch?

Gi'me a sec just testing without, ansible is currently deploying it...

Can you please try the gh-582 branch?

Awesome, after rolling back to my  .bash_profile with "export SHELLOPTS" and a 'git switch gh-582' I can finally start tmux and see your config <3

Thanks I will merge the gh-582 branch which will close this issue.
Not sure export SHELLOPTS is a good idea though 🙂


[Q] How do you set LNB power mode for Transport stream capture

Hi,
I am currently reading about using tsduck to capture a transport stream.
I understand I will have to use tsp command, but I can't see examples on how to set the LNB power to ON or 13v,18v.
The usb tuner I have is : TSB5520SE and the device appears on the system (Linux)

Hi @xfiles2007
You do not need to think in terms of LNB voltage. The user only wants to receive a satellite stream, knowing the signal characteristics, frequency, polarity, modulation, etc. The dvb input plugin will take care of the rest.
Specifically, if the signal polarity is vertical, the plugin will set the LNB power to 13V. And if the polarity is horizontal, the LNB power will be set to 18V. You can see the corresponding Linux-specific code here.
So, if you want a satellite stream as input, just use the dvb input plugin, specify all the characteristics of the signal (RTFM for more details) and don't care about LNB power.

Ok great - good to know!
Thanks :)


feat: Add Content-Type to custom encoder/decoder

Description
This PR adds two fields to Router called encoders and decoders.
These contain a String to BodyEncoderGenerator/BodyDecoderGenerator dictionary.
We have also added a Read(as: Codable, decoder: BodyDecoder) function, which decodes the data from the routerRequest into a codable type using the given Decoder.
The Codable router uses the decoders dictionary to generate a decoder for the Content-type header and then use the new Read function to decode the request using that decoder.
The Codable router uses the Accepts header to select the best encoder from the Encoders dictionary and use that encoder to encode it's response.
Travis pass requirements
This requires a the Query Coders to be able to encode/decoder data which is in this (PR](https://github.com/IBM-Swift/KituraContracts/pull/28)
It also requires the addition of the BodyEncoder/BodyDecoder protocol which is currently being worked on this branch
Motivation and Context
This allows users to customize the JSON encoder and decoder in Codable routes and add their own Encoders/Decoders for encoding/decoding different Content-Type headers.
This is a fix for issue #1298
This follows on from the work in pull request 1221 but is generalized for any custom encoder/decoder
How Has This Been Tested?
This change affects all Codable and Type-Safe routes so tests have been added which use a custom JSONEncoder. The test suite has also been updated to allow custom encoders/decoders to be used.
Checklist:

[x] If applicable, I have updated the documentation accordingly.
[x] If applicable, I have added tests to cover my changes.

The original changes regressed performance quite badly (11%) for a simple Hello World which did not require a decoder/encoder.  Commit 29d83a2 delays the evaluation of the output encoder until we need one (which resolves this regression).
We found a further regression when the request contains an Accept header, as we went through some fairly complex Accept header processing even in the case where we only have a single encoder available.  The fix in 3f6b9e9 avoids processing the Accept header at all when we only have one encoder (which is the default case), and tries to reduce the cost of constructing a MediaType when a Content-Type header is present.
Perf results:
Hello World (no Content-Type / Accept headers)
               | Throughput (req/s)      | CPU (%) | Mem (kb)     | Latency (ms)                   | good
Implementation | Average    | Max        | Average | Avg peak RSS | Average  | 99%      | Max      | iters
---------------|------------|------------|---------|--------------|----------|----------|----------|-------
          Base |    68332.1 |    70271.5 |    97.9 |        20454 |      1.8 |      5.7 |    202.4 |    20
       e1ec693 |    60803.6 |    65512.3 |    98.7 |        22978 |      2.1 |      5.0 |     35.1 |    20
       29d83a2 |    68129.5 |    70385.6 |    97.8 |        24440 |      1.9 |      6.5 |    203.6 |    20

Codable GET (no Content-Type / Accept headers)
               | Throughput (req/s)      | CPU (%) | Mem (kb)     | Latency (ms)                   | good
Implementation | Average    | Max        | Average | Avg peak RSS | Average  | 99%      | Max      | iters
---------------|------------|------------|---------|--------------|----------|----------|----------|-------
          Base |    32700.2 |    33353.0 |    97.9 |        28159 |      3.9 |      4.6 |     28.7 |    20
       e1ec693 |    32046.6 |    32795.3 |    98.5 |        28972 |      3.9 |      5.1 |    201.3 |    20
       29d83a2 |    32258.2 |    32690.4 |    98.6 |        27474 |      3.9 |      4.6 |     26.2 |    20

Codable POST (no Accept header)
               | Throughput (req/s)      | CPU (%) | Mem (kb)     | Latency (ms)                   | good
Implementation | Average    | Max        | Average | Avg peak RSS | Average  | 99%      | Max      | iters
---------------|------------|------------|---------|--------------|----------|----------|----------|-------
          Base |    29489.9 |    29893.9 |    98.4 |        28787 |      4.3 |      6.6 |     30.5 |    20
       e1ec693 |    26609.5 |    27048.2 |    98.0 |        29326 |      4.8 |      6.1 |     32.5 |    20
       29d83a2 |    26635.6 |    27114.8 |    97.8 |        27570 |      4.8 |      6.5 |     30.8 |    20
       3f6b9e9 |    27226.8 |    27655.3 |    97.5 |        22007 |      4.7 |      6.3 |     29.8 |    10

Codable GET (with Accept header)
               | Throughput (req/s)      | CPU (%) | Mem (kb)     | Latency (ms)                   | good
Implementation | Average    | Max        | Average | Avg peak RSS | Average  | 99%      | Max      | iters
---------------|------------|------------|---------|--------------|----------|----------|----------|-------
          Base |    31769.9 |    32195.2 |    98.1 |        28869 |      4.0 |      6.7 |    199.6 |    20
       e1ec693 |    18288.3 |    18688.2 |    99.0 |        29073 |      7.0 |      8.4 |    208.9 |    20
       29d83a2 |    18004.3 |    18400.5 |    99.2 |        28592 |      7.1 |      8.5 |     36.2 |    20
       3f6b9e9 |    31125.3 |    31452.8 |    98.0 |        27566 |      4.1 |      6.9 |     28.1 |    10

Codable POST (with Accept header)
               | Throughput (req/s)      | CPU (%) | Mem (kb)     | Latency (ms)                   | good
Implementation | Average    | Max        | Average | Avg peak RSS | Average  | 99%      | Max      | iters
---------------|------------|------------|---------|--------------|----------|----------|----------|-------
          Base |    28530.1 |    29070.6 |    98.1 |        29092 |      4.4 |      8.8 |     28.7 |    10
       e1ec693 |    16620.7 |    16883.0 |    99.3 |        29714 |      7.7 |      9.1 |     43.3 |    10
       29d83a2 |    16250.5 |    16571.0 |    99.3 |        29056 |      7.8 |      9.3 |     35.9 |    10
       3f6b9e9 |    26858.6 |    27115.1 |    97.7 |        19545 |      4.7 |      8.8 |     26.5 |    10

We are left with a 6-8% regression for a Codable POST (regardless of whether there is an Accept header), and a smaller ~2% regression for a Codable GET.
There may be other opportunities to reduce regression which we haven't explored yet.


Add concept of a Layout node (or similar), like a Mapping but adding no semantics

I have some forms where it doesn't make sense for me to create deep nestings of Mappings, when the actual data for the form is relatively flat. At the moment I'm using Mappings primarily to group fields together for presentational, rather than functional reasons, which means I'm doing extra work to transform data from the mostly flat format my application uses, to the format that react-forms uses.
It would be useful to be able to have field groupings that don't actual alter the data schema. I don't know what the API could be, since the new key-value approach to schema definition means that it's not really possible to have a node that doesn't have a name (at the moment I'm giving them dumb names like 'group1', 'group2' etc).
Alternatively, it might be worth investigating some patterns for defining the layout of a form independently of the data definition.
I've attached a screenshot of the form I'm trying to define.

I'm trying to do something similar with http://kumailht.com/gridforms/
How did it work out and can you provide an full example ?


Name attributes

Closing this because it's already been fixed by 3a73aca. I hope to make a new release in an hour or so.


16-bit depth not supported in PNG file

i get this error message "16-bit depth not supported in PNG file" when trying to use a png file
any idea how should i fix this?
thank you
the image is created using golang png encoder.

i found out that i can just turn this image to greyscale and resolve this issue.
use github.com/harrydb/go/img/grayscale


Add more columns in the list view

If possible, it would be nice for the list view to show more than only the file name, like the last modified data, and/or created date, the file size, etc.
Need to look into the FileInfo API that we have to see what is reasonable to display.

Was done in #32


Release 2.0.0 Not Tagged and Not Published on GitHub

Summary
Release v2.0.0 is available in the Play Store, but that release is missing from Github releases.
Steps to reproduce
Open Github releases page for the project.
Expected behavior
The current latest release on Github is 2.0.0 along with a 2.0.0 tag.
Observed behavior (that appears unintentional)
The current latest release on Github is 1.55.1 and there there are no newer git tags.
Possible fixes
The solution can doing a release on GitHub for 2.0.0 and adding this step to your internal release procedures to make sure it doesn't happen again.

@francisco-core V2 has not been released yet. If you see it in the play store is cause you subscribed as a beta tester.

Ah, I see. Sorry for the confusion. I guess I saw the release date on dec. 15th in some mattermost communication material somewhere and that lead me to conclude it was already released. But you're right, on the play store I was checking for the beta version.
Good luck on the release!


Menu has no children style error

Describe the bug （描述 Bug）
When Menu has no children after change theme style error like

A clear and concise description of what the bug is.
To Reproduce （重现步骤）
Steps to reproduce the behavior:

set router without children and change theme then will see the menu which has no children does not show correct
cause

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior（你期待的是什么?）
A clear and concise description of what you expected to happen.
Thank you for the great component and fix this issue
Screenshots（截图）
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context（附加信息）
Add any other context about the problem here.

also in light mode exist style issue,


web-demo getting 8005 error (User not login error)

Issue: Keep showing "User not login" error toast while try to make contract call, check in log it's 8005 error
Environment: localhost:3000 (by running yarn start:web-demo)
Side notes

I can do the same contract call with same parameters on the official site https://web-demo.particle.network/ using the same account
I've setup .env file properly from Particle dashboard data

https://github.com/Particle-Network/particle-web-demo/assets/26215730/b7548b14-4a15-4b4e-8512-fa00581cb4f7

This issue has resolved after pulling the latest code from master and upgrading @particle-network/auth from 1.1.0 to 1.2.1.


WSLENV Environment Variable Flags Unsupported

According to https://blogs.msdn.microsoft.com/commandline/2017/12/22/share-environment-vars-between-wsl-and-windows/, when setting WSLENV, one can append each variable with flags that configure how the variable is shared. This includes the specifier /u which restricts the environment variable flow from Windows to WSL (which I think makes sense for this use case). Adding such a modifier causes the WSLENV variable check to fail.
Versions: tested on wslgit master on Windows 10 build 17134.345.

Good find, thanks for reporting. Yes, the WSLENV check should allow for such modifiers.

This should be fixed with the latest changes from #78. You can download an alpha build for testing here:
https://ci.appveyor.com/project/andy-5/wslgit/build/artifacts


[BUG] Directly mark replica as failed if the node is deleted

Describe the bug (🐛 if you encounter this issue)
After deleting a node, vc.canInstanceManagerLaunchReplica (code) reports that the node is down and unable to launch a replica. However, the fix implemented in https://github.com/longhorn/longhorn/issues/5508 prevents the replica from being marked as failed.
To improve the handling of this situation, we should distinguish between two cases: node not found and node ready. For the node not found case, we can directly mark the replica as failed.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Perform '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Log or Support bundle
If applicable, add the Longhorn managers' log or support bundle when the issue happens.
You can generate a Support Bundle using the link at the footer of the Longhorn UI.
Environment

Longhorn version:
Installation method (e.g. Rancher Catalog App/Helm/Kubectl):
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version:

Number of management node in the cluster:
Number of worker node in the cluster:

Node config

OS type and version:
CPU per node:
Memory per node:
Disk type(e.g. SSD/NVMe):
Network bandwidth between the nodes:

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal):
Number of Longhorn volumes in the cluster:

Additional context
https://github.com/longhorn/longhorn-tests/pull/1279
https://github.com/longhorn/longhorn-tests/pull/1279#issuecomment-1479004050

Steps that I tried to reproduce:

Having an attached volume w/ one replica
Stop the VM that the volume attached and then delete it with kubectl delete nodes.longhorn.io <node-name> -n longhorn-system
In v1.4.1, both volume health status and replica status are Unknown.

@derekbit Is this issue only for one volume and one replica on the same node, correct? If yes, we need to make the description and title clear. That will be helpful to know the exact context.

No, not just for the single node case. I updated the title for clarification.

Verified on master-head 20230516

longhorn master-head (7680931)
longhorn-manager master-head (cd31ec6)

The test steps
https://github.com/longhorn/longhorn/issues/5542#issuecomment-1512334557
Result Passed

After deleting a node, the expected result for Scenario 1 and Scenario 2 can be verified by running the command kubectl -n longhorn-system get replica -o json | jq -r '.items[] | {name: .metadata.name, failedAt: .spec.failedAt}'. It has been confirmed that this issue has been resolved.

Scenario 1

Scenario 2

Do we need also update our manual test case regarding this ticket?
cc @longhorn/qa


Improvement of Navbar

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Hello PA,
Nav items  in the navbar are looking little clumsy.
Want to increase some gap between them for better UI design.
Expected behavior
Improved navbar for better visibility.
Add ScreenShots
Before:

What browsers are you seeing the problem on?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Anishkagupta04  @varshith257
Kindly assign this issue to me,
Will work on this.

@varshith257
Opened a PR in issue no. #926
Please review and assign level labels accordingly.


Updated with 30-09-2018 facebook

I updated some of the tags with the most recent facebook elements. The version available in the repo failed in a couple of stages. Great job on making this tool!!

Hey @aecc, apologies for the delay getting back to you. I apparently haven't checked my Github in months but trying to be more active. I just merged/made some updates for FB's latest UI as of today. Can you try to pull the latest version and let me know if you still have issues?
Thanks so much for the pull request!


Enable Section for Blog Posts

Description
Closes #1696.

Readiness checklist

[ ] I added unit tests for new functionality or bug fixes.
[ ] I added integration tests for new functionality or bug fixes.
[ ] I added compatibility tests for new functionality or bug fixes.
[ ] I made spot refactorings.
[ ] I updated user documentation.
[ ] I ran task all, and it passed.
[ ] I added/updated comments for both exported and unexported top-level declarations (functions, types, etc).
[ ] I checked comments rendering with task godocs.
[x] I ensured that the title is good enough for the changelog.
[ ] (for maintainers only) I set Reviewers (@FerretDB/core), Assignee, Labels, Project and project's Sprint fields.
[ ] I marked all done items in this checklist.

Is it ready for review?

@noisersup Not yet


Add Shutdown method to registry.Registry

Adds a Shutdown method to registry.Registry. Enables gracefully shutting down an embedded registry HTTP server.
Closes #4337.

Missed some really obvious things. Fixes are pushed, and my own fork is working nicely in my embedding app.

Looks like there's some fix-up commits in the PR that should be squashed; can you clean up commit history a bit?

Looks like there's some fix-up commits in the PR that should be squashed; can you clean up commit history a bit?

Sorry, I'm used to squashing my PRs. I just did so.


HELM-99: make severity color coding column or row

This PR makes the color-coding for severity configurable to either be Row (the entire row, like the current impl when checked), Column (just any column with type='severity'), or Off (like the current impl, when unchecked).
This PR relies on changes made in https://github.com/OpenNMS/opennms-helm/pull/68

Rebased over develop now that HELM-62 was merged.


.Net: Updates for ChatMessageContent to support complex content model

Motivation and Context

Resolves: https://github.com/microsoft/semantic-kernel/issues/3874
Resolves: https://github.com/microsoft/semantic-kernel/issues/3522
Resolves: #3768
This PR contains changes to prepare public API of ChatMessageContent to support extensible content model in the future. With latest OpenAI API it's possible to have content as string or array of content objects (scenario for GPT vision):

Description

Added ChatMessageContentItemCollection to store multiple contents of different types in chat message.
Added ChatMessageContentItemCollection? Items property to ChatMessageContent, made string? Content property nullable.
Added ImageContent class that stores image URI.
Added ADR and diagram of chat and text content models.
Added Example74_GPTVision to demonstrate the functionality with GPT Vision model.

Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

What's the relationship now between ChatMessageContentItemCollection and ChatHistory? They're both a collection of chat message contents, right?

They're both a collection of chat message contents, right?

No, ChatMessageContentItemCollection is a collection of ContentBase items, like TextContent, ImageContent or ChatMessageContent itself, while ChatHistory is a collection of concrete type ChatMessageContent. Yes, it looks like ChatMessageContentItemCollection can be re-used in the same way as ChatHistory, although both classes have different responsibilities.
In our case, ChatHistory is a main container for all chat messages with methods like AddUserMessage, AddSystemMessage etc, while ChatMessageContentItemCollection is a collection of chat message contents, which can be text or image url for example.

No, ChatMessageContentItemCollection is a collection of ContentBase items, like TextContent, ImageContent or ChatMessageContent itself, while ChatHistory is a collection of concrete type ChatMessageContent.

Right, that's my point: ChatHistory is a collection of ContentBase items, just guaranteeing a more specialized ContentBase-derived type. So should that relationship be modeled in the API, so that a ChatHistory can be used anywhere a ChatMessageContentItemCollection is employed?

So should that relationship be modeled in the API, so that a ChatHistory can be used anywhere a ChatMessageContentItemCollection is employed?

I think the reason why it's confusing is because we use TextContent or ChatMessageContent models as response from AI and at the same time we use TextContent inside ChatMessageContent as part of chat message text content. Maybe it's not a problem, but probably that's the reason why hierarchy is not clear. (That's why I've added class diagram to this PR :) ).
From user perspective, I would like to clearly differentiate between ChatHistory from ChatMessageContentItemCollection. In initial ChatMessageContentItemCollection I've added a check that ChatMessageContent (as ContentBase) can't be added to the collection (unline in ChatHistory), because it doesn't make sense to include chat message as content of chat message. But I removed that check, because this restriction doesn't actually provide any benefits, and on top of that, checking ContentBase type every time on adding new item to the collection may be costly.
Answering your question, I'm not sure if using ChatHistory anywhere a ChatMessageContentItemCollection is employed will provide any benefits, but maybe I'm missing something. Some use-cases or scenarios will definitely help.


[Home] Fix for status bar inconsistency #tiny

This fixes the finicky margin between the top of Home and the status bar.

          1 Error

:no_entry_sign:
No CHANGELOG added.

          1 Warning

:warning:
The following files do not have tests: lib/containers/artist.js & lib/containers/home.js

  Generated by :no_entry_sign: danger

🎉 awesome


Pixels are not "selecatable"

First of all, thanks for this tool, I simply do not know why ESRI dont include it in the standard package. Indeed so many headaches would be easily solvable. Anyway, I can't seem to get it to work. It seems to be a general settings problem, as I can't select any pixel, whether for changes or viewing. I click on "Freehand paint" (in the raster painter toolbar) and it is selected, but doesn't hihglight any pixels. Same goes for the Raster Editor. Any fresh ideas as to why it might be exhibitting this type of behavior? Maybe I need to copy the .dll file somewhere in the Arc Installation folders?
In any occasion - mucho gracias!
Momo

What's your version of ArcMap? This addon is pretty sensitive to the version of ArcObject and it is likely to create problems when the version number has major difference with 10.3.
Unfortunately, I am not working with ESRI products now and I have no access to ArcMap. So this project is no longer maintained. If you know how to do it, I would recommend to recompile the addon with the ArcObejct at your computer.
Sorry

Hey Yu,
I was going through the files you made available and within the separate folders for the different ArcGIS versions the ESRI Addin File for the version 10.1 (my version), upon closer inspection reveals itself to be for the 10.0 Version of ArcGIS. So the files are listed as follows:
ArcGIS 10.0 -> 10.0 ESRI Addin File
ArcGIS 10.1 -> 10.0 ESRI Addin File !!!
ArcGIS 10.2 -> 10.2 ESRI Addin File
ArcGIS 10.3 -> 10.3 ESRI Addin File
Could you please send me the 10.1 ESRI addin file, as you seem to have neglected to put it in.
Thanks again!
Momo

Inside the folders you've uploaded here there is no version of your tool for ArcGIS 10.1. Inside the ArcGIS 10.1 folder the addin ESRI file is for the 10.0 Version of ArcGIS. Hence the request - could you please upload/send the 10.1 version? Thank you!

@Momut1 I needs to set up the environment to compile the source code with ArcGIS 10.1. So please give me some time.
If you want to compile it yourself, you need Visual Studio (I used VS 2010 at that time) and ArcObject 10.1, which is included in the ArcGIS 10.1 installation package / disk. What you need to do is to download this repo, open the project (at master branch) with the Visual Studio, change the version number from 10.0 to 10.1 at two files (ARES.csproj and Config.esriaddinx), recompile the project and get the file at ares/ARES/bin/Debug. You may need to import some ESRI ArcObject libraries for compiling (listed at ARES.csproj).

Apologies, I meant no rush at all.
Inside the ArcGIS 10.1 Installation package/disk there are two files: the ARES.dll application extension and the ARES Esri addin file. Which one of these do you refer to as "ArcObject 10.1"? Because there is no such file in the folder...
Thank you for the time you are taking to help me!

ArcObjects is an ESRI product that allows us to develop desktop GIS software.

Hello Yu,
I have some final things to ask of you information-wise in order to really make sure I've got a handle on things:

You said you used VS to compile the tool, but is it necessary to use it now? I don't have the software on my computer and am wondering if I can use any other compiler. The second part of this question being - doesn't Arc itself come with a compiler, inside which you could compile the tool? But yeah - the question is - can I use whichever compiler I want or does it have to be the same one you used to do it in the first place?

What language is the script written in? Python?

Thanks a lot for responding to me.
Warmest regards,
Momo

-------- Оригинално писмо --------

От: Haoliang Yu<EMAIL_ADDRESS>

Относно: Re: [haoliangyu/ares] Pixels are not "selecatable" (#20)

До: haoliangyu/ares

Изпратено на: 23.05.2017 16:10

ArcObjects  is an ESRI product that allows us to develop desktop GIS software.
—
You are receiving this because you were mentioned.
Reply to this email directly,  view it on GitHub , or  mute the thread .
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/haoliangyu/ares","title":"haoliangyu/ares","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/haoliangyu/ares"}},"updates":{"snippets":[{"icon":"PERSON","message":"@haoliangyu in #20: [ArcObjects](http://help.arcgis.com/en/sdk/10.0/arcobjects_net/conceptualhelp/index.html#//0001000002zs000000) is an ESRI product that allows us to develop desktop GIS software."}],"action":{"name":"View Issue","url":"https://github.com/haoliangyu/ares/issues/20#issuecomment-303392529"}}}

@Momut1 This ArcMap add-in is developed in C#.
As for the compiler, this tool needs .Net Framework 4.0, which is shipped in the Visual Studio 2010, or later. This is the only way I know to compile an ArcMap add-in. But I have been away from Windows for a few years, I don't know how thing changes.


Improve sorting performance of Stream

The specification of the test environment is:
OS: Ubuntu 22.04.3 LTS x86_64
CPU: AMD EPYC 7282 (4) @ 2.794GHz
Memory: 7951MiB
The parameters for the benchmark test are:
{batchCount: 2, timestampCount: 500, seriesCount: 100, tagCardinality: 10, startTimestamp: 1, endTimestamp: 1000, scenario: "large-scale"}
{batchCount: 2, timestampCount: 500, seriesCount: 100, tagCardinality: 10, startTimestamp: 900, endTimestamp: 1000, scenario: "latest"}
{batchCount: 2, timestampCount: 500, seriesCount: 100, tagCardinality: 10, startTimestamp: 300, endTimestamp: 400, scenario: "historical"}
Query benchmark results before optimization:
BenchmarkSort/sort-4                   1<PHONE_NUMBER> ns/op<PHONE_NUMBER> B/op 11794139 allocs/op
BenchmarkSort/sort#01-4                1<PHONE_NUMBER> ns/op    368143928 B/op   3009694 allocs/op
BenchmarkSort/sort#02-4                1<PHONE_NUMBER> ns/op    376661456 B/op   3047854 allocs/op
Query benchmark results after optimization:
BenchmarkSort/sort-large-scale-4         	       1<PHONE_NUMBER> ns/op	180651008 B/op	 2610881 allocs/op
BenchmarkSort/sort-latest-4              	       1<PHONE_NUMBER> ns/op	159891208 B/op	 2391475 allocs/op
BenchmarkSort/sort-historical-4          	       1<PHONE_NUMBER> ns/op	159586672 B/op	 2391241 allocs/op

[x] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Fixes apache/skywalking#12160.
[x] Update the CHANGES log.

Query benchmark results after optimization:
BenchmarkSort/sort-large-scale-4 1<PHONE_NUMBER> ns/op 261214432 B/op 3585136 allocs/op
BenchmarkSort/sort-latest-4 1<PHONE_NUMBER> ns/op 415367000 B/op 5739892 allocs/op
BenchmarkSort/sort-historical-4 1<PHONE_NUMBER> ns/op 415185616 B/op 5739826 allocs/op

What were the performance before?

I have added the benchmark results before optimization to the description. The results for sorting small-scale data seem a bit off. I am looking into what might have gone wrong.

I have added the benchmark results before optimization to the description. The results for sorting small-scale data seem a bit off. I am looking into what might have gone wrong.

Is this resolved by the latest commit?

Yes. The benchmark test results have been updated.


Convert all authors to one author

Hi,
Thanks for this great utility. We're looking to migrate a repository with a long history and a many contributing authors. Many of these authors are no longer in contact and we do not have an easy way of looking them up on GitHub, or contacting them and asking them to set up an account if one does not exist. In order to keep the new repository smallish, we were considering migrating from the last revision, and also making all existing files have the same author. Is this possible? Would we have to manually map all existing authors that we do not have info for to a single author in the authors file, or is there a way to set a default?

As a follow up, are email addresses strictly required in the authors file?


BUG: Warnings in timetable upload page tests

Describe the task that needs to be done.
Bug Report: Some tests in the timetable upload page are generating warnings due to a required prop not being passed in to the component.

Describe how a solution to your proposed task might look like (and any alternatives considered).
Add in the required prop, a function, to the UploadPage component. Issue can be considered closed when this warning no longer appears
Notes
This warning was discovered by @shreytailor and reported on the group discord server.

LGTM! Thanks for creating this ticket


[Storage] Consider moving Error Codes to Swagger

Hi @seanmcc-msft, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.


There should be a setting to select the default audio bus for each type of event

Right now, you have to manually select the audio bus, but I think there should be an "Audio" page on the settings with the default busses for each type of event, so that when you create them they are the same as in your settings.
That way, if you want to modify one audio bus later you can change it manually.

I have actually been thinking if we should add a way to change the default value of ANY event setting, because I'm sure some people would want to have characters join at position 234 by default (Character Event), have a default wait time of 3.414 seconds (Wait event) and an alternative choice text of "You cannot choose this you idiot" (Choice event)...
As arguments can be much more easily gathered now anyways (like for the autocompletion of the text editor) something like this would certainly be possible, but it might be easier to add settings for the important ones.
Let me know what you think.

That sounds like a great idea. Having a settings tab with default that lets you overwrite any of the default input fields from any event would be an amazing thing to have. If it is too much we can start slowly adding the ones we already identified.

I will take a look at it. I might combine it with a couple of other module-related stuff I wanted to built, like a way to disable some modules, even when they are installed so people can clear up the event button sidebar to contain only what they really need.


DF-1347 Better Input Validation

DF-1347 Better Input Validation
Changes

Updated  ' _includes/post-macros.html' to add validation class to markup.
Updated  '  static/js/app.js' to validate pagination form on submission.

Notes

This does not handle cases where the user has modified the query parameters directly
@anselmbradford, I'll convert to CommonJS if #499 goes in first.

Test

Navigate to http://localhost:7000/blog/ and enter 'a'.
Click 'Go' (Should do nothing).
Enter '5' (Should navigate to 'http://localhost:7000/blog/?page=5#pagination_content') .

Review

@jimmynotjim
@anselmbradford
@Scotchester
@KimberlyMunoz

LGTM pull in https://github.com/cfpb/cfgov-refresh/pull/499

Also see https://github.com/cfpb/cfgov-refresh/pull/497#issuecomment-98778358 for reference on migrating JS.


README should probably be RST instead of MD

So it renders correctly on PyPI. See agate's README:
https://github.com/onyxfish/agate/blob/master/README.rst

I'll fix this.


Is detecting column width of the terminal on Windows not supported yet?

Thanks for reporting a bug! Please include all of the following:
(1) A description of the issue. A screenshot is often helpful too.
difft prints the diff within 80 columns no matter how wide my terminal is:

(2) A copy of what you're diffing. If you're diffing files, include
the before and after files. If you're using difftastic with a VCS
repository (e.g. git), include the URL and commit hash.
Changed this file as the screenshot above.
(3) The version of difftastic you're using (see difft --version) and
your operating system.
> difft --version
Difftastic 0.33.0

OS: Windows 10 Pro 21H2
Terminal: Windows Terminal 1.14.1962.0
Shell: PowerShell 7.2.6

Another question: I tried to fix this problem by myself, but I failed to run cargo build with an error:
> cargo build
   Compiling difftastic v0.34.0 (C:\t\difftastic)
error: failed to run custom build command for `difftastic v0.34.0 (C:\t\difftastic)`

Caused by:
  process didn't exit successfully: `C:\t\difftastic\target\debug\build\difftastic-c8d1297c37fbb275\build-script-build` (exit code: 1)
  --- stderr

  error occurred: Command "C:\\Program Files (x86)\\Microsoft Visual Studio\\2022\\BuildTools\\VC\\Tools\\MSVC\\14.32.31326\\bin\\HostX64\\x64\\cl.exe" "-nologo" "-MD" "-Z7
" "-Brepro" "-I" "vendor/tree-sitter-cpp-src" "-W4" "/std:c++14" "-FoC:\\t\\difftastic\\target\\debug\\build\\difftastic-193ac9cd16413c45\\out\\vendor/tree-sitter-cpp-src\\
scanner.o" "-c" "vendor/tree-sitter-cpp-src\\scanner.cc" with args "cl.exe" did not execute successfully (status code exit code: 2).

How are you build it on Windows?

I have the same issue on Linux Ubuntu 22.04 (zsh 5.8.1) when invoked like this:
echo 94c99c17d | GIT_EXTERNAL_DIFF='difft --color=always' xargs git show -m --ext-diff | less -R

@balta2ar If my knowledge is correct, in your case the stdout of difft is not a terminal. So automatic column size detection is not possible.

then I guess it would be useful to have it as an argument, then I could detect it manually before xargs and pass it to difft

@balta2ar Try difft --width `tput cols` , which is documented in difft --help


Discord Server

My issue is that there isn't a discord to discuss this project :(

https://discord.gg/axGZT8Nf
This seemed easy enough... not we need more people on it!

If the creator does not wish to maintain one, I am working on this over at discord.gg/puffer (library dedicated to making RL projects like this easy to work with)

At this time it's probably best to move to puffer to centralize some development. When pwhiddy is free we can transfer ownership on the current discord/PokemonRedExperiments server for his keeping.

I'd be happy to have a discord if others are willing to help admin it! I can set one up in the next day or two. In the meantime we can collaborate in the puffer discord.

Heading to the Puffer discord then.

Hey all! there's now a dedicated discord server here!
http://discord.gg/RvadteZk4G


In thermostat_over_climate, overpowering is checked even if thermostat is OFF

This is useless and generate commands to the climate

It is working normally. It was due to underlying Tuya climate not initialized. See many Tuya issues.


Install section not showing on https://rancher.com/docs

Install section not showing on https://rancher.com/docs. Everyone other tab/ link seems to work.
Full link:
https://rancher.com/docs/os/v1.x/en/installation/

There's actually no content on that page, but we need to re-order the navigation of Rancher OS docs to avoid these kind of pages with no content.

Closing this issue because the page has content now.


Adding file persistence bundle and endpoint handling multipart data types

New PR rebased to new master. Due to multiple commits duplication, I decided to flush them out and create new ones, not to trash the code. Issues same as at the last PR. I'm sorry it took me so long, school graduation and matura exam ahead ;)

Closing, because some things need to be implemented first


crypto tickers using polygon are not getting proper calculations

command input to reproduce:
bin/stockclient -s 2023-03-21T00:00:00-05:00 -t x:btcusd

    "1710892800000": {
      "ticker": "X:BTCUSD",
      "close": 67858.8,
      "volume": 49516.82253871595,
      "price-velocity": 5952.530000000006,
      "timestamp": "2024-03-20T00:00:00Z",
      "short-avg-volume": 31791.73091614418,
      "short-avg-volume-ratio": 1.5575377971499744,
      "rvol-short": 0,
      "rvol-short-vel": 0,
      "rvol-short-accel": 0,
      "short-day-rvol-range-percent": 0,
      "short-day-rvol-high": 0,
      "short-day-rvol-low": 0,
      "trade-range-vadj": null,
      "prob-trade-range-vadj": {
        "high": 0,
        "low": 0
      },
      "med-avg-volume": 24254.45321884959,
      "med-avg-volume-ratio": 2.041555919316023,
      "rvol-med": 0,
      "rvol-med-vel": 0,
      "rvol-med-accel": 0,
      "med-day-rvol-range-percent": 0,
      "med-day-rvol-high": 0,
      "med-day-rvol-low": 0,
      "trend-range-vadj": null,
      "prob-trend-range-vadj": {
        "high": 0,
        "low": 0
      },
      "long-avg-volume": 23761.61106648793,
      "long-avg-volume-ratio": 2.0839000520697755,
      "rvol-long": 0,
      "rvol-long-vel": 0,
      "rvol-long-accel": 0,
      "long-day-rvol-range-percent": 0,
      "long-day-rvol-high": 0,
      "long-day-rvol-low": 0,
      "tail-range-vadj": null,
      "prob-tail-range-vadj": {
        "high": 0,
        "low": 0
      }
    }
  }
}

Notice how rvol, rvol velocity, rvol acceleration, the rvol range percent, the rvol high, rvol low, and risk ranges are all zero. It seems that's due to how the data is being stored/retrieved when using polygon. We're storing the x: and I suspect this is a result of that pre-pended state for the ticker.

The related commit addresses the bug here by removing a call using a specific ticker. Instead, the code now just calculates the realized volatility against all tickers included.
The code currently only supports one ticker at a time right now, but multiple ticker support is planned for the future.


Use varhandles when you work with bytebuffers inside of storage pages.

VarHandles are JVM backdoors for performance optimizations of serialization/deserialization operations that work with primitive datatypes on byte[]/ByteBuffer classes. We can very quickly, by changing a single class com.orientechnologies.orient.core.storage.impl.local.paginated.base.ODurablePage to improve the speed of database operations.

This issue relates to #10066


[Bug] Title not showing

Describe
HTML Title is not showing
To reproduce
Steps to reproduce the behaviour:

Go to the website
Look at the title

Expected behaviour
Show the title expected in main.go
Screenshots
https://i.bgpvp.xyz/xSAHu.png
Devices
Every devices

Fixed in 22019a5.


Is security to update ClickHouse server version 1.1.54362 to 1.1.54380 ?

And what is the best way to make this, in production cluster.

In most cases simple 'update package - restart clickhouse' if enough.
But it's always safer to check it on some staging environment, and check the changelog for information about compatibility changes.

Thx a lot!!


added achors, and navigation index letters

Added a contents entry to jump to index letter, also added (top) links to go from anywhere in the contents, and after entry entry to jump back to the top.

This PR has conflicts. I can review once the conflict is resolved.

I see the GItHub page that shows me what the conflicts are. How do I now resolve them?

Aah, ok. No idea how I did it, but glad I did. Lol.
I will try to figure it out later, I have a couple of fixes to do for other
stuff.
It is never ending this, lol
On Mon, 1 Jul 2019 at 17:45, Matt Layman<EMAIL_ADDRESS>wrote:

Merged #176 https://github.com/bnb/awesome-developer-streams/pull/176
into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bnb/awesome-developer-streams/pull/176?email_source=notifications&email_token=AAGXAH43TMLVSEA5EN7E7UTP5IYBXA5CNFSM4H4D7ZH2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOSIR77TA#event-2451832780,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGXAHZX23T5TGYEJVUXTCLP5IYBXANCNFSM4H4D7ZHQ
.


templateUrl &  styleUrls

Could you please show templateUrl &  styleUrls  inside the component ts files?

Yes i did. You must clone repo again, there is a new structure for project.


Move exclude input

Three input on the same line was already a llot but there's no way to accomodate for a fourth so i moved it at the line for date search
akuma u have no eyes

Coverage remained the same at 33.139% when pulling 4ad1ff5a4701c122345c31e39c1bd6cc29f9eb67 on Kiloutre:patch-44 into 53ba669ba6021787003e997552e417ac4c8cd134 on NyaaPantsu:dev.

Coverage remained the same at 33.139% when pulling 07e60a3d0d8c85d0e1bebd7d8c63a4eed333fb9a on Kiloutre:patch-44 into 53ba669ba6021787003e997552e417ac4c8cd134 on NyaaPantsu:dev.

Coverage remained the same at 33.139% when pulling 07e60a3d0d8c85d0e1bebd7d8c63a4eed333fb9a on Kiloutre:patch-44 into 53ba669ba6021787003e997552e417ac4c8cd134 on NyaaPantsu:dev.

Coverage remained the same at 33.139% when pulling 07e60a3d0d8c85d0e1bebd7d8c63a4eed333fb9a on Kiloutre:patch-44 into 53ba669ba6021787003e997552e417ac4c8cd134 on NyaaPantsu:dev.

Coverage remained the same at 33.139% when pulling 6425d1c89f4a08bc861d286d585fdc0f035ac141 on Kiloutre:patch-44 into 1475da3cc0f486ba3b29418ace7c59c6b4c3a0e4 on NyaaPantsu:dev.


Fails to build on Mac OSX for 0.5.0

I wasn't clear whether "git updates > 0.4.0" meant 0.5.0 ? The last commit was about 3 days ago, so it seemed a reasonable guess. But if that is so, there are problems with an OSX build. The main one is an error (there are 6 of them) having to do with overloading "=". The warnings are related to a "converted initializer". Here is a partial dump:
In file included from src/LOGinstruments.cpp:1:
In file included from src/LOGinstruments.hpp:1:
../../include/rack.hpp:34:14: error: no viable overloaded '='
model->tags = {tags...};
~~~~~~~~~~~ ^ ~~~~~~~~~
src/LOGinstruments.cpp:18:14: note: in instantiation of function template
specialization 'rack::createModel<VelvetWidget, const char *, rack::ModelTag>'
requested here
p->addModel(createModel("LOGinstruments", "LOGinstrument...
^
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1/list:862:11: note:
candidate function not viable: cannot convert initializer list argument to
'const std::__1::list<rack::ModelTag, std::__1::allocatorrack::ModelTag >'
list& operator=(const list& __c);
^
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1/list:874:11: note:
candidate function not viable: cannot convert initializer list argument to
'std::__1::list<rack::ModelTag, std::__1::allocatorrack::ModelTag >'
list& operator=(list&& __c)
^

Thanks, at the moment the version that is available on git is not up to date with 0.5.0.
It will be soon, then I'll make a new post here and close the thread.

Now it is ready for 0.5.0


Add a final draft of Contributing.md. Fix #30

related to #30

I'll check this out when I have time this evening. Thanks.

I'll check this out properly tomorrow afternoon.

No problem. Take your time.

The guideline could very well already state this, but just in case I forget to check later:
I think sheme and template file names should be all lowercase. Do you agree?

@alexbooker Sure.

@alexbooker scheme and template file names MUST or SHOULD be all lowercase? I think you want to say MUST instead of SHOULD. Can you confirm?
Also, I quickly wrote a regex to check a scheme/template name:
[a-z0-9.-]+

Debuggex Demo

I wasn't too sure at the time but now I am. It must be lowercase. Thanks.

@alexbooker Done !

@aloisdg Thanks for this.

@alexbooker  you are welcome :)

@aloisdg I want to make some small changes:

Grammar changes
You don't strictly have to create an issue first. That being said, it is recommended.
In my experience, contributors seldom read the contributing.md. Those who do probably wont read it all. It is for that reason that I want to put the most important information at the top.
I think I can make the headings a bit clearer with a TOC
I think I can make the template specification a bit clearer with more examples
I think I can make it all sound a bit more... welcoming.

I agree with all. Can you fix all odd point directly?


No mention of max_bin parameter in Python API

Version
Latest
Description
In the Parameter tuning section of the docs, it is recommended to tune max_bin as a tradeoff between accuracy and speed:

For Faster Speed

Use bagging by setting bagging_fraction and bagging_freq
Use feature sub-sampling by setting feature_fraction
Use small max_bin
Use save_binary to speed up data loading in future learning
Use parallel learning, refer to Parallel Learning Guide

For Better Accuracy

Use large max_bin (may be slower)
Use small learning_rate with large num_iterations
Use large num_leaves (may cause over-fitting)
Use bigger training data
Try dart

However, there is no mention of the max_bin parameter in both the Scikit-Learn and Training API.
Was the parameter removed? Thanks!

If there is no corresponding argument for any parameter from this page https://lightgbm.readthedocs.io/en/latest/Parameters.html, then you can pass it via params dict in training API or via kwargs in sklearn wrapper.
For instance:
lgb.train({'max_bin': 50}, train_set)

lgb.LGBMClassifier(max_bin=50)


DeepClone: Use sourcegenerator to fix trimming issues

This should fix this: https://github.com/MudBlazor/ThemeManager/issues/25#issuecomment-2333822719
But cannot be completely finished due to this problem: https://github.com/MudBlazor/MudBlazor/pull/9434#issuecomment-2359333865

Added a working workaround for MudTheme. It's verbose but there is no better way for now because of the issue above.


:arrow_up<EMAIL_ADDRESS>

This version fixes #10272

@svanharmelen would you mind targeting stable instead for this?  I think the freeze is dangerous enough to warrant a hotfix.

Sure... I'll close this one and create a new PR.


Explore Abstraction of COAP from our samples

Experiments & Questions to answer:

[ ] Attempt to create synchronous COAP API's
[ ] Turn on acknowledge & retransmission for all connections or provide individual control?
[ ] static or dynamic packet allocation?
[ ] Which Golioth Services will first use the abstraction?
[ ] What does full transport abstraction look like?

@mniestroj feel free to keep notes here or an accompanying PR.

@mniestroj continue to use this for the experiments we discussed, which I also added to the description of this issue. Also add any that I missed.


Remix Vite Config Resolution Broken in Monorepos

Opening this to put #850 on your radar.

closing as this was merged! thanks :)


Error about .NET runtime version while exporting mono project

Godot version
4.0-beta10.mono
System information
Windows 11, Ryzen 3700U Vega 10 iGPU
Issue description
I am trying to export a mono project (a simple flappy bird clone). Building and running from both Godot and Visual Studio Code works correctly, however when I try to export the project EXE I get the following error:
Running:  "C:\Program Files\dotnet\dotnet.EXE" publish C:/Users/arena/Projects/20-games-challenge/flappy-bird/FlappyBird.sln -c ExportRelease -r win-x64 --self-contained true -v normal -l:GodotTools.BuildLogger.GodotBuildLogger,C:/Users/arena/Projects/Godot/GodotSharp/Tools\GodotTools.BuildLogger.dll;C:/Users/arena/AppData/Roaming/Godot/mono/build_logs\90eccf62abacd64c2d029fa5dde64602_ExportRelease -p:GodotTargetPlatform=windows -o C:\Users\arena\AppData\Local\Temp\godot-publish-dotnet\12616-ExportRelease-win-x64
MSBuild version 17.3.2+561848881 for .NET
Build started 07/01/2023 09:43:23.
     1>Project "C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.sln" on node 1 (Restore target(s)).
     1>ValidateSolutionConfiguration:
         Building solution configuration "ExportRelease|Any CPU".
       _GetAllRestoreProjectPathItems:
         Determining projects to restore...
       Restore:
         X.509 certificate chain validation will use the default trust store selected by .NET.
         Restoring packages for C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.csproj...
     1>C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.csproj : error NU1100: Unable to resolve 'Microsoft.NETCore.App.Runtime.win-x64 (= 6.0.11)' for 'net6.0'. [C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.sln]
     1>C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.csproj : error NU1100: Unable to resolve 'Microsoft.WindowsDesktop.App.Runtime.win-x64 (= 6.0.11)' for 'net6.0'. [C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.sln]
     1>C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.csproj : error NU1100: Unable to resolve 'Microsoft.AspNetCore.App.Runtime.win-x64 (= 6.0.11)' for 'net6.0'. [C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.sln]
         Writing assets file to disk. Path: C:\Users\arena\Projects\20-games-challenge\flappy-bird\.godot\mono\temp\obj\project.assets.json
         Failed to restore C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.csproj (in 332 ms).

         NuGet Config files used:
             C:\Users\arena\AppData\Roaming\NuGet\NuGet.Config
             C:\Users\arena\AppData\Roaming\NuGet\config\Godot.Offline.Config
     1>Done Building Project "C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.sln" (Restore target(s)) -- FAILED.

Build FAILED.

       "C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.sln" (Restore target) (1) ->
       (Restore target) ->
         C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.csproj : error NU1100: Unable to resolve 'Microsoft.NETCore.App.Runtime.win-x64 (= 6.0.11)' for 'net6.0'. [C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.sln]
         C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.csproj : error NU1100: Unable to resolve 'Microsoft.WindowsDesktop.App.Runtime.win-x64 (= 6.0.11)' for 'net6.0'. [C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.sln]
         C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.csproj : error NU1100: Unable to resolve 'Microsoft.AspNetCore.App.Runtime.win-x64 (= 6.0.11)' for 'net6.0'. [C:\Users\arena\Projects\20-games-challenge\flappy-bird\FlappyBird.sln]

    0 Warning(s)
    3 Error(s)

Time Elapsed 00:00:01.30

My dotnet --info output is:
.NET SDK (reflecting any global.json):
 Version:   6.0.403
 Commit:    2bc18bf292

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.22621
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\6.0.403\

global.json file:
  Not found

Host:
  Version:      6.0.11
  Architecture: x64
  Commit:       943474ca16

.NET SDKs installed:
  6.0.403 [C:\Program Files\dotnet\sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.App 5.0.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 6.0.11 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 5.0.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.11 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 5.0.3 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 6.0.11 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

Download .NET:
  https://aka.ms/dotnet-download

Learn about .NET Runtimes and SDKs:
  https://aka.ms/dotnet/runtimes-sdk-info

Steps to reproduce
Open the attached project and try to export a release for Windows Desktop.
Minimal reproduction project
flappy-bird.zip

I don't know if this bug should be moved to dotnet/sdk, however it seems to be fixed by running:
dotnet nuget add source -n nuget.org https://api.nuget.org/v3/index.json

Previously I had:
PS > dotnet nuget list source
No sources found.

It seems a similar problem to the one solved with https://github.com/dotnet/sdk/issues/20840#issuecomment-927230669

It may have been due to the NuGet configuration in:

C:\Users\arena\AppData\Roaming\NuGet\NuGet.Config
C:\Users\arena\AppData\Roaming\NuGet\config\Godot.Offline.Config

We used to modify the NuGet.Config file in the past. What is the earliest version of Godot 4.0 you've used? Also, have you ever built Godot from source?
Since Godot adds/modifies NuGet configuration, it's possible that this issue is entirely our fault.

I'm pretty sure I have installed at least betas 6, 8 and 10 in this system (don't know for the previous ones since I used to work a lot also with Linux before). I've never build Godot from source.

Oh, I don't think we modify NuGet.Config since beta 1.
I don't have access to a Windows machine but if you could confirm that publishing (dotnet publish) a non-Godot C# project with a fresh .NET 6 SDK installation works, then it must be a Godot issue.

Upon uninstall of the .NET SDK + a fresh install of the latest SDK version I have
PS >  dotnet nuget list source
Origini registrate:
  1.  nuget.org [Abilitato]
      https://api.nuget.org/v3/index.json

Which is different from what I have initially in my previous installation. However, publishing a simple .NET project (non-Godot) seems to work correctly out of the box.

So either a previous .NET SDK installer was buggy and didn't add the nuget.org source or Godot broke the NuGet configuration. If you can export games now with beta10 maybe the bug was fixed, I think we can close this issue and reopen if it happens again.

Yes! I'm closing this, I'll keep an eye on this and will comment here if it happens again


v1.12.x: vendor latest containers/image for parallel pulling

In combination with the recent usage of the pgzip library, pulling an
image is up to 2 times faster.
Signed-off-by: Valentin Rothberg<EMAIL_ADDRESS>Backport of #1964
Cc: @mrunalp

LGTM
/test all
/approve

/ok-to-test

/test e2e_rhel

lgtm


Zooming should have priority over changing pages

First of all great job on this plugin. I am the developer of a similar plugin called pinch_zoom, but I was always struggling with some issues that this plugin doesn't have. Only one issue that my plugin managed to solve by using the InteractiveViewer, this plugin still has. Hopefully you might know a solution to this.
Describe the bug
When zooming in on an image in a PageView, the page will change instead of the image zooming in. Possibly this could be resolved using the OneSequenceGestureRecognizer or MultiDragGestureRecognizer.
In my plugin I managed to solve this even more when using the answer provided in https://stackoverflow.com/questions/51712287/how-to-disable-multi-touch-in-mobile-application-using-flutter/56037327#56037327, which I had to use since I couldn't solve the issue using GestureRecognizer solutions, simply since my plugin uses the InteractiveViewer and not that one.
To Reproduce
Steps to reproduce the behavior:

Add a PageView.builder() with multiple pages
Add an image wrapped by the ZoomOverlay() to one of these pages
Try zooming in on this image. Try with both more vertical and horizontal zooming.
Though it works better with twoTouchOnly, you shall notice that the PageView tends to have priority over the zooming of the ZoomOverlay(). It also works better when zooming vertically.

Expected behavior
The image will zoom in instead of the page changing

I have tried to reproduce it but it seems okay when zooming.

https://user-images.githubusercontent.com/68549184/119984532-16c70800-bff4-11eb-96fb-73b5fad917bb.mp4

@Mayb3Nots You are using ListView, the bug he encounter is within PageView ...

@Mayb3Nots You are using ListView, the bug he encounter is within PageView ... I think you should
This enhancement might solve this issue, disable all existing touch event when 2 finger tapped on the Image
#5

The video I showed you was using PageView. Can you guys provide a simple reproduction code

The following code contains the problem that I tried showing + another weird behaviour. I was about to already file a new report on this, namely that the overlay entry doesn't spawn in the correct place when it is in an expanded widget.
You can't see that in the video, but I constantly tried zooming in with different angles using 2 fingers.
return Scaffold(
      appBar: AppBar(),
      body: PageView(
        children: [
          Container(
            width: MediaQuery.of(context).size.width,
            height: MediaQuery.of(context).size.height,
            color: Colors.blue,
          ),
          Column(
            children: [
              Expanded(
                child: ZoomOverlay(
                  child: Image.network('http://placekitten.com/g/300/200'),
                ),
              ),
            ],
          ),
          Container(
            width: MediaQuery.of(context).size.width,
            height: MediaQuery.of(context).size.height,
            color: Colors.red,
          )
        ],
      ),
    );

Video:
https://user-images.githubusercontent.com/37802918/120060718-9d620080-c059-11eb-99a4-029133b193bb.mp4

Hello! Any update on this? It's even worse when you put the PageView in a ListView.


Can no longer get evaluated JS

I am trying to upgrade my app to the latest RNTurbolinks but it seems to be missing a function my app relies on:
 const script = 'document.documentElement.outerHTML';
 Turbolinks.evaluateJavaScript(script).then((source) => {
           const $ = cheerio.load(source);
           //Extract stuff from the source using cheerio
       });
I can see there is now an 'injectJavaScript' function but that doesn't seem to return a promise or anything else. Is there any chance of adding this back in?
Or how should I be getting the HTML of the page to be able to parse it?

Cheerio is not supported anymore... You should write your js on webpage and use injectJavaScript and use turbolinksMessage event if you want any return value.


Set pagination_class on ApiBlockViewSet to None

Overview
The frontend code does not support pagination and expects to receive a simple data array.
Connects #2056
Testing Instructions

Check out ogr/develop
Log in as<EMAIL_ADDRESS>Browse http://localhost:6543/dashboard/apiblocks and verify that a React error is raised
Check out this branch
Browse http://localhost:6543/dashboard/apiblocks and verify that the page loads without error

Checklist

[x] fixup! commits have been squashed
[ ] CI passes after rebase
[x] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
[x] This PR is targeted at the correct branch (develop vs. ogr/develop)
[ ] ~If this PR applies to both OAR and OGR a companion PR has been created~

Thanks for the review.


store.CanonicalStoreURL == null

Hello uWebshopers,
i get the following error at my uWebshop e-shop site:
"store.CanonicalStoreURL == null"
The only way we found to resolve this error is to republish the entire umbraco content (or edit the web.config file).
Have you ever face this error before?
Any idea how to solve it?
I am using
umbraco: 6.1.6
uWebshop:  <IP_ADDRESS>

Would it be possible to try to upgrade to a newer version of uWebshop?
maybe try <IP_ADDRESS> first? http://uwebshop.com/changelog

Hello Arnold,
i try to upgrade <IP_ADDRESS> version and i am getting the following error:

The problem seems to exist at uWebshop.Payment.CreditCard implementation at CreditCardPaymentResponseHandler.cs file:

is it possible to solve the error above?


assert failure

Hi! I am tryin to fuzz using the below command I am getting the error:
ASSERT FAILURE: c:\work\winafl\source\winafl.c:947: (0) (invalid option)
..\..\DynamoRIO-Windows-7.1.0-1\bin32\drrun.exe -c winafl.dll debug -target_module test_gdiplus.exe -target_method main -fuzz_iteration 10 --nargs 2 -- test_gdiplus.exe in\q
When I try use afl-fuzz - I get the same.
What is the cause of the problem?
I use windows 7 32bit

Hey, as the error message suggests, the command line seems a bit off. What you probably want is
..\..\DynamoRIO-Windows-7.1.0-1\bin32\drrun.exe -c winafl.dll -debug -target_module test_gdiplus.exe -target_method main -fuzz_iterations 10 -nargs 2 -- test_gdiplus.exe in\q

Hey, as the error message suggests, the command line seems a bit off. What you probably want is
..\..\DynamoRIO-Windows-7.1.0-1\bin32\drrun.exe -c winafl.dll -debug -target_module test_gdiplus.exe -target_method main -fuzz_iterations 10 -nargs 2 -- test_gdiplus.exe in\q

Ohhh, sorry.  Just copy/paste to vm didn't works and I didnt reprint correctly. Now it works correctly, thank!!!


Fixes 1

Fixed parsing date while getting tweets
get_tweets got crude support of search/filter parameters (refer to Nitter search URLs for params)
get_tweets got proxy support
get_tweets now parses cards (title, description and image)
get_profiles got proxy support

@dgnsrekt Any chance of merging this pull request? The date parser isn't working anymore for get_tweets. This would fix it.


[10pt] Address Single Pixel Inundation Issue

While developing the Categorical FIM datasets, we have noticed an issue consistent across many AHPS sites where inundation along a portion of the stream is restricted to a single pixel width. There are probably multiple reasons why this is occurring. The purpose of this ticket is to diagnose and fix as many of the occurrences as we can. Further tickets may be created to address this issue depending on the findings of this task.
Screenshots

Check nws_lid='CAMN4' for single pixel inundation related to large drop in thalweg elevation during hydroconditioning.


No error on logging.Logger.warn access [G010]

Before creating this issue I searched for:
G010
logger warn
logger.warn

but could not find previous issue or PR.
Minimal code snippet (ruff_G010_no_call.py) that reproduces the bug:
"""No G010 on logging.Logger.warn access."""
import logging

logger = logging.getLogger()
warn = logger.warn
warn("Hello, world!")

I invoked ruff with:
$ ruff check ruff_G010_no_call.py --isolated --select G010 --target-version py313
$ ruff check ruff_G010_no_call.py --isolated --select ALL --target-version py313

but got no errors.
I expected to get G010 (https://docs.astral.sh/ruff/rules/logging-warn/) on line warn = logger.warn. The code like this is quite wide-spread, here are the results of GitHub search that show code most likely to be invalid in Python 3.13, but that won't be detected with the current version of ruff: https://github.com/search?q=%2F(%3F-i)(logger|logging).warn%24%2F+language%3APython&type=code
I installed ruff from PyPI first, then from https://github.com/astral-sh/ruff/archive/refs/heads/main.zip:
$ python3 --version
Python 3.13.0b3
$ ruff --version
ruff 0.5.1

As a side note, current stable version of mypy does not find this issue either. mypy installed from master can find this issue though.

This is known -- we use a heuristic to detect if a call is a "logging call", and we don't trace name symbols like that. I can't find an obvious issue to link this to though, so I'll leave it as-is.


[UI] No way to save an animation from AnimationPlayer

I spent some time creating an animation in the player. Then I decided it'd be better to transfer it to another scene, so I save the animation... except there's no button in the UI to do that!*
That's really disheartening, specially knowing that it was possible before.
You can create animations, duplicate, rename them and even load them from disk, but no way to save it. If there is, it's very hard to find, because I couldn't.
My workaround was to create a new one from the Inspector panel, save it, load it in the AnimationPlayer, copy the animation from the original and paste on the loaded one.
*Added some dramatic effect, as it was actually a very simple animation.

@vnen I assume you are using nighties? Since from what I remember in 1.1 this button is still there..?

In 1.1 the animation was shown both on the Inspector and on the Resources panels (in which you could save it). Now we don't have the Resources panel (and I don't miss it), but the animation is not opened in the Inspector.
I think the animation should be shown in the inspector and the player should have a save button, since it already have a load button.

+1

+1

Fixed by 56b7567.


Replace usage of push_all and resize with extend

This allows removal of the collections feature gate.
Builds and tests on beta-1.0.0.
Please note that extend is slower then push_all currently: https://github.com/rust-lang/rust/issues/24028#issuecomment-89455384, but the error message says that extend will probably be the future way to go.

I think it can work, but it may be a good idea to wait a bit for extend to get faster, or push_all to be reimplemented, what do you think?

Hm, I tried finding any substance to that claim of extend/iter being slower, and I didn't find any.
I ran your mp4 parser on bigbuck.mp4 and the beta version with this implementation is in the same range, even a bit faster in debug mode (??). Take it with a grain of salt, as I hacked around nom_benchmark a little, but given that extend is stable and ready now, I'd prefer that in this case.

I think now it is a better idea to support beta. The performance hit will only affect producers and consumers, and there are other ways to optimize them.
Thank you for fixing this part of the code :)


Cleaning

cleaning:

NULL -> nullptr
.size() == 0 -> .empty()
typedef -> using
std::map<A, B>::const_iterator -> auto
auto A; A = B; -> auto A = B
Phase1::read_direct(int) -> Phase1::Phase1(int)
Phase1::read_file(F) -> Phase1::Phase1(F)
int 0/1 -> bool false/true
std::vector a; a.resize(int) -> std::vector a(int)
int i; for(i = 0; ...) -> for(int i = 0; ...)
(char*)0 -> nullptr
A::A() {} -> A::A() = default;
virtual A::~A() {} -> A::~A() = default;
class/struct: initialize variable in definition not in ctor
typedef struct A {} A; -> struct A{};

CI_BRANCHES:NEURODAMUS_CORE_BRANCH=sandbox/weji/report_multipop,PYNEURODAMUS_BRANCH=sandbox/weji/report_multipop,NEURON_BRANCH=olupton/nvhpc-t13,

Please retest

Codecov Report

Merging #648 (c172dfa) into master (cea6c0a) will decrease coverage by 0.08%.
The diff coverage is 51.21%.

@@            Coverage Diff             @@
##           master     #648      +/-   ##
==========================================
- Coverage   54.12%   54.03%   -0.09%
==========================================
  Files         102      102
  Lines        8397     8342      -55
==========================================
- Hits         4545     4508      -37
+ Misses       3852     3834      -18

Impacted Files
Coverage Δ

coreneuron/io/core2nrn_data_return.cpp
1.29% <ø> (ø)

coreneuron/io/global_vars.cpp
49.33% <ø> (ø)

coreneuron/io/nrn2core_data_init.cpp
0.64% <0.00%> (ø)

coreneuron/io/phase1.hpp
100.00% <ø> (ø)

coreneuron/io/prcellstate.cpp
1.27% <0.00%> (+0.02%)
:arrow_up:

coreneuron/mechanism/mech_mapping.cpp
55.55% <ø> (ø)

coreneuron/mechanism/patternstim.cpp
0.00% <0.00%> (ø)

coreneuron/mechanism/register_mech.cpp
64.73% <ø> (ø)

coreneuron/permute/balance.cpp
7.69% <0.00%> (+0.54%)
:arrow_up:

coreneuron/sim/scopmath/crout_thread.cpp
0.00% <0.00%> (ø)

... and 23 more

Continue to review full report at Codecov.

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

I merged master and resolved a trivial conflict with https://github.com/BlueBrain/CoreNeuron/pull/652.

Retest this please Jenkins.

Retest this please


Read only boolean not working

// getCellContent()
return {
  kind: GridCellKind.Boolean,
  data: false,
  allowEdit: false,
  allowOverlay: false,
}

// onCellEdited()
([row, col], value) => {
  console.log("SHOULD NOT HAPPEN!")
}

How do I make a readonly Boolean cell?

Also, any reason why its called allowEdit and not readonly like all other cells?

Unfortunately it just predates the read only flag. Let me see if I can fix both these issues.

I can help.
Shall I add an readonly flag, and mark allowEdit as depreciated?

Yes but both flags must work. If both are set readonly takes precedence.
On Sat, Apr 16, 2022 at 11:31 PM Adithya Dsilva @.***>
wrote:

I can help.
Shall I add a readonly flag, and mark allowEdit as depreciated?
—
Reply to this email directly, view it on GitHub
https://github.com/glideapps/glide-data-grid/issues/268#issuecomment-1100814736,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAHN2ZIRK27KSK2DO2X44DVFOV5FANCNFSM5TS3AUWQ
.
You are receiving this because you commented.Message ID:
@.***>

Fixed in 4.0.3, it would be nice to get a PR up for readonly but at least the behavioral issue is fixed.


Sandboxed systemd service

Should resolve #312, relates to discussion #51. Attempts to sandbox chiadog as a systemd service, and safeguard any sensitive chia files in .chia/mainnet that we don't need access to.

Add a scripts/linux/chiadog.service systemd example that attempts to run chiadog in a more isolated environment. Create a new limited user each run (making much of the filesystem readonly), and set .chia/mainnet folders other than log to inaccessible.
Move the offset file - previously, the debug.log.offset file was kept in the chiadog directory, but when running in a read only filesystem we can't write there. Create a temporary directory when running, and store the offset file there. This also means we no longer need to delete the offset file on startup.

A good way to test this is to switch out the ExecStart for an ls -lh .chia/mainnet and see what files are accessible inside the environment, or try to run venv/bin/chia wallet show inside and make sure it fails. Ideally I'd like to have chiadog show a warning when it starts up if it's able to access configuration directories, but I figure that could be added in another PR.
There are a bunch of different strategies in systemd for setting this up, I went with what seems to be the most compatible version (compatible with systemd 235, which I believe is available in Ubuntu 18), explicitly declaring InaccessiblePaths. The other way, using TemporarilyFilesystem to block everything out and binding the one file we need, is maybe a bit more elegant, but needs at least systemd 238 (around Ubuntu 20). Since it's not possible to say "make everything except the logs folder inaccessible" however (declaring .chia as inaccessible makes all child directories inaccessible and can't be overridden by other directives), we need to remember to add any new paths if new versions of chia add additional directories we don't want access to.

Accidentally pointed at main


Error in event handler for (unknown): TypeError: Cannot read property 'left' of undefined

Excuse me,there has a probleme: Error in event handler for (unknown): TypeError: Cannot read property 'left' of undefined
I used multi-select in my project and Chorme console will log this problem，and also in your website too
could you please tell me how to fix it?  thanks！

What action are you taking to trigger this error as I do not see it on page load.


[llava] How to clear imagepaths ?

How to clear the embedded images from the context ?
Apparently, calling executor.ImagePaths.Clear() has no effect on the private variable _embeds

FYI @SignalRT

@dchatel, In my knowledge there is no LlavaAPI to reset images from the context (I could be wrong).
Right now the option that I can think about  is to create a new context / executor when the user prompts new images. Here you have the example modified:
https://github.com/SignalRT/LLamaSharp/blob/LlavaResetIContext/LLama.Examples/Examples/LlavaInteractiveModeExecute.cs
@AsakusaRinne , If this is OK to you I can make a PR with this change.

@SignalRT According to the code, LLavaExecutor clear the image embeds when it has finished to generate the output. Is it possible to fix this issue based on this? (Please correct me if I'm wrong)

I will review if there is a better way to manage this.

I will review if there is a better way to manage this.

Maybe a good way would be to clear the image embeds when ImagePaths is cleared?...
Or reflect ImagePaths state into image embeds?

ImagePath it's clearing right now automatically at the end of InferInternal:
                    _EmbedImagePosition = -1;
                    _imageEmbedHandles.Clear();
                    ImagePaths.Clear();

I'm trying some options searching for "continuity" adding or changing images during conversation.
I get something way better, but I need to solve some cases before to make a PR.

Will llama_kv_cache_seq_rm can help? When embding image/prompt, we can get n_past, will it can be recorded as the positison. (I could be wrong).
`// Removes all tokens that belong to the specified sequence and have positions in [p0, p1)
// seq_id < 0 : match any sequence
// p0 < 0     : [0,  p1]
// p1 < 0     : [p0, inf)
LLAMA_API bool llama_kv_cache_seq_rm(
struct llama_context * ctx,
llama_seq_id   seq_id,
llama_pos   p0,
llama_pos   p1);`

@IntptrMax, it’s what I’m working on, I also introduce the capability to add images in the middle of conversation that’s something that is not possible with master code. I expect to have the complete solution this week.

@dchatel , @IntptrMax  In this PR #664 I introduce the idea to clear the images and to change the images during conversation.


[WIP] Added new getters

Looks good, but maybe add docstrings? Or we can do it in a separate PR, since the rest of the module needs them too.


Get Followers Method gives error

When I call try handler.getUserFollowers(userId: userId, maxId: nextPageId, searchQuery: "", completion: { (result, maxId)
after 298 nd request its gaves error like:
Result(isSucceeded: false, info: SwiftyInsta.ResultInfo(error: SwiftyInsta.CustomErrors.noError, message: "The operation couldn’t be completed. (SwiftyInsta.CustomErrors error 9.)", responseType: SwiftyInsta.ResponseTypes.wrongRequest), value: nil)
PS: I already remove delay. With delay method it is working

Instagram Private API are not intended to "spam" requests, cause they should only be called after user interaction.
It's most likely just Instagram telling you you've made too many requests.
Delay has been implemented to avoid exactly that.

That's because of Instagram rate limit, we warned you about this before.
As @sbertix said, the API designed to work with user interaction. So a rate limit error raised when you sen these number of requests.