FreeBSD/Arm Static Build?

Looking to make this work on a Netgate SG-3100
[https://store.netgate.com/SG-3100.aspx]

Yeah, that would be nice.    Anyways, if you know how to cross compile on FreeBSD or how to get Qemu working on MacOS with FreeBSD/ARM that would help a lot.  Otherwise, honestly, this issue probably going to just sit here.

http://matrossi.blogspot.com/2011/08/cross-compiling-ports-for-arm-under.html
http://ray-freebsd.blogspot.com/2011/09/cross-compiling-ports-for-freebsd.html
https://github.com/freebsd/freebsd-src
But need a larger VM since ran out of disk space running make TARGET=arm toolchain

managed to cross compile freebsd 12.2 arm64 via qemu!
followed: freebsd qemu wiki
downloaded: freebsd 12.2 arm64
# terminal 1
$ brew install qemu telnet
$ qemu-system-aarch64 -m 4096M -cpu cortex-a57 -M virt  \
        -bios edk2-aarch64-code.fd -serial telnet::4444,server -nographic \
        -drive if=none,file=~/Downloads/FreeBSD-12.2-RELEASE-arm64-aarch64.qcow2,id=hd0 \
        -device virtio-blk,drive=hd0 \
        -device virtio-net,netdev=net0 \
        -netdev user,id=net0

# terminal 2
$ telnet localhost 4444
$ pkg install git gmake libpcap go
$ git clone https://github.com/synfinatic/udp-proxy-2020.git
$ cd udp-proxy-2020
$ CGO_ENABLED=1
$ go build -o udp-proxy-2020 cmd/*.go
$ scp udp-proxy-2020<EMAIL_ADDRESS>
thanks for this great project!

Neat.  Thank you @jrc2139 .  I'll see if I can take this and make regular FreeBSD/arm releases.

Neat. Thank you @jrc2139 . I'll see if I can take this and make regular FreeBSD/arm releases.

thanks!  i did come across some docker qemu options.. but i had already spent so much time trying other things, i just wanted to get it working 😅

@jrc2139:  Would you please post your binary so I can try it?

@jrc2139: Would you please post your binary so I can try it?

freedbsd12.2_arm64.zip

@jrc2139:  Thank you!

@jrc2139:  Unfortunately I'm getting "./udp-proxy-2020: Exec format error. Binary file not executable"
I'm trying it on a Netgate SG-3100 running pfSense+ 21.05-RELEASE (arm) on an ARM Cortex-A9 r4p1 running FreeBSD 12.2-STABLE

@jrc2139: Unfortunately I'm getting "./udp-proxy-2020: Exec format error. Binary file not executable"
I'm trying it on a Netgate SG-3100 running pfSense+ 21.05-RELEASE (arm) on an ARM Cortex-A9 r4p1 running FreeBSD 12.2-STABLE. The ARM Cortex-A9 is a 32 bit processor, so that's the problem. Could you generate a 32-bit build?

did you give my instructions a try?  just download the arm flavor of freebsd 12.2

@jrc2139 :   I'm not able to do that.   I'd need somebody to build it for 32 bit ARM

@jrc2139 : I'm not able to do that. I'd need somebody to build it for 32 bit ARM. Here is the link for qemu recipies for armv7
https://wiki.freebsd.org/QemuRecipes
And use the Pandaboard ISO from Here: https://download.freebsd.org/ftp/releases/arm/armv7/ISO-IMAGES/12.1/
(because it's based on the Cortex-A9)

oh that's a bummer that there's no vm-image for arm

Okay, I have some binaries for testing:

arm64
armv6
armv7

Aaron:
Great!  Thank you!!!   I will try out armv7 for you!
Regards,
*B
From: Aaron Turner @.>
Sent: Wednesday, September 29, 2021 7:49 AM
To: synfinatic/udp-proxy-2020 @.>
Cc: Bradley Feldman @.>; Author @.>
Subject: {Disarmed} Re: [synfinatic/udp-proxy-2020] FreeBSD/Arm Static Build? (#53)
Okay, I have some binaries for testing:

arm64 https://synfin.net/misc/udp-proxy-2020-0.0.7-freebsd-arm64-static
armv6 https://synfin.net/misc/udp-proxy-2020-0.0.7-freebsd-armv6-static
armv7 https://synfin.net/misc/udp-proxy-2020-0.0.7-freebsd-armv7-static

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/synfinatic/udp-proxy-2020/issues/53#issuecomment-930102157 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAE5YAWPEBOSEYNKUSDFAMLUEL4KHANCNFSM4XTQJSTA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .   https://s3.amazonaws.com/msv5/images/spacer.gif
[ { @.": "http://schema.org", @.": "EmailMessage", "potentialAction": { @.": "ViewAction", "target": "https://github.com/synfinatic/udp-proxy-2020/issues/53#issuecomment-930102157", "url": "https://github.com/synfinatic/udp-proxy-2020/issues/53#issuecomment-930102157", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { @.": "Organization", "name": "GitHub", "url": "https://github.com" } } ]
This message has been scanned for viruses and
dangerous content by  http://www.mailscanner.info/ MailScanner, and is
believed to be clean.
--
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.

The armv7 binary is running on Netgate SG-3100 (ARM Cortex-A9 ) FreeBSD 12.2-STABLE  pfsenxe 21.05.1-RELEASE
THANK YOU!
*B
From: Aaron Turner @.>
Sent: Wednesday, September 29, 2021 7:49 AM
To: synfinatic/udp-proxy-2020 @.>
Cc: Bradley Feldman @.>; Author @.>
Subject: {Disarmed} Re: [synfinatic/udp-proxy-2020] FreeBSD/Arm Static Build? (#53)
Okay, I have some binaries for testing:

arm64 https://synfin.net/misc/udp-proxy-2020-0.0.7-freebsd-arm64-static
armv6 https://synfin.net/misc/udp-proxy-2020-0.0.7-freebsd-armv6-static
armv7 https://synfin.net/misc/udp-proxy-2020-0.0.7-freebsd-armv7-static

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/synfinatic/udp-proxy-2020/issues/53#issuecomment-930102157 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAE5YAWPEBOSEYNKUSDFAMLUEL4KHANCNFSM4XTQJSTA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .   https://s3.amazonaws.com/msv5/images/spacer.gif
[ { @.": "http://schema.org", @.": "EmailMessage", "potentialAction": { @.": "ViewAction", "target": "https://github.com/synfinatic/udp-proxy-2020/issues/53#issuecomment-930102157", "url": "https://github.com/synfinatic/udp-proxy-2020/issues/53#issuecomment-930102157", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { @.": "Organization", "name": "GitHub", "url": "https://github.com" } } ]
This message has been scanned for viruses and
dangerous content by  http://www.mailscanner.info/ MailScanner, and is
believed to be clean.
--
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.

glad to hear it!


Permission Denied error running supervisor_flask_server.py on pi4

web page at <IP_ADDRESS> tried to load and then this failure happened:
pi@raspberrypi:~/github/ThermostatSupervisor $ python supervisor_flask_server.py

Serving Flask app "supervisor_flask_server" (lazy loading)
Environment: production
WARNING: Do not use the development server in a production environment.
Use a production WSGI server instead.
Debug mode: off
Traceback (most recent call last):
File "supervisor_flask_server.py", line 86, in
threaded=True)  # threaded=True may speed up rendering on web page
File "/usr/lib/python3/dist-packages/flask/app.py", line 943, in run
run_simple(host, port, self, **options)
File "/usr/lib/python3/dist-packages/werkzeug/serving.py", line 814, in run_simple
inner()
File "/usr/lib/python3/dist-packages/werkzeug/serving.py", line 774, in inner
fd=fd)
File "/usr/lib/python3/dist-packages/werkzeug/serving.py", line 660, in make_server
passthrough_errors, ssl_context, fd=fd)
File "/usr/lib/python3/dist-packages/werkzeug/serving.py", line 577, in init
self.address_family), handler)
File "/usr/lib/python3.7/socketserver.py", line 452, in init
self.server_bind()
File "/usr/lib/python3.7/http/server.py", line 137, in server_bind
socketserver.TCPServer.server_bind(self)
File "/usr/lib/python3.7/socketserver.py", line 466, in server_bind
self.socket.bind(self.server_address)
PermissionError: [Errno 13] Permission denied

python supervisor_flask_server.py, is failure on pi4 (linux) but not Windows PC.  Windows PC hangs for issue #136

changes made:

created favicon for supervisor_flask_server.py to eliminate the 404 errors, favicon not showing up in chrome on either linux or windows due to it being local
flask ports < 1024 require root privileges on Linux, that's the root cause for the Permission denied error, running sudo resolves this issue.  Updated wiki to reflect this.

committing enhancements to supervisor_flask_server.py but it's still broken on pi4.
Popen command is not yielding any results, tried stdout=PIPE and STDOUT, neither worked.
working on windows server but not Linux / pi4


Accordion property: collapseAll={true} not working

Describe the bug
When using accordion component, cannot collapseAll doesn't work
To Reproduce
Steps to reproduce the behavior:

Import accordion
Use accordion component with property collapseAll={true}

Expected behavior
Type '{ children: Element[]; collapseAll: boolean; }' is not assignable to type 'IntrinsicAttributes & AccordionProps'.
  Property 'collapseAll' does not exist on type 'IntrinsicAttributes & AccordionProps'.ts(2322)```

**Screenshots**
<img width="1297" alt="Screenshot 2023-01-26 at 5 01 31 PM" src="https://user-images.githubusercontent.com/25948274/214825263-2fa32351-2154-4137-a0c1-fd4c59cd81fd.png">

**Project information:**

- Tailwind: 3.0.0
- Flowbite: 1.4.7
- Flowbite React: 0.1.2
- Type: CRA

Feature wasn't released yet.

I could work on this

This feature is already implemented, it wasn't released yet.


Update entity_linkers.py

Hi,
There were problems when running scispacy linker v0.2. In particular, there was not a config.cfg file to read. I solved that by installing the v0.4.
However, in the new release there are a few changes with respect to nlp.add_pipe.
I had to manually change your source code file entity_linkers.py to make it run:

Thanks for pointing that out!

Please use the earlier version of scispacy to get it working out-of-the-box otherwise it would require a lot of non-trivial changes.


Added Modal Component

Closes #N/A
Summary
Added a reusable modal component to UI made from existing code. The component is called "k-modal".

Local Tests
A combination of all parameters were used all at once to ensure functionality.
Associated Pull Requests
Childs
Description - Repositories that implemented new modal component.

https://github.com/kytos-ng/telemetry_int/pull/134
https://github.com/kytos-ng/mef_eline/pull/507
https://github.com/kytos-ng/topology/pull/210

The new modal component seems to be working fine, at least in the main UI repo. Imma start replacing it within the other napps and do some further testing.

@viniarck @rmotitsuki
Just updated the changelog.

The new modal component seems to be working on mef-eline without any issue.

@viniarck Correct me if I am wrong but I was only able to find 3 other repos that implemented modals within their UI.
It should be these 3:

https://github.com/kytos-ng/telemetry_int/pull/134
https://github.com/kytos-ng/mef_eline/pull/507
https://github.com/kytos-ng/topology/pull/210

@viniarck scratch that, there are two left, another one in topology and one in maintenance

I think that I was able to locate every single modal and update it with the new modal component.
I attached a link to the corresponding PR's to this PR.
All that's left is an additional modal within telemetry_int, but it has an additional check box, so I will have to make modifications to the existing modal component.
I also fixed some buttons within the maintenance UI.


Bugfix: AppManifest as part of release

Description

Azure DevOps PBI/Task reference

AB#[PBI/Task number]
Checklist

[ ] Vehicle App can be started with dapr run and is connecting to vehicle data broker

[ ] Vehicle App can process MQTT messages and call the seat service

[ ] Vehicle App can be deployed to local K3D and is running

[ ] Created/updated tests, if necessary. Code Coverage percentage on new code shall be >= 70%.

[ ] Extended the documentation in Velocitas repo

[ ] Extended the documentation in README.md

[ ] Devcontainer can be opened successfully

[ ] Devcontainer can be opened successfully behind a corporate proxy

[ ] Devcontainer can be re-built successfully

[ ] Release workflow is passing

As discussed, we could keep separation of the upload assets as you have implemented now and the chances on the triggers on Pull Request will be removed after testings.
Thanks!


Source packages failing to install on v51 of the SFDX CLI

Describe the bug
Release v51 of the SFDX CLI uses REST API by default for mdapi depoyments, and this is causing JSON parser errors.
https://github.com/forcedotcom/cli/issues/870
To Reproduce
Run any of the commands that utilise mdapi deployment e.g. sfpowerscripts:orchestrator:deploy, Install Source Package
Expected behavior
The deployment is queued successfully
Screenshots
{
  "status": 1,
  "name": "Error",
  "message": "JSON_PARSER_ERROR: Can not deserialize: unexpected array at [line:1, column:73]",
  "exitCode": 1,
  "commandName": "MdapiDeployCommand",
  "stack": "Error: JSON_PARSER_ERROR: Can not deserialize: unexpected array at [line:1, column:73]\n    at Request._callback (/Users/alan.ly/.local/share/sfdx/client/7.88.4-3b2e55c3f1/node_modules/salesforce-alm/dist/lib/core/force.js:590:29)\n    at Request.self.callback (/Users/alan.ly/.local/share/sfdx/client/7.88.4-3b2e55c3f1/node_modules/request/request.js:185:22)\n    at Request.emit (events.js:315:20)\n    at Request.EventEmitter.emit (domain.js:467:12)\n    at Request.<anonymous> (/Users/alan.ly/.local/share/sfdx/client/7.88.4-3b2e55c3f1/node_modules/request/request.js:1154:10)\n    at Request.emit (events.js:315:20)\n    at Request.EventEmitter.emit (domain.js:467:12)\n    at IncomingMessage.<anonymous> (/Users/alan.ly/.local/share/sfdx/client/7.88.4-3b2e55c3f1/node_modules/request/request.js:1076:12)\n    at Object.onceWrapper (events.js:421:28)\n    at IncomingMessage.emit (events.js:327:22)\n    at IncomingMessage.EventEmitter.emit (domain.js:467:12)\n    at endReadableNT (internal/streams/readable.js:1327:12)\n    at processTicksAndRejections (internal/process/task_queues.js:80:21)\nOuter stack:\n    at Function.wrap (/Users/alan.ly/.local/share/sfdx/client/7.88.4-3b2e55c3f1/node_modules/@salesforce/core/lib/sfdxError.js:171:27)\n    at MdapiDeployCommand.catch (/Users/alan.ly/.local/share/sfdx/client/7.88.4-3b2e55c3f1/node_modules/salesforce-alm/dist/ToolbeltCommand.js:248:46)\n    at MdapiDeployCommand._run (/Users/alan.ly/.local/share/sfdx/client/7.88.4-3b2e55c3f1/node_modules/@salesforce/command/lib/sfdxCommand.js:85:13)\n    at Config.runCommand (/Users/alan.ly/.local/share/sfdx/client/7.88.4-3b2e55c3f1/node_modules/@oclif/config/lib/config.js:173:24)\n    at Main.run (/Users/alan.ly/.local/share/sfdx/client/7.88.4-3b2e55c3f1/node_modules/@oclif/command/lib/main.js:27:9)\n    at Main._run (/Users/alan.ly/.local/share/sfdx/client/7.88.4-3b2e55c3f1/node_modules/@oclif/command/lib/command.js:43:20)\n    at Object.run (/Users/alan.ly/.local/share/sfdx/client/7.88.4-3b2e55c3f1/dist/cli.js:121:21)",
  "warnings": []
}

Platform Details (please complete the following information):

OS: macOS Catalina Version 10.15.7
Version [e.g. CLI Version  eg: 1.6.6]
Salesforce Version: Version 7.88.4
CI Platform: N/A

Additional context
Add any other context about the problem here.

Fixed in  https://github.com/Accenture/sfpowerscripts/releases/tag/Release_19.3


remove MPI dependency when only using LCI parcelport

Previously the LCI parcelport always depends on MPI. This PR removes this dependency.

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)(=)(=)InfoPropertyBeforeAfterHPX Datetime2022-05-31T12:57:29+00:002022-10-09T04:32:24+00:00HPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a955a749596a3d2c5e86c1b4ecb9aa874a39196934bHostnamenid01193nid00840Datetime2022-05-31T15:13:01.357969+02:002022-10-09T06:48:46.678111+02:00Compiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0ClusternamedaintdaintEnvfileComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch-InfoPropertyBeforeAfterHPX Datetime2022-05-31T12:57:29+00:002022-10-09T04:32:24+00:00HPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a955a749596a3d2c5e86c1b4ecb9aa874a39196934bHostnamenid01193nid00840Datetime2022-05-31T15:13:18.026239+02:002022-10-09T06:49:03.641400+02:00Compiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0ClusternamedaintdaintEnvfileComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)(=)(=)Stream Benchmark - Scale(=)(=)(=)Stream Benchmark - Triad(=)(=)(=)Stream Benchmark - Copy(=)(=)(=)InfoPropertyBeforeAfterHPX Datetime2021-11-10T19:14:21+00:002022-10-09T04:32:24+00:00HPX Commit71d8dbe40b737be02ea734f9d98fcc0b1e21f01a5a749596a3d2c5e86c1b4ecb9aa874a39196934bHostnamenid00120nid00840Datetime2021-11-10T20:28:18.266961+01:002022-10-09T06:49:18.855296+02:00Compiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0ClusternamedaintdaintEnvfileExplanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)(=)(=)InfoPropertyBeforeAfterHPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a95f7fa822a96dfcd4301eecdeb34af6c87daeaa464HPX Datetime2022-05-31T12:57:29+00:002022-10-09T21:39:19+00:00ClusternamedaintdaintEnvfileCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0Datetime2022-05-31T15:13:01.357969+02:002022-10-09T23:55:56.124711+02:00Hostnamenid01193nid00492ComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch-InfoPropertyBeforeAfterHPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a95f7fa822a96dfcd4301eecdeb34af6c87daeaa464HPX Datetime2022-05-31T12:57:29+00:002022-10-09T21:39:19+00:00ClusternamedaintdaintEnvfileCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0Datetime2022-05-31T15:13:18.026239+02:002022-10-09T23:56:13.039631+02:00Hostnamenid01193nid00492ComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)(=)(=)Stream Benchmark - Scale(=)(=)(=)Stream Benchmark - Triad(=)(=)(=)Stream Benchmark - Copy(=)(=)(=)InfoPropertyBeforeAfterHPX Commit71d8dbe40b737be02ea734f9d98fcc0b1e21f01af7fa822a96dfcd4301eecdeb34af6c87daeaa464HPX Datetime2021-11-10T19:14:21+00:002022-10-09T21:39:19+00:00ClusternamedaintdaintEnvfileCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0Datetime2021-11-10T20:28:18.266961+01:002022-10-09T23:56:28.149635+02:00Hostnamenid00120nid00492Explanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

Did a rebase on the current master branch.

bors merge


⚠ Mobitopup has degraded performance

In 84c0395, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3709 ms

Resolved: Mobitopup performance has improved in a9ea426 after 419 days, 3 hours, 55 minutes.


[Dialog] Consider using  element

The new dialog element has improved accessibility including focus trapper (if opened as a modal), and the browser support is good.
There are a few questions to answer:

Can we still transition using .show() / .showModal().

Might not be able to use svelte transition, but css transitions/animations.
No way to fade out backdrop on close (::backdrop element immediately removed)
It looks like we can pass the open prop instead of the using the DOM api, which would then allow svelte transitions, but I don't think you can open it as a model, and thus no ::backdrop, focus trapping, etc.

Do we use the built in ::backdrop pseudo element instead of using Backdrop
Clicking the backdrop / outside the dialog does not appear to be supported (still use Backdrop component?)
Scrolling while modal dialog is open is not restricted

Sadly, this doesn't appear to help any with portal-ing, stacking context (z-index, CSS inheritance, etc)
REPL experimentation

openModal action and transitions


if 'decoupled_mode' is set to True (streaming enabled), 'inflight_batcher_llm_client.py' only outputs one token."

"When I follow the ’Llama tutorial‘ and run tensorrtllm triton serve llama, if 'decoupled_mode' is set to False (streaming disabled), there are no issues with the output of 'inflight_batcher_llm_client.py'. However, if 'decoupled_mode' is set to True (streaming enabled), 'inflight_batcher_llm_client.py' only outputs one token."
'decoupled_mode' is set to False:

'decoupled_mode' is set to True:

How you are running the inflight_batcher_client.py script? Are you using streaming?

How you are running the inflight_batcher_client.py script? Are you using streaming?

Yes

I ran through this as well, can you explain how you handled that?


Fixed 2 step session loading from cookie

3 hours of debugging and 10 new lines of code later, finally fixed the auth request for stored sessions. More information about the workaround is in the source file :+1:

I should also add a test for this :+1:

I tracked this a little bit further:

For reference, here's the HTTP Cookie specs: https://tools.ietf.org/html/rfc6265#section-4.1.1
The culprit for stripping quotes is http-cookie: https://github.com/sparklemotion/http-cookie/issues/11
There are a few forks: https://github.com/sparklemotion/http-cookie/network with interesting patches, the main one being https://github.com/uksa/http-cookie/commit/e54c52badce150a4b7dd416f926a8b91be9ae517 (incomplete)

The current fix is a workaround. It has the following issues:

it only works for adding back quotes to the DES... cookie. If a new cookie has the same problem, we will have it as well
we might have other cookie issues in the future. See https://github.com/sparklemotion/http-cookie/issues/9

It might be better to fix http_cookie in the long run.

@lacostej Thanks for looking into this, great finds! I agree with you, it's a workaround, and also it requires the cookie name to start with DES which we don't know if it's always true.
I'll do a fork to the fastlane organisation and point to this one instead. What do you think?

@KrauseFx  I submitted a tentative fix for the quoting issue in http_cookie. See https://github.com/sparklemotion/http-cookie/pull/15
You may want to try it out ? I don't have 2 step session in my accounts so can't really easily try it. I tested it with pseudo code:
value = '"HSARMTKNSRVTWFla1+yO4gVPowH17VaaaxPFnUdMUegQZxqy1Ie1c2v6bM1vSOzIbuOmrl/FNenlScsd/NbF7/Lw4cpnL15jsyg0TOJwP32tC/NguPiyOaaaU+jrj4tf4uKdIywVaaaFSRVT"'

uri = URI.parse 'http://example'
cookie_str = "DES5c148586daa451e55afb017aa62418f91=#{value};Expires=Sun, 06 Nov 2020 00:28:06 GMT;Path=/"

puts cookie_str
c=nil
HTTP::Cookie.parse(cookie_str, uri) { |cookie|
  c = cookie
}

puts c.value
puts c.set_cookie_value

Now the to_yaml is still broken, mostly because I don't know how to make it work with ruby < 1.9
But you should be able t work-around it by serializing the cookie string instead. Not sure if that helps you.

Great @lacostej, I'll give it a try on Monday! :+1:

@KrauseFx I also fixed the YAML export/import i the PR.

I'll merge this for now, and we'll see what if they approve your PR. I didn't squash since the 2 commits are separate from each other :+1:
Thanks again @lacostej :+1:


"Forgot Password" click results in msal:acquireTokenSuccess Event

Library

"@azure/msal-angular": "^1.0.0-beta.5"
"msal": "^1.3.0"

Framework
"@angular/core": "8.2.3",
Description
When I start a Login UserFlow and the User clicks on Forgot your password?, I should get a redirect to my Page with an AuthError like this:
ServerError: AADB2C90118: The user has forgotten their password.
Correlation ID: xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx

The Redirect happens, but if i subscribe to the msal:loginFailure or msal:acquireTokenFailure events in the broadcastService, they are never triggered. Instread I get a msal:acquireTokenSuccess Event.
I guess the error has to be in the handleRedirectCallback method of the MsalService
Regression
I'm not sure why this is happening. It worked with beta4 when I implemented it. Now it isn't even working with beta4.
Configuration
MsalModule.forRoot(
{
	auth: {
		clientId: "**hidden**",
		authority: "**hidden**",
		validateAuthority: false,
		redirectUri: "**hidden**",
		postLogoutRedirectUri: "**hidden**"
	},
	cache: {
		cacheLocation: "localStorage",
		storeAuthStateInCookie: isIE // set to true for IE 11
	}
},
{
	unprotectedResources: [**hidden**],
	popUp: !isIE,
	consentScopes: [
	]
});

Expected behavior
Getting the right event.

@Hampfter Msal just passes back what the service returns. If the service doesn't throw an error, neither will msal. Are you getting a token in the response? What happens when it triggers your acquireTokenSuccess event? Also, want to confirm, this is a b2c scenario?

@Hampfter Can you try reproducing this bug with our Angular B2C sample? And then share the link to the fork here? Thanks!

You only need to change the login method in the app.component.ts and use loginredirect.
  login() {
    if (isIE) {
      this.authService.loginRedirect(loginRequest);
    } else {
      this.authService.loginRedirect(loginRequest);
    }
  }

If you now click Login -> "Forgot your Password?", you will get an acquireTokenSuccess Event. My local storage says different. It has entries like:
AADB2C90118: The user has forgotten their password.
Correlation ID: bfeac638-375e-4719-ab2a-c35201daa383
Timestamp: 2020-04-29 05:54:29Z

So my Implementation did react on a loginFailure Event in which payload i had this errormessage, so i could redirect to my PasswordReset UserFlow. In the acquireTokenSucces Event I'm not getting a token. This is the payload:
{"uniqueId":"","tenantId":"","tokenType":"","idToken":null,"idTokenClaims":null,"accessToken":"","scopes":null,"expiresOn":null,"account":null,"accountState":"f279adc8-dc67-4217-8701-b0f5a2008163","fromCache":false}
@tnorling I#m not getting a Token back. And yes, it is a b2c scenario

This is related to #1406. Will put up a PR to address.


Error when running script line 1: expecting '('     at  (/frida/repl-2.js:1)

When i run example script showing error. Thank you in advance.
import { JNIInterceptor } from "jnitrace-engine";

// Attach to the JNI FindClass method
JNIInterceptor.attach("FindClass", {
    onEnter(args) {
        // called whenever the FindClass is about to be called
        console.log("FindClass method called");
        this.className = Memory.readCString(args[1]);
    },
    onLeave(retval) {
        // called whenever the FindClass method has finished executing
        console.log("\tLoading Class:", this.className);
        console.log("\tClass ID:", retval.get());
    }
});

frida version:
16.1.3
Device:
Nox Emulator, Android Version 7. x86.

any help>>?


S2U-7 Fix typo in debug log message

https://sakaiproject.atlassian.net/browse/S2U-7

Nah, It was well hidden xD


Why is it that for the useAppContext hook all data is exposed, but in useCartContext only select values are?

I was creating a custom component when to my surprise I found that the useCartContext hook behaved very much differently from the useAppContext hook! Why is this the case? Why are only those select values exposed from the useCartContext hook?
useCartContext.js
import { useContext } from 'react';
import _get from 'lodash.get';

import CartContext from '../context/Cart/CartContext';

export default function useCartContext() {
  const [cartData, cartActions] = useContext(CartContext);
  const { getGuestCartInfo, placeOrder } = cartActions;
  const cart = _get(cartData, 'cart');
  const orderId = _get(cartData, 'order.order_number');
  const selectedPaymentMethod = _get(cart, 'selected_payment_method');
  const selectedShippingMethod = _get(cart, 'selected_shipping_method');
  const loaded = _get(cart, 'loaded');

  return {
    orderId,
    placeOrder,
    getGuestCartInfo,
    selectedPaymentMethod,
    selectedShippingMethod,
    loaded,
  };
}

useAppContext.js
import { useContext } from 'react';
import AppContext from '../context/App/AppContext';

export default function useAppContext() {
  return useContext(AppContext);
}

In the same category, why is there no generic useFormikContext hook? My code now looks as follows, which I find very ugly;
import { FormikContext } from 'formik';
import useAppContext from '../../../hook/useAppContext';
import CartContext from '../../../context/Cart/CartContext';

export default function useAddressStepContext() {
  const { errors, values } = useContext(FormikContext);
  const [, { setEmailOnGuestCart }] = useContext(CartContext);
  const { subscribeToNewsletter } = useAppContext();

We are now using a consistent approach for both useAppContext and useCartContext. see this: https://github.com/hyva-themes/magento2-hyva-checkout/blob/main/src/reactapp/src/hook/useCartContext.jsx
export default function useCartContext() {
  const [cartData, cartActions] = useContext(CartContext);

  return {
    ...cartData,
    ...cartActions,
  };
}

Due to this, we can access any data or action by destructuring assignment technique by referring the hook. An example,
const { cart, getGuestCartInfo} = useCartContext();

I also recommend to use this useAppContext and useCartContext in localised AppContext and CartContext.

@peterschriever I am closing this issue as it no longer inconsistent any more. Feel free to re-open this if you feel differently


Bug

Description:
A clear and concise description of what the bug is.
Task version:
Specify the task version
Platform:

[x] Ubuntu
[x] macOS
[x] Windows

Runner type:

[x] Hosted
[x] Self-hosted

Repro steps:
A description with steps to reproduce the issue. If your have a public example or repo to share, please provide the link.
Expected behavior:
A description of what you expected to happen.
Actual behavior:
A description of what is actually happening.

Closing this issue as it appears to be spam.


Can reload from context menu in examples

But according to this reply, the Reload button:

should appear only when the debug attribute of window is set to true, which is not the case by default

This issue on the C lib provides a workaround. I am going to try it.

I pushed the fix in the "reload_menu" branch, it works in OSX VM and Linux. Could you try it on OSX ?
Normally, you should have the context menu when debug set to true (window.set_debug()) and no context menu when false (default)

I can't open the menu anymore.


Feature: Prompt Step type

I think it could be useful to have a Prompt step type, which sets the value to a specified variable.
The main use case I'm thinking of is a confirmation prompt, likely when doing something dangerous. For example, rm -rf <dir>.
There could also be appetite to also allow free text input, or allow the user to select a value from a defined list.
Appreciate this could totally be out of scope though.

We're using inquire for built-in prompts (things like asking for tokens and building changesets). We could definitely expose some of those types through new steps. It's probably best to only add things as folks need them so we don't have to worry about designing things before necessary. So, for this use-case, maybe we add a Confirm step which builds an inquire::Confirm? Would you want a custom message, or should we start with a generic default, like "Continue?" or something.
Also, particularly for confirm, would you also want a command-line option that auto-confirms? So if you run something like knope workflow-that-needs-confirmation -y it doesn't prompt.

I think that would work perfectly, I wonder whether it should be a separate step or whether it should just be an additional argument within the workflow?:
[[workflows.steps]]
type = "Command"
command = "git switch -c release"
confirm = true

Probably better to make it a standalone step, so folks can confirm before doing other built-in steps. Like, confirm before creating a pull request or a GitHub release.


"Error: poetry not found! Install it according to the poetry docs." on Windows 10

Hi,
I'm on Windows10. Serverless environment:

Your Environment Information ---------------------------
Operating System:          win32
Node Version:              14.15.4
Framework Version:         2.42.0
Plugin Version:            5.1.2
SDK Version:               4.2.2
Components Version:        3.10.0

Version of plugin:

5.1.1

I'm using poetry and according to the documentation, this should work fine. From the doc:

If you include a pyproject.toml and have poetry installed instead of a requirements.txt this will use poetry export --without-hashes -f requirements.txt -o requirements.txt --with-credentials to generate them.

But I ran into this error when I tried to deploy:

PS > serverless deploy
Serverless: Generating requirements.txt from pyproject.toml...
Error ---------------------------------------------------
Error: poetry not found! Install it according to the poetry docs.
at ServerlessPythonRequirements.pyprojectTomlToRequirements (C:<path replaced>\node_modules\serverless-python-requirements\lib\poetry.js:34:13)

After some research I found this comment: https://stackoverflow.com/a/54515183/5759828 suggesting to use {shell:true}. As part of my testing I found that the output of spawnSync (line 17 in poetry.js) is:

error: Error: spawnSync poetry ENOENT

I then added shell:true to poetry.js like this:
const res = spawnSync(
    'poetry',
    [
      'export',
      '--without-hashes',
      '-f',
      'requirements.txt',
      '-o',
      'requirements.txt',
      '--with-credentials',
    ],
    {
      cwd: this.servicePath,
      shell: true  <--- added this
    }
  );

and now it works fine.

Same here, this issue was solved when I added shell: true as well.
Is there a fix coming for this?
I believe it tries to run poetry in node env instead of shell and does not find the binary that is already in our path for poetry CLI.


Add request caching to Dockerfile generator

When developing parts of the Dockerfile generation pipeline, and thus regenerating Dockerfiles over and over, many unnecessary HTTP requests are being made to https://api.adoptium.net, since the data is mostly unchanged. Adding a request cache with an expiration time of an hour allows this regeneration to run in under a second in most case, while not compromising on the correctness.

This will exacerbate the timing window already present in this logic, as the retrieval of the assets at the centre of the configuration/version/image_type loop is requested as the 'ga' which may change during the script run if you were unfortunate to be running while a Temurin release is underway - though I accept that is unlikely. The failure would be a skewed set of release images.

We could implement a parameter to skip cache? Then maybe say that for workflow_dispatch runs (e.g manual ones) it skips it? WDYT?

Hmm. I thought that workspace is cleaned before a build and thus the cache file is gone (i's in .gitignore), so that I assumed this patch would only be valid for local development, but not for CI. Do you keep files between builds?

Hmm. I thought that workspace is cleaned before a build and thus the cache file is gone (i's in .gitignore), so that I assumed this patch would only be valid for local development, but not for CI. Do you keep files between builds?

hmm yes this is a good point, this only affects local development. In which case I'm happy

Yes, I'm happy to leave as is based on the original post that ...
"When developing parts of the Dockerfile generation pipeline, and thus regenerating Dockerfiles over and over..."
This would not affect the usual usage. I did momentarily consider whether it should have a guard only set for local development, but that seem unnecessary. Deploying for production and in GHA it is a single run through and we wouldn't be making repeated calls to the same API anyway.


Add IsNullCallbackArg

It took some digging to find out that NULL is passed into custom functions as a nil byte slice: https://github.com/mattn/go-sqlite3/blob/f08f1b6b9ce62b2496d8d64df26c1e278887bc1c/callback.go#L229
It would be nice to have this function s.t. future callers just need to call IsNullCallbackArg to determine whether the argument to their function is null.
Also, some minor auto-formatting.

Codecov Report
Patch coverage: 60.00% and project coverage change: +0.52 :tada:

Comparison is base (f08f1b6) 46.72% compared to head (5ad36e6) 47.25%.

:exclamation: Current head 5ad36e6 differs from pull request most recent head d4bc7eb. Consider uploading reports for the commit d4bc7eb to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1170      +/-   ##
==========================================
+ Coverage   46.72%   47.25%   +0.52%
==========================================
  Files          12       12
  Lines        1526     1528       +2
==========================================
+ Hits          713      722       +9
+ Misses        671      663       -8
- Partials      142      143       +1

Impacted Files
Coverage Δ

sqlite3.go
52.86% <ø> (ø)

callback.go
62.31% <60.00%> (+3.78%)
:arrow_up:

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


Update contribution doc

Title: [MINOR] docs: update and improve contributing guidelines and how-to-build
What changes were proposed in this pull request?
This PR includes the following changes:

Updated Table of Contents: Added new sections for IntelliJ and VS Code setup.
Added IntelliJ Setup Instructions: Detailed steps for setting up IntelliJ on Windows using WSL.
Added VS Code Setup Instructions: Detailed steps for setting up VS Code on Windows using WSL.
Improved Content Structure: Corrected and enhanced the content structure for better readability and usability.
Removed Unnecessary Blank Lines: Cleaned up the formatting for consistency.
Added References to Other Files: Added links to other relevant documentation files (e.g., README.md, GOVERNANCE.md, ROADMAP.md) to guide contributors for better project understanding.

Why are the changes needed?
The changes are needed to:

Provide comprehensive and clear instructions for setting up the development environment using IntelliJ and VS Code on Windows.
Improve the documentation structure and readability, making it easier for new contributors to follow.
Ensure all necessary steps are documented to help contributors set up their development environment correctly, which can reduce setup-related issues and maintain productivity.
Maintain consistency and clarity in the documentation to enhance the overall contributor experience.
Provide references to other key documentation files to help contributors understand the project context better.

Does this PR introduce any user-facing change?
No, this PR does not introduce any user-facing changes. It only updates the documentation for contributors.
How was this patch tested?
The changes were reviewed for clarity, accuracy, and completeness. Since this is a documentation update, no code changes or tests are applicable. The updated documentation was visually inspected to ensure it renders correctly.

Also, please create a related issue and update the PR title like other PRs.

Also, please create a related issue and update the PR title like other PRs.

ok, done!


Can't create new plugin instance

Describe the bug
When trying to create a plugin instance through the dashboard (namely slack-converation and slack-contacts), instead of creating the instance I receive the following error message:

str type expected

This is the HTTP response body (status code is 422):
{
	"detail": [
		{
			"loc": [
				"body",
				"project",
				"name"
			],
			"msg": "str type expected",
			"type": "type_error.str"
		}
	]
}

To Reproduce
Steps to reproduce the behavior:

Go to Projects -> "default" -> Plugins -> New
Choose slack-conversation and fill in details
Click on "save" button

Expected behavior
Plugin Instance is created
Screenshots
Error Message:

Desktop (please complete the following information):

OS: Linux
Browser Tested on Chromium and Firefox
Dispatch release: https://github.com/Netflix/dispatch/releases/tag/v20220915

Additional info
I'm trying to debug this, but I never used Vue so it might take a little of time.

Thanks for the report. We will try to see if we can repro it on our side and circle back as soon as we can.

Just tested on master and it worked, so maybe it was fixed?


Update events (and ther cards) to display message when there's nothing to display

For example, the events card should say something like "There's no upcoming events".

This functionality was added to cards in commit d388cf9e07afdcf1b1829699fe0dd73815f37024 and to the landing pages in commit 8b7538672785c2d4a4883014f6f7c6435717e96e.


Lag / Desync with the latest commits

I updated from June 19th to the latest as of July 21st. Everything looks okay until you play as client. You can see when going from 0 to sprinting there is a lot of obviously visible lag and jitter that was not there before. Suspect to do with the new direction dependant speed?

I confirmed this by installing into a fresh third person template project just now. Same issue everything default
It doesn't happen every time but at least 1 in 3 times when going from dead speed, holding shift, then holding W

https://streamable.com/pyw1sw

https://streamable.com/pyw1sw

Could you please reupload the video?
I tried to reproduce this issue, but lags usually only occur under very bad network conditions, but I am aware that the current implementation of direction-dependent movement speed is not ideal, so I added a setting to disable it in UAlsMovementSettings.

Still can't reproduce this issue, I can only get something like this under bad network conditions, which is actually expected to happen.

I did as you said, but still no luck. There shouldn't be any lags if you have network simulation turned off at all, so make sure it's really off and other actions, like rolling, work without lags.

Fixed in 3d4d50291d1f7bc1fa4c8d2b1f6c0add0820319e.
In ALS, the change of velocity depends on acceleration, and acceleration depends on velocity, so there is a kind of loop that usually causes no problems except in this case.
The CV_Als_AccelerationAndDecelerationAndGroundFriction_Normal curve controls this dependency, but since there was constant interpolation between the acceleration for running and sprinting, for some unknown reason it caused the server to not be able to change the character's speed correctly, and so it pulled the client back thinking the client wasn't sprinting, causing lags.
Removing the constant interpolation seems to have fixed this issue, but I'm still not quite sure why it was causing problems on the server.


End-to-end migration guide

Opening this early so everyone can see the direction it's going.
Fixes #760

(Got a review and an OK from Colton in private chat.)


Fix variable names for $remainingPackages

This PR fixes the issue where new preview packages are not being properly onboarded in the docs.

Resulting changes from running fixed onboarding script -- https://github.com/Azure/azure-docs-sdk-java/pull/2106/files
Docs CI running with this change -- https://apidrop.visualstudio.com/Content CI/_build/results?buildId=250792&view=results
docindex pipeline execution -- https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1077681&view=results

@scbedd @mssfang

@danieljurek  apparently these changes makes it so that we can't NOT error out if there is an issue during the find dependencies stage of the script.
It's completing all the way to the end, then melting.

This is ready for review now @weshaggard
While these changes should go in the complete fix will depend on merging: https://github.com/Azure/azure-sdk-tools/pull/1977
Until then the docindex pipelines can fail if the last invocation of an external command (e.g. mvn in the case of Java) exits with a nonzero exit code.


Limit to Premium

The application data replication setting will only be available for customers with a Premium subscription.
The setting will be applied to all environments that are provisioned, for existing environments the setting will not be updated. The setting is applied to environments with a Standard, Premium or Premium Plus resource pack, it will not be applied to environments with a Basic plan.

@feddovanede Reviewed - shall I publish directly or wait for the go ahead?


Issue managing net.tcp , net.pipe, netmsmq, and msmq.formatname bindings.

The manage_binding.pp file defers to managing iis bindings by using the New-Webbinding cmdlet. However if you attempt to manage the binding types referenced above using this cmdlets you will not get the desired results.
For example if you have a net.tcp binding With Binding Information of 808:* You could create a resource definition like this :
iis::manage_binding { 'webapi_site_binding2':
    ensure       => 'present',
    site_name    => "webapi",
    protocol     => "net.tcp",
    port         => "808",
    ip_address   => '*',
    require      => true,
  }

However this would produce a binding of net.tcp *:808, rather than net.tcp 808:* . Mitigation would be to use the New-ItemProperty cmdlet for these binding types.
New-ItemProperty -Path "IIS:\sites\sitename" \
  -Name Bindings -value @{protocol="net.tcp"; bindingInformation="808:*"}

Going to create some logic in the existing manage_binding.pp file to do this, unless there is already work underway, OR if there is a way to resolve this difference in binding using the New-Webbinding cmdlet.

i'm utterly confuzzled as to what 808:* even means

ip address:port:host
Any request coming in over port 808 route to any hostname. I know TCP does not tag hostname values (which might be why you are "confuzzled")  this is a standard port binding for WAS or WCF implementations. Some articles on it out there.

Hi. I'm going to close this Issue. We deprecated the module and the new version is available here: https://forge.puppet.com/puppetlabs/iis


Add my entry to the guestbook

@shritesh - I'm sorry I didn't see this PR sooner. Thanks for demoing this at the meetup. You're welcome to reopen this PR and sign the guestbook if you like! :)


Add timeout option to CreateContainer, RunContainer, RunPodSandbox

What type of PR is this?

/kind feature
What this PR does / why we need it:
Add timeout option to CreateContainer, RunContainer, RunPodSandbox
This option allows users to test their CRI implementation with a timeout,
similarly to how the kubelet runs requests.
Signed-off-by: Peter Hunt<EMAIL_ADDRESS>Which issue(s) this PR fixes:

Special notes for your reviewer:
Does this PR introduce a user-facing change?

Add `timeout` option to create, run, runp requests, allowing users to cancel CRI requests after a timeout

haha @ the PR number :smiling_imp:


Request time log level

It would be nice if it was possible to configure the log level based off of the total request time.
For example:
Request time < 300ms --> verbose
Request time < 1000ms --> info
Request time < 5000 ms --> warning
etc.
These changes should achieve that.

Hi,
Thank you for your PR 👍
I think the idea makes sense, but if we want to do something like this, I would maybe prefer something more "generic" which would be to allow passing a Function that would decide what ErrorLevel to log to based on all the info we have at the moment of emitting the log event.
For instance, I could imagine that client code would do something like :
SerilogWebClassic.Configure(cfg => cfg
  .UseLogger(myCustomLogger)
  .LogAtLevel((ctx, duration) =>  LogEventLevel.Debug)
));

where`:

ctx would be a `HttpContextBase
duration would be a TimeSpan (clearer than a long where you possibly don't remember if it's miliseconds, seconds or ticks etc)

This means we would have an override of LogAtLevel that accepts a Func<HttpContextBase, TimeSpan, LogEventLevel>. When such a "dynamic" rule is defined, we would then use it to determine the level at which to log in https://github.com/serilog-web/classic/blob/69cb8af00c4c04572893d4426a7b577d5511fcc1/src/SerilogWeb.Classic/Classic/WebRequestLoggingHandler.cs#L47
If you feel like trying such a PR, I could try and help you in the process. Be aware that I don't have a .NET dev environment set up currently, but I'll do my best :P

Hi again,
Thanks for your feedback. I have had a chance to look through all your suggestions now and they all make sense to me but I have run into one scenario where I do not know which path I should take.
ApplicationLifecycleModule.cs
/// <summary>         /// The level at which to log HTTP requests. The default is Information.         /// </summary>         [Obsolete("Obsolete since v4.1 - Use SerilogWebClassic.Configure(cfg => cfg.LogAtLevel(level))")]         public static LogEventLevel RequestLoggingLevel         {             get => Config.RequestLoggingLevel;             set => SerilogWebClassic.Configuration = new SerilogWebClassicConfiguration(                 isEnabled: Config.IsEnabled,                 requestLoggingLevel: value,                 requestFilter: Config.RequestFilter,                 requestContextLogLevel: Config.RequestContextLogLevel,                 formDataLoggingLevel: Config.FormDataLoggingLevel,                 customLogger: Config.CustomLogger,                 logPostedFormData: Config.LogPostedFormData,                 shouldLogPostedFormData: Config.ShouldLogPostedFormData,                 filterPasswordsInFormData: Config.FilterPasswordsInFormData,                 filteredKeywordsInFormData: Config.FilteredKeywordsInFormData             );         }
By removing the RequestLoggingLevel property I am not sure what can be returned from the 'get' of this property as it could be called in places outside of a http request context right? I am not sure if it is safe to remove this property as I believe it will break existing client projects?

Hi @KieranSasseSolarGain

I have run into one scenario where I do not know which path I should take:

Oh yes, good point I didn't think of this.
This property has been deprecated since v4.1 (almost 3 years ago) , and we are now in version v5.x , so I think it's probably ok to introduce a little bit of breaking changes.
In order to minimize the impact, I would maybe remove only the getter of the property , hoping that people is not relying it on it too much. I'm expecting that people is mostly using the setter anyway.
Does that unblock you ?

Hi again @tsimbalar,
I have now had a chance to revisit this PR and I believe I have made the changes you have requested.
I have assumed that the new LogLevelEvaluator function should only be set from the ConfigurationBuilder stuff as everything else on the ApplicationLifecycleModule has been marked deprecated.
I also just removed the getter on the RequestLoggingLevel property as you suggested.
Please let me know if anything else is required,
Cheers

Hi Kieran !
This looks good, thanks so much for going until the end 💪
I haven't done it in a while, so let's see if I remember how it's done, but let me merge this and publish a release :)

Published under v5.1.66 !
https://www.nuget.org/packages/SerilogWeb.Classic/5.1.66


Remove old BB wrappers

 Here is an overview of what got changed by this pull request:

Clones removed
==============
+ varats/varats/projects/c_projects/opus.py  -1
+ varats/varats/projects/c_projects/tmux.py  -1
+ varats/varats/projects/c_projects/lrzip.py  -1

See the complete overview on Codacy


Support passing configuration file name to mkdocs

mkdocs has the -f or --config-file option, this could support multiple configuration files, for translations/etc.

Thank you. +1

+1
Now it's not possible to build MkDocs documentation if th configurations file doesn't lie in doc or docs subdirectory. It would be nice to allow setting the path to the mkdocs.yml file instead of Python configuration file: (Advanced Settings) if MkDocs builder is used.

This was already implemented in https://github.com/rtfd/readthedocs.org/pull/4486

Wow thank you so much :)


added iso1(format 1)

ISO-1 (Format 1)
The ISO-1 PIN block format is equivalent to an ECI-4 PIN block format. The ISO-1 PIN block format supports a PIN from 4 to 12 digits in length. A PIN that is longer than 12 digits is truncated on the right.

It looks good! Let's fix those tiny cosmetic things and merge it! Thank you @mfdeveloper508 !


Hotfix/csl 1031

I don't have rights to push to cardano-sl-0.4-mid, this is what should be done.
Alas, it builds!

🤔

Okay, it seems I have no chances to understand our workflow in this life.

@gromakovsky Sorry for this, but explorer needs to be finished until monday, and I need to have this in order to see if an issue https://issues.serokell.io/issue/CSM-217 is still there.

needs to be finished until monday,

Ok, as I said in #cardano-sl-general this PR shouldn't exist in general, but I don't know how blocked you are (which implies that if you are blocked then exception can be made).

I need to have this in order to see if an issue https://issues.serokell.io/issue/CSM-217 is still there.

This reasoning is strange though. If you needed to test something, why couldn't you just point explorer to some temporary branch which is merge of cardano-sl-0.4-mid and this PR?

Honestly, it didn't even occur to me. :disappointed: Also, there seem to be a lot of issues, and I think it's somehow safer to merge it on the "right" branch and test it from there? Again, I understand the workflow, sorry for messing things up.

Also, there seem to be a lot of issues, and I think it's somehow safer to merge it on the "right" branch and test it from there?

Well, the point is that CSL-1031 will be eventually merged to cardano-sl-0.4 and it in turn will be eventually merged to cardano-sl-0.4-mid.

Again, I understand the workflow

It's good for you if you understand, because I am afraid I don't :)
Actually I see nothing bad in this merge except that it adds some mess to git history (some commits are merged now and then other commits with same changes will be merged later). It shouldn't be done if it's not necessary, but it seems to be necessary now, so I think it's ok.

And also I don't like when PRs are merged before checks finish :rage1:

And also I don't like when PRs are merged before checks finish :rage1:

I agree. Won't happen again.


changed naming of jobs & refactored"

please merge together with:
https://github.com/kiegroup/droolsjbpm-build-bootstrap/pull/495
https://github.com/AppFormer/uberfire/pull/791
https://github.com/mbiarnes/dashbuilder/tree/changeFileNames

This is the naming scheme suggested by @mbiarnes (next time please do share this kind of info directly in the comment, so that others (who are not part of the e-mail discussions) can have the same info)
daily builds:         <jop purpose>-kieAllBuild-<branch>
kie releases:         <job purpose>-kieReleases-<branch>
PR jobs:              <rep>-pullrequest-<branch>
normal build jobs:    <rep>-<branch>

This should be fine as long as we stick to it consistently.

@mbiarnes feel free to merge these changes if you are confident they are OK. I won't have the time to review them this week.


Allow more than 1 server flags packet per connection

Allow updating server feature flags in case we want to make them toggleable in future.

how hard would you think is to make server have feature flags be different per tracker, my original idea was for disabling mags at runtime

how hard would you think is to make server have feature flags be different per tracker, my original idea was for disabling mags at runtime

this is not a feature. "feature" is what server and/or tracker and/or sensor supports. what you're talking is config and should be treated as such.
i also think that this PR isn't needed too, but it's a trivial change so i'll merge it


Line # 9 was a duplicate to # 5

Duplicate line...
#include <string.h>

lol, why did you find this?

I didn't find it... it found me ;-)


sql: introduce SHOW SESSION TRACE, replace EXPLAIN(TRACE) by SHOW TRACE FOR

Roadmap item: Make EXPLAIN(TRACE) useful
PR: https://github.com/cockroachdb/cockroach/pull/16524

Make sure to integrate this into the troubleshooting content, e.g. https://www.cockroachlabs.com/docs/dev/query-behavior-troubleshooting.html

@andreimatei has agreed to help us out (in our time of 1.1 need) to create a first version of these docs! I'll create a stub page with the diagram and structure in place and he'll take it from there.

Hey @andreimatei, have you had time to get started on this?

getting started
On Mon, Oct 2, 2017 at 2:47 PM, Jesse Seldess<EMAIL_ADDRESS>wrote:

Hey @andreimatei https://github.com/andreimatei, have you had time to
get started on this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cockroachdb/docs/issues/1598#issuecomment-333628057,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXBcRk2xT2FynseF819CF3jF_DxBzisks5soS-tgaJpZM4N8SF2
.


CosmosClient: Adds CreateAndInitializeAsync Method

Pull Request Template
Description
Adds a CreateAndInitializeAsync method in CosmosClient. This creates a new client and initializes the containers provided by warming up the caches and connections
Closes: #1706
Sample
using Microsoft.Azure.Cosmos;
List<(string, string)> containersToInitialize = new List<(string, string)>
                                     { ("DatabaseName1", "ContainerName1"), ("DatabaseName2", "ContainerName2") };
CosmosClient cosmosClient = await CosmosClient.CreateAndInitializeAsync("account-endpoint-from-portal",
                                                                               "account-key-from-portal",
                                                                                containersToInitialize)

Hello, how would this work with the newly added Always Encrypted as .WithEncryption() returns a new client.


Load SkyX files on iOS where available

lgtm

Fixes https://github.com/domokit/sky_engine/issues/252


[BEAM-4496] Website merge

New Jenkins job to run with label git-websites and build and push the beam website from master into the asf-site branch of the beam repository.
The website is built by :beam-website:buildWebsite into build/website/content
This is copied into website/content and pushed to the asf-site branch.

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---
---

Run Seed Job

Run Seed Job

Run Website Merge

Commented out the git push, but before I uncomment it and try it, can @swegner PTAL?

Run Seed

Run Seed Job

Run Seed Job

Run Website Publish

@alanmyrvold let me know when this is ready to go in and I can merge it. Were you able to get a successful run on Jenkins? It looks like the most recent couple runs failed.


Caller's IP Address available via a GUC?

Hmmm.  I thought it was possible to retrieve the IP address of the Postgrest request being processed via a postgresql GUC.    Like the claims and http headers.  Seems it isn't.
Any thoughts as to how to obtain this from the postgresql side?

@bradleyf Are you proxying with nginx?
If so, you could extend the config here with:
proxy_set_header   X-Real-IP          $remote_addr;

Then you could get the ip on pg side with: current_setting('request.header.x-real-ip', true)

Yes, I am.   I’ll give it a try.    Thanks!
From: Steve Chavez<EMAIL_ADDRESS>Sent: Wednesday, October 28, 2020 1:53 PM
To: PostgREST/postgrest-docs<EMAIL_ADDRESS>Cc: Bradley Feldman<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: {Disarmed} Re: [PostgREST/postgrest-docs] Caller's IP Address available via a GUC? (#358)
@bradleyf https://github.com/bradleyf  Are you proxying with nginx?
If so, you could extend the config here http://postgrest.org/en/v7.0.0/admin.html#hardening-postgrest  with:
proxy_set_header   X-Real-IP          $remote_addr;
Then you could get the ip on pg side with: current_setting('request.header.x-real-ip', true)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/PostgREST/postgrest-docs/issues/358#issuecomment-718105312 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAE5YASGNJ6ZTRNUBE4PWYTSNBK6NANCNFSM4TCUX7FQ .  https://s3.amazonaws.com/msv5/images/spacer.gif
[ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/PostgREST/postgrest-docs/issues/358#issuecomment-718105312", "url": "https://github.com/PostgREST/postgrest-docs/issues/358#issuecomment-718105312", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } } ]
This message has been scanned for viruses and
dangerous content by  http://www.mailscanner.info/ MailScanner, and is
believed to be clean.
--
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.

That worked!


RDK-39389:Updating the documentation

RDK-39389: Updated the documentation

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.MManja792 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Replace s3 with Azure storage in dev pipeline

Do the following replacement to avoid depending s3 in dev pipeline

Use Azure Storage to store dev artifacts
Use Azure Storage to store terraform state
Use github repo to store current-version

Thanks, please add a credentials-develop.yml


Travis maintenance

Use gcc-8 for build
Don't require travis sudo containers

oh but wabt seems failing

Yep it’s kinda expected : ) I’m using travis to test since I dont have linux machine : )


ImageMagick. NewMagickWandFromImage ?

Hello!
I need to load image from source (stream or bytes, not from file) to Wand
Is there a function for this? I found this:
NewMagickWandFromImage
WandExport MagickWand* NewMagickWandFromImage ( const Image *  image )
But I have problems with parameter 'Image *  image'
How can I implement this to Delphi?
Or maybe there is another way to load image directly to Wand blob in memory?
Thanks.
PS Sorry for my English )

you can use MagickReadImageBlob for this
    if ALImageMagickLib.MagickReadImageBlob(aWand,
                                            aPicStream.Memory,
                                            aPicStream.Size) <> MagickTrue then RaiseLastMagickWandError(aWand);


[question] Blobstore URL for bosh-init

This is more a question than an issue. The documentation at https://github.com/cloudfoundry/bosh-init/blob/master/docs/cli_workflow.md#12-sending-apply-message states
The agent serves a blobstore at `<mbus URL>/blobs` endpoint.

Is that really the case? In my understanding, the blobstore has a different port (as you can also see in the manifest). Nats, being used as the message bus, doesn't really provide any means to store blobs, right? So how does it actually work that blobs are being transferred and served? Can somebody please clarify the above sentence from the documentation?
Thanks!

NATS is not used by the bosh-init.
Agent is configured to use https transport when working with bosh-init (via mbus: "https://mbus:mbus-password@ELASTIC-IP:6868"). Agent in that mode has /blobs endpoint. In NATS mode it does not provide that functionality.
In full Director scenario all agents on deployed VMs are in NATS mode and that's we rely on a dav/external blobstore, from which agents can download blobs.
We are planning to clean up some of these mechanics soon when we switch from NATS to https for all communication, even between Director and agents.

Thanks for clarifying @cppforlife. I guess I'll undo my NATS related changes in https://github.com/cloudfoundry/bosh-init/commit/dbcac78246e5de481b11d22db16eaa81b039ffc7 – sorry for messing that up.


Password Strength Meter

Is your feature request related to a problem? Please describe. I'm always frustrated when I create passwords and am unsure of their strength. Without a clear indication of how strong or weak my password is, it's hard to ensure my accounts are secure.
Describe the solution you'd like I propose adding a password strength meter that visually indicates the strength of the password as the user types. This meter will provide real-time feedback, showing whether the password is weak, medium, or strong. This can help users create more secure passwords.
Describe alternatives you've considered
Third-Party Extensions: Using browser extensions to check password strength, but this approach isn't integrated into the platform and might not be consistent for all users.
Static Guidelines: Providing written guidelines on creating strong passwords. While helpful, they don't offer real-time feedback, which is crucial for immediate improvement.
External Tools: Using external websites for password strength checking, but this requires users to navigate away from PassOP, disrupting their workflow.
Additional context

Password strength varies depending on the context. How can you ensure if a password is strong or weak? For example, if a user is just adding their name with a special character and 2-3 numbers, how would you consider this?

@jinx-vi-0 Password strength depends on length, complexity, and uniqueness. A name with a special character and numbers (e.g., "John@123") is weak. To create a strong password, aim for 12-16 characters, mixing uppercase, lowercase, numbers, and symbols. by Avoiding common words and predictable patterns for enhanced security.

We are already checking for capital letters, special characters, and numbers. If we implement both functionalities, it would be chaotic. We can either provide suggestions to follow the above-mentioned rules or use a strength meter. What do you think about this? We can discuss this over discord dm or in the passop thread.

@jinx-vi-0 I agree that combining both functionalities could be chaotic. Providing clear suggestions for password rules might enhance user experience without overwhelming them. A strength meter can also be effective, but it might complicate things.

Exactly


create table to store jobs applied by grads

Story

As a grad

I Want to store which jobs I have applied for

So That I can have them listed on the dash board

Acceptance Criteria
id (int) pk
job_id (int)
user_id (int)
Implementation Notes

Checklist:

[ ] I have defined the user story
[ ] I have defined some acceptance criteria
[ ] I have included any implementation notes
[ ] I have included all relevant links

completed by refik


Any way to show loading?

I have tweets by ID.  ngTweet works great, but it would be very useful if I could provide a place holder while I wait for Twitter to render the tweet.
For example..
<twitter-widget twitter-widget-id="tweet" twitter-widget-options="{'cards': 'show', 'align': 'center', 'conversation': true, 'theme': 'light'}">
   [[ Loading tweet id {{tweet}} ]]
</twitter-widget>

Will render the loading indicator AND the resulting tweet from twitter appended.  Ideally, the resulting tweet would replace or hide the loading indicator.
Is there anyway to do this?

I'm not able to currently test this, but can you override the transclusion variant? e.g.
<twitter-widget twitter-widget-id="'617749885933232128'">

Sample loading content

</twitter-widget>

You could then try styling that a static placeholder using Twitter's recommended styling.  I'm not sure if the above will actually work.
I agree that a more formal solution would be useful, but I might need to complete #3 in order to do this elegantly. I'll add this to the roadmap, but I can't give a time estimate for implementation.
How would you like this to work? I could have it so you pass in a template URL that I then include, or it can also be transcluded in.  I don't want to force people to use template of my choosing, so I'm hesitant to use a boolean attribute (e.g. ngtweet-loading-template="true").

Edited my example in original question so the HTML shows.  Sorry about that.
What do you mean by "override the transclusion variant"? Setting transclude on the directive to false doesn't seem to work.
For me, ideally, I think the loading template would be fine.

"Override the transclusion variant" == "Replace the transcluded element"
I'll check to see if I can come up with a quick solution when I get the chance this week. Maybe tonight, probably by the weekend.

OK. Thank you.

I dug into this a bit. With the current version of the library, there is no way to do what you require.  That being said, I can implement a portion of the eventing component that should enable you to get a loading prompt.
This implementation would be as such:
<twitter-widget twitter-widget-id="parentCtrl.tweetId" twitter-widget-on-rendered="parentCtrl.doneLoading(parentCtrl.tweetId)">

This is a loading block

</twitter-widget>

Parent controller
parentCtrl.tweetLoading = false;
parentCtrl.tweeId = '1234234235';
parentCtrl.doneLoading = function doneLoading(id) {
    $scope.loading = false;
};

Would this meet your needs? If so, I think I can get a release out this week.

I've implemented this in the feature/factory-eventing branch.

Awesome!  That sounds like it would totally work.  I don't think I'll be able to try it out today, but first thing tomorrow.
Thank you!

Sorry.  I'm just now getting back to this.  So I've pulled ngTweet dependency in using bower.  What would be the best way to pull in the version on the factory-eventing branch?  Just replace what is in my bower_components/ngtweet with https://github.com/arusahni/ngtweet/archive/feature/factory-eventing.zip ?

You should be able to do it via bower install "ngtweet#feature/factory-eventing".

Oh.  Cool.  I didn't know I could do that.  I dl'ed the zip and replaced what was in bower and can confirm it works.  I had multiple tweets loading so I made tweetLoading --> tweetsLoading[] and adjusted the logic.

Cool. Yeah, the sample controller code was just a demo of a callback in action.  Glad it works!  I'll close this issue once it makes it into production.

This was released as part of v0.5.0 .

@arusahni , I just wanted to thank you again.  I used your preview release under #feature/factory-eventing and I've just upgraded to 0.6.  This has been a very useful directive.

@jduprey thanks for the kind words! Happy to hear that the library has proven useful. :+1:


Adds Laravel 6 support.

The following changes allow this to be installed on Laravel 6.x.
The one thing I had to do is update my use statements in each of my migration classes (not sure if this is normal):
from:
use Illuminate\Support\Facades\Schema;

to:
use Fisharebest\LaravelFloats\Schema

The one thing I had to do is update my use statements in each of my migration classes

You don't need to import the facade classes - the laravel autoloader does it for you automatically!
The mappings are defined here: https://github.com/laravel/laravel/blob/master/config/app.php#L221
This package overrides the default mapping here: https://github.com/fisharebest/laravel-floats/blob/master/composer.json#L39
But if you choose to import them, then you must obviously choose which Schema class you want.

I was thinking I didn't need to.  It should have autoloaded but it didn't for me. I even tried adding it to the aliases in app/config but it didn't work as well.  Maybe it is an issue with my version of Laravel (6.2).

Maybe it is an issue with my version of Laravel (6.2).

I haven't used Laravel 6 yet.  I guess I need to test it.

Tag 1.2.0 created.


Allow options ,inline,omitempty

Marshal failed with  https://github.com/kubernetes/api/blob/master/networking/v1/types.go#L418
May consider to support this case.

@dsnet Sure, it make sence.
I just want to confirm the tag options limit here will be landed for json/v2.


[TFLite][Frontend] Support quantized Reverse sequence

Support Reverse sequence quantization operation as part of #15148

Hi @tlopex this is failing on a linting error, can you fix that so perhaps we can progress this PR?


thurstmaster 458 wheel not connect v1.2.1

i installed this yesterday, it worked fine, and came back and it is not connect like on Asseto Corsa it doesn't say nothing connect on the
Xbox controller on Bluetooth devices its says thurstmaster 458 wheel and virtual Xbox game connected I don't know what should i do

I open xboxwheel app and click repair it fix it


Can't get it to launch

Hey! Thanks for working on this, I couldn't believe when I saw it.
I've done everything as in the README, but still nothing happens and the log files aren't even generated.
I'm running on latest everything (nightly neovim), and sourcekit lsp works normally.
Is there anything I could do to troubleshoot?
Also, how am I suppose to use the feline provider? I tried to add it as a provider to a component but it didn't work.

Hey @andrestone,
First can you check if the make command resulted in two valid binaries located in ~/.local/share/xbase/
Next, are you using a generator? what is the basic structure of your project. There is small chance your project wasn't detected.
One thing to try is to navigate to repo root and run make install_debug

Hey @kkharji! Thanks for the prompt reply and sorry for the delay.
Yes, compilation was successful.
 ls -la ~/.local/share/xbase
total 18624
drwxr-xr-x  4 andre  staff      128 Sep 18 12:51 .
drwx------  7 andre  staff      224 Sep 18 12:51 ..
-rwxr-xr-x  1 andre  staff  5975587 Sep 18 12:51 xbase
-rwxr-xr-x  1 andre  staff  3557204 Sep 18 12:51 xbase-sourcekit-helper

And no, I'm not using a generator. I'm opening vim with no arguments in the folder which the .xcodeproj folder is located.
Ideally, I'd like to use .xcworkspace as a root and would be able to navigate all files inside any projects inside that workspace, but I'm not sure if this already supported.
To be clear though, I'm testing it against a single project (no .xcworkspace even present in the folder).
Cheers!

hmm this is odd. Maybe something is broken with auto launcher.
lua require'xbase.server'.register(vim.loop.cwd()), I suggest first trying to run the server binary manually in a different terminal  ~/.local/share/xbase/xbase
Could share with me how you've setup xbase, my an issue there or does it look similar to
use {
  "xbase-lab/xbase",
  config = function ()
    require'xbase'.setup({
      log_level = vim.log.levels.INFO,
      log_buffer = {
        focus  = false,
        default_direction = "horizontal",
      },
      sourcekit = {
        on_attach = require'user.lsp'.on_attach(),
        handlers = require'user.lsp'.handlers,
        capabilities = require'user.lsp'.capabilities
      },
      simctl = {
        iOS = {
          "iPhone 13 Pro",
        },
      },
      mappings = {
        --- Whether xbase mapping should be disabled.
        enable = true,
        --- Open build picker. showing targets and configuration.
        build_picker = 0, --- set to 0 to disable
        --- Open run picker. showing targets, devices and configuration
        run_picker = 0, --- set to 0 to disable
        --- Open watch picker. showing run or build, targets, devices and configuration
        watch_picker = 0, --- set to 0 to disable
        --- A list of all the previous pickers
        all_picker = "<leader>ef", --- set to 0 to disable
      },
    })
  end
}

Yes, xcworkspace is supported to some extent.
Sorry for late response, I lost a family member to a heart condition this week.


Small bug fix

Hello. I have already written a new version of this lib, but doesn't push yet.
Because I realized that no one uses this library...
If you need it, I'll find a some time for it.

Could you still release a new version of the library?

Ok. Will be soon.)


New example showing user download waveform.

Hi reviewers, this is my first attempt to contribute. I wasn't able to do the "Signed-off-by: Name Lastname<EMAIL_ADDRESS>but my name is in there. Is there I way to add this to a commit via VS Code?
Signed-off-by: Gerardo Orozco<EMAIL_ADDRESS>What does this Pull Request accomplish?
This is a new RFSG example to show users how to create, normalize and download a user provided waveform. In this case, is multitone and all the code you need is included on this example.
This is specially important for users coming from Playback library. RFSG has now the functionality to do the same but the steps need to be follow correctly to get the exact power right.
Why should this Pull Request be merged?
It's an example.
What testing has been done?
The algorithm is tested with multiple code documented test cases and the limitations are noted there.
The example is written following as close as possible the same guidelines of the other examples at the time of commit.

Sorry, just now getting to this but you should be able to configure black locally to get the python formatted correctly. You can see this Wiki page we have for setting it up: https://github.com/ni/grpc-device/wiki/Development-Tips-and-Tricks#black-format-on-save

Note that black has configuration options in https://github.com/ni/grpc-device/blob/main/pyproject.toml
Running it locally will automatically use that configuration.

@bkeryan thank you, your pointes to the Black configuration helped.
I used: https://black.vercel.app/
I am not able to get it working with my VS Code for some reason.
Now that is passed there, can I assume you guys will give me comments on anything missing for it to make it to the main?

Hi reviewers, this is my first attempt to contribute. I wasn't able to do the "Signed-off-by: Name Lastname<EMAIL_ADDRESS>but my name is in there. Is there I way to add this to a commit via VS Code?

Honestly, I don't think we've done this across the board and also Squash Merge before committing to main so I need to look into the policy more. But anyways, I think in VS Code you can do it with the three little dot menu next to the repo in the source control tab:

I think there's also a more general setting in VS Code to enable it:

@bkeryan thank you, your pointes to the Black configuration helped. I used: https://black.vercel.app/ I am not able to get it working with my VS Code for some reason.
Now that is passed there, can I assume you guys will give me comments on anything missing for it to make it to the main?

Yes, I reviewed and added some more comments. And we have it set up pretty well with required Checks and policies so conversations have to be resolved and there has to be at least one approving reviewer.
The system tests (run_win_system_tests and run_ubuntu_system_tests) aren't currently marked required because some are intermittent but for this specific change those don't matter because they don't interact with the examples. Mainly just that validate_python step applies to the examples.

Hi reviewers, this is my first attempt to contribute. I wasn't able to do the "Signed-off-by: Name Lastname<EMAIL_ADDRESS>but my name is in there. Is there I way to add this to a commit via VS Code?

Honestly, I don't think we've done this across the board and also we Squash Merge before committing to main so I need to look into the policy more and see if we should enforce it more/less. But anyways, I think in VS Code you can do it with the three little dot menu next to the repo in the source control tab:

I think there's also a more general setting in VS Code to enable it:

Sign-off is for compliance with a DCO (Developer Certificate of Origin) and it is different than commit signing (which uses GPG to cryptographically sign commits).


df -k fails to select 1024-byte blocks

After recent changes (it worked a while ago), df -k fails to select 1024-byte blocks.
cc @fox0
% ./target/release/df | head -2
Filesystem     512-blocks       Used  Available Capacity Mounted on
/dev/disk1s4s1  976490576  152157960  824332616      16% /

% ./target/release/df -k | head -2
Filesystem     512-blocks       Used  Available Capacity Mounted on
/dev/disk1s4s1  976490576  152157952  824332624      16% /

https://github.com/rustcoreutils/posixutils-rs/pull/302

Fixed by #302


Bookmark page's footer is out of place

Issue
Footer on the Bookmarks page is out of place

Solution
CSS needs work

Will fix this

closing as it's a duplicate of #31.


Design the solution

It is necessary to model the solution:
Requirements:

Producers: 1:1
Consumers: 1:n

Feature: Develop a messaging system using Git repositories

    Scenario: Connect
        Given a Broker Client
        When the client trigger a new Event
        Then clone the repository
        And  checkout to that application (Every application will be a Git Branch) in the Broker

    Scenario: Produce an event
        Given a Broker Client
        When the client trigger a new Event
        And    fetch repository
        And    pull
        And    create a new File: EPOCH_MILLISECONDS + _ + NODE_NAME + _ + EVENT_NAME + .json
        And    include in the file the json with the message, if it is necessary
        And    commit the action
        And    push

    Scenario: Produce an event
        Given a Broker Client
        When the client need to review new events from the application
        And    fetch repository
        And    pull
        And    execute a processor
        And    create a new File: EPOCH_MILLISECONDS + _ + NODE_NAME + _ + OK + .json
        And    commit the action
        And    push

https://cucumber.io/docs/gherkin/reference/

Epoch:

https://www.logicbig.com/how-to/java-8-date-time-api/millis-to-date.html

Designed


stripe-cli: 1.21.10 -> 1.22.0

Release notes: https://github.com/stripe/stripe-cli/releases
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 361049

x86_64-linux

:white_check_mark: 1 package built:

stripe-cli

@ofborg: eval

@ofborg: eval

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/2145

@ofborg eval

Thanks for merging this, @RaghavSood, this is very helpful 👍


Updating author profile as per figma mockups

Fixes
Fixes https://github.com/creativecommons/project_creativecommons.org/issues/142  by @brylie
Description
Updated author profile as per figma mockups. Not overriding the vocabulary classes at the moment.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow [best practices][best_practices].
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

@brylie Wordpress is accepting arrays in all keys which is why. We can also just use the first element in the array. I will update the code to do that in case

@brylie @annatuma Here is the look of the page after all the updates.

@shailee-m, how difficult do you think it might be to use a different image size here? From what I can tell, WordPress seems to be using the 'post-thumbnailformat returned bythe_post`. Is that correct?
I don't want the image format to be a blocking issue in this PR so I'm just asking about the difficulty. We can consider whether the image format needs to change and split that into a follow-up task if needed.

This looks good and is ready to merge from what I can see.
The only change would be to resolve the conflict(s).

Resolved conflicts


[proposal] i18n-extract alias should be one letter long

🤖  It is more like improvement
Command

[x ] xi18n

Description
i18n-extract is an alias for xi18n which looks a bit weird as all the rest available CLI commands aliases are like one letter long ref: https://angular.io/cli#command-overview  -and it is make sense.
Proposal
So the proposal is to change i18n-extract alias to one letter alias, for example x

Hi @kuncevic,
Thanks for this feature request. We have discussed this during yesterdays team meeting and we feel that adding an alias is not necessary and can cause confusion here.
Here's some points that were brought up:

Not all commands have aliases
x doesn't really represent a good alias for ng i18n-extract.
A single letter alias can be confusing at times and we'd like to move away from them in the future, in favor of more smarter aliases. Example ng d is not intuitive to which command it an alias of, as it can be of both ng deploy or ng doc.
The alias for i18n extraction will be removed in the next major version.

Yeah i c, alias it is something that easier to type however it is a bit an exercise to type xi18n
Based on that most of the aliases seems like just a first letter of the command and it is make sense. So x seems to me is a good candidate aliasing xi18n as x is the first letter of xi18n command and secondly it is the second letter of the word eXtract.
Speaking of other commands I think short commands like new, doc, etc does not need an aliases as it is very easy to type 3 letters, but if command has more than 3 letters it is a great candidate for aliasing. So based on that the doc commands may not need aliasing but it make more sense to alias deploy command as d

So based on that the doc commands may not need aliasing but it make more sense to alias deploy command as d

This is actually one of the reasons why deploy doesn't have an alias, this is because a single letter alias for a command is ambiguous and users were using ng d as a shortcut for ng doc instead of ng deploy. Hence, we decided to remove the alias from ng deploy as it was harmful as users might have unwilling causing their application to be deployed.
The xi18n is actually also ambiguous by itself for new users and in fact in version 11, we will be removed this command in favor of extract-i18n.
Another reason why certain commands have aliases is that they can are typically used more frequently such as ng serve, ng build and ng generate, while deploy, xi18n, run, config, update are used far less frequently.
We do agree that we should provide some form of aliases to commands, but we feel that a single letter aliases is not ideal at this point.


Violet-theme

add a violet theme
Screenshot of my card:

For assistance: https://help.github.com/en/articles/file-attachments-on-issues-and-pull-requests

My violet theme

Hi! This is a great start, but currently doesn't pass the WCAG requirements to be accessible.
If you could adjust the colors so that the A11y card displays "AA" or "AAA" compliant we can merge it in.
As an additional note: It looks like you added a - in your username, which is causing the image to not render.

Sorry about that, i'm sending again a new color theme, if can you see, I appreciate it.

Looking much better!
Thanks for the addition 😄


add param to 'start' that starts project with custom session name/socket

this allows projects to be reused
fix #143

Has the new gem version been updated? And will there be a new field on the config yaml file?
What I want to know is, is there's an option currently like when you would do:
tmux new-session -t session_name

So that even though it's the same session, you don't have the same windows open.

@magicalbanana I have cut and released 0.6.11.
mux start project someothername should work.

:+1:  thanks for the merge! Been using this the last 6 months and it's an integral part of my dev cycle. Now the only feature that I'm missing is passing params from the command line into argument slots inside the yaml config.


Dart code builders

It would be great to also provide a code builder when registering a type. This way, we could generate Dart code and have better performance too when needed.
I'm seeing here a use case where the XML layout would be used when designing widgets (not only developers), and then the code builders would be used to create Dart widgets for developers.
The code_builder is useful for that purpose.

Looks good, Will be optimized in the future.

Added the builder to generate code in v1.0.0. There is an example in README.md.


Fix silences data update after creating, editing or expiring a silence

Without this fix, the UI does not update to reflect the changes made until the next poll of the silences API.
Also upgrades dynamic-plugin-sdk to v0.0.18 to fix a type error with the Silence type introduced by this change.

/retest-required

/lgtm


Change theme class Typography members to type BaseTypography

Feature request type
Other
Component name
No response
Is your feature request related to a problem?
When overriding the default MudBlazor theme, it's pretty common for me to either want to reuse BaseTypography objects or generate them from a generic factory, but this is not possible because each member of Typography is typed to its implementation instead of relying on its base class.
Describe the solution you'd like
Change the type of each member of Typography to be of type BaseTypography. Since every member of Typography is just an instance of BaseTypography with different default values, this will allow devs to use more generic patterns for generating typography rules. E.g.:
// Current
public class Typography
{
    public Default Default { get; set; } = new Default();
    public H1 H1 { get; set; } = new H1();
    // etc.
}

// Proposed
public class Typography
{
    public BaseTypography Default { get; set; } = new Default();
    public BaseTypography H1 { get; set; } = new H1();
    // etc.
}

Have you seen this feature anywhere else?
Not explicitly, but this is proposal is in line with OOP best practices.
Describe alternatives you've considered
None, this is just a quality of life improvement
Pull Request

[X] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@danielchalmers @henon reopened, pls add to v8 milestone, we should make BaseTypography as abstract to avoid the problem.


docs: Fix namespace typo in install-helm-release.md

A trival fix to misspelled namespace.

This is in more than one place, please replace flux_system in the template also. Thanks

Thanks @stefanprodan I've fixed that as well as signedoff my commits to make the DCO happy.


HDDS-2840. Implement ofs://: mkdir

What changes were proposed in this pull request?
Implement mkdir in ofs://
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-2840
How was this patch tested?
This is work-in-progress. Hasn't been completely tested at the moment.
Writing new test cases and adapting some old test cases to the new FS.

Status update:

For now mkdir works in ozone fs shell. mkdir -p will create volume and bucket if they don't exist. See commit message ebd2bf8e28976f370e8fc17ecf42b1b3c4497a3e how I tested it. Note: Hasn't been tested throughly!
In order for the code to compile after rebase to latest dev branch, I have manually applied HDDS-2777 and HDDS-2785 to OFS classes (as they were copied from existing o3fs classes and some APIs has changed).

Status update:

This PR is ready for review.
This diff should help for the review, the Base is the commit after class copying (except RootedOzoneClientAdapterFactory): https://github.com/smengcl/hadoop-ozone/compare/0bee28acadfb2c358c6f008173e7eca6ed7fa23f...smengcl:HDDS-2840
I have also applied HDDS-2188 to affected OFS classes in order for code to compile.


oneof's with keyword names not escaped

In 2.10.2, if we have any oneof fields with the field name as a Rust keyword, such as
message Message {
  oneof move {
    string s = 1;
  }
}

The generated code will fail to compile with a message similar to:
error: expected identifier, found keyword `move`
    --> src/proto/client.rs:2467:14
     |
2467 |         self.move = ::std::option::Option::None;
     |              ^^^^ expected identifier, found keyword
     |
help: you can escape reserved keywords to use them as identifiers
     |
2467 |         self.r#move = ::std::option::Option::None;
     |              ^^^^^^

The fix seems to be similar to: https://github.com/stepancheg/rust-protobuf/issues/471

Must be fixed now.


App Coming Up With White Screen

Error Description:  Unknown class _TtC7ReSmack22SWRevealViewController in Interface Builder file.

Running on Xcode 9.2 9C40b
MacOS Sierra 10.13.4
iOS Simulator 11.2

Select the SWRevealViewController-> Go To IdentityInspector-> Custom Class-> Module Name -->Remove the Module Name and Run the Application Again That Will Help


Single PR for changes needed for the dev-tag

Upstream PRs:

https://github.com/zephyrproject-rtos/zephyr/pull/30412
https://github.com/zephyrproject-rtos/zephyr/pull/30422

and https://github.com/zephyrproject-rtos/zephyr/pull/30421

Some checks failed. Please fix and resubmit.
checkpatch (informational only, not a failure)
-:610: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#610: FILE: drivers/ieee802154/ieee802154_nrf5.c:417:
+    !defined(CONFIG_NRF_802154_SER_HOST)$

-:857: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#857: FILE: include/net/socket.h:850:
+    char ifr_name[IFNAMSIZ]; /* Interface name */$

- total: 0 errors, 2 warnings, 1930 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

Your patch has style problems, please review.

NOTE: Ignored message types: AVOID_EXTERNS BRACES C99_COMMENT_TOLERANCE CONFIG_EXPERIMENTAL CONST_STRUCT DATE_TIME FILE_PATH_CHANGES MINMAX NETWORKING_BLOCK_COMMENT_STYLE PREFER_KERNEL_TYPES PREFER_SECTION PRINTK_WITHOUT_KERN_LEVEL REPEATED_WORD SPDX_LICENSE_TAG SPLIT_STRING VOLATILE

NOTE: If any of the errors are false positives, please report
      them to the maintainers.

Gitlint issues
Commit 51a884eaf3:
1: UC5 Title exceeds max length (76>75): "[nrf fromtree]: boards: arm: nRF5340: add a note about required nRF CL tools"
Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages.

@shantha-14 added a second commit for the platform updated in the BLE rpmsg sample.

picked up the commit from #403 as well

Some checks failed. Please fix and resubmit.
checkpatch (informational only, not a failure)
-:610: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#610: FILE: drivers/ieee802154/ieee802154_nrf5.c:417:
+    !defined(CONFIG_NRF_802154_SER_HOST)$

-:857: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#857: FILE: include/net/socket.h:850:
+    char ifr_name[IFNAMSIZ]; /* Interface name */$

- total: 0 errors, 2 warnings, 1951 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

Your patch has style problems, please review.

NOTE: Ignored message types: AVOID_EXTERNS BRACES C99_COMMENT_TOLERANCE CONFIG_EXPERIMENTAL CONST_STRUCT DATE_TIME FILE_PATH_CHANGES MINMAX NETWORKING_BLOCK_COMMENT_STYLE PREFER_KERNEL_TYPES PREFER_SECTION PRINTK_WITHOUT_KERN_LEVEL REPEATED_WORD SPDX_LICENSE_TAG SPLIT_STRING VOLATILE

NOTE: If any of the errors are false positives, please report
      them to the maintainers.

Gitlint issues
Commit 8f65f38767:
1: UC5 Title exceeds max length (87>75): "[nrf fromlist] samples: bluetooth: hci_rpmsg: change the allowed platform to nRF5340 DK"
Commit 51a884eaf3:
1: UC5 Title exceeds max length (76>75): "[nrf fromtree]: boards: arm: nRF5340: add a note about required nRF CL tools"
Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages.


Retries for _query_cvd_version_dns. Fixes #49.

Hi @micahsnyder,
I recently noticed that the _query_dns_txt_entry() might fail from time to time and that there are no retries, so I created #49.
Here's a small PR that hopefully fixes the issue.
Would you be so kind and have a look if it's suitable for your project?
Thanks,
Tim

@micahsnyder, no problem at all.
Do you think that this change could be released?
The latest release 1.1.1 is from 2022-07-28.