RN Typedef redefinition with different types ('uint8_t' (aka 'unsigned char') vs 'enum clockid_t')

Description
I am currently building a project with React Native 0.67 and and Monterey MacOS, intel Chip.
I have tried about every possible solution, but yet, the problem persists.
The Problem is the following:
after carefully following the instructions in react setup page  to initialize a new app, when I execute the run-ios command I get the following error on my console, which seems to be related to Flipper:

** BUILD FAILED **
The following build commands failed:
CompileC /Users/dr/Library/Developer/Xcode/DerivedData/Test-gxglauoeoglzeobdubkxzgziurmr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/RCT-Folly.build/Objects-normal/x86_64/SysUio.o /Users/dr/Library/Mobile\ Documents/com~apple~CloudDocs/2.\ Business/2.\ Software\ As\ A\ Service.nosync/socholar-native.nosync/Test/ios/Pods/RCT-Folly/folly/portability/SysUio.cpp normal x86_64 c++ com.apple.compilers.llvm.clang.1_0.compiler (in target 'RCT-Folly' from project 'Pods')
(1 failure)

When I try running it from Xcode I get the following Error in RCT_Folly directory:

RN Typedef redefinition with different types ('uint8_t' (aka 'unsigned char') vs 'enum clockid_t')

Version
0.67
Output of npx react-native info
System:
OS: macOS 12.2.1
CPU: (8) x64 Intel(R) Core(TM) i7-6920HQ CPU @ 2.90GHz
Memory: 48.18 MB / 16.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 14.15.1 - /usr/local/bin/node
Yarn: 1.22.17 - /usr/local/bin/yarn
npm: 7.22.0 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.2 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.2, iOS 15.2, macOS 12.1, tvOS 15.2, watchOS 8.3
Android SDK:
API Levels: 28, 29, 30, 32
Build Tools: 28.0.3, 29.0.2, 30.0.2, 32.0.0, 32.1.0
System Images: android-28 | Google APIs Intel x86 Atom_64, android-29 | Google APIs Intel x86 Atom_64, android-30 | Google APIs Intel x86 Atom, android-30 | Google Play Intel x86 Atom
Android NDK: Not Found
IDEs:
Android Studio: 2021.1 AI-211.7628.21.2111.8139111
Xcode: 13.2.1/13C100 - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_292 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.67.3 => 0.67.3
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
Steps To Reproduce

init a new react project with macOS Montery or Big Sur and React Native 0.67
Execute run-ios command
See It Fail

FYI works on Android
Snack, code example, screenshot, or link to a repository
When Runing with npx react-native run-ios

When running with Xcode in Time.h file

I have also getting the same issue on last 3 days. can you please fixed that issue

Facing same issue

same issue

also troubled by this :-/

also facing same issue

I have set up all things from the scratch then I didn’t get the same issue.

same here

Found the solution guys, apparently it was a bug with that particular version of React Native, I regressed to version 64 and it is working great. I have not tried the newest version 0.67 to check if it works there though!
Hop that helps!
@official-bharat what version are you using?

Still facing the same issue with upgrading React Native for months now. It's blocking the update of the other dependencies that depend on React Native being updated as well

same issue here

@yodarango I had the same issue on RN 0.67.4 and Xcode version 13.1.1
Fixed it by upgrading Xcode to 13.2.
Try to use RN 0.67.4 and Xcode to 13.2.*

in Time.h file you should do below changes at top:-
change this
(TARGET_OS_IPHONE && (__IPHONE_OS_VERSION_MIN_REQUIRED < _IPHONE10_0))
to this
(TARGET_OS_IPHONE && (__IPHONE_OS_VERSION_MIN_REQUIRED < _IPHONE12_0))
just you have to make 10 to 12 and clean the project, run it.
I hope it will solve you issue.

 I couldn't find the actual comment that I got it from but this is a workaround I got from a forum. Just change this in your Podfile

Not working for me with react-native version 0.68.2

 I couldn't find the actual comment that I got it from but this is a workaround I got from a forum. Just change this in your Podfile

Not working for me with react-native version 0.68.2

worked by downgrading RCT-folly deployment target to ios 13.

 I couldn't find the actual comment that I got it from but this is a workaround I got from a forum. Just change this in your Podfile

Found the original comment just to give credit to @mikehardy for the solution I posted earlier: https://github.com/facebook/flipper/issues/834#issuecomment-902886771

https://github.com/facebook/flipper/issues/834#issuecomment-919816118

same thing with react native 0.66.0

Same thing with react native 0.70.0

Same thing with react native 0.70.10

facebook/flipper#834 (comment)

No, event without use_frameworks! I get this error.

 I couldn't find the actual comment that I got it from but this is a workaround I got from a forum. Just change this in your Podfile

This seems to work for me (RN 0.72.5)
`sed -i -e  $'s/__IPHONE_10_0/__IPHONE_13_0/' #{installer.sandbox.root}/RCT-Folly/folly/portability/Time.h`

Same thing with react native 0.72.8


Typescript - Update Node version

:bug: Bug Report
What is the problem?
Workshop uses EOL version of Node Lambda @ https://cdkworkshop.com/20-typescript/30-hello-cdk/200-lambda.html
EOL July 30, 2021: https://docs.aws.amazon.com/lambda/latest/dg/runtime-support-policy.html
Reproduction Steps
N/A
Verbose Log
N/A
Environment

CDK CLI Version: 1.112
Language: TypeScript

Thanks for submitting this report, is this something you'd be willing to create a PR for?

Yes, I have reserved some time for this tomorrow.


Documentation Addition

There are 2 new docs in the docs directory: on how to create a service, and how to use agent handoff
Also added JSDoc comments to the router.

Hey @xtellurian I couldn't see where the documentation is referenced from the other pages. Please add links in relevant places in main readme.md and also on the last documentation page to create a chain to the next document (bottom of https://github.com/MSFTAuDX/BotBoiler/blob/master/docs/testableDialogs.md).

Added a link in the readme
I also removed one of those doco's after reading it again -> it's a duplicate of the createInjectableCoimponent doc


feat: use TRPC for state and remove the spinner flash

Description

If the server has any kind of failure after a message has been sent, then it will fail to send the "state" message
This results in some odd behaviour on the front end
We now have TRPC for loading the initial chat, so... let's get rid of the state command use TRPC to fetch the chat after each message has arrived
This moves the TRPC call inside the ChatProvider, which can then refetch the query
For performance we change the tempLessonPlan to use a ref instead of state, which improves performance during streaming
Plus, by doing so we have more control and can get rid of the flash where the lessonPlan is replaced by the new state, which currently makes each section flip back to the loading state
The end result is that the front end doesn't jump and show spinners after each message, plus we can guarantee after each message that the front end has a matching lesson plan state to what the server has

How to test

Go through the chat flow
Keep an eye on the sections as they appear on the right
You should not see the 500ms spinner animation after the message to the user appears on the left
Reloading the chat you should see the same state
In the Network panel you should see TRPC calls to fetch the chat after each message has completed

:tada: This PR is included in version 1.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


I'm unable to get it to work

The script doesn't appear to be receiving any data from VRChat, the console output never changes.
It sends default data on startup to VRChat and appears in the Debug menu, maybe it's a prefab setup issue, the instructions aren't very clear.
Eventually I'd like to implement the functionality into my Rust program and on all my public avatars.
You should try getting this functionality put into https://github.com/VolcanicArts/VRCOSC

I have created a GUI version of this app, not tested yet as its impossible to grab stuff in desktop
https://github.com/GeekAndProud/OSCLeashGUI
https://i.imgur.com/nDN6QEz.png

I have created a GUI version of this app, not tested yet as its impossible to grab stuff in desktop
https://github.com/GeekAndProud/OSCLeashGUI
https://i.imgur.com/nDN6QEz.png

This could probably be made easier to setup with VRCFury


"element selected as link has no 'href' attribute" when not using any link

I'm getting  the following error:
ERROR rsspls::feed > unable to process RSS item matching 'data'

Caused by:
    element selected as link has no 'href' attribute

When I don't actually use the link element. I just have this in feeds.toml:
[[feed]]
title = "JEWISH NATIONAL FUND v. MINISTER OF NATIONAL REVENUE A-245-24"
filename = "JNFvMNR.rss"
user_agent = "Mozilla/5.0"
[feed.config]
url = "file://[local path redacted]/A-245-24.xml"
item = "data"
heading = "RECORDED_ENTRY"
summary = "DOCNO"
date = "FILING_DATE"

So I'm a bit confused as to why it's searching for a link. Though it is true there are no hrefs anywhere in the source xml file.
For reference, this flows from the same situation described in issue 61

I should say I also have this in the feeds.toml, I just omitted it for brevity:

rsspls currently requires a link element. There is fallback behaviour for compatibility with older versions if link is not specified. The docs have more details: https://rsspls.7bit.org/documentation/#feed-config-link
It seems like link is not required in an RSS feed so I think the requirement for one could be relaxed.


PDF figures do not resize in PDF output

The PDF output of the template does not resize PDF figures to the page size, even when including the {out-width="100%"} option. Large PDF figures will overflow the page. This seems to be related to the template as there is no problem in normal Quarto PDF output. Whether the figure is a stand-alone figure, in a div, or a subfigure, seems to have no effect.
I could only test the figure resizing for Quarto version 1.3.450, because of https://github.com/arcaldwell49/cik-quarto/issues/5.

Note: I will add this to the template. Just add that to the partials for the current manuscript you're working on.

Also, for figure 2, please use the following code chunk
::: {#fig-2,layout-ncol="1"}
![](fig2a.png)

![](fig2b.png)

![](fig2c.png)
**Figure 2:** Time-on-target durations for all experimental groups in the easy task (a), the moderate difficulty task (b), and the hard task (c). Error bars represent 95% confidence intervals.
:::

Thanks, the partial file code worked for me, although I had to put \usepackage{graphicx} in _preamble.txt

Also, for figure 2, please use the following code chunk
::: {#fig-2,layout-ncol="1"}
![](fig2a.png)

![](fig2b.png)

![](fig2c.png)
**Figure 2:** Time-on-target durations for all experimental groups in the easy task (a), the moderate difficulty task (b), and the hard task (c). Error bars represent 95% confidence intervals.
:::

You're right, the {out-width="20%"} were products of my earlier experiments that I missed to delete. I deliberately did not write the caption as a main caption but as the caption of the last subfigure, otherwise I got problems in the html formatting. I'll open a new issue for this.


Added selectionControls property

Added this property from TextField.
This closes #68

Thank you so much!

Good morning @BertrandBev! Will this be merged 🙂 ?

Good morning @BertrandBev! Will this be merged 🙂 ?

Sure thing! Thanks!

Is this released to the pub.dev? The last changelog does not mention it.


Update SITES.md

Thanks for the PR! We've simplified the process of adding your site/repo to the list. Instead of updating SITES.md you would add the utterances topic to your repo.


Max & ideal bit-rate

I'd like to know the ideal and max bit-rate. 8Mbps seems slow to me. I changed bit-rate to 100Mbps and couldn't notice any change. It's similar to 8Mbps one.

The parameter is just forwarded to MediaCodec:
https://github.com/Genymobile/scrcpy/blob/ffa8c66979d4262a95cad584894b9a5fbeea0e84/server/src/main/java/com/genymobile/scrcpy/ScreenEncoder.java#L142
How it is handled depends on your hardware encoder. Typically, above 10~15Mbps it changes nothing.

Thanks for the reply. I've used Vysor and other apps in the past only to be vexed by their functionalities. scrcpy is sleek.

why I cannot set it to 100 or 200M when I connect it via usb3.1?

You can, but the codec decides what to do with the requested bitrate.


Replace periods  within figure names with underscores

@rhiannonlynne this appears obsolete now .. changes are already in main


Add sphinx autodoc for LanguageModel

We want the autodoc so the references thorough the documentation link back to this class:
file:///Users/john/tools/edsl/.temp/docs/language_models.html

They already do?
Top of an .rst page should look like this:
.. _language_models:
And then references back to that page should look like this:
```:ref:`language_models````

done


Over 700 warnings

Is this normal? I just imported everything into Xcode and compiled it into an app and Xcode 9.3 shows over 700 warnings. Also it is not in Swift 4.
Will this be fixed anytime soon?

Hello, we have just released version 3.2, that supports Swift 4. A lot of warnings resolved

Thanks a lot. Will try it immediately.


Using Excalidraw React build in a vanilla JS application

Discussed in https://github.com/excalidraw/excalidraw/discussions/8163

Originally posted by Favourz1 June 21, 2024
Hi, I'm using the build of Excalidraw from React in a vanilla js web application with websocket.
Ideally, I'm using the WebSocket to enable real-time collaboration between many users.

Problem:  Canvas doesn't sync together with the same elements.
I'm using the same websocket implementation for a chat feature which works well, so not a websocket error.

I also tried saving the data to a useref to mimic external communication and restoring on the React app before building and it worked.

I can confirm the data is sending and receiving on both ends but for some reasons, it doesn't update the canvas with current elements on each user screen canvas connected.

An example code of my react app can be found in a  codesandbox example with detailed comments

Also, an overview of my websocket and vanilla js implementation can be found in a codesnadbox example with detailed comments.

I also commented on other methods I tried in the files.

Please help me resolve this as it's an urgent issue for a product I'm building.
cc: @ad1992 & @dwelle

What are you building?


Dynamically set html lang attribute

Question / Discussion
Is there a way to dynamically set html lang attribute?
For example depending on cookie value.
Maybe somehow passing cookies through renderPage in server/index.js.
And generaly server rendering pages depending on cookie

https://brillout.github.io/rules/

Okay, I get it. I haven't provided any information about my own attempts, but that's because I don't even know where to start.
I wanted a push in some direction.
And in any case, it's not hard to just leave at least one sentence, like "Can't help", than to throw a link with the rules at me

https://github.com/brillout/vite-plugin-ssr/discussions/526


Is there a way to reboot the instance before attaching it to the cluster?

Tell us about your request
I want to reboot my node before attaching it to the cluster.
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
I have a requirement to disable ECC for GPU cores of the nodes that requires me to reboot the node before the configuration sticks.
Are you currently working around this issue?
Currently I am planning to use Cluster Autoscaler with Lifecycle hooks to do this.
Additional Context
Please feel free to ask any further queries.
Attachments
No response
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

You could do this today by specifying a reboot command within user-data, but you'd need to use a Custom AMI Family which means you'd need to take care of the bootstrap process as well...
A workaround, that doesn't feel great, is to use at or setup a systemd service as part of your user-data when using the AL2/Ubuntu AMI Family and triggering a reboot after kubelet starts. There's a race when doing this since kubelet could start scheduling pods when it comes online, but a systemd script that runs when kubelet starts would likely win every time since networking still needs to be initialized before pods can actually start. Another option would be to taint the node within the Provisioner and then remove the taint via a systemd startup unit (which you would install in user-data).
We've had some requests for postBootstrap user-data, but I don't think there's any consensus about this and wouldn't be a near term solution.

Hi @bwagner5 - Thanks for your reply on this.
postBootstrap UserData would really resolve this.
One follow up query that I have; there is a line in the documentation - "GPUs are only supported with AL2 and Bottlerocket". So in case I do a custom bootstrap using custom AMI, does it mean that I can't use GPU with Ubuntu/Custom AMI?
I did try the reboot option using the cdk code for launch template below -
gpu_node_template={
            'subnetSelector': {
            'Name': self.stack_name+'/'+vpc.node.id+'/PrivateSubnet*'
            },
            'securityGroupSelector': {
            'aws:eks:cluster-name': cluster.cluster_name
            },
            'amiFamily': 'AL2',
            'userData':
                """
                #!/bin/bash -xe
                if nvidia-smi --query --display=ECC | grep --quiet '^ *\(Current\|Pending\) *: Enabled$'; then
                    nvidia-smi --ecc-config=0
                    sleep 90 && reboot &
                else /bin/echo "ECC disabled" >> /tmp/output.txt
                fi
                """
        }

And it worked. The only problem is the pods start scheduling and start the job and the reboot acts as an interrupt which stops the job. So reboot at or sleep + reboot is not an option.
Do you have any example of using it with systemd script?

I used the following if anybody stumbles upon this -
gpu_node_template={
            'subnetSelector': {
            'Name': self.stack_name+'/'+vpc.node.id+'/privateSubnet*'
            },
            'securityGroupSelector': {
            'aws:eks:cluster-name': cluster.cluster_name
            },
            'amiFamily': 'Custom',
            'amiSelector': {
                'aws::ids' : 'ami-abc123xyzabd'
                },
            'blockDeviceMappings': [
                {
                'deviceName': '/dev/sda1',
                'ebs': {
                    'volumeSize': '100Gi',
                    'volumeType': 'gp3'
                }
                }
            ],
            'userData':
f"""MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="//"

--//
Content-Type: text/x-shellscript; charset="us-ascii"
#!/bin/bash
set -x

cat <<EOF >> /tmp/bootstrapEKS
#! /bin/bash
B64_CLUSTER_CA=$(aws eks describe-cluster --name {cluster.cluster_name} --region {self.region} | jq -r '.cluster.certificateAuthority.data')
API_SERVER_URL=$(aws eks describe-cluster --name {cluster.cluster_name} --region {self.region} | jq -r '.cluster.endpoint')

/etc/eks/bootstrap.sh {cluster.cluster_name} --kubelet-extra-args '--max-pods=29' --b64-cluster-ca $B64_CLUSTER_CA --apiserver-endpoint $API_SERVER_URL --use-max-pods false
systemctl disable bootstrapEKS
EOF

cp /tmp/bootstrapEKS /var/lib/cloud/instance/bootstrapEKS.sh

chmod +x /var/lib/cloud/instance/bootstrapEKS.sh

cat <<EOF >> /etc/systemd/system/bootstrapEKS.service
[Unit]
Description=Bootstrap EKS Service
Before=motd-news.service

[Service]
Type=oneshot
ExecStart=/var/lib/cloud/instance/bootstrapEKS.sh
StandardOutput=journal+console

[Install]
WantedBy=multi-user.target
EOF

systemctl enable bootstrapEKS

if nvidia-smi --query --display=ECC | grep --quiet '^ *\(Current\|Pending\) *: Enabled$'; then
    nvidia-smi --ecc-config=0
    export DEBIAN_FRONTEND=noninteractive
    apt-get --purge remove "*cuda*" "*cublas*" "*cufft*" "*cufile*" "*curand*" "*cusolver*" "*cusparse*" "*gds-tools*" "*npp*" "*nvjpeg*" "nsight*" "*nvvm*" -y
    apt-get --purge remove "*nvidia*" "libxnvctrl*" -y
    apt-get autoremove -y
    reboot
else /bin/echo "ECC disabled" >> /tmp/output.txt
fi
--//--
"""
        }


Rectifying Css for Chapters

Description
The css for Chapters is not eye catching needs some correction to be done.
Screenshot:

Please assign me this under GSSOC 2023

Hey @I-shika , can u pls check this once again.

yes, I recloned the repository right now but the chapter tab is same as my screenshot which I have send earlier

Yea, those bullet list are there, but I have already assigned this issue to others, if there is anything else u would like to work on, pls go ahead!


Where to find ABX triplets for evaluation? (https://github.com/MarvinLvn/CPC2/blob/master/docs/training_and_eval.md)

Instructions at https://github.com/MarvinLvn/CPC2/blob/master/docs/training_and_eval.md state that
"$PATH_ITEM_FILE is the path to the .item file containing the triplet annotations" . However, when I download the ZRC2017 dataset (zrc2017-test-dataset 	9.6 GB 	57a692729215e4b14cd7da5b8d96ff77)  from Zerospeech.com, there is no obvious file that would correspond to the triplet annotations. For each language subfolder, there's simply "10s  120s  1s  english.phn  english.vad.csv  english.wrd", corresponding to different-duration samples and then phone, word and VAD annotation files. Hence, I can't run the ABX eval using the script.

Ah, I got help from a colleague. The item files are "hidden" in the midst of the .wav files in the respective .wav folders, and have .item suffix.


There is no indication in the dashboard that the database is out of date

In supporting a 5.7 install last week, @mnkras and I noticed that their database was out of date. I think we should add some dashboard nag that says "Hay u, you need to update your database!" when the version and the installed version don't match.

This was a good idea – in fact we've gone one better and actually made it so that sites are automatically updated to the proper database version automatically.


Don't fetch ACME certificates for frontends using non-TLS entrypoints (#989)

Fix for #989.

ping @containous/traefik


Deadlock in XKeysymToKeycode

When I mashed the Shift+F9 key during a loading screen in Metro 2033: Redux, the game froze. gdb shows a deadlock with glxosd in the callstack.
First time this happened to me. I will update this ticket if it happens again.
GLXOSD commit b786796dd8feb251b2

(gdb) bt
#0  __lll_lock_wait () at ../nptl/sysdeps/unix/sysv/linux/x86_64/lowlevellock.S:135
#1  0x00007f285f6fb657 in _L_lock_909 () from /lib/x86_64-linux-gnu/libpthread.so.0
#2  0x00007f285f6fb480 in __GI___pthread_mutex_lock (mutex=0x28dc470) at ../nptl/pthread_mutex_lock.c:79
#3  0x00007f285ed86c33 in ?? () from /home/scrawl/Steam/ubuntu12_32/steam-runtime/amd64/usr/lib/x86_64-linux-gnu/libX11.so.6
#4  0x00007f285ede55e1 in _XkbReloadDpy () from /home/scrawl/Steam/ubuntu12_32/steam-runtime/amd64/usr/lib/x86_64-linux-gnu/libX11.so.6
#5  0x00007f285ede5f84 in XKeysymToKeycode () from /home/scrawl/Steam/ubuntu12_32/steam-runtime/amd64/usr/lib/x86_64-linux-gnu/libX11.so.6
#6  0x00007f2860c3628b in glxosd::keyComboMatches(glxosd::KeyCombo, XKeyEvent*) () from /usr/local//lib/x86_64-linux-gnu//glxosd/libglxosd.so
#7  0x00007f2860bcdbdb in glxosd::GLXOSD::osdEventFilter(_XDisplay*, _XEvent*, char*) () from /usr/local//lib/x86_64-linux-gnu//glxosd/libglxosd.so
#8  0x00007f2860bce4e9 in glxosd::osdEventFilter(_XDisplay*, _XEvent*, char*) () from /usr/local//lib/x86_64-linux-gnu//glxosd/libglxosd.so
#9  0x00007f285ed749e9 in XCheckIfEvent () from /home/scrawl/Steam/ubuntu12_32/steam-runtime/amd64/usr/lib/x86_64-linux-gnu/libX11.so.6
#10 0x00007f2860ebfbe6 in XCheckIfEvent () from /usr/local//lib/x86_64-linux-gnu//glxosd/libglxosd-glinject.so
#11 0x00007f2861307e5a in XCheckIfEvent () from /home/scrawl/Steam/ubuntu12_64/gameoverlayrenderer.so
#12 0x00007f2860ecc1ca in handle_buffer_swap(_XDisplay*, unsigned long) () from /usr/local//lib/x86_64-linux-gnu//glxosd/libglxosd-glinject.so
#13 0x00007f2860ecc48a in glXSwapBuffers () from /usr/local//lib/x86_64-linux-gnu//glxosd/libglxosd-glinject.so
#14 0x00007f28612fd0a1 in glXSwapBuffers () from /home/scrawl/Steam/ubuntu12_64/gameoverlayrenderer.so
#15 0x0000000000dcabca in cb_replay(unsigned int*) ()
#16 0x0000000000db077a in backend_layer::kickoff(bool) ()
#17 0x0000000000dd48e1 in crender::render_loader() ()
#18 0x0000000000dd5043 in crender::render_internal(urender::server::presentation_mode) ()
#19 0x0000000000dd4f8d in crender::render(urender::server::presentation_mode) ()
#20 0x0000000000569ca7 in cengine::loader_render_while_loading() ()
#21 0x000000000056a40c in cengine::loader_execute(char const*, int, int) ()

Thanks for reporting this bug. Definitely a mistake on my end, I will try to fix it tomorrow.

Your predicate procedure must decide if the event is useful without calling any Xlib functions.
http://tronche.com/gui/x/xlib/event-handling/manipulating-event-queue/selecting-using-predicate.html

Seems to have been fixed in<PHONE_NUMBER>29935a6101016fb21ee573bbe9fe2d, so we can close this ticket I presume.


Fix list scrolling

@rrolf the "list view does not scroll back" issue should be fixed, can you confirm that?
And how exactly can I reproduce the other issue? I seem to be unable to do so. :thinking:


Correction to command line arguments

The following two commands needed spaces added as they weren't valid, didn't work and were inconsistent with the third command on the same page.

Setup/SkipRules=Cluster_VerifyForErrors/Action=InstallFailoverCluster
updated to
Setup /SkipRules=Cluster_VerifyForErrors /Action=InstallFailoverCluster

Setup/SkipRules=Cluster_VerifyForErrors/Action=CompleteFailoverCluster
updated to
Setup /SkipRules=Cluster_VerifyForErrors /Action=CompleteFailoverCluster

@jantoney : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@jantoney Thanks for your contribution! Merged the PR.


Flutter crash on Android STB when connecting to https

Hi, I have a problem when trying to launch Mobile apps on my Android STB
the problem only happens when trying to load image with "https", but it not crashed when load image with "http"
previously on old flutter, sorry I'm forget the version, there's no issue, but with flutter version 1.12.13+hotfix.8 the apps always crash when loading image
FYI, it's normal in Mobile Devices, but crash in Android STB for TV
this is the output from logcat :
Fatal signal 6 (SIGABRT), code -6 in tid 11506 (Thread-361)
*** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
Build fingerprint: 'ZTE/B860H_Market1/B860H_V1:6.0.1/MHC19J/20180316.183142:user/release-keys'
Revision: '0'
ABI: 'arm'
pid: 11492, tid: 11506, name: Thread-361  >>> com.thecips.prelivemovies_stb <<<
signal 6 (SIGABRT), code -6 (SI_TKILL), fault addr --------
    r0 00000000  r1 00002cf2  r2 00000006  r3 ef13f978
    r4 ef13f980  r5 ef13f930  r6 00000016  r7 0000010c
    r8 e370e1dd  r9 00000001  sl 00000000  fp 00000000
    ip 00000006  sp ef13eaf0  lr f701848d  pc f701a888  cpsr 400e0010
backtrace:
    #00 pc 00042888  /system/lib/libc.so (tgkill+12)
    #01 pc 00040489  /system/lib/libc.so (pthread_kill+32)
    #02 pc 0001cb0f  /system/lib/libc.so (raise+10)
    #03 pc 00019cb1  /system/lib/libc.so (__libc_android_abort+34)
    #04 pc 00017a64  /system/lib/libc.so (abort+4)
    #05 pc 0121e2c5  /data/app/com.thecips.prelivemovies_stb-2/lib/arm/libflutter.so (offset 0xfcf000)
    #06 pc 000409a9  /system/lib/libc.so (pthread_once+54)
    #07 pc 0122beed  /data/app/com.thecips.prelivemovies_stb-2/lib/arm/libflutter.so (offset 0xfcf000)
    #08 pc 0121e175  /data/app/com.thecips.prelivemovies_stb-2/lib/arm/libflutter.so (offset 0xfcf000)
    #09 pc 01215b2d  /data/app/com.thecips.prelivemovies_stb-2/lib/arm/libflutter.so (offset 0xfcf000)
    #10 pc 012159a7  /data/app/com.thecips.prelivemovies_stb-2/lib/arm/libflutter.so (offset 0xfcf000)
    #11 pc 0124c6c7  /data/app/com.thecips.prelivemovies_stb-2/lib/arm/libflutter.so (offset 0xfcf000)
    #12 pc 0124ba7b  /data/app/com.thecips.prelivemovies_stb-2/lib/arm/libflutter.so (offset 0xfcf000)
    #13 pc 012553e5  /data/app/com.thecips.prelivemovies_stb-2/lib/arm/libflutter.so (offset 0xfcf000)
    #14 pc 011f5b4b  /data/app/com.thecips.prelivemovies_stb-2/lib/arm/libflutter.so (offset 0xfcf000)
    #15 pc 011f59d7  /data/app/com.thecips.prelivemovies_stb-2/lib/arm/libflutter.so (offset 0xfcf000)
    #16 pc 012ce8f3  /data/app/com.thecips.prelivemovies_stb-2/lib/arm/libflutter.so (offset 0xfcf000)
    #17 pc 00000ac0  <unknown>
Tombstone written to: /data/tombstones/tombstone_00

need advice, thanks

Hi @indracip
are you experiencing the issue with the dev channel as well?
Could you please provide a reproducible minimal code sample.
Thank you

Ok, I will try in dev channel

hi, @iapicca
I've been change dev channel in flutter, but the output still same, force close when loading images with https
this the sample code When trying to load Image
Image.network(
"https://image.tmdb.org/t/p/w185/8WUVHemHFH2ZIP6NWkwlHWsyrEL.jpg",
// height: MediaQuery.of(context).size.height,
width: MediaQuery.of(context).size.width,
fit: BoxFit.contain,
alignment: Alignment.topRight,
),

Symbolized the stack trace with the android-arm debug build of engine version e1e6ced81d029258d449bdec2ba3cddca9c2ca0c
The abort call is happening here:
https://boringssl.googlesource.com/boringssl/+/4dfd5af70191b068aebe567b8e29ce108cee85ce/crypto/fipsmodule/rand/urandom.c#182
Something is failing when the BoringSSL library tries to invoke getrandom in your environment.

@jason-simmons do you have a solution for this ? What can I do to fix this crash ?

Try building the Flutter engine locally and adding logging to debug inside the BoringSSL library.  Or investigate whether there is something unusual about the STB platform that would affect usage of getrandom or /dev/urandom.

@jason-simmons sorry I never try to building flutter engine locally, could you please help how to this ?
is there any tutorial to build this ?
thank you so much

See https://github.com/flutter/flutter/wiki/Setting-up-the-Engine-development-environment and https://github.com/flutter/flutter/wiki/Compiling-the-engine

Hi, I noticed that some customers of my app experienced this crash.
Stacktrace:
1 #00 pc 0003b134 /system/lib/libc.so (tgkill+12) [armeabi-v7a]
2 #01 pc 00017711 /system/lib/libc.so (pthread_kill+52) [armeabi-v7a]
3 #02 pc 00018327 /system/lib/libc.so (raise+10) [armeabi-v7a]
4 #03 pc 00014bd1 /system/lib/libc.so (__libc_android_abort+36) [armeabi-v7a]
5 #04 pc 00012f58 /system/lib/libc.so (abort+4) [armeabi-v7a]
6 #05 pc 00375f41 libflutter.so init_once (../../third_party/boringssl/src/crypto/fipsmodule/rand/urandom.c:182) [armeabi-v7a]
7 #06 pc 00017d3b /system/lib/libc.so (pthread_once+90) [armeabi-v7a]
8 #07 pc 00389943 libflutter.so CRYPTO_once (../../third_party/boringssl/src/crypto/thread_pthread.c:89) [armeabi-v7a]
9 #08 pc 003759e1 libflutter.so CRYPTO_sysrand (../../third_party/boringssl/src/crypto/fipsmodule/rand/urandom.c:320) [armeabi-v7a]
10 #09 pc 00375745 libflutter.so RAND_bytes_with_additional_data (../../third_party/boringssl/src/crypto/fipsmodule/rand/rand.c:243) [armeabi-v7a]
11 #10 pc 0039b4eb libflutter.so ssl_client_handshake (../../third_party/boringssl/src/ssl/handshake_client.cc:415) [armeabi-v7a]
12 #11 pc 0039a81b libflutter.so ssl_run_handshake (../../third_party/boringssl/src/ssl/handshake.cc:667) [armeabi-v7a]
13 #12 pc 003a4fcd libflutter.so SSL_do_handshake (../../third_party/boringssl/src/ssl/ssl_lib.cc:892) [armeabi-v7a]
14 #13 pc 0034f0d9 libflutter.so Builtin_SecureSocket_Connect (../../third_party/dart/runtime/bin/secure_socket_filter.cc:0) [armeabi-v7a]
15 #14 pc 003ee2db libflutter.so AutoScopeNativeCallWrapperNoStackCheck (../../third_party/dart/runtime/vm/native_entry.cc:215) [armeabi-v7a]
16 #15 pc 003ee13f libflutter.so LinkNativeCall (../../third_party/dart/runtime/vm/native_entry.cc:362) [armeabi-v7a]
17 #16 pc 00004714 /data/app/com.bilibili.comic-1/lib/arm/libapp.so (_kDartVmSnapshotInstructions+10004) [armeabi]

They are all low-end products. It's a pity that I can't reproduce and investigate further without such device.

Hi, I noticed that some customers of my app experienced this crash.
Stacktrace:
1 #00 pc 0003b134 /system/lib/libc.so (tgkill+12) [armeabi-v7a]
2 #01 pc 00017711 /system/lib/libc.so (pthread_kill+52) [armeabi-v7a]
3 #02 pc 00018327 /system/lib/libc.so (raise+10) [armeabi-v7a]
4 #03 pc 00014bd1 /system/lib/libc.so (__libc_android_abort+36) [armeabi-v7a]
5 #04 pc 00012f58 /system/lib/libc.so (abort+4) [armeabi-v7a]
6 #05 pc 00375f41 libflutter.so init_once (../../third_party/boringssl/src/crypto/fipsmodule/rand/urandom.c:182) [armeabi-v7a]
7 #06 pc 00017d3b /system/lib/libc.so (pthread_once+90) [armeabi-v7a]
8 #07 pc 00389943 libflutter.so CRYPTO_once (../../third_party/boringssl/src/crypto/thread_pthread.c:89) [armeabi-v7a]
9 #08 pc 003759e1 libflutter.so CRYPTO_sysrand (../../third_party/boringssl/src/crypto/fipsmodule/rand/urandom.c:320) [armeabi-v7a]
10 #09 pc 00375745 libflutter.so RAND_bytes_with_additional_data (../../third_party/boringssl/src/crypto/fipsmodule/rand/rand.c:243) [armeabi-v7a]
11 #10 pc 0039b4eb libflutter.so ssl_client_handshake (../../third_party/boringssl/src/ssl/handshake_client.cc:415) [armeabi-v7a]
12 #11 pc 0039a81b libflutter.so ssl_run_handshake (../../third_party/boringssl/src/ssl/handshake.cc:667) [armeabi-v7a]
13 #12 pc 003a4fcd libflutter.so SSL_do_handshake (../../third_party/boringssl/src/ssl/ssl_lib.cc:892) [armeabi-v7a]
14 #13 pc 0034f0d9 libflutter.so Builtin_SecureSocket_Connect (../../third_party/dart/runtime/bin/secure_socket_filter.cc:0) [armeabi-v7a]
15 #14 pc 003ee2db libflutter.so AutoScopeNativeCallWrapperNoStackCheck (../../third_party/dart/runtime/vm/native_entry.cc:215) [armeabi-v7a]
16 #15 pc 003ee13f libflutter.so LinkNativeCall (../../third_party/dart/runtime/vm/native_entry.cc:362) [armeabi-v7a]
17 #16 pc 00004714 /data/app/com.bilibili.comic-1/lib/arm/libapp.so (_kDartVmSnapshotInstructions+10004) [armeabi]

They are all low-end products. It's a pity that I can't reproduce and investigate further without such device.

I think our problem is same, still dont have any solution until now, please tell me if you found any solution for this problem, thank you

Is there any solution for this ?

As Jason mentioned, this is a device specific issue where the RNG cannot be initialized which causes failures in BoringSSL (needed during the resolution of a resource via HTTPS). What is the STB model you are using? Can we acquire one? Without being able to debug this locally, we won't be able to make progress on investigating why the __NR_getrandom syscall is failing.

@chinmaygarde this is my STB Model : ZTE ZXV10 B860H 4K
this is indonesia version model, I think it made from china
thank you

I'm still waiting for anyone can solve this

any update?

@indracip any update?
I am also facing the same issue

One simple way to solve this crash is that don't use HTTPS request in Flutter,  instead of the android native by any hybrid bridge.

Could you explain for more detail please ? Any example maybe. :)
Thank you
On Mon, Dec 28, 2020, 6:17 PM julis<EMAIL_ADDRESS>wrote:

One simple way to solve this crash is that don't use HTTPS request in
Flutter, instead of the android native by any hybrid bridge.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/54250#issuecomment-751678614,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKIPW2ET2DNOYFJ336XLKQDSXBSNTANCNFSM4MDTANIA
.

@indracip
Are you still facing this issue using latest Flutter version ?
Since we don't have Android STB device, we won't be able to replicate the issue. Have you looked at this comment / possible solution to see if it works for you ?

Could you explain for more detail please ? Any example maybe. :) Thank you
…
On Mon, Dec 28, 2020, 6:17 PM julis @.***> wrote: One simple way to solve this crash is that don't use HTTPS request in Flutter, instead of the android native by any hybrid bridge. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#54250 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKIPW2ET2DNOYFJ336XLKQDSXBSNTANCNFSM4MDTANIA .

ok, this is my solution, this method put it in diomanager.
Future<void> configSSLCrash() async {     if (Platform.isAndroid) {       final BridgeResult result = await bridge.invoke('/native/SSLCrash'); //will call native method, then get a response by native Net framework(eg. OKHttp, Volley....) request.       final bool isCrashDevice = result.data as bool;       if (isCrashDevice) {         HttpOverrides.global = ProxiedHttpOverrides('<IP_ADDRESS>', '99999');         dio.interceptors.add(InterceptorsWrapper(onRequest: (RequestOptions options) async {           String data = '';           if (options.data is Map) {             data = json.encode(options.data);           } else {             data = options.data as String;           }           final HiBridgeResult result = await hibridge.invoke('/native/request',               <String, String>{'url': options.uri.toString(), 'method': options.method, 'data': base64.encode(utf8.encode(data))});           final Map<String, dynamic> response = json.decode(result.data);           return dio.resolve(response);         }));       }     }   }

where exactly should I put this method? I tried to put this method in my
API request class but an error comes up.
On Wed, Feb 3, 2021 at 9:20 PM julis<EMAIL_ADDRESS>wrote:

Could you explain for more detail please ? Any example maybe. :) Thank you
… <#m_-2853216261289437906_>
On Mon, Dec 28, 2020, 6:17 PM julis @.***> wrote: One simple way to
solve this crash is that don't use HTTPS request in Flutter, instead of the
android native by any hybrid bridge. — You are receiving this because you
commented. Reply to this email directly, view it on GitHub <#54250
(comment)
https://github.com/flutter/flutter/issues/54250#issuecomment-751678614>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKIPW2ET2DNOYFJ336XLKQDSXBSNTANCNFSM4MDTANIA
.
ok, this is my solution, this method put it in diomanager.
Future configSSLCrash() async { if (Platform.isAndroid) { final
BridgeResult result = await bridge.invoke('/native/SSLCrash'); //will call
native method, then get a response by native Net framework(eg. OKHttp,
Volley....) request. final bool isCrashDevice = result.data as bool; if
(isCrashDevice) { HttpOverrides.global = ProxiedHttpOverrides('<IP_ADDRESS>',
'99999'); dio.interceptors.add(InterceptorsWrapper(onRequest:
(RequestOptions options) async { String data = ''; if (options.data is Map)
{ data = json.encode(options.data); } else { data = options.data as String;
} final HiBridgeResult result = await hibridge.invoke('/native/request',
<String, String>{'url': options.uri.toString(), 'method': options.method,
'data': base64.encode(utf8.encode(data))}); final Map<String, dynamic>
response = json.decode(result.data); return dio.resolve(response); })); } }
}
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/54250#issuecomment-772544920,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKIPW2EBDI62FBTNHDBMLCLS5FLSVANCNFSM4MDTANIA
.

@didiiswadi  That is just an example, which tells you that don't use HTTPS request in Flutter, instead of the android native by any hybrid bridge to request HTTPS. for detail, you'd write by yourself because we don't use the same hybrid bridge framework which can let Flutter communicate with Android native.

Ahh, I see then. thank you for your advice :)
On Thu, Feb 4, 2021 at 9:24 AM julis<EMAIL_ADDRESS>wrote:

@didiiswadi https://github.com/didiiswadi That is just an example,
which tells you that don't use HTTPS request in Flutter, instead of the
android native by any hybrid bridge to request HTTPS. for detail, you'd
write by yourself because we don't use the same hybrid bridge framework
which can let Flutter communicate with Android native.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/54250#issuecomment-772975913,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKIPW2AQEY45XUCVTZZFWMTS5IAO5ANCNFSM4MDTANIA
.

@indracip
Are you still facing this issue using latest Flutter version ?
Since we don't have Android STB device, we won't be able to replicate the issue. Have you looked at this comment / possible solution to see if it works for you ?

Yes, currently still have this issue in latest version. Without https, apps not crash, but for safety reason, my apps must need https.
So let it to run without https is not right for me

We've seen quite a lot of _kDartVmSnapshotInstructions related errors in Sentry for multiple customers. Not sure if it's still this exact problem, but it still happens on a daily basis with Flutter 2.2.3.

Yes, currently still have this issue in latest version. Without https, apps not crash, but for safety reason, my apps must need https.
So let it to run without https is not right for me

@indracip Revisiting this issue, since we don't have the said device (Android STB) to verify it correctly whether the crash still occurs or not, is there a way for you to check and confirm if the reported crash still occurs using latest stable version (3.19.5) ?

Closing this based on no activity from OP and others who had this issue and also considering age of the issue at the time it was filed. If anybody thinks this is stil an issue using latest stable version, please file new issue using template so that we can properly address it.


templates with leading YAML separator cause "object not found" errors on deletion

STR

on helm v2.3.0
install a release that has a template file starting with ---, e.g.

---
kind: ConfigMap
apiVersion: v1
metadata:
  labels:
    app: {{ template "fullname" . }}-tcp
    chart: "{{ .Chart.Name }}-{{ .Chart.Version }}"
    component: {{ default "gogs" .Values.service.nameOverride }}
    heritage: "{{ .Release.Service }}"
    release: "{{ .Release.Name }}"
  name: tcp-{{ template "gogs.fullname" . }}-ssh
data:
  2222: default/{{ template "fullname" . }}:ssh

delete the release

Expected

a happy helm deletion occurs

Actual

the release is deleted and helm returns an error:

$ helm delete wiggly-ibis
Error: deletion completed with 2 error(s): object not found, skipping delete; object not found, skipping delete

YAML separators in templates appear to be fine as long as they're not on the first line.

@seanknox Do those objects still exist after it is done deleting?

Everything looks to be deleted, so it's just confusing UX. This breaks CI at https://github.com/kubernetes/charts, however.

@seanknox We had run into these before and @adamreese fixed it. Could it be related @adamreese?

It is probably because the manifest templates are split on ---\n now rather than \n---\n.  We need a check to see if the manifest is blank before raising an error.

In #2190, I suggested using a regular expression that looked for either three hyphens at the beginning of the stream, or three hyphens following a newline. Does that not help here?

The problem isn't the parsing of ---, it's that the manifest is empty. On all other related commands, empty manifests are ignored. For some reason, they are not here.

@seh that said, I'm not opposed to using a regexp instead of the string test.

Understood. I just looked @adamreese's comment above and thought it sounded like a problem we had already eradicated. It turns out that was elsewhere in the codebase.

Thanks @technosophos!


Fix DirectNetworkGuru canHandle checks for lowercase isolation methods

Description
Fixes: #2989
In upgrade to 4.11 RC4 from 4.9.3 it was found that Shared networks could not be created after the upgrade, failing with:

The DirectNetworkGuru refused to design the network due to the isolation method from the physical network was "vlan" and the guru expected "VLAN"
2018-11-06 16:21:37,396 TRACE [c.c.n.g.DirectNetworkGuru] (qtp1096283470-28:ctx-bdc0f3b5 ctx-dacb5ddd) (logid:f3094acc) We only take care of Guest networks of type Shared

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

How Has This Been Tested?

Hi @somejfn can you please test this fix and retry your tests?
@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@blueorangutan test

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3146)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 28327 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3010-t3146-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_public_ip_range.py
Intermittent failure detected: /marvin/tests/smoke/test_templates.py
Intermittent failure detected: /marvin/tests/smoke/test_usage.py
Intermittent failure detected: /marvin/tests/smoke/test_volumes.py
Intermittent failure detected: /marvin/tests/smoke/test_host_maintenance.py
Smoke tests completed. 63 look OK, 4 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_04_extract_template
Failure
128.31
test_templates.py

ContextSuite context=TestISOUsage>:setup
Error
0.00
test_usage.py

test_06_download_detached_volume
Failure
137.52
test_volumes.py

test_02_cancel_host_maintenace_with_migration_jobs
Error
1510.50
test_host_maintenance.py

@somejfn can you please verify this on your setup?

LGTM
Tested with cloud.physical_network_isolation_methods.isolation_method set to 'VLAN', 'vlan' and 'Vlan'.
Shared network created without issue.

Fix is confirmed.  I can create a new network now based on the shared network offering


Metadata declared in jMolecules' @Module not considered

log output
> Logical name: model
> Base package: com.xenoterracide.security.model
> Spring beans:
  o ….controller.RegistrationCtrlr
  o ….user.UserRepository
	at com.xenoterracide.test/com.xenoterracide.test.ApplicationTest.lambda$modulith$0(ApplicationTest.java:26)

my package definition
/**
 * Contains Domain Model classes for security.
 */
@DomainLayer
@BoundedContext(name = "Security")
@Module(name = "Security", id = "security")
package com.xenoterracide.security.model;

import org.jmolecules.architecture.layered.DomainLayer;
import org.jmolecules.ddd.annotation.BoundedContext;
import org.jmolecules.ddd.annotation.Module;

obviously the logical name model isn't going to work. I would think given this output/definition that the logical name would have become security using the id on @Module
If the name/id in @Module isn't supposed to work, is there a way that does? I don't see it in https://docs.spring.io/spring-modulith/reference/fundamentals.html

I've used this ticket to apply an a bit more extensive refactoring to remove a couple of misalignments in semantics. So far, Spring Modulith has always used “name” to refer to an application module's identifier and “display name” as the human-readable variant of that, whereas jMolecules has used “id” and “name”. That's an obvious clash and source of confusion.
In GH-872, I've introduced ApplicationModuleIdentifier to resolve that problem. It also allows us to constrain the tokens to be used as identifiers, as we, for example, use :: as a delimiter between a module name and a named interface in the explicitly declared dependencies in @ApplicationModule(allowedDependencies = …).
On top of that introduction, I've revisited the application module discovery algorithm to properly pick up @Module declarations and the id attribute value of those. I also introduced @ApplicationModule(id = …) as a Spring Modulith-native counterpart of the jMolecules annotation.


Carthage using builds unnecessary dependencies

Nimble and Quick dependencies are builded too, because they have shared scheme in project. But I want to use SWXMLHash and Nimble or Quick are worthless for me. And it takes some time when more frameworks is builded.

I'm not using Carthage at the moment, so forgive my ignorance with this... is the main thing that is necessary to fix this to create two schemes? One for strictly SWXMLHash and one for the tests? Does the fact that I'm using Quick/Nimble as submodules affect this?

Sorry, this should not be a critic. I'm grateful for this framework.
Carthage builds only shared schemes for all targets in project and these targets must be of type "dynamic framework".
So you need to remove Quick and Nimble targets from all shared schemes. Or you can use Cartfile.private, but I don't have any experience with this.

No worries, I just wanted to understand the issue - thanks for helping out! There was another Carthage related issue (see #23) with some comments related to this and I think that's what they were talking about.

Yes, it's same thing.

I can maybe take a look at this tomorrow night. I think since you're not using carthage in the first place that a private file won't really change anything. You should be able to maybe unshare the testing target or something along those lines.

Decided to take a look into this now. But it seems that carthage probably does a recursive checkout, which means that it pulls down your dependencies. Once they exist and since they are available in the workspace they are built as well. I'm not sure of a way to get around this atm.

I did a little more research on this (again, having not used Carthage just yet) and it looks I definitely should look into migrating my current usage of Quick/Nimble away from git submodules. My other options for Quick/Nimble seem to be either using CocoaPods (with a test target specified) or using Carthage (with a Cartfile.private).
I'm open to suggestions between the two. (or alternatives!) Any thoughts?

It depends on how you like your dependency management I suppose. Cocoapods instantly turned me off because it modifies your project file. But if you're just going to use it for your dependencies it might not be a bad idea because people using carthage won't see it. But if you like the way carthage works, then a private file would be good as well. I think it's up to you and how you'd like to have your dependencies handled.

If you don't want to modify your Cartfiles, you can just disable tests being built with your main target (from Xcode's scheme editor). Since Carthage doesn't build for testing, and would then skip your tests' dependencies, no other changes should be necessary.

I just committed https://github.com/drmohundro/SWXMLHash/commit/0cb500a05f500d9f70eba6d92403cc704935b66b to move away from git submodule support over to Carthage for Quick and Nimble. They're only specified in the Cartfile.private file, too.
I believe this should resolve the issue, but I'll keep this open for a few days in case there are still some problems.
There were some references earlier to a shared scheme as well - I didn't change anything about that, so there still may be some issues. I appreciate everyone's help in getting this knocked out!

I tried version 1.0.1 and everything is fine now. Thank you for solving!

:+1: great!


added layout_component_names property

Why it was working before?

@Samuelopez-ansys it's randomly failing. I tried to isolate them and make more robust and independent from other tests


Migrate crypto key from config file to database

Questions:

Unit Test corrected fine? (createContext needs to be run before test so the crypto key gets generated for new tenants)
All dependencies of Cypher.ts covered?
To check about key generation length: 16 or 32?
operation mode: CTR changed to GCM?

(Please ignore previous PR #2030, I went back few commits and created new branch)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Questions:

operation mode: CTR changed to GCM?

Not yet. That's not the goal of that migration.

Questions:

operation mode: CTR changed to GCM?

Not yet. That's not the goal of that migration.

private static async getCryptoSetting(tenantID: string): Promise {
const cryptoSettings = (await SettingStorage.getCryptoSettings(tenantID)).crypto;
if (!cryptoSettings) {

 throw new BackendError({

   source: Constants.CENTRAL_SERVER,

   module: MODULE_NAME,

   method: 'getCryptoSetting',

   message: `Tenant with ID: ${tenantID} does not have crypto settings.`

Yep but a call in the DB just to get the name...
Type to pass the tenant object instead the ID.

Since the information we have before calling Cypher class methods is the tenantID, requiring a Tenant instance will just require similar calls to the DB that could have been avoided before ciphering operations.
The code path here is an error path that should never happen. And if it happens, I think having the tenant name along with the tenantID in the message will ease the debug.  But we can stay with the tenantID alone, that’s fine.
--
Jérôme Benoit aka fraggle
Piment Noir - https://piment-noir.org
OpenPGP Key ID : 27B535D3
Key fingerprint : B799 BBF6 8EC8 911B B8D7 CDBC C3B1 92C6 27B5 35D3

private static async getCryptoSetting(tenantID: string): Promise {
const cryptoSettings = (await SettingStorage.getCryptoSettings(tenantID)).crypto;
if (!cryptoSettings) {

 throw new BackendError({

   source: Constants.CENTRAL_SERVER,

   module: MODULE_NAME,

   method: 'getCryptoSetting',

   message: `Tenant with ID: ${tenantID} does not have crypto settings.`

Yep but a call in the DB just to get the name...
Type to pass the tenant object instead the ID.

Since the information we have before calling Cypher class methods is the tenantID, requiring a Tenant instance will just require similar calls to the DB that could have been avoided before ciphering operations.
The code path here is an error path that should never happen. And if it happens, I think having the tenant name along with the tenantID in the message will ease the debug.  But we can stay with the tenantID alone, that’s fine.
--
Jérôme Benoit aka fraggle
Piment Noir - https://piment-noir.org
OpenPGP Key ID : 27B535D3
Key fingerprint : B799 BBF6 8EC8 911B B8D7 CDBC C3B1 92C6 27B5 35D3


Question: is circle ci need extra config (forked project build fail) ?

I forked this project and try to build this with circle ci, but it failed.
https://circleci.com/gh/gengjiawen/webpack/1

Not really, no.
But it seems the current setup has reached its limit. If you read the error, it says "out of memory". I have that error as well with a larger PR: #1178
I'm currently checking out how to move the tests to circleci v2, which is much more fficient and faster as far as I have experienced. Until then, you are pretty much on your own, sorry.

I first run into this problem with https://github.com/gengjiawen/ts-scaffold. It use the same config as this project. I think the main problem is not lies in memory, but with this in test.sh
yes "" | ./node_modules/.bin/vue init . test

I ssh into the circle ci machine (ubuntu 14.04), execute the command will stuck.
If you execute ./node_modules/.bin/vue init . test in bash, and press enter manually, everything works fine.
I do some search,  change to
echo -ne '\n' | ./node_modules/.bin/vue init . test

But it won't generate the test folder.

I'm closing this since the normal banches do build fine for us, and there's no specific setting you have to use. Nothing we an help you with.


Question: Why is _queries in QueryCollection private?

I would like to execute all queries of my QueryCollection like in the following example:
create_queries = QueryCollection.from_file("src/queries/create.sql")
for _, query in create_queries._queries.items():
    db.execute(query)

Does _queries has to be private? If not, one could access it directly (although it is obviously also now possible to do this, but not recommend because it is private :wink: ) and looping over it.
Alternatively, one could add the following getter function:
def get_queries(self):
    return self._queries

Good point. I'd suggest to make QueryCollection inherit from collections.abc.Mapping that you can do something like:
queries = QueryCollection.from_file("queries.sql")
for key, query in queries.items():
    db.execute(query)


Remove erlsha2 dep

GG WP

@sata I believe base16 needs to be added to applications of current.app.src.

@sata Can you squash the commits together a bit?

@horvand yeah,  I  squashed away 2 commits. I don't mind squashing more tbh.


Drag and drop (Inverse kinematics) issues

Hello :),
I have some issues performing the Drag and drop (aka inverse kinematics) tutorial.

When I try to import the URDF of the mobile ur5 using the command

robot = MobileRobotWrapper(URDF, [PKG])
I get the following error :

AttributeError: 'MobileRobotWrapper' object has no attribute 'createData'

I tried to use the command MobileRobotWrapper.BuildFromURDF(URDF), it works fine, but it imports the 6 DOF ur5 (robot.nq = 6), yet the mobile version of ur5 should be 9 DOF (robot.nq should be equal to 9), so I guess it imported it wrong.

In the same tutorial there are these two commands :

IDX_TOOL = 24
IDX_BASIS = 23

How can indexes be equal to 24 and 23 as the robot has only 9 DOF ?

In the same tutorial, there is this command se3.framesKinematics(robot.model, robot.data), but when I tip it, I have the following error :

AttributeError: 'module' object has no attribute 'framesKinematics'

Thank you very much for your feedback :)

Hello :),
I have some issues performing the Drag and drop (aka inverse kinematics) tutorial.
1. When I try to import the URDF of the mobile ur5 using the command

robot = MobileRobotWrapper(URDF, [PKG])
I get the following error :

AttributeError: 'MobileRobotWrapper' object has no attribute 'createData'

I tried to use the command MobileRobotWrapper.BuildFromURDF(URDF), it works fine

Indeed, the tutorials are very out of date. BuildFromURDF is the current method.
but it imports the 6 DOF ur5 (robot.nq = 6), yet the mobile version of ur5 should be 9 DOF (robot.nq should be equal to 9), so I guess it imported it wrong.
That's because if the base is not fixed the root joint must be provided externally (this info is not contained in the URDF, thus there is no way for Pinocchio to retrieve it. In your case, you need a JointModelPlanar:
`MobileRobotWrapper.BuildFromURDF(URDF,  pkgs, pin.JointModelPlanar())`

The custom constructor of MobileRobotWrapper was taking care of this for you, but you could not use it because MobileRobotWrapper employs an old interface.
Notice that the constructor of MobileRobotWrapper was also doing other stuff, such as adding custom frames, which might be expected in the follow-up of the tutorials.
Therefore, you can either take a look at the constructor of MobileRobotWrapper and repeat its same steps, or fix MobileRobotWrapper itself. In the latter case, the best thing would be to rework the class so that its interface complies more to that of RobotWrapper, but I think a quick fix for you would be simply to replace this line
https://github.com/stack-of-tasks/pinocchio/blob/7765909f1aed41dd9471baaf8e492be5fcc99523/doc/d-practical-exercises/src/mobilerobot.py#L20
with
self.initFromURDF(urdf, pkgs, pin.JointModelPlanar())

1. In the same tutorial there are these two commands :

IDX_TOOL = 24
IDX_BASIS = 23

How can indexes be equal to 24 and 23 as the robot has only 9 DOF ?

That's because those are not indexes of links, but of frames. One link can have an arbitrary number of frames attached to it (including zero). You can take a look in the doc

1. In the same tutorial, there is this command `se3.framesKinematics(robot.model, robot.data)`, but when I tip it, I have the following error :

AttributeError: 'module' object has no attribute 'framesKinematics'

Thank you very much for your feedback :)

Again, the tutorials are out of date. The command would be updateFramePlacements(robot.model, robot.data), or framesForwardKinematics(robot.model, robot.data, q) if you contextually provide q

@gabrielebndn I think you have a correct understanding of what is missing now in the tutorial to be up to date. It would be great if you can correct them in such a way nobody will be annoyed. Can you take it in charge?

I will see what I can do, but I am full of work, so it will be very low priority

Mostly, I think we should find a way to unit test them

Yes, sure for the CI.

From my side, I'm also full and I think you might be much more efficient than me on it. @nmansard @nim65s Do you think that you have some times to fix it?

@nmansard created new tutorials. He is waiting for me to review them. I am planning to do that in a week or 2.
This will come with proper CI integration.

Excellent then!

@gabrielebndn Thank you very much for your feedback, it was really helpful :)
I'm glad to hear that tutorials will be updated. Thank you for your helpful work


added fancy badges

How could we live without those fancy badges! ;)

Unfortunately I don't think the Travis CI will build correctly until vNext of ASP.NET, unless a separate solution is used that only builds + runs the units tests - but I'll add the badge on in the hope it doesn't give the wrong impression.

Well I saw the failing Travis builds even without the badge.
It gives the impression that the repos is still work in progress.
IMO fix or ditch it (Travis CI)


fix: [CHAOS-2883]: Added release notes for 1.23.x Chaos Module

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Screenshot for Reference -

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Attributes entity

This is an experiment for using Nanoc::Int::Attributes instead of Hash.

This isn’t working out as well as I had expected. Mustache integration is making it difficult, too.

Will revisit this some other time.


Make tests resilient to Localization

Fixes #4260

Closing this PR as @rynowak has let me know that there's a better approach.


Add arm64 image build and switch build to use Docker BuildX

Checklist

[x] Implementation
[ ] Tests
[ ] Docs

Implementation:
Switched build system to use Docker BuildX, allowing targeting of multi-arch docker images. Currently only building arm64 and amd64 arch images, but it might be good to add other ARM architectures if anyone wants to run containerised on an older Raspberry Pi.
Tests:
Build process tested against my own Docker repo. I've assumed that the repo secrets DOCKER_PASSWORD and DOCKER_USERNAME are the login details for the inspircd/inspircd-docker Docker Hub repo, and I've made sure that the build works pushing to my Docker Hub. I've verified that the INSP_VERSION env var is respected and added to the BuildX args, so it definitely pulls the correct tagged version of the inspircd source. Can't really test this bit as it's the GitHub actions build script...
Docs:
Don't think any explicit docs are needed. My testing on my own multi-arch k8s cluster shows that the images can be happily used on whatever host, and image pulls will select the correct architecture for the host automatically.
Caveats:
Build time of the GitHub build action is quite long, increasing from ~10mins to ~50mins. I've run a test build on my Raspbery Pi 4, and the build time on that system is not far off that upper ballpark. I've had a look at the caching of the docker/build-publish step, and I don't think it's likely to help much seeing as this image only gets built once in a while.
Merging:
If this PR is merged, then the build action will kick off, and it will REPLACE all currently tagged images in the docker hub repo - so the current amd64 images tagged 3/3.10/3.10.0/latest will be replaced. Not sure it'll be a problem, but just something to be aware of.

So the PR build failed - it's failing to log in to docker hub. The build script uses the following:
        name: Login to DockerHub
        uses: docker/login-action@v1
        with:
          username: ${{ secrets.DOCKER_USERNAME }}
          password: ${{ secrets.DOCKER_PASSWORD }}

Which worked in my fork, because I had two secrets called DOCKER_USERNAME and DOCKER_PASSWORD. I've removed setting the secrets in the env block, as some build actions echo out the entire env block to their log when they run, which is obviously bad. But I can't see if I've got the right secrets. I thought I had but... yeah. I'm a bit baffled as to why this is failing. I could disable the docker login step if it's not a master build, I suppose?

Disabling the docker hub login step unless it's a master build seems to make sense to me - there's no need to log into a service that we're not going to use.
I'm at the end of my rope now - since I have no visibility of this repo's secrets, I can't make sure that DOCKER_USERNAME and DOCKER_PASSWORD will work when used by the docker/login-action@v1 step.

Some tests fail when run locally. After doing docker swarm init and docker build -t inspircd:testing ., the following four tests fail:
customConfigSecrets.sh:, secrets.sh
image inspircd:testing could not be accessed on a registry to record its digest. Each node will access inspircd:testing independently, possibly leading to different nodes running different versions of the image.

default.sh:, mountConfig.sh
SSL handshake has read 2602 bytes and written 709 bytes
Verification error: unable to verify the first certificate

All other tests pass.
The certificate issues, I don't really get.
The other issue -  "image could not be accessed on a registry" - could have a work around. Had a look at this: https://codeblog.dotsandbrackets.com/private-registry-swarm/ and the suggestion is to create a service that runs a registry and force docker to use and trust that registry, then build and publish the image to that registry in the test context, and then run the tests. Unless I've misunderstood this whole process.
Other observations:

The Docker BuildX step doesn't leave the built image lying around
we want to build and test images before merging.
in order for the tests to pass, it looks like docker needs to run an image available in an image repository (??? might have misunderstood this!)
we don't want to leave test images lying around in a public repo, so we don't want to push them to Docker Hub
we could use GitHub Container Registry for test images, setting up a separate build step just for that repo, and delete the image from the repo after testing is finished - see https://github.com/docker/build-push-action/blob/master/docs/advanced/isolated-builders.md and https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry )
this would only work for builds against the main repo, not for forked repos (the auth secrets necessary to login wouldn't be present on forks)

the workflow i'm proposing is:
if building a PR:

modify run_tests.sh to allow overriding of the test image name. locally, it can still be (and defaults to) inspircd:testing
Option A:
build amd64 image only
push it to GHCR as inspircd:testing-$SOME_BUILD_IDENTIFIER
execute run-tests.sh with that image - e.g. export $TEST_IMAGE=gchr.io/inspircd/inspircd:testing-$SOME_BUILD_IDENTIFIER
once test passes/fails, delete the image (<- not sure how to do this!)
Option B:
run a registry service
force docker to use and trust that registry
build the docker image tagged as inspircd:testing and then push to transient registry service

if building a merge to master
don't run the tests, because managing the test images is a pain. If only PRs where the tests pass are merged, then the tests will have been run in the GitHub action when the PR was being built.
I could be a bit wrong though.


view-only mode

Some users want to share their scope with others but don't allow them to modify anything.

I am guessing this is more of a Weave Cloud feature than scope app feature.

OSS doesn't really have the concept of users, so yes I think this is a weave cloud thing.
Would be straight forward to disable controls and pipes for certain users.

I think we should also provide a read-only mode for OSS, through a start flag. I will get this done.


Requirements txt need to be updated

pandas
selenium
bs4
webdriver_manager
spacy
sklearn
lxml
scikit-learn

Thanks @Emmanuel-Samuel!
Updated.

Nice, thumps up. Good script


Jest preprocessor does not correctly hoist jest.mock calls

Description
Using the Jest preprocessor provided by React Native (from jest/preprocessor.js) does not correctly hoist calls to jest.mock(). Using this preprocessor is currently the recommended way to use Jest with React Native: https://github.com/facebook/jest/blob/3d09c5c0c0f336af2e30f5e2e0252913a5cb36dd/examples/react-native/jest.config.js
This is deliberately duplicating #20614 (and possibly #19859, although it's less clear if that is related). They were closed with suggested workarounds, but this is a bug with code provided by React Native and should still be fixed here.
React Native version:
System:
    OS: macOS Mojave 10.14.6
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 3.05 GB / 32.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 10.18.1 - /usr/local/opt/node@10/bin/node
    Yarn: 1.21.1 - /usr/local/bin/yarn
    npm: 6.13.4 - /usr/local/opt/node@10/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 12.4, macOS 10.14, tvOS 12.4, watchOS 5.3
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>91312
    Xcode: 10.3/10G8 - /usr/bin/xcodebuild
  npmPackages:
    react-native: 0.61.5 => 0.61.5
  npmGlobalPackages:
    react-native-cli: 2.0.1

Steps To Reproduce
I've created a self-contained minimal reproduction at https://github.com/nwalters512/react-native-jest-hoisting-bug. After cloning the repository, follow these steps to reproduce both the bug and the expected behavior.

Run yarn to install dependencies.
Run yarn test. This will execute the tests defined in src/index.test.js with React Native's Jest preprocessor.
Observe how the test fails with the exception thrown from src/example.js. This is unexpected, since the implementation of that function should be mocked.
Delete jest.config.js to turn off the react-native preset and use the standard babel-jest transformer.
Run yarn test again.
Observe how the test now passes. This is because the call to jest.mock is correctly hoisted to the top of the file, so the function from src/example.js is now properly mocked.

Proposed solution
Adding babel-plugin-jest-hoist to the list of Babel plugins in jest/preprocessor.js should fix this problem. I'm happy to open a PR with this change if maintainers think this is the right solution.

Same issue. Any solution?


Added GridGap functionality to grid component

Feature/Merge request for Grid Gap functionality described in the TODO features of the project.
I notice their has been a workaround listed in https://github.com/katoid/angular-grid-layout/discussions/43, but wanted to make a formal issue to potentially support this feature.
Change list includes the addition of a gridGap input to the grid component, logic for calculating the grid items top, left, width, and height properties accounting for this new gridGap property, and similar logic for calculating the height of the grid. I also added an input to the playground demo so this can be seen visually.
Feel free to build off this and discuss any potential issues/changes that could be beneficial here

I nearly forgot, before merging it to 'main' branch, you would need to 'squash' all the commits you have made into a single one. To keep the history clean

Thanks for the changes @Devin-Harris . I added some other comments and a small bug I found, when resolved it would be ready to merge!

Hey @Devin-Harris, I have pushed a commit with the fix for the drag and resize mouse coordinates mapping to grid coordinates, with gap support. I think the overall solution can be really simpler than it is now, but I would have to think a little bit longer to get it😅. For now, this fixes seems solid to me. Please, if you can test it and provide feedback would be nice.
After validating these changes are okay, would be nice to squash all those commits into a single one in order to be merged.  Splitting the w, h and x, y calculations makes those methods much clearer to me and seems to resolve those resizing issues I was running into. I have also squashed the commits now 😄

Merged! 👏

@Devin-Harris I see that you don't show up as a contributor in the repo, this may be due that you use different accounts for github and for your commits. If you want to appear as a contributor you would need to fix that. If you like I can undo the merge and redo afterwards with your correct account. Just tell me if you want it.

@Devin-Harris I see that you don't show up as a contributor in the repo, this may be because the email address you used to author the commits isn't connected to your account on GitHub. If you want to appear as a contributor you would need to fix that. If you like I can undo the merge and redo afterwards with your correct account. Just tell me if you want it.

Ah whoops. Its no big deal though. I'm happy just knowing I will be able to use this feature in the future


jabut: Add version 2020.07.31542

Closes #5189

amFCdVRfMjAyMC4wNy4zMTU0Ml9Qb3J0YWJsZV94NjQuemlw is actually Base64 for jaBuT_2020.07.31542_Portable_x64.zip, and this site use % to escape + and /. Is there a better way to update the URI?

I guess it's because I had ran autoupdate before I tested the URI then. Close for now.


requestBody is string "[object Object]"

Given the following fetch:
const formData = new FormData();
formData.append('test', 'true');

fetch('http://localhost/api/test', {method: 'POST', body: formData});

When I mock it with nock:
nock('http://localhost')
  .post('/api/test')
  .reply(function(uri, requestBody) {
    console.log(typeof requestBody, requestBody); // string [object Object]
  });

the requestBody is "[object Object]". Is that normal? How can I inspect FormData posted with nock?

I tried the above with node v6.7.0 and nock v8.0.0 and what I get is:
string ----------------------------640160139954673545871604
Content-Disposition: form-data; name="test"

true
----------------------------640160139954673545871604--

Which versions are you using?

nock 8.0.0 and node 6.3.1

Hmm, I get the same results from your test. I'm using redux-api-middleware in my app, but that doesn't appear to do anything to the body before passing it through to isomorphic-fetch. In any case, it looks like the issue is not in nock. Thanks!

Good to know. Please let me know if you notice anything else fishy.

@evandavis did you find out what the issue was? I've got the same setup with redux-api-middleware, isomorphic-fetch, and nock and am getting the "[object Object]" as my request body within my tests.

@sirdavidwong not exactly; we ended up writing a custom stringifier based on the npm version of form-data for our tests. Not ideal. (I believe the issue is in how FormData converts to string outside the browser.)

Gotcha, ended up doing the same thing :) Thanks @evandavis !

@sirdavidwong did you find out what the issue was?
Thanks!


Use Vite v4.3.9

Bumps Ladle's Vite version to the latest v4.3.9 to fix the recent advisory posted here.

Here is the Vite changelog for this version.  Also please let me know if minor is not correct for this change

Can you change the range to "4.3.9"?


PluginException: ActionUpdateThread.OLD_EDT is deprecated

Apparently the Dash plugin needs an update for newer JetBrains IDEs: "'de.dreamlab.dash.SmartSearchAction' must override `getActionUpdateThread` and chose EDT or BGT. See ActionUpdateThread javadoc."
I've started getting the following exception in IDE Internal Errors, repeatedly:

Exception in plugin Dash (3.3). … Please report the problem to Gerard Delmàs.
stacktrace.txt:

com.intellij.diagnostic.PluginException: `ActionUpdateThread.OLD_EDT` is deprecated and going to be removed soon. 'de.dreamlab.dash.SmartSearchAction' must override `getActionUpdateThread` and chose EDT or BGT. See ActionUpdateThread javadoc. [Plugin: com.paperetto.dash]
	at com.intellij.diagnostic.PluginProblemReporterImpl.createPluginExceptionByClass(PluginProblemReporterImpl.java:23)
	at com.intellij.diagnostic.PluginException.createByClass(PluginException.java:90)
	at com.intellij.diagnostic.PluginException.reportDeprecatedUsage(PluginException.java:125)
	at com.intellij.openapi.actionSystem.ActionUpdateThreadAware.getActionUpdateThread(ActionUpdateThreadAware.java:21)
	at com.intellij.openapi.actionSystem.AnAction.getActionUpdateThread(AnAction.java:199)

(Might be related to #87.)

PyCharm 2024.1
Dash plugin 3.3

Same with the PhpStorm
PhpStorm 2024.1
Build #PS-241.14494.237, built on March 27, 2024
Runtime version: 17.0.10+8-b1207.12 aarch64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 14.4.1
...
Non-Bundled Plugins:
  com.paperetto.dash (3.3)
  ...

Same with the PhpStorm
PhpStorm 2024.1
Build #PS-241.14494.237, built on March 27, 2024
Runtime version: 17.0.10+8-b1207.12 aarch64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 14.4.1
...
Non-Bundled Plugins:
  com.paperetto.dash (3.3)
...

This error will likely show up in the 2024.1 version of every JetBrains IDE. They share a common plugin API.
It looks like the deprecation was first added in the 2022.3 quarterly releases (so, a little less than two years ago). I don't know whether "going to be removed soon" means 2024.2 or 2025.1, but when that occurs the plugin will probably be completely broken.
I think the fix may be as simple as adding getActionUpdateThread to the plugin's shared AbstractMenuAction class:
Index: src/de/dreamlab/dash/AbstractMenuAction.java
===================================================================
@@ -54,4 +54,9 @@

         presentation.setEnabled(PlatformDataKeys.EDITOR.getData(e.getDataContext()) != null);
     }
+
+    @Override
+    public @NotNull ActionUpdateThread getActionUpdateThread() {
+        return ActionUpdateThread.BGT;
+    }
 }

(But it's a very long time since I've done any Java coding, and I'm not set up to build JetBrains plugins to test it.)

I tried Dash the first time today and could not open the IntelliJ Plugin because of this issue.


feat(torii-wasm): updated with libp2p support

Closes #
Introduced changes

Update torii-wasm to new version with Libp2p client support & incorporate libp2p into dojo.js and example
Checklist

[x] Linked relevant issue
[x] Updated relevant documentation
[x] Added relevant tests
[x] Add a dedicated CI job for new examples
[x] Performed self-review of the code

Not sure why the compiling is failing on the github workflow. Maybe a wrong rust version?


Ball search failed action: end_ball

Implement end_ball as ball_search_failed_action in ball_search. It should just end the ball and not the game as end_game.

Hi @jabdoa2! I was looking through ball_search, were you thinking of another if statement in  _compensate_lost_balls()?

Yes,  that would be the right place to implement this. Thanks for looking at this!

This was a feature request from one of our users: https://groups.google.com/forum/m/#!topic/mpf-users/F60DMIKll_g

While this seems to be a quick fix #992, I found that the unit tests written for ball_search seem a bit complicated. I'm not sure how to go about adding one.
On a separate note, I think it might be a good idea to add additional links to the contributing page http://docs.missionpinball.org/en/latest/about/contributing_to_mpf.html as it doesn't list that this project requires python v 3.4 there.

thanks for implementing this @ashanhol


Support disabled nodes in visual editor

https://www.figma.com/file/04DymSq4xElhOBdAyutvvTbq/Composer-SH-06.19-?node-id=4586%3A0

User can disable/enable nodes (single/multi selected) nodes
copy/paste/move does not change nodes state
condition nodes will be taken as "one block" to be disabled

If nodes have been disabled, corresponding LU intent will not be published as well.

TODOs:

[ ] disabled effect: if an action's disabled field set to true, make this node and its child nodes gray (leveragin the css filter)
[ ] In Shell toolbar, provide 2 buttons - Disable | Enable, make them clickable when selected nodes are detected to be able to toggle the state

If all selected nodes are disabled, show | Disable (unclickable) | Enable (clickable) |
If all selected nodes are NOT disabled, show | Disable (clickable) | Enable (unclickable) |
If selected nodes are half disabled and half NOT disabled, show | Disable (clickable) | Enable (clickable) |
If no selected node, show | Disable (unclickable) | Enable (unclickable) |

LU publish part (@hibrenda who is responsible for the LU publish?)

[ ] If nodes have been disabled, corresponding LU intent will not be published as well

@sangwoohaan do you provide what's the icon names of 'Disable' and 'Enable' button?

LU publish part (@hibrenda who is responsible for the LU publish?)

[ ]  If nodes have been disabled, corresponding LU intent will not be published as well

@lei9444 be aware of the lu content for luis publish.


In https://3mdeb.com/contact/ Google Maps has the address Abrahama 1a

https://3mdeb.com/contact/

@Psotas thank for the report.

The map with office location has been fixed.


Fix Prevent Updating

If you don't use the if conditional before unset it will cause a PHP warning.

PHP Warning: Attempt to modify property of non-object in /public_html/wp-content/plugins/w3-total-cache/inc/define.php on line 1652

PR for https://github.com/szepeviktor/fix-w3tc/issues/115#issuecomment-250959447


Safely modify a constraint tree

At the moment we have mutable constraint trees, which is very handy for manipulating complex structures, e.g. changing bounds etc. The problem is that this is a common pattern:
wt = simplified_enzyme_constrained_flux_balance_constraints(
        model;
        reaction_isozymes,
        gene_product_molar_masses,
        capacity = 400.0, # mg/gDW
        interface = :identifier_prefixes,
)
 wt.fluxes[rxn_id].bound = C.Between(-1000, 1000)

but it won't always work, because you also need to do:
wt.fluxes_forward[rxn_id].bound = C.Between(0, 1000)
wt.fluxes_reverse[rxn_id].bound = C.Between(0, 1000)
wt.interface.exchanges[rxn_id].bound = C.Between(-1000, 1000)

This makes is really easy to get incorrect results results if you don't know all the places the original flux bound is also implicitly present...

OK so what we can do:

disable the mutability of the main structures, so that people don't kill it randomly themselves
instead provide something like wt = C.patch(wt, (:fluxes_forward, rxn_id, :bound, :upper), 1000.0) which people need to google so they certainly won't miss the documentation saying that this is a big nono
have the good way documented (if the model is wrong, change it before you convert it to a CT)

does that sound ok?

we'd need lenses to do this properly. :(

This will definitely causes issues, especially in more complex models with more derived bounds. Basically 2 things:

I like being able to quickly modify the constraint types manually, e.g. wt.fluxes[rxn_id].bound = C.Between(-1000, 1000)
I want changes that affect other variables to either warn me or make it obvious that I am missing something

yeah the issue is that 1 and 2 are not cleanly possible to combine because of how julia works. I was digging into that lately but everything is just kinda worse. Let's keep this open for ideas

ok actually, can you test if either of these would do?

https://github.com/JuliaObjects/Accessors.jl
https://github.com/jw3126/Setfield.jl

if all OK we can just recommend these in documentation and avoid pulling in dependencies.

ok the total price tag is:

we need to depend on ConstructionBase.jl
we add this overload:

ConstructionBase.setproperties(x::C.Tree{T}, props::NamedTuple) where T = C.Tree{T}(x..., pairs(props)...)

Okay, but should we now make the trees immutable again?

And this would solve issue 1 nicely, but issue 2 seems unsolvable without some sort of tracking structure, telling you what the dependency structure of all the constraints is?

Okay, but should we now make the trees immutable again?

Not really, I don't want to break it for people who "do" the safety themselves.
re issue 2 you mean "model variables" or "julia variables" ? (but yeah both are unsolvable :D :D )


Bug bounty test - PLEASE DO NOT APPROVE. epcmfi

Bug bounty test - PLEASE DO NOT APPROVE. epcmfi

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


FindANTLR.cmake: should also look in $CLASSPATH

The cmake/modules/FindANTLR.cmake looks in a few hard-coded well-known locations for a potential antlr*complete.jar.
https://github.com/chipsalliance/Surelog/blob/edb409a1f730d59665c312551aadc0f76af1c82e/cmake/modules/FindANTLR.cmake#L13-L21
However, that is not necessarily where the file is.
It would be good if it was also consulting the CLASSPATH (arguably, it even should look there first before going into guess-mode rummaging through directories).
While at it, looking for exactly version 4.12.0 is very specific, can we change that to 4.12.* to be future compatible in case there are new patchlevels coming out ?

I took a look at where debian and homebrew put the asset, I didn't see anything about setting classpath or configuring the users's profile to have that var. Is it the standard for folks to maintain a classpath var like they might a PATH? Maybe antlr already has something to find where the jar is? But yeah easy to implement, just unfamiliar with Java ecosystem.

In NixOS, the jar file ends up in the CLASSPATH (and to show that it is not predictable: in my case, the setting is CLASSPATH=/nix/store/6cnfr2cy5ninj4xnhnqk8x4vkrz8bydn-antlr-4.12.0/share/java/antlr-4.12.0-complete.jar)
In the Java-world, CLASSPATH common - it is what PATH is for binaries.

Ok makes sense, will add


Remove generics from PGPIO8BitInterface.

I assume you'd want to remove the example again, just wanted to see if it compiles :)

Thanks for the PR and the insights. We've decided we're not going this route due negligible benefits. Closing this now.


clamav-http upgrade v0.3.8

Quay scans:
https://quay.io/repository/ukhomeofficedigital/acp-clamav-http?tab=tags
https://quay.io/repository/ukhomeofficedigital/acp-clamav-mirror?tab=tags
https://quay.io/repository/ukhomeofficedigital/acp-clamav?tab=tags

Does the README need updating to reflect any of the changes that you have made?

I believe these changes are backwards and forwards compatible. I can't think of anything to add to the README.
Quay scans pass:
https://quay.io/repository/ukhomeofficedigital/acp-clamav-http?tab=tags
https://quay.io/repository/ukhomeofficedigital/acp-clamav-mirror?tab=tags
https://quay.io/repository/ukhomeofficedigital/acp-clamav?tab=tags

I believe these changes are backwards and forwards compatible. I can't think of anything to add to the README.
Quay scans pass: https://quay.io/repository/ukhomeofficedigital/acp-clamav-http?tab=tags https://quay.io/repository/ukhomeofficedigital/acp-clamav-mirror?tab=tags https://quay.io/repository/ukhomeofficedigital/acp-clamav?tab=tags

It was more from the perspective of any steps to test / validate behaviour. The README seemed a bit thin on how you confirm changes are OK. But if it's all covered by automation, that's fine.

LGTM


Load Testing

Is your feature request related to a problem? Please describe.
Testing the load
Describe the solution you'd like
Testing the load
Describe alternatives you've considered
No response
Additional context
No response

I have created a notification script which sends the notification to the secondary server
Created a monitor script witch monitor the notification sent to the secondary server
Created a dart script which intern creats multiple monitors.
Tested the load using above scripts on my local

Continuing load testing by using the given 2 scripts
Dart Isolates - Uses dart isolates to run multiple monitors at the same time and writes the data from each isolate into a csv
https://gist.github.com/purnimavenkatasubbu/b6a352efe073402348caf5d7257552db
parallel_notification -  Continously sends given number of notifications to the atsign.
https://gist.github.com/purnimavenkatasubbu/d2c6eab2dfc406cd82e8881cbbe5135c


Add the ability to fail tests when using an Envoy deprecated feature

Through the flag -istio.test.deprecation_failure.
Currently it inspects proxy logs looking for deprecated
messages. This can be enhanced in the future to cover other
deprecations.

Then we could have one or more CI jobs with that flag activated. Not marked as required in the beginning.

@jwendell: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
79f34ca60bd34ce54ebc2003d10e518c24236a09
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

does envoy have any thing like metrics when depreciation things are used, fail instead of warning, etc that we could leverage instead of logs?

Yes. Envoy emits a stat of deprecated feature use https://www.envoyproxy.io/docs/envoy/latest/configuration/operations/runtime.html?highlight=runtime deprecated_feature_use#statistics

Yes. Envoy emits a stat of deprecated feature use https://www.envoyproxy.io/docs/envoy/latest/configuration/operations/runtime.html?highlight=runtime deprecated_feature_use#statistics

I guess this doesn't indicate what features we used, so we still may need the logging, but maybe logging + this stat. Not sure.

I thought we just want to fail the test if there is deprecation usage - Do we need feature info as well?

Well we know we use a ton of deprecated features so I'm not sure failing is
useful since it will always fail?
On Sun, Oct 6, 2019, 9:18 PM Rama Chavali<EMAIL_ADDRESS>wrote:

I thought we just want to fail the test if there is deprecation usage - Do
we need feature info as well?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/17599?email_source=notifications&email_token=AAEYGXOXWQMJKWZMPD2FDE3QNK2CPA5CNFSM4I5HDR32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAO7FLY#issuecomment-538833583,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEYGXMXJ3MR7LKQG6LDZKLQNK2CPANCNFSM4I5HDR3Q
.

Based on PR description,  I thought the intention was to fail when run with a flag. But you point is valid, it will always fail -  Was n' completely sure, what is the intention here though?

I ran a few tests locally with that flag on, and had no failures. So, I'm not sure if we use tons of deprecated features or if they just don't show up on Envoy logs.

it's because of the log level, we set misc:error to hide them?

@jwendell: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
f6fba26e0d945419b79a40abd6a7e6dd056b906a
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jwendell: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
f6fba26e0d945419b79a40abd6a7e6dd056b906a
link
/test lint_istio

integ-mixer-k8s-tests_istio
c7d70ee276875a16ca8cb2c4340fe6fbc25e31b4
link
/test integ-mixer-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jwendell: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
f6fba26e0d945419b79a40abd6a7e6dd056b906a
link
/test lint_istio

integ-mixer-k8s-tests_istio
c7d70ee276875a16ca8cb2c4340fe6fbc25e31b4
link
/test integ-mixer-k8s-tests_istio

gencheck_istio
c7d70ee276875a16ca8cb2c4340fe6fbc25e31b4
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jwendell: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
f6fba26e0d945419b79a40abd6a7e6dd056b906a
link
/test lint_istio

integ-mixer-k8s-tests_istio
c7d70ee276875a16ca8cb2c4340fe6fbc25e31b4
link
/test integ-mixer-k8s-tests_istio

gencheck_istio
c7d70ee276875a16ca8cb2c4340fe6fbc25e31b4
link
/test gencheck_istio

pilot-e2e-envoyv2-v1alpha3_istio
c7d70ee276875a16ca8cb2c4340fe6fbc25e31b4
link
/test pilot-e2e-envoyv2-v1alpha3_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Now I'm changing the Envoy log level when the CLI flag is used.
The intention with this PR is to cover https://github.com/istio/istio/issues/16275.

@jwendell: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-tests_istio
c7d70ee276875a16ca8cb2c4340fe6fbc25e31b4
link
/test integ-mixer-k8s-tests_istio

pilot-e2e-envoyv2-v1alpha3_istio
c7d70ee276875a16ca8cb2c4340fe6fbc25e31b4
link
/test pilot-e2e-envoyv2-v1alpha3_istio

integ-istioctl-k8s-tests_istio
95c1866c7bad2e8a00b68b3d9fe670828458646f
link
/test integ-istioctl-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jwendell: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-istioctl-k8s-tests_istio
95c1866c7bad2e8a00b68b3d9fe670828458646f
link
/test integ-istioctl-k8s-tests_istio

integ-mixer-k8s-tests_istio
95c1866c7bad2e8a00b68b3d9fe670828458646f
link
/test integ-mixer-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jwendell: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-istioctl-k8s-tests_istio
95c1866c7bad2e8a00b68b3d9fe670828458646f
link
/test integ-istioctl-k8s-tests_istio

integ-mixer-k8s-tests_istio
95c1866c7bad2e8a00b68b3d9fe670828458646f
link
/test integ-mixer-k8s-tests_istio

pilot-multicluster-e2e_istio
46eb3567bd2a2338fffe233416bc612337160b01
link
/test pilot-multicluster-e2e_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Unknown Problem

<!---   *** BEFORE SUBMITTING: PASTE CLIPBOARD HERE TO COMPLETE YOUR REPORT ***   ---!>

I opened up spyder and then the issue window popped up. Not sure what happened.

Hey @ellbatman, thanks for reporting. Please post the error you're seeing here.

Closing due to lack of response.


Create Reversing_Linked_list.py

Added reversing Linked list
->2 testcases
->time and space complexity
->successfully compiled
#193 issue
@raksha009 @tarun26091999  @plazzy99

@raksha009 Please review my pull request mam.

Plz link the code with respective folder.

I have linked in code-> python
In which specific folder you are talking about to link mam
Please help

Plz link the code with respective folder.

I have linked in code-> python
In which specific folder you are talking about to link mam
Please help
@raksha009

Plz link the code with respective folder.

I have linked in code-> python
In which specific folder you are talking about to link mam
Please help
@raksha009

Heyy, the topic tag in the issue helps to decide which folder to link our file with. Since, the topic tag is LL, plz link it with LinkedList folder.

Plz link the code with respective folder.

I have linked in code-> python
In which specific folder you are talking about to link mam
Please help
@raksha009

Heyy, the topic tag in the issue helps to decide which folder to link our file with. Since, the topic tag is LL, plz link it with LinkedList folder.

@raksha009 @plazzy99 I have changed file location to Linked list as you said