text:
[skip] Testing the OKD 4 lane

What this PR does / why we need it:
Giving the OKD 4 lane a test, not intended to be merged
Starting with some focused tests only
/hold

oh, skip worked great... not :D
/test pull-kubevirt-e2e-okd-4.1.0

/skip

/test pull-kubevirt-e2e-okd-4.1.0

first run timed out while waiting for kubevirt installation
second run while pulling images:
Trying to pull registry:5000/kubevirt/container-disk-v1alpha:devel_alt...Getting image source signatures
Copying blob sha256:910d8f35ad8e06f02edaf13bfe509c4979fa01d16ff369aea7ff842a37573e6c
Copying blob sha256:fb799b424c4f70eee899bc51c333c340d5b25506e275af13341d4a3ef3b54e36
Copying blob sha256:599c158659483586dbeaf1dcf6c58b9b8cd263080674bd696abe8baad025a3e8
Copying config sha256:b7c7ff5b515bce5e65858eab47e480306340c93eda497618d6892ee8b75ce466
Writing manifest to image destination
Storing signatures
b7c7ff5b515bce5e65858eab47e480306340c93eda497618d6892ee8b75ce466
Connection to 127.0.0.1 closed by remote host.
Makefile:107: recipe for target 'cluster-build' failed

/test pull-kubevirt-e2e-okd-4.1.0

/close
new PR: https://github.com/kubevirt/kubevirt/pull/2623


text:
Pull in latest CI cluster

What this PR does / why we need it:
Pull in latest CI cluster with

increased pod limit on os clusters to 110
updated local volume provisioner
new OKD 4.1.2 provider
audit logging enabled
some fixes and additions to gocli

For a complete list see https://github.com/kubevirt/kubevirtci/commits/master
Release note:
NONE

/lgtm
/approve
Let's see if our retester can handle this.

hm, a make generate missing.

/lgtm

/lgtm

/retest

/retest

/override pull-kubevirt-e2e-k8s-1.11.0
/override pull-kubevirt-e2e-k8s-1.13.3

/retest

/test pull-kubevirt-e2e-k8s-1.11.0
/test pull-kubevirt-e2e-k8s-1.13.3

/retest

/test pull-kubevirt-e2e-k8s-1.13.3
/test pull-kubevirt-e2e-os-3.11.0-crio
/test pull-kubevirt-e2e-os-3.11.0
/test pull-kubevirt-e2e-k8s-1.11.0

@slintes We have some issue with the local provisioner
+ grep -v Running
local-volume-provisioner-5ddd8   0/1       CrashLoopBackOff   7         14m

/hold
let's stop testing until this is resolved

@rmohr I also see several tests timing out again. Is that an issue with the CI servers (are they getting slower for some reason), or should we increase timeout again?

I don't see the cri-o issue locally with the CI image of today...
$ k get pod
NAME                             READY   STATUS    RESTARTS   AGE
...
local-volume-provisioner-wvpg7   1/1     Running   0          5h

will update to the latest cluster-up version...

/hold cancel
/lgtm

@slintes @rmohr I hope it does not some flaky issue that will appear and disappear :confused:

/override pull-kubevirt-e2e-os-3.11.0-crio
Results look good.

this test was still running, and when it failed, it overrode the override...
/override pull-kubevirt-e2e-os-3.11.0-crio

/test all

/test pull-kubevirt-e2e-k8s-1.11.0
/test pull-kubevirt-e2e-os-3.11.0
/test pull-kubevirt-e2e-os-3.11.0-multus
/test pull-kubevirt-e2e-k8s-genie-1.11.1
/test pull-kubevirt-e2e-windows2016

source: gharchive/pull-request

authors: cynepco3hahue rmohr slintes

repo: kubevirt/kubevirt

license_type: permissive

license_source: github-api


text:
Include haproxy as a dependency

haproxy will be used to loadbalance requests over the chosen masters, so
we'll need to include the haproxy docker RPM as a dependency.

So - WIP for 2 reasons..

The haproxy image isn't showing up in http://download.suse.de/ibs/Devel:/CASP:/Head:/ControllerNode/standard/x86_64/ - I'm not sure why

Adding this a dependency of the container manifests feels wrong, however, that's where ALL our container images have dependencys...

Unmarking as WIP, the haproxy container RPM now exists, and this is where we specify these for better/worse... If we want to change change that, it's unrelated to adding haproxy.

repo: kubic-project/caasp-container-manifests


text:
Feature/bump core dependencies to angular 8

This PR contains the following changes:

Initial @angular/cli:^8.0.3 update by @DenisaSurdu

Squashed and signed commits into current

Bumping project dependencies to @angular/cli:^2.4.0 compatible versions
Bumping peer dependencies to @angular/cli:^2.4.0 compatible versions
Updating angular.json with compatible tags

- "architect" + "targets"

Because of the scripts-tag in package.json it seems like the primary package manager is yarn. Therefore I reconfigured the angular.json and removed the package-lock.json as well.

to: @kuhnroyal
cc: @DenisaSurdu
fixes: #69

Good to merge? Angular 9 is almost upon us

* polyfills.ts is out of date

no wonder, if nobody merges this within years :D @intellix

Hello, any progress on this issue? What prevents the merge?

authors: Roen00 bbortt intellix

repo: kuhnroyal/mat-datetimepicker

license: MIT


text:
Have you ever gotten this to work on Windows with MSVC?

MSVC seems to not like many of the statements. I'm no c++ expert (which is why i was trying to use this header file obviously). Wonder if you know how to fix?
c:\users\user\documents\github\mexcpp\mexcpp.h(52) : error C2079: 'mexcpp::SigintHandler::sa' uses undefined struct 'mexcpp::sigaction'
c:\users\user\documents\github\mexcpp\mexcpp.h(52) : error C2079: 'mexcpp::SigintHandler::osa' uses undefined struct 'mexcpp::sigaction'
c:\users\user\documents\github\mexcpp\mexcpp.h(63) : error C2228: left of '.sa_handler' must have class/struct/union
type is 'int'
c:\users\user\documents\github\mexcpp\mexcpp.h(64) : error C2514: 'mexcpp::sigaction' : class has no constructors
c:\users\user\documents\github\mexcpp\mexcpp.h(52) : see declaration of 'mexcpp::sigaction'
c:\users\user\documents\github\mexcpp\mexcpp.h(68) : error C2514: 'mexcpp::sigaction' : class has no constructors
c:\users\user\documents\github\mexcpp\mexcpp.h(52) : see declaration of 'mexcpp::sigaction'
c:\users\user\documents\github\mexcpp\mexcpp.h(363) : error C2057: expected constant expression
c:\users\user\documents\github\mexcpp\mexcpp.h(363) : error C2466: cannot allocate an array of constant size 0
c:\users\user\documents\github\mexcpp\mexcpp.h(363) : error C2133: 'fns' : unknown size

Hi,
I have not tried compiling this with MSVC. I do use C++11 features and signal handling to catch Ctrl-C which only works on Unix.
If you are still interested in getting this to work, you could

Make sure your compiler is configured to compile C++11 correctly;
Comment out the code regarding signal handling for Ctrl-C

Best of luck! Let me know what works so we can post it here so others can refer to it later.

Kui

On Sep 10, 2015, at 10:56, pst2154 notifications@github.com wrote:
MSVC seems to not like many of the statements. I'm no c++ expert (which is why i was trying to use this header file obviously). Wonder if you know how to fix?
c:\users\user\documents\github\mexcpp\mexcpp.h(52) : error C2079: 'mexcpp::SigintHandler::sa' uses undefined struct 'mexcpp::sigaction'
c:\users\user\documents\github\mexcpp\mexcpp.h(52) : error C2079: 'mexcpp::SigintHandler::osa' uses undefined struct 'mexcpp::sigaction'
c:\users\user\documents\github\mexcpp\mexcpp.h(63) : error C2228: left of '.sa_handler' must have class/struct/union
type is 'int'
c:\users\user\documents\github\mexcpp\mexcpp.h(64) : error C2514: 'mexcpp::sigaction' : class has no constructors
c:\users\user\documents\github\mexcpp\mexcpp.h(52) : see declaration of 'mexcpp::sigaction'
c:\users\user\documents\github\mexcpp\mexcpp.h(68) : error C2514: 'mexcpp::sigaction' : class has no constructors
c:\users\user\documents\github\mexcpp\mexcpp.h(52) : see declaration of 'mexcpp::sigaction'
c:\users\user\documents\github\mexcpp\mexcpp.h(363) : error C2057: expected constant expression
c:\users\user\documents\github\mexcpp\mexcpp.h(363) : error C2466: cannot allocate an array of constant size 0
c:\users\user\documents\github\mexcpp\mexcpp.h(363) : error C2133: 'fns' : unknown size
—
Reply to this email directly or view it on GitHub.

@kuitang Hello, I met same question. And I use 'Microsoft Visual C++ 2013 Professional' compiler, but it does not work.
Error lists are as follows:
Building with 'Microsoft Visual C++ 2013 Professional'.
Error using mex
fv_cache.cc
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(47) : error C2079: 'context::act' uses undefined
struct 'sigaction'
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(47) : error C2079: 'context::matlab_act' uses
undefined struct 'sigaction'
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(69) : error C2514: 'sigaction' : class has no
constructors
        E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(47) : see declaration of 'sigaction'
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(253) : warning C4800: 'double' : forcing value
to bool 'true' or 'false' (performance warning)
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(254) : warning C4800: 'double' : forcing value
to bool 'true' or 'false' (performance warning)
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(264) : warning C4244: 'argument' : conversion
from '__int64' to 'double', possible loss of data
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(309) : warning C4244: 'initializing' :
conversion from '__int64' to 'int', possible loss of data
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(456) : warning C4244: '=' : conversion from
'size_t' to 'double', possible loss of data
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(612) : warning C4244: 'argument' : conversion
from '__int64' to 'double', possible loss of data
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(678) : warning C4267: 'initializing' :
conversion from 'size_t' to 'int', possible loss of data
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(783) : error C2228: left of '.sa_mask' must have
class/struct/union
        type is 'int'
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(783) : error C3861: 'sigemptyset': identifier
not found
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(784) : error C2228: left of '.sa_handler' must
have class/struct/union
        type is 'int'
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(785) : error C2228: left of '.sa_flags' must
have class/struct/union
        type is 'int'
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(786) : error C2514: 'sigaction' : class has no
constructors
        E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(47) : see declaration of 'sigaction'
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(799) : error C2514: 'sigaction' : class has no
constructors
        E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(47) : see declaration of 'sigaction'

VS2013 support C++11, but why does not it work?

Hi Xuson,
The errors you below are related to the SigintHandler class. It is code to catch sigint (Ctrl-C) on Unix machines. This enables interrupting the middle of a computation. This code requires compiling on a Unix machine.
However, this class is optional to use, and provided only as a convenience.
To compile on a Windows machine, you can simply comment out this class definition as well as the #include <signal.h> line.

Kui

On May 2, 2016, at 10:37, Xusong Chen notifications@github.com wrote:
@kuitang https://github.com/kuitang Hello, I met same question. And I use 'Microsoft Visual C++ 2013 Professional' compiler, but it does not work.
Error lists are as follows:
Building with 'Microsoft Visual C++ 2013 Professional'.
Error using mex
fv_cache.cc
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(47) : error C2079: 'context::act' uses undefined
struct 'sigaction'
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(47) : error C2079: 'context::matlab_act' uses
undefined struct 'sigaction'
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(69) : error C2514: 'sigaction' : class has no
constructors
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(47) : see declaration of 'sigaction'
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(253) : warning C4800: 'double' : forcing value
to bool 'true' or 'false' (performance warning)
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(254) : warning C4800: 'double' : forcing value
to bool 'true' or 'false' (performance warning)
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(264) : warning C4244: 'argument' : conversion
from '__int64' to 'double', possible loss of data
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(309) : warning C4244: 'initializing' :
conversion from '__int64' to 'int', possible loss of data
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(456) : warning C4244: '=' : conversion from
'size_t' to 'double', possible loss of data
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(612) : warning C4244: 'argument' : conversion
from '__int64' to 'double', possible loss of data
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(678) : warning C4267: 'initializing' :
conversion from 'size_t' to 'int', possible loss of data
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(783) : error C2228: left of '.sa_mask' must have
class/struct/union
type is 'int'
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(783) : error C3861: 'sigemptyset': identifier
not found
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(784) : error C2228: left of '.sa_handler' must
have class/struct/union
type is 'int'
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(785) : error C2228: left of '.sa_flags' must
have class/struct/union
type is 'int'
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(786) : error C2514: 'sigaction' : class has no
constructors
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(47) : see declaration of 'sigaction'
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(799) : error C2514: 'sigaction' : class has no
constructors
E:\Projects\160325pedestrain_detection\code\voc-release5\fv_cache\fv_cache.cc(47) : see declaration of 'sigaction'
VS2013 support C++11, but why does not it work?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/kuitang/mexcpp/issues/3#issuecomment-216252760

@kuitang Thanks ! :+1:

source: gharchive/issue

authors: Ocxs kuitang pst2154

repo: kuitang/mexcpp

license: mit

license_source: bigquery


text:
chore: clean up DPP list in service view

feat(services): adds copy button to address cells
fix: data plane list toolbar items misalignment
fix(services): data plane proxy list layout
Fixes the DPP list within the service detail view for external services missing its wrapping KCard.
Signed-off-by: Philipp Rudloff philipp.rudloff@konghq.com
Before:

After:

@johncowen There were a couple of influences breaking this layout. One was the missing KCard. In its absence, different font properties applied sizing the various toolbar items out of proportion. After fixing that, the DPP list in the service detail view and the gateway list view looked mostly fine, but the padding: 12px rule still caused the items to be of a different height because KSelect doesn’t grow in a stretch-aligned flex container. Removing the padding rule fixes this on our end.

Right but my question was, where did it get into this state? It seemed to me to be #1244, which I'm pretty sure it was, I just wanted to see if I could get confirmation of that in case I was wrong, no problem! Oh I'd meant to approve here as well 1 sec.

Right but my question was, where did it get into this state?

It got into this state through the removal of KCard and the addition of padding: 12px on the KFilter input. https://github.com/kumahq/kuma-gui/pull/1244 was pretty much unrelated to this.

authors: johncowen kleinfreund

repo: kumahq/kuma-gui


text:
Fix unknown type name

Hello. Thank you for the merge my PR before.
This is my correction mistake 🙇
Thank you.

Thank you and sorry to be late.


text:
Need to reauthorize every hour

MacLexa uses the implicit authorization method.
This requires the user to reauthorize every hour.
Need to switch over to using the companion app method of Authorization, without creating a companion app

This is a cool app - but yea, a bit annoying to have to constantly re-auth. Hope this gets implemented!


text:
[New game]: PRONTO 👾

🎮 Game Request
Game logic and basic description

A very simple reaction game about not getting off the line.

Point down the features
Game points
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @0xabdulkhalid !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

repo: kunjgit/GameZone


text:
[Enhancement]: Master Typing Score Keeper

Do you want to have the enhancement of existing game ? 😀 Describe yourself..
The game does not provide the user with data on the number of keys that were correctly or incorrectly pressed. We can also give the user information about the key that was pressed incorrectly the most number of times at the end of game.
Describe the solution you'd like
Solution steps

Changes in CSS and HTML to add new counters to keep track of the scores for correct and incorrect key presses.
Changes in JavaScript to update the counters and turn the keys red that were pressed incorrectly the most number of times at the end of the game.

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @KrishnaKalra !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @KrishnaKalra ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

authors: KrishnaKalra kunjgit


text:
Text Color has Changed When Theme is Light

[Bug]: Text is not clearly visible when theme has been changed [4329]
I have change the text color when the theme has been changed
Fixes # <4329>

Mark the task you have completed ✅

[X ] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[ X] I have performed a self-review of my own code or work.
[X ] I have commented my code, particularly in hard-to-understand areas.
[ X] My changes generates no new warnings.
[ X] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[X ] I have added screenshot for website preview in assets/images
[ ] I have added entries for my game in main README.md
[ ] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[ ] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @Heena31 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Hey @Heena31,
Please make sure to link the relevant issue using the appropriate syntax, such as "#issueNumber" 👀.
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗

Thank you  @Heena31 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


text:
Update CI / CD to Quodana

Description
The objective of this PR is to get CI / CD working again w/ quodana.

We are using SonarCloud for this open source project.

authors: hugo-goncalves-kununu tdashton

repo: kununu/collections


text:
Insufficient funds for gas EL geth

I changed the prefund key from the mnemonic applied in network_params.yaml but get an error Insufficient funds for gas EL geth and beaconchain explorer full stop at epoch 3 after being applied, the execution used geth and erigon consensus lighthouse.
WARN [05-05|10:58:39.852] Served eth_sendRawTransaction            conn=172.16.0.20:51168 reqid=229046 duration="318.175µs" err="insufficient funds for gas * price + value: balance 0, tx cost 32020000000000000000, overshot 32020000000000000000"
WARN [05-05|10:58:39.862] Served eth_sendRawTransaction            conn=172.16.0.20:51168 reqid=229047 duration="379.399µs" err="insufficient funds for gas * price + value: balance 0, tx cost 32020000000000000000, overshot 32020000000000000000"
WARN [05-05|10:58:39.871] Served eth_sendRawTransaction            conn=172.16.0.20:51168 reqid=229048 duration="387.445µs" err="insufficient funds for gas * price + value: balance 0, tx cost 32020000000000000000, overshot 32020000000000000000"
WARN [05-05|10:58:39.880] Served eth_sendRawTransaction            conn=172.16.0.20:51168 reqid=229049 duration="330.678µs" err="insufficient funds for gas * price + value: balance 0, tx cost 32020000000000000000, overshot 32020000000000000000"
WARN [05-05|10:58:39.889] Served eth_sendRawTransaction            conn=172.16.0.20:51168 reqid=229050 duration="316.813µs" err="insufficient funds for gas * price + value: balance 0, tx cost 32020000000000000000, overshot 32020000000000000000"
WARN [05-05|10:58:39.897] Served eth_sendRawTransaction            conn=172.16.0.20:51168 reqid=229051 duration="330.558µs" err="insufficient funds for gas * price + value: balance 0, tx cost 32020000000000000000, overshot 32020000000000000000"

VM port is open, but blobscan-api is ECONNREFUSED.
blobscan-api logs:
PUT /blockchain-sync-state 200 - - 3.640 ms
PUT /blockchain-sync-state 200 - - 3.662 ms
PUT /blockchain-sync-state 200 - - 3.120 ms
PUT /blockchain-sync-state 200 - - 2.886 ms
PUT /blockchain-sync-state 200 - - 3.693 ms
PUT /blockchain-sync-state 200 - - 3.638 ms
Error: connect ECONNREFUSED 127.0.0.1:6379
    at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1605:16) {
  errno: -111,
  code: 'ECONNREFUSED',
  syscall: 'connect',
  address: '127.0.0.1',
  port: 6379
}
Error: connect ECONNREFUSED 127.0.0.1:6379
    at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1605:16) {
  errno: -111,
  code: 'ECONNREFUSED',
  syscall: 'connect',
  address: '127.0.0.1',
  port: 6379
}
PUT /blockchain-sync-state 200 - - 3.112 ms
2024-05-05 10:04:16:416 error: [Stats Syncer]: Redis connection error: Error: connect ECONNREFUSED 127.0.0.1:6379
2024-05-05 10:04:16:416 error: [Stats Syncer] Updater daily-stats-syncer: queue error: Error: connect ECONNREFUSED 127.0.0.1:6379

Can you paste your config.yaml so I can try to reproduce it?

authors: avenbreaks barnabasbusa

repo: kurtosis-tech/ethereum-package


text:
Kurtosis service start not working

What's your CLI version?
0.85.1
Description & steps to reproduce
Related to #1684.
When trying to start stopped service kurtosis is throwing an error that it's already started:
kurtosis service start broadcaster el-3-geth-lighthouse
INFO[2023-11-03T12:09:43Z] Starting service 'el-3-geth-lighthouse'
Error:  An error occurred running command 'start'
  Caused by: An error occurred calling the run function for command 'start'
  Caused by: An error occurred starting service 'el-3-geth-lighthouse' from enclave 'broadcaster'
  Caused by: An unexpected error occurred on Starlark for starting service
  Caused by: An error occurred executing instruction (number 1) at DEFAULT_PACKAGE_ID_FOR_SCRIPT[3:20]:
  start_service(name="el-3-geth-lighthouse")
  Caused by: Failed starting service with unexpected error
  Caused by: An error occurred while starting services
  Caused by: Service 'el-3-geth-lighthouse' is already started
root@localhost ➜  ~ kurtosis enclave inspect broadcaster
Name:            broadcaster
UUID:            4782664e9d1a
Status:          RUNNING
Creation Time:   Fri, 03 Nov 2023 02:04:26 UTC

========================================= Files Artifacts =========================================
UUID           Name
8ff23f14105a   2-lighthouse-nethermind-0-63
a48563c349b8   cl-genesis-data
fb8c710fa69d   dora-config
bba721d92500   el-genesis-data
8d1b77df6821   floral-rose
032fdd88b0fa   genesis-generation-config-cl
b2ccb1ec751e   genesis-generation-config-el
63f8586e8d64   geth-prefunded-keys
0570ed90459d   grafana-config
d55b1fb7e8b3   grafana-dashboards
e51df3a8db42   prometheus-config
bc3495029b2d   prysm-password
dbe06f9a9f83   silent-pine
0d01e37f242d   task-1a69c1c8-825b-4431-9e83-d4f7644d7fb5-python-script
7f68aaa6478f   validator-ranges

========================================== User Services ==========================================
UUID           Name                                             Ports                                                  Status
880887055a8e   beacon-metrics-gazer                             http: 8080/tcp -> http://127.0.0.1:35433/               RUNNING
688ca5edef7b   blob-spammer                                     <none>                                                 RUNNING
0c9563f87d68   broadcaster                                      http: 8080/tcp -> http://127.0.0.1:35427/               RUNNING
9602ce28c6bc   cl-1-lighthouse-geth                             http: 4000/tcp -> http://127.0.0.1:35418/               RUNNING
                                                                metrics: 5054/tcp -> http://127.0.0.1:35417/
                                                                tcp-discovery: 9000/tcp -> 127.0.0.1:35416
                                                                udp-discovery: 9000/udp -> 127.0.0.1:33197
2e358d8f595d   cl-2-lighthouse-nethermind                       http: 4000/tcp -> http://127.0.0.1:35421/               RUNNING
                                                                metrics: 5054/tcp -> http://127.0.0.1:35420/
                                                                tcp-discovery: 9000/tcp -> 127.0.0.1:35419
                                                                udp-discovery: 9000/udp -> 127.0.0.1:33198
4c54467926af   cl-2-lighthouse-nethermind-validator             http: 5042/tcp -> 127.0.0.1:35423                      RUNNING
                                                                metrics: 5064/tcp -> http://127.0.0.1:35422/
6ad7811babe1   cl-3-lighthouse-geth                             http: 4000/tcp -> http://127.0.0.1:35426/               RUNNING
                                                                metrics: 5054/tcp -> http://127.0.0.1:35425/
                                                                tcp-discovery: 9000/tcp -> 127.0.0.1:35424
                                                                udp-discovery: 9000/udp -> 127.0.0.1:33199
6836f79cd0e0   dora                                             http: 8080/tcp -> http://127.0.0.1:35434/               RUNNING
100446621efb   eip4788-contract-deployment                      <none>                                                 RUNNING
9ff5585a3f75   el-1-geth-lighthouse                             engine-rpc: 8551/tcp -> 127.0.0.1:35403                RUNNING
                                                                metrics: 9001/tcp -> 127.0.0.1:35402
                                                                rpc: 8545/tcp -> 127.0.0.1:35405
                                                                tcp-discovery: 30303/tcp -> 127.0.0.1:35401
                                                                udp-discovery: 30303/udp -> 127.0.0.1:33194
                                                                ws: 8546/tcp -> 127.0.0.1:35404
ecfbdc9af14d   el-2-nethermind-lighthouse                       engine-rpc: 8551/tcp -> 127.0.0.1:35408                RUNNING
                                                                metrics: 9001/tcp -> 127.0.0.1:35407
                                                                rpc: 8545/tcp -> 127.0.0.1:35410
                                                                tcp-discovery: 30303/tcp -> 127.0.0.1:35406
                                                                udp-discovery: 30303/udp -> 127.0.0.1:33195
                                                                ws: 8546/tcp -> 127.0.0.1:35409
77baf4019285   el-3-geth-lighthouse                             engine-rpc: 8551/tcp                                   STOPPED
                                                                metrics: 9001/tcp
                                                                rpc: 8545/tcp
                                                                tcp-discovery: 30303/tcp
                                                                udp-discovery: 30303/udp
                                                                ws: 8546/tcp
f01d926ea97e   grafana                                          http: 3000/tcp -> http://127.0.0.1:35436/               RUNNING
2864d99c2688   mev-boost-1                                      api: 18550/tcp -> 127.0.0.1:35432                      RUNNING
3bca6f8d6319   mev-custom-flood                                 <none>                                                 RUNNING
259e96f674ce   mev-relay-api                                    api: 9062/tcp -> 127.0.0.1:35430                       RUNNING
8e47568a397b   mev-relay-housekeeper                            <none>                                                 RUNNING
66a74b3d232d   mev-relay-website                                api: 9060/tcp -> http://127.0.0.1:35431/                RUNNING
71dd9b58fad6   postgres                                         postgresql: 5432/tcp -> postgresql://127.0.0.1:35429   RUNNING
0e909b23b82d   prelaunch-data-generator-cl-genesis-data         <none>                                                 RUNNING
7d0c9763f371   prelaunch-data-generator-cl-validator-keystore   <none>                                                 RUNNING
32d826f95eb4   prelaunch-data-generator-el-genesis-data         <none>                                                 RUNNING
5e38798d612f   prometheus                                       http: 9090/tcp -> http://127.0.0.1:35435/               RUNNING
8bc3b9bcbcd7   redis                                            client: 6379/tcp -> 127.0.0.1:35428                    RUNNING
76d0faf26d69   task-1a69c1c8-825b-4431-9e83-d4f7644d7fb5        <none>                                                 RUNNING
a107c00dfa8c   transaction-spammer                              <none>                                                 RUNNING

Desired behavior
The service should start
What is the severity of this bug?
Painful; this is causing significant friction in my workflow.
What area of the product does this pertain to?
CLI: the Command Line Interface

Hey @piwonskp thanks for the bug report. I had a short fix in mind but that doesn't really fix the problem; so I will have to spend more cycles on this for a fix
In the interim I recommend using docker ps -a to get the container id of the dead container & you can start it via docker start containerId

repo: kurtosis-tech/kurtosis


text:
Support for multiline text

Sorry, no pull request.
in RectangleLayer.swift :
            let messageWidth = messageString.size(attributes: [NSFontAttributeName : font]).width
            let messageHeight = messageString.size(attributes: [NSFontAttributeName : font]).height
            path = bezierPathWithStatus(width: messageWidth).cgPath
            let statusLayer = StatusTitleLayer(message: message, 				//TODO: - Perhapse need to extend font in settings
                                               frame: CGRect(
                                                x: messageWidth > rectSide ? -rectSide : 0,
                                                y: 0.0,
                                                width: messageWidth > rectSide ? messageWidth : rectSide,
                                                height: max(25,messageHeight)))

I added messageHeight and changed the height param in the rectangle frame

Tnx for feedback, fixed this issue.  Check it out!

Also you are able to create pull requests by forking my repo.
https://gist.github.com/Chaser324/ce0505fbed06b947d962

Yes I know, I just took a local copy because I needed to change some more things for my project :)
FYI this change isn't 100% correct I noticed. Somewhere in your code you still have a "30" hardcoded for the height of the text part. Don't remember anymore where it was however.

It only influences the vertical centering of the view if I remember correctly. So it won't take the 2nd line into account when calculating the vertical position. Not really a big deal.

authors: ir-fuel kuznetsovVladislav

repo: kuznetsovVladislav/KVSpinnerView


text:
Added zoom to orbit camera

As above, some small changes in the OrbitControls code to add a zoom function to the camera using the mouse wheel:
In the update function, I added a check to input.get_mouse_wheel_movements, then implemented some code to fake the old mouse position as the new one (ie no movement) if the only update was to mouse wheel, then changed scale to 1 + (sum of mouse wheel movements)/1000
It works nicely on the obj example. I'm sure it could be more elegant, but in the meantime, it works...

@bobgates any interest in finishing this PR?

@vitvakatu would you be willing to take that PR and finish it?

What needs to be done before the PR is complete? Just make the changes from the comments?

@alteous yes, there is not much left to do

I'm a little bit busy at the moment, it seems that @alteous want to take care of it.
@kvark sorry, I hope I'll be back soon
Sent from my HONOR DLI-TL20 using FastHub

Sure, I'll take care of it.

Replaced by #73

authors: alteous bobgates kvark vitvakatu

repo: kvark/three-rs


text:
Error when running "Install colmap" in colab

I try to preprocess by my own data but got this issue:
/content/LLFF
Need to run COLMAP
Traceback (most recent call last):
File "imgs2poses.py", line 18, in
gen_poses(args.scenedir, args.match_type)
File "/content/LLFF/llff/poses/pose_utils.py", line 268, in gen_poses
run_colmap(basedir, match_type)
File "/content/LLFF/llff/poses/colmap_wrapper.py", line 35, in run_colmap
feat_output = ( subprocess.check_output(feature_extractor_args, universal_newlines=True) )
File "/usr/lib/python3.7/subprocess.py", line 411, in check_output
**kwargs).stdout
File "/usr/lib/python3.7/subprocess.py", line 488, in run
with Popen(*popenargs, **kwargs) as process:
File "/usr/lib/python3.7/subprocess.py", line 800, in init
restore_signals, start_new_session)
File "/usr/lib/python3.7/subprocess.py", line 1551, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'colmap': 'colmap'

I had the same issue

I think this issue is the same as https://github.com/kwea123/nerf_pl/issues/147


text:
Trace View

Experimental VisPy code here: https://github.com/kwikteam/experimental/tree/master/plot

Duplicate of #16.

repo: kwikteam/phy


text:
Add Singleton Pattern

This project demonstrates the implementation of the Singleton design pattern in Java using two different approaches:
Singleton with Double Check Locking
Singleton using Initialization-on-Demand Holder

OK

repo: kyda-code/learning-java-design-patterns


text:
Update dependencies

Description
Changes proposed in this pull request:

Update compass console dependencies

/retest

repo: kyma-incubator/compass-console


text:
Extensibility UI

Description
Changes proposed in this pull request:

move extensions to a separate UI and revert config maps to their original form

Related issue(s)

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

authors: Lyczeq dariadomagala mrCherry97 qbalukom

repo: kyma-project/busola


text:
Remove Compass references from Provisioner component

Description
Changes proposed in this pull request:

references removed from component, components tests, docs and charts

Related issue(s)
Issue: https://github.com/kyma-project/control-plane/issues/38

/test pre-control-plane-tests-e2e-provisioning

/meow

/test pre-master-control-plane-gke-provisioner-integration

@franpog859 Yes, it failed on deprovisioning timeout, but the configs on provisioned runtimes were verified successfully.

/test pre-master-control-plane-gke-provisioner-integration

@majakurcius , I think we need your approve here 😅

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: rafalpotempa:white_check_mark: kjaksik:white_check_mark: Maladie:x: JackCheng01You have signed the CLA already but the status is still pending? Let us recheck it.

/test pre-master-control-plane-gke-integration

/retest

/test pre-master-control-plane-gke-provisioner-integration

/test pre-master-control-plane-gke-integration
/test pre-master-control-plane-gke-provisioner-integration

/test pre-master-control-plane-gke-provisioner-integration

/test pre-master-control-plane-gke-provisioner-integration

authors: CLAassistant Maladie franpog859 majakurcius rafalpotempa

repo: kyma-project/control-plane


text:
Set global configs for inventory input

Description
Changes proposed in this pull request:

adds shootDomain as global.domainName configuration for every component in the inventory input

/test pre-main-control-plane-gke-integration

authors: adamwalach piotrmiskiewicz


text:
Support different CNI configs

Description
Istio Operator should provide a possibility to configure Istio CNI, because different clusters have a different configurations.
This is an example difference between k3d and GCP cluster:
<       cniBinDir: /opt/cni/bin
<       cniConfDir: /etc/cni/net.d
---
>       cniBinDir: /bin
>       cniConfDir: /var/lib/rancher/k3s/agent/etc/cni/net.d

Reasons
Istio Operator should support different types of clusters.

DoD:

[ ] provide documentation
[ ] test on SKR

closing cause of https://github.com/kyma-project/istio/issues/95

authors: cnvergence strekm

repo: kyma-project/istio


text:
update to alpine 3.14.2

Description
Changes proposed in this pull request:

updated several images to newest alpine version
...
...

Related issue(s)

/retest

/test pre-main-tpi-loki

repo: kyma-project/third-party-images


text:
GH action release workflow

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)
#165

/test all

repo: kyma-project/warden


text:
🛑 senoshaidodelasmanos.love is down

In 95d4ed7, senoshaidodelasmanos.love (https://senoshaidodelasmanos.love) was down:

HTTP code: 0
Response time: 0 ms

Resolved: senoshaidodelasmanos.love is back up in 8a6787b after 566 days, 0 hours, 50 minutes.

repo: kyryl-bogach/upptime


text:
🛑 senoshaidodelasmanos.love is down

In b7b50a8, senoshaidodelasmanos.love (https://senoshaidodelasmanos.love) was down:

HTTP code: 0
Response time: 0 ms

Resolved: senoshaidodelasmanos.love is back up in ca98710 after 12 minutes.


text:
Error running migrations with Vite

This originally came up in a Sveltekit project but reduced to just Vite, in a Vite project (ESM), running a migration script in Linux results in
TypeError [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension ".ts" for xxx.ts
    at new NodeError (node:internal/errors:387:5)
    at Object.getFileProtocolModuleFormat [as file:] (node:internal/modules/esm/get_format:75:11)
    at defaultGetFormat (node:internal/modules/esm/get_format:117:38)
    at defaultLoad (node:internal/modules/esm/load:81:20)
    at nextLoad (node:internal/modules/esm/loader:163:28)
    at ESMLoader.load (node:internal/modules/esm/loader:601:26)
    at ESMLoader.moduleProvider (node:internal/modules/esm/loader:457:22)
    at new ModuleJob (node:internal/modules/esm/module_job:63:26)
    at ESMLoader.#createModuleJob (node:internal/modules/esm/loader:476:17)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:434:34) {
  code: 'ERR_UNKNOWN_FILE_EXTENSION'

The path being used in a module scope is migrationFolder: new URL('./migrations', import.meta.url).pathname
On Windows the path has to be relative for migrationFolder due to file:// shenanigans. A relative path to src/migrations results instead in:
TypeError [ERR_INVALID_MODULE_SPECIFIER]: Invalid module "src\migrations\0001_initial.ts" is not a valid package name imported from C:\GitHub\MyStuff\vite-repo\node_modules\.pnpm\kysely@0.23.3\node_modules\kysely\dist\esm\migration\file-migration-provider.js
    at new NodeError (node:internal/errors:387:5)
    at parsePackageName (node:internal/modules/esm/resolve:777:11)
    at packageResolve (node:internal/modules/esm/resolve:800:5)
    at moduleResolve (node:internal/modules/esm/resolve:901:20)
    at defaultResolve (node:internal/modules/esm/resolve:1115:11)
    at nextResolve (node:internal/modules/esm/loader:163:28)
    at ESMLoader.resolve (node:internal/modules/esm/loader:841:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:424:18)
    at ESMLoader.import (node:internal/modules/esm/loader:525:22)
    at importModuleDynamically (node:internal/modules/esm/translators:110:35) {
  code: 'ERR_INVALID_MODULE_SPECIFIER'

Repro
Vite only: https://codesandbox.io/p/github/drewbitt/vite-kysely-repro/draft/intelligent-meninsky
Sveltekit using Vite by @Terbau: https://github.com/Terbau/kysely-sveltekit-migration-error-recreation
There was also related issue #254

Load the transpiled migrations, not the typescript ones. Since you're using vite, you probably need to implement your own migration provider instead of using a one that loads modules dynamically. See the MigrationProvider interface.

I think this may have more to do with the package being an ESM package than vite. I am experiencing it with a fairly straightforward node.js scenario. I am running the transpiled .js code.
My package is set up with { "type": "module" } in package.json and { "module": "NodeNext","moduleResolution": "NodeNext" } in tsconfig.json.
Seems to be that the loader isn't finding the path, or for ESM, the "URL". This is the alternate loader I wrote to make it work:
class ESMFileMigrationProvider implements MigrationProvider {
    constructor(private relativePath: string) { }

    async getMigrations(): Promise<Record<string, Migration>> {
        const migrations: Record<string, Migration> = { };
        const __dirname = url.fileURLToPath(new URL(".", import.meta.url));
        const resolvedPath = path.resolve(__dirname, this.relativePath);
        const files = await fs.readdir(resolvedPath);

        for (const fileName of files) {
            if (!fileName.endsWith(".js")) {
                continue;
            }

            const importPath = path.join(this.relativePath, fileName).replaceAll("\\", "/");
            const migration = await import(importPath);
            const migrationKey = fileName.substring(0, fileName.lastIndexOf("."));

            migrations[migrationKey] = migration;
        }

        return migrations;
    }
}

Leaving this here in case it might help someone.
For people using SvelteKit with default config I had success writing a custom migrator using the following to import the migrations:
const migrations: Record<string, Migration> = import.meta.glob('./migrations/**.ts', {
	eager: true,
})

@koskimas
The ESMFileMigrationProvider works as expected (with a small fix), is there a way to add a check for being in a ESM module?
Got a similar error trying to run a ESM environment (in Vitest) which included running migrations.

Load the transpiled migrations, not the typescript ones. Since you're using vite, you probably need to implement your own migration provider instead of using a one that loads modules dynamically. See the MigrationProvider interface.

I had no issues running typescript migrations directly with an adjusted FileMigrationProvider, similar to the one provided by @Nick-Riggs (by modifying the condition to continue unless the .ts ending is present).
Is there any specific reason why we should avoid using to typescript migration files directly and refer only to transpiled ones?

To add to Nick's solution:
make sure to import
import { fileURLToPath } from "url";

and I found it wasn't working for relative filepaths for me until I changed the migration import line to include a ./:
const migration = await import(`./${importPath}`);

Try https://github.com/kysely-org/kysely-ctl .

Here's a simplified version - requires ts-node to be installed:
import { promises as fs } from "node:fs";
import path from "node:path";
import type { Migration, MigrationProvider } from "kysely";
import ts from "ts-node";

ts.register({
  transpileOnly: true,
});

export class TypeScriptFileMigrationProvider implements MigrationProvider {
  constructor(private absolutePath: string) {}

  async getMigrations(): Promise<Record<string, Migration>> {
    const migrations: Record<string, Migration> = {};
    const files = await fs.readdir(this.absolutePath);

    for (const fileName of files) {
      if (!fileName.endsWith(".ts")) {
        continue;
      }

      const importPath = path.join(this.absolutePath, fileName).replaceAll("\\", "/");
      const { up, down } = await import(importPath);
      const migrationKey = fileName.substring(0, fileName.lastIndexOf("."));

      migrations[migrationKey] = { up, down };
    }

    return migrations;
  }
}

The absolutePath input should be the full path to the migrations folder.

authors: Jhappy77 Nick-Riggs drewbitt igalklebanov jericirenej koskimas luixo theogravity zerosym

repo: kysely-org/kysely


text:
Writing Unit Test

This is NOT an issue but more of crying for help
We are using Kysely for an enterprise API using AWS Lambda and AWS Aurora (Postgres). One of the requirement is having unit test coverage of at least 80%.
Kysely is AWESOME, but i have been struggling with writing unit tests. (I'm new to Typescript btw). These unit test will run on jenkins, so no Postgres database for us to connect to. I figured that I might need to mock certain calls but don't know how.
One of my functions look like this:
const dialect = new PostgresDialect({ pool: new Pool(databaseConfig)})
const DB = new Kysely<Database>({ dialect })
const results = await DB.withSchema("database_schema")
    .selectFrom("table")
    .where('someId', '=', queryParamSomeId)
    .where('anotherId', '=', queryParamAnotherId)
    .selectAll()
    .execute()

Does anyone have any example that I can learn from?

Hey 👋
Ideally, you'd want to do integration tests against real engines - by spinning up a docker image. See Kysely's node.js tests for reference.

Given something like:
export const db = new Kysely<DB>({
  dialect: new PostgresDialect({  pool: new Pool()  }),
  plugins: [new CamelCasePlugin()],
});

export const fetchAccount = () => db.selectFrom("account").executeTakeFirst()

I'm able to unit test the function and check that the query is correct
const mocks = vi.hoisted(() => ({ query: vi.fn() }));

vi.mock("@/db", () => ({
  __esModule: true,
  ownerDb: new Kysely<DB>({
    dialect: new PostgresClientDialect({  client: mocks  }),
    plugins: [new CamelCasePlugin()],
  }),
}));

it("query", async () => {
    mocks.query.mockResolvedValue({ rows: [{ .... }] });
    await fetchAccount();
    expect(mocks.query.mock.calls).toMatchInlineSnapshot("here you will see the query");
  });

But is it useful?

as unit test I would say yes....if I have a function that create a user and a subscription, something like:
export const createAccount = async () => {
  await db.insert("account").executeTakeFirst()
  await db.insert("subscription").executeTakeFirst()
}

I want to be sure that calling this createAccount() will run 2 particular queries....
Obviously for integrations test we shouldn't mock the connection and just connect to a real db (like wrote in first comment)

Then why unit test if the integration test will cover the same field and more?

mmm maybe it is just a limitation for my experience...but usually I do unit test to be sure that a piece of code does what is meant to do and integration test to test the whole flow....otherwise what would be the point of unit test? but again, I never built myself an app with millions of users, I only worked for companies that do tests in this way, so I might be wrong :)

But what value do you gain from asserting something that will always happen(at least on the happy path, as createAccount isn't a pure function)?
At least the integration test asserts the expected data transformation.

I think the main point is to mock the client so you can run unit tests without having a db running, maybe the assertion can be skipped if you have integration tests, but if I think about a function like this:
const registerBusinsess = ({ db }: { db: Kysely<DB> }) => {
  const account = await db.insert("account").executeTakeFirst()
  const stripeSubs = await stripe.subs.create({ meta: { account: account.uuid }});
  const subscription = await db.insert("subscription").set({ stripeSubs: stripeSubs.id }).executeTakeFirst();
  await sqs.sendMessage({ event: "new-subscription", data: subscription })
}

I would like to have a test like:
it("register a businsess", () => {
   await registerBusinsess({ db });
   expect(dbMock.query).toHaveBeenCalledWith("...");
   expect(stripeSubs.create).toHaveBeenCalledWith("...");
   expect(dbMock.query).toHaveBeenCalledWith("...");
   expect(sqsMock.sendMessage).toHaveBeenCalledWith("..);
})

That test doesn't assert anything that wouldn't happen anyway

authors: francescovenica igalklebanov thelinuxlich vnijat93


text:
codecov.io との連携を見直した方がよい気がする

現状はcommitの横に ❌ のマークを表示するためだけのものになっているように見えます。
coverageの計測は無駄ではないですが、現状では「品質管理がだめそうなプロジェクト」という印象を与える不利益の方が大きそうです。
設定の修正や連携解除を検討してみてほしいです。

やっておきました。特にpatchは0にしました。
https://github.com/kyuridenamida/atcoder-tools/commit/48880c98c3beb272bbbfbbbbb6a36dbe043f2162

よさそう

authors: kmyk kyuridenamida

repo: kyuridenamida/atcoder-tools


text:
Reducing the complexity of the examples

Reducing the complexity of the examples, centralizing dependences in a singleton

I'm sorry I have anticipated creating this PR. In short I do one with the comments you make me in the old PR. I close both PR

repo: kzaher/RxSwift


text:
Installation don't work

Hi,
I running the instructions on windows 10, python 3.6. After editing the "print" command inside launch (because we know that there is a change on it), I keep get the following errors after launch the command:

Traceback (most recent call last):
File "C:\Users\Jorge Castillo\AppData\Local\Programs\Python\Python36\lib\runpy.py", line 183, in _run_module_as_main
mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
File "C:\Users\Jorge Castillo\AppData\Local\Programs\Python\Python36\lib\runpy.py", line 153, in _get_module_details
code = loader.get_code(mod_name)
File "", line 781, in get_code
File "", line 741, in source_to_code
File "", line 219, in _call_with_frames_removed
File "C:\Users\Jorge Castillo(...)\InfVocLDA-master\src\infvoc\launch.py", line 232
gamma_path = os.path.join(output_directory, "gamma.txt")
^
SyntaxError: invalid syntax

I'm launching the command (following the instruction) on cmd windows, in the source established (i.e.,
in InfVocLDA-master\src) and getting the error and can't installing. Also, changing "python" with "py " don't fit the error. What I'm doing wrong? Thanks for help.

The package was originally developed with Python2.7, and the error seems to
be some incompatible syntax error or tab/space conversion.
Let me know if that helps.
On Sun, Sep 30, 2018 at 10:42 PM jorgecastillo2 notifications@github.com
wrote:

Hi,
I running the instructions on windows 10, python 3.6. After editing the
rint command (because we know that there is a change on it), I keep get the
following errors after launch the command:
Traceback (most recent call last):
File "C:\Users\Jorge
Castillo\AppData\Local\Programs\Python\Python36\lib\runpy.py", line 183, in
_run_module_as_main
mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
File "C:\Users\Jorge
Castillo\AppData\Local\Programs\Python\Python36\lib\runpy.py", line 153, in
_get_module_details
code = loader.get_code(mod_name)
File "", line 781, in get_code
File "", line 741, in source_to_code
File "", line 219, in _call_with_frames_removed
File "C:\Users\Jorge Castillo(...)\InfVocLDA-master\src\infvoc\launch.py",
line 232
gamma_path = os.path.join(output_directory, "gamma.txt")
^
SyntaxError: invalid syntax
I'm launching the command (following the instruction) on cmd windows, in
the source established (i.e.,
in InfVocLDA-master\src) and getting the error and can't installing.
Also, changing "python" with "py " don't fit the error. What I'm doing
wrong? Thanks for help.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kzhai/InfVocLDA/issues/4, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAklRpUjbcNrc82UzXrpIxJOQ7RT7SO1ks5ugatHgaJpZM4XBaWF
.

Hi kzhai, thanks for answering
FInally i launch with python 2.7 and fixed the tab/space cinversion via untabify spaces but I keep getting another error type:
Traceback (most recent call last):
  File "C:\Python27\lib\runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "C:\Python27\lib\runpy.py", line 72, in _run_code
    exec code in run_globals
  File "C:\Users\Jorge Castillo\(...)l\InfVocLDA-master\src\infvoc\launch.py", line 248, in <module>
    main()
  File "C:\Users\Jorge Castillo\(...)\InfVocLDA-master\src\infvoc\launch.py", line 218, in main
    batch_gamma = olda.learning(docset);
  File "infvoc\hybrid.py", line 448, in learning
    sufficient_statistics, batch_document_topic_distribution = self.e_step(wordids);
  File "infvoc\hybrid.py", line 335, in e_step
    sufficient_statistics[k][0, id] += temp_phi[k, 0];
IndexError: only integers, slices (`:`), ellipsis (`...`), numpy.newaxis (`None`) and integer or boolean arrays are valid indices

Clearly, I must edit the lines 335 and 448 of hybrid.py in order to recognize valid indexes
I think that in line 335, i.e.,  in  sufficient_statistics[k][0, id] += temp_phi[k, 0]; maybe id isn't recognized as valid index,  and in the other (448) I don't know how to fix it.
Thanks for your help and code developing.

Hi, Jorge,
The reason is the newer version of numpy enforces the data type to be explicitly defined, even creating from an all integer list.
Anyway, please try to run the code from a different repo https://github.com/kzhai/PyInfVoc. It basically contains the exact same code, but much cleaner.
I have just ported this repo to python3, so you should be able to run it without any problem.
kzhai@MININT-LPGA1J7:/mnt/c/Users/kezhai/Workspace/PyInfVoc$ python3 -m launch_train --input_directory=./de-news/ --output_directory=./ --truncation_level=4000 --number_of_pics=10 --number_of_documents=9800 --training_iterations=100 --vocab_prune_interval=10 --batch_size=98 --alpha_beta=100
========== ========== ========== ========== ==========
output_directory=./de-news/18Oct03-231005-D9800-K10-T4000-P10-I10-B98-O100-t64-k0.75-at0.1-ab100/
input_directory=./de-news
corpus_name=de-news
dictionary_file=None
number_of_documents=9800
number_of_topics=10
truncation_level=4000
vocab_prune_interval=10
snapshot_interval=10
batch_size=98
training_iterations=100
tau=64.0
kappa=0.75
alpha_theta=0.1
alpha_beta=100.0
========== ========== ========== ========== ==========
successfully load all training documents...
select documents from 0 to 98
P-step, E-step and M-step take 0, 2, 0 seconds respectively...
vocabulary size = [1725, 1725, 1725, 1725, 1725, 1725, 1725, 1725, 1725, 1725]
training iteration 1 finished in 2.888820 seconds: epsilon = 0.043683
select documents from 98 to 196
P-step, E-step and M-step take 0, 2, 0 seconds respectively...
vocabulary size = [2582, 2582, 2582, 2582, 2582, 2582, 2582, 2582, 2582, 2582]
training iteration 2 finished in 3.002949 seconds: epsilon = 0.043186
select documents from 196 to 294
P-step, E-step and M-step take 0, 2, 0 seconds respectively...
vocabulary size = [3194, 3194, 3194, 3194, 3194, 3194, 3194, 3194, 3194, 3194]
training iteration 3 finished in 2.965397 seconds: epsilon = 0.042702
select documents from 294 to 392
P-step, E-step and M-step take 0, 4, 0 seconds respectively...
vocabulary size = [3654, 3654, 3654, 3654, 3654, 3654, 3654, 3654, 3654, 3654]
training iteration 4 finished in 4.734318 seconds: epsilon = 0.042230
select documents from 392 to 490
P-step, E-step and M-step take 0, 3, 0 seconds respectively...

Thanks for digging into this, and feel free to let me know for any bugs.
Best,
Ke

repo: kzhai/InfVocLDA


text:
A lot of fonts aren't working

I want this font: https://www.reddit.com/r/2007scape/comments/ojepcx/comment/h516xk9/?utm_source=share&utm_medium=web2x&context=3
But it instead gives me a default font, same case with a lot of other fonts. Ones which seems to work are Impact, Georgia, Times New Roman, Arial. Probably some others
Thanks.

For a font to be picked up by the plugin the name must match (but are case insensitive) and the font must have been installed system wide. The font in the reddit post works on my w10 system by entering bankgothic in the font setting (and setting font style to BOLD to reach the same effect)

https://github.com/l2-/template-plugin/issues/21 also see this issue.

Thank you for all your help - appreciated
On Thu, 16 Jun 2022 at 4:37 pm, l2- @.***> wrote:

https://github.com/l2-/template-plugin/issues/21 also see this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/l2-/template-plugin/issues/32#issuecomment-1157808415,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJRO7ZUMV7DBV4S7PNQ67HTVPNC5FANCNFSM5Y6J3HRQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


text:
added all_close function among TensorMaps

Two TensorMaps are considered all close if the have the same keys and their TensorBlocks have the same samples, components, properties (in the same order) and their values pass the allclose test with the given rtol and atol.
It solves #45 how me and @Luthaf discussed.

Could you also use this function to replace the one currently used in the tests?

Could you also use this function to replace the one currently used in the tests?

you mean in the test for the dot/solve ?

authors: DavideTisi Luthaf

repo: lab-cosmo/equistore


text:
Adding lint staged

added lint-staged to react order book

This already got merged in #11... why was this second PR added?


text:
Reverse routing docs and sample

As discussed, here is PR for the docs which includes Echo#Reverse documentation.

Merged 522d98608e223f67c935f65595132a1bee019df8.

authors: skyflyer vishr

repo: labstack/echox


text:
unrecognized arguments: --days 2

I am running your K8s manifest on our cluster and when i want to add the --days 2 argument i get the following error:
time="2023-03-09T14:46:22Z" level=info msg="spawning process: [python -u /app/auto_scan.py --days 2 -d --inline-scanner]"
app=vault-env
usage: auto_scan.py [-h] [--account ACCOUNT] [--subaccount SUBACCOUNT]
[--api-key API_KEY] [--api-secret API_SECRET] [-p PROFILE]
[--proxy-scanner PROXY_SCANNER]
[--proxy-scanner-skip-validation]
[--cache-timeout CACHE_TIMEOUT] [--days DAYS]
[--hours HOURS] [--registry REGISTRY] [--rescan]
[--list-only] [--auto-integrate-inline-scanner]
[--inline-scanner]
[--inline-scanner-path INLINE_SCANNER_PATH]
[--inline-scanner-access-token INLINE_SCANNER_ACCESS_TOKEN]
[--inline-scanner-only] [--inline-scanner-prune]
[--org-level] [-d] [--debug]
auto_scan.py: error: unrecognized arguments: --days 2

I don't think this is an issue with the container itself, more the syntax used for the command and args configuration.  If you don't mind, try something like the following and let me know if it resolves the problem:

args: ["-d", "--inline-scanner", "--days", "2"]

authors: TheAlanNix hawk-stephan-stiefel

repo: lacework-dev/container-auto-scan


text:
Fix some bug

UserStreamListenerのaddToHistory内でEventListAdapterの取得時にキャストされてないため型不適合でコンパイルが通らない問題の修正
StatusCommandTofuBuster.javaの名前の大文字小文字が違うためコンパイルが通らない問題の修正
です。

手動で採用しました


text:
Training Time

It's really a nice work!!
I just want to know what is the approximate training time for the CARZero with the default setting.

Thanks for your comment. We train on an A800 using about 8 hours.

authors: laihaoran maxxxxxxcai

repo: laihaoran/CARZero


text:
bootstrap在vagrant up中跑起来

通过shell provision，可把bootstrap由2步减为1步。

laincloud/lain#137

repo: laincloud/lain-cli


text:
Linking of Chapters broken

Linking of Chapters broken from chapter 3 due to renaming, removing and addition of new/old/junk files and chapters.
Action Required -
The Developer (Me or You who is reading this) needs to clone the code,
Create a new branch named "Link Fixes by ",
scan the links ,
run the live server to see the output,
clicks on the links and buttons to find the issues, then,
fix the broken links.
I am expecting an open source contribution from your side. Since I am in hurry, so I am leaving this as it is.
Will fix it when I get time from work/Job hunting.

Fixed Linking of Chapters broken #76  with #92 PR


text:
Fix: infinite while loop by removing the try except wrapper in the task report

There was a while infinite loop, caused by a double try-except in evaluate.py. One in bench is enough.

Member

Only reason I can think of is if you run this function on its own? Otherwise there is the other try/except that takes care of this?

repo: lamalab-org/chem-bench


text:
Refactored the location of package management

Package management was being done before commands were run. I thought it best to condense this within the bug report. I did this by performing the following:

Creating an array of utilities we are installing
Iterating over this array and creating an array of utilities pending install
Install all of the utilities at once as opposed to doing them one a at a time

My aim was to make this easier to read, more modular, and to reduce the time it takes to run the bug report.

Additionally the commands are not the same as the apps that need to be installed so there needs to be an associative array like this:
# Declare an associative array for utilities and their packages
declare -A utilities
utilities=(
    ["smartctl"]="smartmontools"
    ["ibstat"]="infiniband-diags"
    ["ipmitool"]="ipmitool"
    ["sensors"]="lm-sensors"
    ["iostat"]="sysstat"
    ["lshw"]="lshw"
)

pending_install_packages=()

# Iterate through utilities and create an array of packages pending install
for u in "${!utilities[@]}"; do
    if ! command -v $u >/dev/null 2>&1; then
        pending_install_packages+=("${utilities[$u]}")
    fi
done

authors: bryangwin ryaeng

repo: lambdal-support/lambda-public-tools


text:
--title not adopted in all outputtypes

Hallo,
using Ontospy in doc generating mode with output types 5, 7, 8, 9, 10 does not overtake the given --title
My command:
ontospy gendocs --type 5 --title Metadata4Ing metadata4ing.ttl
Output:

Fixed with https://github.com/lambdamusic/Ontospy/releases/tag/v1.9.9

repo: lambdamusic/Ontospy


text:
Return error code with RuntimeException.

This brings parity with other storage adapters that already include the error code in exceptions.
Signed-off-by: Ere Maijala ere.maijala@helsinki.fi

Q
A

Documentation
no

Bugfix
no

BC Break
no

New Feature
yes

RFC
no

QA
no

I'll add GHA in a PR this weekend. I'll try to rebase this PR afterwards so we can verify the test results here. 👍🏼

Took me a bit but I've added GHA now, added a unit test for your changes and made the tests pass. 👍🏼
Thanks, @EreMaijala!

@EreMaijala For integrity, I've passed the result code to the other RuntimeException creations as well and created tests for that.

@boesing Thanks, I appreciate your extra work and attention to detail. :)

authors: EreMaijala boesing


text:
Provide PHP 8.1 support

Signed-off-by: Abdul Malik Ikhsan samsonasik@gmail.com

Drop php 7.3 support and add 8.1.0 per PR https://github.com/laminas/laminas-servicemanager/pull/103 by @weierophinney
Add .laminas-ci.json
remove bump changelog
commit composer.lock

It seems php 8.1, with latest phpstan, it still error:
PHP Fatal error:  During inheritance of ArrayAccess: Uncaught Return type of Hoa\Protocol\Node\Node::offsetExists($name) should either be compatible with ArrayAccess::offsetExists(mixed $offset): bool, or the #[ReturnTypeWillChange] attribute should be used to temporarily suppress the notice

phar:///github/workspace/vendor/phpstan/phpstan/phpstan.phar/vendor/hoa/protocol/Node/Node.php:134

PHP 8.1 error possibly related with this https://github.com/phpstan/phpstan/issues/5558

Also related: https://github.com/Roave/BetterReflection/issues/701

rebased and re-generated composer.lock from php 7.4

@Ocramius All green 🎉

authors: Ocramius samsonasik


text:
🐛 Fix very long runtimes for Artifact.describe

Identified a few bottlenecks with a profiler and fixed them.
Now
ln.Artifact.get("NiaMP8YQMbmkdOgnAa1T").describe()
ln.Artifact.get("LmlgCMQRhSEHtcbG0000").describe()

runs around 10 sec each instead of several minutes.
Fixed describe when artifact is queried with using.
ln.Artifact.using("laminlabs/cellxgene").get("g0RcSSYe5vQKzSWYkhMc").describe() was failing before.
4-10 sec depending on annotations still feels pretty slow though, so it is probably not the final iteration.
Addresses:

https://github.com/laminlabs/lamindb/issues/2268

@Koncopd does this also improve the memory usage? Can I try it again now?

I hope so, but i haven't checked.

authors: Koncopd Zethson

repo: laminlabs/lamindb


text:
Data log "full" marker is not being written to interface storage

Running this programme on tag v0.2.63, which writes to the data log one byte at a time until the log is full:
#include "MicroBit.h"

MicroBit uBit;

int main() {
    uBit.init();

    uBit.display.print("?");

    while (true) {
        uBit.log.logString("X");
        uBit.sleep(1);

        if (uBit.log.isFull()) {
            uBit.display.print("D");
            uBit.sleep(100);
        }
    }
}

MICROBIT.hex.zip
We should expect the last 4 bytes, which are reserved for this marker, to contain FULL:
https://github.com/lancaster-university/codal-microbit-v2/blob/d54ce9b784ef1bba22c075b281bb877abf53ea18/source/MicroBitLog.cpp#L258
https://github.com/lancaster-university/codal-microbit-v2/blob/d54ce9b784ef1bba22c075b281bb877abf53ea18/source/MicroBitLog.cpp#L738-L742
However, the last 4 bytes are left untouched: MY_DATA.HTM.txt

This is odd - do you know whether the byte-by-byte case ever wrote 'log full'? Does it still work if logging longer lengths? From MakeCode?

It must have worked at some point, as I believe that's how the "log full" message is meant to be displayed:

https://github.com/lancaster-university/codal-microbit-v2/issues/140

I haven't tried it in MakeCode yet.

It does get to this point (even with a long string to fill memory more quickly)
https://github.com/lancaster-university/codal-microbit-v2/blob/master/source/MicroBitLog.cpp#L740
But I think the last byte before flash.getFlashEnd() is blocked by the equals in this test
https://github.com/lancaster-university/codal-microbit-v2/blob/master/source/FSCache.cpp#L119
and
https://github.com/lancaster-university/codal-microbit-v2/blob/master/source/FSCache.cpp#L87
With ">=" changed to ">" in the FSCache test, the I2C transaction seems OK, but it still seems to be zeros in the file.
https://github.com/lancaster-university/codal-microbit-v2/blob/master/source/MicroBitUSBFlashManager.cpp#L369
FSCache::write address 0x1EFFD length 3 getFlashEnd 0x1F000
FUL
MicroBitUSBFlashManager::write 0x1EFFC 4
request 0xB 0x1 0xEF 0xFC 0x0 0x0 0x0 0x4 0xFF 0x46 0x55 0x4C
response 0xB 0x1 0xEF 0xFC 0x0  0x0 0x0 0x4 0xFF

Ah! I forgot the USB must be reconnected to see the latest data... FUL is there!

FUL is saved when the FSCache::write test is changed to ">",

but it doesn't always make LOG FULL appear.

MY_DATA.HTM.txt

Will do!

authors: jaustin martinwork microbit-carlos


text:
Feature: fully add support for table_names paging

SDK
Rust
Description
The table_names method in the python remote client supported paging, but this was undocumented, and only supported on the remote client.  As a result, the parameters didn't get ported over when moving the python client to rust.  We should document it and add a simple in-memory implementation for local tables and then add support for the remote rust client.
Furthermore, until this is fixed, the remote client will only grab the first 10 items from LanceDb Cloud.  This means this feature is probably part of the new API MVP.

Closed by #1059

repo: lancedb/lancedb


text:
FTS Language Tokenisation based on a table column for language

SDK
Python
Description
Right now the tokeniser can only work with one language in FTS, which is ideal for most situations but when working with multiple languages in the same dataset, it might be a problem for text-based search.
Proposal: allow for multiple tokenisers - this base functionality could work with stemming based on multiple languages (however could introduce overlaps which may reduce quality)
Second/further idea: Allow for some form of dynamic tokeniser? For example, one could, at insertion time, add a language string/enum column which would identify the language of the text item beforehand (so if said text was split from a larger document, it would be easier to classify the whole documents language, or each snippet can be classified - language classification would be left up to the end user). Then, when creating the FTS index, different tokenisers and stem patterns can be used based on the language for that item.

thanks for the proposal!
i'm thinking maybe we can support to customize the stemming rules, then the users can construct their own tokenizers to work well no matter what languages the texts are in, is stemming what you are caring? @f1shy-dev


text:
expr: fix memory management

Andy Chu's patch from the mailing list, modified slightly to fit the current source.
http://lists.landley.net/htdig.cgi/toybox-landley.net/2016-March/008113.html
This fixes #147 as well as fixing all remaining memory management issues in expr from what I can see (valgrind now gives no warnings or any indication of issues while previously it reported many invalid reads).

ping

uh I did not mean to do that

(you might want to squash the two commits into one for ease of review. you probably also want to just send the patch on the list, because rob pays more attention to that than to the pull requests...)

repo: landley/toybox


text:
Land Library link top of page links to wrong page

In the top links to the three services, the Land Library does not work. It links to "Land Library old": https://landportal.info/libraryold.
Can we fix this?

Sure!
Inviato da iPhone

Il giorno 08/apr/2016, alle ore 14:03, lisettemeij notifications@github.com ha scritto:
In the top links to the three services, the Land Library does not work. It links to "Land Library old": https://landportal.info/libraryold.
Can we fix this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

authors: lisettemeij semberryltd

repo: landportal/drupal


text:
🛑 Kelsi is down

In dab5e95, Kelsi (https://kelsilesowski.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Kelsi is back up in 06c3f62.


text:
🛑 vipgifts.net is down

In ff2260a, vipgifts.net (https://vipgifts.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: vipgifts.net is back up in 0dbf3b6 after 18 minutes.

repo: lanen/bs-site


text:
🛑 rexingsports.cn is down

In 68cd644, rexingsports.cn (https://www.rexingsports.cn) was down:

HTTP code: 567
Response time: 705 ms

Resolved: rexingsports.cn is back up in bc272ce.


text:
🛑 rexingsports.cn is down

In f9478ad, rexingsports.cn (https://www.rexingsports.cn) was down:

HTTP code: 567
Response time: 1292 ms

Resolved: rexingsports.cn is back up in 9d61f80 after 20 minutes.


text:
🛑 rexingsports.cn is down

In 68b63b3, rexingsports.cn (https://www.rexingsports.cn) was down:

HTTP code: 567
Response time: 452 ms

Resolved: rexingsports.cn is back up in 28d05da after 19 minutes.


text:
🛑 rexingsports.cn is down

In e86e062, rexingsports.cn (https://www.rexingsports.cn) was down:

HTTP code: 567
Response time: 641 ms

Resolved: rexingsports.cn is back up in f6e8ce0 after 1 hour, 28 minutes.


text:
🛑 vipgifts.net is down

In b099f56, vipgifts.net (https://vipgifts.net) was down:

HTTP code: 567
Response time: 277 ms

Resolved: vipgifts.net is back up in 1472600 after 23 minutes.


text:
🛑 rexingsports.cn is down

In 6090b28, rexingsports.cn (https://www.rexingsports.cn) was down:

HTTP code: 567
Response time: 757 ms

Resolved: rexingsports.cn is back up in 02cdbc0 after 25 minutes.


text:
lanetix/issues#1929 - Switch micro services to iojs v1.5

@lanetix/engineers CR Please.
lanetix/issues#1929

Maybe increase the beforeEach timeout? otherwise :ship:

authors: OronNadiv apechimp

repo: lanetix/bookshelf-deep-changed


text:
🛑 Grafana is down

In fb597d5, Grafana (http://langasg1.ddns.net:8082) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Grafana is back up in 55ed94a.

repo: langasg/upptime


text:
Redundant Tool Usage output being returned together with text outputs in the content field of AIMessage in ChatBedrockConverse.

When running the following code
from langchain_core.prompts import ChatPromptTemplate
from langchain_aws import ChatBedrockConverse, ChatBedrock
from langchain_core.runnables import RunnableAssign,RunnablePassthrough
from langchain_core.messages import HumanMessage, AIMessage
from langchain_core.output_parsers import StrOutputParser
import uuid
from langchain_core.tools import tool
from typing import Annotated, Literal

@tool
def math_tool(op: Annotated[Literal["sum" , "sub" , "mul" , "div"], "The math operation to be perfomerd"], x: float, y: float) -> float:
    """Perform mathematical operations."""
    match op:
        case "sum":
            return x + y
        case "sub":
            return x - y
        case "mul":
            return x * y
        case "div":
            return x/y
        case _:
            raise ToolException(f"Operation {op} not recognized.")

model = ChatBedrockConverse(model_id="anthropic.claude-3-haiku-20240307-v1:0", client=client)

prompt = ChatPromptTemplate.from_messages([
("system", """Answer the user's request using relevant tools (if they are available). Before calling a tool, do some analysis within <thinking></thinking> tags. First, think about which of the provided tools is the relevant tool to answer the user's request. Second, go through each of the required parameters of the relevant tool and determine if the user has directly provided or given enough information to infer a value. When deciding if the parameter can be inferred, carefully consider all the context to see if it supports a specific value. If all of the required parameters are present or can be reasonably inferred, close the thinking tag and proceed with the tool call. BUT, if one of the values for a required parameter is missing, DO NOT invoke the function (not even with fillers for the missing params) and instead, ask the user to provide the missing parameters. DO NOT ask for more information on optional parameters if it is not provided."""),
("placeholder", "{messages}")])

chain = {"messages": RunnablePassthrough()} | prompt | model.bind_tools([math_tool])
msg = chain.invoke([HumanMessage(content="How much is 3*3 ?")])
msg.content

The content field of the  AIMessage  returned is:
[{'type': 'text',
  'text': '<thinking>\nThe relevant tool for this request is the "math_tool" which can perform various mathematical operations. The required parameters for this tool are "op" (the operation to perform), "x" (the first operand), and "y" (the second operand).\n\nThe user has requested to find the result of 3 * 3, which corresponds to the "mul" (multiplication) operation with "x" = 3 and "y" = 3. All the required parameters are provided, so I can proceed with invoking the math_tool.\n</thinking>'},
  {'type': 'tool_use',
   'name': 'math_tool',
   'input': {'op': 'mul', 'x': 3, 'y': 3},
   'id': 'tooluse_DOvtVdkMSECOFKJ5usGsgA'}]

A list containing both the text block and the tool call.
But the tool_call field of the AIMessage class is already populate with that tool information:
msg.tool_calls

 [{'name': 'math_tool',
   'args': {'op': 'mul', 'x': 3, 'y': 3},
   'id': 'tooluse_DOvtVdkMSECOFKJ5usGsgA'}]

Is that the expected behavior ? Isn't it redundant to return tool calling information both in the content and tool_calls fields ?

@thiagotps
The text response in your example seems to be a symptom of the prompt you are providing, where you have instructed the model to call the tool; the model seems to infer the function here and actually doing the calculations. For a usual tool call, this should be done by the client, and results passed back to the model. I don't see how this is an issue with the ChatBedrockConverse class. Have you tried running the same prompt with tool calling directly using Bedrock API and compared the results?

I modified your code slightly to remove the system instructions which only returned the tool_call response.
prompt = ChatPromptTemplate.from_messages([("placeholder", "{messages}")])

Output
[{'type': 'tool_use',
  'name': 'math_tool',
  'input': {'op': 'mul', 'x': 3, 'y': 3},
  'id': 'tooluse_QYT9YJl1TO6ewv09Xzh_ZA'}]

@3coins
I think the issue is in line 571 of bedrock_converse.py.
def _parse_response(response: Dict[str, Any]) -> AIMessage:
    anthropic_content = _bedrock_to_anthropic(
        response.pop("output")["message"]["content"]
    )
    tool_calls = _extract_tool_calls(anthropic_content)
    usage = UsageMetadata(_camel_to_snake_keys(response.pop("usage")))  # type: ignore[misc]
    return AIMessage(
        content=_str_if_single_text_block(anthropic_content),  # type: ignore[arg-type]
        usage_metadata=usage,
        response_metadata=response,
        tool_calls=tool_calls,
    )

All of anthropic_content is being passed to the content of AIMessage, despite the tool_calls being extracted above and  being passed separately in the tool_calls field of AIMessage.

I created a pull request  #100  for this.

@thiagotps
Looking at the Anthropic documentation, it seems like content block along with the tool_use info is normal in case a user want to look at the chain of thought reasoning that the model used to decide which tool to use.
https://docs.anthropic.com/en/docs/build-with-claude/tool-use#chain-of-thought
I also looked at the ChatAnthropic and it seems to also add the content as-is.
https://github.com/langchain-ai/langchain/blob/master/libs/partners/anthropic/langchain_anthropic/chat_models.py#L716-L741
Let me know if I am missing anything.

repo: langchain-ai/langchain-aws


text:
add standard params

Here we implement a set of parameters that LangChain is standardizing on to simplify integrations with various LLM monitoring platforms (such as Langsmith). These include standard keys for providers, model names, temperature, stop sequences, etc.
ChatCohere is somewhat distinct in that the .model attribute can be None. Because we always require a string for the model name parameter, I also add a property .model_name that will cache the result of ._get_default_model() if .model is None. I took this from the get_num_tokens method and refactored get_num_tokens to use it. I don't actually know if this is the model that is used behind the scenes when model=None!
Note that we will generally now be calling ._get_default_model() (which makes an additional network call) whenever we first invoke ChatCohere after it's instantiated with no value for model.
We also bump langchain-core to 0.2.0. This is needed to support the new parameters. There are no breaking changes in langchain-core from 0.1 to 0.2, but let me know if you think we'd need to update langchain-cohere to 0.2 as well due to this (we have not been doing this in other integration packages).

Merging as I observe a value for the default model that is consistent with the docs (command-r-plus).
Please let me know if there is an issue with the new model_name property!
cc @harry-cohere

repo: langchain-ai/langchain-cohere


text:
Added prompt wrapping for Claude with Bedrock

Description: Prompt wrapping requirements have been implemented on the service side of AWS Bedrock for the Anthropic Claude models to provide parity between Anthropic's offering and Bedrock's offering. This overnight change broke most existing implementations of Claude, Bedrock and Langchain. This PR just steals the the Anthropic LLM implementation to enforce alias/role wrapping and implements it in the existing mechanism for building the request body. This has also been tested to fix the chat_model implementation as well. Happy to answer any further questions or make changes where necessary to get things patched and up to PyPi ASAP, TY.
Issue: No issue opened at the moment, though will update when these roll in.
Dependencies: None

Discussed internally but posting here as well:
If I’m reading the update right, it’s going to append an Assistant:  tag on the very end  of the prompt if it’s not there?
Claude can actually accept text after the Assistant:  tag (just tested this myself) so I want to make sure this wouldn't add a redundant tag when it's not needed.

Same issue here, many demos are down.
@hwchase17 Would you be able to kindly help me here, who can I contact to prioritize this PR?

Discussed internally but posting here as well:
If I’m reading the update right, it’s going to append an Assistant:  tag on the very end of the prompt if it’s not there?
Claude can actually accept text after the Assistant:  tag (just tested this myself) so I want to make sure this wouldn't add a redundant tag when it's not needed.

Just tested on the Bedrock version of Claude and it did not seem to allow for that. Seems like there is a mismatch between the Anthropic version of Claude and the Bedrock version of Claude.

Discussed internally but posting here as well:
If I’m reading the update right, it’s going to append an Assistant:  tag on the very end of the prompt if it’s not there?
Claude can actually accept text after the Assistant:  tag (just tested this myself) so I want to make sure this wouldn't add a redundant tag when it's not needed.

Just tested on the Bedrock version of Claude and it did not seem to allow for that. Seems like there is a mismatch between the Anthropic version of Claude and the Bedrock version of Claude.

Tested on Bedrock anthropic.claude-v2, this type of prompt works perfectly fine:
json_template = """Human:
Generate a JSON array of exactly 10 items from the below table. The items should resemble something that would belong to """ + customer_name +  """, who is in the """ + customer_industry + """ industry,
Print it in JSON format The ids should number 0-9. No explanatory language please.
{schema}

Assistant:
[
"""

This RAG prompt works fine as well:
template = """
Human: You are a helpful and talkative """ + customer_name + """ assistant that answers questions directly and only using the information provided in the context below.
Guidance for answers:
    - Simply answer the question clearly and with lots of detail using only the relevant details from the information below. If the context does not contain the answer, say "I don't know."
    - Use the royal "We" in your responses.

Now read this context and answer the question at the bottom.

Context: {context}"

Question: "Hey """ + customer_name + """ Chatbot! {question}

Assistant:
Based on the context provided:"""

When I use RAG with Bedrock Claude, it works for the first query and fails after the second iteration.
I am getting:
`ValueError: Error raised by bedrock service: An error occurred (ValidationException) when calling the InvokeModel operation: Invalid prompt: prompt must end with "
Assistant:" turn`

Discussed internally but posting here as well:
If I’m reading the update right, it’s going to append an Assistant:  tag on the very end of the prompt if it’s not there?
Claude can actually accept text after the Assistant:  tag (just tested this myself) so I want to make sure this wouldn't add a redundant tag when it's not needed.

Just tested on the Bedrock version of Claude and it did not seem to allow for that. Seems like there is a mismatch between the Anthropic version of Claude and the Bedrock version of Claude.

Tested on Bedrock anthropic.claude-v2, this type of prompt works perfectly fine:
json_template = """Human:
Generate a JSON array of exactly 10 items from the below table. The items should resemble something that would belong to """ + customer_name +  """, who is in the """ + customer_industry + """ industry,
Print it in JSON format The ids should number 0-9. No explanatory language please.
{schema}

Assistant:
[
"""

This RAG prompt works fine as well:
template = """
Human: You are a helpful and talkative """ + customer_name + """ assistant that answers questions directly and only using the information provided in the context below.
Guidance for answers:
    - Simply answer the question clearly and with lots of detail using only the relevant details from the information below. If the context does not contain the answer, say "I don't know."
    - Use the royal "We" in your responses.

Now read this context and answer the question at the bottom.

Context: {context}"

Question: "Hey """ + customer_name + """ Chatbot! {question}

Assistant:
Based on the context provided:"""

My bad, seems like there needs to be a new line between the Human text and the "Assistant:" token. But only if there is text after the "Assistant:" token

BTW, I tested the code from @jonathancaevans and it fixed the problem for me. All RAGs started to work with Cluade without changing templates.

Discussed internally but posting here as well:
If I’m reading the update right, it’s going to append an Assistant:  tag on the very end of the prompt if it’s not there?
Claude can actually accept text after the Assistant:  tag (just tested this myself) so I want to make sure this wouldn't add a redundant tag when it's not needed.

Just tested on the Bedrock version of Claude and it did not seem to allow for that. Seems like there is a mismatch between the Anthropic version of Claude and the Bedrock version of Claude.

Tested on Bedrock anthropic.claude-v2, this type of prompt works perfectly fine:
json_template = """Human:
Generate a JSON array of exactly 10 items from the below table. The items should resemble something that would belong to """ + customer_name +  """, who is in the """ + customer_industry + """ industry,
Print it in JSON format The ids should number 0-9. No explanatory language please.
{schema}

Assistant:
[
"""

This RAG prompt works fine as well:
template = """
Human: You are a helpful and talkative """ + customer_name + """ assistant that answers questions directly and only using the information provided in the context below.
Guidance for answers:
    - Simply answer the question clearly and with lots of detail using only the relevant details from the information below. If the context does not contain the answer, say "I don't know."
    - Use the royal "We" in your responses.

Now read this context and answer the question at the bottom.

Context: {context}"

Question: "Hey """ + customer_name + """ Chatbot! {question}

Assistant:
Based on the context provided:"""

My bad, seems like there needs to be a new line between the Human text and the "Assistant:" token. But only if there is text after the "Assistant:" token

I found the same thing. The validation seems to be very finicky at this time.

@jonathancaevans
Have you tried swapping Bedrock LLM with BedrockChat and see if that fixes the issue? Also, we should make sure that this change does not break the BedrockChat class.

@3coins
I have and it does.
But, I'm not aware of the full scope of other implementations or classes that might use LLM in the background or more generally, that might not work with BedrockChat. Hence why I'm patching this instead of just recommending folks move to BedrockChat. But if someone could clarify that BedrockChat could encapsulate every implementation or use case, like for example, FewShotPrompts, I would recommend everyone migrate to BedrockChat and just ditch the LLM implementation.
The change does not break BedrockChat as per my testing. There are no unit or integrations tests that I could find though so this testing was very manual and limited.

@jonathancaevans
The reason I am pushing BedrockChat is because it wraps the messages correctly (using LangChain Human/Assistant tags). Given that Anthropic LLM has deprecation warnings in their LLM and asking users to move the ChatAnthropic LLM, should we have the right guidance in the Bedrock LLM as well.
I also noticed that Anthropic LLM has similar code to wrap prompts in their LLM, but this was added a while ago (7 months). Given the urgency of this issue, I think it is fair to add the patch here at this time, as it is unlikely most folks will be able to migrate to the chat model class immediately.

@jonathancaevans The reason I am pushing BedrockChat is because it wraps the messages correctly (using LangChain Human/Assistant tags). Given that Anthropic LLM has deprecation warnings in their LLM and asking users to move the ChatAnthropic LLM, should we have the right guidance in the Bedrock LLM as well.
I also noticed that Anthropic LLM has similar code to wrap prompts in their LLM, but this was added a while ago (7 months). Given the urgency of this issue, I think it is fair to add the patch here at this time, as it is unlikely most folks will be able to migrate to the chat model class immediately.

seconding this. think the correct way to use bedrock and anthropic is via ChatModel interface (meaning BedrockChat). we should still fix the issues here but we should also log a warning if folks use Bedrock LLM with provider="anthropic" recommending they switch to BedrockChat

@jonathancaevans @zack-anthropic @tonycapone @3coins look good to everyone?

@zack-anthropic
Thanks for adding the detailed logic and the test cases.
@jonathancaevans
Do you want to test the current changes, and ensure it fixes the issue you have faced before this is merged?
@baskaryan
Unfortunately, I don't have access to Bedrock anymore to verify these changes fully. Code looks good otherwise.

@hwchase17 Yes raising alternation error > printing it. Thanks.
@baskaryan Yes overall looks good but we should still test live in Bedrock ASAP to make sure it doesn't have unexpected interaction with Bedrock code

I am still experiencing the same issue with Agents.  (Tried ReAct and Planner and Executor) both of them are giving the issue.
ValidationException: An error occurred (ValidationException) when calling the InvokeModel operation: Invalid prompt: prompt must start with "
Human:" turn, prompt must end with "
Assistant:" turn
During handling of the above exception, another exception occurred:

@jonathancaevans The reason I am pushing BedrockChat is because it wraps the messages correctly (using LangChain Human/Assistant tags). Given that Anthropic LLM has deprecation warnings in their LLM and asking users to move the ChatAnthropic LLM, should we have the right guidance in the Bedrock LLM as well.
I also noticed that Anthropic LLM has similar code to wrap prompts in their LLM, but this was added a while ago (7 months). Given the urgency of this issue, I think it is fair to add the patch here at this time, as it is unlikely most folks will be able to migrate to the chat model class immediately.

seconding this. think the correct way to use bedrock and anthropic is via ChatModel interface (meaning BedrockChat). we should still fix the issues here but we should also log a warning if folks use Bedrock LLM with provider="anthropic" recommending they switch to BedrockChat

I tried BedrockChat too and it's failing with same error message.

@Nav25oct
Can you post what prompt is being sent to the LLM. This will help us in correcting the error.

Having the same issue now, using BedrockChat LLM
 response = llm_chain.run(CompletionService.prepare_human_message(completion_dto.human_message, vendor_name, model)

@staticmethod
def prepare_human_message(human_message: str, vendor: AIVendor, model: str):
     if vendor == AIVendor.Bedrock and model.lower().find("claude") != -1:
         return f'\n\nHuman: {human_message}\n\nAssistant:'
     else:
         return human_message

@stanyq4 Can you please post the exact text of the prompt that is being sent to the LLM?

@zack-anthropic now I see that you are formatting the message inside.
After I removed prepare_human_message from my side - it's working, thanks.
QQ though, does it mean that system_message part is not supported anymore?

@stanyq4 Once again it comes down to the actual string that is being sent to the Anthropic API. A "system message" in Anthropic terms is anything that comes before the first human. A prompt string like
"""You are a skillful mathematician.
Human: What is 25 * 18?
Assistant:"""
should work fine.

@zack-anthropic Maybe I am missing something, but how is this fine if there is a check for the beginning of the string to start with a "\n\nHuman: ", we won't be able to put anything before that?

That check should not be there. I'm not sure where in the Langchain codebase it's coming from.

I am still experiencing the same issue with Agents. (Tried ReAct and Planner and Executor) both of them are giving the issue.
ValidationException: An error occurred (ValidationException) when calling the InvokeModel operation: Invalid prompt: prompt must start with "
Human:" turn, prompt must end with "
Assistant:" turn
During handling of the above exception, another exception occurred:

I had exactly the same issue; also, the first request in a session works but fails after that.

@jonathancaevans Just spoke with one of your co-workers -- we have been digging into this since Monday's "internal memo".
I think we have a solution which works, and up to langchain v0.0.303 it worked.
I was trying to figure out why it stopped in v0.0.304 and v0.0.305 and saw this push.
I think we need to undo this.
The prompt:
ANTHROPIC_PROMPT = """\n\nHuman:

Current conversation:
[HISTORY]
{history}
[/HISTORY]

{input}\n\nAssistant:"""

Is what's needed.
That seems to work for both Antrhopic's, and Bedrock's.
Along with 2 other changes -- Making sure any conversation summary does not use the words "Human" and "Assistant" -- note: you can lower case those:
1.) memory = ConversationSummaryBufferMemory(llm=memory_summary_llm, max_token_limit=max_token_count, human_prefix="User", ai_prefix="AI")
and
2.) ```
message = message.replace("Human", r"\human")
message = message.replace("Assistant", r"\assistant")

^ the second is needed to sanitize the user input, as any input that has `Human` and `Assistant` will break it.

authors: 3coins KamilLegault Nav25oct baskaryan jengroff jonathancaevans stanyq4 tonycapone ventz vlmelnyk zack-anthropic

repo: langchain-ai/langchain


text:
Set openai_api_type to be passed as values to leverage azure openai's default

Description:
Recently, this was introduced: https://github.com/langchain-ai/langchain/pull/10707

However, the changes are not taking effect unless you create the AzureOpenAIEmbeddings class with parameter openai_api_type="azure".. In the code it's already getting set by default if openAI version > 1 but it's never getting accessed when the embed_documents is actually happening because it was never surfaced up

Issue: Not so much an issue but leveraging existing default code of api type of azure when used.
Twitter handle: @kevin_neum

Actually, looks like the v1 client doesn't accept an api_type param. https://github.com/openai/openai-python/blob/65e29a2efa455a06deb59e243f27796c4ca2254c/src/openai/_client.py#L43-L61
Closing because this shouldn't go in!

Good catch! Any ideas for leveraging the default value in AzureOpenAIEmbeddings that is currently set but not being leveraged? Thus making this obselete unless it's explicitly set?
https://github.com/langchain-ai/langchain/pull/10707

Nope! And not sure what the linked chunk size issue has to do with this. While we support both v0 and v1, it's not completely obsolete too.


text:
updated async-faiss example

The original notebook has the faiss title which is duplicated in thefaiss.jpynb. As a result, we have two faiss items in the vectorstore ToC. And the first item breaks the searching order (it is placed between A... items).

I updated title to Asynchronous Faiss.

@efriis What if I copy the asynch_fai content into the faiss with correspondent heading? IMO asynchronous code is kind of important and can be referenced when we learn asynchronous usage.

I think it will look very redundant. If you look at the diff between the two files, they look almost identical

If you do want to keep them separate, maybe name this one "FAISS (Async)" so they appear next to each other in the sidebar?

I like idea with "Faiss (Asynch)". It is fast to switch and easy to reference.

Thanks for fixing!

authors: efriis leo-gan


text:
Community: Fix in MultiQuery Retriever to parse the generated queries in the List[str]

Description:  Fixes parsing of generated queries in the MultiQuery Retriever.

Issue: The MultiQueryRetriever was not correctly parsing the generated queries returned by the language model, which should be a list of strings (List[str]).

Fix: This PR updates the code to handle cases where the language model response might not have a "text" key directly. It now uses getattr to access the "text" key or an alternative key specified by the parser_key attribute. This ensures the retrieved queries are always a list of strings.

"""DEPRECATED. parser_key is no longer used and should not be specified."""

Not sure why you are getting this error, but if you use from_llm it should construct the LLMChain in the correct way