EHealth-962#4 - validate ingredient id uniqueness, refactored factories

Changes Unknown when pulling a489b4113e0c37f42c985fba88e042647ed8f330 on EHealth-962-4 into ** on master**.


Generate javadoc

We need to generate the javadoc for the entire project and attach it to our repo somewhere.

Generating isn't a requirement (from part 3)


feat(Order): Galois connection between pred and succ

This was a longstanding TODO. Kudos to Kevin for putting the pieces together!

Only hesitation is the import increase, otherwise LGTM. Happy if someone wants to merge regardless, or to ask for a split.

As the author of Order.SuccPred.Basic, I claim this import increase is fine 😄

bors merge


Setup a Next.js example

At the moment, we don't have an example app demonstrating our project. We should have one. (The example folder in the main branch is broken and doesn't do anything)
I tried creating a Next.js app inside the example folder at the root of the project using yarn create next-app example --ts and ended up with the usual Next.js starter template. I then did yarn add @web3-ui/core inside the example folder. This broke the app. (see SS below)

Can someone help me fix this so that we can import stuff from our packages and this example add doesn't break?

Did you add Web3ui/core as a dependancy?


404 Page Causes Infinite Reload After Using npx create-remix@latest

Reproduction

Install the latest version using one of the following methods:

With npx create-remix@latest
Or goto https://stackblitz.com/edit/remix-run-remix-rwkrfb

Start the development server

Navigate to http://localhost:5173/404 or any other non-existent page

System Info
System:
    OS: Linux 5.15 Arch Linux
    CPU: (16) x64 12th Gen Intel(R) Core(TM) i5-12500H
    Memory: 6.89 GB / 7.61 GB
    Container: Yes
    Shell: 3.7.1 - /bin/fish
  Binaries:
    Node: 20.14.0 - ~/.local/share/nvm/v20.14.0/bin/node
    npm: 10.7.0 - ~/.local/share/nvm/v20.14.0/bin/npm
    pnpm: 9.3.0 - ~/.local/share/pnpm/pnpm

Used Package Manager
pnpm
Expected Behavior
When visiting a 404 page, display the nearest or the root ErrorBoundary without causing an infinite reload
Actual Behavior
Infinity page reload when visiting a 404 page, both in development and in production

FWIW I cannot reproduce this using v2.10.3, but I can with v2.11.0

My codebase runs 2.10.3 and has the same issue:

Infinite 404 loop)
Stops when I disable the <Scripts /> component in my layout

Same here.

After downgrading @remix-run/serve from 2.11.0 to 2.10.3, it's fixed. Look like something is wrong in latest version.

To help narrow down the issue:

Only happens in dev server
Only happens in 2.11.0
I tried reverting commit ef612a729bf6acd6b9a11ba6c3971beb1f1760f1 as a first guess, without success

Yep, can confirm I ran into this very issue today. I had to downgrade to 2.10.x which doesn't have this bug.

It also happens on production servers built and deployed to Cloudflare.

Happens for me in both dev and prod with 2.11.0, doesn't happen with 2.10.3
Note to self: Remember you bumped packages before you encountered weird error and try to debug it

This should be fixed by v2.11.1.
Could you all confirm? 🙏🏼

Fixed for me


push-to-deploy using chef zero and secret management

Move away from hosted chef and client-server setup. It's much nicer to have everything versioned in git (right now, we need to go the the chef.io website to find out which version of cookbooks are used). For secret management, we could use https://vaultproject.io/ (via http://sirupsen.com/production-docker/#secrets)

[x] experiment with chef-solo (--> it's not a solution)
[x] replacement for chef vault? (nope, it needs chef server)
[ ] push to deploy instead of knife cookbook upload?

See also https://www.chef.io/blog/2014/06/24/from-solo-to-zero-migrating-to-chef-client-local-mode/.


Cluster & Edit box overflow makes it unusable on some screens

When clustering, the "Cluster & Edit" dialog box has styling issues: large text string values overflow the box, making the cluster visualizations and the ui buttons appear off screen. The overflowing box can not be dragged or scrolled into a usable position. The only solution is to zoom the browser window out to 50%. See attached images using Windows 10, Refine 2.6rc2. The overflow issue is worse on Chrome, but the box size with inaccessible ui buttons is on both Chrome and Firefox.
on Chrome:

on Firefox:

I work with text tables, so I have this issue regularly. I did not notice it when working on my desktop with large monitor, but with a new 14" laptop (FHD 1920 x 1080) and 13" laptop (3200x1800) it happens constantly making it impossible to cluster at times.
The issue seems to be mentioned in #938, but it is assigned as a minor, low priority issue. However, since this issue prevents features of the software from being usable, it seems like it should be a higher priority bug.

This is a general UI issue that affects many programs, not just OpenRefine.
The issue is really "How do we handle displaying long strings of information to a user?",  "Do we autowrap?", "Do not autowrap and instead create a horizontal scrollbar for the user to pan left or right to review long strings?"  "How far should the pan left or right be allowed, should their ever be a max limit?"
Some of these challenges are answered with a few Jquery plugins.
Here is one that comes to mind that could help with a horizontal scroll that auto updates and recalculates scrollbar sizes on javascript/ajax changes such as what happens during Clustering.
https://gromo.github.io/jquery.scrollbar

There is something clearly broken in the styling on Chrome however that could be an easy fix by looking at why it works in Firefox. Some odd browser prefix thing?
The usability issue for both browsers is that the pop up dialog-frame is set to width 1000px, the height is set by the content, but if it is taller than the browser window size, it is not possible to scroll down to use the full pop up since its container dialog-container is set to position: fixed. A quick fix to allow scrolling is to set dialog-container to position: relative, but that breaks the "draggable" functionality.
It might be an issue with deeper roots, since it looks like when rendering it must query the browser for window size to set the parameters of the page. However, on Win 10 it seems to get the size of the full screen rather than window, since a small portion of the UI is actually below the browser window, where the task bar is, and is inaccessible unless I go to full screen mode with the browser. This maybe an issue specific to HD screens that use scaling to have everything appear normal size on the screen.

As mentioned above, this is a duplicate of #938.


Add styling options for native toolbar

Given the limitations here: https://github.com/bitsdojo/bitsdojo_window/issues/8, it could be nice if there was an API so we can change the color of the native top btns and window bar. Is that possible?
It's really like 3 colors that need to be set to get a much nicer look and match app themes.

Unfortunately, Windows does not offer any clean API to change the color of the native top buttons and window bar so that is the main limitation when trying to implement something like this.

Haha, ya that makes sense! Also explains why I can find literally 0 examples of windows app with custom toolbars, using native menu_bar. Ah well, fact of life I guess :) Thanks for the quick response!


Set version to v6.1.1

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.938%

Totals

Change from base Build<PHONE_NUMBER>7:
0.0%

Covered Lines:
9297

Relevant Lines:
9897

💛  - Coveralls


: error: WRONGTYPE Operation against a key holding the wrong kind of value

What happened:
Trying to gc or fsck end in redis error.

2023/03/24 01:34:31.537100 juicefs[1798186] : found leaked chunk ca6b19fb2021894a35ddbf17caf40357/imagePath-1fba7e075940fa8e318b8efcc0782d2b-user_status-app.svg [redis.go:2277]
2023/03/24 01:34:34.014683 juicefs[1798186] : list slices: WRONGTYPE Operation against a key holding the wrong kind of value [redis.go:2752]
2023/03/24 01:34:34.015778 juicefs[1798186] : error: WRONGTYPE Operation against a key holding the wrong kind of value
goroutine 1 [running]:
runtime/debug.Stack()
/usr/local/go/src/runtime/debug/stack.go:24 +0x65
github.com/juicedata/juicefs/pkg/meta.errno({0x325bc60, 0xc0126ddc10})
/go/src/github.com/juicedata/juicefs/pkg/meta/utils.go:104 +0xc5
github.com/juicedata/juicefs/pkg/meta.(*redisMeta).ListSlices(0xc000292140, {0x3285c30?, 0xc006b32640}, 0xc006b3c270, 0x0, 0xc006667c20)
/go/src/github.com/juicedata/juicefs/pkg/meta/redis.go:2776 +0x1a9
github.com/juicedata/juicefs/cmd.fsck(0xc000bf6800)
/go/src/github.com/juicedata/juicefs/cmd/fsck.go:111 +0x81b
github.com/urfave/cli/v2.(*Command).Run(0xc0008c2900, 0xc000bf6740)
/go/pkg/mod/github.com/urfave/cli/v2@v2.4.0/command.go:163 +0x5dc
github.com/urfave/cli/v2.(*App).RunContext(0xc000682d00, {0x3271840?, 0xc000058090}, {0xc000bf6200, 0x4, 0x4})
/go/pkg/mod/github.com/urfave/cli/v2@v2.4.0/app.go:313 +0xb7d
github.com/urfave/cli/v2.(*App).Run(...)
/go/pkg/mod/github.com/urfave/cli/v2@v2.4.0/app.go:224
github.com/juicedata/juicefs/cmd.Main({0xc000bf6080?, 0x4?, 0x4?})
/go/src/github.com/juicedata/juicefs/cmd/main.go:86 +0x246d
main.main()
/go/src/github.com/juicedata/juicefs/main.go:29 +0x2e [utils.go:104]
2023/03/24 01:34:34.015950 juicefs[1798186] : list all slices: input/output error [fsck.go:113]

What you expected to happen:
That it would delete the bad chunk, or handle this kind of error in redis. How is it possible to get the redis key that make it crash ? This way I can just delete and re-run a gc/fsck. This is only used for backups so I can loose this chunk.
How to reproduce it (as minimally and precisely as possible):
I just need to run juicefs gc or fsck to make it crash
Environment:

JuiceFS version: 1.0.3+2022-12-27.e4bf15a
Cloud provider or hardware configuration running JuiceFS: Custom AMD Ryzen 7 3700X NAS build
OS:  TrueNas Scale Bluefin 22.12.1 (Debian 11 Bullseye)
Kernel : 5.15.79+truenas
Object storage : AWS S3
Metadata engine info: redis/stable,stable-security 5:6.0.16-1+deb11u2
Network connectivity: local network (LXC running on the same host)

The slice keys added by JuiceFS must be a Redis list. You may find the invalid keys with:
for k in `redis-cli --scan --pattern 'c*_*'`; do t=`redis-cli type $k`; if [ $t != 'list' ]; then echo $k $t; fi; done

Hey @SandyXSD ,
Thank you so much for your reactivity!
The redis key was the chunk "ca6b19fb2021894a35ddbf17caf40357/imagePath-1fba7e075940fa8e318b8efcc0782d2b-user_status-app.svg" id. Deleting it with the redis-cli solved it.
I don't know if you want to keep this issue open to handle these kind of errors more gracefully, or is it the expected behaviour ?
Thanks
letic

We'd better log more information to simplify the troubleshooting.


CAMEL-18590 - Camel-Azure components: Define a unique configuration for authentication - Azure Datalake

Description

Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[x] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

/component-test azure


package.json does not specify blessed's version

Had an antiquated version of blessed on my rig. Automatic installation with npm did not catch it, and got an undefined error at git-commander start. npm install blessed fixed it.

Thanks, I'll fix it.


[Mobile Support] Stretch to screen width, but keep height original

Hi guys,
I'm currently using PIXI v3 for a smaller-scale game. I'm used to bigger engines (in terms of size and features) like Unity, but PIXI is more suited for a game like this.
I'm currently testing in-browser but would like to deploy my game to mobile and have been testing in mobile browsers along the way. So far all good, but I would like to implement a full-screen resolution system I'm used to back in Unity.
It worked like this: the width of the screen is always stretched out, but the height was not scaled. Using this technique (which is default behaviour for all Unity games) you only have to take the height into account, and offers a great trade-off between 'lazy' scaling and keeping track of the screen manually, especially if you make a landscape or portrait-only game.
I would like to replicate this behaviour but so far haven't found anything useful.
Thanks in advance for the help!

I figured it out!
initRenderer = ->
  rendererOptions =
    antialiasing: false
    transparent: false
    resolution: window.devicePixelRatio
    autoResize: true

  GAME_WIDTH = 480
  GAME_HEIGHT = 800

  resize = ->
    # Determine which screen dimension is most constrained
    ratio = window.innerWidth / GAME_WIDTH
    # Scale the view appropriately to fill that dimension
    stage.scale.x = stage.scale.y = ratio
    # Update the renderer dimensions
    renderer.resize window.innerWidth, window.innerHeight
    screenHeight = window.innerHeight
    return

  # Create the canvas in which the game will show, and a
  # generic container for all the graphical objects
  @renderer = PIXI.autoDetectRenderer(GAME_WIDTH, GAME_HEIGHT, rendererOptions)
  # Put the renderer on screen in the corner
  renderer.view.style.position = 'absolute'
  renderer.view.style.top = '0px'
  renderer.view.style.left = '0px'
  # The stage is essentially a display list of all game objects
  # for Pixi to render; it's used in resize(), so it must exist
  @stage = new (PIXI.Container)
  # Size the renderer to fill the screen
  resize()
  # Actually place the renderer onto the page for display
  document.body.appendChild renderer.view
  # Listen for and adapt to changes to the screen size, e.g.,
  # user changing the window or rotating their device
  window.addEventListener 'resize', resize

This renders the stage with fixed-width but scales the ehight freely. Hope someone can use this as well!


Add new colors to Discord Dark theme for v1.9.8

New color variables are available in Element v1.9.8 (released today).
Preview of the changes can be found here: https://github.com/matrix-org/matrix-react-sdk/pull/7270#issuecomment-984937239
Fixes #10 too 🙂

Thanks!


Split cpu and reference implementation

It doesn't enable the cpu target yet(I plan to do that in a future PR).

Converts cpu::target to ref::target
Update driver so it can build and verify multiple targets
Update gpu_ops test to run for multiple targets
Split the verify tests into seperate cpp files

I dont think its necessary to add tests fro env functions, so I think this can be merged in as well.

Is this one still pending?  Hard to tell if it passed all required steps.

It looks like the job got cancelled, I am rebuilding it.


optimization of (private) brackets function

This brackets alternative is about 2.x fastest.
Please see the test for performance and results at jsperf.com

Coverage remained the same at 88.78% when pulling d4eeca556d53476b090079a9468eba660b3b4c68 on aMarCruz:patch-2 into 39ba93cef5f3c51a80473750904bd56617e58e38 on muut:master.

Please pull request only on the dev branch


⚠ T:Riza Api has degraded performance

In ce8c8ca, T:Riza Api (https://infra.triza.dev/api) experienced degraded performance:

HTTP code: 200
Response time: 1888 ms

Resolved: T:Riza Api performance has improved in bf4e115.


Correct imports for RN >= 0.40

Fix for build errors in iOS due to React Native 0.40's breaking change: https://github.com/facebook/react-native/commit/e1577df1fd70049ce7f288f91f6e2b18d512ff4d
I believe this will break older versions of RN.

Funny I didn't look here. Seems 3 different ppl have created this PR :D. Fixes #22

Just merged an identical PR that also fixes the statements in the .h.


Reconfigure tutorial

Multiple customers have had a difficult time accessing the tutorial as it is multiple clicks away in the wiki page. Bringing the tutorial directly to the landing page is critical. This will also allow better version control as the readme is the source of the tutorial.

Hi Dan, thanks for the request! I think eventually we'll want to merge the pb-16S-nf workflow to this repo, so it might be better to continue keeping the current tutorial in the Wiki?


TREZOR support for Callisto

This patch enables support for Trezor and Callisto.  It's currently mostly untested (the Callisto Testnet Faucet doesn't work at the moment).  If anyone wants to test, I have a live version on https://test.jochen-hoenicke.de/mew/  (don't use it with private key/mnemonics; it shouldn't phish your passwords, but do you trust me?)
I follow SLIP-44  and use 820 as coin id (and 1 for testnet), so it uses m/44'/820'/0'/0/0 as derivation path for the first address.  I noticed that ClassicEtherWallet uses the same addresses as Ethereum (not the same as Ethereum Classic!) but I think this is a mistake.
I didn't enable Ledger or other hardware wallet support. If this is wanted, this is easy to add.
I didn't commit the automatically generated files to keep the diff small and clean.

JFYI relevant commit to https://github.com/trezor/trezor-common/blob/master/defs/ethereum/networks.json


Line chart should have a minimum height for the chart area most likely

Or perhaps a set height. Right now the height it is given is split up to fit the legend, which can get very tall if there is a ton of ISPs selected. I dont think the legend getting taller should make the chart shorter.

This is not easily doable without getting the legend height in the wrapping component since we need to know how high the chart is and how high the count chart is to set the outer SVG height.


Extending mapped type doesn't allow extra properties when extended type has string literal values

TypeScript Version:  3.2.0-dev.20181011

Search Terms: extends mapped literal widening
Code
// Type '{ a: string; c: number; }' is not assignable to type '{ a: "a"; }'.
//   Object literal may only specify known properties, and 'c' does not exist in type '{ a: "a"; }'. [2322]
const f: <T extends { [P in keyof T]: { a: "a" } }>(literal: T) => void = () => {}
f({one: {a: "a", c: 1}})

// works
const a: "a" = "a"
f({one: {a, c: 1}})

// works
const f_: <T extends { [P in keyof T]: { a: string } }>(literal: T) => void = () => {}
f_({one: {a: "a", c: 1}})

// works
const f__: <T extends {one: { a: "a" }}>(literal: T) => void = () => {}
f__({one: {a: "a", c: 1}})

Expected behavior:
Code compiles for all cases.
Actual behavior:
Code fails when extending a mapped type with string literal properties:
test.ts:4:18 - error TS2322: Type '{ a: string; c: number; }' is not assignable to type '{ a: "a"; }'.
  Object literal may only specify known properties, and 'c' does not exist in type '{ a: "a"; }'.

Removing either the mapping, the string literal or the widenable input makes the code pass type checking.
Also notable is that the one.a property of the function argument is only inferred to be defined by the extended type in the first case. E.g. jumping to definition only does something in the first f() call.
This seems related to the a property being widened when mapped over.
Playground Link: http://www.typescriptlang.org/play/#src=%2F%2F Type '{ a%3A string%3B c%3A number%3B }' is not assignable to type '{ a%3A "a"%3B }'.
%2F%2F   Object literal may only specify known properties%2C and 'c' does not exist in type '{ a%3A "a"%3B }'. [2322]
const f%3A <T extends { [P in keyof T]%3A { a%3A "a" } }>(literal%3A T) %3D> void %3D () %3D> {}
f({one%3A {a%3A "a"%2C c%3A 1}})

%2F%2F works
const a%3A "a" %3D "a"
f({one%3A {a%3A a%2C c%3A 1}})

%2F%2F works
const f_%3A <T extends { [P in keyof T]%3A { a%3A string } }>(literal%3A T) %3D> void %3D () %3D> {}
f_({one%3A {a%3A "a"%2C c%3A 1}})

%2F%2F works
const f__%3A <T extends {one%3A { a%3A "a" }}>(literal%3A T) %3D> void %3D () %3D> {}
f__({one%3A {a%3A "a"%2C c%3A 1}})

Related Issues:

Possibly related: #27421

How does T extends { [P in keyof T]: { a: "a" } } differ from T extends { [s: string]: { a: "a" } in the real code that led to this issue? (In your example, you even just use T, so I assume that the real code behind this issue is more complex.)
Note that from the type system's perspective, T extends { [P in keyof T]: { a: "a" } } is circular and so some things don't work the way you would expect.

The original problem is in react-native's typings (

As you point out in the DefinitelyTyped PR, extending {[key: string]: ...} now requires the type parameter in export function create<T extends NamedStyles<T>>(styles: T): { [P in keyof T]: RegisteredStyle<T[P]> }; to have a string index, precluding the use of a plain interface without the index.
It seems like the pattern the code aims for is "for each property in an object, the value type must extend V", which seems difficult do do without referring to T or requiring an index signature.

I'm closing this issue since the underlying issue is By Design for Typescript itself, and a PR for a fix of @types/react-native is open at https://github.com/DefinitelyTyped/DefinitelyTyped/pull/30564.


[Portuguese] Localize 'Service Discovery'

Signed-off-by: Vasu Singh<EMAIL_ADDRESS>Describe your changes
Related issue number https://github.com/cncf/glossary/issues/1222
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

LGTM
@edsoncelio PTAL


add blob support to GraphQL

Add fields for eip-4844 per execution api change here https://github.com/ethereum/execution-apis/pull/468/files

There is call and estimateGas function on both the Block object and the Pending object.  I think I'd use pending as it's clearer. https://github.com/shemnon/besu/blob/2360908cf5cdaee270bc06eeaf43442687df352f/ethereum/api/src/main/resources/schema.graphqls#L177 and https://github.com/shemnon/besu/blob/2360908cf5cdaee270bc06eeaf43442687df352f/ethereum/api/src/main/resources/schema.graphqls#L254 is the structure for the call data


Prepare for new release of Confluence

Workflow links: https://github.com/atlassian-labs/atlassian-slack-integration-server/actions/runs/9727673343, https://github.com/atlassian-labs/atlassian-slack-integration-server/actions/runs/9727672410

Integration tests pass, so current version of Confluence Server for Slack is marked compatible with Confluence 8.9.3.


Create a Press Release section

Create Press and Logos section:

Logos
Logos with text
Logos in b&w
Different formats (png, jpg, svg, etc)

Modelo: https://about.gitlab.com/press/press-kit/


How to switch to VPN on Windows？

I have successfully installed it on windows10. There is no problem using cmd to ping node host(such as <IP_ADDRESS>). But how to make the nebula work like VPN. I have installed the tap network card of OpenVPN. Thank you.

https://michlstechblog.info/blog/openvpn-connect-to-multiple-vpns-on-windows/

then rename your new tap device to nebula1 in windows config
add
tun:
  # Name of the device
  dev: nebula1

to your nebula config file

You mean you want to use a node like you use a VPN server to redirect traffic ?

You mean you want to use a node like you use a VPN server to redirect traffic ?

How can it be done with Nebula? Writing a "<IP_ADDRESS>/0": true in local_allow_list?


Feature: Add PIN/PW authentication to app opening

Unlock with password, PIN CODE, biometrics (FaceID, fingerprint, etc.) when opening the app

Thanks, appreciate your feedback @baek-sang,
Would you like to contribute this in the feature idea forum so it can be discussed, upvoted and considered for a help wanted ticket?
Please include a link back to this GitHub issue. If you're interested in implementing, please say so and we'll prioritize the review.
You get 10 votes in the feature idea forum, and each one influences the future of the project.

Yes, I'd recommend commenting on Uservoice. Our product teams review feature requests there, but it looks like this particular feature hasn't yet been prioritized on the roadmaps.


vkCmdBuildAccelerationStructureKHR crashes in the recording layer

Tried to make a recording of raytracing function calls and immediately get a crash on vkCmdBuildAccelerationStructureKHR as it is not unwrapping the passed in command buffer to a type the underlying layer understands. Looking in the code there is a TODO in CmdBuildAccelerationStructureKHR in framework\encode\custom_vulkan_api_call_encoders.cpp.
This was with latest dev branch (commit 0837ff0) on Windows.
Thanks for looking into it!

Capture of the ray tracing API is now supported, so this issue should be resolved.


Can.t change password

When he tries to change the password using ssh he gets this message:
root@yi-6fe1:~# passwd
Changing password for root
New password:
Retype password:
passwd: /etc/passwd: Read-only file system
passwd: can't update password file /etc/passwd

you can't change root password, it's on read only partition of yi firmware.

In this case, how do you change the password? I'm trying to follow the instructions at Home Assistant page.
https://www.home-assistant.io/integrations/yi/#changing-the-ftp-password

You can enable username/password by using yi-hack web page. It will activate authorisations for ftp,ssh,telnet,rtsp and http ( same for all )

The username/password setting works only for http, rtsp and onvif.
ssh and telnet use passwd but rootfs is read only, so you can't change it.

Instructions at ha site are for an older version/hack.

I tried without changing the password, but ftp has no video, the image via rtsp but the preview does not work ...

Don't use yi integration on ha. It's for an older version.
See the wiki.

Meanwhilst, a PR has been merged which allows to set the ssh password from the web ui.


Calendar closes on date click using web components and appendTo.

In this pull https://github.com/chmln/flatpickr/pull/510 you forgot to verify if the event path contains the container specified on "appendTo" option.

Pull request: https://github.com/chmln/flatpickr/issues/594

Thanks for fixing this @micku7zu


Building a lyrics search app with vanilla javascript

Article checklist
Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #issue-number

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

@adrianmurage have gone through the article and I can boldly say the code runs now, some lyrics are not returned because they do not exist on the ap

@adrianmurage have gone through the article and I can boldly say the code runs now, some lyrics are not returned because they do not exist on the ap

because they do not exist on the api.

@adrianmurage have gone through the article and I can boldly say the code runs now, some lyrics are not returned because they do not exist on the ap

because they do not exist on the api.

@adrianmurage I have fixed the issue of code not running, it now works without error.

Just sharing this past article to make sure that 1) it does not overlap & 2) to see if it can be referenced at all (if it makes sense).

Is this PR a duplicate?

@hectorkambow it is not a duplicate because I  changed a lot in it, compared to the last PR.

@hectorkambow the last PR was closed.

Is this PR a duplicate?

Yes @hectorkambow. It duplicates a closed PR with the changes I requested @shegz101 to make on their initial submission.

@adrianmurage I have made the necessary corrections

Hey @shegz101    👋 .
First of all, welcome to Section - EngEd. :tada:
We are happy to have you here.
It took a few tries but eventually, we got your article up to par. I am super proud of you for the hard work! 👏
Just a few things I'd like to point out:

I suggest that you check out our resources page which has links to various resourceful tools for writing just to minimize the changes being made by the reviewer.
Lastly, note all the corrections we have made so that you can not repeat the same mistakes.

That's it! :smile:
Once I finalize my part of the review, a final review will be done by a content moderator.
Otherwise, congratulations on publishing your first article🍾.
In case of any question or clarification (currently or in the future), do not hesitate to reach out via Slack or my personal email (adrianmurage21@gmail.com).
I am ALWAYS here to help.
It has been a pleasure reading and reviewing this article.
Looking forward to seeing you contribute more to EngEd. 🚀
Once again good job! 👍
Kind Regards.

@adrianmurage am so happy it is finally coming to life, you have been of great help to me in this journey and am super grateful, I have noted all corrections made and looking forward to write better articles for section which won't be full of errors. I wish you could just be the one reviewing more of my articles.

@adrianmurage it is a pleasure connecting with a great reviewer like you, thanks once again.

@adrianmurage am so happy it is finally coming to life, you have been of great help to me in this journey and am super grateful, I have noted all corrections made and looking forward to write better articles for section which won't be full of errors. I wish you could just be the one reviewing more of my articles.

I am glad I was able to have such a positive impact on your experience and I look forward to seeing more of your work! I assure you that the peer reviewer team is full of amazing reviewers who you will like just as much 😄

@adrianmurage yes I look forward to connecting with other peer reviewers, I know it would be an amazing experience, still planning to see if I can still write one more article before the year runs out, more great articles next year by God's grace.

So I should be looking at the article getting published before the end of this week, early next week.

@adrianmurage I sent you a message on slack

jumping on this one

I started a final review and made some other changes - still noticing a few errors throughout the article.
Most grammatical - please be sure to go through our resources page to get more tips, guides, and resources on how to improve overall technical writing.
We will be closing this pull request.
Thank you.
cc @adrianmurage  there were some errors that were noticed during the final review, and some sentences that were a bit difficult to read through.

@hectorkambow the PR and issue are both closed, does that mean when I feel am good enough, I should open a fresh issue, just want to know the next line of action

@hectorkambow can you please point out example of the errors

@hectorkambow can you just suggest the necessary changes please, have been on this for so long only for it to be totally closed and for me to start afresh

@adrianmurage please what is next, do I have to go through suggesting topic ,and when it is approved rewrite this article, @hectorkambow have started going through the resources you sent, please let me know what is expected of me

@hectorkambow please can you reopen it, I just finished going through the resources you sent me, please give me a chance to make corrections

Left a message via slack

@hectorkambow can you just suggest the necessary changes please, have been on this for so long only for it to be totally closed and for me to start afresh

Our team does not provide revision services - and I believe the reviewers were going an extra step by providing some highlights before.
Please do go through our resources for extra help :rocket:

We can not pin point every single change that is required.
That would not be fair to the students who are submitting proofread content.

OK but am free to start the process afresh, by opening a new issue


Add files via upload

PR types

PR changes

Describe

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


feat(core): Minor UI fixes for mobile; autofocus

feat(core): Minor UI fixes for mobile; autofocus

Social login buttons now 80% of old size, they are too large on small screens such as the iPhone and oversized on the desktop too IMO
Moved the 'sign in failed' warning above the login form, so that it is not cut off on small devices
Fix broken password popover balloon
Added an autofocus directive to automatically move focus to relevant fields on MEAN.js forms. The directive only fires if screen width >= 800px because otherwise a large keyboard may open on a mobile device.
The autofocus directive overwrites the 'autofocus' HTML5 attribute (the attribute is no good for angular apps as it doesn't work across state changes) - I don't believe there's any problem with doing that but I'm open to being corrected

Coverage decreased (-0.2%) to 72.809% when pulling 6a5303c86866de3c178cdb83a14578287fbf2d60 on hyperreality:social-login2 into 17772fe96ad9430645e4c4191d93d921897485b2 on meanjs:master.

Coverage decreased (-0.2%) to 72.809% when pulling 6a5303c86866de3c178cdb83a14578287fbf2d60 on hyperreality:social-login2 into 17772fe96ad9430645e4c4191d93d921897485b2 on meanjs:master.

Coverage decreased (-0.2%) to 72.809% when pulling 6a5303c86866de3c178cdb83a14578287fbf2d60 on hyperreality:social-login2 into 17772fe96ad9430645e4c4191d93d921897485b2 on meanjs:master.

Coverage decreased (-0.2%) to 72.809% when pulling 249fe4e207671384086f332d463ad0279643f3af on hyperreality:social-login2 into 17772fe96ad9430645e4c4191d93d921897485b2 on meanjs:master.

Coverage decreased (-0.2%) to 72.809% when pulling 249fe4e207671384086f332d463ad0279643f3af on hyperreality:social-login2 into 17772fe96ad9430645e4c4191d93d921897485b2 on meanjs:master.

Coverage decreased (-0.2%) to 72.809% when pulling 5fe26403b44745dcfe1d0d77d8c43cc2f4c69875 on hyperreality:social-login2 into 17772fe96ad9430645e4c4191d93d921897485b2 on meanjs:master.

Coverage decreased (-0.2%) to 72.809% when pulling 5fe26403b44745dcfe1d0d77d8c43cc2f4c69875 on hyperreality:social-login2 into 17772fe96ad9430645e4c4191d93d921897485b2 on meanjs:master.

@hyperreality Can you think of a test to write for this new directive? The coverage decreased, so that may be why the coveralls build failed. I think we have a threshold of coverage decreasing.
@lirantal I couldn't find the settings on Coveralls. Do you know what this setting is, and where I can see it?

@mleanos Just realised that you can send keys on Protractor, I'll play around with it.
The coveralls settings are a tiny button on the top left.

Coverage decreased (-0.2%) to 72.809% when pulling bdbc3fec01defb7d378987584be328e4ee0ba950 on hyperreality:social-login2 into 17772fe96ad9430645e4c4191d93d921897485b2 on meanjs:master.

Coverage decreased (-0.2%) to 72.809% when pulling bdbc3fec01defb7d378987584be328e4ee0ba950 on hyperreality:social-login2 into 17772fe96ad9430645e4c4191d93d921897485b2 on meanjs:master.

The test is failing on Travis - I suspect that's because the directive only fires if window width is over 800px. However, trying to get Protractor tests working with different windows sizes was a nightmare I don't fancy revisiting for the time being!

Coverage decreased (-0.2%) to 72.809% when pulling<PHONE_NUMBER>5fcc9f6a1a4359ab3cd299cb6bd2cc on hyperreality:social-login2 into b2a5cb5b6fe51144c3c3e23f6433e17429680d59 on meanjs:master.

@hyperreality Ok. Understood. Perhaps we can just leave that test out then?
I'm surprised the coverage decreased so much by merely adding this one directive.

I'll have one more crack at the test I think.
In the modules directory:
find . -name '*.js' | xargs wc -l reports ~10000 lines of code
wc -l core/client/directives/ is 28 lines
Which is about 0.2%

@hyperreality let's see if we can kick this in before a coming 0.5.0 release

Coverage decreased (-0.1%) to 72.869% when pulling 598711a5c145620c82a9aa7b2f45269ceb06ae2d on hyperreality:social-login2 into b2a5cb5b6fe51144c3c3e23f6433e17429680d59 on meanjs:master.

@hyperreality I left a review, but for some reason it doesn't mark the PR as "updated".

Coverage decreased (-0.3%) to 72.932% when pulling 10d822b9153a06fc996cd7c3353164492e272d93 on hyperreality:social-login2 into fa138045e697aa48fd7b3f37145a52461ee711be on meanjs:master.

Coverage decreased (-0.3%) to 72.932% when pulling bdb5b9d8d008e65ad0901cf23d1fe2215e5dc22c on hyperreality:social-login2 into fa138045e697aa48fd7b3f37145a52461ee711be on meanjs:master.

Coverage decreased (-0.2%) to 73.058% when pulling 33fb7d1e2e454201caf8d85d64dd3878d0eef32b on hyperreality:social-login2 into fa138045e697aa48fd7b3f37145a52461ee711be on meanjs:master.

@hyperreality I'd like to merge this in, but I'm having a hard time reviewing this with all the other commits from your rebase included. I don't know which commits are specific to this PR. Would it be possible for you to clean this up?

@meanjs/contributors @hyperreality
I pulled this branch down locally, and was able to review. Everything seems to be working. I'm merging this now since it introduces a few important fixes, and we need to finalize the other open PR's to get ready for 0.5.0.


Sum selected transactions no longer works

I've tried this in firefox, chrome, and safari on both my personal (OS X) computer and my work (windows 7) computer. The rest of the extension is working as advertised, but not the "sum selected transactions" option. See attached screenshot. I select transactions - one or multiple - and nothing happens. In the past, this has worked.

I can confirm this. It seems to be related to changing account. When YNAB is reloaded it works again. I might need to move around the accounts a few times - but it do stop to work. Also, it seems to be the SAME accounts that make it stop working. If I reload and having one of those open the function won't work no matter what. I can't find any other pattern.

This is most likely related to #339
@aribroo Can you confirm whether this is after switching budgets or not?

@joshmadewell Definitely not after switching budgets, I only have one. Could be after switching accounts, I can't figure out a pattern though, if so.

@aribroo does the feature never work for you or is it a "hit or miss" kind of deal?

@joshmadewell It’s hit or miss. Maybe works 15% of the time, rough estimate.

On Jul 1, 2016, at 5:19 PM, Josh Madewell<EMAIL_ADDRESS>wrote:
@aribroo https://github.com/aribroo does the feature never work for you or is it a "hit or miss" kind of deal?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/toolkit-for-ynab/toolkit-for-ynab/issues/335#issuecomment-230049586, or mute the thread https://github.com/notifications/unsubscribe/AMxHOna_nXx_z-keQSaN_oFL5FQf7FjFks5qRYRUgaJpZM4InpfZ.

@aribroo Alright man, I finally found something that reproduces this and I've fixed it.

I think what's happening is that YNAB is overwriting the header or maybe just re-initializing it with only cleared, uncleared, and working balance. We think we've initialized already so we don't reinitialize.
Hopefully this fixes the issue and you can see it in my PR here: #372.

I'm having trouble with "Selected Transactions" as well. Mac OS X El Capitan, Chrome Version 51.0.2704.103 (64-bit).
When I load YNAB for the first time and switch to an account, selected transaction amount does not show up. If I select an account, then reload the page, the selected transaction will work until I switch to another account.
Thanks!

@Wags391 we have a fix that's gone into master and should be deployed for the next release

came here to report this - i'm using chrome both regular and incognito and selected transactions never works.

@nardz13 please let us know if this is working after our next release!

ok no problem. when will i expect to see the next release ?

Hopefully soon. It's up to @blargity but it should be soon!

i've got the latest release (0.5.0)  but don't seem to be able to see this issue being fixed...

Darn....I'm having loads of issues reproducing this to be honest with you.
Could you provide me with the following:

Operating System
Browser
Other installed extensions
Any errors you see in the Developer Console when selecting transactions (Settings -> More Tools -> Developer Tools OR cmd + shift + i on mac OR (maybe) ctrl + shift + i on windows)

My current setup:

Windows 7 Enterprise
Chrome  47.0.2526.80 m
Google Docs, Google Docs Offline, Google Sheets, Google SLides

Chart.js - Moment.js could not be found! You must include it before Chart.js to use the time scale. Download at http://momentjs.com/
main.js:60 Uncaught TypeError: Cannot read property 'fromMonth' of nullgetVisibleTransactions @ main.js:60enhancedSelectedTotalsCalculate @ main.js:97enhancedSelectedTotalsPoll @ main.js:78
hope this helps.
i can't use the latest version of chrome, as i'm at work and have to wait for central IT to push out the update - i can at home though.

for the record, am on my home laptop now, windows 10, chrome 51.0.2704.106 m, and this is working fine - selected transactions are summing up as expected

If you can figure out what version of Chrome you're on at work, check how old it is. If it's more than 6 months old, yell at your IT department. If it's less than 6 months old it's probably worth checking out what's going on as you're not the only one where IT has disabled updates.

as i said in my earlier message - 47.0.2526.80 m
without looking at version history i'm guessing that's more than 6 months?

Ah, apologies, missed that comment.
Looks like it was released on December 8th, 2015. If it's working for you at home then unfortunately it's probably not going to help many people to spend time looking into it... 😦
I'm going to leave this here for a bit longer to see if anyone else can reproduce on a more recent Chrome. If so we can definitely have a look.

that's fine - i can definitely cop that as a old shitty version of chrome issue.

Alrighty, I'll go ahead and close this for now with the official resolution being, "Yell at your IT department". Booo.


Something's wrong with the result

Hi,
I just download your project and tested it. But there seems to be some problems with the mesh's indexes.  It looks like this=>http://pan.baidu.com/s/1mhXbhOk. Am I doing something wrong or there's a bug with the program?

Hey,
it looks like your Webserver does not support partial requests but always offers the same content independed of which range was requestet. This results in downloading the whole file multiple times and istead of adding the missing parts to the buffer always adding the first blocks of data again. I experienced the same with the simple HTTP server build into Jetbrains Webstorm.
I will probably add a function to detect the content size to fix this for those kind of webservers in the next few days. Please be aware that the whole project is still work in progress and does currently not correctly implement the POP buffer as described by the x3dom team.

I created a ticket ( #2 ) to support also webservers without partial request ability.

Please check if another webserver solves your problem so I can close this ticket.

I implemented the correct handling for servers without partial request, so this problem should be fixed now. Please reopen this issue if the problem persists with the current version of the code.


NO-ISSUE: update midstream main build setup with the latest upstream changes

This PR requires:

[ ] https://github.com/kiegroup/kie-tools/pull/56

There have been changes in upstream which we need to get in our midstream setup manually.
The commits of this PR show the upstream PRs connected to those changes.

as PR https://github.com/kiegroup/kie-tools/pull/56 is merged, please rebase this on main branch so the diff gets easier to review.


Disable when auto move to next git revert changes

VSCode Version: Code 1.19.0 (816be6780ca8bd0ab80314e11478c48c70d09383, 2017-12-14T12:06:43.492Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Extension
Author (truncated)
Version

vscode-intelephense-client
bme
0.8.2

sublime-keybindings
ms-
3.0.3

vscode-docker
Pet
0.0.22

material-icon-theme
PKi
3.1.0

(1 theme extensions excluded)

Steps to Reproduce:

In the git repo, made some changes
when you click the gutter and make revert changes, it'll auto go to next changes

DEMO
This is annoying and we lose our focus on the document.
Also below shortcut is not working. We need to manually click the button
{
  "key": "ctrl+alt+z",
  "command": "git.revertChange"
}

Basically it's should work as PHPStorm.

Reproduces without extensions: Yes/No

#39307

Thanks, Also be note, shortcut keys are not working,


Sample app changes to facilitate Heroku deployment

The sample apps can all be deployed to Heroku as-is, but they won't log to STDOUT. This also wraps the Judoscale initialization in a check for the constant, this way the sample apps can be deployed without Judoscale (to test the installation flow).
This also adds the x86_64-linux platform to the Gemfile for each sample app, which is needed for Heroku.
To deploy a sample app to Heroku, cd into the directory and run:
git init
git add .
git commit -m "prep for Heroku"
heroku create
git push heroku main

For sample apps that require Redis:
heroku addons:create heroku-redis

Heroku-Redis takes a minute to provision. Run heroku logs -t to watch when the app restarts.
To install Judoscale:
# scale up a worker dyno (if necessary) before doing this so Judoscale picks it up
heroku ps:scale sidekiq=1
heroku addons:create judoscale:trial

Converted this to draft. I'm still finding little quirks.

Ah I didn't realize you'd want to run these on heroku... maybe we should bring back the production.rb environment config files that I removed?

Just to follow-up from our call... I don't think we need production.rb. The only config I needed to add was logging to STDOUT, so I just put that in application.rb.

@carlosantoniodasilva FYI - I added instructions to the sample app README's and made a couple updates to the judoscale.rb initializers.


@Nadeko Cleverbot isn't working

My server is running its own NadekoBot, Cleverbot used to work but we tried using the @Nadeko Cleverbot command recently and it doesn't work anymore. We see it typing but after a few second there is no reply sent. Is there any way to fix this?

Must be an issue on Program-O's (Cleverbot API) side.

If you try to talk to program-o itself, (https://www.program-o.com/) it errors.

It doesn't work for me neither. The API is probably down. I know, the old cleverbot API costs money now. But we really need to be able to switch between cleverbot APIs. Nadeko Clone owners can probably afford the packages. Even the free package might be enough.

Please make this a thing @Kwoth 💯

@Midoriin it is already a thing... kinda hidden but you can put ur api key in there. Copy the format from credentials_example.json and paste it in credentials.json. Then, replace all the values u want to replace 😄 . CleverbotApiKey is one of them.
I know this used to work, and it should still work.

Yes. Get the free version of the official cleverbot and add it to your credentials.json, and then your cleverbot command will use official one instead of (poopy) program-o

Wow, I didn't know. That's pretty nice. Thanks!


du on Windows

I'd be interested in having du compile on Windows. Looking at the code and compiler error msgs the only blocker is the use of std::os::unix::fs::MetadataExt. Seems like that could be fixed with a handful of #[cfg]'s. That would of course affect parts of the code that use MetadataExt and ultimately would lead to the following options not being included on Windows:

apparent-size
time
time-style
count-links

I'd personally still get use out of a du without these options. I'm down to work on this if ppl are open to these sorta changes.

These sorts of changes are always welcome!  Of course, if you manage to figure out how to get the equivalent info on Windows that would be preferable to disabling features, but ideally all the utils should be able to run in some form on any OS.

My filesize crate may be of interest - it's already used by several Rust du-alikes and abstracts across Windows and Unix.

Recommend marking as a duplicate of #1622 (which I implemented).

Closing this ticket, fixed in https://github.com/uutils/coreutils/pull/1788


Update for recent vulnerability

Has this script been updated to reflect the vulnerability from yesterday that allowed stealing tokens from custom emoji?

Hello!
No Lemmy-Easy-Deploy update is necessary, LED already lets you update to any custom rc version.
A fix was tagged in the UI repo as 0.18.2-rc.1, so you just need to force an update to that tag:
./deploy.sh -w 0.18.2-rc.1 -f

As of right now, there are no new backend tags, but if you take a look at the --help page (or the README), you can learn how to update to backend RC versions in the same way.


Android build failing

I am building for Android on Ubuntu 14.04 64-bit. It is failing with the following error:-
Assembler messages:
Fatal error: invalid -march= option: `armv7-a'

Appreciate any help.

Again never mind. I tried building for x86, there after progressing for sometime, it failed with error that a library is too short. Then I knew that some of the downloaded files are corrupted. God I wish gclient had integrity checker. I re-downloaded the full webrtc source, now this build completed successfully.


The Circle Icons overlay is in the wrong path

On the Android 12 Beta 4 GSI (Build Number SPB4.210715.014) the icon overlay is meant to be in /system/product/overlay however your magisk module attempts to put it in /system/overlay which does not exist. Manually editing the folders inside the zip according to the folder structure present on my phone has fixed the problem.
Edit: the circle icon theme seems to work in Launchair 11 alpha 6.1, doesn't seem to work in Quickstep or the settings app

Does it show up if you run cmd overlay list? It used to be in /system/product/overlay, but I moved it to /system/overlay in order to fix bootloops on some unofficial semi-GSI ports of the Pixel beta builds.

Yeah it shows up as com.android.theme.icon.circle but I'm using the my edited one (i.e. the one with the overlay in /system/product/overlay)

Can you test it with the default /system/overlay path? It should work as far as I know, but I don't currently have an AOSP GSI installation to test it.

Switched back to your version. The overlay no longer shows when running the command, however for some reason the circle icons stay in Lawnchair.

Just did a fresh install of Android 12 Beta 4 (since the Beta 5 GSI doesn't seem to boot on my phone) and it turns out that Quickstep does in fact have circle icons now (but it only worked after toggling the option twice)
I used the edited module again since the overlay doesn't seem to be working in your version

Fixed in v8.0.0. Thanks for the details!


Automate gh release

Okay, did some trial & error work for setting up a somehow working CI ...

We should have a test status badge on our README
PRs should now been tested by GitHub Actions
If we tag (the master) with sth like 0.1.2, this should trigger a release build with resulting artifact in GH releases.

Regadring npm I found out how we can start (see package.json). I did a manual triggered publish already so you can test with
sudo npm i<EMAIL_ADDRESS>-g
mastodon-bot -h

That looks fantastic, I'm good to merge in if you are. :)

go on your mark :-)
maybe want to try the tagging thing?

Yeah sure, I'll merge in and make a tag then. :)

I bumped up package.json to 1.0.0, and pushed a tag v1.0.0, might as well treat it as the official a 1.0 release. :)

the tags name should either be 1.0.0 or you may insert a 'v' here: https://github.com/yogthos/mastodon-bot/blob/27c99a7d556727089da0d1f7ec486de43fc2a5ff/.github/workflows/release.yml#L4

ah my bad, just fixed it

hmm ... it still is publishing as pre release ...
Will adjust and search how to publish to npm also ... in the next PR :-)


Added 2 nodes of 1 user

Added:

@paulettevanhoesen:matrix.org 2 nodes of the user

@ironoa Element handle of the user was verified


Isn't .U syntax supposed to only take .W arguments?

Type of issue: bug report

Impact: API modification

Development Phase: request
Other information

If the current behavior is a bug, please provide the steps to reproduce the problem:
Put this in a Module:
val bar = 5
val foo = bar.U(7)

Here is a simple Scastie (some commented out code that also all works unexpectedly)
https://scastie.scala-lang.org/nqEQGHEUTziPKNj7NUirBA
What is the current behavior?
This code compiles without error
What is the expected behavior?
I thought it should be an error to have no .W suffix on the arguments inside the .U, since those are not supposed to directly take integer arguments
Please tell us about your environment:
- version: 3.5.1

What is the use case for changing the behavior?
Hard-to-catch unintended user errors

This is something we've discussed in the past, I attempted to fix in https://github.com/chipsalliance/chisel3/tree/deprecate-literal-bit-extract and @albert-magyar had a PR proposed to fix via ScalaFix and linting: https://github.com/chipsalliance/chisel3/pull/1449

I attempted to fix in https://github.com/chipsalliance/chisel3/tree/deprecate-literal-bit-extract

What was the issue with the attempt?

What was the issue with the attempt?

I just ran out of steam and was struggling getting all of the cases working. I think we should redo that branch and just do .U and .S. We also need to make sure those methods don't have parentheses. We can make the suggestion for users who do want to extract immediately to call .extract which is a new method we can add that .apply is an alias for.


Visual Studio 15 adds duplicate import statement for Typescript Project to project file when adding new Typescript file.

Create an empty VS 15 Web Project using ASP.net 4.5.2, not 5 template.
Enter code: export class Test{}
Compile.  You will see error.
Go to project settings, set Module system to "System"
Compile: Error will disappear.
Add new TS file: test2 and add code: export class Test2{}
Error will reappear "Build: Cannot compile modules unless the '--module' flag is provided.
Edit project file by hand.  Remove second reference to Typescript project. Save. Load.
Error will disappear.
Add new TS file, error is back.  Remove second reference, error disappears.

The problem was NOT actually that there is a duplicate import statement.  The resolution is here: http://stackoverflow.com/questions/32170041/cannot-compile-modules-unless-the-module-flag-is-provided/32183183#32183183


Add a similiarty parameter to locateOnScreen function

For exmaple, I need local a input box.
There are three sence:

The background color of the box would change a little sometime

There are some words in that input box sometime.

Even box  not change, but move to another place , locateOnScreen  get None

in Which both pyautogui.locateOnScreen('xxx.jpg') and  pyautogui.locateOnScreen('xxx.jpg', grayscale=True) would fail.

I built similar code (relying on OpenCV) into a different library I've worked on. There are some bonus optimizations, like a pyramid model to speed up matching. Feel free to copy anything helpful for pyautogui.

this is already implemented as confidence parameter


[WIP] Fix curator compatibility

Merged build finished. Test FAILed.

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


Severe Frame Time Increase When Enabling Any Effects Outside of Group

I am experiencing strange behavior. When I try to apply just one effect behind the UI/smoke (just one effect is ticked as the effect to apply in the group) everything works as expected and I get good performance. But when I try to enable any single other shader (not adding to the group, just enabling in ReShade), my fps tanks from 144+ to ~55-60. I have similar GPU/CPU usage as before. If I check the box "Apply for all enabled techniques" instead of selecting just one effect to apply to the group, then there is no performance drop, however I do not want all my effects to run behind the UI/smoke, I want them to run at the end as default (for example, HDR shaders do not work unless they are at the end).
Just MXAO applied by the group, other effect runs as normal

vs.
"Apply for all enabled techniques" (both effects run in the group)

very strange that neither CPU nor GPU time increases much, yet the frame times are so much worse.
I am seeing this behavior in Red Dead Redemption (DX12).

I've done testing on a bunch of prior versions, this issue is present in every version of the add-in that is able to boot the game.


[WIP] Custom Scrollbars

Another pass at integrating custom scrollbars, properly.
Notes:

The custom scrollbars are red while in development, do help differentiate between native and custom
The new editor (still WIP) uses a textarea, which is not compatible with any form us browser-compatible  custom scrollbars, so that uses the default still.
The code needs so much cleaning.

Todo:

[ ] Find correct events to reset the scrollbar (to get the correct size) when:

[ ] Infinte scroll has happened
[ ] A view has changed (new post)

[ ] Clean up code (There's loads of commented out CSS and unfinished styles for scroll containers)
[ ] Test in all the places

Ditto, think this one will be out of date before it moves forward, so gonna close for now


nginx 1.7.7 - nginx-rtmp-module crashes.

nginx-1.7.7
nginx-rtmp-module-git (latest as of nov 17)
nginx-patches-git (latest as of nov 17)
Centos 6.5 (latest patches) & centos 7.0 (latest patches)
My setup:
4 client servers, 16 workers, 1024 connections per worker.
 Without the per-worker-listener we lose the ability to control or stat the servers.
The following message is shown in the message log at this point:
Nov 17 09:59:00 vidclient4 kernel: nginx[11299]: segfault at d1 ip<PHONE_NUMBER>473827 sp 00007fff6a68cad8 error 4 in nginx[400000+bc000]

same for me ...

This issue has also occured on my master server (1 master 3 relays)

i found it only occured when rtmp_auto_push and pull_static used in the same time...

@wdjwxh i have neither of those values and mine was crashing.  Until today, I disabled idle_streams and so far no crashes.

@wdjwxh @mknwebsolutions
I found this problem occurs with any use of the PULL mechanism, static or dynamic, at some point during use it will crash when a pulling stream dissapears.

Do you know how to reproduce the crash quickly? Please tell me the detailed step.

as above ,i found it only carsh when rtmp_auto_push and pull_static used in the same time... so I delete the rtmp_auto_push so it won't crash.  but , the used bandwidth increased 100% even more...

I believe I also encountered this bug:
Program received signal SIGSEGV, Segmentation fault.
0x00007feabe2bff28 in ngx_rtmp_auto_push_publish (s=0x7feabf74eca0, v=0x7feabe545700 <v>)
    at ../nginx-rtmp-module/ngx_rtmp_auto_push_module.c:477
477         ctx->push_evt.log = s->connection->log;
(gdb) bt
#0  0x00007feabe2bff28 in ngx_rtmp_auto_push_publish (s=0x7feabf74eca0, v=0x7feabe545700 <v>)
    at ../nginx-rtmp-module/ngx_rtmp_auto_push_module.c:477
#1  0x00007feabe2aad66 in ngx_rtmp_amf_message_handler (s=0x7feabf74eca0, h=0x7feabf812830, in=0x7feabfad59d8)
    at ../nginx-rtmp-module/ngx_rtmp_receive.c:437
#2  0x00007feabe2a6631 in ngx_rtmp_receive_message (s=s@entry=0x7feabf74eca0, h=h@entry=0x7feabf812830,
    in=in@entry=0x7feabf8147e0) at ../nginx-rtmp-module/ngx_rtmp_handler.c:799
#3  0x00007feabe2a6bc1 in ngx_rtmp_recv (rev=<optimized out>) at ../nginx-rtmp-module/ngx_rtmp_handler.c:464
#4  0x00007feabe223daf in ngx_epoll_process_events (cycle=0x7feabf73fb60, timer=0, flags=1)
    at src/event/modules/ngx_epoll_module.c:822
#5  0x00007feabe21942a in ngx_process_events_and_timers (cycle=cycle@entry=0x7feabf73fb60) at src/event/ngx_event.c:242
#6  0x00007feabe221475 in ngx_worker_process_cycle (cycle=cycle@entry=0x7feabf73fb60, data=data@entry=0x0)
    at src/os/unix/ngx_process_cycle.c:753
#7  0x00007feabe21fdca in ngx_spawn_process (cycle=0x7feabf73fb60, proc=0x7feabe221420 <ngx_worker_process_cycle>, data=0x0,
    name=0x7feabe2d4ed2 "worker process", respawn=0) at src/os/unix/ngx_process.c:198
#8  0x00007feabe222b92 in ngx_reap_children (cycle=<optimized out>) at src/os/unix/ngx_process_cycle.c:621
#9  ngx_master_process_cycle (cycle=0x7feabf73fb60) at src/os/unix/ngx_process_cycle.c:174
#10 0x00007feabe1f9c36 in main (argc=<optimized out>, argv=<optimized out>) at src/core/nginx.c:367
(gdb) print ctx
$1 = (ngx_rtmp_auto_push_ctx_t *) 0x7feabf74eca0
(gdb) print ctx->push_evt
$2 = {data = 0x7feabf74eca0, write = 0, accept = 0, instance = 0, active = 0, disabled = 0, ready = 0, oneshot = 0,
  complete = 0, eof = 0, error = 0, timedout = 0, timer_set = 0, delayed = 0, deferred_accept = 0, pending_eof = 0,
  posted = 0, closed = 0, channel = 0, resolver = 0, cancelable = 0, available = 0, handler = 0x0, index = 0, log = 0x0,
  timer = {key = 0, left = 0x0, right = 0x0, parent = 0x0, color = 0 '\000', data = 0 '\000'}, queue = {prev = 0x0,
    next = 0x0}}
(gdb) print ctx->push_evt.log
$3 = (ngx_log_t *) 0x0
(gdb) print s
$4 = (ngx_rtmp_session_t *) 0x7feabf74eca0
(gdb) print s->connection
$5 = (ngx_connection_t *) 0x0 <- !!

s->connection appears to be NULL, trying to access its member logcauses the segfault.
Running on Debian/jessie:

nginx version: nginx/1.9.15
built with OpenSSL 1.0.1k 8 Jan 2015

nginx-rtmp-module# git branch -v
* master e089592 support for dynamic build

Crash does NOT happen if rtmp_auto_push is set to off.

Worker number needs to be 1, and auto_push needs to be disabled. They are
experimental features and as you see - they doesn't work now. If you really
need so many connected clients (I don't think so), then you can setup
multiple instances and manually set push directives for distributing
streams across them.

Been a while since I was here, but yes I do need so many clients :)
have hit over 20k on my live env (wowza currently)

@DrFlash99 Then you need to cluster itself via pull or push (it's up to you - you need to understand your use case).
I scaled on single server to 8Gbps, so around 15k clients without any problems. I used many separate nginx instances, load balancer and pull's.

@ DrFlash99 فأنت بحاجة إلى تجميع نفسها عن طريق السحب أو الدفع (الأمر متروك لك - تحتاج إلى فهم حالة الاستخدام الخاصة بك).
لقد قمت بالتوسع على خادم واحد إلى 8 جيجابت في الثانية ، أي حوالي 15 ألف عميل دون أي مشاكل. لقد استخدمت العديد من مثيلات nginx المنفصلة وموازن التحميل ومثيلات السحب.

Can you simplify how load Balance works and what is installed on the back-end servers?

@DrFlash99 Then you need to cluster itself via pull or push (it's up to you - you need to understand your use case).
I scaled on single server to 8Gbps, so around 15k clients without any problems. I used many separate nginx instances, load balancer and pull's.

Can you simplify how load Balance works and what is installed on the back-end servers?


cherry-pick: Improve delete device failure logs in driver node

What type of PR is this?
Cherrypick of 477

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Release note:
none

/cc @mkumatag

/cherry-pick release-4.14


Does not work w/ preact-compat components

Components instantiated using React style (extends Component) and while aliasing React to point at react-compat render as [object Object].  Reason being is that components instantiated using react-compat do not have a nodename:
{ '$$typeof': Symbol(react.element),
  type:
   { [Function: Footer]
     childContextTypes: { intl: [Object], locale: [Object] },
     displayName: 'Footer',
     modulePath: '/app/components/Shared/Footer' },
  key: null,
  ref: null,
  props:
   { store:
      { dispatch: [Function],
        subscribe: [Function: subscribe],
        getState: [Function: getState],
        replaceReducer: [Function: replaceReducer] },
     intl: { [Function] cache: [Object], toString: [Function], format: [Function] },
     locale: 'en-CA',
     dispatch: [Function] },
  _owner: null }

Resulting in the following code being triggered in renderToString:
if (!nodeName) {
    return encodeEntities(vnode);
}

@dmitrif It actually looks like you're passing in a React VNode, not a preact-compat VNode. Are you sure your alias is working?

@dmitrif Here's a demonstration showing that what you described works:
https://jsfiddle.net/developit/w13rh1f4/
It definitely looks like you've got a configuration issue somewhere (webpack aliases?) that is resulting in React still being used to create the Virtual DOM Elements instead of preact-compat.
Let me know if you're able to verify.

Seems you are absolutely correct. For some reason I was convinced that the aliasing was happening correctly, but after grokking through the compiled server code, I was proven wrong.
Thank you for your time :)

No problem! Feel free to drop into the Preact Gitter Chat if you are still having issues with the alias!


IOU - Details modal is dismissed after editing title

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v9.0.5-3
Reproducible in staging?: Y
Reproducible in production?: New feature
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Navigate to a workspace chat
Submit a manual expense
Click on the IOU preview component
Click on report header > Title
Edit title & click Save

Expected Result:
User should be returned to Details RHP
Actual Result:
Details RHP modal is dismissed
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/9eb6d15a-0c5f-4fc3-bbcc-9039415340dd

View all open jobs on GitHub

@adelekennedy I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

We think this issue might be related to the #collect project.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Details modal is dismissed after editing title
What is the root cause of that problem?
We are dismissing the modal and navigating to the previous report here
https://github.com/Expensify/App/blob/51dd814bf6a142fbaf590b69f90932dec2fe6507/src/pages/EditReportFieldPage.tsx#L84
What changes do you think we should make in order to solve the problem?
We should change  Navigation.dismissModal(report?.reportID) with Navigation.goBack()`
What alternative solutions did you explore? (Optional)

That makes sense @etCoderDysto but wouldn't that break the feature for the other places where we call this (that existed before we added the possibility of editing the title like this)?

I have been trying to check other pages after applying. I found that different component is used other pages as room's details page. But to make things sure. I will check other places that use the same component.

@iwiznia The component seems to be used only in ModalStackNavigator.

Solution seems good 👍

Should I raise a quick PR?

I am raising a pr asap. And investigating what component was used before.

@iwiznia  I have found a flow where we use EditReportsFieldPage. We use it in MoneyReportView to display editable report fields when the user has enabled 'Report Fields' in more features.
https://github.com/Expensify/App/blob/c5c4ea8fcec6c03ebee5483b6bcacacdfbed9ebf/src/components/ReportActionItem/MoneyReportView.tsx#L83-L86
I have changed the logic to address that we don't change exiting behaviour in MoneyReportView as below. We navigate back only when user is changing report title.
 if (isReportFieldTitle) {
            ReportActions.updateReportName(report.reportID, value, report.reportName ?? '');
            Navigation.goBack();
        } else {
            ReportActions.updateReportField(report.reportID, {...reportField, value: value === '' ? null : value}, reportField);
            Navigation.dismissModal(report?.reportID);
        }

The PR is ready for review.

Ah nice, I knew that code must've been there for a reason 😄

CPd. thanks

@adelekennedy Please assign me to the issue for tracking the payment (reviewed PR)

done @jayeshmangwani !

@adelekennedy the pr was deployed to production 8 days ago.

Thank you @etCoderDysto - the automatic update failed here.

Yes, mevlin is not adding lables.

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ] The PR that introduced the bug has been identified. Link to the PR:
[ ] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[ ] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] A regression test has been added or updated so that the same bug will not reach production again. Link to the GH issue for creating the test here:

Payouts due:

[ ] Contributor: $250 @etCoderDysto (Upwork)
[ ] Contributor+: $250 @jayeshmangwani (NewDot)

@etCoderDysto will you link your upwork profile here?

https://www.upwork.com/freelancers/~01ea78e0164390eb79
Here is a link to my profile.
Thank you!

[x]  [@jayeshmangwani] The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/44671
[x]  [@jayeshmangwani] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/44671#discussion_r1685680379
[x]  [@jayeshmangwani] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
[x]  [@jayeshmangwani] Determine if we should create a regression test for this bug. Yes
[x]  [@jayeshmangwani] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Go to a workspace chat.
Submit a manual expense.
Click on the IOU preview component.
Click on the report header -> Title.
Edit the title and click Save.
Verify that you are navigated back to the Details page.

Do we agree 👍 or 👎

offer sent @etCoderDysto

offer sent @etCoderDysto

I have accepted the offer. Thank you!

Requested $250


[10.x] Add fragment* signatures to view interface

Description
While working on a Blade & HTMX project, I've noticed my editor not being aware of the fragment* methods on the view() helper. The helper returns interfaces that does not implement the method signature for said methods.
During this PR I've noticed several public methods are not on the interface, I am unsure if this is by choice however, so I've left them out (for now).
If this is indeed a breaking change, perhaps another solution could be raised.
Solution
Added signatures for all fragment* methods on Illuminate\Contracts\View\View.
Benefit
Editor/IDE & end users aware of method being available.

An alternative solution would be type-hinting the concrete classes, instead of their contracts/interface, on the view helper.
https://github.com/laravel/framework/blob/9a409cfa38919efa22f8aba32f9bc1b78b329755/src/Illuminate/Foundation/helpers.php#L1003-L1010
I am not saying I think this would be a better solution, I am just suggesting as an alternative solution if adding public methods to the interface should be deferred to the 11.0 release.

Updating the return type of the function was my initial thought, however looking at the other helpers returning interfaces it felt out of place to suddenly change that. So I opted towards maybe the interface should reflect the public API.
In theory updating the function does sound better, as I wonder if updating the interface would break potential implementations of it for other view engines, that users/package maintainers migh have.

Let's see how the maintainers feel about it.
An approach could be adding, not replacing, the concrete classes to the helper's return union type to Laravel 10.x, and add those to the interface in Laravel 11. And also on Laravel 11, remove the concrete classes from the helper return type.
So package authors could anticipate any additions, while this feature can play nice with IDE/static analysis tools.
It is cumbersome, but it is an option.

We can't add methods to contracts on a patch release sorry.

@driesvints Sure, but could we update the return of the view helper then? Like the original post said, perhaps a better solution could be raised from the PR. It's insanely annoying.

@jorqensen it's best that you just attempt a PR so we can look at actual code.


Ensure user-event folder is included into final bundle

What:
With the changes introduced in #24, there's a user-events folder inside the types folder that isn't being added to the final, published, library. This change ensures that this folder is also included.
Why:

Since there's a reference to the folder inside of types/pure.d.ts, the whole types for the userEvent library become any.
How:
By adding types/user-event/**/*.d.ts to the list of included files, we're ensuring the final bundle also includes the content of the types/user-event folder.
Checklist:

[N/A] Tests
[N/A] TypeScript definitions updated
[N/A] Ready to be merged

@crutchcorn are you able to please take a look at this and potentially release a patch of the library? 🙏🏼


AutoML | MSR link update

@nibaccam : Thanks for your contribution! The author(s) have been notified to review your proposed change.


1 remove provider

Pull Request Test Coverage Report for Build 55

32 of 32 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 48:
0.0%

Covered Lines:
802

Relevant Lines:
802

💛  - Coveralls


setOption response is sent only after close

I use TDLib json interface. I send setOption request then I wait 5 seconds for an Ok response but it comes only after I close tdlib as 500: Request aborted error. I tried to increase timeout to 1 minute but result is the same.
Request example:
{
  "@type": "setOption",
  "@extra": {
    "request_id": "80c3ffa46c10489583c1e62c72f42097"
  },
  "name": "disable_persistent_network_statistics",
  "value": {
    "@type": "optionValueBoolean",
    "@extra": {},
    "value": true
  }
}

Log dump (including TDLib "INFO" level)
[ 3][t 4][1640635709.234481096][Td.cpp:3030][#1][!Td][&td_requests]	Receive request 2: setOption {
  name = "disable_persistent_network_statistics"
  value = optionValueBoolean {
    value = true
  }
}
[ 3][t 0][1640635710.129020929][Client.cpp:291][&td_requests]	End to wait for updates, returning object 0 0x0
[ 3][t 0][1640635710.130106925][Client.cpp:278][&td_requests]	Begin to wait for updates with timeout 1.000000
[ 3][t 0][1640635711.131232023][Client.cpp:291][&td_requests]	End to wait for updates, returning object 0 0x0
[ 3][t 0][1640635711.132245063][Client.cpp:278][&td_requests]	Begin to wait for updates with timeout 1.000000
[ 3][t 0][1640635712.133342981][Client.cpp:291][&td_requests]	End to wait for updates, returning object 0 0x0
[ 3][t 0][1640635712.134327173][Client.cpp:278][&td_requests]	Begin to wait for updates with timeout 1.000000
[ 3][t 0][1640635713.134504079][Client.cpp:291][&td_requests]	End to wait for updates, returning object 0 0x0
[ 3][t 0][1640635713.135345935][Client.cpp:278][&td_requests]	Begin to wait for updates with timeout 1.000000
[ 3][t 0][1640635714.136462211][Client.cpp:291][&td_requests]	End to wait for updates, returning object 0 0x0
[ 3][t 0][1640635714.137408018][Client.cpp:278][&td_requests]	Begin to wait for updates with timeout 1.000000
INFO:root:5 sec timeout exceeded inside an app
[ 3][t 0][1640635715.137747049][Client.cpp:291][&td_requests]	End to wait for updates, returning object 0 0x0
[ 3][t 4][1640635715.254472017][Td.cpp:3030][#1][!Td][&td_requests]	Receive request 3: close {
}
[ 3][t 4][1640635715.255259037][Td.cpp:4077][#1][!Td][&td_requests]	Sending result for request 3: ok {
}
[ 2][t 4][1640635715.255663156][Td.cpp:3550][#1][!Td]	Close Td in state 0
[ 3][t 4][1640635715.255665063][Td.cpp:4091][#1][!Td][&td_requests]	Sending error for request 2: error { code = 500 message = "Request aborted" }
[ 3][t 4][1640635715.256119251][Td.cpp:4062][#1][!Td][&td_requests]	Sending update: updateAuthorizationState {
  authorization_state = authorizationStateClosing {
  }
}
[ 3][t 4][1640635715.256139039][Td.cpp:3396][#1][!Td]	Have no request actors
[ 3][t 4][1640635715.256140232][Td.cpp:4062][#1][!Td][&td_requests]	Sending update: updateAuthorizationState {
  authorization_state = authorizationStateClosed {
  }
}

You didn't init TDLib. There is no way it can complete the request before instance is initialized.

I think I did. Because I've already sent getAuthorizationState by this moment.
There are logs before setOption:
getAuthorizationState {"@type":"getAuthorizationState","@extra":{"request_id":"updateAuthorizationState"}}
[ 3][t 0][1640638404.088001012][Client.cpp:278][&td_requests]	Begin to wait for updates with timeout 1.000000
[ 3][t 4][1640638404.088825225][Td.cpp:2668][#1][!MultiTd]	Create Td with layer 135, database version 13 and version 35 on 4 threads
[ 3][t 4][1640638404.089335203][Td.cpp:4062][#1][!Td][&td_requests]	Sending update: updateOption {
  name = "version"
  value = optionValueString {
    value = "1.7.11"
  }
}
[ 3][t 0][1640638404.089835166][Client.cpp:291][&td_requests]	End to wait for updates, returning object 0 0x6000027c02a0
[ 3][t 4][1640638404.089843034][Td.cpp:4062][#1][!Td][&td_requests]	Sending update: updateAuthorizationState {
  authorization_state = authorizationStateWaitTdlibParameters {
  }
}
[ 3][t 4][1640638404.090131044][Td.cpp:3030][#1][!Td][&td_requests]	Receive request 1: getAuthorizationState {
}
[ 3][t 4][1640638404.090355157][Td.cpp:4077][#1][!Td][&td_requests]	Sending result for request 1: authorizationStateWaitTdlibParameters {
}
[ 3][t 0][1640638404.090353965][Client.cpp:278][&td_requests]	Begin to wait for updates with timeout 1.000000
[ 3][t 0][1640638404.090360164][Client.cpp:291][&td_requests]	End to wait for updates, returning object 0 0x600002bc40b0
[ 3][t 0][1640638404.090782165][Client.cpp:278][&td_requests]	Begin to wait for updates with timeout 1.000000
[ 3][t 0][1640638404.090786218][Client.cpp:291][&td_requests]	End to wait for updates, returning object 1 0x600002bc40d0

As mentioned in options docs some of parameters must be set  before setTdLibParameters, so I tried to set them right before setTdLibParameters request

They must be set before setTdlibParameters, but then you need to init the TDLib instance, processing authorizationStateWaitTdlibParameters and authorizationStateWaitEncryptionKey. The instance isn't inited before calls to setTdlibParameters and checkDatabaseEncryptionKey.

Thank you


No validation of timezone when using annotated datetime with annotated-types

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
I have a datetime that I want to be validated as aware and in UTC. I'm following the docs here to annotate the datetime as such. I expect the initialization of the model to fail, but it succeeds. When I try a similar approach by annotating an integer as greater than a value the validation works as expected.
I hope this is clear enough, and that I'm using the library correctly.
I'm on annotated-types==0.7.0 if it matters.
Thanks for this great library by the way. Excellent stuff!
Example Code
from datetime import datetime, timezone
from typing import Annotated

from annotated_types import Gt, Timezone
from pydantic import BaseModel, ValidationError

class Event(BaseModel):
    date: Annotated[datetime, Timezone(timezone.utc)]

class Adult(BaseModel):
    age: Annotated[int, Gt(18)]

try:
    Adult(age=12)
except ValidationError:
    pass  # This is expected

try:
    Event(date=datetime(2024, 1, 1, 12, 0, 0))
except ValidationError:
    pass  # This is expected
else:
    raise Exception("No validation error 😢")

Python, Pydantic & OS Version
pydantic version: 2.9.2
        pydantic-core version: 2.23.4
          pydantic-core build: profile=release pgo=false
                 install path: /home/tgurion/code/pydantic-annotated-types/.venv/lib/python3.13/site-packages/pydantic
               python version: 3.13.0rc2 (main, Sep  9 2024, 22:13:26) [Clang 18.1.8 ]
                     platform: Linux-6.8.0-47-generic-x86_64-with-glibc2.35
             related packages: typing_extensions-4.12.2
                       commit: unknown

Using AwareDatetime might help, but it won't solve your fundamental issue of enforcing a specific TZ.
https://docs.pydantic.dev/latest/api/types/#pydantic.types.AwareDatetime
(Also, you probably want to use Strict mode. The non-strict mode (e.g., Event(date=1) might be a bit surprising.)
Here's a workaround.
from zoneinfo import ZoneInfo
from datetime import datetime, timezone
from typing import Annotated

from annotated_types import Gt, Timezone
from pydantic import BaseModel, ValidationError, AwareDatetime, Strict, AfterValidator

ZI_UTC = ZoneInfo("UTC")

def require_utc(dt: AwareDatetime) -> AwareDatetime:
    assert dt.tzinfo == ZI_UTC, f"Datetime must be provided in UTC. Got {dt.tzinfo}"
    return dt

class Event(BaseModel):
    date: Annotated[AwareDatetime, Strict(), AfterValidator(require_utc)]

In REPL.
In [2]: Event(date=datetime.now().astimezone(ZI_UTC))
Out[2]: Event(date=datetime.datetime(2024, 10, 31, 0, 23, 34, 692493, tzinfo=zoneinfo.ZoneInfo(key='UTC')))

In [3]: Event(date=datetime.now().astimezone(ZoneInfo("Africa/Accra")))
ValidationError: 1 validation error for Event
date
  Assertion failed, Datetime must be provided in UTC. Got Africa/Accra [type=assertion_error, input_value=datetime.datetime(2024, 1...nfo(key='Africa/Accra')), input_type=datetime]
    For further information visit https://errors.pydantic.dev/2.9/v/assertion_error

In [4]: Event(date=1)
ValidationError: 1 validation error for Event
date
  Input should be a valid datetime [type=datetime_type, input_value=1, input_type=int]
    For further information visit https://errors.pydantic.dev/2.9/v/datetime_type

In [5]: Event(date=datetime.now())
ValidationError: 1 validation error for Event
date
  Input should have timezone info [type=timezone_aware, input_value=datetime.datetime(2024, 1... 30, 17, 26, 29, 170166), input_type=datetime]
    For further information visit https://errors.pydantic.dev/2.9/v/timezone_aware

Also, note that the Timezone in Annotated doesn't appear to be checked or validated.
In [6]: class A(BaseModel):
   ...:     t0: Annotated[datetime, Timezone("DRAGONS")]
   ...:

In [7]: A(t0=1)
Out[7]: A(t0=datetime.datetime(1970, 1, 1, 0, 0, 1, tzinfo=TzInfo(UTC)))

Thank you very much for the response @mpkocher!
I totally missed the AwareDatetime type. This, and your suggestions, completely solve my problem. I still think the behaviour in my example is confusing, so maybe there's an action there (potentially just docs update). If you agree I'm happy to help and contribute an improvement. Otherwise, feel free to close the case.

@Nagasaki45,
Thanks for your question!
We actually have a pretty detailed example in the docs showing how to do validation with a constrained timezone.
I'm hoping that helps to clarify the issue here. That being said, if you think an additional docs update would be helpful here, feel free to open a PR and I'd be happy to review!
One other point - we don't support Timezone as a constraint at the moment, hence your initial example not working :)

That could probably be added as a separate feature request, though if I recall correctly, we ran into some difficulties with that in the past. Doesn't mean it can't be done!

Hi @sydney-runkle,
Thank you for your comments. I should better familiar myself with the docs, I guess. It sounds like @mpkocher and yourself don't see any problem with the current behaviour nor the docs, and my problem is solved (thanks again @mpkocher!), so I'm happy closing the case.


[feature request] Allow to hide "Disable/Ignore" suggestions

We are planning on using this extension at Stripe, but want to discourage users from using the "Disable"/"Ignore" suggestions, instead of them being easily accessible through quick fixes.
It would be great to have a setting to hide those suggestions.

Thx, will check and release it!

Any updates?


carapace: 0.26.0 -> 0.27.0

Description of changes
https://github.com/rsteube/carapace-bin/releases/tag/v0.27.0
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.


Browse the device

Hey,
I want to get the USB path, for browsing the device through my application.
How do I get this done ?

USB path has different meaning on different OS. You'd better check the implementation of https://github.com/woodemi/libusb.dart

This plugin is for desktop (windows, linux and macos)
I'm searching one for Android

quick_usb is mainly to provide cross-platform access to USB devices.
You could wrap your own plugin with Android UsbManager

This plugin is for desktop (windows, linux and macos) I'm searching one for Android

This plugin supports Android and uses Android UsbManager under the hood which is again Android wrapper for libusb.
It's unclear what you mean by "browsing" but if you're reffering to a path of USB disk devices to be browsed by some kind of file explorer than you'll either need to do some extra work by yourself or look in another place.
Please provide more info about your application and what you want to do.

I'd like to close it if no further info


Remove a type of Post Pipeline instances, not only a single instance

This PR (delete as applicable)

Adds a new feature/bug fix

Describe the changes below:
A post-fix pipeline class can be added to game object many times, via different instances.
Previously, removePostPipeline(pipelineClass) method only removes a single pipeline instance. It won't clear all instances of pipelineClass. This PR will remove all instances of pipelineClass.

👍

👍


Dw/cloudwatch scaledown alarms

Closes #79
Can someone please verify this works

Average count was being used which maybe the result of the failure to 0 nodes Changed to Maximum
Added Cloudwatch Alarms for Scale down
Added the Cloudwatch Alarm actions

I will test this on AWS

Fixed Cloudwatch Alarms


Uniswap Permit2 exploit

Name
Uniswap Permit2
Description
The exploit specifically involves the "Permit2" contract, which is used to route trades on the Uniswap exchange.
Addresses
Not sure
Meta Article URLs
https://blockworks.media/uniswap-users-warned/

Scammers on social media tried to scare users into thinking that Permit2 was compromised, so they can send you to their fake Revoke.cash clone website. Permit2 and Uniswap are not compromised.


Grouped "Clients" page by subscriptions

This is my attempt to clean up the clients page. Having to click around and apply filters or type in search boxes wasn't sufficient for my management, so they asked me to group clients.
Users can now view the status of entire subscription groups rather than (only) individual clients. If one of the machines in the group has a problem, the group will show up as having a problem.
Now you can see the overall status of a product at a glance, and not have to filter or search for the machines in question. Simply clicking on the name of the subscription group will fold/unfold the groups so you can see which machines have a problem and what that problem is.
This is useful if you have a product you are monitoring that spans multiple machines (such as having a database on one machine and a web server on another, for example). All management has to do to check the status of a product is click the bookmark link to the clients page. If product-XYZ has something wrong with it, they don't have to see the machine or the problem, they can just call whoever needs to fix it.
It is hard for me to imagine a reason that people wouldn't like this feature, but it may need to be modified further before it is ready to be commited to the master.
In a feature request discussion, palourde mentioned perhaps putting this in another view. I was thinking  a checkbox in the settings menu that says "Group the Clients page by subcriptions" might also work...  in case people don't want this feature they can toggle it.
I am submitting the work I have done thus far to get some input and thoughts from the other developers and community. I am not an experienced web developer, so the way I did things might not have been the most perfect or clean (although I think I did a pretty good job...).
I would be happy to get input or development from others.

Okay, here we go:

I updated my uchiwa to the latest version that adds support for sensu enterprise dashboard and status filtering (hence a lot of the differences between files)
I returned the clients page to the way it was
I copy/pasted the "Clients" section from controllers.js to a new "Groups" section in the same file, updated the page header text, and added the functions needed to get the groups page to behave correctly
I added "groups.html" in bower_components/uchiwa-web/partials/views
I added a new icon in the sidebar for the groups page and linked to groups.html
I changed app.js so the above link would resolve correctly
I even added fancy +/- icons into the accordion to make it more obvious that the sections are expandible/collapsible

The last update I did changed the tooltip of the sidebar groups icon to show the number of groups with critical/warning/unknown statuses instead of number of clients.
For example, if you have one client with a critical status and that client is in two groups, the groups tooltip will show "Critical: 2".
I had previously just copy/pasted the clients tooltip and this was bothering me. The code I wrote works perfectly but might not be super-pretty-angular-style, if someone with higher web development knowledge wants to refactor it.

You should think about fixing html comments '<! --' to '<!--'. btw: great work.

Looks interesting, but it appears that you have partials as a subdirectory of itself. Might be worth rebasing against master also.

That's nice!
As mentioned before, maybe look at the partials folder and re-generate the CSS files with grunt sass.

This submission has been moved to pauloconnor's "Grouping clean up" mentioned directly above.

This is exactly what we were looking for but instead of grouping by 'subscriptions' grouping by some client defined key. For example if there was a host_group array/key in the client hash.  At our org all the teams use the same base subscription (check_http, check_port, check_process) and pass in data that is interpolated on the client.
Would it be possible to extend it to that? Or is that a very different PR?

Try asking in #80. The person who has the power to merge this hasn't had time to refactor it, maybe they could modify it in the process.

+1

+1

Definitely like this idea - hoping it gets refactored/merged soon.

@traviswdev should this PR be closed in favour of #80 so comments aren't split across both?

Good point. Closing PR. Comments should be posted on #80.


Cherrypick recent CLI changes (1)

Cherrypick the following two changes:
https://github.com/temporalio/temporal/commit/8c17ae8cdf174b8fa3e16df4a90007d58cb951e0
https://github.com/temporalio/temporal/commit/2fa70b3be0cdb7106c1b3d9fd399e779b865ccee

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

Coverage decreased (-0.03%) to 67.077% when pulling c7274ceda0103457d8eabacd338078132a37315a on meiliang86:cli into 5791f0752a3ff85c6b5b32438d3cad271ea1817e on uber:master.


anbox: cleanup

Use trivial builders for all the text manipulation and use runtimeShell
Motivation for this change
Didn't get a chance to review #57826 so I thought to submit the changes I would have requested anyway.
Things done

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

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

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

cc @gebner

LGTM.


Check out the RQ task results for anything interesting

AB#21

Lots of the known 'Text file busy' errors. Moving this all to DO Functions should resolve.


RDK-42619: Cherry Video Plane/AudioMixing - HdmiInput Thunder changes

Reason for change: changes done in thunder layer for secondary video plane and audio mixing
Test Procedure: Build and Verify.
Risks: Low
Signed-off-by: Aishwariya B<EMAIL_ADDRESS>

Closing older PRs. Pls bring it to latest sprint/main branches if required.


Remove version: VirtualGL.TurboVNC version 3.0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/turbovnc/files/3.0/TurboVNC-3.0-x64.exe/download
Status Code: 200
(Automated message - build 897)

URL: https://sourceforge.net/projects/turbovnc/files/3.0/TurboVNC-3.0-x64.exe/download
Status Code: 200
(Automated message - build 897)

URL: https://sourceforge.net/projects/turbovnc/files/3.0/TurboVNC-3.0-x64.exe/download
OK
(Automated message - build 923)

Close with reason: Package still available;


Is there a reason why we pushed node_modules?

title :)

Github actions are loaded as a plain js file, not as a module, so in order to use modules such as remark, we need to have their source in the repo in some form. Committing node_modules is the easiest way to do that, some other actions do it (eg https://github.com/dawidd6/action-download-artifact).
But we could take the time to implement a build step, so that we just need to commit the resulting dist js. It might actually be easiest to use TypeScript.

Actually, ncc looks like the easy route. PR incoming