Add support for the embassy UCPD driver (STM32)

Hello,
thank you for writing this library!
I am in the process of using this with the UCPD peripheral on STM32 (I will test with STM32G4) on embassy for implementing a Sink.
I have some ideas in mind for changes to the sink's Driver trait, because I have some trouble to use UCPD with it as is:

Replace poll/get_pending_message with receive_message, because in UCPD, both steps are handled with a single call to receive().await on the PD phy instance. I can also refactor this in the FUSB302 driver. The functions are always used together in Sink::poll(), so I think it makes sense.
Add a Result return type to send_message and receive_message and propagate errors to the user application (might drop the driver and make a new one on error)

Some general ideas:

Remove did_change_protocol from the Driver trait and instead check if the state of the Driver changed from within the Sink.

What do you think?

I added a PR for showing those ideas.

Thanks so much, yeah that all sounds great:) I can't test any FUSB301B regressions, but I'll merge the PR and fix anything that breaks later, if that helps unblock your work.

It's not blocking me at all, I can just use my fork :D
My intention is to make your library a bit more generic, if you agree to the proposed changes. Unfortunately, I also have no way of testing on FUSB301B.
I will likely open another PR later with some more things that we can discuss.

@elagil I’m fully on board, I’d love to make this correctly-abstracted so that core PD logic can be shared with as many device implementations as possible.

I will likely open another PR later with some more things that we can discuss.

Sounds great!

@elagil IIRC, those are required for timing out if negotiations don't occur within 500ms, and toggling CC lines every 20ms

@fmckeogh I am asking because they are not actually used in the library. Am I missing something?

@elagil My apologies, I thought you were referring to the Instant timer logic, if you mean the usb_pd::timers module, @bentwire added that I assumed for future USB PD standard work

Yes, sorry I've been out of the loop.  Only doing this as a hobby and life got in the way.
Those timers were supposed to be for the pd state machine yes.


Banner redesign

The banner needs to be redesigned.
Adding to the backlog for now since we'll probably need new assets from design when the time comes.

We got assets! No longer blocked :p

We got assets! No longer blocked :p

https://www.figma.com/file/X7ZuxzntCJwG6Ae8kGztcm/HOTH-8?node-id=0%3A1

https://www.figma.com/file/X7ZuxzntCJwG6Ae8kGztcm/HOTH-8?node-id=0%3A1


[rcl.logging_rosout]: Publisher already registered for provided node name.

[WARN] [1703083924.789427952] [rcl.logging_rosout]: Publisher already registered for provided node name. If this is due to multiple nodes with the same name then all logs for that logger name will go out over the existing publisher. As soon as any node with that name is destructed it will unregister the publisher, preventing any further logs for that name from being published on the rosout topic.

tony@iqr-turtlebot4-121:~/EE211_ws/src/tb4_find_pickup_place$ /bin/python3 /home/tony/EE211_ws/src/tb4_find_pickup_place/navigation/pose_reach.py
[INFO] [1703213070.097758493] [basic_navigator]: amcl/get_state service not available, waiting...
[INFO] [1703213071.099617823] [basic_navigator]: amcl/get_state service not available, waiting...

发现有多个节点重名：/basic_navigator、/camera/camera、/left_wheel_drop_stf、/modbus_rtu_driver_node、/right_wheel_drop_stf、/robot_state_publisher、/rplidar_node、/serial_bridge_node。
tony@iqr-turtlebot4-121:~$ ros2 node list
WARNING: Be aware that are nodes in the graph that share an exact name, this can have unintended side effects.

参考教程：

https://github.com/ros-planning/navigation2/issues/2283
https://github.com/ros2/rviz/issues/671
https://github.com/turtlebot/turtlebot4_tutorials/issues/2

疑似/amcl节点被占用，该Bug并不能稳定复现。
参考教程：

https://github.com/ros-planning/navigation2/issues/2283
https://github.com/ros2/rviz/issues/671
https://github.com/turtlebot/turtlebot4_tutorials/issues/2


Fix shell portability issue on NetBSD

You realize this cset patches 'src/gui/tcltk/tcl/win/configure' for NetBSD compatibility and yet
that is the config file for building on Windows?
We have NetBSD in our build cluster and ship NetBSD binaries.
That said, I agree using == with 'test' on a shell script is bad practice.

pkgsrc scans shell scripts and blocks when finds common portability problems. It blocked the build due to this '=='.

Interesting.
OK. I will make the change in our bk source repository and get it included in the next release.

The current release includes this patch.  Thanks!


fix(readme.md): update readme

:tada: This PR is included in version 1.42.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


feat: secscan cli command to run known-bad model checks

Description
This PR fixes an issue that was reported to me, that wasn't inside LocalAI itself, but rather the model.
As we provide ways for users to browse and install models, its for the best if we have some way to warn users if the model in question is known to be vulnerable to security issues.
While it's an impossible goal to catch ever possible issue, HuggingFace at least tags some repos when they determine them to be unsafe - so for now, that's as far as this check goes. As an example (DO NOT USE THIS MODEL) https://huggingface.co/tulayaka/loadtest/tree/main
@mudler is working on some new gallery pages, so I prioritized finishing this very limited PR up. There is now a manual cli command that will scan all galleries for installed vulnerable models, and mudler's new local-ai models install passes the gallery through the filter before install as a sanity check.

Closing this PR in favor of using the scan endpoint (example)


New zsh session throws "fasd: function definition file not found"

Please check the existing issues to make sure you're not duplicating a report.
For bug reports, please provide the following information:
In a terminal, run zmanage info and paste the output below:
Zim commit ref:   abbeeb9
Zsh version:      zsh 5.4.2 (x86_64-apple-darwin16.7.0)
System info:      Darwin YS-MacBook-Pro.local 17.2.0 Darwin Kernel Version 17.2.0: Fri Sep 29 18:27:05 PDT 2017; root:xnu-4570.20.62~3/RELEASE_X86_64 x86_64

gist of .zimrc:

#################
# CORE SETTINGS #
#################
#
# Zim settings
#
# Select what modules you would like enabled.
# The second line of modules may depend on options set by modules in the first line.
# These dependencies are noted on the respective module's README.md.
zmodules=(directory environment git git-info archive debug ssh history input utility meta custom \
          syntax-highlighting history-substring-search prompt spectrum completion fasd)

###################
# MODULE SETTINGS #
###################

#
# Prompt
#

# Set your desired prompt here
zprompt_theme='gitster'

#
# Completion
#

# set an optional host-specific filename for the completion cache file
# if none is provided, the default '.zcompdump' is used.
zcompdump_file=".zcompdump-${HOST}-${ZSH_VERSION}"

#
# Utility
#

# Uncomment to enable command correction prompts
# See: http://zsh.sourceforge.net/Doc/Release/Options.html#Input_002fOutput
setopt CORRECT

#
# Input
#

# Uncomment to enable double-dot expansion.
# This appends '../' to your input for each '.' you type after an initial '..'
zdouble_dot_expand='true'

#
# Syntax-Highlighting
#

# This determines what highlighters will be used with the syntax-highlighting module.
# Documentation of the highlighters can be found here:
#   https://github.com/zsh-users/zsh-syntax-highlighting/blob/master/docs/highlighters.md
# For (u)rxvt, termite and gnome-terminal users,
# removing the 'cursor' highlighter will fix the disappearing cursor problem
zhighlighters=(main brackets cursor)

#
# SSH
#

# Load these ssh identities with the ssh module
zssh_ids=(id_rsa)

#
# Pacman
#

# Set (optional) pacman front-end.
#zpacman_frontend='powerpill'

# Load any helper scripts as defined here
#zpacman_helper=(aur)

# Customize to your needs...
bindkey -s '^l' 'clear\n'
# tmux
alias tmux='tmux -2'
alias tma='tmux attach -t -d'
alias tmn='tmux attach -t $(basename $(pwd)) || tmux new -s $(basename $(pwd))'
alias tml='tmux list-sessions'
alias zshconf="vim ~/.zshrc"
alias la="ls -la"
alias less=$PAGER
alias zless=$PAGER
alias ccat='pygmentize -g $1'
alias fuck='eval $(thefuck $(fc -ln -1 | tail -n 1)); fc -R'
alias gr='[ ! -z `git rev-parse --show-cdup` ] && cd `git rev-parse --show-cdup || pwd`'
alias ems="emacs --daemon"
# # IP addresses
# alias ip="dig +short myip.opendns.com @resolver1.opendns.com"
alias localip="ipconfig getifaddr en0"

export LC_ALL="en_US.UTF-8"
# Add homebrew to the completion path
# enable colored output from ls, etc
export CLICOLOR=1
# expand functions in the prompt
setopt prompt_subst
# why would you type 'cd dir' if you could just type 'dir'?
setopt AUTO_CD
# Now we can pipe to multiple outputs!
setopt MULTIOS

# This makes cd=pushd
setopt AUTO_PUSHD

# This will use named dirs when possible
setopt AUTO_NAME_DIRS

# If we have a glob this will expand it
setopt GLOB_COMPLETE
setopt PUSHD_MINUS

# No more annoying pushd messages...
setopt PUSHD_SILENT

# blank pushd goes to home
setopt PUSHD_TO_HOME

# this will ignore multiple directories for the stack. Useful? I dunno.
setopt PUSHD_IGNORE_DUPS

# 10 second wait if you do something that will delete everything. I wish I'd had this before...
setopt RM_STAR_WAIT

# use magic (this is default, but it can't hurt!)
setopt ZLE

setopt NO_HUP

# only fools wouldn't do this ;-)
# export EDITOR="subl -n -w"
setopt IGNORE_EOF

# If I could disable Ctrl-s completely I would!
setopt NO_FLOW_CONTROL

# Keep echo "station" > station from clobbering station
setopt NO_CLOBBER

# Case insensitive globbing
setopt NO_CASE_GLOB

# Be Reasonable!
setopt NUMERIC_GLOB_SORT

# I don't know why I never set this before.
setopt EXTENDED_GLOB

# hows about arrays be awesome? (that is, frew${cool}frew has frew surrounding all the variables, not just first and last
setopt RC_EXPAND_PARAM

# Who doesn't want home and end to work?
bindkey '\e[1~' beginning-of-line
bindkey '\e[4~' end-of-line

# Additional zsh plugins
test -e "${HOME}/.iterm2_shell_integration.zsh" && source "${HOME}/.iterm2_shell_integration.zsh"

# zsh-completions
fpath=(/usr/local/share/zsh-completions $fpath)

# zsh-syntax-highlighting
source /usr/local/share/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh

# zsh-autosuggestions
source /usr/local/share/zsh-autosuggestions/zsh-autosuggestions.zsh

# zsh-navigation-tools
fpath+=( /usr/local/share/zsh-navigation-tools )
source /usr/local/share/zsh-navigation-tools/zsh-navigation-tools.plugin.zsh

# zsh gulp autocompletion
# source /Users/yeongsheng/.gulp-autocompletion-zsh/gulp-autocompletion.zsh

alias naliases=n-aliases ncd=n-cd nenv=n-env nfunctions=n-functions nhistory=n-history
alias nkill=n-kill noptions=n-options npanelize=n-panelize
autoload znt-history-widget
zle -N znt-history-widget
bindkey "^R" znt-history-widget
zle -N znt-cd-widget
bindkey "^B" znt-cd-widget
zle -N znt-kill-widget
bindkey "^Y" znt-kill-widget

# aws-cli zsh completion
# source /usr/local/share/zsh/site-functions/_aws

# zsh-bd
source ${HOME}/.zsh/plugins/bd/bd.zsh

# fasd
alias a='fasd -a'        # any
alias s='fasd -si'       # show / search / select
alias d='fasd -d'        # directory
alias f='fasd -f'        # file
alias sd='fasd -sid'     # interactive directory selection
alias sf='fasd -sif'     # interactive file selection
alias z='fasd_cd -d'     # cd, same functionality as j in autojump
alias zz='fasd_cd -d -i' # cd with interactive selection

gist of .zshrc:

#
# User configuration sourced by interactive shells
#

# Source zim
if [[ -s ${ZDOTDIR:-${HOME}}/.zim/init.zsh ]]; then
  source ${ZDOTDIR:-${HOME}}/.zim/init.zsh
fi
# The following lines were added by compinstall
zstyle :compinstall filename '/Users/yeongsheng/.zshrc'
fpath=("/usr/local/bin/" $fpath)

export TERM=xterm-256color

autoload -Uz compinit
compinit
# End of lines added by compinstall
zstyle :compinstall filename '${HOME}/.zshrc'

fasd_cache="${HOME}/.fasd-init-zsh"
if [ "$(command -v fasd)" -nt "${fasd_cache}" -o ! -s "${fasd_cache}" ]; then
  fasd --init posix-alias posix-hook zsh-hook zsh-ccomp zsh-ccomp-install zsh-wcomp zsh-wcomp-install >| "${fasd_cache}"
fi
source "${fasd_cache}"
unset fasd_cache

# SCALA_HOME
export SCALA_HOME=/usr/local/opt/scala/idea

# DOCKER HOST
export DOCKER_HOST="tcp://<IP_ADDRESS>:4243"

# Add /usr/sbin for new location of mosquitto bin
export PATH=${PATH}:/usr/local/sbin

# Add imagemagick to PATH
# export PATH=${PATH}:/usr/local/opt/imagemagick/bin
export PATH="/usr/local/opt/imagemagick@6/bin:$PATH"
export PKG_CONFIG_PATH=/usr/local/opt/imagemagick/lib/pkgconfig

# Activate erlang@20 erlang shell history
export ERL_AFLAGS="-kernel shell_history enabled"

# yarn
export PATH="${HOME}/.yarn/bin:${PATH}"

# Rust/racer/rustfmt
export PATH="${HOME}/.cargo/bin:${PATH}"

# chruby
# source /usr/local/chruby/share/chruby/chruby.sh
source /usr/local/share/chruby/chruby.sh
source /usr/local/share/chruby/auto.sh
# source "$HOME/.rvm/scripts/extras/chruby.sh"
# source /Users/yeongsheng/.gulp-autocompletion-zsh/gulp-autocompletion.zsh

# zsh-auto-completion
source ~/.zsh/zsh-autosuggestions/zsh-autosuggestions.zsh
# zsh-syntax-highlighting
export ZSH_HIGHLIGHT_HIGHLIGHTERS_DIR=/usr/local/share/zsh-syntax-highlighting/highlighters
source /usr/local/share/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh

# Increase ulimit for mongodb, redis and riak
ulimit -n 65536 200000

test -e "${HOME}/.iterm2_shell_integration.zsh" && source "${HOME}/.iterm2_shell_integration.zsh"

# emacs-client with elixir 1.5.x oepn command
export ELIXIR_EDITOR="emacsclient-elixir +__LINE__ __FILE__"

# fzf via Homebrew
# fzf via local installation
_has() {
  type "$1" &> /dev/null
}

if [ -e ~/.fzf ]; then
  _append_to_path ~/.fzf/bin
  source ~/.fzf/shell/key-bindings.zsh
  source ~/.fzf/shell/completion.zsh
fi

# fzf + ag configuration
if _has fzf && _has ag; then
  export FZF_DEFAULT_COMMAND='ag --nocolor -g ""'
  export FZF_CTRL_T_COMMAND="$FZF_DEFAULT_COMMAND"
  export FZF_ALT_C_COMMAND="$FZF_DEFAULT_COMMAND"
  export FZF_DEFAULT_OPTS='
  --color fg:242,bg:236,hl:65,fg+:15,bg+:239,hl+:108
  --color info:108,prompt:109,spinner:108,pointer:168,marker:168
  '
fi

[ -f ~/.fzf.zsh ] && source ~/.fzf.zsh
# Options to fzf command
export FZF_COMPLETION_OPTS='+c -x'

Description

Each new zsh session throws below warning/error:

${HOME}/.zim/modules/fasd/init.zsh:1: fasd: function definition file not found

Steps to reproduce

Install zim (git hash abbeeb9d375560b0e10c782e64970f22ff0dce11)
Enable zmodule 'fasd'
Launch a new zsh session

Images or other information

Nil.

I got the error on the first shell session after adding fasd to zmodules, but not on subsequent ones. This may be because of how we zcompile the functions in login_init.zsh, that is sourced by .zlogin.
@yeongsheng-tan, make sure you've updated your .zlogin file to match the latest zlogin template.
Still, this may be one of the only scenarios where a function zcompiled by login_init.zsh (sourced by .zlogin) is being called by a module initialized by init.zsh (sourced by .zshrc). The .zshrc is executed before the .zlogin (see http://zsh.sourceforge.net/Intro/intro_3.html), hence the error.
TL;DR We are zcompiling code after initialization
@Eriner, thoughts? (Maybe we should be zcompiling code before initialization)

Hi @ericbn ,
thank you for the tip.
My ~/.zlogin looks like:

#
# startup file read in interactive login shells
#
# The following code helps us by optimizing the existing framework.
# This includes zcompile, zcompdump, etc.
#

(
  # Function to determine the need of a zcompile. If the .zwc file
  # does not exist, or the base file is newer, we need to compile.
  # These jobs are asynchronous, and will not impact the interactive shell
  zcompare() {
    if [[ -s ${1} && ( ! -s ${1}.zwc || ${1} -nt ${1}.zwc) ]]; then
      zcompile ${1}
    fi
  }

  zim_mods=${ZDOTDIR:-${HOME}}/.zim/modules
  setopt EXTENDED_GLOB

  # zcompile the completion cache; siginificant speedup.
  for file in ${ZDOTDIR:-${HOME}}/.zcomp^(*.zwc)(.); do
    zcompare ${file}
  done

  # zcompile .zshrc
  zcompare ${ZDOTDIR:-${HOME}}/.zshrc

  # zcompile some light module init scripts
  zcompare ${zim_mods}/git/init.zsh
  zcompare ${zim_mods}/utility/init.zsh
  zcompare ${zim_mods}/pacman/init.zsh
  zcompare ${zim_mods}/spectrum/init.zsh
  zcompare ${zim_mods}/completion/init.zsh
  zcompare ${zim_mods}/fasd/init.zsh

  # zcompile all .zsh files in the custom module
  for file in ${zim_mods}/custom/**/^(README.md|*.zwc)(.); do
    zcompare ${file}
  done

  # zcompile all autoloaded functions
  for file in ${zim_mods}/**/functions/^(*.zwc)(.); do
    zcompare ${file}
  done

  # syntax-highlighting
  for file in ${zim_mods}/syntax-highlighting/external/highlighters/**/*.zsh; do
    zcompare ${file}
  done
  zcompare ${zim_mods}/syntax-highlighting/external/zsh-syntax-highlighting.zsh

  # zsh-histery-substring-search
  zcompare ${zim_mods}/history-substring-search/external/zsh-history-substring-search.zsh

) &!

Do I just stick the latest zlogin template at the top of the line above?

@ericbn I went ahead and stuck the latest zlogin template to the top of my ~/.zlogin and my first fresh shell session threw the below:
/Users/yeongsheng/.zim/modules/fasd/init.zsh:1: fasd: function definition file not found
zrecompile:zcompile:99: zwc file has wrong version (zsh-5.3.1): /Users/yeongsheng/.zim/modules/custom/init.zsh.zwc
zrecompile:zcompile:99: zwc file has wrong version (zsh-5.3.1): /Users/yeongsheng/.zim/modules/spectrum/init.zsh.zwc
➜ ~ zrecompile:zcompile:99: zwc file has wrong version (zsh-5.3.1): /Users/yeongsheng/.zim/modules/fasd/init.zsh.zwc
zrecompile:zcompile:99: zwc file has wrong version (zsh-5.3.1): /Users/yeongsheng/.zim/modules/prompt/functions/prompt_liquidprompt_setup.zwc
zrecompile:zcompile:99: zwc file has wrong version (zsh-5.3.1): /Users/yeongsheng/.zim/modules/syntax-highlighting/external/highlighters/brackets/brackets-highlighter.zsh.zwc
zrecompile:zcompile:99: zwc file has wrong version (zsh-5.3.1): /Users/yeongsheng/.zim/modules/syntax-highlighting/external/highlighters/cursor/cursor-highlighter.zsh.zwc
zrecompile:zcompile:99: zwc file has wrong version (zsh-5.3.1): /Users/yeongsheng/.zim/modules/syntax-highlighting/external/highlighters/line/line-highlighter.zsh.zwc
zrecompile:zcompile:99: zwc file has wrong version (zsh-5.3.1): /Users/yeongsheng/.zim/modules/syntax-highlighting/external/highlighters/main/main-highlighter.zsh.zwc
zrecompile:zcompile:99: zwc file has wrong version (zsh-5.3.1): /Users/yeongsheng/.zim/modules/syntax-highlighting/external/highlighters/pattern/pattern-highlighter.zsh.zwc
zrecompile:zcompile:99: zwc file has wrong version (zsh-5.3.1): /Users/yeongsheng/.zim/modules/syntax-highlighting/external/highlighters/root/root-highlighter.zsh.zwc
zrecompile:zcompile:99: zwc file has wrong version (zsh-5.4.1): /Users/yeongsheng/.zim/modules/syntax-highlighting/external/zsh-syntax-highlighting.zsh.zwc
zrecompile:zcompile:99: zwc file has wrong version (zsh-5.3.1): /Users/yeongsheng/.zim/modules/history-substring-search/external/zsh-history-substring-search.zsh.zwc

Subsequent sessions came up clean and I no longer have the above errors/warnings, nor am I anymore plagued by the original issue:
${HOME}/.zim/modules/fasd/init.zsh:1: fasd: function definition file not found

👍 💯 😍

@ericbn could zmanage update have taken care of updating ~/.zlogin appropriately?

@yeongsheng-tan, you can replace the whole content of your .zlogin with the contents of the new template (instead of prepending them), since all you have is zim's previous code there.
The main reason we did these changes are to allow next updates to be taken care of by zmanage update.
Still, there's the function definition file not found error in the first shell session after cache is cleaned that needs to be addressed, so let's keep this issue open.


Java SDK - IndexOutOfBounds exception thrown when

In OpenFgaClient#writeTransaction(String storeId, ClientWriteRequest request, ClientWriteOptions writeOptions), if the write transactions and delete transactions are null, an IndexOutOfBounds exception is thrown when trying to get the response, because we call get(0):
var futureResponse = this.writeNonTransaction(storeId, transactions.get(0), options);

This issue was discovered in #307.
We should not fail with an IndexOutOfBounds exception in this case.

Fixed with #307


Version several tasks to match current sprint number

I'm not sure we should do this.  We should only do it if the task changed in that sprint.  Doing it arbitrarily every sprint has huge storage consequences (along with build impact, agent disk space etc...).
We also shouldn't do it unless we are consistent across all the tasks ... and only do it if we're changing and bumping the task anyways.

We don't intend to bump these versions unnecessarily every sprint, but want to set a baseline so that when we look at a task, we know in what sprint it last changed.  I want to set this baseline in S111 so that from TFS 2017 Update 1 and on, we have better correlation between a task's version and when it last changed.  Also, when a customer says they're on version 1.111.0, we'll have a sense for which quarterly update that aligns with, compatibility, etc.  I asked @ericsciple if we should do the same thing for other tasks: set a baseline, sprint-aligned version starting S111.

I'm on board knowing when it changed.  So guideline is when changing ...

Makes sense


End of Day

Date: 4/26/2016
Time: 8:11
Day started yesterday at 5:09 PM - I did not press end day button because it was not available at 10:00 PM when I went to bed. I assumed it would show me a start day button automatically after 12:00 AM but it did not. I just pressed End Day and will now have to wait the 4 hrs to see the start day button.

@DanielaGarza This is the current expected behavior.  A future version will contain the new SOD/EOD implementation that will address this concern.


Expose a metric with the current value of sample_limit per scrape pool

For target limit there is a metric prometheus_target_scrape_pool_target_limit that can be used to check how close given pool is to the limit. There is no such metric for sample limit, which makes it hard to query for scrape pools close to configured sample limit.
Add prometheus_target_scrape_pool_sample_limit exposing the value of sample_limit scrape config option.
Signed-off-by: Łukasz Mierzwa<EMAIL_ADDRESS>

Thanks for your PR. This is making the incorrect assumptions that Prometheus is scraping itself, and that the name of the scrape config matches the job label of the target. Put another way, you can't mix ingested per-target metrics with per-Prometheus metrics like this usefully.
To be useful this would have to be ingested with the scrape, however that's an additional overhead for every single target for all users of Prometheus which is too high a cost for the benefits this would bring.

Both sample_limit and target_limit scrape config options are set globally for the entire scrape job. If it's valid to expose target_limit as a metric then why isn't it valid to do the same for sample_limit?
Seems like you're saying I wouldn't be able to use this metric in any practical way, which is probably related to the fact that prometheus_target_scrape_pool_sample_limit  metric would have scrape_job label, while scrape_samples_post_metric_relabeling that I could use to query for the number of samples scraped from each target will have job label. The mismatch makes it harder but it feels justified - sample_limit metric describes the scrape config configuration while scrape_samples describes results from specific target of that scrape config, so these are on 2 different level of detail, hence the mismatch. But simple label_replace function would allow me to use these two metrics in a single query.
I do get the argument that it's not a straightforward for my intended use,  but I do believe it's still a useful metric (what if you just want to expose it on a dashboard?).

Both sample_limit and target_limit scrape config options are set globally for the entire scrape job. If it's valid to expose target_limit as a metric then why isn't it valid to do the same for sample_limit?

Target_limit applies to the scrape config as a whole, sample_limit applies individually to each target within the scrape config. There are no per-target metrics on the /metrics of Prometheus.

Seems like you're saying I wouldn't be able to use this metric in any practical way

Yes, it's not generally usable.

But simple label_replace function would allow me to use these two metrics in a single query.

Not in general, there's no restrictions in relation to what the job label can be.

I do get the argument that it's not a straightforward for my intended use, but I do believe it's still a useful metric (what if you just want to expose it on a dashboard?).

I don't see how that's of much use on a dashboard, and that is a very weak justification as it could be applied to anything. It's a hardcoded constant coming from your config file which doesn't relate to any other value on the /metrics.
This metric doesn't make sense to add, and would likely cause more confusion than anything else due to the above issues.

Fair enough

Is there some other way to query (for example) all targets that are >= 80% of sample_limit?

Sample limit is meant to be a coarse emergency relief valve, so will tend to have a very small number of potential values in a given setup. So hardcoding it in your alerting rule is typically fine.

Is there an alternative change I could work on?

Per the above, I don't see any reasonable way to expose this.

We looked at this in the bug scrub, and per the above there's no reasonable way to do this.


[Storage] Prune Block and Transaction Data

Changes

[x] refactor block identifier store/get so it can be reused for oldest block identifier
[ ] track oldest available block in block storage
[ ] Implement Prune method in BlockStorage
[ ] Create periodic pruning goroutine in stateful syncer

Pull Request Test Coverage Report for Build 7573

12 of 17 (70.59%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 77.749%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

storage/block_storage.go
12
17
70.59%

Files with Coverage Reduction
New Missed Lines
%

storage/block_storage.go
1
68.88%

Totals

Change from base Build 7556:
-0.02%

Covered Lines:
5940

Relevant Lines:
7640

💛  - Coveralls


Does GridDB support User Defined Function?

I am currently working on a benchmark of time series database systems for analytical (complex) queries. These queries are algorithms that cannot (easily) be implemented in SQL language so they require a third-party programming language such as C, C++, Python, Java, etc.
I am writing to you to ask if GridDB supports such a feature of a third-party programming language for User Defined Functions?

Please refer to griddb/griddb#360 .


Enhance JSON index to support nested array

Description
Enhance JSON index to support top-level value, top-level array, and nested array (multi-dimensional array)
Change JSON_MATCH filter expression to be JSONPath compatible (e.g. '"$.a[*].b.c[0]"=123')
Other misc fixes including null and empty array/object handling
In order to support nested array:

Json index format is changed and a new version (2) is added. Version 1 is deprecated, but still supported. New generated json index will be of version 2.
Array access must be explicit. In order to match any element in an array, [*] must be added to the filter, e.g. '"$.array[*]"=''foo'''.

Upgrade Notes
Please upgrade the servers before uploading the segments with the new index.
Release Notes
With the new JSON index, array wildcard access must be explicit

Codecov Report

Merging #6877 (cce9b2c) into master (1c09b78) will decrease coverage by 0.00%.
The diff coverage is 68.25%.

@@             Coverage Diff              @@
##             master    #6877      +/-   ##
============================================
- Coverage     65.48%   65.48%   -0.01%
  Complexity       12       12
============================================
  Files          1421     1421
  Lines         69980    69997      +17
  Branches      10112    10119       +7
============================================
+ Hits          45825    45835      +10
- Misses        20874    20878       +4
- Partials       3281     3284       +3

Flag
Coverage Δ
Complexity Δ

unittests
65.48% <68.25%> (-0.01%)
12.00 <0.00> (ø)

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

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


Rebuild for python311

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

matplotlib main builds failed, digging more though.

The failures are a known issue with Matplotlib 3.6.1, and should be fixed by Matplotlib 3.6.2. Regardless, there's nothing wrong here, and the packages that are failing aren't even relevant to this rebuild.

@dopplershift Is this the issue then? matplotlib/matplotlib#24094

Yes, and conda-forge/matplotlib-feedstock#332.

Wow! We are in cartopy already! This migration may be the first one to end in a few months instead of +1 year!


Question: Can the code be executed  on serverside?

Hi, iam very new to Angular and Javascript/Typescript.
In our app-browser which has some security modifications the lib methods are leading to crashes which can not be changed.
As the title says iam interessted to know if the library functions can be excuted on serverside ?

Yes, you can use this library on a server using Node or Deno.

you find the answer on the very first section in the documentation. https://github.com/Hopding/pdf-lib#motivation

@btecu and @alwins0n are spot on 👍


Relax requirements for generic conversion to TestCaseError

This permits using alternative error types which doesn't implement std::error::Error, like failure::Error directly in proptest blocks.
This is accomplished by relaxing the requirements on the blanket conversion mechanism to only require fmt::Display. Unfortunately this would cause TestCaseError to conflict with the blanket implementation, so it had to be reworked.

The breaking change is pretty unfortunate, though this otherwise looks like a good thing to have. For now, I'm going to keep this open under consideration for the next time we have a breaking release.

This got deleted when i switched from a master to main branch. I've cut a ticket to track this at #507


Accept objects supporting the file system path protocol in the Image class

Although this MR has goodwill, it would introduce undefined behavior, and cause more harm/confusion than value.
The pathlib is a great library for managing path context, but it's the users responsibility for casting to string, not wand's constructor. How would Wand know that the author intended for a relative path, or absolute path?
Accepting this change will also give a false sense of support for image protocols & Pseudo-image Formats that pathlib simple can not support.
For example...
 RSVG:<filename>
 http://example.org/image.png
 CAPTION:"Hello world!"
 xc:#FF00FF

Perhaps you can add a description that states the use-case, value/benefit addition, and business justification for this proposed updated.

Hey @emcconville, thanks for your comments. Actually this merge request is a bit too specific, it should be 'Accept objects supporting the file system path protocol' rather than specifically pathlib instances. I will fix this now.
As for your comments: Per PEP 519 (https://www.python.org/dev/peps/pep-0519/) any object with a __fspath__ dunder is a file system path. I think support for this should be added to wand purely on the basis that it is an accepted PEP. All os.path functions, open builtin and other stdlib modules are being updated to accept any object that supports the path protocol.
In short: the filename parameter accepts a file path, but rejects any file path objects that are not strings (such as a pathlib instance).
IMO it's pretty simple to add, you can use the os.fspath function to get the path string (or bytes) from the object. I mean, if you really don't want to add support for this then that's reasonable, but it's nice to treat objects that are telling you they are file paths as file paths.

I've updated the merge request. It boils down to adding this line:
filename = filename.__fspath__() if hasattr(filename, "__fspath__") else filename
Regarding your comments about pseudo image formats: I would say these are not file paths, so don't fall under the remit of this merge request. They can be passed as strings as-is.
This isn't saying 'Only pathlib instances can be passed', it's just allowing them to be passed in as-is. It makes code look a bit neater, which is always good.

Cool. Thanks for the justification, it does help understand what the goal of the change is. I think the __fspath__ check is definitely worth implementing.
I'm not against this, just throwing out a reminder that filename= methods are not PEP compliant, but ImageMagick's. This is a hybrid situation, and does require some additional care (even if it's just a one-liner).
I would suggest:

Create additional test cases over revising existing. This will give you the ability to mark a test case as skipped in the event of unsupported feature(s).
Be consistent about the change. Both reading & writing methods should be addressed, and not in the constructor. Perhaps closer to the MagickReadImage & MagickWriteImage methods (we're already copying/encoding the argument for ctypes, so it may be more appropriate there).
Documentation. What good is implementing a PEP if nobody knows about it.

True true! I'll work on doing what you suggest tomorrow. I was going to rebase on my 'fix-travis' branch to ensure it all works as expected.

I've included this, but moved it from the Image constructor to to the encode_filename. Sorry for 2 year delay


added faq; updated g oauth

Docs Build status updates of commit d1e4897:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Advertising/scripts/TOC.yml
:white_check_mark:Succeeded

Advertising/scripts/examples/authenticating-with-google-services.md
:white_check_mark:Succeeded

Advertising/scripts/guides/faq.md
:white_check_mark:Succeeded

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


Replace project links with shields

looks better (imo) and much easier to use on phones.

thanks, I hadn't seen the project page shield before :-)


Report Proper Function Name on Error

Now that we are moving to FL integration through the use of pointfree and helper functions, we need a way to signal to the user the path in which the error occurred. If using the FL methods, it should be reported as such.
That means if an error occurred using the "public" or "fluent" version of the function it should read:
Either.map: Function required
But if using the FL method, typically through a helper/pointfree function, the error should read:
Either.fantasy-land/map: Function required
This will require that the type functions that have FL equivalents must be partial applied with the name that will be in the method name space:
[Type].[methodName]: Error Message

@bennypowers are you still looking for something? what about this?

This shipped. Closing


Keyboard not behaving consistently on input focus

I have a major problem with the native.keyboardshow events not firing when a text input is focussed on Android. It seems to work fine on Sumsung devices, but not on stock Android ROMS. It is causing big problems, since it creates a very inconsistent user experience. Sometimes it fires, but sometimes only the focus event fires, but not the keyboard show event. Even if the keyboard is shown.
Any ideas? This is happening on Android only.

I'm experiencing the same issue. How are you implementing the code? I think this may have something to do with adding event listeners to not a global level (ex. inside a component instead of a global level store event). Any thoughts?

Our problem was that our app was running full-screen on Android; full
screen mode causes the code inside of Cordova to fail as it looks like
there's a watch on the screen size. The events are fired when the screen
size changes, but when running full-screen, the sizes mismatch sometimes
and the events don't seem to fire. That's my speculation. Are you running
fullscreen?

I'm having similar issues on hide. When hiding the soft keyboard about 20% of the time, the hide event doesn't fire. I'm also in fullscreen mode.
I've done some debugging in android studio and I'm noticing that the resultBottom variable isn't accurate. The onGlobalLayoutListener gets fired every time, but sometimes rootView.getWindowVisibleDisplayFrame(r) results in an r.bottom that is the old value rather than the new value.
 However sometimes, rootView.getWindowVisibleDisplayFrame(r) will show an r.bottom of 736 when dismissing the keyboard, or a 246 when getting the keyboard. This would make the difference 0 and so neither the shown or hidden events are triggered.
My assumption is this probably has to do with the keyboard we're using. If the keyboard app takes a bit longer to be fully loaded and shown, then the screen is still its old height when we're calling rootView.getWindowVisibleDisplayFrame(r).
Is the author of the plugin still actively maintaining it?

A really really dirty workaround that you should never use but seems to be working for me so far is this: in IonicKeybaord.java add import android.os.SystemClock; to the list of Imports at the top. Then add SystemClock.sleep(300); as the first line in the onGlobalLayout() function.
All this does, is wait 300 milliseconds before doing the check, which seems to usually stop the issue.
Note that this is poorly tested and adding these kinds of wait things to workaround timing issues is really a bad idea in general.

@c00 @saschwarz I tested this hack but doesn't work.

I might be seeing a different issue. If I programmatically set focus to an ion-input and tap it the keyboard appears. Or if I use a regular HTML input <ion-item><input type="text" placeholder="here"/></ion-item> and tap it the keyboard appears.
But using any variation of ion-input:
        <ion-item>
            <ion-label>Test</ion-label>
            <ion-input type="text" placeholder="tap here"></ion-input>
        </ion-item>

I can't get the input to accept focus and, of course, the keyboard doesn't appear.

I know that isn't the best solution, but I solved my problem with this:
    $(document).on('focus click', 'input',  function(e){
        setTimeout(function(){
          if (!cordova.plugins.Keyboard.isVisible) {
            cordova.plugins.Keyboard.show();
            cordova.fireWindowEvent('native.keyboardshow', {'keyboardHeight': +262});
            cordova.plugins.Keyboard.isVisible = true;
          }
        },300);
    });

    $(document).on('focusout', 'input',  function(e){
      setTimeout(function(){
        if (cordova.plugins.Keyboard.isVisible) {
          cordova.plugins.Keyboard.close();
          cordova.fireWindowEvent('native.keyboardhide');
          cordova.plugins.Keyboard.isVisible = false;
        }
      },300);
    });

The issue for me was the tablet on which I was testing was old and had an old default browser. I installed crosswalk and everything worked perfectly afterward.


Remove index value for existing object

Let $object be \Riak\Object. It is easy to add new values to be indexed via secodary index (available for leveldb):
$object->addIndex($indexName, $indexValue);

When the $object already have some index values, it is also possible to read them, such as:
$indexMap = $object->getIndexMap();

However, there is no option to remove an existing index. This would be particularly useful when updating an existing record (re-writing $object retrieved via $object = $riakEntity->getFirstObject();). If I need to remove an existing index, I need to create a new Riak\Object and copy all values. It would be nice to have a method to remove an existing index, such as:
$object->removeIndex($indexName);

If you agree with the proposed change, I can try to implement it.

Hi
Sound like a good idea, I do not have much time to work on this project at the moment, so you very welcome to try and implement it.


ASU-1180 | Hitas contract PDF

Codecov Report

Merging #174 (c5df16c) into develop (2ce845f) will increase coverage by 0.12%.
The diff coverage is 95.56%.

@@             Coverage Diff             @@
##           develop     #174      +/-   ##
===========================================
+ Coverage    94.72%   94.84%   +0.12%
===========================================
  Files           97       99       +2
  Lines         2846     2990     +144
  Branches       235      245      +10
===========================================
+ Hits          2696     2836     +140
- Misses         109      111       +2
- Partials        41       43       +2

Impacted Files
Coverage Δ

application_form/pdf/haso.py
100.00% <ø> (ø)

application_form/api/sales/views.py
96.66% <81.81%> (-3.34%)
:arrow_down:

apartment_application_service/pdf.py
97.50% <92.85%> (+3.21%)
:arrow_up:

application_form/pdf/hitas.py
96.82% <96.82%> (ø)

application_form/pdf/__init__.py
100.00% <100.00%> (ø)

invoicing/api/serializers.py
96.92% <100.00%> (-0.10%)
:arrow_down:

invoicing/utils.py
100.00% <100.00%> (ø)

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 2ce845f...c5df16c. Read the comment docs.


Acom 5902 course map on ed page

Description
Add course_map method to handle google course map data (.kml file)

Code looks good 👍
Let's wait a day or two. If all set, let's get it merged.


Generic Button to toggle state.

To work better with the action manager we do want a more generic way to
represent buttons that toggle between two states; currently this is only
done in two places: QtNDisplayButton and QtGridViewButton whcih are
checkboxes and not Button; thus they do not have the same interface.
Instead of subclassing checkbox and target the qss on the new class;
create generic QtBistateButton that sets the mode for qss targetting and
has a well defined set of parameters.
There is an increase in number of indirection; so a small increase of
complexity; but this is likely reusable for other button ; for example
the toggle console button is a push button and coudl reflect wether the
console is shown or not.
Type of change

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

How has this been tested?
Manually opening existing examples and checking that buttons still looks ok and behave.
Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

I'm not good at naming; so better name welcome.

there is a built-in checkable property on QPushButton... it's False by default, but can be used to turn a QPushButton into a 2-state (toggle) button... would that be useful here? or still not quite?  (It's possible the original implementer didn't know it was available when they used QCheckBox)

there is a built-in checkable property on QPushButton... it's False by default, but can be used to turn a QPushButton into a 2-state (toggle) button... would that be useful here? or still not quite? (It's possible the original implementer didn't know it was available when they used QCheckBox)

I think I was the original implementor and I didn't know that. If that would do the trick then we should use the QPushButton instead of QCheckBox. It might be nicer than having to make our own QtBistateButton (or at least we can make that based off a QPushButton if it brings about more unification and more simplification), I don't mind - but the original choice of QCheckBox may just have been ill-informed

I can have a look, I did not knew about the "checked" value; we might still need a custom subclass anyway as we need to synchronize the checked state with something else. I'll also look as whether this is atargettable property from QSS.

I'll also look as whether this is atargettable property from QSS.

I believe you can hit with the :checked pseudo-state
QPushButton:checked

I've pushed changes to use QPushButton; and :checked pseudo-state; this shorten the code a bit more, and same for the qss. Though github seem to be having an outage and changes are not reflectdin in the PR yet.

Github seem to be messed up and show 90+ commits in here, many of which are already in master.

going to try to close and reopen.

This is a breake change without deprecation :(
@sofroniewn


Typo

I know this sounds pernickety but the plural of a compute would be computes, right?
Just noting because you called the folder "computeds" ... good work though.
Been upgrading to ember-cli the last 24 hours and its great :+1:

Short for computed property macros, I think computes would convey something entirely different.


Revert "Use Open3 for integration tests instead of manually forking."

Reverts Homebrew/brew#1967

CC @reitermarkus

Ok, so keg is required after os/mac/keg, hence overriding require_relocation?.


TF Rule: Lambda permission should not have a wildcard principal

Add a new Terraform rule:
AWS Lambda permissions should not use a wildcard principal.

This rule is covered in cli/assets/terraform.yml and cli/assets/terraform12.yml with rule LAMBDA_PERMISSION_WILDCARD_PRINCIPAL


fix: fileName can take an absolute path

closes #87

Codecov Report

Merging #88 into 1.x will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##              1.x      #88   +/-   ##
=======================================
  Coverage   98.59%   98.59%
=======================================
  Files           2        2
  Lines          71       71
=======================================
  Hits           70       70
  Misses          1        1

Impacted Files
Coverage Δ

lib/plugin.js
98.57% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Ability to alert on local MUF

(suggestion from a user)
Ability to alert when the MUF rises above a certain frequency for a user's location.

https://pskreporter.info/cgi-bin/psk-freq.pl can be used to query the 'best' frequency for digital modes based on location, although this is on demand rather than constant. Initially at least, this might be query-based rather than automatic.

From pskreporter.info:-

There is also a web service to provide the 'best frequency' to listen on to hear PSK31 traffic in your grid square at the current time. > This is useful if you leave your radio unattended, yet controlled by software.
The basic URL is https://pskreporter.info/cgi-bin/psk-freq.pl which will return data indicating the best frequency for your location (based on geolocating your IP address). If there is insufficient data, then no frequency will be returned.
The parameters to this script are:
grid= the two letter grid square. This will override the geolocation of your ip address.
freq= comma seperated list of frequencies (in hz) that you are prepared to listen to. This should be the VFO frequency for each band. The response frequency will be one of these.


[Feature Request] - md5sum of the .img file

Would be great if there's an option to calculate the md5sum of .img file right within Etcher.
Then we don't need a separate utility to perform the md5sum calculation.

Duplicate of #643  (but see also https://github.com/resin-io/etcher/issues/993#issuecomment-275706782 )


MacOS service discovery issue

Hi,
I am trying to run example code on MacOS 10.15.7, however during service discovery I am getting
Unhandled Exception: FormatException: The format is invalid

This fails on Guid creation at bluetooth_characteristics.dart line 35. It seems that Guid _fromString input is empty but not null.
Does anyone have an idea what could be the issue and how to solve it?
P.S. Very same code runs fine on iOS.
/Erik

Duplicate of #657


Replace 32 bit sizes by size_t

Correct size_t/uint32_t miss alignment between header and source.

This patch was a divergence between the android and github plugin.

@dawagner @oznog Should be merge this ?

I just forced push the my other pull request on this one :(
Restoring it...

:+1:

:shipit:


[Enhancement] Optimize scan range placement of hdfs backend selector

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Optimzation include:

take raw string as hash paremeter. no need to decode.
don't put basePath into hasher because all scan range's are the same.
use simple shuffle instead of ScanRangeSorter to avoid data skew(in extreme cases)

CoorPrepareExec time has been reduced from 7746ms -> 1662ms

Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

run starrocks_fe_unittest

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

@Mergify backport branch-2.5

@mergify backport branch-3.0


progress bar modules on npm

Should look at using the same modules that npm itself uses:

https://www.npmjs.com/package/are-we-there-yet

https://www.npmjs.com/package/gauge

ETA discussion on gauge issue tracker: https://github.com/iarna/gauge/issues/22


NameError: name 'log_args' is not defined

from fastai2.tabular.all import *
Traceback (most recent call last):
File "", line 1, in
File "D:\projects\fastai\fastai2\fastai2\tabular\all.py", line 1, in
from ..basics import *
File "D:\projects\fastai\fastai2\fastai2\basics.py", line 1, in
from .data.all import *
File "D:\projects\fastai\fastai2\fastai2\data\all.py", line 2, in
from .core import *
File "D:\projects\fastai\fastai2\fastai2\data\core.py", line 8, in
from .load import *
File "D:\projects\fastai\fastai2\fastai2\data\load.py", line 59, in
@log_args(but='dataset,wif,create_batch,create_batches,create_item,retain,get_idxs,sample,shuffle_fn,do_batch')
NameError: name 'log_args' is not defined

You need latest version (git) of fastcore

Yes, and please search closed issues before opening a new one since you could see the answer there :)


display multiple linemode options

currently linemode can only display a single mode - there is no way to display both size and mdate simultaneously
make linemode a comma delimmeted list of options

With https://github.com/sxyazi/yazi/pull/1257, you can easily create your custom linemode (here's an example https://github.com/sxyazi/yazi/discussions/1288#discussioncomment-10033306), or add multi linemode alongside the preset one through Linemode:children_add()
Closing as already supported.


New search filter

@marcincichon OR

@ollietreend wasn't OR after all but fixed now and available on dev2.

@marcincichon  looks good.

Looks good @marcincichon 👍


Basic fail points impl, test case that uses them to cause crashes when attempting to write to DB

Note that I'm not using https://docs.rs/failpoints/latest/failpoints/ (used elsewhere in the code) because it doesn't allow user-defined failpoint behavior, but only a limited set of hard-coded actions (such as panicking).

Looking forward to this. cc @arun-koshy for fail points.


cargo-test-support: Add features to the default Cargo.toml file

What does this PR try to resolve?
Found in https://github.com/hi-rustin/cargo-information/pull/32#issuecomment-1815625144.
I think we need to add features to the default Cargo.toml file.
How should we test and review this PR?

[x] Tested in https://github.com/hi-rustin/cargo-information/pull/32

Additional information
Do you which Cargo command can cover this case? I want to add a unit test for it.
It seems I can use cargo metadata to cover it. I can add a dep to the package and try to check the output JSON.

@bors r+

:pushpin: Commit 504c4b2fbb1bd652d6ac93a947079d7ecb95b955 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 504c4b2fbb1bd652d6ac93a947079d7ecb95b955 with merge 65d0eb536dd4a7ae6ddc50ca14556896e450c4ff...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 65d0eb536dd4a7ae6ddc50ca14556896e450c4ff to master...


Improve documentation search results for important keywords

Searching in existing doc for words such as:

--seek-file
-f
doit.cfg
DOIT_SEEK_FILE

provides no results.
Try to improve Sphinx based documentation so that this kind of keywords are described and found. Sphinx provides many "roles" such as :envvar:, :token:, :option:, :term:, :abbr:, :command:, :file: which are likely to put related words into search index.

@dkennedy2 and I would like to take a look at this. We'll start by tracking down the command line options and environment variables in the docs and tag them with the correct role.

@xordspar0 @dkennedy2 thanks, that would be awesome.
There are 2 problems with current docs. First is the need to add Sphinx roles into existing references as you mentioned. Second is that many command line options are not even mentioned in docs... I was thinking about having a doit command that would generate a Sphinx formatted output of doit help <cmd>, and maybe even write a directive for that!

Cool, I like the idea of documentation for the cli interface being generated by code.
We've started adding roles in https://github.com/pydoit/doit/pull/289, but there's still more work to do there. Would it make sense to add all the roles in one PR and then make a second PR with the docs generation?

I made a PR specifically to add the :option: roles: https://github.com/pydoit/doit/pull/359

Progress of adding roles to existing documentation:

[x] :envvar: – There currently aren't any environment variables in the documentation that I can see. We will write a doit command for generating that documentation.
[x] :token: – This is for BNF grammars. doit doesn't have any BNF grammars in its documenation, so we won't do this.
[x] :option: – We have a PR open for this: https://github.com/pydoit/doit/pull/359
[ ] :term: – This is for defining glossary terms that must be defined. "If you use a term that’s not explained in a glossary, you’ll get a warning during build." This seems useful.
[ ] :abbr:
[x] :command: – I don't think this role does anything special, it just marks a command. Most of the docs already use .. code-block:: console directive instead, so I don't think it's necessary.
[ ] :file:


[T3A3][W11-A2] Zhi Jiang

Ready for Review

@zavfel Please review the comments and close PR. We are now advised to follow this protocol for PR closing to ensure student read our comments.

updated the code base on your comments


Fix time format string not to ignore trailing zeros

Small bug fix. Current time format strings remove trailing zeros on frame's filename but they are important in Windows filename order.
hh-mm-ss-FFFF => hh-mm-ss-ffff
reference

Alright, thanks.


build: bump go version of runtime/cloudformation from 1.8 to 1.9

Signed-off-by: Francesco Racciatti<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area tests

/area proposals

What this PR does / why we need it:
This PR bumps the go version of runtimes/cloudformation from 1.8 to 1.9.

Fixes #
Special notes for your reviewer:

@francesco-racciatti: The label(s) area/build cannot be applied, because the repository doesn't have them.

In response to this:

Signed-off-by: Francesco Racciatti<EMAIL_ADDRESS>What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area tests

/area proposals

What this PR does / why we need it:
This PR bumps the go version of runtimes/cloudformation from 1.8 to 1.9.
Fixes #
Special notes for your reviewer:

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: francesco-racciatti
Once this PR has been reviewed and has the lgtm label, please assign admiral0 for approval by writing /assign @admiral0 in a comment. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @gnosek

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: francesco-racciatti, gnosek
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [gnosek]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Upgrade terraform-provider-openstack to v1.52.1

This PR was generated via $ upgrade-provider pulumi/pulumi-openstack.

Upgrading terraform-provider-openstack from 1.51.1  to 1.52.1.
Fixes #295
Fixes #293
Upgrading pulumi-terraform-bridge from v3.41.0 to v3.53.0.

Not sure what's happening with the dotnet sdk output changing back and forth, I don't remember if we've seen something similar before

Perhaps need to rebuild SDKs locally? The CI detects non-empty changes.

@rquitales did your tooling on auto-rebuilding SDKs land in prod? We could adopt it from AWS across all repos like this one.

Perhaps need to rebuild SDKs locally? The CI detects non-empty changes.

We were missing a dependency, which caused inconsistent builds. Fixed by https://github.com/pulumi/pulumi-openstack/pull/308/commits/1020d540501090a0684aeed1b6a4312dcf2fe8d1.


bug: lazygit Failed to get lazygit config directory

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.11.0-dev+315-g46187117c
Operating system/version
Fedora 40
Describe the bug
After updating lazy.nvim the following error occur on launching lazygit with keymap <leader>gg
lazygit Failed to get **lazygit** config directory.
Will not apply **lazygit** config.

# Error:
~/.repro/plugins/lazy.nvim/lua/lazy/async.lua:157: In coroutine without async context

Steps To Reproduce

Update everything to latest version.
Start LazyVim
Navigate to any git project dir and open any file.
Try to start LazyGit with the default keymap provided in LazyVim

Expected Behavior
LazyGit should start without any error.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  "folke/LazyVim",
  {
    'LazyVim/LazyVim',
    import = 'lazyvim.plugins',
  },
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Fixed by https://github.com/folke/lazy.nvim/commit/3513227a9a41c8e6366e1719f4cefbe891ca73d2


broadcastRawTransaction failing for deployments

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (3ff3d05 2024-10-01T00:25:45.808130533Z)
What command(s) is the bug in?
forge script
Operating System
Linux
Describe the bug
When trying to deploy a pre-signed transaction via a script I'm getting an error: No ``to`` field found
...
├─ [0] VM::broadcastRawTransaction(0xf8a58085174876e800830186a08080b853604580600e600039806000f350fe7fffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffe03601600081602082378035828234f58015156039578182fd5b8082525050506014600cf31ba02222222222222222222222222222222222222222222222222222222222222222a02222222222222222222222222222222222222222222222222222222222222222)
    │   └─ ← [Revert] vm.broadcastRawTransaction: transact_from_tx: No `to` field found

The above example is using the same value as specified in foundry tests here. https://github.com/foundry-rs/foundry/blob/master/testdata/default/cheats/BroadcastRawTransaction.t.sol#L342.

hey @ScreamingHawk could you please share the code or a minimal repro for this issue, would help for figuring out where the problem is. thank you!

Thanks @grandizzy
Here's a minimum reproducible example:
https://github.com/ScreamingHawk/mre-forge-broadcastrawtransaction

thank you! yes, indeed that tx is missing the to field
cast dt 0xf8a58085174876e800830186a08080b853604580600e600039806000f350fe7fffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffe03601600081602082378035828234f58015156039578182fd5b8082525050506014600cf31ba02222222222222222222222222222222222222222222222222222222222222222a02222222222222222222222222222222222222222222222222222222222222222
{
  "type": "0x0",
  "nonce": "0x0",
  "gasPrice": "0x174876e800",
  "gasLimit": "0x186a0",
  "value": "0x0",
  "input": "0x604580600e600039806000f350fe7fffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffe03601600081602082378035828234f58015156039578182fd5b8082525050506014600cf3",
  "r": "0x2222222222222222222222222222222222222222222222222222222222222222",
  "s": "0x2222222222222222222222222222222222222222222222222222222222222222",
  "v": "0x1b",
  "hash": "0xeddf9e61fb9d8f5111840daef55e5fde0041f5702856532cdbb5a02998033d26"
}

as per https://github.com/foundry-rs/foundry/pull/4931#issuecomment-1546652889 seems this is a create tx, try encoding with to as 0x address
There are other samples that should work (or you could create your own raw tx), e.g.
cast dt 0xf869808502540be40083030d4094f39fd6e51aad88f6f4ce6ab8827279cfffb922668204d28082f4f6a061ce3c0f4280cb79c1eb0060a9a491cca1ba48ed32f141e3421ccb60c9dbe444a07fcd35cbec5f81427ac20f60484f4da9d00f59652f5053cd13ee90b992e94ab3
{
  "type": "0x0",
  "chainId": "0x7a69",
  "nonce": "0x0",
  "gasPrice": "0x2540be400",
  "gasLimit": "0x30d40",
  "to": "0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266",
  "value": "0x4d2",
  "input": "0x",
  "r": "0x61ce3c0f4280cb79c1eb0060a9a491cca1ba48ed32f141e3421ccb60c9dbe444",
  "s": "0x7fcd35cbec5f81427ac20f60484f4da9d00f59652f5053cd13ee90b992e94ab3",
  "v": "0xf4f6",
  "hash": "0xd25911fa4ea892d775652a4417b11530d33cea00537fb0607756c6afab872c2c"
}

Thanks @grandizzy
Unfortunately since I'm hoping to use this for presigned transactions I am unable to re-encode as it will change the deployment address. See https://eips.ethereum.org/EIPS/eip-2470#deployment-transaction
As the transaction is valid and can be submitted on chain I believe this is a bug in forge scripts.


Add utility to parse TMCF + CSV into node MCF

This is a port of the flume-based implementation found internally here: https://source.corp.google.com/piper///depot/google3/datacommons/import/util/ext_table_util.cc;l=48-400;bpv=0;bpt=1

Thanks for the review!


AuthStateForbidden: Wrong state parameter given.

Can anyone help with "AuthStateForbidden: Wrong state parameter given." issue? I don't know how to reproduce this issue but getting this error occasionally.
Internal Server Error: /sso/social/complete/linkedin-oauth2/
Traceback (most recent call last):
File
"/virtualenvs/pyappd/lib/python2.7/site-packages/django/core/handlers/base.py",
line 132, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
File
"/virtualenvs/pyappd/lib/python2.7/site-packages/django/utils/decorators.py",
line 145, in inner
return func(*args, **kwargs)
File
"/virtualenvs/pyappd/lib/python2.7/site-packages/django/views/decorators/csrf.py",
line 58, in wrapped_view
return view_func(*args, **kwargs)
File
"/virtualenvs/pyappd/lib/python2.7/site-packages/social/apps/django_app/utils.py",
line 52, in wrapper
return func(request, backend, *args, **kwargs)
File
"/virtualenvs/pyappd/lib/python2.7/site-packages/social/apps/django_app/views.py",
line 20, in complete
redirect_name=REDIRECT_FIELD_NAME, *args, **kwargs)
File
"/virtualenvs/pyappd/lib/python2.7/site-packages/social/actions.py",
line 43, in do_complete
user = backend.complete(user=user, *args, **kwargs)
File
"/virtualenvs/pyappd/lib/python2.7/site-packages/social/backends/base.py",
line 40, in complete
return self.auth_complete(*args, **kwargs)
File
"/virtualenvs/pyappd/lib/python2.7/site-packages/social/backends/oauth.py",
line 357, in auth_complete
state = self.validate_state()
File
"/virtualenvs/pyappd/lib/python2.7/site-packages/social/backends/oauth.py",
line 83, in validate_state
raise AuthStateForbidden(self)
AuthStateForbidden: Wrong state parameter given.

I also get this frequently.

I had this issue. Put domain name in nginx and django settings, not IP of server. That error gone.

@amigo00678  Can you be more specific? Where to put it?
I think I eventually got this solved, by NOT passing a "scope" parameter to Linkedin's auth.

I also had this issue when using twitter, but not all my users got this error.
@amigo00678 could you give a snippet for django settings of yours to fix this?

The setting that fixed that for me was BASE_URL = 'mydomain.com' instead of BASE_URL = '212.x.x.x'.


[draft] react-router 7 support

Description
Tested https://github.com/remix-run/react-router-templates/tree/main/default with ntl dev + ntl serve + ntl deploy and they all seem to work well. Edge was not tested.
Notes:

There is still work to be done to remove remix references from the react-router-adapter but this is meant to serve as a starting point for us

There is an outstanding typescript issue with our plugin not being recognized as a valid vite.Plugin type

If we want full separation, we will likely need to also add a react-router-runtime as I've done here, however it is worth noting that the following variables are not currently exported from @react-router/server-runtime

broadcastDevReady
logDevReady
UNSAFE_SingleFetchRedirectSymbol
UNSAFE_SingleFetchResults
UNSAFE_SingleFetchResult

Further on separation, we need to decide if react-router

At the moment there is a fair amount of code that can be shared, however this may change as react-router continues to be developed. It's not something we can easily predict.
Do we want to move the react-router-adapter to its own repository, or keep it here for the time being?

TODO:

Test/Build Edge runtime
Demos/Fixtures
Framework Detection in (build)

Related Tickets & Documents

Related Issue #
Closes #

QA Instructions, Screenshots, Recordings
Please replace this line with instructions on how to test your changes
For us to review and ship your PR efficiently, please perform the following steps:

[ ] Open a bug/issue before writing your code 🧑💻. This
ensures we can discuss the changes and get feedback from everyone that should be involved.
[ ] Read the contribution guidelines 📖. This ensures your code follows our style
guide and passes our tests.
[ ] Update or add tests (if any source code was changed or added) 🧪
[ ] Update or add documentation (if features were changed or added) 📝
[ ] Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

Replaced by https://github.com/netlify/remix-compute/pull/472


fix number type

fix number type in java sdk, use long for big integer.

@netweng @Yuyz0112 PTAL

LGTM


[Autocomplete] add Mui-expanded class

[x] I have followed (at least) the PR section of the contributing guide.

Solves #33258

The expanded attribute in autocomplete.json doesn't look like the other ones. Not sure how to generate it properly.

The changes look good. You need to run yarn docs:api to generate documentation based on JSDoc comments.


TooManyTemplateParams on morphOne

Describe the bug
This PR added support for polymorphic relations, but morphOne seems to be missing.
Impacted Versions
vimeo/psalm 4.4.1
psalm/plugin-laravel v1.4.2

Duplicate of #158, and fixed by #159.
@riasvdv Please try upgrading psalm/plugin-laravel to 1.4.8 or later.


DSP block usage of oneapi-asp??

In board_spec.xml
https://github.com/OFS/oneapi-asp/blob/1c8a325178d71df0e9db6416aeaa08a1d436a70d/n6001/hardware/ofs_n6001/board_spec.xml#L26C1-L27C1
it is reported that 2284 DSP blocks in AGF014 FPGA is already utilized? however I couldn't find any DSP related logic OFS-FIM and oneapi-asp (this repo)
https://github.com/OFS/ofs-agx7-pcie-attach
is it due to N6001 floor plan limitation?
https://github.com/OFS/ofs-agx7-pcie-attach/blob/release/ofs-2023.2/syn/board/n6001/setup/pr_assignments.tcl
or
https://github.com/OFS/ofs-agx7-pcie-attach/blob/release/ofs-2023.2/syn/board/n6001/setup/pr_assignments_slim.tcl

You are correct - the ASP itself doesn't use any DSP blocks, but the blocks that are in the static region aren't accessible to the ASP/kernel so they considered to be consumed. Those numbers in the xml file are difficult to keep up-to-date - the baseline is defined by the floorplan (the ASP+kernel can only utilize the resources available to the afu_main region), and then the ASP itself consumes logic and M20K blocks so they need to be subtracted from the number available to the afu_main region to find the values used in the xml file.
Basically, to calculate numbers for a specific FIM+ASP combination:

Find the resources consumed by the FIM's floorplan in the FIM's Fitter reports. Subtract the resources available to the afu_main region from the total chip resources. R(total device) - R(available to afu_main) = R(fim floorplan).
Find the resources consumed by the ASP components (excluding the kernel_system module). You can find this by subtracting the kernel_system resources from the afu_main resources - this leaves the ASP resources. R(consumed in afu_main region) - R(consumed in kernel_system module) = R(consumed by asp).
Add R(fim floorplan) to R(consumed by asp) to get R(consumed by FIM+ASP). Put that number in the xml file for each of the resource-types.

i was wondering if calucation can be automated by synthesizing following reference design:
https://github.com/oneapi-src/oneAPI-samples/tree/master/DirectProgramming/C%2B%2BSYCL_FPGA/ReferenceDesigns/board_test
The Board Test sample is a reference design that contains tests to check FPGA board interfaces and reports the following metrics:

Host-to-device global memory interface bandwidth
Kernel clock frequency
Kernel launch latency
Kernel-to-device global memory bandwidth
Unified Shared Memory bandwidth


fix: stop propagating request to backend if not valid

Issue
https://github.com/gravitee-io/issues/issues/7301

Gravitee.io Automatic Deployment
🚀 A prerelease version of this package has been published on Gravitee's private artifactory, you can:

use it directly by updating your project with version: 1.2.2-issues-7301-stop-propagate-request-3-10-SNAPSHOT
download it from Artifactory here

:tada: This PR is included in version 1.2.2 :tada:
The release is available on:

1.2.2
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.3.2 :tada:
The release is available on:

1.3.2
GitHub release

Your semantic-release bot :package::rocket:


Superstream extension url

Source information
Superstream 14.11
Source language
English
Source new URL
https://www.showbox.media/
Other details
Extension broken please fix it, the actual url is https://www.showbox.media/
Thanks
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL is not updated yet.
[X] I will fill out all of the requested information in this form.

dupe closed in favor of #103


Error whilst running extractor.parse(doc)

I'm getting an error when trying to run extractor.parse(doc). I've tried on both my text and also the example text and it occurs both times.
Error as follows:
Could not handle incoming annotation
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/local/Cellar/python/3.6.4_4/Frameworks/Python.framework/Versions/3.6/lib/python3.6/threading.py", line 916, in _bootstrap_inner
    self.run()
  File "/usr/local/lib/python3.6/site-packages/Giveme5W1H/extractor/extractor.py", line 20, in run
    extractor.process(document)
  File "/usr/local/lib/python3.6/site-packages/Giveme5W1H/extractor/extractors/abs_extractor.py", line 41, in process
    self._evaluate_candidates(document)
  File "/usr/local/lib/python3.6/site-packages/Giveme5W1H/extractor/extractors/action_extractor.py", line 104, in _evaluate_candidates
    if any(doc_coref.values()):
AttributeError: 'list' object has no attribute 'values'

Thanks!

Version is giveme5w1h==1.0.12

Let me know if there's anything else you need... This is my first issue!

The error seems to be coming from Stanford CoreNLP, not our tool. Could you provide a full, minimal code example, with which I can reproduce the error, including also an article text, please?

Hi,
I faced similar issue too. I ran giveme5w1h-rest command, then at the page, I ran 'run example' and faced this error.
Output at run giveme5w1h-rest console
  File "/usr/local/lib/python3.6/site-packages/Giveme5W1H/extractor/extractors/abs_extractor.py", line 41, in process
    self._evaluate_candidates(document)
  File "/usr/local/lib/python3.6/site-packages/Giveme5W1H/extractor/extractors/action_extractor.py", line 104, in _evaluate_candidates
    if any(doc_coref.values()):
AttributeError: 'list' object has no attribute 'values'

Output at corenlp console
Caused by: java.lang.ClassNotFoundException: javax.xml.bind.JAXBException
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:583)
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
	... 53 more

OS: MacOS 10.14.3
Python Version 3.6.2
Giveme5W1H Version 1.0.13
Stanford CoreNLP Version
Java Version 8 Update 201

Let me know if there's anything else you need... This is my first issue!

I am experiencing the exact same error code. Did you find a fix for this?


index: fix hash_info type assertion

fixes https://github.com/iterative/dvc/actions/runs/3216435563/jobs/5258323572

Codecov Report
Base: 52.95% // Head: 52.95% // No change to project coverage :thumbsup:

Coverage data is based on head (74fed5a) compared to base (a6aa2e6).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #197   +/-   ##
=======================================
  Coverage   52.95%   52.95%
=======================================
  Files          47       47
  Lines        2723     2723
  Branches      466      383   -83
=======================================
  Hits         1442     1442
- Misses       1215     1222    +7
+ Partials       66       59    -7

Impacted Files
Coverage Δ

src/dvc_data/index/index.py
69.64% <100.00%> (ø)

src/dvc_data/cli.py
32.10% <0.00%> (ø)

src/dvc_data/hashfile/build.py
17.74% <0.00%> (ø)

src/dvc_data/hashfile/state.py
34.48% <0.00%> (ø)

src/dvc_data/hashfile/db/index.py
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add velocityWorld Setpoint

@whoenig can you take a look?

actually we should probably test it on hardware first...

Looks good. If there is an issue with HW, it would likely be in the cpp-layer.


Remove minUTxOValue from PParams

The minUTxOValue field was removed from PParams some eras ago, so it shouldn't be in Conway PParams.

I've moved this into a hidden section with the comment that it's retired. I'd like to keep it until we have separate directories for older eras.


Using the CLIMATE_FAN_QUIET instead of DIFFUSE

ervee commented
Perhaps using the CLIMATE_FAN_QUIET instead of DIFFUSE. CLIMATE_FAN_QUIET should also be available in climate and esphome and is perhaps a more logical choice for ultra-low fan speed.
CLIMATE_FAN_QUIET is not be showing up in Home Assistant Card like CLIMATE_FAN_MIDDLE and CLIMATE_FAN_FOCUS.
I already opened a feature request at ESPHome: https://github.com/esphome/feature-requests/issues/2149

I think that would be a breaking change at Esphome if you were to insert a new value bumping the other +1.
I wonder why Quiet is not available in the climate card.

Wouldn't it be better to use the CLIMATE_FAN_QUIET which is already there, and update HomeAssistant climate.py to propperly use FAN_QUIET for Quiet, and add Diffuse (and possibly middle while we are at it):
AT_TO_HA_FAN_SPEED = {
"Quiet": FAN_DIFFUSE,
"Low": FAN_LOW,
"Medium": FAN_MEDIUM,
"High": FAN_HIGH,
"Powerful": FAN_FOCUS,
"Auto": FAN_AUTO,
"Turbo": "turbo",
}

oh, i think it now is in home assitant:
https://github.com/home-assistant/core/commit/d14000f7049f955a08b9c164798f5f78335e54b6

Everything's done for me


Introduce Process structure to allow runtime to get useful info about the container's process

This PR adds the Process structure so that any runtime implementation can update PID related to the started process on the container. Also it can retrieve the token necessary for the shim to connect the proxy.

Coverage decreased (-0.4%) to 67.288% when pulling 48a4b635b986d7d027035eef351dc82e8c3ed899 on sboeuf/adapt_virtcontainers_to_fit_runtime into cadef0c6cceee71078cd7a08e1dea0867a5ef0be on master.

LGTM

Coverage decreased (-0.4%) to 66.276% when pulling a10b902b3d9c1054ce8a896d46a69aabdcfedbe0 on sboeuf/adapt_virtcontainers_to_fit_runtime into 9d57e22e0c66a671bf931dd43666dbbeff2e2cf4 on master.


Haproxy map support to speed up vhost to backend lookup.

Currently vhost to backend lookup happens in the traditional way, where each rule is scanned to lookup the corresponding backend, this can add considerable latency when the number of apps/vhosts are large in number (especially for apps/vhosts which appear at the rear end of the rules comparison).
Ex:
frontend xyz
{other_lines}
use_backend backend1 if { hdr(Host) -i myapp.domain1.com }
use_backend backend2 if { hdr(Host) -i myapp.domain2.com }
....
....
....
use_backend backendN if { hdr(Host) -i myapp.domainN.com }
Here the lookups for domainN, N-1, N-2 etc . can take considerable time, since these comparisons are sequential. Hence this PR supports a --haproxy-map , which creates a lookup map which is supported by haproxy, so for the above example when we use haproxy-map, we get
frontend xyz
{other_lines}
use_backend %[req.hdr(host),lower,map_dom(/path/to/map)]
and /path/to/map contains:
myapp.domain1.com backend1
myapp.domain2.com backend2
...
...
...
myapp.domainN.com backendN,
which is a hash based lookup dict, this way domainN, N-1 , N-2 etc, lookups will be faster.
Currently it just works for host header and x-marathon-app-id header, it retains the rule based comparison for bath based routing and auth.

Few functions are not supported in python3. Ignore this PR, will fix and open a new PR.


Firstboot service conflicts with cloud-init.

reconfiguring the openssh server during the first boot scripts driven by the first boot service, conflicts with the execution of any package install in cloud-init that wants to use debconf.
Firstboot and cloud-init need to operate sequentially not in parallel.

Reopening, still WIP https://github.com/hypriot/image-builder-rpi/pull/227


Like email subject lines

To discuss about code, bugs and anything else. That is why issues are used for.

hello


Uncaught SyntaxError: Failed to construct 'WebSocket': The URL's scheme must be either 'ws' or 'wss'. 'http' is not allowed.

I'm following the documentation and when I'm trying to create a connection I get this error.
Running Django 1.6
omnibus added to  installed apps
template_context_processors added
template tags included
omnibusd started

It would be helpful to know what you were doing wrong. Do you remember what it was?

Sorry, I should have stated what it was back then. I have no idea now.


VerneMQ Webhook Unsubscribe not working

I have configured vernemq with webhooks everyhting works as expected apart from unsubscribe hook.

[

](url)

Closing in favour of: https://github.com/vernemq/vernemq/issues/2036


Readme.md update with new sections

Infos :

Addition: New sections that contains all our functions (only ones callable by package users).
Addition: Small visual modifications.

Notes :
@m-henderson
The link to the Discord server is dead, wanna remove Contributor Community section ?
Regenerate a link ? or create new server ?

I updated the link to the coding discord server community. Give this one a try https://discord.gg/F77g42ZNFa
I can set it up with a relevant picture and channels later

Thanks ! I will update the readme with this one
Yeah, no worries for the server structure ;)


Add ATLAS into HWIMO-BUILD, Remove duplicated template-packages.json, Update README

this work is based on request of @changev
@PengTian0  @RackHD/corecommitters

test this please

Hi, @changev  @cgx027 @PengTian0 ,
I add a new commit to address your comments :  https://github.com/RackHD/RackHD/pull/497/commits/a9ff37af315e7636253199e8ea11a019ad52e8b4

BUILD RackHD #972 : FAILURE

Hi, @RackHD/corecommitters , Can you approve this PR?

@panpan0000 please resolve the conflicts

BUILD RackHD #1010 : UNSTABLE

** BUILD build-config-setup  #171667: SUCCESS

conflicts resolved. Thanks.


dashboard/app: bisect bugs to find guilty commits

If a bug has a reproducer we should try to bisect commits to find a guilty one. Having the commit introducing the bug is always useful for debugging.
Potential problems:

unreliable reproducers
triggering of unrelated bugs (happens frequently)
flakes (bugs in GCE, crashes in tty on first ssh connection, etc)
bugs (races) that manifest in multiple different ways
bugs that will be attributed to tools improvements (e.g. KASAN, fault injection improvements)
reproducers that need slight changes on different kernel revisions

Some problems with bisection uncovered so far:

Kernels past v4.11 do not build with gcc 7 and 8 (undefined reference to `____ilog2_NaN').
v4.1 and back have only compiler-gcc5.h.
v3.17 and back have only compiler-gcc4.h.
v3.6 and back do not have makeolddefconfig.
Compat socket calls can't be bisected past "x86/entry/syscalls: Wire up 32-bit direct socket calls" (v4.10):
https://syzkaller.appspot.com/bug?id=b5b150e322d5f48c869bcf1528cdbee08d1421cb

Re 1: Encountered this, this patch fixes the issue: https://patchwork.kernel.org/patch/9380181/

v2.6.28 and below does not work with modern make:
*** mixed implicit and normal rules: deprecated syntax

v3.8 build fails:
Can't use 'defined(@array)' (Maybe you should just omit the defined()?) at kernel/timeconst.pl line 373.
kernel/Makefile:134: recipe for target 'kernel/timeconst.h' failed

v2.6.28 and below does not work with modern make:

make 3.81 works for v2.6.28 and also seems to work for current HEAD

Building v2.6.28 with gcc-4.9.4:
First build is broken with our fat config:
include/linux/kvm.h:240:9: error: duplicate member ‘padding’

but even defconfig fails:
  VDSO    arch/x86/vdso/vdso.so.dbg
gcc: error: elf_x86_64: No such file or directory
gcc: error: unrecognized command line option ‘-m’

It seems that we also need old binutils.

For v3.8 and below we need perl-5.14.4.
Unfortunately this or any manually built perl doesn't work for later kernels:
Can't locate strict.pm in @INC

and I don't know how to build and pre-package perl packages.

Building and using own binutils and all dependencies (glibc) turns out to be too painful. I think we need to stop at v3.9 which still can work with modern make/perl/binutils.

We also can't used too old syzkaller reproducers, because e.g. on 6a0246bf72830c3461e9784c104894882738541f we don't have make target and everything is different.

Let's consider this fixed by now. We do bisection. Lots of improvements are possible, but it's better to track them in separate finer-grained issues.


fix(typings): the return type of factory of defer should be ObservableInput

should be able to defer a ArrayLike result
Observable.defer(() => [1, 2, 3])

Closes https://github.com/ReactiveX/rxjs/issues/2586


Merge with AmuNMT

A shallow merge for now. Amun and Marian communicate through the model file, which should be ok as long as we do not have a Marian scorer in Amun.
What should we do with the repos? New repo? New organization? New name?

An organisation definitely.
Amun should be Amun, or AmuNMT consistently otherwise it causes confusion. Amun is prob better.
Can you git pull when merging so that the git history is retained. Need to convince the bosses I'm doing some work and not sitting on a beach drinking pina colada

Sure. I have this domain, "unrolled.ai". Would that be an organization name?

might as well

might be best to just rename amun. Or somehow bring the issues and pull request page to the new repo. The discussions on there is useful for the long term

rename -> move

https://github.com/IQAndreas/github-issues-import

At first I will just transfer the ownership of the repos, everything is being move then. Maybe I will keep amun and marian apart for now.

suggestion: amun and marian uses the same 3rd parties code: spdlog, cnpy, etc. I think it's worth to make it common. I mean to have three directories: amun, marian, 3rd_party.

@tomekd yes, but let's wait with this a bit.


"Out of gas" error on finalize transaction

I am having trouble to finalizing my first won auction. I am using MyEtherWallet, which I successfully used to bid and reveal. However, when I try to finalize, I get an out of gas error. It doesn't refund me the appropriate amount over the second place bid, and doesn't list me as the owner on Etherscan, etc.
Any idea if I'm doing something wrong?
Here is one instance of a failed transaction (I tried 3X). https://etherscan.io/tx/0x9e0cb70f1befb0c80eab81f180dc4b7ec0b43414d540d87ebcba2fe97410f1ef
It is for coinality.eth (For the record, I am not Dan from Coinality, but bid on this name with his blessing).

I think the gas limit is too low, how about increasing it to 300000?

This was a brief bug with MEW; it should work now.


bump work_version to enable simulator

I'm not sure if the goal is to change each integration to use 3.x of the worker eventually. This change enables the publishing to the simulator channel.
App PR: https://github.com/segmentio/app/pull/3058
cc @segment-integrations/core

@ndhoule @thehydroimpulse any chance you can review? I'll take this over for John if anything needs to change

@stephenmathieson we don't need this PR anymore. Master is already on the 3.x version :)


Made Dark Mode Ready

Made Dark Mode Ready
Fixed App Icon missing. (Used to show a black icon on devices earlier)
Center-aligned GitHub Icon on ContentView (left-aligned icon appeared as if some text follows it)
Fixed Typo: LightGreen to PaleGreen

Yash, so cool that you fixed the icon and enabled dark mode! I'm merging your contribution now. In 2 weeks I'll do another release to send that to the App Store.

Thanks 😃
And congratulations on this fabulous app. It's really useful 👍🏼


TERM setting not good with mutt

I ssh to my mailserver and run mutt.  With default TERM=qansi there are serious visual errors when moving about the index screen.  Changing to TERM=vt100 makes it work but of course removes colour support.

Yes, this is a (unfortunate) known thing. In the cases I have investigated, this is due to differences in the qansi definition on the BlackBerry device and on the server that you ssh into. In some cases the definitions are very different, and things break pretty badly. Had I known at the beginning that qansi was not well standardized across distributions I would have perhaps selected another terminal definition to implement. The fix is to either switch Term48 from qansi to something more standardized like xterm, or launch Term48 into a terminal multiplexer like tmux or screen, which will present a TERM=screen interface to the rest of the world, while using the local TERM=qansi definition to draw to the screen on the device. Launching into a terminal multiplexer would also have other benefits, such as multiple windows, scrollback, and copy, but also comes with it's own challenges. Either way, it is on my list of things to fix.
Thanks for reporting! Please feel free to report any other issues, either here or via email - it's helps me prioritize things to work on.

Could you provide a terminfo file I can install on my servers?

If you want to have qansi definitions that match what is on the BlackBerry, they are stored at /usr/lib/terminfo/q and the termcap is at /etc/termcap. The terminfo is compiled, so I am not sure if there would be problematic binary differences if you just copied it over to another system. The termcap is just text though, so you should be able to copy that over without too much issue, provided you can coerce your server side curses library to use it.

I just uploaded a new version to BB World. It is in draft while it goes through some testing, but if you're interested in having a look and verifying it fixes your mutt issue, then I have added you as a tester (using the same email you used to email me the other day). It may take a day or so to propagate, but you should be able to retrieve it from BB World like so:

Open the BlackBerry World App
Swipe down from the top and choose Settings
Choose Development Mode
In the Content ID box, enter ‘term48’ (case sensitive, no quotes)
You should be brought to the app page, showing the new version 0.4.

This version also bundles the ssh, scp, sftp and ssh-keygen commands.
I am still working a couple of minor visual issues in some curses apps, but Term48 now declares/implements TERM=xterm-color (and ships a private terminfo database to get the definition), which should fix the compat issue you reported with mutt. The xterm emulation is incomplete, but it's passing the first and second suites in vttest (and some others), which is the base functionality that's really needed.
If you can't get it from BB World, then I can email you the .bar to sideload.

I getting a download error from bbworld. Will try again later

Ack, me too. Will check it again later in BB World. I had a lot of problems getting this version uploaded, so there may be some funkiness on their backend.

Got it installed finally.  the bundle ssh works, thanks!  But if I try to run mutt the whole term48 app crashes

                                                                                Hmm. That's not good, and I haven't had any crashes with mutt during my testing. Can you tell me which version of mutt and on what platform? I will try to reproduce over the weekend. Thanks!                                                                                Todd                                                                                From: Stephen Paul WeberSent: Friday, December 11, 2015 16:35To: mordak/Term48Reply To: mordak/Term48Cc: mordakSubject: Re: [Term48] TERM setting not good with mutt (#4)Got it installed finally.  the bundle ssh works, thanks!  But if I try to run mutt the whole term48 app crashes

—Reply to this email directly or view it on GitHub.

Mutt 1.5.20 (2009-06-14)
Linux iweb 2.6.32-5-amd64 #1 SMP Mon Feb 25 00:26:11 UTC 2013 x86_64 GNU/Linux
Debian GNU/Linux 8 \n \l

works if I run mutt under screen

Though running screen makes my font reaaaaaly tiny...

                                                                                Okay, this helps a lot. There is some difference between the terminfo definitions that I am not handling correctly. Is your screen setting 80 or 132 column mode? I put this in as part of xterm, but perhaps it should be disabled... If you hit metamode-s then it will reset the font size, though it can still be changed if a program wants it.                                                                                Todd                                                                                From: Stephen Paul WeberSent: Friday, December 11, 2015 16:43To: mordak/Term48Reply To: mordak/Term48Cc: mordakSubject: Re: [Term48] TERM setting not good with mutt (#4)Though running screen makes my font reaaaaaly tiny...

—Reply to this email directly or view it on GitHub.

echo $COLUMNS says 45 when I open the app.  When I run screen and it shrinks things down, mutt will run under screen, but if I exit screen (and the font stays small) and I run mutt it crashes still.

                                                                                Yeah, there is clearly something unexpected going on when mutt thinks it is talking to xterm-color. I will try to reproduce. Are you running any mutt plugins or anything?

I don't think so.  my server's muttrc isn't even very crazy

                                                                                Neither is mine, but I understand I am missing out. :-) Thanks again for the detailed info - I will try to get it sorted over the weekend.

I tried setting TERM=vt100 and that still works

Okay, I have a Debian 8.0.0 install going here. I can confirm that screen is explicitly setting columns=80, which is why the font gets so small (to fit 80 columns across). You can reset the font size with Metamode-s. I can set a preference to let the user enable / disable this behaviour.
I cannot reproduce getting Term48 to exit with mutt, but I also cannot replicate your exact setup. My fresh debian 8.0.0 install comes with a newer kernel and mutt 1.5.23. Will keep working it.

If you send me an ssh public key I could give you an account on my actual box

Oh, this made things a million times faster to fix. I think this should do it:
https://github.com/mordak/Term48/commit/d0f8a90de2e66387ac3c9083327287064bc066d4
Thanks again! I'll push a new build shortly. :-)

Build <IP_ADDRESS> is showing in BlackBerry World for me now. Can you confirm the fix?

Working! Thanks!


Allow user to post a feedback for a given repo

User Story: Given a single logged in user, he should be able to vote for a given repository. The voting should be based on a feedback text and the score equivalent to Documentation, Design, Support and Maturity.

Once the score has been posted, the user should be able to see his feedback on the repo page. Also the repo score should be updated accordingly.

Technical:
Create the backend infrastructure responsible for receiving a request with the feedback and score of a Repo and saving it on the database. Also, once a repo has been updated, it's contributors should have their karma updated accordingly. This should be done asynchronously to allow scaling, using the Actor design pattern and Play Framework.

Feature updated on dev branch