Incompatibility with create-react-app

This is followup to PR  https://github.com/sindresorhus/ip-regex/pull/12
sindresorhus/ip-regex is in dependency chain of this package
sindresorhus/public-ip > sindresorhus/is-ip  > sindresorhus/ip-regex
In PR you wrote that this sindresorhus/ip-regex is targeted for node (which i totally understand) and same probably goes for sindresorhus/is-ip, but this package and sindresorhus/public-ip are targeted also for browser.
Right now you can't use this package or sindresorhus/public-ip in scenario with webpack + uglifyjs without explicitly setting up babel for this modules or using babel-engine-plugin like you suggested in PR.
Problem is when you have React app build using https://github.com/facebookincubator/create-react-app
where by default you can't change webpack without ejecting whole configuration.
More details here
Ejecting while configuration only to use one module is a bit overkill in my opinion.
So my question is do you have any other idea how this module could be useable with react-scripts without making this package (and packages in dependency chain) published with es2015 versions ?
Or could you at least improve README with info that it requires babel-engine-plugin to work with uglifyjs@2?

This is a shortcoming of create-react-app and should be resolved there. Everyone always just wants to solve their own immediate problem quickly instead of thinking bigger and solving the underlying problem. I'm not going to change anything for webpack or create-react-app, sorry.
See https://github.com/sindresorhus/ama/issues/446 and https://github.com/facebookincubator/create-react-app/issues/1125.


IpHelper::getCidrBits() enh #6

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Tests pass?
❌

Fixed issues
#6

Depend on
#4

I see less sense in the check method as the getter itself checks.
The invalid IP test does not work properly because it depends on #4.

Seems to fail tests...

Seems to fail tests...

After https://github.com/yiisoft/network-utilities/pull/8 PR the test will be correct.

#8 was merged. Can you push recent master into this pull request branch?

Merged. The tests run correctly.

Merged. Thank you!


Config

Move all vulnerable data to a separate configuration file from which to read the table identifier, telegram bot token, name of the file with access to the Google API.

https://github.com/MeinLiX/NotificationAndPlanner/pull/3

closed in pr https://github.com/MeinLiX/NotificationAndPlanner/pull/3


call of overloaded 'abs(long unsigned int)' is ambiguous

When i tried to compile. its gave me lots of "call of overloaded 'abs(long unsigned int)' is ambiguous" error

I am getting the same results?  How can we fix this.  Thanks

I have the same problem too.......

It is because the program tries substracting two non-negative numbers from eachother without conversion. Removeing the abs (with leaving the brackets in place) from the lines where it throwes an error would fix the problem without breaking anything.
I think.


DocuConverter.jar giving error on compile

Error: MethodHandle.invoke and MethodHandle.invokeExact are only supported starting with Android O (--min-api 26)
When I compile app this error comes im a lil new please help
All I did was add jar file in libs folder in android project please help
This is my min and max api
applicationId "com.example.myapplication"
minSdkVersion 17
targetSdkVersion 29
versionCode 1
versionName "1.0"
it will be fix if I do minSdkVersion 26 but if I do that there be no point for converter since it will decrease range too much so does this docuConverter only works for latest smartphone ?

The library is build on java 8, so it can not be used on android below API 26


Titanfall 2 + Northstar crashes while loading dxvk

Software information
Game: Titanfall 2 + Northstar (Regular game works fine)
Software: Pop_OS, Steam Proton (tried every version from Experimental to 5.0_10, and 7.3-GE-1), DXVK 1.10.1
Launcher: Steam
System information

GPU: 1080ti
Driver: 510.69.02
Wine version: in Software information
DXVK version: 1.10.1 also tried master branch

Apitrace file(s)
not traceable, file isn't even created.
Log files
none are created, seems like the game has trouble actually loading dxvk. here's the northstar log file if that'd help
https://pastebin.com/ngBj7f5u
Northstar does work with wined3d (obviously very poorly). Also, regular game works fine. I can provide more logs, however by myself I only could get the northstar log. Seems like the northstar launcher crashes while loading dxvk.

Are you trying to start the client or server version?

Are you trying to start the client or server version?

Client version.

This is caused by Northstar shipping with a broken DXVK release. You need to copy the latest release's dlls into Titanfall2/bin/x64_retail, and then installing the latest DXVK release into your prefix. Here's the latest release of Northstar running on DXVK:

Wait, I'm a bit confused. The only file we ship in that directory is the wsock32.dll (Titanfall2/bin/x64_retail/wsock32.dll) which shouldn't be related to DXVK in any way...

we're chatting about this on discord, let's see what wacky stuff is going on here

To follow this up based on discussions on Discord, it seems the "botched" DXVK version is being downloaded by Steam itself. I'm putting "botched" in quotes as it works fine with vanilla game but Northstar does a few things differently that cause it not to work, the cause of which is yet to be determined.
Northstar only ships d3d11.dll for dedi and that's it. bin/ folder of v1.7.2 release for example:
$ tree bin/
bin/
├── x64_dedi
│   ├── d3d11.dll
│   ├── GFSDK_SSAO.win64.dll
│   └── GFSDK_TXAA.win64.dll
└── x64_retail
    └── wsock32.dll

2 directories, 4 files

The d3d11 version they ship for the dedicated server is a pure stub.

Wine should be default configured to use native D3DX libraries and none of what any game ships with

Turns out the DXVK dlls were created by ReShade not Steam.

It was ReShade causing it that I previously installed on my Windows machine. Damn.


No support for MIME message encoding for both headers and entity body

Description:
In ballerina.mime, the function encode(blob content) (blob) bounded for MimeBase64Encoder struct, only supports encoding a blob type. So according to the current implementation, we are only able to encode an entity body part with a blob content.
        mime:Entity textBodyPart = {};
         textBodyPart.setBlob(bodyText.toBlob("UTF-8"));
         var getBlobResponse = textBodyPart.getBlob();
         mime:MimeBase64Encoder encoder = {};
         match getBlobResponse {
                 blob msgBlob => {
                         blob blobEncode = encoder.encode(msgBlob);
                 }
                 mime:EntityError err => io:println(err);
         }
However, the Entity struct represents both headers and body of a message. Since there is no support for encoding whole message with both headers and entity body, we are not able to use ballerina.mime support for sending a base64 encoded email message with headers, body and message attachments in ballerina gmail connector.
Steps to reproduce:
Affected Versions:
0.964
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Please note that base64 is one of the values we provide in "content-transfer-encoding" header which indicates the type of transformation that has been used in order to represent the body in an acceptable manner for transport. That does not include headers. Only the body should be encoded. So it is only the entity body that you should encode not the whole entity.
And can you please provide more info regarding your requirement so that i can see what the exact issue is.
Here's the spec btw > https://www.w3.org/Protocols/rfc1341/5_Content-Transfer-Encoding.html
Thanks.

@dushaniw
A new method called "getBodyPartsAsChannel()" has been introduced to convert the given parts as a channel as it this is needed for gmail connector API.
Can you please verify whether this is working.

Output of 'getBodyPartsAsChannel()' on a multipart entity wraps the whole message with a boundary which is not needed for Gmail connector. Therefore, a new method is needed to convert the Entity to a byte channel. Will inform you once that is implemented Dushani.
Sample
--556b0b8b339b42ef
To<EMAIL_ADDRESS>Subject: Test
From<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Bcc<EMAIL_ADDRESS>content-type: multipart/mixed;boundary=adce710aa16eebb8
--adce710aa16eebb8
content-type: multipart/related;boundary=8dfa22047e1fd565
--8dfa22047e1fd565
content-type: multipart/alternative;boundary=a564970075426c29
--a564970075426c29
content-type: text/plain
text body
--a564970075426c29
content-type: text/html
HTML BODY
--a564970075426c29--
--8dfa22047e1fd565--
--adce710aa16eebb8--
--556b0b8b339b42ef--

Since gmail connector is not using MIME, we are closing this.


Error occurred when executing MaskComposite

Hi I'm trying to test the workflow json, but I'm get this error, I don't know what I'm doing wrong!
Thanks for share your work, i really appreciated all this knowledge!
Error occurred when executing MaskComposite:
too many indices for tensor of dimension 2
File "C:\Users\Usuario\ConfyUI\ComfyUI_windows_portable\ComfyUI\execution.py", line 153, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
File "C:\Users\Usuario\ConfyUI\ComfyUI_windows_portable\ComfyUI\execution.py", line 83, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
File "C:\Users\Usuario\ConfyUI\ComfyUI_windows_portable\ComfyUI\execution.py", line 76, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
File "C:\Users\Usuario\ConfyUI\ComfyUI_windows_portable\ComfyUI\comfy_extras\nodes_mask.py", line 244, in combine
destination_portion = destination[:, top:bottom, left:right]

Will fix soon, thank you for reporting!

Should be fixed in https://github.com/Sxela/ComfyWarp/releases/tag/v0.2.0


TPC-H benchmark

The scope of this PR is to build a benchmark for TPC-H plans in Velox.
Both DWRF and Parquet file formats are supported. Since DWRF does not support the Date type, the filters are adjusted accordingly.
The TPC-H data must be specified using the data_path option.
Each table data must be placed in a hive-style partition.
Sample invocation
cmake-build-release/velox/benchmarks/velox_tpch_benchmark --num_drivers 16 --data_format parquet --data_path /data/tpch10
============================================================================
velox/benchmarks/TpchBenchmark.cpprelative  time/iter  iters/s
============================================================================
q1                                          2.93s  340.75ms
q6                                         926.40ms     1.08
q18                                        12.56s   79.65m
============================================================================

cmake-build-release/velox/benchmarks/velox_tpch_benchmark --num_drivers 16 --data_format orc --data_path /data/tpch10
============================================================================
velox/benchmarks/TpchBenchmark.cpprelative  time/iter  iters/s
============================================================================
q1                                          1.81s      553.58m
q6                                          566.12ms     1.77
q18                                        9.21s  108.53m
============================================================================

The benchmark also supports run_query_verbose option that can be used to print the plan along with execution statistics for a particular query.

We can benchmark TPC-H queries over a bigger scale factor periodically.

@pedroerp This benchmark is currently disabled in CI. We can enable it if we add TPC-H data generation as part of the CI job.

@9prady9 I compared q1 against DuckDB on my local laptop for 16 threads and parquet data for SF10. The time reported by DuckDB was Total Time: 2.48s. So Velox Parquet was slower, but DWRF was faster.
Let me know your thoughts on this refactor as well.

@mbasmanova can you take another look? thanks.

@mbasmanova thanks very much! I will move the TpchQueryBuilder to a separate one.

@majetideepak Deepak, thank you for iterating on the PR. There are some macos-build failures. Would you take a look?

@majetideepak Deepak, do you have some instructions on how to run this benchmark, specifically, what should be the layout of the data_path?
CC: @Yuhta

@mbasmanova I wrote in the description that the layout should be hive-style partitioning.
Example: if --data_path=/mnt/data then the layout will look like. It is probably best to add a comment to the code regarding this. I will add one.
/mnt/data/nation
/mnt/data/orders
......

@majetideepak Thank you, Deepak.


Add google analytics measurement id

Can we confirm that the twitter card and the other metadata is visualised properly prior to merging


IC-1324 / IC-1326 / IC-1328 cancel a referral with a reason

What does this pull request do?

add contract and service layer method for cancelling a referral

add contract and service layer method for getting referral cancellation reason

What is the intent behind these changes?
allow referrals to be cancelled by the PP with a specific reason

@rgforsyth added you here for visibility about the 'additionalComments' arg


Add lxml dependency to README

Hi, nice project!
When trying to execute the notebook I found an error about lxml missing, so I thought that could be something worth mentioning on the README

Great spot @rogererill. I only realised this for myself earlier in the week as I started work on making this a CLI app. I'm hoping to finish it this weekend so would love your opinion on it.


Fix line break in Ubuntu installation instructions

Signed-off-by: Robert Lemke<EMAIL_ADDRESS>

bot, test pull request

@rh-atomic-bot r+

bot, add author to whitelist

bot, retest this pleaaw

bot, retest this please

@robertlemke You need to sign your PR.
git commit -a --amend -s
git push --force
Should fix this.

@robertlemke Are you still interested in fixing this?

I don't think this is a bug.


Add Nick to staff

On the staff page, add Nick Krzemienski.
Nick is a US Marine veteran and full stack mobile engineer. He is the founder of Koda Labs (http://kodalabs.com), a boutique mobile development studio. He is volunteering his time to help out and jumpstart our Deploy program.

Just submitted a pull request for this. Let me know what you think.


Modified the README.md to include a second example

Modified the README.md to include a second example where the account SID and
auth token are embedded in the code rather than stored as environmental
variables. This would allow a developer to compile a self-contained executable
that could be run by end users without the need to create or understand
environment variables. Also modified both examples to define sending and
receiving phone number bindings to make it easier for the developer to
read/configure.

This change is

@markandrus Hi Mark. It seems like this PR is over a year old. Would like to know if you want to update it and try to merge it into master, or if it is a good candidate for closing.
(P.S., haven't actually looked at it, just the date it was opened).

Sorry, I kind of dropped the ball on @asparks1805 PR. Some notes:

@asparks1805 identified a useful function, parseCredentials, that we hadn't exported. It'd probably be useful to export this.
I don't think we should land the "using SID and secret embedded in the code" example, since this is kind of an anti-pattern (someone could extract the credentials from the binary). Twilio's other examples prefer environment variables here.
Otherwise, I do think binding the variable names in the example is useful (e.g., toPhone, etc.)—I get the order confused whenever I re-read it.

I'm also reminded of this suggestion to improve some of our examples.
I don't think we should merge this as-is. Maybe we can run with some of the bullet points above, though?


 It looks like contour was not able to watch the api server.  Is your cluster healthy? Does it pass sonobouy conformance tests?

Reply to this email directly, view it on GitHub, or mute the thread.

@davecheney

Is your cluster healthy?

Everything works as expected yes, I don't believe it is related to the cluster itself, it seems that everything stems from the errors from envoy.

Does it pass sonobouy conformance tests?

I will try, but I'm still in 1.12.8 so I have either to upgrade it to 1.13 or disable the preflight checks.
I will get back on you on that

Thank you. What is driving me towards the API server are these log lines
from contour
"2019-07-17T20:35:58.392Z","E0717 20:35:58.392346       1
reflector.go:134]
pkg/mod/k8s.io/client-go@v0.0.0-20190226174127-78295b709ec6/tools/cache/reflector.go:95:
Failed to list *v1beta1.TLSCertificateDelegation: Get
https://<IP_ADDRESS>:443/apis/contour.heptio.com/v1beta1/tlscertificatedelegations?limit=500u0026resourceVersion=0
https://<IP_ADDRESS>/apis/contour.heptio.com/v1beta1/tlscertificatedelegations?limit=500u0026resourceVersion=0:
dial tcp <IP_ADDRESS>:443: connect: connection refused "
What is <IP_ADDRESS>, is it your api server?
On Thu, 18 Jul 2019 at 20:15, Victor Noël<EMAIL_ADDRESS>wrote:

@davecheney https://github.com/davecheney
Is your cluster healthy?
Everything works as expected yes, I don't believe it is related to the
cluster itself, it seems that everything stems from the errors from envoy.
Does it pass sonobouy conformance tests?
I will try, but I'm still in 1.12.8 so I have either to upgrade it to 1.13
or disable the preflight checks.
I will get back on you on that
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/heptio/contour/issues/1250?email_source=notifications&email_token=AAABYA7CGK3RXX2YW3GHQELQAA7E7A5CNFSM4IEYNNBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2IAQ6A#issuecomment-512755832,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAABYAZIANHC3ZWOZDCN7FTQAA7E7ANCNFSM4IEYNNBA
.

@davecheney I don't know, I thought it was the ip of the envoy container on the same pod, but I suppose you are right, the error messages seem to be about the API server! I will investigate this, thank you for the clarification

Thank you. Sorry about the error message, it comes from the k8s client library and isn’t very good. Please let us know what you find out.

On 18 Jul 2019, at 20:31, Victor Noël<EMAIL_ADDRESS>wrote:
@davecheney I don't know, I thought it was the ip of the envoy container on the same pod, but I suppose you are right, the error messages seem to be about the API server! I will investigate this, thank you for the clarification
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hey @victornoel, were there any errors when applying the manifest to your cluster? I'm wondering if the service account or RBAC permissions script failed causing the error.

@stevesloka no there wasn't, but let me clarify something: contour works perfectly all the time, except sometimes, this happens, it crashes and then starts again and everything works again perfectly.

That's interesting @victornoel. I'm not sure this is a Contour issue. It seems like the probes are doing their job and restarting Contour. The only other approach I'd maybe recommend is switching to the split model where Contour is deployed as a deployment and Envoy as a daemonset.
In this model, if this problem happens again, the instance of Contour will be restarted and Envoy should continue to serve traffic until Contour comes back online.

@stevesloka what you say makes sense yes.
There are two things I'm not clear on:

why envoy would print those errors
why kubernetes tells me contour was restarted because of error but at the same time the probes fail

Anyway, I'm going to close this for now, and reopen it when I have more info and if it contradicts those conclusions.
Thank you for your help.
By the way, @stevesloka is running envoy as a deamonset the recommended architecture?


Mandatory operands for vsetvli, vsetivli could be better explained

Looking at the lastest GitHub released spec, under "Vector Tail Agnostic and Vector Mask Agnostic vta and vma" there is this text:
The assembly syntax adds two mandatory flags to the `vsetvli` instruction

...followed by a table of the valid vta and vtm flags.
A first suggestion is to change "adds ... to" in the text above to "has ... in", as it's not apparent what is being added to.
A second suggestion is to add vsetivli to the sentence, presuming these flags are also mandatory for it.
Accompanying the above table is a note:

Prior to v0.9, when these flags were not specified on a vsetvli, they defaulted to mask-undisturbed/tail-undisturbed. The use of vsetvli without these flags is deprecated, however, and specifying a flag setting is now mandatory. The default should perhaps be tail-agnostic/mask-agnostic, so software has to specify when it cares about the non-participating elements, but given the historical meaning of the instruction prior to introduction of these flags, it was decided to always require them in future assembly code.

Are references to pre-ratified specifications of value in the ratified spec? Presuming not, a third suggestion is to remove this note.
Regardless, this note further confirms that the vta and vtm flags are required.
Unfortunately, some later examples of the use of vsetvli/vsetivli do not obviously include the mandatory flags. In (33.6):
 vsetvli rd, rs1, vtypei   # rd = new vl, rs1 = AVL, vtypei = new vtype setting
 vsetivli rd, uimm, vtypei # rd = new vl, uimm = AVL, vtypei = new vtype setting
 vsetvl  rd, rs1, rs2      # rd = new vl, rs1 = AVL, rs2 = new vtype value

Thus, a final suggestion is to add some indication or immediate explanation that vtypei is a series of flags (not one operand as shown), including some mandatory flags. This explanation comes in a subsection about a page later after content about encodings and the vtype register layout.

I support the proposed changes.
I haven't checked how broadly various tools have deprecated the old assembly syntax. I generally use LLVM's (integrated) assembler, modern versions of which error on the old syntax. So I'm OK with deleting the note.
Over time, I'm hoping that such details of assembly language syntax will be migrated to https://github.com/riscv-non-isa/riscv-asm-manual/blob/main/riscv-asm.md . But that is a broader issue.

I support the proposed changes.

Thanks, @nick-knight!

Over time, I'm hoping that such details of assembly language syntax will be migrated to https://github.com/riscv-non-isa/riscv-asm-manual/blob/main/riscv-asm.md . But that is a broader issue.

In the quest for "one source of truth", I certainly hope the RISC-V Sail specification informs the content of the RISC-V Assembly Programmer's Manual, if it is not generated (or partly generated) from it. The Sail specification needs to retain its role as the canonical reference.


qmake-qtrpi not work in CLI

Hi,
Thanks for your nice work. I successfully completed the guide, and get things installed. But error occurs when I use qmake-qtrpi in CLI, and I can not add the qmake-qtrpi to Qt creator qt version.
The error is:
sqn@sqn-T460p:/opt/qtrpi/bin$ qmake-qtrpi -v
 qmake-qtrpi: /usr/lib/x86_64-linux-gnu/libstdc++.so.6: version `CXXABI_1.3.8' not found (required by qmake-qtrpi)
Btw, I DO have a libstdc++.so.6 in /opt/qtrpi/raspbian/sysroot-minimal/usr/lib/arm-linux-gnueabihf but it's not linked.
I'm using Ubuntu14.04 64bit with rpi-zeroW.

Hi,
Thanks for your message.
"qmake-qtrpi" should be a symbolic link to "/opt/qtrpi/raspi/qt5/bin/qmake", you can check it with:
ls -l /opt/qtrpi/raspi/qt5/bin/qmake

Also, can you post the console output of:
/opt/qtrpi/raspi/qt5/bin/qmake -query

Please also notice that:

using qtrpi with a rpi-zero is untested (we only performed some tests on the rpi 1, 2 and 3)
qtrpi only support Raspbian Jessie (not Raspbian Stretch)

Hi, @GuillaumeLazar
Thanks for your quick reply. I upgrade my host pc GCC version from 4.8 to 4.9, and get things worked.
Now, the output of :
/opt/qtrpi/raspi/qt5/bin/qmake -query
sqn@sqn-T460p:/opt/qtrpi/bin$ /opt/qtrpi/raspi/qt5/bin/qmake -query QT_SYSROOT:/opt/qtrpi/raspbian/sysroot QT_INSTALL_PREFIX:/opt/qtrpi/raspi/qt5pi QT_INSTALL_PREFIX/dev:/usr/local/qt5pi QT_INSTALL_ARCHDATA:/opt/qtrpi/raspi/qt5pi QT_INSTALL_ARCHDATA/dev:/usr/local/qt5pi QT_INSTALL_DATA:/opt/qtrpi/raspi/qt5pi QT_INSTALL_DATA/dev:/usr/local/qt5pi QT_INSTALL_DOCS:/opt/qtrpi/raspi/qt5pi/doc QT_INSTALL_DOCS/dev:/usr/local/qt5pi/doc QT_INSTALL_HEADERS:/opt/qtrpi/raspi/qt5pi/include QT_INSTALL_HEADERS/dev:/usr/local/qt5pi/include QT_INSTALL_LIBS:/opt/qtrpi/raspi/qt5pi/lib QT_INSTALL_LIBS/dev:/usr/local/qt5pi/lib QT_INSTALL_LIBEXECS:/opt/qtrpi/raspi/qt5pi/libexec QT_INSTALL_LIBEXECS/dev:/usr/local/qt5pi/libexec QT_INSTALL_BINS:/opt/qtrpi/raspi/qt5pi/bin QT_INSTALL_BINS/dev:/usr/local/qt5pi/bin QT_INSTALL_TESTS:/opt/qtrpi/raspi/qt5pi/tests QT_INSTALL_TESTS/dev:/usr/local/qt5pi/tests QT_INSTALL_PLUGINS:/opt/qtrpi/raspi/qt5pi/plugins QT_INSTALL_PLUGINS/dev:/usr/local/qt5pi/plugins QT_INSTALL_IMPORTS:/opt/qtrpi/raspi/qt5pi/imports QT_INSTALL_IMPORTS/dev:/usr/local/qt5pi/imports QT_INSTALL_QML:/opt/qtrpi/raspi/qt5pi/qml QT_INSTALL_QML/dev:/usr/local/qt5pi/qml QT_INSTALL_TRANSLATIONS:/opt/qtrpi/raspi/qt5pi/translations QT_INSTALL_TRANSLATIONS/dev:/usr/local/qt5pi/translations QT_INSTALL_CONFIGURATION:/opt/qtrpi/raspi/qt5pi/etc/xdg QT_INSTALL_CONFIGURATION/dev:/usr/local/qt5pi/etc/xdg QT_INSTALL_EXAMPLES:/opt/qtrpi/raspi/qt5pi/examples QT_INSTALL_EXAMPLES/dev:/usr/local/qt5pi/examples QT_INSTALL_DEMOS:/opt/qtrpi/raspi/qt5pi/examples QT_INSTALL_DEMOS/dev:/usr/local/qt5pi/examples QT_HOST_PREFIX:/opt/qtrpi/raspi/qt5 QT_HOST_DATA:/opt/qtrpi/raspi/qt5 QT_HOST_BINS:/opt/qtrpi/raspi/qt5/bin QT_HOST_LIBS:/opt/qtrpi/raspi/qt5/lib QMAKE_SPEC:linux-g++ QMAKE_XSPEC:devices/linux-rasp-pi-g++ QMAKE_VERSION:3.0 QT_VERSION:5.7.0
But I still dont know why this worked?

I'm glad to hear that it works.
Effectively GCC 4.8 is quite old. A recent GCC version is always a good thing.
The issue was maybe an incompatibility with a too old version of GCC.
We should add GCC version 4.9 at the minimum requirement in the qtrpi documentation

Thanks for your work again.
And it's proved that qtrpi is compatible with Rpi-zeroW.

It's a good news. Can you share with us the qtrpi environment variables used for your rpi-zerow?
QTRPI_QT_VERSION
QTRPI_TARGET_DEVICE

It's:
QTRPI_QT_VERSION='5.7.0'  QTRPI_TARGET_DEVICE='linux-rasp-pi-g++'
Rpi ZeroW use the same Core type as Rpi: RaspberryPI models comparison


Update release process

Description

Motivation and Context

Have you tested this? If so, how?

Thank you very much, appreciated. :-)


This checks for a negative position on pageYOffset.

Problem: On Safari iOS, if pageYOffset has a negative value, waypoint will automatically fire.
Solution: Add check to see if pageYOffset is greater than 0.
Gif of problem:

Page example of problem (please view in Safari on an iOS device):
http://ryanhagerty.com/waypoints.html

Thank you for this PR! You've uncovered a neat edge case bug that I've now detailed in #499. I wouldn't have seen it were it not for this in your code:
window.addEventListener("scroll", function() {
  console.log(window.pageYOffset);
  var waypoint = new Waypoint({

You're creating a new Waypoint with every scroll event, and the bug you're seeing is in a very specific case, when creating a waypoint during that last bit of bounce from -1 to 0 scrollTop. If you create one Waypoint...
window.addEventListener("scroll", function() {
  console.log(window.pageYOffset);
});

var waypoint = new Waypoint({
  // ...
});

...you should see your page behave as you expect. Besides what's in #499, Waypoints will handle negative scrolling fine. You could even, conceivably, use it as a part of a pull-to-refresh component.


Add custom properties to i18n languages (additionally to domains)

What problem does this feature solve?
I have further properties that are linked to languages.
In my case I have a siteHandle related to my languages. Whenever I make an apollo graphql request I need to send the correct siteHandle to get the correct content.
It would be great to something like this:
https://cmty.app/nuxt/nuxt-i18n/issues/c240
But with a custom property. (see below)
What does the proposed changes look like?
// nuxt.config.js

['nuxt-i18n', {
  locales: [
    {
      code: 'en',
      domain: 'mydomain.com',
      siteHandle: 'default',
      anyProperty: 'xyz',
    },
    {
      code: 'es',
      domain: 'es.mydomain.com',
      siteHandle: 'mySpanishSiteHandle',
      anyProperty: 'abc',
    }
  ],

And then I would have access to the current siteHandle (or any other property) by:
this.$i18n.locale.siteHandle

This feature request is available on Nuxt community (#c331)

Hm I tried that, but it seemed not to work. Thanks for your hint though, I will try again.
Maybe I tried to access $i18n.locale.siteHandle directly instead of $i18n.locales.find(l => l.code === $ii18n.locale

Works like a charm. Thanks. I wrongly assumed that this.$i18n.locale would also be an object containing the code and whatever property.


Add support for slim models

slim or "alex" type models use 3px arms, and renders look weird, since they are always using 4px arms.
Example using id c69f8316c60a4f8ca922bda933e01acd:

For reference, see Steve vs Alex:

Steve
Alex

Mojang's session API returns model: slim for alex type models:
{
  "timestamp":<PHONE_NUMBER>754,
  "profileId": "c69f8316c60a4f8ca922bda933e01acd",
  "profileName": "Doomblah",
  "textures": {
    "SKIN": {
      "url": "http://textures.minecraft.net/texture/72e11c1a3527aa62501988d8f577f5760c3edfcb6d2729b4ff4494b14bc",
      "metadata": {
        "model": "slim"
      }
    }
  }
}

I should add that we get this information for uuids only, not for usernames.
Maybe we can add a field to redis that defines whether or not a skin (hash) is slim.
However, we first need to confirm that the same skin file, uploaded once as 'alex' and once as 'steve', generates a different hash.

Just tested, the skin hash does not change, so we need to store per-userid because two users could share the same with one of them using the slim type and the other one the normal size.

Actually, we could use the profileName field of the response and update that in our cache as well.


CI: Larger Mac Runner?

Mac OS CI times are now exceeding 1hr.

CI times seem more reasonable lately -- the runners definitely seemed slower during the 2.0 release week.  Closing for now.


Push/Pull apis

Jadeite will need the Rowan push to git and pull from git api's when available.

push pull and log added to jadeite


Wrong tooltip for loot settings

Description

Version Number
7.12
Steps to Reproduce
Open "Loot" setting and hover over "GroupLoot".
Expected Behavior
Tooltip that makes sense :)
Actual Behavior
Tooltip makes no sense.
P.S. Also "Auto loot" is unclear, does that mean loot need when possible then greed? Any option for DE?

Thanks for catching this! I will fix this today! <3

Fixed https://github.com/Kkthnx-WoW/KkthnxUI/commit/9fd2e95e021a33132d726f0586c40961ccbfd944


[ci skip] adding user @ludaavics

Hi! This is the friendly automated conda-forge-webservice.
I've added user @ludaavics as instructed in #7.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #7

Hello @ludaavics, would you still be willing to take on maintenance for this feedstock?

I would be happy to, yes
On Fri, Jan 26, 2024, 04:45 Bela Stoyan @.***> wrote:

Hello @ludaavics https://github.com/ludaavics, would you still be
willing to take on maintenance for this feedstock?
—
Reply to this email directly, view it on GitHub
https://github.com/conda-forge/dash-mantine-components-feedstock/pull/8#issuecomment-1911760009,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAON3LBQCQBFGKSUD3HFNELYQN3KJAVCNFSM6AAAAABASEJPFOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJRG43DAMBQHE
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

@conda-forge-admin, please rerender


Playback feature?

Hi Mattia,
I'm trying to have students send me a record of their coding steps (so I can see their thinking process and make sure they didn't just paste code from elsewhere). vs-code-recorder would be great for this, but it would be even better if there was a way to play back a recorded session. Is that feasible?
--Barry

Hi Barry,
It's in my intasion to create a web player. When it's ready I'll link it in the exension README.md.


disk memory fills up on OSX

Hi GpiLab gurus,
Lately I kept running out of disk memory. I found that the folder com.gpilab.GPI still had 126GB even after having closed all canvases.  I assume those files remained after a couple crashes I experienced last week. Deleting all files in that folder resolved the problem. Just thought this may be helpful for other users to know. Here is the hidden path where that folder is on my computer:
/private/var/folders/66/dw_7jnhd44jg_x1rhhhrfmwm0000gn/T/com.gpilab.GPI/
Thanks!
Mike

This "tmp" directory should also clear up on restart in both Linux and OSX.

Thanks Nick, turns out I wasn't able to empty the trash, and only restarting did the job in the end.

Thanks Nick, turns out I wasn't able to empty the trash, and only restarting did the job in the end.


Added more documentation

So far:

[x] add

Ok, I've finished going through all of the commands for the first time. Now is the harder bit; which is, namely, figuring out why some don't work in the API at all, where the inconsistencies are in the documentation, and how to clean up and make this, as a whole, better. For now, though, I think this is a massive step forward. Can someone give a brief overview of my work here, point out any glaring issues?

This, of course, relates and possibly closes #7.

cc @whyrusleeping @diasdavid @lgierth @mappum for review

I have now linted this, and it works in apiary. Whether or not it is 100% accurate is up for debate; but, this means we have something to work with, as far as three columns and apiary goes.

Okay, I will start reviewing this and then we can merge this for now and have any bikeshedding discussions later, as the most important thing for now is to get the IS state documented

@Dignifiedquire Want to take another pass?

Removed 'needs review' pending current changes ongoing after the API talk.

Please do not close or delete this branch. I am using it to stub out future PRs. This PR will not and should not be merged.


Feature/welcome redesign

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: aleksanderfedorov:white_check_mark: LinneyS:x: IrinaMalyshevaYou have signed the CLA already but the status is still pending? Let us recheck it.


Vagrant workaround fixes

Small workaround fixes to get vagrant up to work. These fixes are not long term.

Next version of Vagrant v1.8.6 should have fixed its bug a/c to https://github.com/CiscoCloud/mantl/issues/1829
Conflicting Mesos versions should be addressed in the Vagrant base box built by Packer

Working on bringing this up now! it is currently running ansible playbooks. ⌚

I am unable to start this the ansible play hangs on
TASK: [kubernetes | pull hyperkube docker image] ******************************
😢

@ChrisMcKenzie might be a transient network issue or is it persisting across multiple runs?

I have run twice and I am running again right now. but same result both times prior.

😞

I have hunch! the kubeworkers option in the vagrantfile is set to 0 I bet it is still trying to provision it even though it doesn't exist testing this now.

nope! :(

This is working beautiful now I must have done something wrong at first! feel free to merge


VUE - v-for not working

hellow.
LeaderLine.setLine(this.$refs.start, this.$refs.end) - not working if this.$refs.start or this.$refs.end was created in v-for
not working function isElement
element.nodeType  - is undefined
  	typeof element.getBoundingClientRect  - is undefined

vue create "nodeType" and "getBoundingClientRect" in the nextTick

UPD:
Temp workout this probleb:
add str if(newOption.nodeType === Node.ELEMENT_NODE && newOption[0]!=undefined && newOption[0].nodeType === Node.ELEMENT_NODE ){ newOption = newOption[0]; }
after code
// anchorSE
['start', 'end'].forEach(function(optionName, i) {
var newOption = newOptions[optionName], newIsAttachment = false;

Hi @Sanoend, thank you for the comment.

LeaderLine.setLine(this.$refs.start, this.$refs.end) - not working

Yes, of course that should not work because the setLine is unknown method.
See document: https://anseki.github.io/leader-line/

No reply came, then this abandoned issue is closed.


Add dependabot to microservices repo

Not sure exactly how to do this in a monorepo but ideally it would work with microservices implemented in languages other than Python. Support microservices in a monorepo (and multiple languages) by adding a entry for each microservice in the dependabot.yml file.

Done in commit 5a4a37a06a3031e701895b2f0b89bd6ac41929d5. Closing.


Missing migration for the report table

The code in master is out of sync with database.
HHH000397: Using ASTQueryTranslatorFactory Hibernate: select report0_.id as id1_4_, report0_.android_version as android_2_4_, report0_.brand as brand3_4_, report0_.content as content4_4_, report0_.date as date5_4_, report0_.installation_id as installa6_4_, report0_.phone_model as phone_mo7_4_, report0_.stacktrace_id as stacktr10_4_, report0_.user_comment as user_com8_4_, report0_.user_email as user_ema9_4_ from report report0_
WARN 17640 --- [           main] o.h.engine.jdbc.spi.SqlExceptionHelper   : SQL Error: 1054, SQLState: 42S22
ERROR 17640 --- [           main] o.h.engine.jdbc.spi.SqlExceptionHelper   : Unknown column 'report0_.stacktrace_id' in 'field list'

The problem with the com.faendir.acra.liquibase.change.ReportColumnsChange#afterChange.
It tries to use new database structures from not applied migrations.
I have to comment it out to create a database structure.


Development vdc monitorstack

Fixes for grafana / prometheus

https://github.com/threefoldtech/js-sdk/issues/1921

https://github.com/threefoldtech/js-sdk/issues/1921


Use os.path.join and os.path.sep instead of "/"

Using "/" literal does not work on Windows/MinGW.
Also use __file__ to be able to run from different directories, such as python amalgamate/merge_all.py

I didn't notice #279 before. Closing.


Improve detection of StatefulSet changes

By adding current StatefulSet Spec as a source of computed hash we can
improve detection of manual changes to prometheus SS and force their
rollback.
/cc @brancz @LiliC

Rebased to include CI changes from #2804

lgtm on green


WMS ID: 11445 - OAS SSO With IAM Domain via App Gateway

OAS SSO With IAM Domain via App Gateway
11445
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[x] All filenames are lower case (including folders, images, files, etc.)
[x] Filenames are descriptive
[x] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[x] Image references in markdown contain an alternate text

---automatically closed due to code freeze---

After how much time we should do it again to get it approved?


"home" could not be copied

After running for the first time.

Also, this, which looks worrying:

The default config has vmClonePath: /Users/torarne/vmclone, but it looks like the code expects it to be / terminated, otherwise it tries to create  /Users/torarne/vmclonehome, /Users/torarne/vmcloneusr, etc

This is scary:

Would you mind sharing your full cilicon.yml file so I can try to look into it (feel free to redact any confidential info)?

I installed the app, ran it for the first time, got this:

Entered a local tart VM name (ventura-vanilla:13.4) based on
❯ tart list
Source Name                         Size Running
local  monterey-vanilla:12.6        18   false
local  sonoma-vanilla:14.0-23A5257q 31   false
local  sonoma-vanilla:14.0-beta     18   false
local  ventura-vanilla:13.4         16   false

Which generated:
❯ cat ~/cilicon.yml
provisioner:
  type: script
  config:
    run: echo Hello World && sleep 10 && echo Shutting down
hardware:
  ramGigabytes: 8
  display:
    width: 1920
    height: 1200
    pixelsPerInch: 80
  connectsToAudioDevice: false
directoryMounts: []
source: ''
vmClonePath: /Users/torarne/vmclone
editorMode: false
retryDelay: 3
sshCredentials:
  username: admin
  password: admin

Which put this in my home dir:
drwxr-xr-x@   2 torarne  staff     64 Jun 27 16:19 vmclone
lrwxr-xr-x@   1 torarne  staff     25 Jun 27 12:08 vmclonehome -> /System/Volumes/Data/home
drwxr-xr-x@   3 torarne  staff     96 Jun 27 16:19 vmcloneusr

That'll likely be the issue. If you want to use a local (non-cached OCI) image from tart you will need to enter its path (~/.tart/vms/...). It doesn't currently support taking names directly as used in tart but it would be a good addition in the future.

Ah, that fixed it yes! Thanks! A very strange failure mode though :D Would perhaps make sense to fix/improve regardless of whether plain named tart images are supported. Perhaps a quick check whether the path exists if it's not an oci image.

Great, is it working? If so we can close the issue.


Allow undefined from ProxyHandler.getOwnPropertyDescriptor

From https://github.com/Microsoft/TypeScript/pull/10550
Currently, with strictNullChecks enabled, the getOwnPropertyDescriptor hook cannot be correctly implemented as all code paths must return a valid PropertyDescriptor.
Reference: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Proxy/handler/getOwnPropertyDescriptor

Fixed by https://github.com/Microsoft/TypeScript/pull/10550

As I commented on the #10550 PR,
it looks like a later commit has undone this patch. It is not present in the current version
interface ProxyHandler<T> {
    //...
    getOwnPropertyDescriptor? (target: T, p: PropertyKey): PropertyDescriptor;
    //...
}


Add 'emacs-forward-word'

This commit adds the 'emacs-forward-word' widget to the list of partial accept widgets.

Thanks!


Benchmarking Zephyr vs. RIOT-OS

Hello everyone,
I'm Luiz Villa, a researcher at the University of Toulouse. I'm working with software defined power converters and my team is looking into using an RTOS to manage real-time micro-controllers.
We ran a benchmark between Zephyr and RIOT OS that we would like to share with you.
Spoiler: Zephyr performed poorly.
Here's the summary:

Our test code is available at https://gitlab.laas.fr/owntech/zephyr/-/tree/test_adc_g4
Our target : Nucleo-G474RE
Our main conclusions:

Zephyr has an overhead of almost 27us compared with 10us from RIOT
Zephyr consumes 34% of CPU cycles compared with 14% from RIOT
Zephyr has a period of 12us when we flood the shell compared with 1us from RIOT
Zephyr has a minimum thread period with an active shell of 38us compared with 40us from RIOT

Our questions for the community:

Does anyone has the same issues?
Have you managed to bring the execution time of Zephyr below this threshold?
Are we missing something on our code?

Originally posted by @luizvilla in https://github.com/zephyrproject-rtos/zephyr/discussions/32870

Interesting results, thanks for sharing @luizvilla!
Note that In the adc example, I'd use a hardware timer to trigger the ADC in a real application (it is possible on STM32). Regarding footprint sizes, it would be nice to compare rom reports in both cases to see the big difference, in particular for ADC and shell. Have you published any paper with some more insights?

@luizvilla I've cloned your project, and I've seen a few things that can be disabled or improved. A few things: use minimal lib C (if possible by the application), disable some shell features, reduce some stack sizes, etc. Without testing on a real board I quickly got ~37K ROM and ~9K RAM by doing a few tweaks, so I'm sure there is room for improving the footprint at least.

@luizvilla As mentionned by @gmarull, the first thing to do would be to understand the configuration used for the zephyr target. I guess this can be minimized in various ways, but point is to stay on par with RIOT configuration.
For instance, on test_adc_g4, we can disable LOG, I2C, tweak stacks sizes, use minimal libc, and quickly divide footprint by 2.
Though, doing this maybe we remove features provided by the RIOT firmware.
Can you elaborate on the configuration options you selected (basically all the 'y' features in prj.cnf) ?

Can you elaborate on the configuration options you selected (basically all the 'y' features in prj.cnf) ?

You can see them here. As @gmarull pointed out, the set selected here is certainly not optimized in any way.
https://gitlab.laas.fr/owntech/zephyr/-/blob/test_adc_g4/zephyr/prj.conf

You can see them here. As @gmarull pointed out, the set selected here is certainly not optimized in any way.

I've seen that. I just want to know if this configuration was made on purpose, to be on par with RIOT configuration.

@luizvilla Also, the project documentation to get the things build is great. Though, I haven't found info on how the latency measurements were made. Can you elaborate on this part ?

I noticed that you are defining the trigger thread's priority to 6. Have you tried making this a high-prio cooperative thread instead?

Interesting results, thanks for sharing @luizvilla!
Note that In the adc example, I'd use a hardware timer to trigger the ADC in a real application (it is possible on STM32). Regarding footprint sizes, it would be nice to compare rom reports in both cases to see the big difference, in particular for ADC and shell. Have you published any paper with some more insights?

Thank you for your quick return!
So far we have not published this. We came to you to be sure we are not forgetting something.

You can see them here. As @gmarull pointed out, the set selected here is certainly not optimized in any way.

I've seen that. I just want to know if this configuration was made on purpose, to be on par with RIOT configuration.

The idea was to create a benchmark between both systems. So we did not try to optimize the code, but rather test equivalent functionality and performance for these functitonalities.

@luizvilla Also, the project documentation to get the things build is great. Though, I haven't found info on how the latency measurements were made. Can you elaborate on this part ?

We toggle a pin and measure the time between toggles.

I noticed that you are defining the trigger thread's priority to 6. Have you tried making this a high-prio cooperative thread instead?

Not really. We wanted to test if the threads had similar performances on comparable conditions.
We are proofreading an internal report we made and we'll share with you as soon as we can.

The idea was to create a benchmark between both systems. So we did not try to optimize the code, but rather test equivalent functionality and performance for these functitonalities.

Sure, it's not about optimizing the code, but rather about defaults. If you want an apples to apples comparison you should enable the same features on both RTOS in my opinion. If it turns out that one of the two enables more functionality by default than the other one, then you should probably disable it in order to come to a meaningful comparison, unless you want to compare "default configurations".
I suggest you use west build -t rom_report to get a better idea of where your flash code is coming from. Same goes to RAM with west build -t ram_report.

Agree with @carlescufi, a meaningful comparison should enable a comparable feature set on both sides.

Thanks, everyone, for your valuable input.
Just to clarify, there are two main issues at hand:

The footprint - Being sure that the same amount of features are active on both RTOS as to have a clear comparison metric of their footprint. I'll get back to you on this as soon as I talk to our software engineer.
The performance - The systems had only their adc acquisitions active within a single repeating thread which had a variable sleep length. This test was conducted with and without the shell. This is, I think, independent from the system's footprint and the results are comparable.

We'll get back to you with more details. But I already thank you a lot for your reactivity and interest.

Also, note the documentation for k_usleep:
/*
 * This function is unlikely to work as expected without kernel tuning.
 * In particular, because the lower bound on the duration of a sleep is
 * the duration of a tick, @option{CONFIG_SYS_CLOCK_TICKS_PER_SEC} must be
 * adjusted to achieve the resolution desired. The implications of doing
 * this must be understood before attempting to use k_usleep(). Use with
 * caution.
 */

Zephyr's k_usleep ~always does a "true" sleep, whereas it appears from the documentation (I'm not familiar with RIOT-OS) that
xtimer_usleep will busy-wait for small periods.
For small periods xtimer_usleep would then be more comparable to k_busy_wait.
(Also, is Zephyr running in tickless mode? Is RIOT-OS? Are you running Zephyr with CONFIG_USERSPACE? Is the Zephyr thread preemptive? Is the RIOT-OS thread preemptive?)

Yeah, looking at the code here, it seems like indeed the main loop is just asking for too much: https://gitlab.laas.fr/owntech/zephyr/-/blob/test_adc_g4/src/adc_g4/adc.c#L180
Zephyr's sleep calls are blocking primitives, they will register a timer interrupt for that duration in the future, pend the current thread, and then context switch to another thread (likely the idle thread in this app) to do other work while it waits.  Trying to do that at > 30 kHz on a CPU like this isn't going to work well, your app is likely spending most of its time in context switch and interrupt handling.  (It's also trying to do that more than twice as often as under RIOT).
Similarly, I see you're asking for a tick rate of 1 MHz.  That's not impossible to do as long as you configure 64 bit ticks, but in some sense that's false precision.
Can you post the build/zephyr/.config file from your application?  There is no doubt a ton of tuning we could suggest.

OK, having studied the source to the test (and a tiny bit of digging in RIOT-OS), some more notes:

As mentioned, the resulting .config file from your build would be helpful.

Are you sure your performance deltas aren't just measurement error? At least according to the source at the HEAD commit, you're waking the RIOT-OS up every 30us, but the Zephyr thread every 13us.  So Zephyr is being asked to do 130% more work?

As mentioned by Carles upthread, the usage pattern isn't really right for what Zephyr expects.  You have your ADC polling thread configured at a low priority, so it's always going to be preempted by the shell thread.  If you want reliable polling in timers like this, you want the timers set at a high priority so the OS services the hardware and not the interactive/bursty shell thread.

This explains why "flooding the shell" (if I understand your phrasing correctly) locks out the ADC handling.  Zephyr has a real time scheduler, that is correct behavior.  You told the OS explicitly not to run the ADC thread if the shell had work to do!

(That does imply that the shell can't handle a 115kbps UART stream without saturating a 170 MHz Cortex M, which does indeed seem like a performance bug.  But I guess it depends on what commands you're trying to run with the flood of bytes.)

Crude sleep polling like this frankly isn't what we optimize for. If you really have hard real time polling requirements like ADC sampling and you want to run them off of the CPU SysTick timer (and not a hardware trigger, or external counter device, etc...) then the Right Thing in Zephyr is to use a k_timer, which allows you to hook the timer ISR directly with a repeating interrupt that is hardened against things like delayed interrupts and won't glitch or skip except when the interrupt gets masked out or preempted.

Deliberately cranking the sleep timer past the interrupt handling rate is REALLY not the right thing to do, on either OS.  It looks like you arrived at the values you have not because that's the frequency you want but because those are the smallest values you could set before something broke.  This isn't likely going to be testing what you think it's testing; you're probing the edge cases and not the desired operating regime.  Honestly: I think Zephyr comports itself fairly well here, given that you can give it more pathological input than RIOT-OS and still get useful behavior.

Broadly, I guess what I would hope to see here would be (given the ADC-sampling app), a comparison of the maximum sampling rate achievable under each OS under various IPC paradigms (sleeping, timer hooks, busy-waiting, etc...), and a statistical measure of each OS's variance (i.e. how regular are the sampling times).
But the even shorter summary is that you really want to be using k_timer for this use case.  That's what it's designed for.

I will say though, that in RIOT-OS's defense their ARMv7 context switch code is a masterpiece of simplicity when compared with ours.  It saves to the stack!  It's local, no PendSV!  No CONFIG_SWAP_NONATOMIC!  It would make an excellent starting point for an arch_switch() if anyone wanted to try.  I don't know that it's necessarily faster than our context switch code, but it's definitely cleaner.

Thank you everyone for your returns! This has been super useful!
We'll rework our application, simplify our Zephyr and re-think our scheduling.
We'll get back to you if we have some more interesting stuff.
Thanks again!

Thanks @luizvilla! this is very interesting. Did you consider Contiki?

I would suggest ADC reads using DMA to check the performance and ease of programming. Sampling a block at 48 kHz and then transferring it to the host while keeping up with all the work. Through how many hoops do you have to jump, to get this going with either Operating System.
That would look at a real use case, while only using the shell before and after the measurements. You could also look at power management/drain for these use cases.

Thanks @luizvilla to share this topic!
Could you share the used code on Github? It seems the LAAS GitLab is not open to public access anymore.
I'm curious to do tests with an STM32 and Zephyr and your benchmark could be a good starting point.

Hi, there @luizvilla! This was an interesting topic to read. But by any chance could you make the code public or maybe shared it somewhere else? I would really appreciate it. Thanks!

Even though this discussion has been closed for more than 3 years, I really enjoyed the content 😄.
For what it’s worth, at least we know that OwnTech Foundation decided to use Zephyr in this: https://github.com/owntech-foundation/Core. 🫡


Fix error thrown when inserting keys

Closes #69

Checks for ordering only when more than one key is present

[x] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.
Most changes should add an entry to the changelog and may need to rev the pubspec package version.
Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

@jonasfj Before I can add new test files. It seems many of the old tests were based on this bug  e.g.
https://github.com/dart-lang/yaml_edit/blob/31919348bd2a1bbb805b4eb88a6b7f50d4ab247e/test/editor_test.dart#L23-L35
The fix now returns the new correct SourceEdit as the correct offset for the edit 32 not 0 anymore.

By all means fix all the old tests too.
But are you sure wise to support reverse alphabetical ordering?
It might be unlikely that someone will do that intentionally.
Especially if there is only 2 properties then reverse alphabetical ordering is really just any order that isn't alphabetical :)

Oh, yeah. Sorry for the overkill. I'll fix that and the tests. Thanks.

@jonasfj I'm still debugging the spliceList method. It seems something broke somewhere.

By the way, the test that is failing is somewhere in test/random_test.dart.
I'd suggest trying to add more print statements and extract the YAML + the modification being made into a separate test.
Don't try to debug things in random_test.dart, try to extract the test case it generates instead. We'll want it as a hardcoded test case anyways :D

I noticed a few weird hacks with prints in random_test.dart and filed: https://github.com/dart-lang/yaml_edit/pull/81
After that the failing test case should print:
Failed to call YamlModificationMethod.splice on:

environment: true

0.03167456183131434:
  - - true
    - |-
        false
    -<PHONE_NUMBER>
  - null
  -<PHONE_NUMBER>
  - true
  - - false
  -<PHONE_NUMBER>
  -<PHONE_NUMBER>
  - 0.9662696464087974
  - null
  - "9[hd24=s438Y7a8_!M&}s<3cTr{\"e%<1I7vq&xp4p\"9jH#NJv"
762977104: null

with the following arguments:
[0, 1, [0.7396543051030952, null, {P/wQ'@G`$A%6LA$aXk0Uy&f36Pi[~:905@`DJ~nKUs!2:$45hj[vk7V<Qt~\2M^Uk$, vZEjAW=tOT=Q&dpH,cN'hX$_wHR=:q?UvHzC;+8*bLX_5Rgvp)L{E, {null: null, {3368549847: null, 0.9104247116161583: dcN&=EiSL?{sWLMRg -W]%
[=~Ggy@Z9F<77:3dY!{wE;m\uC9R9S2sIv<LyIUK, JRBV\|1]O<K;Wosal33 f30@`?*{}?}ke>^Eg''oUXw+JXq%C|{Ck._BGk: true}: ヽ༼ຈل͜ຈ༽ﾉ ヽ༼ຈل͜ຈ༽ﾉ},<PHONE_NUMBER>]]
and path:
[0.03167456183131434, 4]

Error Details:
Assertion failed: (package:yaml_edit) Modification did not result in expected result.

# YAML before edit:
>
> environment: true
>
>
> 0.03167456183131434:
>   - - true
>     - |-
>         false
>     -<PHONE_NUMBER>
>   - null
>   -<PHONE_NUMBER>
>   - true
>   - - false
>   -<PHONE_NUMBER>
>   -<PHONE_NUMBER>
>   - 0.9662696464087974
>   - null
>   - "9[hd24=s438Y7a8_!M&}s<3cTr{\"e%<1I7vq&xp4p\"9jH#NJv"
> 762977104: null
>

# YAML after edit:
>
> environment: true
>
>
> 0.03167456183131434:
>   - - true
>     - |-
>         false
>     -<PHONE_NUMBER>
>   - null
>   -<PHONE_NUMBER>
>   - true
>     -<PHONE_NUMBER>
>   - - false
>   -<PHONE_NUMBER>
>   -<PHONE_NUMBER>
>   - 0.9662696464087974
>   - null
>   - "9[hd24=s438Y7a8_!M&}s<3cTr{\"e%<1I7vq&xp4p\"9jH#NJv"
> 762977104: null
>

Please file an issue at:
https://github.com/dart-lang/yaml_edit/issues/new?labels=bug

#0      YamlEditor._performEdit (package:yaml_edit/src/editor.dart:579:7)
#1      YamlEditor.insertIntoList (package:yaml_edit/src/editor.dart:339:5)
#2      YamlEditor.spliceList (package:yaml_edit/src/editor.dart:376:7)
#3      _Generator.performNextModification (file:///workspace/yaml_edit/test/random_test.dart:209:20)
#4      main.<anonymous closure>.<anonymous closure> (file:///workspace/yaml_edit/test/random_test.dart:47:27)
#5      _ReturnsNormally.typedMatches (package:matcher/src/core_matchers.dart:153:8)
#6      FeatureMatcher.matches (package:matcher/src/feature_matcher.dart:16:42)
#7      _expect (package:matcher/src/expect/expect.dart:138:30)
#8      expect (package:matcher/src/expect/expect.dart:56:3)
#9      main.<anonymous closure> (file:///workspace/yaml_edit/test/random_test.dart:46:9)
#10     Declarer.test.<anonymous closure>.<anonymous closure> (package:test_api/src/backend/declarer.dart:215:19)
<asynchronous suspension>
#11     Declarer.test.<anonymous closure> (package:test_api/src/backend/declarer.dart:213:7)
<asynchronous suspension>
#12     Invoker._waitForOutstandingCallbacks.<anonymous closure> (package:test_api/src/backend/invoker.dart:258:9)
<asynchronous suspension>

00:03 +15 -1: testing with randomly generated modifications: test 15 [E]
  Expected: return normally
    Actual: <Closure: () => void>
     Which: threw _YamlAssertionError:<Assertion failed: (package:yaml_edit) Modification did not result in expected result.

            # YAML before edit:
            >
            > environment: true
            >
            >
            > 0.03167456183131434:
            >   - - true
            >     - |-
            >         false
            >     -<PHONE_NUMBER>
            >   - null
            >   -<PHONE_NUMBER>
            >   - true
            >   - - false
            >   -<PHONE_NUMBER>
            >   -<PHONE_NUMBER>
            >   - 0.9662696464087974
            >   - null
            >   - "9[hd24=s438Y7a8_!M&}s<3cTr{\"e%<1I7vq&xp4p\"9jH#NJv"
            > 762977104: null
            >

            # YAML after edit:
            >
            > environment: true
            >
            >
            > 0.03167456183131434:
            >   - - true
            >     - |-
            >         false
            >     -<PHONE_NUMBER>
            >   - null
            >   -<PHONE_NUMBER>
            >   - true
            >     -<PHONE_NUMBER>
            >   - - false
            >   -<PHONE_NUMBER>
            >   -<PHONE_NUMBER>
            >   - 0.9662696464087974
            >   - null
            >   - "9[hd24=s438Y7a8_!M&}s<3cTr{\"e%<1I7vq&xp4p\"9jH#NJv"
            > 762977104: null
            >

            Please file an issue at:
            https://github.com/dart-lang/yaml_edit/issues/new?labels=bug
            >

  package:matcher             expect
  test/random_test.dart 46:9  main.<fn>

And from that we should be able to extract the test case.
Once extracted we can craft a minimal example from the test case.
I think it makes sense to have the failing test cases and minimum example as hardcoded test cases going forward.
random_test.dart does make consistent test cases, but the RNG might change in the future and then it'll test other things.

Side note: it's pretty cool that random_test.dart actually found an issue. It was really a shot in the dark when we made it :rofl: I had no idea that it would actually prove useful.
(Okay, we don't know that the issue it found is a useful error, but it hopefully is something worth fixing).

@kekavc24 I landed https://github.com/dart-lang/yaml_edit/pull/81 I'd suggest rebasing this branch on main, then it'll be much easier to figure out what's wrong in random_test.dart.

Also please do consider adding an entry to CHANGELOG.md.
Once we've landed this, I think we should cut a release, it's a pretty important bugfix!
I filed https://github.com/dart-lang/yaml_edit/pull/82 to start the CHANGELOG.md.

hmm, I'm not sure I entirely follow.
But I have a few questions:

Is this only an issue given the bug fix we have in this PR?

I'm guessing this bug has been there a while, but that this PR just made it surface, right?

Are we only discovering the issue because of the bug fix we have in this PR?
Because we no longer make 1 length maps sorted when inserting the second key.
And this just happens to cause this bug to surface now.

I think if we can make test pass by sorting 1 length maps again, then we should do so.
AND then file an issue to fix the bug in spliceList. That way we can do it in a separate PR.

If on the other hand, the bug fix we made here, cause this bug in spliceList, then we should fix it in this PR.

I'm pretty sure this is an existing bug that just surfaced, try the test case.
SLICE LIST IN NESTED BLOCK LIST
---
key:
- true
- - false
---
- [splice, [key, 1], 0, 1, ['test']]

Just:

Create test/testdata/input/splicelist_in_nested_block_list.test with the contents above.
Run dart test test/golden_test.dart

It fails both on main and on this PR.

Question is why it only surfaces in random_test.dart in this PR.

If we switch the implementation to:
int getMapInsertionIndex(YamlMap map, Object newKey) {
  final keys = map.nodes.keys.map((k) => k.toString()).toList();

  // We can't deduce ordering if list is empty, so then we just we just append
  if (keys.isEmpty) {
    return map.length;
  }

  for (var i = 1; i < keys.length; i++) {
    if (keys[i].compareTo(keys[i - 1]) < 0) {
      return map.length;
    }
  }

  final insertionIndex =
      keys.indexWhere((key) => key.compareTo(newKey as String) > 0);

  if (insertionIndex != -1) return insertionIndex;

  return map.length;
}

Then we've still fixed the bug we set out to fix, right? @kekavc24
We'll be inserting in sorted order, even when there is only 1 entry in the map, but that's also how it worked before.
And with this random_test.dart just happens to pass. It's deterministically random, but it certainly doesn't try all combinations.

If we do that, then we've done the smallest possible change to fix the bug we're trying to fix. Then we can land this and move on the next bug. It's much easier if we don't let PRs grow too big.
I filed https://github.com/dart-lang/yaml_edit/issues/83 for the issue with spliceList. And it'd be great if we can fix that too -- just easier to do in a separate PR.

It's not from this fix.

@jonasfj Wow, I'm sorry. I really fumbled this. I committed a file I was meant to delete.
My golden tests seem to be failing with no cause. Can you take a look?

Make sure you delete the output files before running golden tests... Then review the newly generated output.
Otherwise, I can look tomorrow maybe.

Currently on it. @jonasfj


bug: Coinbase Wallet account stays logged in after refresh

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
^0.12.12
Current Behavior
Steps to reproduce:

log into wagmi using a coinbase wallet
log out using useDisconnect.disconnect
refresh the page

The account is still logged in
Expected Behavior
upon refresh, no accounts should be logged in
Steps To Reproduce

log into wagmi using a coinbase wallet
log out using useDisconnect.disconnect
refresh the page

import {configureChains, createClient} from 'wagmi';
import * as wagmiChains from 'wagmi/chains';
import {jsonRpcProvider} from 'wagmi/providers/jsonRpc';

const url = new URL({web3JsonRpcUrl});
url.protocol = 'wss';
const webSocketUrl = url.toString();

const {chain, provider} = configureChains(
        [wagmiChains.sepolia],
        [
          jsonRpcProvider({
            rpc: (_chain) => ({
              http: {web3JsonRpcUrl},
              webSocket: webSocketUrl,
            }),
          }),
        ]
      ),

const connector = new CoinbaseWalletConnector({
        chains,
        options: {
          appName: 'Test App',
          headlessMode: true,
        },
});

const client = createClient({
      autoConnect: true,
      provider: provider,
      connectors: [connector],
    });

<WagmiConfig client={wagmiClient}>< App /></WagmiConfig>

Logging in:
import {useConnect} from 'wagmi';

const {connect} = useConnect();
const login = connect

log out:
import {useDisconnect} from 'wagmi';

const {disconnect} = useDisconnect();
const logout = disconnect;

Link to Minimal Reproducible Example (StackBlitz, CodeSandbox, GitHub repo etc.)
No response
Anything else?
No response

@fred-dinari can you please make this runnable using new.wagmi.sh. It's not a minimal reproduction unless we can use it without having to set things up.

@tmm I can reproduce this even in the link you sent me without touching any code. If you follow the steps above you can see that the user is still logged in after a refresh despite disconnecting. Below is a recording
https://www.loom.com/share/b6f51df878bd4370a820a2540c7d3118


The FileOpenPicker's method PickSingleFileAsync throws ComException

Describe the bug
Hi, just tried to pick an image but got an exception - System.Runtime.InteropServices.COMException: 'Invalid window handle. (0x80070578)' To reproduce try the following code.
Steps to reproduce the bug
            FileOpenPicker filePicker = new FileOpenPicker();
            foreach(string ext in new []{ ".jpeg", ".jpg", ".png", ".bmp" })
                filePicker.FileTypeFilter.Add(ext);

            filePicker.ViewMode = PickerViewMode.Thumbnail;
            filePicker.SuggestedStartLocation = PickerLocationId.PicturesLibrary;
            filePicker.SettingsIdentifier = "ImageFilePicker";
            filePicker.CommitButtonText = "Insert";

            StorageFile file = await filePicker.PickSingleFileAsync(); //throws exception

Expected behavior

Screenshots

Version Info

NuGet package version:
[WinUI 3 - Project Reunion 0.5: 0.5.7]

Windows app type:

UWP
Win32

Yes

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

October 2020 Update (19042)

May 2020 Update (19041)
Yes

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

Additional context

You need to cast the picker to IInitializeWithWindow and call it's Initialize with the window handle to use it in a desktop app.

See #2716

https://docs.microsoft.com/en-us/uwp/api/windows.storage.pickers.fileopenpicker.-ctor?view=winrt-19041#remarks

Thanks all for the answers. But it still looks like a bug. The WA is ok but believe should be fixed in the future versions.


CreatingClientSecretsFile Step 6

Hi,
I'm creating "CreatingClientSecretsFile" for the first time and step 6 says to click on "Consent Screen" which is no longer there. Will it be a problem to skip this step or is there another way to access this part. Thank you please let me know

Consent is now under credentials - it's a tab on the top of that screen.
On Fri, Aug 21, 2015 at 9:52 AM, Michael B<EMAIL_ADDRESS>wrote:

Hi,
I'm creating "CreatingClientSecretsFile" for the first time and step 6
says to click on "Consent Screen" which is no longer there. Will it be a
problem to skip this step or is there another way to access this part.
Thank you please let me know
[image: image]
https://cloud.githubusercontent.com/assets/13904431/9409710/594278d2-47ea-11e5-9cdb-eee710842f2c.png
—
Reply to this email directly or view it on GitHub
https://github.com/jay0lee/GAM/issues/100.

Thank you


Remove some unused CSS and images

I largely used PurgeCSS to locate potentially unused CSS, but there were many false positives so this was still largely a manual process.

I didn't grep for everything, but critically I did skim every deleted line for things that might be involved in some sort of "generated CSS class name" construct, and didn't note any.
I think probably we should just merge, deploy to chat.zulip.org, and fix forward rather than trying to do more preemptive examination.

Merged, thanks @andersk!


Remove useCustomCrashReportsEndpoint method

What does this PR do?
This was a leftover and this method had no effect anyway: crashes are sent to the RUM and Logs features via message bus.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1510 (e7433ec) into feature/sdkv2 (34b0ddd) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@                Coverage Diff                @@
##           feature/sdkv2    #1510      +/-   ##
=================================================
+ Coverage          83.34%   83.42%   +0.08%
=================================================
  Files                429      428       -1
  Lines              14638    14618      -20
  Branches            2217     2212       -5
=================================================
- Hits               12200    12195       -5
+ Misses              1855     1836      -19
- Partials             583      587       +4

Impacted Files
Coverage Δ

...atadog/android/core/configuration/Configuration.kt
91.18% <100.00%> (-5.34%)
:arrow_down:

.../kotlin/com/datadog/android/v2/core/DatadogCore.kt
84.18% <100.00%> (+0.56%)
:arrow_up:

... and 17 files with indirect coverage changes


settings pages need pagination and search

Note: This issue is in the product/ux phase and is not yet ready for engineering work, though as always, comments are welcome.
Overview
Coming soon...
Token Context
@msorens This came up in #a2-support on May 24 that a customer with 5000+ tokens could not display the tokens list page due to timeout errors. That was the second time I am aware of this materialized in the field. It is likely there are other instances I'm not aware of.
Inevitably our simplistic, initial approach of just return all-the-things was bound to hit this issue. Sounds like we need to consider adding either pagination or infinite scrolling, starting with the tokens list. For this latest customer, the policies list would also be an issue, as they created a separate policy for each token though that can be ameliorated by aggregating multiple tokens into a subjects list on a single policy.

this work has been deprioritized, so closing.


docs: add info about merging strategy in App Config

🔗 Linked issue
#21287
❓ Type of change

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

📚 Description
Add info about merging strategy in App Config
reference: https://github.com/nuxt/nuxt/issues/15649#issuecomment-1397429223
📝 Checklist

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

@pi0 can you give feedback regarding this? Thanks

Approved too fast, actually you have to make sure that the layer has a default value as array and then only in the last app.config.ts you can use a function merger. Could make sense to use a ::code-group and show both the layer app config and main app.config.ts
See example on https://stackblitz.com/edit/github-qogr8r?file=app.config.ts,.playground%2Fapp.config.ts

Approved too fast, actually you have to make sure that the layer has a default value as array and then only in the last app.config.ts you can use a function merger. Could make sense to use a ::code-group and show both the layer app config and main app.config.ts
See example on https://stackblitz.com/edit/github-qogr8r?file=app.config.ts,.playground%2Fapp.config.ts

Thanks for the explanation!
It's a bit hard to grasp, but let's hope some documentation helps :)

Good point about function merger limitation. I have made an upstream to track if we can overcome it: https://github.com/unjs/defu/issues/97

Thank you for your contribution @wilsonpinto !


There is an issue with Signup

Request URL:
https://banking-jet.vercel.app/sign-up
Request Method:
POST
Status Code:
200 OK
.
But Not able to Sign up

I think you should check your environment variables once to resolve this issue at Vercel.

good


Update using-ssh-over-the-https-port.md

The user for connecting must be "git"

Why:
Improve documentation
What's being changed:
doc
Check off the following:

[ ] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[ ] For content changes, I have reviewed the localization checklist
[ X ] For content changes, I have reviewed the Content style guide for GitHub Docs.

@bon77 Thanks so much for opening a PR! I'll get this triaged for review ⚡

@bon77 Thanks so much for opening a PR! I'll get this triaged for review ⚡

It looks like our new action inadvertently closed this 🙃 Sorry about that! Reopening and triaging this for review 💛


Imported Ecto.Changeset to Ecto.Model

In one of the previous commits import Ecto.Changeset and import Ecto.Query, only: [from: 2] were deleted and I don't know about the latter but I think the former must still be present because otherwise ecto gives an error and doesn't compile

You should import it explicitly, it is not Ecto's responsibility.

oh ok, my mistake, I am sorry

no worries, thank you for the contributions!


devoxx-tekton-demo to 0.0.4

Promote devoxx-tekton-demo to version 0.0.4

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Added a skill for identifying conjunctions.

If your PR is related to a contribution to the taxonomy, please, fill
out the following questionnaire. If not, replace this whole text and the
following questionnaire with whatever information is applicable to your PR.
Describe the contribution to the taxonomy

Identifies conjunctions in sentences provided for context
Five different examples
Answer is the conjunction itself

Input given at the prompt

   Identify the conjunction in this sentence

Response from the original model

  ' '

Response from the fine-tuned model

  I have not tested the model

Contribution checklist

[ ] The contribution was tested with lab generate
[ ] No errors or warnings were produced by lab generate
[x] All commits are signed off (DCO)
[x] The qna.yaml file contains at least 5 seed_examples
[] The qna.yaml file was linted and prettified (yaml-validator can do both)

Thank you for your contribution to InstructLab! Unfortunately, once a Github repo is made public, all open PRs are automatically closed since they link against a private repo. We have detected that your PR might have been one of the ones affected by this change. If you are still interested in contributing your improvement, please fill out the following short form by no later than May 3rd, and we will get back to you with the additional steps necessary once we have had time to assess the PRs of those still interested:
https://forms.gle/V7SrPPMZDo6iGDYu8


fix: replaced deprecated vim. functions

This PR just replaces the deprecated functions with the recommended one.

ooops... did not see a PR https://github.com/MunifTanjim/nui.nvim/pull/365 already created for this. apologies for the noise


feat: add prettier

Description

Install Prettier.
Add Prettier pre-commit hooks.

Related Issues
#106
Checklist

[ ] Install Prettier.
[ ] Add pre-commit hook.

Screenshots

@brandeddavid Resolve conflicts.

@shyamtawli done


Add LSQ quantizer

This PR contains an implementation of LSQ quantizer (Learned Step Size Quantization, ICLR 2020, see here). It uses gradients to update quantization scales and can achieve sound results in our production environment, especially for lower bits.
In the mnist experiment, it can get about 99.20% top1 acc. And the results on imagenet-1k is on going.

I have added codes about using the first batch data to initialize activation scale. After that, we can get about 99.20% top1 acc with the provided example.
Also, I have tested exporting the model to TensorRT. The transformed TensorRT model get almost the same acc with the PyTorch model.

Looks good. Completing related doc is necessary. Please refer to overview, quantization and Quantizer. Feel free to ask me if have any questions.

Docs have been added:)

@chenbohua3 looks great, thanks for your contribution!


Phidget Spatial IMU as a supported sensor

Describe the the feature you would like
My team recently obtained a Husky through the MyBotShop european distributor, and the robot is outfitted with a PhidgetSpatial 3/3/3 Basic IMU and a Hokuyo UST-10 LIDAR. While the UST-10 configuration works just by adding
lidar2d:
  - model: hokuyo_ust
    urdf_enabled: true
    launch_enabled: true
    parent: base_link
    xyz: [0.0, 0.0, 0.0]
    rpy: [0.0, 0.0, 0.0]
    ros_parameters:
      urg_node:
        laser_frame_id: lidar2d_0_laser
        ip_address: <IP_ADDRESS>
        ip_port: 10940
        angle_min: -2.356
        angle_max: 2.356

to the robot.yaml configuration, we'd like to also have the PhidgetSpatial IMU supported as well. Will these sensors be supported in the roadmap?
Other notes
Since I am working on equipping our own Husky with this sensor, I am willing to contribute to the development and test the configurations.

Hi @fbottarel we currently do not but would consider it for the future.  It would be the same as the UM6/7 so duplicating those sections for PhidgetSpatial should be straight forward.

Thank you @tonybaltovski, would you kindly point me to the relevant files to edit/create? I can try doing it myself and if successful I can open a PR for it.

I needs to be added:

https://github.com/clearpathrobotics/clearpath_config/tree/main/clearpath_config/sensors
https://github.com/clearpathrobotics/clearpath_common/tree/humble/clearpath_sensors_description
https://github.com/clearpathrobotics/clearpath_robot/tree/main/clearpath_sensors

Closing due to inactivity.  Please feel free to re-open if issue persists.


Sync operation terminates before destination files are deleted

I am using Veeam Backup and Replication software. My backup job runs a post job batch script. The script runs FastCopy command line Sync to mirror backups to another local storage location. When VBR retention runs, it deletes older backups. This means that FC will delete backups from the destination that no longer exist in the source. The full backups are quite large resulting in very large file deletions.
What happens is that FC and the script terminate before the all destination files are deleted. The files are eventually deleted but can take 7-10 minutes after the script has terminated. This gives a false sense of completion.
I am guessing you are using an API call to do the deletions. I suspect that your code does not wait until the file is actually deleted but just checks the request to delete was successful.
Is there a way that your code can wait for all files to be deleted before FC terminates?

This means that FC will delete backups from the destination that no longer exist in the source
But FC does not delete destination files.   what app or program does the deletion in your scenario?
FC does not do mirroring, i.e. does not delete the target file it the file does not exist in the source.  It only copies files from source to target.

Thanks for the speedy reply. The source is the VBR repository containing backups. It is a large Raid Array. The destination is where the production backups are copied to via FC Sync. It is also a large Raid Array.
Normally, FC will copy the new nightly backups from the source to the destination.
However, when VBR retention runs, VBR deletes backups from the repository (Source) that are older than 2 weeks.
In this case FC will copy new backups from the source to the destination and delete backups from the destination that no longer exist in the source.
So FC is doing the deletions. Here is what the FC documentation says about Sync...
Sync (Size/date) - Copy if size or date is different, or it does not exist.
In addition, delete all destination files/directories that do not exist in Source.

"> Sync (Size/date) - Copy if size or date is different, or it does not exist.
In addition, delete all destination files/directories that do not exist in Source.
"
please show me where you get this.

https://fastcopy.jp/help/fastcopy_eng.htm#mode

https://fastcopy.jp/help/fastcopy_eng.htm#mode

This is directly from FC documentation. Having trouble with embedding the link. Please copy and paste in your browser.
https://fastcopy.jp/help/fastcopy_eng.htm#mode

it is a different project. That is a windows application which I don't think is open source.

My apology. I now see that this is a different program with the same name.


Merge dev into master

Rebased files manually due to conflicts.

Hello
Does anyone have to pass me the full installer? The full archives do not appear to be on the official website. I need hopebilling to install it from scratch.


Xamarin Simple Logger Not Implemented for Windows Phone 8.1 Application

Xamarin Simple Logger Not Implemented for Windows Phone 8.1 Application
An exception of type 'System.NotImplementedException' occurred running in a Windows Phone 8.1 Application.

That's expected behaviour. Apart from UWP the Windows platform is not supported yet.

Is there a road map or time frame when it will be implemented?

No, it's not. It's not difficult to do, but have a lot of work on another projects.


Allow configuration of paths to dependent programs so we don't need to run atom from the terminal?

I think running Atom as an app could work if there were configurable options for the location of necessary binaries

Actually, some kernels (e.g. Julia) already work without the $PATH. Hydrogen goes and finds where the kernels are listed, and those kernelspecs contain the instructions for running the kernel, so kernels that contain fully-qualified paths work no matter what.
You can fix any kernels that don't use fully-qualified paths by modifying the kernelspecs (probably in ~/.ipython/kernels/ or /usr/local/share/jupyter/) to contain fully-qualified paths (/usr/local/bin/node instead of node). You'll also need to create a kernelspec for Python by running ipython kernelspec install-self; the Python one uses a fully-qualified path already.
This is kind of a pain, and probably those kernels should just be doing this already. The good news is that all the kernels are open-source and you can send them a pull request to fix it!
However, updates and installation are still going to need the $PATH, because I can't control how npm installs things (especially zmq).

@willwhitney The problem is that Atom can't even find my IPython without running from the shell, because I don't have IPython installed system-wide. I would at least like for Hydrogen to offer a configurable IPython command. This is separate from IPython finding commands for running its subprocesses.

Hydrogen doesn't use IPython at all. It starts the kernels itself and is a
replacement for the IPython notebook server.
The only time it uses the ipython command is when starting a Python
kernel, and you can avoid that by installing the Python kernelspec as
stated above.
On Wed, Jul 1, 2015 at 12:15 PM Christopher Armstrong <
<EMAIL_ADDRESS>wrote:

@willwhitney https://github.com/willwhitney The problem is that Atom
can't even find my IPython without running from the shell, because I don't
have IPython installed system-wide. I would at least like for Hydrogen to
offer a configurable IPython command. This is separate from IPython finding
commands for running its subprocesses.
—
Reply to this email directly or view it on GitHub
https://github.com/willwhitney/hydrogen/issues/71#issuecomment-117733789
.

@willwhitney Oh, I totally misunderstood. Thank you. Sorry, this is my first time ever using Jupyter, so I wasn't familiar with all this kernel business. It turns out install-self installed some crap system-wide :-( but I was able to then move /usr/local/share/jupyter/kernels to ~/.ipython/kernels to disinfect my system and Hydrogen is now working great when run from the desktop. Thanks :-)

No worries! Glad I could help.


QUIC: rename SSL_tick()

Rename SSL_tick() to SSL_process_events() or SSL_handle_events().  Also rename SSL_get_tick_timeout() appropriately.  Finally, fix all the documentation.

Already done in #20879


puppeteer Error on logout api:  Requesting main frame too early!

Environment

**Wppconnect-server version(s):**2.3.0
Browser: [e.g. Chrome 87, Chromium 85]
OS: Ubuntu
Node version:  Node 20

Steps to Reproduce

call  generate  token
get qr code
and logout api

Log Output
Error: Requesting main frame too early!
    at assert (/home/whatsuser/Desktop/WhatsAppApi-2.3.0/node_modules/puppeteer-core/lib/cjs/puppeteer/util/assert.js:28:15)
    at FrameManager.mainFrame (/home/whatsuser/Desktop/WhatsAppApi-2.3.0/node_modules/puppeteer-core/lib/cjs/puppeteer/cdp/FrameManager.js:223:32)
    at CdpPage.mainFrame (/home/whatsuser/Desktop/WhatsAppApi-2.3.0/node_modules/puppeteer-core/lib/cjs/puppeteer/cdp/Page.js:415:35)
    at CdpPage.addScriptTag (/home/whatsuser/Desktop/WhatsAppApi-2.3.0/node_modules/puppeteer-core/lib/cjs/puppeteer/api/Page.js:510:31)
    at /home/whatsuser/Desktop/WhatsAppApi-2.3.0/node_modules/@wppconnect-team/wppconnect/dist/controllers/browser.js:307:47
    at step (/home/whatsuser/Desktop/WhatsAppApi-2.3.0/node_modules/@wppconnect-team/wppconnect/dist/controllers/browser.js:83:23)
    at Object.next (/home/whatsuser/Desktop/WhatsAppApi-2.3.0/node_modules/@wppconnect-team/wppconnect/dist/controllers/browser.js:64:53)
    at fulfilled (/home/whatsuser/Desktop/WhatsAppApi-2.3.0/node_modules/@wppconnect-team/wppconnect/dist/controllers/browser.js:55:58)
    at runNextTicks (node:internal/process/task_queues:60:5)
    at process.processTimers (node:internal/timers:509:9)
verbose:  [Session1:client] wapi.js failed

Please, update your project


Resolve Upper Bound issues, update Groovy

[x] - Update parent POM
[x] - Add upper bound enforcement
[x] - cleanup issues (mostly commons-*)
[x] - Update Groovy to 2.4.11, so it is the same as the core baseline

All versions are compatible with the Jenkins core.
@reviewbybees @jglick

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

Oops:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-site-plugin:3.3:site (default-site) on project stapler-parent: failed to get report for org.apache.maven.plugins:maven-javadoc-plugin: Failed to execute goal org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M1:enforce (default) on project stapler: Execution default of goal org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M1:enforce failed. NullPointerException -> [Help 1]

Works on my machine (c). No sure what happens there

Reproducible locally:
java.lang.NullPointerException
	at org.apache.maven.plugins.enforcer.EnforceBytecodeVersion.isBadArtifact(EnforceBytecodeVersion.java:195)
	at org.apache.maven.plugins.enforcer.EnforceBytecodeVersion.checkDependencies(EnforceBytecodeVersion.java:182)
	at org.apache.maven.plugins.enforcer.EnforceBytecodeVersion.handleArtifacts(EnforceBytecodeVersion.java:109)
	at org.apache.maven.plugins.enforcer.AbstractResolveDependencies.execute(AbstractResolveDependencies.java:77)
	at org.apache.maven.plugins.enforcer.EnforceMojo.execute(EnforceMojo.java:202)

You just need to update extra-enforcer-rules to 1.0-beta-4 or later to pick up the fix of https://github.com/mojohaus/extra-enforcer-rules/issues/13.

@jglick ptal

It seems NumberConverter regressed sometime between 1.7.0 and 1.8.0. 1.8.3 is also affected. But 1.9.3 is even worse—it does not convert String[]s properly either. Any I think I have identified a bug in how Stapler was calling BeanUtils that is responsible. Will file a follow-up PR.


floor recharger examine charge rate

About the PR

this just floors it
Media

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

Changelog
no cl no fun

why not 36W or 36,7W?

That's exactly what this PR does

this pr changes it to be 36W
cbf to do the / * thing and its not really important

this pr changes it to be 36W cbf to do the / * thing and its not really important

Ahhh, sorry, I'm stupid
from the picture it was not to understand

screenshot was just the previous behaviour

test fail erroneous


Pin cuda-version for CUDA 12.x series

Description

Closes #61
Closes #62
Closes #60

Todos

[X] Pin to cuda-version instead of cudatoolkit for the 12.x series
[ ] Test

@jaimergp this would be great to get out the door, do you need help with testing or anything?

I can also confirm that this fix work!

Sorry about the delay. Let's see how this works.


Global parameter -g yields "flag provided but not defined"

Describe the bug
I'm attempting to install Goss across a large environment (Mostly Redhat 7 machines) as a service using systemd. Debugging this took quite some time because Goss doesn't support distinct exit codes yet, but in my problem solving I found that Goss doesn't find the global parameter -g --gossfile acceptable.
How To Reproduce

Execute Goss v0.3.10 with the commands noted below in "Actual Behavior"
Expected Behavior

Expected Goss to behave according to the documentation, in that the options -g and
--gossfile would read in the provided YAML file without issues (excluding YAML formatting or FileNotFound errors).
# ./goss -h
NAME:
   goss - Quick and Easy server validation

USAGE:
   goss [global options] command [command options] [arguments...]

VERSION:
   v0.3.10

GLOBAL OPTIONS:
   --gossfile value, -g value  Goss file to read from / write to (default: "./goss.yaml") [$GOSS_FILE]
   --vars value                json/yaml file containing variables for template [$GOSS_VARS]
   --vars-inline value         json/yaml string containing variables for template (overwrites vars) [$GOSS_VARS_INLINE]
   --package value             Package type to use [rpm, deb, apk, pacman]
   --help, -h                  show help
   --version, -v               print the version

Actual Behavior

Attempted to run base command from /opt/goss/ as working directory, using both the -g and --gossfile parameters, varying the order.

./goss serve -f json --gossfile /opt/goss/goss.yml
2020/04/03 11:57:51 flag provided but not defined: -gossfile

./goss serve -f json -g /opt/goss/goss.yml
2020/04/03 12:01:32 flag provided but not defined: -g

./goss serve --gossfile /opt/goss/goss.yml -f json
2020/04/03 12:02:01 flag provided but not defined: -gossfile

./goss serve -g /opt/goss/goss.yml -f json
2020/04/03 12:05:46 flag provided but not defined: -g

Environment:
Goss Version: v0.3.10
OS Version: Red Hat Enterprise Linux Server release 7.6 (Maipo)
Uname Output: Linux ****************** 3.10.0-957.21.2.el7.x86_64 x86_64 x86_64 x86_64 GNU/Linux
Ansible Version: v2.8.0

I'd like to also note, the service still works because the default action is to look in the working directory for a gossfile, but we'd like to be able to specify it in the future.

Not on a computer to send exact command at the moment.
Given:
goss [global options] command [command options] [arguments...]
Does the following work?
goss -g /opt/goss/goss.yml serve ...

Wow, RTFM to me.
Thank you, that's what I was doing wrong.

Lol, all good. Glad I was able to help/unblock you. :)


Added cludden/protoc-gen-go-temporal

I do not now if you update plugins manually or automatically. Anyway, to ease the task to add cludden/protoc-gen-go-temporal plugin I added a PR.
It's a great plugin that helps a lot with temporal workflows. Would appreaciate if you can add it to the community plugins.
This solves #694.

Once a PR is merged the added plugin(s) go into an automated workflow, and eventually get pushed to https://buf.build/plugins.
However, we're a bit selective with the public plugins we push to the BSR.
At this moment, it doesn't appear this is a widely-used plugin and we're unlikely to accept the PR, but we should keep the issue open and reevaluate if demand increases.
Side note, users on a paid plan can upload any Protobuf plugin and use them privately within their organization.
https://buf.build/docs/bsr/remote-plugins/custom-plugins

Thank you for the contribution, but at the present we're going to close this PR. There is not a significant demand at the moment for this plugin. We may consider adding it at some point in the future.
In the meantime, users on paid plans could upload this as a custom plugin. See https://buf.build/docs/bsr/remote-plugins/custom-plugins for more details.


Create Python Role, aggregate Python pkgs...

So,
Python is a must to have.

Also, we aren't installing Python at the beginning of the playbooks, we're assuming that it's already there.
So, the idea will be to create a "python" role and aggregate all Python packages there, in one place. This might make it easier to go to Python 3, that will happen in a year.
Cheers!

Insights:
https://gist.github.com/gwillem/4ba393dceb55e5ae276a87300f6b8e6f