Terminal screen redraw issues when ssh-ing from WSL2 console after recent Windows updates

Environment
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd]
Windows Terminal version (if applicable): 1.6.10571.0

Platform ServicePack Version      VersionString
-------- ----------- -------      -------------
 Win32NT             10.0.19042.0 Microsoft Windows NT 10.0.19042.0

Any other software?
WSL2

Steps to reproduce
My windows machine updated the other night and now I'm having weird terminal issues when I ssh to a remote host from a local WSL2 terminal.
Plain old shell generally seems to work on the remote end, but the moment it needs to go to some form of full screen redraw interface (e.g. less, top, vim, etc.) the screen can't refresh anymore.  In some cases, long output (e.g. ps -AH u) will cause it to stall output as well.
Seems to work okay for local sessions though.
The remote machines also work okay from a different Windows laptop's WSL that I have (for the moment since I haven't rebooted to apply the update yet), so it doesn't appear to be the remote end point that's the problem.
It also works okay if I ssh from a Putty terminal emulator on my problematic workstation, so it appears to really be a problem with the terminal emulator on this machine.  I have tried starting bash from

the various Windows Console programs terminal emulator (Win-R -> bash, pwsh, cmd)
the powershell terminal
the new "Windows Terminal"
These all exhibit the same behavior.

When the problem occurs I can enter the magic ~.~. sequence to kill the ssh session and return to the host terminal session.  At this point the cursor itself is gone (no longer a white block, underscore, etc.), though I can type again.  Running reset in the local bash session recovers the local behavior without needing to restart the console window, but I still get the same issue once I try to ssh to a remote host again.
Expected behavior
Free flowing characters over ssh :)
Actual behavior
Characters stop flowing :(
Links
Feedback hub report: https://aka.ms/AAbi4mw

Hmm. HMM. This sounds a lot like a window resize issue.
Can you run stty -a > stty.log before SSH, and then stty -a >> stty.log (note the >>) blindly after you get back to local bash after ~.~.?
Then paste the contents of stty.log. :smile:

So, here's what I think you asked for:
(redirect stty -a output to a file, both before and after an ssh session that causes the problematic issue and is terminated with ~.~.)
speed 38400 baud; rows 56; columns 194; line = 0;
intr = ^C; quit = ^\; erase = ^?; kill = ^U; eof = ^D; eol = <undef>;
eol2 = <undef>; swtch = <undef>; start = ^Q; stop = ^S; susp = ^Z; rprnt = ^R;
werase = ^W; lnext = ^V; discard = ^O; min = 1; time = 0;
-parenb -parodd -cmspar cs8 -hupcl -cstopb cread -clocal -crtscts
-ignbrk brkint ignpar -parmrk -inpck -istrip -inlcr -igncr icrnl ixon -ixoff
-iuclc -ixany imaxbel -iutf8
opost -olcuc -ocrnl onlcr -onocr -onlret -ofill -ofdel nl0 cr0 tab0 bs0 vt0 ff0
isig icanon iexten echo echoe echok -echonl -noflsh -xcase -tostop -echoprt
echoctl echoke -flusho -extproc
speed 38400 baud; rows 56; columns 194; line = 0;
intr = ^C; quit = ^\; erase = ^?; kill = ^U; eof = ^D; eol = <undef>;
eol2 = <undef>; swtch = <undef>; start = ^Q; stop = ^S; susp = ^Z; rprnt = ^R;
werase = ^W; lnext = ^V; discard = ^O; min = 1; time = 0;
-parenb -parodd -cmspar cs8 -hupcl -cstopb cread -clocal -crtscts
-ignbrk brkint ignpar -parmrk -inpck -istrip -inlcr -igncr icrnl ixon -ixoff
-iuclc -ixany imaxbel -iutf8
opost -olcuc -ocrnl onlcr -onocr -onlret -ofill -ofdel nl0 cr0 tab0 bs0 vt0 ff0
isig icanon iexten echo echoe echok -echonl -noflsh -xcase -tostop -echoprt
echoctl echoke -flusho -extproc

Did a vimdiff assuming the break was at speed ... for the before/after ssh on local session.  Seems to be the same.
Here's the version on ssh if I redirect it (again, prior to doing something that sets the terminal off):
speed 38400 baud; rows 56; columns 194; line = 0;
intr = ^C; quit = ^\; erase = ^?; kill = ^U; eof = ^D; eol = <undef>;
eol2 = <undef>; swtch = <undef>; start = ^Q; stop = ^S; susp = ^Z; rprnt = ^R;
werase = ^W; lnext = ^V; discard = ^O; min = 1; time = 0;
-parenb -parodd -cmspar cs8 -hupcl -cstopb cread -clocal -crtscts
-ignbrk -brkint ignpar -parmrk -inpck -istrip -inlcr -igncr icrnl ixon -ixoff
-iuclc -ixany imaxbel -iutf8
opost -olcuc -ocrnl onlcr -onocr -onlret -ofill -ofdel nl0 cr0 tab0 bs0 vt0 ff0
isig icanon iexten echo echoe echok -echonl -noflsh -xcase -tostop -echoprt
echoctl echoke -flusho -extproc

Here's the version after an initial ssh to the remote host (copy and pasted, no redirect):
speed 38400 baud; rows 56; columns 194; line = 0;
intr = ^C; quit = ^\; erase = ^?; kill = ^U; eof = ^D; eol = <undef>; eol2 = <undef>; swtch = <undef>; start = ^Q; stop = ^S; susp = ^Z; rprnt = ^R; werase = ^W; lnext = ^V; discard = ^O;
min = 1; time = 0;
-parenb -parodd -cmspar cs8 -hupcl -cstopb cread -clocal -crtscts
-ignbrk -brkint ignpar -parmrk -inpck -istrip -inlcr -igncr icrnl ixon -ixoff -iuclc -ixany imaxbel -iutf8
opost -olcuc -ocrnl onlcr -onocr -onlret -ofill -ofdel nl0 cr0 tab0 bs0 vt0 ff0
isig icanon iexten echo echoe echok -echonl -noflsh -xcase -tostop -echoprt echoctl echoke -flusho -extproc

Aha, these are starting to look different (redirected version is different from interactive version).  Is that normal?
However, I see nearly the same results on the local stty as remote (for both bare or redirected comparisons).  The only difference is the brkint setting which is -brkint in the remote cases.
Anyways, not sure if that helps you at all.  Let me know if you need more details.
Thanks much for the speedy response!

Here's another interesting way to get it to stop outputting:
echo {a..z}{a..z}

(no output)
However, this doesn't cause an issue:
echo {a..z}{0..9}

(full cross-product output)
Tried that after trying to count how many lines or characters ps -ef output stopped at (9 and 593).
If I login again to that remote machine from another console window, the previous one's processes are as expected (e.g. nothing hung/spinning.  Previous ps has already terminated, though its parent bash and ssh haven't).

Closing this.  Pretty sure there's an MTU issue in WSL2 land now, not the terminal.
If I do the following:
ping -Mdo -s 1373 $remote_host

The packets stop responding, though -s 1372 works fine.
A simple workaround for the moment is to do the following in the WSL2 VM (at least until I can track down where it's really going wrong):
ip link set mtu 1400 dev eth0

Just to close the loop on this, here's a couple of other references for more details/fixes regarding MTU path discovery issues with VPN and WSL2:
https://github.com/microsoft/WSL/issues/4246#issuecomment-766877500
https://github.com/microsoft/WSL/issues/6602
https://github.com/microsoft/WSL/issues/5638
https://github.com/microsoft/WSL/issues/4698#issuecomment-712890576


fix: prevent poller undefined issue

There was ADO bug 2135 which has since been closed and not reproducable but these changes might mitigate if it happens again.
As comment says, I believe there is weird condtion where the array has an undefined injected into it which is then processed later. This shouldn't be possible based on single threaded code and executing till completion which ensure the undefineds are filtered out before processed however it's the only way I think this bug can occur.

:tada: This PR is included in version 0.361.6 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Implements dark mode

How to add dark mode to a Gatsby site

Easy Dark Mode (and Multiple Color Themes!) in React | CSS-Tricks - CSS-Tricks

usehooks-ts - npm

useDarkMode | usehooks-ts

javascript - Next.js custom class on body using _document.js - Stack Overflow


Using transform_graph to quantize yolo-tiny-v2, performance & mAP drops.

I am using tranform_graph to quantize darkflow's tiny-yolo-v2 *.pb
https://github.com/thtrieu/darkflow
found several issues

when i use video to do inference (or images) the performance drops to very low with gpu enable
from 5.xx fps to 1.xx fps, and i can feel the images inference speed drops, by input with lots of pictures.

after use tranform_graph's quantization, the mAP drops from 68% to 13% (with my 500 pics selcted from pascal_voc2012)
the function i used list below

../tensorflow-src/tensorflow/bazel-bin/tensorflow/tools/graph_transforms/transform_graph --in_graph=_151875_12_28_02_36_51_tiny-yolo-v2.pb  --inputs='input' --outputs='output' --out_graph=_151875_12_28_02_36_51_tiny-yolo-v2.quantize_node.pb --transforms='remove_nodes(op=Identity, op=CheckNumerics)
  fold_constants(ignore_errors=true)
  fold_batch_norms
  fold_old_batch_norms
  quantize_weights
  quantize_nodes
  strip_unused_nodes
  merge_duplicate_nodes
  sort_by_execution_order'

after quantize mAP drops, i use "insert_logging" & "freeze_requantization_ranges"
tried to bring back mAP with that pascal_voc selected 500 pictures
but the mAP drops to 0%.

i wrote a script try to fix the logging file
I go thorough logging with single image, one by one
if the mAP goes up, i merge with the last highest mAP ...
until the last image
so comes out with a valid logging file, which bring back the mAP to 62%

now the questions are

is that possible to quantize yolo-tiny-v2?
because the mAP and performance drops so much (even thou the mAP can be adjust back)
are there any other neural network models that you suggest me to use?
my target is to create some quantized small models for running on ic chips
does that means logging and freeze_requantization_ranges might not suitable for tiny-yolo-v2?

Thanks for your time for reading this
== cat /etc/issue ===============================================
Linux 8fbb96ca119d 4.15.0-34-generic #37~16.04.1-Ubuntu SMP Tue Aug 28 10:44:06 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
VERSION="16.04.5 LTS (Xenial Xerus)"
VERSION_ID="16.04"
VERSION_CODENAME=xenial
== are we in docker =============================================
Yes
== compiler =====================================================
c++ (Ubuntu 5.4.0-6ubuntu1~16.04.10) 5.4.0 20160609
Copyright (C) 2015 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
== uname -a =====================================================
Linux 8fbb96ca119d 4.15.0-34-generic #37~16.04.1-Ubuntu SMP Tue Aug 28 10:44:06 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
== check pips ===================================================
numpy                 1.15.4
protobuf              3.6.1
tensorflow-gpu        1.12.0
== check for virtualenv =========================================
False
== tensorflow import ============================================
tf.VERSION = 1.12.0
tf.GIT_VERSION = v1.12.0-0-ga6d8ffae09
tf.COMPILER_VERSION = v1.12.0-0-ga6d8ffae09
Sanity check: array([1], dtype=int32)
== env ==========================================================
LD_LIBRARY_PATH /usr/lib:/usr/local/nvidia/lib:/usr/local/nvidia/lib64
DYLD_LIBRARY_PATH is unset
== nvidia-smi ===================================================
Sat Jan 26 05:56:00 2019
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 390.87                 Driver Version: 390.87                    |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  Quadro GV100        Off  | 00000000:18:00.0 Off |                  Off |
| 40%   54C    P0    50W / 250W |      1MiB / 32508MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
|   1  Quadro GV100        Off  | 00000000:3B:00.0 Off |                  Off |
| 40%   54C    P0    50W / 250W |      0MiB / 32508MiB |      5%      Default |
+-------------------------------+----------------------+----------------------+
+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID   Type   Process name                             Usage      |
|=============================================================================|
|  No running processes found                                                 |
+-----------------------------------------------------------------------------+
== cuda libs  ===================================================
/usr/local/cuda-9.0/targets/x86_64-linux/lib/libcudart_static.a
/usr/local/cuda-9.0/targets/x86_64-linux/lib/libcudart.so.9.0.176

We're deprecating the graph transform tool, and recommend TF Lite for quantization, so I'm closing this one.


Implement a test suite for the bounty board

We do not currently have a test suite nor any automated test hooks built into the Bounty board, beyond the typescript compile time checks on commit.
This is something we will need to address as the application grows in complexity.
Looking at other banklessDAO projects, a proposal could be:
Stage 1:

Implement a unit test suite in Jest to test the base functionality of the application
Agree on a target coverage metric and begin writing tests to achieve said coverage

Stage 2:

Add a pre-commit hook and/or github actions stage to run a suite of tests against the application on various OS
This is so that we keep on top of testing as part of a continuous process

Stage 3:

Add Cypress or another E2E testing suite, this does not need to be in any hook, but any developer can run the application and get a visual overview of the base functionality. At the same time, Cypress solves the problem of manually clicking and validating test cases.

Implemented in most recent release


support projects using ESM by adding a package.json with type commonjs

I hope this won't break existing projects, I so, I'll revert 🤞

Thanks!


tweaking example to something that works on my local

I was getting an error using the implementation
const implementation = {
    helloworld: {
	Greeter: {
	    sayHello,
            sayGoodbye
	}
    }
}

but
const implementation = {
    helloworld: {
	Greeter: {
	    SayHello: sayHello,
        SayGoodbye: sayGoodbye
	}
    }
}

works.

I think recent changes to grpc fix this, maybe deps needs an update. I remember there were other issues blocking a version increment, but I can take a look.

Since the proto isn't documented, this isn't a bug, it means you didn't match the case to the expected case. This will probly work for your proto, for example:
const grpcnode = require('grpcnode').server
const grpc = require('grpc')

function SayHello (call, callback) {
  const message = `Hello ${call.request.name}`
  callback(null, {message})
}

function SayGoodbye (call, callback) {
  const message = `Goodbye ${call.request.name}`
  callback(null, {message})
}

const implementation = {
  helloworld: {
    Greeter: {
      SayHello,
      SayGoodbye
    }
  }
}

I will check on updating grpc, but the issue here is that the case of your implementation doesn't match. I used to have the example-code in this repo, but decided to link to it (in the README.) Did you have  look at that?

I am closing for now, as the structure of the callbacks has changed to match across the board. Now you should use the case that is in the proto:
module.exports = {
  helloworld: {
    v1: {
      Greeter: {
        SayHello: ({request: { name }}) => ({message: `Hello ${name}`}),
        SayGoodbye: ({request: { name }}) => new Promise((resolve) => {
          setTimeout(() => {
            resolve({message: `Bye, ${name}`})
          }, 1000)
        })
      }
    }
  }
}


Fix shorthand property assignment breaking refactor move

Fixes #59131

@microsoft-github-policy-service agree

Hi @sheetalkamat, would you be so kind to see if any additional review from you or your colleagues would be required for this PR to be merged?


"An error occurred while loading Azure resources: Requests from this account have been throttled"

Issue Type: Bug
Unable to see all Azure resources, whenever I try to browse I get this error message
"An error occurred while loading Azure resources: Requests from this account have been throttled. To retry, please select a smaller number of subscriptions."
I have two Azure identities, one with a single subscription and one with access to 12 subscriptions.
For all other tools (Azure Storage Explorer for example) this works fine.
But with Azure Data Studio I am throttleded.
Can this be handle in a more user-friendly way?
Azure Data Studio version: azuredatastudio 1.40.1 (3a32679ab590c1023ab4de6f191769e7e127d25b, 2022-11-22T18:30:25.850Z)
OS version: Windows_NT x64 10.0.22621
Restricted Mode: No
Preview Features: Enabled

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz (8 x 2803)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.73GB (17.68GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (4)

Extension
Author (truncated)
Version

dacpac
Mic
1.11.0

schema-compare
Mic
1.17.0

sql-database-projects
Mic
0.20.0

vscodeintellicode
Vis
1.2.1

Hi @yooakim
We're aware of this issue in the Browse tab and will hopefully fix this soon.
In the meantime, you may access resources in the Connections > Azure tree view, and let us know if you face any issues.


Build of stable images failed

Looks like it failed to install packages during the build https://github.com/factoriotools/factorio-docker/actions/runs/10584438516/job/29328687156#step:4:200
Any chance you could run the build action again?

I retried it

Looks like that did it. Thanks.


Macro cleanup + Additions

Macros are pretty messy. Usage of ARR_X to prevent Quote catching , as separate arguments isn't really wide spread.
Need to get file line numbers working.

Add Logging in general to this

Added to veh compat since on PR


[Connector-V2] [E2E] Add missed ElasticSearch E2E module.

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

LGTM


CIF-2982 - Support product filter hooks in ProductList component

Description

Moved extendProductFilterWith up from SearchResults to ProductCollection so it can also be used by the ProductList component.
Added unit test.

How Has This Been Tested?

Unit test
Proxy component in Venia

Types of changes

[ ] 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 change)

Checklist:

[X] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[X] I have added tests to cover my changes and the overall coverage did not decrease.
[X] All unit tests pass on CircleCi.
[X] I ran all tests locally and they pass.

With this approach we now have two ways to extend the query of the product list component: by using the productQueryHook of the ProductList#getCategoryRetriever() and by using the newly introduced filter hook.
Wouldn't it make more sense to add the product filter hook to the AbstractCategoryRetriever? That way the extension pattern remains the same. Also that the SearchResults and the ProductList use the same queries is an implementation detail, it does not mean that any ProductCollection would allow to do that.

@Buuhuu I think we have a few consistency issues here:

SearchResults doesn't use any retriever, so we cannot use the same pattern here as with the ProductList.
AbstractCategoryRetriever is used in the context of a category query. While extending category, category filter and products make sense here, extending the product filter does not. This retriever would not be using it for anything, but we would only have an external consumer.
I agree that moving the extensions to ProductCollection might not be the best place, since as you said, we cannot expect any subclass to allow the same grade of extensibility.

I don't have a good answer here, since frankly the SearchResultsService is a mess.


Wrong arrow position on big elements

First of all, thanks for the great work on this plugin!
I'm having an issue using it on large elements. Basically, I have a div element with dynamic height and everytime the height is big enough, the arrow's position is wrong.
It seems like the arrow's y position is in the middle of the div element, while the popover's y position remains on the top of the element.
Sometimes it works as expected on elements that are not close to the top of the page.
Here are some screenshots describing this issue:
If this element has 50px of height, the popover appears as expected.

But, if I increase the height to 100px, this is the result:

And, if I have a 500px height, this is what I get:

And here, the second popover works as expected

And this is the code that I'm using for this test:
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

  <link rel="stylesheet" href="jquery.webui-popover.css" />

I'm testing on Safari 8.0.3, Chrome 40.0.2214.93 and Firefox 35.0.1

Oh,this looks a bit strange, I'll debug your example and try to fixed the bug
Thanks for your feedback

@paulomoreno  This problem is solved, and I add your  code to file test.html ,hope you don't mind.

That's great! I tried it here and it's working flawless now. That's ok, you can use it!
And again, thanks for the great work!

@sandywalker what was the solution because i have the same effect with the lastest code ?


neba-127 React to runtime changes of resource type hierarchies in the…

#127

Coverage decreased (-0.02%) to 91.027% when pulling 2ce8054e1d39789e3d1264d6c27508b7597fd328 on neba-127 into cf40472424f53e6ad8a2acbd751c9e3a7b09fd3a on develop.

Coverage increased (+0.09%) to 91.144% when pulling b27f4cb113442a46a8839bb75db1e2479fe6d655 on neba-127 into cf40472424f53e6ad8a2acbd751c9e3a7b09fd3a on develop.


ranged enemy movement behavior

I tested this out on the branch, looks pretty good. Both ranged and melee enemies are moving as expected.


Improved screenshot

The python screenshot module no longer touches disk

Hey peto01 - Curious: why did you take the save to file option out on the server side?

Nice addition, btw

Hi malcolmvetter. Correct me if I'm wrong, but the SavePath argument stated where the temporary screenshot file would be saved on the victim side. My changes remove the need for a temporary file on disk. In my testing the resulting .PNG file still gets saved on the Empire server under the downloads/AGENT_ID/screenshot folder.

peto01 Add that back in and test on your side, that's where the framework drops files from modules.  For example, take a look at the macro modules.  If not specified (Mac macro default as an example) it will print to screen, if specified it should redirect output to a file.
Also take a look at the windows version, we would want to make sure the file name doesn't stay static and overwrite previous captures.

Hi,
My 2 cents, but maybe should have changed the native_screenshot module instead, in order to keep two screenshot modules based on two diffrent techniques.
TPWSOS 🌻

Or just set a flag in config for which option and have both in a single module.

I originally implemented this change on the EmPyre project and was asked to submit it here. Now that I have looked at Empire I agree with TPWSOS, the native_screenshot module does the same thing and is much more concise.


Fail to load dynamic library

After installing the packages, I have received the following issue,
Error
rishi@blackbox:~ ] shellen
Traceback (most recent call last):
File "/usr/local/bin/shellen", line 7, in
from shellen.main import main
File "/usr/local/lib/python3.6/site-packages/shellen/main.py", line 3, in
from shell import Shellen
File "/usr/local/lib/python3.6/site-packages/shellen/shell.py", line 18, in
from asms.disasm import DisassemblerWrapper
File "/usr/local/lib/python3.6/site-packages/shellen/asms/disasm.py", line 6, in
from capstone import *
File "/usr/local/lib/python3.6/site-packages/capstone/init.py", line 230, in
raise ImportError("ERROR: fail to load the dynamic library.")
ImportError: ERROR: fail to load the dynamic library.
Packages Install Validation:
rishi@blackbox:~ ] sudo pip3 install shellen
Requirement already satisfied: shellen in /usr/local/lib/python3.6/site-packages
Requirement already satisfied: pygments in /usr/local/lib/python3.6/site-packages (from shellen)
Requirement already satisfied: termcolor in /usr/local/lib/python3.6/site-packages (from shellen)
Requirement already satisfied: requests in /usr/local/lib/python3.6/site-packages (from shellen)
Requirement already satisfied: terminaltables in /usr/local/lib/python3.6/site-packages (from shellen)
Requirement already satisfied: colorama in /usr/local/lib/python3.6/site-packages (from shellen)
Requirement already satisfied: prompt-toolkit in /usr/local/lib/python3.6/site-packages (from shellen)
Requirement already satisfied: keystone-engine in /usr/local/lib/python3.6/site-packages (from shellen)
Requirement already satisfied: capstone in /usr/local/lib/python3.6/site-packages (from shellen)
Requirement already satisfied: chardet<3.1.0,>=3.0.2 in /usr/local/lib/python3.6/site-packages (from requests->shellen)
Requirement already satisfied: urllib3<1.23,>=1.21.1 in /usr/local/lib/python3.6/site-packages (from requests->shellen)
Requirement already satisfied: idna<2.7,>=2.5 in /usr/local/lib/python3.6/site-packages (from requests->shellen)
Requirement already satisfied: certifi>=2017.4.17 in /usr/local/lib/python3.6/site-packages (from requests->shellen)
Requirement already satisfied: six>=1.9.0 in /usr/local/lib/python3.6/site-packages (from prompt-toolkit->shellen)
Requirement already satisfied: wcwidth in /usr/local/lib/python3.6/site-packages (from prompt-toolkit->shellen)
Thanks!

Check out this issue. pip installed capstone's libraries in a wrong directory. Move this libraries in your /usr/local/lib/python3.6/site-packages/capstone/ path, or just install capstone manually.

Thanks, that solved the capstone error I guess, but now this:
rishi@blackbox:~ ] shellen
Traceback (most recent call last):
  File "/usr/local/bin/shellen", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.6/site-packages/shellen/main.py", line 9, in main
    shell = Shellen()
  File "/usr/local/lib/python3.6/site-packages/shellen/shell.py", line 54, in __init__
    self.__asm = AssemblerWrapper(X86_32)
  File "/usr/local/lib/python3.6/site-packages/shellen/asms/asm.py", line 57, in __init__
    self.executor = Assembler(self.arch)
  File "/usr/local/lib/python3.6/site-packages/shellen/asms/asm.py", line 16, in __init__
    super().__init__()
  File "/usr/local/lib/python3.6/site-packages/shellen/asms/baseexc.py", line 28, in __init__
    self._archs   = self.avail_archs()
  File "/usr/local/lib/python3.6/site-packages/shellen/asms/asm.py", line 28, in avail_archs
    ARM32:   (KS_ARCH_ARM,     KS_MODE_ARM),
NameError: name 'KS_ARCH_ARM' is not defined

Try this issue. It suggests installing keystone manually as well :D
Check these instructions for installing keystone manually.


[SUGGESTION] Compat for mods that add some of the same gemstones as you

An example would be Tech Reborn, as it has a few variants of garnet.

I've had this thought before, and am certainly open to the idea. I haven't gotten around to figuring out exactly how I'd handle it at the end of the day.
One issue is that certain gems are much easier to obtain in other mods than they are assumed to be in AI. Citrine from Spectrum, for example, is found all over in geodes, while  it can only be obtained naturally in the Nether in AI. These gems also don't have common/fabric tags at the moment, so the implementation would be a bit tricky.
One direction I've considered taking is an "Amethyst Additions" style of addon mod that adds brand new recipes aimed at integrating other gem mods, especially Spectrum, which is a similarly themed mod overall.


Safeguard against redirects on PUT request

A redirect on a PUT request will make the http client perform a GET request to the signposted URL. This will (probably) return a 200, which the code will then interpret as a successful PUT. This check ensures that the method the response relates to is the same as the one we invoked.

Codecov Report
Attention: Patch coverage is 33.33333% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 38.67%. Comparing base (3c58af4) to head (bbf2f52).
Report is 33 commits behind head on main.

Files
Patch %
Lines

client/http/witness_client.go
0.00%
4 Missing :warning:

internal/distribute/rest/distribute.go
50.00%
0 Missing and 1 partial :warning:

internal/feeder/pixelbt/pixel_feeder.go
0.00%
1 Missing :warning:

internal/feeder/rekor/rekor_feeder.go
0.00%
1 Missing :warning:

internal/feeder/serverless/serverless_feeder.go
0.00%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #191       +/-   ##
===========================================
- Coverage   51.05%   38.67%   -12.38%
===========================================
  Files          11       21       +10
  Lines         903     1148      +245
===========================================
- Hits          461      444       -17
- Misses        374      613      +239
- Partials       68       91       +23

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[icons] What font is used in Material Icons?

I would have made a discussion, but the "Submit" button simply disables, and nothing is created.
What font is used in the Material Icons that have letters in them?

Hey @AlbinoGeek, thanks for opening the issue! Can you point to some examples?

Small blocky font: Abc, Atm, Co2, EightK, EightKPlus, EightMp, EighteenMp, EightteenthMp, ElevenMp, FiberDVR, FiberNew, FiberPin, FifteenMp, FiveK, FiveKPlus, FiveMp, FivteenMp, FourKPlus, FourMp, FourteenMp, Hd, HdrAutoSelect, HdrEnhancedSelect, HdrOff, HdrOffSelect, HdrOn, HdrOnSelect, HdrPlus, HighQuality, Hevc, Hls, ...
Different small blocky font: 5G, FourGMobiledata, FourGPlusMobiledata, GMobileData, HMobileData, HPlusMobileData, LteMobileData, LtePlusMobileData,
Numbers: Filter1, Filter2, Filter3, Filter4, Filter5, Filter6, Filter7, Filter8, Filter9, Filter9Plus, Looks1, Looks2, Looks3, Looks4, Looks5, Looks6
Different Numbers: Timer10, Timer10Select, Timer3, Timer3Select,
Currencies
Punctuation (a stretch): AssignmentLate, Error, Help, QuestionMark

After having a look through, it's actually amazing how inconsistent some of these are.
When it comes to the MobileData or brand names, sure, but otherwise, why are there at least three distinct sets of numbers?

This is genuinely a good question, however, I don't think we (MUI) will be able to answer or solve that as far as Material UI goes. These are Material Design icons—nothing custom (as in made by ourselves, as far as I know) is added there, we just make sure that Material UI is up to date with https://fonts.google.com/icons.
Maybe you should head over to https://github.com/google/material-design-icons and ask for further clarification there!

Closed for https://github.com/google/material-design-icons/issues/1306


nodePackages.rtlcss: use buildNpmPackage

Description of changes
See #229475.
Maintainer needed.

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

is there any reason why not to keep it in nodePackages?

nodePackages is unmaintainable. A change for one package can and usually will cause merge conflicts for other PRs. It's massive size means autogeneration can take hours. See #229475.


Idle Off not working for me

I'm not sure why this stopped working but I can't seem to figure out the issue. I've completely uninstalled/installed the newest version. I even deleted the device and added it again. I have confirmed IP address is correct. I set it to go black/off after 1 minute of idle.
Any suggestions?

Usually this means something prevents idle and the most common reason seem to be a joystick or a ps5 controller. Open a command prompt and navigate to the installation directory and execute "LGTVdaemon.exe -show", watch the idle counter in the window that opens and start unplugging devices till the counter starts to increase. This should give some insight to what is going wrong.  Let me know how it goes please

Hello, looks like it's my Fanatec Handbrake that is causing this. It's part of my sim rig. Is there a way to keep this plugged in and still use LG Companion?

Unfortunately some controllers do this. You will likely find it prevents the normal windows screensaver/screen off power stteigs too. Please see my input at the bottom of #58 for some additional info

I see, thanks for the info

Closing the issue

Just leaving this here as there is some progress on the issue: https://www.techradar.com/news/its-finally-game-over-for-one-of-windows-11s-oldest-and-annoying-bugs

awesome thanks! I just upgraded to W11 last week too. My sim rig stuff was
causing the issue. I have since just left it disconnected when not using it
to work around it.
On Wed, Feb 15, 2023 at 12:47 PM JPersson77 @.***>
wrote:

Just leaving this here as there is some progress on the issue:
https://www.techradar.com/news/its-finally-game-over-for-one-of-windows-11s-oldest-and-annoying-bugs
—
Reply to this email directly, view it on GitHub
https://github.com/JPersson77/LGTVCompanion/issues/110#issuecomment-1431846310,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPRFKQNJ5S3IGQD7U5E5STWXUQE3ANCNFSM6AAAAAAUJZZPEU
.
You are receiving this because you authored the thread.Message ID:
@.***>

No worries! I got a report that KB5022845 fixed the similar issue he was experiencing with his 8bitdo controller 👍

I have a question. I have a LG C1 and C2. I never used the companion app on
the C1 as most use was with a PS5. However I'm now using the companion app
for both TV's since I do some PC gaming on both. The companion app is
turning my C1 black (assuming idle timer) even though I'm playing games on
my PS5. Is there a way to stop the idle function when I'm on my PS5?
On Fri, Feb 17, 2023, 3:05 AM JPersson77 @.***> wrote:

No worries! I got a report that KB5022845 fixed the similar issue he was
experiencing with his 8bitdo controller 👍
—
Reply to this email directly, view it on GitHub
https://github.com/JPersson77/LGTVCompanion/issues/110#issuecomment-1434342903,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPRFKWRBAVETHMHY3VTJ23WX45PDANCNFSM6AAAAAAUJZZPEU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Nevermind my last question. Guess I should look in options before asking! I
see I can set it to work when only certain HDMI port is in use.
On Fri, Feb 17, 2023, 3:05 AM JPersson77 @.***> wrote:

No worries! I got a report that KB5022845 fixed the similar issue he was
experiencing with his 8bitdo controller 👍
—
Reply to this email directly, view it on GitHub
https://github.com/JPersson77/LGTVCompanion/issues/110#issuecomment-1434342903,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPRFKWRBAVETHMHY3VTJ23WX45PDANCNFSM6AAAAAAUJZZPEU
.
You are receiving this because you authored the thread.Message ID:
@.***>


not work! not work!

The plugin does not work. Sketch 42

Hi, @DopDrop,
I'm using it in Sketch 42 without problems. Please make sure you have the latest version of the plugin.


Parse embedded multipart messages

Currently MIME parts that are embedded into other multi-part MIME parts are not parsed.
Change this.

First implementation: 2fff9ee72071de437ec64b8260275db5504fb2f8
Currently it works with multipart bodies that envelope all other contents. Will see if I even get any multipart emails where this doesn't stand.


Istanbul University added.

@muhammetsafak I am sorry, the domain ogr.iu.edu.tr is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Not correct for VS 2022

The page instructions for the creation of a WinUI app does not appear to be correct for VS 2022.  There is no WinUI type.

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

ID: 9564158a-dfea-b690-7b5e-5f6987019710
Version Independent ID: 1a90b127-6ad9-a28e-0ee4-16170021b104
Content: Create a new project that uses the Windows App SDK - Windows apps
Content Source: hub/apps/winui/winui3/create-your-first-winui3-app.md
Product: reunion
Technology: winui-conceptual
GitHub Login: @Karl-Bridge-Microsoft
Microsoft Alias: kbridge

Sorry for the delay in responding.
The problem was probably that this topic didn't have a link to Install tools for the Windows App SDK. The instructions there are tailored for the version of Visual Studio you're using. For example, for VS2022 17.1, in the Installation details pane of the installation dialog, for the C# and/or C++ workload, you'd need to check **Windows App SDK C#/C++ Templates" (at the bottom of the list). Once you've done that, you'll have the WinUI 3 project and item templates in Visual Studio. For other versions of VS, you'll need to install the VSIX. All details are at the link above.
I've fixed both topics now.
-Steve


cue/go: cuevalue.Fields report  non-concrete value struct

What version of CUE are you using (cue version)?

$ cue version
 v0.5.0-alpha.1.0.20221022113314-dd1169c686c9

Does this issue reproduce with the latest release?
yes
What did you do?

Here's the repro:
# tip
go mod tidy
go run main.go

-- go.mod --
module test-cue/openapi-resolve

go 1.19

require cuelang.org/go v0.5.0-alpha.1.0.20221022113314-dd1169c686c9

require (
	github.com/cockroachdb/apd/v2 v2.0.2 // indirect
	github.com/google/uuid v1.2.0 // indirect
	github.com/mpvl/unique v0.0.0-20150818121801-cbe035fff7de // indirect
	github.com/pkg/errors v0.8.1 // indirect
	golang.org/x/net v0.0.0-20220722155237-a158d28d115b // indirect
	golang.org/x/text v0.3.8 // indirect
	gopkg.in/yaml.v3 v3.0.1 // indirect
)
-- main.go --
package main

import (
	"fmt"

	"cuelang.org/go/cue"
	"cuelang.org/go/cue/cuecontext"
	"cuelang.org/go/cue/format"
)

var test = `
useless: string

#Params: {
	image: string
}

parameter: #Params | {
	containers: [...#Params]
}
`

func main() {
	cueV := cuecontext.New().CompileString(test)
	v := cueV.LookupPath(cue.ParsePath("parameter"))
	b, err := format.Node(v.Syntax())
	if err != nil {
		panic(err)
	}
	fmt.Println(string(b))
	_, err = v.Fields()
	if err != nil {
		panic(err)
	}
}

What did you see instead?
> go run main.go
[stdout]
{
        PARAMS.#x | {
                containers: [...PARAMS.#x]
        }

        //cue:path: #Params
        let PARAMS = {
                #x: {
                        image: string
                }
        }
}

[stderr]
panic: parameter: non-concrete value struct

This is a blocker for KubeVela. CUE Team to investigate.

This one's priority is high because there's no workaround. I'll say this is a user-facing problem since we're generating docs from cue parameters for our users, and this issue blocks the generator. Hope this can be fixed in v0.6.

+1

@FogDong apologies for the delay in replying here.
To build on @mpvl's answer.

parameter is not concrete as it is a disjunction.

To build on this, the critical point is that because parameter is a disjunction (that cannot be resolved to a single item) it is, by definition, not a struct value. Even though it is a disjunction of two struct values, it is still ambiguous as to what Fields() could/should return in this situation.

AFAICT, you did not specify what you expected to see.

To build on this too, we need to understand what list of fields you're expecting to see in this situation and why.

@FogDong: Could you specify what you expected to see in this case? AFAICT, the output is correct.


Enabling access to containers outside localhost

Hi,
I'm trying to enable access to container outside localhost inside the network. I'm doing this for our development and testing environment since we need more people to access business central but also other applications using web service to be able to target our solution.
Can you advice what would be the best option to accomplish this?
I am currently trying to do port mapping for container but without success. The script I'm using looks like this:
$imageName = 'bcinsider.azurecr.io/bcsandbox-master:13.0.23638.0-ca'
$cred = Get-Credential -UserName $env:USERNAME -Message "Enter your Credentials"
$containerName = 'mycontainer'

New-NavContainer -containerName $containerName `
-imageName $imageName `
-accept_eula `
-Credential $cred `
-auth NavUserPassword `
-licensefile "C:\myfolder\mylicense.flf" `
-includecside `
-multitenant `
-restart always `
-enableSymbolLoading `
-doNotExportObjectsToText `
-additionalParameters "--env PublicDnsName=$containerName --publish 8081:8080 --publish 444:443 --publish 7146-7149:7046-7049"

I would suggest using Transparent network instead of NAT.
https://docs.microsoft.com/en-us/virtualization/windowscontainers/container-networking/architecture

@Koubek Thanks for reply.
Sorry for my lack of expertise in networks but I'm not sure to what exact solution you are referring from this article or where to start actually. Can you please provide more detailed steps?
Is there any simple and out-of-the-box solution we can use without having to do lots of network configurations?

This seems like a common scenario where you would want your Business Central solution available across network and not just locally. We also considered installing NAV 2018 / Business Central as a standalone application since we could easily make clients available throughout network but the build we are working on is only available as docker image... So we need a way to make our containers available just the same.

Using static IP - the example is coming from here: https://docs.microsoft.com/en-us/virtualization/windowscontainers/container-networking/advanced
# Example: Create a transparent network using static IP assignment
# A network create command for a transparent container network corresponding to the physical network with IP prefix <IP_ADDRESS>/23
C:\> docker network create -d transparent --subnet=<IP_ADDRESS>/23 --gateway=<IP_ADDRESS> MyTransparentNet
# Run a container attached to MyTransparentNet
C:\> docker run -it --network=MyTransparentNet --ip=<IP_ADDRESS> windowsservercore cmd

And for example here you can find also DHCP enabled network example (it's actually easier to start with but I had some problems when quickly rotating containers with the same names):
https://www.ntweekly.com/2017/03/30/how-to-create-a-transparent-network-with-windows-containers/

With static IP I should:

First, create a transparent network using static IP assignment

docker network create -d transparent --subnet=<IP_ADDRESS>/23 --gateway=<IP_ADDRESS> MyTransparentNet

Second step: Run a container attached to MyTransparentNet - and this is actually my nav container? So I can run command:

$imageName = 'bcinsider.azurecr.io/bcsandbox-master:13.0.23638.0-ca'
$cred = Get-Credential -UserName $env:USERNAME -Message "Enter your Credentials"
$containerName = 'mycontainer'

New-NavContainer -containerName $containerName `
-imageName $imageName `
-accept_eula `
-Credential $cred `
-auth NavUserPassword `
-licensefile "C:\myfolder\mylicense.flf" `
-includecside `
-multitenant `
-restart always `
-enableSymbolLoading `
-doNotExportObjectsToText `
-additionalParameters "-it --network=MyTransparentNet --ip=<IP_ADDRESS>"

Or do I still need to run container from windowsservercore and then run my nav container?
docker run -it --network=MyTransparentNet --ip=<IP_ADDRESS> windowsservercore cmd

I don't use NavContainerHelper but I think you should be fine calling New-NavContainer and passing
-additionalParameters "--network=MyTransparentNet --ip=[free_ip_from_the_subnet]"

I was using a cmdlet that was automatically adjusting DNS registers so I was able to access containers using their hostname.

Hi ivandjordjevic,
We are also trying to setup similar environment at our end.
1st container created through Mrs. Freddy's script which is working fine and accessible outside the host too.
cannot create 2nd container as it keep "Error response from daemon: failed to create endpoint DEMO on network nat: HNS failed with"

Networking with docker containers is very powerful and allows you to do everything really. With NAV containers you can any of the network settings.
Any use of networking must be compatible/integrated with your infrastructure. In Microsoft we cannot use transparent networking as our infrastructure won't allow a new unknown machine access.
Only out-of-the-box solution is NAT - if you need other network settings, you will need to read about networking and docker. Just like setting up NAV from a DVD doesn't really include any network settings on the server.
So, I cannot describe how to setup a second container on a machine that just always works. For that, the infrastructure and settings are simply too different.
I can explain how to setup a second container on an Azure VM (created by http://aka.ms/getnav) as this is a known starting point. I will do that and post the script/necessary steps here and in a blog post.

I fully agree with @freddydk, everything really depends on your needs and there so many combinations that one that fits one company doesn't work for others.
Usually, the best way is to involve your sysadmin/network admin.

I have added a script and sample on how to do this on azure vms here:
https://github.com/Microsoft/nav-docker/issues/254


Scrollbars are grabbable by a pixel!

There should be an option to set the size of the divider area that is draggable; a 'borderGrabWidth' dimension option or something to go with 'borderWidth' perhaps?
A problem that is especially apparent when using scroll bars is that when they have a draggable divider immediately to the right or directly below them, are barely accessible because the area where the cursor changes to divider dragging behaviour covers most of the scroll bar - in Firefox you get about a pixel that you can, with a steady hand, just about grab, and in Chrome you get about 3 pixels or so because the scrollbar is fatter.
Codepen example:
http://codepen.io/wolframhempel/pen/b8d8f23d4354d58f6a53f96a3e83753d

Agreed they can be hard to scroll with the dividers in the way. Normally, I've just used css to adjust the width / position

Yes, ok, so I managed to turn it back into 5px so it doesn't exceed the default divider size with:
.lm_drag_handle {
     width: 5px !important;
     left: 0 !important;
 }

Couldn't get it to see the CSS without the !important flags though, a 'borderGrabWidth' or some such would be best I think.

Thanks, the CSS in previous comment fixed this bug for me.

If you want to make a pull request of this fix, please make sure that you edit LESS source code and not generated CSS one.

Changing the default CSS / LESS would be a simple fix, and resolve the problem, but it would remove the 'feature' discussed in the docs though whereby the grab area is larger than the divider, so that a developer may use a tiny divider and still have the user be able to grab it easily.
Arguably this might be a better default since it it should be unambiguously trouble free and consistent regardless of scroll bar presence or not, where as obviously the current operation is not always trouble free.
However, it is a change that could trip up anyone who decided to implement 1px dividers. It would therefore, very mildly, a backwards incompatible change.
I just looked at this again, and, the current 'master' revision of GL actually breaks the 'borderWidth' dimensions config! I think it was broken before and I didn't notice, but the less generated CSS seems to reduce the size of the grab handle relative to the splitter, which mitigates this (issue 240) problem a bit, but it made me really notice that the drag handle does not change size with 'borderWidth' because suddenly my handle wasn't centred on my splitter any more!
Anyway, the reason I suggested a 'borderGrabWidth' was mainly because it wouldn't have the same backwards compatibility problems.

I created branch 'border-grab-width' and pull request 270 to fix all this. Behaviour in terms of backwards compatibility should be good as anyone with a 1px divider will get a 15px drag handle, and anyone with dividers 15px or above will get a drag handle the size of their divider, and 15px is big enough not to be too painful. Anyone who wants a bigger one can then adjust it upwards.

Closing: we merged (with thanks!) @mwri 's changes. Reopen if you disagree


Custom Scenarion in encryption at rest feature throws AttributeError

Hi,
I wrote a custom Scenario shown below to test aws_emr_security_configuration encryption at rest
`Feature: Resources should use encryption at rest while they are created
In order to improve security
As engineers
We'll enforce encryption at rest
Scenario: AWS EMR Encryption at Rest
Given I have AWS EMR Security Configuration instance defined
When it contains configuration
Then encryption must be enabled`
main.tf
`resource "aws_emr_security_configuration" "foo" {
name = "emrsc_other"
configuration = <<EOF
{
"EncryptionConfiguration": {
"AtRestEncryptionConfiguration": {
"S3EncryptionConfiguration": {
"EncryptionMode": "SSE-S3"
},
"LocalDiskEncryptionConfiguration": {
"EncryptionKeyProviderType": "AwsKms",
"AwsKmsKey": "arn:aws:kms:us-west-2:187416307283:alias/tf_emr_test_key"
}
},
"EnableInTransitEncryption": false,
"EnableAtRestEncryption": true
}
}
EOF
}
when I run my test against the above template with my feature, I get AttributeError: 'str' object has no attribute 'keys' console outputRunning tests.
Feature: Resources should use encryption at rest while they are created  # C:\Training\AWS\Cloud-CustomRules\Terraform-Compliance\rules\encryption_at_rest.feature
In order to improve security
As engineers
We'll enforce encryption at rest
Scenario: AWS EMR Encryption at Rest
    Given I have AWS EMR Security Configuration instance defined
    When it contains configuration
    Then encryption must be enabled
      AttributeError: 'str' object has no attribute 'keys'

1 features (0 passed, 1 failed)
1 scenarios (0 passed, 1 failed)
3 steps (2 passed, 1 failed)
Run<PHONE_NUMBER> finished within a moment
Cleaning up.`
Not sure what is wrong here, i guess may be I am not reading the EnableAtRestEncryption property correctly?
any help is appreciated. Thanks!

Thanks for reporting @vrbcntrl. The problem is due to inline configuration in terraform. Unfortunately, inline configuration is not supported, since it may vary a lot!
This feature might be supported after having #72 and #66.

Thank you @eerkunt for the response. I'll wait for the update from you then :)

This should be fixed in 1.0.0. Please let us know if it doesn't work for you.

Hi @eerkunt ,
I have again tested this use case with 1.0.4, unfortunately it still throws me a different error now
 Scenario: AWS EMR Encryption at Rest         Given I have aws_emr_security_configuration defined         When it contains configuration         Then encryption must be enabled           KeyError: 'type'
Please find my tf template, out file and the results in the attachments.
In this case also, I have 2 encryption properties that I want to test, but I suppose I cant use the above scenario, so I have also tried the scenario outline just like I have tried in #93 but that also throws me error...
aws_emr_security_configuration.zip

Thanks for trying it again. #101 introduces a fix and enhancement on these kind of conditions. Now we support heredoc json strings. Please have a try with 1.0.5 and let me know if it doesn't work for you ?

Thanks @eerkunt  for fixing the issue. The new version 1.0.5 works fine if I wanted to test a single encryption property i.e 'aws_emr_security_configuration': 'EnableAtRestEncryption' because this is configure in init.py , however I have 2 encryption properties in the same json as shown below
"EnableInTransitEncryption": false, "EnableAtRestEncryption": false
so I tried to test them using the same technique used for #93 , however it didn't work :(
` Scenario Ouline: AWS EMR Encryption at Rest
Given I have aws_emr_security_configuration defined
When it contains configuration
Then it must contain
Examples:
| encryption |
| EnableInTransitEncryption |
| EnableAtRestEncryption |`
I got the below error:
`Error Oracle says:
There is no step defintion for 'Then it must contain '.
All steps should be declared in a module located in c:\users\vrb.cntrl\appdata\local\programs\python\python37\lib\site-packages\terraform_compliance\steps.
For example you could do:
@step(r"Then it must contain ")
def my_step(step):
raise NotImplementedError("This step is not implemented yet")`
so, if you could let me know how to test the 2 or 3 related properties in the same resource, that would be really helpful...thanks in advance!

There are multiple problems here.

Your BDD test is wrong, there are both some typos and it s not structured correctly. I think you are trying to achieve this ;

  Scenario Outline: AWS EMR Encryption at Rest and In-Flight
    Given I have aws_emr_security_configuration defined
    When it contains configuration
    Then it must contain <encryption>
    Then its value must match the "<value>" regex

    Examples:
      | encryption                | value |
      | EnableInTransitEncryption | true  |
      | EnableAtRestEncryption    | true  |

There was a case where Boolean values were not checked on Then its value must match the "<value>" regex step. It is not fixed on #103.

Could you please try with 1.0.6 and the BDD test above together ?

Thank you so much @eerkunt .
I have tested the above BDD with v 1.0.7 and both positive and negatives tests are working as expected.
I also tested the use case mentioned in #93 and it is also working fine.
Thanks again!


Arsstb 370

This will prevent ui tests from breaking (hopefully) when LOA change is merged in main

wrong branch!


Circuit Breaker using Hystrix-go

Hystrix-go is a library by NETFLIX. This library has been used to implement the circuit breaker for dmux

reopen


Add support for Power (ppc64le) Architecture

Given that GO is fully supported on Power, it would be great to include the official support for it.
For building and tests you can use Travis (recently added support for Power), using os: linux-ppc64le. Also, you can request a free Power VM at http://openpower.ic.unicamp.br/minicloud/.

Built from this branch, built on x86 and tested on a ppc64le system. Everything seemed to work fine.
https://github.com/ceseo/dep/tree/pr1867


New fields added to People post

@gregoryspragginsjr we have a few new fields in the People post type that need frontend support. Those fields are:

fax
mailing_address
office_hours

Fax and mailing address will need icons and need to be located in the section with phone and building.
Office hours can live above the "Current Courses" section.

@gregoryspragginsjr I've done this. Closing.


Execution Observability with OpenTelemetry

Summary
OpenTelemetry is a CNCF standard for cloud observability. It instruments HTTP calls, DB calls, etc as traces. Together with OpenLLMetry, which provides instrumentations for LLM providers like OpenA, vector DBs like Chroma and LLamaIndex - it gives a great way to view and debug agent exceutions.
It's natively supported by all major observability platforms like Datadog, Honeycomb, New Relic and others, so if we install the SDK in the repo it allows users to see and debug executions of OpenDevin.
Motivation
Debug and see executions of OpenDevin, including LLM calls, vector DB calls, HTTP calls, and others.
Technical Design
We can offer an option to initialize an SDK, like the one provided by OpenLLMetry so if the user enables it they can view execution traces.
Alternatives to Consider
None
Additional context

We already have https://github.com/OpenDevin/OpenDevin/pull/1944 that tracks costs. We could incrementally build more, and eventually, refactor them to be OpenTelemetry compatible.

Just curious, is there any other popular open source solution like OpenLLMetry?

@li-boxuan happy to contribute this if you give me the green light :)
@yufansong OpenLLMetry is the most popular one and is fully compliant with OpenTelemetry standards (I know that, because I'm defining them these days :) )

@nirga that would be awesome, please go ahead!

@nirga that would be awesome, please go ahead!
Update: sorry I didn’t realize it’s a commercial product. Would you mind sharing some plans first before you start any dev work?

I test it locally, it seems require me to register and get the api_key before using sdk. If in this case, I think maybe not suitable to intergate into opendevin. Or at least I need to discuss with other core maintainers.

Hey @li-boxuan and @yufansong! it's not a commercial product. OpenLLMetry is completely open source Apache 2.0 project. The data it outputs can be connected to 20 different platforms for visualization, some of them oss (like signoz or hyperdx) and some not. Our platform, traceloop, is one of these.
See this list of integrations - https://www.traceloop.com/docs/openllmetry/integrations/introduction
The error you've got @yufansong is basically asking you to connect it to some destination to see the traces.

@li-boxuan what are your thoughts on this?

Hey @li-boxuan and @yufansong! it's not a commercial product. OpenLLMetry is completely open source Apache 2.0 project. The data it outputs can be connected to 20 different platforms for visualization, some of them oss (like signoz or hyperdx) and some not. Our platform, traceloop, is one of these.
See this list of integrations - https://www.traceloop.com/docs/openllmetry/integrations/introduction
The error you got @yufansong is basically asking you to connect it to some destination to see the traces.

Ah, sorry for the late reply. My previous problem is, when I run Traceloop.init(disable_batch=True), it will throw the following error:
Error: Missing Traceloop API key, go to https://app.traceloop.com/settings/api-keys to create one
Set the TRACELOOP_API_KEY environment variable to the key

Warning: Traceloop not initialized, make sure you call Traceloop.init()

Do you mean actually I don't need to register and can directly following like this way to avoid this initialization error?

Update: I also find these two tools for LLM Observability, but it seems both of them are commercial products.
https://www.helicone.ai/
https://www.langchain.com/

@yufansong exactly! There are 15 different platforms you can connect to, some of them (HyperDX and SigNoz) are completely OSS

I'm trying Phoenix.
With LiteLLM and Vertex, I have this issue:
https://github.com/Arize-ai/phoenix/issues/5813


Widen installable versions of 99designs/phumbor

This PR widens the installable versions of 99designs/phumbor.
There is a newer v1.2.0 release that can't currently be installed alongside this bundle.

Done. Thank you.


Support Page: Link Button is not active (blue) when on page

Reproduce:

Go to https://dao.hypha.earth/demoxdaox/support
Check button colour

Expected behaviour:
The button should be active colour (blue) because I am on the page
Experienced behaviour:
The button is white (not active)

@leonieherma1 at the moment there is no button here, this is plain text, could you give me a link to which the user should go?

@evgenibir I mean the "question mark icon" - it should be blue when I am on the page.

Same behavior as the "gear icon" when I am on the configuration page:


Update sql project file move to use DacFx api

This switches moveFile() used for drag and drop to use the DacFx api exposed in STS to move the file on disk and in the .sqlproj. Also moves the move part to a helper function so both rename and moveFile can call it.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

16 of 19 (84.21%)  changed or added relevant lines in 2 files are covered.
7 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.01%) to 41.978%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/sql-database-projects/src/controllers/projectController.ts
15
18
83.33%

Files with Coverage Reduction
New Missed Lines
%

extensions/sql-database-projects/src/common/httpClient.ts
1
66.18%

extensions/sql-database-projects/src/dialogs/publishDatabaseDialog.ts
2
52.88%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
28766

Relevant Lines:
63636

💛  - Coveralls


Minor refactoring to support filtering exports

Problem
#1578
It is not possible to modify the Admin UI to include a custom field for export.  This is possible for import but not export.
The bug is due to the fact that the form is removed from kwargs hence cannot be used in custom Resources to filter the queryset.
Solution

changed the pop() call so that the form remains in kwargs
Added a new filter_export() hook to make it a lot easier to dynamically filter querysets for export
Updated the example app to demonstrate using 'E Book'

Acceptance Criteria

Ran integration tests
Manual testing
Updated Documentation

Other refactoring

Split the test fixtures into 'book', 'categories' and 'author'
Split import / export tests in test_admin_integration.py for better readability

Coverage: 100.0%. Remained the same when pulling 406787893bcee0e2de6bbb89ade2a157d4ead57e on matthewhegarty:issue-1578-filtering-export-from-form into 0ff664cf646f310a9a7f76b1b7b10af5d8bd5b32 on django-import-export:main.

Hi, can you please make a release with this fix? I can't customize the export form without this.

yes, my plan is to release it this week

Latest release (3.3.0) now available https://pypi.org/project/django-import-export/


Provide information regarding revenues of Estonian companies

Related Task:

https://github.com/buerokratt/Common-Services/issues/41

Had a meeting with Mr.Rainer and we checked the current flow, will be merging the current state


Provided a fix for cron not working out of the box.

As the reporter of #345 stated, cron does not work out of the box in the 0.9.19 baseimage.
This PR incorporates the fix. I have tested it and it seem to work with no side-effects.

I started making an updated fork of this project, containing Ubuntu 16.04 fixes, including this one. Have a look here:
https://github.com/hyperknot/baseimage16


Add Pug files support

Would be great to add support for Pug .pug files.
There is already a prettier-pug plugin which works great. But it doesn't do anything with Tailwind

Seems to be possible to be done easily as the official prettier-pug plugin has a parser
import prettierParserPug from '@prettier/plugin-pug'

prettierParserPug.parsers.pug.parse

But I didn't manage to get it working with this extension :( the pug code never gets handled by this extension for some reason.


Add client implementation for Numbers API, support 7.1

Notes
#108 should be merged first, this PR currently includes commits from it.
Motivation
Add resources related to the Numbers API.
Changes

Add Numbers API-related objects
Add Numbers API-related resources
Add related integration tests
Broaden support back to 7.1

@felipecoppola Addressed comments and added support back to 7.1 due to an internal request.
Can you give this one a final look? Thanks!


Add support for recaptcha

It would be great if you could configure which captcha solution was in.
That way I could easily drop in ones such as: https://github.com/ambethia/recaptcha

Devise uses recaptcha support as default captcha. It would be nice if this could work for this gem as wel.
See https://github.com/plataformatec/devise/wiki/How-To:-Use-Recaptcha-with-Devise

See fa8f158f4661c0d7fb09c2a23d781c06894e8cee


Namespaces within a Tenant cannot be selected in NetworkPolicies

Hi everyone!
I'm trying the brand new v0.0.5 release and found out that, due to the issue #200, there's no way to reference (using namespaceSelector) a particular Namespace within a tenant in a NetworkPolicy.
As stated in the mentioned issue and in the official documentation, targeting of namespaces by name in NetworkPolicies is not possible and, with the added functionality, there's no way to tag each Namespace separately.
To be able to reference each Namespace in a NetworkPolicy, a possible solution would be to automatically tag each Namespace with a label with its own name.
Something like this:
apiVersion: v1
kind: Namespace
metadata:
  labels:
    capsule.clastix.io/tenant: mytenant
    name: mytenant-ns
  name: mytenant-ns

Looks good to me: @bsctl @MaxFedotov any remark on this?
I also think this can be a really good first issue, any contribution is highly appreciated! ❤

We are already doing this in order to support Calico GlobalNetworkPolicies.
But we are using namespacesMetadata.additionalLabels in Tenant spec:
  namespacesMetadata:
    additionalLabels:
      capsule.clastix.io/tenant: {tenant_name}

I need to identify each Namespace within a tenant separately, not all the Namespaces belonging to a particular tenant.

I think the use-case here is different: we don't have to replicate the tenant name but, rather, the Namespace name. Labeling a Namespace is either disabled due to the reported bug/enhancement, we have just to rely on the namespacesMetadata.additionalLabels but these are going to be applied across all the Tenant Namespace resources.
I guess the need for this feature is to take advantage of the Capsule NetworkPolicy resources reconciliation: these ones cannot either be deleted by the Tenant Owner and ensure to select the single Namespace and isolate itself rather than at Tenant level.
Please, @stg-0, can you confirm this or, rather, provide a sort of user-story for this enhancement?

That's exactly it. A simple use-case would be to have layer segregation by Namespace within a tenant. Off the top of my head, a possible scenario would be a tenant with a Namespace dedicated to my DBs and a second Namespace with some consumers, then I want to be able to only access those DBs (and no other Namespaces) from the consumers. I hope that clarifies a bit the need.


add xfsprogs to ros

Feature request to add XFS binaries ros.

Here is the test ISO: http://releases.rancher.com/os/test/rancheros-xfs.iso

Will this feature rolls out on next release 1.5 ? In that case
Will my query will get resolved which I update on Slack
On RancherOS I want to Additional Mount Disk
for Example I created persistent disk and would like to attache to RancherOS I know I could
mounts:

["/dev/vdb", "/mnt/s", "ext4", ""]   ----> Will it support other than ext4 I want to disk to be mount as xfs

@sjkeerthi That should work, you can try v1.5.0-rc1.

Version - rancher/os:v1.5.0-rc1 11/30
Verified fixed


Ipv6

Add ipv6 and AAAA records support
This will probably need a version bump as the interface is slightly modified
It now takes IpAddr and if_addr::Interface types in most places instead of the V4 ones
host_ipv4 argument renamed to ip
AsIpv4Addrs renamed to AsIpAddrs

[x] dns_parser: support "AAAA" address record.
[x] responder: support register ServiceInfo with IPv4 only or IPv6 only, as well as both IPv4 and IPv6 in one register.
[x] responder: recv and process queries on IPv6 interfaces if IPv6 address is configured.
[x] responder: send response on IPv6 interface if IPv6 address is configured.
[x] querier: send query on IPv6 interface if IPv6 address is configured.
[x] querier: recv and handle answers on IPv6 interfaces if IPv6 address is configured.
[x] Default setting: both IPv4 and IPv6 is enabled (if addr is configured). Provides API to disable IPv6 (or IPv4). i.e. allow the ServiceDaemon to be IPv6 only or IPv4 only.
[x] Tests

The reviewer should look at the diff per commit as it is easier to read
Fix #61

I'm not sure why it doesn't work on windows :/

I'm not sure why it doesn't work on windows :/

I tried it out on Windows 11.  It seems like IPv6 IN6ADDR_ANY does not like set_scope_id on Windows.  I tried the following diff, and the test passed on Windows:
diff --git a/src/service_daemon.rs b/src/service_daemon.rs
index 08053e9..23c1e2e 100644
--- a/src/service_daemon.rs
+++ b/src/service_daemon.rs
@@ -642,9 +642,9 @@ fn new_socket_bind(intf: &Interface) -> Result<Socket> {
             Ok(sock)
         }
         IpAddr::V6(ip) => {
-            let mut addr =
+            let addr =
                 SocketAddrV6::new(Ipv6Addr::new(0, 0, 0, 0, 0, 0, 0, 0), MDNS_PORT, 0, 0);
-            addr.set_scope_id(intf.index.unwrap_or(0));
+            // addr.set_scope_id(intf.index.unwrap_or(0));
             let sock = new_socket(addr.into(), true)?;

             // Join mDNS group to receive packets.

Also I found another test failure on macOS Monterey. It seems depending on what kind of interfaces exist on the system. But anyway, the issue is that in my macOS system, there are two IPv6 interfaces with different names, but the same IPv6 address.
I applied the following diff in the test for it to pass:
diff --git a/tests/mdns_test.rs b/tests/mdns_test.rs
index 7350c4a..9d825e1 100644
--- a/tests/mdns_test.rs
+++ b/tests/mdns_test.rs
@@ -2,7 +2,7 @@ use if_addrs::{IfAddr, Interface};
 use mdns_sd::{
     DaemonEvent, IntoTxtProperties, ServiceDaemon, ServiceEvent, ServiceInfo, UnregisterStatus,
 };
-use std::collections::HashMap;
+use std::collections::{HashMap, HashSet};
 use std::net::{IpAddr, Ipv4Addr, Ipv6Addr};
 use std::sync::{Arc, Mutex};
 use std::thread::sleep;
@@ -22,7 +22,8 @@ fn integration_success() {
         .unwrap();
     let instance_name = now.as_micros().to_string(); // Create a unique name.

-    let my_ifaddrs: Vec<_> = my_ip_interfaces().iter().map(|intf| intf.ip()).collect();
+    let ifaddrs_set: HashSet<_> = my_ip_interfaces().iter().map(|intf| intf.ip()).collect();
+    let my_ifaddrs: Vec<_> = ifaddrs_set.into_iter().collect();
     let my_addrs_count = my_ifaddrs.len();
     println!("My IP addr(s): {:?}", &my_ifaddrs);

(Note that ServiceDaemon already uses a set when resolving, so didn't have two entries with the same IPv6 addrs).

Nice catch!
I applied the MacOs patch.
For the scope_id, it is extremely confusing, In ipv6 multicast, the kernel needs to know on which interface the packet should be sent because all ip6 interface are required to get a link-local addr (fe80::) and the user might not want to broadcast on all ifaces.
It the end it is similar to the -I option in ping or other tools for ipv4.
Windows documentation seems to agree with that https://learn.microsoft.com/en-us/windows/win32/wininet/ip-version-6-support
It is confusing because for multicast addr, part of the address is called scop in the RFC https://datatracker.ietf.org/doc/html/rfc4291#section-2.7 to give a hint as how far the multicast should go.
Maybe windows only accept a scope_id when the socketaddr is a multicast address, because when it is not a multicast windows already knows on which interface it should send packets?
In conclusion I'll applied your patch, the socketaddr iface_id is already set twice right after with
sock.join_multicast_v6(&GROUP_ADDR_V6, intf.index.unwrap_or(0))
sock.set_multicast_if_v6(intf.index.unwrap_or(0))

Default setting: both IPv4 and IPv6 is enabled (if addr is configured). Provides API to disable IPv6 (or IPv4). i.e. allow the ServiceDaemon to be IPv6 only or IPv4 only.

I missed to ask about this point in your description. How is ServiceDaemon allowed to be IPv6 only or IPv4 only? (I couldn't see it now).

It works if the interface has no IPv6 or no IPv4 configured, the ServiceDaemon won't try to broadcast on an un-configured ip version.

It works if the interface has no IPv6 or no IPv4 configured, the ServiceDaemon won't try to broadcast on an un-configured ip version.

That's true.  I was wondering about the cases when the user wanted to explicitly disable IPv4 or IPv6 even if they are configured on the interface. It would be helpful for one of my use cases. I'll take a look.

There are two levels of this kind of control:

at the ServiceDaemon level, effects everything in the daemon. There is an open PR to limit the daemon to a single interface, which is a concrete related case. Setting IPv4 or IPv6 only is another case. We might consider a more generic way to support these.

at the individual service level.  For example it probably be helpful to add a method to ServiceInfo to allow the user register a service only for IPv4 or IPv6, without affecting the whole daemon / other services.  (I'm thinking of implement this first.)

For browsing,  the current API is simple and does not allow much options.  But at least a user can filter out the resolved IPv4 or IPv6 addresses based their needs.
Any suggestions or comments based on your experience involving both IPv4 and IPv6?

Actually I've been using mdns-sd on an Ipv6 only host, I agree that the user should be able to choose the interface and ip versions, but the most important thing it to have a good default behavior.


Mini player video shows up in live stream view after app backgrounded in unrelated live stream view

Play a video in the mini player
Enter another video that is also marked as a live stream (that ended two days ago in this case, and furthermore wasn't playable — unresponsive play button over a black screen)
Background the app
Foreground the app. The video that played in the mini player is now shown in the live view above a completely unrelated chat log.

You'll also find that tapping the mini player after this will take you to the live view, again associating the video with an unrelated chat log.

Thank you for the report!


Inclusion of catalog files inside other catalog files

brooklyn.catalog:
  brooklyn.libraries: http://artifactory/my.jar
  items:
  - classpath://yaml-inside-my-jar.bom
  - id: sibling-item
    version: 0.1.0
    item:
      type: BasicEntity

Docs in https://github.com/apache/brooklyn-docs/pull/49

TODO: Support relative to the current context URLs.

This all looks good to me.  I've cloned this PR and done various tests of my own, similar to the ones in the integration tests, and it all seems to work.  👍

Looks good to me too - cloned, built and tested with consul blueprint (from Consul PR).

excellent.  merging.
re the ability to reference items relative to the BOM's location:  an easy way to do this is by uploading a ZIP containing all of it and we automatically make that a library.  harder will be to know the remote URL and try to resolve things relative to it.
i tend to think it's the former.  do you agree?

Thanks @ahgittin. Yes I meant the former in the TODO above.

Correction, I meant the latter - resolving URLs relative to the current source.


http://www.eolang.org/ redirects endlessly

When I try to open http://www.eolang.org/ I just get redirected endlessly.

@yegor256/z please, pay attention to this issue

@serranya/z this project will fix the problem faster if you donate a few dollars to it; just click here and pay via Stripe, it's very fast, convenient and appreciated; thanks a lot!

@Serranya fixed, thanks!

Job gh:yegor256/eo#152 is not assigned, can't get performer

This job is not in scope


Add a force option to allow deletion without user confirmation

Hi,
I'm pretty confident about the assets I'm not using and I'd like to run php please assets:clear with Laravel Scheduler, or as part as a deployment script.
It would be great to have the ability to execute the command without any interaction.
Thanks.

Done :) You can have the command run without asking for confirmation by adding the --force flag at the end.

That's great news! Thank you!


cmd+d to do multiselect / multicursor and then apply neovim commands to each occurrence does not work

When I press viw to select a word, then press cmd+d to select the other instances, it first activates Match Case and Match Whole Word settings, then starts highlighting the other occurrences.
However, when I then press something like ciw to change the words while all the occurrences are highlighted, only the first one changes.

How do I begin selecting occurrences immediately, without first activating those settings?
How do I apply the neovim keys to each currently selected occurrence instead of just the first one?

Multi cursor not supported in visual mode

Multi cursor not supported in visual mode

Normal vscode vim with jumpy2 plugins works wonders

Can provide a function that encapsulates editor.action.addSelectionToNextFindMatch to support this feature

That would be nice, if we wrapped all the muticursor commands in normal mode like "select all word under cursor" and all the other ones, that would nicely transition you into insert mode.

It's a little janky but seems to work. This is what I came up with:
  vim.keymap.set("v", "<D-d>", function()
    local sel_start = vim.fn.getpos("v")
    local sel_end = vim.fn.getpos(".")
    vim.api.nvim_feedkeys(vim.api.nvim_replace_termcodes("<esc>", true, false, true), "v", false)
    vim.defer_fn(function()
      vim.cmd.startinsert()
      vscode.notify_range_pos("editor.action.addSelectionToNextFindMatch", sel_start[2], sel_end[2], sel_start[3],
        sel_end[3], true)
    end, 20)
  end, opts)

To work it needs to forward the cmd+d keystroke from vscode:
  {
    "command": "vscode-neovim.send",
    "key": "cmd+d",
    "when": "editorTextFocus && neovim.mode != insert",
    "args": "<D-d>"
  },
  {
    "key": "cmd+d",
    "command": "editor.action.addSelectionToNextFindMatch",
    "when": "editorFocus && neovim.mode != 'visual'"
  },
  {
    "key": "cmd+d",
    "command": "-editor.action.addSelectionToNextFindMatch",
    "when": "editorFocus"
  }

First keybinding sends cmd+d to vscode-neovim if in visual mode. second keybinding allows normal vscodes cmd+d behaviour once in insert mode. The thrid disables the default vscode keybinding.
It seems to work very well for me. Only undo functionality is a little wierd. If I select and edit using this method and then hit undo it will undo only one typed character at a time.
If however i go into insert mode normally select the word/phrase using shift+arrow keys (vscode selection) and then start hitting cmd+d to get my selection, Undo will undo everything i typed at once (what I would expect)

After the next release, plugin vscode-neovim/vscode-multi-cursor.nvim will be ready for use. Not sure if it meets everyone’s needs, but it should make multi-cursor easier to use.


[COMMUNITY] Qingchao Shen -> Reviewer

Please join us to welcome @jikechao as a new Reviewer in TVM. He has been continuously improving frontend support and fixing bugs.

Commits History
Code Review

@Hzfengsy @quic-sanirudh Thanks a lot!


pow.ffs.setDefaultStorageConfig(customConfig)

my step below
const customConfig = {
      "hot": {
        "enabled": true,
        "allowUnfreeze": false,
        "unfreezeMaxPrice": 0,
        "ipfs": {
          "addTimeout": 30
        },
      },
      "cold": {
        "enabled": true,
        "filecoin": {
          "repFactor": 5,
          "dealMinDuration": 1036800,
          "excludedMiners": null,
          "trustedMiners": [
            "t016304",
            "t016304",
            "t016305",
            "t016306",
            "t016309"
          ],
          "countryCodes": null,
          "renew": {
            "enabled": false,
            "threshold": 0
          },
          "addr": "t3xao2uyz37six5dkyeuaq7n5krgzoo5tc3khtqmfjcelxt36mjfdkuwnm7tnpjgh6fysujekrpg4qnw5tn3cq",
          "maxPrice": 0,
          "fastRetrieval": false
        }
      },
      "repairable": false
    }
    const pow = createPow({ host })
    const { token } = await pow.ffs.create()
    pow.setToken(token)
    pow.ffs.setDefaultStorageConfig(customConfig)
source code -> util.js -> line32 -> hot.setEnabled(obj.enabled)
error below
Error: default cid config is invalid: hot-ipfs config is invalid: invalid ipfs config: add timeout should be greater than 0 seconds, got 0
Error: default cid config is invalid: hot-ipfs config is invalid: invalid ipfs config: add timeout should be greater than 0 seconds, got 0
i wanna know the addTimeout is 30, but why error catch?

Thanks for reporting. Will take a closer look and report back here.

@kingHawkWin can you tell me what version of @textile/powergate-client you are using and what Powergate host you are connecting to? Thanks.

Also, just looking at the property names on your customConfig object... some of the names are not correct. How are you determining how that config object should be shaped and what the property names are? If you're not using Typescript, your best bet is to call await pow.ffs.defaultStorageConfig(), and inspect the returned object's defaultStorageConfig property to see exactly what the StorageConfig object looks like. You can event just modify it then pass it into await pow.ffs.setDefaultStorageConfig(customConfig).

Ok, I have a good final answer for you. The problem specifically with the addTimeout is because the Powergate server you're connected to has been updated and the @textile/powergate-client hasn't been updated to account for a breaking gRPC change. That change can be seen here where the IpfsConfig index was changed. This is a bug and @textile/powergate-client needs to be updated to fix it. I'll do that now and report back here when the release is out.
But also, the things I mentioned in the previous comment will be problems for you as well. Be sure to address those things as well.

Ok 2.0.0 is released. Please update.

Great~ Thanks for your fast supports!
I'm his friend and will tell him what he should to check, :)

Hi asutula， Thanks for your helps to sovled this issue and this really helpful to me.
I totally follow your instrution to call await pow.ffs.defaultStorageConfig() and get the default config with correct names. But the banlance of default addr is 0, it cannot uplaod anything , so I update with my addr with some banlance values, it may work to pay for miner, and then the new error happens as below:
please see https://github.com/textileio/js-powergate-client/issues/135


chore: bump jest and use mjs

TODO:

wait for the ts-jest stable release

I've checked the tests and couldn't find any use of  jest-environment-jsdom or jest-jasmine2 but as they are present in resolution, I've added them. Should I remove them?
Edit: Blocked by https://github.com/kulshekhar/ts-jest/issues/3453

Yes, remove them from resolutions. Skip-dependency is to do just that, force skip a dependency. But that is no longer necessary.

And you can also remove the explicit dependency on jest-circus and the testRunner: 'jest-circus/runner', line from the jest config.


matplotlib error due to missing environment variables

Hello,
first, I'm not sure if this is the correct place to ask for this. I'm trying to use matplotlib with CodeRunner in Moodle connecting to a Jobe server hosted in Digital Ocean. Simple scripts with python3 work well but if I try to use matplotlib I get this error:
SCRIPT:
import matplotlib.pyplot as plt
import numpy as np

# Aquí el estudiante escribe su código para generar el gráfico
x = np.arange(5)
y = np.random.randint(1, 10, size=5)
plt.bar(x, y)
plt.xlabel('Eje X')
plt.ylabel('Eje Y')
plt.title('Gráfico de barras')
plt.show()

OUTPUT:
Error in question
Bad output from grader:
Run result: Error
Output:
{"fraction": 1.0, "epiloguehtml": "<img src=\"data:image/png;base64,iVBORw0KGgoAAAANSU ..... \" alt=\"Gr\u00e1fico de barras\">"}

Matplotlib created a temporary config/cache directory at /tmp/matplotlib-b49jl89b because the default path (/home/jobe00/.cache/matplotlib) is not a writable directory; it is highly recommended to set the MPLCONFIGDIR environment variable to a writable directory, in particular to speed up the import of Matplotlib and to better support multiprocessing.
. Your program execution may have aborted (e.g. a timeout or memory limit exceeded).

I've tryed to configure the environment variables for "jobe" user in the jobe server but the user used to execute the job in the server does not get these values.
Username: jobe00
matplotlib uses /home/jobe00/.config/matplotlib
MPLCONFIGDIR None
MATPLOTLIBRC None
XDG_CONFIG_HOME None
MATPLOTLIBDATA None
How can I resolve that?
Thanks!

Please see https://github.com/trampgeek/jobeinabox/issues/13
The issue should, I admit, be fixed in Jobe itself. And I thought there had been a pull request from someone which did that, but it's clearly not there right now. I'll look into it.


Made some changes for python 3 compability:

These are changes to improve async for python3 compability.

print needs brackets now
Exception as exception instead of Exception, exception
basestring does not exist anymore

This is probably not all.

I'm going to close this because I don't think we can get Python 3 compatibility in this way because we have to support Python 2.
I'd be happy to take changes that make the 2 to 3 conversion clean though.


chore(es/minifier): add missing #[doc(hidden)]

Description:

BREAKING CHANGE:

Related issue (if exists):

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

Can you sign the CLA?

@kdy1 Sorry, I just went to dinner and wasn't at the computer. I have signed the CLA now.


github-pages 1.4.1 broken

Hello,
It seems github-pages plugin is broken in 1.4.2. I reverted to 1.3.2 which seems to work.
The problem is that I get Caused by: org.eclipse.jgit.api.errors.RefAlreadyExistsException: already exists when trying to push a website.
My config:
plugins {
  id 'org.ajoberstar.github-pages' version '1.4.1'
}
githubPages {
  repoUri = 'git@github.com:ieugen/ieugen.ro.git'
  pages {
    from 'output'
  }
}

The error:
Caused by: org.ajoberstar.grgit.exception.GrgitException: Problem checking out.
	at org.ajoberstar.grgit.operation.CheckoutOp.call(CheckoutOp.groovy:115)
	at org.ajoberstar.grgit.operation.CheckoutOp.call(CheckoutOp.groovy)
	at java_util_concurrent_Callable$call.call(Unknown Source)
	at java_util_concurrent_Callable$call.call(Unknown Source)
	at org.ajoberstar.grgit.util.OpSyntaxUtil.tryOp(OpSyntaxUtil.groovy:45)
	at org.ajoberstar.grgit.Grgit.methodMissing(Grgit.groovy:188)
	at org.ajoberstar.gradle.git.ghpages.GithubPagesPlugin$_createPrepareTask_closure1_closure4.doCall(GithubPagesPlugin.groovy:69)
	at org.gradle.api.internal.AbstractTask$ClosureTaskAction.execute(AbstractTask.java:554)
	at org.gradle.api.internal.AbstractTask$ClosureTaskAction.execute(AbstractTask.java:535)
	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeAction(ExecuteActionsTaskExecuter.java:80)
	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeActions(ExecuteActionsTaskExecuter.java:61)
	... 68 more
Caused by: org.eclipse.jgit.api.errors.RefAlreadyExistsException: already exists
	at org.eclipse.jgit.api.CheckoutCommand.processOptions(CheckoutCommand.java:497)
	at org.eclipse.jgit.api.CheckoutCommand.call(CheckoutCommand.java:202)
	at org.eclipse.jgit.api.CheckoutCommand.call(CheckoutCommand.java:126)
	at java_util_concurrent_Callable$call$0.call(Unknown Source)
	at org.ajoberstar.grgit.operation.CheckoutOp.call(CheckoutOp.groovy:112)
	... 78 more

Can you give 1.4.2-rc.1 a try?

Still does not work:
Caused by: groovy.lang.MissingPropertyException: Could not find property 'repo' on task ':prepareGhPages'.
	at org.gradle.api.internal.AbstractDynamicObject.propertyMissingException(AbstractDynamicObject.java:43)
	at org.gradle.api.internal.AbstractDynamicObject.getProperty(AbstractDynamicObject.java:35)
	at org.gradle.api.internal.CompositeDynamicObject.getProperty(CompositeDynamicObject.java:97)
	at org.gradle.api.tasks.Copy_Decorated.getProperty(Unknown Source)
	at org.ajoberstar.gradle.git.ghpages.GithubPagesPlugin$_createPublishTask_closure2_closure7.doCall(GithubPagesPlugin.groovy:91)
	at org.gradle.api.internal.AbstractTask$ClosureTaskAction.execute(AbstractTask.java:554)
	at org.gradle.api.internal.AbstractTask$ClosureTaskAction.execute(AbstractTask.java:535)
	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeAction(ExecuteActionsTaskExecuter.java:80)
	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeActions(ExecuteActionsTaskExecuter.java:61)
	... 68 more

Looks ok. It seemed to be an issue with my setup. Maybe gradle build daemon.

Cool, I'll get that released tonight.


[Feature Request] take input from stdin

Being able to set the cookie from stdin would make automation much easier

https://stackoverflow.com/questions/22744443/check-if-there-is-something-to-read-on-stdin-in-golang/26567513#26567513
should be easy enough

+1 this please ian


I don't know- always bad

a don't know, what's wrong... i tried to st up Perm.Ex, but when i restart server, it's always broken.. ;_;
perm.ex.txt

You used the tab key in a lot of the lines.  Don't do that.  ;)
Follow this example for indentation, settings, etc.  And be sure every group has an inheritance; if they are not inheriting, give them line 19.

thx

You're welcome. :)  Let me know if you need any more help.  (You can reply to this ticket, even though it's closed.)


Incorrect terminology: URL ≠ Domain

Description of Problem
When visiting the Cookie-Whitelist section of the add-on preferences, all the whitelist entries are described as URLs, which they certainly are not (URLs must have at least a scheme and a hostname value separated by :// or just :). The correct word you're looking for is Domain name or, shorter, Domain. :slightly_smiling_face:
This is particularly confusing if you, at the same time, open the Firefox cookie settings "dialog-thing" which contains actual URLs/addresses (with HTTP or HTTPS scheme).
Screenshot in which the bug can be seen

Steps for anyone to reproduce the bug

Open the extension's settings
Open the Cookie-Whitelist section

Your settings

OS/version: Debian 9
Browser/version: Firefox 54.0
Cookie AutoDelete version: 1.3.1

The German translation is out of date since 1.2.0 (still waiting on @Strayer but feel free to send in a PR [See #57] ). So some instances of that have been modified, but some still exists, so I'll probably edit the locales for the next cycle 1.5.0.

The "Add URL" button still calls them URL, even in the updated translation. Decided to rather stay true to the original English translation file. (See PR #63)
I agree with the issue though, the wording is confusing.

Well as long as there's an issue, I'll be reminded every time I come here. 😄

Closing as there are no more references to URL at least in the en locale.


Different number of masked positions  of  samples in  the same batch  for RoBERTa

❓ Questions and Help
In pretraining of Roberta, what is the effect of adding a random number to the number of masked positions( 0.15*seq_len +np.random.rand()) instead of fixed length for all samples in a batch?
e.g.
for a batch of 8 samples, the seq_len after padding is 339
masked_tokens.size:  torch.Size([8, 339])
masked_token_len tensor([50, 50, 51, 51, 51, 51, 51, 51], device='cuda:0')
but the number of masked tokens is not always 50 or 51
            num_mask = int(                 # add a random number for probabilistic rounding                 self.mask_prob * sz + np.random.rand()             )

Please provide the full command you used to launch your training job.  What is your mask_prob?  Are you using --mask-whole-words?

mask_prob is 0.15,   do not use mask-whole-words. will mask-whole-words leads to the uneven number of mask_tokens even  without random number for probalilistic rounding
`
TOTAL_UPDATES=125000    # Total number of training steps
WARMUP_UPDATES=10000    # Warmup the learning rate over this many updates
PEAK_LR=0.0005          # Peak learning rate, adjust as needed
TOKENS_PER_SAMPLE=512   # Max sequence length
MAX_POSITIONS=512       # Num. positional embeddings (usually same as above)
MAX_SENTENCES=8        # Number of sequences per batch (batch size)
UPDATE_FREQ=32          # Increase the batch size 16x
DATA_DIR=data-bin/wikitext-103
mkdir checkpoint
cur_save=roberta_wiki103
python3 train.py --fp16 $DATA_DIR
--task masked_lm --criterion masked_lm
--arch roberta_base --sample-break-mode complete --tokens-per-sample $TOKENS_PER_SAMPLE
--optimizer adam --adam-betas '(0.9,0.98)' --adam-eps 1e-6 --clip-norm 0.0
--lr-scheduler polynomial_decay --lr $PEAK_LR --warmup-updates $WARMUP_UPDATES --total-num-update $TOTAL_UPDATES
--dropout 0.1 --attention-dropout 0.1 --weight-decay 0.01
--max-sentences $MAX_SENTENCES --update-freq $UPDATE_FREQ  --max-update $TOTAL_UPDATES --ddp-backend=no_c10d
--log-format json --tensorboard-logdir checkpoint/${cur_save}  --save-dir checkpoint/${cur_save} --save-interval 10  2>&1 | tee checkpoint/${cur_save}.txt
`

I doubt it makes a big difference. We're just trying to keep the percentage of masked tokens close to 0.15, but if you do num_mask = int(self.mask_prob * sz) then you'll always be rounding down. For example if sz=50 then you'll always mask 7 tokens, which is 14% of tokens. By adding the random number we'll sometimes pick 7 and sometimes pick 8, which on average will yield 15% masks.

I doubt it makes a big difference. We're just trying to keep the percentage of masked tokens close to 0.15, but if you do num_mask = int(self.mask_prob * sz) then you'll always be rounding down. For example if sz=50 then you'll always mask 7 tokens, which is 14% of tokens. By adding the random number we'll sometimes pick 7 and sometimes pick 8, which on average will yield 15% masks.

so it is equal to  num_mask = int(self.mask_prob*sz +0.5)  ?

No, because then for sz=50 it would always mask 8 tokens (=16%). Here are some details about stochastic rounding: https://en.wikipedia.org/wiki/Rounding#Stochastic_rounding


Revert "Handle API errors to trigger force upgrades of the app "

Reverts Expensify/App#32326
Straight revert

@situchan confirmed this fixes the android crash. https://github.com/Expensify/App/issues/35049
I have been having hard time to build android, moving along as this is unblocking dev enviroments for everyone

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to production by https://github.com/thienlnam in version: 1.4.31-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


docker-compose.yml and run.sh for dev environment

@felipewind,

docker-compose.yml, with two SGBDs and build the Quarkus application;
run.sh, to executes with a single command;
Dockerfile.dev, to compile and run Quarkus application in dev mode

For issue:
https://github.com/felipewind/poc-quarkus-multiple-datasources/issues/3

Thank you very much @eduardomrodrigues
I really appreciate your help!
closes #3


Docker images release to Dockerhub on stable github release

Experimental release - PR into main with -beta post fix on release number.
Stable release - PR into main with no post fix on release number.

docker images
release to docker hub - bear in mind Flux will update on push to latest tag