pip installation failed

I've struggle to install pydbhub via pip.
The installation seem to be finished, but i was't able to import pydbhub, and pydbhub wan't shown in the package list using pip list.
@ Python 3.7.9
>>>pip install pydbhub --user
Collecting pydbhub
  Downloading pydbhub-0.0.3-py37-none-any.whl (3.0 kB)
Collecting rich
  Downloading rich-10.15.2-py3-none-any.whl (214 kB)
Requirement already satisfied: requests in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from pydbhub) (2.25.1)
Requirement already satisfied: python-dateutil in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from pydbhub) (2.8.1)
Requirement already satisfied: six>=1.5 in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from python-dateutil->pydbhub) (1.14.0)
Requirement already satisfied: chardet<5,>=3.0.2 in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from requests->pydbhub) (3.0.4)
Requirement already satisfied: idna<3,>=2.5 in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from requests->pydbhub) (2.10)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from requests->pydbhub) (1.26.4)
Requirement already satisfied: certifi>=2017.4.17 in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from requests->pydbhub) (2021.5.30)
Collecting colorama<0.5.0,>=0.4.0
  Downloading colorama-0.4.4-py2.py3-none-any.whl (16 kB)
Collecting typing-extensions<5.0,>=3.7.4
  Downloading typing_extensions-4.0.1-py3-none-any.whl (22 kB)
Collecting pygments<3.0.0,>=2.6.0
  Downloading Pygments-2.10.0-py3-none-any.whl (1.0 MB)
Collecting commonmark<0.10.0,>=0.9.0
  Downloading commonmark-0.9.1-py2.py3-none-any.whl (51 kB)
Installing collected packages: typing-extensions, pygments, commonmark, colorama, rich, pydbhub
Successfully installed colorama-0.4.4 commonmark-0.9.1 pydbhub-0.0.3 pygments-2.10.0 rich-10.15.2 typing-extensions-4.0.1

But can't access the library:
>>> import pydbhub
Traceback (most recent call last):
  File "<console>", line 1, in <module>
ModuleNotFoundError: No module named 'pydbhub'

And also I've found just a folder "pydbhub-0.0.3.dist-info" but no package folder.
If any further information is needed I'll try to serve it.
Thanks in advance,
Martin

What is the result if you install directly from Git?

Thanks for the fast reply, but unfortunately the result keep the same :-/
>>> pip install --upgrade git+git://github.com/LeMoussel/pydbhub.git
Defaulting to user installation because normal site-packages is not writeable
Collecting git+git://github.com/LeMoussel/pydbhub.git
  Cloning git://github.com/LeMoussel/pydbhub.git to c:\users\mdotza~1\appdata\local\temp\pip-req-build-1tltb6g5
  Running command git clone --filter=blob:none -q git://github.com/LeMoussel/pydbhub.git 'C:\Users\MDOTZA~1\AppData\Local\Temp\pip-req-build-1tltb6g5'
  Resolved git://github.com/LeMoussel/pydbhub.git to commit 501ea2c0ec7785bc06a38961a1366c3c04d7fabd
  Preparing metadata (setup.py): started
  Preparing metadata (setup.py): finished with status 'done'
Requirement already satisfied: requests in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from pydbhub==0.0.3) (2.25.1)
Requirement already satisfied: python_dateutil in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from pydbhub==0.0.3) (2.8.1)
Requirement already satisfied: rich in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from pydbhub==0.0.3) (10.15.2)
Requirement already satisfied: six>=1.5 in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from python_dateutil->pydbhub==0.0.3) (1.14.0)
Requirement already satisfied: certifi>=2017.4.17 in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from requests->pydbhub==0.0.3) (2021.5.30)
Requirement already satisfied: chardet<5,>=3.0.2 in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from requests->pydbhub==0.0.3) (3.0.4)
Requirement already satisfied: idna<3,>=2.5 in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from requests->pydbhub==0.0.3) (2.10)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from requests->pydbhub==0.0.3) (1.26.4)
Requirement already satisfied: typing-extensions<5.0,>=3.7.4 in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from rich->pydbhub==0.0.3) (4.0.1)
Requirement already satisfied: commonmark<0.10.0,>=0.9.0 in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from rich->pydbhub==0.0.3) (0.9.1)
Requirement already satisfied: colorama<0.5.0,>=0.4.0 in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from rich->pydbhub==0.0.3) (0.4.4)
Requirement already satisfied: pygments<3.0.0,>=2.6.0 in c:\users\mdotzauer\appdata\roaming\python\python37\site-packages (from rich->pydbhub==0.0.3) (2.10.0)

>>> import pydbhub
Traceback (most recent call last):
  File "<console>", line 1, in <module>
ModuleNotFoundError: No module named 'pydbhub'

best,
Martin

Strange. This seems to be a concern of your Python environment.

May be. I'll try to update to the latest version of python 3.7.
But many thanks for your feedback.


grpc-json transcoder: HTTP 200 on large response failure

Title: Large response that fails (due to exceeding per_connection_buffer_limit_bytes) is reported as HTTP status 200 instead of 5xx
Description:
See also #6004. Situation is similar: when using gRPC-JSON transcoder and JSON-encoded response exceeds per_connection_buffer_limit_bytes, this results in empty HTTP  response with status 200. Also, there is no ERROR record in the logs that would clearly indicate the failure.
Why a 200 would be returned when the request clearly failed? This is extremely confusing to any HTTP client.

This is the problem with http1. The filter has to send the header first before transcoding the response body.   For http2, the error status is send in trailing headers.

Thank you, @qiwzhang. So what does it mean in practice? Is there a solution for HTTP/1.1 clients?


P520 Missing highlight

Missing highlight on the first snippet, on the line of:
var nodeLinkFn;

Actually, on the same snippet, the if (nodeLinkFn) and its closing key is not highlighted as well.

Fixed, thanks.


Tunnelling doesn't work in v3.1.0

Description
With the new version of needle (3.1.0) the current approach (here) to set up tunnelling doesn't work anymore (version 3.0.0 works fine). Scanning the network using Wireshark shows that needle ends up looping CONNECT requests to the proxy. I suspect that needle tries to use both the tunnel agent while also passing proxy parameters from the environment (HTTP_PROXY and HTTPS_PROXY) and thus ends up in this weird state.
We were previously using global agent together with _PROXY environment variables to force needle to do CONNECT requests but since the new version that also doesn't work (which might be a different issue)
How to reproduce

setup local mtimproxy (or any other proxy)
set up tunnelling using the tunnel agent as describe in the documentation, pointing proxy and port to the mtimproxy.
set up HTTP_PROXY and HTTPS_PROXY environment variables to point to the same proxy
try and make a network call using needle

Expected behaviour
When HTTP_PROXY and HTTPS_PROXY is set, and tunnel is configured, needle should make a CONNECT request to the proxy and establish a tunnel
Actual behaviour
needle tries to establish CONNECT request but doesn't succeed

Here's the code snippet:
var needle = require('needle');
var tunnel = require('tunnel');
var myAgent = tunnel.httpOverHttp({
  proxy: { host: '<IP_ADDRESS>', port: 8080 }
});

needle.get('https://github.com/status', {agent: myAgent} ,function (error, response) {
        if (!error && response.statusCode == 200)
                console.log(response.body);
        else console.log(error);
});

Make sure to npm install needle tunnel and then set _PROXY environment variables to point to the same proxy (in my case<IP_ADDRESS>:8080) as the agent configuration above.
When I run this with needle version 3.1.0 installed, in Wireshark, I see attempts to CONNECT <IP_ADDRESS>:8080 HTTP/1.1 and HTTP/1.1 502 Bad Gateway  (text/html) whereas with version 3.0.0, I can see CONNECT github.com:443 HTTP/1.1 and HTTP/1.1 200 Connection established - which is what I would expect. Mind that both result in Server disconnected but the symptoms are the same ones we are experiencing in our own system so I think this is a good example.
Let me know how it goes, I will also try and debug though I am not very familiar with the codebase of needle.

Let me add a bit more context and details.
What we are trying to achieve is to use needle with an HTTP/S proxy. In secure setup, HTTP clients are expected to send proxied requests for HTTPS resources through a HTTP CONNECT tunnel.
As far as I understand, needle doesn't support CONNECT requests. Therefore we are using https://github.com/gajus/global-agent to patch node's http agent to provide CONNECT-capable proxy support. There's similar libraries like https://github.com/koichik/node-tunnel (deprecated) or https://github.com/TooTallNate/node-http-proxy-agent.
The introduction of https://github.com/tomas/needle/pull/382 picks up HTTP_PROXY/HTTPS_PROXY from environment variables and does not allow needle to be used without proxy if those environment variable are present.
We're looking for a way to opt-out of needle picking up proxy configuration from environment variables.

@tomas , any chance you could have a look at this, especially PR #427 as a suggestion to disable this behaviour?

Yes, sorry. I'll make some time this week to take a look into this. :)

Hey @tomas , any chance you could have a look at #427 to optionally disable needle from automatically picking up environment variables?

During the analysis and debugging, I noticed that no consistent distinction is made between http_proxy and https_proxy. It is sufficient if one of the two is set, then this is used for all connections. If both are set, the http_proxy is used.

My test snipped:
var needle = require('needle');
needle.get('https://github.com/status', function (error, response) {
        if (!error && response.statusCode == 200)
                console.log(response.body);
        else console.log(error);
});

The results (via proxy: export HTTPS_PROXY=http://localhost:8888):

squid
tinyproxy

HTTPS websites
❌
⚠

HTTP sites
✅
✅

For HTTPS pages, the connection goes through the tinyproxy, but the proxy tries to connect the destination via HTTP. An attempt with CURL through the tinyproxy works without problems.
Some return values:
error from squid
CacheErrorInfo - ERR_READ_ERROR&body=CacheHost: d4e570ebcbe2
ErrPage: ERR_READ_ERROR
Err: [none]
TimeStamp: Fri, 29 Dec 2023

ClientIP: 10.10.x.x
ServerIP: github.com

HTTP Request:
GET /status HTTP/1.1
Accept: */*
User-Agent: Needle/3.3.0 (Node.js v18.17.1; linux x64)
Host: github.com
Connection: close

some output from node:
_header: 'GET https://github.com/status HTTP/1.1\r\n' +
  'accept: */*\r\n' +
  'user-agent: Needle/3.3.0 (Node.js v18.17.1; linux x64)\r\n' +
  'host: github.com\r\n' +
  'Connection: close\r\n' +
  '\r\n',
method: 'GET',
path: 'https://github.com/status',
host: 'localhost',
protocol: 'http:',
statusCode: 502,
statusMessage: 'Bad Gateway',

I cannot see how is send the CONNECT request.
curl example
curl https://github.com/status -v
* Uses proxy env variable HTTPS_PROXY == 'http://localhost:8888'
*   Trying <IP_ADDRESS>:8888...
* Connected to (nil) (<IP_ADDRESS>) port 8888 (#0)
* allocate connect buffer!
* Establish HTTP proxy tunnel to github.com:443
> CONNECT github.com:443 HTTP/1.1
> Host: github.com:443
> User-Agent: curl/7.81.0
> Proxy-Connection: Keep-Alive

I have found a workaround for me:
var { ProxyAgent } = require('proxy-agent');
var needle = require('needle');
needle.get('https://github.com/status',{ agent: new ProxyAgent(), use_proxy_from_env_var: false }, function (error, response) {
        if (!error && response.statusCode == 200)
                console.log(response.body);
        else console.log(response);
});

This is a similar setting we're using needle. We're using proxy-agent or global-agent, but the earlier changes for needle to pick up the env variables broke this.
@tomas with use_proxy_from_env_var: false implemented, in my opinion, we can close this issue.
@dklimpel happy to leave this open if your case isn't fully covered yet.

IMHO this is open. Needle supports:

HTTP Proxy forwarding, optionally with authentication

And that is not the case. There is no support for https_proxy at the moment.


[BUG]: Cannot create migrations when using: import "server-only"

What version of drizzle-orm are you using?
0.29.3
What version of drizzle-kit are you using?
0.20.13
Describe the Bug
Clone repo https://github.com/gnllucena/drizzle-import-server-only
bun install
bun run migration:generate
The command will only create the meta folder and the _journal.json file on src/server/data/migrations/ folder.
If you remove the import “server-only” from src/server/data/schema/customers-schema.ts you’ll notice that the migrations are generated normally.
Expected behavior
The migrations should be created when schema uses import "server-only"
Environment & setup
OS: macOS 14.2.1 23C71 arm64
Kernel: 23.2.0
Shell: fish 3.5.0

I'm encountering the same problem when running drizzle-kit studio where my schema.ts file has import "server-only"

Issue persists on "drizzle-kit": "^0.20.14"
Repo updated

As a workaround, I was able to bypass the server-only error when running drizzle-kit commands. I'm not sure if this works with bun though.
NODE_OPTIONS="--conditions=react-server" drizzle-kit ...
Relevant docs:

https://nodejs.org/api/packages.html#conditional-exports
https://github.com/reactjs/rfcs/blob/main/text/0227-server-module-conventions.md#poisoned-imports


allow set debounceTime

allow to change default 500ms at line 1583, that will be called at line 1529

Hi debounceTime === 0 is same with setTimeout 0 and will be executed on
process next tick
On Wed, Jul 21, 2021 at 14:30 Icebob @.***> wrote:

@.**** requested changes on this pull request.

In src/index.js
https://github.com/moleculerjs/moleculer-web/pull/260#discussion_r673728695
:

@@ -1580,10 +1583,11 @@ module.exports = {

  	this.settings.routes.forEach(route => this.addRoute(route));

  // Regenerate all auto aliases routes

const debounceTime = this.settings.debounceTime >= 0 ? parseInt(this.settings.debounceTime) : 500;

this.settings.debounceTime >= 0 is not correct. Just this.settings.debounceTime

0

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/moleculerjs/moleculer-web/pull/260#pullrequestreview-711340118,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHX6SL7YJMKJ5KWAZ2OQZNTTYZZQTANCNFSM5AHYQNUA
.

Yeah, just null >= 0 is true, so if somebody set it to null it should be used the default 500 value.

Oh, new knowledge lol
On Wed, Jul 21, 2021 at 14:36 Icebob @.***> wrote:

Yeah, just null >= 0 is true, so if somebody set it to null it should be
used the default 500 value.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/moleculerjs/moleculer-web/pull/260#issuecomment-883963144,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHX6SLY3MYPEIPDFP7F5RW3TYZ2IHANCNFSM5AHYQNUA
.


Search in VE

Search edit box
A user types desired variable name in search box.
expectation: it searches variable name among top level, and expanded children level of variable tree.
question: does it have to search all spaces of children tree, although user has not expanded the variable tree?

Only search within current information available in VE at the moment (names)

https://github.com/Microsoft/RTVS/commit/468732e509cab0e9ff2a8d509e9afc2059d2174d


[Feature/Bug Report] Discourage/disable "./" as one of the "moduleDirectories"

🐛 Bug Report
This is a bug report pertaining to the use of "./" as one of the moduleDirectories. In particular, I was using a scoped package that had the same subdirectory name as a non-scoped package (e.g. @my-scope/my-package relies on my-package as an external/peer dependency). This prevented @my-scope/my-package from having access to my-package.
To Reproduce
See this repo
Expected behavior
Ideally, this behavior is not allowed. We should instead encourage people to use moduleNameMapper.
Link to repl or repo (highly encouraged)
Again see https://github.com/davidroeca/jest-shadowing-issue
envinfo
  System:
    OS: Linux 5.4 Linux Mint 20 (Ulyana)
    CPU: (8) x64 Intel(R) Core(TM) i5-8350U CPU @ 1.70GHz
  Binaries:
    Node: 14.9.0 - ~/.asdf/installs/nodejs/14.9.0/bin/node
    Yarn: 1.22.4 - ~/.asdf/shims/yarn
    npm: 6.14.8 - ~/.asdf/installs/nodejs/14.9.0/bin/npm
  npmPackages:
    jest: ^26.4.2 => 26.4.2

Apologies, hit enter too soon. Edited the issue with the correct info.

I have also run into this issue (@emotion/react having react as a dependency).
@davidroeca I think the Needs Repro label can be removed as there seems to be a reproduction.

just for the record:
module.exports = {
  // ...
-  moduleDirectories: ["node_modules", "./"]
+  moduleDirectories: ["node_modules", "<rootDir>"],
};

solves the issue

I see no mentions/cautions here. I believe it is still at the very least worth discouraging somewhere in the docs.


Stop using javascript window.open in template

Questions
Answers

Description?
Three templates in module blockreassurance displayBlock.tpl, displayBlockProduct.tpl, displayBlockWhite.tpl are using JavaScript window.open method to emulate a link.  We don't have any validation with links there so replace the JavaScript window.open by a pure HTML a tag will be better.

Type?
improvement

BC breaks?
Does it break backward compatibility? no

Deprecations?
Does it deprecate an existing feature? no

Fixed ticket?
Issue 30055.

How to test?
After applying PR, check blockreassurance block on Product page to make sure the block view and link work as usual.

@NeOMakinG I don't know either. matks and nicosomb suggested this kind of change in PR 449 - widget, so I guess we should wait their final decision there before proceeding with this PR :pray:

Idk the historical reason about this but looks fine to me

I suspect the reason for that is you can't have a tag inside another a tag and since you are allowed to add it in the content through editor options then it's prone to break. It's the same with imageslider where you specify the link for whole slide, but still can add links inside slide description. Templates need to be changed to wrap only images/icons with links, not the text content. I can't see any other way atm.

@SharakPL I have just understood your comment today. Be back when you have time to protect this valuable trick :)

So wdyt? Do we keep the trick or do we apply @leemyongpakvn changes, which I think are interesting, and consider the module user must make sure he does not input bad HTML with a tags inside a tags?

I would keep the trick for now.
I changed my themes to wrap only image with link, but it's not ideal either.

@matks SharakPL clarified the use case of onclick="window.open('{$block['link']}') and confirmed he want to keep the trick. Can I close this PR ?

@matks SharakPL clarified the use case of onclick="window.open('{$block['link']}') and confirmed he want to keep the trick. Can I close this PR ?

If it was the only one purpose of this PR, I guess yep! And sorry for the time, at least we are aware of it now!

Close as the use case of the trick is clarified.


<urlopen error [Errno 11004] getaddrinfo failed>

I m running youtubedl from a third party tool. When I run in usual DOS command line, Its working fine. But when I run from a tool, its throwing below error message.
Using: Win7

Command: E:\YoutubeDownloader\bin\youtube-dl.exe  --output "D:\Youtubes\Qtest\Test.%%(ext)s" --recode-video mp4 "https://www.youtube.com/watch?v=ujFcyRNta7U"
Working Dir: E:\YoutubeDownloader\bin
Process Status: Starting
Process Status: Running
WARNING: The url doesn't specify the protocol, trying with http
[generic] watch?v=ujFcyRNta7U": Requesting header
WARNING: Could not send HEAD request to http://--output "D:\Youtubes\Qtest\Test.%%(ext)s" --recode-video mp4 "https://www.youtube.com/watch?v=ujFcyRNta7U": <urlopen error [Errno 11004] getaddrinfo failed>
[generic] watch?v=ujFcyRNta7U": Downloading webpage
ERROR: Unable to download webpage: <urlopen error [Errno 11004] getaddrinfo failed> (caused by URLError(gaierror(11004, 'getaddrinfo failed'),))
Process Status: NotRunning (Could be completed/Crashed/Stopped/Not yet Started)
--ProcessCompleted-ExitCode:--1

Same output for verbose too

When I run in usual DOS command line, Its working fine. But when I run from a tool, its throwing below error message.

Apparently the problem is in that tool. Most likely it didn’t pass correct arguments to youtube-dl.


Add more esoteric categories for the explore page

The page at https://flathub.vercel.app/apps almost entirely duplicates the home page. Which is a shame!

We need new categories to expand on this page. I will curate some new lists of more esoteric things. As noted in #171 investment in curatorial tools and processes would help app discovery here and justify browsing.

I would put 5 top/random apps of each category here instead

That's worth experimenting with for sure.

That's top apps

Has been landed and I'm pretty happy with it so far, I'm going to close this. Feel free to reopen, not sure how much we want random apps instead.


Fixing thingies

@lawiet47 I removed the exceptions based on responses (we can just use the HTTPError defined by the requests package.

I added that because I thought we would require a slightly better way of exception handling (parsing error messages & stuff) in the future

I added that because I thought we would require a slightly better way of exception handling (parsing error messages for specific endpoints & stuff) in the future

I think this changes achieve just that. TBH I only see value in adding exceptions for whenever the response does not match what we expect - meaning, handle the HTTP errors with the requests package.


VCA envelope tails are a bit abrupt

When the VCA envelope is decaying, the sound does not decay as smoothly as we would like.
This is due to resistor R407 on the VCF/VCA board, which ensures that when zero volts is applied to the CV input, the output signal is completely silenced. It "pushes down" the CV input by a tiny bit.
But if too much current flows through this resistor, it can make the long tails of the volume envelope end abruptly. The final decay doesn't sound as smooth as it should.
Right now the volume pot turns up and down the control voltage that goes to the final VCA. This means when the volume is turned down low, the ADSR signal is small relative to the impact of the "push-down" resistor R407. If instead we always sent the ADSR signal to the VCA CV full-strength, and then attenuated the signal afterwards, I think the un-smooth decay would effect would be reduced.
Since the volume knob is way over on the Ribbon board, we should probably add another VCA just for the master volume. The Ribbon board can send two VCA signals, one is the volume ADSR, and the other is a [0, 5v] master volume control voltage. The final master volume VCA could be exponential control (IE a single SSI216x section plus an opamp), so we can get a nice audio taper from a linear potentiometer on the Ribbon board.
TL;DR add an expo control VCA after the main VCA. The first VCA gets the volume ADSR as a control signal, the final VCA is controlled from the master volume pot.

The new ADSR and VCA setup fixes this.


Resolves #247 implement scroll widget

Description
This PR adds the scroll widget.
Usage
The scroll widget lets its child be scrolled if it overflows its parent's width / height (as set by the :width / :height props).
The :hscroll and :vscroll props are booleans dictating whether can scroll vertically and/or horizontally, defaults to true.

Checklist
Please make sure you can check all the boxes that apply to this PR.

[x] All widgets I've added are correctly documented.
[ ] I added my changes to CHANGELOG.md, if appropriate.
[x] I used cargo fmt to automatically format all code before committing

very cool, thanks a lot!

@viandoxdev I'm not so sure why your implementation doesn't not work at all.
Using the code snippet you've supplied to test, does not work.

@viandoxdev I'm not so sure why your implementation doesn't not work at all. Using the code snippet you've supplied to test, does not work.

Make sure your eww is up to date, and that the current running deamon matches that (you may need to restart it with eww kill).

Prior to using git branch it works, Thanks.


Initial implementation of a TracerResolver for Jaeger

This TracerResolver implementation uses the ConstSampler and NullStatsReporter. Configuration for other implementations will be provided as separate PRs. This PR is intended to provide a basic resolver implementation.

Codecov Report

Merging #175 into master will increase coverage by 0.52%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #175      +/-   ##
============================================
+ Coverage     79.66%   80.19%   +0.52%
- Complexity      447      461      +14
============================================
  Files            76       77       +1
  Lines          1751     1772      +21
  Branches        205      206       +1
============================================
+ Hits           1395     1421      +26
+ Misses          267      264       -3
+ Partials         89       87       -2

Impacted Files
Coverage Δ
Complexity Δ

...er/jaeger/tracerresolver/JaegerTracerResolver.java
100% <100%> (ø)
10 <10> (?)

...java/com/uber/jaeger/reporters/RemoteReporter.java
81.25% <100%> (+0.29%)
6 <1> (+1)
:arrow_up:

.../uber/jaeger/samplers/RemoteControlledSampler.java
91.54% <0%> (-1.41%)
18% <0%> (-1%)

...jaeger/reporters/protocols/ThriftUdpTransport.java
83.05% <0%> (+5.08%)
16% <0%> (+2%)
:arrow_up:

...c/main/java/com/uber/jaeger/senders/UdpSender.java
76.36% <0%> (+5.45%)
13% <0%> (+2%)
:arrow_up:

Continue to review full report at Codecov.

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

@yurishkuro Just to be clear - move everything to the Configuration class, with environment var based config via a default constructor and then the TracerResolver simply use the default constructor and return the tracer?

move all config-related code to Configuration (env var declaration, instantiating config objects)
but not to default constructor, that would not be backwards compatible - to a static method Configuration.fromEnv(): Configuration


Configurable linebreaks (in description)

I often break longer descriptions into several lines, for better readability in the code itself. If I add an additional empty line docstrap seems to automatically add a paragraph in the resulting html - which is good. But for each single linebreak in the comment a
 tag is added as well, and this looks weird most of the time in the resulting html, since it would be better - at least for me - if the resulting text would just wrap.
So I would really like to have a configurable option which does not generate these
 tags at least in the description.

Ah nevermind, I just saw what caused this. I use the markdown parser, and "hardwrap" was set to true. Disabling this seems to solve my problem.


Add fields to LOI

Adds isOpportunistic to track whether LOI is created by the user in Suggest LOI flow
Adds ownerEmail field to keep track of which user created the LOI

Fixes #1805

/gcbrun


[LICENSE] [v1.x] Change intgemm to a submodule instead of fetch.

Description
This resolves the issue that LICENSE referred to 3rdparty/intgemm/LICENSE which was not present in the source because it was downloaded:
https://lists.apache.org/thread.html/r669576b6506f67ea96e9b1bd385b04f59c4e6e7fa1b13a6dc0abc7d9%40<general.incubator.apache.org>
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage
[x] Code is well-documented

Changes

[x] Remove fetch from CMakeLists.txt
[x] Add submodule in 3rdparty
[x] Mention catch in top-level LICENSE
[x] Exclude intgemm from header license checks

Note
Sibling #19406 addresses v1.8.x.  This is v1.x.  master is coming.

@mxnet-bot run ci [unix-cpu, unix-gpu]
Testing is really flaky today https://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/PR-19407/1/pipeline
[2020-10-22T20:01:24.591Z] ======================================================================

[2020-10-22T20:01:24.591Z] FAIL: test operators which don't implement FComputeEx or FStatefulComputeEx

[2020-10-22T20:01:24.591Z] ----------------------------------------------------------------------

[2020-10-22T20:01:24.591Z] Traceback (most recent call last):

[2020-10-22T20:01:24.591Z]   File "/usr/local/lib/python3.5/dist-packages/nose/case.py", line 198, in runTest

[2020-10-22T20:01:24.591Z]     self.test(*self.arg)

[2020-10-22T20:01:24.591Z]   File "/usr/local/lib/python3.5/dist-packages/nose/util.py", line 620, in newfunc

[2020-10-22T20:01:24.591Z]     return func(*arg, **kw)

[2020-10-22T20:01:24.591Z]   File "/work/mxnet/tests/python/gpu/../unittest/common.py", line 218, in test_new

[2020-10-22T20:01:24.591Z]     orig_test(*args, **kwargs)

[2020-10-22T20:01:24.591Z]   File "/work/mxnet/tests/python/gpu/../unittest/test_sparse_operator.py", line 1757, in test_sparse_storage_fallback

[2020-10-22T20:01:24.591Z]     check_operator_with_temp_resource(shape, lhs)

[2020-10-22T20:01:24.591Z]   File "/work/mxnet/tests/python/gpu/../unittest/test_sparse_operator.py", line 1752, in check_operator_with_temp_resource

[2020-10-22T20:01:24.591Z]     check_numeric_gradient(test, location)

[2020-10-22T20:01:24.591Z]   File "/work/mxnet/python/mxnet/test_utils.py", line 1238, in check_numeric_gradient

[2020-10-22T20:01:24.591Z]     ("NUMERICAL_%s"%name, "BACKWARD_%s"%name))

[2020-10-22T20:01:24.591Z]   File "/work/mxnet/python/mxnet/test_utils.py", line 749, in assert_almost_equal

[2020-10-22T20:01:24.591Z]     raise AssertionError(msg)

[2020-10-22T20:01:24.591Z] AssertionError:

[2020-10-22T20:01:24.591Z] Items are not equal:

[2020-10-22T20:01:24.591Z] Error 1.254846 exceeds tolerance rtol=1.000000e-04, atol=1.000000e-03 (mismatch at least 11.000000%).

[2020-10-22T20:01:24.591Z] Location of maximum error: (0, 0), NUMERICAL_x=0.82617188, BACKWARD_x=0.82753056

[2020-10-22T20:01:24.591Z]  ACTUAL: array([[0.8261719, 0.8261719, 0.8261719, ..., 0.8261719, 0.8261719,

[2020-10-22T20:01:24.591Z]         0.8261719],

[2020-10-22T20:01:24.591Z]        [0.8261719, 0.8261719, 0.8261719, ..., 0.8261719, 0.8261719,...

[2020-10-22T20:01:24.591Z]  DESIRED: array([[0.82753056, 0.82753056, 0.82753056, ..., 0.82753056, 0.82753056,

[2020-10-22T20:01:24.591Z]         0.82753056],

[2020-10-22T20:01:24.591Z]        [0.82753056, 0.82753056, 0.82753056, ..., 0.82753056, 0.82753056,...

[2020-10-22T20:01:24.591Z] -------------------- >> begin captured stdout << ---------------------

[2020-10-22T20:01:24.591Z]

[2020-10-22T20:01:24.591Z] *** Maximum errors for vector of size 100:  rtol=0.0001, atol=0.001

[2020-10-22T20:01:24.591Z]

[2020-10-22T20:01:24.591Z]   1: Error 1.254846  Location of error: (0, 0), NUMERICAL_x=0.82617188, BACKWARD_x=0.82753056

[2020-10-22T20:01:24.591Z]   2: Error 1.254846  Location of error: (0, 1), NUMERICAL_x=0.82617188, BACKWARD_x=0.82753056

[2020-10-22T20:01:24.591Z]   3: Error 1.254846  Location of error: (0, 2), NUMERICAL_x=0.82617188, BACKWARD_x=0.82753056

[2020-10-22T20:01:24.591Z]   4: Error 1.254846  Location of error: (0, 3), NUMERICAL_x=0.82617188, BACKWARD_x=0.82753056

[2020-10-22T20:01:24.591Z]   5: Error 1.254846  Location of error: (0, 4), NUMERICAL_x=0.82617188, BACKWARD_x=0.82753056

[2020-10-22T20:01:24.591Z]   6: Error 1.254846  Location of error: (0, 5), NUMERICAL_x=0.82617188, BACKWARD_x=0.82753056

[2020-10-22T20:01:24.591Z]   7: Error 1.254846  Location of error: (0, 6), NUMERICAL_x=0.82617188, BACKWARD_x=0.82753056

[2020-10-22T20:01:24.591Z]   8: Error 1.254846  Location of error: (0, 7), NUMERICAL_x=0.82617188, BACKWARD_x=0.82753056

[2020-10-22T20:01:24.591Z]   9: Error 1.254846  Location of error: (0, 8), NUMERICAL_x=0.82617188, BACKWARD_x=0.82753056

[2020-10-22T20:01:24.591Z]  10: Error 1.254846  Location of error: (0, 9), NUMERICAL_x=0.82617188, BACKWARD_x=0.82753056

[2020-10-22T20:01:24.591Z]

[2020-10-22T20:01:24.591Z] --------------------- >> end captured stdout << ----------------------

[2020-10-22T20:01:24.591Z] -------------------- >> begin captured logging << --------------------

[2020-10-22T20:01:24.591Z] common: WARNING: Error seen with seeded test, use MXNET_TEST_SEED=1917660267 to reproduce.

[2020-10-22T20:01:24.591Z] --------------------- >> end captured logging << ---------------------

[2020-10-22T20:01:24.591Z]


Fix missing URL in RestApi for address notifications (#305)

Closes CityOfZion/neo-python/#305
/v1/notifications/addr/<addr> is now responding.
Now in the right branch hopefully, apologies for the misunderstanding.

Coverage remained the same at 78.235% when pulling a7e5471609fc18f88f5e6350a4247f02d65f7011 on MediaServe:development into 1c04f57245f457a93c03736172c0b03fe739cbaf on CityOfZion:development.

Nice, thanks! Could you add a line about this to the changelog please?

Nice, thanks! :+1:


Remove Connect and ConnectAsync methods from IDistributedCache interface

@lodejard brought up a good point of the need of having a Connect method on the interface. They seem to be implementation details for the particular distributed cache implementation (ex: RedisCache). SqlServerCache and LocalCache(distributed MemoryCache) do not require them.
https://github.com/aspnet/Caching/blob/dev/src/Microsoft.Extensions.Caching.Abstractions/IDistributedCache.cs#L10-L12
cc @Tratcher @muratg @Eilon @davidfowl
And regarding related issue https://github.com/aspnet/Caching/issues/131, RedisCache can implement an IDisposable interface and in the dispose method can close the connections. Currently RedisCache is registered as singleton and the DI would call the Dispose method on the singleton instance(I verified this is happening)

I have the following PR with the changes
https://github.com/aspnet/Caching/pull/136

Guys why are we removing the Connect methods from IDistributedCache ?
I understand that SqlServerCache and LocalCache(distributed MemoryCache) do not require them but in reality they are not distributed caches.
Redis, NCache or any other distributed caches are very different and support different needs and they do not provided a connection pool therefore a Connect/Initialize method is a must.

@basitanwer If you look at the PR you will see how it works for Redis too. NCache won't be different. You can still do your own pool if necessary.

I read the PR, and I think I understand it.  But I don't think the response to @basitanwer is complete.
Because connections are now done as a part of every Get and Set (and async variants) operation, how does one now handle failure scenarios?  I.e., when a Get gives you a null result, how do you know when that's because the key doesn't exist in the cache and when it's because the cache isn't even there?

@aggieben either way it's a cache-miss. Your app should function even when the cache is unavailable, even if less efficiently.
Connection pooling, if any, should happen inside the cache provider, that's not a detail that should be exposed the the application.

@Tratcher yes, it's a cache-miss either way, but we need to know what kind of cache miss so that we can implement fallback strategies.  In my particular case, if the redis isn't even there, then I fallback to a different caching strategy.  With these changes it is now impossible to correctly implement such a thing without having to re-write code that the framework already has but is now inaccessible.

If your redis server isn't there then I'd expect that to be a deployment / configuration error, not something you'd try to deal with at runtime.


Enable user to collect detailed Inventory of ESXi host(s) in a given vSphere cluster

This PR will enable user to collect detailed inventory of all ESXi host(s) in a given vSphere cluster, this PR also let user know the current storage and network configuration and installed ESXi version. As user I want to know my Connection and health status of any esxi host before any datastore operation is invoked on the target host in vCenter.
The changes in this PR are as follows:

Collect detailed ESXi host inventory.
Know the Connection State and Health Status of the host
Know  accessible datastore(s) to each ESXi host in the given cluster
Know Storage Adapter (hbas) of a host.

I have read the contributor guidelines and have completed the following:

[x] Formatted the code using VSCode default formatter for PowerShell.
[x] Tested the code end-to-end against an SDDC.
[x] Documented the functions using standard PowerShell markup and applied AVSAttribute to newly exported functions.

@srajmohan24 can you merge it ?


[Feature] Creating frontend using Stremlit in python

Integrate a frontend for TwinTrim using Streamlit and authenticate with streamlit, a Python library for creating interactive web applications. This frontend should allow users to perform key actions like selecting directories to scan, viewing duplicate files in an organized manner, and confirming or adjusting auto-removal settings.

@Kota-Karthik kindly assign me this issue along with the labels of gssoc extended,hacktoberfest and level

@KGupta2601
Thanks for raising this issue
But as our prject is a CLI tool
I don't really think this issue will be of any use to us
But we encourage you to raise other issues


Diagnostic Settings

In Diagnostics settings section of Virtual network there is a "VMProtectionAlerts" log type. May I know what this is used for?

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

ID: f03f79a2-923f-3e1a-5e8f-8ce1f5913a56
Version Independent ID: 6330e3e6-d903-b8f8-370d-e61ff4384d5e
Content: Azure Virtual Network
Content Source: articles/virtual-network/virtual-networks-overview.md
Service: virtual-network
GitHub Login: @anavinahar
Microsoft Alias: anavin

@CharmanderJieniJieni Thanks for the feedback! I do not believe we have any documentation about what exactly it is, all I could find is this doc about diagnostic log schema.
I have assigned the issue to the content author to evaluate and update as appropriate.

@TravisCragg-MSFT any update here? I still do not find any documentation on it and actually the same is true for many other Diagnostics for other Azure resources

As per the documentation "Virtual Networks = Schema not available" and similar for Express route which is for sure not true. so there is a update for documentation required.

Looking for the same info, is there any update done to the documentation ?

I have handled a number of support cases related to the diagnostic settings for Virtual Networks. The last info I heard is that neither VMProtectionAlerts nor AllMetrics are actually functional. I believe they were enabled as part of a preview test that is not moving forward.

it is nearly the end of the first quarter of 2022, and we still can't resolve the mystery of "VMProtectionAlerts"

I'm still not able to find any documents describe VMProtectionAlerts, is there any update?

In my understanding this log has never worked for public consumption, but it still isn't removed from the Portal.


feat: Lightning Gateway registers configurable number of route hints

Same PR as https://github.com/fedimint/fedimint/pull/2647 except this makes the number of route hints configurable from a gatewayd CLI parameter and it adds a test using LDK Node.

Test for this feature is failing.

Test for this feature is failing.

I believe there is a race when setting up the channels and registering the gateway. Need to refactor a bit

Taking me a bit longer to debug than I would have liked. I think there is currently a bug or race condition when getting CLN's route hints. Sometimes it is returning 1 route hint when it should be returning 2.

LDK Node was causing the test I was writing to be pretty flaky. The channel would sometimes close between CLN and LDK Node which caused the wrong number of route hints to be returned.
Given that this PR is not dependent on LDK node, I just took that part out for now and will return to the LDK Node issues later. The test should be reliable now.


"Coțofănești" commune is listed in the drop-down menu

Description:
The commune is listed in the drop-down menu according to the law
Preconditions :the website (https://peviitor-ro.github.io/ui.orase/) is up and runing.
After every step delete the text you typed from the search bar.

Step 1
Type "Coțofănești" in the search bar.
Expected results
The location is listed in the drop-down menu as "Coțofănești, Bacău"


Ethereum RPC credentials

Hello, I found not how to configure the "user" and "password" in the RPC configuration wallet in MPOS
/**
 * Local wallet RPC
 *  RPC configuration for your daemon/wallet
 *   https://github.com/MPOS/php-mpos/wiki/Config-Setup#wiki-local-wallet-rpc
 **/
$config['wallet']['type'] = 'http';
$config['wallet']['host'] = '<IP_ADDRESS>:8545';
$config['wallet']['username'] = '';
$config['wallet']['password'] = '';

Knowing that I activated RPC in ETH console
admin.startRPC("<IP_ADDRESS>", 8545, "*", "web3,db,net,eth")

Do you have a suggestion to solve this?

You'll have FAR (!) more todo than this :) The RPC from ETH is totally different from bitcoin (and its clones) - you have to rewrite quite some classes of MPOS to get it running with ETH. bitcoinwrapper.class.php for example and much more..


Add DTS-HD + DSUR as a sound type

I've been getting this warning in home assistant when using the integration with my Denon AVC-X3700H:
Not able to match sound mode: 'DTS-HD + DSUR', returning raw sound mode.
From looking at #106 , it seems that the issue is a simple matter of a missing enum. This ought to fix that.

thanks 👍


HLSL: Implicit derivatives in non-uniform control flow

GLSL and SPIR-V define implicit derivatives as undefined in the specification in contrast to HLSL which seems to like these cases (e.g Sample in non-uniform control flow) and try to unroll the loop.
Would there be a way to workaround this, considering the derivatives are undefined? For example, detecting requirement for implicit derivatives and set hardcoded ones like emitting SampleLod(0) instead of Sample?

Uh ... How would HLSL give you sensible (i.e. non-undefined) derivatives in non-uniform control flow?
I'm not sure I see the issue here, SPIR-V requires that control flow is uniform on implicit LOD, so compiling to HLSL or any other target should just ... work? HLSL -> SPIR-V sounds like the problem case here if HLSL is more lax than SPIR-V, but that's not SPIRV-Cross' issue.

Hm, the offending shader is here: https://github.com/SaschaWillems/Vulkan/blob/master/data/shaders/parallax/parallax.frag#L73-L80
Compiling from spv -> HLSL gives me shader errrors due to HLSL trying to unroll the loop but fails after 1024 unrolls

Wait, so, HLSL has to assume the worst, and even try to unroll "correct" code even with dynamic loop counter? ... Wow :(
SampleLod(0) is just plain incorrect, so that's not a solution.

right.. I just saw that Code motion section which prohibits implicit derivatives in non-uniform control flow (TIL). Couldn't find a corresponding restriction in the GLSL specification so far beside being undefined, but need to look again.

There is definitely such a restriction in GLSL as well.

Reconfirmed that the generated spv code is illegal https://github.com/KhronosGroup/SPIRV-Headers/issues/58#issuecomment-371019137
The corresponding code sections have been changed https://github.com/SaschaWillems/Vulkan/issues/425
Closing this, thanks!


how to assign class to a textFieldGroup?

The following code not working!
echo $form->textFieldGroup($model, 'Email',array('class'=>'input-group input-group-lg'));
I am not able to assign the class to the above input.
Please if anyone can help?

Collins wants you to check out Tenderpreneur.net
Hi:
I would like to add you to my network on Tenderpreneur.net.
Follow this link to join my network:
http://tenderpreneur.net.
Have a great day!
Explore Tenderpreneur.net and create network of your friends.===============================================
App Uprising, Inc., 20 Jay street, Brooklyn, NY, 11201
Powered by ReadyGraph
https://www.readygraph.comIf you'd like to unsubscribe and stop receiving these emails click here: http://email.readygraph.com/wf/unsubscribe?upn=s5UH8bhnBaopYjthcNSRce1IxazJ5upXwSo1UjuFbGfULCvr0VD9v-2FjRiAQ4yfo-2F8Z6P-2FGddAZqvVvzwce0TPNSliCYeiRIQjqESWEfcZyvWrrh2YFxAzdCNSaAusYUxfhVQGw0sXLURZSezQcUjRQV14rBZ-2FqN7RHqZtIKUDBH6tIEwytCHShKCBkE-2FHqoLpbVV4KzxqeEyDiSI-2F3F51Wg7swisn8gH4Ap2SS16xi6b5-2Bf0w9PgAnQpPG-2FDSeSzbLRM3YH98-2FCIDPWWj1N1GzF6pI1tIkw4t-2FlzlY8-2F3M3lqaM8ycuIoovv3q-2BX-2B6Ca5HRPAmMlmkeYD1-2Bw-2BLRxoA-3D-3D.


[DCM-KIT] Onboarding Video

Description

Introduce an DCM Onboarding Video to be able to have an easy access to the DCM Use Case, especially for new adopters. This will support an initial and better understanding and adaption of DCM in a few minutes (similar to the Catena-X video on the Catena-X Homepage).
Important: There are currently licensing issues for the development of an onboarding video.
The current risk is that the timeline for 24/08 is not met and that, if necessary, only an interim result of an onboarding video can be handed over.
Update @MWADCM Risk is valid. Will not be finished by Rel. 24/08
Impact
Easier access to the DCM use case making the whole idea more touchable especially for new potential adopters who have not read any other DCM content yet. On top the target is to catch the attention of visitors of the DCM KIT without forcing them to read and click through many documents.
Tasks

[ ] Solving license issues concerning Video content/software
[ ] Formal Handover of individual topics from DCM Consortia Group
[ ] Expert Group internal alignment and understanding on Consortia Group handover topics
[ ] Final Preparation of DCM Onboarding (Video) content for DCM KIT
[ ] Initial Pull Request GITHUB until 11.07.2024
[ ] Final Pull Request GITHUB until 25.07.2024
[ ] Publication in DCM KIT on Tractus-X 08/24

Was presented in open planning ⇾ contributors and committer are involved

Hello @MWADCM ,
Since the feature is a 24.08 feature and the development phase is coming to an end, we need a status on the feature.

[ ] Currently you are assigned (Responsible) → Is this correct? If not, please assign the correct contact person
[ ] Please check whether the status (backlog, work in progress ...) is set correctly
[ ] Please check whether there is an assignment to a committee or expert group (supported-by) to be found on the board
[ ] Please comment on the current status of the feature
[ ] Are all SubTasks (issues from other repositories that deal with the feature) linked? → The easiest way is to mention the feature here in the issue (via the ID) so we can see which teams/repositories are involved.

If you need any clarification, please get in touch, thank you very much.
Stephan

Update @MWADCM Risk is valid. Will not be finished by Rel. 24/08

@Grand-Thibault  This was a feature for milestone 24.08. It has the status NOT in Release, which is OK, but is planned with this feature? 24.12? 25.03?

@MWADCM @LukasSchmetz see Stephans comment above

@stephanbcbauer Now further work on this feature planned currently. Therefore we put it on "Not in Release" as there was on close option.

Closed by NSC because no progress. Please reopen and assign an upcoming release if needed.


MM-44685: Render app bar icons on top of RHS background

Summary
When the RHS is open and expanded, it renders a position: absolute div that covers the whole screen. This div handles any click on it to collapse the RHS again. The end result is that the user can click anywhere outside of the RHS to collapse it.
When the App Bar was implemented, this was not taken into account, so such RHS background renders also on top of the App Bar icons, making it impossible to click on them when the RHS is expanded.
There are two simple solutions for this:

Make the App Bar icons to render on top of the background as it is today (this very PR).
Shift the RHS background to the left, so that it does not cover the App Bar. This can be accomplished by aligning it to the right of the RHS with the following change:

--- a/sass/layout/_sidebar-right.scss
+++ b/sass/layout/_sidebar-right.scss
@@ -38,7 +38,7 @@
         position: absolute;
         z-index: 5;
         top: -70px;
-        left: -500%;
+        right: 0;
         // Since the element (sidebar--right__bg) is tied to the sidebar, we need to expand beyond it, we're given an arbitrary large width and left positioning so that it may take the width completely
         width: 1000%;
         height: 110%;

Option 2 has the advantage of being much cleaner, as we don't mess with z-indexes. However, it has a negative consequence: the whole App Bar is no longer covered by that background, so clicking on an empty space of the App Bar does not collapse the RHS. This may be confusing for the end user, who knows that clicking anywhere outside of the RHS collapses it.
That's why I chose option 1. The complexity problem of having to deal with z-indexes is relieved with the comments added.
I could have also implemented another solution: remove the RHS background altogether and detect clicks outside of the RHS by adding a listener to the mousedown event and inspecting it, checking whether the RHS outer container was clicked or not (something like
containerRef.contains(event.target)). This solution was discarded because it drastically changes the original design of the RHS and this commit is being merged just a few days ago from a major release, so minimizing risks is imperative.
Ticket Link
https://mattermost.atlassian.net/browse/MM-44685
Related Pull Requests
--
Screenshots
https://user-images.githubusercontent.com/3924815/171682317-25a2c0e7-f18c-4a4d-ba4d-328212a866a6.mp4
Release Note
NONE

@amyblais, fix for a v7.0 bug.

It would definitely be nice to do this without messing with z-indexes, but I agree that this is the easiest solution for now. I don't think removing the div that triggers this (the third option) would be a good idea though. The whole behaviour of having the RHS collapse if you click outside of it is so bizarre to me, so we'll probably want to revisit that in the future, maybe by having that only happen if you click inside the centre channel instead of anywhere that's not the RHS

Trying to share the load among QA instead of throwing everything to @DHaussermann :)

@saturninoabril, please note that this should be merged before code freeze (tomorrow EOD).

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/203764

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Cherry pick is scheduled.

/cherry-pick cloud

Cherry pick is scheduled.


Keyboard shortcut documentation

Looks like there is a default mapping for "Quick fix" CMD + .. It might be sense to mention this and other keyboard shortcuts in README.

There are no custom shortcuts added. Quick fix shortcut is native to VS Code and it’s different in different environments (windows, mac).


U+012B ( ī ) - new glyph

PR #23

added as of https://github.com/source-foundry/alt-hack/commit/35eb8e5c89d37a5630adc31498e034b99443f348
cc: @dok42


add docs

Codecov Report

Merging #25 (bbe0d7a) into master (53d215a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #25   +/-   ##
=======================================
  Coverage   73.13%   73.13%
=======================================
  Files          11       11
  Lines        1396     1396
=======================================
  Hits         1021     1021
  Misses        375      375

Continue to review full report at Codecov.

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


Parsing problem of APKBUILD when empty line in list of sources

ans@hans-VirtualBox:/tmp/apa$ make abuild
...

musl: Checking sha512sums...
musl-v1.2.3.tar.gz: OK
handle-aux-at_base.patch: OK
relr-1.patch: OK
relr-2.patch: OK
relr-3.patch: OK
relr-4.patch: FAILED
sha512sum: can't open 'relr-4.patch': No such file or directory

source="musl-$_commit.tar.gz::https://git.musl-libc.org/cgit/musl/snapshot/$_commit.tar.gz
handle-aux-at_base.patch
    relr-1.patch
    relr-2.patch
    relr-3.patch
    relr-4.patch

    ldconfig
    __stack_chk_fail_local.c
    getconf.c
    getent.c
    iconv.c
    "

relr-4.patch is missing in parsed data
            {
                "remote": "relr-3.patch",
                "local": "relr-3.patch"
            },
            {
                "remote": "ldconfig",
                "local": "ldconfig"
            },

There is some unit tests in the test directory
bomres/services/sbom_resolver/service/test
python3  ../build/bomres/bomres/lib/create_apkcache.py

The issue is related to mismatch with the git state in APKINDEX and aports repository.
"remote": {
"type": "git",
"commit": {
"hash": "1016b47723",
"tag": "v3.16.3"
},
"url": "git://git.alpinelinux.org/aports",
"path": "main/musl/APKBUILD"
git checkout -b 016b47723   -> lists three entries
In the same resolver.conf another git hash is listed
"repo": {
"main": {
"raw": "v3.16.3-99-g1016b47723",
"tag": "v3.16.3",
"build-number": "99",
"hash": "1016b47723"
},

With better understanding how APKINDEX and aports tags aligns this should not be an issue any more.


update GI to reflect current hive block weight

The community has seen that GI does not show accurate view of profit potential.  This is primarily due to number of hive blocks consistently at 33.3% instead of 50.0% as originally planned.  The community has created a "Honey" index that is based on this new index to accurately show potential profits in hives.  I am suggesting import of this index as well as the future index (based on number of current immature bees that will mature) for users to make more accurate decisions when purchasing.  It will also cause GI to not inflate as much and people to not understand why they are not profiting when GI says they will.  I understand this can take time to analyze and potential other reasons, but being on the 3rd wave of bee purchases I don't think this will even out unless it is changed in the wallet.  This will also allow community members to grow and be able to understand profits without having to go to community for explanation that GI is more like gambling at this point.   It is my understanding that hive to PoW block ratio will not change based on the time it takes for PoW blocks to come in versus consensus of hive blocks, and it has been consistent since hive activation on mainnet.

Don't change nomenclature pointlessly -- too many knock-on effects. Just revise GI down to the new expectation.

Consensus is that this is a bug and changing the GI won't fix the problem just hide it, utilize other community indexes until a resolution is found (very close) 👍

Forcing POW block would require idle POW so GI should be modified as well as honey pot like in my original requset.

fixed in 16.0.5 :)


Add Storm SQL docs to index page for 1.x branch

Rearrange a bit and add documentations about some missing Storm SQL.

It also needs to be addressed for 1.1.0, but since release vote is ongoing, I'll handle it manually if vote passes and site doc for 1.1.0 is up.

Since #1955 is superset of this, and it contains comments which can be also applied to this, I'll update this as well, and merge.


(feature) Ability to set custom path to APK/AAB file

use case: Our build system creates APK file in its own "non-standard" path, and the plugin expects the apk in its "usual" place outputs/apk/release.
It would be nice to have the ability to specify apk path as a command line parameter.

--buildFile

Thank, problem solved


"Add Video" means "Add video device"

in v3.3 the "Add Video" menu option actually means "add an additional video input" (add an additional webcam). But "add video" apparently confuses users that are looking for the still missinng ability to send a recorded video file streaming... "play this video recording".
You should rename the option to "Add new video input"
Further issue: you can select an infinite number of times the same video input device (webcam)... and send the same image many many times to all recipients "video-spamming" your image. This should not be allowed (to add a video input which is already added) because it could cause a security attack issue by a participant which disrupt a call by sendingg his image multiple times.

English and Hungarian translation is updated on develop branch.
I'm looking into the camera duplication issue.

@so010 , @havfo , @bidzik  please update the translation on your language too. I want to close this issue after it! Thanks!

Done regarding project 3.5


Feature/include merged refs

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


Are there any plans to add later .NET Core runtime support?

Hi,
I was wondering if there would be  NET Core 2.1 runtime support? I am in the evaluation phase and  just wondering how much support .NET will be getting. The NuGet package for instance was last updated in April 2018:

and with the formatting and lack of error messages coming out when using .NET core (see https://github.com/kubeless/kubeless/issues/941) I just wanted to be sure there would be some help for any .NET issues I run into.

Hi,
I think there are not any short term plan to add .NET 2.1 (pinging @allantargino in case he plans to do that as maintainer).
Regarding issues that you may find, you can open issues here or try to get help in the #kubeless channel in the K8s slack.

@allantargino if you are going to spend any more time it may as well be for .NET Core 2.2 when it comes out.

Hi @natiki!
For this NuGet package you shouldn't expect much changes over time - since it only holds a simple contract the entities Event and Context - which is basically this project: https://github.com/kubeless/kubeless/tree/master/docker/runtime/dotnetcore-2.0/src/Kubeless.Functions
From the other side, the Runtime itself, is being updated in the last weeks. I'm just about to release an update to .NET Core 2.1. (You can check the branch here (and fell free to send contributions!): https://github.com/allantargino/kubeless/tree/dotnetcore21
Today, I'm focusing on refactoring the runtime core to reflect the compile feature that kubeless implements. As soon as it is stable (I'm start adding unit tests now - expect a PR this week), we can move on features like .NET Core 2.2 and also #927.
Thanks for testing it out and helping us make it better!

@natiki - .NET Core 2.1 is here :)
@andresmgot - Can you please close this issue?
Thanks

Sure! Thanks @allantargino!

@allantargino I will test and see if I can close #941 once @andresmgot makes the next release available as it has the namespace support in it?


Add CSV/JSON output for raw samples

Added --csv-file-raw flag, which outputs a CSV file containing all raw sample measurements in milliseconds. Columns are benchmarks, rows are samples. First row is a header column with benchmark names. For example:
foo, bar, baz
1.2, 5.5, 9.4
1.8, 5.6, 9.1
1.3, 5.2, 9.8

Added samples output to the existing JSON file output, containing the same raw sample measurements in milliseconds.

Prepare to release v0.4.20.

Fixes https://github.com/Polymer/tachometer/issues/162

Oh interesting... Is the idea that someone could use csv-raw or json samples to do custom stats on the data?

Oh interesting... Is the idea that someone could use csv-raw or json samples to do custom stats on the data?

Yeah exactly, we're interested in doing some analysis/visualization of the latency distributions for lit-html, and rather than trying to build more analysis into tachometer, it makes more sense to dump out the raw data and look at it in R or Sheets etc. Seems like a useful escape-hatch for other people who want to do further analysis too.


What is the input_size

[Enter feedback here]
snipet code require create dummy input with argument "input_size". What is it, how i can find out ?

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

ID: b236fa85-23be-f5ec-5c5f-332937187f04
Version Independent ID: 781552b8-a796-a963-05d3-ed98f509dbd4
Content: Convert your PyTorch training model to ONNX
Content Source: docs/windows-ml/tutorials/pytorch-convert-model.md
Product: machine-learning
GitHub Login: @QuinnRadich
Microsoft Alias: quradic

Replace dummy_input = torch.randn(1, input_size, requires_grad=True)  with dummy_input = torch.randn(1, 3, 32, 32, requires_grad=True)
torch.randn is used to create a randomized tensor to feed into the model. The first parameter is the batch size, it is set to 1 here but I believe it can be set to other values as well. The second parameter is number of channels. In this tutorial, the first convolutional neural network layer has 3 channels, so it is expecting an input tensor to have the same value, and thus it is set to 3.
Lastly, the last two parameters are height and width. Since it is using images in the CIFAR10 dataset, the height and width are both 32.
If you then run the Convert_ONNX function, you should get the proper output.


[@azure/search-documents] span.context is not a function

Package Name: @azure/search-documents
Package Version: 11.2.0
Operating system: Windows
[x] nodejs

version: 14.17.4

[x] browser

name/version: Microsoft Edge Version 92.0.902.78 (Official build) (64-bit)

[x] typescript

version<EMAIL_ADDRESS>

Is the bug related to documentation in

[ ] README.md
[x] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
import { SearchClient } from '@azure/search-documents';
...
const searchResults = await searchClient.search('*', searchOptions);

Will throw an exception as follows:
(node:28376) UnhandledPromiseRejectionWarning: TypeError: span.context is not a function
at TracingPolicy.<anonymous> (c:\repo\project\src\node_modules\@azure\core-http\src\policies\tracingPolicy.ts:79:32)
at step (c:\repo\project\src\node_modules\@azure\core-http\node_modules\tslib\tslib.js:143:27)
at Object.next (c:\repo\project\src\node_modules\@azure\core-http\node_modules\tslib\tslib.js:124:57)
at c:\repo\project\src\node_modules\@azure\core-http\node_modules\tslib\tslib.js:117:75
at new Promise (<anonymous>)
at Object.__awaiter (c:\repo\project\src\node_modules\@azure\core-http\node_modules\tslib\tslib.js:113:16)
at TracingPolicy.sendRequest (c:\repo\project\src\node_modules\@azure\core-http\dist\index.js:4430:22)
at ProxyPolicy.sendRequest (c:\repo\project\src\node_modules\@azure\core-http\src\policies\proxyPolicy.ts:158:29)
at OdataMetadataPolicy.<anonymous> (c:\repo\project\src\node_modules\@azure\search-documents\src\odataMetadataPolicy.ts:47:29)
at Generator.next (<anonymous>)
at c:\repo\project\src\node_modules\tslib\tslib.js:117:75
at new Promise (<anonymous>)
at Object.__awaiter (c:\repo\project\src\node_modules\tslib\tslib.js:113:16)
at OdataMetadataPolicy.sendRequest (c:\repo\project\src\node_modules\@azure\search-documents\dist\index.js:1871:22)
at SearchServiceClient.ServiceClient.sendRequest (c:\repo\project\src\node_modules\@azure\core-http\src\serviceClient.ts:303:25)
at SearchServiceClient.<anonymous> (c:\repo\project\src\node_modules\@azure\core-http\src\serviceClient.ts:518:34)
at step (c:\repo\project\src\node_modules\@azure\core-http\node_modules\tslib\tslib.js:143:27)
at Object.next (c:\repo\project\src\node_modules\@azure\core-http\node_modules\tslib\tslib.js:124:57)
at c:\repo\project\src\node_modules\@azure\core-http\node_modules\tslib\tslib.js:117:75
at new Promise (<anonymous>)
at Object.__awaiter (c:\repo\project\src\node_modules\@azure\core-http\node_modules\tslib\tslib.js:113:16)
at SearchServiceClient.ServiceClient.sendOperationRequest (c:\repo\project\src\node_modules\@azure\core-http\dist\index.js:4704:22)

To Reproduce
Steps to reproduce the behavior:

Trying to repro in a smaller project was actually not possible, the small project worked as expected.
Here is my current nodejs endpoint for reference
Must be related to yarn.lock dependencies that are different from current node projet and smaller repro projet

import { FastifyPluginAsync } from 'fastify';
import S from 'fluent-json-schema';
import _ from 'lodash';
import { ISearchedAssetReply, SearchedAsset } from '../common/src/types';

import { SearchIndexClient, SearchClient, AzureKeyCredential, odata } from '@azure/search-documents';
import { SearchOptions } from '@azure/search-documents/types/search-documents';

interface IQuerystring {
  searchQuery: string;
}

const schema = {
  querystring: S.object().prop('searchQuery', S.string().required()),
};

// Getting endpoint and apiKey from .env file
const endpoint = '...';
const apiKey = '...';

export const handler: FastifyPluginAsync = async (fastify) => {
  fastify.get<{
    Querystring: IQuerystring;
    Reply: ISearchedAssetReply | string;
  }>('/api/search/assets2', { schema }, async (request, reply) => {
    let query = request.query;

    const indexName = 'assets';
    const searchClient = new SearchClient<SearchedAsset>(endpoint, indexName, new AzureKeyCredential(apiKey));

    let searchOptions: SearchOptions<keyof SearchedAsset> = {
      top: 10,
    };

    try {
      let searchResults = await searchClient.search('*', searchOptions);

      let totalCount = searchResults.count as number;
      const assets = searchResults.results; // FIXME async search iterator

      fastify.log.info(`Search assets for '${query.searchQuery}': assets.length=${assets}`);

      reply.status(200).send({
        assets,
        totalCount,
      });
    } catch (ex) {
      reply.status(400).send(ex);
    }
  });
};

Expected behavior
Should be able to get the searchResults object from searchClient.search() without any exception
Additional context
Unblocking solution: By using dev version @azure/search-documents: 1.3.0-alpha.20210816.1 the issue is gone !

Thanks for the report @Ludovic71!
@maorleger Could this be because 11.2.0 of the search package is depending on 1.0.0-preview.11 of core-tracing? We ensured all of the other packages are on atleast 1.0.0-preview.12 or above.

It does appear to be the case with 11.2.0 - it still depends on core-tracing preview.11 and core-http 1.2.6
I will work on getting a release out there as soon as possible.

@Ludovic71<EMAIL_ADDRESS>has been released which brings it up to<EMAIL_ADDRESS>- please let me know if you continue to see that same issue.


Reloadable config

If the config is loaded from a file - or from a K8S ConfigMap - changes to the config should reconfigure the SDK if possible.
Some fields may not be re-configurable - but at least 'disabled' and trace ratio should be.
I don't know what are the plans for the logging config: but if something similar to traces is added to allow sampling and/or
customization of log level - this should also be part of the minimal set of reconfigurable settings.
Env variables obviously require restarting the app to change the trace sampling rate - which doesn't work very well.

This is probably better suited as a spec level issue, since this repository is concerned with the schema of the configuration. The expectations for SDKs with respect to file configuration are expected  in the specification file configuration document.
I'm transferring this issue to the spec repository.
But some thoughts on this specific request. Long term, we want SDKs to be able to update their configuration. This is aligned with the goals of OpAMP, which aims to one day allow SDKs to connect to servers for remote management. The current thinking is that the file configuration scheme will be used to encode the desired SDK configuration sent from servers to clients. The idea you're discussing is similar, but instead of from a remote server, the ConfigMap can be changed and the SDK expected to be reconfigured accordingly.
I think we will eventually get there, but there are a number of non-trivial milestones we need to accomplish first. We need to nail down the basic primitive API operations of file configuration, iterate and stabilize the schema, figure out how to configure SDK extension components, figure out environment variable substitution (and if / how file configuration interacts with the environment variable schema). If you're interested in helping, any hand you can give prototyping and working through the basics will help expedite the timetable for the more complicated interactions like updating config.


refactor: clean out variables that are being assigned to but not being read

Clears out private properties and local variables that are declared and assigned values, but aren't actually being read at any point. This will start being caught by tslint once we update to TS 2.6.

tslint will start catching them when we update to TS 2.6 @mmalerba. I didn't bump to TS 2.6 in this PR, because there are some build issues that we'll need to sort out.

@crisbeto please rebase

Updating to cla: yes as the non-cla'd commit was done by me via github.

Rebased @josephperrott.


JSX: Convert encoded HTML entities in attributes into UTF-8

From https://github.com/Microsoft/TypeScript/issues/10492
Encountered another difference with babel regarding encoded HTML in attributes:
TypeScript Version:  2.0.0
Code
function MyComponent(props) {
    return (
{props.myAttr}
);
}

elsewhere:

<MyComponent myAttr="more…" />

Expected behavior:
Outputs the html

more…

Actual behavior:
Outputs the html

more&hellip;

Make sure that &# and &#x works too!
Be careful of the following subtlety
<MyComponent myAttr="This is &# encoded, like in HTML. \ has no effect" />
<MyComponent myAttr={"This is \\u encoded, like in JavaScript. & has no effect"} />

Fixed by #10747


[ Wait for #2578 ] [ hgemm ] Support beta case

This commit allows hgemm to get beta condition as well.
Note that beta for here is as follow:
C = alpha * A * B + beta * C
In addition add zero-init code for beta = 0.F case. According to recent model profiling result, even for initialization, minimizing instruction is quite helpful more overall model latency reduction.

Self evaluation:

Build test:     [X]Passed [ ]Failed [ ]Skipped
Run test:     [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: skykongkong8<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2585. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


feat: various Asset Locks improvement

What was done?

remove dependency of Asset Lock txes on CCreditPool
new case for functional tests of Asset Locks - more than one output for Asset Lock tx.

How Has This Been Tested?
Run unit/functional tests
Breaking Changes
Slightly changes behaviour of TxMempool. Tx can be accepted in mempool even if Asset Unlock transaction with same index is already mined. But final consensus rules are same.
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation
[x] I have assigned this pull request to a milestone

Tx can be accepted in mempool even if Asset Unlock transaction with same index is already mined.

what happens if you invalidate the block with the original tx (both the original and the duplicate one should be in mempool at that point, right?) and try to mine another block after that?

what happens if you invalidate the block with the original tx (both the original and the duplicate one should be in mempool at that point, right?) and try to mine another block after that?

yes, both of them can be in mempool, right.
Only one of them can be mined in block.
They are validated agains creditPoolDiff here:
            if (!creditPoolDiff->ProcessTransaction(iter->GetTx(), state)) {
                if (fUsingModified) {
                    mapModifiedTx.get<ancestor_score>().erase(modit);
                    failedTx.insert(iter);
                }
                LogPrintf("%s: asset-locks tx %s skipped due %s\n",
                          __func__, iter->GetTx().GetHash().ToString(), state.ToString());
                continue;
            }

at first look it may looks like transaction can be accepted, because:
if (!CheckAssetLockUnlockTx(tx, pindex, pool.indexes, state)) {

CheckAssetLockUnlockTx validates only against indexes from previous block.
But it is not true, because it's just simple basic validation of transaction. Proper validation happens on next step, when validated both "transaction index" and "credit pool unlock limits":
CCreditPoolDiff::Unlock:
    if (sessionUnlocked + toUnlock > pool.currentLimit) {
        return state.Invalid(TxValidationResult::TX_CONSENSUS, "failed-creditpool-unlock-too-much");
    }

    if (pool.indexes.Contains(index) || newIndexes.count(index)) {
        return state.Invalid(TxValidationResult::TX_CONSENSUS, "failed-creditpool-unlock-duplicated-index");
    }

So, index is tested against both list: list of indexes that have been ever used since Tip() and list of indexes that are freshly introduced in just this block.
Unfortunately, I can't add a functional test for it because:

same transaction will have same hash -> refused to be added in mempool
no quorum can sign transaction with same index but with different hash -> impossible to get 2nd tx with same index.
And because at least one of this tx is invalid it can't be mined - not because same index, but by other reasons (wrong signature or same hash).

Any ideas how to test it otherwise?


PERF: parse and timedelta ops improvements, #6755

closes #6755
-------------------------------------------------------------------------------
Test name                                    | head[ms] | base[ms] |  ratio   |
-------------------------------------------------------------------------------
dtype_infer_timedelta64_1                    |   7.0527 | 119.7317 |   0.0589 |
timedelta_convert_string                     |  13.3094 | 108.9200 |   0.1222 |
timedelta_convert_string_seconds             |  18.2626 |  85.0617 |   0.2147 |
dtype_infer_timedelta64_2                    |   9.4047 |   9.4063 |   0.9998 |
timedelta_convert_int                        |   0.1300 |   0.1260 |   1.0315 |
-------------------------------------------------------------------------------
Test name                                    | head[ms] | base[ms] |  ratio   |
-------------------------------------------------------------------------------

Ratio < 1.0 means the target commit is faster then the baseline.
Seed used: 1234

Target [da92dc0] : PERF: timedelta and datetime64 ops improvements
Base   [d8a2f30] : Check for size=0 before setting item
Fixes #10193

This is a cython conversion of the timedelta parsing routines. Was much more involved that it looked, as the original was using regexes, this is a string parsing state machine in cython.
However, I think this could actually be done in c (or maybe using char * pointers in cython), rather than the list build up. This could be another 10x perf improvement on this (my benchmark is the c parser for ISO datetimes).
if any of the c gurus are interested, pls comment!


Update to Sarama 1.31.1

Fixes #

Proposed Changes

updates to latest Sarama client lib for the control plane

Release Note

Control Plane is using Sarama 1.31.1

Docs

@pierDipi I forgot to run the gofmt :see_no_evil:
I've upodated the PR now

/test pull-knative-sandbox-eventing-kafka-broker-unit-tests


Support for 1 channel Inching/Self-locking Switch 5V 12V

Hello!
Thank you for your great work - integration works fantastic! The only problem I have is support for https://ewelink.coolkit.cc/?p=143 1 channel Inching/Self-locking Switch 5V 12V. I can't get entity of this device.
Is this device supported? How can I integrate it?
Thank You!

It should be supported. Show debug logs.

Yes, but please let me know how can I filter log events - is this the right way:
logger:
default: debug
logs:
custom_components.sonoff: debug
Or do you want complete log?

Check debug mode in readme

Here it is. Problematic device is:
Device ID: 100002ca68
FW Name: PSA-B01-GL
In log: 2020-12-12 10:33:44  INFO     sonoff_local  100002ca68 <= Local1 | No devicekey for device
Log.txt

Actually, this is a problem:
DEBUG    sonoff_cloud  8 devices loaded from the Cloud Server
It should load 9 devices, 9th is missing in HA.
When trigger device, this INFO happens:
INFO     sonoff_local  100002ca68 <= Local1 | No devicekey for device
Also find out:
DEBUG    sonoff_local  Zeroconf Removed: eWeLink_100002ca68._ewelink._tcp.local.
If it is useful: Alex can control this device; device is connected via WiFi range extender (no NAT, getting IP from the same DHCP server). I can PING it. Device is shred with another user. LAN control doesn't change anything.

Does anyone have an idea, what is the meaning of No devicekey for device message?
So far I have found out:

Local or Cloud mode: no effect, same error all the time
file .sonoff.json has definitions for all devices except for the missing one
I have devicekey but I have no idea how can I force HA to detect the device?

Does anyone have an idea, what is the meaning of No devicekey for device message?
So far I have found out:

Local or Cloud mode: no effect, same error all the time
file .sonoff.json has definitions for all devices except for the missing one
I have devicekey but I have no idea how can I force HA to detect the device?

Maybe device have been added to other ewelink account?

Maybe device have been added to other ewelink account?

Of course ... many thanks for magic word "other ewelink account"!
Since transition from Peter's project to your's, I have connect N1 in other network segment. I have blamed this change, but in fact I just forget to share N1 device from phone's account to HA's account.
Thank You again!

Of course ... many thanks for magic word "other ewelink account"!
Since transition from Peter's project to your's, I have connect N1 in other network segment. I have blamed this change, but in fact I just forget to share N1 device from phone's account to HA's account.
Thank You again!


[FLINK-29229][hive] Fix ObjectStore leak when different users has dif…

…ferent config

What is the purpose of the change
The connection may leak when users have different config. Here, we fix this like hive does to use the same config to create the HiveMetaStoreClient.

Thanks for your review. Because the comment is not releated to the code, I will this offline.

The failed test is not releated to the fix. Merging...


Solo.getCurrentViews

From<EMAIL_ADDRESS>on September 13, 2012 06:46:21
What steps will reproduce the problem? 1. It happens from time to time What is the expected output? What do you see instead? expected output: get current views.
actual: Exception What version of the product are you using? On what operating system? 3.4.1.
Android 2.3.3
02:08:45: Result code = Success     Result parameters = VERSION.CODENAME = REL
VERSION.RELEASE = 2.3.3
VERSION.INCREMENTAL = JHKG2
VERSION.SDK_INT = 10
BOARD = GT-I9100
BOOTLOADER = unknown
BRAND = samsung
CPU_ABI = armeabi-v7a
CPU_ABI2 = armeabi
DEVICE = GT-I9100
DISPLAY = GINGERBREAD.JHKG2
FINGERPRINT = samsung/GT-I9100/GT-I9100:2.3.3/GINGERBREAD/JHKG2:user/release-keys
HARDWARE = smdkc210
HOST = DELL84
ID = GINGERBREAD
MANUFACTURER = samsung
MODEL = GT-I9100
PRODUCT = GT-I9100
SERIAL = unknown
TAGS = release-keys
TIME =<PHONE_NUMBER>000
TYPE = user
USER = root Please provide any additional information below. java.lang.NullPointerException: null: com.jayway.android.robotium.solo.ViewFetcher.getNonDecorViews(ViewFetcher.java:169)
com.jayway.android.robotium.solo.ViewFetcher.getAllViews(ViewFetcher.java:85)
com.jayway.android.robotium.solo.ViewFetcher.getViews(ViewFetcher.java:194)
com.jayway.android.robotium.solo.Solo.getCurrentViews(Solo.java:1534)
I have ddms log of this exact moment. I can't find anything related to solo there.
Original issue: http://code.google.com/p/robotium/issues/detail?id=327

From<EMAIL_ADDRESS>on September 13, 2012 07:18:45
Thanks for reporting this. It will be fixed in the next release.
Status: Accepted
Owner<EMAIL_ADDRESS>

From<EMAIL_ADDRESS>on October 10, 2012 22:11:50
Thanks again. This has been fixed in Robotium 3.5.
Status: Verified


pythonPackages.neovim: 0.2.4 -> 0.2.5 & gem-neovim: 0.6.2 -> 0.7.0

Motivation for this change
In neovim, :CheckHealth:
WARNING: Latest nvim-python-neovim is NOT installed: 0.2.5
WARNING: Gem "neovim" is out-of-date. Installed: 0.6.2, latest: 0.7.0

Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Can someone explains me how to get around this:
ERROR: Command error (job=16): /nix/store/3akdg6ln41i8p8pl73jv5m59fp60qvp7-neovim/bin/nvim-ruby --version
    Output: /nix/store/bn1g8vrxx1sxkx641xxsdhk8sjbf71rk-ruby-2.4.3/lib/ruby/site_ruby/2.4.0/bundler/spec_set.rb:88:in `block in materialize': Could not find neovim-0.6.2 in any of the sources
     (Bundler::GemNotFound)      from /nix/store/bn1g8vrxx1sxkx641xxsdhk8sjbf71rk-ruby-2.4.3/lib/ruby/site_ruby/2.4.0/bundler/spec_set.rb:82:in `map!'   from /nix/store/bn1g8vrxx1sxkx641xxsdh
    k8sjbf71rk-ruby-2.4.3/lib/ruby/site_ruby/2.4.0/bundler/spec_set.rb:82:in `materialize'        from /nix/store/bn1g8vrxx1sxkx641xxsdhk8sjbf71rk-ruby-2.4.3/lib/ruby/site_ruby/2.4.0/bundler/
    definition.rb:170:in `specs'   from /nix/store/bn1g8vrxx1sxkx641xxsdhk8sjbf71rk-ruby-2.4.3/lib/ruby/site_ruby/2.4.0/bundler/definition.rb:237:in `specs_for'   from /nix/store/bn1g8vrxx1sx
    kx641xxsdhk8sjbf71rk-ruby-2.4.3/lib/ruby/site_ruby/2.4.0/bundler/runtime.rb:20:in `setup'       from /nix/store/bn1g8vrxx1sxkx641xxsdhk8sjbf71rk-ruby-2.4.3/lib/ruby/site_ruby/2.4.0/bundle
    r.rb:109:in `setup'      from /nix/store/3akdg6ln41i8p8pl73jv5m59fp60qvp7-neovim/bin/nvim-ruby:16:in `<main>'
  - ERROR: Failed to run: /nix/store/3akdg6ln41i8p8pl73jv5m59fp60qvp7-neovim/bin/nvim-ruby --version

Well, I solved this by manually editing the Gemfile.lock. Why do I need to edit it manually?

@GrahamcOfBorg build neovim


Replace StringBuffer with StringBuilder

And also a few other minor code cleanups.

Codacy just finds "new" issues in old code, so... can ignore that for this PR.


Amalgamate sources into a single .cc file when bootstrapping

This PR needs some work still, but I wanted to get feedback before polishing/fixing it.
The ultimate goal here is to lower the barrier to entry to using Ninja. You should be able to just drop configure.py and ninja-bootstrap.cc into your project, and now your project can use Ninja without worrying about whether end-users have installed Ninja or not (or have the right version). You bootstrap in this case using simply: ./configure.py --bootstrap_from_amalgamation.
To reduce the number of code-paths/variations, I made regular bootstrapping build from the amalgamation also. And to make things more principled/consistent, ninja.bootstrap is always built from ninja-bootstrap.cc and ninja is always built in the regular way, instead of spawning a ninja that will attempt to overwrite itself.
As a side benefit, this makes bootstrapping faster (since it's only compiling one file):
Right now:
$ time ./configure.py --bootstrap
bootstrapping ninja...
wrote build.ninja.
bootstrap complete.  rebuilding...
[27/27] LINK ninja

real    0m10.444s
user    0m20.639s
sys     0m1.579s

With my change:
$ time ./configure.py --bootstrap
bootstrapping ninja...
wrote build.ninja.
bootstrap complete.  rebuilding...
[26/26] LINK ninja

real    0m6.364s
user    0m16.623s
sys     0m1.148s

The --bootstrap_from_amalgamation flag was added for the case where you want to bootstrap directly from the amalgamation (the case I described above where you want to use Ninja in your project).

I fixed the Python indentation.

PTAL -- let me know what you think about the flag.
If you prefer to check in ninja-bootstrap.cc and update it on every checkin, I can try to make that work. It seems like it would be more fiddly though, since ninja-bootstrap.cc would be both a source and an artifact within the Ninja repo. You'd probably want an explicit "regenerate" operation to update it, and a Travis test that it is up-to-date.

FYI the main thing I still have to fix here is: the amalgamation needs to include all .cc files for all platforms, since the amalgamation should build on all platforms without needing to regenerate it. This will take a little bit of #ifdef, but I think it should be pretty straightforward.  If you are ok with the approach, I'll make it robust across platforms.

FWIW I don't understand the motivation for this change. It seems to make things more complicated in more ways than it makes things simpler.

I tried to explain the motivation of this change in my first message. I do personally think this can simplify things a bit, but that is not the primary goal. The primary goal is to make Ninja easier to embed directly into source trees so the user doesn't need to install Ninja themselves manually.
If the change doesn't make sense to you though, I'll just close the PR and use Ninja as a Git submodule through the regular bootstrapping process.

(Though perhaps I would create a repo that also has a statically-build Windows binary to cover that case, since Python is not preinstalled on that platform).


Bring back fortresses

The Fortress class and related classes are currently untested since game states were implemented.

New commit just brought them back! However, they don't do anything at the moment. All that's left to do is implement the defense bonus that the fortress gives. Because it's not done yet, I will leave this issue open.


[BUG] TypeError: 'NoneType' object not iterable

Before submitting a bug report...

[X] This bug wasn't already reported.
(I have checked every bug report on GitHub)

Describe the bug
After the undetected chrome update the program does not open chrome and just crashes.
Copy and paste your error

Screenshots
The chrome window does not open
Value of dashboard variable
Program does not get that far

Did you used a proxy ?

No I did not

I've been getting this problem too, also with the same error messages as OP.

I have the same problem after updating

Would someone of you copy and paste here the error when you replace lines 20 to 24 in main.py from
        try:
            executeBot(currentAccount)
        except Exception as e:
            logging.error(f"{e.__class__.__name__}: {e}")

to just
        executeBot(currentAccount)

2023-08-12 15:13:25,280 [INFO] Using default request storage Traceback (most recent call last):   File "C:\Users\Downloads\temp_farmer\Microsoft-Rewards-Farmer\main.py", line 144, in <module>     main()   File "C:\Users\Downloads\temp_farmer\Microsoft-Rewards-Farmer\main.py", line 20, in main     executeBot(currentAccount)   File "C:\Users\Downloads\temp_farmer\Microsoft-Rewards-Farmer\main.py", line 105, in executeBot     with Browser(          ^^^^^^^^   File "C:\Users\Downloads\temp_farmer\Microsoft-Rewards-Farmer\src\browser.py", line 41, in __init__     self.webdriver = self.browserSetup()                      ^^^^^^^^^^^^^^^^^^^   File "C:\Users\Downloads\temp_farmer\Microsoft-Rewards-Farmer\src\browser.py", line 80, in browserSetup     driver = webdriver.Chrome(              ^^^^^^^^^^^^^^^^^   File "C:\Users\AppData\Roaming\Python\Python311\site-packages\seleniumwire\undetected_chromedriver\webdriver.py", line 32, in __init__     config = self._setup_backend(seleniumwire_options)              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\AppData\Roaming\Python\Python311\site-packages\seleniumwire\webdriver.py", line 41, in _setup_backend     self.backend = backend.create(                    ^^^^^^^^^^^^^^^   File "C:\Users\AppData\Roaming\Python\Python311\site-packages\seleniumwire\backend.py", line 24, in create     backend = MitmProxy(addr, port, options)               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\AppData\Roaming\Python\Python311\site-packages\seleniumwire\server.py", line 56, in __init__     **build_proxy_args(get_upstream_proxy(self.options)),                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\AppData\Roaming\Python\Python311\site-packages\seleniumwire\utils.py", line 57, in get_upstream_proxy     merged.update(proxy_options) TypeError: 'NoneType' object is not iterable Exception ignored in: <function Chrome.__del__ at 0x00000264327ACD60> Traceback (most recent call last):   File "C:\Python311\Lib\site-packages\undetected_chromedriver\__init__.py", line 800, in __del__     self.quit()   File "C:\Users\AppData\Roaming\Python\Python311\site-packages\seleniumwire\webdriver.py", line 67, in quit     self.backend.shutdown()     ^^^^^^^^^^^^   File "C:\Python311\Lib\site-packages\undetected_chromedriver\__init__.py", line 763, in __getattribute__     return super().__getattribute__(item)            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ AttributeError: 'Chrome' object has no attribute 'backend'
this is what i got

Just fixed it ! (that was an error from myself, very easy to correct)


Option to disable pagination when results are on single page

Would be great to have an option to don't show the pagination when there is only 1 page with results.

I'm not sure what you mean by this. If you're suggesting that we should return the raw result set rather than a Scrivener.Page if the result is a single page, that's not something I'll consider. We want consistent return values from the paginate call.

I meant when I do
<%= pagination_links @page, hide_when_single_page: true %>

it would render nothing.

I moved this issue to the proper repo https://github.com/mgwidmann/scrivener_html/issues/65


Azure CLI example to get id by providing the id makes no sense

There is an example shown on this page for how to get the object id of a service principal using the azure cli.
It's written as:
az ad sp show --id XXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX --query id --out tsv

This makes no sense. The X's are an admission to the fact you have to supply the object id here, in order to get the object id..
I am searching for documentation because having used the azure cli to create a service principal like so:
$servicePrincipalName="TestDeleteMe"
az ad sp create-for-rbac -n $servicePrincipalName

I am now having a hard time understanding how I then obtain it's object id after creation. The information returned from the above call does not include the object id.

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

ID: 5ee09f0a-cf58-5592-29d8-7c86113eb304
Version Independent ID: 87f2d2f4-7f4d-1e50-c3ee-1e73f34be216
Content: Find identity object IDs for authentication - Azure API for FHIR
Content Source: articles/healthcare-apis/azure-api-for-fhir/find-identity-object-ids.md
Service: healthcare-apis
Sub-service: fhir
GitHub Login: @EXPEkesheth
Microsoft Alias: kesheth

@dazinator
Thanks for your feedback! We will investigate and update as appropriate.

@dazinator  and @SaurabhSharma-MSFT  thanks for reporting the issue. Sharing the documentation for az-ad-sp-show https://learn.microsoft.com/en-us/cli/azure/ad/sp?view=azure-cli-latest#az-ad-sp-show. This command provides object id once you provide application id.
If not action needed, we can close the ask.

#please-close


Dependency graph

I generated this dependency graph while trying to get an idea of how different functions were connected.
Thought I'd share it in case anyone else finds it useful, even though some parts are congested.
It currently ignores local functions because it's possible for different local functions to have the same name, and that's too much of a hassle to sort out.

Cool!! Is that made with a tool on MATLAB? What version are you using?

We could probably add that to the wiki, and updated it with release versions. @agahkarakuzu what do you think?

It's using GraphViz to make the figure, but everything is done through a MATLAB function.
It's based on M2HTML which I've modified for documenting my own code.
The practical application is to have something simpler, like the figure below, on a webpage where clicking on any of the function names will open a new page showing the function description and source code.

@mathieuboudreau we can use Sphinx to automatically generate this. We can actually dedicate a Developer section for this on ReadTheDocs.
On the other hand, semantics of the above graph is a bit puzzling. Yet, it is still useful. For example, vfa_t1 does not point to checkanteriorver, reminded me to ask you about incrementing from 2.0.13 to 2.0.14 after  #314. Given that voxelwise property has changed and also the implementation.

reminded me to ask you about incrementing from 2.0.13 to 2.0.14 after #314. Given that voxelwise property has changed and also the implementation.

That version numbering is only for patches (we really need to consolidate this, or rename that version number). Even if voxelwise changed, because the Travis test and manual GUI tests did not fail, no patch was needed, so no need to increment that version number or implement a patch (which is where checkanteriorover would get called).

I'll close the issue since it was only meant to share the graph.
I was hoping it would look like an umbrella.
It kinda does if you look at it sideways, I guess.


Adapt WebSocketHandler to  integrate with a starlette WebSocket route

As discussed on https://github.com/kytos-ng/kytos/discussions/337, was partly introduced and implemented to export some logs to the UI web console with socket.io, this was partly functional, but since flask-socketio will be removed, then WebSocketHandler needs to be adapted to use a  starlette WebSocket route, we'll likely need a buffer queue too.
I was going to take care of this in a single PR, but since this was only partly working, I'll leave this for a subsequent PR, it's still scheduled for 2023.1 though, so prioritizing this way, at least the primary PR unblocking starlette can land with the major ones, and then I'll get back to this.
Related ui issue https://github.com/kytos-ng/ui/issues/42 depends on this one here

As mentioned on https://github.com/kytos-ng/ui/pull/44, the ui part turned out to require much more work than anticipated to be production grade https://github.com/kytos-ng/ui/issues/42, this part specifically will be defered to a future release.


[report] Add plain-text output of k8s resource queries

Description
I've recently updated the report command to generate the cluster dump as provided by kubectl:
kubectl cluster-info dump --output-directory=/path/to/output/dir --output=json

which dumps debugging information about all resources (incl. container logs). The change affected the usability of logs as plain-text resource descriptions (pods/nodes/events etc.) are also helpful.
Additionally, with the move to logrange, report is no longer capturing the collected logs.
Suggestion

Revert the changes to continue dumping resources descriptions in plain-text as before (alongside the cluster-info dump).
Capture the logs as collected on the cluster and include in the debug report.

Closed by https://github.com/gravitational/gravity/issues/1264.


RSK wallet needs access to generic 44' path.

The RSK wallet needs access to other paths, I believe this is the way to enable the generic 44' path in the application upload script.

Can you be more specific in the path required ? Please link your source.

Can you be more specific in the path required ? Please link your source.

After some deliberations, we can use a smaller path, so I will make a new PR. What do you mean by "link your source" ?


Add server connection port to client structure and return in INFO requests

Currently the "port" returned in INFO CONNECTIONS results is the remote port of the client, which is not particularly useful.  The server port is determined in client.c and should be stored and used in the connection results.

Add a uint16_t serverport field of ClientInfo and set it to the server port value
Return this value in the connection info results

Actually, the remote port is used to complete the "address" of the client.
The server port used by the client should be added as additional information.

Fixed in 26d29c5c6356e152f961d9b4afce4f0c7ad46e6d


Fixing vehicle and refresh for BagBarToggle

Unfortunately it looks like the event needs to stay, because moving any item shows Keyframe :/ also added support for vehicle, because button stays even though all UI was changed

Ah, i see! Can reproduce it, will merge.


HW1 Setup

I encountered two issues when trying to follow the setup instruction for HW1.

Latest version of mujoco-py wants MuJoCo 1.50 rather than MuJoCo 1.31.
XXX-v1 environment is no longer available from latest version of gym; I had to use XXX-v2 instead.

Should be fixed in the Fall 2018 version of HW1.