The command '/bin/sh -c apk update' returned a non-zero code: 139

I get the error message The command '/bin/sh -c apk update' returned a non-zero code: 139 when I run the following dockerfile
FROM gliderlabs/alpine:3.3
RUN apk update
RUN apk add curl bash

This is running on ubuntu 16. I recently changed my kernel version which I initially thought was the problem but running other images seems to be fine, so I dont think this is the issue.
Any ideas?

Seeing the exact same problem on 16.04 with kernel 4.6.0-040600-generic #201606030904. Not sure if it's related to alpine.

I have same problem. I reported it in docker repos https://github.com/docker/docker/issues/23471

I'm actually getting a 139/SIGSEGV with a much more simple invocation on Alpine:
➜  /tmp docker run -ti --entrypoint=/bin/sh alpine /bin/sh
➜  /tmp echo $?
139

I've added more context on docker/docker#23471 as well.

I can't reproduce this on Docker 1.12.1. Is this still an issue?
root@ubuntu16:~# cat /etc/os-release
NAME="Ubuntu"
VERSION="16.04.1 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.1 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
UBUNTU_CODENAME=xenial
root@ubuntu16:~# uname -a
Linux ubuntu16 4.4.0-21-generic #37-Ubuntu SMP Mon Apr 18 18:33:37 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
root@ubuntu16:~# docker version
Client:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   23cf638
 Built:        Thu Aug 18 05:33:38 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   23cf638
 Built:        Thu Aug 18 05:33:38 2016
 OS/Arch:      linux/amd64
root@ubuntu16:~# docker pull alpine:latest && docker run -ti alpine:latest /bin/sh
latest: Pulling from library/alpine
Digest: sha256:ca7b185775966003d38ccbd9bba822fb570766e4bb69292ac23490f36f8a742e
Status: Image is up to date for alpine:latest
/ # echo "Hi from Alpine"
Hi from Alpine
/ # exit
root@ubuntu16:~#

Still busted for me:
➜  ~ docker --version
Docker version 1.12.1, build 23cf638
➜  ~ cat /etc/os-release
NAME="Ubuntu"
VERSION="16.04.1 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.1 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
UBUNTU_CODENAME=xenial
➜  ~ uname -a
Linux atacama 4.6.7-xanmod9 #1 SMP Thu Aug 18 01:45:56 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
➜  ~ docker pull alpine:latest && docker run -ti alpine:latest /bin/sh
latest: Pulling from library/alpine

c0cb142e4345: Pull complete
Digest: sha256:ca7b185775966003d38ccbd9bba822fb570766e4bb69292ac23490f36f8a742e
Status: Downloaded newer image for alpine:latest
➜  ~ echo $?
139

Is it related to the XanMod kernel per https://forum.xanmod.org/thread-55.html? Are you able to try kernel 4.7 per the last post?

I installed latest 4.8-based xanmod and now the above command works.

Going to close this as it is related to a custom kernel which we can't really support or account for. It sounds like later versions of this kernel are working better with Docker.

root@zz:~/rvm-package# dpkg -l | grep docker
ii  docker.io                      1.12.1-0ubuntu13~16.04.1           arm64        Linux container runtime
root@zz:~/rvm-package# uname -a
Linux zz.local.lan 4.4.0-38-generic #57-Ubuntu SMP Wed Sep 7 10:19:14 UTC 2016 aarch64 aarch64 aarch64 GNU/Linux

root@zz:~/rvm-package# docker build -t rvm-package .
Sending build context to Docker daemon 69.63 kB
Step 1 : FROM ajeetraina/centos7-arm
latest: Pulling from ajeetraina/centos7-arm
5584ea4c92c5: Pull complete
a3ed95caeb02: Pull complete
Digest: sha256:10989b1b7a3bdf826857ba5b3348956d495b9be1ced644a3aa7321cfbd705b04
Status: Downloaded newer image for ajeetraina/centos7-arm:latest
 ---> 495db0a341d8
Step 2 : MAINTAINER Petr Archakov<EMAIL_ADDRESS> ---> Running in 0ff448edc7a5
 ---> b5183f89eeaf
Removing intermediate container 0ff448edc7a5
Step 3 : RUN yum install -y wget curl tar make gcc-c++ rpm-build nano which   libyaml-devel autoconf readline-devel zlib-devel libffi-devel openssl-devel   automake libtool bison sqlite-devel
 ---> Running in 773809bcf027
The command '/bin/sh -c yum install -y wget curl tar make gcc-c++ rpm-build nano which   libyaml-devel autoconf readline-devel zlib-devel libffi-devel openssl-devel   automake libtool bison sqlite-devel' returned a non-zero code: 139

i think this thing is running the standard kernel.
root@zz:~/rvm-package# dpkg -l | grep linux-
ii  linux-base                     4.0ubuntu1                         all          Linux image base package

it's a cavium box at packet.net, you can reproduce there quite easily.

Seeing the exact same problem on 16.04 with kernel 4.6.0-040600-generic #201606030904. Not sure if it's related to alpine.
EDIT: Running fine on 4.4.0-23-generic #41-Ubuntu

Hi, I am not use alpine,  following is my host info:
$: uname -a
Linux zw963-PC 4.14.0-deepin2-amd64 #1 SMP PREEMPT Deepin 4.14.12-2 (2018-01-06) x86_64 GNU/Linux

I got exact same error in this system, but same Dockerfile, it worked in my another linux laptop (4.4.115)

@andyshinn ,  @johanbrandhorst , do you think this is a issue which still not fixed?

I haven't seen it for ages but I'm not using kernel 4.6 anymore either.


FQCN of assertion classes are not replaced in assertions entry point classes when using the generateAssertionsInPackage option.

I use generateAssertionsInPackage option like following.
Project structure
|-src/main/java
|  `-sample/
|    `-Foo.java
`-pom.xml

pom.xml
<configuration>
  <packages>
    <param>sample</param>
  </packages>
  <generateAssertionsInPackage>sample.assertion</generateAssertionsInPackage>
</configuration>

Assertion class is generated as sample.assertion.FooAssert.
FooAssert.java
package sample.assertion;

import sample.Foo;

...

public class FooAssert extends AbstractFooAssert<FooAssert, Foo> {

However assertions entry point classes are generated like following.
Assertions.java
package sample.assertion;

...
@javax.annotation.Generated(value="assertj-assertions-generator")
public class Assertions {

  ...
  @org.assertj.core.util.CheckReturnValue
  public static sample.FooAssert assertThat(sample.Foo actual) {
    return new sample.FooAssert(actual);
  }

FooAssert is written as sample.FooAssert (not sample.assertion.FooAssert).

Its a bug then, thanks for reporting this !

Is there any updates for it?

Nope, I unfortunately don't have the time to work on the generator project these days :(


DateParser is not a dependency of SearchLight

I tried to add it but I the package manager forced me to update a bunch of packages and i don't know if that is desired. That is why I did not make a PR.
And also, DateParser is not being imported in SearchLight.jl

Thanks - I'll add it

I think this can be safely closed now. The API has changed a lot and the standard way is to add specialized methods to Base.convert. The place for adding these is config/initializers/converters.jl for Genie apps.


created doc string for to_pandas function

Added docstring for to_pandas function.

@camharris22 you have a merge conflict with @wiwang's “to_color” function, I think because you added yours in the same line as where he put his. You should be able to resolve the merge conflict in the online editor.


How to use with LM Studio?

Issue
I just installed LM Studio because it supports MLX on Apple Silicon.
I configured the OpenAI compatible endpoint and am trying to call this. This is my command line:
OPENAI_API_KEY=whatever aider --openai-api-base http://localhost:1324 --model openai/qwen2.5-coder-14b-instruct

All I get is:
litellm.APIError: APIError: OpenAIException - Connection error.
Retrying in 0.2 seconds...

If I use http://localhost:1234/v1 nothing changes.
I verified that I can call the endpoint:
$ curl http://localhost:1234/v1/models
{
  "data": [
    {
      "id": "mistral-nemo-instruct-2407",
      "object": "model",
      "owned_by": "organization_owner"
    },
    {
      "id": "qwen2.5-coder-14b-instruct",
      "object": "model",
      "owned_by": "organization_owner"
    },
    {
      "id": "meta-llama-3.1-8b-instruct",
      "object": "model",
      "owned_by": "organization_owner"
    },
    {
      "id": "text-embedding-nomic-embed-text-v1.5",
      "object": "model",
      "owned_by": "organization_owner"
    }
  ],
  "object": "list"
}

What am I missing?
Version and model info
Aider 0.64.1
Followup
It doesn't even seem to contact the endpoint. I started with --verbose and got this:
Too soon to check version: 2.4 hours
Command Line Args:   --openai-api-base http://localhost:1324 --model openai/qwen2.5-coder-14b-instruct --verbose --no-show-model-warnings

Environment Variables:
  OPENAI_API_KEY:    ...ever

Defaults:
  --model-settings-file:.aider.model.settings.yml
  --model-metadata-file:.aider.model.metadata.json
  --env-file:        /Volumes/ExtSSD/aidertest/.env
  --cache-keepalive-pings:0
  --map-refresh:     auto
  --map-multiplier-no-files:2
  --input-history-file:/Volumes/ExtSSD/aidertest/.aider.input.history
  --chat-history-file:/Volumes/ExtSSD/aidertest/.aider.chat.history.md
  --user-input-color:#00cc00
  --tool-error-color:#FF2222
  --tool-warning-color:#FFA500
  --assistant-output-color:#0088ff
  --code-theme:      default
  --aiderignore:     /Volumes/ExtSSD/aidertest/.aiderignore
  --lint-cmd:        []
  --test-cmd:        []
  --encoding:        utf-8
  --voice-format:    wav
  --voice-language:  en

Option settings:
  - aiderignore: /Volumes/ExtSSD/aidertest/.aiderignore
  - analytics: None
  - analytics_disable: False
  - analytics_log: None
  - anthropic_api_key: None
  - apply: None
  - apply_clipboard_edits: False
  - assistant_output_color: #0088ff
  - attribute_author: True
  - attribute_commit_message_author: False
  - attribute_commit_message_committer: False
  - attribute_committer: True
  - auto_commits: True
  - auto_lint: True
  - auto_test: False
  - cache_keepalive_pings: 0
  - cache_prompts: False
  - chat_history_file: /Volumes/ExtSSD/aidertest/.aider.chat.history.md
  - chat_language: None
  - check_update: True
  - code_theme: default
  - commit: False
  - commit_prompt: None
  - completion_menu_bg_color: None
  - completion_menu_color: None
  - completion_menu_current_bg_color: None
  - completion_menu_current_color: None
  - config: None
  - dark_mode: False
  - dirty_commits: True
  - dry_run: False
  - edit_format: None
  - editor: None
  - editor_edit_format: None
  - editor_model: None
  - encoding: utf-8
  - env_file: /Volumes/ExtSSD/aidertest/.env
  - exit: False
  - fancy_input: True
  - file: None
  - files: []
  - git: True
  - gitignore: True
  - gui: False
  - input_history_file: /Volumes/ExtSSD/aidertest/.aider.input.history
  - install_main_branch: False
  - just_check_update: False
  - light_mode: False
  - lint: False
  - lint_cmd: []
  - list_models: None
  - llm_history_file: None
  - load: None
  - map_multiplier_no_files: 2
  - map_refresh: auto
  - map_tokens: None
  - max_chat_history_tokens: None
  - message: None
  - message_file: None
  - model: openai/qwen2.5-coder-14b-instruct
  - model_metadata_file: .aider.model.metadata.json
  - model_settings_file: .aider.model.settings.yml
  - openai_api_base: http://localhost:1324
  - openai_api_deployment_id: None
  - openai_api_key: ...ever
  - openai_api_type: None
  - openai_api_version: None
  - openai_organization_id: None
  - pretty: True
  - read: None
  - restore_chat_history: False
  - show_diffs: False
  - show_model_warnings: False
  - show_prompts: False
  - show_release_notes: None
  - show_repo_map: False
  - skip_sanity_check_repo: False
  - stream: True
  - subtree_only: False
  - suggest_shell_commands: True
  - test: False
  - test_cmd: []
  - tool_error_color: #FF2222
  - tool_output_color: None
  - tool_warning_color: #FFA500
  - upgrade: False
  - user_input_color: #00cc00
  - verbose: True
  - verify_ssl: True
  - vim: False
  - voice_format: wav
  - voice_language: en
  - weak_model: None
  - yes_always: None

Checking imports for version 0.64.1 and executable /nix/store/qw9bwqbwcai9iaqiyir18knz91sff9am-python3-3.11.10/bin/python3.11
Installs file: /Volumes/ExtSSD/.aider/installs.json
Installs file exists and loaded
Not first run, loading imports in background thread
No model settings files loaded
Searched for model settings files:
  - /Volumes/ExtSSD/.aider.model.settings.yml
  - /Volumes/ExtSSD/aidertest/.aider.model.settings.yml
Model info:
{}
Aider v0.64.1
Model: openai/qwen2.5-coder-14b-instruct with whole edit format
Git repo: .git with 0 files
Repo-map: disabled
Use /help <question> for help, run "aider --help" to see cmd line args
────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
> Write a Wurmi game in Python with PyGame. Wurmi is a variant of the Snake formula where there are multiple pieces of food on the screen at once, with
additional pieces of poison one must not run into. The level is cleared when all pieces of food have been eaten.

SYSTEM Act as an expert software developer.
SYSTEM Take requests for changes to the supplied code.
SYSTEM If the request is ambiguous, ask questions.
SYSTEM
SYSTEM Always reply to the user in the same language they are using.
SYSTEM
SYSTEM
SYSTEM Once you understand the request you MUST:
SYSTEM 1. Determine if any code changes are needed.
SYSTEM 2. Explain any needed changes.
SYSTEM 3. If changes are needed, output a copy of each file that needs changes.
SYSTEM
SYSTEM To suggest changes to a file you MUST return the entire content of the updated file.
SYSTEM You MUST use this *file listing* format:
SYSTEM
SYSTEM path/to/filename.js
SYSTEM ```
SYSTEM // entire file content ...
SYSTEM // ... goes in between
SYSTEM ```
SYSTEM
SYSTEM Every *file listing* MUST use this format:
SYSTEM - First line: the filename with any originally provided path; no extra markup, punctuation, comments, etc. **JUST** the filename with path.
SYSTEM - Second line: opening ```
SYSTEM - ... entire content of the file ...
SYSTEM - Final line: closing ```
SYSTEM
SYSTEM To suggest changes to a file you MUST return a *file listing* that contains the entire content of the file.
SYSTEM *NEVER* skip, omit or elide content from a *file listing* using "..." or by adding comments like "... rest of code..."!
SYSTEM Create a new file you MUST return a *file listing* which includes an appropriate filename, including any appropriate path.
SYSTEM
SYSTEM

USER Change the greeting to be more casual

ASSISTANT Ok, I will:
ASSISTANT
ASSISTANT 1. Switch the greeting text from "Hello" to "Hey".
ASSISTANT
ASSISTANT show_greeting.py
ASSISTANT ```
ASSISTANT import sys
ASSISTANT
ASSISTANT def greeting(name):
ASSISTANT     print(f"Hey {name}")
ASSISTANT
ASSISTANT if __name__ == '__main__':
ASSISTANT     greeting(sys.argv[1])
ASSISTANT ```

USER I switched to a new code base. Please don't consider the above files or try to edit them any longer.

ASSISTANT Ok.

USER I am not sharing any files that you can edit yet.

ASSISTANT Ok.

USER Write a Wurmi game in Python with PyGame. Wurmi is a variant of the Snake formula where there are multiple pieces of food on the screen at once, with additional pieces of poison one must not run into. The level is cleared when all pieces of food have been eaten.
litellm.APIError: APIError: OpenAIException - Connection error.
Retrying in 0.2 seconds...
litellm.APIError: APIError: OpenAIException - Connection error.
Retrying in 0.5 seconds...

Thanks for trying aider and filing this issue. This doc may be helpful:
https://aider.chat/docs/llms/lm-studio.html

This fails with:
LM_STUDIO_API_KEY=whatever aider --model lm_studio/qwen2.5-coder-14b-instruct --verbose --no-show-model-warnings
Config files search order, if no --config:
  - /Volumes/ExtSSD/aidertest/.aider.conf.yml
  - /Volumes/ExtSSD/.aider.conf.yml
────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
Too soon to check version: 2.5 hours
Command Line Args:   --model lm_studio/qwen2.5-coder-14b-instruct --verbose --no-show-model-warnings
Defaults:
  --model-settings-file:.aider.model.settings.yml
  --model-metadata-file:.aider.model.metadata.json
  --env-file:        /Volumes/ExtSSD/aidertest/.env
  --cache-keepalive-pings:0
  --map-refresh:     auto
  --map-multiplier-no-files:2
  --input-history-file:/Volumes/ExtSSD/aidertest/.aider.input.history
  --chat-history-file:/Volumes/ExtSSD/aidertest/.aider.chat.history.md
  --user-input-color:#00cc00
  --tool-error-color:#FF2222
  --tool-warning-color:#FFA500
  --assistant-output-color:#0088ff
  --code-theme:      default
  --aiderignore:     /Volumes/ExtSSD/aidertest/.aiderignore
  --lint-cmd:        []
  --test-cmd:        []
  --encoding:        utf-8
  --voice-format:    wav
  --voice-language:  en

Option settings:
  - aiderignore: /Volumes/ExtSSD/aidertest/.aiderignore
  - analytics: None
  - analytics_disable: False
  - analytics_log: None
  - anthropic_api_key: None
  - apply: None
  - apply_clipboard_edits: False
  - assistant_output_color: #0088ff
  - attribute_author: True
  - attribute_commit_message_author: False
  - attribute_commit_message_committer: False
  - attribute_committer: True
  - auto_commits: True
  - auto_lint: True
  - auto_test: False
  - cache_keepalive_pings: 0
  - cache_prompts: False
  - chat_history_file: /Volumes/ExtSSD/aidertest/.aider.chat.history.md
  - chat_language: None
  - check_update: True
  - code_theme: default
  - commit: False
  - commit_prompt: None
  - completion_menu_bg_color: None
  - completion_menu_color: None
  - completion_menu_current_bg_color: None
  - completion_menu_current_color: None
  - config: None
  - dark_mode: False
  - dirty_commits: True
  - dry_run: False
  - edit_format: None
  - editor: None
  - editor_edit_format: None
  - editor_model: None
  - encoding: utf-8
  - env_file: /Volumes/ExtSSD/aidertest/.env
  - exit: False
  - fancy_input: True
  - file: None
  - files: []
  - git: True
  - gitignore: True
  - gui: False
  - input_history_file: /Volumes/ExtSSD/aidertest/.aider.input.history
  - install_main_branch: False
  - just_check_update: False
  - light_mode: False
  - lint: False
  - lint_cmd: []
  - list_models: None
  - llm_history_file: None
  - load: None
  - map_multiplier_no_files: 2
  - map_refresh: auto
  - map_tokens: None
  - max_chat_history_tokens: None
  - message: None
  - message_file: None
  - model: lm_studio/qwen2.5-coder-14b-instruct
  - model_metadata_file: .aider.model.metadata.json
  - model_settings_file: .aider.model.settings.yml
  - openai_api_base: None
  - openai_api_deployment_id: None
  - openai_api_key: None
  - openai_api_type: None
  - openai_api_version: None
  - openai_organization_id: None
  - pretty: True
  - read: None
  - restore_chat_history: False
  - show_diffs: False
  - show_model_warnings: False
  - show_prompts: False
  - show_release_notes: None
  - show_repo_map: False
  - skip_sanity_check_repo: False
  - stream: True
  - subtree_only: False
  - suggest_shell_commands: True
  - test: False
  - test_cmd: []
  - tool_error_color: #FF2222
  - tool_output_color: None
  - tool_warning_color: #FFA500
  - upgrade: False
  - user_input_color: #00cc00
  - verbose: True
  - verify_ssl: True
  - vim: False
  - voice_format: wav
  - voice_language: en
  - weak_model: None
  - yes_always: None

Checking imports for version 0.64.1 and executable /nix/store/qw9bwqbwcai9iaqiyir18knz91sff9am-python3-3.11.10/bin/python3.11
Installs file: /Volumes/ExtSSD/.aider/installs.json
Installs file exists and loaded
Not first run, loading imports in background thread
No model settings files loaded
Searched for model settings files:
  - /Volumes/ExtSSD/.aider.model.settings.yml
  - /Volumes/ExtSSD/aidertest/.aider.model.settings.yml
Model info:
{}
Aider v0.64.1
Model: lm_studio/qwen2.5-coder-14b-instruct with whole edit format
Git repo: .git with 0 files
Repo-map: disabled
Use /help <question> for help, run "aider --help" to see cmd line args
────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
> Write a Wurmi game in Python with PyGame. Wurmi is a variant of the Snake formula where there are multiple pieces of food on the screen at once, with
additional pieces of poison one must not run into. The level is cleared when all pieces of food have been eaten.

SYSTEM Act as an expert software developer.
SYSTEM Take requests for changes to the supplied code.
SYSTEM If the request is ambiguous, ask questions.
SYSTEM
SYSTEM Always reply to the user in the same language they are using.
SYSTEM
SYSTEM
SYSTEM Once you understand the request you MUST:
SYSTEM 1. Determine if any code changes are needed.
SYSTEM 2. Explain any needed changes.
SYSTEM 3. If changes are needed, output a copy of each file that needs changes.
SYSTEM
SYSTEM To suggest changes to a file you MUST return the entire content of the updated file.
SYSTEM You MUST use this *file listing* format:
SYSTEM
SYSTEM path/to/filename.js
SYSTEM ```
SYSTEM // entire file content ...
SYSTEM // ... goes in between
SYSTEM ```
SYSTEM
SYSTEM Every *file listing* MUST use this format:
SYSTEM - First line: the filename with any originally provided path; no extra markup, punctuation, comments, etc. **JUST** the filename with path.
SYSTEM - Second line: opening ```
SYSTEM - ... entire content of the file ...
SYSTEM - Final line: closing ```
SYSTEM
SYSTEM To suggest changes to a file you MUST return a *file listing* that contains the entire content of the file.
SYSTEM *NEVER* skip, omit or elide content from a *file listing* using "..." or by adding comments like "... rest of code..."!
SYSTEM Create a new file you MUST return a *file listing* which includes an appropriate filename, including any appropriate path.
SYSTEM
SYSTEM

USER Change the greeting to be more casual

ASSISTANT Ok, I will:
ASSISTANT
ASSISTANT 1. Switch the greeting text from "Hello" to "Hey".
ASSISTANT
ASSISTANT show_greeting.py
ASSISTANT ```
ASSISTANT import sys
ASSISTANT
ASSISTANT def greeting(name):
ASSISTANT     print(f"Hey {name}")
ASSISTANT
ASSISTANT if __name__ == '__main__':
ASSISTANT     greeting(sys.argv[1])
ASSISTANT ```

USER I switched to a new code base. Please don't consider the above files or try to edit them any longer.

ASSISTANT Ok.

USER I am not sharing any files that you can edit yet.

ASSISTANT Ok.

USER Write a Wurmi game in Python with PyGame. Wurmi is a variant of the Snake formula where there are multiple pieces of food on the screen at once, with additional pieces of poison one must not run into. The level is cleared when all pieces of food have been eaten.

litellm.BadRequestError: LLM Provider NOT provided. Pass in the LLM provider you are trying to call. You passed
model=lm_studio/qwen2.5-coder-14b-instruct
 Pass model as E.g. For 'Huggingface' inference endpoints pass in `completion(model='huggingface/starcoder',..)` Learn more:
https://docs.litellm.ai/docs/providers

I got it to work with the OpenAI compatible endpoint, using the IP address <IP_ADDRESS> instead of the hostname localhost.

I installed it with nix-darwin with a modified version of this Nix file.
Here is my aider.nix:
{ lib
, stdenv
, python311
, fetchFromGitHub
, gitMinimal
, portaudio
, vim
}:

let
  python3 = python311.override {
    self = python3;
    packageOverrides = _: super: { tree-sitter = super.tree-sitter_0_21; };
  };
  version = "0.64.1";
  aider-chat = python3.pkgs.buildPythonApplication {
    pname = "aider-chat";
    inherit version;
    pyproject = true;

    src = fetchFromGitHub {
      owner = "Aider-AI";
      repo = "aider";
      rev = "refs/tags/v${version}";
      hash = "sha256-sCLIFp9OTaA/UcHLrNG3Wwiar58ayC9TUdCZdplS0LY=";
    };

    pythonRelaxDeps = true;
    doCheck = false;

    build-system = with python3.pkgs; [ setuptools-scm ];

    dependencies = with python3.pkgs; [
      aiohappyeyeballs
      aiohttp
      aiosignal
      annotated-types
      anyio
      attrs
      backoff
      beautifulsoup4
      certifi
      cffi
      charset-normalizer
      click
      configargparse
      diff-match-patch
      diskcache
      distro
      filelock
      flake8
      frozenlist
      fsspec
      gitdb
      gitpython
      grep-ast
      h11
      httpcore
      httpx
      huggingface-hub
      idna
      importlib-resources
      jinja2
      jiter
      json5
      jsonschema
      jsonschema-specifications
      litellm
      markdown-it-py
      markupsafe
      mccabe
      mdurl
      multidict
      networkx
      numpy
      openai
      packaging
      pathspec
      pexpect
      pillow
      prompt-toolkit
      psutil
      ptyprocess
      pycodestyle
      pycparser
      pydantic
      pydantic-core
      pydub
      pyflakes
      pygments
      pypandoc
      pyperclip
      python-dotenv
      pyyaml
      referencing
      regex
      requests
      rich
      rpds-py
      scipy
      smmap
      sniffio
      sounddevice
      soundfile
      soupsieve
      tiktoken
      tokenizers
      tqdm
      tree-sitter
      tree-sitter-languages
      typing-extensions
      urllib3
      wcwidth
      yarl
      zipp

      # Not listed in requirements
      mixpanel
      monotonic
      posthog
      propcache
      python-dateutil
    ];

    buildInputs = [
      vim
      portaudio
    ];

    nativeCheckInputs = (with python3.pkgs; [ pytestCheckHook ]) ++ [ gitMinimal ];

    disabledTestPaths = [
      # Tests require network access
      "tests/scrape/test_scrape.py"
      # Expected 'mock' to have been called once
      "tests/help/test_help.py"
    ];

    disabledTests =
      [
        # Tests require network
        "test_urls"
        "test_get_commit_message_with_custom_prompt"
        # FileNotFoundError
        "test_get_commit_message"
        # Expected 'launch_gui' to have been called once
        "test_browser_flag_imports_streamlit"
        # AttributeError
        "test_simple_send_with_retries"
        # Expected 'check_version' to have been called once
        "test_main_exit_calls_version_check"
      ]
      ++ lib.optionals stdenv.hostPlatform.isDarwin [
        # Tests fails on darwin
        "test_dark_mode_sets_code_theme"
        "test_default_env_file_sets_automatic_variable"
      ];

    preCheck = ''
      export HOME=$(mktemp -d)
      export AIDER_CHECK_UPDATE=false
    '';

    optional-dependencies = with python3.pkgs; {
      playwright = [
        greenlet
        playwright
        pyee
        typing-extensions
      ];
    };

    passthru = {
      withPlaywright = aider-chat.overridePythonAttrs (
        { dependencies, ... }:
        {
          dependencies = dependencies ++ aider-chat.optional-dependencies.playwright;
        }
      );
    };

    meta = {
      description = "AI pair programming in your terminal";
      homepage = "https://github.com/paul-gauthier/aider";
      changelog = "https://github.com/paul-gauthier/aider/blob/v${version}/HISTORY.md";
      license = lib.licenses.asl20;
      maintainers = with lib.maintainers; [ taha-yassine ];
      mainProgram = "aider";
    };
  };
in
aider-chat

And this is how I include it into my home-manager configuration:
{ config, pkgs, lib, ... }:
let
  aider-chat = pkgs.callPackage ./software/aider.nix { };
in
{
  # rest omitted
  home.packages = [
    aiderchat
    # rest omitted
  ];
}

Chances are you got the wrong dependencies by installing it that way. For future reference:
https://aider.chat/docs/troubleshooting/imports.html

This was only a first test anyway. I thought of something like that as the source for the problem already.
In the future, I am going to use Exo with aider anyway because I plan to set up a Mac mini cluster, and the OpenAI-like API endpoint from Exo works flawlessly with Aider.
I am using the Nix approach to keep my system relatively tidy, therefore I first looked into the Nix way. However I might be better off just getting bare-bones Python from Nixpkgs and use the classic venv / pip approach in the future.


ARROW-2293: [JS] Print release vote e-mail template when making source release

@wesm I welcome any suggestions to modify the template, might as well get it right now. The current template is essentially copy-pasted from your 0.3.1 rc0 email

thanks @TheNeuralBit!


Integration tests

We need puppeteer tests for

[ ] webpack
[ ] next.js
[x] vite
[ ] snowpack
[ ] nollup

Just thinking about this more (still trying to get the current tests running on my laptop), do you think it might be worthwhile having a look to have a common set of test cases and run the same ones for each bundler? Something like the following:
[vite, snowpack, nollup].forEach(engine => {
    beforeEach(() => {
          // start dev server
     });

    describe('some test case', () => {
         it ('should update');
    });
});

Would reduce a lot of duplication. Fixtures might have to be duplicated though because of different index.html files though. Or do you think it would be better to keep tests separate for the time being?

I think there's value in having a general test suite so we can test it all at once but it's going to be really hard to find the right abstraction to make all integrations work.


Attaching a node to document.body freezes the page

Hi
The component I'm working on is supposed to attach a node to doument.body when it's being rendered and it works perfectly fine outside of playground, but as soon as playgorund tries to run it the entire page freezes and I have to close the tab.
I get this message on the sources page paused before potential out of memory crash with this line highlighted in the react-dom-server.browser.development.js file:
while (out.length < bytes) { and the value of bytes is infinity
Thanks for the help :)


docs: Provide new website template

Relates to https://github.com/arcus-azure/arcus.github.template

@tomkerkhove Done

Small remark: can we indent the "Azure Blob Storage", "Azure File Share" and "Azure Table Storage" pages to become a sub-page of "Azure Storage"?

@mbraekman Was a bit harder than expected but storage files are now sitting inside an extra directory and I've added redirect rules for the old urls.


Update http4s-blaze-client, http4s-circe, ... to 0.21.18

Updates

org.http4s:http4s-blaze-client
org.http4s:http4s-circe
org.http4s:http4s-client
org.http4s:http4s-core
org.http4s:http4s-dsl
org.http4s:http4s-server

from 0.21.17 to 0.21.18.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch

Superseded by #110.


soft delete

when i am deleting a record from a table and redirect to the list i am getting an error. To show the list i have used this package. it was working fine before the soft delete.
ErrorException in AliasLoader.php line 63: Class 'yajra\Datatables\Facades\Datatables' not found
but in my aliases array in app.php there is 'Datatables' => yajra\Datatables\Facades\Datatables::class
added. :( help me.

i do not know what happen when i am adding Facade like this way
'Datatables' => yajra\Datatables\Facades\Datatables::class
but now i am adding the facade like this way it is working.
'Datatables' => yajra\Datatables\Datatables::class
but my question is why this behaviour.

@omar-faruk-sust , thanks for reporting. 'Datatables' => yajra\Datatables\Datatables::class should be the correct alias. updated the read me. Thanks!


Update MD file on new branch 2017-11-29_09-41-07

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/content1.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.


[BUG] Leaves space on right side

Description
Below the 640px breakpoint, a space on the right side of the page appears which takes up some of the space of the page. I have provided a screenshot for better understanding :) This could not be a bug if that space has been left on purpose, if so plz close this issue😁
Screenshots

Do you want to work on this issue?
None
Additional information
No response

Thank you for the suggestion. I think this has now been fixed as part of another issue, so I will close for now. Also we will be redesigning the whole app soon also


Image downloader

The current Booru downloader is broken, and it should either be fixed, or another method to download images should be added to Edgeware.

I just made a pull request that should at the very least restore basic function to this feature but i have not extensively tested it #41


Nether/End Ore blocks vs generation

Minecraft version & Mod Version: 1.12.2  and 1.11.2
Single player or Server:  Single
Describe problem (what you were doing / what happened):
I disabled Nether and End ore spawn and the blocks were removed from the game..
Crash log (if any):  None
Make an option to keep the ore blocks, just disable the ore gen.  I am using COFH World for all of my ore gen and would like to use the ores from this mod for Netherack/endstone places.
Thanks,
IMarv

Oh I see what you mean, thats a good idea.
Ok ill add a new flag to allow you to "enable" the ore block as true or false, so that way if its 'true' but the spawn count is zero then the block still exists for other worldgen.

Update: turns out there is already a way to do this in config file.
So if you set "Nether Ore"=false , then yes it totally removes the blocks.
However, if you leave "Nether Ore"=true, you can still disable the world gen using the blockCount___ = 0. For example
`        netherorecustom {
# Approximate ore vein size.  Zero means no spawns. [range: 0 ~ 32, default: 8]
I:blockCountCoal=0
        # Approximate ore vein size.  Zero means no spawns. [range: 0 ~ 32, default: 8]
        I:blockCountDiamond=0

        # Approximate ore vein size.  Zero means no spawns. [range: 0 ~ 32, default: 8]
        I:blockCountEmerald=0

        # Approximate ore vein size.  Zero means no spawns. [range: 0 ~ 32, default: 8]
        I:blockCountGold=0

        # Approximate ore vein size.  Zero means no spawns. [range: 0 ~ 32, default: 8]
        I:blockCountIron=0

        # Approximate ore vein size.  Zero means no spawns. [range: 0 ~ 32, default: 8]
        I:blockCountLapis=0

        # Approximate ore vein size.  Zero means no spawns. [range: 0 ~ 32, default: 8]
        I:blockCountRedstone=0`

Excellent!
Sorry for my derp.


AWS Glue's python shell & apache spark jobs as batch processing

I am currently using Zappa for the Django application to make serverless and manually maintaining aws glue's python jobs for replacing the batch server celery.  can you please make aws-glue as asynchronous batch processing for Zappa framework?
I mean,  I just write normal python functions and i will decorate with some decorators then that functions should automatically become awsglue jobs and callable from Django views.

any responce ?


Square spiral

Supercedes https://github.com/NSLS-II/bluesky/pull/653 (from @awalter-bnl's master branch) and adds cleanup. Needs:

[ ] Tests
[ ] Entries added the plans documentation page

Codecov Report

Merging #676 into master will decrease coverage by 1.06%.
The diff coverage is 3.33%.

@@            Coverage Diff             @@
##           master     #676      +/-   ##
==========================================
- Coverage    82.6%   81.53%   -1.07%
==========================================
  Files          20       20
  Lines        4386     4446      +60
==========================================
+ Hits         3623     3625       +2
- Misses        763      821      +58

Impacted Files
Coverage Δ

bluesky/plan_patterns.py
57.6% <1.92%> (-39.67%)
:arrow_down:

bluesky/plans.py
92.71% <12.5%> (-0.67%)
:arrow_down:

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 b8df766...3af108f. Read the comment docs.

Is #646 related?

Yes.

Hi Guys,
So my attempt to use this yesterday refreshed my memory about the issue that Tom raised. He asked why it was necessary to have both n and m odd or both even for an n x m grid. At the time I mentioned that I was not able to think of a way to define the path without this requirement. However fresh eyes on the problem yesterday and today means I have resolved this issue. Attached is a new version of the square_spiral_pattern, I can add it to GitHub and make the other changes (to the documentation sections of square_spiral and relative_square_spiral) but given I made a mistake in the way to add code last time I wanted to check if I should just ‘edit’ these files on GitHub or fork it first?
cheers
Andrew
Andrew Walter
Assc. Scientist
NSLS II, Photon sciences directorate
Brookhaven National Laboratory
Building 741, Rm 108
Upton New York, USA, 11973
Email: awalter<EMAIL_ADDRESS>Phone:
office:<PHONE_NUMBER>
On Feb 28, 2018, at 2:34 PM, Dan Allan <notifications<EMAIL_ADDRESS>wrote:
Yes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/NSLS-II/bluesky/pull/676#issuecomment-369356465, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYlCpWW80BwEq3RDXt75gNDASSM7r6T5ks5tZapQgaJpZM4MuySi.

I wanted to check if I should just ‘edit’ these files on GitHub or fork it first?

The code in this pull request currently resides on the square-spiral branch of my (danielballan) fork. I suggest fetching that and opening up a new pull request:
git clone https://github.com/awalter-bnl/bluesky  # you might have already done this part
git remote add danielballan https://github.com/danielballan/bluesky
git fetch danielballan
git checkout square-spiral
git checkout -b square-spiral-2
# Copy over your edits to replace the existing files, and commit.
git push origin square-spiral-2

Then open https://github.com/NSLS-II/bluesky in your browser and open a new pull request.

@awalter-bnl In case it doesn't go without saying -- feel free to ask follow-up questions. Somewhere on GitHub there is a public record of other open-source developers holding my hand through this exact process once up a time, and I'm more than happy to pay it forward. GitHub Flow is effective but confusing at first.

Thanks @danielballan I will have a go at this tomorrow, Maksim offered to go over it on Monday so will work it out by then if I am still having issues;-)
On Mar 3, 2018, at 5:45 PM, Dan Allan <notifications<EMAIL_ADDRESS>wrote:
@awalter-bnlhttps://github.com/awalter-bnl In case it doesn't go without saying -- feel free to ask follow-up questions. Somewhere on GitHub there is a public record of other open-source developers holding my hand through this exact process once up a time, and I'm more than happy to pay it forward. GitHub Flow is effective but confusing at first.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/NSLS-II/bluesky/pull/676#issuecomment-370185955, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYlCpQXWit8abRrQ-AhhJJed-O0yrE3vks5taxztgaJpZM4MuySi.

I suspect this PR was superseded by #1001.

Closing due to #1001.

@mrakitin is correct, both #646 and #676 are superseded by #1001

#646 will be automatically closed as a bound issue for PR #1001.
I've just closed this outdated PR #676.


Initial commit of fish-app

Initial commit of the standalone app of the fish shell version 2.1.1.

Shouldn't the name of the Cask just be fish?

Well, I thought that since there are different versions available for download - and this is the app version - I would name it accordingly.

Hi!
Yes, the generate_cask_token script would assign the token fish.  But we would use the same name if we Cask fish.pkg from the same source (which we should).
So let's keep this unusual token for disambiguation.  Avoiding future renames is the most important reason for standardizing.
Thanks, @krestenjacobsen.  The contribution is much appreciated !

On second thought, I have a small tweak in #9731.


Merge ModelApi into Model.

ModelApi was mostly a bunch of named constructors for Model, but there
isn't a particularly good way to handle type annotations for named
constructors that apply only to a subclass rather than the class they're
defined in.
Addresses #93.

Tests and pytype pass btw.


Turn the BLE on and off

Is there a way to turn the BLE on and off when an event happens? When a command is sent via UART for example

Currently, there is no function implemented to stop and start BLE again.
I will try to implement it.

Okay, thank you very much for your response 🙏🏽
I'll be on the lookout in case there's any release of this feature.
🙌🏽


fix: Change default value for allowUnsecureRandom.

Change:

Disables unsecure random by default.

Pre-launch Checklist

[x] I read the Contribute page and followed the process outlined there for submitting PRs.
[x] This update contains only one single feature or bug fix and nothing else. (If you are submitting multiple fixes, please make multiple PRs.)
[x] I read and followed the Dart Style Guide and formatted the code with dart format.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///), and made sure that the documentation follows the same style as other Serverpod documentation. I checked spelling and grammar.
[x] I added new tests to check the change I am making.
[x] All existing and new tests are passing.
[x] Any breaking changes are documented below.

If you need help, consider asking for advice on the discussion board.
Breaking changes
Default value for method changes which might cause the random call to throw.

Added upgrade instructions for this change here: https://github.com/serverpod/serverpod_docs/pull/77/commits/8cee3c29a8ddcbd022675462c1fc273677b7bb08


Create a Logo/Icon/Favicon

can i work on this ?

Hey, sure! 🌈

Hi @buzz-js . How is this logo supposed to look like?

Hi @buzz-js . How is this logo supposed to look like?

I haven't thought of something specific, what do you think?


Remove iOS support flag

Hey thanks for creating a plugin for NativeScript.
I'm the guy behind PNR (https://plugins.nativescript.rocks), and PNR and NativeScript itself; uses the https://github.com/milannankov/nativescript-radio/blob/master/src/package.json#L10 ios flag to detect if the plugin actually supports iOS.    Looking at the code base, I don't see any ios support.     So if you don't mind please remove that in the next release so that the plugin shows up properly as Android only on the plugin sites...   :-)    At the point you do add iOS support, then you re-add the flag and the data will automatically be fixed on PNR.   Please note I do have some documentation on publishing flags at https://plugins.nativescript.rocks/create

Sure, I will do that. Thanks for giving me heads up.


Shade coursier in jar

Fixes https://github.com/sbt/sbt/issues/6402
After shading the launcher.jar contains:
LICENSE  META-INF  module.properties  NOTICE  org  scala  xsbt  xsbti

Before shading it contained:
concurrentrefhashmap  coursier  LICENSE  META-INF  module.properties  NOTICE  org  scala  xsbt  xsbti

@eed3si9n this is ready for review

Actually I think I can simplify this further

Done


cilium-cli: set latest stable version based on stable.txt

Instead of letting renovate update the latest stable Cilium version, use the version defined in stable.txt during build.

i guess we need to do something similar in https://github.com/cilium/cilium-cli/blob/main/Makefile? we still build release binaries from there 💭

Good point. Will send the corresponding change there as well.

i guess we need to do something similar in https://github.com/cilium/cilium-cli/blob/main/Makefile? we still build release binaries from there 💭

Good point. Will send the corresponding change there as well.

https://github.com/cilium/cilium-cli/pull/2783


chore(docs): add CONTRIBUTING doc

What this PR does / why we need it:
chore(docs): add CONTRIBUTING doc
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #122
Reference to #

Special notes for your reviewer:
/cc @gaocegege @simon-cj

Release note:

NONE

/lgtm


opening the vc projects

Hi, I want to modify the source code a little bit. After downloading the source code, I used visual studio 2015 to edit it. But it seems not able to load the projects correctly. Could you please guide me on how to modify the solution? Thanks.

Hi,
That is likely caused by dependencies not being resolved correctly. Could
you tell me the error exactly ? Did you load the sln file or the project
file?
On Tue, May 2, 2017 at 11:10 PM mingguichen<EMAIL_ADDRESS>wrote:

Hi, I want to modify the source code a little bit. After downloading the
source code, I used visual studio 2015 to edit it. But it seems not able to
load the projects correctly. Could you please guide me on how to modify the
solution? Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/oculomatic/oculomatic-release/issues/2, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AIQUIvakxOiUpjCBX2Hwj4sld0NfzAMZks5r1_ArgaJpZM4NO6sX
.

Please see the screenshots below. I tried both sln and vcproj files, the results are the same. Thanks.

oculomatic.vcxproj : error  : Unable to read the project file "oculomatic.vcxproj".
The imported project openFrameworksRelease.props" was not found. Confirm that the path in the  declaration is correct, and that the file exists on disk.
openframeworksLib.vcxproj : error  : Project openframeworksLib.vcxproj could not be found.
Uh, it's related to openframeworks. It 's my first having issues with lib dependents when loading the projects. I'll try to fix it first.

Please check the readme and make sure all he dependencies are installed.
Let me know if you need help.
On Tue, May 2, 2017 at 11:29 PM mingguichen<EMAIL_ADDRESS>wrote:

oculomatic.vcxproj : error : Unable to read the project file
"oculomatic.vcxproj".
The imported project openFrameworksRelease.props" was not found. Confirm
that the path in the declaration is correct, and that the file exists on
disk.
openframeworksLib.vcxproj : error : Project openframeworksLib.vcxproj
could not be found.
Uh, it's related to openframeworks. It 's my first having issues with lib
dependents when loading the projects. I'll try to fix it first.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/oculomatic/oculomatic-release/issues/2#issuecomment-298815807,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIQUIunMs8B6D1wc-s3qepvDxO-1g5gDks5r1_SFgaJpZM4NO6sX
.

What I am missing now is the files within the folder ofxDatGui\src\templates. I searched online, and it seems to be discontinued. Could you please send me your ofxDatGui at<EMAIL_ADDRESS>Thanks.

Just delete the ofxdatgui ref from the project. I only used it for a
voltage display that is currently not enabled anyhow.
On Wed, May 3, 2017 at 1:23 AM mingguichen<EMAIL_ADDRESS>wrote:

What I am missing now is the files within the folder
ofxDatGui\src\templates. I searched online, and it seems to be
discontinued. Could you please send me your ofxDatGui at
<EMAIL_ADDRESS>Thanks.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/oculomatic/oculomatic-release/issues/2#issuecomment-298825461,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIQUIpWq5fM-8VlCcNPgkbzgTnUjnKqhks5r2A9sgaJpZM4NO6sX
.


custom root ca support.

Pivotal has a lot of enterprise customers who have proxies which perform SSL re-encryption, which is like an authorized man-in-the-middle (MITM). Most companies do this to ensure their employees aren't downloading viruses, browsing illegal content, etc., so it's a common use case.
This was an easily solvable problem when the CLI had support for --skip-ssl-validation, but then created a new security risk. This PR adds transport support for a new flag in pivotal-cf/pivnet-cli, --root-ca, which allows for a customer to pass in their company's root CA, so the SSL connection is validated.
Signed-off-by: Mike Lloyd<EMAIL_ADDRESS>

This is a great idea! But unfortunately, it's missing tests. Can you add tests for this feature?

@xtreme-debbie-chen can you point me to the core client tests? I wasn't able to find them.

Is this what you're looking for? https://github.com/pivotal-cf/go-pivnet/blob/master/pivnet_test.go

@pnikonowicz thanks for taking a look at it, I'm not quite done with the testing yet.
Based on my understanding around SSL reencryption, if you reencrypt the traffic at the proxy and tell the client that about the proxy CA cert, it shouldn't throw an error about bad certificates. I'm really not sure why it's throwing that error and I've been doing a bit of research into it.
As far as why I'm not using Ginkgo/Gomega, I can't get easily extend the test framework to do MITM, so I have to stand up the tests manually since there is a lot of setup for a local web server. I totally understand it's not ideal, but I'm not sure how else to go about it.

No problem. I'm going to close the PR request for now and you can reopen when you are ready for us taking a look at it again.
Don't worry about ginkgo, we can and are more than willing to preform that refactor when needed.
I looked into the problem but I time boxed the effort and couldn't figure it out in time. Sorry I can't offer any more assistance, but the general idea looks good from a top level perspective.

I'm going to reopen as I might be able to work on adding tests for this.
We are also considering adding it as a feature but will need to investigate how popular this feature would be.
If in the meantime, we were able to add tests, then this PR could be merged.

@pnikonowicz I haven't made any progress on this since it was closed since I'm not entirely sure why the tests are failing for the reasons I mentioned above. I'm more than happy to work on it if you can help me figure out what is being done incorrectly.


SemAct extentions #enter/#exit hooks

It seems like the Extension enter and exit hooks are only called in the EachOf operator. The documentation says (and I agree) that they should be called when arriving at and leaving a TripleConstraint.
Specifically, I'm trying to implement a Ruby version of ShexMap, and it's important to be able to group variable mappings into "domains" that separate repeated bindings of a variable. Eric's JS implementation has a kind of intermediate representation with lists of objects, where the JSON objects are analogous to the domains I'm talking about.
This is mostly an intent-to-PR, but I wanted to be sure that I'm reasoning correctly here and that there isn't a reason that semantic_actions enter and exit aren't called by every TripleConstraint.

Have at it. The extension stuff was prospective, and can't really handle the real semantics.
I haven't had time to work on this gem for quite a while, and don't know when I'll get back to it, so you're welcome to jump in.


Fixed #33586 -- Reloaded delayed apps cache during backwards migrations for RunPython operation

Ticket #33586

Closing per comment.


Python 3: fix a bytes/str runtime issue

On python3, in grpc._links.service._Kernel._on_service_acceptance_event,
there is a runtime TypeError:
_on_service_acceptance_event
    group, method = service_acceptance.method.split('/')[1:3]
TypeError: 'str' does not support the buffer interface

It is fixed by using a bytes literal (b'/') instead of a string literal.
This exposed another issue in grpc.beta._server where an exception was being
raised with a bytes literal for a message (a string should be used instead.)

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

This is OK to test.

This looks great; thank you for the fix.

Aaand the explicit: lgtm


Add --allow_hpa flag

Hello @aantn,
One suggestion, the —allow_hpa flag could be —allow-hpa instead to follow the format of the other flags, what do you think?

I'd love to. But it's tricky at the moment because of how cli flag names
are autogenerated for strategy variables. I want to get it fixed but
probably not in this PR. Sorry :(
On Sun, Mar 10, 2024, 23:41 dgdevops @.***> wrote:

Hello @aantn https://github.com/aantn,
One suggestion, the —allow_hpa flag could be —allow-hpa instead to follow
the format of the other flags, what do you think?
—
Reply to this email directly, view it on GitHub
https://github.com/robusta-dev/krr/pull/226#issuecomment-1987371538, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADYUBZXPIIJVRT2OC2NRUDYXTHRTAVCNFSM6AAAAABELOVID2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBXGM3TCNJTHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I made a fix for the name of the strategy settings https://github.com/robusta-dev/krr/pull/235

Nice work, thank you.
Just a heads up that I'm flying to Paris for kubecon now, so don't worry if
it takes longer than usual to review and merge.
On Sun, Mar 17, 2024, 19:53 Mykola Martynov @.***>
wrote:

I made a fix for the name of the strategy settings #235
https://github.com/robusta-dev/krr/pull/235
—
Reply to this email directly, view it on GitHub
https://github.com/robusta-dev/krr/pull/226#issuecomment-2002554138, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADYUB2TEJ2YJYTLDZ2OF3DYYXKBZAVCNFSM6AAAAABELOVID2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBSGU2TIMJTHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Heat map error: index pattern does not contain any of the following field types: geo_point

Describe the bug
Heat map vis failing with an error
No Compatible Fields: The logstash-syslog-dns* index pattern does not contain any of the following field types: geo_point
Screenshots

Why is geoip.location not found? I see it in some of the records in the JSON. I do not see it in my index pattern though.

thank you again for your detailed testing @aviationfan !
I suspect it is generally found but not with the correct fied type geo_point so you see it in the jsons but kibana cant use it as a vis source for geographic data.
Please try the following template and test again. Do you have the geoip plugin enabled?
https://github.com/nin9s/elk-hole/pull/15/commits/726efb61c2198810d3479f7142e25548832554f9

Would that change to the template only affect new data or would it fix old data too?

It would only affect a new index, New data in the existing index isn't sufficient.
Index templates are only applied during index creation unfortunately

@aviationfan fixed?

Looks like the heatmap is working but other stuff may not be. I am seeing 79 fields in the index pattern, is that right?

It’s difficult to answer (at least for me) if 79 are correct. I have 100 where 11 are conflicts produced by changes in the field types in the different indexes
Are you expecting something special to not work correctly?
I will clear all indexes but the latest and check back the field types counter

79 fields is correct!

Great, that matches then.


Fix compilation and scaladoc warnings

Fixes #1114, #1115, and #1149.
This cleans up compilation and scaladoc warnings in the build.

build

build


add bib() function to vibration

Summary
vibration issue https://github.com/Expensify/App/issues/4355#issuecomment-895584421
bip() function is added in to https://github.com/Expensify/react-native/blob/master/Libraries/Vibration/Vibration.js react native fork

Changelog

[CATEGORY] [TYPE] - Message
Test Plan

Closing this PR per https://github.com/Expensify/App/issues/4355#issuecomment-903922319


A third mode: 'filesandtags', to aggregate same tag entries in 'file' mode as well

When mode is 'fileandtags', aggregate same tag entries in each file instead of listing each tag as it occurs in the respective file.

Yeah, this makes more sense to an extra mode, the tree of file > tag > todo will have more depth and doesn't really replace the file mode.

New mode implemented with the name "file_tag" in this commit https://github.com/drmargarido/TodoTreeView/commit/deaeda63fc6af2965aa5b3a6d6a7901806983544.

Working as described. Thanks!


Use Chrome skins?

Hi everyone, do you think it would be possible to add support to chrome skins?

If you're talking about your standard Chrome themes, as @Excigma mentioned, they aren't technically supported (you can't install from Chrome web store) but the themes are essentially composed in the exact same way now so you can get the author of the theme to upload it to AMO and you'll be able to use it (or if you're really desperate there's probably some way to obtain the theme and manually install it). This kind of theme is supported by MaterialFox but there are a few elements here and there which will differ from Chrome. Most notably, for very light themes, tabs won't have a border around them. For the most part though, they'll look very similar.
If you're talking about some other kind of theme then no, they won't be supported.
I'll close this now but happy to re-open if we didn't answer your question.


Split up converter.rb?

I'm loving Asciidoctor -- and going directly to PDF with asciidoctor-pdf to in particular. I tested out changing some of the behavior (change the title page layout, header/footer placement, etc), but converter.rb is a pretty large file and hard to find my way around.
It seems to me that it should probably have it's functionality split up into a bunch of other files. Does anybody else think so? Or have any ideas about how to do so?

I agree. In fact, the comment at the header of that file reminds me of this every time I open the file.
This is certainly a goal. However, I'm not willing to refactor this file until #37 is solved because we have no way to verify if we break anything (aside from manual testing).
We also want to join this with #53, because we have to rewrite everything anyway to fit in that new model. It would be a perfect time to address both issues.

Hmm, a unit test suite, that would be great... and make refactoring safer. Seems like a reasonable prerequisite.

I regret putting it off. Trust me.

Btw, the test suite is there now, with > 700 tests.
I still want to see this wrapped up as part of #53 since we don't want to do the work twice.

At this point, I don't have any plans personally to break up converter.rb. But we do have a test suite that is approaching 2,000 tests. That's the backstop I need right now to ensure stability and accommodate refactoring.
Now that we have a test suite, I'm comfortable with someone coming along and proposing different ways to organize the code. But I ask that you keep the changes focused on a particular part of the code to avoid one monolithic change.


More documentation is needed for capabilities/steps/etc.

Documentation should cover around ;

How to contribute
Detailed installation instructions
Detailed running instructions like clone, pip, docker, etc.
Common scenarios to use
How to run in a Jenkins Pipeline
How to create new Features/Scenarios (more like a radish-bdd link ?)
How to create new steps supporting new Features/Scenarios (more like a radish-bdd link ?)

Where can I find the :
How to contribute
Detailed installation instructions
Detailed running instructions like clone, pip, docker, etc.
Common scenarios to use
How to run in a Jenkins Pipeline
How to create new Features/Scenarios (more like a radish-bdd link ?)
How to create new steps supporting new Features/Scenarios (more like a radish-bdd link ?)

How to contribute
Detailed installation instructions
Detailed running instructions like clone, pip, docker, etc.
Where can I find these steps:
Common scenarios to use
How to run in a Jenkins Pipeline
How to create new Features/Scenarios (more like a radish-bdd link ?)
How to create new steps supporting new Features/Scenarios (more like a radish-bdd link ?)


Simplify request handling code

The change to write the incoming request to file, and map that into memory (using ByteBuffer) introduces a new opportunity - continue the refactoring to remove the BufferedReader that we attach back on  that file, and switch to reading from the ``ByteBuffer``` directly.  Currently the code looks a little odd as it drops between both schemes.
Note: The profiling the application it was obvious that there's a speedup to be had as multipart boundaries are traversed.  Instead of reading line by line and throwing the data away, try skipping a number of bytes in a single seek() call!

Also, the in.readLine() calls in decodeMultipartData could conceivably lead to memory exhaustion if a multipart post request is malformed.


Log dehydrated output to a file

This allows us to read it even if the process is killed due to a timeout. The file is deleted on success to avoid clutter.
@chriddyp Please review.

💃  once you consider my comments


Exclude some media directories option

It would be useful to skip some media directories from backuping.
E.g. Cache directories

That's also a great idea! :)

We need before to list all media files via Django built-in API.
And after there is fnmatch (https://docs.python.org/2/library/fnmatch.html#fnmatch.fnmatch) for match file with wildcard.
We must ensure storages are able to list all files in all directories, files' names must be an absolute path from storage's root. Maybe add a walk function to storages.

Now DBbackup can walk through storage, excludes are coming soon !

covered in #300


Fix transparent notification on lock screen

From this:

To this:

Unfortunately I think that this might also take away the ripple drawable, since it's drawn on the same layer that the lockscreen applies the transparency to. Oh well, I'll merge for now and add that to #3.

What ripple?

@BrianValente When you press the notification (> API 21 only), as long as the contentIntent isn't null it should show a ripple effect as feedback. Your change to set the color as the background of a different view also blocks the ripple animation.


Add branded type support

Resolves #54.

Coverage increased (+0.01%) to 99.479% when pulling 9aaf6c176a629715390f433d296ff4e717873931 on yuhr:master into c1f9b1971786a9bb07e641110f2178d35b6f87b5 on pelotom:master.

Usage:
const A = String.withBrand('A');
type A = Static<typeof A>;

// Also `Brand(name, runtype)` can be used
const B = Brand('B', String.withConstraint(x => /NYANPASU/.test(x)));
type B = Static<typeof B>;

// Merging is possible
const C = Brand('A', String.withBrand('B'));
type C = Static<typeof C>;

// FAIL: A static type-checking error happens here
const a1: A = 'NYANPASU';

// PASS: These are OK
const a2 = 'NYANPASU~' as A;
const a3: A = a2;

// PASS: The type of this should be infered as `A`
const a4 = a2;

try {
  // PASS: No error thrown here, because `Brand` is for static type-checking at all
  const b1: B = B.check('NYANPASU' as A);
  const b2: B = B.check(a4);
  const b3: B = B.check(a4 as C);
} catch (err) {
  // Hence should not reach here
}

// FAIL: These should emit errors
const c1: C = A.check('NYANPASU!');
const c2: C = B.check('NYANPASU!!!') as A;

// PASS: These are possible, because `C` is equivalent to `A & B`
const c3: C = A.check('NYANPASU...') as C;
const c4: C = B.check('NYANPASU.') as A & B;
const c5: C = A.withBrand('B').check('NYANPASU?');
const c6: C = B.withBrand('A').check('NYANPASU~*');
const c7: C = C.withBrand('A').check('*NYANPASU*');

Thanks!

In your above merging example:
const C = Brand('A', String.withBrand('B'));
type C = Static<typeof C>;

C is inferred to be
type C = string & {
    [RuntypeName]: "B";
} & {
    [RuntypeName]: "A";
}

which is an uninhabitable type - TypeScript currently doesn't compute that to never but you could imagine a future version of TypeScript doing that (as currently e.g. boolean & string is converted to never).


How to set duration from outside the widget

Can anyone help me setting the duration from outside the widget. I didn't find that in the documentation. I also looked into the source code and saw there is setDuration() function but couldn't find how to use it in documentation. Also I'm not sure if it performs the function I'm looking for.

You can access the widget from outside the same way just like any other jQuery widget. So what you can do is something like this:
const picker = $('#widget').timeDurationPicker('instance');

picker.setDuration('PT3H25M45S');
picker._save();
console.log(picker.translate());

You can access the widget from outside the same way just like any other jQuery widget. So what you can do is something like this:
const picker = $('#widget').timeDurationPicker('instance');

picker.setDuration('PT3H25M45S');
picker._save();
console.log(picker.translate());

Thanks. This does help me achieve what I wanted to do.


Does ioredis handle connections?

Hi, I have a nodejs app that connects to redis via ioredis at various different times (e.g. save, read, delete etc).  I'm not sure what is the best practice for connections.

Should I create a connection at the beginning of starting the app.  Reuse that redis instance whenever I need to?
Create and close a redis connection every time I use it?
It doesn't matter ioredis will detect open connections and use any available?

Apologies if someone has already asked that question, I couldn't find it anywhere

I think you should always use a single redis instance to communicate with redis if your nodejs is a single instance.
you should always use the existing connection ( like connection pool for database), since creating and closing connection will add overhead
that is a good question, unfortunately I do not have knowledge to answer it now, probably someone will answer it for you

thanks @zhongshixi 👍
I'll wait for someone to answer question 3 before closing this issue

Thanks @vflopes


Exception setting "OutputEncoding"

Version: 1.4.1
When I run "Invoke-DeeplTranslateText" in the Powershell 7 console everything is ok.
However, when I use the command in Powershell Studio 2024 with PS7, I get the following error message:
Invoke-DeeplTranslateText.ps1 (254): ERROR: At Line: 254
ERROR: |
ERROR: 254 | [Console]::OutputEncoding = [System.Text.UTF8Encoding]::new()
ERROR: | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
ERROR: | Exception setting "OutputEncoding": "The handle is invalid."
I have commented out line 254 in the Invoke-DeeplTranslateText.ps1 file. Then the error is gone and the application runs properly

Thanks for the hint.
This line actually was a leftover from testing on PowerShell 5.1. But since the module's minimum required PowerShell Version is set to 7.0 due to some other dependencies.
It has been removed now and the module version has been updated to 1.4.2.


[STYLING] Fixed Styling inconsistencies in the form

Fixed minor inconsistencies in the styling of the converter form.
Margin on the Reset Button:
Current:

Fixed:

Size and rounded corners on the Currency select
Current:

Fixed:

@imhardikdesai Did you take a look on this PR? It's a simple change which will increase the styling consistency.


add cut url function support

Please provide Issues links to:

Issues: #

Provide test code:

Description
...

Codecov Report

Merging #86 (0446abb) into master (d3b3c9b) will decrease coverage by 0.91%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #86      +/-   ##
==========================================
- Coverage   91.69%   90.78%   -0.92%
==========================================
  Files          11       11
  Lines         698      705       +7
==========================================
  Hits          640      640
- Misses         37       44       +7
  Partials       21       21

Impacted Files
Coverage Δ

trim.go
83.75% <0.00%> (-8.04%)
:arrow_down:

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 d3b3c9b...2cbb011. Read the comment docs.

Codecov Report

Merging #86 (0446abb) into master (d3b3c9b) will decrease coverage by 0.91%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #86      +/-   ##
==========================================
- Coverage   91.69%   90.78%   -0.92%
==========================================
  Files          11       11
  Lines         698      705       +7
==========================================
  Hits          640      640
- Misses         37       44       +7
  Partials       21       21

Impacted Files
Coverage Δ

trim.go
83.75% <0.00%> (-8.04%)
:arrow_down:

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 d3b3c9b...2cbb011. Read the comment docs.


Add type exactly mapped to hb types (see #175)

Add type exactly mapped to hb types (see #175)

The idea is OK, but this needs continuous updates from harfbuzz mainstream.
And, which enum values are we restricting ourselves to? E.g.: ttf_script_t
added here is missing values present in hb >= 3.0.0.
Also why are _TTB and _BTT directions are commented out?

we don't do support _TTB _BTT as there are vertical direction.

this is the value from our external/harfbuzz-2.8.0/

indeed, there not the one from newer lib

we could add a real ttf_script_t + ttf_direct_t but that's somehow also an api breaking. or is this ok ?

this is the value from our external/harfbuzz-2.8.0/

The thing is, distros can and do build with --disable-harfbuzz-builtin, so they
are not restricted to our embedded version of harfbuzz.

I've updated with real own TTF type
just need to add more hb_script and defines with HB version so that it compiles everywhere

It now changes api, do we want that?

It changes the API but it looks cleaner this way. ttf 2.0.18 is still new so it doesn't seem so an issue if 2.0.20 comes soon.
We can also revert to the previous version ..
@slouken ?

If we really do change api, it would be better to enumerate TTF_SCRIPT_INVALID as -1, so that we can insert more new values before it later.

I'm thinking we should not change the API. If a new version of HarfBuzz comes out the application can pass through new values that SDL_ttf doesn't know about (and shouldn't care about.)
Don't change this yet, let's think this through a little more.

Indeed TTF_SCRIPT_INVALID would be -1.
I think there are already lot of languages added: ~150 scripts. 1 script cover more than 1 language.
They won't add a huge amount of it.
Eventually, we could change the 'default' of the switch, so that if we don't recognize it as an TTF_Script, we just forward it to HB as  a raw value. Seem possible since TTF_Script doesn't seem to overwrite HB_TAG (4 chars)
so I wait, just ping me when we choose one solution.

Reposting my relevant thoughts from #175 here since this seems like where the discussion is happening:

To avoid the maintenance burden of all the TTF_SCRIPT_ constants, I think it's fine as an end-user  to just have the TTF_TAG macro that converts the standardized ISO 15924 4-character codes to SetScript's required input format. All the possible 4-letter codes for different script styles are already documented on unicode.org, the only issue I can think of is TTF's HarfBuzz not supporting a newly-added ISO script tag and having no way to check for an error if it isn't available (though the lack of error checking is a problem with the current release as well).

@sezero @slouken does that solution sound more workable?
Also @1bsyl I haven't actually done a visual test yet so it could be totally broken, but my unit test for TTF_SetDirection in PySDL2 seems to show top-to-bottom rendering actually working:
def test_TTF_SetDirection(font):
    # Define HarfBuzz direction constants
    HB_DIRECTION_LTR = 4
    HB_DIRECTION_TTB = 6

    # Try setting the script directiony
    ret = sdlttf.TTF_SetDirection(HB_DIRECTION_LTR)
    assert ret == 0

    # Try changing the script direction to see if it has any effect
    w1, h1, w2, h2 = c_int(0), c_int(0), c_int(0), c_int(0)
    sdlttf.TTF_SizeText(font, b"Hi there!", byref(w1), byref(h1))
    ret = sdlttf.TTF_SetDirection(HB_DIRECTION_TTB)
    assert ret == 0
    sdlttf.TTF_SizeText(font, b"Hi there!", byref(w2), byref(h2))
    sdlttf.TTF_SetDirection(HB_DIRECTION_LTR) # Reset direction
    assert w1.value > w2.value
    assert h1.value < h2.value

Assuming it actually does what it's supposed to, is there any reason not to allow people to pass through those HarfBuzz values even if they're not officially supported?

Vertical TTB and BTT aren't working currently.
but maybe it can be done ... needs to change horiAdvance, vertAdvance  and update the y position with it ...

@a-hurst now this should be working to draw in vertical direction: https://github.com/libsdl-org/SDL_ttf/commit/119abf2e8958f7e73e67c3d0534115ad3eb6e167

@slouken @sezero
maybe that the function
extern DECLSPEC int SDLCALL TTF_SetDirection(int direction); /* hb_direction_t */
extern DECLSPEC int SDLCALL TTF_SetScript(int script); /* hb_script_t */

Should take the TTF_Font *font parameter in ? but that would change the API ...

@slouken @sezero maybe that the function
extern DECLSPEC int SDLCALL TTF_SetDirection(int direction); /* hb_direction_t */
extern DECLSPEC int SDLCALL TTF_SetScript(int script); /* hb_script_t */

Should take the TTF_Font *font parameter in ? but that would change the API ...

Personally, I don't like changing the api, therefore I'd suggest adding a comment to the existing
functions like These functions set/change the direction and script globally within SDL_ttf
and adding new functions like TTF_FontSetDirection and TTF_FontSetScript, instead.
As for script and direction enums, I'd suggest an easy and ugly way like adding a note to
the top of our header -- something like:
diff --git a/SDL_ttf.h b/SDL_ttf.h
index 468359f..1122334 100644
--- a/SDL_ttf.h
+++ b/SDL_ttf.h
@@ -29,8 +29,14 @@
    confusion. */

+/* Note: In order to use direction and script for text shaping, define
+   SDL_TTF_HAVE_HARFBUZZ in your program before including SDL_ttf.h  */
+
 #ifndef SDL_TTF_H_
 #define SDL_TTF_H_

 #include "SDL.h"
+#ifdef SDL_TTF_HAVE_HARFBUZZ
+#include <hb.h>
+#endif
 #include "begin_code.h"

@sezero I've added TTF_FontSetDirection() and TTF_FontSetScript()   (FontSet* vs SetFont to match current naming).  https://github.com/libsdl-org/SDL_ttf/commit/1b2f2d7b142ac090519b03a8213977e091f724dc
I agree to generally not to change the API. Especially if the api is older, with lot of dependencies. But for a very recent API, I think it should not be so strict. Even biggest project have the notion of 'deprecated' api.
About the script and direction enums. It's not about simply writing "#include <hb.h>" to have the hb type.
but configuring a project (android, cmake, linux, visual, xcode, etc..) and get harfbuzz header included for it.
Not big deal, but more convenient if one has not to do so.

@sezero I've added TTF_FontSetDirection() and TTF_FontSetScript() (FontSet* vs SetFont to match current naming). 1b2f2d7

Draw_Line might need adjusting? It currently uses the global state instead of the per-font direction.

Instead of using int, you can use strings and use the relevant _from_string() API in HarfBuzz.

@sezero thanks fixed https://github.com/libsdl-org/SDL_ttf/commit/9a2cb0e452a52045419c3554e4c6696a3cd0a714, your reviewing is better than my test-cases
@khaledhosny thanks ! indeed, that may the best solution

After thinking about it some more, we shouldn't add a script enumeration to our header, instead we should add a function that sets the script by name. People who want to use the enumeration can include the harfbuzz header themselves, and people who want to look it up by name can do so:
extern DECLSPEC int SDLCALL TTF_SetFontScriptName(TTF_Font *font, const char *script);

@slouken
I have reverted not to change the old deprecated API (TTF_SetFont, TTF_SetScript).
Exposed also vertical direction TopToBottom and BottomToTop.
Change to TTF_SetFontScriptName
Doing some test seems that's ok.
if you merge, please 'squash' so we have only 1 commit

It looks like this patch set causes conflicts right now. Can you rebase and force push a single commit with your changes?

@slouken Not sure but it says "This branch has no conflicts with the base branch"

@slouken Not sure but it says "This branch has no conflicts with the base branch" when selecting "squash and merge"

But not when wants to rebase and merge:

That usually seems to happen when one merges master into the branch once or multiple times

Merged!


Download mdtoc directly

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Download the binary directly and add a toc to the README.md.
Which issue(s) this PR fixes:

None
Does this PR have test?
None

Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

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

Project coverage is 42.06%. Comparing base (11d77f4) to head (21f7fd0).
Report is 346 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2354      +/-   ##
==========================================
- Coverage   45.50%   42.06%   -3.44%
==========================================
  Files          79      109      +30
  Lines        7782    15863    +8081
==========================================
+ Hits         3541     6673    +3132
- Misses       4099     8696    +4597
- Partials      142      494     +352


[Core] New Wallet feature

Hello, i have implement the new Wallet feature that we have discussed on the #3202 .  I have implement it this way because i don't want to have many compatibility issues at the beginning. Until more people find it useful and start to implement it on other exchanges it is best to not break the previous implementation. Please check it and tell me if you like it.

I have implement the solution with your new insight and i really like it. It is MUCH better. I will make a PR shortly.

With this new Builder pattern you can easily add whatever feature you want in the Wallet.class and i believe it would be much simpler to extend new functionality in the future if needed.

Looking good. I think this warrants a version bump to 4.5.0.

Thanks for the reply @timmolter.

Can we merge this?

I'll bump the version.


Won't be able to pinch/rotate the map For MapView.Animated

Hi, All... I'm facing some issues in MapView.Animated, I am unable to zoom in or zoom out for the maps. If anyone gets the solution or idea how to fix this. please let me know. I'm using same code given in AnimatedViews.js

I fixed this by changing paddingTop or marginTop in AnimatedView.js styles, hope this fixes it!


Azure Table Names

If you follow the example to create a Knowledge Store https://docs.microsoft.com/en-us/azure/search/knowledge-store-create-rest then the Azure Table Names are hotelReviewsDocument, hotelReviewsKeyPhrases and hotelReviewsPages.

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

ID: bb77e148-a7e4-7d5f-4243-e39154647f4b
Version Independent ID: de828e74-0172-9655-4bac-fecee40acebe
Content: Connect to a knowledge store (preview) with Power BI - Azure Cognitive Search
Content Source: articles/search/knowledge-store-connect-power-bi.md
Service: cognitive-search
GitHub Login: @LisaLeib
Microsoft Alias: v-lilei

Hi @buildbod, was there an issue with the table names in the docs? Trying to understand your comment. Thanks!

Hi @Grace-MacJones-MSFT the table names on the page are:
hotelReviewsSsDocument, hotelReviewsSsKeyPhrases, and hotelReviewsSsPages
yet the tables created in the example https://docs.microsoft.com/en-us/azure/search/knowledge-store-create-rest are:
hotelReviewsDocument, hotelReviewsPages, hotelReviewsKeyPhrases, hotelReviewsSentiment, hotelReviewsInlineDocument, hotelReviewsInlinePages and hotelReviewsInlineKeyPhrases
Step 6 of the related article https://docs.microsoft.com/en-us/azure/search/knowledge-store-connect-power-bi uses table names:
hotelReviewsSsDocument, hotelReviewsSsKeyPhrases, and hotelReviewsSsPages
The variation in the table names make the examples hard to follow. Also the Power BI template https://github.com/Azure-Samples/cognitive-search-templates does not work as it is expecting table names that do not match those created by https://docs.microsoft.com/en-us/azure/search/knowledge-store-create-rest .

Thank you @buildbod for the detailed feedback. Upon further review we feel your feedback would be best received by the doc author to determine if an update to the current doc is necessary, or to create a new doc if they align with the feedback.

@buildbod   The Power BI template is only useful for Azure tables created by the wizard because the wizard feeds in projection information that describes the shape of the data.  I'll make sure that the docs do a better job of saying that.
The REST walkthrough needs to use the manually created Power BI report. I'll make that point is clear also.
I'll leave this issue open for now while I work through these doc items.

@buildbod  The docs have been updated to clarify the instructions for using Power BI with the other walkthroughs.  I'm closing this issue but feel free to re-open if you have more feedback about this issue or how it was addressed.  Thanks!
#please-close


Add Elo bins

Coverage remained the same at 98.684% when pulling 822256878ef41cabc3c50a526c2a23590e60e93b on guilhermearaujo:elo into 5ab238f24be6f0ff59c6faaadab28a7a6555061c on hugolantaume:master.

@hugolantaume can you release a new version?

Released v0.0.3


ESA 2018 5-hr Workshop

pre existing content
5 hrs
Next steps:
[ ] AB to provide links to content in April 2018
[ ] ADC team can help promote
[ ] emphasize this is an entry level training for data management

@kameyer Can you add this to our training page?

Wksp information not yet online so copying some details below. We do not need to create an arcticdata.io page for this one, but rather link to the ESA page when open since that is how people will register. In the interim, we can put up a summary and indicate "full details coming soon" or some such.
WK 5: "Managing Ecological Data for Effective Use and Re-Use: A Workshop for Early Career Scientists"
Sunday, August 5, 2018
8:00 AM - 11:30 AM
Abstract: While graduate students in ecology learn about methods for collecting ecological data, there is less emphasis on managing and using the resulting data effectively. This is an increasingly important skill set; many funding agencies require data management plans, journals are requiring that data and resulting software be accessible, and society is increasingly expecting that research be reproducible. Ecologists with good data skills will be able to maximize the productivity of their research program, effectively and efficiently share their data and other research products with the scientific community, and potentially benefit from the re-use of their data by others.
The purpose of this workshop is to give attendees a set of practical tools for organizing and sharing their data through all parts of the research cycle. The target audience is early-career scientists but is open to any researchers who would benefit from developing better data management skills, including faculty who would like to teach best practices for preparing data. Topics will include data structure, quality control, data documentation, the importance of good data management practices for data sharing, collaboration, and data re-use. The workshop will be an interactive combination of presentation, discussion and activities. Participants must bring their own laptop to work on exercises. A related Inspire Session is planned and a Special Session on “Creating Effective Data Management Plans for Ecological Research” may also be useful to participants.
Speakers: Amber Budden, Matt Jones, Vanessa Raymond, Bill Michener, Stephanie Hampton, Dave Vieglais

Added to the training page


OS X null pointer crash on font missing family attribute

Thanks for this and for fontkit, both are great libraries!
One of our designers ran into a crash on OS X that we tracked down to a font that was missing a "family" attribute.
Specifically, this line, where CTFontDescriptorCopyAttribute returns null, which eventually blows up when the resulting FontDescriptor gets coerced to JavaScript world.
Options for a fix seem to be:

Checking the string attributes for null and falling back to a default (e.g., "No Family Detected"), or

Ignoring fonts that have missing attributes. The approach I'd take on this is to allow createFontDescriptor to return null if the font is busted, and explicitly handle this null in the places where it's called (getAvailableFonts, findFonts, and substituteFont).

The reasoning for the second option is that a font that's missing a family is probably broken in other ways.
(Suitcase Fusion 6 describes the font that crashed font-manager as "corrupted".)
Happy to submit an OS X patch for whatever resolution you prefer.
(Would also be happy if you want to just code it up yourself.)
Thanks again for the great libraries!

@lynaghk You wouldn't happen to have the font that caused your crashes hanging around, would you?

I do not, sorry. The font was from a client and I didn't save a copy after we resolved the issue on our own fork of this library. For what it's worth, the font was frutiger65_bold.ttf, though I have no idea where it was purchased/downloaded, etc.

@fahrio Check out https://www.npmjs.com/package/font-scanner. It's a fork of this package with a few fixes we discovered when deploying this library in GitKraken.

Should be fixed by #35.
@implausible if there are other fixes you guys made in your fork, we'd love for you to contribute them back!

@implausible also, if you'd like to be a maintainer of font-manager itself, I'd add you to the repo.

@devongovett I'd definitely take you up on that. We've gotta fix the Electron issue https://github.com/foliojs/font-manager/issues/42


Support for Azure Time Series Insights

Is there any plans to support azure's native timeseries insights DB
https://azure.microsoft.com/en-gb/services/time-series-insights/
Since REST API for this available, this should be possible.
https://docs.microsoft.com/en-us/rest/api/time-series-insights/preview-query

interested in TSI (V2) support.

Really curious about this too. TSI is looking promising.

Thumbs up for this!

Has this progressed at all?

Interested in this as well. Would be really nice to be able to use this data source.


set final_sparsity=0.45, after 50 epochs, the sparsity of each layer is about 0.39

hi, everyone, when i used sparseml to sparse my model, i got a problem.
after finished training based on a pretrained checkpoint, the sparsity of each layer is about 0.39, much less than the final_sparsity 0.45.
my recipe is as follow:
version: 1.1.0
modifiers:

!EpochRangeModifier
start_epoch: 0.0
end_epoch: 50.0

!GMPruningModifier
params:
ALL_PRUNABLE
init_sparsity: 0.05
final_sparsity: 0.45
start_epoch: 1.0
end_epoch: 50.0
update_frequency: 0.5
global_sparsity: False
inter_func: cubic
leave_enabled: True
mask_type: unstructured

my training strategy is as follow:
optimizer:
name: AdamW
lr: 0.0001
weight_decay: 0.05
lr_schedule:
name: CosineAnnealingLR
T_max: 50
eta_min: 0.00005
Has anyone had a similar problem? thanks

Hi @yueyunqingwu what training script are you using? Unexpected final sparsity can be a result of steps per epoch being out of sync with the actual number of update steps that occur.

@yueyunqingwu
Since some time has passed, we'll go ahead and close this issue. But if you're able to supply the requested info, please do! We're happy to assist. Thank you.
Jeannie / Neural Magic


Could pixi-display cause event propagation issues?

I'm hunting an event propagation issue, even though I don't .stopPropagation, the event doesn't bubble under an entity
I've created a small test case, however, in the test case, event bubbles as expected
I'm just wondering whether the way pixi-display shuffles entities could cause the event issue I'm experiencing

Yes, it has its own interaction cycle. Please look inside and tell me if there's something wrong in it , compared to original pixi code.

Sorry, I only just managed to replicate the issue, replied on https://github.com/pixijs/pixi.js/issues/2921#issuecomment-245039445
If you want we can move the discussion here


feat: replicate support

closes #242
/claim #242

Thanks so much for this @Kartik1397! Wrote a couple of comments. I'd also suggest actually testing this against app.traceloop.com because I'm not sure it will work as expected currently. You can build a new simple sample app for that.

Thanks, @nirga for reviewing the PR! I've incorporated your feedback in my latest commit. Please let me know if there's still anything missing or can be done in better way. This is my first time using OpenTelemetry SDK, I may have overlooked some straightforward aspects. Your guidance is greatly appreciated!
Testing on api.traceloop.com
https://github.com/traceloop/openllmetry/assets/24036721/1df68b94-5fb8-4613-a38a-21d08f82b566

Perfect! I see that poetry install is failing because of a problem with the lock file (see checks below), other than that if the CI works afterwards we're good to go.
Thanks! 🙏

Perfect! Can you just add the sample app you used?

Should I add it to https://github.com/traceloop/openllmetry/tree/main/packages/sample-app/sample_app?

Perfect! Can you just add the sample app you used?

Should I add it to https://github.com/traceloop/openllmetry/tree/main/packages/sample-app/sample_app?

Yes!

Done @nirga. Also reduced minimum python version required for  opentelemetry-instrumentation-replicate to 3.8 as replicate support python 3.8. https://pypi.org/project/replicate/

@Kartik1397 when installing the SDK there's a warning
Warning: poetry.lock is not consistent with pyproject.toml. You may be getting improper dependencies. Run poetry lock [--no-update] to fix it. (see checks logs)

@Kartik1397 when installing the SDK there's a warning
Warning: poetry.lock is not consistent with pyproject.toml. You may be getting improper dependencies. Run poetry lock [--no-update] to fix it. (see checks logs).

This should be fixed now.

Also, I'm not sure your code will work for streaming - have you tested it?

I can see traces of stream function calls are getting consumed by traceloop(with propmpts). Do we need to capture response in case of streaming API?

{
    "name": "replicate.stream",
    "context": {
        "trace_id": "0x0e03350bb8f375674076a774eb4d964f",
        "span_id": "0x84224c4e5915e5b4",
        "trace_state": "[]"
    },
    "kind": "SpanKind.CLIENT",
    "parent_id": "0x21c7475439b11a24",
    "start_time": "2023-12-18T14:36:54.448065Z",
    "end_time": "2023-12-18T14:36:54.448092Z",
    "status": {
        "status_code": "OK"
    },
    "attributes": {
        "llm.vendor": "Replicate",
        "llm.request.type": "completion",
        "traceloop.workflow.name": "chat_stream_generator",
        "llm.request.model": "meta/llama-2-70b-chat:02e509c789964a7ea8736978a43525956ef40397be9033abf9fd2badfe68c9e3",
        "llm.prompts.0.user": "tiny robot"
    },
    "events": [],
    "links": [],
    "resource": {
        "attributes": {
            "service.name": "sample_app/replicate_streaming.py"
        },
        "schema_url": ""
    }
}

@Kartik1397 when installing the SDK there's a warning
Warning: poetry.lock is not consistent with pyproject.toml. You may be getting improper dependencies. Run poetry lock [--no-update] to fix it. (see checks logs).

This should be fixed now.

Also, I'm not sure your code will work for streaming - have you tested it?

I can see traces of stream function calls are getting consumed by traceloop(with propmpts). Do we need to capture response in case of streaming API?

```
{
    "name": "replicate.stream",
    "context": {
        "trace_id": "0x0e03350bb8f375674076a774eb4d964f",
        "span_id": "0x84224c4e5915e5b4",
        "trace_state": "[]"
    },
    "kind": "SpanKind.CLIENT",
    "parent_id": "0x21c7475439b11a24",
    "start_time": "2023-12-18T14:36:54.448065Z",
    "end_time": "2023-12-18T14:36:54.448092Z",
    "status": {
        "status_code": "OK"
    },
    "attributes": {
        "llm.vendor": "Replicate",
        "llm.request.type": "completion",
        "traceloop.workflow.name": "chat_stream_generator",
        "llm.request.model": "meta/llama-2-70b-chat:02e509c789964a7ea8736978a43525956ef40397be9033abf9fd2badfe68c9e3",
        "llm.prompts.0.user": "tiny robot"
    },
    "events": [],
    "links": [],
    "resource": {
        "attributes": {
            "service.name": "sample_app/replicate_streaming.py"
        },
        "schema_url": ""
    }
}
```

Yes, of course :) Same as we do in OpenAI.

Okay, will add support for streaming.

Pushed changes to capture response streams.

Failed as Github doesn't recognize newly added Replicate API token. Tested locally.
Merging.


ci: add clean cache and pull request title validation workflow

Description

This PR adds

clean cache workflow. https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#force-deleting-cache-entries.
Validate PR title workflow

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
[ ] Ideally, include a test that fails without this PR but passes with it.
[ ] Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

[x] Run the tests with pnpm test:ci.

Documentation

[ ] If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

[x] Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.

The team decided that this is not needed. The cache is already in place and we control the name of the PR when merging


do not undef =~ unless defined

Fixes ruby 3.2 support. Similar to https://github.com/pry/pry/pull/2149

Closing in favor of https://gitlab.com/puppet-debugger/puppet-debugger/-/merge_requests/70


Added requirements.txt in repository root

The docs/requirements.txt, .travis.yml, and readme files were updated to reference this file, so there aren't dependencies specified in multiple places.
@jmathai I'm not sure how you want to handle version pinning. I just pinned to >= the most recent version and < the next major version (hoping for people following semantic versioning). Please let me know if you'd like me to change that.

Moved mock and nose into elodie/tests/requirements.txt.


Commit message truncated after '#' (hash) symbol

If the commit message contains a hash, then the commit message is truncated after the hash (as if the hash was interpreted as a bash comment).
I noticed this misbehavior because I was not seeing my commits and issues being cross-referenced in Github. If this can help circumscribing the time frame of the issue, here's a screenshot of a recent commit history (screenshot taken around July 1 2016 09:40 GMT):

Sorry, nevermind. I was using 5.16.0. It's working great on 5.16.1.
:smile:


[Bug]: WriteToBigQuery with file_loads and dynamic table destination doesn't load after first File Load

What happened?
I was able to reproduce a bug in the WriteToBigQuery function in the apache beam source code for Dataflow Streaming and the File Loads setting.  This issue is primarily caused by the impulse nodes feeding into the WaitForBQJobs task used in four separate nodes in the pipeline.
This bug manifests when using file loads to write data to different tables, set via a lambda reading the row inputs. The issue occurs after the first file load, where the impulses (which should either be periodic, or through a different method) no longer fire again, resulting in the side inputs into the nodes that load to a temp table, check if the load is complete, load a schema change to the intended table, check if that change is complete, copy from the temp table to the intended table, check that is complete, then remove the original temp table.
The use of the single impulse makes sure that this set of work gets executed once, but after it is executed once, and more data comes in, none of that data gets actioned.
I reported this issue directly to the Customer Support at the Dataflow team at GCP, but after not seeing it being actioned, I'm reporting it here.
Here is the Google Doc I created for them that walks through reproduction of the bug.
This has been reproduced on both Apache beam 2.32 and 2.40, running on dataflow.
Issue Priority
Priority: 2
Issue Component
Component: io-py-gcp

Thanks @Abacn
Thank you @rizenfrmtheashes for that document, it was really helpful in understanding the underlying issues. I'm working on a solution in #23012 (currently just writing tests and changing other tests that relied on the previous implementation).

.remove-labels 'awaiting triage'

@ahmedabu98 Oh I absolutely missed this.  I'm glad it's getting actioned. I took a look at the PR and it looks good at addressing the issue with the WaitForBQJobs single impulse, as well as removing the single impulse dependency on the tasks following. Looking forward to seeing its inclusion in the next release!

Thanks for taking a look! I'm looking to get it merged before the end of the week, though I'm not sure if it will make version 2.42.0 as that release branch has already been cut.


Fix uber-merge-using-staging-tag to preserve revision description

Commit message, author and revision description was not preserved when --uber-merge-using-staging-tag flag was used.

@Pawka have you tested these changes on your end ?

@Pawka have you tested these changes on your end ?

In the morning sync with Povilas, he said yes. This is to verify in SQ. Once you do sanity check and land this change then i can help testing it whether addresses actual issue.