Unable to install sqlite3 npm package

Describe the bug
I am unable to install the sqlite3 npm package, because there is a loop of the extension asking to install it.
I am able to install the sqlite3 npm package outside of vscode in an arbitrary directory using npm install<EMAIL_ADDRESS>
[1610487207492] INFO  (ext): EXECUTING COMMAND => sqltools.selectConnection
[1610487207492] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1610487207492] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1610487207494] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1610487207495] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1610487207496] INFO  (ls): REQUEST RECEIVED => connection/ConnectRequest
[1610487207496] INFO  (ls): Connection instance created for database.
    ns: "conn-manager"
[1610487207502] ERROR (ls): Connecting error: {"code":1000,"data":{"notification":"Core/missingModule","dontNotify":true,"args":{"conn":{"previewLimit":50,"driver":"SQLite","database":"${workspaceFolder:project}/data/database.sqlite","name":"database","isActive":false,"isConnected":false,"id":"database|SQLite||${workspaceFolder:project}\\data\\database:sqlite"},"action":"install","deps":[{"type":"package","name":"sqlite3","version":"4.2.0"}]}}}
    ns: "conn-manager"
[1610487207505] INFO  (ext): Connection explorer changed. Will be updated.
    ns: "conn-man:explorer"
[1610487207507] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1610487207507] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1610487209186] INFO  (ls): REQUEST RECEIVED => DependencyInstaller/install
[1610487209186] INFO  (ls): Will install sqlite3 package
    ns: "dep-man"

up to date, audited 69 packages in 2s

1 package is looking for funding
  run `npm fund` for details

found 0 vulnerabilities
[1610487211461] INFO  (ls): Finished sqlite3 script
    ns: "dep-man"
[1610487211461] INFO  (ls): Finished installing deps
    ns: "dep-man"
[1610487212553] INFO  (ext): EXECUTING COMMAND => sqltools.selectConnection
[1610487212553] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1610487212553] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1610487212554] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1610487212554] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1610487212555] INFO  (ls): REQUEST RECEIVED => connection/ConnectRequest
[1610487212555] INFO  (ls): Connection instance created for database.
    ns: "conn-manager"
[1610487212560] ERROR (ls): Connecting error: {"code":1000,"data":{"notification":"Core/missingModule","dontNotify":true,"args":{"conn":{"previewLimit":50,"driver":"SQLite","database":"${workspaceFolder:project}/data/database.sqlite","name":"database","isActive":false,"isConnected":false,"id":"database|SQLite||${workspaceFolder:project}\\data\\database:sqlite"},"action":"install","deps":[{"type":"package","name":"sqlite3","version":"4.2.0"}]}}}
    ns: "conn-manager"
[1610487212561] INFO  (ext): Connection explorer changed. Will be updated.
    ns: "conn-man:explorer"
[1610487212575] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1610487212575] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest

To Reproduce
Steps to reproduce the behavior:

Install sqltools and sqlite3 driver extensions
Connect to sqlite database
Message: "You need to install<EMAIL_ADDRESS>to connect to .". Click "Install Now"
Message<EMAIL_ADDRESS>installed! Go ahead and connect!". Click "Connect to ".
goto 3

Expected behavior
Installs sqlite3 package and opens database
Screenshots
N/A
Desktop (please complete the following information):

SQLTools Version 0.23.0
VSCode Version: 1.52.1
OS: Windows
Driver:

[ ] PostgreSQL/Redshift
[ ] MySQL/MariaDB
[ ] MSSQL/Azure
[x] SQLite 0.2.0
[ ] Other? Which...

Database version: sqlite

Additional context
N/A

I am experiencing the same issue. Below is my log of trying to connect to the sqlite database, running the install prompt, and then trying to connect again. Basically, it is stuck in a loop where it cant seem to find the installed dependency, but there isn't much in the way of errors coming back, except for a failed connection.
[1611124698600]` INFO  (ext): EXECUTING COMMAND => sqltools.selectConnection
[1611124698600] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1611124698601] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1611124698602] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1611124698602] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1611124698602] INFO  (ls): REQUEST RECEIVED => connection/ConnectRequest
[1611124698603] INFO  (ls): Connection instance created for Football Sim Position Map.
    ns: "conn-manager"
[1611124698604] ERROR (ls): Connecting error: {"code":1000,"data":{"notification":"Core/missingModule","dontNotify":true,"args":{"conn":{"previewLimit":100,"database":"${workspaceFolder:football-sim}/backend/database/position_map.db","driver":"SQLite","name":"Football Sim Position Map","isActive":false,"isConnected":false,"id":"Football Sim Position Map|SQLite||${workspaceFolder:football-sim}\\backend\\database\\position_map:db"},"action":"install","deps":[{"type":"package","name":"sqlite3","version":"4.2.0"}]}}}
    ns: "conn-manager"
[1611124698607] INFO  (ext): Connection explorer changed. Will be updated.
    ns: "conn-man:explorer"
[1611124698611] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1611124698611] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1611124700551] INFO  (ls): REQUEST RECEIVED => DependencyInstaller/install
[1611124700551] INFO  (ls): Will install sqlite3 package
    ns: "dep-man"

up to date, audited 69 packages in 476ms

1 package is looking for funding
  run `npm fund` for details

found 0 vulnerabilities
[1611124701097] INFO  (ls): Finished sqlite3 script
    ns: "dep-man"
[1611124701097] INFO  (ls): Finished installing deps
    ns: "dep-man"
[1611124703092] INFO  (ext): EXECUTING COMMAND => sqltools.selectConnection
[1611124703093] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1611124703093] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1611124703094] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1611124703094] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1611124703095] INFO  (ls): REQUEST RECEIVED => connection/ConnectRequest
[1611124703095] INFO  (ls): Connection instance created for Football Sim Position Map.
    ns: "conn-manager"
[1611124703097] ERROR (ls): Connecting error: {"code":1000,"data":{"notification":"Core/missingModule","dontNotify":true,"args":{"conn":{"previewLimit":100,"database":"${workspaceFolder:football-sim}/backend/database/position_map.db","driver":"SQLite","name":"Football Sim Position Map","isActive":false,"isConnected":false,"id":"Football Sim Position Map|SQLite||${workspaceFolder:football-sim}\\backend\\database\\position_map:db"},"action":"install","deps":[{"type":"package","name":"sqlite3","version":"4.2.0"}]}}}
    ns: "conn-manager"
[1611124703098] INFO  (ext): Connection explorer changed. Will be updated.
    ns: "conn-man:explorer"
[1611124703101] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1611124703102] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest

I am experiencing the same issue. Below is my log of trying to connect to the sqlite database, running the install prompt, and then trying to connect again. Basically, it is stuck in a loop where it cant seem to find the installed dependency, but there isn't much in the way of errors coming back, except for a failed connection.
[1611124698600]` INFO  (ext): EXECUTING COMMAND => sqltools.selectConnection
[1611124698600] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1611124698601] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1611124698602] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1611124698602] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1611124698602] INFO  (ls): REQUEST RECEIVED => connection/ConnectRequest
[1611124698603] INFO  (ls): Connection instance created for Football Sim Position Map.
    ns: "conn-manager"
[1611124698604] ERROR (ls): Connecting error: {"code":1000,"data":{"notification":"Core/missingModule","dontNotify":true,"args":{"conn":{"previewLimit":100,"database":"${workspaceFolder:football-sim}/backend/database/position_map.db","driver":"SQLite","name":"Football Sim Position Map","isActive":false,"isConnected":false,"id":"Football Sim Position Map|SQLite||${workspaceFolder:football-sim}\\backend\\database\\position_map:db"},"action":"install","deps":[{"type":"package","name":"sqlite3","version":"4.2.0"}]}}}
    ns: "conn-manager"
[1611124698607] INFO  (ext): Connection explorer changed. Will be updated.
    ns: "conn-man:explorer"
[1611124698611] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1611124698611] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1611124700551] INFO  (ls): REQUEST RECEIVED => DependencyInstaller/install
[1611124700551] INFO  (ls): Will install sqlite3 package
    ns: "dep-man"

up to date, audited 69 packages in 476ms

1 package is looking for funding
  run `npm fund` for details

found 0 vulnerabilities
[1611124701097] INFO  (ls): Finished sqlite3 script
    ns: "dep-man"
[1611124701097] INFO  (ls): Finished installing deps
    ns: "dep-man"
[1611124703092] INFO  (ext): EXECUTING COMMAND => sqltools.selectConnection
[1611124703093] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1611124703093] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1611124703094] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1611124703094] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1611124703095] INFO  (ls): REQUEST RECEIVED => connection/ConnectRequest
[1611124703095] INFO  (ls): Connection instance created for Football Sim Position Map.
    ns: "conn-manager"
[1611124703097] ERROR (ls): Connecting error: {"code":1000,"data":{"notification":"Core/missingModule","dontNotify":true,"args":{"conn":{"previewLimit":100,"database":"${workspaceFolder:football-sim}/backend/database/position_map.db","driver":"SQLite","name":"Football Sim Position Map","isActive":false,"isConnected":false,"id":"Football Sim Position Map|SQLite||${workspaceFolder:football-sim}\\backend\\database\\position_map:db"},"action":"install","deps":[{"type":"package","name":"sqlite3","version":"4.2.0"}]}}}
    ns: "conn-manager"
[1611124703098] INFO  (ext): Connection explorer changed. Will be updated.
    ns: "conn-man:explorer"
[1611124703101] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1611124703102] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest

Further info,
I setup a new Ubuntu 20.04 Virtual machine, and installed npm and node via the APT package manager (as opposed to following the info on https://nvm.sh), and after restarting my vscode session, I was able to get sqlite to work properly.
My guess is the script is looking in a certain place for the sqlite node modules but cant find them there and then it fails. This would explain why it can successfully install them correctly (npm handles the installation, not this package). I have not figured out where the sqlite library is looking for the sqlite node module however.

Further info,
I setup a new Ubuntu 20.04 Virtual machine, and installed npm and node via the APT package manager (as opposed to following the info on https://nvm.sh), and after restarting my vscode session, I was able to get sqlite to work properly.
My guess is the script is looking in a certain place for the sqlite node modules but cant find them there and then it fails. This would explain why it can successfully install them correctly (npm handles the installation, not this package). I have not figured out where the sqlite library is looking for the sqlite node module however.

My installation of sqltools was working great until about a week ago when I updated my npm/node installation. After that, sqltools sqlite driver stopped working with the error as described in this issue - an infinite installation loop.
I'm on macOS so my steps will be different than others, but I found this previous issue that gave me some guidance - see #638
My fix was to remove the installed npm files for this extension and re-install.
cd ~/Library/Application\ Support/vscode-sqltools/
rm ./package-lock.json
rm -rf ./node_modules
npm i

npm -i installed the npm module<EMAIL_ADDRESS>and all required dependencies. I then restarted VSCode and everything worked fine. For people on Windows, I would guess that the vscode-sqltools directory would be located at something like C:\Users\{Your User Name}\AppData\vscode-sqltools .
If any one can confirm or provide the correct OS specific path, please update this issue!

My installation of sqltools was working great until about a week ago when I updated my npm/node installation. After that, sqltools sqlite driver stopped working with the error as described in this issue - an infinite installation loop.
I'm on macOS so my steps will be different than others, but I found this previous issue that gave me some guidance - see #638
My fix was to remove the installed npm files for this extension and re-install.
cd ~/Library/Application\ Support/vscode-sqltools/
rm ./package-lock.json
rm -rf ./node_modules
npm i

npm -i installed the npm module<EMAIL_ADDRESS>and all required dependencies. I then restarted VSCode and everything worked fine. For people on Windows, I would guess that the vscode-sqltools directory would be located at something like C:\Users\{Your User Name}\AppData\vscode-sqltools .
If any one can confirm or provide the correct OS specific path, please update this issue!

As suggested by @magic-lantern, I've fixed this by deleting the "node_modules" and "package-lock.json" from C:\Users\<username>\AppData\Local\vscode-sqltools\Data
However, I don't get any tables, which is a separate issue.

As suggested by @magic-lantern, I've fixed this by deleting the "node_modules" and "package-lock.json" from C:\Users\<username>\AppData\Local\vscode-sqltools\Data
However, I don't get any tables, which is a separate issue.

Here are the paths:
macOS: ~/Library/Application Support/vscode-sqltools
Windows: %LOCALAPPDATA%\vscode-sqltools\Data (for example, C:\Users\USERNAME\AppData\Local\vscode-sqltools\Data)
Linux: ~/.local/share/vscode-sqltools (or $XDG_DATA_HOME/vscode-sqltools)
This can vary based on each machine setup, I hope making the path explicity here can help you guys fix/debug it so we can find a better solution for this problem.
I'm doing an improvement on dependencies installer code to use vscode terminal instead of spawning a shell to install deps. This will fix issues for most of the setups

This is the package I use to generate the paths btw: https://www.npmjs.com/package/env-paths

Looks like there is no solution for the issue.. Ubuntu 21.10, node 16

ok

Same problem.  I don't use SQLite often, but it was working for me in late 2021 and now I'm getting the endless "Install<EMAIL_ADDRESS>loop.

I think have found a solution in regards to NVM linux.

Go to respective vscode-sqltools directory (in my case $HOME/.local/share/vscode-sqltools running Manjaro 5.15) remove everything except Bookmarks.json.
Open vscode from the terminal $ code (I think this works because nvm is registered to export on shell launch idk).
Try connecting to SQLite database.
Install deps from prompt<EMAIL_ADDRESS>

I think have found a solution in regards to NVM linux.

Go to respective vscode-sqltools directory (in my case $HOME/.local/share/vscode-sqltools running Manjaro 5.15) remove everything except Bookmarks.json.
Open vscode from the terminal $ code (I think this works because nvm is registered to export on shell launch idk).
Try connecting to SQLite database.
Install deps from prompt<EMAIL_ADDRESS>

I tried this bro. It still isn't working for me. The same error keeps showing.

This error still persist on WSL2 Ubuntu 20.04. The pop up of<EMAIL_ADDRESS>won't go away.

For anyone encountering the issue. The solution that worked for is
$HOME/.local/share/vscode-sqltools delete all files except the Bookmarks.json and then close the uninstall the SQLTools SQLite driver extension then reinstall it. And open your SQLite database again. It's weird it just work like that for me.

For anyone encountering the issue. The solution that worked for is
$HOME/.local/share/vscode-sqltools delete all files except the Bookmarks.json and then close the uninstall the SQLTools SQLite driver extension then reinstall it. And open your SQLite database again. It's weird it just work like that for me.

This worked like magic. Thanks a lot bro.

Glad it also worked for you mate cheers!


When to "help-along" a page remove SMO?

There are many choices: When we see a page remove delta; when we see a merge delta; or both. But in paper http://research.microsoft.com/pubs/246926/p2067-makreshanski.pdf only "seeing a page remove delta" is mentioned as triggering the help-along protocol.
Details
When we see a page remove delta, we find left sibling's PID, then get its physical pointer, check its logical upper bound to make sure it equals the logical lower bound of the page being removed, and then CAS a merge delta containing the logical lower bound of the removing node onto the left sibling.
When we see a page merge delta, we find the parent of the current page, and post an index term delete to the parent using CAS.
Is that the correct way to do so?

When we see remove node, we jump to left sibling


[FEATURE]   Add new metadata elements other than Database and Schema on the Data Source Pane

I need to add new metadata elements other than Database and Schema on the Data Source Pane.
I was trying vendor-defined fields but didn't get any success.
Is there any feature to add a new metadata element on the Data Source Pane between Database and Schema?
Name: Akash Singh
Company: USEReady

Hi Akash,
Currently there is not a way to do that. This will be a feature request. I added it to our backlog. Hence, changing the story to "Enhancement".
Thanks & Regards,
Vasanthi Neelagiri,
Product Manager,
Tableau.


Save Base64encoded string

Hi,
How can I store my image that I cast it to Base64encode in database?bigger than string I need.
Thanks

not a bug, closing.


Gameboy CPU Core

Implement the Gameboy CPU Core

We shouldn't have individual items for tracking milestones, just add the individual items


ARTEMIS-918 - use requested routing type when client defined

with amqp and address doesnt exist
https://issues.apache.org/jira/browse/ARTEMIS-918

@andytaylor  this is breaking topic integration with AMQP. Run JoramAMQPAggregationTest and you will se the failures.


Add a 'Replace unused destructured parameter with _' quick fix

From https://github.com/microsoft/vscode/issues/76583

TypeScript Version<EMAIL_ADDRESS>
Search Terms:

quick fix
remove destructuring
parameter
function

Code
function f({x}, y) { return y; }

Trigger quick fixes on {x}
Bug:
A quick fix which removes the destructing is returned, but there is no quick fix to convert the first paramter to a _. Replacing a parameter with _ is available for normal parameters:
function f(x, y) { return y; }

@mjbvz seems like the OP is a complaint that we broke their code by observably changing the function parameter ordering
Changing { x } to _ has observable runtime effects (calling with null no longer throws, for example). We should change it to { } and leave it alone after that; I don't see any benefit to changing it to _ - it just creates new scope pollution

Related: #32196

function f({ x }, y) { return y; }

For the first parameter should be two quick-fix options

Remove destructuring
Remove unused declaration for: 'x'?

@RyanCavanaugh Is it true?
What about the following case?
function f({ x, x1, x2 }, y) { return y; }

I think it should swap from singular to all available to remove:

Remove unused declaration for: 'x'?

to

Remove unused declarations: 'x, x1, x2'?

and the remove destructuring should stick around


Unable to import fields

Only certain fields are available for importing bulk lists.  For example, the field rbacGroupNames are not available when using the CSV to import.  This is important if you want to assign the indicators to a specific group.  Maybe this will be available in the public release?

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

ID: eef05866-969b-5c37-8a49-f316d4165101
Version Independent ID: 9c81d2d1-e216-1cef-c511-9daacde024b4
Content: Indicator resource type
Content Source: windows/security/threat-protection/microsoft-defender-atp/ti-indicator.md
Product: w10
Technology: windows
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@mjcaparas @levinec Would you be able to provide clarifications for this question? Authoritative input might be needed. Thanks.

Hi, checking with team.

Response from the team indicates that it's not supported.

Will this be a future release?
Get Outlook for iOShttps://aka.ms/o0ukef

From: jcaparas<EMAIL_ADDRESS>Sent: Tuesday, October 15, 2019 5:14:19 PM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: Garfield de Freitas<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Unable to import fields (#5049)
Closed #5049https://github.com/MicrosoftDocs/windows-itpro-docs/issues/5049.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/5049?email_source=notifications&email_token=ANKFLWVXWAVI6AT6O7YURC3QOYXCXA5CNFSM4I27U56KYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUHNPOVY#event-2715481943, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANKFLWXHMPIZCFAR5SCNXP3QOYXCXANCNFSM4I27U56A.

@corsen55 : I recommend using the Feedback Hub to influence the developers in adding your suggestion to their future feature list.


Upgrade Helix to 1.4.2

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 56.08%. Comparing base (59551e4) to head (a41117d).
Report is 1460 commits behind head on master.

:exclamation:  There is a different number of reports uploaded between BASE (59551e4) and HEAD (a41117d). Click for more details.
HEAD has 48 uploads less than BASE

Flag
BASE (59551e4)
HEAD (a41117d)

integration
7
0

integration2
3
0

temurin
12
2

java-21
7
2

skip-bytebuffers-true
3
1

skip-bytebuffers-false
7
1

unittests
5
2

java-11
5
0

unittests2
3
0

integration1
2
0

custom-integration1
2
0

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #14648      +/-   ##
============================================
- Coverage     61.75%   56.08%   -5.67%
- Complexity      207      705     +498
============================================
  Files          2436     2119     -317
  Lines        133233   111889   -21344
  Branches      20636    17775    -2861
============================================
- Hits          82274    62754   -19520
+ Misses        44911    44165     -746
+ Partials       6048     4970    -1078

Flag
Coverage Δ

custom-integration1
?

integration
?

integration1
?

integration2
?

java-11
?

java-21
56.08% <ø> (-5.54%)
:arrow_down:

skip-bytebuffers-false
56.03% <ø> (-5.71%)
:arrow_down:

skip-bytebuffers-true
56.02% <ø> (+28.29%)
:arrow_up:

temurin
56.08% <ø> (-5.67%)
:arrow_down:

unittests
56.08% <ø> (-5.67%)
:arrow_down:

unittests1
56.08% <ø> (+9.19%)
:arrow_up:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

The failures are caused by a behavior change from Helix: https://github.com/apache/helix/issues/2982

The failures are caused by a behavior change from Helix: apache/helix#2982

This is similar on the lines of our last upgrade try


Testing Issue # 2615

If this submits it is successful.

Bug Report Dump (Auto-generated)

Version 1.27.2 (SEE BUILD PHASE)
Device: Simulator (iOS 12.1)
TestFlight: false

Yay
Sent with GitHawk


feat: add support for new ContainerAuthenticator

This PR adds an authenticator and token manager for the new "container" flavor of authentication. This style retrieves a compute resource token from the file system and uses that to retrieve an IAM access token, which is used to authenticate with the service.
Note: I realized that I refactored the IAM token manager but not the authenticator. It really should be both, so as part of this PR I refactored the authenticator to match the refactoring in the token manager.

@padamstx

I was a little surprised to not find much testing of the requestToken() method in terms of the various scenarios with token expiration, etc. but after poking around it looks like the jwt-token-manager.test.js script contains tests for those types of scenarios in sort of a generic sense (i.e. not specific to cp4d, iam or container authenticators). Is that right?

That's right - everything but the actual request for the token is abstracted away into the shared parent, the JWT token manager. So, that behavior is all captured in the JWT token manager tests.

:tada: This PR is included in version 2.12.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[iPad] Share window is displayed at the bottom of the screen, just very little part of it is visible

What is your question?
Hello,
I have the following code as per examples and notes regarding iPad specific requirement. I receive no errors but the share window is popped up at the bottom of the screen, only a small part of it visible.
final box = context.findRenderObject() as RenderBox?;
....
Share.shareXFiles(
[file],
sharePositionOrigin: box!.localToGlobal(Offset.zero) & box.size,
);

Checklist before submitting a question

[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I am using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

I'm facing the same issue


Use external directive-handler library

This makes Scala CLI use directive handler classes from an external project, rather than from its sources. This allows other projects to handle using directives with the same high level API as Scala CLI, rather than the rather low level using_directives. I'm trying to use this library from Almond, for example.

I'd be ok giving you access to the directive-handler repo, or having you fork it under the VirtusLab org, and manage it from there.

I'd be ok giving you access to the directive-handler repo, or having you fork it under the VirtusLab org, and manage it from there.

I think it would be best to fork it under VirtusLab since it's a core part of ScalaCLI and it would be easier for use manage access for people in the team

@romanowski could you fork the repository?

Sure: https://github.com/VirtusLab/directive-handler

@alexarchambault where do we stand with this? Could we instead publish the library from the usual Scala CLI repo?

Closing this, there'll be another approach to this soon.
To be tracked under #2443.


Excessive numer of compiler warning from libAtomVM when compiling stm32 port

When compiling the stm32 port with the arm-none-eabi-gcc compiler there are 10105 compiler warnings, originating from 905 unique lines of code in libAtomVM. The majority of the problems are sign-conversion accounting for 627 of the unique issues.
Breakdown of warning types:

warning type
unique issues
total warnings

sign-conversion
627
8458

cast-qual
87
497

conversion
50
291

switch-default
30
35

cast-align
18
282

bad-function-cast
16
26

missing-prototypes
14
14

undef
13
15

strict-overflow
12
33

switch-enum
9
9

shadow
8
326

inline
6
105

format-nonliteral
5
5

sign-compare
4
4

float-equal
2
2

suggest-attribute=format
2
2

discarded-qualifiers
1
1

pedantic
1
1

The attached archive contains a complete log file, and two folders.
stm32_compiler_warnings.tar.gz

by-type_details folder contains warning separated into files by warnings. These files contain every occurrence of each warning with the follow-up lines for context.
by-type_sorted_unique folder is similarly divided by warning type, but sorted by source file and limited a single occurrence of each problem, and without the extra lines of details.

I hope that this can help to triage some of these issues. This is really a meta issue in itself and part of the larger #255

Since #985 only relevant warnings are printed.


python 3.8

C:\Users\user\Desktop\Νέος φάκελος\ert-downloader-master>python ert-downloader.py https://www.ertflix.gr/*****
Traceback (most recent call last):
File "C:\Users\user\Desktop\Νέος φάκελος\ert-downloader-master\ert-downloader.py", line 1, in
from extractors import archive_extractor, ertflix_extractor, star_extractor, megatv_extractor
File "C:\Users\user\Desktop\Νέος φάκελος\ert-downloader-master\extractors\archive_extractor.py", line 1, in
import requests
ModuleNotFoundError: No module named 'requests'

You have not run on the terminal pip install -r requirements.txt

Thanks for the prompt reply
Running python ert-downloader.py https://www.ertflix.gr/****
Many small snapshots are downloaded that in the end do not merge.

The second time I tried it came down normally, thank you very much!!!!


Files offline?

Hi,
I am a Linux newbie. Just to confirm: this application does not save all of the files from google drive on my local machine?
Mong H. Ng

Yes, correct. The local cache stores only some of the remote files to reduce latency.


Add action to listener

Adding action to listener will allow you to subscribe for actions in much more simpler  way than custom middleware.

@zhDmitry For this purpose use event emitters. It was discussed many times.
https://github.com/reactjs/redux/issues/2081
https://github.com/reactjs/redux/issues/1971
https://github.com/reactjs/redux/pull/1795
https://github.com/reactjs/redux/issues/1786
https://github.com/reactjs/redux/pull/1697
https://github.com/reactjs/redux/issues/1243
https://github.com/reactjs/redux/issues/1057
https://github.com/reactjs/redux/issues/347


Wrong behavior when using var on readonly fields in Kotlin

Related to #680
When using the modifier var instead of val on implicit or explicit read-only fields, no setter methods get generated if you use abstract classes resulting in an error that they are missing. However, if you use interfaces they will get generated. In both cases Kapt generates method stubs for getters and setters.
Requery should result in an error if var instead of val is used on read-only fields. This should also apply for interfaces. One just needs to test whether a specific setter is present or not.
Correct behavior would be:

test whether field should be read-only
1.1 if it is check for present setters and throw an error or
1.2 overwrite and proceed as normal

Example:
@get:Key
@get:Generated
var id: Int

results in (by kapt)
@io.requery.Generated()
@io.requery.Key()
public abstract int getId();

public abstract void setId(int p0);

but setId() wont get overriden because id is generated and therefore read-only.

Any way to overcome this?
I get an error: "error: Element "id" is read-only but has setter (Kotlin: change var to val)" .
It was possible before to change the id.
Now I can't.
Example of what I tried:
@Entity
interface TLThemeResource : Parcelable, Persistable {
    @get:Key
    @set:Key
    @get:Generated
    var id: Long
...
}


[Flutter Web] SelectableText selection background is off when a height is set

Steps to Reproduce

Set a value to the height parameter in TextStyle
class MyWidget extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Column(
      children: <Widget>[
        SelectableText(
          'SelectableText with no height',
          style: TextStyle(),
        ),
        SizedBox(height: 16),
        SelectableText(
          'SelectableText with height',
          style: TextStyle(height: 1),
        ),
      ],
    );
  }
}

Expected results:

Actual results:

Logs

[✓] Flutter (Channel stable, 2.2.0, on macOS 11.2.2 20D80 darwin-x64, locale en-AU)
    • Flutter version 2.2.0 at /Users/edwin/flutter
    • Framework revision b22742018b (2 weeks ago), 2021-05-14 19:12:57 -0700
    • Engine revision a9d88a4d18
    • Dart version 2.13.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/edwin/Library/Android/sdk
    • Platform android-30, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 39.0.3
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] VS Code (version 1.56.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.22.0

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 91.0.4472.77

@wliumelb
I not sure if I understand exactly what's the issue here is. Can you elaborate ?

The g in second text is not completely covered by background selection.

 flutter doctor -v
[✓] Flutter (Channel stable, 2.2.1, on Mac OS X 10.15.4 19E2269 darwin-x64,
    locale en-GB)
    • Flutter version 2.2.1 at /Users/dhs/documents/fluttersdk/flutter
    • Framework revision 02c026b03c (4 days ago), 2021-05-27 12:24:44 -0700
    • Engine revision 0fdb562ac8
    • Dart version 2.13.1

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.55.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (4 available)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 91.0.4472.77

• No issues found!

CC @chunhtai

This sounds like a framework issue, so I'm going to remove the web label. However, if this needs an engine fix, please feel free to add the web label back.


Brain phantom

Looks good. I think you just need to merge main into your branch to get the tests fixed. It looks like the failing one has been removed.


fix: fix hover example

when I visit monaco editor website， I find this example is error。 can't get playground.html, and 404。
Then I git clone monaco editor and run dev。 finally I find that after website restructure， the file playground.html  at root directory。

Thanks!


pending force closing channel with negative blocks_til_maturity

Background
I force close a channel, and it stuck in pending with negative block_till_maturity
Opening tx : "eb0be30a7b0f9f44844893f3485b6f5ea56f526351a2e226faeb0d275e5b6f2b"
Also closing tx exist and been confirmed "73becb3977d126dfce2a1f45241692f2c6f87eb34c0a925a6f76e93f7c7d367b"
I think this is the receiving address "bc1qsz33v8nrcgnc8ucrz96rcq0mkp345m9lrzh3sygd0er6d2xgv0ds34zgsp"
and it say unspend but is not present on the list unspent.
"pending_force_closing_channels": [
        {
            "channel": {
                "remote_node_pub": "0395033b252c6f40e3756984162d68174e2bd8060a129c0d3462a9370471c6d28f",
                "channel_point": "eb0be30a7b0f9f44844893f3485b6f5ea56f526351a2e226faeb0d275e5b6f2b:0",
                "capacity": "40000",
                "local_balance": "6749",
                "remote_balance": "0"
            },
            "closing_txid": "73becb3977d126dfce2a1f45241692f2c6f87eb34c0a925a6f76e93f7c7d367b",
            "limbo_balance": "6749",
            "maturity_height": 571319,
            "blocks_til_maturity": -37,
            "recovered_balance": "0",
            "pending_htlcs": [
            ]
        }
    ]

Your environment
lnd version 0.6.0-beta commit=v0.6-beta-rc4
Raspberry Pi3b linux
bitcoind v0.17.1
Log
2019-04-12 16:07:44.661 [INF] LTND: Waiting for chain backend to finish sync, start_height=571345
2019-04-12 16:07:44.720 [INF] LTND: Chain backend is fully synced (end_height=571345)!
2019-04-12 16:07:44.810 [INF] NTFN: New block epoch subscription
2019-04-12 16:07:44.810 [INF] HSWC: Starting HTLC Switch
2019-04-12 16:07:44.815 [INF] NTFN: New block epoch subscription
2019-04-12 16:07:45.140 [INF] NTFN: New block epoch subscription
2019-04-12 16:07:45.208 [INF] UTXN: (Re)-sweeping 1 heights below height=571345
2019-04-12 16:07:45.211 [INF] UTXN: Attempting to graduate height=571319: num_kids=1, num_babies=0
2019-04-12 16:07:45.211 [INF] UTXN: Sweeping 1 CSV-delayed outputs with sweep tx for height 571319
2019-04-12 16:07:45.211 [INF] SWPR: Sweep request received: out_point=73becb3977d126dfce2a1f45241692f2c6f87eb34c0a925a6f76e93f7c7d367b:0, witness_type=CommitmentTimeLock, time_lock=144, size=0.00006749 BTC
2019-04-12 16:07:45.212 [INF] UTXN: UTXO Nursery is now fully synced
2019-04-12 16:07:45.214 [INF] NTFN: New spend subscription: spend_id=1, outpoint=73becb3977d126dfce2a1f45241692f2c6f87eb34c0a925a6f76e93f7c7d367b:0, height_hint=571345
2019-04-12 16:07:46.534 [INF] NTFN: New confirmation subscription: txid=73becb3977d126dfce2a1f45241692f2c6f87eb34c0a925a6f76e93f7c7d367b, num_confs=1
2019-04-12 16:07:46.534 [INF] NTFN: New confirmation subscription: conf_id=1, txid=73becb3977d126dfce2a1f45241692f2c6f87eb34c0a925a6f76e93f7c7d367b, height_hint=571175
2019-04-12 16:07:47.354 [INF] NTFN: Dispatching 1 confirmation notification for txid=73becb3977d126dfce2a1f45241692f2c6f87eb34c0a925a6f76e93f7c7d367b
2019-04-12 16:07:47.355 [INF] NTFN: New spend subscription: spend_id=11, outpoint=73becb3977d126dfce2a1f45241692f2c6f87eb34c0a925a6f76e93f7c7d367b:0, height_hint=571345
2019-04-12 16:07:47.355 [INF] CNCT: *contractcourt.commitSweepResolver(eb0be30a7b0f9f44844893f3485b6f5ea56f526351a2e226faeb0d275e5b6f2b:0): waiting for commit output to be swept
2019-04-12 16:07:47.418 [INF] BRAR: Starting contract observer, watching for breaches.
2019-04-12 16:07:47.418 [INF] DISC: Authenticated Gossiper is starting
2019-04-12 16:32:18.691 [INF] UTXN: NurseryReport: building nursery report for channel eb0be30a7b0f9f44844893f3485b6f5ea56f526351a2e226faeb0d275e5b6f2b:0

Also
channelbalance
"pending_open_balance": "0"
walletbalance
"unconfirmed_balance": "0"
txns
"tx_hash": "73becb3977d126dfce2a1f45241692f2c6f87eb34c0a925a6f76e93f7c7d367b",
            "amount": "0",
            "num_confirmations": 192,
            "block_hash": "000000000000000000016d64a8286b20b6ffe54fef6c858021517f774a91a2e7",
            "block_height": 571175,
            "time_stamp": "1554983975",
            "total_fees": "0",
            "dest_addresses": [
                "bc1qsz33v8nrcgnc8ucrz96rcq0mkp345m9lrzh3sygd0er6d2xgv0ds34zgsp",
                "bc1qtpkxhae6fwqq6n2suyfrma6xr8z0c44u2nafwx"
            ]

The output's somewhat small so given the current high fees it might take some time to be able to sweep it.

Hi!
I have same problem
LND is 0.6.0-beta commit=v0.6.0-beta-116-g985902be2779a5dcc7ac1d5b3d7eb7929608816d
For example
LND-34:
{
"channel": {
    "remote_node_pub": "03bc9337c7a28bb784d67742ebedd30a93bacdf7e4ca16436ef3798000242b2251",
    "channel_point": "6cd7a9364d87dd145c2688d5f1997af0aafef151a05a959ec37491964c4f0e29:0",
    "capacity": "16777215",
    "local_balance": "16751786",
    "remote_balance": "0"
},
"closing_txid": "e762434a9f9d123e459a6e35c05dcf3e68cc893895fb63193a807e8d02fcb13d",
"limbo_balance": "817",
"maturity_height": 570310,
"blocks_til_maturity": -3762,
"recovered_balance": "16751786",
"pending_htlcs": [
    {
	"incoming": false,
	"amount": "817",
	"outpoint": "63dd2e62ff9432cecc71cdb642aaa48ba1e8d64e1db9179ec741a63ca285027e:0",
	"maturity_height": 570311,
	"blocks_til_maturity": -3761,
	"stage": 2
    }
]
},

LND-32:
{
"channel": {
    "remote_node_pub": "03bc9337c7a28bb784d67742ebedd30a93bacdf7e4ca16436ef3798000242b2251",
    "channel_point": "0fadc94bf3845cc723f3ec049c6f052787105fa3bceeafecab80b405197c0d3d:1",
    "capacity": "16777215",
    "local_balance": "16761926",
    "remote_balance": "0"
},
"closing_txid": "8eeaddd2ba3f19f6c088c7a902dac71c64e5af8d2e64802f2639cf9989b3fd1d",
"limbo_balance": "3268",
"maturity_height": 570310,
"blocks_til_maturity": -3762,
"recovered_balance": "16761926",
"pending_htlcs": [
    {
	"incoming": false,
	"amount": "817",
	"outpoint": "d8f1b521a0be0ab85f9130019d477a60145a087f7d5d4bbe1217d0f4b1e73709:0",
	"maturity_height": 570311,
	"blocks_til_maturity": -3761,
	"stage": 2
    },
    {
	"incoming": false,
	"amount": "817",
	"outpoint": "2208bad382348c33d5b81d7bb26c6c86e1099f86e41664473a9f46ebd8474931:0",
	"maturity_height": 570311,
	"blocks_til_maturity": -3761,
	"stage": 2
    },
    {
	"incoming": false,
	"amount": "817",
	"outpoint": "e16350621e9a5490a78ca61ce385e57579c5bf39fb082525dba1b3b5268a6977:0",
	"maturity_height": 570311,
	"blocks_til_maturity": -3761,
	"stage": 2
    },
    {
	"incoming": false,
	"amount": "817",
	"outpoint": "a7b9339afa4409d418dc2bdb5ade2896fc666a368c17375d7f82c4dfd3a91cca:0",
	"maturity_height": 570311,
	"blocks_til_maturity": -3761,
	"stage": 2
    }
]
},

As i understand it's because small amount of pending HTLC?
My balance there is big but i cannot sweep. How can i resolve this? To wait? Is there some force way to sweep this?

Strange. I checked some closing transactions from above listed ones
Amounts (~16M) were swept successfully by my LNDs and even used.
So funds are not locked as i understand correctly. Right?

"limbo_balance": "817",

This is the balance that is still pending. We're not able to sweep it because the output is too small for the current fee rates - you'll need to wait until fees are lower. Your commitment output has already been swept as you can see on-chain, so there's no issue here.


Could a license be added ?

Hi,
very helpful. Would it be possible that you add a license ?
Thanks.

Hi @hawk66,
sure, will do
thanks


Add turkish language

Hi @mertmit . Thanks a lot for the translations! For the friendly name we use the language origin name. I would assume that would be Türkçe correct?

@mertmit Could you please take a quick look on your translations in action and give me a feedback if we can release it like that?
https://github.com/ransome1/sleek/releases/tag/v1.1.1-rc.1
Thanks

Yes "Türkçe" is true. I checked the release everything seems correct. Thanks for the great application.


default k for unions

Coverage decreased (-0.05%) to 98.137% when pulling b181b1aed5ba0ab4f34131e7b7d1e86ef79b83ad on default-k-union into 3619578bd11c5ee2efd6c93ff60338cef6026991 on master.


unique_key config for snapshots

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

What page(s) or areas on docs.getdbt.com are affected?
unique_key config for snapshots

What changes are you suggesting?
I think for the snapshot hudi table, the primaryKey should be ‘dbt_scd_id’, which is used in the ‘merge into on’ statement generated by ‘dbt snapshot’ command. And the dbt_scd_id column is related to the primary key of the source table and the update_at referenced column when we reference to timestamp strategy.
But now the primaryKey of the snapshot CTAS hudi table, which is created as we run the first ‘dbt snapshot’ command, is the same with the primary key of the source table.
In my experiment, I change two parts of the sql generated by the dbt, one is the primaryKey of the snapshot CTAS hudi table using ‘dbt_scd_id’, the other one is the ‘insert [referenced columns]’ in the 'merge into insert ’ statement instead of ‘insert *’. After these changes, I get the scd2 table, stored in hudi.
Additional information
In my solution, I copy the files include/spark/macros/adapters.sql and include/spark/macros/materializations/snapshot.sql  to the macro directory in my own project

And, I change the related code, that works.

Hi @darcyMaJiyue thank you so much for opening this issue! We will try to update the docs as soon as I have a clear understanding what to update.
@dbeatty10 Can you help me understand what the change to the docs should be?

@runleonarun I can't tell if this is a request to update the docs or a request to update the implementation in dbt-spark.
It looks more like the latter, so I'm going to transfer this issue to dbt-spark -- we can always transfer it back again if needed 😎


V3.1

Current coverage is 73.33%

Merging #65 into master will not affect coverage as of 572afad

@@            master     #65   diff @@
======================================
  Files            4       4
  Stmts           15      15
  Branches         0       0
  Methods          0       0
======================================
  Hit             11      11
  Partial          0       0
  Missed           4       4

Review entire Coverage Diff as of 572afad

Uncovered Suggestions

+13.33% via ...utils/ApolloUtils.js#17...18
+13.33% via ...utils/ApolloUtils.js#5...6

Powered by Codecov. Updated on successful CI builds.


Can you provide a requirements file to install the required conda environment?

Can the authors provide a conda environments file to duplicate the environment?
I am currently in dependency hell. Openmm / Openmm-plummed refuses to be imported no mater what I try.

Hey guys, was just wondering if anyone could help with this if they have some spare time. Thanks in advance!

Hi ackbar03, thanks for your interest in our protocol, and sorry about the late response, most of us were away over the long weekend!
Assuming you are using our colab notebook, my best guess is that you are not using a fresh kernel. However, if that is not the issue if you could answer a couple of questions, we can probably help more.

Are you using the colab or trying to do this locally?
Could you paste here your error messages and install logs?

Thank you,
Bodhi

Hi,

I am doing this locally. I git cloned the files and was trying to run through the heavydemo notebook to see if everything works. I was stuck on the "#@title Install OpenMM with Plumed" section.

The error was that after installing openmmplumed, openmm, and fixpdb through conda install -c conda-forge openmm-plumed and conda install -c conda-forge openmm and conda install -c omnia pdbfixer respectively, it refuses to import
from openmm.app import *.

I googled the problem and it seems that the issue is with the version of openmm, as per https://github.com/deepmind/alphafold/issues/175.
After installing the correct version via conda install -c conda-forge openmm=7.5.1, it gives my an error when I try to import openmm.
However, I see there were some fixes in the code, and the latest version does not cause any problems using  openmm=7.5.1. Thanks for making the change.
I am still encountering some other problem but I will open up a new issue.


onstream event on firefox

Hi Muaz Kahn,
I am using your RTCMultiConnection to implement a video+audio+data broadcast. It's working fine on google chrome but I have an issue when I want to use it one firefox : the "onstream" event is not fired when someone is joining the room (so I am not able to add the media element at this moment..). Do you have an idea of what could happen?
Thanks in advance for your help

You should try this:
connection.enableFileSharing = false; // optional
connection.session = {
    audio: true,
    video: true,
    data: true,
    broadcast: true // DO NOT use "oneway:true"
};

Also please make sure that you are NOT trying to capture single camera simultaneously both form Chrome and Firefox.
connection.onMediaError = function() {
    alert(JSON.stringify(arguments, null, '\t'));
};

This new configuration is not working.
And I have a problem with it because it ask the person who is joining the room to also share his video. Only the initiator should share his video. Which is not happening when I use "oneway: true". So I should keep using "oneway" and not using "broadcast".
I am using the 3.2.89 version of your rtcmulticonnection-v3 library. It's maybe something you improve in newer version ?

I updated the rtcmulticonnection-v3 library to the newest version (3.2.98).
But the problem is persistent. I should maybe precise you the configuration I use :
From the broadcaster side :
`	$scope.connection = new RTCMultiConnection();
$scope.connection.session = {
audio: true,
video: true,
data: true,
broadcast: true
};
$scope.connection.sdpConstraints.mandatory = {
    OfferToReceiveAudio: false,
    OfferToReceiveVideo: false
};

$scope.connection.enableScalableBroadcast = true;
$scope.connection.enableFileSharing = false;

`
From the listener side :
`	$scope.connection = new RTCMultiConnection();
$scope.connection.session = {
audio: false,
video: false,
data: true,
oneway: true
};
$scope.connection.sdpConstraints.mandatory = {
	OfferToReceiveAudio: true,
	OfferToReceiveVideo: true
};

$scope.connection.enableScalableBroadcast = true;
$scope.connection.enableFileSharing = false;
$scope.connection.dontCaptureUserMedia = true;
$scope.connection.autoReDialOnFailure = true; `

Do you have another idea why this is only working on Google Chrome ?

Please try this:
$scope.broadcaster = new RTCMultiConnection();

$scope.broadcaster.session = {
    audio: true,
    video: true,
    data: true,
    oneway: true
};

$scope.broadcaster.sdpConstraints.mandatory = {
    OfferToReceiveAudio: false,
    OfferToReceiveVideo: false
};

$scope.broadcaster.enableScalableBroadcast = false;

Use oneway:true instead of broadcast:true.
Set enableScalableBroadcast=false

If you're planning one-to-many broadcast rather than oneway streaming, then:
1.Set  OfferToReceiveAudio==true and OfferToReceiveVideo==true (you are using wrong values)
2. Set enableScalableBroadcast=false
3. Set broadcast:true (as you are already doing)
enableScalableBroadcast MUST be false in all situations.
OfferToReceive Audio/Video MUST be true so that broadcaster can also receive videos.

It's still not working on firefox.

Updated rtcm to the last version.
The event is now triggered on firefox.

I'm using the latest version of rtcmulticonnection, using a many-to-many connection with vue and when someone join to the room, the event onNewParticipant is triggered, but onstream not.


Update pom.xml (lombok version update)

Hi! lombok version needs to be updated to latest. Cheers o/

Closing for the same reason as the other one. New version soon.


Allow numbers and strings not beginning with "L" and "H"...

...to be valid values for cmdLow and cmdHigh
eg.: for Rohnson R-029, dpRotationSpeed can be used for heating power (1-3) or for mode ("smart"/"auto"), but with the limitation of the values by the first characters "l" or "h" it's not usable at all.

Thank you!


The resource you are looking for has been removed, had its name changed, or is temporarily unavailable.

When trying to add the communicator author app in a team's channel, I'm getting the following message: "The resource you are looking for has been removed, had its name changed, or is temporarily unavailable."
:(

App service deployment might failed. Check the fix which was mentioned here: https://github.com/OfficeDev/microsoft-teams-apps-company-communicator/wiki/Troubleshooting#1-code-deployment-failure

App service deployment might failed. Check the fix which was mentioned here: https://github.com/OfficeDev/microsoft-teams-apps-company-communicator/wiki/Troubleshooting#1-code-deployment-failure

Thanks, I was to get it working by following the steps on the migration from v3-v4.  I was creating it from scratch at the beginning /having multiple instances of the project.

Thanks, I was to get it working by following the steps on the migration from v3-v4.  I was creating it from scratch at the beginning /having multiple instances of the project.

I deployed an Angular App using Github Actions, and I get this error The resource you are looking for has been removed, had its name changed, or is temporarily unavailable. whenever I refresh the page. I have a web.config file in my root folder but that is not working.
This error started when I started deploying using Github Actions.
Can you help me understand and fix the error?


Add Language: USD

Language name
USD, developed by Pixar Animation Studios
URL of example repository
https://github.com/PixarAnimationStudios/USD - Example
https://github.com/ColinKennedy/USD-Cookbook - See any **/usda folder for examples. Here's an example
URL of syntax highlighting grammar
VSCode
https://github.com/AnimalLogic/AL_usd_vscode_extension
Pygments
General Syntax
List Of Keywords
Full disclosure, I wrote this lexer and it isn't perfect (for example, not catching .connect as part of an attribute name). Screenshot of what the looks like, here
Text Editors
A simpler, regex-based example
Vim's example some small issues in my experience but it works pretty well overall
Most popular extensions

.usd (7K+)
.usda (13K+)
.usdc (3.9K+)

Grammar Features
This gist shows a fairly complex USD file and some of its syntax features.
Detected language
None

Language name
USD, developed by Pixar Animation Studios
URL of example repository
https://github.com/PixarAnimationStudios/USD - Example
https://github.com/ColinKennedy/USD-Cookbook - See any **/usda folder for examples. Here's an example
URL of syntax highlighting grammar
VSCode
https://github.com/AnimalLogic/AL_usd_vscode_extension
Pygments
General Syntax
List Of Keywords
Full disclosure, I wrote this lexer and it isn't perfect (for example, not catching .connect as part of an attribute name). Screenshot of what the looks like, here
Text Editors
A simpler, regex-based example
Vim's example some small issues in my experience but it works pretty well overall
Most popular extensions

.usd (7K+)
.usda (13K+)
.usdc (3.9K+)

Grammar Features
This gist shows a fairly complex USD file and some of its syntax features.
Detected language
None


api-server needs same service perms as stream-loader

the api-server sends messages to the queue similar to how stream-loader does, so the perms have to match

PR Merged


[TechDocs] Images not rendered consistently from docs and entity page

Expected Behavior
When adding some docs, you might need to insert images on it and be rendered by the techdocs package used by backstage.
Actual Behavior
After an strange behavior we noticed in our company, I did a couple tests, and I found a misleading behavior depending if the docs are loaded from the docs page (/docs), or from the entity page (/{entity}/docs).
Steps to Reproduce

Inside your /docs folder, add an image whose name has white spaces on it, like image with spaces.jpg, and another one withour white spaces.
In your index.md add the following lines:  Image without formatting:
  ![image with spaces](./image with spaces.png)

  Image with formatting:
  ![image with spaces formatted](./image%20with%20spaces.png)

  Image without spacing:
  ![image without spacing](./image_without_spaces.png)

When the new documentation is loaded. Go to the docs page and open your new docs. The 3 images are displayed as expected.
Now open the entity page and navigate to the docs. In that case, only the last image (the one without white spaces) is loaded successfully.

In these 2 screenshot you can see what we get in each case:

First, the output when using the docs from the sidebar (as the images are big, you can only see the first one. The order is the same as described in the step 2):

And then, from inside the entity page (as you can see, only the one without spaces in the name is rendered properly):

Context
Logs when opening the docs tab inside the EntityPage:
2022-06-13T12:54:13.905Z backstage info GET /api/techdocs/static/docs/default/Component/backstage-empty-project-wizard-test/assets/stylesheets/palette.e6a45f82.min.css 304 type=request status=[object Object] request=[object Object] timings=[object Object] tracing=[object Object]
2022-06-13T12:54:13.913Z backstage info GET /api/techdocs/static/docs/default/Component/backstage-empty-project-wizard-test/image_without_spaces.png 304 type=request status=[object Object] request=[object Object] timings=[object Object] tracing=[object Object]
2022-06-13T12:54:13.914Z backstage info GET /api/techdocs/static/docs/default/Component/backstage-empty-project-wizard-test/assets/stylesheets/main.50e68009.min.css 304 type=request status=[object Object] request=[object Object] timings=[object Object] tracing=[object Object]
2022-06-13T12:54:14.018Z backstage error Encountered error object without a message, got 'InvalidAccessKeyId: null' type=errorHandler stack=Error: Encountered error object without a message, got 'InvalidAccessKeyId: null'
    at Object.assertError (/app/node_modules/@backstage/errors/dist/index.cjs.js:29:11)
    at /app/node_modules/@backstage/plugin-techdocs-node/dist/index.cjs.js:808:16
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
2022-06-13T12:54:14.019Z backstage info GET /api/techdocs/static/docs/default/Component/backstage-empty-project-wizard-test/image%2520with%2520spaces.png 500 type=request status=[object Object] request=[object Object] timings=[object Object] tracing=[object Object]

When accessing via the Docs, no logs are displayed in info mode.

Just updated to backstage 1.3.0, now this issue seems to be solved, so I'm closing it.


Reopen module as class

class Kernel; end

This code currently crashes with an assertion error, it should raise the same TypeError as MRI.

It looks like there are no upstream specs for this code (at least the error message cannot be found in the specs repo), so it might be worth it to add those upstream as well.


Update yiisoft/http and yiisoft/error-handler dependencies.

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

:+1:


#272 Mod Support

Description
Contributions

Added mod downloader
Added type
Closes #272
Refactored rust main to be simpler.
Added preset env for cron jobs to carry over cron data.
Moved to directory variables.

Checklist

[x] I added one or multiple labels which best describes this PR.
[x] I have tested the changes locally.
[x] This PR has a reviewer on it.
[x] I have validated my changes in a docker container and on Ubuntu. (Only needed for Odin or Docker Changes)

Still sifting through all the changes. I should have a review finished up tomorrow!


Add support for weave 2.6.5

Weave 2.6.5 is a bugfix release. In particular, it fixes one nasty bug where weave will CrashLoopBackOff if ipv6 is disabled on the host. This is preventing installation of kurl at a company that disables ipv6 out of the box on all hosts.
https://github.com/weaveworks/weave/releases/tag/v2.6.5

2.6.5 is now supported.


Install v2 patient/caregiver app at OHSU

@swmuir to add notes on exactly where we are.

Sean created a new docker and OHSU is going to test it.


Fixed aspect ratio in image preview, removed text under polaroid

A couple style nits, r+ with those.


About Evaluation on Val Split

I have run the code following these configurations:
MODEL:
  WEIGHT: "/content/gdrive/MyDrive/SMOKE/tools/logs/model_final.pth" # # "catalog://ImageNetPretrained/DLA34" #
INPUT:
  FLIP_PROB_TRAIN: 0.5
  SHIFT_SCALE_PROB_TRAIN: 0.3
DATASETS:
  DETECT_CLASSES: ("Car", "Cyclist", "Pedestrian")
  TRAIN: ("kitti_train",)
  TEST: ("kitti_test",)
  TRAIN_SPLIT: "train"
  TEST_SPLIT: "val"
SOLVER:
  BASE_LR: 2.5e-4 # 2.5e-4 for DLA training
  STEPS: (10, 1000, 10000, 18000) # The original is 10000, 18000
  MAX_ITERATION: 25000 # 25000
  IMS_PER_BATCH: 1

running this command:
!python tools/plain_train_net.py --eval-only --config-file "configs/smoke_gn_vector.yaml"
and I got these results which are too much higher than the ones mentioned in the paper:
car_detection AP: 99.253822 90.208725 90.030869
car_orientation AP: 99.235359 90.185669 89.979942
pedestrian_detection AP: 71.481392 71.233459 63.760303
pedestrian_orientation AP: 68.498894 67.344612 60.299564
cyclist_detection AP: 90.288994 90.166794 89.574661
cyclist_orientation AP: 89.407166 89.463783 88.851265
car_detection_ground AP: 83.296700 82.894707 75.743301
pedestrian_detection_ground AP: 43.819630 38.038414 36.839706
cyclist_detection_ground AP: 45.897942 52.835869 46.894695
car_detection_3d AP: 70.499840 68.271675 61.957222
cyclist_detection_3d AP: 35.539608 40.786041 39.733009
pedestrian_detection_3d AP: 40.907085 35.998760 30.820993
Also, the training finishes really fast and the scores are always exactly the same, for all models that are generated from different iterations, even if I try to train from scratch, I wonder what is the problem? can anyone help me please?
Thank you.

I have run the code following these configurations:
MODEL:
  WEIGHT: "/content/gdrive/MyDrive/SMOKE/tools/logs/model_final.pth" # # "catalog://ImageNetPretrained/DLA34" #
INPUT:
  FLIP_PROB_TRAIN: 0.5
  SHIFT_SCALE_PROB_TRAIN: 0.3
DATASETS:
  DETECT_CLASSES: ("Car", "Cyclist", "Pedestrian")
  TRAIN: ("kitti_train",)
  TEST: ("kitti_test",)
  TRAIN_SPLIT: "train"
  TEST_SPLIT: "val"
SOLVER:
  BASE_LR: 2.5e-4 # 2.5e-4 for DLA training
  STEPS: (10, 1000, 10000, 18000) # The original is 10000, 18000
  MAX_ITERATION: 25000 # 25000
  IMS_PER_BATCH: 1

running this command:
!python tools/plain_train_net.py --eval-only --config-file "configs/smoke_gn_vector.yaml"
and I got these results which are too much higher than the ones mentioned in the paper: car_detection AP: 99.253822 90.208725 90.030869 car_orientation AP: 99.235359 90.185669 89.979942 pedestrian_detection AP: 71.481392 71.233459 63.760303 pedestrian_orientation AP: 68.498894 67.344612 60.299564 cyclist_detection AP: 90.288994 90.166794 89.574661 cyclist_orientation AP: 89.407166 89.463783 88.851265 car_detection_ground AP: 83.296700 82.894707 75.743301 pedestrian_detection_ground AP: 43.819630 38.038414 36.839706 cyclist_detection_ground AP: 45.897942 52.835869 46.894695 car_detection_3d AP: 70.499840 68.271675 61.957222 cyclist_detection_3d AP: 35.539608 40.786041 39.733009 pedestrian_detection_3d AP: 40.907085 35.998760 30.820993
Also, the training finishes really fast and the scores are always exactly the same, for all models that are generated from different iterations, even if I try to train from scratch, I wonder what is the problem? can anyone help me please?
Thank you.

I have a question for you. I also got similar output results as yours, but the indicators in the original paper include not only 3D but also Bev. My question is, how can we get the indicators about Bev?


Fix check for connection_persistent with phpredis factory

Fixes #537
This condition is now exactly the same as the one to check for connection method, see https://github.com/snc/SncRedisBundle/blob/86b5d1e4a82701bb172800106bb1c719a03bb1bd/Factory/PhpredisClientFactory.php#L62

@starred-gijs Could you add some tests to PhpredisClientFactoryTest please?

Ah yes will do that, did not realise there was a testcase for this class. Will do it shortly

Hmmz now that I have seen the testcase, Im not sure how to write a test for this, any pointers?

@starred-gijs You will need to check what's the value of $client->getPersistentID() in current test cases.

Also 'connection_persistent' => true, looks wrong as it should be an integer.

And my bad it should be string according to phpredis doc https://github.com/phpredis/phpredis#pconnect-popen

Merging it. I will add some tests later before the release tomorrow morning.
Thanks a lot @starred-gijs!


update sctp mc

update sctp mc as discussed in previous PR
From prior PR comment:

The current version of load-sctp-module machineConfig is using different value for ignition version and file permission comparing to Red Hat Official Documentation for Openshift 4.14.
Updating load-sctp-module machineConfig to align with Red Hat Official Documentation for Openshift 4.14
Enabling SCTP on OCP 4.14

/ok-to-test


Can i use it in web worker?

Can i use it in web worker?How to do it?

use-cannon runs cannon-es in a webworker and still uses the es-debugger. https://github.com/pmndrs/use-cannon/blob/master/src/Debug.tsx

Yes, this can be used while cannon-es is in a webworker.
If you use react, use Debug from use-cannon, which is already running in a web worker.
If you don't use react, check out the cannon-es webworker and webworker with SharedArrayBuffer examples.
NOTE: the library itself doesn't need to go in the serviceworker. It works with three.js which is usually on the main worker. You need to put only cannon-es in the webworker.


upgrade prettier

upgrade prettier
remove pretty lint

@wolverineks, let's resolve conflicts here plz, and then we will merge right away 🙏 Sorry the PR got out of sync that much

removes dependency on prettylint
requires adding ci step yarn prettier --list-different ./src/**/*.js

@wolverineks , resolve conflicts please, after the hocs PR got merged


Investigate simple code splitting in webpack bundle

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

Action Performed:

Open a new incognito window
Open the developer console, network tab
Throttle your network connection to Slow 3G
Go to NewDot web (production or staging)

Expected Result:
The site should load and become visible reasonably quickly, even over a slow network.
Actual Result:
The site takes about 55 seconds to load.
Workaround:
n/a
Platform:
Where is this issue occurring?

Web
Mobile Web

Issue reported by: @roryabraham
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1666947230935069
View all open jobs on GitHub

Created this issue specifically for @kidroca to investigate implementing the simple "no downside" improvements described in the slack thread. Not as interested in investing in more lazy loading than that right now. Let's see how much we can improve this without having to influence the main code development.

Sure thing
Configuring vendor chunks would also aid webpack development speed as caching would be more effective

Just to be clear:
The improvements with vendor chunks, would probably not help much for new users (incognito mode) loading the web app
It's intended to help existing users refreshing/reopening after an update has been deployed
The improvement in load time we might see for this (vendor chunks) change is - fresh loading time might improve a bit from fetching smaller chunks in parallel

Assuming no update here, which is fine. I think @kidroca has a few other higher priority issues

Same as last week ... @kidroca when do you think you can get a PR up to split out vendor code into separate chunk(s)? Seems like it might be reasonably quick and would be good to have this going in parallel

Yeah, I've been putting this off, because it's not a priority
If it's not a problem that someone would have to review/test it, I can squeeze a PR next week

Current load times

simulated slow 3g
incognito

No cache - 60sec.

With Browser cache - 6.53 sec.

Yeah, that is not great. This isn't a high priority so only work on it where you have spare cycles for now but keep me posted and I can review a PR.

With bundle splitting

one main.js file of 1.78 MB
one big vendors.js file of 11.28 MB

Initial load times didn't change. It still takes about 60 sec. to load App on a "slow 3g", but partial caching is now possible - meaning every time we're only making changes to App code, users would have to download only the smaller main.js file
vendors.js would change when we make package.json dependencies changes - updating/adding/removing external libraries including expnesify-common or react-native-onyx

There are other strategies like configuring Webpack to extract a js file for each 3rd party library, but this would only improve partial caching, while load times stay the same.
Webpack maintainers recommend to stay with the built in strategy that creates one vendors.js chunk and instead focus on code splitting (using async import)

this would create more chunks naturally from modularized App code
it promotes a modular and more maintainable App structure

Only code splitting would improve initial load speed for uncached loads

One more thing worth mentioning - the backend seems to be configured to expire .js files in 4hrs
When those 4hrs expire we'll be downloading the whole app again, even if it didn't change
When using Webpack we don't need to configure the server to expire .js files, webpack appends unique hashes based on the script content. We can just serve the .js files and browsers would cache them, when files change they would be served with a different hash and invalidate browser cache

With code splitting (just one chunk) (2x faster initial load)

one main.js file of 1.78 MB
one big vendors.js file of 6.1 MB
one big onfido-sdk.js file of 5.11 MB
"slow 3g" first time loading - 30.8s, loading the following in parallel:

10.76s for main.js
24.33s for vendors.js
n/a onfido - not loaded on the initial page

Onfido SDK is almost 1/2 of the total App size

Before code splitting

With code splitting

This 5.11 MB code chunk is now only loaded when we're at the Onfido step

POC for code splitting
Simple code splitting using async code and dynamic import
https://github.com/Expensify/App/blob/22e0a24177f7a2909866d06d53026c394ab52f68/src/components/Onfido/BaseOnfidoWeb.js#L20-L26
Component mounts and runs async code
Optionally the component can render something until the import() promise resolves
Code splitting using React.lazy
Use lazy import
https://github.com/Expensify/App/blob/429554d3e66e5e403780d404b1ce2910382f1320/src/components/Onfido/index.website.js#L6-L9
Replace a regular import statement with a React.lazy one liner
(Almost) everything about this lazy component would be captured in a separate bundle and loaded on demand
Use <Suspense> when rendering the lazy component
https://github.com/Expensify/App/blob/429554d3e66e5e403780d404b1ce2910382f1320/src/components/Onfido/index.website.js#L26-L37
When the lazy component have not been or is still loading a fallback content can be displayed
Preload demo
This video show App loading in 30 sec on a slow network and only then continues to load the remaining (Onfido) content
(App remains usable in the meantime)
https://user-images.githubusercontent.com/12156624/205067384-2496d7d2-45f6-4b3d-9a0c-51c5d00fb6e8.mp4

the backend seems to be configured to expire .js files in 4hrs

How did you discover that? Maybe I can look into this piece

Onfido SDK is almost 1/2 of the total App size

This is good to know, since we're getting rid of it at some point soon. Internal issue link here

This POC looks good to me, but given that we'll be getting rid of the Onfido SDK soon, I'd say let's keep it as just a POC and only move forward with https://github.com/Expensify/App/pull/13235, because that doesn't really establish a new code pattern that we need broader consensus on

the backend seems to be configured to expire .js files in 4hrs

How did you discover that? Maybe I can look into this piece

Just load/reload new.expensify.com with devtools opened on the Network tab
Optionally enable filtration to only see JS requests
Find the main.js script and inspect response headers:
cache-control: public, max-age=14400
expires: Sat, 03 Dec 2022 00:54:15 GMT
                // now is 21:05:00 GMT

Seems to come from Cloudflare dashboard setting

I've asked someone to help us look into cloudflare's settings to see where that max-age is being set b/c I don't see it anywhere in our other config settings.

hmm though the hash on the main.js bundle seems to always be the same db426a36985fca224c47 - this is currently the hash I get on production and staging and the same thing from your screenshot (at least from what I can see)
prod:

staging:

hmm though the hash on the main.js bundle seems to always be the same db426a36985fca224c47 - this is currently the hash I get on production and staging and the same thing from your screenshot

Interesting, perhaps a new version haven't been released yet
I'll inveterate why staging and prod have the same hash - hash should be based on the content and even though these envs are pretty much the same there are slight differences
Even if the hash for staging and prod is the same as long as it changes when new build is published it should be OK

Yeah that's what I was trying to verify as well, at the very least with local dev when I changed something it generated a new hash

we should have a new deploy today right that gets pushed to staging so we should be able to verify once that's done that the deploy process is properly changing hashes as well?

@bondydaa
Hashes are different for staging and prod, in your comment here: https://github.com/Expensify/App/issues/12235#issuecomment-1337692
You see the file from new.expensify.com - not the one from staging.new.expensify.com

556
I guess you first opened new. then staging.new., this would keep the request in the network history, so when you search for db426a36985fca224c47 - it's the only thing you see
You should instead look for main-

oh lol doh 🤦 good catch

discussing how we want to handle the CF cache setting, I think we can probably just up it to like 30 days or a year (a year is what we have on old dot since we do something similar with hashes for the JS files).
That way we get the benefit of caching but then anytime a new hash is deployed the user will have to redownload it.

discussing how we want to handle the CF cache setting

@bondydaa Should we create a separate issue to handle this in a follow-up?

@aimane-chnaif can you please accept the job and reply here once you have?
https://www.upwork.com/jobs/~019fbb45bb5ac42437

Paid @aimane-chnaif .  This doesn't look like it needs regression test steps added, comment/reopen if anyone disagrees

we upped the cloudflare cache TTL to 1 day for now on https://github.com/Expensify/Expensify/issues/251424.
we can always tweak it more if needed.


Design - Create User Interview Guide

Create a document to guide user interviews to answer any questions we may have.
Questions should aim to extract user behaviors while using the application, and eventually drive prioritization of issues and design decisions down the road.

I'm happy to give this a review once you've got a first version

Was this issue closed by #586 @johnrobbjr?

@alexsoble not yet, that pull request was for adding the document, but the full interview guide has not been written yet. I hope to get to it soon once life gets a bit less hectic!

This issue seems dormant, closing.


useLingui hook implementation

https://github.com/lingui/js-lingui/pull/1859

Any way we can help get this over the line? (sadly I don't have any real Rust expertise)

I also didn't have any Rust expertise but learned specifically for that project. So it's not an excuse 🤪
Honestly i don't have enough motivation / time to continue working on that. Implementing this in Rust is much more complicated, since SWC and Rust ownership model doesn't give the same tools as babel do. So everything related to scopes, traversing, finding references, replacing references you should do on your own inside the Visitor's loop. You could not, for example, find a useLingui statement, get all bindings referencing this statement and transform them in a loop.
I think, it might be a good task for ChatGPT, we have input, we have output, and we simply need to write a transform.

Available in 5.0.0-next.1.


fix(connectors): fix connector patch validation

Description
Fixed connector patch validation by fixing connector spec element name while parsing patch request.
Verification Steps
Updating connector validation test.
Checklist (Definition of Done)

[X] All acceptance criteria specified in JIRA have been completed
[X] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
- [ ] Documentation added for the feature
[ ] CI and all relevant tests are passing
[ ] Code Review completed
- [ ] Verified independently by reviewer
[ ] All PR comments are resolved either by addressing them or creating follow up tasks
- [ ] Required metrics/dashboards/alerts have been added (or PR created).
- [ ] Required Standard Operating Procedure (SOP) is added.
- [ ] JIRA has been created for changes required on the client side

fyi @lburgazzoli @valdar this fixes connector validation.

@lburgazzoli Mind a review?

In reviewing this I have opened two other issues:

https://issues.redhat.com/browse/MGDCTRS-975
https://issues.redhat.com/browse/MGDCTRS-976

Added a couple of tests for good measure.


Refactor: AlignedMemory::resize()

Follow up to #360 and designed in collaboration with @alessandrod.

Renames resize() => fill_write()
Optimizes away memset 0
Adds the flag zero_up_to_max_len to keep track of optimization potential
Renames new => new_with_capacity
Renames new_with_data => new_from
Renames new_with_size => new_zero_filled
Adds new_with_capacity_zeroed

Codecov Report

Merging #368 (4a4d51d) into main (f52bfa0) will decrease coverage by 0.12%.
The diff coverage is 76.19%.

@@            Coverage Diff             @@
##             main     #368      +/-   ##
==========================================
- Coverage   89.88%   89.76%   -0.13%
==========================================
  Files          24       24
  Lines        9000     9019      +19
==========================================
+ Hits         8090     8096       +6
- Misses        910      923      +13

Impacted Files
Coverage Δ

src/aligned_memory.rs
80.40% <72.97%> (-7.20%)
:arrow_down:

src/call_frames.rs
97.80% <100.00%> (ø)

src/elf.rs
91.96% <100.00%> (ø)

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


Enum error: Index signature parameter type must be string or number

Hi,
I have an object that has an Enum as key. The Enum values get eventualy compiled to number in JS.
I'd like to be able to specify this:
var mapping: { [mappingKey: MappingKey]: string; }
(MappingKey is Enum)
This doesn't work and throws compile errors.
Specifying mappingKey: number is not the correct behavior, as I would be able to do this in TS code then:
mapping[5] // this should throw compile error
This is a bit related to this issue:
https://github.com/Microsoft/TypeScript/issues/1415

Duplicate of #2491


METRON-1182: Refactored AlertList View to abstract displaying of individual alerts…

…, this will help us to have different views for displaying individual alerts
Contributor Comments
This PR is an effort to Refactor alert-list-view. If we can abstract display of individual alerts we can plug-in different types of views to display individual alerts.
Testing
The PR doesn't add any new functionality since this is just a refactoring e2e tests should be fine and running to test the PR.
Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[x] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && build_utils/verify_licenses.sh

[x] Have you written or updated unit tests and or integration tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[x] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

Lots of improvements in this PR, I like it.  Spun this up in full dev and everything continues to work.  Nice job.  +1


Linux support

I am working on an LWJGL3 renderer for Ardor3d with the aim to use it with SWT and really need linux support.  I would love to use and potentially contribute to the work you have done so far.  Can you describe the current blockers to having Linux support?

There are no blockers, technically. It's just that I did not yet have the need for a Linux version and haven't had the motivation and time to implement one. You are very welcome to contribute it!

Ah, that is good to hear. :)  I hope to look at that area in a couple
months then.
On Mon, Jul 9, 2018 at 1:37 PM Kai Burjack<EMAIL_ADDRESS>wrote:

There are no blockers, technically. It's just that I did not yet have the
need for a Linux version and haven't had the motivation and time to
implement one. You are very welcome to contribute it!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LWJGLX/lwjgl3-swt/issues/2#issuecomment-403578573,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB1w23QISf5A14Hms9RKdkpuU01hHF9bks5uE6LWgaJpZM4VIMFN
.


MINOR... but changing maxDepth in graphqlconfig.yaml  doesn't trigger an update signal to amplify push

Describe the bug
MINOR, but still worth fixing... When changing the depth using aws config codegen it does not generate a dirty state to know to push the graphql api.
The work around is to just add a space into your schema.graphql to trigger an update to make the amplify push succeed.
To Reproduce
do an amplify push to make sure nothing is pending.
Change the depth using aws config codegen to something different.
do a amplify push and it will say no changes.
Expected behavior
The amplify push should say that API of graphql needs to be updated.
What is Configured?
my graphqlconfig.yaml went from not having a entry for maxDepth to now having maxDepth: 8

@jsgphd Thanks for opening this issue, you could always use amplify push --force to push undetected changes

This is not a bug, there are no API changes needed when depth is changed. This only changes the codegen queries. You can always queries to arbitrary depths regardless of the depth property set for codegen purposes


EMI Calculator

Description 📜
It's a EMI calculator which is made using HTML,CSS,JS . Here are 3 options -->

Loan Amount
interest rate/year
Duration
These 3 will be taken from users.
And all the output will be shown into different Div consisting ->
Monthly Installment
Monthly Interest
Total Repayment
Total Interest Cost

Please include a summary of the change and which issue is fixed.
Fixes #926

Type of change 📝
Please delete options that are not relevant. And in order to tick the check box just put x inside them for example [x] like this

[x] Add a new project

Domain of Contribution 📊

[x] Project using HTML, CSS & JS

Checklist ✅
Please delete options that are not relevant.And in order to tick the check box just but x inside them for example [x] like this

[x] I have performed a self-review of my own code or work.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GSSoC'22 contributor

Screenshots / Gif (Recommended) 📸
https://user-images.githubusercontent.com/77873383/164891388-74ac890e-2275-4f1f-9a51-d951eab22198.mp4

@khushi-purwar , please check


identifier "_Ctype_CvRect" may conflict

Hello,
Thanks for go-opencv!
When I run your hellocv.go example I got this error:
github.com/go-opencv/go-opencv/opencv
../opencv/cvaux.go:66:21: identifier "_Ctype_CvRect" may conflict with identifiers generated by cgo
I am runing go 1.12
Please help,
Thanks

Same issue

Same issue

I'm running it on a mac if that helps, how about you @zmajew and @Cloud-Chen-laofeizhu?

Any news here? @lazywei Would you be so kind to review and, possibly, change attached PR?

Same issue... Any ETA? :)

same problems, help please


Add Alibaba Nacos as a new services discovery backend

Alibaba Nacos is Service Discovery and Management,and more and more chinese company use it.
So i want add nacos as a new istio services discovery backend
Describe alternatives you've considered
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Additional context

This should be done as an MCP server rather than integrating directly into pilot code

This should be done as an MCP server rather than integrating directly into pilot code

could u provider some docs for me to start?

Here is an example WIP for a nacos MCP server https://github.com/nacos-group/nacos-istio and here is one for consul https://github.com/costinm/istio-consul


add content folder + files

see three-dev-tool content folder -- adapt this

start with ThreeDevTools.js

#17 #16


expectPrivate should work without specifying method name

From<EMAIL_ADDRESS>on June 01, 2009 17:16:48
Should work based on parameter types as well (like invokeMethod in Whitebox).
Original issue: http://code.google.com/p/powermock/issues/detail?id=113

From<EMAIL_ADDRESS>on July 02, 2009 06:34:56
Fixed from revision r845
Status: Fixed


Can you support script generation through recording?

What are you trying to achieve?
Ability to generate out-of-the-box tests like Playwright
https://playwright.dev/docs/codegen
By generating the code, then I can edit it directly. without writing down all the code.

nice to have this feature too

I had ideas for that but don't have resources


fix(cli): hard restart with communication channel

🔗 Linked issue
Followup unresolved fixes from #18641
❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Aligning (more) with next version of CLI, this PR improves dev wrapper implementation:

Use fork to open a communication channel
Request hard restart via process.send signaling
Stop the listener and gracefully stop and unallocated nuxt before restarting
Small fixes for restart handling edge cases
Make sure old child is closed before starting new one

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Experimenting with https://github.com/nuxt/nuxt/pull/19394, I'm getting this error:
 ERROR  [uncaughtException] Channel closed                                                                                15:03:45

  at new NodeError (node:internal/errors:393:5)
  at target.send (node:internal/child_process:740:16)
  at packages/nuxi/src/commands/dev.ts:99:21
  at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Following that error, programmatic restarts no longer work.

Can you directly reproduce it in this branch? What is your node version?

I can reproduce reliably after a couple of saves. Reproduction:
git checkout feat/modules-dir
git merge --squash origin/fix/cli-restart
pnpm nuxi dev test/fixtures/basic
# modify test/fixtures/basic/modules/example.ts

Running on Node v18.12.1

Try again with latest commit :)

Seems to be working well 👍


Database: sourceUrl should be in the source table, not the item table.

[x] Update database
[x] Change python layer

Once this is done update url for source will be much easier.
Update: update sources set url = 'http://waitbutwhy.com/feed' where lookupId = '3173';

[ ] Add foreign key constraint for sourceId in Item table
[ ] Change mapping between sources and sourceGroupAssigmnet table to be done by id rather than url.

update sources set url = 'http://www.smbc-comics.com/rss.php' where lookupId = 760;

[ ] Add isAggregator boolean value to sources table


Documentation: error in Document API example code on the web.

The example in http://orientdb.com/docs/master/Document-Database.html#usage has an error:
ODatabaseDocumentTx db = new ODatabaseDocumentTx(url);
db.open("admin", "admin_passwd
------------------------------^ where's the rest of this line?
try {
   db.begin();
   // YOUR CODE
   db.commit();
} finally {
   db.close();
}

Fixed, it will be online in <1h. Thanks.


Touch bar support

Show timer on touch bar
Change slide through touch bar directly

Thanks for the ideas 👍 Unfortunately I don't have a MacBook with touch bar so I'm not sure how easily I can work on this. If anyone lese could help out that would be great.

I think there is a touch bar simulator in Xcode for developers who don't have a MacBook with touch bar !


should pch and lty vary by group?

Personally, I think it would be a bad idea to have them vary automatically. Groups should be identified by one aesthetic dimension whenever possible. Otherwise the plot becomes very distracting.
However I do think it's about important to offer a simple way to change the aesthetic that varies. But we can already do that by setting col=1 and pch=1:ngrp.
This should probably be documented prominently.

Thanks @vincentarelbundock. Totally agree that these shouldn't vary by default. (The issue title isn't clear; my bad.) I was thinking instead of offering a convenience way for users to opt into this. I have a PR incoming shortly that I'll tag you as a reviewer for to look at.


Moar modules

Follow up of #84, make sure that's merged first.
This extracts all app features into their own component, leaving the main app module trim and with a quickly identifiable manifest of features. This makes testing and development generally easier.
Fixes #71

Built the app with these changes and all works well.

I'm going to merge this now so not to have a long-running branch, with the intention to add some automated tests immediately following (unit or e2e).


Refactoring of the REST-Update method. [WIP]

This PR addresses the REST-PUT interface.

[x] The PUT-Interface should be able to work with partially changes of an object.

Application sampledata 1:
{
  "id": 57,
  "name": "Peter"
}

[ ] The interface should be able to work with nested data.

The current state (e62227e) can only handle changes at the top level. Changes to childproperties e.g. the name of the viewport in an application leed to an error when data does contain an id (sampledata 2) or an empty child if data doesn't contain id (sampledata 3).
Application sampledata 2:
{
  "id": 57,
  "name": "Peter",
  "viewport": {
    "id:" 56,
    "name": "Silke"
  }
}

Application sampledata 3:
{
  "id": 57,
  "name": "Peter",
  "viewport": {
    "name": "Silke"
  }
}

If we can solve this problems the update method would be really nice.

The solution for partial updates on the top level seems very nice to me!
But i am not sure if we really want to allow the updating of nested objects/collections as this would (currently) only work if the nested object is annotated with @Cascade(CascadeType.SAVE_UPDATE). As we are talking about the AbstractRestController here, we cannot assume that all nested objects (of arbitrary entities) are annotated like that.
To update the viewport of an application, wouldn't it be a more RESTful approach to PUT
"viewport": {
    "id:" 56,
    "name": "Silke"
}

against something like
http://localhost:8080/{{shogun2-app}}/rest/applications/{appId}/viewport

or
http://localhost:8080/{{shogun2-app}}/rest/applications/{appId}/viewport/{{vpId}}

This would lead to an implementation in the ApplicationRestController. Client applications would then possibly have to fire multiple PUT requests...

I just added an enhancement of the current state and fixed the tests

But i am not sure if we really want to allow the updating of nested objects/collections as this would (currently) only work if the nested object is annotated with @Cascade(CascadeType.SAVE_UPDATE)

We allready have this annotation on nearly every property e.g.:
https://github.com/terrestris/shogun2/blob/master/src/shogun2-core/shogun2-model/src/main/java/de/terrestris/shogun2/model/Application.java#L82-L83
https://github.com/terrestris/shogun2/blob/master/src/shogun2-core/shogun2-model/src/main/java/de/terrestris/shogun2/model/map/MapConfig.java#L70-L71
I would appreciate to be able to update nested objects/collections with only one PUT request. We could add errorhandling if the annotation on an edited property is missing.
The adavantage would be the you could easily update applications from the Client side.
This might be an disadavantag too, as you could even easily send a malformed json the corrupts your whole data.
Would be cool if some other devs could submit there opinion on this. @marcjansen @weskamm @dnlkoch @chrismayer

We allready have this annotation on nearly every property

Yes, but these annotations are only there because it was easy and fast (-> saved code) when we were developing the initialization service. I dont think that it is best practice (e.g. regarding performance) and the Cascade annotations should only be used/configured if we have a good reason for that. (Which means on the other side that these annotations should be removed from the code someday)
Its a hard topic and i dont have a strong meaning on this. But if you really want to update all nested elements, i dont think that we will get this running with the changes contained in this PR

I'm done from my side. Any more comments? @buehner

As i dont see any conflicts here and as far as i know its already working fine in our latest project, i am going to merge this one..


Add redirect for single non-conforming URL

This PR:

Adds a redirect for the single legacy Blogger blog post whose path on the current site is non-conforming.

/fyi @kbarnard10, @idvoretskyi

/assign
/approve
/hold
/lgtm
@zacharysarah wasn't sure if you wanted to wait to merge this, or if it was ready to go - so pre-approved, and we can drop the hold whenever you're ready to see it live

Thanks @heckj!
/hold cancel


Cleans up ClothingShoesBootsMagSyndie in magboots.yml

About the PR

uh... For some reason the ClothingShoesBootsMagSyndie had an Explosive component and there does not seem to be a way to set it off
Why / Balance

:)
Technical details

remove the Explosive component from ClothingShoesBootsMagSyndie
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

no cl

They have hidden explosives so hard to find you can't even activate them

i think its because it can be made into a maxcap like jetpacks/gas tanks

i think its because it can be made into a maxcap like jetpacks/gas tanks

mmm... ok... that makes sense. I checked the jetpack yml but forgot to look at its children...

i believe it can also inherit magboots just fine to remove even more duplication

mmm... I broke something...

i believe it can also inherit magboots just fine to remove even more duplication

It would remove the tag duplication, that's about it. I'm more nervous about doing it for the magboots because it has the Appearance + GenericVisualizer components and I'm not sure what they would affect

validboots dont have jetpack visuals its just copy pasted magboot visuals

validboots dont have jetpack visuals its just copy pasted magboot visuals

the bloodred mags don't have an item sprite attached to them, so they inherit the sprite from the parented base jetpack, I found a fix so its whatever

Mmm... I need to do more cleanup on this file...


CA-245333 (api-version autogeneration); plus some small refactoring

This PR depends on thexs-opam repo having the mustache package.

A new xs-opam has been released. I think it is worth to add a travis job also for the sdk. You can probably simply copy any generic xapi one, like https://github.com/xapi-project/xcp-rrd/blob/master/.travis.yml

Apart from the files that are autogenerated completely from scratch, the SDK includes several "hardcoded" files where there is only a small number of variables. The values of the latter are taken from the datamodel, so it makes sense to move the subsitution within the OCaml code instead of defining these values manually and updating them for every release (experience has shown it's error prone).

Yes, it makes perfect sense. I wasn't being critic, I quite like the approach

The commit that has just arrived is the final installment on this ticket.
This PR has to be merged together with https://github.com/xapi-project/xen-api/pull/3038 to prevent a build breakage.

@mseri the link https://github.com/xapi-project/xen-api/pull/3038 has been approved. If all ok here we can go ahead with merging.

Go ahead!