PS4 Game activity not matched to PS5

Description of the issue
Just setup the addon and it seems to have connected except my activity isn't updating.
Addon version
1.0.0
Platform
Home Assistant core - 2022.8.2
Supervisor 2022.08.3
Operating System 8.4
Frontend 20220802.0 - latest
Logs of the issue (if applicable)
No response

You'll need to provide more information. Please provide:

Add-on Configuration (redact any passwords)
What do you mean by 'updating'? Does the sensor ever receive a value? Does it not change after receiving a value? Etc.
Describe expected behaviour
Add-on logs (the issue template doesn't specify them for fun 😉)

Configuration:
mqtt:
host: <IP_ADDRESS>
port: "1883"
user: admin
pass: removed
logger: "@ha:ps5:*"
device_check_interval: 1000
device_discovery_interval: 60000
account_check_interval: 4000
include_ps4_devices: false
psn_accounts:

username: davidnestico
npsso: removed

Like I turned on my PS5 and opened God of War and it just stayed idle.
LOGS:
Starting PS5-MQTT...
2022-08-16T19:17:37.532Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310
2022-08-16T19:17:37.631Z @ha:ps5:webserver Server listening on PORT: 62286
2022-08-16T19:17:40.651Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310
2022-08-16T19:20:15.283Z @ha:ps5:checkDevicesState Update HA
2022-08-16T19:20:15.301Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310
2022-08-16T19:20:42.572Z @ha:ps5:error SyntaxError: Unexpected end of JSON input
at JSON.parse ()
at /app/server/dist/redux/sagas/check-devices-state.js:75:38
at step (/app/server/dist/redux/sagas/check-devices-state.js:35:23)
at Object.next (/app/server/dist/redux/sagas/check-devices-state.js:16:53)
at next (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1161:27)
at currCb (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1255:7)
at runSelectEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:735:5)
at runEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1208:7)
at digestEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1275:5)
at next (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1165:9)
2022-08-16T19:20:42.590Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310
2022-08-16T19:20:51.671Z @ha:ps5:checkDevicesState Update HA
2022-08-16T19:20:51.714Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310
2022-08-16T19:38:01.162Z @ha:ps5:error SyntaxError: Unexpected end of JSON input
at JSON.parse ()
at /app/server/dist/redux/sagas/check-devices-state.js:75:38
at step (/app/server/dist/redux/sagas/check-devices-state.js:35:23)
at Object.next (/app/server/dist/redux/sagas/check-devices-state.js:16:53)
at next (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1161:27)
at currCb (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1255:7)
at runSelectEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:735:5)
at runEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1208:7)
at digestEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1275:5)
at next (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1165:9)
2022-08-16T19:38:01.181Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310
2022-08-16T19:38:07.766Z @ha:ps5:checkDevicesState Update HA
2022-08-16T19:38:07.782Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310
2022-08-16T21:03:34.369Z @ha:ps5:error SyntaxError: Unexpected end of JSON input
at JSON.parse ()
at /app/server/dist/redux/sagas/check-devices-state.js:75:38
at step (/app/server/dist/redux/sagas/check-devices-state.js:35:23)
at Object.next (/app/server/dist/redux/sagas/check-devices-state.js:16:53)
at next (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1161:27)
at currCb (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1255:7)
at runSelectEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:735:5)
at runEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1208:7)
at digestEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1275:5)
at next (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1165:9)
2022-08-16T21:03:34.396Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310
2022-08-16T21:03:40.429Z @ha:ps5:checkDevicesState Update HA
2022-08-16T21:03:40.441Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310
2022-08-16T21:03:56.475Z @ha:ps5:error SyntaxError: Unexpected end of JSON input
at JSON.parse ()
at /app/server/dist/redux/sagas/check-devices-state.js:75:38
at step (/app/server/dist/redux/sagas/check-devices-state.js:35:23)
at Object.next (/app/server/dist/redux/sagas/check-devices-state.js:16:53)
at next (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1161:27)
at currCb (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1255:7)
at runSelectEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:735:5)
at runEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1208:7)
at digestEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1275:5)
at next (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1165:9)
2022-08-16T21:03:56.484Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310
2022-08-16T21:04:05.761Z @ha:ps5:checkDevicesState Update HA
2022-08-16T21:04:05.779Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310
2022-08-16T21:04:55.333Z @ha:ps5:error SyntaxError: Unexpected end of JSON input
at JSON.parse ()
at /app/server/dist/redux/sagas/check-devices-state.js:75:38
at step (/app/server/dist/redux/sagas/check-devices-state.js:35:23)
at Object.next (/app/server/dist/redux/sagas/check-devices-state.js:16:53)
at next (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1161:27)
at currCb (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1255:7)
at runSelectEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:735:5)
at runEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1208:7)
at digestEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1275:5)
at next (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1165:9)
2022-08-16T21:04:55.352Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310
2022-08-16T21:05:04.438Z @ha:ps5:checkDevicesState Update HA
2022-08-16T21:05:04.447Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310
2022-08-16T21:07:10.518Z @ha:ps5:error SyntaxError: Unexpected end of JSON input
at JSON.parse ()
at /app/server/dist/redux/sagas/check-devices-state.js:75:38
at step (/app/server/dist/redux/sagas/check-devices-state.js:35:23)
at Object.next (/app/server/dist/redux/sagas/check-devices-state.js:16:53)
at next (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1161:27)
at currCb (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1255:7)
at runSelectEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:735:5)
at runEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1208:7)
at digestEffect (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1275:5)
at next (/app/node_modules/@redux-saga/core/dist/redux-saga-core.dev.cjs.js:1165:9)
2022-08-16T21:07:10.532Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310
2022-08-16T21:07:16.998Z @ha:ps5:checkDevicesState Update HA
2022-08-16T21:07:17.011Z @ha:ps5:mqtt MQTT Message received ps5-mqtt/78C8818CE310

Is your account online and not appear offline?
Have you tried setting the privacy setting:
Who can see your online status and what your currently playing. to anyone?
Have you tried starting a PS5 only title? Like Astro's playroom, returnal, etc.?
A restart of the add-on might be required after making these changes.

I started God of War aka a PS4 title, wonder if that's why, and will try restarting addon soon.
And I just changed my gaming history to "anyone" everything else was already on "anyone" except that was "friends only" until now. I will test this tonight.

@FunkeyFlo
Update:
I made all my settings "anyone" and I restarted addon, it looks like it works but only with PS5 games damn, any reason why it can't work with PS4 games and it just shows as idle?
With that being said how can I make my card look like yours with the game name and also cover?

Unfortunately not all PSN API's have been documented (by the community). It's not something that Sony officially supports themselves, outside of official partners like discord.
The coming weeks I probably won't have time to debug this so it might take a while for a fix to arrive.
P.s.: please don't clutter the thread with questions on different topics. For questions/issues related to the add-on open a new GitHub issue. How to use the entities the add-on supplies is probably a question for the discord.

Alright, can I open a discussion with the question then instead? sorry.
And I mostly play PS5 games now anyways I just sometimes go back to PS4 games for trophy hunting reasons or if the game doesn't have a PS5 upgrade like warzone, so not a big deal for how long it takes to fix!

Fix has been implemented an included in the next release

Awesome!! Thanks for a quick fix!


Entries are obscured by the keyboard

The entry is covered by the keyboard, and WindowLevel is invalid by setting it. How can I make the entry in front of the keyboard

Having Same issue.

Can do with this .  it's in the documentation.
let offset = EKAttributes.PositionConstraints.KeyboardRelation.Offset(bottom: 10, screenEdgeResistance: 20)
let keyboardRelation = EKAttributes.PositionConstraints.KeyboardRelation.bind(offset: offset)
attributes.positionConstraints.keyboardRelation = keyboardRelation


Let ChangesetBuilder.invalidated search even container elements

Pull Request Description
Will fix #6221 by making sure IR element with UUID is found even it is not a leaf.
Important Notes
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[ ] All code follows the
Scala,
Java,
style guides.
All code has been tested:

[ ] Unit tests have been written where possible.

... understanding how it works. Can you write it down as a comment on the function

Using descriptive identifier names since 07fc59c


No GLSurfaceView support for Android

Hi, there is no support for GLSurfaceView component in the current version, it's important for rendering video (for example).

We accept pull requests :)

Ok, let us check what we can make with it

Hey there! Closing this issue since it's a feature request and not a bug.
We are trying to keep GitHub issues strictly for bugs.
Here is a great library you can try for videos.
https://github.com/brentvatne/react-native-video
Hope this helps.


Scorecard - Not copied by mastercontroller when in subdirectory

All property files support subdirectories. The scorecard feature does support that too (reading) but it appears that the MC cannot deal with a definition file in a subdirectory within config when preparing the final results.

Well, it seems that XLT copies only files at 1st level below config plus those that are included using property includes. So, this is not related to the Scorecard feature but a general issue.

Don't we copy included properties with subdirs?
On Mon, 12 Aug 2024 at 18:19, h-arlt @.***> wrote:

Well, it seems that XLT copies only files at 1st level below config plus
those that are included using property includes. So, this is not related to
the Scorecard feature but a general issue.
—
Reply to this email directly, view it on GitHub
https://github.com/Xceptance/XLT/issues/521#issuecomment-2284400015, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AANV6QEDH3OST2GAL4T3ZW3ZRDOBTAVCNFSM6AAAAABML7BLNWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBUGQYDAMBRGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
René Schwietzke
CEO/Managing Director
Email: @.***
Phone: +49-3641-376-300
Mobile: +49-172-7986-359
Xceptance Software Technologies GmbH
Leutragraben 2-4
07743 Jena
Germany
Xceptance Software Technologies, Inc.
One Broadway
Cambridge, MA 02142
United States
Geschäftsführer: Roy Sarnoch, René Schwietzke
Sitz der Gesellschaft: Jena
Registergericht: Amtsgericht Jena
HRB-Nr.: 210087
www.xceptance.com

XLT copies only files at 1st level below config plus those that are included using property includes.

You have to include it explicitly when it resides in a sub-directory. Otherwise, it won't be copied (although users might expect it to be).
Assume for example, users store their load test profiles in a sub-directory named 'profiles'. Then, only the currently active load test profile as configured by property com.xceptance.xlt.testPropertiesFile will be copied but not the others. Whether or not this is expected depends on the point of view. As long as the other files are not part of the effective load test configuration, they don't have to be copied. Nevertheless, they are part of the overall load test configuration and may be copied as well.
Regarding scorecard configuration files, they are no properties files and thus cannot be included. So, the copy-when-included path is not applicable for them. As a consequence, scorecard definition files have to be located at first level below config in order to get copied. So far for the explanation of the current state.
Question now is: how do we want to proceed from here? Do we want to handle scorecard definition files somewhat specifically? Or, do we want to rethink the current approach (e.g. copy all files inside of config directory that match a certain file filter instead of those at 1st level only)?


Update index.md to correct table for amazonec2

The table for AMI ID for the amazonec2 had a small markdown error.

Thanks!  Would you mind signing your commit (https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work).

Thanks! LGTM


R.merge is not compatible with interfaces

This code
import * as R from 'ramda'

interface User {
  readonly name: string
  readonly age: number
}

const u1: User = {name: 'John', age: 25}
const u2: User = {name: 'Jane', age: 25}

const u3 = R.merge(u1, u2)

produces the following error:
Argument of type 'User' is not assignable to parameter of type 'Struct<{}>'.
  Type 'User' is not assignable to type 'ArrayLike<{}>'.
    Property 'length' is missing in type 'User'

I'm not quite sure what's wrong with the "obvious" signature (merge<T1, T2>(a: T1, b: T2): T1 & T2), but the current one is definitely problematic.

Yeah, this is a variant of our biggest problem right now (e.g. #78), caused by the state of TS; progress for a solution can be found here.

What is really wrong with a signature in case of merge?
merge<T1, T2>(a: T1, b: T2): T1 & T2;?

Or why not, if want to introduce constraints that it is a  structure:
merge<T1 extends Struct<any> , T2 extends Struct<any>>(a: T1, b: T2): T1 & T2;

Why type Struct<T> = Obj<T> | List<T>; is used when ramda's merge actually expects objects (Obj)?

Sorry, you're right this signature contained more generics than it could really put to good use.
I guess technically Ramda also accepts the use-case of merging arrays, though I'll admit it's a bit unexpected there it's still going to treat everything as objects anyway:
R.merge([1],[2,3]);
Object {0: 2, 1: 3}

My initial instinct has been to not immediately prohibit use-cases that could technically be legitimate. That said, I'll admit I haven't actually used this array version here myself either...


Add a safe.Atomic wrapper for handler switchers

I believe the safe wrapper around the HTTP/TCP/UDP Handlers is here to protect them against data races during configuration changes.
I don't think we need to ensure synchronization for them so using the atomic primitives instead of sync ones could improve performances.
Any thoughts ?

Hello @sylr,
Thanks for your contribution!
We took the time to review the changes and found that we will not accept them.
We are not aware of any performance issue on the handlerSwitcher mechanism, and this PR does not clarify what the issue is.
Therefore, since there is no justification for this PR, and since it is not trivial, we are closing it.


Probe: actor system

Explore implementing a probe into the actor framework.
idea: add USDT (dtrace) probes into actor framework and then use dtrace for the actual runtime tracing

Grabbing this one.

idea: add USDT (dtrace) probes into actor framework and then use dtrace for the actual runtime tracing

Hmm, but wouldn't that make actor probing work only on MacOS?
But agreed making this suited for a particular library feels too specific, so I'll also look if I can easily hook in something more general, i.e.:

#8
tracing interface
metrics interface

This can end up being related to https://github.com/tonarino/actor/issues/7 and https://github.com/tonarino/actor/issues/27

Hmm, but wouldn't that make actor probing work only on MacOS?

True. No one said it's a good idea 😅 (I just wanted to include a complete context.)


Discord Invites Are Invalid

I don't know if this is really the correct place to file this kind of "issue", but all of the Discord links that have been posted (in the README, in the initial discussion about starting a Discord, and in Gitter) all appear to be invalid or expired.

Ooof, sorry about that, my mistake. I'll open a PR now, and in the meanwhile, new (non-expiring) link is https://discord.gg/33kcChzH7f


Outdated Consumer example in README

An example of consumer implementation provided in README doesn't implement failed method, so it doesn't compile :disappointed:

Fixed in ac40d7550a040cd0f53929f1bc5d9c536746422e, thanks!


Cannot compile kvrocks

Bug Report
I cannot compile the kvrocks correctly by following the steps mentioned in the example/kvrocks/README
I found that the README is updated 9 months ago. However, the patch is updated three days ago.
Current behavior
When I compile with running ‘make’, there's an error:
incubator-kvrocks/external/kvdk/include/types.hpp:7:10: fatal error: libpmemobj++/string_view.hpp: No such file or directory

This error could be fixed by copying libpmemobj++/ directory from kvdk to the kvrocks.
But there is another error:
In file included from redis_bitmap_string.cc:5:
redis_string.h:9:10: fatal error: namespace.hpp: No such file or directory
#include "namespace.hpp"

I found that you are developing kvdk and there is no namespace.h file in the latest version. I would be grateful if you fixing the issue and update the README.

Thank you for reporting this issue. It was fixed in commit 9b7598, please try it out.

It works! Thank you very much!

You're welcome. I will close it.


[react-native-windows] Missing Publisher Details in the details tab under dll Properties

The CheckboxWindows.dll is missing versioning/publisher details set in their metadata.
Currently, the publisher information is not available in the properties (screenshot below), and this absence of publisher details is causing issues with AppLocker (a security tool). AppLocker can sometimes block updates to DLLs if they do not have proper versioning/publisher details set in their metadata.

Thanks for your PR


chromium: add patch to fix performance regression with fonts

Motivation for this change
This fixes a font-related performance regression in Chromium 77: https://bugs.chromium.org/p/chromium/issues/detail?id=1003997
The issue can be very annoying, with 4.5 second pauses and an inability to scroll through bookmark folders.
It is not clear exactly which fonts (or how many) need to be installed to see this problem, but I reverted the relevant commit and confirmed that this fixed the issue in chromium, chromiumBeta, and chromiumDev.
The reverted commit appears to be self-contained and not related to any security bugs. This patch will be hopefully be removed by the time Chromium 78 is released.
If accepted to master, please also cherry-pick to release-19.09.
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"
[x] 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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @bendlas
and previous committers @andir @samueldr

Updated to use fetchpatch.

Chromium author says "NOTE: It is SAFE to revert that CL on Stable or any channels to fix the issue." https://bugs.chromium.org/p/chromium/issues/detail?id=1003997#c13


Kyspark 3.3.x 4.x cherry pick

In order to synchronize the 4.6.13 version code of Yinglong and keep the spark function consistent, this pr cherry-picks the commits of spark branches kylspark-3.2.x-4.x and kylspark-3.3.x-4.x-qa.
Among them, 3 commits encountered conflicts in the synchronization kylspark-3.2.x-4.x, and I have dealt with them.
Conflicting commit ID and git log:

94a7be5094a67e6dbef480bd12ce88a853c692d8 KE-40948 Add rowGroup filters info (#596)
19fef37022f54cac5f6f856bf15d7bcf9334f679 KE-40433 add page index filter log (#619) (#624)
ea75c337d2dbf714edbf8e6f597e32187a174cd1 KE-41399 spark-42388 Avoid parquet footer reads twice in vectorized

Ways to resolve conflicts:
I added the getFileReader function to the org.apache.spark.sql.execution.datasources.parquet.SpecificParquetRecordReaderBase.ParquetRowGroupReaderImpl class, so that we can get the ParquetFileReader. In spark3.3, it was combined into the ParquetRowGroupReader class so that we Unable to get, so queryMetrics cannot be recorded.
code show as below:
   private static class ParquetRowGroupReaderImpl implements ParquetRowGroupReader {
     private final ParquetFileReader reader;

       @Override
     public ParquetFileReader getFileReader() {
         return reader;
     }

retest this please


Scheme lambda function highlighting is incorrect

In the snippet
(lambda (x y z) (+ y z))

x is highlighted as if it were a function call (instead of just an argument).
I can work around this by adding spaces around arguments, i.e.
(lambda ( x y z ) (+ x y))

but it would be nice if this were handled correctly.

Huh, good to know… Are there any other context beside (lambda where this is the case?

From a little more experimentation, the same mistake happens with quoted strings:
'(1 2 3) will highlight 1 in a different color from 2 and 3

Thanks for spotting this too. We do handle quoting in generic Lisp syntax but not in Scheme for some reason.

Looks fixed?

Nice, thanks for the quick fix!

btw, we're going to release the new version in a week on February 23


SSL via Let's Encrypt

What do you think of using Let's Encrypt for https://libretaxi.org/?

Yea, this is important one. Should be implemented

Fixed


Arweave-js dependency needs to be updated

Hello guys! Today I updated my js libraries in my project and I noticed that redstone-smartweave v0.5.9 is using arweave-js 1.10.23 but the latest Arweave-js version is 1.11.4.
https://github.com/redstone-finance/redstone-smartcontracts/blob/main/package.json#L69
This is causing the next error in this line:
const smartweave = SmartWeaveWebFactory.memCachedBased(arweave)
    .useRedStoneGateway( {confirmed: true} )
    .build();

The error:
error TS2345: Argument of type 'import("node_modules/arweave/node/common").default' is not assignable to parameter of type 'import("node_modules/redstone-smartweave/node_modules/arweave/node/common").default'.
  Types of property 'api' are incompatible.
    Type 'import("node_modules/arweave/node/lib/api").default' is not assignable to type 'import("node_modules/redstone-smartweave/node_modules/arweave/node/lib/api").default'.
      Types have separate declarations of a private property 'mergeDefaults'.

My guess is that a solution would be to update the arweave-js dependency for the next release. Thanks in advance, I love your tools!!

Hey @luckyr13 :) I've tried updating arweave-js in the sdk but our tests failed due to the arlocal error. I reported it on their gh and will wait for their reply. As for now, you could simply add following property to the package.json:
"resolutions": {
    "redstone-smartweave/arweave": "1.10.23"
  }

It works under yarn, I believe newest npm's equivalent is overrides
"overrides": {
"redstone-smartweave/arweave": "1.10.23"
},

@luckyr13 fixed in 0.5.11


Password check

@snedelkoski now we both contributed and need petar to approve :D
Do you want to take another look first, before I send to petar to aprove?


fix handlePing function

as the docs: https://github.com/shadowsocks/shadowsocks/wiki/Manage-Multiple-Users
server should send 'pong' when receive 'ping'.

LGTM

I found that there is still a problem.
The python version of shadowsocks would send 'stat: {"xxx": ooo}' every minute, but go version not.

@gyteng I think we have obied the protocol

Yes, what message about the protocol should I provide to you?

I thing this id enough

This is the behaviour of python version:
when I send 'ping', it will back 'pong', and will back 'stat: ......' every 10 seconds until the udp socket is end.

Nope, after I clarify the protocol, I found we both made mistake.
The response of Ping is stat{xxx:xxx}

What version did you use?
the protocol of libev is different from python

https://github.com/shadowsocks/shadowsocks/wiki/Manage-Multiple-Users

I will find out

while True:
print(cli.recv(1506))  # when data is transferred on Shadowsocks, you'll receive stat info every 10 seconds

@gyteng

That's it.
if this option is true. Status will be transfer to client every 10 seconds.
I will find more implements about this.

@gyteng I think here needs another merge request add the status reply. Is that ok?

Yes, I am not very familiar about golang, maybe it will take long time for
me.
I will take another pr later.
By the way, how to upgrade a go program which is build from the source, for
example, I install shadowsocks-go:
go get github.com/shadowsocks/shadowsocks-go/cmd/shadowsocks-server
and there is a new commit.
2018-03-09 14:04 GMT+08:00 Arthur-Lee<EMAIL_ADDRESS>
@gyteng https://github.com/gyteng I think here needs another merge
request add the status reply. Is that ok?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/shadowsocks/shadowsocks-go/pull/270#issuecomment-371722205,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABdi0fRrl3UHjw6DMeClCr95TUVAPbqZks5tchtzgaJpZM4Sf6iQ
.

That's ok. I can do it.
But we should make sure this change is suit to ss_manager and followed protocol

Thanks for your help,
there is the response of ping in python version
https://github.com/shadowsocks/shadowsocks/blob/master/shadowsocks/manager.py#L130


presentPaymentSheet crashes the app in android

Describe the bug
The app crashes after calling presentPaymentSheet.
To Reproduce
I followed the example in the integration guide here.
My checkout page:
import React, { useState, useEffect } from "react";
import { useStripe } from '@stripe/stripe-react-native';
import useAPIError from '../../common/hooks/useAPIError';
import stripeSheet from '../../common/api/stripe';
import { Button, VStack, Text } from 'native-base';

export default function CheckoutScreen() {
  const { initPaymentSheet, presentPaymentSheet } = useStripe();
  const [loading, setLoading] = useState(false);
  const [apiIntent, setApiIntent] = useState('');
  const [apiCustomer, setApiCustomer] = useState('');
  const [apiEphemeralKey, setApiEphemeralKey] = useState('');
  // const [clientSecret, setClientSecret] = useState();
  const { addError } = useAPIError();

  const fetchPaymentSheetParams = async () => {
      try{
        const response = await stripeSheet(addError);
        const { paymentIntent, ephemeralKey, customer }  = response.result;
        console.log(paymentIntent);
        console.log(ephemeralKey);
        console.log(customer);
        setApiIntent(paymentIntent);
        setApiEphemeralKey(ephemeralKey);
        setApiCustomer(customer);
        return {
          paymentIntent,
          ephemeralKey,
          customer,
        };

      }catch(e){
          // TODO: handle error
          return {
            paymentIntent: null,
            ephemeralKey: null,
            customer: null,
          }
      }
  };

  const initializePaymentSheet = async () => {
    const {
      paymentIntent,
      ephemeralKey,
      customer,
    } = await fetchPaymentSheetParams();

    const { error } = await initPaymentSheet({
      customerId: customer,
      customerEphemeralKeySecret: ephemeralKey,
      paymentIntentClientSecret: paymentIntent,

    });
    if (!error) {
      setLoading(true);
    }else{
    }
  };

  const openPaymentSheet = async () => {
    const { error } = await presentPaymentSheet(); // app crashes here
    if (error) {
      addError(`Error code: ${error.code}`, error.message);
    } else {
      addError('Success', 'Your order is confirmed!');
    }
  };

  useEffect(() => {
    initializePaymentSheet();
  }, []);

  return (
    <VStack>

      <Button colorScheme="primary" onPress={initializePaymentSheet}>Retry</Button>
      <Text>Disabled is {(!loading) + ''}</Text>
      <Text>intent is {apiIntent}</Text>
      <Text>customer is {apiCustomer}</Text>
      <Text>ephemeralKey is {apiEphemeralKey}</Text>
      <Button
        colorScheme="primary"
        disabled={!loading}
        onPress={openPaymentSheet}
      >Checkout</Button>

    </VStack>
  );
}

Screenshots

Smartphone (please complete the following information):

Device: Android Huawei GR3 2017
OS: Android 8.0

Additional context
Developing app with expo and using nativebase for UI (I tried pure react native UI but the issue is still there).

Expo does support the older version of this library (0.1.4), this means that there are differences in initializing the PaymentSheet. Please follow our example which points to this specific version: https://github.com/stripe/stripe-react-native/blob/v0.1.4/example/src/screens/PaymentsUICompleteScreen.tsx

Thanks, I solved the issue by downgrading the package to 0.1.4 with:
yarn upgrade<EMAIL_ADDRESS>
Then adding these missing args to initPaymentSheet:
      customFlow: false,
      merchantDisplayName: 'Example Inc.',
      style: 'alwaysDark',

@obayit you don't need to manually specify stripe-react-native version in your project, as docs says you just need to run
expo install @stripe/stripe-react-native
https://github.com/stripe/stripe-react-native#expo

@obayit there is one more difference https://github.com/stripe/stripe-react-native/blob/v0.1.4/example/src/screens/PaymentsUICompleteScreen.tsx#L36
you need to pass clientSecret also to the presentPaymentSheet method

@arekkubaczkowski thanks again

@obayit Downgrading was the only thing that worked for me. I know @arekkubaczkowski said you don't need to manually specify, but unless I downgraded to specifically that version, it would crash with no logs.

I also had to downgrade*, otherwise, it will throw an error something along the line "presentPaymentSheet expects 3 arguments but got only 2".
Also, I replaced the StripeProvider withinitStripe and the app doesn't crash anymore (Android)
*as it says in this tutorial:
https://youtu.be/qiEwO_UrOro?t=576

customFlow: false,
merchantDisplayName: 'Example Inc.',
style: 'alwaysDark',

Same! Fixed.


Custom attributes striped from select options

I have a select list that requires custom data attributes to be read from the selected option on change.
If an item is selected a function runs using the custom option data.
If I apply the lib to a select list all options are striped out and all data attributes are removed.
Is there a way to keep the data attributes intact on the options?
If not, can you point me to where the option in being constructed?
Thanks.

@DieterGribnitz there is no way to keep them with the current version of Choices, but you can recreate them by overriding option property on callbackOnTemplateCreate

Thanks I will give it a try.

Is there still no easy way to do this ? Thanks.

How can I re-write this in JS, not Typescript? Thank you


vscode.dev NLS script loading can finish after code loading

Testing #216888

safari
incognito mode
open https://insiders.vscode.dev/?vscode-version=43f70eca0c428e6f1a5bfad59a9b2a66f99b748d
page doesn't load, console contains NLS missing error but also errors about duplicated modules

[Error] Error: !!!
[Warning] Duplicate definition of module 'vs/editor/common/model/mirrorTextModel' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/model/textModelPart' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/model/textModelSearch' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/model/pieceTreeTextBuffer/pieceTreeBase' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/model/pieceTreeTextBuffer/pieceTreeTextBuffer' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/model/pieceTreeTextBuffer/pieceTreeTextBufferBuilder' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/model/textModelText' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/model/utils' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/modelLineProjectionData' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/services/findSectionHeaders' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/services/semanticTokensDto' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/services/treeViewsDnd' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/services/unicodeTextModelHighlighter' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/standalone/standaloneEnums' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/textModelBracketPairs' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/textModelEvents' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/textModelGuides' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/model/guidesTextModelPart' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/tokenizationRegistry' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/tokenizationTextModelPart' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/model/bracketPairsTextModelPart/bracketPairsTree/bracketPairsTree' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/model/bracketPairsTextModelPart/bracketPairsImpl' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/tokens/lineTokens' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/languages/supports/indentationLineProcessor' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/tokens/contiguousTokensEditing' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/tokens/contiguousMultilineTokens' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/tokens/contiguousMultilineTokensBuilder' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/tokens/contiguousTokensStore' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/tokens/sparseMultilineTokens' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/tokens/sparseTokensStore' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/viewEvents' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/viewEventHandler' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/browser/view/dynamicViewOverlay' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/browser/view/viewPart' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/viewLayout/linePart' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/viewLayout/linesLayout' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/viewLayout/viewLinesViewportData' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/viewModel' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/viewLayout/lineDecorations' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/viewModel/glyphLanesModel' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/viewModel/modelLineProjection' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/viewModel/overviewZoneManager' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/viewModel/viewContext' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/common/viewModelEventDispatcher' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/caretOperations/browser/moveCaretCommand' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/codeAction/common/types' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/colorPicker/browser/colorPickerModel' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/comment/browser/blockCommentCommand' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/comment/browser/lineCommentCommand' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/dnd/browser/dragAndDropCommand' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/find/browser/replaceAllCommand' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/find/browser/replacePattern' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/folding/browser/foldingRanges' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/folding/browser/foldingModel' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/folding/browser/hiddenRangeModel' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/folding/browser/indentRangeProvider' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/folding/browser/syntaxRangeProvider' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/format/browser/formattingEdit' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/hover/browser/contentHoverTypes' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/hover/browser/hoverTypes' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/hover/browser/marginHoverComputer' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/inPlaceReplace/browser/inPlaceReplaceCommand' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/indentation/common/indentUtils' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/inlayHints/browser/inlayHints' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/inlineCompletions/browser/commandIds' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/inlineCompletions/browser/ghostText' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/inlineCompletions/browser/singleTextEdit' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/inlineCompletions/browser/utils' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/inlineEdit/browser/commandIds' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/linesOperations/browser/copyLinesCommand' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/linesOperations/browser/sortLinesCommand' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/semanticTokens/common/semanticTokensConfig' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/smartSelect/browser/bracketSelections' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/smartSelect/browser/wordSelections' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/snippet/browser/snippetParser' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/inlineCompletions/browser/provideInlineCompletions' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/stickyScroll/browser/stickyScrollElement' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/editor/contrib/suggest/browser/suggestOvertypingCapturer' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/hover/hoverWidget' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/selectBox/selectBoxCustom' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/selectBox/selectBox' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/splitview/paneview' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/common/actions' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/actionbar/actionViewItems' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/actionbar/actionbar' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/button/button' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/dropdown/dropdown' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/dropdown/dropdownActionViewItem' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/inputbox/inputBox' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/icons/iconSelectBox' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/menu/menu' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/menu/menubar' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/toggle/toggle' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/dialog/dialog' (insiders.vscode.dev, line 5)
[Warning] Duplicate definition of module 'vs/base/browser/ui/findinput/findInputToggles' (insiders.vscode.dev, line 5)
[Error] Error: !!! NLS MISSING: 10309 !!!
	n (insiders.vscode.dev:6:5132)
	h (insiders.vscode.dev:6:4998)
	(anonymous function) (workbench.web.main.js:622:26996)
	_invokeFactory (insiders.vscode.dev:5:1497)
	complete (insiders.vscode.dev:5:1721)
	_onModuleComplete (insiders.vscode.dev:6:1060)
	_resolve (insiders.vscode.dev:6:528)
	defineModule (insiders.vscode.dev:5:5594)
	f (insiders.vscode.dev:6:1783)
	(anonymous function) (workbench.web.main.js:622:26805)
	Global Code (workbench.web.main.js:3163:10024)

reloading again fixed this

I see a 10% repro rate on this.

As we figured out later, the way how we append a script tag to the head element from the body of the page is probably causing the browser to load that in parallel to the script execution of the body. If the loader manages to start loading code before the script tag for NLS has finished loading, things will break badly.

Was there work done here? Any way to verify?

Reload a couple of times with a URL such as https://insiders.vscode.dev/?vscode-version=7bdcef6dce8eb50e511846808e1b13bb7bb039fa

I confirmed that I get the following error once per reload but that the workbench renders properly:
GET https://main.vscode-cdn.net/insider/7bdcef6dce8eb50e511846808e1b13bb7bb039fa/out/vs/workbench/workbench.web.main.nls.js net::ERR_ABORTED 404 (Not Found)

Yeah we still load the old NLS file, but that is fine to ignore.


Allows single- and double-dashed args with the same name

Fixes #231. Note that this changes the error text slightly, to include the single/double-dash prefix.
Checklist

[x] I've added at least one test that validates that my change is working, if appropriate
[x] I've followed the code style of the rest of the project
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary

@swift-ci Please test


Distance calculation

This is  a question/feature request.
Description
In my app I have implemented a drivers log for our company.
At the moment we are calculation distance between start and end of journey manually.
It would be nice if I have 2 input fields with

suggestions from google places api
automatic distance calculation

Do you know how I can start with this?
Do you know packages to use here, e.g. https://www.npmjs.com/package/vue2-google-maps?
Thx.

I think that we can safely close this, it's not related to Enso in any way.


What are drawbacks of having debug toolbar in production settings ?

INTERNAL_IPS or DEBUG=False being not set means DDT is disabled… Or at least out of sight.
But I am wondering what are actual (performance or security) drawbacks of having DDT listed in INSTALLED_APPS and MIDDLEWARE settings when INTERNAL_IPS is not set and DEBUG=False.
In other words : How far DDT is to being a no-op when DEBUG=False and INTERNAL_IPS is not set ?
Asking because it can be handy to have same INSTALLED_APPS and MIDDLEWARE settings for dev/prod and vary only on DEBUG and INTERNAL_IPS ; it offers simpler settings management and the ability to enable it temporarily (ex: on test/pre-production server).
Thanks by advance for your input :)
Oh and thanks for that great piece of software that rocks jazzes !

Thanks for your input !
At the moment, I ended up with the following compromise (a bit a magic don't hurt to be confy and safe at the same time !)
settings.py
from django.core.exceptions import ImproperlyConfigured

from .base_settings import *

# Override parameters with local_settings.py
try:
    from .local_settings import *
except ImportError:
    raise ImproperlyConfigured(
        'You need to create a local_settings.py, see README.md'
    )

if DEBUG:
    # Enable debug toolbar only if DEBUG=True
    INSTALLED_APPS = INSTALLED_APPS +  ('debug_toolbar',)
    MIDDLEWARE = ['debug_toolbar.middleware.DebugToolbarMiddleware',] + MIDDLEWARE

base_settings.py
[…]
DEBUG = False

INSTALLED_APPS = (
    'django.contrib.auth',
   […]
    # enabled on DEBUG=True ; see settings.py
    # 'debug_toolbar',
)

MIDDLEWARE = [
    # enabled on DEBUG=True ; see settings.py
    # 'debug_toolbar.middleware.DebugToolbarMiddleware',
    'django.contrib.sessions.middleware.SessionMiddleware',
    […]
]

[…]

local_settings.py (not git-versioned, example for development setup)
DEBUG = True

Thanks for your input !
At the moment, I ended up with the following compromise (a bit a magic don't hurt to be confy and safe at the same time !)
settings.py
from django.core.exceptions import ImproperlyConfigured

from .base_settings import *

# Override parameters with local_settings.py
try:
    from .local_settings import *
except ImportError:
    raise ImproperlyConfigured(
        'You need to create a local_settings.py, see README.md'
    )

if DEBUG:
    # Enable debug toolbar only if DEBUG=True
    INSTALLED_APPS = INSTALLED_APPS +  ('debug_toolbar',)
    MIDDLEWARE = ['debug_toolbar.middleware.DebugToolbarMiddleware',] + MIDDLEWARE

base_settings.py
[…]
DEBUG = False

INSTALLED_APPS = (
    'django.contrib.auth',
   […]
    # enabled on DEBUG=True ; see settings.py
    # 'debug_toolbar',
)

MIDDLEWARE = [
    # enabled on DEBUG=True ; see settings.py
    # 'debug_toolbar.middleware.DebugToolbarMiddleware',
    'django.contrib.sessions.middleware.SessionMiddleware',
    […]
]

[…]

local_settings.py (not git-versioned, example for development setup)
DEBUG = True

I believe the logger also gets monkey-patched regardless if it's enabled or not.
I'm torn about wanting to have this installed and be a no-op at the production level because of my perceived prevalence of devs struggling to get the toolbar to show on a non-local, staging type machine and default to setting SHOW_TOOLBAR_CALLBACK to always return True. If we screw something up and there's a leak, the toolbar will track all requests and be at risk of exposing a lot of sensitive information. Especially if we introduce a way to persist the data to support multi-threaded servers or Channels.
An alternative is to make it easier to configure SHOW_TOOLBAR_CALLBACK. Perhaps a predefined function that returns true if request.user.is_superuser == True? We'll likely want to sanitize the locals and request data similar to what Django does with the error emails with the option to show those values.

I believe the logger also gets monkey-patched regardless if it's enabled or not.
I'm torn about wanting to have this installed and be a no-op at the production level because of my perceived prevalence of devs struggling to get the toolbar to show on a non-local, staging type machine and default to setting SHOW_TOOLBAR_CALLBACK to always return True. If we screw something up and there's a leak, the toolbar will track all requests and be at risk of exposing a lot of sensitive information. Especially if we introduce a way to persist the data to support multi-threaded servers or Channels.
An alternative is to make it easier to configure SHOW_TOOLBAR_CALLBACK. Perhaps a predefined function that returns true if request.user.is_superuser == True? We'll likely want to sanitize the locals and request data similar to what Django does with the error emails with the option to show those values.

Perhaps a predefined function that returns true if request.user.is_superuser == True

I think this could go a long way. Although I might suggest making a new permission if possible instead of directly using the is_superuser flag.

Although I might suggest making a new permission if possible instead of directly using the is_superuser flag.

I can see a future in which that's reasonable where the toolbar has the serializable branch done, can run in production with the metrics logged somewhere, and then accessed via the web app. However, that's a very optimistic perspective on the toolbar's feature development.
I'd be more in favor of keeping it very basic until it needs to be expanded.

OK, I can see that. Adding the permission would be a backward-compatible change, after all. It feels like the "right" way to do it, but this is more about making sure the production failure mode is mitigated to an acceptable degree than anything else.


Support for Windows 7, 8

This commit fixes #108

Think I'm having this problem as well. Why was this never merged back in 2014?


[BUG/FEATURE] Disable "Open with JCE" on unauthorised sheets.

Describe the bug
Players are able to open the JCE Editor on any sheet they have access to. While this is harmless as they cannot edit the sheet, they do and can read the entire HTML of it including the <secret> tags, which is obviously a breach of what that tag is supposed to do
To Reproduce
Steps to reproduce the behavior:

Create a journal
Right click and open JCE as another user who does not have writing permissions, only to view

Expected behavior
There is no button to open the JCE or at the very least, the button opens with a permission error, rather than have the code be shown.
Foundry VTT version:
Core Version: 0.8.7
System: dnd5e 1.3.5

This is also an issue if you enable Autoopen, so I think my favorite solution would be to add a setting to for minimum permission level needed (i.e. Trusted, Assistant, GM) for the module's functions to be available.
Alternatively, I could also check permissions before opening and make an error message appear like you suggested, but I feel like that's messy. It might work well in tandem with the setting described above.
Unfortunately, I don't think I can easily selectively change the context menu for each document, since it's per sidebar tab, but I will look into this.
Thanks for bringing this to my attention

It now only works for the GM


Specify k6 container when reading logs

Signed-off-by: Marcos Candeia<EMAIL_ADDRESS>See #164

 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.


Functional tests for .NET Core CLI 2.1.X

FYI @mikeharder I cherry-picked a fix from 2.2 onto this PR to fix one of the mismatched prodcon variables.


Remove P210 and P410 formats

According to https://docs.microsoft.com/en-us/windows/win32/medfound/10-bit-and-16-bit-yuv-video-formats#p216-and-p210, P210 is 2 planes (Y plane + interleaved UV plane and each sample is stored as a WORD.  There is no definition for P410, but one would assume it follows the same pattern.
Unfortunately, P210 and P410 in vaapi-fits are implemented incorrectly as 3 planes (Y plane + U plane + V plane).  And, AFAICT, ffmpeg and gstreamer do not have appropriate formats to handle the official P210 and unofficial P410 formats.
Fortunately, #164 and #163 will add support for Y210 and Y410 which have better support across the media SW stacks (drivers & middleware).  Once these patches are approved and merged, we should remove P210 and P410 code and tests in vaapi-fits.

BTW, ffmpeg Y210 and Y410 support are still pending:
https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=102
<EMAIL_ADDRESS>Regardless, we don't need to wait for those patches to be accpeted and should proceed with removing P210 and P410 here.

Sure, I will

#169 #174


GPMDP freezes after ~10 seconds of playing music

OS:
Windows 10 build 14393.693
Issue Descriptions:
The desktop player keeps freezing when I play music and go to another program (e.g., Chrome) for about 10 seconds.
Steps to Reproduce:

Go to any album from any artist
Press the play button on the left side of the song
Unfocus on GPMDP for about 10-15 seconds
Go back to GPMDP and notice that it's frozen and will not respond

debug_info.zip

Disable desktop notifications.  This is a windows bug

Thanks


Defaults for Destructuring Assignment

Idea
  options = { price: 30, markdown: 3.2 }
  { type = 'Create' , price, markdown } = options
  console.log(type) #Create

Motivation
For functions that take in multiple arguments, using an options argument has no simple way to make defaults for missing properties.  However this could be helpful when destructuring anything.  Thoughts?

There was a very similar proposal at #810. I like the idea, but to quote @jashkenas:

My opinion is that this adds way too much complexity into the already-difficult-to-mentally-parse pattern matches. I'd prefer to extract my variables in one step, and then default them in another, even if this syntax did exist.

Closing as duplicate.

By the way, the approach I'd suggest for your scenario is to use Underscore's extend or defaults functions, e.g.
{type, price, markdown} = _.extends {}, {type: 'Create'}, options

Why not use CoffeeScript's extend?
options = { price: 1.00, markdown: 0.3 }
{ helpers: { extend } } = require "coffee-script"
{type, price, markdown} = extend { type: 'Create' }, options
console.log [ type, price, markdown ]

@bigeasy: and add CoffeeScript as a runtime dependency? That's a ton of overhead. Why would anyone ever do that? You'd be much better off using underscore and taking advantage of all the other convenience methods they provide.

@bigeasy It's such a small function, I would consider just subsuming it into your own personal helpers library.  This way you break the dependency with very little duplication:
# Extend a source object with the properties of another object (shallow copy).
extend = exports.extend = (object, properties) ->
  for key, val of properties
    object[key] = val
  object

There's no guarantee coffee-script will keep helpers.coffee structured as it is now.  That's why I recommend that you either copy the function into your own codebase or use underscore.js (which makes more of a promise of a stable API).


My custom shortcut key is invalid because of the now python plugin

Environment data

VS Code version: 1.50.1 (system set up)
Extension version (available under the Extensions sidebar): v2020.9.114305
OS and version: Win7 x64 6.1.7601 & Win10 Pro 2004
Python version (& distribution if applicable, e.g. Anaconda): PYthon 3.8.3 64 bit
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions: N/A
Relevant/affected Python-related VS Code extensions and their versions: N/A
Value of the python.languageServer setting: N/A

Expected behaviour
I hope that in the global situation, Ctrl+; key combination is available, but it is not available now, it is intercepted by default shortcut keys of python plugin
Actual behaviour
In the global scope, Ctrl+; shortcut keys is invalid
Steps to reproduce:
I have used VS CODE for many years, and I have also defined a unified key combination to edit C/C++, Java, Python, C#, etc.,
Such as using the combination key Ctrl+; replaces the END button, This is shown in the keybinding.json file：
    {         "key": "ctrl+;",         "command": "cursorEnd",         "when": "textInputFocus"     },     {         "key": "end",         "command": "-cursorEnd",         "when": "textInputFocus"     },
but recently, I updated the Python plugin and found ctrl + ; is invalid. The reason is that the default shortcut key of the new version of python plug-in will intercept Ctrl+; it makes this button invalid.
I tried to delete the default shortcut key, but it still doesn't work.
Even if I delete the default shortcut key of the python plug-in, it is still displayed in the keybindings.json file and will intercept Ctrl+;， like this：
    {         "key": "ctrl+; c",         "command": "-python.datascience.changeCellToCode",         "when": "editorTextFocus && python.datascience.featureenabled && python.datascience.hascodecells && !notebookEditorFocused"     },     {         "key": "ctrl+; m",         "command": "-python.datascience.changeCellToMarkdown",         "when": "editorTextFocus && python.datascience.featureenabled && python.datascience.hascodecells && !notebookEditorFocused"     },     {         "key": "ctrl+; x",         "command": "-python.datascience.deleteCells",         "when": "editorTextFocus && python.datascience.featureenabled && python.datascience.hascodecells && !notebookEditorFocused"     },     {         "key": "ctrl+; a",         "command": "-python.datascience.insertCellAbove",         "when": "editorTextFocus && python.datascience.featureenabled && python.datascience.hascodecells && !notebookEditorFocused"     },     {         "key": "ctrl+; b",         "command": "-python.datascience.insertCellBelow",         "when": "editorTextFocus && python.datascience.featureenabled && python.datascience.hascodecells && !notebookEditorFocused"     },     {         "key": "ctrl+; s",         "command": "-python.datascience.insertCellBelowPosition",         "when": "editorTextFocus && python.datascience.featureenabled && python.datascience.hascodecells && !notebookEditorFocused"     },     {         "key": "ctrl+; d",         "command": "-python.datascience.moveCellsDown",         "when": "editorTextFocus && python.datascience.featureenabled && python.datascience.hascodecells && !notebookEditorFocused"     },     {         "key": "ctrl+; u",         "command": "-python.datascience.moveCellsUp",         "when": "editorTextFocus && python.datascience.featureenabled && python.datascience.hascodecells && !notebookEditorFocused"     },
I think the python plug-in custom default The shortcut key of is not a good idea,
it will make the shortcut key is also invalid when I editing java, c# code, this is a global scope, will have a great impact

Logs

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

Hi @ryantian2000, thanks for reporting a bug and sorry for the trouble caused. These new keybindings do not have global scope--they are active only when working in a Python file containing cell markers (which are # %% by default), as specified in the 'when' clauses in the screenshot above.
You should be able to override the default keybindings that we introduced by adding your custom bindings in keybindings.json as described here: https://code.visualstudio.com/docs/getstarted/keybindings#_advanced-customization. Please let us know if that works.

I found that other programs intercepted the key combination, not caused by the python plug-in.


Outdated references

Which Cloudflare product does this pertain to?
Zero Trust
Existing documentation URL(s)
https://developers.cloudflare.com/cloudflare-one/connections/connect-networks/get-started/create-remote-tunnel/
What changes are you suggesting?

Log in to Zero Trust and go to Networks > Tunnels.

There is no longer a Network menu entry in the Zero Trust dashboard. It's called Access instead.
Additional information
No response

This was a dashboard bug that has since been fixed.


When you sleep/suspend and wake up sometimes hyprland changes workspace

When I sleep my system with systemctl if I am on the first workspace then when I resume Hyprland is on the 3rd (which in this case is the fisst workspace without application windows open), their is no animation shown both when sleeping and resuming.
I would expect Hyprland to stay on the workspace that it was on when I suspended.
Config file:
hyprland.conf.txt
Log file:
hyprland.log

dupe of #1980

Still have this problem. I use 1 monitor

Imy having this issue as well. When waking up it starts on ws5 and I only have 4 configured. Also the apps on the first ws are gone but that is another issue I think...


wifi-faq.txt Rough Draft Update

Did this work?!

As discussed in https://www.reddit.com/r/wifi/comments/eunz7y/if_you_have_a_question_read_the_wiki_before/fyqqtl6


Problem when installing

Description
I am trying to install this module from the PSGallery. But it fails and I am unable to figure out why
Environment
PS> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.14393.953
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.953
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Screenshots

Are you able to install any modules or is this specific to my modules?

your modules

Can you check to see what *Px modules you currently have installed in your %ProgramFiles%\WindowsPowerShell\Modules folder as well as in your Documents\WindowsPowerShell\Modules folder? I'm curious if you have old versions installed in the %ProgramFiles% folder, and if PowerShellGet wants to put updated versions there as well as a result, but cannot because you're not elevated. Just a hunch so far.


Anyone did solve this issue and figure it out? How this challenge work? Please help!

Challenge Word Blanks has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

function wordBlanks(myNoun, myAdjective, myVerb, myAdverb) {
  var result = "";
  // Your code below this line

  // Your code above this line
  return result;
}

// Change the words here to test your function
wordBlanks("dog", "big", "ran", "quickly");

Hi,
Please use the forums or chatroom for help!
https://forum.freecodecamp.com/t/how-to-report-a-bug/19543
Cheers!

@rabinrai44
Thank you for reporting this issue.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "Help" button on the challenge on freeCodeCamp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification. Thank you, and happy coding.

Okay. I got that solution. Thank you for reply 👍


npx shadcn-ui@latest doesn't add Input and Button

npx shadcn-ui@latest doesn't add Input and Button
While I understand that using Input and Button is a choice of user, but it should be stated in the Form guide to install those if you want to use those components.
Can be fixed by simply adding a note in the Form guide to install those components if you're going to use them.

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Fix "Could not find method nexusStaging()"

This fixes a build error introduced by the recommended way to configure nexusStaging.
See Codearte/gradle-nexus-staging-plugin#16 if you want more details.

@PLNech So if I get it right, you introduced the Nexus plugin to automate the publication to Maven Central, but this introduces a dependency that is not available in some contexts, making the build fail? And this PR fixes it?

@clement-leprovost-algolia: Exactly, the default setup required any project to add it in its root build.gradle to compile.
With this fix we only apply the plugin conditionally with the nexus property, as in here, when releasing.


Create directive that prints out violation message

To simplify the custom violation message templates, we should provide a directive that just prints the violation message from the current scope.
With this directive, the users of valdr would not need direct knowledge of the structure of the violation object to create custom message templates.
suggested by @B8li

:+1: for this


Include the js folder in typescript config to fix "npm run tsc"

Summary of Changes
This PR fixes npm run tsc, which was error-ing previously.
Addresses this issue
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes, including screenshots of any UI changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public python functions and the classes in the PR contain docstrings that explain what it does

[x] PR passes all tests documented in the developer guide
[x] I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message"

Codecov Report

Merging #151 into master will increase coverage by 20.8%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #151      +/-   ##
==========================================
+ Coverage   53.13%   73.93%   +20.8%
==========================================
  Files         106       20      -86
  Lines        2650      890    -1760
  Branches      273       76     -197
==========================================
- Hits         1408      658     -750
+ Misses       1213      206    -1007
+ Partials       29       26       -3

Impacted Files
Coverage Δ

...ication/static/js/components/common/Tabs/index.tsx

...ndsen_application/static/js/ducks/search/api/v0.ts

...ication/static/js/components/common/Flag/index.tsx

...ts/common/ResourceListItem/TableListItem/index.tsx

...sen_application/static/js/ducks/allTags/reducer.ts

...tatic/js/components/common/ScrollTracker/index.tsx

...pplication/static/js/ducks/popularTables/api/v0.ts

...js/components/TableDetail/WatermarkLabel/index.tsx

...sen_application/static/js/config/config-default.ts

...n/static/js/components/common/InfoButton/index.tsx

... and 76 more

Continue to review full report at Codecov.

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

is there a way to re kickoff that CLA check?

@alran just click the blue Details link in the right side of the CLA ckeck section, and you’re lead through signing by using your GitHub account as far as I can remember!

@alran I think @jhettler encountered this issue before in (https://github.com/lyft/amundsendatabuilder/pull/23) . My understanding is that you need to config the github user.name and user.email locally first(could confirm with git config --list), then push to the branch which should work. There are some similar discussions in https://github.com/lyft/amundsendatabuilder/pull/23 . Let us know if it works or not.

nice 👍

Cool @alran - not having spent a lot of time with TypeScript I didn't know about tsc
Contrats on your first Amundsen PR 🎉


fix travis

Seems like the travis setup is broken at the moment. We should fix this!

This seems to work again. Closing this.


Invalid prisma.course.findUnique() invocation while trying to fetch

Discussed in https://github.com/prisma/prisma/discussions/22325

Originally posted by Akhil7K December  9, 2023
Question
While trying to find the unique course which is an unique field defined in the prisma schema file.
id is defined as INT in course model in prisma schema file.
courseId has been defined as number in page.tsx (error arises here).
Error in terminal:
Invalid `prisma.course.findUnique()` invocation:

{
  where: {
    id: "1",
        ~~~
    userId: "user_2YTcfLAelOkdGjy6SAaUpztHPxc"
  },
  include: {
    chapters: {
      where: {
        isPublished: true
      },
      orderBy: {
        position: "asc"
      }
    },
    attachments: {
      orderBy: {
        createdAt: "desc"
      }
    }
  }
}

Argument `id`: Invalid value provided. Expected Int, provided String.

As it can be seen in the above provided error message id is supposed to be a number but a string is being returned which I believe is the cause for the error. I've been through my schema.prisma file and other related files where I have used courseId as a param. page.tsx is the sole file throwing this error.
Expected behavior (optional)
Course creation page is supposed to load without any errors.
Information about Prisma Schema, Client Queries and Environment (optional)

// Add your schema.prisma

// Add any relevant Prisma Client queries here

This is the concerned course model, as it can be seen id has been defined as an INT type, and the subsequent relation betweeen models also maintains the same INT type.

OS: Windows 11

Database: MySQL

Node.js version: v20.7.0

Run prisma -v to see your Prisma version and paste it

prisma                  : 5.7.0
@prisma/client          : 5.7.0
Computed binaryTarget   : windows
Operating System        : win32
Architecture            : x64
Node.js                 : v20.7.0
Query Engine (Node-API) : libquery-engine 79fb5193cf0a8fdbef536e4b4a159cad677ab1b9 (at<EMAIL_ADDRESS>Schema Engine           : schema-engine-cli 79fb5193cf0a8fdbef536e4b4a159cad677ab1b9 (at<EMAIL_ADDRESS>Schema Wasm             : @prisma/prisma-schema-wasm 5.7.0-41.79fb5193cf0a8fdbef536e4b4a159cad677ab1b9
Default Engines Hash    : 79fb5193cf0a8fdbef536e4b4a159cad677ab1b9
Studio                  : 0.495.0
Preview Features        : fullTextIndex, fullTextSearch

Closing as this is just an active discussion turned into an issue.


Additional config.verbose check for startup message

Great library! Reduced our webpack build times by 200%
Our app has a lot of console messages on startup and it can get a bit cluttered with all the happypack logs, so I'm hoping to keep them hidden if config.verbose is set to false :)

[Current coverage][cc-pull] is 86.52%

Merging [#51][cc-pull] into [master][cc-base-branch] will increase coverage by <.01%

@@             master        #51   diff @@
==========================================
  Files            20         20
  Lines           763        764     +1
  Methods         164        164
  Messages          0          0
  Branches        132        133     +1
==========================================
+ Hits            660        661     +1
  Misses          103        103
  Partials          0          0

Powered by Codecov. Last updated by [8b58708...0832c26][cc-compare]
[cc-base-branch]: https://codecov.io/gh/amireh/happypack/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/amireh/happypack/compare/8b5870865837eaf0de0f899066e76a3e54257577...0832c267e72cc8320170317af3715a20438cd62c
[cc-pull]: https://codecov.io/gh/amireh/happypack/pull/51?src=pr

Thank you! I agree it could get noisy sometimes, especially if you have multiple plugins defined.
Will have this in in the next minor release and will update this PR once that is published.

Perfect - thanks for the quick merge.


Add action pod lib lint

Add action pod lib lint

This probably should use bundle exec as the cocoapods action: https://github.com/fastlane/fastlane/blob/master/fastlane/lib/fastlane/actions/cocoapods.rb#L17

Thanks for the PR @thierryxing, could you also include the bundle exec prefix, similar to how CocoaPods does it? https://github.com/fastlane/fastlane/blob/master/fastlane/lib/fastlane/actions/cocoapods.rb#L17

OK, I'll add it right now.

All done!

Looks great, however the Actions.md change is broken. Please update the PR and let me know once it's ready to merge 👍

Got it!

All Done!
It's ready to merge.
Thank you

This is great, thank you so much! Your action will be in the next release. Thanks
@thierryxing 👍

Thank you very much for helping me finish this PR.@KrauseFx,@marcelofabri,@asfalcone
Fastlane is an awesome tool, and it has been helping us to build our own CI and CR system and save quite a lot of time for us.
It is my great pleasure to contribute back to Fastlane, I will keep doing this as possible as I can.


Add missing setup call

While DijkstraOneToMany evaluates the timeout, it's missing the setup.

Currently we're sort of doing both? The visited nodes are reset on each call:
https://github.com/graphhopper/graphhopper/blob/6f56fc37dd73417c3c788bbf0cab0f6892f5b928/core/src/main/java/com/graphhopper/routing/DijkstraOneToMany.java#L150
IMHO we should reset all limits per run. Enforcing them over multiple passes should be up to the caller.

I agree that the time limit should rather be used per call, because there might be unrelated things running in between the calcPath call that likely should be excluded from the time measurement. For the visited nodes and the maximum weight I think the global limits make sense, because the current weight and visited nodes can only increase from call to call anyway. Or do you have an actual use case for per-call limited weights and nodes?

I'm currently using DijkstraOneToMany to compute a distance matrix. While CH-based routing would be an order of magnitude faster, it's unfortunately not an option for my use case, which involves more dynamic custom models.
Global limits would have to scale with the number of destinations for which the algorithm instance is to be reused. The problem is that some destinations might simply not be reachable, and I don't want GraphHopper touching the whole graph to come to that conclusion. In the event that a limit is reached while calculating the path for a destination, I would still want to continue with the remaining destinations, which is not possible with a global limit in place.

In the event that a limit is reached while calculating the path for a destination, I would still want to continue with the remaining destinations, which is not possible with a global limit in place.

You mean if you are trying to calculate a path to node A which is only found at, say, weight=1000, and it isn't found because the limit is 100 for example you aren't able to find node B at weight 50? That shouldn't be of course. I think the way it should work is:

start the search to some node, which grows the Dijkstra tree until it is found or the weight limit is reached
search for another node. If it was discovered already return it, otherwise keep growing the tree (up to the weight limit if necessary)

This algorithm is basically left over from a previous CH preparation algorithm, so it is quite possible that it doesn't fit more general usages.

search for another node. If it was discovered already return it, otherwise keep growing the tree (up to the weight limit if necessary)

As long as paths for already discovered nodes are still returned even when the algorithm has reached one of its limits, it would be fine.

Ok, but you are saying this is currently not the case? Anyway as far as I am concerned we can merge this PR.

I think the current state is fine.


use KAFKA_GROUP_ID instead of GROUP_ID

I would occasionally get "0" from GROUP_ID which caused a kafka group
of "0" to get created, oops.
This sets the name to KAFKA_GROUP_ID. We also select a reasonable
default for the kafka group ID instead of leaving it blank.

it was s2i that was setting GROUP_ID
src/root/opt/app-root/etc/generate_container_user:GROUP_ID=$(id -g)

Run create-manifest.py and you're good to go


Add missing onKeyDown function to CalendarDay

Eep.
Fixes https://github.com/airbnb/react-dates/issues/899
@ljharb

Coverage decreased (-0.2%) to 85.487% when pulling 4d3b312a5367c7711a78d9e9e4c43824fef08bbf on maja-fix-on-key-down-error into 94e18523fa8559fbf68d667b6f42697d8db3cdf4 on master.


Updating cluster config uses wrong component version

Description
When updating cluster configuration through reconciler API the reconciler reports error.
{"level":"DEBUG","time":"2021-12-03T09:46:02.128Z","message":"Bookkeeper evaluated reconciliation (schedulingID:fb7c515e-c0dd-4808-b94a-757eed445615--3a4cb5bf-0003-493e-99d4-d8565b4dc1fd) for cluster 'fb7c515e-c0dd-4808-b94a-757eed445615' to cluster status 'reconciling': Done=CRDs,cluster-essentials,istio-configuration,certificates,cluster-users,rma,busola-migrator,service-catalog-addons,serverless,eventing,api-gateway,helm-broker,monitoring,ory,rafter,connectivity-proxy,application-connector,service-manager-proxy,compass-runtime-agent,service-catalog,logging / Error= / Other=btp-operator[error: Failed to get manifest for component 'btp-operator' in Kyma version '2.0.0-rc7' using repository 'https://github.com/kyma-incubator/sap-btp-service-operator/releases/download/v0.1.18-custom/sap-btp-operator-0.1.18.tar.gz' : stat /tmp/reconciler/2.0.0-rc7-btp-operator/btp-operator/Chart.yaml: no such file or directory],sc-migration[error: Failed to get manifest for component 'sc-migration' in Kyma version '2.0.0-rc7' using repository 'https://github.com/kyma-incubator/sc-removal/releases/download/0.5.0/sap-btp-operator-migration-v0.5.0.tgz' : stat /tmp/reconciler/2.0.0-rc7-sc-migration/sc-migration/Chart.yaml: no such file or directory]"}

reconciler_payload.json
Expected result
Reconciler to install BTP Operator
Actual result
Reconciler failed to install BTP Operator
Steps to reproduce
https://github.com/kyma-project/control-plane/blob/2b282dc12b723ca238ec75c308a563caecd98123/components/kyma-environment-broker/internal/process/update/apply_reconciler_configuration.go#L41-L53
Troubleshooting

fwiw, same bug with rc8
{"level":"WARN","time":"2021-12-07T15:33:52.052Z","message":"Runner: failing reconciliation of 'sc-migration' in version '2.0.0-rc8' with profile 'Production': Failed to get manifest for component 'sc-migration' in Kyma version '2.0.0-rc8' using repository 'https://github.com/kyma-incubator/sc-removal/releases/download/0.5.0/sap-btp-operator-migration-v0.5.0.tgz' : stat /tmp/reconciler/2.0.0-rc8-sc-migration/sc-migration/Chart.yaml: no such file or directory","correlation-id":"756cd798-e500-4f1b-a21d-dd03fd6b6fc2--535a3304-1f5b-4471-adda-7ca8e6e2c52d","component-name":"sc-migration"}

hey @m00g3n, further debugging shows that reconciler accepts only a subset of valid helm charts. Currently there appears to be a requirement that helm chart is packed nested under a directory named exactly as the helm chart. It should accept helm charts without any subdir.


Published posts section should be collapsed upon loading

I find the list of published posts to be annoying since I have to scroll beyond it to see the actual stats. Note that this list doesn’t exist in the Android app, and on the web it’s collapsed.

cc: @nbradbury

To be in par with the web version we should remove the Published module.

Closing as we removed this section.


[Proposal] Allow #if to guard switch case clauses

https://lists.swift.org/pipermail/swift-evolution/Week-of-Mon-20170508/036568.html

Closing.
We don't need full evolution process for this feature.


CDKTF project panics (golang) in container run from linux but not mac

Describe the bug
I have a CDKTF project running in a Docker image. The image is pretty light: focal-slim with terraform, nodejs, and cdktf installed -- and then my golang application that uses cdktf/jsii to create AWS resources.
When I run a statically-linked go binary, it works fine in my local mac environment (even in a Docker container) but gives errors when ran from the same container in a Kubernetes linux environment.
I know this is a lot of unrelated technology, but I do believe the underlying issue is most closely related to jsii as the problem only occurs when an AWS resource (via jsii) is encountered in the code.
Expected Behavior
A barebones Docker image behaves similarly between mac and linux environments.
Current Behavior
Running my Docker image locally (mac) and executing my golang binary generates the expected cdktf output. However, when the image is run in a Kubernetes environment (linux) and I exec in to run my binary, I get the following error:
panic: no response received from child process

goroutine 1 [running]:
github.com/aws/jsii-runtime-go/runtime.Load({0xf23b0d, 0x13}, {0xf168d2, 0x6}, {0x3190e40, 0x12662a9, 0x12662a9})
        external/com_github_aws_jsii_runtime_go/runtime/runtime.go:43 +0x91
github.com/hashicorp/cdktf-provider-aws-go/aws/v9/jsii.Initialize()
        external/com_github_hashicorp_cdktf_provider_aws_go_aws_v9/jsii/jsii.go:27 +0x85
github.com/hashicorp/cdktf-provider-aws-go/aws/v9.NewAwsProvider({0x7f033f190348?, 0xc000498800}, 0xc000498ba0, 0xc0004c69a0)
        external/com_github_hashicorp_cdktf_provider_aws_go_aws_v9/aws.go:63618 +0x45
[...]

Reproduction Steps
N/A
Possible Solution
No response
Additional Information/Context
I assume there is some sort of dependency needed that ins't included in the binary itself, even with Golang? I did verify that things work fine up until the moment I try to create an AWS resource (via github.com/hashicorp/cdktf-provider-aws-go/aws/v9).  I believe this is related to when the child node process is created?
I appreciate any guidance on the direction to explore for this type of issue. I'm a little stumped given the container runs correctly in a local environment.
SDK version used
github.com/aws/jsii-runtime-go v1.63.2
Environment details (OS name and version, etc.)
Mixed, details above

Apologies, not a jsii issue afterall. This was a Kubernetes container resource limit -- but because the spawned child process isn't the primary process, the pod was not killed and it just silently failed.


No compatible HID devices found with DTU-1141B connected

Dear devs,
i tried using the SDK, cloning this repo, getting the adequate files from wacom dev support and the keys, yet the software simple demo does not see my DTU-1141B as a compatible device?
I asked support, and they told me that the 1141B is supported for this SDK. Is it so?
Thank you in advance.

Wich operating system are you using? I am on Linux Mint (Ubuntu based distro) and i have modified /dev/hidraw* permission to get my device viewed by library...

Hey, i am using win11 at the moment, with the latest drivers.

so turns out, it does support it, but not as a hid device, but as a generic device.


Create connection to DB

Create a connection to a local database for testing connection and sql statements on test data.

Issue closed with create-connection-to-db pull request #45


Unclear about what if Right-hand-side (RHS) of has operator is not a pure alphanumeric word

I checked these 2 docs but it is unclear what will happen if Right-hand-side (RHS) of has operator is not a pure alphanumeric word:

https://docs.microsoft.com/en-us/azure/data-explorer/kusto/query/datatypes-string-operators
https://docs.microsoft.com/en-us/azure/data-explorer/kusto/query/has-operator

print("https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourcegroups/test.rg000001?api-version=2021-04-01" has "resourcegroups/test.rg000001")
true

print("https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourcegroups/test.rg000001?api-version=2021-04-01" has "/resourcegroups/test.rg000001?")
true

print("North America" has "North America")
true

The tests show that if Right-hand-side (RHS) of has operator is not a pure alphanumeric word and contains several parts like resourcegroups/test.rg000001 or /resourcegroups/test.rg000001?, the has operator can still do a word match.

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

ID: b54a3e89-5278-6712-42b8-146c4f6c57dd
Version Independent ID: 6d773e2a-596d-5c8f-28d1-9b61389f1089
Content: The case-insensitive has string operator - Azure Data Explorer
Content Source: data-explorer/kusto/query/has-operator.md
Service: data-explorer
GitHub Login: @orspod
Microsoft Alias: orspodek

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

We have assigned the issue to the content author. They will evaluate and update as appropriate.

@orspod - Could you please help loop relevant product owner who can clarify on this ask. Thank you

@jiasli Thanks for contacting us with feedback about our product documentation. Depending on the complexity, it could take from a few business days to a few weeks to look further into your question, issue, or suggestion. We'll contact you if we need further information, and you'll receive an email message each time we "@mention" you in a comment.

@jiasli Thanks again for your feedback. The RHS can be any scalar or literal expression as mentioned in the arguments section. I'm closing this issue, but we'll continue to monitor if there is a need to update the descriptions.
#please-close


Wrong link in Rasa Contribute page

Issue:
On Rasa Contribute page, at the "Like to write code?" tab, the link to Rasa Docs is wrong. It is redirecting the user to the Report a Bug instead.
Error (including full traceback):

In the bottom left corner of the image, you can see that the link added is https://github.com/RasaHQ/rasa/issues/new/choose instead of https://rasa.com/docs/

I don't know if this issue should've been open here, but I didn't find another place where I could've done that, so.... 😅

Hello @virtualroot may I make PR for this issue?

Thanks for raising this issue, @mvielkind will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗

@lunelson are you able to update this?

I think this is intentional, but I agree it's a bit confusing: opening a new issue is actually the right way to contribute to the docs, but maybe @karen-white has some thoughts on what this should say?

@lunelson How about, "Open a Feature Request issue to suggest new code examples for the Rasa docs"

Another thought: keep the language the same but link here

@karen-white I think that's an elegant solution—PR is up for review


node

Hi all,
Is there any way for me to get node information during the solver searching the optimal solution? I want to plot a graph where the x-axis is the time step and the y-axis is the best found solution at a particular time point.

thank you
Xiangyi

For this you might want to use the Eventhdlr plugin to collect the data whenever a certain event is triggered, e.g. a new solution is found. Look here for an example implementation: https://github.com/mattmilten/TreeD

For this you might want to use the Eventhdlr plugin to collect the data whenever a certain event is triggered, e.g. a new solution is found. Look here for an example implementation: https://github.com/mattmilten/TreeD

Hey! Thanks a lot. What you have suggested really works. :)


added draw_polygon

Identical to area but with four lines deleted, and a new doc comment.
I think I'm going to try also doing a second PR that's the same but that pulls out the code that area and draw_polygon into an internal function.  If github will allow two separate PRs (??), then you can choose.
(I understand the value of having different functions that duplicate code.  It's simpler to understand, and if you want to change one but not the other, it's easier.  However, in this case draw_polygon really is area with four lines deleted, so maybe it's reasonable to have them call the same internal function to run the code that's the same in both functions.)
This is in response to #107.

OK, as I suspected, I can only have one PR live at the same time.  If you want, take a look at this:
https://github.com/mars0i/owl/blob/master/src/owl/owl_plot.ml#L1014
It pulls out the code that's common to area and draw_polygon and puts it in a function _area_fill that area and draw_polygon call.  I'll update this PR with that code if you prefer that coding style for area and draw_polygon.  Either way of writing these functions seems fine to me.

Thanks Marshall. IMO, the first PR looks better. I know there is some overlap between the two functions but it looks clearer and each is self-contained. Let's abstract the common part out in future if there are more similar functions added.

@mars0i Can you revert to the first PR so I can merge?

Oh, sorry.  I forgot that when I pushed the change it would go into the PR.  That behavior goes against my intuitions that I continually forget, even when I have just been reminded of it.

Many thanks @mars0i , this is a really nice PR, great job. 👍
I am still open to discussion about introducing the specific functions you mentioned. We just need a bit time to have a look at how others do things. But this draw_polygon is definitely very general and much needed imo.

That all sounds good.  Thanks.  I'm very happy to have this in the master branch, and will be using it immediately.  It's the sort of thing that's difficult to add without inserting it into owl_plot.ml, where all of the necessary definitions have been set up, but maintaining a separate branch eventually becomes problematic.


add packaging CI to upstream repo

It would be nice if we had a CI running on upstream repo (like OpenROAD) to be aware that one of their PR in review is potentially breaking packaging (see #259 and #261)
/cc @msaligane

We should discuss if that workflow should be in upstream repo on conda-eda's.


Allow using ISO 639-2 (three-letter) language codes

Proposed change

Allow using ISO 639-2 (three-letter) language codes
Type of change

[ ] New country/market holidays support (thank you!)
[ ] Supported country/market holidays update (calendar discrepancy fix, localization)
[x] Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency upgrade (version update)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] This PR is filed against beta branch of the repository
[x] This PR doesn't contain any merge conflicts and has clean commit history
[x] The code style looks good: make pre-commit
[x] All tests pass locally: make test, make tox (we strongly encourage adding tests to your code)
[x] The related documentation has been added/updated (check off the box for free if no updates is required)

coverage: 100.0%. remained the same when pulling bf6d60a0d1bc2566bf25369328a7a167da3805e9 on arkid15r:allow-639-2-language-codes into bf54ecdeb526e082d90e076a204c162f4e4f1e61 on dr-prodigy:beta.

Nice, this would really help with pap Papiamento/Papiamentu language for #1303 and #1309

Nice, this would really help with pap Papiamento/Papiamentu language for #1303 and #1309

I was surprised you hadn't tackled it yourself. You're mature/familiar enough w/ the codebase to make such kind of changes. And you always have an option to discuss first if in doubt.
My point is... I'm  encouraging you start making deeper/wider changes when needed.
🚀


[Doc Enhancement] Add a table for Platform Support (Desktop/Web/iOS/Android) for better visibility

Type of issue
Typo
Feedback
[Doc Enhancement] Add a table for Platform Support (Desktop/Web/iOS/Android) for better visibility
Currently we have added a note like below:
AI label, sensitivity label, citations, and feedback buttons are available only in Teams web and desktop clients.
It's better to have a table like below for better visibility:

Or like below:

Page URL
https://review.learn.microsoft.com/en-us/microsoftteams/platform/bots/how-to/format-ai-bot-messages?branch=pr-en-us-10798&tabs=after%2Cbotmessage#add-sensitivity-label
https://learn.microsoft.com/en-us/microsoftteams/platform/m365-apps/teamsjs-support-m365
Content source URL
https://github.com/MicrosoftDocs/msteams-docs/blob/main/msteams-platform/bots/how-to/format-ai-bot-messages.md
Author
@surbhigupta
Document Id
1de6f2d1-f7ee-b07c-fd5a-dc6661aa0696

Hello @surbhigupta12 - Please help to update the documentation.


Update port to Photoprism<PHONE_NUMBER>af848

Hi,
Is it possible to update the port to the newest Photoprism release https://github.com/photoprism/photoprism/releases/tag/240711-2197af848
Maybe, issue #70 is fixed as well, if the -r parameter for the cp is changed to -R or it is done in other way it works in FreeBSD.
Thanks!!

@tinsukE Could you build g20240420 correctly? I opened the issue #70 since, during the process, it copies files and the -r parameter is not supported in FreeBSD, but in Linux.

@dml33 I have investigated the build failure and wrote my findings on that issue.


How to create a morphology with mut and SomaType.SOMA_NEUROMORPHO_THREE_POINT_CYLINDERS ?

I am trying to create a simple morphology with a 3 points soma with morphio, but it seems to not give the correct parent id assignment for the 3rd point. It should be to id 1, but it is on id 2.

Just to put more context:
Alexis is able to create such a morphology:

However, it is not compliant to the spec:
" The second and third soma points, as well as all starting points (roots) of dendritic and axonal arbors have this first point as the parent (parent ID 1)."
from this page: https://neuromorpho.org/SomaFormat.html
referenced by morph-io: https://morphio.readthedocs.io/en/latest/specification.html#id2 for 3 points soma.

@arnaudon: can you make a snippet of what you are doing, so I can have a look?
The soma formats are constantly a headache because when they have 3 points, they can either be a proper (IMO) set of 2 stacked cylinders, OR the 3 point soma from neuromorpho.
Clearly, this needs to be cleaned up, regardless.

Yes, I noticed, I want to make a 3 point some as in neuromorpho. But probably the best is just to save a str directly, instead of creating a morphio object and somehow tell morphio it should be a 3 point neuromorpho thing

@jdcourcol , in fact, why do we want placeholder morph to be neuromorpho 3 points, and not 2 cylinder version?

in fact, why do we want placeholder morph to be neuromorpho 3 points, and not 2 cylinder version?

What about a single point; then it's just a sphere that has the nice property that its surface area is the same as a cylinder (not including the endcaps)

@jdcourcol , in fact, why do we want placeholder morph to be neuromorpho 3 points, and not 2 cylinder version?

We don't want that: this is what the cell team provided to us. I want the format to be correct. Then if it a 2 cylinders soma, I am fine with it too.

ok, so the morph created here is a valid one with 2 cylinders, not a neuromorpho 3 points, right? https://github.com/BlueBrain/BluePyEModel/blob/main/bluepyemodel/tools/morphology.py


Nav bar hairline not visible

Might be related to the nav bar clipping maybe?

Bug Report Dump (Auto-generated)

Version 1.12.0 (1106)
Device: iPhone 7 Plus (iOS 11.0.3)
TestFlight: true

@Sherlouk “GitHawk upload by Sherlock”. Did you hardcode this 😂
Sent with GitHawk

Na I uploaded your image for you? 😂
Sooooo basically in amidst my refactor I remember "temporarily" changing it to Sherlouk while I fixed a bug. If you check like all of the first commits I had it coming from the login manager. Evidently I forgot to put that logic back in 😂
OOPSSS SORRY

I have no idea how this happened. We don’t mess with the nav bar. @BasThomas can you consistently repro? If not let’s close.
Sent with GitHawk

No, but it’s happening quite a bit. Seems like once it’s happening, only a force quit and relaunch fixes it. I’ll keep an eye on it and see if I can narrow it down.
Sent with GitHawk

Only in the issue view? You might be able to use FLEX and find out where it went.
Sent with GitHawk