fix action manager disposal when shared

Related forum issue: https://forum.babylonjs.com/t/how-do-you-dispose-the-actions-of-an-actionmanager/9985/7
PG to test: #J19GYK#567

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14267/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14267/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14267/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14267/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14267/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14267/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14267/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14267/merge#BCU1XR#0


Crash?

Hi
didnt realize that this is why it´s propably often not working.
Also the iPhone is often not triggering the "apple value add" thing which triggers the HomeKey to be presented.
Does this panic does help in any directon?
`I (3797) HKAuthCtx: Ephemeral Key generated
E (4025) HKAuthCtx: Response Status not 0x90, something went wrong!
Guru Meditation Error: Core  1 panic'ed (LoadProhibited). Exception was unhandled.
Core  1 register dump:
PC      : 0x400d45de  PS      : 0x00060d30  A0      : 0x800e557e  A1      : 0x3ffc42a0
A2      : 0x3ffc4418  A3      : 0x3ffc4458  A4      : 0x00000001  A5      : 0x00000000
A6      : 0x00000fb9  A7      : 0x3f405164  A8      : 0x800d45da  A9      : 0x3ffc4280
A10     : 0x00000000  A11     : 0x3ffc42fc  A12     : 0x00000004  A13     : 0x3ffc4320
A14     : 0x3ffc42f0  A15     : 0x3ffc434c  SAR     : 0x0000001b  EXCCAUSE: 0x0000001c
EXCVADDR: 0x00000001  LBEG    : 0x400014fd  LEND    : 0x4000150d  LCOUNT  : 0xfffffffd
Backtrace: 0x400d45db:0x3ffc42a0 0x400e557b:0x3ffc4400 0x400f563f:0x3ffc45a0 0x400f5747:0x3ffc4670 0x400d4d76:0x3ffc4690 0x4011791d:0x3ffc46b0
#0  0x400d45db:0x3ffc42a0 in AuthenticationContext::fast_auth(bool) at src/auth/authContext.cpp:415
#1  0x400e557b:0x3ffc4400 in LockMechanism::loop() at src/main.cpp:179
#2  0x400f563f:0x3ffc45a0 in Span::pollTask() at .pio/libdeps/wemos_d1_mini32/HomeSpan/src/HomeSpan.cpp:278
#3  0x400f5747:0x3ffc4670 in Span::poll() at .pio/libdeps/wemos_d1_mini32/HomeSpan/src/HomeSpan.cpp:164
#4  0x400d4d76:0x3ffc4690 in loop() at src/main.cpp:871
#5  0x4011791d:0x3ffc46b0 in loopTask(void*) at /Users/thorsten/.platformio/packages/framework-arduinoespressif32/cores/esp32/main.cpp:50
ELF file SHA256: 30765105d23cddf3
Rebooting...
ets Jul 29 2019 12:21:46
`

Hi,
The crash occurred due to an oversight from my part, it will be fixed, however, it only got there because the device didn’t agreed with what was sent as can be seen by the error before the crash.
Could you please provide a debug level log by using @LD to get more context? Also do you have express mode on? And could clarify what is the "apple value add" thing?

Thanks.
I will create the logs later
With "apple value add" thing i meant the VAS and ECP
=> https://github.com/kormax/apple-vas
=> https://github.com/kormax/apple-enhanced-contactless-polling

or in other words: the home-key didnt show up in many cases. Sometimes it showed up - but didnt authenticate successful... and rarely i was able to authenticate.

Ill post fresh logs later. Thanks for all the effort!
Best
Thorsten

Ok, great.
So is this also what was happening a couple days ago as mentioned in #5 ?
From the behavior you are describing, there might possibly be an issue with the PN532 unit, consider trying another one, however, you might know that there’s an issue with PN532 clones that use cheap resistors and stuff, kormax also talked about this in his ECP repo.
Unfortunately, the library for the PN532 does not currently implement a way to identify if there is an RF I/O issue like it is with nfcpy in Python which is something I’ll deal with at some point among other things.

Some updates:
I've replaced the PN532 - and it works much better.
With the watch almost every time - on my phone i was still struggling to identify the HomeKey and showed "PASSIVE TARGET SELECTED". After a while i figured out i had the express mode also activated for a travel card (Had "Clipper card" enabled - https://www.clippercard.com/ClipperWeb/apple-pay ). After disable express mode for Clipper card i have the same high success rate on the iPhone.
I have still here and there
I (1041693) HKAuthCtx: Ephemeral Key generated E (1041817) HKAuthCtx: Response Status not 0x90, something went wrong! W (1041875) LockMechanism: Authentication Failed, lock state not changed
But also a good success rate.
Do you need the same @LD logs as before - or do you see the same behave on your side?
MQTT does also works fine... (a BIG thank you!)  so I'm very close to replace my existing "stupid" NFC reader at my door with this new one :-) :-)
Best
Thorsten

Awesome!
If things work for you now, no logs are needed anymore also I haven’t been able to replicate this behaviour so you might still have some issues with the module, faulty connections maybe?


Add sigalg/elliptic curve tests with s_client

This change adds integration tests for supported signature
algorithms and supported elliptic curves. I noticed that we
could just use s_client for all of our Openssl client tests.
The python ssl handshake tests have been removed and ported
to use s_client.

The diff looks really nasty in unified view. Split view is more reasonable.


Merge valuesyml

Description

This PR updates our approach to the generation of values.yaml from versions.yaml.
Before, we were appending it to the end of values.yaml, which contains the base settings. But this approach means the step is not idempotent.
This PR moves the base settings from values.yaml to a new base_values.yaml file. The ruby script reads that file, combines it with versions.yaml, and writes it to values.yaml.
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit 9bdfc65be36ca8fce6e5a58738b71573dd3802f6
https://deploy-preview-2536--calico.netlify.com

Closing since we decided to put our values inlib.rb


⚠ SceneTime has degraded performance

In a366713, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1891 ms

Resolved: SceneTime performance has improved in 53997e7 after 9 minutes.


Write a Python program to print number to words in indian numbering system

Description
Write a Python program to print number to words in indian numbering system
Input  : 123456789
Output : One Hundred Twenty Three Crore Forty Five Lakh Sixty Seven Thousand Eight Hundred Ninety

How to contribute

Save the solution in print-number-to-words-in-indian-numbering-system.py file
Add print-number-to-words-in-indian-numbering-system.py file in print-number-to-words-in-indian-numbering-system folder

please assign to me

!assign


Use a sensible license

Wikipedia says:

The license was approved as a GPL-compatible free software license by the Free Software Foundation but its use is “not recommended.”

Please relicense your code under a better open source license. Alternatively, you could put the code in the public domain.

reconsider sensible.

I know you closed this, but I'd like to add arguments to why I think this request makes sense and I would like to +1 it.
While the WTFPL license would be perfectly fine for some jurisdictions, it is very much akin to "public domain" which in many countries simply does not exist or is not a valid contract.
The vague wording and lack of warranty disclaimer unfortunately make it explicitly forbidden to use WTFPL-licensed software in many corporate environments. If you want your work to be truly universally available for reuse please, please reconsider using one of the OSI-approved, permissive, copyfree licences like BSD, MIT, Apache 2.0 or even CC BY.
I'd love to use your lib, but unfortunately cannot due to WTFPL :(


Add E2E test support for ERC20/ERC721

Depends on https://github.com/hyperledger/firefly-tokens-erc20-erc721/pull/33

I attempted to add ERC20/ERC721 tokens to the E2E matrix for GitHub PRs - will see if it works and runs here. Open to whether this is needed or not (obviously we only want to run a subset of all combinations on every PR).

Codecov Report

Merging #611 (6dc62e6) into main (2e464e7) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #611   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          305       305
  Lines        18196     18266   +70
=========================================
+ Hits         18196     18266   +70

Impacted Files
Coverage Δ

internal/database/sqlcommon/tokenpool_sql.go
100.00% <100.00%> (ø)

internal/events/token_pool_created.go
100.00% <100.00%> (ø)

internal/tokens/fftokens/fftokens.go
100.00% <100.00%> (ø)

internal/database/sqlcommon/chart_sql.go
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

And there's a failing E2E test 😕

It's weird that Fabric + ERC1155 even executes (and passes?)... does it just ignore the token provider? I know it's definitely not actually using it.
And yea the failures in ERC20 are expected until the dependent PR noted is merged and tagged in the manifest.

Opened https://github.com/hyperledger/firefly-cli/issues/162 to track the fact that -t is silently ignored by Fabric stacks.


[198282] Set Application ID in nodes

This PR includes the app id property in participant QoS to identify the SustainML nodes as part of the SustainML framework in the Fast DDS Monitor.
This PR MUST be merged right after:

#25

@mergifyio rebase


Book Unit testing

Hello 👋 @AmanWorku, In this milestone;

Added unit test for the Book method
Added Unit test for the Label method.

Thank you for reviewing this pull request 💯


Registering a DAQmx callback after a task has started crashes the gRPC server and prevents future connection

In testing out various things related to https://github.com/ni/grpc-device/discussions/398, I discovered this unfortunate bug. If you attempt to register a DAQmx callback after the task has already been started, the gRPC server crashes.
I've attached two example python scripts (using betterproto, though it seemingly shouldn't matter what the client is). One script which works daqmx_register_working.py (registering the callback happens before starting the task) and one which causes the crash described above  daqmx_register_crash.py (the task is started and underway when the registration is attempted). The files have txt extensions since GitHub doesn't allow for uploading .py files.
The key part / difference is the logic near the bottom. This excerpt is from the crash version:
        print("Starting task")
        await raise_if_error(await daq_service.start_task(task=task))

        print("Sleeping")
        await asyncio.sleep(2)

        print("Trying to register callbacks")
        # THIS WILL FAIL AND CAUSE THE GRPC SERVER TO CRASH
        acquisition = asyncio.gather(read_data(), wait_for_done())
        await acquisition

When running the "crash" script, the client will eventually see:
Starting task
Sleeping
Trying to register callbacks
ERROR: Connection lost
...traceback...

And the gRPC server will no longer be running (seemingly having crashed). Further attempts to connect to the server will fail until it's restarted manually. We didn't see any logs or messaging on the gRPC server side. We're running the server executable on Windows 10.
Perhaps registering a callback after a task has started is not expected to work, but it seems the server should return an error to the client and avoid crashing, since this takes down the full gRPC functionality and requires human intervention to restart.
Thanks in advance for any help/attention. Really appreciate the work on the NI gRPC functionality!

This bug is fixed in DAQmx for the next release (not sure when that is scheduled).

Awesome, thanks astarche!


swarm/update should process the query parameters

Now swarm/update accepts version, rotateWorkerToken and rotateManagerToken.

:+1:
Will ping when published :)

Published in v2.3.1


finfo::finfo(): Failed to load magic database at '(null)'

finfo::finfo(): Failed to load magic database at '(null)'
Just to say I'm aware and again, it's not my fault... So previews of images won't work. A fix is to remove php7.1-fileinfo.

Temporary fix : https://github.com/Wonderfall/dockerfiles/blob/master/nextcloud/11.0/Dockerfile#L77-L79

Issue :
https://bugs.alpinelinux.org/issues/6945


MIssing export property

This is not an issue, per se, but I'm trying ti understand a behaviour.
If I create a custom object and add it to an empty array, then add a similar object with an additional property, then export from export-excel, the additional property is not exported.
If on the other hand, I add an object with the property to an empty array, then add one that doesn't have it, the object missing the additional property simply export it as empty.
How are the exported properties determined?

https://github.com/dfinke/ImportExcel/blob/master/Export-Excel.ps1#L665
Works similar to the PS object flow engine.
[PSCustomObject]@{a=1;b=2}
[PSCustomObject]@{a=3;b=4;c=6}

Prints
a b
- -
1 2
3 4

Update-FirstObjectProperties is there to fix this.
The properties of the first object set the headings for the sheet.

Excellent!  Thank you
\Greg

From: jhoneill<EMAIL_ADDRESS>Sent: Tuesday, November 5, 2019 6:21:38 PM
To: dfinke/ImportExcel<EMAIL_ADDRESS>Cc: uSlackr<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dfinke/ImportExcel] MIssing export property (#715)
Update-FirstObjectProperties is there to fix this.
The properties of the first object set the headings for the sheet.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/dfinke/ImportExcel/issues/715?email_source=notifications&email_token=AGEO6ZY5MKWXKA5BCLUDPRTQSH5YFA5CNFSM4JJIKBXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDEWBVY#issuecomment-550068439, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGEO6Z4P5VNS2743D7WAJLTQSH5YFANCNFSM4JJIKBXA.


Closing client socket with auto reconnect enabled

When closing client socket with 'reconnect' enabled using socket.end() scheduled reconnect function will crash the application (~line 322 in nssocket.js, when removing listeners from socket).
If I try socket.destroy() reconnect function would recreate the internal socket and connect to server, but it shouldn't, application requested closing of socket.

any chance you could provide us a snippet of the code?

Currently I'm working on a big project with really short deadline, that's how I ran into it, but I'll make a test code when I'll get some time (probably in few days). Just a quickie how to reproduce:

Create a nssocket with 'reconnect' option enabled and I believe it was type set to 'ipv4' (just like in example)
Let it connect
Call socket.end() - this will throw a exception after some time
OR - Call socket.destroy() - this will reopen the internal socket after some time

As I said earlier I'll write some code when I'll get some time, if it'll still be necessary.
I got around that issue by writing custom reconnect logic on application level.

Try putting an on 'error' handler like below
socket.on('error', function (err) {
  log.error('client error', err);
});

@enbyted any updates? tnx!

Thanks for the ping, I forgot about that in the mass of work lately, here is a failing test code:
var net = require('net'),
    nssocket = require('nssocket');

net.createServer(function (socket) {
  console.log('client connected');
}).listen(8345);

var socket = new nssocket.NsSocket({
  reconnect: true,
  type: 'tcp4',
});

socket.on('start', function () {
  console.dir('start');
});

socket.on('error', function (err) {
  console.log('client error', err);
});

socket.connect(8345);
setTimeout(function() {
	socket.end();
}, 250);

So, that will crash with:
<workspace>/node_modules/nssocket/lib/nssocket.js:323
    self.socket.removeAllListeners();
                ^
TypeError: Cannot call method 'removeAllListeners' of null
    at doReconnect (<workspace>/node_modules/nssocket/lib/nssocket.js:323:17)
    at tryReconnect [as _onTimeout] (<workspace>/node_modules/nssocket/lib/nssocket.js:351:5)
    at Timer.listOnTimeout [as ontimeout] (timers.js:112:15)

If you change the socket.end() to socket.destroy() it will not crash, but it'll reconnect without calling on.start (you'll see on program's output one 'start' ond two 'client connected')
So, there is no way to close an autoreconnect socket.
Also, I'm using version '"0.5.3" from NPM repo, I think that's the newest.

+1 for merging this. @enbyted can you make a pull request?

@nucleardreamer There already is one (https://github.com/foreverjs/nssocket/pull/34).

Oh awesome, thank you for writing that!


Axes fixed for 8BitDo N30 Pro 2 / Windows

Mappings created with latest firmware version (v6.21) for 8BitDo N30 Pro 2 and SDL 2.0.10.
The axes are weird on Windows: leftx is a0, lefty is both a1 and a2 (same value on both axes),
rightx is a3, and righty is both a4 and a5 (same value on both axes).
I thought this was a bit weird, so double-checked with Steam gamepad configuration, and got the same result.
(I get some strange results on Linux and macOS as well, but there, dpad left/right triggers the same axes (in addition to dpad movement) as left/right triggers (in addition to buttons for the triggers. So on all platforms, the device behaves a bit strange but not equally so. Could be buggy firmware causing undefined behavior or something)

I was a bit interested in figuring out the strange behavior (since the axis duplication didn't happen on Linux and macOS). I found the problem, and explained it here https://discourse.libsdl.org/t/duplicated-joystick-axes-with-sdl-dinputjoystick-and-8bitdo-n30-pro-2-with-suggested-fix/27247 along with a suggested fix for SDL, in case you're interested.
(The "strange behavior" on Linux and macOS where the dpad left/right and triggers shares axes are actually a firmware problem not OS/SDL, I debugged the HID reports being sent from the device).


when creating an RStudio user, also create user in middle

migrated to tryagainconcepts/detalytics-middle#147


3/04 PM Publish

Docs Build status updates of commit dbd3bf9:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-conceptual/azps-5.5.0/release-notes-azureps.md
:warning:Warning
View (azps-5.5.0)
Details

docs-conceptual/azps-0.10.0/release-notes-azureps.md
:warning:Warning
View (azps-0.10.0)
Details

docs-conceptual/azps-5.6.0/release-notes-azureps.md
:warning:Warning
View (azps-5.6.0)
Details

docs-conceptual/azps-2.8.0/release-notes-azureps.md
:warning:Warning
View (azps-2.8.0)
Details

docs-conceptual/azps-3.8.0/release-notes-azureps.md
:warning:Warning
View (azps-3.8.0)
Details

docs-conceptual/azps-4.8.0/release-notes-azureps.md
:warning:Warning
View (azps-4.8.0)
Details

azps-5.4.0/Az.ADDomainServices/Az.ADDomainServices.md
:white_check_mark:Succeeded

azps-5.4.0/Az.ADDomainServices/Get-AzADDomainService.md
:white_check_mark:Succeeded

azps-5.4.0/Az.ADDomainServices/New-AzADDomainService.md
:white_check_mark:Succeeded

azps-5.4.0/Az.ADDomainServices/New-AzADDomainServiceForestTrust.md
:white_check_mark:Succeeded

azps-5.4.0/Az.ADDomainServices/New-AzADDomainServiceReplicaSet.md
:white_check_mark:Succeeded

azps-5.4.0/Az.ADDomainServices/Remove-AzADDomainService.md
:white_check_mark:Succeeded

azps-5.4.0/Az.ADDomainServices/Update-AzADDomainService.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Add-AzEnvironment.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Az.Accounts.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Clear-AzContext.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Clear-AzDefault.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Connect-AzAccount.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Disable-AzContextAutosave.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Disable-AzDataCollection.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Disable-AzureRmAlias.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Disconnect-AzAccount.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Enable-AzContextAutosave.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Enable-AzDataCollection.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Enable-AzureRmAlias.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Get-AzAccessToken.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Get-AzContext.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Get-AzContextAutosaveSetting.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Get-AzDefault.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Get-AzEnvironment.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Get-AzSubscription.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Get-AzTenant.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Import-AzContext.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Invoke-AzRestMethod.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Register-AzModule.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Remove-AzContext.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Remove-AzEnvironment.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Rename-AzContext.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Resolve-AzError.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Save-AzContext.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Select-AzContext.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Send-Feedback.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Set-AzContext.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Set-AzDefault.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Set-AzEnvironment.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Accounts/Uninstall-AzureRm.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Advisor/Az.Advisor.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Advisor/Disable-AzAdvisorRecommendation.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Advisor/Enable-AzAdvisorRecommendation.md
:white_check_mark:Succeeded

azps-5.4.0/Az.Advisor/Get-AzAdvisorConfiguration.md
:white_check_mark:Succeeded

This comment lists only the first 50 files in the pull request.

docs-conceptual/azps-5.5.0/release-notes-azureps.md

Line 2955, Column 1: [Warning-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
Line 4195, Column 1: [Suggestion-sensitive-language-wl] Term 'whitelist' is sensitive and should not be used in content or code. Usage: '* Added new sizes to the whitelist of VM sizes for which accelerated networking will be turned on when using the simple param set for '

docs-conceptual/azps-0.10.0/release-notes-azureps.md

Line 1281, Column 1: [Warning-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
Line 2521, Column 1: [Suggestion-sensitive-language-wl] Term 'whitelist' is sensitive and should not be used in content or code. Usage: '* Added new sizes to the whitelist of VM sizes for which accelerated networking will be turned on when using the simple param set for '

docs-conceptual/azps-5.6.0/release-notes-azureps.md

Line 3048, Column 1: [Warning-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
Line 4288, Column 1: [Suggestion-sensitive-language-wl] Term 'whitelist' is sensitive and should not be used in content or code. Usage: '* Added new sizes to the whitelist of VM sizes for which accelerated networking will be turned on when using the simple param set for '

docs-conceptual/azps-2.8.0/release-notes-azureps.md

Line 502, Column 1: [Warning-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
Line 1742, Column 1: [Suggestion-sensitive-language-wl] Term 'whitelist' is sensitive and should not be used in content or code. Usage: '* Added new sizes to the whitelist of VM sizes for which accelerated networking will be turned on when using the simple param set for '

docs-conceptual/azps-3.8.0/release-notes-azureps.md

Line 1376, Column 1: [Warning-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
Line 2616, Column 1: [Suggestion-sensitive-language-wl] Term 'whitelist' is sensitive and should not be used in content or code. Usage: '* Added new sizes to the whitelist of VM sizes for which accelerated networking will be turned on when using the simple param set for '

docs-conceptual/azps-4.8.0/release-notes-azureps.md

Line 2294, Column 1: [Warning-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
Line 3534, Column 1: [Suggestion-sensitive-language-wl] Term 'whitelist' is sensitive and should not be used in content or code. Usage: '* Added new sizes to the whitelist of VM sizes for which accelerated networking will be turned on when using the simple param set for '

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


CaptionManagement does not exist or replaced by other Codeunit

[Enter feedback here]
In latest version of BC i am not able to find CaptionManagement codeunit.

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

ID: 256aabb4-8d5d-cc06-6c1f-5558c668491b
Version Independent ID: 51d1427c-130d-b5c0-3d04-89c01caa89f8
Content: Event Types - Business Central
Content Source: dev-itpro/developer/devenv-event-types.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Hello, Thanks for the feedback.
These events were moved to codeunit 42 "Caption Class" of the System Application in BC 2019 release wave 2. You can this in Breaking Changes.
The article will be updated. I hope this answers your question.


Rework UVM package to use vm package abstractions

Rework the UVM package to make use of the abstractions around virtstack interactions.
Signed-off-by: Daniel Canter<EMAIL_ADDRESS>

@jstarks After rework galore, going to leave this as a draft until I can get some stuff answered regarding some of the below and also until I run our test suite 100 times with this 😄.
There was some recent changes that after rebasing were a pain to handle. Most of the annoyances are in wcow lol. The main pain point with this is that we still "support" the internal gc connection (the HCS managed one). Right now this won't work at all if you wanted to use the internal guest conn
Here's some of the ugly that if we can agree that we don't see a need to keep around the internal conn can get rid of:

https://github.com/microsoft/hcsshim/pull/1037/files#diff-897d0148db083dffc20d984ea4d9aa1dc4f2dd4b36b1bf7c61952e6cb51f5da6R296-R309
For this it was just some branching logic to grab the gcs capabilities through HCS itself, not sure how that would fit into the UVM interface.

Then launching a container in the UVM: https://github.com/microsoft/hcsshim/pull/1037/files#diff-897d0148db083dffc20d984ea4d9aa1dc4f2dd4b36b1bf7c61952e6cb51f5da6L270-R308
Gross. There's probably a better way to handle that but I left it in just to show what can be avoided if we can just always assume we have a conn in-proc.

I think we kept the fallback to the internal conn originally in case there was any issues when swapping to external, but I don't think we've seen any issues since the swap so.. 😬
Here's the biggest thing I want feedback on: https://github.com/microsoft/hcsshim/pull/1037/files#diff-25e9fbfbd912fbf50e566d845edbc79a704c57df49456d8c1c6e2e6c011d8506R8
For the builder (I don't think the same pattern can work for runtime adds) what do you think about building up the document using this pattern of "WithX" and then applying them in Create?
I'd originally made the CreateOpt type to handle all of the weird "what would this even look like on the UVM interface" HCS stuff like registry additions, some extra things you need to set if the VM is going to be saved as a template etc. but honestly this pattern might be better suited to create the entire document.
For instance, instead of:
 if uvm.scsiControllerCount > 0 {
	scsi, ok := uvmb.(vm.SCSIManager)
	if !ok {
		return nil, errors.Wrap(vm.ErrNotSupported, "stopping SCSI setup")
	}
	for i := 0; i < int(uvm.scsiControllerCount); i++ {
		if err := scsi.AddSCSIController(uint32(i)); err != nil {
			return nil, errors.Wrap(err, "failed to add scsi controller")
		}
	}
}

if uvm.vpmemMaxCount > 0 {
	vpmem, ok := uvmb.(vm.VPMemManager)
	if !ok {
		return nil, errors.Wrap(vm.ErrNotSupported, "stopping VPMem setup")
	}
	if err := vpmem.AddVPMemController(uvm.vpmemMaxCount, uvm.vpmemMaxSizeBytes); err != nil {
		return nil, errors.Wrap(err, "failed to add VPMem controller")
	}
}

you can have:
if uvm.scsiControllerCount > 0 {
	for i := 0; i < int(uvm.scsiControllerCount); i++ {
		cOpts = append(cOpts, vm.WithSCSIController(uint32(i)))
	}
}

if uvm.vpmemMaxCount > 0 {
	cOpts = append(cOpts, vm.WithVPMemController(uvm.vpmemMaxCount, uvm.vpmemMaxSizeBytes))
}

and the With functions can do the typecasting inside and just call the corresponding method
func WithSetSerialConsole(port uint32, listenerPath string) CreateOpt {
	return func(ctx context.Context, uvmb UVMBuilder) error {
		serial, ok := uvmb.(SerialManager)
		if !ok {
			return vm.ErrNotSupported
		}
		return serial.SetSerialConsole(port, listenerPath)
	}
}

Ok after consulting with the team, seems like we're all fine with axing the internal connection completely. huzzah, things get prettier

Just going to draft this, this is so out of date and has fallen off the priority list.

Closing to declutter the PR list view. Will revisit when it's back on the priority list


Captcha doesn't validate letters in Images

Hello.
People of QA in my proyect detect that this kind of captchas (letters in images) doesn't get validated as well as other kind of captchas. they could write anything and the captcha pass. We want to know if is an Issue with you or with google.

weird.
My code just wraps net.tanesha.recaptcha.ReCaptchaImpl:.isValid for validation.
Maybe something changed at google.

I close this issue, because a newer version is available with Recaptcha 2.0 support.


Python Debug Console doesn't start in conda environment

I'm working with remote docker with python. The launcher fails to open python debug console when setting conda environment while python environment works fine.

/extPython


Fix for ES6 Js Use to Import

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #41528

@CoolBlueDesigns Why did you create another PR?  I left instructions on your other PR on how to resolve.  Anyway, you can close the other PR since you have this one now.

@RandellDawson I have closed my first one. Thank you for the advice.


Fix platform-specific build failure due to missing libpthread [phab:D1361]

Created by j3parker at 2017-02-22 13:35:23:
Add -lpthread to glog and gflag linker options and add Jacob Parker to
AUTHORS and CONTRIBUTORS

j3parker wrote at 2017-02-22 13:38:15:
Corresponding change in gflags HEAD: https://github.com/gflags/gflags/pull/189
Aside: would it be worth changing to using the native Bazel support for gflags via a git_repository rule?

shahms wrote at 2017-02-22 16:19:45:

! In (earlier comment), j3parker wrote:
Corresponding change in gflags HEAD: https://github.com/gflags/gflags/pull/189

Aside: would it be worth changing to using the native Bazel support for gflags via a git_repository rule?

Assuming the upstream BUILD file actually works now (they didn't previously, hence the comment in the WORKSPACE file), it might be worth switching to using http_archive directly.  git_repository is not the recommended way of importing external repositories from github (or any other provided which can provide stable archives) as they unnecessarily download all of the commit history.

j3parker wrote at 2017-02-22 23:25:10:
Cool, thanks for the context and the tip RE: git_repository

j3parker wrote at 2017-03-01 23:46:29:
Ping :)

shahms wrote at 2017-03-01 23:50:13:
Sorry, still listed as "Changes Planned", was awaiting those changes.

shahms accepted this revision at 2017-03-01 23:50:13

j3parker wrote at 2017-03-01 23:56:29:
Ah, my bad. I think I set that accidentally (new to Phabricator/arc)

j3parker wrote at 2017-03-03 15:09:13:
Anything else I need to do? I'm not sure how to remove "Changes Planned". As a random guess I'm setting the action on this comment to "Request Review" :P
EDIT: huzza!

shahms wrote at 2017-03-03 15:28:41:
I've landed this, thanks!

shahms closed via commit 18bbadc2be29962bdfb631879e8824dda6cc3412 at 2017-03-03 15:29:02


Move Wiki from .github render-engine

for two reasons.

I originally added it to .github as I thought that would make it available in the org main. Since it doesn't it doesn't make sense to be added to it there.

I keep looking for it there anyway and it would help if it were in the root.

@kjaymiller What do you mean by making it in the root? where do you want to move it? here in the https://github.com/render-engine/render-engine repo or somewhere else?

I've been trying to figure out the best place for the greater ecosystem to live.
My natural inclination is some render-engine site that people can contribute to, but that's another thing to manage.
I had a wiki in the .github repo that I've been adding things to (Probably outdated now) and was saying it didn't make sense for it to be in the .github repo because folks are looking there. They're likely going to start in the render-engine repo so it would be better if the wiki was there.
Of course this may be a non-issue as again I'm not confident that the wiki is the long-term solution.

@kjaymiller can you enable wikis from the setting of this repo so I can move it here?

fixed by #569


Extract marriage-abroad payment information from templates

Trello card: https://trello.com/c/K2bUzopZ
The Trello card is about removing payment information and links to consular fees from outcomes that don't display services/fees.
This branch removes the payment information but not the links to consular fees. I plan to deal with that in a separate branch.
Overview of the changes in this branch:

Add full set of ceremony countries to regression test data. This allows me to use the marriage-abroad-countries-leading-to-outcome.rb script to determine the countries that might be affected by a change to the outcome template.
Extract the payment information from the templates to the marriage_abroad_services.yml file. Update the regression test artefacts where this extraction results in changes to the outcomes.
Introduce the _payment_for_services partial to display the relevant payment information for the ceremony country.
Move payment partials to the payment_information_partials directory.

I would've preferred to store a set of acceptable payment methods in the YAML file (rather than the partial name) but that doesn't seem possible at the moment given how varied the payment information can be.
I've only added the payment information to the ceremony countries that already have entries in marriage_abroad_services.yml (i.e. the services/fees section is displayed for at least some of the outcomes). This ensures that we're no longer displaying the payment information on outcomes where the services/fees section isn't displayed.
While working on this branch, I used the complete set of regression test artefacts for marriage-abroad so that I could sanity check the changes I was making. I've removed these from the commits now that I'm happy with the changes that have been made.
This branch has exposed a couple of other issues.

Outcomes for a number of ceremony countries display services/fees but no payment information.
Outcomes for a number of ceremony countries were displaying different payment information depending on the type of ceremony. I think (hope) this is an oversight as the mechanism in this branch only allows for payment information to be specified for the country and not at a more granular level.

I've added comments to the Trello card to ask how we address both of these issues.
I plan to add another Trello card to work out whether we can reduce the number of payment information partials in the system.

As you will probably have gathered from my previous comments, I found this PR a bit hard to follow:

I may have missed it, but it would help if there was a general explanation of what the marriage-abroad-countries-leading-to-outcome.rb script does and how you used it.
I think most commits in this PR are of 2 (or possibly 3) types, but all of their commit note subject lines start with the word "Extract". This makes it hard to discriminate between the different types of commit.
The commit notes didn't seem to make it clear how you decided that the changes to the regression test artefacts were acceptable.
I think my confusion about the changes to the regression test artefacts was made worse, because I initially overlooked the fact that the purpose of the PR is to "remove payment information from outcomes that don't display services/fees", so I was expecting this PR to just be a refactoring.
It feels as if there is some sense in which single commits are doing both refactoring and making a change to the behaviour. Or have I got that wrong?
In general I'm in favour of the refactoring aspect of this PR - I think it's much better if we store data about each country rather than conditionals which use lists of countries which share some random characteristic.
It's great that you've identified some discrepancies. I hope you get some answers relatively quickly.
I reckon it would be worthwhile flagging up some of the (accidental?) variation in the payment partials to see whether this list can be further consolidated and ideally converted into list of acceptable/unacceptable payment types for each country. The benefit to the FCO & the Content Team would be that it would be much easier to update this data.
All in all it looks great (good work!), but I think I need to come back to it once you've explained more about the  marriage-abroad-countries-leading-to-outcome.rb script and the changes to the regression test artefacts. Hence I'm not marking it as LGTM for now.

Thanks for reviewing, @floehopper and @pmanrubia.
I'm going to close this open a new PR that addresses your feedback.

I may have missed it, but it would help if there was a general explanation of what the marriage-abroad-countries-leading-to-outcome.rb script does and how you used it.

@floehopper: I've attempted to explain the use of this script in the following paragraph in the description of PR #2507:

For outcomes that are rendered for multiple countries, I used the marriage-abroad-countries-leading-to-outcome.rb script to get a list of those countries before updating the add-payment-partials-to-marriage-abroad-services-data.rb script.

I've also got a separate note to add some more information about that script. Either to the script itself, or to some docs.

I think most commits in this PR are of 2 (or possibly 3) types, but all of their commit note subject lines start with the word "Extract". This makes it hard to discriminate between the different types of commit.

@floehopper: I've attempted to improve this slightly in PR #2507 by making it clearer when the commit extracts a partial, versus when it extracts payment information from a template. I can see that it's still not very clear but hopefully it's slightly better.

The commit notes didn't seem to make it clear how you decided that the changes to the regression test artefacts were acceptable.

@floehopper: I've attempted to improve this in PR #2507. See https://github.com/alphagov/smart-answers/pull/2507/commits/9a91b1f78fb731d57813435ce0a6eb42c16585f2 for an example.

I think my confusion about the changes to the regression test artefacts was made worse, because I initially overlooked the fact that the purpose of the PR is to "remove payment information from outcomes that don't display services/fees", so I was expecting this PR to just be a refactoring.

@floehopper: I've renamed the branch in PR #2507 to (hopefully) make its purpose clearer.

It feels as if there is some sense in which single commits are doing both refactoring and making a change to the behaviour. Or have I got that wrong?

@floehopper. That's correct in the case of some commits. I think it would've been possible to create a branch that just refactored how the payment information was displayed before then separately removing it for those countries that shouldn't display it. While possible, I think this would've been quite a lot more work as I would've had to support displaying different payment information for a single country. I believe this only exists by mistake (i.e. templates have been split causing differences between the information displayed for the same country).


add subscribe email functionality to website for the Mailchimp stuff

@MoyaSparklefarts  Please assist @ScreamingHawk .

`

Subscribe
* indicates required

First Name  *

Last Name  *

Email Address  *

`

subscribe-code.txt

Thanks. Ezpz to add. On which page?

Probably the contact page for now, definitely the blog page once it's up.

Added. Worth testing out. The test on the page you get directed too will be editable on MailChimp. There's different text if you forget to fill a field.
Please close the ticket if you're happy with it

It works fine, closing now 👍


YOLOv8 rebuilding CUDA engine not finishing

When executing the commands from the Quickstart the following comes up and inference never seems to start:
2024-02-27 14:15:34.115 WARN  /data/jenkins/ros/workspace/ly_debian-compatible-release-2.1/isaac_ros-dev/ros_ws/src/isaac_ros_dnn_inference/isaac_ros_tensor_rt/gxf/tensor_rt/tensor_rt_inference.cpp@278: Rebuilding CUDA engine /tmp/yolov8s.plan (forced by config). Note: this process may take up to several minutes.
I waited over an hour on an Orin NX without any further messages. As this is probably the TensorRT conversion I also tried exporting to TensorRT with yolo export model=yolov8s.pt format='engine' outside the container, which works just fine. However, using the resulting TensorRT model inside the container did not work as well. Any ideas on this?

Hi @leon-seidel ,
Could you please provide the full logs up to and including the message you sent? It's possible there was a failure earlier in the logs that led to a different node in the launch graph failing, which is why you don't see any further messages.
Also, can you check if any engine plan file is created at the end of that waiting period? Or is /tmp/ still empty?
When you say that the plan created outside the container didn't work "as well" inside the container, do you have a specific metric you're using, or any observations about how the performance is different?
This information will help us help you. Thanks!

Thanks for your answer and sorry for the late reply! Looking at the full logs show that the problem seems to be related to lipnvvpi, as the tensor_rt_container doesn't even start.
[ERROR] [1709216981.120842565] [tensor_rt_container]: Failed to load library: Could not load library dlopen error: /opt/nvidia/vpi2/lib64/libnvvpi.so.2: version `VPI_2.3' not found (required by /opt/ros/humble/lib/libisaac_ros_nitros_image_type.so), at /tmp/binarydeb/ros-humble-rcutils-5.1.4/src/shared_library.c:99
[ERROR] [launch_ros.actions.load_composable_nodes]: Failed to load node 'dnn_image_encoder' of type 'nvidia::isaac_ros::dnn_inference::DnnImageEncoderNode' in container '/tensor_rt_container': Failed to load library: Could not load library dlopen error: /opt/nvidia/vpi2/lib64/libnvvpi.so.2: version `VPI_2.3' not found (required by /opt/ros/humble/lib/libisaac_ros_nitros_image_type.so), at /tmp/binarydeb/ros-humble-rcutils-5.1.4/src/shared_library.c:99

Regarding the other questions:
There is a plan file created in /tmp/ and with "as well" I meant that it's not working at all, either.
We are using Jetpack 5.1.1 for this.
Full logs are here

Update: Upgrading to Jetpack 5.1.2 fixed this


Cross compiling fails

Is cross compiling supported?
Reading the turorial, I got the impression that it should be supported. But I couldn't build my project for macOs or Linux on my Windows machine.
This is the error I got for Linux:
'"C:\Users\[user]\.nuget\packages\runtime.linux-x64.microsoft.dotnet.ilcompiler\1.0.0-alpha-26223-02\tools\ilc"' is not recognized as an internal or external command,
  operable program or batch file.

And I got a similar error for mac.

Cross compiling is not supported yet.

Ok. Thanks!
Is there a timeframe for this feature?

will the IL2CPP work when using ubuntu on windows, with a linux-64 target ?

Yes, CppCodeGen should work in Ubuntu on Windows. Note that CppCodeGen is experimental less mature option. You may have more success with using the default RyuJIT codegen.

@jkotas Thanks, I'm looking at IL2CPP so that I may compile using a non llvm compiler for a target currently not natively supported by .NET CORE with very limited resources and therefore it won't have a runtime with JIT capability given this is RyuJIT still the better an option? am I missing something ?

Ok, CppCodeGen is the right option for that.

@jkotas cool wasnt sure if I missed something and if RyuJIT could be used for none llvm compiler and target other types of compilers like gcc or generate libraries

A year later, and it's still unsupported
@jkotas is it still planned to be supported or should we consider it a distant future?

I am not aware of anybody actively working on it.

@jkotas where would a developer start if they wanted to implement this? How does dotnet core go from VM IR to x86?

We are moving this project to https://github.com/dotnet/runtimelab/tree/feature/NativeAOT . The new place is fork of dotnet/runtime repo that has the build infrastructure for cross-compilation. Getting the NativeAOT branch building for x86 would be the first step.

How does dotnet core go from VM IR to x86?

RyuJIT is used on all platforms. https://github.com/dotnet/runtime/tree/master/src/coreclr/src/jit.


Add implementation of Tweet API

Functionality to be implemented includes:

Tweets lookup
Manage Tweets
Timelines
Search Tweets(Already implemented)
Tweet counts
Filtered stream
Volume stream
Retweets
Likes
Hide replies

Done

Tweets lookup


Ignore the unreachable code detection error about a tail throw statement as a tail unreachable statement assertion

Unreachable code detection cannot to be the perfect in JavaScript because JavaScript cannot segregate the main statement block(for detect the tail statement of main flow in function) and the declaration block like a where statement of Haskell. We cannot find the tail statement without an assertion such as return, throw or assert(false).
So unreachable code detection should ignore once about a tail throw statement:
expected:
function f(n) {
    switch (n) {
        case 0:
            return 0;
        default:
            if (n > 0) {
                return n;
            }
            else {
                return -n;
            }
    }
    // tail unreachable statement assertion
    throw new Error; // should not be unreachable code error
    throw new Error; // unreachable code error

    function g() {
    }
    // many functions...
    function z() {
    }

    // bad hidden code
    return n; // unreachable code error
}

fault but no error:
function f(n) {
    switch (n) {
        case 0:
            return 0;
        default:
            if (n > 0) {
                return n;
            }
            // forgotten else...
            //else {
            //    return -n;
            //}
    }
    // tail unreachable statement assertion
    // throw new Error;
    // throw new Error; // unreachable code error

    function g() {
    }
    // many functions...
    function z() {
    }

    // bad hidden code
    return n; // not unreachable code error
}

This fault code cannot detect by --noImplicitReturns and unreachable code detection but those language functions reject the tail unreachable statement assertion. It is bad conflict.

I'd say that the reason for this issue is specific coding style and thus it can be captured by syntactic linting rules (something like function declarations at the end of function body should not be mixed with statements).

this issue is specific coding style ... something like function declarations at the end of function body should not be mixed with statements

Yes, but it is not the point.

it can be captured by syntactic linting rules

It is good rules but how can I get it error?
Those language functions cannot reject that mistakes caused by that coding style, and reject the correct counter measure. Those constraints makes a new risk. Will you accept it risk and weakness as a spec? I think, it is not good as a language.

We can't pick apart what would be considered an "acceptable" snippet of code that's unreachable. Better than a throw statement would be a filler comment like ////////////////////////////.

Alternatively, you can use a fail function for now:
function fail() {
    throw "Unreachable";
}

How do I use it function? It seems useless.
function f() {
  if (true) return;
  fail(); // unreachable error
  function fail() {
    throw "Unreachable";
  }
}


https://youtu.be/QXf8nzQaCac

Checklist

[X] I'm reporting a broken site support
[X] I've verified that I'm running youtube-dl version 2021.12.17
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones

Verbose log

root@postal:~# youtube-dl -U
youtube-dl is up-to-date (2021.12.17)
root@postal:~# exit
logout
theo@postal:~/Videos$ youtube-dl "https://www.youtube.com/watch?v=QXf8nzQaCac"
[youtube] QXf8nzQaCac: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
theo@postal:~/Videos$ youtube-dl "https://youtu.be/QXf8nzQaCac"
[youtube] QXf8nzQaCac: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
theo@postal:~/Videos$ youtube-dl --verbose "https://youtu.be/QXf8nzQaCac"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'https://youtu.be/QXf8nzQaCac']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 2.7.18 (CPython) - Linux-5.4.0-126-generic-x86_64-with-Ubuntu-20.04-focal
[debug] exe versions: ffmpeg 4.2.7, ffprobe 4.2.7
[debug] Proxy map: {}
[youtube] QXf8nzQaCac: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

WRITE DESCRIPTION HERE
I couldn't download it ...

Nor could you follow the issue template correctly...

[x] I've searched the bugtracker for similar issues including closed ones

Could these be connected?


Browserbase Integration

A simple browserbase implementation built based upon puppeteer

Thank you for taking the time to build this! We've now made some changes to how we'll manage the servers repo going forward.
TLDR: We're restructuring the MCP servers repository to focus on reference implementations only. New server implementations should be developed in separate repositories and can be added as links to our README, rather than submitted as PRs to this repository.
In line with that new approach, I'll close out this PR, but we would love to include a link to this server once it's in its own repo. Please do submit a new PR when it's ready!


Update scalafmt-core to 3.7.12

About this PR
📦 Updates org.scalameta:scalafmt-core from 2.7.5 to 3.7.12 ⚠
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]

labels: library-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #322.


Add support for subcsription AutoDeleteOnIdle

Allow library users to configure AutoDeleteOnIdle-property
MSDN

Can be handled by overriding configurations.


This updates all the controls

Can you please update the documentation. If you have an attribute, with multiple controls, if you removeOption on a single control, it removes from all the controls of that attribute.

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

ID: da2fd88e-934d-9bfc-9888-9d15b97e8e4d
Version Independent ID: 5a98fca8-60c7-ccc1-6edc-2ef16d2ebf8c
Content: addOption (Client API reference) in model-driven apps - PowerApps
Content Source: powerapps-docs/developer/model-driven-apps/clientapi/reference/controls/addOption.md
Service: powerapps
GitHub Login: @KumarVivek
Microsoft Alias: kvivek

@fhburney Thanks for contacting us. Can you confirm if you're still facing the issue.

Thanks for contacting us. As we haven’t heard back from you, I am closing this issue. If you need further help, please reopen the issue or open a new issue.


Support a mechanism to force updating periodically updated channels

If a bad version of a channel is pushed out, and the CI system aggressively caches the cache directory, CI can be broken for the entire duration of the update period (eg. 24h) because Hermit will not check for new versions.
I propose that hermit sync should be extended to check channels in the active environment for updates, in addition to pulling down latest sources.
CI scripts can then run hermit sync before anything else to ensure they're up to date.

Fixed in #226


ci: Don't install things to home directory, add workflow to test release pipelines

Cherry-pick PR: #706
Summary:
Noticed we were getting extra conda packges from the home directory on
the M1 runners. Making this PR to reduce the amount that we're using the
home directory.
Will probably make it unwritable in the future so this is a pre-emptive
step to that.
Also adds a workflow to test the release pipelines without actually releasing a new binary.
Signed-off-by: Eli Uriegas<EMAIL_ADDRESS>Pull Request resolved: https://github.com/pytorch/data/pull/706
Reviewed By: ejguan
Differential Revision: D38406845
Pulled By: seemethere
fbshipit-source-id: 1d312c3e1db34497c546a9b842ac7ba01d2a7ee3
Please read through our contribution guide prior to
creating your pull request.

Note that there is a section on requirements related to adding a new DataPipe.

Fixes #{issue number}
Changes

There are CI errors for both wheel and conda. Any idea why?
Looks like it is deal to ImportError: cannot import name '_check_lambda_fn which exists in 0.4.0 but I think has since been changed to _check_unpickable_fn. Perhaps there is a mismatch between the version of PT Core and TorchData?
I think we might also need to cherry pick #546 or correct the version of PT Core.

Oh. Let me fix that. I think the test release workflow is using nightly PyTorch but it should use 1.12.1

It fails on m1 with Python 3.8.
Looks like it is due to subprocess.CalledProcessError: Command '['cmake', '--version']' died with <Signals.SIGABRT: 6>. Is it just flaky?

I guess it's flaky. The root cause was some missing system dependency. I think it's fine now as the test is passed in this PR link.


Updated documentation for AWS

The current documentation does not work anymore in AWS. Things are made a lot easier in EKS these days. With this update you can get that going. I did not spend a  lot of time on descriptions.

Hi sorry for the late response.
Thanks so much for the update on this docs 👍
Do you know if there is an easy way with this setup using a LoadBalancer to expose MultiJuicer directly to also support https? At the moment this setup as described in this guide would be http only, right? Or can you configure the AWS Loadbalancer to also handle tls termination?

haha no worries @fwijnholds  :D
I left the AWS Guide open and added a note that it would be nice to include in the guide on how to set up https on AWS.
We also have excatly the same issue open for azure, might be easier to just link to a guide on how to set up nginx-ingress and cert-manager and dont have this section be plattform specific.


Ran terraform fmt

This is with 0.7.7

Not a huge deal, but I use this submodule a lot and terraform always picks it up as something new to format after a terraform get

Good idea. Shameless plug - https://github.com/antonbabenko/pre-commit-terraform :)


Database not properly closed when .close() is called while in "busy" state

Hi @WebReflection,
I think I may have found a little bug in the current version of the library.
This is related to the close() method: if you call this method while the object is marked as busy, then the ".exit" command is queued but never gets run, because the notWorking flag is set immediately, and then when query() is called again by next(), the method exits before actually running the command.
This sequence of events can happen for instance if you run a select query that raises a fatal error (that makes sqlite output to stderr but not to stdout, so the busy flag is not reset), and then try to close the database.
As a (rather ugly) workaround, setting ignoreErrors==true and adding a call to a successful select before closing solves the problem. E.g.:
db.ignoreErrors= true;
db.query("select 1", function(err, data) {
	db.close();
});

(This "resets" the busy state before the call to close(), so that .exit will be properly run regardless of what happened before).
This is quite a minor issue, as the object is unusable anyway after the call to close(), but it can interfere when doing unit tests with lots of sequential database openings and closings, as the db file stays open (thus locked) until the dblite object gets garbage-collected, instead of being explicitly freed when close() is called.
Please let me know if you need more information to reproduce this.
Bye!

this is a very valid concern and I'll try to wrap my head around a better solution. Thanks

can yu please tell me if using this login instead all works as expected?
  if (
      // notWorking is set once .close() has been called
      // it does not make sense to execute anything after
      // the program is being closed
      notWorking &&
      // however, since at that time the program could also be busy
      // let's be sure than this is either the exit call
      // or that the last call is still the exit one
      !(string === '.exit' || queue[queue.length - 1][0] === '.exit')
    ) return onerror('closing'), self;

This would be the very beginning of method self.query = function(string, params, fields, callback)
Thanks a lot

Hello,
I just tested this on the version I'm currently using (0.6.1), and it looks like it does the job: all the unit tests that used to timeout with the original code now pass properly.
Thanks a lot :-)
Cheers!

thanks for testing, it's up and running on 0.7.1


Exception is thrown on using composition patter in schema

Describe the bug
When using composition patter in the object schema (allOf), exception is thrown.
Error while resolving allOf schema:  Error: Could not resolve values for path:"enum". They are probably incompatible. Values:
[
  "incomplete",
  "completed",
  "refunded",
  "expired",
  "error",
  "pending_stellar",
  "pending_external"
]
[
  "pending_sender",
  "pending_receiver",
  "pending_transaction_info_update",
  "pending_customer_info_update"
]
    at throwIncompatible (<project>/node_modules/json-schema-merge-allof/src/index.js:95:9)
    at <project>/node_modules/json-schema-merge-allof/src/index.js:320:11
    at Array.forEach (<anonymous>)
    at mergeSchemas (<project>/node_modules/json-schema-merge-allof/src/index.js:301:13)
    at merger (<project>/node_modules/json-schema-merge-allof/src/index.js:334:18)
    at Object.resolveAllOf (<project>/node_modules/@paloaltonetworks/openapi-to-postmanv2/lib/deref.js:102:14)
    at Object.resolveRefs (<project>/node_modules/@paloaltonetworks/openapi-to-postmanv2/lib/deref.js:179:19)
    at <project>/node_modules/@paloaltonetworks/openapi-to-postmanv2/lib/deref.js:104:23
    at Array.map (<anonymous>)
    at Object.resolveAllOf (<project>/node_modules/@paloaltonetworks/openapi-to-postmanv2/lib/deref.js:103:26)

As a result (?), composition object is not rendered correctly. However, this could be related to #574 (issue happens in the same place, but with a slightly different setup)
Expected behavior
No error
Current behavior
Error
Possible solution
N/A
Steps to reproduce
Clone project in the correct branch and run  yarn api:gen
Screenshots
ReDoc

(Using this plugin, v1.7.2)

Context
See #574
Your Environment
Branch
OpenAPI yaml file
Package.json
Working ReDoc link

Hi @Ifropc, it appears that @paloaltonetworks/openapi-to-postmanv2 is throwing the exception. Since it's a forked package/library, my thought was that we should try to parse your OpenAPI spec using the original library/CLI:
https://github.com/postmanlabs/openapi-to-postman
It could also be helpful to scrub their issues/PRs to see if this behavior has been already reported or patched.

It looks like it's affected by https://github.com/postmanlabs/openapi-to-postman/issues/417
FYI: I wrote that the page page doesn't render the content correctly at first, but in reality nothing is generated (code 1 is returned)
I updated description.
Out of curiosity, why is this library used?

Out of curiosity, why is this library used?

It's used for generating the code snippets.


Adding a lot of versions to a definition curation hides the OK/Cancel buttons

If I add a bunch of versions to a definition I'm curating the OK/Cancel options get hidden. Clicking away from the screen doesn't save my work.

Hi @fossygirl ,
I believe the issue is the fact that the modal was closing when there are versions added to the input field.
Would this behaviour be better (see GIF below)?:

When clicking outside the modal (dark backdrop) don't close the Modal if there are any changes made (versions added in the input field)
Close the modal if the input field is empty

Sure, this is a fine change.


Added support for * subdomains

I fixed the regex that prevented you from adding * domains to your proxy list

This is an automated message from CI:
Docker Image for build 1 is available on DockerHub as jc21/nginx-proxy-manager:github-pr-1127
Note: ensure you backup your NPM instance before testing this PR image! Especially if this PR contains database changes.

In conflict with https://github.com/jc21/nginx-proxy-manager/pull/1181


Failsafe/fail-fast in case tags are missing

If tags are missing, for example because they are missing in a fork - https://github.com/scala/scala-dev/issues/331#issuecomment-293216198, then the plugin will happily fallback to commit sha + timestamp mode.
A user of sbt-dynver might prefer it to fail-fast instead, and perhaps this should be the out-the-box experience for sbt-dynver (with an opt-out).

Documented as https://github.com/dwijnand/sbt-dynver#sanity-checking-the-version.


update docs folder links

What steps will reproduce the problem?
http=>https
What is the expected result?
What do you get instead?
Additional info

Q
A

Version
1.0.?

PHP version

Operating system

hello. Could you explain what this issue is about? I don't understand what you mean


Where can I obtain an API key for Discord_Token

Hello ,
Where can I obtain an API key for Discord_Token?
I am looking forward to hearing back from you!
Best Regards
Peter Westin

Hi Peter I have added a new section in the readme where you can find the answers to this very good question!
Thanks for asking, let me know if you have any more questions 😄


DB: Optimise query pattern for InstanceList

This PR fixes the instance list speed regression introduced by https://github.com/lxc/lxd/commit/fe37c21bae5ab70f65a7e6639de5c759588fb339 and goes further to optimise the query pattern than was in LXD 5.2.
The main problem was that the InstanceToArgs function loads the profile(s) (and its config and devices) for every instance, even if multiple instances use the same profile(s). Additionally the GetDevices() and GetConfig() prepare a statement on every call, which is inefficient.
This PR updates InstanceList() to use its own optimised queries.
On LXD 5.2 (before that commit), my test box could create 128 images:ubuntu/jammy containers on ZFS (with pre-warmed image volume) in: 1m27.566s, 1m28.397s, 1m28.810s
On LXD 5.3 (after that commit), the results are as follows: 2m2.597s, 2m2.060s, 2m3.763s
On LXD edge channel with this PR, the results are as follows: 1m24.483s, 1m26.036s, 1m27.209s

Nice! Good to see InstanceList optimized. I had made a note in the  initial PR that this might be necessary.
If the goal is mainly efficiency, then I think this looks good.
Alternatively to avoid the complexity added by the custom queries, I can update the generated methods for Devices and Config to have pre-prepared statements, similar to the other entities. And for profiles, we could add a cachedProfiles parameter to InstanceToArgs, which will then only fetch new profiles if there is no associated profile ID for the instance  in cachedProfiles, which is then added to cachedProfiles for the next iteration.
So something like
func InstanceList() {
  //...
  cachedProfiles map[int]api.Profile
  for _, instance := range dbInstances {
    instanceArgs, err := InstanceToArgs(ctx, tx, instance, cachedProfiles)
    if err != nil {
      return err
    }
  }
}

// ...
func InstanceToArgs(ctx context.Context, tx *sql.Tx, inst *Instance, cachedProfiles map[int]api.Profile) (*InstanceArgs, error) {
  // Return a list of `profile_id` from `instances_profiles`.
  profileIDs, err := cluster.GetInstanceProfileIDs(ctx, tx, instanceID)

  for _, profileID := range profileIDs {
    apiProfile, ok := cachedProfiles[profileID]
    if !ok {
      // Fetch a new profile and store it in the cache.
      dbProfile, err := cluster.GetProfiles(ctx, tx, cluster.ProfileFilter{ID: &profileID})
      apiProfile, err = dbProfile.ToAPI(ctx, tx)
      cachedProfiles[profileID] = apiProfile
    }

     inst.Profiles = append(inst.Profiles, apiProfile)
  }
}

This would still be less efficient than your implementation here, since there's more individual queries. One thing that might be helpful though is if this logic for efficiently creating a list of InstanceArgs could be split into its own helper. I can see 2 other cases in the codebase at the moment that calls InstanceToArgs in a loop that could be helped by this.

Yes good idea I will take a look and add to this pr.

I've added support for a --project flag to lxd-benchmark.

LXD edge with PR
lxd.benchmark init --count 128 images:ubuntu/jammy
Test environment:
  Server backend: lxd
  Server version: 5.3
  Kernel: Linux
  Kernel architecture: x86_64
  Kernel version: 5.4.0-121-generic
  Storage backend: dir | zfs
  Storage version: 1 | 0.8.3-1ubuntu12.14
  Container backend: lxc | qemu
  Container version: 5.0.0 | 7.0.0

Test variables:
  Container count: 128
  Container mode: unprivileged
  Startup mode: normal startup
  Image: images:ubuntu/jammy
  Batches: 32
  Batch size: 4
  Remainder: 0

[Jul  5 18:58:50.081] Found image in local store: 1e98e1a26b1be1075bb1b0b08a31cdac00807683e4bd60940593c3d10549745c
[Jul  5 18:58:50.081] Batch processing start
[Jul  5 18:58:50.921] Processed 4 containers in 0.840s (4.765/s)
[Jul  5 18:58:51.857] Processed 8 containers in 1.776s (4.504/s)
[Jul  5 18:58:53.890] Processed 16 containers in 3.809s (4.200/s)
[Jul  5 18:58:57.555] Processed 32 containers in 7.474s (4.282/s)
[Jul  5 18:59:05.656] Processed 64 containers in 15.575s (4.109/s)
[Jul  5 18:59:22.793] Processed 128 containers in 32.712s (3.913/s)
**[Jul  5 18:59:22.793] Batch processing completed in 32.712s**

LXD 5.3:
lxd.benchmark init --count 128 images:ubuntu/jammy
Test environment:
  Server backend: lxd
  Server version: 5.3
  Kernel: Linux
  Kernel architecture: x86_64
  Kernel version: 5.4.0-121-generic
  Storage backend: zfs | dir
  Storage version: 0.8.3-1ubuntu12.14 | 1
  Container backend: lxc | qemu
  Container version: 5.0.0 | 7.0.0

Test variables:
  Container count: 128
  Container mode: unprivileged
  Startup mode: normal startup
  Image: images:ubuntu/jammy
  Batches: 32
  Batch size: 4
  Remainder: 0

[Jul  5 18:59:56.211] Found image in local store: 1e98e1a26b1be1075bb1b0b08a31cdac00807683e4bd60940593c3d10549745c
[Jul  5 18:59:56.211] Batch processing start
[Jul  5 18:59:57.246] Processed 4 containers in 1.035s (3.865/s)
[Jul  5 18:59:58.169] Processed 8 containers in 1.959s (4.084/s)
[Jul  5 19:00:00.119] Processed 16 containers in 3.908s (4.094/s)
[Jul  5 19:00:04.589] Processed 32 containers in 8.378s (3.819/s)
[Jul  5 19:00:17.036] Processed 64 containers in 20.825s (3.073/s)
[Jul  5 19:01:02.699] Processed 128 containers in 66.488s (1.925/s)
**[Jul  5 19:01:02.699] Batch processing completed in 66.488s**

LXD 5.2
lxd.benchmark init --count 128 images:ubuntu/jammy
Test environment:
  Server backend: lxd
  Server version: 5.2
  Kernel: Linux
  Kernel architecture: x86_64
  Kernel version: 5.4.0-121-generic
  Storage backend: dir | zfs
  Storage version: 1 | 0.8.3-1ubuntu12.14
  Container backend: lxc | qemu
  Container version: 4.0.12 | 7.0.0

Test variables:
  Container count: 128
  Container mode: unprivileged
  Startup mode: normal startup
  Image: images:ubuntu/jammy
  Batches: 32
  Batch size: 4
  Remainder: 0

[Jul  5 19:02:56.730] Found image in local store: 1e98e1a26b1be1075bb1b0b08a31cdac00807683e4bd60940593c3d10549745c
[Jul  5 19:02:56.730] Batch processing start
[Jul  5 19:02:57.686] Processed 4 containers in 0.956s (4.183/s)
[Jul  5 19:02:58.609] Processed 8 containers in 1.879s (4.257/s)
[Jul  5 19:03:00.769] Processed 16 containers in 4.039s (3.962/s)
[Jul  5 19:03:04.777] Processed 32 containers in 8.047s (3.977/s)
[Jul  5 19:03:13.456] Processed 64 containers in 16.726s (3.826/s)
[Jul  5 19:03:32.769] Processed 128 containers in 36.039s (3.552/s)
**[Jul  5 19:03:32.769] Batch processing completed in 36.039s**

@stgraber tested the lxd-benchmark with launch mode for LXD 5.2 and LXD edge with my PR:
lxd.benchmark launch --count 128 images:alpine/3.16
LXD 5.2: Batch processing completed in 118.958s
LXD edge with PR: Batch processing completed in 217.692s
The init times are better than before, but the launch times are worse (although still better than LXD 5.3), this can be explained by the extra start time checks we had to add in order to accommodate allowing instances with duplicate IPs to exist at create time.
See https://github.com/lxc/lxd/pull/10546

@tomponline
--- FAIL: TestInstanceList (0.12s)

@stgraber it occurred to me today that as the https://github.com/lxc/lxd/pull/10546 PR kept the duplicate IP check in the bridged`` device's validateConfig` function then it should have previously been called both at create time and at start time, so really that PR should not be the cause of any start time slow downs. Lets see what the charts show after this has been merged and revisit if start time is still higher than before.


Update browser_adapter.dart

avoid print calls in production code.
New Pull Request Checklist

[x] I have read the Documentation
[x] I have searched for a similar pull request in the project and found none
[x] I have updated this branch with the latest develop to avoid conflicts (via merge from master or rebase)
[-] I have added the required tests to prove the fix/feature I am adding
[-] I have updated the documentation (if necessary)
[-] I have run the tests and they pass

This merge request fixes / refers to the following issues: #1589
Pull Request Description
the print call leads to an annoying output in console.

Can you change it so the line is gone instead of commented? Then I'll merge it :)

already done
(this is my first-time-ever-contribution in an open source project.... sorry!)


restore mono_build.sh which got deleted by a previous commit and mae t work if OLPATH contains whitespace characters

You probably forgot to add the moved file when you moved it from GameMod/
to linux/ ;)
Also fix issues if OLPATH contains whitespaces, which is the case for the
default GOG installation path of "$HOME/GOG Games/"

Argh, you're completely right - thanks!


Move test to example

Move the old test to the examples folder. Aligns better with our new format. Fixed a typo in raw.rs where default keyword was accidentally on a new line.

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

Comparison is base (58b92c8) 61.12% compared to head (33a3833) 61.12%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #337   +/-   ##
=======================================
  Coverage   61.12%   61.12%
=======================================
  Files          23       23
  Lines        1407     1407
=======================================
  Hits          860      860
  Misses        547      547

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

@bconn98 needs rebase


IPC

I assume it’s only designed for Linux ?
Would be cool to be able to use pic on desktop but I assume that needs native golang code with build tags .
what environment are you designing for
really interesting project btw !!

Hi @gedw99 and welcome
I'm currently finishing off the following features which will be propagated to ControllerBus soon (tm):

BroadcastChannel (in web browser) and WebWorkers
Sub-process (linux sub-process) - named pipes and/or stdin/stdout
Networked (k8s pod)
Wasm sub-process (via wasmer-go)

Developers will be able to declare a topology of configs to run on available Bus. The Bus are bridged using "bridge controllers" which forward directives conditionally up/down the Bus tree.

Hey @paralin
sorry about late reply.
The wasm stuff for browser and server sounds very useful for me .
I probably got the wrong end of the stick when I started asking about IPC. To make this easier let me reframe my question …
does the server part ( sub processes is what you called it above ) support Windows and MAC ?
the reason I ask is because I am working on a system that runs on desktops where the system is modular in that just like the Service Factory pattern ( Thankyou Martin Fowker we worship you :) ) the implementation can be swapped out at runtime.
I am getting the feeling that wasm is your engine to allow swapping out on the server / desktop . Like a poor man’s docker .
Sorry about all the vague questions but this system is pretty unique and so searching for it’s use case fit a bit .

@gedw99 Check out Bifrost, which uses controller-bus: https://github.com/aperturerobotics/bifrost

More IPC info coming soon. closing this issue for now


Update Codecov development dependency

Codecov Report

Merging #8 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master        #8   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines          318       318
=========================================
  Hits           318       318

Continue to review full report at Codecov.

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


How do I get html content by using prevall method?

For example,a document as follows:

Now I want to get all content within the 'article' div,include the innerhtml(such as p and b),but except the last "b".The result would like this:

Anybody can tell me how to do this?

You can't use prevAll(), because it don't work with text nodes - just as in jQuery.
To accomplish this you can select last bold element, then exclude it from Select(".article").Contents() using Not() method:
using Microsoft.VisualStudio.TestTools.UnitTesting;
using NUnit.Framework;
using Assert = NUnit.Framework.Assert;

namespace CsQuery.Tests.Issues
{
    [TestFixture, TestClass]
    public class Issue185 : CsQueryTest
    {
        private const string html =
           "
"+
                "Hello World!"+
                "
I am using CsQuery library.
"+
                "I am feeling bold"+
                "I am the second bold"+
                "What about"+
                "you?"+
            "
";

        private const string expected =
            "Hello World!" +
            "
I am using CsQuery library.
" +
            "I am feeling bold" +
            "I am the second bold" +
            "What about";

        [Test, TestMethod]
        public void Issue185Test()
        {
            CQ document = CQ.Create(html);

            var lastBold = document.Select(".article b").Last();
            var contents = document.Select(".article").Contents().Not(lastBold);
            var actual = contents.RenderSelection();

            Assert.AreEqual(expected, actual);
        }
    }
}


Inference performance questions

Issue type
Performance
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
v2.13.0-17-gf841394b1b7 2.13.1
Custom code
No
OS platform and distribution
Linux Ubuntu 20.04.6 LTS
Mobile device
No response
Python version
3.8.10
Bazel version
5.3.0
GCC/compiler version
9.4.0
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Server have 72 cpu cores. I'm unable to get more than 60% cpu utilization when using more than 16 threads via the --tensorflow_inter_op_parallelism parameter.
I'd like to use one tensorflow_model_server instance that would utilize all the HW cpu cores. Using tensorflow_inter_op_parallelism > 16, the utilization per thread drops as low as 30-40% utilization and the overall performance doesn't improve.
I'm using:
tensorflow_model_server --port=5051 --model_name=taboola_serving --model_base_path=/opt/taboola/models/cvr/ --tensorflow_intra_op_parallelism=32 --tensorflow_inter_op_parallelism=16
Also, attached is a flamegraph. I fail to understand why the ThreadPoolDevice::Compute part is only 38.5% of the overall time.
Is there some other perfromance parameter that I'm missing?

Standalone code to reproduce the issue
As described above

Relevant log output
No response

Hi @eyalhir74 ,
In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here.
Thank you!

Thanks @Venkat6871 !
I'm just loading our model to tensorflow_model_server and have a client that does infer calls against it.
So not much code that I can supply.
Attached are two screenshots that hopefully better demonstrate it.
The first image shows a top breakdown when the server was started with --tensorflow_inter_op_parallelism=8 - you can see 8 threads that reach ~92% utilization
Second image is when running with --tensorflow_inter_op_parallelism=16 and top shows 16 threads with 60% utilization.
Now, since the server have 72 cpu cores, I'd like to open one big tensorflow server running on all cpu cores, all reaching 90+% utilization. No matter how many clients/load I put on the server with tensorflow_inter_op_parallelism > 16, I can't reach full utilization and the HW just sits idle then.

Thanks @Venkat6871 !
I'm just loading our model to tensorflow_model_server and have a client that does infer calls against it.
So not much code that I can supply.
Attached are two screenshots that hopefully better demonstrate it.
The first image shows a top breakdown when the server was started with --tensorflow_inter_op_parallelism=8 - you can see 8 threads that reach ~92% utilization
Second image is when running with --tensorflow_inter_op_parallelism=16 and top shows 16 threads with 60% utilization.
Now, since the server have 72 cpu cores, I'd like to open one big tensorflow server running on all cpu cores, all reaching 90+% utilization. No matter how many clients/load I put on the server with tensorflow_inter_op_parallelism > 16, I can't reach full utilization and the HW just sits idle then.

@Venkat6871 any idea what could be wrong here?

Hi @eyalhir74 ,
Apologies for the delay, and thank you for the information. However, without the proper code, it is difficult for us to debug the issue. To expedite the troubleshooting process, we need a code snippet to reproduce the reported issue.
Thank you!


[JPPM] Fatal error

Console log:
$ jppm app:run

Fatal error: Uncaught Error: Class 'plugins\AppPlugin\AppPluginPlugin' not found
 in res://packager/Packager.php on line 62, position 32
Stack Trace:
#0 <internal>() called at [res://JPHP-INF/.bootstrap.php:1]
#1 packager\cli\ConsoleApp->main() called at [res://JPHP-INF/.bootstrap.php:9]
#2 packager\Packager->loadPlugins() called at [res://packager/cli/ConsoleApp.php
:63]
#3 {main}
  thrown in res://packager/Packager.php on line 62

JPPM build from last sources.

@MWGuy your package.php.yml incorrect, is not a bug!
See plugins section, here must be similar to:
plugins:
  - AppPlugin

or
plugins: [AppPlugin]


multios syntax when the option is unset

Should cat < * be highlighted as an error/warning when the MULTIOS option is unset? It's syntactically valid, and has an unexpected meaning to someone used to the default option settings.

Sorry, revised suggestion: cat < * should highlight the * as default rather than globbing.  (I assumed the * would be expanded but it's not.)


Moving StorageClusterInit logic to StorageCluster

Previously the noobaa service was created by the StorageClusterInitialization object, which did not track the status and health of noobaa. Instead that object just ensured that noobaa is created. In this PR, I've moved the noobaa service to the StorageCluster object, and the StorageCluster's status is now an aggregate of both the CephCluster and noobaa status.
This means that when the StorageCluster indicates that it has completed installing, that means both the CephCluster and Noobaa are online.
In addition to the NooBaa change, all StorageClusterInit objects have moved to StorageCluster. StorageClusterInit now only acts as a means of triggering the re-reconcile of those initialization objects when StorageClusterInit is deleted.

After introducing this change, I discovered that noobaa v2.0.2 doesn't actually come online for us. We didn't catch this because the StorageCluster was indicating everything was good. Now that the noobaa status is reflected in the StorageCluster, setup ocs setup fails due to noobaa failures.
The noobaa issue is being tracked here.  https://github.com/noobaa/noobaa-operator/issues/90

@jarrpa @obnoxxx FYI, This is an API altering change. If we want to go for something like this, our window of opportunity is pretty small here. Once we release ocs-operator with Noobaa being tracked by the storageclusterinitialization object, it will be more difficult to make this kind of change.

/hold
This needs some discussion before merging.

Adding @nimrod-becker @dannyzaken

/hold
This PR is being revised to move the ownership of all StorageClusterInit objects to the StorageCluster. Deletion of the StorageClusterInit object will then trigger the re-reconcile of those initialization objects

I moved the StorageClusterInitialization logic to StorageCluster. Still working on unit/functional tests though.

@davidvossel I think you missed updating noobaa to 2.0.3 in go.mod.
other than that LGTM

@jarrpa should be ready for review again. StorageClusterInitialization logic has been moved to StorageCluster. The StorageClusterInitialization object is still created but is functionally a no-op.  Deleting the StorageClusterInitialization object forces the StorageCluster object to re-reconcile the initialization objects.
I'll update the description shortly to reflect the large scope this PR is taking on.
I'd like to leave the hold on the pr for a bit longer. There's some details with the unit testing I want to look at in a little more detail.

@davidvossel can we move the upgrade to noobaa 2.0.3 out to a separate PR to unblock other PRs? (if my understanding is correct, this is needed for e.g. PR #194 to proceed...)

Leaving this on hold until I can integrate the changes from PR #230 here

I'm wondering if we could have done this in a different way and avoid the need to bump API version.
We could retain the StorageClusterInit resource and controller. The controller could track the status of noobaa, and keep the status of the StorageClusterInit updated.
The StorageCluster controller would create a StorageClusterInit object as we do now. It would then track the status of this child resource and keep status of StorageCluster and of the ocs-operator updated, as it does with CephCluster.
Also, as the already tracks the status of CephCluster,  it could even create the StorageClusterInit resource after CephCluster is successfully deployed. And then begin tracking its status. This could help simplify the status tracking logic in the storagecluster controller.

/test ocs-operator-e2e-aws

/retest

/test ocs-operator-e2e-aws

/retest


Errors

Error:(2, 0) Plugin with id 'com.github.dcendents.android-maven' not found.
I am getting this error after importing it into android studio, how do I import it and not get this error..
I am new to developing for android,..Sorry if this is a bad question, but I am unable to get rid of this error :(

answered to you in other thead


Islessthan

Allow IsGreaterThan, IsLessThan and IsEqual to support passing in SObjectFields as both params.

Urrgggh I forgot to rebase so it showing two commits 👎


Side menu hard loads when selected menu item renders Shesha setting forms

For example, if you navigate to Settings, Theme, and Entity Configurations, upon selecting the menu item, the side menu hard loads. However, if you navigate to a menu item that renders a dynamic form, it does not hard load
Steps to reproduce the behavior:

Go to 'https://function-adminportal-test.shesha.dev/dynamic/shesha/forms'
Click on 'all the side menu' from form templates going down and monitor the behaviour 'entity configuration', 'settings' and 'theme'
Actual Results: The side menu hard loads when the menu item selected is rendering the Shesha settings form and would cache the behaviour and affect the next menu item you would select

Expected behavior
Upon selecting a menu item rendering the Shesha settings form should not hard load. The behaviour should be the same as when rendering the dynamic forms
See attached video
https://github.com/user-attachments/assets/1b7e0872-aba9-433f-8f8e-e99bf01253e4

Hi @AlexStepantsov, the hard reload is not there anymore, however when you click the menu items that had the issue the menu scrolls up and hide the selected item: https://jam.dev/c/1117ccef-0959-4177-9793-4c8771717747. Is there something we can do about that?


[5.1] Allow for setting an "as" prefix for an entire group.

Does as_prefix make sense for the name here? The fact that something else at this level is named prefix might make this problematic. I'm up for better ideas!

Example usage:
// assessment_routes.php

$router->get('widgets', ['as' => 'widgets', function () {
    return view('Assessment::widgets');
}]);

$router->get('gizmos', ['as' => 'gizmos', function () {
    return view('Assessment::gizmos');
}]);

<?php
// RouteProvider.php
Route::group(['prefix' => '/skills', 'as_prefix' => 'assessment.'], function (Router $router) {
    require __DIR__.'/assessment_routes.php';
});

My Widgets

My Gizmos

Although the following might be more consistent with the way that View namespaces work:
<?php
// RouteProvider.php
Route::group(['prefix' => '/skills', 'as_prefix' => 'Assessment::'], function (Router $router) {
    require __DIR__.'/assessment_routes.php';
});

My Widgets

My Gizmos

I really like the idea :+1:
However I'm not so sure about as_prefix. What about just as? or would that not be clear enough?

That key exists already, to set a route name.

That key exists already

But only for routes themselves not for groups. I agree that it could lead to some confusion though...

We allow middleware in route groups as well as individual routes. I agree it should just be as and not as_prefix. Thanks :heart:

I think you should be using route() instead of url() in the examples. :+1:
How about group_as, instead of as_prefix?

Just use "as"... No need for another name.
On Saturday, June 6, 2015, Ibrahim AshShohail<EMAIL_ADDRESS>wrote:

I think you should be using route() instead of url() in the examples. [image:
:+1:]
How about group_as, instead of as_prefix?
—
Reply to this email directly or view it on GitHub
https://github.com/laravel/framework/pull/9104#issuecomment-109617363.

@taylorotwell if we just don"as" (which I'm fine with) should we automatically include a delimiter like "::" or "." so that you can just do "as" => "Assessment?" "::" seems consistent with view namespaces.
I can finish up the changes later today.

I'm rather against a default delimiter. If we have one I'd suggest .. In my opinion, :: is problematic with multiple nested groups. e.g. user::profile::posts.index doesn't look very nice.
Also if this is happening, it should definitely be configurable. Not everybody likes their route names the same way...

I also vote against a default delimiter.

I think that's better. Use as, also the final dot unnecessary. The router syntax would be intact.
Route::group(['prefix' => '/skills', 'as' => 'assessment'], function (Router $router) {
    // anything
});

I think you should be using route() instead of url() in the examples.

Yeah, that was a part of my initial troubles getting named routes to work.

In my opinion, :: is problematic with multiple nested groups. e.g. user::profile::posts.index doesn't look very nice.

Isn't that exactly how namespaced views look, though? I currently have a template that has this at the top:
@extends('Assessment::_layouts.default')

I also vote against a default delimiter.

Then I won't use one. :) Code updated and basically just changes the as_prefix key to as.

Thanks!


How to access the login page ?

I have followed all the steps given in the read me till run the .SQL file and how you explore the project but what is login route how do I login into the system ?
I am getting this error
Call to undefined method CI_session::has_userdata()

your-project-url/login is your login route and your-project-url/logout is the logout route.
please check whether you  load session library using $this->load->library('session'); It houses in application/config/autoload.php


Is this role supported on EL platforms?

As the title asks, I can only see Debian and Ubuntu listed on Ansible galaxy but can't see any obvious reason why this would not work on EL.

Hi, the reason why it's not on the list is cause I haven't tested on it.
Please feel free to do so and send me a PR. THX

OK thanks- I will do just that as this seems perfect for my needs.

Happy to report that I haven't encountered any issues with this on EL7. Thanks for the great work!

@jamieburchell thanks for testing


Remove stubbed subcommands Stop and Restart

Updating subcommand docs and found these were unimplemented. We can add them again later should we decide we need them. Please confirm that we're not expecting these for any special systemd startup type support. /cc @tyler-smith @cpacia @szollo

Coverage remained the same at 35.635% when pulling aeaea462c5d1a61decf69ae0d2439b8bb945ea4c on remove-stubbed-commands into fc81139985cdd9cc5d0610f2f3577e3ce910d2df on master.


encode file chunks to base64

Hello! I want before file sending encode it to base64. How i can make this? I am not understand how i can get file to encode it?
when i write code like this:
preprocess :function(chunk){
chunk = $.base64.encode(chunk);
chunk.preprocessFinished();
},
I get error:
Uncaught TypeError: Object W29iamVjdCBPYmplY3Rd has no method 'preprocessFinished'
so how i can encode to base64 chunk content? Is it impossible?

We are discussing how to implement a feature that would allow this in a library that is a fork of this one in this ticket: https://github.com/flowjs/flow.js/issues/42.
Once done it would be quite simple to back-port it to this library too.
Comments and feedback from the resumable.js community would be very useful and appreciated.
/cc @AidasK


Installing on hostinger

I have a hosting account at hostinger. Is it possible to install it on my hosting?

👋 @ugintl
If they support containers, then it should be possible.


[lspconfig] Cannot access configuration for mdx_analyzer.

Description
I installed mdx_analyzer via Mason and it says it is not in server_configurations.md but actually is. I use Lunarvim 1.3.
require "lspconfig".mdx_analyzer.setup({})

[lspconfig] Cannot access configuration for mdx_analyzer. Ensure this server is listed in `server_configurations.md` or
 added as a custom server.

I installed and let other LSP clients to be attached successfully. The code below is using Lunarvim API, but as to mdx_analyzer it does not even recognize, so I have to use lspconfig API directly.
require("lvim.lsp.manager").setup("gopls")
require("lvim.lsp.manager").setup("lua_ls")

can't reproduce . you can try to update your plugins first.

Turns out that the branch of nvim-lspconfig was fixed to the old one from last year. Managed to update it by git chekout master and get the latest server configurations. It was a bug of either Lunarvim or Lazy.nvim.


Add docs for running a proxy when using JFRD as daemon

We should document for customers on how to configure JFRD as a daemon to work with their proxy.
This will involve setting up a proxy, testing, and documenting.

See: https://github.com/newrelic/newrelic-jfr-core/issues/147

The current docs point out the generic way to set proxy configurations that affect all connections on a JVM.  This issue is for setting up a proxy config specific to the jfr-daemon and the underlying Telemetry SDK #167


HSEARCH-3398 Move PojoWorkPlan to SPI (for real this time...)

In ab07d7d5fd6652b7e92e3adfd9b0820980feb631 I did everything necessary
to move PojoWorkPlan to SPI (i.e. I made sure to wrap it with APIs where
needed), but I forgot to move the interface itself to SPI...

Pull Request Test Coverage Report for Build 1

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

Totals

Change from base Build 74:
0.0%

Covered Lines:
11693

Relevant Lines:
13484

💛  - Coveralls


"X-Forwarded for header handling" setting in Proxy Server has no effect

Env
16.1.3
LAN->Proxy Server->OpenVPN GW
Steps to reproduce

Got to Proxy server
Turn on Advanced mode
Set X-Forwarded for header handling to none
Click on the Apply button
Make a request to a web server you control
Check the value of HTTP_X_FORWARDED_FOR

Expected result
X-Forwarded-For: unknown
Actual result
X-Forwarded-For: <random public IP>, <LAN IP>

I've checked the config at /usr/local/etc/squid/squid.conf and there is no sign of forwarded_for

your using the wrong option, you need to set "Set forward header to unknown (off)".
The field is optional, so "none" is default.

Got it, thanks.