error running test

Hi,
I installed last version of lammps-interface present in github repository, but running the example I receive this error:
Traceback (most recent call last):
File "/usr/local/bin/lammps-interface", line 33, in
sys.exit(load_entry_point('lammps-interface==0+unknown', 'console_scripts', 'lammps-interface')())
File "/usr/local/bin/lammps-interface", line 25, in importlib_load_entry_point
return next(matches).load()
File "/usr/lib/python3.10/importlib/metadata/init.py", line 171, in load
module = import_module(match.group('module'))
File "/usr/lib/python3.10/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1050, in _gcd_import
File "", line 1027, in _find_and_load
File "", line 1006, in _find_and_load_unlocked
File "", line 688, in _load_unlocked
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "/usr/local/lib/python3.10/dist-packages/lammps_interface-0+unknown-py3.10.egg/lammps_interface/cli.py", line 3, in
from .lammps_main import LammpsSimulation
File "/usr/local/lib/python3.10/dist-packages/lammps_interface-0+unknown-py3.10.egg/lammps_interface/lammps_main.py", line 11, in
from . import ForceFields
File "/usr/local/lib/python3.10/dist-packages/lammps_interface-0+unknown-py3.10.egg/lammps_interface/ForceFields.py", line 25, in
from .Molecules import *
File "/usr/local/lib/python3.10/dist-packages/lammps_interface-0+unknown-py3.10.egg/lammps_interface/Molecules.py", line 7, in
from .structure_data import MolecularGraph
File "/usr/local/lib/python3.10/dist-packages/lammps_interface-0+unknown-py3.10.egg/lammps_interface/structure_data.py", line 14, in
from .mof_sbus import InorganicCluster, OrganicCluster
File "/usr/local/lib/python3.10/dist-packages/lammps_interface-0+unknown-py3.10.egg/lammps_interface/mof_sbus.py", line 1553, in
add_distance_matrix(InorganicCluster['Cu']['Cu Paddlewheel'])
File "/usr/local/lib/python3.10/dist-packages/lammps_interface-0+unknown-py3.10.egg/lammps_interface/mof_sbus.py", line 11, in add_distance_matrix
if(float('.'.join(nx.version.split('.')[:2])) >= 2.0):
ValueError: could not convert string to float: '3.0rc1'
How could I fix.it?
Regards
Alberto

Hi Alberto,
looks like the networkx version you are using has a version that lammps interface does not understand. Perhaps try installing it with "networkx<3"?
In any case, I am not using lammps-interface at the moment (and haven't done so in a while).
This package is looking for a new maintainer (in my last email exchange with Pete Boyd, the original author of this package also expressed this intention).
@peteboyd Shall we add a note to the README about this?

I suspect my last pull request will have fixed this https://github.com/peteboyd/lammps_interface/pull/60
You can try installing from source and let me know whether it works


Shopify AWS EventBridge for Webhook

Hi @osiset @Kyon147 @lucasmichot
I'm registering the webhook via AWS EventBridge, I have added the ARN for the webhook address.
Example:
'webhooks' => [
         [
            'topic' => 'app/uninstalled',
            'address' => "arn:aws:events:eu-west-2::event-source/aws.partner/shopify.com/9999999/staging-event-bus"
        ],
],

While I'm installing the app in the Shopify store, I got this error: https://prnt.sc/8KxBgePrZSFC
Can you please check and let me where I'm wrong?
Thank you.

Hi @veed76
I can't say much for your AWS set up but that does not look like a proper qualified url.
The error says what the issue is Invalid URL so I believe that is your problem.

Thanks @Kyon147 for reply. just want to know in this package is it possible to pass AWS ARN in webhook Address?

The webhook needs to be a URL, it's not a requirement of the package but a requirement of Shopify when setting up webhooks using the API.

In shopify Documentation they mentioned that we can pass ARN

There is a support for webhookSubscriptionCreate. I check in osiste/Laravel Package and there is no support for eventBridgeWebhookSubscriptionCreate GraphQL.

There is a support for webhookSubscriptionCreate. I check in osiste/Laravel Package and there is no support for eventBridgeWebhookSubscriptionCreate GraphQL.

Hi @veed76 , indeed there is no support.
Feel free to make a PR or just ask a Shopify developer to do it

Interesting, good testing -it looks like webhookSubscriptionCreate uses type URL https://shopify.dev/api/admin-graphql/2022-01/scalars/URL
Which is why eventBridgeWebhookSubscriptionCreate exists to be able to use the ARN routes.
As @lucasmichot - the package does not have that GraphQL support in it, as you've found yourself. So feel free to provide a PR.

Hi @lucasmichot @Kyon147
I have added the support for eventBridgeWebhookSubscriptionCreate, can you please review the PR: https://github.com/osiset/laravel-shopify/pull/1114
Thank you.
Vishal Gohil


Slack connector stops working after 1 day

Hi,
Thanks for this app.
I use the Slack connector to synchronize my status with iOS Focus mode.
It works... for one day. After that I need to re connect to Slack.
I guess it is a session issue but I can't tell why it happens.
I am not a Slack admin so I can't tell if there is a policy doing this.
Do you know a way to debug this?

I close this since the project seems to be archived.


can't run if ruby is already installed

I think it is an issue to not let ruby be installed in WSL before being able to run this tool. Is this something that can be fixed easily? or do I need to make a pull request?

Sorry for late reply. The issue is fixed recently(Well, I think...) can you try it again?


add installation.md

we probably want to add some information about running
1/ the tests locally
2/ graphql locally

we probably want to add some information about running
1/ the tests locally
2/ graphql locally

Done.


Graph DB Access Error

Cloned the repo and ran:
go run server.go
Running on port 8080, database is at neo4j+s://demo.neo4jlabs.com
error querying graph: Neo4jError: Neo.ClientError.Security.Forbidden (Database access is not allowed for user 'movies' with roles [PUBLIC, movies].)

Thanks for the report, it should be fixed now. It was due to a slight error when dealing with default configuration values.


[SPARK-21330][SQL] Bad partitioning does not allow to read a JDBC table with extreme values on the partition column

What changes were proposed in this pull request?
An overflow of the difference of bounds on the partitioning column leads to no data being read. This
patch checks for this overflow.
How was this patch tested?
New unit test.

Test build #80130 has started for PR 18800 at commit 7de8ccc.

Test build #80131 has started for PR 18800 at commit 9587bf1.

Test build #80130 has finished for PR 18800 at commit 7de8ccc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80130/
Test PASSed.

Test build #80131 has finished for PR 18800 at commit 9587bf1.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80131/
Test FAILed.

Test build #3867 has started for PR 18800 at commit 9587bf1.

Test build #3867 has finished for PR 18800 at commit 9587bf1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.


Fixes classLocation and classfileLocation

This is a backport of https://github.com/sbt/io/pull/189

To validate this pull request against other sbt modules, please visit this link.


Fix forgotten re-assignment of line with replaced newline characters

It was preventing any lines after the first line being logged.

Fix for #41

LGTM


[Solved] Motion problems in 0.0.70+ , A/a behaves as i, I doesn't work

Hello,
I cannot append to end of line using the A key while inside a { } block.
Here is a gif demonstrating this behaviour

The same behaviour happens with uppercase I (insert) too. It should insert to the beginning of the line but it does not.
The issue still arises:

with all extensions disabled
without init.vim specified
when editing bogus file in another folder
when editing other file types (HTML)
on Insiders and Stable

The issue does not arises:

when using neovim in Terminal.app
when the line ends with a (, ), { or }

Notes:

latest marketplace version of vscode-neovim
~I'm just going to try to restart my computer and see what happens~ Restarting the computer does not help.

Successful SHIFT+A: (forgot to trim, "Meta" keypress unrelated)

Faulty SHIFT+A:

Workaround for poor souls encountering this issue: use VSCodeVim in the meantime.

Can confirm!
Happened after recent #461.
@asvetliakov, sorry, could you take a look?

Can also confirm this bug is occurring, doesn't seem to be bracket based though.

Normal cursor placed on the y of the adjacency on the third line

Shift-A works as expected, placing the insert cursor after the last closing brace
Shift-I places the insert cursor immediately before the y

Normal cursor placed on the B of BLACK

Shift-A and Shift-I both place the insert cursor immediately before B

    def tick(self):

        adjacency: Dict[Coord, int] = {}

        for coord, tile in self.tiles.items():
            if tile == BLACK:
                for neighbour in self.get_neighbours(coord):
                    adjacency[neighbour] = adjacency.get(neighbour, 0) + 1

Happens with this code in it's original file and also pasted to an enew untitled buffer, both set to "Plain Text" and "Python"

Yup I thought I broke something but have the same bug as you guys. A and I don't work as expected, it simply goes into insert mode under the cursor.

Can anyone with with issue try with default vscode settings? I can't seem to reproduce by using both code snippets

Also try to update neovim to latest nightly

@asvetliakov, solved the issue for me!

Tried :

Using an empty init.vim ; no effect
Reverting to an empty settings file (bar the settings required to get the plugin working, path and key dispatch (remapped escape key) ) : no effect
Using code --extensions-dir /tmp/nvim and then installing this extension only : no effect (in conjunction with fresh settings as above)

nvim is 0.5.0+ubuntu2+git202011201111-480b04122-d569569c9-523645f61~ubuntu20.04.1 ; this is from the "Neovim Unstable" PPA for Ubuntu. 480b04 is from 20th November.

Downloaded nvim.appimage from nightly releases page
Added +x and placed on path
Changed nvim path setting in extension to point at nvim.appimage

Confirm that all the cases in my comment above now behave properly.

awesome, so the problem looks like in neovim ui event differences between latest nightly and old versions. 0.0.63 only worked by win_viewport event, in 0.0.70+ that event can be ignored if there is no followed normal cursor goto/scroll event

Had the exact same problem. Getting the latest version of Neovim fixed it for me.

It looks like neovim upstream is migrating builds from Travis to Github Actions and several releases aren't done yet. I'm using the snap nightly v0.5.0-654-g3acfefb63 and experience the motion issues.
Neovim migration tracker
A really simple workaround is to change your version of the extension in vscode:

And much :heart: to @asvetliakov & contributors -- this is an amazing extension!

Anyone is having this problem and can't upgrade to latest neovim nightly (especially when win builds are not available yet) - can try to install this build:
https://github.com/asvetliakov/vscode-neovim/suites/1740998941/artifacts/33041509
This is from #473
But my personal recommendation is to use just WSL in this case.

TIL vscode silently auto update extensions :joy: I kept staring at the config file to see if I mistyped something 😂

But my personal recommendation is to use just WSL in this case.

I've been thinking of using WSL for some other vim add-ons that don't work on windows. what Linux disto would you recommend?

Had the exact same problem. Getting the latest version of Neovim fixed it for me.

Same here

I've been thinking of using WSL for some other vim add-ons that don't work on windows. what Linux disto would you recommend?

Personally i'm using ArchLinux when on Win/WSL. But that's just personal preference

I got 0.0.73 version. But my issue didn't solved. My A(append) motion always show prepend motion. How can solve it... T.T

@ycy3723, have you tried to updating Neovim nightly to the latest master?

Yes, latest version is 0.0.73 and I already installed it.

Yes, latest version is 0.0.73 and I already installed it.

No, I'm talking about Neovim itself.

Yes, latest version is 0.0.73 and I already installed it.

No, I'm talking about Neovim itself.

@Shatur95 Thx for your service. Finally I fixed with your tip 👏 👍

Not sure if anyone else ran into this, but I had an issue with upgrading neovim to latest nightly.
Previously I was using brew install --HEAD neovim on macOS, but when I upgraded to latest HEAD, nvim would crash on startup every time, which was even worse than this cursor motion issue, but I fixed it via:
brew uninstall neovim
brew tap jason0x43/homebrew-neovim-nightly  # this just pulls from the neovim github release page
brew install neovim-nightly

Note that since these are unsigned binaries, the first time you run it will complain that it cannot be opened – go to System Preferences > Security & Privacy > General and click "Open Anyway". I had to repeat the process 3 types for some unsigned shared libraries also installed by the package.
Hope this helps someone else who was struggling with it like I was!

Previously I was using brew install --HEAD neovim on macOS, but when I upgraded to latest HEAD, nvim would crash on startup every time, which was even worse than this cursor motion issue, but I fixed it via:

Probably need brew install luajit --HEAD here too. But for some reason brew install --HEAD neovim doesn't work for me at all. I found it much easier to drop tarball somewhere at home and adjust PATH

Neovim nightly builds for win are finally available:
https://github.com/neovim/neovim/releases/nightly

Neovim nightly builds for win are finally available:
https://github.com/neovim/neovim/releases/nightly

I downloaded the appimage and zip from the latest nightly build NVIM v0.5.0-dev+992-gf3a8c930a (the one that just came with Windows builds) but am still encountering this issue on both Windows and linux even with all extensions disabled and empty init.vim
I then proceeded to build neovim master revisions from around 2 days ago, and each revision seemed to work fine with 0.0.70+ up until commit b01a60b. This commit seems to have reverted a past commit.
Perhaps we should fall back to 0.0.63 for now?

I downloaded the appimage and zip from the latest nightly build NVIM v0.5.0-dev+992-gf3a8c930a (the one that just came with Windows builds) but am still encountering this issue on both Windows and linux even with all extensions disabled and empty init.vim
I then proceeded to build neovim master revisions from around 2 days ago, and each revision seemed to work fine with 0.0.70+ up until commit b01a60b. This commit seems to have reverted a past commit.
Perhaps we should fall back to 0.0.63 for now?

@SpicyRicecaker Interesting... does this build work for you with latest nightly? https://github.com/asvetliakov/vscode-neovim/suites/1740998941/artifacts/33041509

@SpicyRicecaker Interesting... does this build work for you with latest nightly? https://github.com/asvetliakov/vscode-neovim/suites/1740998941/artifacts/33041509

@asvetliakov that build seems to work just fine on linux with the latest nightly appimage

@asvetliakov that build seems to work just fine on linux with the latest nightly appimage

@SpicyRicecaker
do you have repro snippet? I've just reinstalled neovim through brew (finally started to work for me) and it's working fine. Thanks!
If it's consistent then i'll just merge #473 and release update

@SpicyRicecaker
do you have repro snippet? I've just reinstalled neovim through brew (finally started to work for me) and it's working fine. Thanks!
If it's consistent then i'll just merge #473 and release update

@asvetliakov
Sorry, I'm new to contributing, so I don't really know what you mean by repro snippet...
Do you mean how I installed this extension, or neovim?

@asvetliakov
Sorry, I'm new to contributing, so I don't really know what you mean by repro snippet...
Do you mean how I installed this extension, or neovim?

A sample text and steps to reproduce behaviour, same as here: https://github.com/asvetliakov/vscode-neovim/issues/467#issuecomment-752016314

A sample text and steps to reproduce behaviour, same as here: https://github.com/asvetliakov/vscode-neovim/issues/467#issuecomment-752016314

@asvetliakov
Gotcha
Running NVIM v0.5.0-dev+992-gf3a8c930a with https://github.com/asvetliakov/vscode-neovim/suites/1740998941/artifacts/33041509, empty init.vim, settings.json empty except for nvim path, no extensions

Normal cursor placed on the y of the adjacency on the third line

Shift-A seemes to be slightly bugged, placing the insert cursor on the beginning of the fourth line, while $ moves to the end of the bracket
Shift-I works as expected, placing the insert cursor before the a

Normal cursor placed on the B of BLACK

Shift-A and Shift-I both work as expected

def tick(self):

    adjacency: Dict[Coord, int] = {}

    for coord, tile in self.tiles.items():
        if tile == BLACK:
            for neighbor in self.get_neighbors(coord):
                adjacency[neighbor] = adjacency.get(neighbor, 0) + 1

Occurs in plaintext file

@asvetliakov
Gotcha
Running NVIM v0.5.0-dev+992-gf3a8c930a with https://github.com/asvetliakov/vscode-neovim/suites/1740998941/artifacts/33041509, empty init.vim, settings.json empty except for nvim path, no extensions

Normal cursor placed on the y of the adjacency on the third line

Shift-A seemes to be slightly bugged, placing the insert cursor on the beginning of the fourth line, while $ moves to the end of the bracket
Shift-I works as expected, placing the insert cursor before the a

Normal cursor placed on the B of BLACK

Shift-A and Shift-I both work as expected

def tick(self):

    adjacency: Dict[Coord, int] = {}

    for coord, tile in self.tiles.items():
        if tile == BLACK:
            for neighbor in self.get_neighbors(coord):
                adjacency[neighbor] = adjacency.get(neighbor, 0) + 1

Occurs in plaintext file

@SpicyRicecaker Sorry , i mean with normal 0.0.73 not with that build

@SpicyRicecaker Sorry , i mean with normal 0.0.73 not with that build

@asvetliakov
Running NVIM v0.5.0-dev+992-gf3a8c930a appimage on Fedora 33, with 0.0.73, empty init.vim, settings.json empty except for nvim path, no extensions except for neovim

Normal cursor placed on the w of the world

Shift+a puts cursor before w
Shift+i puts cursor before w
a puts cursor before w
i puts cursor before w

Hello world

In a basic text.txt file

@asvetliakov
Running NVIM v0.5.0-dev+992-gf3a8c930a appimage on Fedora 33, with 0.0.73, empty init.vim, settings.json empty except for nvim path, no extensions except for neovim

Normal cursor placed on the w of the world

Shift+a puts cursor before w
Shift+i puts cursor before w
a puts cursor before w
i puts cursor before w

Hello world

In a basic text.txt file

Ok, thank you. Curious, why it's working on mac?

Ok, thank you. Curious, why it's working on mac?

Ok, finally reproduced with:
message ChatEntryNoteData {
    // Note
    string note = 1;
}

Cursor at // [Cursor]Note

Ok, finally reproduced with:
message ChatEntryNoteData {
    // Note
    string note = 1;
}

Cursor at // [Cursor]Note

Fixed and released 0.0.74

Fixed and released 0.0.74

Fixed and released 0.0.74

Thank you!

Fixed and released 0.0.74

Thank you!

@asvetliakov  Some issues for wellle/targets.vim. Before 0.0.70+, everything works. After 0.0.70+, text object motion when unicode character present in the same line breaks the motion.
Bugs after 0.0.70+
Normal cursor placed on a of par1, press via to select 'inside argument' text object par1

Without unicode, par1 is selected as expected
With unicode ŷ, par is selected without the 1

y = function(par1)  # Without unicode
ŷ = function(par1)  # With unicode

Normal cursor placed on a of par1, press cia to change 'inside argument' text object par1 and cursor positioned inside bracket (<my_cursor>)

Without unicode, par1 is deleted and cursor positioned inside bracket y = function(<my_cursor>) as expected
With unicode ŷ, par is deleted and cursor positioned outside bracket to the left by 1 column ŷ = function<my_cursor>()

y = function(par1)  # Without unicode
ŷ = function(par1)  # With unicode

Similar bugs for other related motion...
Versions
OS: Windows with WSL2
asvetliakov/vscode-neovim 0.0.75: https://github.com/asvetliakov/vscode-neovim/commit/20cba2206f1fd04db4eaccc716b1028f96411194

Local Windows NVIM v0.5.0-dev+992-gf3a8c930a
NVIM v0.5.0-dev+992-gf3a8c930a
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: C:/Program Files (x86)/Microsoft Visual Studio/2017/Enterpr
ise/VC/Tools/MSVC/14.16.27023/bin/Hostx86/x64/cl.exe /DWIN32 /D_WINDOWS
/W3 /MD /Zi /O2 /Ob1 /DNDEBUG /W3 -D_CRT_SECURE_NO_WARNINGS -D_CRT_NONST
DC_NO_DEPRECATE -DWIN32 -D_WIN32_WINNT=0x0600 -DINCLUDE_GENERATED_DECLAR
ATIONS -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LE
VEL=3 -ID:/a/neovim/neovim/build/config -ID:/a/neovim/neovim/src -IC:/pr
ojects/nvim-deps/usr/include -ID:/a/neovim/neovim/build/src/nvim/auto -I
D:/a/neovim/neovim/build/include
Compiled by runneradmin@fv-az152-708
Features: -acl +iconv +tui
See ":help feature-compile"
 system vimrc file: "$VIM\sysinit.vim"
fall-back for $VIM: "C:/Program Files/nvim/share/nvim"

Run :checkhealth for more inf

@asvetliakov  Some issues for wellle/targets.vim. Before 0.0.70+, everything works. After 0.0.70+, text object motion when unicode character present in the same line breaks the motion.
Bugs after 0.0.70+
Normal cursor placed on a of par1, press via to select 'inside argument' text object par1

Without unicode, par1 is selected as expected
With unicode ŷ, par is selected without the 1

y = function(par1)  # Without unicode
ŷ = function(par1)  # With unicode

Normal cursor placed on a of par1, press cia to change 'inside argument' text object par1 and cursor positioned inside bracket (<my_cursor>)

Without unicode, par1 is deleted and cursor positioned inside bracket y = function(<my_cursor>) as expected
With unicode ŷ, par is deleted and cursor positioned outside bracket to the left by 1 column ŷ = function<my_cursor>()

y = function(par1)  # Without unicode
ŷ = function(par1)  # With unicode

Similar bugs for other related motion...
Versions
OS: Windows with WSL2
asvetliakov/vscode-neovim 0.0.75: https://github.com/asvetliakov/vscode-neovim/commit/20cba2206f1fd04db4eaccc716b1028f96411194

Local Windows NVIM v0.5.0-dev+992-gf3a8c930a
NVIM v0.5.0-dev+992-gf3a8c930a
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: C:/Program Files (x86)/Microsoft Visual Studio/2017/Enterpr
ise/VC/Tools/MSVC/14.16.27023/bin/Hostx86/x64/cl.exe /DWIN32 /D_WINDOWS
/W3 /MD /Zi /O2 /Ob1 /DNDEBUG /W3 -D_CRT_SECURE_NO_WARNINGS -D_CRT_NONST
DC_NO_DEPRECATE -DWIN32 -D_WIN32_WINNT=0x0600 -DINCLUDE_GENERATED_DECLAR
ATIONS -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LE
VEL=3 -ID:/a/neovim/neovim/build/config -ID:/a/neovim/neovim/src -IC:/pr
ojects/nvim-deps/usr/include -ID:/a/neovim/neovim/build/src/nvim/auto -I
D:/a/neovim/neovim/build/include
Compiled by runneradmin@fv-az152-708
Features: -acl +iconv +tui
See ":help feature-compile"
 system vimrc file: "$VIM\sysinit.vim"
fall-back for $VIM: "C:/Program Files/nvim/share/nvim"

Run :checkhealth for more inf


stable/prometheus-operatory : Fix #21169 - Also delete thanosrullers

The Delete section does not specify we also have to delete thanosrulers.monitoring.coreos.com. This file change will just add it to the list of CRD to delete.

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it: Improve documentation
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #21169

Special notes for your reviewer:  This change just impact the documentation. There is no change into the chart.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @bismarck

@angegar Even though this is just a documentation change, still need to sign your commits and bump the chart version. See the Contributing Guidelines.

I just sign the commit.
My apologize to have not read the CONTRIBUTING guide.
Please let me know if something else is missing.


Fix image in  can exceeds container width on Firefox

An image in <figure class="figure"> wider than the container no longer exceeds it
fix #21886

Closing since this has since been addressed. https://github.com/twbs/bootstrap/issues/21886#issuecomment-287808975


Path source: relative paths without leading dot do not work

Make sure you have done the following

[X] I have updated to the latest version of blink.cmp
[X] I have read the README

Bug Description
dirA
|____dirB
       |_____file.txt

with the working directory set as dirA, entering ./dirB/ triggers completion with the suggestion file.txt. Omitting the dot (ie. entering dirB/) does not work.
The completion is rejected here: https://github.com/Saghen/blink.cmp/blob/02528e8ccbe4d0cef5e1df52eda419c5ed557ad3/lua/blink/cmp/sources/path/lib.lua#L32
I am not really sure if this is a bug or a feature, but it took me by surprise.
Relevant configuration
No response
neovim version
nightly
blink.cmp version: branch, tag, or commit
v0.7.6

This might be tricky because it could activate when you wouldn't want it to (i.e. typing foo/bar in markdown, or str/trim in clojure). We should likely enable this per language with a treesitter query to ensure we're in a comment/string

Would this also be an issue if foo/bar or str/trim does not exist in the current working directory?

It wouldn't be an issue in that case but it seems like a good idea to keep it disabled by default, in case a directory does exist, if we we're not checking the treesitter node


[release-4.10][manual][PR#371] implement strict alignment check for the noderesourcetopology filter …

…plugin (#347)

nrt: filter: use IsZero()

Use resource.Quantity.IsZero() helper instead of comparing
with zero quantity. This is a bit nicer to read, but most notably
saves us a relatively costly MustParse call in each function run.
No intended changes in behaviour.
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>
nrt: filter: isolate the per-NUMA decision logic

Extract the inner logic to check if a NUMA Node is suitable
in its own function.
This change enables further cleanups, and easier changing
of the filtering logic.
It will also make adding more logs easier down the line
No intended changes in behaviour.
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>
nrt: filter: split complex if in a sequence of ifs

Split the complex condition, to make the code easier
to read, change, and possibly add (and log) the
NUMA node reject reason in the future.
No intended change in behaviour.
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>
nrt: filter: qos=guaranteed requires align for mem/hp

In kubernetes >= 1.23, which is already out, we can
depend on the memory manager for alignment of memory and
hugepages. So we make the alignment check consistent to
what we do for CPU.
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>
nrt: filter: make test more extendible

Make it easier to add new test cases without intended
changes in behaviour
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>
nrt: flter: add more test cases

Improve the test coverage of the filter plugin
Signed-off-by: Francesco Romani<EMAIL_ADDRESS><!--  Thanks for sending a pull request!  Here are some tips for you:

1. If this is your first time, please read our contributor guidelines: https://git.k8s.io/community/contributors/guide/first-contribution.md#your-first-contribution and developer guide https://git.k8s.io/community/contributors/devel/development.md#development-guide
2. Please label this pull request according to what type of issue you are addressing, especially if this is a release targeted pull request. For reference on required PR/issue labels, read here:
https://git.k8s.io/community/contributors/devel/sig-release/release.md#issuepr-kind-label
3. Ensure you have added or ran the appropriate tests for your PR: https://git.k8s.io/community/contributors/devel/sig-testing/testing.md
4. If you want *faster* PR reviews, read how: https://git.k8s.io/community/contributors/guide/pull-requests.md#best-practices-for-faster-reviews
5. If the PR is unfinished, see how to mark it: https://git.k8s.io/community/contributors/guide/pull-requests.md#marking-unfinished-pull-requests

Is there a reason for combining all the commits together?

Is there a reason for combining all the commits together?

it was squashed upstream (maintainer decision).


[BUG] Watchdog is not disabled in bootloader

Describe the bug
When operating the OAK-D PoE devices over unreliable connections, the docs recommend setting environment variables to modify the XLink watchdog timeout. However, these variables only modify the watchdog timeouts DeviceBase.cpp, but not in DeviceBootloader.cpp.
When one connects to a device over XLink, it flashes a bootloader anyway. If a device connection has high latency, it'll fail during that part of the connection regardless of whether the DEPTHAI_WATCHDOG environment was set to a more conservative value than the default.
Minimal Reproducible Example
Just connect to a device via a high-latency connection (e.g. over 4G), and try to flash a bootloader. It'll fail with a connection timeout, even if you set the DEPTHAI_WATCHDOG environment variable to 0.
   import depthai as dai

    try:
        device_info = dai.DeviceInfo(
            name=camera_ip,
            mxid="",
            state=dai.XLinkDeviceState.X_LINK_FLASH_BOOTED,
            protocol=dai.XLinkProtocol.X_LINK_TCP_IP,
            platform=dai.XLinkPlatform.X_LINK_MYRIAD_X,
            status=dai.XLinkError_t.X_LINK_SUCCESS,
        )
        bootloader = dai.DeviceBootloader(device_info)
    except RuntimeError:
        device_info = dai.DeviceInfo(
            name=camera_ip,
            mxid="",
            state=dai.XLinkDeviceState.X_LINK_BOOTLOADER,
            protocol=dai.XLinkProtocol.X_LINK_TCP_IP,
            platform=dai.XLinkPlatform.X_LINK_MYRIAD_X,
            status=dai.XLinkError_t.X_LINK_SUCCESS,
        )
        bootloader = dai.DeviceBootloader(device_info)

    progress = lambda p: print(f"Flashing progress: {p*100:.1f}%")
    bootloader.flash(progress, dai.Pipeline())

Expected behavior
The watchdog timeout should be changed also in the bootloader section of the code.
Here is an example solution that we're working with now (it only disables the watchdog, doesn't change the value):
https://github.com/luxonis/depthai-core/compare/main...mrmorawski:depthai-core:no_bootloader_watchdog
Attach system log
DEPTHAI_LEVEL=debug DEPTHAI_WATCHDOG=0 poetry run python
Python 3.10.7 (main, Nov 24 2022, 19:45:47) [GCC 12.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
> import depthai as dai
> [2023-03-18 12:18:33.686] [debug] Python bindings - version: <IP_ADDRESS> from  build: 2023-01-31 23:58:49 +0000
> [2023-03-18 12:18:33.686] [debug] Library information - version: 2.20.2, commit: 4ff860838726a5e8ac0cbe59128c58a8f6143c6c from 2023-01-31 22:20:03 +0200, build: 2023-01-31 23:34:39 +0000
> [2023-03-18 12:18:33.687] [debug] Initialize - finished
> >>> [2023-03-18 12:18:33.757] [debug] Resources - Archive 'depthai-bootloader-fwp-0.0.24.tar.xz' open: 1ms, archive read: 68ms
> [2023-03-18 12:18:34.049] [debug] Resources - Archive 'depthai-device-fwp-8c3d6ac1c77b0bf7f9ea6fd4d962af37663d2fbd.tar.xz' open: 1ms, archive read: 360ms
>
> >>> info = dai.DeviceInfo("<IP_ADDRESS>")
> >>> dev = dai.Device(info)
> [2023-03-18 12:19:41.424] [debug] Found an actual device by given DeviceInfo: DeviceInfo(name=<IP_ADDRESS>, mxid=194430108152761300, X_LINK_BOOTLOADER, X_LINK_TCP_IP, X_LINK_MYRIAD_X, X_LINK_SUCCESS)
> [2023-03-18 12:19:41.424] [debug] Device - OpenVINO version: universal
> [2023-03-18 12:19:41.424] [warning] Watchdog disabled! In case of unclean exit, the device needs reset or power-cycle for next run
> [2023-03-18 12:19:41.424] [debug] Device - BoardConfig: {"camera":[],"emmc":null,"gpio":[],"logDevicePrints":null,"logPath":null,"logSizeMax":null,"logVerbosity":null,"network":{"mtu":0,"xlinkTcpNoDelay":true},"nonExclusiveMode":false,"pcieInternalClock":null,"sysctl":[],"uart":[],"usb":{"flashBootedPid":63037,"flashBootedVid":999,"maxSpeed":4,"pid":63035,"vid":999},"usb3PhyInternalClock":null,"watchdogInitialDelayMs":null,"watchdogTimeoutMs":0}
> libnop:
> 0000: b9 10 b9 05 81 e7 03 81 3b f6 81 e7 03 81 3d f6 04 b9 02 00 01 ba 00 00 be bb 00 bb 00 be be be
> 0020: be be be be 00 bb 00
> [2023-03-18 12:19:41.438] [debug] Searching for booted device: DeviceInfo(name=<IP_ADDRESS>, mxid=REMOVED, X_LINK_BOOTLOADER, X_LINK_TCP_IP, X_LINK_MYRIAD_X, X_LINK_SUCCESS), name used as hint only
> [2023-03-18 12:19:42.591] [debug] Connected bootloader version 0.0.24
> [2023-03-18 12:19:57.595] [warning] Monitor thread (device: REMOVED [<IP_ADDRESS>]) - ping was missed, closing the device connection
> [2023-03-18 12:19:59.346] [debug] DeviceBootloader about to be closed...
> [2023-03-18 12:19:59.346] [debug] XLinkResetRemote of linkId: (0)
> [2023-03-18 12:20:00.599] [debug] DeviceBootloader closed, 1252
> Traceback (most recent call last):
>   File "<stdin>", line 1, in <module>
> depthai.XLinkWriteError: Couldn't write data to stream: '__bootloader' (X_LINK_ERROR)

Additional context
For our particular application, we're running the cameras in standalone mode,  so we don't need an XLink connection to stream frames. We only need to be able to update pipelines on cameras, which I assume will be the usecase of most people running PoE devices in standalone mode.

I don't know the depthai-core library well enough to submit a full PR, but the hack listed in 'expected behaviour' works well for us.
Would just mirroring the adjustable watchdog code from DeviceBase.cpp work for DeviceBootloader.cpp as well?

Thanks for the proposed solution @mrmorawski  - there is only one issue with this approach, that it is not completely "WD disabled" as BL still has its own WD being counted down. Though BL bumps its WD on any comms that it receives/sends successfully, so the occasion of it timing out is smaller.
Perhaps it could be reworked / put under DEPTHAI_BOOTLOADER_WATCHDOG variable - or in general, the WD timeout just extended further. Have you tried with the latter and what was the time which worked for you?

thanks for the quick reply @themarpe . I'm aware it's not a full solution, just a quick hack that I put in to make sure it'll work at all. I'll maybe try to experiment with different timeouts next week.


Get Size of file using AVFormatContext

Hi, first of all, this is a really cool project.
I am using this lib to get some information of files like ffprobe, and i need the size of a file. I just used this ffprobe as an example, and implemented here.

Codecov Report

Merging #45 into master will increase coverage by 0.1%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master      #45     +/-   ##
=========================================
+ Coverage   33.37%   33.47%   +0.1%
=========================================
  Files           9        9
  Lines        2652     2656      +4
=========================================
+ Hits          885      889      +4
  Misses       1719     1719
  Partials       48       48

Impacted Files
Coverage Δ

avformat/avformat.go
60.04% <100%> (+0.33%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks. Actually we can get this already. Look:
var ctx *Context = ... some value...
if ioCtx := ctx.IOContext(); ioCtx != nil {
  size := ioCtx.Size()
  ... use size ...
}

Sorry for not seeing that.
Thats better, i will close this PR.

No prob. Keep it coming when you have something new.


goruntine was blockde when get entry , sometime

the numbers of goruntine was increasing by all the time

the numbers of goruntine was increasing by all the time

That's weird an maybe unrealted to bigcache as it creates only one gorutine when cache is created
https://github.com/allegro/bigcache/blob/c9426ba4b01ea37f412904df29a670711306e31e/bigcache.go#L91-L107

I think the goruntine blocked when I get a value and bigcache try to get a lock

yes,I think I get the blocked reason.
it about that I set a new vallue when the callback of remove was called.


Add a new widget to change the template to display hits

I'd need a new widget to change the template used to display hits on the page.
Small parenthesis, for now, I made a custom system, but I encounter an issue with the "currentRefinedValues" widget.
To change the mode of displayal, I change the template in the "hits" widget, and I call a search.start();
This works well, but after a "clear filter", when I call the search.start();, some facets are reappearing in the "currentRefinedValues" widget (as if they were still in the object memory).
As this feature could be usefull to someone else, I prefer suggest to add the widget instead of correcting a very specific bug.
End of the parenthesis.
The expected result could be something like this :

API
search.addWidget(
  instantsearch.widgets.hitsDisplayMode({
    container: '#hits-display-mode',
    templates: {
      small: '
Hit {{objectID}}
',
      medium: '
Hit {{objectID}}: {{{_highlightResult.name.value}}}
',
      full: '

Hit {{objectID}}
 {{{_highlightResult.name.value}}}
'
    },
    default: 'medium',
    cssClasses: {
        active: 'active',
        iconSmall :'fa-th',
        iconMedium :'fa-bars',
        iconFull :'fa-th-large',
        small: 'small',
        medium: 'medium',
        full: 'full'
    }
  })
);

The requirements will be the small, medium and full template. Maybe we could use the template specified in "hits" as the medium one, I don't know.
We could also specify css classes, and the default template to apply (but this would be in conflict with the "hits" widget again, as they are complementary )
The rended html could be something like this :

TODO

[ ] find a cool name for the widget
[ ] determine the better way to do that without conflicting with the "hits" widget
[ ] affine the customizable options (css, templating)

Thanks a lot

Hi, nice feedback!
Indeed what we would need here is maybe this: https://github.com/algolia/instantsearch.js/issues/753#issuecomment-168701879

A way to update the configuration of a widget
A way to force a render of the interface or a render of a specific widget

Both are not so easy to do for now but ultimately this is what we would need. See https://github.com/algolia/instantsearch.js/issues/852
As for your widget that's a very good one indeed, it would be great to have it as a community widget!
We will try to discuss those features soon, in the meantime you can comment on it and give us your feedback

Hi,
Thanks. Indeed, if I can force the renderer, that would do the trick.
I'm trying to work on it but I'm not very familiar with React, so it can be long to have a good result ;)
I'm gonna watch the issues, I'll tell you if I have something to push.

@gregquat in instantsearch.js V2 we are planning to let you do the whole hit rendering yourself (say, with jQuery) along with being able to force a render.
Thanks!
Moved to #1071 (somehow)


Making images responsive

Checklist

[x] I have read the tutorials and know the correct effect of the functional design.
[x] There are no similar question on existing issues (including closed ones).
[x] I searched the Internet for related problems, but still couldn't solve it.
[x] My question is based on the latest code of master branch.

Description
Is there a way to make image size responsive for images embedded in posts without adding a new plugin?

Example. The whole image should be visible but only the top left 2/3rds or so is

@bcaddy This can be achieved through CSS.

Could you elaborate on that? I'm not a web dev and the whole point of me using Jekyll themes is to not deal too much with CSS and HTML.

Images in posts appear to be responsive but not in tabs. Is there a simple way to just make images in tabs responsive as well?

Images in posts appear to be responsive but not in tabs. Is there a simple way to just make images in tabs responsive as well?

That's a good point, I will update the page layout later. BTW, I am not a web dev either :)

Thanks!
You're closer than I am :) I just write scientific software for simulations and barely know the difference between HTML and CSS

Done, please checkout the latest code.

That fixed it. Thanks!


occluded 2d kps

Hi, I notice that the person center is computed from 'visible' keypoints, which I think refers to the body joints that we can literally see in the image, instead of the occluded joints.
But for some datasets, the annotation is made by machine, which means that even the 2 person is heavily occluded, there might be full set of 2d keypoints in the annotation file. In this case, the 'person_center' of the person in the back will actually be on the first person's body:

I think ROMP has the ability to predict the shape from the center of 'visible body parts'. And when the two center comes too close to each other, the CAR mechanism will be activated to push them away. But there is no guarantee that the center will be pushed to a pixel that belongs to the occluded person. In this case, is it really possible to infer 2 different people using pixels from the same person?
So, in the end, can we really use the massive machine annotated data for training, concerning the above situation ? Also, can we train ROMP with bounding box center?
I'm not sure if this is technically feasible. But I think the center based representation is actually an alternative of bounding box, these single points might carry equal information as bboxes due to the receptive field, it's no longer a single point, but a point that can represent an area. So to some extend, this is feasible, am I understanding correctly ?
But actually, in my previous works of feature matching / pixel-wise descriptor learning. This is actually difficult problem, we would normally filter those occluded points as it would carry ambiguity. For example, we can't let a pixel of a man to be matched to a car. even if there is an implicit man's pixel behind the car

Hi, I see your points.
But if you can visualize some samples in crowded datasets and their annotations, you will find that the cases which two people share the same center position is quite rare. Even in occlusion, they might have different height, meanwhile, they mush be at different 3D position. Therefore, in most occlusion cases, the 2D center position of two people in occlusion might be close, while can be handled by CAR.
I don't think we have to guarantee that the center will be pushed to a pixel that belongs to the occluded person. Why should we guarantee that? We just need to garantee that they are discriminative enough to be learnt by our model. CNN has enough receptive field to see the entire body.
Training ROMP with bounding box annotation for better detection is performed during pretraining. We really need these datasets for stable detection in crowd.

Yes, I see, that's what I mean. Thanks!
By the way, there is also a bug in cropping method https://github.com/Arthur151/ROMP/issues/175. You just accidently set leftTop, rightBottom = np.clip(box[0], 0, width), np.clip(box[1], 0, height), you might also have a quick look on that one if you have time. Bye for now~


Disable constraint/trigger during cleanup

I am shooting in the dark here, but there is actually a constraint thanks to a FK on the table that is very slow for deletions:
            Table "public.commit_installed_packages"
        Column        |  Type  | Collation | Nullable | Default
----------------------+--------+-----------+----------+---------
 installed_package_id | bigint |           | not null |
 commit_id            | bigint |           | not null |
Indexes:
    "commit_installed_packages_pkey" PRIMARY KEY, btree (installed_package_id, commit_id)
Foreign-key constraints:
    "fk_commit_installed_packages_commit" FOREIGN KEY (commit_id) REFERENCES commits(id)
    "fk_commit_installed_packages_installed_package" FOREIGN KEY (installed_package_id) REFERENCES installed_packages(id)

Instead of deleting the foreign key completely, which is quite bit of work because GORM does not allow to disable FKs per table only globally, I am trying a trick that is mentioned in several articles and solutions on the internet. Since FK constraints are implemented via triggers, it could help deletion significantly.

Sure, passes locally on pgsql and unit tests just warn on sqlite, so I'm good with this, it's nicely limited and even if it fails, it just warns.


Writing numpy array to cogs?

Is there a way to write a numpy array read using rio-tiler back to a cog file?
For eg, exporting the numpy array from part() method to a geo-referenced cog

:wave:  I've done this recently!  You can save the numpy array to a rasterio in-memory file and pass the file handle as the source argument to [rio_cogeo.cog_translate](https://github.com/cogeotiff/rio-cogeo/blob/master/rio_cogeo/cogeo.py#L42-L61.  There is a good example of this here.

👋 I've done this recently! You can save the numpy array to a rasterio in-memory file and pass the file handle as the source argument to rio_cogeo.cog_translate. There is a good example of this here.

In "Using the API with in MemoryFile" section:
What is dst_transform in transform=dst_transform ?

@purijs the dst_transform is the transform information for the GeoTIFF (origin, resolution). From the rio-cogeo doc example
# Create GeoTIFF profile
bounds = mercantile.bounds(mercantile.Tile(0,0,0))
src_transform = from_bounds(*bounds, 1024 1024)

the transform is created using rasterio helper function (from_bounds) by passing the bounds of the data and the height and width.
I think with this we are good to close. Feel free to reopen if there is a proper bug 🙏


feat: enable require_trailing_commas lint rule

This rule helps in prompting trailing commas thereby improving readability of code
Signed-off-by: Ayush P Gupta<EMAIL_ADDRESS>Description
This PR aims at enabling require_trailing_commas lint rule to improve readability of code by adding trailing commas as per Docs
Fixes #(issue)
Screenshots (if appropriate):
Type of change
Linter related change
Please tick the relevant option by putting an X inside the bracket

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

Checklist:

[x] My code follows the guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] All current GitHub actions pass
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings
[x] I have read the CONTRIBUTING document

Thank you a lot for this PR, I agree with this change so I've replaced the Flutter Lints package with the Lint package which follows the Effective Dart: Style Guide in the dev branch, the require_trailing_commas rule is turned on by default.
I will merge this PR but keep in mind that I'm planning on replacing the contents in the main branch with the ones in dev branch soon so if you're looking to collaborate, please send any PRs to the dev branch for now.
Have a nice day and thanks again!


[Q] does the dektec output plugin doesn't terminate tsp ?

when using the "until" plugin with a DekTec output plugin (in ASI mode), the output plugin doesn't terminate and as a consequence the tsp process doesn't terminate.
is this normal behavior of DekTec output plugin?
Izhar

is this normal behavior of DekTec output plugin?

Definitely not. In fact, this is not under control of the output plugin. An output plugin just implements start(), send() and stop() primitives.
The orchestration is done at tsp level. When a plugin in the middle (the until plugin here) decides to terminate, it returns that information to tsp. Then, tsp propagates the termination both ways. Downward to the output plugin, an "end of input" condition is propagated, as if the input plugin had no more data.
As long as the "output executor" (the tsp thread that calls the output plugin) receives packets, it calls the send() function in the plugin. When it receives an "end of data", the output executor thread calls the stop() function and terminates. This is as simple as that.
To investigate what's going on, I suggest tsp -d or even tsp -d10 (debug level 10). It is very verbose, so take care, use a short until condition. Please post the last part of the log, starting a bit before the last waitWork message of the output plugin.

here you go;
 Debug[2]: ip: received UDP packet, source: <IP_ADDRESS>:11111, destination: <IP_ADDRESS>:51105, timestamp: 1,621,336,174,884,761
* Debug[10]: ip: passPackets(count = 7, bitrate = 5,199,983, input_end = false, aborted = false)
* Debug[10]: ip: waitWork(min_pkt_cnt = 1, ...)
* Debug[10]: ip: waitWork(min_pkt_cnt = 1, pkt_first = 17,878, pkt_cnt = 71,362, bitrate = 0, input_end = false, aborted = false, timeout = false)
* Debug[10]: until: waitWork(min_pkt_cnt = 1, pkt_first = 17,871, pkt_cnt = 7, bitrate = 5,199,983, input_end = false, aborted = false, timeout = false)
* Debug[10]: until: passPackets(count = 7, bitrate = 5,199,983, input_end = false, aborted = false)
* Debug[10]: until: waitWork(min_pkt_cnt = 1, ...)
* Debug[10]: dektec: waitWork(min_pkt_cnt = 1, pkt_first = 17,871, pkt_cnt = 7, bitrate = 5,199,983, input_end = false, aborted = false, timeout = false)
* Debug[10]: dektec: passPackets(count = 7, bitrate = 0, input_end = false, aborted = false)
* Debug[10]: dektec: waitWork(min_pkt_cnt = 1, ...)
* Debug[2]: ip: received UDP packet, source: <IP_ADDRESS>:11111, destination: <IP_ADDRESS>:51105, timestamp: 1,621,336,174,886,780
* Debug: ip: input: got bitrate 5,199,983 b/s
* Debug[10]: ip: passPackets(count = 7, bitrate = 5,199,983, input_end = false, aborted = false)
* Debug[10]: ip: waitWork(min_pkt_cnt = 1, ...)
* Debug[10]: until: waitWork(min_pkt_cnt = 1, pkt_first = 17,878, pkt_cnt = 7, bitrate = 5,199,983, input_end = false, aborted = false, timeout = false)
* Debug: until: plugin requests termination
* Debug[10]: until: passPackets(count = 1, bitrate = 5,199,983, input_end = true, aborted = true)
* Debug[10]: ip: waitWork(min_pkt_cnt = 1, pkt_first = 17,885, pkt_cnt = 71,355, bitrate = 0, input_end = false, aborted = false, timeout = false)
* Debug: until: packet processing thread terminated after 17,880 packets, 17,879 passed, 0 dropped, 0 nullified
* Debug[10]: dektec: waitWork(min_pkt_cnt = 1, pkt_first = 17,878, pkt_cnt = 1, bitrate = 5,199,983, input_end = true, aborted = false, timeout = false)
* Debug[10]: dektec: passPackets(count = 1, bitrate = 0, input_end = false, aborted = false)
* Debug[10]: dektec: waitWork(min_pkt_cnt = 1, ...)
* Debug[10]: dektec: waitWork(min_pkt_cnt = 1, pkt_first = 17,879, pkt_cnt = 0, bitrate = 5,199,983, input_end = true, aborted = false, timeout = false)
* dektec: terminating DTA-2174B output
* Debug[2]: ip: received UDP packet, source: <IP_ADDRESS>:11111, destination: <IP_ADDRESS>:51105, timestamp: 1,621,336,174,888,805
* Debug[10]: ip: passPackets(count = 7, bitrate = 5,199,983, input_end = false, aborted = false)
* Debug[10]: ip: waitWork(min_pkt_cnt = 1, ...)
* Debug[10]: ip: waitWork(min_pkt_cnt = 1, pkt_first = 17,892, pkt_cnt = 71,348, bitrate = 0, input_end = false, aborted = true, timeout = false)
* Debug: ip: input thread aborted after 17,892 packets
^C* tsp: user interrupt, terminating...
* Debug: aborting all plugins...
* Debug: aborting plugin ip
* Debug: ip: aborting IP input
* Debug: aborting plugin until
* Debug: aborting plugin dektec
^C

Hi @izharm,
Thanks for the traces. The termination is processed correctly. All plugins terminate. The dektec output plugin also terminates but it seems that the Dektec API hangs in the final detach operation.
Does it occur all the time with the DTA-2174B?
Could you try the option -O dektec --instant-detach ...? As indicated in the documentation, this option is recommended on Dektec modulators which sometimes hang during detach while ASI devices usually don't. Maybe the DTA-2174B has a detach behaviour which is similar to the modulators.
The only issue with --instant-detach is that the current output operation may be interrupted before all bytes are output. So, the last packets may not be output on the ASI port. Depending on what you are expecting, this may be a problem or not. If this option solves your problem, then this is the same issue as with some Dektec modulators and I see no other option.

Does it occur all the time with the DTA-2174B?

yes it does.

Could you try the option -O dektec --instant-detach ...?

yes, this solved the problem.
thanks.


Function startstop (rebased)

This fixes the issue "scene in a very quick chaser may stay ON after the end of the chaser"
Also, if several "sources" start a function (a chaser, a collection, whatever), all the sources have to stop the function for the function to actually stop.
This allows having a function in several collections at the same time.
If several collections contain the function, they will have to all be stopped for the function to stop.
-> The function will not be stopped when stopping only one collection.

The function-started-multiple-times issue will be fixed separately from https://github.com/mcallegari/qlcplus/pull/480


Issue with plugin replacementMap not being cleared

Issue raised by @Tony-S.
I've just done a brand new install of Winter using composer. I've added various plugins using e.g. php artisan plugin:install Winter.User
In the backend I get the following warning:
Some issues have been detected and require attention:
The Winter.Builder plugin replaces RainLab.Builder, please remove RainLab.Builder to ensure there are no conflicts
The Winter.User plugin replaces RainLab.User, please remove RainLab.User to ensure there are no conflicts

I have not installed the Rainlab versions. They don't appear in the list of plugins or in the manage plugins page and they don't show as files in the plugins files area. Is this warning therefor a bug? Things seem to be working otherwise, I can create users ok.

This problem is resolved in develop branch of winter.
Release 1.1.4 will contain this fix.


LND with Bitcoind connection refused

Starting:
bitcoind -txindex -server -daemon -rpcallowip=::/0 -zmqpubrawblock=tcp://<IP_ADDRESS>:28332 -zmqpubrawtx=tcp://<IP_ADDRESS>:28332 -rpcuser=xxx -rpcpassword=xxx -datadir=/bitcoin -externalip=<IP_ADDRESS> -rpcbind=<IP_ADDRESS> -rpcport 8332
lnd --logdir=/data --bitcoin.active --bitcoin.mainnet --bitcoin.node=bitcoind --bitcoind.rpcuser=xxx --bitcoind.rpcpass=xxx --bitcoind.rpchost=<IP_ADDRESS>:8332 --bitcoind.zmqpath=tcp://<IP_ADDRESS>:28332 --externalip=<IP_ADDRESS>
The result:
2018-03-31 10:34:26.397 [INF] LNWL: Opened wallet
unable to start wallet: dial tcp <IP_ADDRESS>:28332: connect: connection refused
unable to create chain control: dial tcp <IP_ADDRESS>:28332: connect: connection refused
dial tcp <IP_ADDRESS>:28332: connect: connection refused
What's can be wrong with it?

This would mean that the node isn't actually listening on the target port. You can use commands like lsof to detect if the node is actually listening. You'll also need to ensure that your bitcoind node was compiled with zmq. Otherwise it will silently fail to create the server.


Cannot read property 'topLeft' of undefined

Blueprint: https://pastebin.com/kjMB4YDC
After a little editing (deleted a few items at the top) - trying to copy results in an error: "Cannot read property 'topLeft' of undefined".

This has nothing to do with the rails.
I found a very simple way to reproduce this error.

I see what the problem is. You are deleting the first entity in the immutable entity array. At export it needs all blueprint corners to calculate the final entity positions and the default value for map.reduce is the first entity which got deleted. I will fix it in a bit.

BTW, Thanks for reporting this!


maybe cloudflare change there code again

`var target = new Uri("https://kissasian.sh/");
                var cf = new CloudflareSolver
                {
                    MaxTries = 3,
                    ClearanceDelay = 3000
                };

                var handler = new HttpClientHandler();
                var client = new HttpClient(handler);

                var result1 = cf.Solve(client, handler, target).Result;

                if (!result1.Success)
                {
                    Console.WriteLine($"[Failed] Details: {result1.FailReason}");
                    return;
                }

                // Once the protection has been bypassed we can use that HttpClient to send the requests as usual
                var content = client.GetStringAsync(target).Result;
                Console.WriteLine($"Server response: {content}");
                break;`

Unhandled exception. System.AggregateException: One or more errors occurred. (Input string was not in a correct format.)
---> System.FormatException: Input string was not in a correct format.
at System.Number.ThrowOverflowOrFormatException(ParsingStatus status, TypeCode type)
at System.Double.Parse(String s)
at CloudflareSolverRe.Types.Javascript.JsFuck.DecodeNumber(String encodedNumber)
at CloudflareSolverRe.Types.Javascript.JsCalculation.Solve()
at CloudflareSolverRe.Types.Javascript.JsCalculation.get_Result()
at CloudflareSolverRe.Types.Javascript.JsChallenge.ApplyCalculation(Double number, IJsCalculation calculation)
at System.Linq.Enumerable.Aggregate[TSource,TAccumulate](IEnumerable1 source, TAccumulate seed, Func3 func)
at CloudflareSolverRe.Types.Javascript.JsChallenge.Solve()
at CloudflareSolverRe.Solvers.JsChallengeSolver.SolveChallenge(String html)
at CloudflareSolverRe.Solvers.JsChallengeSolver.Solve()
at CloudflareSolverRe.CloudflareSolver.SolveJavascriptChallenge(Nullable1 jsDetectResult)    at CloudflareSolverRe.CloudflareSolver.SolveWithJavascript(Int32 tries)    at CloudflareSolverRe.CloudflareSolver.Solve()    at CloudflareSolverRe.CloudflareSolver.Solve(HttpClient httpClient, CloudflareHandler cloudflareHandler, Uri siteUrl, CancellationToken cancellationToken)    at CloudflareSolverRe.CloudflareSolver.<>c__DisplayClass30_0.<<Solve>b__0>d.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at CloudflareSolverRe.Utilities.SemaphoreLocker.LockAsync[T](Func1 worker)
at CloudflareSolverRe.CloudflareSolver.Solve(HttpClient httpClient, HttpClientHandler httpClientHandler, Uri siteUrl, CancellationToken cancellationToken, Boolean randomUserAgent)
--- End of inner exception stack trace ---
at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
at System.Threading.Tasks.Task1.GetResultCore(Boolean waitCompletionNotification)    at System.Threading.Tasks.Task1.get_Result()
at KissAsianData.Program.Main(String[] args) in C:\Users\Long charmroeun\source\repos\kisscambodia\KissAsianData\Program.cs:line 287
at KissAsianData.Program.(String[] args)
C:\Users\Long charmroeun\source\repos\kisscambodia\KissAsianData>

they change
function(p){var p = eval(eval(e("ZG9jdW1l")+(undefined+"")[1]+(true+"")[0]+(+(+!+[]+[+!+[]]+(!![]+[])[!+[]+!+[]+!+[]]+[!+[]+!+[]]+[+[]])+[])[+!+[]]+g(103)+(true+"")[3]+(true+"")[0]+"Element"+g(66)+(NaN+[Infinity])[10]+"Id("+g(107)+")."+e("aW5uZXJIVE1M"))); return +(p)}();
to
function(p){return eval((true+"")[0]+".ch"+(false+"")[1]+(true+"")[1]+Function("return escape")()(("")["italics"]())[2]+"o"+(undefined+"")[2]+(true+"")[3]+"A"+(true+"")[0]+"("+p+")")}(+((!+[]+(!![])+!![]+!![]+!![]+!![]+!![]+!![]+[])))));FsLhXWH.nhhfXxJAslmk-=+((!+[]+(!![])+!![]+!![]+!![]+!![]+!![]+!![]+[])+(!+[]+(!![])+!![])+(!+[]+(!![])+!![]+!![]+!![]+!![]+!![]+!![])+(!+[]-(!![]))+(!+[]+(!![])+!![]+!![])+(+!![])+(!+[]+(!![])+!![]+!![]+!![]+!![])+(!+[]+(!![])+!![]+!![])+(!+[]+(!![])+!![]+!![]+!![]+!![]+!![]))/+((!+[]+(!![])+!![]+!![]+!![]+!![]+!![]+!![]+!![]+[])+(!+[]+(!![])+!![]+!![]+!![])+(!+[]+(!![])+!![]+!![])+(!+[]+(!![])+!![]+!![]+!![]+!![])+(!+[]-(!![]))+(!+[]+(!![])+!![]+!![])+(!+[]+(!![])+!![]+!![]+!![]+!![])+(!+[]+(!![])+!![]+!![]+!![]+!![]+!![])+(!+[]-(!![])));

Would be awesome if you could push an Update like you did last week when you got it working  :)

it should be a new CalculationType

Do you know what exactly needs to be changed in the code?
I tried a few things, but unlike last week I couldn't bring it to work yet.

I know some change but my regular expression is not good xd

Same for me. xD
Everything that goes past basic regex is a no for me.
Well, maybe someone else gets a solution in a few hours. Time to be patient :)

Someone has the old HTML challenge to compare with the new one?

@ngosang
https://drive.google.com/open?id=1hjyZzGHgNz0sQxnzcZwMNIeyPy8P-9q5

i change
private static readonly Regex JsChallengeRegex = new Regex(@"


Add meta fields section to ReadTheDocs

In this PR, I add meta fields to readthedocs.

Here is the issue: https://github.com/kensho-technologies/graphql-compiler/issues/604


How do I write a "small" telegram

Hi Envy,
I try to use this great library to switch a lamp but it does not seem to work.
The log of KNXD shows these messages when I push a switch or dim with the same switch (see the first 3 log-lines). The same can be done by using the knxtool groupswrite command (I assume "s" stands for "small"):
LPDU: BC 11 82 10 0E E1 00 81 AE :L_Data low from 1.1.130 to 2/0/14 hops: 06 T_DATA_XXX_REQ A_GroupValue_Write (small) 01
LPDU: BC 11 83 10 0E E1 00 89 A7 :L_Data low from 1.1.131 to 2/0/14 hops: 06 T_DATA_XXX_REQ A_GroupValue_Write (small) 09
LPDU: BC 11 84 10 0E E1 00 88 A1 :L_Data low from 1.1.132 to 2/0/14 hops: 06 T_DATA_XXX_REQ A_GroupValue_Write (small) 08
LPDU: BC 11 85 10 0E E2 00 80 09 A2 :L_Data low from 1.1.133 to 2/0/14 hops: 06 T_DATA_XXX_REQ A_GroupValue_Write 09
LPDU: BC 11 82 18 0E E2 00 80 80 24 :L_Data low from 1.1.130 to 3/0/14 hops: 06 T_DATA_XXX_REQ A_GroupValue_Write 80

The last 2 lines are from the knxtool groupwrite command (without the "s").
Now, if i use the esp-knx-ip library and write a 1 byteInt value, the knxd log shows:
LPDU: BC 11 C8 08 0E E2 00 80 14 EA :L_Data low from 1.1.200 to 1/0/14 hops: 06 T_DATA_XXX_REQ A_GroupValue_Write 14
LPDU: BC 11 C8 08 0E E2 00 80 00 FE :L_Data low from 1.1.200 to 1/0/14 hops: 06 T_DATA_XXX_REQ A_GroupValue_Write 00

It looks like a "groupwrite".
So, my question is: how do I send a "small" telegram?
Here are the specs for a "small" telegram:

The 1byteInt looks like these specs:

Missed the sendBit() and writeBit() functions..


Update Composer dependencies (2024-08-13-00-18)

Visual regression test passed!
View the visual regression test report


RunTractor.py ERROR Phenotype ID must match with Hapdose file ID

Hi
After running RunTractor.py (version 0.0.1), I am getting an error but also an output summary stats file. I think my output summary stats is reasonable, and this could just be a minor bug but I'm not sure.
------
MEG_phase2.chr22.phase.anc0.hapcount.txt
------
MEG_phase2.chr22.phase.anc1.hapcount.txt
------
MEG_phase2.chr22.phase.anc0.dosage.txt
------
MEG_phase2.chr22.phase.anc1.dosage.txt
------
Notice:
Tractor drop one local ancestry term for regression. Therefore, MEG_phase2.chr22.phase.anc1.hapcount.txt will not be used.
------
ERROR: Phenotype ID must match with Hapdose file ID
END of calculation

wc -l summ_using_PCsAPOL/Phe.eskd2021_nicole.txt
     2775 summ_using_PCsAPOL/Phe.eskd2021_nicole.txt
head -1 summ_using_PCsAPOL/Phe.eskd2021_nicole.txt
IID	y

head -1 MEG_phase2.chr22.phase.anc0.dosage.txt |wc -w
2779   (CHROM	POS	ID	REF	ALT	1000560	1000625	1000628	1000633	1000634 ..... )
head -1 MEG_phase2.chr22.phase.anc0.hapcount.txt |wc -w
2779   (CHROM	POS	ID	REF	ALT	1000560	1000625	1000628	1000633	1000634 ..... )

You can see the Error message. But as you can see the header of the pheno file, and the number of samples in the pheno file and hapdosage files are all tallying correctly.
I just want to make sure that everything is correct.
Thank You.

Yes, please use the most recent version.


Support for Composite primary keys

Hello I am loving your library , thank you very much for creating it .
This is a question rather that an issue as I want to know if there is support for composite primary keys in the Repository. I am having some trouble understanding this as
IRepository<TEntity, TPk> seems to ahve one key unless I am missing something very obvious.
Thanks again

Hi @Pikou. Thanks for that :-)
Yes I made the repository only to support 1 primary key. But I use FastCRUD with dapper that does support composite key.
So the Get method will support it.
Otherwise you can do overrights in your implementation and use e.g. Session.Find etc from FastCRUD and it should work with composite.
The actual reason I based my library to utilize FastCRUD was because it's methods supported more than just the key.
In actual fact the TPk is only used for GetKey and SaveOrUpdate. So just override what you need :-)

Hello generik0
Thanks for your very fast reply :)


[1.19.2] Request for Gobber ore drop compatibility

hello,
Very nice mod.  Would you consider adding compatibility with my Gobber mod, so there are drops for the gobber ore drops?
https://www.curseforge.com/minecraft/mc-mods/gobber-fabric
https://github.com/kwpugh/gobber_fabric/tree/1.19.2/src/main/java/com/kwpugh/gobber2
Compatibility would be awesome.
Thank you for considering.
Regards,
kwpugh

We would probably accept a pull request implementing compat if you want to do it.


[WFRP4e System] Transferring an item

There are several problems when transferring items between 2 actors.

If you transfer an item from actor A's sheet to a container located in Actor B's sheet, the item disappears from sheet A, but does not appear in sheet B. I would like it to be possible to put the item directly into the container.
If you try to move a container with objects from actor A's sheet to actor B's sheet, then actor B will have only a container, and all the objects from it will disappear. They will disappear completely from the actor's list.From: mrak_xfoundry: 12.331wfrp4e: 8.2.2wfrp4e-altdorf: 6.0.1wfrp4e-archives1: 6.0.1wfrp4e-archives2: 6.0.1wfrp4e-archives3: 6.0.0wfrp4e-core: 6.0.1wfrp4e-dotr: 6.0.1wfrp4e-eis: 6.0.2wfrp4e-empire-ruins: 6.0.1wfrp4e-horned-rat: 6.0.1wfrp4e-middenheim: 6.0.2wfrp4e-owb1: 6.0.1wfrp4e-owb2: 6.0.1wfrp4e-pbtt: 6.0.0wfrp4e-rnhd: 6.0.2wfrp4e-salzenmund: 6.0.1wfrp4e-soc: 6.0.2wfrp4e-starter-set: 6.0.1wfrp4e-ua1: 6.0.1wfrp4e-ua2: 6.0.0wfrp4e-up-in-arms: 6.0.2wfrp4e-wom: 6.0.2wfrp4e-zoo: 6.0.0Active Modules: about-face, babele, combat-tracker-dock, dice-calculator, easy-ruler-scale, forien-actors-summary, forien-armoury, forien-quest-log, gm-screen, lib-wrapper, ru-ru, ru-wfrp4e, sequencer, socketlib, token-attacher, warhammer-lib, wfrp4e-gm-toolkit

Are you using Sheet V1 or Sheet V2?
Also it's weird, are you sure no module is interfering? Not only items do not disappear on the "source actor" when you drag them to other Actor, but when you add item to a new actor, it's "location" is scrubbed so they will appear..

V1.
I have now checked in a world without localization and without modules, only Core. The same thing. You can easily reproduce and see =)

That's the issue, I cannot reproduce.
When you drag items they get duplicated, always. And their location is scrubbed, so they always appear outside of containers.
Here is proof of fresh world with no modules (sorry, I lied, there is 1 module, because library is mandatory)

Maybe the auto-translator transmitted the wrong message. Create a container in Sheet A. Put the items in the container in sheet A. After that, drag the container with the items (not the items individually)   in sheet B).

Maybe the auto-translator transmitted the wrong message. Create a container in Sheet A. Put the items in the container in sheet A. After that, drag the container with the items (not the items individually) in sheet B).

this will only duplicate container, without its contents. But no items will disappear from the original actor.

I'm sorry))) I slept all evening, but we figured out that it works.
Drag the item from Sheet A to the Container located in Sheet B. The item will disappear.

Yes, being tired definitely doesn't make me clearer. The disappearance of the container with things happens as in the first scenario - when you transfer the container with items in sheet A to sheet B and put it in a container located in sheet B. Most likely this is 1 problem, it turns out, it's just a special case that I dragged the container separately.


Bug: No reset of metrics after each check run

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Currently the sparrow is exposing metrics for each check. In this example, I will reference the latency check but this is related to all the checks.
The latency check provides metrics with the statuscode as metadata in the metric itself, eg.:
sparrow_latency_duration_seconds{status="200",target="https://xyz.telekom.de/"} 0.1750766
If in one of the previous runs/check intervall the status was not 200 (eg. 500) the metric will be exposed with status="500".
If the current run is getting a 200 status code the metric will be exposed with status="200". The metric with status="500" is still present. The problem is that the sparrow is not exposing the last state. This is misleading since the status 500 is not the last state.
Expected Behavior
The prom metrics should represent the last state of the check.
Therefore the old metrics should be cleared before updating with the current state.
Steps To Reproduce
No response
Relevant logs and/or screenshots, environment information, etc.

Who can address the issue?
No response
Anything else?
No response

@y-eight While this bug is not exactly the same as #77, it's quite similar and I'd see it as one bigger issue that should be addressed in the near future.

The problem here is quite another; we are probably misusing prometheus metrics. A more appropriate solution would be to use a counter with the status and url labels (to count how many requests were successful or not) and just a latency_duration_seconds, which just returns how much time the last call needed to finish.
This way, we always return the core information of the check (which is the latency) and also expose some additional information.


specify node version in package.json

https://docs.npmjs.com/files/package.json#engines

added a couple of checks. should fail pretty explicitly now if you're on less than 8.5.0.


Asymmetric matcher message is not printed

Describe the bug
Given a custom matcher, when assertion fails, I expect to see the message which I've set up. Currently it's not present.
expect.extend({
  maybeAny(received, constructor) {
    if (received === null) {
      return {
        pass: true,
        message: () => 'Received value is equal to null',
      };
    }

    try {
      expect(received).toEqual(expect.any(constructor));
      return {
        pass: true,
        message: () => `Received value matches ${constructor} constructor`,
      };
    } catch {
      return {
        pass: false,
        message: () => 'Some custom message',
      };
    }
  },
});

Reproduction
https://stackblitz.com/edit/vitest-dev-vitest-f9qkgy?file=vitest.setup.ts
// when I try to run it in stackblits, I get an error Error: For a percentage based memory limit a percentageReference must be supplied, however on my machine it works fine
System Info
System:
    OS: macOS 13.4.1
    CPU: (8) arm64 Apple M1 Pro
    Memory: 94.03 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 19.8.1 - /opt/homebrew/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 9.5.1 - /opt/homebrew/bin/npm
  Browsers:
    Chrome: 115.0.5790.170
    Safari: 16.5.2
  npmPackages:
    @vitest/ui: latest => 0.34.1
    vite: latest => 4.4.9
    vitest: latest => 0.34.1

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

The behavior looks consistent to Jest (jest has better diff):
 FAIL  test/basic.test.ts
  ✕ Prints custom message (4 ms)

  ● Prints custom message

    expect(received).toEqual(expected) // deep equality

    - Expected  - 1
    + Received  + 0

      Object {
        "a": Any<Number>,
    -   "b": maybeAny<function Number() { [native code] }>,
      }

      19 |   expect({
      20 |     a: 1
    > 21 |   }).toEqual({
         |      ^
      22 |     a: expect.any(Number),
      23 |     b: expect.maybeAny(Number),
      24 |   })

      at Object.toEqual (test/basic.test.ts:21:6)

The message is meant for actual matchers.


Fix/pdf viewer

Description :sparkles:
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:

Additional notes :spiral_notepad:

Codecov Report

Merging #919 (caaf9f8) into develop (fd33b99) will decrease coverage by 40.07%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##           develop     #919       +/-   ##
============================================
- Coverage    62.98%   22.90%   -40.08%
============================================
  Files          868      102      -766
  Lines        23355     2484    -20871
  Branches      2333      636     -1697
============================================
- Hits         14709      569    -14140
+ Misses        8246     1848     -6398
+ Partials       400       67      -333

Impacted Files
Coverage Δ

...s/forms/application/step6/ApplicationFormStep6.tsx
0.00% <ø> (ø)

...applicant/src/components/pdfViewer/PdfViewer.sc.ts
0.00% <0.00%> (ø)

...t/applicant/src/components/pdfViewer/PdfViewer.tsx
0.00% <0.00%> (ø)

...s/forms/application/step2/ApplicationFormStep2.tsx
0.00% <0.00%> (ø)

frontend/shared/src/components/wizard/Wizard.tsx

frontend/shared/src/hooks/useSetQueryParam.ts

...ontend/shared/src/components/messaging/Message.tsx

...ntend/shared/src/components/link-text/LinkText.tsx

...et/admin/src/components/login/InfoboxContent.sc.ts

...rontend/tet/admin/src/components/editor/Editor.tsx

... and 760 more

Continue to review full report at Codecov.

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


[SelectField] Keyboard navigation does not focus/highlight current/first item

When using keyboard to navigate to, and use, a SelectField, when activating the field to open the dropdown, the currently selected item is not highlighted, nor in focus.
Example: https://www.youtube.com/watch?v=swz82ul6Hao
On clicking enter to open the dropdown, nothing is currently keyboard highlighted - on clicking "down" on the keyboard, the 2nd item is selected.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

We would expect the first/current item to be focussed on opening the dropdown.
Current Behavior

Steps to Reproduce (for bugs)

Tab to a SelectField
Click enter to activate/open the field
Note that nothing is currently focussed/selected
Pressing down selects the next item in the list.

Context

Getting complaints from users during testing that to select the first item, we need to click down, then up again, and confusing that nothing is focussed on activating the dropdown.
Your Environment

Tech
Version

Material-UI
0.18.0

React
15.6.1

browser
Any

etc

This issue has been fixed in the v1-beta branch.

Thanks Oliver.


Add debug symbol transformation for AArch64

Feature
Debug symbol transformation for AArch64. That is, when running wasmtime compile -g <path_to_wasm>, wasmtime does not exit with an error message and debug symbols are included in the resulting .cwasm file.
Benefit
It enables developers writing code targeting a Wasm environment to be able to step debug their code in a Wasm environment if they are using Apple Silicon.
Implementation
https://github.com/jeffcharles/wasmtime/pull/1/files. This uses a similar approach to how the symbols are mapped for x86_64. I can open that PR on this repo instead. The contribution guidelines mentioned I should open an issue here first to discuss.
The PR above seems to work for a number of test cases that I've tried.
Alternatives
I'm not sure what the alternatives are.

I had not seen that issue. I can take a look at those details.
Even without changes to the debug crate though, given my change, I'm currently able to get a reasonable debugging experience on AArch64 as opposed to an error message with the status quo.

I see the PR just makes aarch64 do exactly what x64 and s390x already do. I don't know much yet about Cranelift's target backends, but it seems plausible to me that inst::unwind::systemv::map_reg would be the right choice for aarch64 as well. @akirilov-arm, is there any reason not to merge this?
Also, maybe we should edit the contributor guidelines to say that if it's easier to explain your issue by just opening a pull request showing what you want changed, then you don't need to open an issue first? Anyway I'd say go ahead and open a PR here with your proposed changes.

@akirilov-arm, is there any reason not to merge this?

I don't know, usually I don't look at PRs that have not been opened in this repository. From a cursory glance it looks OK, but I am not really familiar with that part of the code.

I've opened a PR at #4468.

Merged, thank you!


Transform request body

What is the feature you are proposing?
Routes get request body with functions like await c.req.json(). In middleware you should be able to intercept and change the value the route gets with something like c.req.setJson(someTransformMethod(await c.req.json())).

If you would like to do that in the meantime, you can use c.set() / c.get() and pass the updated data to your route that way instead

As @NicoPlyley said, we can use c.set() / c.get(). Closing this.

@yusukebe c.get() doesn't have same benefit. If some routes body need to be transformed and others don't it is inconsistent. Some will have to use c.get().json and others use c.req.json() which is confusing and bring mistakes.
If possible to transform a body before getting to route all route can use c.req.json() and know it is right data.

@cdtut
This way is not recommended because this API is for internal usage, but you can make it with the code:
app.use('*', async (c, next) => {
  const data = await c.req.json()
  //
  c.req.bodyCache.json = 'transformeddata'
  await next()
})

app.post('/', async (c) => {
  const data = await c.req.json()
  return c.json(data)
})

Can internal c.req.bodyCache.json = data have external API like c.req.setJson(data) so can be used reliably? For all body requests not just json.

Can internal c.req.bodyCache.json = data have external API like c.req.setJson(data) so can be used reliably?

No. Basically, the Request body should be immutable, and we don't want to add more methods to the c.req.

Since the author of the library intends to keep the request body immutable, for @cdtut or anyone else who needs a consistent way to interact with and modify the body, here's a solution:
You can create a root middleware to handle the request body. Capture the value using await c.req.json() and store it in a variable like body.
const app = new Hono();

app.use(async (c: Context, next: Next) => {
    const body = await c.req.json();
    c.set('body', body);
    await next();
});

From this point onward, you can consistently access and interact with the body throughout the application using the stored variable. Retrieve the body with c.get('body'), and update it similarly with c.set('body', newBody).


Provide Feedback On Initially Expanded Rows To Parent Component

I've run into a bit of a problem using the row expansion feature in my own React SPA. Each of the records I'm passing to the table have an id, and I'm using react-router-dom to change the url as I expand and collapse rows. The base url is /mods and if a row is expanded then the url is /mods/:recordID. This is mostly working, but I haven't been able to handle the case where someone loads the page with an id in the url that doesn't exist. I'd like the router to navigate to /mods, but I can't think of a good way to do this that doesn't involve going through all of the records myself.
Instead of duplicating the logic inside of the table, I propose allowing the parent to pass in a callback function that will be called after the rows are initially expanded. The callback function would be passed an array of records that had been expanded by the function passed to initiallyExpanded. This would allow the parent to react to cases where nothing expanded, and other cases, without forcing them to execute the initiallyExpanded function against the records for a second time outside of the table.
I'm happy to take on the implementation of this, but I welcome feedback on the idea and potentially better ways of solving the problem.

After thinking a little bit more, maybe we could just expose expandedRecordIds in some fashion. That would be much simpler to do, and it would still solve my problem.

Duplicate of #73. My bad


[CORE-550] Pretty-print egress (#7598)

Cherry-picking from v2.2.x:

Pretty-print egress

Add tests for job & pipeline egress details

Add test cases for simple URL egresses

Per https://github.com/pachyderm/pachyderm/pull/7598#pullrequestreview-964707018

Remove extraneous newline

Per https://github.com/pachyderm/pachyderm/pull/7598#discussion_r866866813

Already reviewed in #7598.


Deploy to target already existing cloudfront

What is the problem this feature will solve?
I am already running cloudfront in our production environment and our domain is connected.
When creating a new cloudfront distribution and connecting the domain, it takes time to complete the connection, so I wish I could connect to the existing cloudfront instead of creating a new cloudfront distribution.
What is the feature that you are proposing to solve the problem?
I wish there was a feature that could deploy to an existing Cloudfront ID as a target.
What alternatives have you considered?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Sensitive Information Declaration

[X] I confirm that neither PII/PID nor sensitive data are included in this form

This would certainly be possible. The Cloudfront module only outputs the aws_cloudfront_distribution and aws_wafv2_web_acl resource, and these could be passed in as variables and a flag created to skip the Cloudfront module.
The Cloudfront module is integral to the function of the module, so a warning here that bringing your own resources would come at your own risk.


add fuzz test in chaos_experiment_run

Proposed changes
add fuzz tests in /chaoscenter/graphql/server/pkg/chaos_experiment_run
Types of changes
What types of changes does your code introduce to Litmus? Put an x in the boxes that apply

[ ] New feature (non-breaking change which adds functionality)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices applies)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] I have signed the commit for DCO to be passed.
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[x] I have added necessary documentation (if appropriate)

Dependency

Please add the links to the dependent PR need to be merged before this (if any).

Special notes for your reviewer:

fuzz build is failing can you check?


Trials are 14 days long, not 15 - Incorrect information in some views

https://github.com/CartoDB/cartodb/search?utf8=✓&q=15+days+free+trial
A user just reported this. For example, in the account settings, the "You are close to your limits" message shows 15 days, but trials are 14 days long.

Not sure who would manage this. @xavijam? @matallo?

@xavijam is your man here, there are some cases in central, too
ps: I went through the commercial website, and all occurences are "14 days", thanks for spotting these

:+1:, thanks @iriberri


Message handler is too slow; dropping message during tECDSA signing

#3418 fixed the problem with dangling message handlers no longer processing messages and having their buffers full. During the tests of #3404 on commit 9b28e73aff86bd52e1feb9e976e00b9149d26778, so after the fix was reverse-merged, I noticed similar behavior on one of the clients.
The second client (happy-path-2.txt) at 2022-11-26T00:22:25.120+0100 started reporting a massive number of message handler is too slow; dropping message warnings. This problem was not present on other nodes at that time. The number of warnings suggests it could be - again - some problem with a dangling receiver no longer processing messages.
The logs from all 4 clients running were uploaded to google drive given the size of the files:
https://drive.google.com/drive/folders/11MU2L6gDbUHgQ_b3X_m9cFjZLE3_mjxS?usp=sharing

I am not sure if this is the root cause of the observed problem but I think the control loop of a receiver is still broken:
https://github.com/keep-network/keep-core/blob/3a446af41e00fc3b35c9db9b3abdfe80e85b597a/pkg/net/libp2p/channel.go#L138-L163
If the handlers get blocked for whatever reason (full buffer for example), c.removeHandler(messageHandler) will never get called, and messageHandler.channel writer will have to start dropping messages leading to the logs I observed.
I opened #3420 with a proposition of fixing the API but I think the refactoring will require more thinking to do this right and we could potentially fix this problem sooner by splitting the handler goroutine into two - one piping messages between channels and one controlling the lifecycle of a handler.

One interesting observation is that so far, this problem was always activating after the announcement phase launches:

2022-11-28T15:23:10.079+0100	INFO	keep-tbtc	tbtc/signing_loop.go:211	[member:61] starting announcement phase for attempt [1]	{"wallet": "0x048da8d0ff8bfbb5c59d1b87519457cf90438266594319c82833cac04348c3db860af9e6325debf91c44e3a35c201642c9433b86d5cf03039e2ed81776f128ab58", "message": "0x66a8773a610c219e571734ad2ad614608b5888b6602e5a1a4178283088f30196", "signingStartBlock": 148626, "signingTimeoutBlock": 148831}
2022-11-28T15:23:10.204+0100	WARN	keep-libp2p	libp2p/channel.go:357	message handler is too slow; dropping message


ui: changed the remove tier button

Describe your changes:

I worked on changing the remove tier button for better UX
https://user-images.githubusercontent.com/51777795/233342177-cda9e15d-0e98-4276-a3a8-2fcebfc4676e.mov

Type of change:

[x] Improvement

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@aniketkatkar97 @chirag-madlani looks good


Tutorial request: k3s for developers, from container to scale (on own servers, without k8s knowledge)

Dear k3s team,
it would be great to get a developer friendly tutorial how to use k3s and its preferred tools to move from a dev container to a production cluster. I am looking for a bare metal ("dedicated server" with linux) simple command line tutorial without the need of learning Terraform, Vagrant, Ansible, Helm, BGP and so on and so forth.
Currently we have an external high-available load-balancer, 3 ingress servers running Traefik and Docker Swarm Masters, and multiple worker nodes. We spin up 3+ containers for each tennant, they automatically register with Traefik and are available load-balanced via their sub-domain.

Docker Swarm has served us well as a very easy to use tool to get started, but we seem to have some minor issues with high loads. At the same time no-one seems to be using Docker Swarm nowadays, related GitHub projects are mostly dormant. K3s seems to be a good alternative, especially as the initial setup seems as easy as Docker Swarm.
What I haven't found so far is an overarching tutorial of how to scale a web container with k3s on own servers. Yes, it is k8s compatible, but at the same time it has some tools build in and/or preferred, so a dedicated tutorial would be helpful. And it would be great to NOT have a lot of external tools which need to be understood and maintained. For starters it's just easier to setup a new server every month by hand instead of learning yet another tool to do this automatically. And for GDPR and money saving reasons our infrastructure is on dedicated servers, even A18z recognised cloud is expensive. To make live easier our scenario just uses an external load balancer.
It should include (using Docker Swarm terms here)

[x] setup of k3s cluster
[ ] setup of ingress with traefik and (purchased) SSL certificates (on 3 servers with fixed IP:port for high availability)
[ ] start a service with x containers with sub-domain as label and node-constraint (on workers only)
[ ] view status of cluster (maybe with a GUI)
[ ] do a rolling update of a service
[ ] advanced: collect logs with Prometheus/Grafana or Beats/Elastic/Kibana
[ ] more advanced: make it high-available without external LB but with external Floating IP

For reference a post which gets Traefik and a web service up and running on Docker Swarm with two config files and 3 CLI commands.

Have you looked at the Rancher Academy course on k3s?
https://community.suse.com/courses/4522316?utm_source=manual


[Docs] Note on running metric in dp

What does this PR do?

Fixes https://github.com/PyTorchLightning/pytorch-lightning/issues/4353
Metrics needs to be treated differently in DP mode (until we have statefull DP #1895). This PR adds a note to the docs what the user can do for now.
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

[ ] Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

@tchaton Only solution is to make a custom version of DataParallel (as PR #1895 tried to do) where we can insert hooks between these lines of code:
https://github.com/pytorch/pytorch/blob/73e121de1c69bba6863f38bea4b0044a920fdb20/torch/nn/parallel/data_parallel.py#L167-L168
such that metrics can accumulate their internal state after the data is send through the model, but before the results are gathered.


is there way to store/upload SSL Client Certificate via AT Command on ESP32 MCU

is there way to store/upload SSL Client Certificate on ESP32 MCU
via AT Commands with no need for Downloader Tools

Hi,
Maybe AT+SYSFLASH is helpful for you.

please can you provide a little sample
how to store certificate using this AT+SYSFLASH
0.delete existing cert,key,...
1.upload and save new cert,key,....

AT+SYSFLASH=<operation>,<partition>,<offset>,<length>
Function:
Read/write the user partitions in flash.

- <operation>:
0: erase sector
1: write data into the user partition
2: read data from the user partition
- <partition>: name of user partition
- <offset>: offset of user partition
- <length>: data length

For example：
// read 4096 bytes from the "ca_data" partition offset 0.
AT+SYSFLASH=2,"ca_data", 0,4096
// write 4096 bytes to the "ca_data" partition offset 0.
AT+SYSFLASH=1,"ca_data",0,4096
// erase 1 sector from the "ca_data" partition offset 0.
AT+SYSFLASH=0,"ca_data",0,4096
And about SSL Client Certificate format for ESP32 AT, please refer to https://github.com/espressif/esp32-at/blob/master/tools/README.md

// write 4096 bytes to the "ca_data" partition offset 0.
AT+SYSFLASH=1,"ca_data",0,4096

In this example what are the data that are being written to partition ca_data? I don't see a  parameter. Where do I enter the data that I want to write? For example, suppose I want to write a binary to an app partition. I would want to write the contents of hello-world.bin to partition ota_0. How can I do this using AT+SYSFLASH

// write 4096 bytes to the "ca_data" partition offset 0.
AT+SYSFLASH=1,"ca_data",0,4096

In this example what are the data that are being written to partition ca_data? I don't see a data parameter. Where do I enter the data that I want to write? For example, suppose I want to write a binary to an app partition. I would want to write the contents of hello-world.bin to partition ota_0. How can I do this using AT+SYSFLASH

https://www.espressif.com/sites/default/files/documentation/esp32_at_instruction_set_and_examples_en.pdf
page 97

Thanks, yes I see now how it works. After sending the command the UART waits for the data to follow.


Fix errors in DBG when compile with ENABLE_DASH_DEBUG defined.

.

Thanks! That makes me wonder - does anyone use DBG at all? 🤔


[FP8] Fix sharding for per-tensor-quantization

I tried doing it and see it brings some other commits not in this rebased branch. If they are rebase conflicts then I can manage them. If they are just extra commits I'm fine to bring them in in this PR or through others, (or committing directly). Your call.