Bedrock Mistral not working as agent

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
from langchain import hub
from langchain_community.chat_models.bedrock import BedrockChat
from langchain.agents import AgentExecutor, create_structured_chat_agent

from langchain_community.tools.tavily_search import TavilySearchResults

prompt = hub.pull("hwchase17/structured-chat-agent")

llm_model = BedrockChat(
        region_name="us-east-1",
        model_id="mistral.mistral-7b-instruct-v0:2",
    )

tools = [TavilySearchResults()]
agent = create_structured_chat_agent(llm_model, tools, prompt)

agent_executor = AgentExecutor(
    agent=agent,
    tools=tools,
    verbose=True,
    handle_parsing_errors=True,
    return_intermediate_steps=True,
)

agent.invoke({"input": "What is the president of USA?"})

Error Message and Stack Trace (if applicable)
Exception: ValueError: Error raised by bedrock service: An error occurred (ValidationException) when calling the InvokeModelWithResponseStream operation: Malformed input request: #: extraneous key [stop_sequences] is not permitted, please reformat your input and try again.
Description

I am using Mistral model of AWS Bedrock as an agent
It doesn't work

System Info
System Information

OS:  Darwin
OS Version:  Darwin Kernel Version 23.3.0: Wed Dec 20 21:31:00 PST 2023; root:xnu-10002.81.5~7/RELEASE_ARM64_T6020
Python Version:  3.10.0 (default, Nov 11 2023, 18:46:15) [Clang 15.0.0 (clang-15<IP_ADDRESS>)]

Package Information

langchain_core: 0.1.34
langchain: 0.1.13
langchain_community: 0.0.29
langsmith: 0.1.33
langchain_google_vertexai: 0.1.2
langchain_openai: 0.0.8
langchain_text_splitters: 0.0.1
langchainhub: 0.1.15

Packages not installed (Not Necessarily a Problem)
The following packages were not found:

langgraph
langserve

Hello , I have the same issue , do you find a solution ?

You can try this
agent = create_structured_chat_agent(llm_model, tools, prompt, stop_sequence=False)


Add support for tagging EBS volumes

Looks good

https://github.com/mpostument/awstaghelper/releases/tag/0.19.0

Awesome, thanks!
BTW, hope you don't mind I've used ChatGPT for this, I basically gave it the EC2 instances tagging code and asked it to convert it to EBS volumes, and did this file by file.
It converted the calls to EBS and worked like a charm with just a couple of manual fixes because some of the functions were named differently and had to be renamed to match.

No problem with ChatGPT at all :)


[upb] Fix uwp build

Fix uwp build, add some compile options.
Related: #9980.

I took a look at: JackBoosY:dev/jack/9980
It fails to build with this error:
CMake Error at scripts/cmake/vcpkg_fail_port_install.cmake:89 (message):
openssl-windows is only for Windows Desktop
Target 'UWP' not supported by openssl-windows!
Besides, this is not the current 1.27 release of grpc.

@Noemata Did you build grpc:x64-uwp?

I did try to build grpc:x64-uwp from your repo.  Build failed at openssl part.

@Noemata Sorry for late.
Can you provide all the information output from the command line / powershell?

@Noemata The openssl part will fix in #10366.

Thanks for the notification @JackBoosY.  What I'm curious about is the fact that someone is already building gRPC for uwp in order to produce this (https://www.nuget.org/packages/Grpc/) nuget package.  Can some of this activity not be coordinated?  I'm guessing Google is doing this work on behalf of Microsoft?  It's great to be able to leverage VCPKG instead of having to rely on other parties to build such libs.  Also, if Google is able to produce lib builds for Microsoft platforms, I should think Microsoft can do so even better.

@Noemata The official nuget package provided by grpc does not use the vcpkg dependencies, so we still need to fix this issue.

Thank you for the clarification @JackBoosY.

LGTM, thanks!

Tried building gRPC, same issue as before with abseil.  VCPKG gets tripped up by whatever is wrong with the ninja build setup.  I did describe this in detail in another posting.
Very disappointing.
Extracting git...
-- Note: abseil only supports static library linkage. Building static library.
-- Downloading https://github.com/abseil/abseil-cpp/archive/06f0e767d13d4d68071c4fc51e25724e0fc8bc74.tar.gz...
-- Extracting source D:/vcpkg/downloads/abseil-abseil-cpp-06f0e767d13d4d68071c4fc51e25724e0fc8bc74.tar.gz
-- Applying patch fix-lnk2019-error.patch
-- Applying patch fix-uwp-build.patch
-- Using source at D:/vcpkg/buildtrees/abseil/src/4e0fc8bc74-c40446395f
-- Configuring x64-uwp
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:72 (message):
Command failed: ninja -v
Working Directory: D:/vcpkg/buildtrees/abseil/x64-uwp-rel/vcpkg-parallel-configure
Error code: 1
See logs for more information:
D:\vcpkg\buildtrees\abseil\config-x64-uwp-out.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_configure_cmake.cmake:295 (vcpkg_execute_required_process)
ports/abseil/portfile.cmake:14 (vcpkg_configure_cmake)
scripts/ports.cmake:90 (include)
Error: Building package abseil:x64-uwp failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: abseil:x64-uwp
Vcpkg version: 2020.02.04-nohash-external

This is using the 10005 commit.

@Noemata Here is my console output:
PS F:\vcpkg> .\vcpkg.exe install grpc:x64-uwp
Computing installation plan...
The following packages will be built and installed:
  * abseil[core]:x64-uwp
    grpc[core]:x64-uwp
  * upb[core]:x64-uwp
Additional packages (*) will be modified to complete this operation.
Starting package 1/3: upb:x64-uwp
Building package upb[core]:x64-uwp...
-- Note: upb only supports static library linkage. Building static library.
-- Using cached F:/vcpkg/downloads/protocolbuffers-upb-9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz
-- Using source at F:/vcpkg/buildtrees/upb/src/b291e32482-5b4f85036a
-- Configuring x64-uwp
-- Building x64-uwp-dbg
-- Building x64-uwp-rel
-- Installing: F:/vcpkg/packages/upb_x64-uwp/share/upb/copyright
-- Performing post-build validation
-- Performing post-build validation done
Building package upb[core]:x64-uwp... done
Installing package upb[core]:x64-uwp...
Installing package upb[core]:x64-uwp... done
Elapsed time for package upb:x64-uwp: 1.094 min
Starting package 2/3: abseil:x64-uwp
Building package abseil[core]:x64-uwp...
-- Note: abseil only supports static library linkage. Building static library.
-- Using cached F:/vcpkg/downloads/abseil-abseil-cpp-06f0e767d13d4d68071c4fc51e25724e0fc8bc74.tar.gz
-- Using source at F:/vcpkg/buildtrees/abseil/src/4e0fc8bc74-c40446395f
-- Configuring x64-uwp
-- Building x64-uwp-dbg
-- Building x64-uwp-rel
-- Installing: F:/vcpkg/packages/abseil_x64-uwp/share/abseil/copyright
-- Performing post-build validation
-- Performing post-build validation done
Building package abseil[core]:x64-uwp... done
Installing package abseil[core]:x64-uwp...
Installing package abseil[core]:x64-uwp... done
Elapsed time for package abseil:x64-uwp: 2.505 min
Starting package 3/3: grpc:x64-uwp
Building package grpc[core]:x64-uwp...
-- Note: grpc only supports static library linkage. Building static library.
-- Using cached F:/vcpkg/downloads/grpc-grpc-v1.27.3.tar.gz
-- Using source at F:/vcpkg/buildtrees/grpc/src/v1.27.3-191df9b830
-- Configuring x64-uwp
-- Building x64-uwp-dbg
-- Building x64-uwp-rel
-- Installing: F:/vcpkg/packages/grpc_x64-uwp/share/grpc/copyright
-- Performing post-build validation
-- Performing post-build validation done
Building package grpc[core]:x64-uwp... done
Installing package grpc[core]:x64-uwp...
Installing package grpc[core]:x64-uwp... done
Elapsed time for package grpc:x64-uwp: 9.167 min

Total elapsed time: 12.77 min

The package grpc:x64-uwp provides CMake targets:

    find_package(gRPC CONFIG REQUIRED)
    # Note: 5 target(s) were omitted.
    target_link_libraries(main PRIVATE gRPC::gpr gRPC::grpc gRPC::grpc++ gRPC::grpc++_alts)

    find_package(modules CONFIG REQUIRED)
    target_link_libraries(main PRIVATE c-ares::cares)

PS F:\vcpkg>

So, please provide failure logs such as D:\vcpkg\buildtrees\abseil\config-x64-uwp-out.log.

My apologies, I must have gotten the repo too soon.  I did a fresh clone of the vcpkg master repo this morning and the results were much better.  However, Now I get this:
Building package openssl-uwp[core]:x64-uwp... done
Installing package openssl-uwp[core]:x64-uwp...
Installing package openssl-uwp[core]:x64-uwp... done
Elapsed time for package openssl-uwp:x64-uwp: 3.113 min
Starting package 3/8: openssl-windows:x64-uwp
Building package openssl-windows[core]:x64-uwp...
CMake Error at scripts/cmake/vcpkg_fail_port_install.cmake:93 (message):
  openssl-windows is only for Windows Desktop

  Target 'UWP' not supported by openssl-windows!

Call Stack (most recent call first):
  ports/openssl-windows/portfile.cmake:1 (vcpkg_fail_port_install)
  scripts/ports.cmake:90 (include)

Error: Building package openssl-windows:x64-uwp failed with: BUILD_FAILED

Not sure if someone pushed in something new the break the openssl build for UWP, since this was an older problem.  Suggest @JackBoosY have another look.  Looks like someone added a condition to prevent building openssl for UWP.

I did notice in the build log @JackBoosY provides above, openssl is never listed as part of the build process?

In case this helps, I was able to build the following individually:
.\vcpkg.exe install upb[core]:x64-uwp --recurse
.\vcpkg.exe install protobuf[core]:x64-uwp --recurse
.\vcpkg.exe install abseil[core]:x64-uwp --recurse
.\vcpkg.exe install zlib[core]:x86-uwp --recurse
It was nice to see upb and abseil in there.

@Noemata Openssl does have dependency issue, and it will fix in #10366:
You can check changes here.


credential events are not been stored by following module

module which are not storing credentials

http/basic_auth_weak_password/
ssh/weak_password

While using module http/basic_auth_strong_password/ which works completely fine and storing credential events while above mentioned modules credential events are not stored.

Hi,
This is by design. The weak password modules are used to monitor hacker activities like what commands does the hacker perform after getting logged in easily. The strong password modules are used to monitor what does hacker provides as username/passwords to get into the system

Hi,
This is by design. The weak password modules are used to monitor hacker activities like what commands does the hacker perform after getting logged in easily. The strong password modules are used to monitor what does hacker provides as username/passwords to get into the system

Thanks for commenting, I misinterpreted the wiki guidelines.

Thanks for commenting, I misinterpreted the wiki guidelines.


Bazel can't fetch numeric-1.2.6.min.js because numericjs.com has expired

INFO: Downloading from http://www.numericjs.com/lib/numeric-1.2.6.min.js: 0B

When i opened it on my broswer, it prompted that numericjs.com has expired

same problem here

Hey @dsmilkov where can we download this from GitHub? I found https://github.com/sloisel/numeric/blob/master/src/numeric.js but it doesn't appear to be the same code that we have checked in internally, even though the versions are the same.
Or better yet, is there another library we could use?
cc: @danmane

https://github.com/sloisel/numeric/blob/master/src/numeric.js is just a portion of the library (uncompiled), and the author doesn't store the compiled/bundled one on github.
We can link to:
https://cdnjs.cloudflare.com/ajax/libs/numeric/1.2.6/numeric.min.js
Cloudfare CDN should be pretty stable. Let me know your thoughts

Thanks Submitted an internal change where numericjs gets pulled from cloudflare cdn. The internal change should show up in github within a day.
Thanks @sharkdtu and @natlachaman  for the bringing this issue to our attention!


Update returned exit code not zero while installing sift in Remnux

Tried installing Sift Cli in Remnux VM and it is displaying error codes below
Update returned exit code not zero
Error: Update returned exit code not zero
at ChildProcess. (/snapshot/sift-cli/sift-cli.js:556:23)
at ChildProcess.emit (events.js:400:28)
at maybeClose (internal/child_process.js:1058:16)
at Socket. (internal/child_process.js:443:11)
at Socket.emit (events.js:400:28)
at Pipe. (net.js:686:12)
saltstack.log

It looks like this is an issue with importlib_metadata. I'll look to see if I can duplicate then resolve this issue.

Sift CLI  - 1.14 and the python version is python 2.7 so do i need to update it to Python3 -V and try installing it ?

Hi @digitalsleuth
Now I tried it is working thanks
Cheers!

I'm getting the same error while trying to install on a REMnux box (using sudo sift install --mode=packages-only) and if I'm reading my saltstack.log correctly it's also an issue with importlib-metadata, however when I run sudo python3 -m pip install importlib-metadata<=5.0.0 I get the following response:
bash: =5.0.0: No such file or directory

I'm on an Ubuntu 20.04 vm running in VirtualBox

Hi @pshef , the command is actually sudo python3 -m pip install importlib_metadata<5.0.0
Make note of the underscore, and ensure there is no = sign either. That should fix the issue for you.

sudo python3 -m pip install "importlib-metadata<5.0.0" is what ended up working. Thank you so much!


[stdlib] {Set,Dictionary}.init(minimumCapacity:) Update docs

Avoid describing the exact algorithm that these collections use to allocate storage. (It is a private implementation detail that we want to be able to change.)
rdar://problem/36619317

@swift-ci smoke test


Problem with folds

Hi,
when working with this (btw. very great) plugin and very big files, I found the following bug:
Because my blocks to compare are very long, it's convenient to fold them first and then do the Linediff.
That causes Linediff to remove the end of the file (i.e. after the second block).
I'll do a step by step example:

Open vim with an empty buffer. Type / paste the following code:

def one
  two
end

def two
  three
end

something at the end of the file

Fold the two blocks of codes (vim ex commands):

:1,3fold
:5,7fold

Select each block for Linediff (the cool thing is, this works with V on the closed fold!):

:1,3Linediff
:5,7Linediff

Change something in the second (right) diff-buffer, and save it

Result (at least for me):

The change is correctly copied to the original buffer (that's good)
The fold is gone (that's not perfect but ok)
The text after the second fold is gone (that's the bug ;-) )

Thank you for the detailed bug report, I really appreciate the example.
Unfortunately, I can't seem to reproduce the issue :/. My guess is that there's some setting that I have that's making the difference. Could you show me your .vimrc? I might be able to find the setting like that. Let me know what your Vim version is as well, just in case.

Thank you for your fast reply!
You're right, with a minimal vimrc, it doesn't occur.
My vimrc is quite big and a lot of plugins, so I don't want you to work through all of that ;)
I'll do my own investigations and I'll report back if I find the culprit!
Thanks anyway!

It's likely that it's a setting and not a plugin, because i can't think of any plugin that could affect this one. If you manage to narrow it down to whatever setting is causing this, do let me know. There might be something I could do on the plugin side to prevent people from being affected by it.

Ok, I think I tracked it down, it was quite a tough one ;)
It happens only if

Nerd-Tree-Tabs plugin is enabled (https://github.com/jistr/vim-nerdtree-tabs)
The Nerd-Tree is open when starting the linediff

The Nerd-Tree-Tabs plugin which tries to make a NerdTree buffer persistent over all tabs. It's probably a quite hacky abuse of the way vim handles buffers/tabs, so that's probably the reason it interferes here :(
OK, at least I have a workaround now (just close Nerdtree).
But of course, if you want to have a closer look at it and find a proper solution (or maybe a bug in NerdtreeTabs), that would be even better :)

OK now again with automatic testcase :)

download / clone linediff, nerdtree, nerdtree-tabs to ~/.vim/bundle/
create a new folder with three files:

vimrc
set nocompatible
filetype off
set runtimepath=~/.vim/bundle/linediff.vim/
set runtimepath^=~/.vim/bundle/nerdtree/
set runtimepath^=~/.vim/bundle/vim-nerdtree-tabs/
filetype plugin indent on

test.txt
def one
  two
end

def two
  three
end

something at the end of the file

script.vim
NERDTreeTabsOpen
" move the cursor out of the NERDTree buffer
winc l
1,3fold
5,7fold
1,3Linediff
5,7Linediff
" change something in the right diff buffer
winc l
normal ciwabc
w
" back to the original test.txt buffer
normal gt
/something

run the command in this folder: vim -u vimrc test.txt -c 'source script.vim'

You have reproduced the bug, if you get an error message saying that 'something' was not found.
By commenting the first line in script.vim 'something' is found.

Okay, I've managed to reproduce it, and I think the key is the setting g:nerdtree_tabs_synchronize_focus. If I set it to 0, the issue doesn't seem to happen. My guess is that, when I switch around buffers, an autocommand catches the event and switches to the NERDTree instead. I can't really think of how I could fix this on my end, to be honest, but if this setting works, it might be the simplest way to avoid this happening.

@AndrewRadev, see :help autocmd-disable. There are 'eventignore' and :noautocmd. Sounds like they might be useful here.

Thanks, @xaizek, it might be something to experiment with. I'm not sure it'll work, since I think it's autocommands that get triggered outside the plugin that do this. Still, it's something to try.


Typo in 2.2.17

Describe the Issue
when doing de audit to rhel 8 with squid installed
TASK [--: 2.2.17 | PATCH | Ensure web proxy server services are not in use | Remove package] ***
fatal: [server: FAILED! => {"msg": "The conditional check 'rhel8cis_rule_2_217' failed. The error was: error while evaluating conditional (rhel8cis_rule_2_217): 'rhel8cis_rule_2_217' is undefined. 'rhel8cis_rule_2_217' is undefined\n\nThe error appears to be in '/etc/ansible/roles/enx_rhel8_cis/tasks/section_2/cis_2.2.x.yml': line 563, column 9, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n  block:\n      - name: "2.2.17 | PATCH | Ensure web proxy server services are not in use | Remove package"\n        ^ here\n"}
Expected Behavior
pass this rule
Actual Behavior
it stops with an error
Control(s) Affected
What controls are being affected by the issue
Environment (please complete the following information):
branch being used: [e.g. devel]
devel
Ansible Version: [e.g. 2.10]
ansible [core 2.15]
Host Python Version: [e.g. Python 3.7.6]
3.10
Ansible Server Python Version: [e.g. Python 3.7.6]
3.10

Additional Details:
offending line:  https://github.com/ansible-lockdown/RHEL8-CIS/blob/241557565deed3b92008a6102c7929e4a47a5df8/tasks/section_2/cis_2.2.x.yml#L552
rhel8cis_rule_2_217 --> rhel8cis_rule_2_2_17

hi @enx-roy-scheepers
Many thanks for this finding and subsequent PR. This has been merged.
Many thanks
uk-bolly

Merged to main
many thanks
uk-bolly


Add spacemacs quick start instructions

I noticed the project said it supported spacemacs, but I coudln't find instructions for how to set it up in the spacemacs environment.
So here are my instructions for how to install nord-emacs using the your spacmacs configuration file ~/.spacemacs.

@arcticicestudio, any chance we can merge this soon?

Thanks for this <3

Closing due to inactivity.


fishOfThieves incompatible with enviromentZ

as it says in the name. polar bears with enviromentZ mod normally drop polar bear fur
with fishOfThieves installed polar bears only drop fish from this mod and never drop polar bear fur

I've tested. Polar Bear Furs are dropped. That's probably a bad luck for you then.


New package: DebugDataWriter v0.1.0

Registering package: DebugDataWriter
Repository: https://github.com/rssdev10/DebugDataWriter.jl
Created by: @rssdev10
Version: v0.1.0
Commit: 89d06c3241eb5bd7c3a7fcbcbcdb3391361b0a7e
Git reference: HEAD
Release notes:

The package provides a simple way for saving debug data into structured separate files in different human readable formats.

Failed due to absence of the [compat] version for JSON package. But there is no difference for the version number of that. JSON is used for human readable serialization only.
[noblock]


Account picker dialog spins forever if no clouds are selected

If a user unselects all "Enable cloud" settings and then opens the account picker dialog then it sits there with a "Loading accounts" spinner forever, without displaying any error. (there is one in the console though)
We should show an error to the user in this case so they know how to fix it (or even better don't let them unselect ALL clouds, always requiring at least one be selected)

Looked into this a bit since I was curious why the "no-floating-promises" rule wasn't catching this, and it's because this is being called through an event handler.
https://github.com/microsoft/azuredatastudio/blob/514b4546436acc838d1ce7af3fc4e829d155c1ca/extensions/azurecore/src/azureResource/tree/connectionDialogTreeProvider.ts#L33
We're calling getAllAccounts here (and correctly awaiting it), but then that's throwing the "No account providers registered" error on https://github.com/microsoft/azuredatastudio/blob/514b4546436acc838d1ce7af3fc4e829d155c1ca/src/sql/workbench/api/common/extHostAccountManagement.ts#L67 So the await above rejects, but there's nothing handling that error. One possible fix would be to make the event handle catch any errors itself and then handle them appropriately.

fixed in #24004


Chrome passing WPT tests for "console.log" scenario

[x] Folder wpt-metadata/webdriver/tests/bidi/log is removed.
[ ] WPT.fyi tests are passing.

UPD: Added the second point to fix the wpt.fyi.

As of 2022-11-23 the WPT log tests have 0 / 45 passes. Hypothesis is it is due to the WPT downloads the old version of ChromeDriver, which should be fixed after ChromeDriver Beta release after 2022-12-01.

@sadym-chromium is there a nightly version of chromedriver available for download? Might be an option to have the Nightly tests on wpt run with that version instead. We do that for geckodriver.

Not officially, but the following trick can be done:

Get the latest version.
Get binary from https://commondatastorage.googleapis.com/chromium-browser-snapshots/index.html?prefix=Linux_x64/{the latest version}/

James actually told me that for Chrome the dev channel is used. As such the idea that I had in my last comment is not doable, and we will have to wait.

So it means that it's actually not that bad and changes in Chrome/chromedriver could be tested earlier? Maybe even for the dev channel?

So it means that it's actually not that bad and changes in Chrome/chromedriver could be tested earlier? Maybe even for the dev channel?

The next dev release should contain the patch. We are prioritizing the early WPT ChromeDriver runs.

UPD: the dev release on 2022-12-15 should contain the patch.

The WPT still uses ChromeDriver wihtout the fix: Starting ChromeDriver 110.0.5464.0. Investigating.

Attempt to fix the ChromeDriver revision issue: https://github.com/web-platform-tests/wpt/pull/37577

The wpt-versions/pinned_chromium_revision is updated. Waiting for the next WPT run.

Looks like this works now:
https://wpt.fyi/results/webdriver/tests/bidi/log/entry_added?label=master&product=firefox[experimental]&product=chrome[experimental]&product=edge[experimental]&product=safari&aligned&view=subtest

Yes, except 3 tests. Should be fixed by this: https://github.com/web-platform-tests/wpt/pull/37675.

Done.


python311Packages.radio-beam: 0.3.4 -> 0.3.6; rename from radio_beam

Description of changes
Diff: https://github.com/radio-astro-tools/radio-beam/compare/v0.3.4...v0.3.6
part of #245383

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 265383 at 056d3fba run on x86_64-linux 1

2 packages marked as broken and skipped:

python310Packages.spectral-cube
python311Packages.spectral-cube

2 packages built successfully:

python310Packages.radio-beam
python311Packages.radio-beam

Result of nixpkgs-review pr 265383 run on x86_64-linux 1

4 packages marked as broken and skipped:

python310Packages.spectral-cube
python310Packages.spectral-cube.dist
python311Packages.spectral-cube
python311Packages.spectral-cube.dist

4 packages built:

python310Packages.radio-beam
python310Packages.radio-beam.dist
python311Packages.radio-beam
python311Packages.radio-beam.dist

Result of nixpkgs-review pr 265383 run on x86_64-darwin 1

4 packages marked as broken and skipped:

python310Packages.spectral-cube
python310Packages.spectral-cube.dist
python311Packages.spectral-cube
python311Packages.spectral-cube.dist

4 packages built:

python310Packages.radio-beam
python310Packages.radio-beam.dist
python311Packages.radio-beam
python311Packages.radio-beam.dist


Add more usable error for new-line chars

I think I've seen it in docs already, but this seems to be a better approach to just explicitly tell a user what to do rather than expecting that the user will dig throught the whole documentation or google for that. Although the error is quite readable for people who work with the python often, it's completly unusable for beginners who encounter kv-lang for the first time and at the same time are python beginners.
I made pep8 fixes too. Please help me test L448, because I'm not sure how to trigger it. ^^

Do you have a test case for this PR? I still get a different error when I use \n in a kv string I load using Builder and a kv file works correctly.

@udiboy1209 What error do you get?
single file:
from kivy.lang import Builder
from kivy.base import runTouchApp
from kivy.uix.boxlayout import BoxLayout
Builder.load_string('''
<Test>:
    canvas:
        Color:
            rgb: 1, 0, 0
        Rectangle:
            size: self.size
            pos: self.pos
    Button:
        text: 'test\ntest'  # triggers the error
''')
class Test(BoxLayout): pass
runTouchApp(Test())

multiple files (main.py, my.kv):
from kivy.app import App
from kivy.lang import Builder
from kivy.uix.boxlayout import BoxLayout
Builder.load_file('answertesting.kv')
class Test(BoxLayout): pass
class My(App):
    def build(self):
        return Test()
My().run()

<Test>:
    canvas:
        Color:
            rgb: 1, 0, 0
        Rectangle:
            size: self.size
            pos: self.pos
    Button:
        text: 'test\ntest'  # wont trigger, probably even shouldnt

@KeyWeeUsr I am running an example similar to the first one. This is the error I get:
[INFO   ] [Logger      ] Record log in /home/udiboy/.kivy/logs/kivy_16-12-07_28.txt
[INFO   ] [Kivy        ] v1.9.2-dev0
[INFO   ] [Python      ] v3.5.1+ (default, Mar 30 2016, 22:46:26)
[GCC 5.3.1 20160330]
[INFO   ] [Factory     ] 193 symbols loaded
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_gif, img_sdl2 (img_pil, img_ffpyplayer ignored)
 Traceback (most recent call last):
   File "kvtest.py", line 14, in <module>
     TestApp().run()
   File "/home/udiboy/.apps/venvs/kivy3/lib/python3.5/site-packages/kivy/app.py", line 802, in run
     root = self.build()
   File "kvtest.py", line 11, in build
     return Builder.load_string(kv)
   File "/home/udiboy/.apps/venvs/kivy3/lib/python3.5/site-packages/kivy/lang/builder.py", line 348, in load_string
     parser = Parser(content=string, filename=fn)
   File "/home/udiboy/.apps/venvs/kivy3/lib/python3.5/site-packages/kivy/lang/parser.py", line 401, in __init__
     self.parse(content)
   File "/home/udiboy/.apps/venvs/kivy3/lib/python3.5/site-packages/kivy/lang/parser.py", line 505, in parse
     objects, remaining_lines = self.parse_level(0, lines)
   File "/home/udiboy/.apps/venvs/kivy3/lib/python3.5/site-packages/kivy/lang/parser.py", line 563, in parse_level
     '%s spaces' % spaces)
 kivy.lang.parser.ParserException: Parser: File "<inline>", line 4:
 ...
       2:Label:
       3:    text: 'Some
 >>    4: very long line which will be cut'
 ...
 Invalid indentation, must be a multiple of 4 spaces

and this is the code:
from kivy.app import App
from kivy.lang import Builder

kv = '''
Label:
    text: 'Some \n very long line which will be cut'
'''

class TestApp(App):
    def build(self):
        return Builder.load_string(kv)

if __name__ == '__main__':
    TestApp().run()

Right, that will be here no matter what. The indentation is because of "\n ", which just puts a single space at the beginning, which messes up the whole thing.
I may try to catch those things too if there is \n

From what I think, there are only two possible errors which can arise from not escaping newlines

Unexpected EOL in string
Indentation error

We can output a warning with the EOL error to check for unescaped newlines and missed ending  quotes '.
And the Indentation error outputs the actual KV text being processed which makes it obvious that the newline needs to be escaped, so we can leave that as it is.
I think this is better than predicting whether there was an unescaped newline. What do you think?

Seems reasonable and shouldn't really slow things down. The edit should be a little bit better catching even the EOL that it didn't before. Hopefully now remains only the indentation error, which I also think is rather more obvious that user broke something and if the user tries to fix the indentation error, sooner or later the EOL will be triggered, therefore even the warning would be visible.

The indentation error shouldn't be given higher priority over unexpected EOL. Because in some cases I also get the error Only one root object is allowed by .kv . Does the parser run through the entire KV once before evaluating it? I think the final step would be to ensure EOL is thrown when it is clearly the cause of problems.

It should now fetch both indentation and root error. I guess there's nothing to slow down at all if an exception is already going to be thrown, so let's at least make something good with it 😋

This reminds me of https://mail.python.org/pipermail/python-ideas/2016-November/043848.html. While error suggestions are good when we know for sure what causes the errors, I think guessing is probably not a good idea overall since it may also mislead. But it's been a while since I looked at lang so I'm not sure if other issues could cause the errors at location they were changed.

Yeah, I saw something similar before writing this, but then again a beginner user of kv lang might not guess that a newline causes the error and that might just backfire with opening issues. Seems like none was opened recently, but I think I saw such question at least on IRC.
I think it's only \n character that causes such issues (that's why the condition for that), but if not, then it raises just a casual error without an additional message.

This change might be useful for beginners, because the output from KV parser might be not very readable in the beginning especially if Python's traceback throws stuff into console too. However, if you feel the change is not really needed, then feel free to close it.

I'll close for now because I do think this could raise exceptions where the error mentioned isn't the true cause. If there were extensive tests I would probably feel more comfortable with this though.

@KeyWeeUsr I am running an example similar to the first one. This is the error I get:
[INFO   ] [Logger      ] Record log in /home/udiboy/.kivy/logs/kivy_16-12-07_28.txt
[INFO   ] [Kivy        ] v1.9.2-dev0
[INFO   ] [Python      ] v3.5.1+ (default, Mar 30 2016, 22:46:26)
[GCC 5.3.1 20160330]
[INFO   ] [Factory     ] 193 symbols loaded
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_gif, img_sdl2 (img_pil, img_ffpyplayer ignored)
 Traceback (most recent call last):
   File "kvtest.py", line 14, in <module>
     TestApp().run()
   File "/home/udiboy/.apps/venvs/kivy3/lib/python3.5/site-packages/kivy/app.py", line 802, in run
     root = self.build()
   File "kvtest.py", line 11, in build
     return Builder.load_string(kv)
   File "/home/udiboy/.apps/venvs/kivy3/lib/python3.5/site-packages/kivy/lang/builder.py", line 348, in load_string
     parser = Parser(content=string, filename=fn)
   File "/home/udiboy/.apps/venvs/kivy3/lib/python3.5/site-packages/kivy/lang/parser.py", line 401, in __init__
     self.parse(content)
   File "/home/udiboy/.apps/venvs/kivy3/lib/python3.5/site-packages/kivy/lang/parser.py", line 505, in parse
     objects, remaining_lines = self.parse_level(0, lines)
   File "/home/udiboy/.apps/venvs/kivy3/lib/python3.5/site-packages/kivy/lang/parser.py", line 563, in parse_level
     '%s spaces' % spaces)
 kivy.lang.parser.ParserException: Parser: File "<inline>", line 4:
 ...
       2:Label:
       3:    text: 'Some
 >>    4: very long line which will be cut'
 ...
 Invalid indentation, must be a multiple of 4 spaces

and this is the code:
from kivy.app import App
from kivy.lang import Builder

kv = '''
Label:
    text: 'Some \n very long line which will be cut'
'''

class TestApp(App):
    def build(self):
        return Builder.load_string(kv)

if __name__ == '__main__':
    TestApp().run()

A fix to this error is by replacing every \n with \\n  and it will compile without errors.
A fix to your code is below:
from kivy.app import App
from kivy.lang import Builder

kv = '''
Label:
    text: 'Some \n\n very long line which will be cut'
'''

class TestApp(App):
    def build(self):
        return Builder.load_string(kv)

if __name__ == '__main__':
    TestApp().run()


Outdated cask: intellij-idea-eap

Link to cask: intellij-idea-eap

New version is: 2017.1 EAP 171.3780.15

$ brew cask _stanza version intellij-idea-eap
2016.3.5,163.13906.16
$ brew cask install intellij-idea-eap
==> Downloading https://download.jetbrains.com/idea/ideaIU-163.13906.16.dmg
######################################################################## 100.0%
curl: (22) The requested URL returned error: 403 Forbidden
Error: Download failed on Cask 'intellij-idea-eap' with message: Download failed: https://download.jetbrains.com/idea/ideaIU-163.13906.16.dmg
The incomplete download is cached at /Users/myuser/Library/Caches/Homebrew/Cask/intellij-idea-eap--2016.3.5,163.13906.16.dmg.incomplete
Error: nothing to install

Some relevant links pointing to the new version:
Old (dead) EAP link: https://confluence.jetbrains.com/display/IDEADEV/IDEA+2016.3+EAP
New EAP location:    https://confluence.jetbrains.com/display/IDEADEV/EAP
New EAP landing page: https://www.jetbrains.com/idea/nextversion/
Direct download link: https://download-cf.jetbrains.com/idea/ideaIU-171.3780.15.dmg
SHA 256 checksum: 26071b76c684797a3720fbcec116c0616950ee0d42e762215ebe188c39de9277
I've followed all the instructions here, before reporting this issue.

Just Submitted a PR with update for this https://github.com/caskroom/homebrew-versions/pull/3446

@arvindsv First of all You reported issue to wrong repository, all EAPs lives in caskroom/homebrew-versions and there is intellij-idea-next-eap which You should use.
EAP for current release (2016.3) of IntelliJ IDEA is not dead until release of next version (2017.1).

Ok, thank you. Good to know.


Architecture diagram

The paper has been "coming soon" for ages, so draw up an architecture diagram to better understand the architecture.

This is the print output of the pretrained model, maybe add the last linear layer to the diagram? Make Attention and MLP diagrams?
Pretrained falcon model: RWForCausalLM(
(transformer): RWModel(
(word_embeddings): Embedding(65024, 4544)
(h): ModuleList(
(0-31): 32 x DecoderLayer(
(input_layernorm): LayerNorm((4544,), eps=1e-05, elementwise_affine=True)
(self_attention): Attention(
(maybe_rotary): RotaryEmbedding()
(query_key_value): Linear(in_features=4544, out_features=4672, bias=False)
(dense): Linear(in_features=4544, out_features=4544, bias=False)
(attention_dropout): Dropout(p=0.0, inplace=False)
)
(mlp): MLP(
(dense_h_to_4h): Linear(in_features=4544, out_features=18176, bias=False)
(act): GELU(approximate='none')
(dense_4h_to_h): Linear(in_features=18176, out_features=4544, bias=False)
)
)
)
(ln_f): LayerNorm((4544,), eps=1e-05, elementwise_affine=True)
)
(lm_head): Linear(in_features=4544, out_features=65024, bias=False)
)


graalvm8-ee: 19.1.1 -> 19.2.0

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @hlolli @NeQuissimus

19.2.0 has a regression with unpacking .so-files from jars
It is not 100% reproducible, sometimes those programs work, sometimes crash with:
java.lang.UnsatisfiedLinkError: Can't obtain InputStream for com/sun/jna/linux-x86-64/libjnidispatch.so
        at com.sun.jna.Native.loadNativeDispatchLibraryFromClasspath(Native.java:962)
        at com.sun.jna.Native.loadNativeDispatchLibrary(Native.java:922)
        at com.sun.jna.Native.<clinit>(Native.java:190)

I am not sure if the problem is NixOS-only or global, I am using graalvm only on NixOS.
19.1.1 has no problem, so downgrade is the solution for one who hit the issue and has no time for investigating it.


csi-provisioner rbac add resources nodes get, list, watch

csi-provisioner 1.0 implement support VolumeNodeAffinity, it need  resources nodes get, list, watch
Fixes #293

#293

@huaizong thanks for contribution, can you fix the same thing in external-provisioner too?

@huaizong thanks for contribution, can you fix the same thing in external-provisioner too?

@rootfs external-provisioner as the required changes already https://github.com/kubernetes-csi/external-provisioner/blob/master/deploy/kubernetes/rbac.yaml#L50

@huaizong thanks for contribution, can you fix the same thing in external-provisioner too?

external-provisioner as the required changes at branch master, but not at branch release-1.0
https://github.com/kubernetes-csi/external-provisioner/blob/release-1.0/deploy/kubernetes/rbac.yaml#L42
https://github.com/kubernetes-csi/external-provisioner/blob/master/deploy/kubernetes/rbac.yaml#L50

lgtm


Control_L to Control_L (+ when you type Control_L only, send Escape)

Is there a way I can get a command like that above working with the new JSON format?

https://github.com/tekezo/Karabiner-Elements/issues/8 :)

Is it possible then?

Hi, Is this being worked on?

Or, maybe this should be closed as dupe of #8?

This feature is implemented in a fork #247.

Sorry for adding noise. Is this feature implemented or I still need to install wwwjfy/Karabiner-Elements fork?

It is implemented in the latest karabiner-elements. You take can take guidance from the main README.md and docs.

See https://pqrs.org/osx/karabiner/complex_modifications/#caps_lock 👍


As HE in MIB, no option to concede to ME at top level

since then you'd immediately lose

Resolved in #650


docs: extend client generation with buf

### Definition of Ready

- [ ] I am happy with the code
- [ ] Short description of the feature/issue is added in the pr description
- [ ] PR is linked to the corresponding user story
- [ ] Acceptance criteria are met
- [ ] All open todos and follow ups are defined in a new ticket and justified
- [ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
- [ ] No debug or dead code
- [ ] Critical parts are tested automatically
- [ ] Where possible E2E tests are implemented
- [ ] Documentation/examples are up-to-date
- [ ] All non-functional requirements are met
- [ ] Functionality of the acceptance criteria is checked manually on the dev system.

Can somebody review this please @mffap @hifabienne @dakshitha

Yes it's on my list of things, no need to ping so soon :)

It was meant to not only ping you but rather others 😁


DST functionality in iran

From 21 March 2023, the government of Iran will no longer move the clock forward
Therefore, please give an update to the library

This change was released in the IANA time zone database version 2022b, which is included in moment-timezone package version 0.5.36. If you're not seeing the correct times, you might need to upgrade your version of moment-timezone.


[HOLD for payment 2024-11-18] [HOLD for payment 2024-10-17] [Direct Feeds] [Backend] Create and remove beta for direct feeds

This is part of the Third party workspace feeds V2 (direct connections) project.
Implement the canUseDirectFeeds beta for direct feeds and then clean it up
Issue OwnerCurrent Issue Owner: @joekaufmanexpensify

Not overdue

This can be monthly

not there yet

No need checklist

$125 to @DylanDylann for testing

Jus removing the beta so no need for checklist

Will do!

I think we can close it and handle payment later

I have already noted all polish issues in the third-party workspace feed that I worked on. And we can handle all payments in one after allis  done

Works for me! Closing then


[BUG] fvm 2.2.5 cannot find dart

fvm running error
okello@okello-ThinkPad-X240:~$ fvm
/home/okello/.pub-cache/bin/fvm: 8: dart: not found
okello@okello-ThinkPad-X240:~$

Describe the bug
Installation of fvm as per the official documentation using the global package installation style.
dart pub global activate fvm
installation is successful even path configuration but when running fvm command i get dart not found.
Attempts

Edit path configuration in .bashrc
Run dart pub cache repair
dart pub global deactivate fvm
dart pub global activate fvm

Desktop (please complete the following information):

OS: Ubuntu 20.04
FVM Version : 2.2.5

Flutter Doctor
okello@okello-ThinkPad-X240:~$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.5.0, on Ubuntu 20.04.3 LTS 5.11.0-41-generic,
locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] Connected device (1 available)
• No issues found!
okello@okello-ThinkPad-X240:~$ flutter -v doctor
[✓] Flutter (Channel stable, 2.5.0, on Ubuntu 20.04.3 LTS 5.11.0-41-generic,
locale en_US.UTF-8)
• Flutter version 2.5.0 at /home/okello/Flutter/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 4cc385b4b8 (3 months ago), 2021-09-07 23:01:49 -0700
• Engine revision f0826da7ef
• Dart version 2.14.0
[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
• Android SDK at /home/okello/Android/Sdk
• Platform android-32, build-tools 32.0.0
• Java binary at: /opt/android-studio/jre/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
• All Android licenses accepted.
[✓] Chrome - develop for the web
• Chrome at google-chrome
[✓] Android Studio (version 2020.3)
• Android Studio at /opt/android-studio
• Flutter plugin version 63.0.1
• Dart plugin version 203.8452
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
[✓] Connected device (1 available)
• Chrome (web) • chrome • web-javascript • Google Chrome 96.0.4664.93
• No issues found!
okello@okello-ThinkPad-X240:~$

I would like to close this issue since i was able to find a solution by reading through this answer 1 in stack overflow post https://stackoverflow.com/questions/66868564/command-not-found-pub-after-flutter-installation


Allow passing absolute path to seed.import

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

I'll redo this on alpha.


improving the ui of hacktoberfest page buttons

Is your feature request related to a problem? Please describe.
the hacktoberfest repo buttons are not visible.

Describe the solution you'd like.
i would like to improve the ui and add hover effect and better font color and design for better user experience
Describe alternatives you've considered.
improving font color weight size and background color
Add any other context or screenshots about the feature request here.
issue:

Sure @mridulsaggi

I would like to work on this issue, please assign it to me under GSSoC'23.

@MohamedMafaz Please give this issue to me. I possess all the abilities this bug needs. I participate in GSSoC 2023. Kindly put this in GSSoC 2023.

I would like to work on this issue under GSSoC'23.Assign this issue to me under GSSoC'23.

@Kapileswar-Moharana i have already been assigned this issue and i have made the changes and already made the pull request.


Rename rollover crates

The v_{1,2}_0_0 suffixes are noisy and too verbose. And using the word "deprecated" correctly flags the older version as one that will be phased out soon.
We leverage pub use current::* so that we don't have to add current as part of any imports.

If a version 3 of rollover happens, would support for v1 be removed?

Yep, that's why I think this is better: I think we are only ever going to support to versions of any given protocol at the same time, according to ADR019.

bors r+


implement --format-json JSON option. It's usefull for post-processing…

This implementation limit dependencies as json dependency will not be required except if you try to export as JSON. In order to allow this, the import json has been moved to the unique method requiring json : JsonPrettyTable.get_string()

@raimon49 the CI fails for python 3.3 for an unknown reason - not related to the current MR. I may help if requested, but I really don't know the reason for this failure

@lebouquetin Thanks for the good JSON support.
It fails in Python 3.3, probably because it dropped 3.3 support in pip. pip-licenses will also drop support for Python 3.3 in the near future.
I am planning to merge your patch.

@lebouquetin Released version 1.8.0 with your added option enabled.
https://pypi.org/project/pip-licenses/1.8.0/
Thanks for your contribution.


Buy more servers you cowards

Is your feature request related to a problem? Please describe.
You haven't bought enough servers and there's hardly ever an available one open. I love these servers but can only get on them once in a blue moon, often on a server halfway around the world.
Describe the solution you'd like
Please, if possible, buy at least two or three more servers in the US. They're great and I just want to be able to join one more often. I'm dying out here.
Describe alternatives you've considered
An alternative would be to maybe convert some existing servers to 32 player, again if possible.
Additional context
If any of the Dane staff (or Dane himself) end up actually seeing this, thank you so much for all you do, and keep up the great work (Hopefully with a few more servers?)

It’s up to Dane if we add more servers… not really a money issue. He just doesn’t want too many servers with no players on them, so we carefully add more servers as the existing ones are consistently full throughout the day.

If the servers are full only during peak hours, then we won’t add more until all are full for more of the day


Use web test to implement a functional test suite

We need some functional tests to ensure that Warehouse functions at a high level. Ideally this would use webtest to avoid needing to parse http and simply use the wsgi application.

I'd balance between selenium (backed by robot framework) and webtest depending on amount of JavaScript we have. I don't think we'll have that much so webtest should make it easy.

This has been started now, so I'm going to go ahead and close this.


template-haskell 2.17

The new template haskell in ghc 9 offered these migration instructions for TyVarBndr
https://gitlab.haskell.org/ghc/ghc/-/wikis/migration/9.0#template-haskell-217
I've naively attempted to resolve this by setting TyVarBndrSpec here
https://github.com/locallycompact/composite/commit/73cbba09d166187b5a6ae3343741e1ee61a52edf
But I get one type error.
composite-base                > /home/lc/Source/github.com/composite/composite-base/src/Composite/TH.hs:163:59: error:
composite-base                >     • Couldn't match type ‘()’ with ‘Specificity’
composite-base                >       Expected: Control.Lens.Getter.Getting
composite-base                >                   (base-<IP_ADDRESS>:Data.Semigroup.Internal.Endo
composite-base                >                      [(Name, [TyVarBndrSpec], Type)])
composite-base                >                   [Dec]
composite-base                >                   (Name, [TyVarBndrSpec], Type)
composite-base                >         Actual: ((Name, [Language.Haskell.TH.Lib.Internal.TyVarBndrUnit],
composite-base                >                   Type)
composite-base                >                  -> Data.Functor.Const.Const
composite-base                >                       (base-<IP_ADDRESS>:Data.Semigroup.Internal.Endo
composite-base                >                          [(Name, [TyVarBndrSpec], Type)])
composite-base                >                       (Name, [Language.Haskell.TH.Lib.Internal.TyVarBndrUnit], Type))
composite-base                >                 -> [Dec]
composite-base                >                 -> Data.Functor.Const.Const
composite-base                >                      (base-<IP_ADDRESS>:Data.Semigroup.Internal.Endo
composite-base                >                         [(Name, [TyVarBndrSpec], Type)])
composite-base                >                      [Dec]
composite-base                >     • In the first argument of ‘toListOf’, namely ‘(each . _TySynD)’
composite-base                >       In the second argument of ‘(.)’, namely ‘toListOf (each . _TySynD)’
composite-base                >       In the second argument of ‘(.)’, namely
composite-base                >         ‘map fieldDecMay . toListOf (each . _TySynD)’
composite-base                >     |
composite-base                > 163 |   let fieldDecs = catMaybes . map fieldDecMay . toListOf (each . _TySynD) $ decs

I can't really see what's going on here, but _TySynD doesn't look like something I want to change, but if I change these all to TyVarBndrUnit, then I get an error trying to use ForallT, which also doesn't look like like something I want to change.
composite-base                > /home/lc/Source/github.com/composite/composite-base/src/Composite/TH.hs:220:30: error:
composite-base                >     • Couldn't match type ‘()’ with ‘Specificity’
composite-base                >       Expected: [TyVarBndr Specificity]
composite-base                >         Actual: [TyVarBndr ()]
composite-base                >     • In the first argument of ‘ForallT’, namely ‘lensBinders’
composite-base                >       In the second argument of ‘SigD’, namely
composite-base                >         ‘(ForallT lensBinders lensContext lensType)’
composite-base                >       In the expression:
composite-base                >         SigD lensName (ForallT lensBinders lensContext lensType)
composite-base                >     |
composite-base                > 220 |     , SigD lensName (ForallT lensBinders lensContext lensType)
composite-base                >     |                              ^^^^^^^^^^^
composite-base                >
composite-base                > /home/lc/Source/github.com/composite/composite-base/src/Composite/TH.hs:240:31: error:
composite-base                >     • Couldn't match type ‘()’ with ‘Specificity’
composite-base                >       Expected: [TyVarBndr Specificity]
composite-base                >         Actual: [TyVarBndr ()]
composite-base                >     • In the first argument of ‘ForallT’, namely ‘prismBinders’
composite-base                >       In the second argument of ‘SigD’, namely
composite-base                >         ‘(ForallT prismBinders prismContext prismType)’
composite-base                >       In the expression:
composite-base                >         SigD prismName (ForallT prismBinders prismContext prismType)
composite-base                >     |
composite-base                > 240 |     , SigD prismName (ForallT prismBinders prismContext prismType)

I went the long way round, I tried th-abstractions but it didn't seem to make since since we were using a type synonym binding, and that couldn't be then made to be polymorphic. This at least type checks but I haven't tested the new version.
https://github.com/ConferOpenSource/composite/pull/49


Issue With CodeMirror & Fighwheel

When figwheel hot reloads, submitting to format no longer works. I think this is because the "state" object is lost and there is a reference to the CodeMirror Editor Instance there.
This is only a problem with development workflow.

This shouldn't have a major effect on production builds but will cause a re-deployment.


west_commands: core: run_telnet_client() ignores Ctrl+C

Describe the bug
While improving the west rtt command, I ran into an issue where everything worked successfully, but when I pressed Ctrl+C (which sends SIGINT / KeyboardInterrupt), nc, openocd, ninja, cmake and other processes were not terminated correctly.
I traced this to netcat (nc) which ignores SIGINT, (at least under Ubuntu 22.04).

What target platform are you using? Any
What have you tried to diagnose or workaround this issue? traced the problem to run_telnet_client()
Is this a regression? unsure

To Reproduce
Steps to reproduce the behavior:

use west rtt in (insert link to PR)
press Ctrl+C
See error (shown in "logs and console output")

Expected behavior
Child processes of west (including nc) would be terminated
Impact
It's kind of a showstopper, since attempting to run more than 1 invocation of west rtt will fail until the user manually terminates all of the lingering processes from the previous run. Note: this isn't a problem with west rtt, but with run_telnet_client().
Logs and console output
Successfully running west rtt (via build)
$ west build -p -S rtt-console -b nucleo_l496zg  -t rtt samples/hello_world/
...
target halted due to debug-request, current mode: Thread
xPSR: 0x01000000 pc: 0x080008a4 msp: 0x20001640
Start address 0x080008a4, load size 17250
Transfer rate: 18 KB/sec, 1725 bytes/write.
Info : Unable to match requested speed 500 kHz, using 480 kHz
Unable to match requested speed 500 kHz, using 480 kHz
Info : Unable to match requested speed 500 kHz, using 480 kHz
Unable to match requested speed 500 kHz, using 480 kHz
Info : rtt: Searching for control block 'SEGGER RTT'
rtt: Searching for control block 'SEGGER RTT'
Info : rtt: Control block found at 0x20000410
rtt: Control block found at 0x20000410
Info : Listening on port 5555 for rtt connections
Listening on port 5555 for rtt connections
Detaching from program: /home/cfriedt/zephyrproject/zephyr/build/zephyr/zephyr.elf, Remote target
[Inferior 1 (Remote target) detached]
Debugger detaching: resuming execution
Warn : target not halted
Error executing event gdb-detach on target stm32l4x.cpu:

Info : dropped 'gdb' connection
Info : accepting 'rtt' connection on tcp/5555
*** Booting Zephyr OS build v4.0.0-rc3-3-gfba2b206c8ae ***
Hello World! nucleo_l496zg/stm32l496xx

Leftover processes after running west rtt and terminating it with Ctrl+C
$ ps -Afw | grep "nc\|telnet\|openocd\|rtt"
...
cfriedt    61034       1  0 09:39 pts/7    00:00:00 /usr/bin/cmake --build /home/cfriedt/zephyrproject/zephyr/build --target rtt
cfriedt    61035   61034  0 09:39 pts/7    00:00:00 /usr/bin/ninja rtt
cfriedt    61036   61035  0 09:39 pts/7    00:00:00 /usr/bin/cmake -E env /opt/zephyr/.venv/bin/python3 -m west rtt
cfriedt    61039   61036  1 09:39 pts/7    00:00:00 /opt/zephyr/.venv/bin/python3 -m west rtt
cfriedt    61484   61039  2 09:39 ?        00:00:00 /opt/zephyr/zephyr-sdk-0.17.0/sysroots/x86_64-pokysdk-linux/usr/bin/openocd -s /home/cfriedt/zephyrproject/zephyr/boards/st/nucleo_l496zg/support -s /opt/zephyr/zephyr-sdk-0.17.0/sysroots/x86_64-pokysdk-linux/usr/share/openocd/scripts -f /home/cfriedt/zephyrproject/zephyr/boards/st/nucleo_l496zg/support/openocd.cfg -c tcl_port 6333 -c telnet_port 4444 -c gdb_port 3333 -c init -c targets -c halt
cfriedt    61634   61039  0 09:39 pts/7    00:00:00 nc localhost 5555
...

Also a problem with the JLink runner
> ps -Afw | grep -i "jlink\|rtt"
cfriedt   969192       1  1 16:09 pts/4    00:00:00 JLinkRTTLogger -Device NRF52833_XXAA -If SWD -Speed 4000 -RTTChannel 0 -USB<PHONE_NUMBER>76 /tmp/rtt_output.log
cfriedt   969198  969192  0 16:09 pts/4    00:00:00 [JLinkGUIServerE] <defunct>
cfriedt   969245  958956  0 16:09 pts/4    00:00:00 grep --color=auto -i jlink\|rtt

Environment (please complete the following information):

OS: Ubuntu 22.04 (and likely others)
Toolchain: Zephyr SDK
Commit SHA: c50777a8431 (v4.0.0-rc3)

Additional context
Discovered while working on #81156

I think I'll separate the fix for this from the other issue. No need to tie them together.


LongPolling always returns 'Rate limit reached' error

LongPolling always returns error_code: 29, error_message: 'Rate limit reached'.
Callback API works fine though, but it would be nice to quickly test some ideas with longpolling without setting up a server.

It was a VK server-side problem, LongPolling works fine now.


test new functions in post_processing

          test these functions:

get_pnl
get_pnl_no_mock
get_pnl_excluding_agent_0
get_pnl_excluding_agent_0_no_mock

Originally posted by @wakamex in https://github.com/element-fi/elf-simulations/pull/161#discussion_r1097784895

functions are removed; issue is no longer relevant


Add rounded style on navigation component.

Description
Add rounded-r-xl style on navigation component.
Issue reference
null
Checklist

[x] Code compiles correctly
[x] Created/updated tests
[x] Extended the documentation

Thanks for your PR. Please provide the rounded navigation screenshot. This will improve the review efficiency.
And why you set it rounded? We designed a outer border for it.

Hi @doddgu , screenshot attached.
This improvement is used to make the style of navigation component more uniform with other components.

@CuteLeon Hi, flat is by design. If you like rounded, you can add a settings for it.

Hi, @doddgu, I Implemented a NavigationStyle config to control navigation bar style. And kept default option as current Flat approuch.

@CuteLeon Indent 8 spaces?

My IDE replaced white-space characters automatically, I have reverted them;

@CuteLeon Indent 8 spaces?

Online Demo cannot setting rounded style？

@pccai let me check

@pccai Updated.


Highlight still visible after calling Window.makeWindowFullyTransparent()

Calling Window.makeWindowFullyTransparent() (introduced in #48) is supposed to make the window fully transparent (with only flutter-drawn widgets, the title, and the traffic light buttons remaining visible), however, a thin, white line, which is supposed to represent a highlight on top of the window, still remains visible:

Since this is unintended, it is currently considered a bug. Ideally, there would be methods called Window.enableHighlight() and Window.disableHighlight() which could be called by Window.makeWindowFullyTransparent() to remove the line, however, I could not find a way to achieve this.
For this reason, I am opening this issue with a “help wanted” label. If anybody knows of a way to disable the highlight of an NSWindow within Swift, please reach out to me.

window_manager's setAsFrameless removes the highlight (though not the border at the left). You might wanna look at that as a start.

Don't worry about the border, that one is caused by the visual effect subview of the sidebar. setAsFrameless removes the .titled styleMask, however, doing so also has other effects, such as removing the traffic light buttons and the window's title, as well as (I believe) preventing the window from receiving keyboard events. That said, adding a function that allows the user to modify the styleMask wouldn't hurt, I suppose.


Add --enable-static configure flag to compile a statically linked binary

Fixes #630.
Usage:
$ ./autogen.sh
$ ./configure --enable-static
$ make

thanks!

released in 1.6.3

released in 1.6.3


build(deps): upgrade open-vulnerability-clients

nvd-lib, gh-advisory-lib, and data-source have been combined into open-vulnerability-clients
Several imports have been updated as classes were moved
NvdCveApi has been repackaged as NvdCveClient
Unit tests have been updated to correctly compare double values

In addition to the above changes, the updated open-vulnerability-clients has improved the error handling for both the GHSA and NVD APIs.

I'll update the PR to resolve the errors within a day or two.

Thanks for the PR @jeremylong, much appreciated!
The failures appear to be due to flakiness of the workflow. All tests succeed locally. The second failure was related to SonarCloud. I just triggered it again, let's see what happens.

@nscuro I will be submitting another PR in a day or two. I wasn't happy with the Float data types not being precise (duh) in the unit tests. The open-vulnerability-clients version 4.0.1 has been released that changed all the Float data types to BigDecimal and added data feeds for the CISA Known Exploited Vulnerability Catalog and First's EPSS data.


Channels not specified with new "Quickstart" install

Heyo guys
So I am currently unable to start Jasper becase of this error :
Traceback (most recent call last):
  File "/home/pi/jasper/jasper.py", line 148, in <module>
    app.run()
  File "/home/pi/jasper/jasper.py", line 115, in run
   self.mic.say(salutation)
  File "/home/pi/jasper/client/mic.py", line 262, in say
    self.speaker.say(phrase)
  File "/home/pi/jasper/client/tts.py", line 472, in say
    self.play_mp3(tmpfile)
  File "/home/pi/jasper/client/tts.py", line 100, in play_mp3
    wav.setframerate(mf.samplerate())
  File "/usr/lib/python2.7/wave.py", line 358, in setframerate
   raise Error, 'bad frame rate'
wave.Error: bad frame rate
Exception wave.Error: Error('# channels not specified',) in <bound method Wave_write.__del__ of
<wave.Wave_write instance at 0xfdc2b0>> ignored

aplay and arecord both show my sound devices (jack audio & usb mic) and both are working (tried them with recording something with arecord and playback with aplay).
Can anyone help me ?
Thx for your responses

Fixed it with changing tts from google to ivona


Fix bug in Download-BcNuGetPackageToFolder

Error occurs when using -downloadDependencies with value 'own' because of an invalid reference
This fixes that by correcting the reference, allowing to download dependencies where the author matches

Thanks


Encrypt password used for binddn

Hello!
I have installed the plugin successfully, but realised that the password for the bind is stored in clear in the DB. Could it be possible to change current way of storing bind password, or let user bind with its own credentials?
Because my sites are reachable through FTP, a user with bad intentions could potentially upload PHP code allowing him to get the LDAP info, including bind account credentials and further request other info on the LDAP server using that account. I would like to prevent that.

Hi.
Yes, I can see that that is a potential security hole. But: What way would you prefer to use for encrypting the password for the initial bind? Because at one point it definitely needs to be decrypted. So a malicious user could also upload a script that hooks into the decryption and can still gain access to the cleartext password. At least to my knowledge. That for sure is a risk I am aware of.
But having other people upload untrusted files is per se a security risk. How do you know they don't upload a script that hooks into wordpress' authentication filter and passes the users password along with their username. That is a far more serious issue than gaining access to a password that should only be usable from a certain machine...
So I'm not really seing an absolutely secure way of storing the password (that is needed in cleartext for the ldap_bind) in a way that someone with access to the server can't read it. So as long as there is no secure way I'm not going to elude someone into a false sense of security.  I might be wrong on that and if you can correct me that'd be great.

Thanks for the quick reply, really appreciated.
As I was looking more into this, I was reaching to the same conclusions.
The only way I would see this working would be do the bind on the fly with the user credentials. That would imply to store somewhere what the RDN should be for users so that it can be appended to the username when submitting credentials. And then, basically using the user's credentials to bind and validate itself.
No binddn credentials stored any more.
I realise this would probably induce a big change in the code though...
Maybe I should look into other possibilities, but it looks like allowing users to manage plugins is pretty much letting them do anything they want one way or another. I am just starting on WordPress, I could appreciate any insights you might have on this.

You will still need binddn-credentials to search for the DN of the user as user can be stored somewhere within the directory-tree. Therefore you need to bind to be able to search for the user that tries to log in. As by now most LDAP-servers don't allow an anonymous bind you will need the binddn that is used to sesrch for the user that tries to log in.
Only when you know exactly where the users are located within the Directory tree and the users are using the last part of the DN to log in you might be able to skip that first bind and only use the users credentials. But that is explicitly not the scope of this plugin. This plugin gives you the freedom to use whatever directory structure that you might encounter and whatever you want the user to use for logging in. This freedom though comes with the price that you need a binddn.
You (or your directory admin) could create a user for this initial binding that can only read the necessary attributes and only allow binds from the WordPress-server to the LDAP to not allow people to read other informations once they gain access to the server.

Hmm, I understand. Indeed my idea was to dramatically reduce the scope of the plugin in order to keep both the plugin and the ability to install/delete plugins for my users. But as you point out this would be counter-productive.
I can look further into LDAP ACLs to see how far I could get reducing the rights of the bind user a bare minimum. Otherwise my guess is to either:

remove the authLDAP plugin and use local user + FTP / plugins access
or keep the plugin and prevent ftp access + control which plugins could be installed.

In any case, thank you for the prompt answers, and kudos for the plugin, it works asolutely great, and I realise that the limitation I am facing come from WordPress design itself.

Hey,
A bit of a feedback. I've been playing with LDAP ACL following your suggestion and I think I pinned it. I put it here in case it may prove useful for others:

I created a special entry for the LDAP bind: cn=wordpress,ou=bind,dc=example,dc=com
I then added an ACL rule and edited the next one (after the first one filtering access to passwords) :

olcAccess: {1}to dn.subtree="ou=people,dc=example,dc=com" attrs=entry,uid by ssf=128 dn="cn=administrator,,dc=example,dc=com" write by ssf=128 dn.children="ou=bind,dc=example,dc=com" read by ssf=128 users read by ssf=128 anonymous auth by * none
olcAccess: {2}to dn.subtree="ou=people,dc=example,dc=com" by ssf=128 dn="cn=administrator,dc=example,dc=com" write by ssf=128 dn.children="ou=bind,dc=example,dc=com" none by ssf=128 users read by ssf=128 anonymous auth by * none

Now I can use that user to search for a uid, and that's all really. It is also not possible to filter all uids using the bind user. That looks ok! Thanks again!


Possible to add hints as to what stations will sound like?

It'd be nice to not have to go to https://soma.fm to see what a station will sound like (station names rarely help so I end up just listening to one or two):

Is it asking too much to add these hints (on hover or a little clickable info icon) as to what a station will sound like?
If you're interested in something like this I could probably help by converting the select box to a div dropdown and adding in the relevant descriptions. If not, no biggie, still a cool extension. 👏

I agree! Beyond just station descriptions, showing some artist names like on http://somafm.com/groovesalad/artists.html would be helpful. Hat tip to @rustyhodge for pointing me at this page.

I could probably help by converting the select box to a div dropdown

I have the station descriptions as tooltips in HuxleyFM in a div menu instead of a select menu, so the code could be copied from there, if you want.

tbh, I listen to Soma because I don't know any of the musicians so the station descriptions would be better for me anyway. I'm also not sure what the UI of a huge musician list in a small Chrome extension would look like.
I didn't realize there was a desktop app for this. I'd definitely switch if there was a way to minimize it to the taskbar, but I suppose that should be it's own issue/request.


A multi-language README

Hi!, I think geoserver-node-client is an awesome Project ! if it has a mult-language version ReadME, it can be known by more developer from many other countries~

Hi @MrCare thanks for your interest.
Do you only want to translate the README, or the whole documentation of the library?
I am not sure if translating the README would help much, because the documentation of the library is in English as well, and also the GeoServer documentation is only available in English (as far as I know). Moreover, keeping translations in sync is quite difficult to my experience.

Hi @JakobMiksch ! I can start from README ! I am a web developer from China devoted to Smart Agriculture. We use drones to get data from farm and then use some openSource software to publish the Data and do analysis. Geoserver is one of these openSource softwares, As far as I know Geoserver is a lot popular in developers from China~, however they always use Java a lot（there is a project named geoserver-manager). I think this node.js project is much valuable , it means A JavaScript developer team can use one single language to cover both front end and back end! it's very suitable to a 3-5 people team!

I use this project to achieve the tiff publish function like above!

Thanks for sharing your use cases. Great to hear that many people use this project.
What do you think about creating an additional README file Readme_zh-CN.md that we can link from the main README?
As it is done here: https://github.com/tj/commander.js

@MrCare are there any updates on this? see my comment above.


Investigate SNI and Host header mismatch behavior

See https://twitter.com/hanno/status/859796242678902786

I am not sure why we are not setting SNI_server_name for interfaces with only one vhost. (if it's not named, then sure. But when it has a name, we still ignore it… why?!)


[TACHYON-1650] Add the Chinese version of documentation page 'Running-Tachyon-Fault-Tolerant-on-EC2'

https://tachyon.atlassian.net/browse/TACHYON-1650
Add the Chinese version of documentation page 'Running-Tachyon-Fault-Tolerant-on-EC2'

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

Thanks for the translation. I left one question.

@yupeng9 I also see this. In fact, many pages share some sentences, and it just happens that this sentence seems to be longer. Now our design is sharing same code fragments and table elements in the same pages of different languages. Thus for different pages of one particular language they are independent. If they also share snippets under _includes/, All pages may get tightly coupled. What's more, maybe this sentence is little longer so it seems necessary to extract it out. However, what about a shorter sentence (since many short sentences are shared among different pages)? I think it's better to follow a simple rule, rather than depending on the sentence' length.

Fair points. Thanks.

LGTM


How to know that my data set is well done?

I am trying to train a model to use it with EasyOCR in an ANPR project, as a first test I used a dataset that is provided in the EasyOCR repository, this is an example of the images used.

The dataset contains approximately 1000 similar images in different sizes, after a training of 10,000 iterations I got poor results but since it was a training test I didn't give it more importance.
After that I tried to train my own model following the same steps I used to train the previous model, create my .txt file with the labels and run the script to create the mdb dataset, but after training I don't get any result, it stays on the same prediction and it never gets better.
Example of images:

Another thing I noticed is that after the training script runs the dataset filter, I only had about 100 samples left out of 500 that the dataset has.
The training seems as if it was stuck since it never achieved a correct prediction, unlike with the first dataset, the training after 2000 iterations already gave good results.
These are my logs and results of failure training.
exp_name: None-VGG-BiLSTM-CTC-Seed1111
train_data: data
valid_data: data/val
manualSeed: 1111
workers: 0
batch_size: 192
num_iter: 10000
valInterval: 100
saved_model:
FT: False
adam: False
lr: 1
beta1: 0.9
rho: 0.95
eps: 1e-08
grad_clip: 5
baiduCTC: False
select_data: ['train']
batch_ratio: ['1']
total_data_usage_ratio: 1.0
batch_max_length: 25
imgH: 32
imgW: 100
rgb: False
character:<PHONE_NUMBER>-abcdefghijklmnopqrstuvwxyz
sensitive: False
PAD: False
data_filtering_off: False
Transformation: None
FeatureExtraction: VGG
SequenceModeling: BiLSTM
Prediction: CTC
num_fiducial: 20
input_channel: 1
output_channel: 512
hidden_size: 256
num_gpu: 0
num_class: 38
dataset_root: data
opt.select_data: ['train']
opt.batch_ratio: ['1']
dataset_root:    data	 dataset: train
sub-directory:	/train	 num samples: 102
num total samples of train: 102 x 1.0 (total_data_usage_ratio) = 102
num samples of train per batch: 192 x 1.0 (batch_ratio) = 192
Total_batch_size: 192 = 192
dataset_root:    data/val	 dataset: /
sub-directory:	/.	 num samples: 1
[8400/10000] Train loss: 0.00001, Valid loss: 9.07022, Elapsed_time: 46794.15422
Current_accuracy : 0.000, Current_norm_ED  : 0.14
Best_accuracy    : 0.000, Best_norm_ED     : 0.14
Ground Truth              | Prediction                | Confidence Score & T/F
4nt5526                   | 2103j6                    | 0.0474	False
[8500/10000] Train loss: 0.00001, Valid loss: 9.07421, Elapsed_time: 47353.40529
Current_accuracy : 0.000, Current_norm_ED  : 0.14
Best_accuracy    : 0.000, Best_norm_ED     : 0.14
Ground Truth              | Prediction                | Confidence Score & T/F
4nt5526                   | 2103j6                    | 0.0477	False
[8600/10000] Train loss: 0.00001, Valid loss: 9.07817, Elapsed_time: 47907.41048
Current_accuracy : 0.000, Current_norm_ED  : 0.14
Best_accuracy    : 0.000, Best_norm_ED     : 0.14
Ground Truth              | Prediction                | Confidence Score & T/F
4nt5526                   | 2103j6                    | 0.0479	False
[8700/10000] Train loss: 0.00001, Valid loss: 9.08219, Elapsed_time: 49519.87970
Current_accuracy : 0.000, Current_norm_ED  : 0.14
Best_accuracy    : 0.000, Best_norm_ED     : 0.14
Ground Truth              | Prediction                | Confidence Score & T/F
4nt5526                   | 2103j6                    | 0.0481	False

Each training requires 192 images. But you only have 102 photos. Is it too little?
I don't understand why it can run?

Each training requires 192 images. But you only have 102 photos. Is it too little? I don't understand why it can run?

(I'm the OP but with other account)

I also have the same question with you. I don't know my dataset have enough good to train. You can print the label in train set to know what filter.
https://github.com/clovaai/deep-text-recognition-benchmark/blob/68a80fe97943a111ff1efaf52a63ad8f0f1c0e5d/dataset.py#L158

@e-click  after deploying Easyocr how about the accuracy of the prediction i'm facing poor performance


Quick'n'Dirty fix for command reflection exceptions after upgrading to 5.5-dev branch of Laravel

I just noticed that the package breaks with 5.5-dev because the fire method was changed into handle. (see https://laravel.com/docs/master/upgrade)

[ReflectionException]
Method TwigBridge\Command\TwigBridge::handle() does not exist

I'am not a hardcore Laravel developer, but adding this to the 3 commands (TwigBridge,Clean and Lint) fixed my reflection errors after I upgraded from 5.4 -> 5.5 issue.
Guess this way the same package can be used by 5.4 and 5.5.
public function handle() {
    return $this->fire();
  }

Can you make a PR to rename to fire?

Really? A PR for a change that can be done using the github editor?
To be honest, it would take me more time to clone, setup local git, change code, push it and make a pull request than it would take somebody with write access on this repo to open the 3 files and change the name of the method ;)

You can make the edits through the github editor also. Just edit the file and choose to fork it. Then go to your repo to the patch-x branch, edit the other files and click the PR button.

Come to think of it, renaming the methods would cause more damage then adding the above code since renaming would render the bridge useless for projects that can't make the move to 5.5.

I think older versions support both names.

This commit to Laravel is the first instance of both fire and handle being acceptable: https://github.com/laravel/framework/commit/970836c5368642d975c56aa71f8aecafddcd3fd8#diff-2aaa23817a4aeafde63888bd631364e7
This change seems to have been applicable since Laravel 5.0.0 (up until the current version of Laravel, 5.5 of course).
With this, would it be best to just go with the pull request garret-gunter pushed, or change the method names from fire to handle?

Hi, can you fix(rename fire->handle) it and release new version ?
for twig:lint and twig:clean
Method TwigBridge\Command\Lint::handle() does not exist
now it;s not possible to use TwigBridge package in newest Laravel 5.5 because alias handle>fire disappeared.
Thx


gf in front of car

Describe your bug here.
gf is just infront of car
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

Its not a bug

Its not a bug

Its a choice

You can move her back with a lua script
function onCreate()
setObjectOrder('gfGroup', 6)
end
but why would you want to


[GH-#32] Coinbase Transactions

closes #32

This also solves #40

@d-velev can you try to fix the issues after merging master?


Alias does not work for imported module.

Hi there! 👋
I'm running into an issue where importing a module, that then imports an aliased dependency, throws an error. I'm not sure if this is an issue with this project or the upstream Microbundle.
Logs

npm run dev
npm run dev

><EMAIL_ADDRESS>dev /Users/JHAMILTON/work/micro
> microenvi --alias react=preact-compat,react-dom=preact-compat

microenvi running on: 3000
node-resolve: setting options.module is deprecated, please override options.mainFields instead
node-resolve: setting options.jsnext is deprecated, please override options.mainFields instead
Watching source, compiling to public/scripts:
Error: Could not load preact-compat (imported by /Users/JHAMILTON/work/micro/node_modules/@material-ui/core/esm/styles/useTheme.js): ENOENT: no such file or directory, open 'preact-compat'
    at /Users/JHAMILTON/work/micro/node_modules/rollup/dist/rollup.js:21723:19
    at async Promise.all (index 1)
    at async Promise.all (index 8)
    at async Promise.all (index 0)
    at async Promise.all (index 0)

Reproduction

Install dependencies

npm init --yes

npm install --save-dev microenvi preact preact-compat @material-ui/core

Update the package.json with a new main and a script to run microenvi.

{
  "main": "public/scripts/bundle.mjs",
  "scripts": {
    "dev": "microenvi --alias react=preact-compat,react-dom=preact-compat"
  }
}

Create html file public/index.html.

<!DOCTYPE html>

        <meta charset="UTF-8" />
        <meta name="viewport" content="width=device-width, initial-scale=1.0" />
        <meta http-equiv="X-UA-Compatible" content="ie=edge" />
        <title>Hello World</title>
        <link
            rel="stylesheet"
            href="https://fonts.googleapis.com/css?family=Roboto:300,400,500,700&display=swap"
        />

Create js entry file src/index.js.

import { createMuiTheme } from "@material-ui/core/styles";

const theme = createMuiTheme({});

Run Microenvi.

npm run dev

Addendum, running npx microbundle build -i src/index.js --alias react=preact-compat,react-dom=preact-compat --no-compress --define process.env.NODE_ENV=production doesn't seem to throw an error. It'd seem the problem lies somewhere on the Microenvi side.

Did a little more digging and it looks like this may be an issue with how dependencies are structured. After install, running rm -rf node_modules/microenvi/node_modules makes the build work 🤔


Reduce spammy logs from sidecar-injector

Currently every time a pod is injected we log about 50 lines of json. We
also are logging the same thing multiple times for the watches, when we
can consolidate into one.

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

unit-tests_istio
b1473b4d6ace2671fb21d8b3097275b417cacc51
link
/test unit-tests_istio

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

/retest


Integrates Github pull requests status

Summary
Now everytime that a pull request is triggered, codestats analyzes all the metrics and if any isn't correct, it marks the pull request as failed. I added a minimum column to the metrics. For know it will work only with numeric metrics. It compares the value of the metrics with its minimum value. We can add the type of metric (integer, boolean, etc) later and compare according to that
E.g.:
{
   "metric": {
       "branch_name": "test",
       "name" :"coverage",
       "value": "20",
       "minimum": "70"
   }
}

will evaluate as invalid.
TODO: See how evaluate this only when all the metrics are updated. When the pull request is triggered, probably none of the metrics where calculated yet. I can think of two workarounds:
1 - Trigger this evaluation from the CI. All the metrics must be calculated already
2 - Wait for a webhook of status and see if the context matches CI. So if TravisCI finishes for example. It will trigger a status webhook with the context TravisCI. After that happens, we trigger the evaluate
Screenshots

👍


Feat(workflow): add apply application workflow step definition

What this PR does / why we need it:
add apply application workflow step definition
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
/cc @yangsoon

/hold


Create Supplemental Area Overlap table

Predicted suitable area and percentage of suitable area that overlaps with areas suitable for at least one host plant species. For now, just do this for contemporary climate predictions
| species | Area (km2? Ha?) | Area >= 1 host | % >= 1 host |

For now, this is done in src/summary/summary-7-create-area-table.R; may become part of create-manuscript-objects.R.


[Automation] Generate Fluent Lite from storagemover#package-2022-07-01-preview

[Automation] Generate Fluent Lite from storagemover#package-2022-07-01-preview

/azp run java - storagemover - ci

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-resourcemanager-storagemover


fix travis

Coverage increased (+1.7%) to 87.941% when pulling e80fbff207a894fc9560ef60dd3a9c89976bcc7e on ashpa:master into fcab8995aadb15bbaee92636f9b5c5dac07927d1 on CS2103JUN2017-T4:master.


devcontainer: Make installation not interactive

Specifically when installing pnpm, if symlink doesn't work, don't wait for user input [Y/n] before using the fallback option
This bug was found (and the solution was suggested) by @sjawhar
Tested:
My own devcontainer got stuck here, and that was solved after adding the --force flag.
(I ran it in docker-compose.override.yml, which is supposed to be copied from docker-compose.dev.yml according to https://github.com/METR/vivaria/blob/main/CONTRIBUTING.md#development-setup )

It's possible #377 will fix the issue

Another option is to delete the node_modules folder, but you'd have to already know that's why the server container isn't starting

Do you mean "the user will notice the problem and remove node_modules? (If so: I prefer something that doesn't involve the user)

I prefer something that doesn't involve the user

So would I. As I said, it's possible #377 fixes this

So @sjawhar , do you recommend we wait for your PR?

So @sjawhar , do you recommend we wait for your PR?

Can you reliably reproduce the issue? If so, I recommend testing if #377 does indeed fix it

Oh it's merged.
Ok, so @sjawhar , I'll check if your PR solves the problem first.
If not - then I'll check if pnpm has a flag like --true.
Otherwise I'll personally recommend we merge this, unless someone has a better idea. (though I do resonate with @tbroadley 's point that --force also does things we don't like, but it seems better than not-working-at-all)

ok wow you replied really quickly

I ran "rebuild devcontainer without cache" and it worked. It's possible that there's some cache that wasn't actually purged, but I'm going to assume this is resolved, and I'm closing the PR.
Thanks @sjawhar

Something similar happened again:
https://evals-workspace.slack.com/archives/C07KLBPJ3MG/p1728315025225569


Unable to import methylize

Currently unable to import methylize with conda 4.10.3 and python 3.8.8

Should be fixed with methylize v0.9.4 and v0.9.5. Open another issue if not resolved.


IEmailSender should have explicitly named parameters for html vs plain text

From @joeaudette on September 27, 2017 14:32
There have been a number of people reporting bugs about the email confirmation link in the StarterWeb template not working.
This is happening because people implement IEmailSender and don't realize that the message being passed in is html. When they send the email as plain text the confirmation link does not work because of the url encoding.
I think the "message" parameter of SendEmailAsync should be renamed to htmlMessage and possibly should have an additional parameter for "plainTextMessage" and both should be passed in from AccountController.
This would make it more clear to implementors of IEmailSender what kind of string(s) they are dealing with and how to correctly implement it.
related issues:
https://github.com/aspnet/Home/issues/2152
https://github.com/aspnet/Home/issues/2217
Copied from original issue: aspnet/Templates#868

@Eilon What would be the value of having another parameter "plainTextMessage"? If we do the conversion from plain text to html if htmlMessage is not provided, I can see it being useful. But is that really a scenario that should be supported?

Hmm good point. Maybe just rename the existing parameter to be clearer about what it is? I don't think we need 2 parameters because then it's confusing which one to use and what happens if you use both.
So, does the current parameter expect it to be a string containing HTML, as in,
This is a confirmation email!
?

Yeah something like this
Please confirm your account by <a href='<link>'>clicking here.

Yeah then I think just rename the existing parameter is all we need.

I changed the parameter names in IEmailSender and EmailSender. Will send a PR out soon. I noticed that we do not encode the link for ForgotPassword in AccountController here but perhaps we should like we do here


Siamese BERT seems to be a cross encoder rather than a bi encoder.

siamese vs cross encoder
cross encoder
I believe you are not implementing a siamese network here but rather a cross encoder in case of Siamese BERT example.

Hi, Thanks for pointing it out. You are correct. I intend to add Siamese BERT to this repo at a later point in time.


Some commands are not fully functional or don't work as expected

Some pipenv commands are not fully operational or have known issues
This is a meta-issue for tracking known issues

I totally agree. I propose that you can send options to the pip subcommand from pipenv for example --timeout. In me it is almost important because with a slow connection to execute pipenv lock I get an error. I hope among all to find a solution. Regards!
Read More ... [ https://github.com/pypa/pipenv/issues/3122 ]


java-openliberty stack not replacing variable inside server.xml with environment variable

Describe the bug
I am updating this tutorial to switch from using the java-microprofile application stack to the java-openliberty stack: https://developer.ibm.com/tutorials/connecting-appsody-java-microprofile-applications-to-jdbc-data-sources/
The one thing that keeps giving me trouble, and only inside a container in a pod, is that the resolution of variables with the value of environment variables is not working.
This problem does not happen with appsody run or with reading values from bootstrap.properties. It only happens with reading environment variables when the code is running inside a pod container after  appsody deploy
Whenever the REST endpoint for accessing the database is hit, the request fails and the following message is issued in the logs:
Caused by: java.sql.SQLNonTransientException: [jcc][t4][10380][11951][4.25.13] Required property "${db_server}" is unknown host. ERRORCODE=-4222, SQLSTATE=08001 DSRA0010E: SQL State = 08001, Error Code = -4,222

Note that it says "${db_server}" instead of the actual value of the variable.
If I ssh into the container in the pod, I can see the db_server environment variable set in the container, and if I change anything in the server.xml file, such as modifying an unrelated value, that change forces the server to restart and then it somehow performs the variable replacement correctly and everything works fine.
I went as far as adding a statement like System.out.println("DB Server:" + System.getenv("db_server")); in the source code to prove that the db_server environment variable is set before the server starts and that statement prints the correct value right before the exception, so it seems somehow the Liberty server is "seeing" the wrong value even though the JVM has the correct value.
To Reproduce
Steps to reproduce the behavior:

Clone the "java-openliberty" branch of this application: https://github.com/IBM/icpa-jee-tutorial/tree/java-openliberty
Create a secret in the cluster:

cat<<EOF > .db2.temp.env
host=doesntmatter.liberty.fails.before.this
port=50001
user=doesntmatter
password=doesntmatter
db=BLUDB
EOF

kubectl create secret generic sample-jee-db-secret --from-env-file=.db2.temp.env

Request the response:

curl host:port/starter/database

See error

Caused by: java.sql.SQLNonTransientException: [jcc][t4][10380][11951][4.25.13] Required property
 "${db_server}" is unknown host. ERRORCODE=-4222, SQLSTATE=08001 DSRA0010E: SQL State =
08001, Error Code = -4,222

Expected behavior
The ${db_server} variable should be resolved to doesntmatter.liberty.fails.before.this and the server should complain about not finding that bogus server name (it is valid in the real case, but that is besides the point)
Actual behaviour
Somehow, the ${db_server} variable is not resolved.
Environment Details (please complete the following information):

Kubernetes on Docker Desktop and Kubernetes on IBM Cloud

If applicable please specify:

CLI version: appsody 0.5.9
Stack you are using (including the version):
incubator/java-openliberty 0.2.3

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
If I ssh into the container and modify some other constant in the server.xml file directly in the container, the Open Liberty re-reads the server.xml file and then the REST endpoint works without issue.
I read about this issue (https://github.com/OpenLiberty/open-liberty/issues/10674) and wondered whether it is possible the server cached the wrong value upon startup.
Update post initial creation of the issue:I just found out that If I replace ${db_server} with ${env.db_server}, then my code works! Obviously still a serious regression (relative to java-microprofile) that will cost people a lot of time, but at least I can make progress.

Update post initial creation of the issue:I just found out that If I replace ${db_server} with ${env.db_server}, then my code works! Obviously still a serious regression (relative to java-microprofile) that will cost people a lot of time, but at least I can make progress.

I think this is a known issue fixed in <IP_ADDRESS> ... you can try replacing the stack.yaml if you have time, before we get a chance to look at it to confirm:

libertyversion: '<IP_ADDRESS>'

(With the current tagging scheme you need to use a 3,6,9,12 quarterly fixpack )

I had to tear down the environment and publish the tutorial with the workaround. Will try and come back to this next week.


Fixes #9 and #10

Beautiful comments will now work again. This PR fixes both #9 and #10.
A new class was introduced BeautifulComments which keep the settings variables and the preference definition.
In addition, it holds some render logic which is controlled by those variables, and which allow the smoke test to run.

Btw - check out the class comments of the two browser plugins at some time.

Nice...and thanks

Working on paper and more :) like a rabbit full speed.


Neovim: fix binary path

Using rescritls on Neovim client with following configuration raise a error.
require'lspconfig'.rescriptls.setup = {
  cmd = {
    'node',
    '/home/pedro/Desktop/Projects/rescript-vscode/server/out/server.js',
    '--stdio',
  },
  init_options = {
    extensionConfiguration = {
      askToStartBuild = false,
    },
  },
}

Steps to reproduce:

Open a project
Open .res file

LSP Log:
[START][2022-07-21 17:02:07] LSP logging initiated
[ERROR][2022-07-21 17:02:08] .../vim/lsp/rpc.lua:420	"rpc"	"node"	"stderr"	"/home/pedro/Desktop/Projects/rescript-vscode/server/out/server.js:805\n                    extensionConfiguration.binaryPath[0] === \"~\") {\n                                                     ^\n\nTypeError: Cannot read property '0' of undefined\n    at StreamMessageReader.onMessage [as callback] (/home/pedro/Desktop/Projects/rescript-vscode/server/out/server.js:805:54)\n    at /home/pedro/Desktop/Projects/rescript-vscode/server/node_modules/vscode-jsonrpc/lib/common/messageReader.js:162:26\n    at processTicksAndRejections (node:internal/process/task_queues:94:5)\n"

Thank you!


Consistent return type for size()

Description
As mentioned in #2641, the math::size() overloads should have a return type consistent with Eigen's size() member to avoid compiler errors when calling code such as:
  Eigen::VectorXd a(2);
  a << 1, 2;

  std::max(a.size(), stan::math::size(a));

Which resolves to:
std::max(size_t, Eigen::Index);

Current Version:
v4.4.0

Thanks, @andrjohns.  Is there code that produces compiler errors now?  If so, it'd help to say what they are.  That may be the std::max call, but I couldn't tell.
The fundamental problem is that Eigen returns signed sizes/indexes and std::vector returns unsigned ones.
I think the problem with std::max is that it can't infer a template parameter if the types don't match.  We could solve this by implementing our own max that just works on int types like Stan does.

In terms of c++, just the example I posted in the issue. Im not sure if there are other examples or wider impacts, since this a subset of the issues addressed in the linked PR by @SteveBronder - he might have a better idea

I think having our own max for int types would be the way to go. Though I'm not sure how to handle the return for mixes. Maybe just always long int? Cuz then I think size_t is not a problem


Most options not working on hvnc

Hi again when I use hvnc on most clients the only options that work are chrome, run, cmd, and powershell, the rest such as edge and explorer when i try it does nothing just a black screen. The only time edge worked was when i tested it on my other PC

yea currently looking into both issues, the explorer one is because its running it as admin which causes some issues, working on a fix. and for the edge, I got no clue, but I'm looking into it.

yea currently looking into both issues, the explorer one is because its running it as admin which causes some issues, working on a fix. and for the edge, I got no clue, but I'm looking into it.

still looking into the edge problem, but for the explorer issue, there is now an option called de-escalate to user, explorer has some issue launching as admin