deployment on ubuntu server with nginx causes 504 timeout

I am running a ollama server which runs a llama2 on the cloud server with ubuntu.
curl -X POST http://localhost:11434/api/generate -d '{
>   "model": "llama2",
>   "prompt":"Why is the sky blue?"
> }'

the result is starting to stream in 3 seconds
{"model":"llama2","created_at":"2023-09-01T22:35:43.271181437Z","response":" The","done":false}
....
....
However, for the public visit there is a nginx server connecting to this ollama server, but everytime visiting the service in cloud IP address always causes 504 timeout.
the configuration of nginx server timeout doesn't work.
server {
        listen 9003;
        server_name xx.xx.xx.xx;

        location / {
                proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
                proxy_set_header   Host      $http_host;
                proxy_set_header X-Forwarded-Proto $scheme;

                proxy_read_timeout 150000;
                proxy_connect_timeout 150000;
                proxy_send_timeout 150000;
     }
}

usually timeout 150000 should be enough to solve the timeout issue since ollama inference is only within 3 seconds.
any ideas on the deployment

investigate more on 'event-stream',
 header:{
  'Content-Type': 'text/event-stream'
}

therefore
proxy_http_version 1.1;  version 1.1 supports long connection
       proxy_set_header Connection 'keep-alive';
       proxy_set_header Cache-Control 'no-cache';
       proxy_set_header Content-Type 'text/event-stream';

proxy_buffering off;  disable the buffering to make sure the real time connection


Update requirements.md

Description
We test 3.16 against k8s 1.19, so we should say that in the docs.

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit 983082fb61fc15d2a1d76a58f38630f1fc4421e5
https://deploy-preview-4172--calico.netlify.app


Release 2.3

[x] Bump version number
[x] Update ChangeLog
[x] Update devDependencies (JSCS, Istanbul, ESLint, ...)
[x] Check for performance regression
[x] Test some important downstream projects
[x] Tag the repository
[x] Publish to npm
[x] Update web site

Performance check using V8 4.3:
2.2:
    Underscore 1.5.2    42.5 KiB      6.79 ms ± 3.45%
      Backbone 1.1.0    58.7 KiB      5.96 ms ± 0.61%
      MooTools 1.4.5   156.7 KiB     30.29 ms ± 1.05%
        jQuery 1.9.1   262.1 KiB     31.04 ms ± 1.10%
          YUI 3.12.0   330.4 KiB     29.57 ms ± 2.87%
 jQuery.Mobile 1.4.2   442.2 KiB    126.83 ms ± 8.16%
       Angular 1.2.5   701.7 KiB     86.80 ms ± 6.85%
                     ------------------------
                      1994.3 KiB    317.29 ms ± 6.38%

2.3:
    Underscore 1.5.2    42.5 KiB      6.82 ms ± 3.26%
      Backbone 1.1.0    58.7 KiB      6.00 ms ± 0.69%
      MooTools 1.4.5   156.7 KiB     30.90 ms ± 0.87%
        jQuery 1.9.1   262.1 KiB     31.98 ms ± 1.39%
          YUI 3.12.0   330.4 KiB     28.37 ms ± 0.76%
 jQuery.Mobile 1.4.2   442.2 KiB    123.92 ms ± 7.33%
       Angular 1.2.5   701.7 KiB     89.50 ms ± 6.65%
                     ------------------------
                      1994.3 KiB    317.48 ms ± 5.83%

No functional regression was observed on the following downstream projects:

Istanbul (current master)
JSCS (current master)
escomplex-js (current master)

This is completed, 2.3.0 is now available.


Up to date ocm plugin

Signed-off-by: Takumi Yanagawa<EMAIL_ADDRESS>What's changes

Make C2P for OCM and C2P for Kyverno consistent in input, output, directory structure, and documentations

I found the reason why merge is blocked. I have added my signature to the commit but I haven't signed it. Now I sign all commits and the merge turns into green.


Internal Server Error when fetching Go track

When I try to fetch the Go track I get an internal server error. I can fetch other tracks and friends of mine can fetch the Go track. Command output:
~/exercism ᐳ exercism fetch go
2015/07/06 12:33:46 an internal server error was received.
Please file a bug report with the contents of 'exercism debug' at: https://github.com/exercism/x-api/issues
~/exercism 1 ᐳ exercism debug

**** Debug Information ****
Exercism CLI Version: 2.2.0
Exercism CLI Latest Release: 2.2.0
OS/Architecture: darwin/amd64
Build OS/Architecture darwin/amd64
Home Dir: /Users/jwalker
Config file: /Users/jwalker/.exercism.json
API Key: XXXXXXXXXXXXXXXXXXXX
API: http://exercism.io [connected]
XAPI: http://x.exercism.io [connected]
Exercises Directory: /Users/jwalker/exercism

Other (potentially misleading) information: before this came up I had just submitted the secret-handshake problem. Looking at the Go track config it looks like I somehow skipped the error-handling problem. Maybe this is relevant?

Oooh. The error handling problem is new as of yesterday. I'm in a meeting but will look into this as soon as I'm out.

OK, it turns out that the API is not dealing with the binary data in the file. Up until now all the files have been utf-8 encoded, and it hasn't been necessary to handle anything more complex. We're going to need to base64 encode the files, I think; but this will require a change to the CLI as well.
For now I've simply redeployed the API without the error-handling problem.
I'm leaving this issue open until I can create a new issue in the api repo to deal with it (I'm traveling this week).

I put in a workaround for this so that the go track is fetchable. See https://github.com/exercism/x-api/issues/64


[RELAY][BACKEND] Enable PlanMemory in the graph runtime.

This PR implements PlanMemory for graph runtime codegen backend of Relay. It also contains a few other improvements

[x] add show_meta_data option to astext

Can be helpful to display text when there is a huge meta-data

The algorithm is basically the same from NNVM. We do need to introduce a storage token and have an initialization phase that propagates and calculate expected reference count of the token, before we run the greedy allocation algorithm.

@tqchen quick question - are there plans to allow dynamic memory allocation in the graph runtime, which would allow variable shapes? I believe that's not currently supported, but was curious if you had plans there.

@ajtulloch yes, I think the plan is to write a new runtime system soon ™. A few of us are working on a PLDI submission, and expect to ship a bunch of improvements/fixes/features post deadline.

Overall looks good to me, a bit tired from the PLDI push so maybe someone else should do a pass.

@tqchen Quite busy recently. But I will try my best to spend sometime to do a round tonight if it is not too late.

When we start to move into Relay we have this clear separation of compiler and runtime. The migration starts as two-phase process, and we are in the first step that moves the compiler, but keeps the old graph runtime.
I think we can expect the static graph runtime to exist for a while, but we can also explore the possibility of new backends that breaks different assumptions(e.g. dynamic memory alloca, control flow). Luckily the IR is expressive enough to represent all these workloads.
There is also a tradeoff here, depending on whether we want to allow JIT, how big the runtime is, etc. So I can imagine it could be possible that we build several of them. @ajtulloch I think it is a good time to hear opinions from everyone on what do we need

@tqchen that sounds great. Is there an existing RFC you'd like us to contribute feature suggestions to?

there is not an existing RFC, how about we open a new one?

opened in #2122

@ajtulloch RFC seems like a great idea, would look forward to figuring out what everyone is interested in, and what people are looking to do.

@tqchen I only have some nit comments. Overall LGTM.

Thanks, @ajtulloch @yzhliu @zhiics, i have addressed the comments.

Thanks @ajtulloch @yzhliu @zhiics this is merged


Add conda support [Help Wanted]

@spiritofpigeon , i don't think you can install this library with conda, conda install is available for lifelines though,
do pip install lifetimes in your conda environment, hope that helps,

In the next release, I'll set up conda

I don't use conda, so I'm uncomfortable doing this. If someone else would like to, I would appreciate that very much!

I don't know if this is what you mean, but isn't it possible to install it on top of the Python version being used by Anaconda by using the --user option? Something like pip install --user lifetimes.

Hi @CamDavidsonPilon , I'm trying to add the publish of lifetimes to conda-forge channel. Let me know if you are okay with this. https://github.com/conda-forge/staged-recipes/pull/10871

okay with me :)

Could be helpful to merge #359 before the next release. I can change the conda package recipe to include correct LICENSE by then. Not blocking though


No connection/execution with the raspberry

I am trying to executing the example: pico_micro_ros_example.c following this guide
The building seems ok: it returns from visual studio code this:

[proc] Executing command: /usr/local/bin/cmake --build /home/kappa95/micro_ros_ws/src/micro_ros_raspberrypi_pico_sdk/build --config Debug --target all -j 10 --
[build] [  1%] Performing build step for 'ELF2UF2Build'
[build] [  3%] Built target bs2_default
[build] [  6%] Built target bs2_default_padded_checksummed_asm
[build] [100%] Built target elf2uf2
[build] [  7%] No install step for 'ELF2UF2Build'
[build] [  8%] Completed 'ELF2UF2Build'
[build] [ 15%] Built target ELF2UF2Build
[build] Consolidate compiler generated dependencies of target pico_micro_ros_example
[build] [100%] Built target pico_micro_ros_example
[build] Build finished with exit code 0

I generate the uf2 file and I have uploaded it as in the guide.
I have followed all the next steps of the guide and after:
micro-ros-agent serial --dev /dev/ttyACM0 baudrate=115200
I get:
/bin/bash: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8)
[1619029391.389393] info     | TermiosAgentLinux.cpp | init                     | running...             | fd: 3

The raspberry pico doesn't blink and I don't find the topics and nodes. i.e. :
$ ros2 node list
//

I am using ROS2 Dashing with Ubuntu 18.04.5 LTS.
Update:
I don't know if it could be useful for solving the issue, but I checked the return value from rmw_uros_set_custom_transport() function using a simple if condition for blinking the led if the return value is false. And it shows that it is false.
    bool goofy = rmw_uros_set_custom_transport(
        true,
        NULL,
        pico_serial_transport_open,
        pico_serial_transport_close,
        pico_serial_transport_write,
        pico_serial_transport_read);

    gpio_init(LED_PIN);
    gpio_set_dir(LED_PIN, GPIO_OUT);

    if (!goofy)
    {
        for (size_t i = 0; i < 20; i++)
        {
            gpio_put(LED_PIN, 1);
            sleep_ms(250);
            gpio_put(LED_PIN, 0);
            sleep_ms(250);
        }
    }

Hello @kappa95, I have replicated the issue and it is related to connection timing.
The function rclc_support_init tries to connect with the agent with a timeout of 10 seconds, if the agent is not reachable in that time the scripts fails, not even reaching the LED activation. You can check this with the return value of the function, which returns false (RCL_RET_OK) on a successful connection.
I have modified the example to solve this problem (PR #29), now it will wait for a reachable agent for up to 2 minutes using the function rmw_uros_ping_agent. This should give enough time to start the agent and establish the connection.
Please give it a try and let us know the results

Hello @kappa95, I have replicated the issue and it is related to connection timing.
The function rclc_support_init tries to connect with the agent with a timeout of 10 seconds, if the agent is not reachable in that time the scripts fails, not even reaching the LED activation. You can check this with the return value of the function, which returns false (RCL_RET_OK) on a successful connection.
I have modified the example to solve this problem (PR #29), now it will wait for a reachable agent for up to 2 minutes using the function rmw_uros_ping_agent. This should give enough time to start the agent and establish the connection.
Please give it a try and let us know the results

I pull the repo, I have seen the changes in the code. But I get the same issue

Could you please try with the dockerized foxy agent?
docker run -it -v /dev:/dev --privileged  microros/micro-ros-agent:foxy serial --dev /dev/ttyACM0 -v6

Could you please try with the dockerized foxy agent?
docker run -it -v /dev:/dev --privileged microros/micro-ros-agent:foxy serial --dev /dev/ttyACM0 -v6

Same results:
docker run -it -v /dev:/dev --privileged microros/micro-ros-agent:foxy serial --dev /dev/ttyACM0 -v6
[1619164483.342955] info     | TermiosAgentLinux.cpp | init                     | running...             | fd: 4
[1619164483.343359] info     | Root.cpp           | set_verbose_level        | logger setup           | verbose_level: 6

Second terminal:
ros2 node list
//

Could you check the serial port output without the agent, using miniterm or similar?
Disconnect and reconnect the board, then for the first 2 minutes there should appear a periodic message each second, like this:
sudo miniterm /dev/ttyACM0 115200
--- Miniterm on /dev/ttyACM0  115200,8,N,1 ---
--- Quit: Ctrl+] | Menu: Ctrl+T | Help: Ctrl+T followed by Ctrl+H ---
~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀
��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀
��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀
��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀
��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀
��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀
��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀

If nothing appears, check that your raspberry pico is actually on the /dev/ttyACM0 port, by disconnecting it and checking that it is not on the /dev/tty list using ls /dev/ttyACM*.

Could you check the serial port output without the agent, using miniterm or similar?
Disconnect and reconnect the board, then for the first 2 minutes there should appear a periodic message each second, like this:
sudo miniterm /dev/ttyACM0 115200
--- Miniterm on /dev/ttyACM0  115200,8,N,1 ---
--- Quit: Ctrl+] | Menu: Ctrl+T | Help: Ctrl+T followed by Ctrl+H ---
~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀
��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀
��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀
��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀
��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀
��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀
��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀	��␂␀␀␀�␒~␀␀␐␀�␀␀␀␂␀␀

If nothing appears, check that your raspberry pico is actually on the /dev/ttyACM0 port, by disconnecting it and checking that it is not on the /dev/tty list using ls /dev/ttyACM*.

With miniterm, nothing appear: no messages.
And the only device is the raspberry:
ls /dev/ttyACM*
/dev/ttyACM0

I tried the hello world program from the raspberry site and using minicom worked.

After some investigation, the problem relies on the arm-none-eabi-gcc installed version: Ubuntu 18.04 uses 6.3.1, while the micro-ros library is compiled using 9.2.1. To check your actual version: arm-none-eabi-gcc --version
This commands will download a compatible version from source and set the cmake compiler path.
wget --directory-prefix=$HOME https://developer.arm.com/-/media/Files/downloads/gnu-rm/9-2020q2/gcc-arm-none-eabi-9-2020-q2-update-x86_64-linux.tar.bz2
tar -xvf $HOME/gcc-arm-none-eabi-9-2020-q2-update-x86_64-linux.tar.bz2 -C $HOME
echo "export PICO_TOOLCHAIN_PATH=$HOME/gcc-arm-none-eabi-9-2020-q2-update" >> ~/.bashrc
source ~/.bashrc

Anyway, check the version output on the cmake command is at least 9.x.x:
-- Defaulting build type to 'Release' since not specified.
PICO compiler is pico_arm_gcc
PICO_GCC_TRIPLE defaulted to arm-none-eabi
-- The C compiler identification is GNU 9.3.1
-- The CXX compiler identification is GNU 9.3.1
-- The ASM compiler identification is GNU

After some investigation, the problem relies on the arm-none-eabi-gcc installed version: Ubuntu 18.04 uses 6.3.1, while the micro-ros library is compiled using 9.2.1. To check your actual version: arm-none-eabi-gcc --version
This commands will download a compatible version from source and set the cmake compiler path.
wget --directory-prefix=$HOME https://developer.arm.com/-/media/Files/downloads/gnu-rm/9-2020q2/gcc-arm-none-eabi-9-2020-q2-update-x86_64-linux.tar.bz2
tar -xvf $HOME/gcc-arm-none-eabi-9-2020-q2-update-x86_64-linux.tar.bz2 -C $HOME
echo "export PICO_TOOLCHAIN_PATH=$HOME/gcc-arm-none-eabi-9-2020-q2-update" >> ~/.bashrc
source ~/.bashrc

Anyway, check the version output on the cmake command is at least 9.x.x:
-- Defaulting build type to 'Release' since not specified.
PICO compiler is pico_arm_gcc
PICO_GCC_TRIPLE defaulted to arm-none-eabi
-- The C compiler identification is GNU 9.3.1
-- The CXX compiler identification is GNU 9.3.1
-- The ASM compiler identification is GNU

Thank you so much! Something now is changed:
docker run -it -v /dev:/dev --privileged microros/micro-ros-agent:foxy serial --dev /dev/ttyACM0 -v6
[1619178075.701850] info     | TermiosAgentLinux.cpp | init                     | running...             | fd: 3
[1619178075.702175] info     | Root.cpp           | set_verbose_level        | logger setup           | verbose_level: 6
[1619178075.714481] info     | Root.cpp           | create_client            | create                 | client_key: 0x2B5411CA, session_id: 0x81
[1619178075.714548] info     | SessionManager.hpp | establish_session        | session established    | client_key: 0x726929866, address: 0
[1619178075.714607] debug    | SerialAgentLinux.cpp | send_message             | [** <<SER>> **]        | client_key: 0x2B5411CA, len: 19, data:
0000: 81 00 00 00 04 01 0B 00 00 00 58 52 43 45 01 00 01 0F 00
[1619178075.717420] debug    | SerialAgentLinux.cpp | recv_message             | [==>> SER <<==]        | client_key: 0x2B5411CA, len: 24, data:
0000: 80 00 00 00 00 01 10 00 58 52 43 45 01 00 01 0F 2B 54 11 CA 81 00 FC 01
[1619178075.717486] info     | SessionManager.hpp | establish_session        | session re-established | client_key: 0x726929866, address: 0
[1619178075.717542] debug    | SerialAgentLinux.cpp | send_message             | [** <<SER>> **]        | client_key: 0x2B5411CA, len: 19, data:
0000: 81 00 00 00 04 01 0B 00 00 00 58 52 43 45 01 00 01 0F 00
[1619178075.719486] debug    | SerialAgentLinux.cpp | recv_message             | [==>> SER <<==]        | client_key: 0x2B5411CA, len: 24, data:
0000: 80 00 00 00 00 01 10 00 58 52 43 45 01 00 01 0F 2B 54 11 CA 81 00 FC 01
[1619178075.719580] info     | SessionManager.hpp | establish_session        | session re-established | client_key: 0x726929866, address: 0
[1619178075.719652] debug    | SerialAgentLinux.cpp | send_message             | [** <<SER>> **]        | client_key: 0x2B5411CA, len: 19, data:
0000: 81 00 00 00 04 01 0B 00 00 00 58 52 43 45 01 00 01 0F 00
[1619178075.824008] debug    | SerialAgentLinux.cpp | recv_message             | [==>> SER <<==]        | client_key: 0x2B5411CA, len: 24, data:
0000: 80 00 00 00 00 01 10 00 58 52 43 45 01 00 01 0F 2B 54 11 CA 81 00 FC 01
[1619178075.824408] info     | SessionManager.hpp | establish_session        | session re-established | client_key: 0x726929866, address: 0
[1619178075.824503] debug    | SerialAgentLinux.cpp | send_message             | [** <<SER>> **]        | client_key: 0x2B5411CA, len: 19, data:
0000: 81 00 00 00 04 01 0B 00 00 00 58 52 43 45 01 00 01 0F 00

The Led on the Raspberry now is on
But still:
ros2 node list
//

Thank you for the feedback!

I uses Ubuntu 20.04 and ros2 foxy. Is it possible to run the example code correctly?


Unable to follow FAQ tutorial to allocate more memory for Mac

When reporting a bug please provide the following information to help reproduce the bug:
Version of OpenRefine used (Google Refine 2.6, OpenRefine2.8, an other distribution?):
OpenRefine2.8
Operating Systems and version:
macOS 10.12.3
Browser + version used - Please note that OpenRefine doesn't support Internet Explorer but works OK in most cases:
Chrome
Steps followed to create the issue:
Follow the FAQ to allocate more memory for Machttps://github.com/OpenRefine/OpenRefine/wiki/FAQ%3A-Allocate-More-Memory
If you are allowed and are OK with making your data public, it would be awesome if you can include the data causing the issue or a URL pointing to where the data is (if your concerned about keeping your data private, ping us on our mailing list):

Current Results:
there is not "java" section, no "VMOptions", no "${java.options}". See the screenshot obove.
Expected Results:
from the FAQ:
expand the "java" section at the bottom
double click on the "VMOptions" content (the right end side)
change "${java.options}" with "-Xms256M -Xmx1024M"

Thanks for this David!  Do you care to update our Wiki FAQ ?  I'll give you write access to fix this up and then let us know in this issue and we'll close it out.

@griii2 Actually, you should already have EDIT rights for our Wiki.  Look for the Edit button in the top right corner of any Wiki page.

Hi Thad, there is a misunderstanding, I have no idea how to allocate more memory - I was unable to follow the FAQ and I don't know what to do.

The tutorial is probably a bit outdated. Simply change “-Xmx1024M” into something else, eg “-Xmx2048M” or “-Xmx8G”.

@griii2 if you have further problems just let us know on our mailing list.


How to hide cd echo

My current directory is /path/to/foo, and then I ran commands cd /; cd -.
Now my current directory is still /path/to/foo, but I got  console echo "/path/to/foo", this is the directory path.
How can I hide it?

You just have to close STDOUT like this cd / ; cd - 1>&- .


Unit tests in src/proxy/http/remap/unit-tests not built/run by cmake

They do not appear in any CMakeLIsts.txt file.

Why is this open and done?

Doesn't look done.
wkaras ~/REPOS/TS
O$ find . -name test_PluginDso.cc
./src/proxy/http/remap/unit-tests/test_PluginDso.cc
wkaras ~/REPOS/TS
O$ grep -F test_PluginDso.cc $(find . -type f -name '*ake*' )
wkaras ~/REPOS/TS
O$


Consider using actions/cache

Is there a particular reason why actions/cache is not included in the workflows?
Besides vendor addressing the module cache in this particular case, persisting the build cache could optimize build times across the org (and dependent repositories) IMHO.
https://github.com/actions/cache/blob/main/examples.md#go---modules

I ran some (non-scientific) experiments with eventing and VSphereSources and could not see a huge performance gain from adding a cache.
From what I can tell this is due to:

Using vendor which absorbs a lot of the typical go mod download time (in a test repo without vendor I do see gains as expected)
Sharing the cache with different workflows (using same cache key for unit tests, lint, e2e, etc.) leads to a situation where apparently not all Go cached files are uploaded to the cache destination. Sometimes I only saw 5MB cache size compared to 1.2GB in a local run (for pkg mod and gobuild cache)

Haven't looked deeper into the latter, i.e. why the cache is not the similar size I am seeing after a workflow run in the local GOCACHE GOMODCACHE folders.
Closing this issue for now. Apparently, the community is also pushing for actions/setup-go to support caching without an extra step (see https://github.com/actions/setup-go/issues/130).


[Bug] Code validation (syntax error) does not work for php language

Reproducible in vscode.dev or in VS Code Desktop?

[ ] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[ ] Not reproducible in the monaco editor playground

Monaco Editor Playground Link
No response
Monaco Editor Playground Code
<?php
// The next line contains a syntax error:
if () {
  return "The parser recovers from this type of syntax error"
}
?>

Reproduction Steps
Write a syntax error, like missing condition in if statement or missing a ;
Actual (Problematic) Behavior
Code validation (syntax error) does not work for php language
It does not work in demo either (microsoft.github.io/monaco-editor)
If you select PHP language, the first comment is "the next line contains a syntax error". But the "next line" (which has a syntax error) is not highlighted
Expected Behavior
Code validation works
Additional Context
No response

@SteelManITA Monaco does not support validating PHP syntax.

@rcjsuen I don't understand... Why then on the page https://microsoft.github.io/monaco-editor is there actually an example in PHP that refers to validation?
In any case, is there currently no way to have even basic validation of PHP syntax?

@rcjsuen I don't understand... Why then on the page https://microsoft.github.io/monaco-editor is there actually an example in PHP that refers to validation?

Which example are you referring to?

In any case, is there currently no way to have even basic validation of PHP syntax?

I doubt it personally but I don't work for Microsoft. You'd probably need to hook it up with a language server.

@rcjsuen

@rcjsuen I refer to the example that appears when selecting php on the homepage (screnshot below)

I agree it's kind of a strange design choice but unfortunately the reality is that it does not have PHP syntax validation out-of-the-box.

Ok, thanks for clarifying. Do you know any packages I can use to achieve my
goal?
Il mar 17 dic 2024, 12:34 Remy Suen @.***> ha scritto:

@rcjsuen https://github.com/rcjsuen I refer to the example that appears
when selecting php on the homepage (screnshot below)
I agree it's kind of a strange design choice but unfortunately the reality
is that it does not have PHP syntax validation out-of-the-box.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/monaco-editor/issues/4781#issuecomment-2548220765,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEDUJDYGDMPJPYJ4RSINUZ32GAD5XAVCNFSM6AAAAABTP35N2GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBYGIZDANZWGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ok, thanks for clarifying. Do you know any packages I can use to achieve my goal?

You should read up on the language server protocol and then look at TypeFox/monaco-languageclient. They have examples that integrate with Java and Python language servers that you can reference.


Error in setup

Dear Chris
The following error is received during build:
root@objectstorage-01:~# cd seaweedfs/weed && make install go install -ldflags="-s -w" go: downloading github.com/aws/aws-sdk-go v1.51.30 go: downloading google.golang.org/api v0.176.0 go: downloading github.com/pierrec/lz4/v4 v4.1.18 go: downloading github.com/klauspost/compress v1.17.7 build github.com/seaweedfs/seaweedfs/weed: cannot load cmp: malformed module path "cmp": missing dot in first path element make: *** [Makefile:10: install] Error 1

Please check the golang version.
go version
go version go1.22.1 darwin/arm64

and run
go mod tidy

Thank you for your reply @kmlebedev
go version go version go1.13.8 linux/amd64
after running "go mod tidy":
        testing/slogtest: malformed module path "testing/slogtest": missing dot in first path element


Create Research section

With the research done earlier this year it would be great to include a research section and within that section:

Larger overview of all research done.
Separate pages outlining the findings and information from research that has been done

Is this still a draft?


Add ADR for per-namespace gateways.

I fixed the merge conflicts which means we're going to have to reapprove. 🙈

aaaand @Krenair pointed out that this isn't mergeable unless Daniel signs it. We can wait until tomorrow, or close it and re-open it instead.

Signed and force-pushed.


Remove Docker GPG key before dearmor-ing it

What this PR does / why we need it:
#3482 introduced a regression that prevents subsequent kubeone apply runs from succeeding because the Docker GPG key already exists and gpg --dearmor command is interactively asking for permission to overwrite it. Because there's no tty, this fails and hence KubeOne can't proceed past this step.
I opted in to remove the GPG key instead of using gpg --dearmor --yes because this is considered a more secure practice. GPG can fail for other reasons too (e.g. issues with the key) which should be fatal, i.e. KubeOne shouldn't proceed. Using the --yes flag would ignore these important errors.
What type of PR is this?
/kind regression
Does this PR introduce a user-facing change? Then add your Release Note here:
NONE

Documentation:
NONE

/assign @kron4eg

/approve

/cherrypick release/v1.9


Latest bits are not in dist folder nor jsdelivr

This commit (https://github.com/haacked/aspnet-client-validation/commit/692acb52a4e0c5853d3de5d86f240a93cd397b1a) is not in the /dist folder - see here https://github.com/haacked/aspnet-client-validation/blob/5008fe31af9c0e78d655d76c64e98ac21e2805d5/dist/aspnet-validation.js#L416C16-L416C46
Perhaps build was not run after merge?
In turn, bits on jsdelivr are not updated as well.
Updating the bits, fixes issue with PageRemote/Remote (https://github.com/haacked/aspnet-client-validation/issues/101).
@haacked please help :)

I just published a new release! Please try it out and let me know if it addressed the issues you mentioned.


Problems with parent.relativePath on rhel for io.rhiot:rhiot-camel

I'm trying to build the current rhiot git repo on a rhel 7.2 system. I have tried the stock mvn 3.0.5 as well as the current 3.3.9 download, and both are failing when
with mvn 3.0.5:
[sstark@sstark rhiot]$ mvn install
[INFO] Scanning for projects...
Downloading: http://nexus.codehaus.org/snapshots/org/codehaus/groovy/groovy-eclipse-batch/maven-metadata.xml
[WARNING] Could not transfer metadata org.codehaus.groovy:groovy-eclipse-batch/maven-metadata.xml from/to codehaus-snapshots (http://nexus.codehaus.org/snapshots/): nexus.codehaus.org: unknown error
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project io.rhiot:rhiot-camel:0.1.5-SNAPSHOT (/home/sstark/rhiot/camel/pom.xml) has 1 error
[ERROR]     Non-resolvable parent POM: Could not find artifact io.rhiot:rhiot-cloudplatform:pom:0.1.5-SNAPSHOT and 'parent.relativePath' points at wrong local POM @ line 6, column 13 -> [Help 2]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
[ERROR] [Help 2] http://cwiki.apache.org/confluence/display/MAVEN/UnresolvableModelException
[sstark@sstark rhiot]$ vi camel/pom.xml
[sstark@sstark rhiot]$ mvn --version
Apache Maven 3.0.5 (Red Hat 3.0.5-16)
Maven home: /usr/share/maven
Java version: 1.8.0_45, vendor: Oracle Corporation
Java home: /home/sstark/jdk1.8.0_45/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "3.10.0-327.13.1.el7.x86_64", arch: "amd64", family: "unix"
[sstark@sstark rhiot]$ type mvn
mvn is hashed (/usr/bin/mvn)
[sstark@sstark rhiot]$

with mvn 3.3.9:
Downloading: http://nexus.codehaus.org/snapshots/org/codehaus/groovy/groovy-eclipse-batch/maven-metadata.xml
[WARNING] Could not transfer metadata org.codehaus.groovy:groovy-eclipse-batch/maven-metadata.xml from/to codehaus-snapshots (http://nexus.codehaus.org/snapshots/): nexus.codehaus.org: unknown error
Downloading: https://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-container-default/1.5.4/plexus-container-default-1.5.4.jar
Downloading: https://repo.maven.apache.org/maven2/org/apache/maven/maven-toolchain/1.0/maven-toolchain-1.0.jar
Downloaded: https://repo.maven.apache.org/maven2/org/apache/maven/maven-toolchain/1.0/maven-toolchain-1.0.jar (33 KB at 47.5 KB/sec)
Downloaded: https://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-container-default/1.5.4/plexus-container-default-1.5.4.jar (212 KB at 292.2 KB/sec)
Downloading: https://repo.maven.apache.org/maven2/com/google/guava/guava/14.0.1/guava-14.0.1.pom
Downloaded: https://repo.maven.apache.org/maven2/com/google/guava/guava/14.0.1/guava-14.0.1.pom (0 B at 0.0 KB/sec)
Downloading: https://repo.maven.apache.org/maven2/com/google/guava/guava-parent/14.0.1/guava-parent-14.0.1.pom
Downloaded: https://repo.maven.apache.org/maven2/com/google/guava/guava-parent/14.0.1/guava-parent-14.0.1.pom (0 B at 0.0 KB/sec)
[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[WARNING] 'parent.relativePath' of POM io.rhiot:rhiot-camel:[unknown-version] (/home/sstark/rhiot/camel/pom.xml) points at io.rhiot:rhiot instead of io.rhiot:rhiot-cloudplatform, please verify your project structure @ line 6, column 13
[FATAL] Non-resolvable parent POM for io.rhiot:rhiot-camel:[unknown-version]: Could not find artifact io.rhiot:rhiot-cloudplatform:pom:0.1.5-SNAPSHOT and 'parent.relativePath' points at wrong local POM @ line 6, column 13
[WARNING] 'build.plugins.plugin.version' for org.apache.maven.plugins:maven-jar-plugin is missing. @ io.rhiot:rhiot-cloudplatform-paas-adapters:[unknown-version], /home/sstark/rhiot/cloudplatform/paas/adapters/pom.xml, line 103, column 12
[WARNING] 'build.plugins.plugin.version' for org.apache.maven.plugins:maven-jar-plugin is missing. @ io.rhiot:rhiot-cloudplatform-paas-services:[unknown-version], /home/sstark/rhiot/cloudplatform/paas/services/pom.xml, line 90, column 21
[WARNING] 'build.plugins.plugin.version' for org.apache.maven.plugins:maven-jar-plugin is missing. @ io.rhiot:rhiot-cloudplatform-paas-single:[unknown-version], /home/sstark/rhiot/cloudplatform/paas/single/pom.xml, line 99, column 12
[WARNING] 'build.plugins.plugin.version' for org.apache.maven.plugins:maven-jar-plugin is missing. @ io.rhiot:rhiot-gateway-app:[unknown-version], /home/sstark/rhiot/gateway/gateway-app/pom.xml, line 66, column 21
[WARNING] 'build.plugins.plugin.version' for org.apache.maven.plugins:maven-jar-plugin is missing. @ io.rhiot:rhiot-tooling-shell:[unknown-version], /home/sstark/rhiot/tooling/shell/pom.xml, line 70, column 21
[WARNING] 'build.plugins.plugin.version' for org.apache.maven.plugins:maven-jar-plugin is missing. @ io.rhiot:rhiot-tooling-cmd-core:[unknown-version], /home/sstark/rhiot/tooling/cmd-core/pom.xml, line 66, column 21
[WARNING] 'build.plugins.plugin.version' for org.apache.maven.plugins:maven-jar-plugin is missing. @ io.rhiot:rhiot-cmd:[unknown-version], /home/sstark/rhiot/tooling/cmd/pom.xml, line 60, column 21
 @
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project io.rhiot:rhiot-camel:[unknown-version] (/home/sstark/rhiot/camel/pom.xml) has 1 error
[ERROR]     Non-resolvable parent POM for io.rhiot:rhiot-camel:[unknown-version]: Could not find artifact io.rhiot:rhiot-cloudplatform:pom:0.1.5-SNAPSHOT and 'parent.relativePath' points at wrong local POM @ line 6, column 13 -> [Help 2]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
[ERROR] [Help 2] http://cwiki.apache.org/confluence/display/MAVEN/UnresolvableModelException
[sstark@sstark rhiot]$ mvn --version
Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5; 2015-11-10T08:41:47-08:00)
Maven home: /usr/local/share/apache-maven-3.3.9
Java version: 1.8.0_45, vendor: Oracle Corporation
Java home: /home/sstark/jdk1.8.0_45/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "3.10.0-327.13.1.el7.x86_64", arch: "amd64", family: "unix"
[sstark@sstark rhiot]$

I have gotten much further on osx, so there is some platform specific resolution of the .. value of the relative path. It gets past this problem if the relativePath is more explicit:
[sstark@sstark rhiot]$ git diff
diff --git a/camel/pom.xml b/camel/pom.xml
index acc1c57..9e42c96 100644
--- a/camel/pom.xml
+++ b/camel/pom.xml
@@ -7,7 +7,7 @@
         <groupId>io.rhiot</groupId>
         <version>0.1.5-SNAPSHOT</version>
         <artifactId>rhiot-cloudplatform</artifactId>
-        <relativePath>..</relativePath>
+        <relativePath>../cloudplatform</relativePath>
     </parent>
     <name>Rhiot :: Camel components</name>
     <artifactId>rhiot-camel</artifactId>

I am running Fedora 19/Maven 3.3/JDK 8 and have the same issue. Your more explicit ../cloudplatform worked for me. Thanks,


SaveAll Linux keybinding Ctrl+Alt+S minimizes VS Code on Ubuntu

VSCode Version: 1.7.0 insiders
OS Version: Linux Ubuntu 14.04

We added a keybinding for workbench.action.files.saveAll in the 1.7 release to Ctrl+Alt+S for Linux.
{ "key": "ctrl+alt+s",            "command": "workbench.action.files.saveAll" }
For Ubuntu, this is bound to OS command to minimizes the window and VS Code is minimized (not what the user is expecting).
Suggestion: bind Files: SaveAll to chord Ctrl+K S like on Windows.

This issue is similar to #13567 which targets windows platform. That issue was initially assigned to @chrmarti first so I did so as well.

It actually seems to do nothing on Ubuntu 16.10, either at a OS level or saving all. What was the reason this keybinding was added anything? Is ctrl+k s used in any other editors?

Should have checked more carefully. The reason for adding this was: Users with auto-save disabled will end up with multiple dirty editors after global Replace All, Rename across files, possibly others. Binding Save All seemed like a good idea.

There's a bit of background to this, many people actually wanted it to be bound to ctrl+shift+s at launch because I believe VS is like that. We decided to align with Sublime's keybinding in this respect.
Given what we now know about ctrl+alt+s, I'm not sure there's much value in hiding it being a chord keybinding that isn't used in other programs.

Atom and ST don't bind it on Ubuntu 14.04 or Windows 10, only on OS X. They might not use multiple dirty editors in their regular workflow as much as we do.

I'd say align with Atom and ST in this respect, a new non-standard keybinding wouldn't be discoverable and people can always fallback on a custom keybinding if they need to.
On a personal note I don't use auto save or save all, when I use TS refactoring I go through my quick open editors list and save the files after making sure. Not the most efficient method but I'm concerned I have either untitled files or changes I don't want to save hanging around from earlier.


Correcting the 30U/360 day-count calculation in the Java 6 version of…

… the code.
When the startDateMonth and endDateMonth variables were created, the value they contain was
set to the maximum value of the month (e.g. the month value for 2015-01-15 was 31, instead of 1)
instead of the day of the month.

Thank you! The bug was introduced when switching to joda time (going from 1.3.x to 2.0.0).
Thx!


Publication

@LinguList Just to verify: Does this dataset refer to the same publication as chaconarawakan, because it is concerned with the areal word list by Huber and Reed Vocabulario Comparativo: Lista de PalabrasSelectas En Lenguas Indigenas de Colombia? (in contrast to the Swadesh 100 for the other part). Or am I missing something?

If the metadata quotes the same paper: yes. I think I received two files from T. Chacon by then, after I asked him to put the data online.

In fact, the data is different, though: I coded the arawakan Data independently from a spreadsheet Tiago had prepared from sources for him in edictor format. Then, I also gave him access to Huber1992, which is where this dataset comes from. So the Arawakan sub-part was coded twice. And both datasets supplement the paper, although the paper itself does not really make clear what was used when. Something we have to live with...

Yup, that's what confused me. Thanks for clarifying it!

Done in #5 and Concepticon.


traefik modify response headers

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
I hava a backend service. It provides a custom response header name 'encryptKey', the response message obtained through traefik access has become 'Encryptkey'.
What did you see instead?
The direct access pod message is as follows

# The traefik message is as follows

What version of Traefik are you using?
v2.9.5
What is your environment & configuration?
# (paste your configuration here)

Add more configuration information here.
If applicable, please paste the log output in DEBUG level
No response

Hello,
I think this is a duplicate of #466
You have a complete answer on this one.

Hello,
I think this is a duplicate of #466
You have a complete answer on this one.

thanks


Some alerts include text which is not written in Sentence case.

Add (or fix the) CSS to capitalise the first character of alerts

@kwwii can you assign this to me I would like to work on this!

@gaktive - I assumed that this would be done programmatically such that the first letter of the first word of every alert message is capitalised.

Hi, can you assign this issue to me? I would like to try to sort this out.

Please update the issue with specifics of which alerts need fixing


Integrate LLVM at llvm/llvm-project@b2505ca2ece3

Please wait for the build to finish before merge as I haven't built locally. Wanted to start the build on github as it will take some time.


Rewards Functionality

Requirement(s): Needs payment functionality to be completed

Update: We are falling behind on this right now. Frontend team has no updates on this and backend team is not yet tracking rewards.


Feedback

Feedback included

Thanks ThibaultLesuisse. It was very  challenging to  fix errors. I will continue to explore more.


Comments Count Feature in Movie App

MOVIE APP

[x] This is a MOVIE APP we will complete this with HTML, CSS & JS with advanced syntax (ES6) Modules.
[x] We use Gitflow and Webpack environment for this MOVIE APP.
[x] We added Header in this Movie APP
[x] We add a Popup Function in this Project.
[x] We added amazing  Style in this Project.
[x] We added a Comment Display Feature in this Project.
[x] We added a Comment Count Feature to this Project.
[x] We added a Like Featuere in this Project.
[x] We added a Like Count Feature in this Project.

APPROVED
Hi [@Atril33]),
Your project is complete! So it's time to merge it
Congratulations! 🎉
Highlights
There are no linter errors in the project. ✔
This pull request meets all project requirements. Keep rocking! ✔
The comment counter is working perfectly ✔
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.


SQL Error [208] [S0002]: Error while trying to export data to .CSV from temporary tables with single hashtag.

Description
Suddenly, even without trying to upgrade, Dbeaver started throwing an error which says: SQL Error [208] [S0002]: Invalid object name '#tablename'
When I created a temp table with double hashtags (global temp table), then I was able to export the data without errors.
Why sudden change?
DBeaver Version
22.1.3
Operating System
Windows 7
Database and driver
SQL Server DB
Steps to reproduce
No response
Additional context
No response

I had to uncheck the option called "open new connection" in "Extraction Settings" of "Export Data" window and that worked.


Waypoint: Manage Packages with NPM checklist error

Challenge http://www.freecodecamp.com/challenges/waypoint-manage-packages-with-npm has an issue.
This waypoint should be updated to match cloud9's updates, however, there is a legitimate issue that could confuse a beginner on item 12. The command listed in item 12 should be changed to 'how-to-npm' in order to run the tutorial program.

Hey @Damon-Myers thanks for posting.
Your first issue is a duplicate of #1318, keep an eye there for updates.
Your second issue seems to have been fixed


1.13.2?

Plugin will be updated for 1.13.2?

Hi,
I’m a little busy right now and don’t even know what is different in 1.13 (or if it directly impacts my plugin). I get an email like this every update, but rarely need to make actual adjustments to most of my plugins. Most plugins should just work, and I’ve even taken additional precautions by designing IntelliDoors to be relatively functional even in the event a new update is released. So if you haven’t tried it yet, just install the plugin anyway to see if it works and monitor your console for errors.
If it’s a major issue I’ll look into it in general here soon, but right now my testing server isn’t even online, and I don’t even have any players of my own to work on testing out the plugin with.
Most updates shouldn’t cause anything to break with IntelliDoors, but even ones that add new blocks (e.g. new door types, fence gates, etc) can be added to the plugin by just adjusting the config file to include the new blocks. However I can understand an update making it more convenient by just having new blocks work without changing anything.
Thanks,
Matt
Matthew Oltendorf
(925) 548-2096
On Mar 16, 2019, 14:47 -0700, BaxAndrei<EMAIL_ADDRESS>wrote:

Plugin will be updated for 1.13.2?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


[ci] Temporarily disable running of checks on Daint

For the duration of the PE upgrade.

@jenkins-cscs retry daint


Fix block location order

There is a regression in getFileBlockLocations() API where the client does not return the block locations in their original order in file. This is causing Presto queries to fail because they are relying on the block orders.  This PR refactored the native API so it returns a list instead of an unordered map.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2229/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2233/
Test PASSed.

@calvinjia PTAL.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2244/
Test PASSed.

@calvinjia Thanks! I updated the PR. PTAL.


Build failure with disabled SNMP support

Describe the bug
When trying to build ltfs as of today's current master on Debian 11/amd64 with --disable-snmp passed to configure, ld fails due to libdl not getting linked via AM_LDFLAGS.
To Reproduce
Steps to reproduce the behavior:

git clone 'https://github.com/LinearTapeFileSystem/ltfs.git' && cd ltfs
./autogen.sh
./configure --disable-snmp
make

Expected behavior
The build succeeds and yields binaries/artifacts without SNMP support.
Desktop (please complete the following information):

OS: Debian 11 amd64

Additional context
In configure.ac, when building with SNMP support, AM_LDFLAG is ultimately expanded to include the output of the net-snmp-config --agent-libs command, which on my system happens to include the required -ldl argument for the linker to succeed.
I think passing that argument to all build steps where the linker targets objects that had their corresponding source file(s) (at least src/libltfs/ltfslogging.c and src/libltfs/plugin.c) use libdl features should fix the problem.
FWIW, as a workaround, I configured the source with LIBS=-ldl ./configure --disable-snmp, which succeeds.

In my RHEL7 env, -ldl is included into the configuration of ICU.
ICU_MODULE_CFLAGS=' -D_REENTRANT    -I/usr/include '
ICU_MODULE_LIBS='-Wl,-z,relro   -lpthread -ldl -lm   -L/usr/lib64 -licui18n -licuuc -licudata  -lpthread -ldl -lm   '

Did you use custom /usr/bin/icu-config described into #153? Or does Debian11 provide /usr/bin/icu-config into ICU packages?

Debian 11 does not provide icu-config:
$ dpkg -S icu-config
dpkg-query: no path found matching pattern *icu-config*

However, I did not prepare my build env (my desktop machine) in any way to build ltfs. Both configuration and build phase work for me without icu-config present, with the above-mentioned limitations when disabling SNMP support.

I can confirm current master builds successfully on Debian 11. Thanks very much for the fix!


Building on OSX causes paket errors

It works fine on windows, but on OSX I'm getting paket install errors which break the build.
/Users/rachel/Code/Third Party/Cricket/.paket/paket.targets: Error: Command 'mono --runtime=v4.0.30319 /Users/rachel/Code/Third Party/Cricket/.paket/paket.exe install' exited with code: 2. (Cricket)

@rachelreese @plumhead  Does the latest build of this work on OS X?

Build Yes:
All Tests run: No
Detail below:
Tests run: 98, Errors: 0, Failures: 1, Inconclusive: 0, Time: 8.677544
seconds
Not run: 3, Invalid: 0, Ignored: 3, Skipped: 0
Errors and Failures:

Test Failure : Cricket.Tests.Given an Message Handler.I can scan for a
message
No result timeout

at Cricket.Tests.Given an Message Handler.I can scan for a message ()
[0x00000] in :0
Tests Not Run:

Ignored : Cricket.Tests.Given an event stream.I can remove a
subscription post and subscribe to events by key
Failing on build servers

Ignored : Cricket.Tests.Given an event stream.I can remove a
subscription post and subscribe to events by type
Failing on build servers

Ignored : Cricket.Tests.Given an Message Handler.I can recieve a message
Failing on build servers

Running build failed.
Error:
System.Exception: NUnit test failed (1).
at Fake.NUnitSequential.NUnit (Microsoft.FSharp.Core.FSharpFunc2 setParams, IEnumerable1 assemblies) [0x00000] in :0
at<EMAIL_ADDRESS>(Microsoft.FSharp.Core.Unit _arg5)
[0x00000] in :0
at<EMAIL_ADDRESS>(Microsoft.FSharp.Core.Unit unitVar0) [0x00000] in :0
at Fake.TargetHelper.runTarget@314 (System.String targetName) [0x00000]
in :0

Build Time Report

Target            Duration

Clean             00:00:00.0103774
RestorePackages   00:00:00.0711791
AssemblyInfo      00:00:00.0094526
Build             00:00:24.0219756
Total:            00:00:36.9810771
Status:           Failure

System.Exception: NUnit test failed (1).

at Fake.NUnitSequential.NUnit (Microsoft.FSharp.Core.FSharpFunc2 setParams, IEnumerable1 assemblies) [0x00000] in :0
at<EMAIL_ADDRESS>(Microsoft.FSharp.Core.Unit _arg5)
[0x00000] in :0
at<EMAIL_ADDRESS>(Microsoft.FSharp.Core.Unit unitVar0) [0x00000] in :0
at Fake.TargetHelper.runTarget@314 (System.String targetName) [0x00000]
in :0

On 4 February 2015 at 11:50, Colin<EMAIL_ADDRESS>wrote:

@rachelreese https://github.com/rachelreese @plumhead
https://github.com/plumhead Does the latest build of this work on OS X?
—
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Cricket/issues/40#issuecomment-72842068.

@plumhead is that with the latest? They pass on travis.??

Clean pull from master.
Sent from my iPad

On 4 Feb 2015, at 13:46, Colin<EMAIL_ADDRESS>wrote:
@plumhead is that with the latest? They pass on travis.??
—
Reply to this email directly or view it on GitHub.

Fixed...


fix time series inconsistency between continuous and discrete time simulations

As pointed out in issue #239, there is an inconsistency between the way that continuous time simulations and discrete time simulations return their outputs for single output systems.  In continuous time, the code follows the documentation and the following works:
t, y = step_response(sys)
plot(t, y)

However, in discrete time simulations the returned value for y had shape (1, len(t)), which meant that you had to do the following for discrete time systems:
t, y = step_response(sysd)
plot(t, y[0])

The problem was that for continuous time simulations forced_response() (the ultimate function that gets called) had an np.squeeze() at the end but this wasn't present for discrete time simulations. This PR fixes that problem so that the behavior is consistent, resolving the bug in issue #239.
Other small changes:

Added unit tests that catch the original bug

Added a squeeze keyword that controls whether this behavior is implemented (default = True).

Did some PEP8 cleanup on timeresponse.py

Note that this may make some existing code break since discrete time simulations can now return an output that is a 1D array instead of 2D.  The fix is either to remove the output index (y[0] becomes y) or set the squeeze keyword to False).

Coverage increased (+0.006%) to 78.228% when pulling 3120e01a47f23164248aabe4c8907056ad6af07f on murrayrm:fix_time_series into d8d0a9c6a60230b0b3dad0bdaa2dd3004bdd254b on python-control:master.


How to rename default passport cookie name 'laravel_token'?

First off, thank you for putting this together, it's a very nice package!
I was looking for a way to overwrite the default passport cookie laravel_token name to something else like custom_token for example. I only found this resource, but when I try something like:
Passport::cookie('custom_token'); as suggested in the example, it returns the new cookie name in the response to the browser, but it does not Authenticate or work for the next request.
I placed that snippet in the login area before the response, but maybe it's not the right place for it to work.
Any idea how to have the custom token name work with this package and authenticate requests successfully?
Note: The default laravel_token cookie name authenticates perfectly, so only when renamed it breaks authentication.

Hey, thank you very much for the kind words about the boilerplate!
I think it might be an issue with where you are placing the custom naming cookie code, it seems that by placing it in the login area, passport will only change the cookie name on that particular route.
Could you try placing Passport::cookie('custom_token'); in your AppServiceProvider (specifically the register method) and tell me if it works for you?

That worked right away! Thank you so much again and keep up the great work!


[wip] Bloop compile to correct JVM version

I'd like to somehow cleaup and organize this code now. I have no idea what's the right way.
Should I just extend acceptBloopVersion field function in BloopRifleScala in that way, it checks also bloop's jvm and restarts it if it's invalid?
Should I download a newer jvm via couriser lib and spawn bloop with it in that case in Operations.startServer?


[SPARK-47561][SQL] Fix analyzer rule order issues about Alias

What changes were proposed in this pull request?

We found two analyzer rule execution order issues in our internal workloads:

CreateStruct.apply creates NamePlaceholder for unresolved NamedExpression. However, with certain rule execution order, the NamedExpression may be removed (e.g. remove unnecessary Alias) before NamePlaceholder is resolved, then NamePlaceholder can't be resolved anymore.
UNPIVOT uses UnresolvedAlias to wrap UnresolvedAttribute. There is a conflict about how to determine the final alias name. If ResolveAliases runs first, then UnresolvedAlias will be removed and eventually the alias will be b for nested column a.b. If ResolveReferences runs first, then we resolve a.b first and then UnresolvedAlias will determine the alias as a.b not b.

This PR fixes the two issues

CreateStruct.apply should determine the field name immediately if the input is Alias
The parser rule for UNPIVOT should follow how we parse SELECT and return UnresolvedAttribute directly without the UnresolvedAlias wrapper. It's a bit risky to fix the order issue between ResolveAliases and ResolveReferences as it can change the final query schema, we will save it for later.

Why are the changes needed?

fix unstable analyzer behavior with different rule execution orders.
Does this PR introduce any user-facing change?

Yes, some failed queries can run now. The issue for UNPIVOT only affects the error message.
How was this patch tested?

verified by our internal workloads. The repro query is quite complicated to trigger a certain rule execution order so we won't add tests for it. The fix is quite obvious.
Was this patch authored or co-authored using generative AI tooling?

no

cc @viirya @gengliangwang

Looks good to me.

I backported it to 3.5 directly since there is no merge conflicts. Since it's very hard to hit this bug, I didn't backport it further. I've updated the JIRA ticket as well.


Make KUBECONFIG a template variable that is interpolated in templates (config map, cli, env)

Problem
Prometheus requires the kube configuration file to be present for service discovery with Kubernetes. The only process who has access to this file on the nodes is the agent. It cannot be set by the operators.
Proposed solution
Make KUBECONFIG a variable known by the agent  similarly to {{packageroot}} or {{configroot}}. Allow operators to use this variable in templates and replace it with the appropriate path.
Example
The monitoring operator creates a configmap with the fils prometheus.yaml. Therein the k8s service discovery is configured like this snipped shows:
    kubernetes_sd_configs:
      - role: pod
        kubeconfig_file: {{KUBECONFIG}}
        namespaces:
          names:
            - {}
        selectors:
          - role: pod

After more discussions, we agree that this is not the best way to proceed for multiple reasons:

it makes the agent diverge even more from the k8s kublet behaviour.
the kube config file used by the agent should not be shared with operators for security reasons.


fix(backend): Preserve url protocol when joining paths

Description
Internally when joinPaths was used to join strings that included a url protocol like https:// the utility function would remove the duplicate / from the protocol resulting in a string with this form 'https:/api.lclclerk.com/v1/organizations/org_2YRy2Bcrc05EMTY0nMY3qHTjbwj'
When this string was passed in a URL constructor like new URL(...) the URL would either be fixed automatically or throw an "Invalid URL" error.
I believe it came down to node version and different implementations of the URL class under the hood. In any case this PR ensures that absolute url string that is generated from joinPaths will keep the protocol intact.

Fixes https://github.com/clerk/javascript/issues/2706
Checklist

[ ] npm test runs as expected.
[ ] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[x] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[x] @clerk/backend
[ ] @clerk/chrome-extension
[ ] @clerk/clerk-js
[ ] @clerk/clerk-expo
[ ] @clerk/fastify
[ ] gatsby-plugin-clerk
[ ] @clerk/localizations
[ ] @clerk/nextjs
[ ] @clerk/clerk-react
[ ] @clerk/remix
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/themes
[ ] @clerk/types
[ ] build/tooling/chore

Yes, this maybe have to do with edge vs node runtime. I'm still investigating. The fix is valid but I will merge after I've successfully reproduced.

!snapshot

!snapshot

!snapshot

Hey @octoper - the snapshot version command generated the following package versions:

Package
Version

@clerk/backend
1.0.1-snapshot.vf0e076d

@clerk/chrome-extension
1.0.1-snapshot.vf0e076d

@clerk/clerk-js
5.0.1-snapshot.vf0e076d

@clerk/clerk-expo
1.0.1-snapshot.vf0e076d

@clerk/fastify
1.0.1-snapshot.vf0e076d

gatsby-plugin-clerk
5.0.1-snapshot.vf0e076d

@clerk/localizations
2.0.1-snapshot.vf0e076d

@clerk/nextjs
5.0.1-snapshot.vf0e076d

@clerk/clerk-react
5.0.1-snapshot.vf0e076d

@clerk/remix
4.0.1-snapshot.vf0e076d

@clerk/clerk-sdk-node
5.0.1-snapshot.vf0e076d

@clerk/shared
2.0.1-snapshot.vf0e076d

@clerk/themes
2.0.1-snapshot.vf0e076d

@clerk/types
4.0.1-snapshot.vf0e076d

Tip: Use the snippet copy button below to quickly install the required packages.
@clerk/backend
npm i<EMAIL_ADDRESS>--save-exact

@clerk/chrome-extension
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-js
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-expo
npm i<EMAIL_ADDRESS>--save-exact

@clerk/fastify
npm i<EMAIL_ADDRESS>--save-exact

gatsby-plugin-clerk
npm i<EMAIL_ADDRESS>--save-exact

@clerk/localizations
npm i<EMAIL_ADDRESS>--save-exact

@clerk/nextjs
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-react
npm i<EMAIL_ADDRESS>--save-exact

@clerk/remix
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-sdk-node
npm i<EMAIL_ADDRESS>--save-exact

@clerk/shared
npm i<EMAIL_ADDRESS>--save-exact

@clerk/themes
npm i<EMAIL_ADDRESS>--save-exact

@clerk/types
npm i<EMAIL_ADDRESS>--save-exact

We manage to replicate this in CF workers. It seems like the implementation of the URL constructor does not auto fixe the url as node or browser runtimes do. Thanks @octoper for replicating.


Added Azure connection string

#4 Added Azure connection string

That way you can connect to it in dev mode using the config instead of always setting the environment variable.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Beshoy Hanna<EMAIL_ADDRESS>Sent: Monday, May 27, 2019 2:49:45 AM
To: fadyham/hazzat-api
Cc: fadyham; Author
Subject: Re: [fadyham/hazzat-api] Added Azure connection string (#14)
@behoyh commented on this pull request.
Just a little bit confused on why we need the mssql://${dbUser}:${dbPass}@${dbServer}/${dbName}?encrypt=true URL if the entire connection string is coming from Azure
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/fadyham/hazzat-api/pull/14?email_source=notifications&email_token=AD3ENNHMKOWSJPXOEKWNU2DPXOVDTA5CNFSM4HPY74PKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOBZXZFZA#pullrequestreview-242193124, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AD3ENNH5P3IM7JYLXO7AEP3PXOVDTANCNFSM4HPY74PA.

We may want to have an env variable or something decide the environment in the future, and different configs for each. I'll approve and add an issue.


[fix] handle files from tenant root solving #1328

Implemented additional tests and central logic for relative urls and tenant root. Solving #1328

Thanks! We'll review it shortly 👏

Will have a look :)!

A local test with o365 spo page get -u https://contoso.sharepoint.com -n home seem to be working after the suggestions you did. Looks good but I can't explain why I did fail in the first place.. So I am not 100% confident that this solves the issue.. Can you do a sanity check?

Will do. Thanks!

Works like a charm indeed. Thank you! 👏

Merged manually. Thank you! 👏


test(osmomath): log tests around max spot price

Closes: #XXX
What is the purpose of the change
This is related to: https://github.com/osmosis-labs/osmosis/issues/3540
Upon adding tests around max spot price for geometric twap, overflows started to occur.
To eliminate the possibility of log function being at fault, the tests around max spot price values are added directly to the log. Benchmark is also added to ensure that large values do not inflate the time it takes to run the log. The difference was indeed negligible.

Hi @czarcas7ic . Could you please review this change when you get the chance?


Feature Request: condition to check if a Github release  to a given tag exists

Is your feature request related to a problem?
I'm trying to sync the release of 3 different repositories. For that, I want to run the updatecli script when all the 3 repositories have a release to a given tag. If there are such releases, do whatever I want to do.
Solution you'd like
I would like to have a condition that checks if a given github repository has a release for a given tag.
Alternatives you've considered
I'm using the condition to check if the tag exists in a git repository. But this does not guarantee that the github release exists. This is a problem for me because I need to download an asset from the release of one of the repositories. Therefore, I need to check if the release is there using other tools.
Anything else?
No response

Thanks for opening this issue
I just did a quick check and I think you spot a UX issue and I am open to suggestion
Currently the githubrelease condition compares the value from the spec.tag (default to source output), and the versionfilter. I think versionfilter should only be used in source at least that's what we do for the other resources
So the Github release should check that a tag exist and only that
I tried the following manifest
Manifest
name: Test checking that both github release and gittag exist

scms:
  default:
    kind: github
    spec:
      owner: updatecli
      repository: updatecli
      branch: main
      token: '{{ requiredEnv "GITHUB_TOKEN" }}'

sources:
  shell:
    name: Get version from shell command output
    kind: shell
    spec:
      command: echo v0.40.1
conditions:
  gittag:
    name: Test that gittag exist based on source output
    sourceid: shell
    kind: gittag
    scmid: default
  githubrelease:
    name: Test that github release tag exist based on source output
    kind: githubrelease
    sourceid: shell
    spec:
      owner: updatecli
      repository: updatecli
      token: '{{ requiredEnv "GITHUB_TOKEN" }}'
      versionfilter:
        kind: regex
        pattern: "v0.40.1"
targets:
  version:
    name: Only run if conditions succeeded
    kind: shell
    spec:
      command: echo release

Output
 ~  updatecli diff --config /tmp/test.yaml                                                                                 ✔ │ took 13s

+++++++++++
+ PREPARE +
+++++++++++

Loading Pipeline "/tmp/test.yaml"

SCM repository retrieved: 1

++++++++++++++++++
+ AUTO DISCOVERY +
++++++++++++++++++

++++++++++++
+ PIPELINE +
++++++++++++

###########################################################
# TEST CHECKING THAT BOTH GITHUB RELEASE AND GITTAG EXIST #
###########################################################

SOURCES
=======

shell
-----
The shell 🐚 command "echo v0.40.1" ran successfully with the following output:
----
v0.40.1
----

CONDITIONS:
===========

githubrelease
-------------
Searching for version matching pattern "v0.40.1"
✔ Github Release version "v0.40.1" found

gittag
------
Source input value detected: using it as spec.versionfilter.pattern
Searching for version matching pattern "v0.40.1"
✔ Git tag "v0.40.1" matching

TARGETS
========

version
-------

**Dry Run enabled**

The shell 🐚 command "echo release v0.40.1" ran successfully with the following output:
----
release v0.40.1
----

=============================

REPORTS:

⚠ Test checking that both github release and gittag exist:
	Source:
		✔ [shell] Get version from shell command output (kind: shell)
	Condition:
		✔ [githubrelease] Test that github release tag exist based on source output (kind: githubrelease)
		✔ [gittag] Test that gittag exist based on source output (kind: gittag)
	Target:
		⚠ [version] Only run if conditions succeeded (kind: shell)

Run Summary
===========
Pipeline(s) run:
  * Changed:	1
  * Failed:	0
  * Skipped:	0
  * Succeeded:	0
  * Total:	1

@jvanz If it's good for you
If I understand correctly you would expected the following snippet to work
conditions:
  test:
    kind: githubrelease
    spec:
       owner: updatecli
       repository: updatecli
       tag: 1.0.0

@jvanz If the proposed manifest is fine to you then I'll go merge the pullrquest #1087

@olblak Yes, that's fine to me. JFYI, the documentation should be updated as well. I've did not test the githubRelease in the condition configuration because the documentation show that it does not have the condition feature.

@olblak Yes, that's fine to me. JFYI, the documentation should be updated as well. I've did not test the githubRelease in the condition configuration because the documentation show that it does not have the condition feature.

Thanks for the feedback, I didn't realize that I forgot to update the doc website...


fix: modify wrong url magic-string to magical-string

This is a PR to fix : https://github.com/jeantimex/javascript-problems-and-solutions/issues/341

Thanks for the fix!


Can i export as pdf

Hi,
If I need pdf export, can I use
Thanks

If you see examples and plugin structure, pdf or any other format support you need to add in your table, plugin is independent of the same.

If you see examples and plugin structure, pdf or any other format support you need to add in your table, plugin is independent of the same.

You mean, need to add pdf package and then use in Table. Right?

Yes

Yes
Ok, thanks,
react-table-plugins only for collect table row.
If we want to export, we need to use the export type related package.
Right?
There is no inbuild package?

Yes

Hi @gargroh
I am facing an error in useExportData file
Error: React Table: Export Blob is mandatory


B404-B603- Categorisation as warnings

Hi
Im running bandit against my python project (its a great tool, i love it!) but there are some things that could be improved
Im using subprocess.call, but with shell set to False, so im not vulnerable to command injection there. However, i have two errors in my build.
B404 - as im using subprocess
B607 - as, despite not being vulnerable to command injection, i need to be careful
Those recommendations are great, but in my opinion should be categorised as warnings rather than vulnerabilities. I have been reading and most of the people ignore those rules by default
Would be possible to have this amended?
Thanks

I agree, as oftentimes it's necessary to shell out to the OS, and (unless I'm missing something) subprocess is the current suggested way of doing this, so flagging just an import of subprocess as a issue is problematic.


Building on Linux without multiple .csproj files

@fabriciomurta Following your instructions in #268, I developed a better approach.
Please ignore #268.

Are you able to run this on linux?
../../Resources/JSBuilder/JSBuildConsole.exe /path="../../Resources/Bridge.jsb"
I was expecting (at least) a mono (...)JSBuildConsole.exe or an a.out version of this. :)

I expect you to get a
bash: ./Resources/JSBuilder/JSBuildConsole.exe: cannot execute binary file
Every time you try to run this win32 file on linux or OS X. That would not be a problem if you're on mingw or cygwin though.

Alright, I can run it with mono.


Couldn't find appropriate backend to handle uri example.wav and format None.

Describe the bug
Here's a detailed description of the issue you're encountering along with the relevant error message, translated into English:

Issue Description
I am following the tutorial from this Colab notebook step by step. However, I encountered a problem at the first step of the code execution. Below is the code and the pyproject.toml configuration I am using.
Code
import speechbrain as sb
from speechbrain.dataio.dataio import read_audio
from IPython.display import Audio

from speechbrain.inference.ASR import EncoderDecoderASR

asr_model = EncoderDecoderASR.from_hparams(
    source="speechbrain/asr-crdnn-rnnlm-librispeech",
    savedir="pretrained_models/asr-crdnn-rnnlm-librispeech",
)
asr_model.transcribe_file("./content/example.wav")
signal = read_audio("./content/example.wav").squeeze()
Audio(signal, rate=16000)

pyproject.toml
[tool.poetry]
name = "s2st-with-speechbrain"
version = "0.1.0"
description = ""
authors = ["Frederic"]
readme = "README.md"

[tool.poetry.dependencies]
python = "^3.11"
speechbrain = "^1.0.0"
ipython = "^8.25.0"
torch = "^2.3.0"

[build-system]
requires = ["poetry-core"]
build-backend = "poetry.core.masonry.api"

Environment

ffmpeg version: 7.0.1
macOS version: 15 (ARM version)

Error Message
torchvision is not available - cannot save figures
SpeechBrain could not find any working torchaudio backend. Audio files may fail to load. Follow this link for instructions and troubleshooting: https://pytorch.org/audio/stable/index.html
SpeechBrain could not find any working torchaudio backend. Audio files may fail to load. Follow this link for instructions and troubleshooting: https://pytorch.org/audio/stable/index.html
Traceback (most recent call last):
  File "/Users/zeta/workspace/s2st_with_speechbrain/test.py", line 11, in <module>
    asr_model.transcribe_file(
  File "/Users/zeta/Library/Caches/pypoetry/virtualenvs/s2st-with-speechbrain-wrzyle28-py3.11/lib/python3.11/site-packages/speechbrain/inference/ASR.py", line 77, in transcribe_file
    waveform = self.load_audio(path, **kwargs)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zeta/Library/Caches/pypoetry/virtualenvs/s2st-with-speechbrain-wrzyle28-py3.11/lib/python3.11/site-packages/speechbrain/inference/interfaces.py", line 282, in load_audio
    signal, sr = torchaudio.load(str(path), channels_first=False)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zeta/Library/Caches/pypoetry/virtualenvs/s2st-with-speechbrain-wrzyle28-py3.11/lib/python3.11/site-packages/torchaudio/_backend/utils.py", line 204, in load
    backend = dispatcher(uri, format, backend)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zeta/Library/Caches/pypoetry/virtualenvs/s2st-with-speechbrain-wrzyle28-py3.11/lib/python3.11/site-packages/torchaudio/_backend/utils.py", line 116, in dispatcher
    raise RuntimeError(f"Couldn't find appropriate backend to handle uri {uri} and format {format}.")
RuntimeError: Couldn't find appropriate backend to handle uri example.wav and format None.

Hello, can you try following those steps in our docs? https://speechbrain.readthedocs.io/en/latest/audioloading.html (looks like I had a copy-paste fail in those logs, it meant to point to this rather than the torchaudio docs...)
We haven't had feedback from macOS users so I am curious if you can find a solution. You mention your ffmpeg version, but odds are that torchaudio isn't finding it. This ffmpeg version should be supported by torchaudio.

I know the reason for this error is because when torchaudio checks the version of ffmpeg, it only supports 6, 5, and 4, and I have version 7 installed locally.
So, install soundfile to solve the ffmpeg version mismatch.

Ah, I misremembered the supported ffmpeg version range. Anyway, since the issue is resolved, I will close this issue since it is setup-related rather than a bug in SpeechBrain code/docs.


Anchors don't work properly

I can't link to specific sub-sections using the anchor tag anymore.
Example:
http://zalando.github.io/restful-api-guidelines/common-data-objects/CommonDataObjects.html#must-use-common-error-return-objects doesn't bring me to the respective sub-section.
I also don't see the little "anchor" symbol next to headlines/sub headlines

This section was renamed (http://zalando.github.io/restful-api-guidelines/common-data-objects/CommonDataObjects.html#must-use-problem-json). We should see if we can check the internal links during the build process.
But the anchor symbol missing is a new problem, likely introduced by #148. Willi will have a look at this.

The missing anchor symbol is possibly caused by incompatible library upgrades, and thus could possibly be fixed by upgrading to Gitbook 3 (#106).

They still don't work. I investigate.

Depends on Gitbook update, which breaks other things. Therefore decided not to do this now.


Addressing Deserialization Bug in Signal PDU

What does this PR do?

[ ] Documentation
[ ] Code changes

How did you verify your code works?

@znoble360 The change in this branch may address the bug you were experiencing. If so, I'll ensure to add checks to other PDUs that could experience similar issues during deserialization.

I just got a chance to test out the update and it seems to take care of the issue, thanks!

@znoble360 Glad to hear that! I apologize for the delay in addressing this. If you run into any other issues, please feel free to continue submitting issues and/or PRs. Thanks!


Nick: Watch fubeauy.vn

Nick requests the watch of the watch_keyword fubeauy\.vn. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

fubeauy\.vn has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


HierarchicalModel can't be instantiated easily with AutoModel.from_pretrained

Since HierarchicalModel uses a separate config class for its encoder heads, it does not play nice with AutoModel.from_pretrained.
>>> model = AutoModel.from_pretrained(<path/to/pretrained/model>, config=<CnlpConfig>)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File ".../site-packages/transformers/models/auto/auto_factory.py", line 463, in from_pretrained
    return model_class.from_pretrained(
  File ".../site-packages/transformers/modeling_utils.py", line 2156, in from_pretrained
    model = cls(config, *model_args, **model_kwargs)
TypeError: HierarchicalModel.__init__() missing 1 required positional argument: 'transformer_head_config'

This config needs to be the same as it is in training, so it needs to be part of what gets exported when saving the model. The most straightforward approach would be to incorporate it into CnlpConfig as an optional dict attribute.

is this done?


Fix typo

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context.
chck  -> check
Type of change

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

@abrilurena Thanks ✨


Fixes various GitHub issues (CSV parsing and Documentation)

I added detailed comments explaining the changes to each commit.

@tpetricek looks good! Travis CI failure looks bogus... can submit manual rerun?

Hmm, this failed again - I think it might be because the build server gets F# Data files using Paket, which calls the GitHub REST API and we exceeded the number of allowed requests. /cc @forki ?

nope this one looks not like an API rate limit error.

This is probably not related - but Deedle still uses the old Objective-C profile for Travis. We should probably update to the C#/Mono profile. Who knows how to do this :-) ?

@tpetricek: helpful? http://stackoverflow.com/questions/16751772/how-do-i-use-travis-ci-with-c-sharp-or-f

Now this is a more useful failure - I forgot to delete some experiments.

Hmm, I see http://docs.travis-ci.com/user/languages/csharp/
Otherwise, no experience tinkering too much with .travis.yaml

@adamklein The build is working now. Thanks for asking about it - there was, indeed, a bug!

Awesome!


onSelect function does not working

I think There is a problem with onSelect function. it only returns Untick and tick by writing down them to console in latest version.
it does not return anything about current item and does not print number of  selected items as you intended to code in example usage.
I am currently using 1.3.4 and it works as expected.
Thank you and have a nice day.
Best regards,
Beton.

I'm also encountering this issue. Did you find any solution?


express instrumentation does not construct span name correctly when ignoring middlewares

Example - middlewares activated for a given path:
const instrumentation = new ExpressInstrumentation({
  ignoreLayersType: [ExpressLayerType.MIDDLEWARE],
});
const app = express();
app.use('/mw', (req, res, next) => { next(); });
app.get('/mw', (req, res) => { res.send('ok'); });

When doing a request to the /mw endpoint, the span name will be GET /mw/mw.
This is repeated for every middleware on the same path and the final span name is a concatenation of middleware paths + the final route path and can cause some wonky route names, e.g. GET /mw/mw/mw/mw/mw/mw

Closing it as the fix was merged


🐛 [Bug] Compilation failure for HuggingFace T5-base Model

Bug Description
When compiling the T5-base network (https://huggingface.co/t5-base), the following error is encountered:
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu! (when checking argument for argument index in method wrapper__index_select)

To Reproduce
Steps to reproduce the behavior:

Run torch_tensorrt.compile with t5-base model as input, using fp32 precision.
Choose two fixed-size inputs of shape [1, 128] and [1, 128] and enable truncate_long_and_double with 12 GB workspace.
Pass in model keyword args to disable attention and hidden state outputs
Run inference using the compiled model on two sample inputs.

Expected behavior
Model should successfully compile with Torch-TRT. Specifically, internal device mismatch issues should either be addressed with a warning at compile time, or should otherwise not cause errors.
Environment

Torch-TensorRT Version: 1.4.0.dev0+f43be5b6
PyTorch Version: 1.14.0.dev20221114+cu116
CPU Architecture: Intel Xeon CPU
OS: Ubuntu 20.04
How you installed PyTorch: pip
Build command you used: python setup.py develop
Are you using local sources or building from archives: local
Python version: 3.8.13
CUDA version: 11.6

Additional context
The problem seems related to #1416 which was intended to address device mismatch issues of this sort. Since this case is not caught by that PR, it likely arises in a different area, for example as a result of an internal computation in a Torch block.

Root cause is related to various model-internal auxiliary tensors being initialized on CPU. Running model.cuda() and putting both input tensors on GPU resolves the compilation issue.
This model is one operator away from full TensorRT support (only requiring aten::full_like), however full compilation is not currently functional since the model outputs are in Tuple form which is not currently supported by Torch-TensorRT, and could warrant a new feature, as in #629.

Dheeraj assigned changes to George.


Search "DB Full-Text" doesn't close connections

Description
Search over many databases leads to many opened sessions and the next attempt to open another one tab with query failed with exception.
To fix the issue, user has to reopen the program.

DBeaver Version
Enterprise Edition 24.2.1-202409122322
Operating System
Windows 10
Database and driver
PostgreSQL 16
PostgreSQL JDBC Driver 42.7.4
Steps to reproduce

open one additional tab with the query: SELECT * FROM pg_stat_activity; and check the overal number of sessions
create connection to postgres with option "Show all databases" = true
open "DB Full-text" search (Ctrl + H)
choose the connection you've created (the more databases you have the faster you will reach the default limit on connection)
type anything and start searching
close search tab
re-execute the query with pg_stat_activity on your tab
you will see two opened sessions for each database

But it is expected all these sessions are closed after search was done.
If you have more than 50 databases in your postgres server, it leads to 100 postgres sessions (and the default postgres limit is 100 sessions).
When run out of available sessions, the exception will be occur on the next attempt to establish a new connection:
FATAL: sorry, too many clients already
Additional context

please also check other database engines
please also check other tabs with search

Thank you for the bug report.


Create dockershim shortcode

Creating a shortcode to put on pages that mention "Docker Engine" to alert people of the Dockershim removal and point them to the "Dockershim Removal FAQ", per Issue #32805.
I've not done one of these before, so I assume that I'll have to work with it for a bit to get it right before this PR is ready for review.

Here is a list of documentation files in content/en/docs that include the term "Docker Engine":
./concepts/services-networking/service.md
./reference/command-line-tools-reference/feature-gates.md
./reference/glossary/docker.md
./reference/node/topics-on-dockershim-and-cri-compatible-runtimes.md
./setup/production-environment/container-runtimes.md
./tasks/administer-cluster/kubeadm/adding-windows-nodes.md
./tasks/administer-cluster/migrating-from-dockershim/migrating-telemetry-and-security-agents.md
./tasks/administer-cluster/migrating-from-dockershim/migrate-dockershim-dockerd.md
./tasks/administer-cluster/migrating-from-dockershim/change-runtime-containerd.md
./tasks/network/customize-hosts-file-for-pods.md

Should any of these include the dockershim note?

Preview (see the note at the top of the page): https://deploy-preview-32826--kubernetes-io-main-staging.netlify.app/docs/setup/production-environment/container-runtimes/

/lgtm

/area web-development

@chrisnegus ideally, squash this down to 2 commits:

adding the shortcode
actually using the shortcode

That split will help localization teams

@chrisnegus is this ready for review?

@chrisnegus is this ready for review?

Sure. I'll remove the hold. I was going to also include the shortcode in a whole bunch of pages. But let's review this and get the shortcode merged. Then I'll add the shortcodes to the necessary pages in a separate PR.

@chrisnegus ideally, squash this down to 2 commits:

adding the shortcode
actually using the shortcode

That split will help localization teams

Okay.

Here you can see the shortcode showing up as a note at the top of the container-runtimes page.

/lgtm

/approve


[bug] can return deactivated user

@brianjgeiger
Steps

Go to an account on staging2 that has at least a couple of components, files, and at least one contributor. Save the account user ID for your account.
Deactivate the account on staging2 (ping Steve or Sara to expedite the process)
Go to: "https://staging2.osf.io/api/v2/users/{deactivated_user_ID}/"

Expected
It is unknown what is expected. If a user wanted to delete their account, they would deactivate their account because it states that the action is “irreversible”. If a user deactivates an account and tries to create a new account with the email that was associated with the deactivated account, it can not be done. The error message states "The email X is already associated with an account". As a user, this makes me think my deactivated account is synonymous to a project registration: a frozen version of my account where other users can access the public components I have contributed to/created up until that point and I would assume that I could reactivate my account. If that is how we define a deactivated user then from a developer's perspective, I would assume that you should be able to return a deactivated user using the API. On the other hand, if a deactivated user should not be accessible then a user might expect that the email associated to the old account be free. This presents an off boarding issue that should be considered.
Actual
The deactivated user information is retrieved- 200 status code. API Links associated with the account are also active and point to the user's respective projects (public and private), and files (which can all be downloaded). The "html" link that points to the OSF staging2 account returns a 410 (GONE) status code.

@lbanner

I would think we would want to just dis-allow this (?) i.e. retrieving data from deactivated accounts. But this note too begs an interesting question around what it means to 'deactivate' an account as opposed to a full deletion, 'If a user deactivates an account and tries to create a new account with the email that was associated with the deactivated account, it can not be done. The error message states "The email X is already associated with an account." ' @saradbowman @jlcohoon ?

Agreed, the API should definitely not return any info about a deactivated user. It should essentially be the same solution as #3551

OK good -

Whether we fully delete or deactivate, shouldn't the user be able to sign up with the same email? If it's just deactivation, they resume where they left off. If it's full deletion, they start from scratch.

I think I might be overruled in my previous comment. I'm going to make a summary of account alerts/emails when you try to perform various actions (forgot password, try to log in with a deactivated account) and then we should be able to make a more consistent decision on the alert/behavior the user sees.

this is a great idea, @jlcohoon .... we discussed with ErinM the need for this kind of documentation and this would be a good first step.

Some of this is captured here
https://docs.google.com/document/d/1bLNVsHLktmRFhW1TJ-K4kR6PQa39aQP3iFw3shgk_vk/edit#
(though
not all).
On Tue, Jul 14, 2015 at 11:31 AM, Gary Kriebel<EMAIL_ADDRESS>wrote:

this is a great idea, @jlcohoon https://github.com/jlcohoon .... we
discussed with ErinM the need for this kind of documentation and this would
be a good first step.
—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/3565#issuecomment-121282726
.

woah, that's great @saradbowman

I have a concern about this. Currently, there's no standard procedure for what happens upon deactivating an account; cases are evaluated as they come in dependent on various factors (public vs private projects, project has other contributors or does not, etc.). Happy to discuss scenarios further privately.
Is there a proposal or text written to explain what happens when a user is deactivated via the API? Is this something that needs to be run by community?
The update to the FAQ page also makes claims about account deactivation: https://osf.io/faq/#common
To my knowledge, those also have not been discussed.

Users will not be deactivated via the API. The only thing this issue explicitly will be addressing is ensuring that deactivated users are not returned by the API. This issue should be resolved soon, and further discussions about how deactivated users will be handled will likely want to move somewhere else.

Ah, okay. Thanks!


[ci] add v4096-l8-b4 and v4096-l32-b4 into default CI tests

This PR also replace original configs directory with the configgen binary. The run directory is also deprecated.

Blocked by #423


build netcdf with CMAKE under MSVC 2022

Hello
I am trying to build netcdf with cmake under MSVC 2022
first I built hdf5 with zlib
then in CMAKE i get the following error
Selecting Windows SDK version 10.0.22000.0 to target Windows 10.0.22621.
CMake Error at C:/software/hdf5/bin/hdf5-config.cmake:145 (include):
include could not find requested file:
C:/software/hdf5/cmake/hdf5-targets.cmake

Call Stack (most recent call first):
CMakeLists.txt:743 (FIND_PACKAGE)
Using HDF5 include dir: C:/software/hdf5/include;
CMake Error at CMakeLists.txt:887 (MESSAGE):
HDF5 was built without zlib.  Rebuild HDF5 with zlib.
Configuring incomplete, errors occurred!
See also "C:/software/netcdf/bin/CMakeFiles/CMakeOutput.log".
See also "C:/software/netcdf/bin/CMakeFiles/CMakeError.log".
It seems that the cmake script is not aware of the fact hdf5 was built with zlib
any help would really be appreciated
best
jac
To report a non-security related issue, please provide:

the version of the software with which you are encountering an issue
environmental information (i.e. Operating System, compiler info, java version, python version, etc.)
a description of the issue with the steps needed to reproduce it

If you have a general question about the software, please view our Suggested Support Process.

Thank you; I'll take a look at this, sorry for the delay in getting back to you! I'm able to build with hdf5-1.10 under Windows, but have not yet tried with newer HDF5 versions.

hi
thanks for your reply.
I made some progress in the meantime.
I was able to compile netcdf-c (downloaded from the github repository) together with hdf-1.12
the netcdf archive that can be downloaded is apparently incomplete
hdf-1.12 creates configurration files (hdf-targets,cmake etc) in directories that change from version to version
in version 1.12 these were placed
in  ..\build_CPack_Packages\win64\ZIP\HDF5-1.12.1-win64\share\cmake\hdf5
in version 1.14 this structure has changed and I cold not figure out where the correct files could be placed.
I then rolled back to version 1.12
best
jac

I am working to stand up a Windows/Visual Studio dev environment to test on.


fix:Expose elastic search credentials in helm deploy

Summary of Changes

Adds the ability to specify a user and password for elasticsearch when you bring your own cluster. I added this because I am using a hosted elasticsearch on AWS and I need to set my user and password to the empty string ('').
Documentation

Added some description in the values.yaml file.
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely, including a title prefix.
[x] PR includes a summary of changes.
[x] My commits follow the guidelines from "How to write a good git commit message"

lgtm, thanks, we set these two configs as well.


Update https.rst

nginx

I think this may be correct, but I think this documentation is very outdated and needs to be significantly revised. For example, Helm charts under the helm chart repo referred to as stable is since long not maintained.


Adds custom Header to CE DSL

I noticed the CE DSL was missing the ability to add custom headers.

Thank you!


bumping monochrome version to beta.3

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

@marionleborgne


MetricConstraints with example notebook

Description
Initial metric constraints API with lambda support and some examples.
General Checklist

[x] Tests added for this feature/bug
if it was a bug, test must cover it.
[x] Conform by the style guides, by using formatter
[ ] Documentation updated
[ ] (optional) Please add a label to your PR

API suggestion if you're open to it:

If you're open to reducing number of classes (fine if not), maybe something like: ConstraintsBuilder().add_custom_constraint(constraint_name, column_name, condition) [or .add_custom_constraint(constraint_name, column_name, metric, condition)but I'm not sure why we need to specify the metric here vs. in the condition, maybe necessary for implementation] for the use case in this PR? PerhapsConstraintsBuilder().add_constraint(constraint_name, column_name, condition=MaxLessThanEqualConstraint(less_than_equal=7))` is a more generic API for this where condition: Union[Callable, ConstraintHelperMethod].

Hey Bernease, we chatted a little outside of this PR but trying to loop back for visibility. My explanation was that the number of classes is not ideal and I agree with your feedback that we should simplify this part of how people will define constraints, but that this is the core API that we will build the simplifications on top of, and this will remain a valid way to define constraints (for extensibility purposes).
Murilo's test shows the kinds of helpers we could create, do you think we should just pull some of these test methods no_null and greater_than_number into the main API now?, e.g.
...
    builder.add_constraint(not_null(column_name="weight"))
    builder.add_constraint(not_null(column_name="animal"))
    builder.add_constraint(not_null(column_name="legs"))
    builder.add_constraint(greater_than_zero(column_name="weight"))
    builder.add_constraint(greater_than_zero(column_name="legs"))
    builder.add_constraint(greater_than_number(column_name="weight", number=10))
    builder.add_constraint(greater_than_number(column_name="legs", number=20))
    ...

I was thinking we could have the building blocks in initial launch and then scale out convenience methods to make this easier? Thoughts?


Update gradle version for Android Studio 2.3

Android Studio 2.3 has been released and it wants you to upgrade the gradle version used in the project.

I totally agree with the comments from @passsy .
@jonathan-caryl are you still working on it?

@StefMa I've updated the gradle-wrapper.jar as requested by @passsy and rebased.

@jonathan-caryl thank you for the contribution 👍


Update README.md

just read and approve :pray:

Edits look good to me 👍

irrelevant since theres conflicts with the pr, therefore i don't care about this anymore (yes, i'd rather have this close than to fix conflicts)


[Question] use WithValidation concern rules which rule constraint available inside cell

Versions

PHP version: 7.1
Laravel version: 5.7.28
Package version: 3.1

Description
when import using WithValidation concern, sometimes we need to build rules that rule constrain available inside row/cell.
For example, we want to check if user credit not exceed credit limit in WithValidation concern rules.

we have table below:

-------------------
User	Add Credit
-------------------
1	100
2	53
3	88

and we have rules() below:

public function rules(){
	$user = 'read user from User column';
	$creditLimitExceed = function($attribute, $value, $fail) use($user){
		$limit = $user->getCreditLimit();
		if ($value>$limit){
			$fail(trans('credit_limit_exceed_message.'));
		}
		return true;
	}
	return [
		'add_credit'=> [$creditLimitExceed],
	];
}

how to access/read $user value from row?
how to write it properly?
thx

Not sure if it would be possible the way you propose, since the rules() method doesn't get the row number. The rules() method is not used on a per row base usage, but validates in batches.
Please do note that support for custom validation rules has been added recently.

Closing due to inactivity. Please feel free to reopen if you need additional support.


[poc] middlewares: tcp: add rate limit

What does this PR do?
Add a TCP rate limiter based on the HTTP one (I couldn't find what to improve to better fit TCP). It limits
Motivation
Fixes #5999
More

[ ] Added/updated tests - will do once design is approved
[ ] Added/updated documentation - partially perhaps

Additional Notes
Previous PRs in this direction were declined, so I'm not sure what else should be done to properly adapt to TCP. This implementation seems fine, only filtering connection attempts

It's actually not mergeable, but requires a design review from maintainers to proceed. Are draft PRs reviewed?

By default, draft PRs are not reviewed.
Your PR is more draft than a mergeable PR.
Because it's a POC, the PR will be reviewed (at least partially).

any news on this feature ? is it being implemented ?

Design needs to be reviewed by maintainers, as I want to be sure the feature is correctly implemented and only then document it and fulfill all the requirements

ok, by the way is this rate limit working only on a number of session basis or can it work also on a tot. number of transmitted byte basis ?

The thing I did limits connection attempts, not amount of data transmitted