Support flake8-todos

https://github.com/orsinium-labs/flake8-todos

[ ] T001: use TODO instead of FIXME (or BUG) for consistency.
[ ] T002: add author into TODO (motivation).
[ ] T003: add link on issue into TODO.
[ ] T004: missed colon in TODO.
[ ] T005: missed text in TODO.
[ ] T006: write TODO instead of ToDo (use upper case).
[ ] T007: missed space after colon in TODO.

Related:

https://github.com/charliermarsh/ruff/issues/3859

I can try to implement this!

We can close this issue.
Congrats @evanrittenhouse.


GUACAMOLE-723: Add interface for switching between multiple active connections.

This change replaces the connection header in the Guacamole menu with a dropdown menu:

Clicking the menu reveals a list of all available connections. The user may use this to switch connections without navigating all the way back to the home screen:

As with the list of connections on the home screen, this list both paginated and filterable:

When multiple connections are opened this way within the same tab, connections running in the background are displayed as thumbnails in a collapsible panel at the bottom-right corner of the client interface:

Each thumbnail updates in real-time and may be manually closed from the panel by clicking the "X" in the upper-right corner of the thumbnail. If a connection running in the background encounters an error or is disconnected by the server, a warning icon is displayed to call the user's attention to the connection:

The notification containing the details of the failure will only be visible once the user navigates back to that connection. Alternatively, if the failure is expected, the user can simply click the "X" to dismiss the connection and remove the thumbnail.
The arrow button on the left edge of the panel functions as a show/hide toggle, allowing the panel to be collapsed against the right edge of the screen if it is preventing interaction with the display underneath.
If so many connections are open that the panel expands to the width of the browser window, the panel will cease expanding and become scrollable. Scrolling with the mouse scroll wheel will work even though the scroll direction is horizontal, as vertical scrolling via the mouse has been manually redirected horizontally with JavaScript.

Interesting, but I suppose that's not an unwelcome effect. I didn't test against QuickConnect, but I expect this must be due to the <meta ...> tag inserting the connection form.
As long as that form works, I have no issue with this.

QuickConnect appears to work fine from there.  Going to review this once more but I suspect it'll get merged here, shortly...


Add pandas-highcharts as requirements

hello,
Please add pandas-highcharts on requirements.txt
Thank you,

Oops. Thanks for notifying

also requests :)
pip install requests

Done!


typo at tutorial-monitoring.md

@nnmer : Thanks for your contribution! The author, @cynthn, has been notified to review your proposed change.

Verified editorial corrections.


feat: --show-error-tracebacks CLI option to display errors' tracebacks in the output

Resolves #391

Codecov Report

Merging #396 into master will decrease coverage by 0.16%.
The diff coverage is 93.33%.

@@            Coverage Diff             @@
##           master     #396      +/-   ##
==========================================
- Coverage     100%   99.83%   -0.17%
==========================================
  Files          25       25
  Lines        1791     1797       +6
  Branches      282      284       +2
==========================================
+ Hits         1791     1794       +3
- Misses          0        2       +2
- Partials        0        1       +1

Flag
Coverage Δ

#unittests
99.83% <93.33%> (-0.17%)
:arrow_down:

Impacted Files
Coverage Δ

src/schemathesis/cli/__init__.py
100% <100%> (ø)
:arrow_up:

src/schemathesis/utils.py
100% <100%> (ø)
:arrow_up:

src/schemathesis/runner/events.py
100% <100%> (ø)
:arrow_up:

src/schemathesis/cli/output/default.py
99.53% <85.71%> (-0.47%)
:arrow_down:

src/schemathesis/_compat.py
87.5% <0%> (-12.5%)
:arrow_down:

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 c352fdf...d5cb6d8. Read the comment docs.


How to set callbacks for Docker deployment?

What would you like to share?

docker部署方式，我把源码里的webhooks.yml文件拷贝过来，回调没有成功，是要怎么设置回调呢？
Additional information
No response

https://github.com/openimsdk/openim-docker/issues/136


Automatically generate stellar-cli help page.

Here's a screenshot of what it looks like:

Here's the preview link: https://developers-pr755.previews.kube001.services.stellar-ops.com/docs/build/smart-contracts/stellar-cli

Preview is available here:http://developers-pr755.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr755.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr755.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr755.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr755.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr755.previews.kube001.services.stellar-ops.com

The lack of indentation in the sidebar is a bummer; I've created https://github.com/ConnorGray/clap-markdown/issues/31 to address it. I see you also created https://github.com/ConnorGray/clap-markdown/issues/30, @fnando — I vote we add these issues to our own project board so we can ensure they get done quickly, @janewang.

Location suggestion: Can we pull this out into its own section, so it would go:
Tools Overview
SDKs
Developer Tools
Stellar CLI Manual
And then rename it to the Stellar CLI Manual so it's shorter? LMK what you think!

Preview is available here:http://developers-pr755.previews.kube001.services.stellar-ops.com

@briwylde08 great suggestion! The new title is way nicer!

Preview is available here:http://developers-pr755.previews.kube001.services.stellar-ops.com


fix InterceptorDefinitionTest.testInterceptorsImplementInterceptorInterface

This test uses an internal ParameterizedTypeImpl class, which isn't guaranteed to be present in the set of interface types implemented by an interceptor metadata class. The JDK implementation of ParameterizedType may be present as well.
This commit uses the public interface ParameterizedType to perform this test.
Fixes #455

Draft until #455 is accepted.


Modify ARCore download link

I saw moved to ARCore preview2 a bit ago, haven't noticed any issues with as almost identical to preview2.

Thank you for this proposal Michael. Please, resolve the conflicts and I will approve and merge it.

ARCore preview 1 shows up as "Tango Core" in Parameters/Applications. And ARCore preview 2 shows up as ARCore. WebARonARCore master doesn't seem to work at all if ARCore preview 1 (Tango Core) is not installed, even if ARCore preview 2 or ARCore 1.0 from Play Store is installed.

You are completely correct @vincentfretin . I will roll back to always point to the correct APKs needed by the current state of the project. We are working on a complete overhaul based on latest ARCore to try to resolve these issues but it will take some time still until we get there. Thank you for your input.


[WIP] Generates stubs with Buf using protobuf-ts

https://github.com/timostamm/protobuf-ts

Use BigInt to represent signed and unsigned 64 bit integral types since browser support is good:

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/BigInt
https://caniuse.com/?search=bigint

Protobuf map inputBlindingKey and outputBlindingKey are now converted to object { [key: string]: Uint8Array } instead of javascript Map

https://github.com/timostamm/protobuf-ts/discussions/286

Is this PR need #136 to be merged first?

@tiero pr needs to be tested with daemon 0.8.9

Is this PR need #136 to be merged first? @Janaka-Steph

Is it not closing 136 ? Otherwise I misunderstood the issue

sorry I meant #137

sorry I meant #137
Is it duplicated of this?

Ha yes it's a duplicate but using different protoc plugins. Will be closed once we can merge this one.

  * [Protobuf map conversion timostamm/protobuf-ts#286](https://github.com/timostamm/protobuf-ts/discussions/286)

I recall we have done someting in LDK for protobuf native Maps, cc/ @louisinger
Are sure this change will not impact other apps?

I recall we have done someting in LDK for protobuf native Maps, cc/ @louisinger

Yes, the API functions inputs don't use Map but Record (so a JS object and then we create the proto request from it..

@Janaka-Steph what's wrong with the integration test?

@tiero @louisinger @altafan Tests are passing \o/


enable-master breaks docker

Hey I'm trying to get just the master node up and running. I did kube-config install then kube-config enable-master and rebooted. Before enabling the master docker works fine, after it's broken. When I do a status I get the following, have you run into this?
[root@k8s-master ~]# systemctl start docker.service
Job for docker.service failed because a configured resource limit was exceeded. See "systemctl status docker.service" and "journalctl -xe" for details.
[root@k8s-master ~]# systemctl status docker.service
* docker.service - Docker Application Container Engine
   Loaded: loaded (/usr/lib/systemd/system/docker.service; enabled; vendor preset: disabled)
  Drop-In: /usr/lib/systemd/system/docker.service.d
           `-docker-flannel.conf
   Active: failed (Result: start-limit)
     Docs: https://docs.docker.com

Nov 20 15:36:27 k8s-master systemd[1]: Failed to start Docker Application Container Engine.
Nov 20 15:36:27 k8s-master systemd[1]: docker.service: Failed with result 'start-limit'.
Nov 20 15:37:52 k8s-master systemd[1]: docker.service: Failed to load environment files: No such file or directory
Nov 20 15:37:52 k8s-master systemd[1]: docker.service: Failed to run 'start-pre' task: No such file or directory
Nov 20 15:37:52 k8s-master systemd[1]: Failed to start Docker Application Container Engine.
Nov 20 15:37:52 k8s-master systemd[1]: docker.service: Failed with result 'resources'.
Nov 20 15:37:55 k8s-master systemd[1]: docker.service: Failed to load environment files: No such file or directory
Nov 20 15:37:55 k8s-master systemd[1]: docker.service: Failed to run 'start-pre' task: No such file or directory
Nov 20 15:37:55 k8s-master systemd[1]: Failed to start Docker Application Container Engine.
Nov 20 15:37:55 k8s-master systemd[1]: docker.service: Failed with result 'resources'.

Hello @stevesloka!
Glad to see you are testing this out.
Yeah, this is a known issue, that reboots doesn't work properly.
I've fixed it in bd95c0e4871a1c7f9542631042482417056eeeee
The only thing you will have to do:
nano /etc/kubernetes/dynamic-dropins/docker-flannel.conf
# Add these two lines
Restart=on-failure
RestartSec=2

The issue happens because when docker starts, it want a file that flannel provides.
However, flannel requires some secs to start up, so docker fails too many times, before flannel is ready
This patch makes docker wait longer for flannel
This code is already merged in dev, which will be released quite soon.
Generally speaking, you may always revert enable-master with kube-config disable-node

Thanks for the heads up. I copied in those two lines, but still no go. Docker is still not starting.
[root@k8s-master ~]# systemctl status docker.service
* docker.service - Docker Application Container Engine
   Loaded: loaded (/usr/lib/systemd/system/docker.service; enabled; vendor preset: disabled)
  Drop-In: /usr/lib/systemd/system/docker.service.d
           `-docker-flannel.conf
   Active: activating (auto-restart) (Result: resources) since Fri 2015-11-20 20:00:36 EET; 1s ago
     Docs: https://docs.docker.com

Nov 20 20:00:36 k8s-master systemd[1]: docker.service: Failed to run 'start-pre' task: No such file or directory
Nov 20 20:00:36 k8s-master systemd[1]: Failed to start Docker Application Container Engine.
Nov 20 20:00:36 k8s-master systemd[1]: docker.service: Unit entered failed state.
Nov 20 20:00:36 k8s-master systemd[1]: docker.service: Failed with result 'resources'.

Which docker version are you using?
Can you output kube-config info?

[root@k8s-master ~]# docker ps
CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS              PORTS               NAMES
[root@k8s-master ~]# docker info
Containers: 1
Images: 27
Server Version: 1.9.0
Storage Driver: overlay
 Backing Filesystem: extfs
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.1.13-1-ARCH
Operating System: Arch Linux ARM (containerized)
CPUs: 4
Total Memory: 923.1 MiB
Name: k8s-master
ID: GVUL:M3IB:D7QC:AAGZ:4F3O:LM3H:6O5L:R6OY:WQYP:JEHC:Z3YS:CZF7

[root@k8s-master ~]# kube-config info
Architecture: armv7l
Kernel: Linux 4.1.13
CPU: 4 cores x 900 MHz

Used RAM Memory: 41 MiB
RAM Memory: 923 MiB

Used disk space: 1.5GB (1557644 KB)
Free disk space: 53GB (55302780 KB)

SD Card was built: 19-11-2015 22:13

kubernetes-on-arm:
Latest commit: ccea1ef
Version: 0.5.8

systemd version: 227
docker version:
operation not permitted
Error response from daemon: Cannot start container fafa71e602e21b66fc162b4caa0855731cb169820115920b2cf0d4cc0590b811: [8] System error: operation not permitted
kubernetes version:

Oh, you've run on the same problem as I. I ran into it a few minutes ago
I have no idea why this is happening.
At least, it's because of docker 1.9.0, which was updated for pacman a few days ago.
The bad thing with pacman is that you can't downgrade easily
My issue: #18125

Can I define the version to install for now? If I have to blow away my sd cards I'm cool with that, it's not too too painful.

The problem is that one can't specify to install when using pacman. So, not either a sd card refresh would do it

On 20 Nov 2015, at 21:12, Steve Sloka<EMAIL_ADDRESS>wrote:
Can I define the version to install for now? If I have to blow away my sd cards I'm cool with that, it's not too too painful.
—
Reply to this email directly or view it on GitHub.

Ok gotcha, no worries. Thanks for the help!

@stevesloka See this: https://github.com/docker/docker/issues/18125#issuecomment-159736200
It solved the problem for me.

Looks like with docker 1.10 same issue:
[root@k8s-master ~]# docker version
Client:
Version:      1.10.0
API version:  1.22
Go version:   go1.4.3
Git commit:   590d5108-unsupported
Built:        Fri Feb 12 16:35:07 2016
OS/Arch:      linux/arm
An error occurred trying to connect: Get http:///var/run/docker.sock/v1.22/version: read unix /var/run/docker.sock: connection reset by peer
Any suggestions before I start debugging? Rollback for docker?

Which version of Kubernetes on ARM are you running @Mangirdaz?
If it's 0.7, please open a new issue.


Error using tooltip recipe

Using the tooltip recipe I get the following error:
Uncaught TypeError: Failed to execute 'getComputedStyle' on 'Window': parameter 1 is not of type 'Element'.

Steps to reproduce:

ember install:npm ember-pop-over
ember g ember-pop-over
ember g component tool-tip
Add the tooltip recipe from the README to app/components/tool-tip.js
Add the tooltip handlebars from the README to any template
Hover over the tooltip in the browser

@patrickberkeley Seems like I need to write better error handling for dom-ruler. Once done with that, debugging this should be a piece of cake.
Also, this is beta software. I would love any help I can get on this.


What for is BOOT_KEYS_DIR directory?

Hi all,
I'm switching from sample keys to user SIGNING_MODEL and according to meta-signing-key/README.md  the vital definitions include:

SIGNING_MODEL := "user"
Prohibit using the sample keys for signing the images.

UEFI_SB_KEYS_DIR := "<path>"
Point to the location of user keys used for UEFI secure boot.

MOK_SB_KEYS_DIR := "<path>"
Point to the location of user keys used for MOK secure boot. Note that
MOK secure boot is on top of UEFI secure boot so creating the user keys
for MOK secure boot only will still introduce the security risk in your
product.

IMA_KEYS_DIR := "<path>"
Point to the location of user keys used for IMA appraisal.

USER_KEY_SHOW_VERBOSE = "1"
Optional. Used to enable the verbose output for debugging purpose.

As I'm not using ima DISTRO_FEATURES yet. I'm also not setting IMA_KEYS_DIR only the UEFI_SB_KEYS_DIR and MOK_SB_KEYS_DIR. Now the issue I have occured is that i'm failling to enroll dbx during Automatic Certificate Provision in addition there is an unknown error during loading of LockDown.efi. So my question is what am I missing? Do I need BOOT_KEYS_DIR as well? And what is it being used for?
Because I haven't found any description about it in the documentation. I've seen in the code of user-key-store.bbclass that LockDown.efi is signed by uks_boot_sign which just calls boot_sign... I thought that BOOT_KEYS_DIR is just for grub password isn't it?

OK I have found it but it was well hidden. The LockDown.efi is providing all the uefi sb materials and it is signed by this gpg key. Everything is now clear to me.


Generic inference doesnt work with PropType

This issue references a discord discussion: https://discord.com/channels/793943652350427136/1109024933402640456
As of now, vues compiler cannot handle complex types for generics. So a component like this doesnt work correctly:
<script lang="ts">
type SwitchByBoolean<T extends boolean, V> = false extends T ? V : V[];
</script>

<script lang="ts" setup generic="T extends boolean, V extends SwitchByBoolean<T, string>">
const props = defineProps<{
  modelValue?: V;
  multiple?: T;
}>();

This component accepts a string array if multiple is true, otherwise it accepts only a string.
Since vue cant analyze the types properly, it will not cast the props.
e.g. usually when writing this:
<MyCompoennt boolean-flag />

It will result in props.booleanFlag === true. However, in the above example, multiple would be an empty string.
The solution would be to use PropType:
<script setup lang="ts">
const props = defineProps({
  modelValue: {
    type: [String, Array] as PropType<V>,
  },
  multiple: {
    type: Boolean as PropType<T>,
    default: false,
  },
});
</script>

This correctly enforces the boolean flag. However, type inference is broken now.
The types dont enforce the array-string / string

IMO it's a vue core problem

Closed due to unable to reproduce in latest version.

@johnsoncodehk can you please also add a test for only passing "multiple" without true or false? And the check if it is cast to "true" correctly or to an empty string? Because that was the problem I had

Could you show me the full expected test code? Thanks.

Usage as Boolean prop:
<template>
	<HelloWorld :multiple :modelValue="['a']" />
        <HelloWorld :modelValue="a" />
        <!-- @vue-expect-error -->
	<HelloWorld :modelValue="['a']" />
	<!-- @vue-expect-error -->
	<HelloWorld :multiple :modelValue="'a'" />
</template>

<script lang="ts" setup>
import HelloWorld from './component.vue'
</script>

Component maybe with extra check:
<script lang="ts">
type SwitchByBoolean<T extends boolean, V> = false extends T ? V : V[];
</script>

<script lang="ts" setup generic="T extends boolean, V extends SwitchByBoolean<T, string>">
const props = defineProps<{
	modelValue?: V;
	multiple?: T;
}>();

// not sure how to produce a testfailure
expect(typeof props.modelValue).toBe('boolean')
</script>

The extra expect is neccessary because vue is casting boolean props to true/false depending on if there are present or not.
However, with generic prop types, vue is not able to figure out the PropType of modelValue and will use its default behavior (so it will be undefined or an empty string instead)

@johnsoncodehk tagging you because I am not sure if you are notified about closed issues


Binder build failing

Suspect the conda env has a versioning problem (a package has changed it's API in a breaking way in a later version, the env does not specify a version, so conda chooses the latest version).
Todo:

[x] Confirm the nature of the problem
[x] Specify package versions in env

Apparently solved with https://github.com/memphis-iis/datawhys-content-notebooks-python/commit/7bc8b12caea2a6cc33dbc637df0ead18c3660b59 and forgot to close


'Reindent Lines' is not working in VS Code for Mac

Issue Type
Bug
Description

select some code blocks
select 'Reindent Lines' from Command Palette
nothing is not indented.

VS Code Info
VS Code version: Code 1.20.1 (f88bbf9137d24d36d968ea6b2911786bfe103002, 2018-02-13T15:23:28.607Z)
OS version: Darwin x64 17.5.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz (4 x 2300)

Load (avg)
2, 2, 2

Memory (System)
16.00GB (6.95GB free)

Process Argv
/private/var/folders/tl/mt5w_jv57b70h1ch11xcmq940000gn/T/AppTranslocation/C4D0DB56-5381-4B97-8B62-10A8470DBCDB/d/Visual Studio Code.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (3)

Extension
Author (truncated)
Version

python
ms-
2018.4.0

team
ms-
1.136.0

LiveServer
rit
5.1.1

Reproduces only with extensions

Can you provide more details, like code samples, to help troubleshoot?

This problem was solved. Thanks.


navigator.vibrate

https://developer.mozilla.org/en-US/docs/Web/API/Navigator/vibrate
seems to only work for chrome on android?

already there
http://caniuse.com/#search=vibration


Removed cluster tools banner from dashboard page

Summary
Fixes #9229

Removed new cluster tools banner from the dashboard page
How to test
Go to cluster dashboard > cluster tools banner should not be there

Fixed


Enable DML leak detection.

GPGMM will ASSERT if device allocation leaks are present.
Roll third_party/gpgmm/ c52956998..1e5ae0071 (16 commits).

@wangli69087


FIx CODEOWNERS

The sub-folders for code ownership are not working as intended.

This should be fixed now: https://github.com/reactiveops/charts/pull/81


empty name in 'adfDashboardChanged' listener

There is an issue with empty name in 'adfDashboardChanged' listener. I belive it is caused by late adf-dashboard name attribute assignment.
I declare adf-dashboard this way:
<adf-dashboard name="{{dashboard.id}}" collapsible="true" structure="6-6" adf-model="dashboardConfig" />
At start dashboard.id not assigned, it will be assigned by $http.get callback.
We can fix this by broadcasting $scope.name value instead name.

Coverage remained the same at 93.153% when pulling 8234370abfe484c84ad3eefb149994e86e435a6c on tko-inform:master into 0a1b1b68f207d64b3f6a1ac6c50553ea7b128c75 on angular-dashboard-framework:master.

Coverage remained the same at 93.153% when pulling 51a1febc5b8dca5c9713e111b7e1e5a2325746b5 on tko-inform:master into 0a1b1b68f207d64b3f6a1ac6c50553ea7b128c75 on angular-dashboard-framework:master.


Does reset method have the same effect as setting coeff to 0?

Hi, thanks for your greater work!
I just wonder if calling model.reset() would be the same as calling model.set_control(control_vector, coeff=0)?

model.reset() is the same as calling model.set_control(None)--it actually clears the vectors completely from the model. it's better to use reset() before training a new vector, and inference with a reset model is slightly faster, but otherwise the behavior should be identical to using a zeroed vector.


<n-image> aspect ratio

This function solves the problem (这个功能解决的问题)
Support aspect-ratio prop on <n-image> so the height or width can be set automatically.
Expected API (期望的 API)
<n-image src="..." width="100%" aspect-ratio="16 / 9" />

you use:
  <n-image
    :img-props="{ style: { width: '100%', 'aspect-ratio': '16 / 9' } }"
    width="100"
    src="blablabla"
    fallback-src="https://07akioni.oss-cn-beijing.aliyuncs.com/07akioni.jpeg"
  />

This issue does not have any recent activity. If you are still experiencing similar problems, open a new error, including a minimal copy of the problem


Bloc doc intial documentation structure

Signed-off-by: talwinder.kaur<EMAIL_ADDRESS>

build directory should not be included in commit.


Code doe not seem to work.

I pasted the code in the URL, pressed enter as instructed, and nothing happened.

Which one?

I have the same issue, and I was trying to use the chest esp... I'm sure that it's just a little extra thing I need to do

that hack is broken, we woll be working on a fix


Dimmer Module

Is it possible to add a Dimmer Module to the system, so that you can say "Alexa set the light to 20%"
if yes, could you post an example?

Sorry, dimmer modules are not supported. This project supports only turning on/off.
To Alexa set the light to xxxx you have to make an Amazon Skill.
I am in the process of building a smart-home-skill for everyone and about 90% of the work is done now. I will update this issue when it's completed.

@kakopappa Thanks! I got sinric to work with the example blind code. would you be able to make some
code based on that so i can open and close the blinds? essentially control the servo?
Thanks,
Arman

@kakopappa I tried downloading the switch_example.ino and when I try to compile it i get the following error "json' was not declared in this scope" against this line  String deviceId = json ["deviceId"];.
I am a novice. Kindly advise on the way forward.
Thanks
Ramesh

You are missing the json library. Install ArduinoJson library first
On Mon, 9 Mar 2020 at 12:15 AM rramesh68<EMAIL_ADDRESS>wrote:

@kakopappa https://github.com/kakopappa I tried downloading the
switch_example.ino and when I try to compile it i get the following error
"json' was not declared in this scope" against this line String deviceId =
json ["deviceId"];.
I am a novice. Kindly advise on the way forward.
Thanks
Ramesh
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kakopappa/arduino-esp8266-alexa-multiple-wemo-switch/issues/33?email_source=notifications&email_token=ABZAZZTFQUXVXUFAJAZTF43RGPHBPA5CNFSM4EJ25ZU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOE3SJY#issuecomment-596228391,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZAZZVOIRYCLH2SIKPGWQ3RGPHBPANCNFSM4EJ25ZUQ
.


Add Growatt solar inverter

RE-CHECK!

Automated adapter checker
ioBroker.growatt

[ ] :heavy_exclamation_mark: [E300] Not found on travis. Please setup travis
[ ] :eyes: [W400] Cannot find "growatt" in latest repository

Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.growatt

[ ] :heavy_exclamation_mark: [E300] Not found on travis. Please setup travis
[ ] :eyes: [W400] Cannot find "growatt" in latest repository

Add comment "RE-CHECK!" to start check anew

Why doesn't that work? What do I have to do so that Travis is recognized.
I've done everything that is in the forum and in the instructions.
Traves checks as OK.

Ok, thx!

RE-CHECK!

Automated adapter checker
ioBroker.growatt

[ ] :heavy_exclamation_mark: [E300] Not found on travis. Please setup travis
[ ] :eyes: [W400] Cannot find "growatt" in latest repository

Add comment "RE-CHECK!" to start check anew

Why doesn't that work? What do I have to do so that Travis is recognized.
I've done everything that is in the forum and in the instructions.
Travis checks as OK.

The same...

RE-CHECK!

Automated adapter checker
ioBroker.growatt

[ ] :heavy_exclamation_mark: [E300] Not found on travis. Please setup travis
[ ] :eyes: [W400] Cannot find "growatt" in latest repository

Add comment "RE-CHECK!" to start check anew

How do I get this adapter into the ioBroker repository?

@PLCHome with me or Bluefox finding time to review the adapter :-) Sorry, but I'm currently on vacation, so time is very rare

thanx

RE-CHECK!

Automated adapter checker
ioBroker.growatt

[ ] :heavy_exclamation_mark: [E145] No news found for actual version 0.0.6
[ ] :heavy_exclamation_mark: [E300] Not found on travis. Please setup travis
[ ] :eyes: [W400] Cannot find "growatt" in latest repository

Add comment "RE-CHECK!" to start check anew

@PLCHome Here some review comments

as checker told aboe news are missing for 0.0.6
with     "@iobroker/adapter-core": "^2.4.0", the js-controller dep needs to be >=2.0.0!
io-package native defined 5 values, admin sets 7
it seems to store a password, please do this encrypted - please see Repo Readme for best practices!

Please check and fix and inform us when done

RE-CHECK!

Automated adapter checker
ioBroker.growatt

[ ] :heavy_exclamation_mark: [E118] Versions in package.json and in io-package.json are different
[ ] :heavy_exclamation_mark: [E145] No news found for actual version 0.0.6
[ ] :heavy_exclamation_mark: [E300] Not found on travis. Please setup travis
[ ] :eyes: [W400] Cannot find "growatt" in latest repository

Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.growatt

[ ] :heavy_exclamation_mark: [E118] Versions in package.json and in io-package.json are different
[ ] :heavy_exclamation_mark: [E145] No news found for actual version 0.0.6
[ ] :heavy_exclamation_mark: [E300] Not found on travis. Please setup travis
[ ] :eyes: [W400] Cannot find "growatt" in latest repository

Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.growatt

[ ] :heavy_exclamation_mark: [E300] Not found on travis. Please setup travis
[ ] :eyes: [W400] Cannot find "growatt" in latest repository

Add comment "RE-CHECK!" to start check anew

@Apollon77 fixed:

as checker told aboe news are missing for 0.0.6
with "@iobroker/adapter-core": "^2.4.0", the js-controller dep needs to be >=2.0.0!
io-package native defined 5 values, admin sets 7
it seems to store a password, please do this encrypted - please see Repo Readme for best practices!

RE-CHECK!

Automated adapter checker
ioBroker.growatt

[ ] :heavy_exclamation_mark: [E300] Not found on travis. Please setup travis
[ ] :eyes: [W400] Cannot find "growatt" in latest repository

Add comment "RE-CHECK!" to start check anew

@Apollon77:
All issues from review are fixed.
[E300] Not found on travis. Please setup travis:
The repro is in Travis and OK

@Apollon77:
All issues from the review have been fixed. I would be happy if the review would be completed and the adapter could be finished.
One would be happy about it:
https://github.com/PLCHome/ioBroker.growatt/issues/1#issuecomment-706708855

@Apollon77 @GermanBluefox
All issues from the review have been fixed. I would be happy if the review would be completed and the adapter could be finished.

I've been trying to get this adapter into the ioBroker for more then 3 months.
If it is not wanted, I can also stop the project. Should I?
I assume that at least the users and supporters of the adapter are against abort:
@yunagear @kopierschnitte

It would be a pity if this project will be aborted. The code works without any problems since I've started using it a few months ago and I would definitely call it stable.

@PLCHome No please donot stop. Yes we need to catch up with the reviews ... sorry for that

Also for us a very important project!! As we want to use it for several plants together with iobroker and Grafana.


No reply for "az spring-cloud build-service builder create ..."

There is no reply for the command:
 az spring-cloud build-service builder create -n ${CUSTOM_BUILDER} \
       --builder-file azure/builder.json \
       --no-wait

Is this normal?
My env:  Windows 11, Git-bash

Refer: https://docs.microsoft.com/en-us/cli/azure/spring-cloud/build-service/builder?view=azure-cli-latest#az-spring-cloud-build-service-builder-create
So this is expected.


Parameter check needed

Returned true when called with non-string identifier argument (like 12)
Return true for empty string

If "49927398716" is a valid identifier, it seems to me that<PHONE_NUMBER>6 should be considered valid also.


Chisel erroring out when importing an external sol file (interface in this case)

Component
Chisel
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (ed9298d 2023-03-09T00:04:28.809303436Z)
What command(s) is the bug in?
No response
Operating System
Linux
Describe the bug
Start up chisel:
import "iSimpleDefiStaking.sol"
address a = 0x7f9Ef04582Fe662c19A0d016b9FF27E2ca2ADfc6;
Last line generates error:
Failed to find REPL contract!
Welcome to Chisel! Type !help to show available commands.
➜ import "tmp/iSimpleDefiStaking.txt"
➜ address a = 0x7f9Ef04582Fe662c19A0d016b9FF27E2ca2ADfc6;
Failed to find REPL contract!
➜ !so
// SPDX-License-Identifier: UNLICENSED
pragma solidity ^0.8.17;
import {Cheats} from "forge-std/Vm.sol";
import "iSimpleDefiStaking.txt";
contract REPL {
Cheats internal constant vm = Cheats(address(uint160(uint256(keccak256("hevm cheat code")))));
/// @notice REPL contract entry point
function run() public {}

}
iSimpleDefiStaking.txt is really iSimpleDefiStaking.sol (github extension limitation)
iSimpleDefiStaking.txt

@clabby mind taking a look at this?
perhaps this is the same error as https://github.com/foundry-rs/foundry/issues/4481

not able to repro with provided file, though after removing the function stakes(address _user) external returns (SimpleDefiStaking.returnStakes[] memory) line as SimpleDefiStaking is not declared
Welcome to Chisel! Type `!help` to show available commands.
➜ import "iSimpleDefiStaking.txt"
➜ address a = 0x7f9Ef04582Fe662c19A0d016b9FF27E2ca2ADfc6;
➜ !so
// SPDX-License-Identifier: UNLICENSED
pragma solidity ^0.8.26;

import {Vm} from "forge-std/Vm.sol";

import "iSimpleDefiStaking.txt";

contract REPL {
    Vm internal constant vm = Vm(address(uint160(uint256(keccak256("hevm cheat code")))));

    /// @notice REPL contract entry point
    function run() public {
        address a = 0x7f9Ef04582Fe662c19A0d016b9FF27E2ca2ADfc6;
    }
}

going to close for now, please reopen if still an issue. thank you!


ISPN-1753 Fix nearcache demo and update deploy plugin for AS 7.1

https://issues.jboss.org/browse/ISPN-1753

Thanks


⚠ Bofrost.at has degraded performance

In 18213b0, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1202 ms

Resolved: Bofrost.at performance has improved in 447615a.


[3.0] Memory leak in Mesh.add_surface_from_arrays()

Tested on Godot 3.0 af282b56feb8d84049f911dd1d91d82bd2211727
Windows 10 64 bits
The following script shows a memory leak:
extends Node

func _process(delta):
	var mesh = Mesh.new()
	var arrays = [
		[Vector3(0,0,0)],
		null,
		null,
		null,
		null,
		null,
		null,
		null,
		null
	]
	mesh.add_surface_from_arrays(Mesh.PRIMITIVE_POINTS, arrays)

	get_node("Label").set_text(str(OS.get_dynamic_memory_usage()))

The memory doesn't increase if I remove add_surface_from_arrays.
I found it because it has a massive impact on my voxel game, where meshes are generated at runtime, 1 Mb is leaked at each chunk update.

I ran Valgrind's memcheck on your example. Here's a paste of some of its output https://pastebin.com/WQtTGhTk
Perhaps it's helpful to someone knowledgeable about that code.

Ported terrain sculpting to Godot 3.0, and now this leak makes Godot run out of memory in less than 30 seconds. @reduz any idea?

I narrowed it down further, and I found this line causes a leak on its own in ArrayMesh::add_surface_from_arrays():
		PoolVector<Vector3> vertices = p_arrays[ARRAY_VERTEX];

After some midnight debugging I found that PoolVector ends up doing copy-on-write in its own destructor, due to an atttempt to call destructors of all items using a Write access:

The test done is the following, put directly into Node's PROCESS notification, and debugging the whole process of resizing, then destruction at scope exit:
{
	PoolVector<Vector3> pv;
	pv.push_back(Vector3());
}

Solution: Change line 150 to         if (old_alloc->refcount.unref() == false) {
yup that was it a true to a false.
Well done HumanSheeple you have done it again :)

Not sure what the real fix is, needs @reduz confirmation. Also keep in mind this happens because of copy on write, so not only the alloc was leaked, but PoolVector is calling destructors of copied objects instead of the originals.


Don't scroll outside of context menu when it is open

1.0.0-beta5

Select text
Open context menu
Move cursor outside of the context menu
Scroll the mouse wheel

Expected:
Content behind the context menu shouldn't scroll

Fixed in 1.0.0-rc3

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Room.findAll() can not get all the rooms

At first, findAll() found only one room. Now after about 1 hour, it can find three.
I have joined more than 30 rooms. It doesn't make sense to restart or re-login.
Why did it load so slowly? Is there any precondition for loading a room?

Thx for issue, we are trying to resolve this problem now.

How's it going？

It's due to the mac protocol, the room will be loaded after message come.


Gradient clipping by norm has different semantics in tf.keras.optimizers against keras.optimizers

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: No
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.13.1
Python version: 3.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: CUDA 10.0 + cuDNN 7.4
GPU model and memory: NVIDIA Tesla V100 32GB

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
The gradient clipping mechanism is implemented in the abstract class, tf.keras.optimizers.Optimizer (keras.optimizers.Optimizer) so that every optimizer inherited from the class supports it.
We find different semantics of the implementations between tf.keras and keras.io.
In tf.keras, the gradient norm is calculated and clipped per gradient tensor.
https://github.com/tensorflow/tensorflow/blob/bb8cf258bc87f68612a5e032d5b4def0d3c52566/tensorflow/python/keras/optimizers.py#L98-L99
But in keras.io, the gradient norm is calculated globally across all gradient tensors.
        if hasattr(self, 'clipnorm') and self.clipnorm > 0:
            norm = K.sqrt(sum([K.sum(K.square(g)) for g in grads]))
            grads = [clip_norm(g, self.clipnorm, norm) for g in grads]

Code link: https://github.com/keras-team/keras/blob/eab1b5bcdf105746ede02d2eb8a5cb3ca359b1b5/keras/optimizers.py#L96-L98
IMO, tf.keras should adopt the latter method since
(1) The paper introducing gradient clipping (https://arxiv.org/abs/1211.5063) suggests the latter one, and
(2) tf.keras should be compliant with keras.io.

I already provided comment in the PR. Closing this.

Hi @tanzhenyu , thanks for your reply.
But where can I find the comment you left?

@chenchc in the PR, go to files changes tab (or the regular page) you should be able to see it

@tanzhenyu I cannot see the comments in both tabs. Maybe they are invisible to my account?

@chenchc My major comment is that I don't see where in this paper describing it should use global clip instead of local clip.


Battery level

It's possible with the web socket to know the battery level of the nuimo?

Hi @moifort – it's possible but not yet supported. Let me see if I can quickly add it.

If you can implement it, it would be awesome! But if it's take too much time for you I will see if I can make a pull request :)
Thibaut.

Le 23 août 2016 à 07:33, Lars Alexander Blumberg<EMAIL_ADDRESS>a écrit :
Hi @moifort – it's possible but not yet supported. Let me see if I can quickly add it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Firmware version (prefix V) and battery level (prefix %) are now immediately reported when you connect your web socket(s). Additionally, battery level is reported every 3 minutes.

PS: We also made an compiled app available at senic.com/developers

Oooh! Thank you!!! You are awesome \o/!!!!
Thibaut.

Le 24 août 2016 à 17:55, Lars Alexander Blumberg<EMAIL_ADDRESS>a écrit :
PS: We also made an compiled app available at senic.com/developers
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Use a null when serializing Response Ids equal to None

Right now, when the Response's id is set to None, the resulting JSON response is invalid:
{"jsonrpc":"2.0","error":{"code":-32700,"message":"Parse error"},"id"} // note the missing value for "id"

With a tiny tweak, the correct response JSON is produced instead:
{"jsonrpc":"2.0","error":{"code":-32700,"message":"Parse error"},"id":null}

Oh this is quite unexpected, I would think that serialize_key would result in null, but it seems serde-json is tricky...
Thanks for noticing it

Can I ask you to write unit test?
I had test for deserialize but not serialize...
https://github.com/DoumanAsh/json-rpc-types/blob/master/tests/response.rs#L37

Ok, there you go; in return I'd like to ask for a 1.0.1 release soon :slightly_smiling_face:.

1.0.1 has been released
1.0.0 is yanked


Fixed DisplayFolder bugs. Changed default shortcut keys

Fixed DisplayFolder bugs : import from CSV and directly in the UI grid.
Changed default shortcut keys to avoid Ctrl+C and Ctrl+S which are usually associated with Copy and Save.

Thank you!! The Ctrl+C one was extremely frustrating.


chore: fix android build issues and upgrade IVS to 1.13.0

Description of changes:
Fixed possible build issues with RN 0.69 and updated IVS to 1.13.0 on iOS and Android.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I'll get this into the release branch, thanks @hirbod

@hirbod You mind rebasing on top of the release/1.3.0 branch?

@alexmdodge done


add connect timeout; reinstate speed limit (#126)

This adds a connection timeout of 30 seconds, closing #125
It re-instates the lower speed limit that was briefly added in #85 and
was accidentally reverted in the following pull request (#86)

In combination, this should prevent downloads from hanging, whether
because the server is unavailable and so the connection is never
established, or because the server never sends any data after the
connection is established. This does not put an absolute timeout on
downloads since a very large download can take arbitrarily long while
still making progress.
(cherry picked from commit 84e948c02b8a0625552a764bf90f7d2ee97c949c)

Codecov Report

Merging #127 (84e948c) into release-1.4 (7a74621) will increase coverage by 0.07%.
The diff coverage is 100.00%.

:exclamation: Current head 84e948c differs from pull request most recent head 6ef5a66. Consider uploading reports for the commit 6ef5a66 to get more accurate results

@@               Coverage Diff               @@
##           release-1.4     #127      +/-   ##
===============================================
+ Coverage        92.96%   93.03%   +0.07%
===============================================
  Files                5        5
  Lines              483      488       +5
===============================================
+ Hits               449      454       +5
  Misses              34       34

Impacted Files
Coverage Δ

src/Curl/Curl.jl
100.00% <ø> (ø)

src/Downloads.jl
84.37% <ø> (ø)

src/Curl/Easy.jl
94.19% <100.00%> (+0.13%)
: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 7a74621...6ef5a66. Read the comment docs.


Remove MutationObserver, overload setAttribute instead

I have no idea why I didn't think of this sooner. Where's the catch?
Summary:

Removes MutationObserver completely
Instead, we overload BaseElement's setAttribute method
We still need to apply all available attributes once when the element is connecting (as we've been doing all the time)

Pros:

Simplifies the code base -- one less moving part to worry about it
Possibly better for performance? But we don't have any good data so far
Attributes are forwarded to object properties immediately and not in the next JavaScript tick (not a big deal in practice, but in our tests we need to queue a lot of microtasks...)

Cons:

setAttribute is not triggered when the user manually modifies an attribute value in the DOM.

Blocked until I have a way to do better performance tests.

Blocked until I have a way to do better performance tests.

Losing the ability to edit stuff from the DOM is a big deal IMO. Being able to show folks that are new to JS and HTML how to mess around with the devtools has been really powerful in getting them accustomed to the environment. Having to change source + reload all the time doesn't flow as nicely.
If this gets changed, would it be possible to enable the old behavior behind a flag or API call?

Losing the ability to edit stuff from the DOM is a big deal IMO. Being able to show folks that are new to JS and HTML how to mess around with the devtools has been really powerful in getting them accustomed to the environment. Having to change source + reload all the time doesn't flow as nicely.
If this gets changed, would it be possible to enable the old behavior behind a flag or API call?

If this gets changed, would it be possible to enable the old behavior behind a flag or API call?

Yeah, I think I'd like to at least offer an observe toggle in <three-game> that opts in to the Mutation Observer. I like that feature a lot, too :)

If this gets changed, would it be possible to enable the old behavior behind a flag or API call?

Yeah, I think I'd like to at least offer an observe toggle in <three-game> that opts in to the Mutation Observer. I like that feature a lot, too :)

I've done a bit of experimenting and light benchmarking and have some good data now.
I've created a new example that spawns 100 meshes and animates them in every frame.
I have deliberately chosen an approach where the position.x, .y and .y attributes of every of these elements are changed in every frame (100 elements * 3 attributes * up to 60 frames per second.) This is not something you would do in a real project (where, instead, you would rather mutate the Three.js scene object directly), but it did give me an opportunity to test the current implementation (MutationObserver) vs. the setAttribute thing in this PR.
Result: the setAttribute approach performs significantly better. If I said "by an order of magnitude", it would probably be an understatement. Where in this demo the MutationObserver approach is essentially prohibitively slow, you don't even see the setAttribute make any noise in the profiler.
For this reason, I will be merging this PR soon. This unfortunately means that we'll be losing the ability to have a user fiddle around in their devtools, play with attributes, and see the changes reflected in the scene immediately. A couple of thoughts:

I love this "feature" would be sad to lose it.
At the same time, I see it as mostly a novelty. It's something fun to tell newcomers to the library, but it's nothing that you would need in any actual project. This minimizes my motivation to optimize for it.
However, I do like the idea of making the observing of "user fiddling" opt-in. I can't, however, make a decision about this today. I do know that I want to merge this change into main and release it with 0.3, but I want to think some more about if, how and where exactly to implement opt-in observing.

If you have thoughts on this, please voice them!

I've done a bit of experimenting and light benchmarking and have some good data now.
I've created a new example that spawns 100 meshes and animates them in every frame.
I have deliberately chosen an approach where the position.x, .y and .y attributes of every of these elements are changed in every frame (100 elements * 3 attributes * up to 60 frames per second.) This is not something you would do in a real project (where, instead, you would rather mutate the Three.js scene object directly), but it did give me an opportunity to test the current implementation (MutationObserver) vs. the setAttribute thing in this PR.
Result: the setAttribute approach performs significantly better. If I said "by an order of magnitude", it would probably be an understatement. Where in this demo the MutationObserver approach is essentially prohibitively slow, you don't even see the setAttribute make any noise in the profiler.
For this reason, I will be merging this PR soon. This unfortunately means that we'll be losing the ability to have a user fiddle around in their devtools, play with attributes, and see the changes reflected in the scene immediately. A couple of thoughts:

I love this "feature" would be sad to lose it.
At the same time, I see it as mostly a novelty. It's something fun to tell newcomers to the library, but it's nothing that you would need in any actual project. This minimizes my motivation to optimize for it.
However, I do like the idea of making the observing of "user fiddling" opt-in. I can't, however, make a decision about this today. I do know that I want to merge this change into main and release it with 0.3, but I want to think some more about if, how and where exactly to implement opt-in observing.

If you have thoughts on this, please voice them!


Fix #1325, return correct sql state when connection is closed

When driver detects the connection close, it is not returning the correct sql/xopen compliant state. This is causing the connection pool managers like hikari to detect a connection closed state correctly. This fixes issue #1325

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

Thanks for your contribution @anilkbachola, the team will review this PR and let you know if it will be merged in.


path error in error message fix

This provides a way for the errors to fail properly and not to fail on the failure of a missing err.stack

Coverage remained the same at 77.586% when pulling 2654ada10b9ac08463c5cdd9f4d49a76d0918c64 on jesusalc:master into 9e911ebdfc814c579eb1e52c43fe1608266d56ca on mongodb-js:master.

Fantastic! Thanks! Published in<EMAIL_ADDRESS>


PaymentRequest.show() Not working || Error: Payment process canceled by user

const paymentRequest = new PaymentRequest(
      [IOS_PAYMENT_METHOD_DATA],
      DETAILS,
    );
    paymentRequest
      .canMakePayment()
      .then(async (canMakePayment: boolean) => {
        console.log('canMakePayment', canMakePayment);
        if (canMakePayment) {
          try {
            const paymentRes = await paymentRequest.show();
            console.log('paymentRes', paymentRes);
          } catch (error) {
            console.log('error', JSON.stringify(error));
          }
        } else {
          showTransactionError();
        }
      })
      .catch(() => showTransactionError());

I keep getting this error on iOS 17.5
{
  "code": "payment_error",
  "message": "Payment process canceled by user.",
  "domain": "RCTErrorDomain",
  "userInfo": null,
  "nativeStackIOS": [
    "0   React                               0x0000000105b937c8 RCTJSErrorFromCodeMessageAndNSError + 112",
    "1   React                               0x0000000105b37208 __41-[RCTModuleMethod processMethodSignature]_block_invoke_2.73 + 152",
    "2   alrajhicapital                      0x0000000100f1645c -[Payments rejectPromise:message:error:] + 196",
    "3   alrajhicapital                      0x0000000100f13b98 __56-[Payments paymentAuthorizationViewControllerDidFinish:]_block_invoke + 56",
    "4   UIKitCore                           0x00000001936a22e4 1741FA37-4E53-371E-8DAE-D611AAB0043D + 2851556",
    "5   UIKitCore                           0x0000000193c3c554 1741FA37-4E53-371E-8DAE-D611AAB0043D + 8725844",
    "6   UIKitCore                           0x00000001936a1d88 1741FA37-4E53-371E-8DAE-D611AAB0043D + 2850184",
    "7   UIKitCore                           0x00000001936fb9ac 1741FA37-4E53-371E-8DAE-D611AAB0043D + 3217836",
    "8   UIKitCore                           0x000000019379fe30 1741FA37-4E53-371E-8DAE-D611AAB0043D + 3890736",
    "9   UIKitCore                           0x000000019379fab4 1741FA37-4E53-371E-8DAE-D611AAB0043D + 3889844",
    "10  UIKitCore                           0x000000019347a91c __UIVIEW_IS_EXECUTING_ANIMATION_COMPLETION_BLOCK__ + 36",
    "11  UIKitCore                           0x000000019347a800 1741FA37-4E53-371E-8DAE-D611AAB0043D + 591872",
    "12  UIKitCore                           0x0000000193479518 1741FA37-4E53-371E-8DAE-D611AAB0043D + 587032",
    "13  UIKitCore                           0x0000000193486b14 1741FA37-4E53-371E-8DAE-D611AAB0043D + 641812",
    "14  UIKitCore                           0x0000000193486b84 1741FA37-4E53-371E-8DAE-D611AAB0043D + 641924",
    "15  QuartzCore                          0x0000000192828c50 A4B65B35-9EEB-38E4-9D4F-7146AAE1E28C + 318544",
    "16  libdispatch.dylib                   0x00000001086d27bc _dispatch_client_callout + 20",
    "17  libdispatch.dylib                   0x00000001086e2d58 _dispatch_main_queue_drain + 1084",
    "18  libdispatch.dylib                   0x00000001086e290c _dispatch_main_queue_callback_4CF + 44",
    "19  CoreFoundation                      0x00000001911bf710 00E76A98-210C-3CB5-930B-F236807FF24C + 354064",
    "20  CoreFoundation                      0x00000001911bc914 00E76A98-210C-3CB5-930B-F236807FF24C + 342292",
    "21  CoreFoundation                      0x00000001911bbcd8 CFRunLoopRunSpecific + 608",
    "22  GraphicsServices                    0x00000001d606c1a8 GSEventRunModal + 164",
    "23  UIKitCore                           0x00000001937f490c 1741FA37-4E53-371E-8DAE-D611AAB0043D + 4237580",
    "24  UIKitCore                           0x00000001938a89d0 UIApplicationMain + 340",
    "25  alrajhicapital                      0x0000000100ce49ec main + 96",
    "26  dyld                                0x00000001b486de4c 71846EAC-EE65-3697-BF7D-790B6A07DCDB + 249420"
  ]
}

having the same issue, did you managed to solve it ?

Here's a refined version of your comment:

I'm facing the same issue. Were you able to solve it, @TheAhmadSami?

I looked into the Merchant IDs section in Xcode and checked these two lines:

Here's the updated code that worked for me:
const applePayRequest2 = async () => {
  const DETAILS = {
    displayItems: [
      {
        label: 'Amount',
        amount: { currency: 'USD', value: '500' },
      },
      {
        label: 'Fees',
        amount: { currency: 'USD', value: '10' },
      },
      {
        label: 'VAT',
        amount: { currency: 'USD', value: '5' },
      },
    ],
    total: {
      label: 'Company Name',
      amount: { currency: 'USD', value: '515' },
    },
  };

  const paymentRequest = new PaymentRequest(
    [IOS_PAYMENT_METHOD_DATA({})],
    DETAILS,
  );
  paymentRequest
    .canMakePayment()
    .then((canMakePayment) => {
      if (canMakePayment) {
        paymentRequest.show().then((paymentResponse) => {
          paymentResponse?.complete(PaymentComplete.SUCCESS);
          const { paymentData, paymentMethod, transactionIdentifier } =
            paymentResponse?.details?.applePayToken;

          // Continue processing the payment
        });
      } else {
        showTransactionError();
      }
    })
    .catch(() => showTransactionError());
};

Here's my IOS_PAYMENT_METHOD_DATA function:
export const IOS_PAYMENT_METHOD_DATA = ({
  supportedNetworks,
}: {
  supportedNetworks?: SupportedNetworkEnum[];
}): IosPaymentMethodDataInterface => {
  return {
    data: {
      merchantIdentifier: CURRENT_ENV?.APPLE_PAY_MERCHANT_ID,
      supportedNetworks: supportedNetworks || [
        SupportedNetworkEnum.Visa,
        SupportedNetworkEnum.Mastercard,
        SupportedNetworkEnum.Mada,
      ],
      countryCode: 'US',
      currencyCode: 'USD',
    },
    supportedMethods: PaymentMethodNameEnum.ApplePay,
  };
};

@TheAhmadSami  Thank you, my case was similar, the merchant ids were enabled for release only.


WAV file to SD card

Hi!
First of all - your solution is magic, thanks for your work!))
Can you please provide an example of how I can write ADC samples as to *.wav file to SD card?

I'll second this request 👍

Currently trying the same thing, your code is a goldmine for well thought-out C++ practices. However, writing to SD comes with a few timing and buffer issues which are quite hard to spot if you don't know how SD-cards work, e.g. it can only write 512 bytes at a time, how is a wav-file structured/padded and giving your SD-card large buffers at slower rates, otherwise the SD-card will slow down the reading of samples. On that note, i don't quite understand your "i2sReaderTask": It features a xQueueReceive from a queue that never stored anything, since i can't find a "xQueueSend". Im guessing you are using it as a method of synchronisation, since just polling for samples can fluctuate wildly in timing and accuracy. Sadly, most tutorials online use that method. But how does it work in your case? The queue should be filled with an event, a notification so to speak, but what triggers this? I'd expect a feature like with the STM32 HAL, where you would configure a register to throw an event into a pre-named queue, but i can't find that in your code. I'd really love to hear you answer and discuss a bit, as well as with other users. Cheers

Hi
The queue comes from this line of code in I2SSampler.cpp line 63.
i2s_driver_install(m_i2sPort, &i2sConfig, 4, &m_i2sQueue);

It's not documented very well (if at all) but if you dig into the code here - https://github.com/espressif/esp-idf/blob/master/components/driver/i2s.c
You can see on line 985-991 they create a queue for you.
Events are pushed onto the queue by the i2s driver. The one we are interested in is the event it sends when it has filled one of the DMA buffers - which is I2S_EVENT_RX_DONE.
In my code there are two layers of buffering - we have the DMA buffers which the i2s driver reads samples into. Every time one of those gets filled our i2sReaderTask wakes up and empties the contents over to our audio buffer.
I've tried to decouple the writing of the samples from the DMA by having my own quite large audio buffers. So we can have a pretty fast function to keep pulling the samples from the DMA buffers. And a much slower function to actually send the samples somewhere.
We leave the reading of the samples to the i2s device driver and DMA so we should get a pretty consistent sampling rate. However, this does mean that there is some latency in the system. This will depend on the size of the DMA buffers you configure the i2s driver with.

Smaller buffers = lower latency - but at the cost of having to empty the buffers more often so more CPU load
Larger buffers = higher latency - but lower CPU load

Hope that helps!

I need to actually get an SD card breakout board - which just thinking about makes me think that I probably have one somewhere!
I'll dig through the boxes and see.

ooh so that's the "register" - thing i mentioned in reference to the STM32 HAL.  Very newbie-hostile. Goes to show that i don't know the espressif i2s source code too well yet. Thank you for your quick and well explained answer. My code now jumps to a breakpoint set inside the if-statement which asks for the queue-event. Would you recommend pinning the reader task to a core and manage everything else over the other or is freeRTOS more efficient in managing itself freely?
I didn't know about the open source code up until half an hour ago when a user on stackoverflow pointed it out. I asked similar questions there. I hope i can progress with my code and then open a repo of my own, since working with esp32 and i2s is very cool but still kind of obscured.
Again thanks for the quick advice and your videos, they sure do help de-mystify the whole topic

To be honest, it's not really documented anywhere and you shouldn't have to go spelunking through the code to find how it works!
Regarding pinning to a core - it really depends on what else you have going on. Arduino puts a lot of stuff (WiFi and Bluetooth) on core 1, so it's often good to avoid that and pin your own stuff on core 0. But I've also heard people say not to bother and let freeRTOS sort it out - so it may be a case of trying it and see if it makes any difference.
You should definitely put your code up when it's done - it's great to have a portfolio of stuff you've worked on that you can show people.

Hi again, i figured i'd notify you or other users since you helped me so quickly and kindly. I've created a repository for the entirety of my fieldrecorder-project here: https://github.com/jake-is-ESD-protected/I2S-MEMS-mic-field-recorder
Maybe someone is interested in this unfinished code, which i will be updating every few days. The saving of wav-files works, but comes with every audio-issue i could think of...

very low gain
negative offset
loss of samples (short timing intervals)
loss of samples and replacement with 0s (long timing intervals)
loss of every second sample it seems, because the recording is twice as fast as the RL-sound but is not pitched higher

It's my job the next few days to figure out these issues and fix them, although the sheer amount of knobs and gears i could turn here are a bit overwhelming. The best combination of buffer-size, speed and program-flow is out there and i'm trying to find it.

There's a good example here as well:
https://github.com/atomic14/esp32_sdcard_audio


User struct missing the groups key

Checklist

[X] I have looked into the README and have not found a suitable solution or answer.
[X] I have looked into the documentation and have not found a suitable solution or answer.
[X] I have searched the issues and have not found a suitable solution or answer.
[X] I have upgraded to the latest version of this SDK and the issue still persists.
[X] I have searched the Auth0 Community forums and have not found a suitable solution or answer.
[X] I agree to the terms within the Auth0 Code of Conduct.

Description
The User struct is not returning the groups value when called via the management API, however this gets returned when calling the API directly.
Would you be ok with me sending a PR adding this?
Expectation
The User struct includes a groups property
Reproduction

Given I use the users search management API via this SDK
I can't read the value of the groups because the User struct is not including it even thought the response is

Auth0 Go SDK version
v0.14.0

For now I got around this limitation by calling the API directly bypassing the SDK but it would be nice for the SDK to support retrieving all the fields the API returns and not just a subset

Hey @robertmarsal 👋🏻
Thanks for opening an issue. Would you mind offering a bit more context into the ask as this is not part of the public API schema found at https://auth0.com/docs/api/management/v2#!/Users/get_users_by_id. Something similar to this https://github.com/auth0/go-auth0/pull/153#issue-1557542390 would be great 🙇🏻
Thanks!

Hi @sergiught I probably don't have a solid argument on adding this to the User struct as this is an internal custom field used by my company however this is being returned by the API when calling it directly without using the SDK so there should be a way to read these values with the SDK as well for parity and consistency.
Maybe the User struct can contain an extra map field with all the non standard fields and consumers can read their custom values from there?

Hey @robertmarsal thanks for the extra info! Indeed the User struct at the moment only supports the Auth0 API json tags and doesn't take into account that other identity providers could enhance the struct with additional fields.

Maybe the User struct can contain an extra map field with all the non standard fields and consumers can read their custom values from there?

This is something we'll have to discuss internally and find a solution for. Until then I'll set this issue as a feature request instead of a bug, as it would be more appropriate. I'll circle back when I have an update on this.

Hey @robertmarsal, we've added an example of how this can be achieved by creating a custom struct and using the Request func exposed by the management SDK to make requests rather than the SDKs User funcs. In a future release we might look to improve upon this but we want to make sure we expose this functionality in the right way.
You can check it out here and let me know if you have any questions!


Fix using item links together with remote items

This fixes a new bug report that came in. When I made remote items also immediately acquire the items on touch, that entailed then discarding self->self item receives, which come from the network a moment later. It turns out, self->self item receive is also used for a separate mechanism called item_links.
Briefest overview of item links aka groups (post-seed-rolling):
When an item link item is collected, it gets sent to the AP server with target player ID == item link ID. This is a player ID that is actually a very simple group of other player IDs, which are all real players.
E.g.,
player id 0 = 'Archipelago'
player id 1 = 'p1'
player id 2 = 'p2'
player id 3 = 'itemlinkname'
groups[3] = {1, 2}

p1 wanting to send an item to the link, sends it to player id 3. Then the server splits that into N messages sent to the players in the link:
p1 -> p1
p1 -> p2

Thus the item link id is no longer used on the receive side; the items always appear as if they came directly from the player who found them.
Implementation (ROM)
Add a new item destination type $0002 = SM item link item that sends to the current player and others. This gets:

New dialog box showing the item is for self and others
Item acquiring code is run immediately
Additional handling of item received messages from the network

Look up player id -> player index early on (now somewhat optimized, former O(n) was expensive and might cause lag)
Allow config: remote_items=disabled worlds to collect p1 -> itemlink -> p1 items via network. I think this would require a client change to finish making this happen. Do you think that would be worthwhile? I think it's a bit weird otherwise: you'd collect, say, a Speed Booster for all the Super Metroid players along with yourself, but if you die, you'd be the only one in the item link without that item. A client change may be needed for this case regardless (it might send a doubled item?), which I haven't made yet. I have updated Generate locally per below.

Follow-up
A bit more testing to be done tomorrow. I'll be filing the corresponding PR updating AP's worlds/sm/__init__.py to:

sort the player id list
include player names of any player who happens to be able to send items to this world via item links only (currently missing). This also removes any item link names that this world is part of but doesn't have any items to send to, as those would never appear in-game.

This should be ready. 2nd commit resolves the issue with non-remote-items:
Since the client thru current 0.3.5 limits what data is shared with the non-remote-items games, we have the following choices:

When under non-remote-items config: drop all p1->p1, itemloc=0 messages received, or
When under non-remote items config: go remote-only for item link items (the way it is today, with 2 message boxes; but kind of ironic/unintuitive esp. after the rest of the new PR makes these items work local+remote when remote items is on), or
Update the client for this situation, and deny 0.3.5 and older clients from connecting to 0.3.6+ worlds, or just to 0.3.6+ non-remote-item worlds with item links.

I went with option (1). This does mean that the player who found the item link item can deprive themself of it by dying or reloading to an earlier save, if they have remote items disabled. But generally it makes item link items behave like local items from the perspective of the remote items=disabled player, so it may be more in line with their expectations than being able to receive the item from themselves, not less.
Tested in a 6player multi (5 sm, 1 non-sm) with 4 item links groups. It went completely smoothly.

That seems pretty good and thorough to me. Thank you!


deb: explicitly install docker-ce-cli, scan-plugin, and docker-ce

internal ticket: POS-27
supersedes / replaces https://github.com/docker/docker-install/pull/195 (except for the "containerd.io" part)
closes https://github.com/docker/docker-install/pull/195
The scripts is using "--no-install-recommends" to provide a more minimal install, and to prevent installing "latest" version of (e.g.) the engine when installing the CLI.
This patch:

adds some utilities to compare versions
skips CLI-version discovery on older versions (before 18.09)  which did not have separate packages for the CLI
on versions 20.10 and up (or if no version was specified), also install the scan cli plugin (support for cli plugins was added   in docker 19.03, but not installing on that version by default))

@chris-crone @glours PTAL

rebased, after https://github.com/docker/docker-install/pull/236 was merged


(Testing, Do Not Merge) ROX-13378 - PR to test UI

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

/retest

/retest

Receiving a plethora of notifications. Since this PR is used for testing, removing @stackrox/core-workflows from the reviewer. Add it back when testing is complete or when necessary.


The Android Gradle plugin supports only Kotlin Gradle plugin version 1.5.20 and higher.

FAILURE: Build failed with an exception.

What went wrong:
The Android Gradle plugin supports only Kotlin Gradle plugin version 1.5.20 and higher.
The following dependencies do not satisfy the required version:
root project 'android' -> org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.50

same

Same

This basically means that the gradle version used in the plugin itself is outdated.
I was able to solve this error by manually upgrading the org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.50 in the AR plugin itself. -> you can edit the source code of plugins in Android Studio. I set it to 1.5.20
Steps I took:
1)Open project in Android Studio,
2) Below project there should be a folder "External Libraries"
(If you cant see the external libraries, make sure project is selected in the dropdown)

Open External Libraries>Flutter plugins>ar_flutter_plugin>android>build.gradle
In that file, update the ext.kotlin.version.

buildscript {
    ext.kotlin_version = '1.5.20'
}

Hope this solves it for you!

This basically means that the gradle version used in the plugin itself is outdated.
I was able to solve this error by manually upgrading the org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.50 in the AR plugin itself. -> you can edit the source code of plugins in Android Studio. (as far as i know, not in VsCode)
Steps I took: 1)Open project in Android Studio, 2) Below project there should be a folder "External Libraries" (If you cant see the external libraries, make sure project is selected in the dropdown)

Open External Libraries>Flutter plugins>ar_flutter_plugin>android>build.gradle
In that file, update the ext.kotlin.version.

buildscript {
    ext.kotlin_version = '1.5.20'
}

Hope this solves it for you!

thanks

This basically means that the gradle version used in the plugin itself is outdated.
I was able to solve this error by manually upgrading the org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.50 in the AR plugin itself. -> you can edit the source code of plugins in Android Studio. (as far as i know, not in VsCode)
Steps I took: 1)Open project in Android Studio, 2) Below project there should be a folder "External Libraries" (If you cant see the external libraries, make sure project is selected in the dropdown)

Open External Libraries>Flutter plugins>ar_flutter_plugin>android>build.gradle
In that file, update the ext.kotlin.version.

buildscript {
    ext.kotlin_version = '1.5.20'
}

Hope this solves it for you!

This basically means that the gradle version used in the plugin itself is outdated.
I was able to solve this error by manually upgrading the org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.50 in the AR plugin itself. -> you can edit the source code of plugins in Android Studio. (as far as i know, not in VsCode)
Steps I took: 1)Open project in Android Studio, 2) Below project there should be a folder "External Libraries" (If you cant see the external libraries, make sure project is selected in the dropdown)

Open External Libraries>Flutter plugins>ar_flutter_plugin>android>build.gradle
In that file, update the ext.kotlin.version.

buildscript {
    ext.kotlin_version = '1.5.20'
}

Hope this solves it for you!

I am not getting the option to edit it. Where i can get?

I need the actual solution. Anyone?


Imagestream error on experimental mode

What type of PR is this?
missing integration test
What does does this PR do / why we need it:
Adding integration test coverage for imagestream error on experimental mode
Which issue(s) this PR fixes:
Fixes - missing integration test of https://github.com/openshift/odo/pull/3265
How to test changes / Special notes to the reviewer:
make test-cmd-devfile-create

as per discussion, added changes of this PR on https://github.com/openshift/odo/pull/3322


VRaw is not hydratable

Problem
Now that #2842 is implemented we can finally use raw HTML using server-side rendering 🎉
Unfortunately, it seems like hydration is not supported at this time when using this feature.
I would like to understand why this is the case and as a follow-up question: should it be supported?
Steps To Reproduce

Use VNode::from_html_unchecked
Enable SSR + Hydration
Error: VRaw is not hydratable (raw HTML string cannot be hydrated)

Expected behavior
The best outcome would be that hydration is supported but I have no clue wether that's possible.
Environment:

Yew version: master
Rust version: 1.65.0

Questionnaire

[x] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[x] I don't have time to fix this right now, but maybe later

@hamza1311 Can you maybe provide some insight on this?

VRaw can't be hydrated because there's nothing to hydrate. It's a simple HTML string
Maybe panicking is a bad idea. Would you mind PRing a fix to remove the panic?

During hydration, one would still have to figure out the number of children to collect and perhaps in debug mode assert that the inner_html matches (although the latter part might cause unexpected mismatches if done too naively, because the parsed html is sometimes slightly "fixed" by the browser, e.g. the insertion of tbody tags and more). The former part could be done by inserting additional comments like is done around <Suspense>.

VRaw can't be hydrated because there's nothing to hydrate. It's a simple HTML string

Check, that makes sense

Maybe panicking is a bad idea. Would you mind PRing a fix to remove the panic?

Yes, will do by the end of this week!

During hydration, one would still have to figure out the number of children to collect and perhaps in debug mode assert that the inner_html matches (although the latter part might cause unexpected mismatches if done too naively, because the parsed html is sometimes slightly "fixed" by the browser, e.g. the insertion of tbody tags and more). The former part could be done by inserting additional comments like is done around <Suspense> - would have to be matched on server-side during SSR too.

I will have a look. Are you okay with removing the panic first? Or do you want to implement this before that?

I think simply removing panicking would not work as next_sibling will also need to be pointed properly for the component / element before VRaw or any update to the element before VRaw will cause it to be inserted to the end of the child nodes of their common parent.

I'll take some time this week to figure out how this works. Seems like it's not that easy as I initially thought but that's life!


Default syslog configuration doesn't work

If the Syslog setting in openarc.conf (default yes) is not explicitly set to yes, no logs will be output.
Even messages such as the following are not output in starting or stopping:
OpenARC Filter v1.0.0 starting…
OpenARC Filter v1.0.0 terminating…
In openarc/openarc.conf.sample, the default value for the "Syslog" setting is "yes".
Therefore, I expected that Syslog would be output if "Syslog" was not explicitly configured.

The documentation in that file is incorrect. The default value for this setting is false.


Removed empty value from default custom_remote in attributes

Description
When checking to see if you add a 49-remote.conf file using custom_remotes attribute, the first value in the array is checked, but because Chef merges the attributes the first one will always be blank if you choose not to have a default custom_remote.
Issues Resolved
Not having a default custom_remote but having one in a role/environment will now work correctly
Check List

[ ✔ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ✔ ] New functionality includes testing.
[ ✔ ] New functionality has been documented in the README if applicable
[ ✔ ]  All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

@Sparwilk This DCO sign-off must be included in the commit message of each of these commits before we can merge this change.

Merging under the obvious fix policy


WIP change std::size_t to xt::index_t

This adds a new type xt::index_t which replaces the use of std::size_t with a integer type that can be choosen at configuration time to be signed or unsigned.
A lot of our tests pass, but there will be many warnings (currently disabled) and some tests fail + xio hangs somewhere in an endless loop (so printing is not possible).
cc @ukoethe

Also some new index_ts should probably be legitimate std::size_t ...

Locally, all tests were building fine. I guess now we have to decide if we want to enable this in xtensor @JohanMabille @SylvainCorlay :)
I think it's cool to have the option to switch.

https://github.com/isocpp/CppCoreGuidelines/pull/1115

Closing this PR in favor of #925


Chapter 2 - Version of Protocol

The text mentions the "Version of Protocol" header element, but the screenshots do not contain it. I'm assuming there's a default value.
While it may not be central to the book's subject matter, since it is mentioned, you may want to include information or link to information on what happens with regard to protocol version if none is specified.
On the other hand, HTTP 2.0 is all about reducing latency, so maybe it is almost central to the book's subject matter.
-Pat

Click to see Hill for issue #230

Click to see Hill for all issues

created by Async

@callahanp If this is still relevant, would you please clarify your question? Please make sure you check book since content got a major update.


Unable to update alert webhook from CLI

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az monitor metrics alert update
Errors:
'MetricAlertAction' object has no attribute 'webhook_properties'
Traceback (most recent call last):
python3.8/site-packages/knack/cli.py, ln 233, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 659, in execute
    raise ex
cli/core/commands/__init__.py, ln 722, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
...
cli/core/commands/arm.py, ln 527, in handler
    instance = custom_function(instance=instance, **custom_func_args)
command_modules/monitor/operations/metric_alert.py, ln 89, in update_metric_alert
    match.webhook_properties = action.webhook_properties
AttributeError: 'MetricAlertAction' object has no attribute 'webhook_properties'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

None
az monitor metrics alert update --name {} --add-action {} --resource-group {}

Expected Behavior
Environment Summary
Linux-5.4.72-microsoft-standard-WSL2-x86_64-with-glibc2.29
Python 3.8.5
Installer: PIP

azure-cli 2.18.0 *

Extensions:
scheduled-query 0.2.1
interactive 0.4.4

Additional Context

monitor

That's because you didn't install property requirements of CLI. You can run this command to update CLI and install requirements automatically
az upgrade


Adding a param description at [Config Wiki]

Describe
Adding a param description at Config Wiki to set the maximum connection time
**connectTimeout**
number - the maximum connection time

Reason
in my development environment, the handshake time is about 20s, but the default time in the plugin is 10s.
I find the parameter 'connectTimeout' when i debug the conection，but this parameter is not metioned in the config wiki.
Screenshots
in debug console:
[info] config at f:\****** {"remotePath":"xxxx","uploadOnSave":true,"downloadOnOpen":false,"ignore":[],"concurrency":4,"protocol":"sftp","connectTimeout":10000,"interactiveAuth":false,"secure":false,"remoteTimeOffsetInHours":0,"name":"My Server","host":"xxxx","port":22,"username":"******","password":"******"}

Done.


Display what components / build builds work (nodejs, ruby, java, etc.) and detect custom ones.

ocdev create should provide better help / description outlining what you can do with the components. For example, listing that:

Ruby
Node.JS
Java
PHP

All work with OpenShift.
Also, instead of erroring out, ocdev should also check OpenShift for what custom Image Builds are available and list them all.

@cdrage part of this can be resolved with using ocdev catalog list -
$ ocdev catalog list
The following components can be deployed:
- httpd
- nodejs
- perl
- php
- python
- ruby
- wildfly

Do you suggest that we add more description with every component?

Also, instead of erroring out, ocdev should also check OpenShift for what custom Image Builds are available and list them all.

So besides the error, do you suggest that we list all the available components?

@containscafeine Yes on both. I suggest we improve the description as well as the error out better :+1: I'll assign this to myself.

PR: https://github.com/redhat-developer/odo/pull/335

This has been merged :+1:


Output masking exception when outputting sequences

I get the error:
ERROR: test_graph_seq (test_loss_weighting.TestLossWeighting)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/blake/src/keras/tests/auto/test_loss_weighting.py", line 162, in test_graph_seq
    model.fit(data, nb_epoch=1, batch_size=2, verbose=1, sample_weight=masks)
  File "/home/blake/src/keras/env/lib/python3.4/site-packages/keras/models.py", line 678, in fit
    shuffle=shuffle, metrics=metrics)
  File "/home/blake/src/keras/env/lib/python3.4/site-packages/keras/models.py", line 211, in _fit
    outs = f(*ins_batch)
  File "/home/blake/src/keras/env/lib/python3.4/site-packages/theano/compile/function_module.py", line 513, in __call__
    allow_downcast=s.allow_downcast)
  File "/home/blake/src/keras/env/lib/python3.4/site-packages/theano/tensor/type.py", line 169, in filter
    data.shape))
TypeError: ('Bad input argument to theano function with name "/home/blake/src/keras/env/lib/python3.4/site-packages/keras/models.py:616"  at index 2(0-based)', 'Wrong number of dimensions: expected 3, got 2 with shape (2, 4).')

when attempting to use output masking when outputting sequences. Either I'm doing something wrong, or there's a bug. I have created unit tests that exposes this issue in https://github.com/brainwater/keras/tree/outmasktest
n.b. This happens in both python 2 and python 3
It can be run with:
$ python -m unittest test_loss_weighting.TestLossWeighting.test_graph_seq
or
$ python -m unittest test_loss_weighting.TestLossWeighting.test_sequential_seq

Was it be fixed?

I think this works if one just makes each weight a list of length 1, e.g. for the sample weights for my example tests:

       'output': [[1.,1.,1.,1.],

                  [1.,1.,0.,0.]],

       'output': [[[1.],[1.],[1.],[1.]],

                   [[1.],[1.],[0.],[0.]]],

Which seems like a bug to me, but this may be the intended behavior.

Hi,guys! @brainwater @tttwwy @bmabey @jfsantos @kashif
Would you help me to solve this problem when I was getting the feature of Flatten layer.#431 Any help will be much appreciate!


[cxx-interop] Cast "data" to the correct type in _swift_dispatch_data_apply.

The implicit conversions are OK in C but C++ will error. To make this header valid in both C and C++ we should just always cast.
This and https://github.com/apple/swift-corelibs-foundation/pull/2895 fix the Linux builds for #32721.

@swift-ci test

Build failed
Swift Test OS X Platform
Git Sha - fe2f99f33b65193de4f3e1bbeb2953aa355a2ead

Java error. Unrelated.
@swift-ci please test OS X Platform.

@swift-ci please test OS X.

Thanks, @varungandhi-apple. I'll commit this tomorrow unless anyone else wants to review.

It's possible that this [1] build failure was caused by this patch. A (full) OS X test did succeed for this patch before it landed, though. Should I revert or wait until the next build also fails (or doesn't fail). I'll plan on the latter unless I hear otherwise.
[1] = https://ci.swift.org/job/oss-swift-incremental-RA-osx/12891/

I think the failure:
<unknown>:0: error: fatal error encountered during compilation; please file a bug report with your project and the crash log
<unknown>:0: note: Cannot read '/Users/buildnode/jenkins/workspace/oss-swift-incremental-RA-osx/buildbot_incremental/swift-macosx-x86_64/./lib/swift/iphoneos/layouts-arm64.yaml'

is unrelated to this patch. I think we've seen it sometimes non-deterministically, I suspect it is an incremental build issue.

I think we've seen it sometimes non-deterministically, I suspect it is an incremental build issue.

I think you're right. That makes the most sense. I just saw all the other commits were all lldb stuff and this was a failure while building the stdlib. Thanks for looking into it :)


request: use declare

Is there an existing issue or pull request for this?

[X] I have searched the existing issues and pull requests

Feature description
A "problem" with overridable types. This one is just my personal solution to have accurate types wherever possible.
Command#piece appears to be Store<Piece<PieceOptions>> still when it should be CommandStore instead.
Desired solution
Ever considered using the declare modifier for class properties? Not much of a big deal anyway since the developer could just assert it to a specific store type but IMO it's helpful since it offers a shorter way of accessing the store that holds the piece.
Example:
// Command.ts
export class Command extends Piece {
  public declare store: CommandStore;
}

// help.ts
export default class extends Command {
  public override messageRun(message: Message) {
    const commands = [...this.store.values()]; // Command<Args, CommandOptions>[]
    console.log(commands.map(c => c.name));
  }
}

Alternatives considered
Use the common way of obtaining a store.
container.stores.get('store');

Additional context
No response

Why do we need this? It just seems more complicated.

Why do we need this? It just seems more complicated.

It's not. This overrides the types from the parent class to the inherited one. It just doesn't look right if you access the command store inside a command as Store<AliasPiece<PieceOptions>> when it should be CommandStore instead.

@sapphiredev pack


I want toolbar to be designed by me to add tabs for example

Thank you very much for this great project and very good documentation.
I just found that Toolbar is included out of the box while I need to design it in XML to add tabs to handle this.
I would do that by setActionBar but I dont want to break your functionality.
Also I think I can getToolbar and modify it by code. But I think it is better to have a way for customizing Action Bar for common scenarios such as tabs.
Thank you,
Hisham

If you Tabs example is activity than you can simple extend ActionBarActivity and initialize Toolbar by your own layout and style.
If it's an fragment, you can setSectionColor(,,,) method...
If you want to change toolbar when new tab is selected, then.. I simple don't know :laughing:

If you Tabs example is activity than you can simple extend ActionBarActivity and initialize Toolbar by your own layout and style.
If it's an fragment, you can setSectionColor(,,,) method...
If you want to change toolbar when new tab is selected, then.. I simple don't know :laughing:

Well, With new Toolbar the tabs should not be included in it. There is a reason if google separates it.
When I tried to create the materialTabs library for the first time I done this error. Because if you extends Toolbar for inflating your custom tabs you can't do it. Toolbar class supports only to have one view line like the LinearLayout. So I suggest you to create a custom tabs externally extending a Linear or ScrollableView, and use it adding it with the same primaryColor used in your theme at the top of your fragment (attached to the section as target). This new method is more simplier than the old tabs attached to the action bar and takes you the possibility to adds at the top, bottoms, or everywhere you want your tabs.

Well, With new Toolbar the tabs should not be included in it. There is a reason if google separates it.
When I tried to create the materialTabs library for the first time I done this error. Because if you extends Toolbar for inflating your custom tabs you can't do it. Toolbar class supports only to have one view line like the LinearLayout. So I suggest you to create a custom tabs externally extending a Linear or ScrollableView, and use it adding it with the same primaryColor used in your theme at the top of your fragment (attached to the section as target). This new method is more simplier than the old tabs attached to the action bar and takes you the possibility to adds at the top, bottoms, or everywhere you want your tabs.

Convinced and will close it. Thank you.


zero filling missing genes in the query

hi @Cottoneyejoe95, could you please add that as an option if the query data has some missing genes then it should be zero filled? add and option call zero_fill=False in load_query function and explain if True it will fill in zeros for missing genes in query data

I added this functionality in the branch where the new version of scarches is stored atm. marco/trvae_refactor


Webpack 4 / MiniCssExtractPlugin issue

Is this a bug?
In production mode html markup works fine but css file prints -> background: url(path/from/disk/to/file/mysvg.svg).
It supposed to print: "sprite-467f4d.svg#mysvgid".
In dev mode where i don't use MiniCssExtractPlugin the console log prints the correct url.
Git: https://github.com/eglaost/webpack-boilerplate

Node.js version: 8.11.1
webpack version: 4.15.1
svg-sprite-loader version: 3.8.0
OS type & version: Win 10

Same here.

I try to use 3 css extractors with Webpack 4 with the same results:

extract-css-chunks-webpack-plugin
mini-css-extract-plugin
extract-loader + file-loader

I try to use 3 css extractors with Webpack 4 with the same results:

extract-css-chunks-webpack-plugin
mini-css-extract-plugin
extract-loader + file-loader

here I come...
@vortizhe
It should be in the file loader chain. because background images were all in file loader chain, and think about fonts, they are the same.

and if I put the publicPath in SVG-sprite-loader, the SVG path which is been used become 'publicPath/path/to/SVG'.
seems that sprite file name was replaced by the mini-css-extract-plugin.

@kisenka
hi, it's me again.
here Is the same repo, and when building with mini-css-extract-plugin the sprite URL was replaced with the real file path.
here is the result:

hope helps.

Hey! I'am planning to deprecate and remove extract mode from svg-sprite-loader. To replace it I wrote separate plugin which perform SVG extracting only – extract-svg-sprite-webpack-plugin. Webpack 3 and 4 are supported. It's under heavy development and I suggest you to take part in beta-testing of it. I love my users and want to make them happy! :)

@vicvinc @vortizhe @eglaost and, of course, I need your feedback, if something going wrong with this plugin.

@kisenka  me, troublemaker is glad to try this. got any situation will be feedback ASAP.

@kisenka, thanks for you work!

I'am planning to deprecate and remove extract mode from svg-sprite-loader. To replace it I wrote separate plugin which perform SVG extracting only – extract-svg-sprite-webpack-plugin

Could you please elaborate?

If extract mode is removed from svg-sprite-loader, what else remains there? I thought it's the main usecase? Does it mean that svg-sprite-loader is going to be deprecated entirely?
Do you recommend to completely switch to extract-svg-sprite-webpack-plugin? I'm asking, because Its weekly downloads counter looks low at npmjs, also there are some bugs (e.g. this PR which fixes a typo is not merged since August: https://github.com/JetBrains/svg-mixer/pull/69/files, also this issue https://github.com/JetBrains/svg-mixer/issues/59 is the 1st thing I see when I try using extract-svg-sprite-webpack-plugin by doing "import i from ".../xyz.svg").

same

@kisenka
hi, it's me again.
here Is the same repo, and when building with mini-css-extract-plugin the sprite URL was replaced with the real file path.
just use yarn build:production and you will see it.
here is the result:

hope helps.

Same to you. And did you found the solution finally?

@kisenka, thanks for you work!

I'am planning to deprecate and remove extract mode from svg-sprite-loader. To replace it I wrote separate plugin which perform SVG extracting only – extract-svg-sprite-webpack-plugin

Could you please elaborate?

If extract mode is removed from svg-sprite-loader, what else remains there? I thought it's the main usecase? Does it mean that svg-sprite-loader is going to be deprecated entirely?
Do you recommend to completely switch to extract-svg-sprite-webpack-plugin? I'm asking, because Its weekly downloads counter looks low at npmjs, also there are some bugs (e.g. this PR which fixes a typo is not merged since August: https://github.com/JetBrains/svg-mixer/pull/69/files, also this issue JetBrains/svg-mixer#59 is the 1st thing I see when I try using extract-svg-sprite-webpack-plugin by doing "import i from ".../xyz.svg").

I can't answer all of that, but I removed this plugin an switched to https://github.com/JetBrains/svg-mixer/tree/master/packages/extract-svg-sprite-webpack-plugin and everything now works perfectly

Wrote a fix for Webpack 5, so in case you stumble upon this issue or you need inspiration for the fix: https://github.com/JetBrains/svg-sprite-loader/issues/465#issuecomment-990993090


feat: Improve the help

This is heavily borrowed from pulumi command help.
go run ./pkg/cmd/ help
Nitric - The fastest way to build serverless apps

To begin working with Nitric, run the 'nitric new' command:

    $ nitric new

This will prompt you to create a new project for your cloud and language of choice.

The most common commands from there are:

    - nitric run   : Run a nitric stack locally for development or testing
    - nitric up    : Deploy code to a cloud and/or update resource changes
    - nitric down  : Tear down your stack's resources entirely from the cloud

For more information, please visit the project page: https://nitric.io/docs

Usage:
  nitric [command]

Available Commands:
  completion  Generate the autocompletion script for the specified shell
  down        Destroy an existing stack and its resources from the cloud
  help        Help about any command
  list        list stacks for a project
  new         create a new nitric project
  run         run a nitric stack
  stack       Manage stacks
  target      work with target objects
  up          Deploy code to a cloud and/or update resource changes
  version     Print the version number of this CLI

Flags:
      --config string          config file (default is $HOME/.nitric-config.yaml)
  -h, --help                   help for nitric
  -o, --output stringEnumVar   output format (default table)
  -v, --verbose int            set the verbosity of output (larger is more verbose) (default 1)

Use "nitric [command] --help" for more information about a command.

fixes #142

I have left out "stack new" as it does not exist yet..


Preparing for release.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


BUG: [GH 11738] Fix for end_time when multiple of a frequency is requested

Added a new test with the multiple of a frequency.
Rearranged some tests to improve readability of the code.

looks good. can you add a whatsnew note (bug fix), ping when green

@jreback Sorry, didn't follow contributing guidelines carefully in this PR. Please see PR #11779.

ok, np. yeah, just keeping the original PR is much better / easier to follow the flow


⚠ Thumbnails API Endpoint has degraded performance

In 9b5de6b, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1383 ms

Resolved: Thumbnails API Endpoint performance has improved in 9632ad2.