Error in command: AL: Package

I want to build the app using the AL: Package command.
What could be the reason of this error?
Microsoft (R) AL Compiler version 0.16.19969
Copyright (C) Microsoft Corporation. All rights reserved

Process is terminated due to StackOverflowException.
[Info  - 14:40:58] Connection to server got closed. Server will restart.
[Error - 14:40:58] Request textDocument/codeLens failed.
Error: Connection got disposed.
	at Object.dispose (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\main.js:825:25)
	at Object.dispose (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-languageclient\lib\client.js:57:35)
	at LanguageClient.handleConnectionClosed (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-languageclient\lib\client.js:1864:38)
	at LanguageClient.handleConnectionClosed (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-languageclient\lib\main.js:106:15)
	at closeHandler (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-languageclient\lib\client.js:1852:18)
	at CallbackList.invoke (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\events.js:71:39)
	at Emitter.fire (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\events.js:135:36)
	at closeHandler (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\main.js:221:26)
	at CallbackList.invoke (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\events.js:71:39)
	at Emitter.fire (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\events.js:135:36)
	at StreamMessageReader.AbstractMessageReader.fireClose (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\messageReader.js:135:27)
	at Socket.<anonymous> (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\messageReader.js:188:62)
	at emitOne (events.js:101:20)
	at Socket.emit (events.js:191:7)
	at Pipe._handle.close [as _onclose] (net.js:510:12)
[Error - 14:40:58] Request textDocument/definition failed.
Error: Connection got disposed.
	at Object.dispose (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\main.js:825:25)
	at Object.dispose (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-languageclient\lib\client.js:57:35)
	at LanguageClient.handleConnectionClosed (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-languageclient\lib\client.js:1864:38)
	at LanguageClient.handleConnectionClosed (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-languageclient\lib\main.js:106:15)
	at closeHandler (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-languageclient\lib\client.js:1852:18)
	at CallbackList.invoke (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\events.js:71:39)
	at Emitter.fire (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\events.js:135:36)
	at closeHandler (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\main.js:221:26)
	at CallbackList.invoke (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\events.js:71:39)
	at Emitter.fire (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\events.js:135:36)
	at StreamMessageReader.AbstractMessageReader.fireClose (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\messageReader.js:135:27)
	at Socket.<anonymous> (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\messageReader.js:188:62)
	at emitOne (events.js:101:20)
	at Socket.emit (events.js:191:7)
	at Pipe._handle.close [as _onclose] (net.js:510:12)

Looked together with Bart to this issue. Seems to be a specific setting in an object, but we couldn't nail it down yet. Whatever it is, it causes a crash, while it should produce a compilation error.

Hi @bvbeek,
If you can produce a small example that would be the best case, but isn't there any stack traces that looks more like C# code?

Hi @thpeder
Tried with one object, then it works.
How can I find the source problem here in 450+ objects?

isn't there any stack traces that looks more like C# code?

Where should I look?

We have a log file here:
"C:\Users<User>.vscode\extensions\microsoft.al-\bin\EditorServices.log"

Hi Thomas,
This is the log output.
I don't see any Event Viewer log or something.
12-7-2018 13:37:23 [NORMAL] - Method "SetupEnvironment" at line 161 of C:\a\73\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Host\Program.cs

    Editor Services Host v0.16.19969 starting (pid 7532)...

12-7-2018 13:37:23 [NORMAL] - Method "Main" at line 127 of C:\a\73\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Host\Program.cs

    Editor Services Host started!

12-7-2018 13:39:39 [NORMAL] - Method "SetupEnvironment" at line 161 of C:\a\73\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Host\Program.cs

    Editor Services Host v0.16.19969 starting (pid 14796)...

12-7-2018 13:39:39 [NORMAL] - Method "Main" at line 127 of C:\a\73\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Host\Program.cs

    Editor Services Host started!

Output in VS Code:
Microsoft (R) AL Compiler version 0.16.19969
Copyright (C) Microsoft Corporation. All rights reserved

Process is terminated due to StackOverflowException.
[Info  - 13:39:38] Connection to server got closed. Server will restart.
[Error - 13:39:38] Request textDocument/hover failed.
Error: Connection got disposed.
	at Object.dispose (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\main.js:825:25)
	at Object.dispose (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-languageclient\lib\client.js:57:35)
	at LanguageClient.handleConnectionClosed (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-languageclient\lib\client.js:1864:38)
	at LanguageClient.handleConnectionClosed (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-languageclient\lib\main.js:106:15)
	at closeHandler (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-languageclient\lib\client.js:1852:18)
	at CallbackList.invoke (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\events.js:71:39)
	at Emitter.fire (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\events.js:135:36)
	at closeHandler (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\main.js:221:26)
	at CallbackList.invoke (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\events.js:71:39)
	at Emitter.fire (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\events.js:135:36)
	at StreamMessageWriter.AbstractMessageWriter.fireClose (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\messageWriter.js:57:27)
	at Socket.<anonymous> (C:\Users\bvbeek\.vscode\extensions\ms-dynamics-smb.al-0.16.19969\node_modules\vscode-jsonrpc\lib\messageWriter.js:79:63)
	at emitOne (events.js:96:13)
	at Socket.emit (events.js:191:7)
	at Pipe._handle.close [as _onclose] (net.js:510:12)

I don't think your issue is connected to 2867 since that is a problem with the debugger and this is when you are building the package.
Can you try and build your project with the command line tool Alc.exe and see if it also crashes.

@bvbeek Do you have access to Collaborate, if so could you please try with a recent Docker image.

Hi Thomas, I made this docker-vm on 5 July , so I think it's really recent right?

Hey Bart,
Do you also get the exception when compiling just the file?
I don't think having an arbitrary limit of what is a big object is a great experience either, we should rather see what we can do to make your project compile even with files that size.

Hi Thomas,
I investigate this issue as wel, and the size of the object is not the problem.
The reason is a large OutStream.WriteText(large string).
The large string contains an base64 image to import it in a demo-item.
The string is 72 characters long and counts 3637 lines

Hey Bart,
Thanks for investigating it, but I don't quite know what you mean by 3637 lines, since AL doesn't support to have multiline text literals, and I can't reproduce the problem locally.

More detailed investigation:
174675 characters will compile
174676 characters will not compile

Thanks, I'm now able to reproduce it.

Hi @bvbeek,
As a workaround for now, you should be able to write it all out in one line instead of doing all concatenations.

Hi Thomas,
Thanks for the suggestion, good luck to solve this!


Fix for StreamUtils.tar and Yarn LogCopier

Fix for StreamUtils.tar there were some problems with un-tarring the resulting object on the GNU tar distribution vs. the BSD tar distribution
Fix for LogCopier.Stopwatch

@liyinan926 can you review?

LGTM.


Not able to use it

I imported the file app.js in code.
and tried passing
`var state = {
token: '',
debug: window.location.search.indexOf('debug')>-1,
mode: 'driving',
direction:'divergent',
threshold: 3600,
resolution: 1.5,
batchSize: 400,
inspector: false
};
isochrone([12.957194522,77.5682954863], state, function(output){
console.log(output);
})`
but I am getting an error
Uncaught TypeError: Cannot read property 'durations' of undefined
at Object. (app.js:157)
at Dispatch.call (app.js:2266)
at XMLHttpRequest.respond (app.js:2516)
my call is
https://api.mapbox.com/directions-matrix/v1/mapbox/driving/12.9572,77.5683;12.9572,77.5683;12.9572,77.5873;13.0454,77.5683;13.0454,77.5873?sources=0&destinations=all&access_token=
I am new to this and jsin particular kindly help me.

I've been getting this same error intermittently. (maybe 1 time out of 5) Did you ever resolve this issue?

Same here. I logged the error associated with the call and found this:
"message":"Too Many Requests"
@peterqliu did the platform team feel this module was hammering the API too hard and lock usage down?

@mikelinc this is likely because you're rate-limited -- non-enterprise plans are unfortunately capped at 60 Matrix requests per second

I just merged a PR that addresses/alleviates some of these relevant troubles #14


fix: enable hot reload on windows

Description
Concerned with issue/bug #283
Changes

fix(dev_event_handler.py): use platform specific python3 alias to spawn processes

Documentation
No documentation update is required
How I've tested my work

26/26 tests passed (Debian 10)
25/26 tests passed (Windows 10) (global_index_request() failed)

Remarks

Although this minute code change gets the dev_index_request() test passed on windows, for some reason hot reload fails to work on my system.

Windows: Inside dev_event_handler.py, function on_any_event() gets called every time I make a change in the base_routes.py file but I cannot see changes being reflected on the browser.
Debian:  Inside dev_event_handler.py, function on_any_event() does not get called when I make a change in the base_routes.py file and thus I cannot see changes being reflected on the browser.
I just want to make sure if this is a problem that everyone else is experiencing or is it only me?

Failure of global_index_request() test persists

Hi @guilefoylegaurav , thanks for your PR.
I don't have an access to a windows machine till the 26th so can't really tell you about the windows issue.
Dev server reflects the right changes on my osx machine but is not killed properly on KeyboardInterrupt. I will check my linux PC to give you updates about the debian state.

@guilefoylegaurav , I think you pushed the same commit.

Noticing that there were some updates to the upstream, I pulled the updates down from upstream to the fork's main and rebased by feature branch with the main and push that back, actually

@guilefoylegaurav , are you working on converting duplicate code to a property?

My apologies
I am, just pushed the updates

Just one last thing @guilefoylegaurav , have you run the pre-commit instructions in the README here?
https://github.com/sansyrox/robyn#-to-develop-locally

@sansyrox
I did not run it until now
Ran it now and it did not reveal any formatting issues
I was under the impression that the hooks would run automatically on commit: https://pre-commit.com/

Ah, okay. This PR looks good. But I will do a final review tomorrow morning 😄
Also, the link that you shared is dead in the last comment.

Thanks @sansyrox
My bad, I updated the link
I am embarrassed beyond words - I messed the link up while copying it seems

I am embarrassed beyond words - I messed the link up while copying it seems. I will take care from now on to ensure that this does not happen again

Not to worry :smile: Happens with all of us 😄


[Request] Add android-studio-canary

URL to application: https://developer.android.com/studio/preview and https://developer.android.com/studio/archive
Latest version: 2020.3.1
Graphical interface: Yes
Portable version: Yes (zip)

The manifest for the stable version already exists here: https://github.com/lukesampson/scoop-extras/blob/master/bucket/android-studio.json

I had a go, but annoyingly I can't get regex to match a version number, I've got this so far if anyone wants to try:
"checkver": {
    "regex": "Download \\w+ \\(([\\d.]+)\\) Canary (\\d+)",
    "replace": "${0}.${1}"
},

Turns out it's because the site changes the HTML if you visit it with a browser while checkver downloads the original HTML before any changes. Making a PR now.

@Jojo4GH it's added now, try installing with scoop install android-studio-canary


Mitigates audio stutter

By making ubihealthd more polite and less verbose.
The -v3 flag reduces the logging level
Nice=19 makes ubihealthd have lower priority.

Thanks a lot!


⚠ Bofrost.at has degraded performance

In 0d2023d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2503 ms

Resolved: Bofrost.at performance has improved in 96cf222 after 6 minutes.


git clone not working when using ssh git url (add ssh client)

user reported that using a bash script in dockerskeleton they can't git clone a url using ssh scheme
git clone git@github.com:apache/incubator-openwhisk-runtime-docker.git

I'm guessing is because the image is missing the ssh client or some ssh client side config.
We should not enable/install ssh server/deamon/service, but look if it's possible to just add the minimum to make the git clone work over ssh.

It looks like openssh-client package in alpine only adds client no deamon.
And this is what's require to get git clone working with ssh access.
Here is an example:
docker run --rm -v ~/.ssh:/ssh  dockerskeleton bash -c "export GIT_SSH_COMMAND=\"ssh -i /ssh/id_rsa -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no\";git clone --depth 1 git@github.com:apache/incubator-openwhisk.git;cat incubator-openwhisk/README.md"


Make triage bot remove the label automatically

🚀 Feature

Motivation
It is annoying when you have to remove the triage label manually. It is also not at the top of the list.
Pitch
Make the bot remove it automatically when other labels get added by a member.
Alternatives
Keep
cc @borda

Just realized, there is no bot for this. So this is probably not possible.


Nutanix: Add support for proxy URL setting

What this PR does / why we need it:
Adds support for setting a proxy URL for Nutanix via NUTANIX_PROXY_URL or a field in the spec. This will be helpful for our E2E testing, and it's nice to support this use case for users as well (although I'm unsure how frequent that will be).
Which issue(s) this PR fixes (optional, in fixes #<issue number> format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Optional Release Note:

NONE

Signed-off-by: Marvin Beckers<EMAIL_ADDRESS>

/override pull-machine-controller-e2e-vsphere
vCenter is currently down

Another PR that requires your services sire @xrstf

/retest

/test pull-machine-controller-e2e-azure

/lgtm cancel
to prevent needless retests.

/test pull-machine-controller-e2e-azure

/override pull-machine-controller-e2e-vsphere
/override pull-machine-controller-e2e-vsphere-resource-pool
/lgtm

/retest


tailwind download

It would be helpful if _download_cli_if_not_exists was exposed as an optional tailwind download sub-command.
This would make it possible in a Dockerfile to download the binary (without architecture hoops) when the image is built so that it's not later downloaded when the tailwind build command is run.
I currently run python -m manage tailwind --skip-checks build in my Dockerfile, which is fine to get the binary file installed, but it tends to create a bunch of extra layers that seem to add up rather quickly.

Sounds like a good idea. I will add it.

Added a "download_cli" subcommand to handle this request. Before releasing it, I need to add some documentation and also want to refactor some parts of the management command in general.

🎉  Thank you for the quick reply and feature addition.

download_cli is part of 2.10.0 released today.


Poll your move doesn't work..

Hi mate,
Poll your move option in settings doesn't work for me for some reason. I don't get any notifications even tho it's set to poll once an hour...
Android 7.1.2,  LineageOS on Oneplus 2

I can confirm having the same issue.
Android 7.1.2, Google Pixel, bootloader unlocked and rooted, but otherwise stock OS.


Update docker-compose setup

Simplify the copying of go files into docker context when building the gateway
Pin the example services to golang:1.17 and node:18

Codecov Report
Base: 70.70% // Head: 70.70% // No change to project coverage :thumbsup:

Coverage data is based on head (8c8f08f) compared to base (8b8e104).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #173   +/-   ##
=======================================
  Coverage   70.70%   70.70%
=======================================
  Files          26       26
  Lines        2690     2690
=======================================
  Hits         1902     1902
  Misses        665      665
  Partials      123      123

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


⚠ Bofrost.at has degraded performance

In 2c6d728, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2123 ms

Resolved: Bofrost.at performance has improved in 0fb4eec after 6 minutes.


added pow builtin

addresses #525.  We could make it return a float per #531 also, but that might haver perf implications.

oops, I accidentally pushed this on the same branch as the fix for #525 ... The the changes in 3a6b391 and c233a70  address #532.  The generated code for a with statement now looks like this:
var ctxmgr = SomeContextManager ();
if (ctxmgr.__enter__){
	ctxmgr.__enter__ (ctxmgr);
};
body_of_with_block();
if (ctxmgr.__exit__){
	ctxmgr.__exit__ (ctxmgr);
} else {
	ctxmgr.close ();
};

So that calling with on a context-manager class with an __enter__ and __exit__ will call those on either side of the code in the with block.

Updated with exception handling that matches the python idiom:
	{
		var fred = ContextMgr (2);
		if (fred.__enter__){
			fred.__enter__ ();
		}
		try {
                            context_block_body_goes_here()
			if (fred.__exit__){
				fred.__exit__ ();
			} else {
				fred.close ();
			}
		}
		catch (_with_blk_exc_) {
			let was_handled = false;
			if (fred.__exit__){
				was_handled = fred.__exit__ (_with_blk_exc_);
			}
			if (was_handled !== true) {
				throw __except0__;
			}
		}
	};

This matches the usual Python context manager behavior, although there are a couple of minor divergences.  In Python the __enter__ must return a reference to self, in TS it does not;  and in Python exceptions within the with block return a tuple of exception_type, exception_object, traceback which is not possible in TS.

Hi Steve,
Having context manager functionality: Most welcome
Code overall: Looks good.
About the anonymous name generator:  Maybe I didn't dive in deep enough, but this seems to duplicate the automatic generation of things like temp0,  temp1 etc. that is already in the compiler.
I'd like to merge this manually in the next version (3.7.1) rather then pull it into 3.6, making some changes on the fly. The result will then become 3.7.2.
KR
Jacques

I've built a slightly simplified version of your code into the modules branch, which will become 3.7.3.
The autotests run OK, but it may be possible to slim it down a bit more.
Will take a fresh look at it at first occasion.
Good to have this feature in, thanks for the creativity and effort!

Context managers now built in 3.7.3.
Turned out that scoping of variable in with header should be non-local.
So I had to remove the function and generate in-line body.
Also took out the else {close ()} option, it was there to make the autotester compile with both TS and CPython. With the real  context managers in place it isn't needed anymore.
Added some more autotests, e.g. with nested context manager.


Implement DICG

This is a FrankWolfe variant without active set, see this paper https://proceedings.neurips.cc/paper/2016/hash/daca41214b39c5dc66674d09081940f0-Abstract.html .

Needed interface functions:

fixing the face
changing the fixing

covered in #460


chore: replace heptio/contour with project contour

Signed-off-by: kushthedude<EMAIL_ADDRESS>fixes https://github.com/projectcontour/contour/issues/2966

Hi @youngnick @sunjayBhatia I have did the requested changes, please review

Ah, I don't have official powers in this org, so someone else will have to approve officially


Token Invalid issue

I followed the instructions and got the token. Here is that i did on high level.

removed the robot from the mi home app ios.
reset the wifi.
I connected my mac mini to the robot wifi.
used Miio discover command
noted the result.
disconnected the robot.
connected it to the mi home app.
check if its working no issues with the mi home app.

Homebridge on load still throws the error token invalid. my robot is a Xiaomi mi s1 i got shipped from china since its not available in USA.
any help here?

disconnected the robot.

connected it to the mi home app.

This is the problem: The moment to re-connect the robot to a new app, a new token is issued, revoking any other one. I'd suggest you to use the backup method to retrieve the token. I think it's the most reliable one.

Thanks for the quick reply. can you help with the right link. there are too many just don't want to get confused.

This is the one that worked for me: https://python-miio.readthedocs.io/en/latest/discovery.html#tokens-from-backups :)

thanks all i got this working. now great appreciate all the help


Pin code not displayed

Hello, the installation of the integration stops at the stage of issuing a pin code. Pin code is not displayed on the TV. In my TV model  MQTT works without SSL, TLS and certificates, I checked the commands through the MQTT service on homeassistant and on mqtt explorer, command  works and connection is ok.
I tried in different browsers and on different operating systems.
connection hisense
address <IP_ADDRESS>:36669
username hisenseservice
password multimqttservice
clientid HomeAssistant
bridge_insecure false
try_private false
start_type automatic
topic /remoteapp/# both 0
2022-07-15 08:54:07 DEBUG (MainThread) [custom_components.hisense_tv.config_flow] async_step_user - user_input is None
2022-07-15 08:54:32 DEBUG (MainThread) [custom_components.hisense_tv.config_flow] async_step_user - set task_mqtt
2022-07-15 08:54:32 DEBUG (MainThread) [custom_components.hisense_tv.config_flow] _check_authentication - publish gettvstate
2022-07-15 08:54:32 DEBUG (MainThread) [custom_components.hisense_tv.config_flow] _check_authentication - publish sourcelist

Same here

The config workflow is waiting for any of these two topics:

%s/remoteapp/mobile/%s/ui_service/data/authentication
%s/remoteapp/mobile/%s/ui_service/data/sourcelist
First %s should be your prefix. The second %s should be HomeAssistant

Since your log don't show _async_pin_needed or _async_pin_not_needed it looks like you didn't receive any of those two topics.
Please check with MQTT Explorer (or any other client) if your TV is reacting to the MQTT topics during setup.


Fix FactionsUUID-related bank lookup lag

This patch changes "factions top" plugin lookups on large servers from literally minutes to seconds.
The root of the problem is generating new OfflinePlayer each lookup, vs using a cached one with the non-deprecated method that takes the Faction directly.

Thanks for this, I'll review in the morning and merge if all tests are completed correctly.
(Current time for me: 22:10)

All tests have passed, I will be keeping a close eye on this functionality to ensure Forks do not break.

Thanks,
Callum

Thanks! :)
A few questions:

Approximately when will this go live on Spigot? That way I can make a post to the relevant Discord channel about this update.
In the future, there's going to be a severely breaking update to FUUID. Would you be welcoming of a PR for this? It may be breaking enough to be worth having an entirely new implementation (still sorting out how much I'm smashing with a hammer). 😄

I'll be posting this update shortly, i just wanted to ensure that the implementation of Medieval Factions was completed, but as you've improved the efficiency, I'll push 1.1.4 tomorrow. In regards to the new implementation, I'd require a jar or a way to test the changes as I'd like this project to be as bug free as possible, but if you're willing to PR and help me out, it would be great. A lot of people still use FactionsUUID B249, most people use this as the base of their forks so a new implementation would be preferred.
Thanks,
Callum

Sounds good to me! :)


[Chore] Add on-tag workflow to deploy to npm

This should be done on the "release" creation, which is done after creating a tag. I believe this will make the overall flow like:

Make changes
Create tag and push to github
Create release using the github UI, create changelog

see: https://docs.github.com/en/repositories/releasing-projects-on-github/automatically-generated-release-notes#creating-automatically-generated-release-notes-for-a-new-release

github-actions will deploy changes to npm

see: https://docs.github.com/en/actions/publishing-packages/publishing-nodejs-packages#publishing-packages-to-the-npm-registry

Initial version added with:
25288218db9d366ae8b312bdcff21e4bbfaa223d

This should be fixed with the above, and 0.0.3 will be the first version deployed with the new flow. Version 0.0.2 isn't deployed yet.


Use Lombok's NonNull annotation for null argument checks in LevenshteinDistance.

Hi @Isira-Seneviratne
We have, so far, resisted using dependencies like Lombok in all of Apache Commons. If you think this is neccessary, feel free to start a discussion on the developer's mailing list https://commons.apache.org/mail-lists.html

Hi @Isira-Seneviratne
We have, so far, resisted using dependencies like Lombok in all of Apache Commons. If you think this is neccessary, feel free to start a discussion on the developer's mailing list https://commons.apache.org/mail-lists.html

Ah, I see. Thank you for the information.
In that case, I guess it's not necessary.


point-mutations:  verify ordering of actual/expected

In the Jasmine testing framework, it is convention to:
expect(actual).fn(expected)

But some test suites have the following:
expect(expected).fn(actual)

This causes the error messages to be confusing.
Please verify the point-mutations exercise to make sure that the expectations are ordered correctly. If they are, then this can be closed.
This issue will help to close #111

Got it

This one is OK and can be closed

Neat!


deno: 1.17.2 -> 1.17.3

Motivation for this change
Bump deno to 1.17.3
I also played around with the tests but it was a bit of a mess
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 155310 run on x86_64-linux 1

1 package built:

deno


Improving UI of the Navbar.

The Navbar needs improvement to enhance UI/UX.
I would like to add better hovering effect and change spacing and layout in the Navbar.

assign to me


TestG0StackOverflow timeout on windows/amd64 in CI

Upstream issue for the same test having issues on windows/arm64:
https://github.com/golang/go/issues/63938

Lately in microsoft/main CI, a random set of Windows jobs is sometimes specifically bad at TestG0StackOverflow and some other runtime tests, and fails multiple retries within the same job:
https://dev.azure.com/dnceng-public/public/_build/results?buildId=466102&view=results

Attempt 1:
go tool dist: FAILED
skipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier errorskipped due to earlier error
=== Failed
=== FAIL: runtime:cpu124 TestG0StackOverflow (517.40s)
=== FAIL: runtime:cpu124 TestWERDialogue (0.08s)
=== FAIL: runtime:cpu124 TestCgoTracebackContext (unknown)
=== FAIL: runtime:cpu124 TestReturnAfterStackGrowInCallback (unknown)
=== FAIL: runtime:cpu124 TestCgoCallbackGC (unknown)
=== FAIL: runtime:cpu124 TestCgoExternalThreadPanic (unknown)
=== FAIL: runtime:cpu124 TestMemmoveOverlapLarge0x120000 (unknown)
=== FAIL: runtime:cpu124 TestConcurrentReadsAfterGrowth (unknown)
...

Then, job timeout.

Retrying the whole pipeline sometimes makes it succeed, sometimes not. (Might correspond to randomly getting a particular host?)
I'm not able to repro on my local machine.

not seen for a while


Show RegularExpressionAttribute pattern in hints

SwaggerWcfProperty.Pattern doesnt use regular expression for checking.
So when we need to check data by Regexp and want to avoid duplication of pattern in SwaggerWcfPropertyAttribute and RegularExpressionAttribute, just use RegularExpressionAttribute.

thanks, merged on #161
I changed the attribute to a custom one, to avoid adding dependencies to the project


Can't Install On Mac

Describe the bug
I can't install quick.db
To Reproduce
Steps to reproduce the behavior:
Try installing
Expected behavior
Gives me a load of node.js errors in the console
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Mac Catalina
Version: Newest

Did you follow the instructions in the README?


Short lived tokens

Also would be great to include the short-lived token related tests, like in https://github.com/dropbox/dropbox-sdk-python/pull/183/


DiscordAPIError: Missing Access

When running the bot in wsl ubuntu I get the error DiscordAPIError: Missing Access, I gave the bot administrator permission and the application.commands scope.
Start-up and error message:
[INFO] 12:06:05 ts-node-dev ver. 1.1.8 (using ts-node ver. 9.1.1, typescript ver. 4.5.5)
Robot Trip: Connected
DiscordAPIError: Missing Access
    at RequestHandler.execute (/home/bas/test/robot-trip/code/node_modules/discord.js/src/rest/RequestHandler.js:154:13)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at RequestHandler.push (/home/bas/test/robot-trip/code/node_modules/discord.js/src/rest/RequestHandler.js:39:14)
    at Message.crosspost (/home/bas/test/robot-trip/code/node_modules/discord.js/src/structures/Message.js:487:5)
[ERROR] 12:10:15 DiscordAPIError: Missing Access

package-lock.txt

Did you fill out a correct .venv file?

I tried 3 different .env configurations:
TOKEN='OTQ2MzE5OTgxMDMxODY2Mzg4.Yhc_KQ.Q-GkCMqFzcMV6gaLLHhBZKWyKXY'
CHANNEL_NEWS_ID=946321733428858892
CHANNEL_BINNENLAND_ID=946322028640759818
CHANNEL_BUITENLAND_ID=946322045711568997

TOKEN='OTQ2MzE5OTgxMDMxODY2Mzg4.Yhc_KQ.Q-GkCMqFzcMV6gaLLHhBZKWyKXY'
CHANNEL_NEWS_ID=946321733428858892

and
TOKEN=OTQ2MzE5OTgxMDMxODY2Mzg4.Yhc_KQ.Q-GkCMqFzcMV6gaLLHhBZKWyKXY
CHANNEL_NEWS_ID=946321733428858892
CHANNEL_BINNENLAND_ID=946322028640759818
CHANNEL_BUITENLAND_ID=946322045711568997
CHANNEL_SPORTS_ID=946322045711568997
CHANNEL_REGIONAAL_ID=946322045711568997
CHANNEL_POLITIEK_ID=946322045711568997
CHANNEL_ECONOMIE_ID=946322045711568997
CHANNEL_KONINGSHUIS_ID=946322045711568997
CHANNEL_TECH_ID=946322045711568997
CHANNEL_CULTUUR_ID=946322045711568997
CHANNEL_OPMERKELIJK_ID=946322045711568997

All of them result in the same error.

It's probably the elections channel. Let me know if 87a8d0ecb18c5b2b98130082fd07b11393572d4b fixed it.

Unfortunately, that didn't fix my problem, but I think I found what the problem is.
in BotManager.ts message.crosspost() is used which publishes a message in an announcement channel to all channels following it. But that means that the channel to which the bot is configured to send messages has to be a news channel.
I tried making that command conditional with an if statement like is example, but because typeguards are used, that if statement creates an error that says: the if statement wouldn't work because it will always be a "TextChannel".
But when I print the .type of that channel it returns "news" when it is a news channel even though a "TextChannel" typeguard is used.


libraryVC title don't modify when change album

YPPickerVC.swift
    func navBarTapped() {

        let vc = YPAlbumVC()
        vc.noVideos = !self.configuration.showsVideoInLibrary
        let navVC = UINavigationController(rootViewController: vc)

        vc.didSelectAlbum = { [weak self] album in
            self?.libraryVC?.setAlbum(album)
            self?.libraryVC?.title = album.title   //add this code to fixed issue
            self?.libraryVC?.refreshMediaRequest()
            self?.setTitleViewWithTitle(aTitle: album.title)
            self?.dismiss(animated: true, completion: nil)
        }
        present(navVC, animated: true, completion: nil)
    }

@boboxiaodd You mean the title property right ?
Can you confirm the displayed title is indeed updates on your side ?
Cheers,

@s4cha
first in library page, change Album,
then nav to camera  ,
and back to library , the navbar 's titlte restore to old
in func updateUI
https://github.com/Yummypets/YPImagePicker/blob/80f49bc8579f7b761a17be037bed31f494e96e83/Source/YPPickerVC.swift#L269

Ahhhhh!
My bad, I couldn't reproduce it at the time, I was missing the navigate and come back step :)
Your fix works as expected I am going to bump the version with it.
Nice one ;)

@boboxiaodd that should be fixed in 2.7.1
https://github.com/Yummypets/YPImagePicker/releases/tag/2.7.1
Thanks for taking the time to report this and finding the fix :)


Uploadable benchmarks

The page at /benchmark/visualize now allows for upload and visualization of a json file with benchmark values.
Still to do:

[x] Fix design time benchmarks

I just linked #37 and #43 to (under development) to the PR.


Remove some options from the org. admin list of 'features'

Related with https://github.com/CartoDB/cartodb/issues/4466
Remove private maps, private datasets and sync tables, since those things are inherited from the plan. I don't see why you shouldn't avoid people to have privacy or sync tables in a corporate environment. The simpler, the better.

cc/ @andrewxhill let me know why you would like to add this. More features, more mainteinance and more things to teach / explain

Since we've not added anything yet, let's discuss what should be added at #4466.

nope, i agree here, good closed


Cannot instantiate actor from imported module

It is not possible to instantiate actors from imported modules!
kll@Boxy:~/terastream/acton/compiler/test/actonc/regressions/import_actor$ ~/terastream/acton/dist/bin/actonc build
Compiling foo.act for release

******************** Compilation error
Module foo does not export 'Bar$newact'
kll@Boxy:~/terastream/acton/compiler/test/actonc/regressions/import_actor$

I committed this regression in #666, so simply try to build that for a reproduction

@nordlander is this similar to the not-possible-to-instantiate classes or different?
This one is not as critical for me, but if it's of similar character and easily fixed...!?

Hmm, where has the example gone? There's no test/actonc/regressions/ directory anymore...

test/regression_auto/import... I think

Problem fixed by PR #731. The test case still fails, though, seemingly because of a bad root argument in the test config. Leaving the issue open for that reason (for @plajjan  to fix!).

Test case now working after #737 fixes.


Rough SSH Bastion Host Support (relates to #2144)

Added ParamikoSSHBastionClient as a subclass of ParamikoSSHClient, with overridden connect and close methods for connection bouncing.
Use ParamikoSSHBastionClient if a ! exists in the host. The left side of the ! is the bastion host, right side is the target host.
All connection parameters for the target host are used for connecting to the bastion host. See https://github.com/lattwood/st2/blob/7cf7ba0938d23ddba29ce6c55d39ac6e49f7aaea/st2actions/st2actions/runners/ssh/paramiko_ssh_bastion.py#L43-L51

Rudimentary testing was done with https://gist.github.com/lattwood/4c52ba7b8583ae4565eb, I'll be doing more extensive testing using st2workroom today. This touches on #2144
This is the most python I've written so I apologize in advance if anything isn't done the python way.
Is a CLA required or would Apache licensing suffice?

I am taking a look at this PR but we should wait for @lakshmi-kannan and @Kami.

High level this is great. most of my comments are really cosmetic things.
Few thoughts -

Instead of overriding the hosts property perhaps a bastion_host property would make sense on the runner?
Perhaps a special runner make more sense and keep things explicit for users. This could be a consumption hinderance so maybe this is not such a good idea.

Also, perhaps a good idea to add some units -
https://github.com/StackStorm/st2/blob/master/st2actions/tests/unit/test_parallel_ssh.py

I was thinking about a bastion_host property, but my understanding is that it would require the modification of any actions using it to add the bastion_host(s) parameter. I would like to make it explicit, but that's a large drawback.
If there is a way to implicitly add the bastion_host(s) parameter to all actions supporting the remote-shell-cmd, how do you run a single command on mixed groups of hosts? (ie, how do you indicate hosts a,b,c are behind bastion1, d,e,f are behind bastion2, and g,h are directly connectable)

but my understanding is that it would require the modification of any actions using it to add the bastion_host(s) parameter.

Actually if we make bastion_host(s) an optional runner_parameter then all actions get it for free.

how do you run a single command on mixed groups of hosts?

This is a tricky one. Unless we come up with a host_group concept this will not work very nicely. For the near term I would suggest the for each execution we only allow 1 bastion host i.e. all hosts for that execution would be assumed to be behind the same bastion.
To run against multiple groups would require multiple executions. If practically it turns out to be the common case then we can look at options as the next step.

@manasdk much cleaner this way, just need to figure out unit tests

It's looking much better now in terms of code and tests.

Docs - Please add some info about bastion host in ssh docs. https://github.com/StackStorm/st2/blob/master/docs/source/troubleshooting/ssh.rst

Changelog - Add an entry to CHANGELOG with your name in the "ind evelopment " section. https://github.com/StackStorm/st2/blob/master/CHANGELOG.rst

One other thing I thought was if bastion host had a non standard SSH port, we'll need to add ability to handle that. But we'll make the person requiring the feature to add that ;).
Thanks for being patient. We'll merge this soon!

@lakshmi-kannan something something security something obscurity :)

LGTM. Merging now.

@lakshmi-kannan or @lattwood  I has started using stack storm recently. My environment is accessed by only one jump box. My stack storm is installed on a new box where i have passwordless connectivity to jump server. i have to use my jump server as bastion host and then trigger my scripts on target hosts. can you please post me some examples how to use or configure the bastion host details in stackstorm. i desperately need your help.


[ROCm] Adding a wrapper macro to declare dynamic shared memory

@chsigg
This is a follow-up PR for your request in PR #28451
It creates a macro (with CUDA/ROCm specific implementations) for declaring dynamic shared memory on GPUs.

@tatianashp @whchung

I took a look at the logs for the failing CI runs. The errors in the CI runs do not seem to be related to the changes in this PR


SVG sprites don't work in IE11

OL 3.8.2
The sprite (inline SVG image) generates exception in File: ol-debug.js, Line: 59315, Column: 11 when called from IE11, no problem sin Chrome.
Details:
The referenced line is "context.drawImage(image, originX, originY, width, height,..."
Looks like either sizes or origin aren't defined despite of the following lines in the code:
function getSVGIcon() {
var DOMURL = window.URL || window.webkitURL || window;
var img = new Image(20, 20);
var svg = new Blob([ICON_RECT90()], { type: 'image/svg+xml;charset=utf-8' });
img.src = DOMURL.createObjectURL(svg);
return img;
}
...
var iconStyle = new ol.style.Style({
image: new ol.style.Icon({
anchor: [10, 10],
origin: [0, 0],
anchorXUnits: 'pixels',
anchorYUnits: 'pixels',
rotateWithView: true,
img: getSVGIcon(),
imgSize: [20, 20],
})
});
iconFeature.setStyle(iconStyle);
MAP.getLayers().item(0).getSource().addFeatures([iconFeature]);

I think the problem is that IE11 can't tell the difference between "safe" and "unsafe" SVG code. Not sure if this can be solved with the Canvas renderer.
http://svgopen.org/2010/papers/62-From_SVG_to_Canvas_and_Back/#security_issues

I confirm, IE11 can't use SVG code.
OL3's Canvas renderer can render SVG if you encode it as base64 string. But you will have security error (only in IE) if you use interraction like foreachfeatureatpixel..

Exactly my exprience, have a working example with inline SVG icons here http://spatialillusions.com/milsymbol/examples/openlayers3/imageicons.html and it works fine in IE11. But if you try to add drag interactions you will get a security warning in IE, but it will work fine in Chrome.

@alexpir @DavidHequet I found how to fix this and sent a pull request, hopefully it will be merged to the master. If you want to get it working you just have to add one line and move one. (and make sure that your imageSize is set to two integers and not floats.)

I rebased the pull and created a new one #4164 . Hopefully it will be accepted, I have a bunch of developers of military systems waiting for this.

Thank you for the help. However I discovered that the problem is deeper than looked initially: firstly as correctly suggested there is cross-domain security issue in IE (the solution seems to be simply give up on limited interactivity as in Edge it works fine), secondly and more importantly there are some problems in drawing inline SVG onto canvas completely destroying the picture... what's bad that the second is eqally applicable to IE and FF.
I managed to reproduce the second issue not only with my site but also with http://spatialillusions.com/milsymbol/examples/openlayers3/imageicons.html  and tend to believe that the problem is generic. The trick to reproduce it to zoom in /shift area in the very begining (while data is still loading) so that all icons are outside of the viewport (OL3 seems to not load images for features outside of the viewport). Later when you scroll those icons into the viewport the the browser tries to acess images and failes (silently - there is nothing in console) the picture falls apart... looks terrible.
So, my solution was to give up in IE and FF, detect them at initial load (bad practice but stilll) and use pre-fenerated static PNGs for those browsers.,For the rest of browsers continue to use SVGs. Using PNGs I loose the ability to dynamically change the icon (my icon need to depend on "speed" parameter and should change the length of an arrow depending on it) but at least it is stable.
Would be nice if there is better way to solve the issue and stay with SVGs regardless the browser...

Can you create a jsfiddle to trigger this behavior? (Or modify my test case http://jsfiddle.net/spatialillusions/5jx0dzpu/ ) maybe I can try to isolate the issue then.
Sent from my iPhone

On 2 okt. 2015, at 15:08, alexpir<EMAIL_ADDRESS>wrote:
Thank you for the help. However I discovered that the problem is deeper than looked initially: firstly as correctly suggested there is cross-domain security issue in IE (the solution seems to be simply give up on limited interactivity as in Edge it works fine), secondly and more importantly there are some problems in drawing inline SVG onto canvas completely destroying the picture... what's bad that the second is eqally applicable to IE and FF.
I managed to reproduce the second issue not only with my site but also with http://spatialillusions.com/milsymbol/examples/openlayers3/imageicons.html and tend to believe that the problem is generic. The trick to reproduce it to zoom in /shift area in the very begining (while data is still loading) so that all icons are outside of the viewport (OL3 seems to not load images for features outside of the viewport). Later when you scroll those icons into the viewport the the browser tries to acess images and failes (silently - there is nothing in console) the picture falls apart... looks terrible.
So, my solution was to give up in IE and FF, detect them at initial load (bad practice but stilll) and use pre-fenerated static PNGs for those browsers.,For the rest of browsers continue to use SVGs. Using PNGs I loose the ability to dynamically change the icon (my icon need to depend on "speed" parameter and should change the length of an arrow depending on it) but at least it is stable.
Would be nice if there is better way to solve the issue and stay with SVGs regardless the browser...
—
Reply to this email directly or view it on GitHub.

I changed a Math.ceil to a Math.floor and got everything working in Firefox, don't have a IE computer to test with at home. Here is an updated example with drag interactions on all icons. Works without any problems in Firefox for me. http://spatialillusions.com/milsymbol/examples/openlayers3/imageicons_with_interactions.html

I, i tried with IE 11 at works, it don't event load the map, error here:
ol.render.canvas.Replay.prototype.replay_ = function(
....
context.drawImage(image, originX, originY, width, height,
x, y, width * pixelRatio, height * pixelRatio);
The error in french is "Demande d'accès à la méthode ou aux propriétés inattendue" , a think it's
"Unexpected call to method or property access."

Make sure to round down your svg's size to whole pixels:
imgSize: [Math.floor(mysymbol.width), Math.floor(mysymbol.height)]
I had the same problem as you, but this solved it. Then the question is if it should be up to the user to round the pixels or if openlayers should do it.

I just opened youre sample ( http://spatialillusions.com/milsymbol/examples/openlayers3/imageicons_with_interactions.html ) with IE, didn't try anithing else

Found the cause of this, will update the code later today. It's because of my use of style functions, and you can get around that, but I will have to investigate further if it can be fixed in openlayers. (Send me an email if you want direct contact about this, you'll find my email on my homepage.)

Now should the code be reverted back to a version that works in IE11, there is still some trouble with style functions, but I'll see if I can solve this. If you want a version using my patched open layers so you can use interactions in IE11 let me know, otherwise just wait for ol 3.10.

I spent some time looking into this and my best guess at the moment of what is happening is that since a stylefunction is updated every time you interact with the map we will try to draw icons to the canvas before they have been generated, and then IE will throw the error because the image does not contain any data.
https://msdn.microsoft.com/en-us/library/ff975414(v=vs.85).aspx
If we simply slow down the draw by placing the drawimage in a try catch and print the error to the console in the catch, everything will work. But I would say that you shouldn't use inline generated icons and stylefunctions together. Apply the style to the features individualy and update them when nessesary.

@spatialillusions Can I assume that this issue is related to the military icons library you are using? Should we partially revert #4164, i.e. move the drawImage call outside the try/catch block again?

@ahocevar No keep #4164, it still solves the tainting of the canvas issue. This drawImage error is ol.render.canvas.Replay.prototype.replay_ and only happens when you are using style functions, since the style function will try to render new SVG every time you interact with the map. (Even on pan) And the error is triggered if you are using interactions or not, but not for all icons, just when open layers is trying to draw them faster than they can be created, or so it seams like.
I don't remember exactly at the moment, but I think the error I got in the drawImage in my pull was another one, and it occurred for every Icon. So by keeping the drawImage inside the try block we should be on the safe side. The errors produced by IE11 is however a bit diffuse and makes it hard to figure out exactly what the issue is. But I think in this case it is more of a security issue.
To sum everything up.

#4164 solves the tainting of canvas issue when using SVG icons with interactions.
If you are using complex inline SVG you should not use style functions, and instead attach the style to the features when they are initiated.

•If you are using complex inline SVG you should not use style functions, and instead attach the style to the features when they are initiated.

This is where I think you're doing something wrong. Because the style function does not make a difference here. Your inline SVG images should not have a status of 'loaded' initially when you deliver them as data uri. So set them as url in the icon style, not as image. Because the latter assumes that the image is loaded already.

@ahocevar I'll look into this and get back to you if I manage to solve it in another way. I managed to use the src and size, but did run into a lot of other IE problems, but I think I might be able to trace it.

@ahocevar Hi, so I spent a bunch of hours trying to figure this out, and I thought that I would report my findings.
I found a lot of strange things, like that the ol.render.canvas.Replay.prototype.replay_ is fired several times if I use a style function, and the image the drawImage with is different; not different like background map and icon, but the icon in my case was triggered several times with different values, like the standard pushpin icon from KML, my source is not KML it's GeoJSON, so strange things are happening. I also think it tired to draw the point without an icon at all, and drawImage in IE 11 will throw an error if you try to draw an image with the size [0,0] to the canvas, and this is the error we have been seeing. So, something fishy is going on there.
But the strangest part was that I did manage to draw the initial set of icons to the canvas without triggering the ol.render.canvas.Replay.prototype.replay_ and then everything worked, like the initial map worked, but as soon ol.render.canvas.Replay.prototype.replay_ was triggered things started to go wrong.
And then I tried using src and size instead of image and imgSize, but then I found that size is not passed on down to the image object in the same way as imgSize is (in one place in the code imgSize is renamed to size and this is confusing); and if you try to set imgSize when you have set src you will get an error since you arn't supposed to do that. And then you will run into trouble since IE11 has problems with finding the size of the actually loaded image if the image source isn't a url but instead a base64 encoded string.
But the strangest thing is that it works on drawing the icons before you start to interact with the map, how this is possible I still haven't been able to figure out, since it really shouldn't, since I never was able to see an icon with the correct image size assigned by the load image function.
Most of this could be solved by introducing a bunch of special solutions for solving this IE problems, like passing on the size down to the image object, but I started to see how messy my code started to look. And when I finally got it working I ended up with something where I was able to use src and size to get my map displayed with my icons, and everything looked fine until i started to interact with the map when the icons started to strobe flash all the time, since because of something the style function was triggered ALL the time, and then the map became quite useless.
So the conclusion is that we could get a better and more general solution for IE if we did a lot of recoding, introducing stuff that reduce performance in all browser except old versions of IE, and there still is a bunch of loose ends to find and figure out. Or we could say that style functions and inline generated image icons don't play well together, use the workaround that is in place in 3.10 and everything will work. (And unfortunately I'm not able to spend much more time on this, since I just had to find a solution to a specific use case, and that use case has now been solved.)
Best regards
Mån

Thanks for reporting your findings @spatialillusions. Closing this as resolved then.

Very interesting topic. I tested with version v3.10 (checked that above patch is included) using Style/Icon with URI src and integer size. SVG icons display correctly in IE11 as soon as size is specified, but forEachFeatureAtPixel does not work. No error messages in IE11, it just doesn't "see" the feature.
Did I miss something in the comments - or should I open a new bug report?

@kaij Like mentioned above, src and size will end up in some problems with the current implementation since the "image" isn't loaded from anywhere. Have a look at my jsfiddle above if you want to get it working. Before the patch interactions didn't work even if you used the workaround.
I tried to see if it would be possible to get src and size to work, but that will require a big code rewrite and I don't know if it is worth the trouble if you can have a working workaround.

Thanks for the quick response! I also tried it that way, but get a "SecurityError" on every mousemove. Not just in my own code, but already at http://spatialillusions.com/milsymbol/examples/openlayers3/imageicons_with_interactions.html or http://jsfiddle.net/spatialillusions/5jx0dzpu/. Are these the right examples (which should work in IE11)?

@spatialillusions I checked again to match exactly your jsfiddle with OL 3.10. I think there was something wrong with my data uris before - it works now fine in IE11! Thanks!


There is no create/add Collaborator

I think we are missing a method, add/create Collaborator. let me know, if I am missing something, or I can create the PR to add this.
http://github-tools.github.io/github/docs/3.1.0/Repository.html

See #462


Documentation is not up-to-date (lacking): Deployment of non-production Amundsen on AWS ECS using aws-cli

Hi guys
Step #3 from https://www.amundsen.io/amundsen/installation-aws-ecs/aws-ecs-deployment/ is not valid.
$ ecs-cli up --keypair JoaoCorreia --extra-user-data userData.sh --capability-iam --size 1 --instance-type t2.large --cluster-config amundsen --verbose --force --aws-profile $AWS_PROFILE
>>> FATA[0002] Error executing 'up': open userData.sh: no such file or directory

What to do with it?
I've seen some comment: in ~//amundsenfrontendlibrary/docs/instalation-aws-ecs but there is no such dir instalation-aws-ecs
The same issue to step #4 from https://www.amundsen.io/amundsen/installation-aws-ecs/aws-ecs-deployment/
ecs-cli compose --cluster-config amundsen --file docker-ecs-amundsen.yml up --create-log-groups
No such file. I searched over two repos https://github.com/amundsen-io/amundsen and https://github.com/amundsen-io/amundsenfrontendlibrary
I haven't found such files. What am I missing?

While walking thought the issues I found a proper location of the needed files: https://github.com/amundsen-io/amundsen/tree/master/docs/installation-aws-ecs

While walking thought the issues I found a proper location of the needed files: https://github.com/amundsen-io/amundsen/tree/master/docs/installation-aws-ecs

pr is merged.

pr is merged.


There is an error when clicking the copy related actions on Linux

Storage Explorer Version: 1.19.0-dev
Build Number: 20210330.6
Branch: main
Platform/OS: CentOS 7.6.1810 (Core)
Architecture: ia32/x64
Regression From: Previous release(1.18.1)
Steps to Reproduce

Create a CentOS 7.6.1810 (Core) Linux OS -> Install and launch Storage Explorer.
Select one storage account -> Right click it -> Click 'Copy Primary Key' action.
Check there is no error.

Expected Experience
There is no error.
Actual Experience
There is an error.

Additional Context

This issue reproduces for all copy related actions.
Storage Explorer crashes when performing the copy related actions on Linux Ubuntu 16.04/18.04.

@v-limu I can reproduce the long hang after hitting Copy related actions. Is that what you mean by crash? As for the dependency issue, it shouldn't happen since it is one of the core C++ libraries that native modules need. There is nothing we can do if it is missing.

Hi @JasonYeMSFT  Yes. I hit Copy related actions on Linux Ubuntu 16.04/18.04. The result is the long hang and can't work.
On the same CentOS 7.6.1810 (Core) Linux OS. We verify this issue in the release 1.18.1. No error occurs and everything works well.

Could you try manually adding the path to the lib and see if that workarounds the problem? https://unix.stackexchange.com/a/532121


Associate "firestore.rules" filename with firebase icon

Changes proposed:

[x] Add

Can you please post a link so we can get more info on the filename? I suspect that this may need its own icon.

Cloud Firestore (Firestore) is a NoSQL database under the Firebase family of products.
https://firebase.google.com/products/firestore/
firestore.rules are the security rules for Firestore database access.
I agree with you and have made a Firestore-specific icon based on the existing firebase.svg icon.
It retains the Firebase root but includes the child Firestore logo in the corner like the TS or JS on Angular icons.
Hope this is more useful.

Thank you for doing that. I will be more observant next time.

Thank you! And thank you for a great plugin :)


feat: add avatar in chat header

Avatar fallback works ok. I think you can include the actual avatar image with very little modification. The profile picture sits inside the profile.image property of each contact. You can use the AvatarImage component like so to display the b64 image.
  const contactName = selectedContact?.localDisplayName;
  const contactAvatar = selectedContact?.profile.image;

  return (

          <Avatar className="h-8 w-8 shrink-0">
            <AvatarImage
              src={contactAvatar}
              alt={selectedContact?.localDisplayName}
            />
            <AvatarFallback>
              {contactName ? contactName.charAt(0).toUpperCase() : "D"}
            </AvatarFallback>
          </Avatar>
          ...
  );

Additionally, I would throw the Avatar component from both Sidebar and header into a custom component that takes in the contact data as a prop. This way its consistent throughout and you only need to change it once.

@DaniloArantesF, you're right. That makes much sense for us.


Fix RuntimeContext cannot store anything when the trace is not creat

Please answer these questions before submitting pull request

Why submit this pull request?
[x] Bug fix
[ ] New feature provided
[ ] Improve performance

Coverage increased (+0.009%) to 24.603% when pulling cc0bae203a7573069cb97ec0de85f61bd8124641 on ascrutae:feature/runtime-context into 1c3606a8328a795a1b4cf06db6f4cd95bb5f9a49 on apache:master.

@wu-sheng  Done!


ResourceOwnerPasswordValidator in AccountController

We currently have a custom database for user storage and we are moving to implement IdentityServer4 as soon as it is ready!  I have a project up and running and it works great using Postman to test etc.  I grabbed the UI from one of the Quickstarts but its using the InMemoryUserLoginService etc in the AccountController.  Is there an example of an AccountController that uses RecourceOwnerPasswordValidator? Is that even what I should be looking for? So far so good from the server project though... keep up the good work and sorry if this is a simple one .. I am definately a newbie!

There's a different interface for that flow: https://identityserver4.readthedocs.io/en/dev/topics/resource_owner.html

Thanks for the quick reply.. yes.. I have already implemented that and have it set up in startup.cs
services.AddTransient<IResourceOwnerPasswordValidator, Configurations.ResourceOwnerPasswordValidator>();
services.AddTransient<IProfileService, Configurations.ProfileService>();
This works great from Postman but not in the UI I grabbed from one of the quickstarts you provided.
The AccountController does not like InMemoryUserLoginService... so I "think" I need to use the ResourceOwnerPasswordValidator from there too.. maybe I'm confused.. well wait.. yes.. I am confused!
Is this the correct approach from within my AccountController?

Ah, ok. Now I understand. In the UI you'd just rip out the InMemoryUserLoginService and do your "own thing" (presumably check the same backend user store your IResourceOwnerPasswordValidator is using).


Update test-tokens.mdx

Description
Replace EOA address with EOA to make it more aligned with rest of documentation.
Changes

Replace EOA address -> EOA

Thank you for the fix!


Overflowing bitshift on architectures where long is shorter than index_t

The 1l in the definition of max_simplex_index should probably be (index_t)1, otherwise the code fails on a bunch of platforms where 1l is 32 bit.

Thank you! It is fixed now.


COLOR_PAIRS is bigger than max short

this is causing dvtm to hang on startup
changes some types from short to int, but it might make more sense to clamp COLOR_PAIRS down to some maximum, maybe output an error if it's too big.

this fixes #83 and #84 for me, can anyone else confirm?

I believe this also supersedes #82

Thank you, this fixes #83 and #84 for me.

cc @martanne
Please review; if there's any changes you'd like, let me know.
This fixes #83 which makes dvtm unusable, and #84.

Thanks for looking into this and sorry for the unresponsiveness. I hope to have a bit more time for dvtm in the future.


Bot doesn't leave channel

I believe the bot should automatically leave the voice channel after a period of inactivity. However, it currently remains connected indefinitely.

fixed in f79e9d39d9abfc90cc234f0c85ef9b978b856e3c


[beta] Backport cargo-util version fix

Backport of #9745 to fix the version for cargo-util.  This should help if someone does cargo update -p cargo to ensure they get the correct version of cargo-util.

I don't think this justifies publishing a new 0.55.1 on its own, but if someone feels it is important, it shouldn't be too hard to prepare.

r? @alexcrichton
(rust-highfive has picked a reviewer for you, use r? to override)

:warning: Warning :warning:

Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.55.0. Please double check that you specified the right target!

@bors: r+

:pushpin: Commit 002f43d9bdb385e0226c596eff8f0c93494e3fbe has been approved by alexcrichton

:hourglass: Testing commit 002f43d9bdb385e0226c596eff8f0c93494e3fbe with merge 1f76a218bc7f326606dda811b58c42b7e1e21168...

:sunny: Test successful - checks-actions
Approved by: alexcrichton
Pushing 1f76a218bc7f326606dda811b58c42b7e1e21168 to rust-1.55.0...


Pregenerate some aspects of progress reports

Pregenerate workdone.md snippets based off of events, + kids events that are put up and taken down automatically. Email these to a specified email

Add settings to days of the week, be able to adjust send time as well (defaults to M/T/F @ 2:45pm)
Should encompass itself and last days not worked (M=S/U/M, T=T, F=W/R/F)

No longer relevant


Working Inconsistently, is there a repair?

Z suddenly appears to not be working properly for me, not finding directories I have been to a hundred times. Is there a way to rebuild, repair or reset?

removing ~/.z solves. +1 for an option in tho.
OPTIONS
       -c     restrict matches to subdirectories of the current directory
+
+      -d     destroy learned frecency
+
       -e     echo the best match, don't cd


Updated material ui to version 5

Changes

Updated material ui to latest major version 5

Is anyone merging this? I am also in need of MUI v5. @zachawilson If nobody merge this, do you mind setting up your branch on public npm?
Don't know if this repo is maintained anymore?

Hello good.
Are you contemplating publishing these changes to NPM?


Windows AD authentication using Username, UPN, or NetBIOS name

Question
I have successfully setup AD auth using a person's username (e.g. marty.mcfly), but how could I auth against their UPN<EMAIL_ADDRESS>and/or NetBIOS name (netbiosname\marty.mcfly) as well? This is for both Pode 1.8.4 and Pode 2.0 that I've experienced this where the username will work, but the upn or domain\username format will give me a Pode 500 error. To make things simple we can stick with Pode 2.0 that I'm using:
New-PodeAuthScheme -Form | Add-PodeAuthWindowsAd -Name 'Login' -Fqdn 'domain.local' -Groups "app_group" -SuccessUrl '/' -FailureUrl '/login' #I've replaced domain.local with my domain
Add-PodeRoute -Method Get -Path '/login' -Authentication 'Login' -login -ScriptBlock {
    Write-PodeViewResponse -Path 'auth-login' -FlashMessages
}
Add-PodeRoute -Method Post -Path '/login' -Authentication 'Login' -login

I'm using the html PatternFly Login Component for the login. If the only auth variable allowed is the username (and password), would I then need to parse the username field in the Add-PodeRoute -Method Post somehow? Thanks in advance!

Hey @ericpritchett,
This is definitely something Pode should be handling, and I can get that fixed in a 2.0.1 - using parsing on \ and @.
As for how to get around it at the moment, it should be possible in both 1.8.4 and 2.0.0; both requires setting up a global middleware to parse the username first. If you want to limit either to just /login then supply -Route '/login' onto either.
1.8.4
Add-PodeMiddleware -Name 'ParseFormUsername' -ScriptBlock {
    param($e)

    if ($e.Data.Username) {
        $e.Data.Username = (($e.Data.Username -split '@')[0] -split '\\')[-1]
    }

    return $true
}

2.0.0
Add-PodeMiddleware -Name 'ParseFormUsername' -ScriptBlock {
    if ($WebEvent.Data.Username) {
        $WebEvent.Data.Username = (($WebEvent.Data.Username -split '@')[0] -split '\\')[-1]
    }

    return $true
}


Can i send sms from my google voice using app-notify

Not at this time. If someone wants to submit a PR I'll consider adding it.


Having some problems following the instructions to get the receiver working.

I have been following the instructions in the README.md file -
I got to point 5 -
Using the Chromecast setup application, make sure [x] send your serial number to Google is checked. This is the only way that you can access your unpublished receiver
I can't find that option on the Google Home app - but equally, I'm not sure what the [x] signifies - does that mean that this is not longer required?
I carried on with the instructions.
Step 7 says :
Enter the App ID of your receiver application into your sender application or one of our sample sender applications listed above.
there are no sample sender applications listed above.  Clicking on the link to https://developers.google.com/cast/docs/sample-apps (am I missing something?)

Thanks for the report. The README has been updated to be hopefully more clear and up to date.
As for the sample sender apps, all of the samples listed that do not say "receiver" are samples that can be used for testing:
https://developers.google.com/cast/docs/sample_apps#reference_apps


Draw doesn't work on timeseries figure

I think this is similar to #161 which was closed by #162. The error experienced is the same as #161 where after drawing icons the tool doesn't reset and the returned object doesn't include the selected points.

Reproduce with:
folium                    0.15.1
streamlit                 1.29.0
streamlit-folium          0.17.4

and
import folium
from streamlit_folium import st_folium
from folium.plugins import Draw
import streamlit as st

m = folium.Map(location=[35.68159659061569,<PHONE_NUMBER>6151428], zoom_start=16)

# Lon, Lat order.
lines = [
    {
        "coordinates": [
            [139.76451516151428, 35.68159659061569],
            [139.75964426994324, 35.682590062684206],
        ],
        "dates": ["2017-06-02T00:00:00", "2017-06-02T00:10:00"],
        "color": "red",
    },
]

features = [
    {
        "type": "Feature",
        "geometry": {
            "type": "LineString",
            "coordinates": line["coordinates"],
        },
        "properties": {
            "times": line["dates"],
        },
    }
    for line in lines
]

folium.plugins.TimestampedGeoJson(
    {
        "type": "FeatureCollection",
        "features": features,
    },
    period="PT1M",
    add_last_point=True,
).add_to(m)
Draw(export=True).add_to(m)
m.save('draw.html')
c1, c2 = st.columns(2)
with c1:
    output = st_folium(m, width=700, height=500)

with c2:
    st.write(output)

Note all we get expected behaviour with:
folium                    0.14.0
streamlit                 1.29.0
streamlit-folium          0.14.0

Note this behaviour still exists with
folium==0.15.1
streamlit==1.31.1
streamlit-folium==0.18.0

in Python 3.12.2


Fail to install behind a corporate proxy.

Behind the a corporate proxy the npm install of web-component-tester fail.
The cause is wtc-saucer installation fail in post-installtion script with a direct http.request cf https://github.com/Polymer/wct-sauce/issues/5#issuecomment-78175279.
This dependency is needed as mandatory ? could be an optional dependency ?
cf the error installation
user@host:/tmp$ npm install web-component-tester

><EMAIL_ADDRESS>install /tmp/node_modules/web-component-tester/node_modules/socket.io/node_modules/engine.io/node_modules/ws
> (node-gyp rebuild 2> builderror.log) || (exit 0)

make: Entering directory `/tmp/node_modules/web-component-tester/node_modules/socket.io/node_modules/engine.io/node_modules/ws/build'
  CXX(target) Release/obj.target/bufferutil/src/bufferutil.o
  SOLINK_MODULE(target) Release/obj.target/bufferutil.node
  SOLINK_MODULE(target) Release/obj.target/bufferutil.node: Finished
  COPY Release/bufferutil.node
  CXX(target) Release/obj.target/validation/src/validation.o
  SOLINK_MODULE(target) Release/obj.target/validation.node
  SOLINK_MODULE(target) Release/obj.target/validation.node: Finished
  COPY Release/validation.node
make: Leaving directory `/tmp/node_modules/web-component-tester/node_modules/socket.io/node_modules/engine.io/node_modules/ws/build'

><EMAIL_ADDRESS>postinstall /tmp/node_modules/web-component-tester/node_modules/wct-sauce/node_modules/sauce-connect-launcher
> node scripts/install.js

><EMAIL_ADDRESS>install /tmp/node_modules/web-component-tester/node_modules/socket.io/node_modules/socket.io-client/node_modules/engine.io-client/node_modules/ws
> (node-gyp rebuild 2> builderror.log) || (exit 0)

make: Entering directory `/tmp/node_modules/web-component-tester/node_modules/socket.io/node_modules/socket.io-client/node_modules/engine.io-client/node_modules/ws/build'
  CXX(target) Release/obj.target/bufferutil/src/bufferutil.o
  SOLINK_MODULE(target) Release/obj.target/bufferutil.node
  SOLINK_MODULE(target) Release/obj.target/bufferutil.node: Finished
  COPY Release/bufferutil.node
  CXX(target) Release/obj.target/validation/src/validation.o
  SOLINK_MODULE(target) Release/obj.target/validation.node
  SOLINK_MODULE(target) Release/obj.target/validation.node: Finished
  COPY Release/validation.node
make: Leaving directory `/tmp/node_modules/web-component-tester/node_modules/socket.io/node_modules/socket.io-client/node_modules/engine.io-client/node_modules/ws/build'

><EMAIL_ADDRESS>postinstall /tmp/node_modules/web-component-tester/node_modules/wct-local
> node scripts/postinstall.js

----------
selenium-standalone installation starting
----------

---
selenium install:
from: http://selenium-release.storage.googleapis.com/2.45/selenium-server-standalone-2.45.0.jar
to: /tmp/node_modules/web-component-tester/node_modules/wct-local/node_modules/selenium-standalone/.selenium/selenium-server/2.45.0-server.jar

---
chrome install:
from: http://chromedriver.storage.googleapis.com/2.14/chromedriver_linux64.zip
to: /tmp/node_modules/web-component-tester/node_modules/wct-local/node_modules/selenium-standalone/.selenium/chromedriver/2.14-x64-chromedriver

-----
selenium-standalone installation finished
-----

><EMAIL_ADDRESS>postinstall /tmp/node_modules/web-component-tester/node_modules/wct-sauce
> node scripts/postinstall.js

Prefetching the Sauce Connect binary.
Missing Sauce Connect local proxy, downloading dependency
This will only happen once.

events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: connect ETIMEDOUT
    at errnoException (net.js:905:11)
    at Object.afterConnect [as oncomplete] (net.js:896:19)

npm ERR<EMAIL_ADDRESS>postinstall: `node scripts/postinstall.js`
npm ERR! Exit status 8
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is most likely a problem with the wct-sauce package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node scripts/postinstall.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls wct-sauce
npm ERR! There is likely additional logging output above.
npm ERR! System Linux 3.13.0-46-generic
npm ERR! command "/home/a000cqp/opt/nodejs/bin/node" "/home/a000cqp/opt/nodejs/bin/npm" "install" "web-component-tester"
npm ERR! cwd /tmp
npm ERR! node -v v0.10.36
npm ERR! npm -v 1.4.28
npm ERR! code ELIFECYCLE
npm ERR! not ok code 0

I'd definitely like to make wct-sauce an optional dependency for WCT 3.0 - but don't want to drop it just yet for fear of breaking existing users

But you block some new users to use it :

All Users that work in big company that could not use it if they are behind a corporate firewall
And in some company that have  Business Continuity Plan (BCP) constraint, and in which one that the development environment should installable even if a no internet connectivity.

Do you have a plan for the release of the WCT 3.0 ?

Soon!
wct-local and wct-sauce are now optional dependencies on master

The new user, thank you !!!

Install still fails for my team behind the corp proxy. I see that act-sauce is an optional deep but I still get error: "Could not find WCT plugin named "sauce""

+1

+1

I don't know exactly why but when I initialize an element using polymer init from my Polymer Starter Kit clone i get error: "Could not find WCT plugin named "sauce""
But when I initialize it outside everything is going well.
I guess it's because the polymer init script detect a bower installation and assume that all packages are already installed, sauce in particular .

+1

+1

+1

👍

+1

+1


getModNotes fails if the user has any mod notes that do not contain a link

When adding a mod note, mods have the option to not include a link to the contextual post or comment:

However if you then try and retrieve mod notes for the user this crashes. Example code:
const notes = await context.reddit.getModNotes({
    subreddit: "fsvsandbox",
    user: "fsv",
}).all();

The error is Error: Expected thing id to start with t1_, t2_, t3_, t4_, t5_, t6_ got }.
In the UserNote type, redditId is defined as redditId: T1ID | T3ID | T5ID; which means that if there is no linked post/comment/subreddit the object cannot be built.

Duplicate of #33


fix build

Description
Side-effect of linked PR, need to upgrade cwlavro to avoid nightly cron builds from failing
Review Instructions
Check and see if the nightly CLI cron build passes
Issue
https://github.com/dockstore/dockstore/pull/5564
Security
If there are any concerns that require extra attention from the security team, highlight them here.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic style checks and unit tests by running ./mvnw clean install
[x] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

Snyk warning seems like a false positive, where we haven't upgraded to junit 5, we're using 4.13.2 (past the recommended version)


Clicking on the create new project button results in the application taking sometimes ~20 seconds before it loads.

@overZellis133 is there any way you can describe the conditions to make this bug repeatable. I assume it goes away when you do localStorage.clear() in the console ?

I wonder if using Chrome devtools would help us debug @Laurian ? https://developers.google.com/web/tools/chrome-devtools/storage/localstorage

afaik this was fixed


If a task with few categories, such as 3 or 4，will the clustering performance be good?

A task with few categories, such as 3 or 4, whether the model will work ?

In the code, what is difference between pair_dataset and class_dataset?

Hello,
1, I cannnot be sure whether this work is applicable on few-class scenarios.
2, there is no big different two dataset, the class-dataset is with a balanced-sampling.

Thank you!


Property 'env' does not exist on type 'typeof AdminJS'

Typing issue with env property on the AdminJS object when referencing it in a react TSX file.

Frontend envs should be available via window.AdminJS.env.<env property>

@dziraf I just tried that and typescript is throwing  this error Property 'NODE_ENV' does not exist on type 'Number', I don't get any errors when I use process.env so it leads me to believe there  is a typing issue with window.AdminsJS.env.

@dziraf  why isenv type coming up as a Number?


Preview Message

When I enable this control for activities, when I go to use the new control I see a message that is a preview feature and I must sign-in to enable preview features. But there is no explanation in this documentation as to where/how that is done.

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

ID: 12b353db-27dd-23ca-60a9-22a9007d94c3
Version Independent ID: b4e77474-01be-f5a2-54a5-1f2a16324137
Content: Add the Kanban control to Opportunity or Activity entities (Dynamics 365 Sales)
Content Source: ce/sales-enterprise/add-kanban-control.md
Service: dynamics-365-sales
GitHub Login: @shubhadaj
Microsoft Alias: shujoshi

@joholmes123 : Thank you for the feedback! Can you confirm whether you're still seeing this issue?

Just tested it last week, it is no longer giving me that message and I can configure the control for both activities and opportunities. Thank you!

Closing this issue as the problem doesn't persist.


Disable animations for BasicPlatform (offcreen)

Closes #9749
Closes #9750
Ref https://github.com/chartjs/Chart.js/pull/9750#issuecomment-940203994
Also enables any other changes to config that a platform would require.

This change breaks animation generation on nodejs.


[Feature Request] configure time for email polling

not an issue, but would be a good feature to implement to save on machine resources
Describe the solution you'd like
Would like to see a way to configure a time range for the polling of the email box that way there is no unnecessary polls around the clock.

Problem is shippers do not always send their emails "from 9 to 5".
Is there a problem with it checking your email at regular intervals? You could always increase the interval as well.

no issue, was just noticing it updating the time all day and thought it was unnecessary

Based on exploring multiple components in Home Assistant, it doesn't seem like there's a way to implement this. All components poll at XX frequency. Feel free to adjust your polling to longer times if you like.


flutter_localizations dependency conflict with intl v0.16.1

Because my_app depends on flutter_localizations any from sdk which depends on intl 0.16.0, intl 0.16.0 is required.

Hi @brizaldi
see https://github.com/flutter/flutter/issues/50782#issuecomment-586138656
Update is in current master, dev and beta channels.
Closing as fixed and duplicate,
if you disagree please write in the comments and I will reopen it
Thank you


this page is missing settings for provider = apple and OpenID Connect

this page is missing settings for provider = apple and OpenID Connect

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

ID: e194e752-fc11-446c-e043-31311238edca
Version Independent ID: 52455ec5-3a4c-f256-05db-dec0dac6c9eb
Content: OAuth tokens in AuthN/AuthZ - Azure App Service
Content Source: articles/app-service/configure-authentication-oauth-tokens.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

Internal item created: https://dev.azure.com/msft-skilling/Content/_workitems/edit/288625
#please-close


fix: 🐛 changing to correct font size for p tags

👋 Thanks for creating a pull request! 🚀 Checkout the storybook we've created for it: chlorophyll


Add tests for discovery ACL functionality

This PR secures the consul test instance with a default ACL policy of 'deny', and then runs a functional service registration test with a given ACL.
For cleanliness, the secured consul instance should allow registrations for the other tests without an ACL. Other tests do currently pass, so this isn't necessarily required given the current test design.

Suspect build is failing due to #83
The build worked on my fork in travis after commenting out the ./docs/src/main/asciidoc/ghpages.sh command

Failing due to #84
A script in the PR is removed by ghpages.sh

I haven't forgot about this. I need to make it so these tests only run on the travis build, not on a local build.

I think the tests will still work on a local build providing you haven't hardened your local consul instance with different acl keys. The test basically makes sure the tokens works against a hardened consul instance.

Thats the thing, I don't want to run it with acls.

Squashed, merged with a bit of polish via 12af0f532e1b60c51bf3e6e64fc0daacffa197e6

@bpgray this is merged, but the config tests fail now, so for now they are disabled.


Added a Love Module

Short description of what this resolves:
Addresses Issue: #341
This module detects if the user query has a 'love' intent and
returns an appropriate response which is either a response to
the user's love or a definition of
Example output:
Input: "Do you love me?"
Output: {"text": "I truly love you more than I have ever loved anyone."}
Input: "What is love?"
Output: {"text": "Love is patient, love is kind. It does not envy, and it does not boast."}
Changes proposed in this pull request:

Added a local JSON file for love responses
Added a love module test
Checks if the user query has love intent.

Fixes: #

@swapagarwal


[HOLD for payment 2024-02-20] [$500] 0 member is formatted weirdly when leaving a room and revisiting room details

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

Version Number: 1.4.33-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @blimpich
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1706559325923909
Action Performed:

create a workspace
create a room with workspace visibility
open the details pane and copy the link in the url to your clipboard
leave the room
paste the link you copied into the URL

Expected Result:
We should see the members menu item with a correctly formatted zero representing zero members
Actual Result:
The zero is there but its formatted really strangely
Workaround:
unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/decb658a-8ce3-4605-9eca-98b83e54b214

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~018691b573ff8ea80e
Upwork Job ID:<PHONE_NUMBER>965654528
Last Price Increase: 2024-01-30
Automatic offers:

dukenv0307 | Contributor | 28137708
shubham1206agra | Contributor | 28137709

Hey 👋🏻
new to contributing to Expensify, is there any way I can take this issue?

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~015f3056aaa4d9252e

new to contributing to Expensify, is there any way I can take this issue?

@jyash97 welcome! We ask that you provide proposals for the issue, in a similar format to the one here. Then the C+ will review to see which proposals will best fit the issue 👍

@shubham1206agra any thoughts on the proposal here?

@dukenv0307's proposal looks good.
🎀👀🎀 C+ reviewed

I messed up assignment 😭
hope it didn't break anything...

@hayata-suenaga no worries!  All good!
So to confirm, @dukenv0307 has been hired as the contributor and @shubham1206agra has been hired as the C+? Just want to make sure I have that correct. Thanks!

yes that's right! thank you!

Payment is scheduled for next week! 🎉

@shubham1206agra it looks like we're still waiting for you to accept the proposal in Upwork here — can you let me know once you accept!
@dukenv0307 you have been paid via Upwork, so you are all set!

Perfect! Payment has been made! Once you have completed the checklist @shubham1206agra — we can get this closed. Thank you!

@shubham1206agra any chance you can have the checklist completed by EOD today?

Perfect! Thanks so much!


chore(deps) bump lua-resty-http from 0.15 to 0.16.1

Summary
Upgrades lua-resty-http from 0.15 to 0.16.1 and removes the deprecated ways of using it.

Don't like so many ssl_verify = false though, we should fix those

I'll create a backlog card for that.


Feature: Add Master Master replication

Thank you for your contribution.

I'll pull a new request with my new code


Add type='button' to clear button to avoid form submit

Clicking a <button> inside a <form> will submit the form by default. This means when using the datepicker inside an HTML form, the clear button will also submit the form...
This PR adds type='button' to the clear button to override the default behaviour to submit the form (see https://developer.mozilla.org/en-US/docs/Web/HTML/Element/button#attr-type).

@krystalcampioni hey,
I know this is an old issue, however I'm encountering it in v2.6.0. I have the datepickers inside a form element and it keeps submitting it instead of just toggling them on/off. ( adding type="button" would fix it )

@c0sm1n87 Sorry, It should be fixed now on v2.6.2.


BUG: No Link Attach to Languages tab in Navbar

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
by clicking on languages tab in navbar there is no link attach to go to Languages page.
Record

[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue

@andoriyaprashant  Please assign this task to me.
Please add following Labels:

gssoc
gssoc-extd
hacktoberfest

### No URL is given for Langaues tab adding that in navbar for every page

Solution I want to Add:

@andoriyaprashant please review and merge the PR.


initial dolly conversation; transition seamlessly into money checking tree

    p "meeting dolly first time" # TODO: initial dolly conversation; transition seamlessly into money checking tree

https://github.com/pixequil/oso-game-1/blob/40b3ef34aba3e06d0637f7e4e341ca7f6518cc1b/game/money.rpy#L26

https://github.com/pixequil/oso-game-1/blob/c9953ec3e6480bd1dfa989c34c13f39644383997/game/money.rpy#L25-L27

initial dolly conversation; transition seamlessly into money checking tree. Dolly is a Shifty businesswoman with a shoddily built storefront lining the street, and whatever she offers to sell you a “loot box” full of what can surely be wonderful riches, provided you can give her "a lot of money".

label .dfirst:
show posty suspicious
p suspicious "What's this?"
dolly "Ah! About time you made it to my stand!"
dolly "I am Dolly, the sole proprietor of this fine establishment."
dolly "How's the delivery going, Posty?"
p happy "It's going fine I suppo-"
p astonished "Wait a minute, how do you know my name?"
dolly "I have connections to your higher-ups, darling."
p confused "Wha-"
dolly "But anyway, what brings you here today?"
p neutral "I guess I'm just browsing."
dolly "In that case, allow me to offer you something truly exsquisite."
show lootbox at truecenter with moveinbottom
dolly "Behold, a one-of-a-kind, very special, very rare ''loot box.''"
p suspicious "What's in it?"
dolly "Who can say?"
dolly "Perhaps it's filled with wonderful riches to make you exceptionally wealthy!"
dolly "Or it could be filled with an everyday household object."
dolly "It's all chance, honey! Buy one today and you can win practically anything."
p happy "Alright, you've convinced me! How much?"
dolly "How much do you have?"
$ saw.dolly = True
jump .money_check


Exposing entity builder

Hi.
I'd like to request that EntitiyBuilder be exposed to the API.
I'm writing a library where most (but not all) entities have a couple of the same components, so it would be nice to be able to write a wrapper that would manage that (say, having a method called new_domain_entity that would return a entity builder after having called with  both of these components)

Hello @pedrohjordao!
Just doing a litte bit refactoring, so I'll include a fix for that. Could you please post a code snippet where I can see how you would imagine the usage of the requested API?

Right now I'm trying to wrap everything from specs and only expose the significant parts for the library (e.g. I don't think I'll expose the World struct), so i'd probably end up doing something like this:
 struct Manager<C> {
  planner: Planner<C>,
  ...
 }

impl<C: 'static> Manager {
  pub fn new_empty() -> Planner<C> {...} // hides the world existence

pub fn new_shared_entity(&mut self) -> EntityBuilder {
  self.planner.world_mut().create_now().with::<Shared>().with::<Managed>()
}

pub fn new_local_entity(&mut self) -> EntityBuilder {
  self.planner.world_mut().create_now()
}

sorry if I got anything wrong with the specs api, i'm going out of the top of my head.

Ah now I got it, the method you need already exists, you just want the type to be reexported. Which, I think, wasn't left out intentionally. So a very easy issue!

I'm trying to wrap everything from specs and only expose the significant parts for the library

Do you want to hide the dependency or do you think the API could be better? If it's the latter, please tell me how you'd imagine the ideal API for specs.
Thanks for your explanations and time!

Just hiding the dependency. I'm trying to emulate another library at work, so I'm trying to keep the same api(ish) and hiding implementation details.
Can't say that I found anything missing or wrong from the specs api, but this is my first day of use.
Maybe runXwYr could have a better name? A few more examples would be nice too, but the new sys example actually answered most of my questions.
If you want me to make me a happy (and lazy) user you can add an example with serialization + networking (with a server receiving requests to create entities and sending them to clients, while clients update the entity and send back to servers (kinda like a multiplayer game)), but I think I can work it out myself xD.

@pedrohjordao Unfortunately, I didn't use the serialization stuff yet. If you have a specific question, just ask on https://gitter.im/slide-rs/specs and we'll help you out!

Maybe runXwYr could have a better name?

Are you using the run* methods? I'm just integrating dependencies into specs, which lets you specify that A has to run before B, however it wouldn't play well with the run* methods.
Would you like that feature? I'm just wondering if we should keep the older way of running systems (so the run* methods) and just add my feature...

I'm implementing systems. I just thought the method name was confusing the
first time I saw it.
Coincidently I was just thinking about how I would do dependencies, glad to
know it's being worked on.
Em sáb, 13 de mai de 2017 15:58, Thomas Schaller<EMAIL_ADDRESS>escreveu:

@pedrohjordao https://github.com/pedrohjordao Unfortunately, I didn't
use the serialization stuff yet. If you have a specific question, just ask
on https://gitter.im/slide-rs/specs http://Gitter and we'll help you
out!
Maybe runXwYr could have a better name?
Are you using the run* methods? I'm just integrating dependencies into
specs, which lets you specify that A has to run before B, however it
wouldn't play well with the run* methods.
Would you like that feature? I'm just wondering if we should keep the
older way of running systems (so the run* methods) and just add my
feature...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/slide-rs/specs/issues/146#issuecomment-301267850, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFkDZMMIfoKh0JasC2OfK_BpfgiglaUCks5r5f1RgaJpZM4NaFQZ
.

--
Att.
Pedro Jordão

Oh, just thought of something. I think the impl of components could be made
easier if there were other component traits with a set storage.
Something like impl ComponentVec for Position is easier to write then also
having to set the storage associated type. It would probably be better to
give a name that describes the use case of the storage.
Maybe this is a language limitation?
If a user wants to use their own storage (s)he can use the Component as is.
Em sáb, 13 de mai de 2017 15:58, Thomas Schaller<EMAIL_ADDRESS>escreveu:

@pedrohjordao https://github.com/pedrohjordao Unfortunately, I didn't
use the serialization stuff yet. If you have a specific question, just ask
on https://gitter.im/slide-rs/specs http://Gitter and we'll help you
out!
Maybe runXwYr could have a better name?
Are you using the run* methods? I'm just integrating dependencies into
specs, which lets you specify that A has to run before B, however it
wouldn't play well with the run* methods.
Would you like that feature? I'm just wondering if we should keep the
older way of running systems (so the run* methods) and just add my
feature...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/slide-rs/specs/issues/146#issuecomment-301267850, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFkDZMMIfoKh0JasC2OfK_BpfgiglaUCks5r5f1RgaJpZM4NaFQZ
.

--
Att.
Pedro Jordão

@pedrohjordao
So you want to write
impl VecComponent for Vel {}

over
impl Component for Vel {
    type Storage = VecStorage<Vel>;
}

?
I dislike the idea because

it hides the fact that you can easily change the storage type
it hides the fact that it is indeed a Component, not only a VecComponent
even though the interface is the same, there's a different trait
more items in the crate, making it more difficult to overview it
it's unnecessary code in specs

Fair enough.
Em sáb, 13 de mai de 2017 16:33, Thomas Schaller<EMAIL_ADDRESS>escreveu:

@pedrohjordao https://github.com/pedrohjordao
So you want to write
impl VecComponent for Vel {}
over
impl Component for Vel {
type Storage = VecStorage;
}
?
I dislike the idea because

it hides the fact that you can easily change the storage type
it hides the fact that it is indeed a Component, not only a
VecComponent
even though the interface is the same, there's a different trait
more items in the crate, making it more difficult to overview it
it's unnecessary code in specs

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/slide-rs/specs/issues/146#issuecomment-301270025, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFkDZJ0EvV0VVcYvQhLyO0_PWzGuwbPiks5r5gWjgaJpZM4NaFQZ
.

--
Att.
Pedro Jordão