Undefined symbol: DeleteGpuDelegate(_TfLiteDelegate*)

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version:
Python version:
Installed using virtualenv? pip? conda?:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

Describe the problem
Provide the exact sequence of commands / steps that you executed before running into the problem
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
I had built tensorflowlite.a for ios follow offcial website, build successful!
but not working for gpu, below is error msg with xcode:
Undefined symbol: DeleteGpuDelegate(_TfLiteDelegate*)

Offcial website building instruction do not enable gpu feature, how to do this?
Below is website about how to build tensorflowlite for ios
https://www.tensorflow.org/lite/guide/build_ios

Please have a look on #28468 and let us know if that resolves the issue. Also can you please provide us the code snippet and information about the TensorFlow version you are using. Thanks!

@achandraa
thanks!
I came cross a strange issues only occurred on HuaWei Phone.
For phone infomation with this image.

At first time run inference no crash, It is always crash at second time. below is crash info.

Below is processed crash info with ndk-stack, but unable to locate in tensorflow source as build tflite from source with no debug symbols and i do not know how to build with debug symbol
I have tried many methods for building tflite with debug symbol but not successed
For example
1、 bazel build -c dbg --strip=never --compilation_mode=dbg --per_file_copt=//tensorflow/lite/.*.cc@-g,-O0  //tensorflow/lite:libtensorflowLite.so --crosstool_top=//external:android/crosstool --cpu=armeabi-v7a --host_crosstool_top=@bazel_tools//tools/cpp:toolchain --cxxopt="-std=c++11"
with -c dbg --strip=never  --compilation_mode=dbg

@weinixuehao
Sorry for the late response.  I've been out of town.
I need a little bit of clarification.  This bug started with linking error "undefined symbol of DelegateGpuDelegate" but now, you're experiencing a crash that is not related to the GPU delegate, right?

@impjdi Right!
This is two issues

@weinixuehao
I meet the same crash problem on zuk z2 and mi6. My code is almost same as yours.
It crashed at the DepthWiseConv.
Have you fixed the problem? Thank you!

@jianchong-chen
I used tflite java version to deal with the problem because having no time to consider it.
if you still want to use c++ api which building tflite yourself maybe is a good start
such as building it with xcode or android studio directly which has a benefits to allow you debugging  this source and to watch some variable when them vary

@weinixuehao @jianchong-chen  hello,I met a similar problem.The app crush when running at the invoke-code.But I use libtensorflowtflite_jni.so in official-prebuilt AAR.I now have another issue,how do you init your tflite model? And how do you detect the image?Do you put your interpreter and input/output tensor in global variable at JNI c++ file?I doubt this maybe a Android  error but I don't know how to solve it.Looking forward to your reply

@weinixuehao Could you please let us know if this issue still persists ? If it is resolved then please feel free to move this issue to close status ? Thanks!


Unlike Media/Post

I could not find an InstagramUnlikeRequest to unlike a post. This feature will be great to have!
Or is there already a way to unlike a media ?

maybe try to like same photo twice?
see if it removes it

I don't think it should work because on the instagram web, the endpoint invoked for the unlike is different from that for the like operation.
But still will try @patriotaki

I think it should not be a difficult task to have the unlike operation. We just need to replace the /like with /unlike in the endpoint.
Although I am not sure about the response parsing. @brunocvcunha


Live collaboration?

Hello there,
I was looking at the app before deciding to sponsor it and saw some live collaboration capabilities which looked good. In reality however, this is nothing more than a glorified diff system which messes up the formatting by inserting some huge red // around the changes. You can remove them but then the formatting is a mess and you end up reconciling differences manually, which is not very effective.
Is this the current capability for collaboration or have I done something to break it?
Thanks!

You won't see any red // in the v5. But yes, there is no real time collaboration, only a periodic diff/merges mechanism.


Fixes Issue #1595 Added red color to header text

Pull Request Type
What kind of change does this PR introduce?
[x] Solved an issue/task
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:
Issue Number: #1595

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Interacting at all with the winget source hangs

Brief description of your issue
Attempting to list installed packages updates hangs, or update sources hangs
Steps to reproduce

winget list --verbose or winget update --verbose

Expected behavior
List installed packages or updates.
Actual behavior
The indicator simply swirls around indefinitely.
Log attatched.
WinGet-2024-06-21-15-51-26.570.log
Environment
Windows Package Manager v1.7.11261
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.26241.5000
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.22.11261.0

Winget Directories
---------------------------------------------------------------------------
Logs                               %LOCALAPPDATA%\Packages\Microsoft.Deskt…
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.Deskt…
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages
Installer Downloads                %USERPROFILE%\Downloads

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Enabled
LocalArchiveMalwareScanOverride           Disabled

Could you try running winget source reset --force from an administrative terminal and then check again?

Nope. Running winget update --verbose still hangs.
Output:
C:\Users\ךינשגכהד>winget update --verbose-logs
Failed in attempting to update the source: winget
Failed when opening source(s); try the 'source reset' command if the problem persists.
An unexpected error occurred while executing the command:
0x8007139f : The group or resource is not in the correct state to perform the requested operation.

Here's the attatched log:
WinGet-2024-06-22-15-23-44.971.log

How about updating the source manually through the command:
Add-AppxPackage -Path "https://cdn.winget.microsoft.com/cache/source.msix"

(If for some reason that command doesn't work, you can download the source package manually from the URL and double-click on the .msix file to run)

Did't work. Tried the pwsh command and manual download.
Here are the logs:
WinGet-2024-06-23-09-03-08.165.log
WinGet-2024-06-23-09-01-17.434.log
AppInstaller-2024-06-23-09-00-52.010.log
AppInstaller-2024-06-23-09-00-52.882.log
AppInstaller-2024-06-23-09-00-45.856.log
AppInstaller-2024-06-20-21-30-37.317.log
WinGet-2024-06-23-08-50-56.829.log
WinGet-2024-06-23-08-49-48.980.log

All the above logs end abruptly at
"2024-06-23 08:50:57.381 [REPO] Adding to aggregated source: winget"
One of them having
"2024-06-22 15:26:30.547 [FAIL] C:__w\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(520)\WindowsPackageManager.dll!00007FFA71743B79: (caller: 00007FFA7174546A) Exception(1) tid(3d14) 8007139F The group or resource is not in the correct state to perform the requested operation.
"
Which indicates the sources are in a bad state.
Could you run "winget source reset --force" again and share the logs? Worst case, we may need to investigate some additional data files at: C:\Users<user>\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\Settings

Resetting the source this time allowed listing updates and searching. Here are the logs
WinGet-2024-06-24-07-43-18.540.log
WinGet-2024-06-24-07-42-54.702.log
WinGet-2024-06-24-07-42-47.744.log

Resetting the source this time allowed listing updates and searching. Here are the logs
WinGet-2024-06-24-07-43-18.540.log WinGet-2024-06-24-07-42-54.702.log WinGet-2024-06-24-07-42-47.744.log

Thanks. Glad it's working again. The logs do show reset is successful (user_sources and sources_metadata are removed). I can only guess the previous reset attempt did not complete.


Icon documentation

Description
For some reason 'Icon' is missing from the roadmap showing "Material 1.0 | Sep 2023" features and components.
I do see that documentation now exists for icon in the source but it is not reflected in the "https://material-web.dev/" menu
The documentation says that two axes (optical size and weight) are handled automatically but does not describe the mechanism in how those axes are determined.
Browser/OS Environment
Not pertinent to this issue.

Duplicate of https://github.com/material-components/material-web/issues/5206
https://fonts.google.com/icons is a good resource to learn a bit more about the axes. We recommend those two axes be managed with font-weight and font-size to reduce the payload size of importing the icons.


Extension of manual

Added a description of changes in function GetHandle()

You need to squash commits, in the second you just added a dot


chore: delete snownet-tests

When snownet was first being developed, these tests ensured that hole-punching as well as connectivity via a relayed works correctly. We have since added extensive tests that ensure connectivity works in many scenarios via tunnel_test. tunnel_test does not (yet) have a simulated NAT so hole-punching itself is not covered by that.
UDP hole-punching is shockingly trivial though because all you need to do is send UDP packets to the same socket that the other party is sending from. This isn't done by our own code but rather by str0m's implement of ICE (as long as we add the correct candidates).
The snownet-tests themselves are quite fragile because they need to set up their own event loop and manually construct an IP packet. They haven't caught a single bug to my knowledge so I am proposing to delete them for ease of maintenance.
For example, in https://github.com/firezone/firezone/actions/runs/10449965474/job/28948590058?pr=6335 the tests fail because we no longer directly force a handshake when the connection is established. This is unnecessary now because the buffered intent packet will directly force a handshake from the client to the gateway. Yet, snownet-tests event loop would need adjusting to also do that.

Integration tests should cover here.

They don't cover the actual hole-punching bit because everything is on the same network within our docker-compose file. But any smoke test we do locally will test that so I don't think we effectively lose anything here.
@jamilbk Could you update the required checks to remove the deleted jobs?


cannot setup IOS following current directions

Environment

Dev OS:  macos ventura Version 13.3.1 (22E261)
Xcode 14.3
Build version 14E222b

Steps to reproduce
brand new project with 1 commit of adding library w/ pod modifications
https://github.com/1mike12/rnmapbox_broken
the 1 commit after making installation changes
https://github.com/1mike12/rnmapbox_broken/commit/1dcc0c269b84c3633034d346f6117ebf9289e9c9
try to install pods and get
/Users/u/SoftwareProjects/rnmapbox_broken/node_modules/@rnmapbox/maps/rnmapbox-maps.podspec:95: warning: already initialized constant Pod::MapboxImplVersion
/Users/u/SoftwareProjects/rnmapbox_broken/node_modules/@rnmapbox/maps/rnmapbox-maps.podspec:95: warning: previous definition of MapboxImplVersion was here
[!] CocoaPods could not find compatible versions for pod "MapboxMaps":
  In Podfile:
    rnmapbox-maps (from `../node_modules/@rnmapbox/maps`) was resolved to 10.0.6, which depends on
      MapboxMaps (~> 10.13.1)

None of your spec sources contain a spec satisfying the dependency: `MapboxMaps (~> 10.13.1)`.

You have either:
 * out-of-date source repos which you can update with `pod repo update` or with `pod install --repo-update`.
 * mistyped the name or version.
 * not added the source repo that hosts the Podspec to your Podfile.

whoops, totally missed the fix
just run
pod repo update
https://github.com/rnmapbox/maps/discussions/2830#discussioncomment-5762648


[Bug]: max_tokens in openai_model.py wrongly set

Language
Python
Version
1.2.1
Description
In the complete function in openai_model.py the following assignment is used in line 126:
max_tokens = template.config.completion.max_input_tokens
later this variable is used in line 197 as input for the OpenAI model to set the max. number of output tokens, leading to not being able to set the max_input_tokens to higher than 4k.
Reproduction Steps
1. set the max_input_tokens in config.json > 4k
2. call the OpenAI model
3. Error about the max_tokensize will occur
...

C# version - #1800


[FEATURE] Display Language of the Project

Description
When displaying the language, the project is written in, its easier for people finding a good issue which is related to their techstack
Screenshots
No response
Additional information
No response

closed with #73


fix(cache) same invalidation sent multiple times

Summary
A single patch on e.g. service entity may cause four cluster events fired, out of which all four are the same, though 2 operating on different cache because the #5114 introduced the split.
This PR checks that if the new and old entity cache keys are same it doesn't send the other 2 invalidation events.

This is a general lua testing question -- is it possible to mock cache:invalidate and then assert that it was only called X number of times?

@hutchic it is, I think some of our tests do actually that. Let me try to add that.

update, I'll try to add test on monday, a bit more complex than I otiginally thought.


❓ Support : Maximum performance guidance

Performance metrics provides performance using 1 thread and batch size equal to 1, which is totally fine for a benchmark, but as a fresh new user I am having difficulties understanding how I could get maximum performance (use case : run the VAD on dozens of thousands of recorded audio files, having high core counts machines for that), apart from using ONNX instead of JIT.
Should I set session.intra_op_num_threads and session.inter_op_num_threads  to the number of cores I have available, and maybe more importantly how do I set the batch size after torch.hub.loading the model/utils ?
Adding an example in that direction in the wiki/notebook would be great.
Thanks a lot

run the VAD on dozens of thousands of recorded audio files, having high core counts machines for that

Hi, our VAD was not designed for this purpose.

I do understand that it is for streaming first, and I do have a streaming use case coming later on, so I would still like to know the expected effect of changing num_threads to something different than 1, and how to change the batch size without forking the source code if that is possible. Thanks


Add phpCE elePHPant

Ping @dseguy

@akiojalehto Looks like this is another one not manufactured by Custom Plush Toys.  Submitted #97 to remove it.
https://phpug.slack.com/archives/C195M60A2/p1540753148007100

I'm not in the PHP UG Slack. Can you tell me what the message is from there?

I am finding some extra information on this question.  :)


Error in compilation

Environment
**git rev: **:
Binding:
Environment:

OS: Ubuntu

Bug
What happened:
Compiling code is breaking with the following error:
Compiling deltalake v0.4.1 (https://github.com/delta-io/delta-rs#17999d24)
error[E0599]: no method named `with_allow_http` found for struct `MicrosoftAzureBuilder` in the current scope
   --> /home/kesav/.cargo/git/checkouts/delta-rs-fb509102fe888c8e/17999d2/rust/src/builder.rs:404:35
    |
404 |                 builder = builder.with_allow_http(allow);
    |                                   ^^^^^^^^^^^^^^^ method not found in `MicrosoftAzureBuilder`

error[E0599]: no method named `with_client_secret_authorization` found for struct `MicrosoftAzureBuilder` in the current scope
   --> /home/kesav/.cargo/git/checkouts/delta-rs-fb509102fe888c8e/17999d2/rust/src/builder.rs:559:27
    |
559 |         builder = builder.with_client_secret_authorization(client_id, client_secret, tenant_id);
    |                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ method not found in `MicrosoftAzureBuilder`

error[E0599]: no method named `with_allow_http` found for struct `MicrosoftAzureBuilder` in the current scope
   --> /home/kesav/.cargo/git/checkouts/delta-rs-fb509102fe888c8e/17999d2/rust/src/builder.rs:563:51
    |
563 |         builder = builder.with_use_emulator(true).with_allow_http(true);
    |                                                   ^^^^^^^^^^^^^^^ method not found in `MicrosoftAzureBuilder`

For more information about this error, try `rustc --explain E0599`.
error: could not compile `deltalake` due to 3 previous errors

What you expected to happen:
How to reproduce it:
Add the deltalake dependency using:
deltalake = { git = "https://github.com/delta-io/delta-rs", features = ["azure"] }

The current main branch relies on a patch of the object_store crate. Could you add the following to your Cargo.toml. The root one in case you are in a workspace ...
[patch.crates-io]
object_store = { git = "https://github.com/apache/arrow-rs", rev = "86446ea88b6fc3e11b47aeedb9eb24f4d69b2f13" }

Yes figured out that.


Ingress (Traefik) - Daemonset not binding properly

What kind of request is this (question/bug/enhancement/feature request): Bug
Steps to reproduce (least amount of steps as possible):

Install daemonset version of https://docs.traefik.io/user-guide/kubernetes/
Attempt to upgrade/redeploy the daemonset

Result:
The ports it bound on (80) will no longer be bonded
Other details that may be helpful:
(I have to manually delete the pods but I believe that's because rolling updates are not turned on for the daemonset)
Prior to upgrade: https://you-have-a.bigno.se/4f2f9ba.png
After upgrade: https://you-have-a.bigno.se/c01b5e1.png
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.1.1
Installation option (single install/HA): Single installation

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Hosted
Machine type (cloud/VM/metal) and specifications (CPU/memory): Metal - 2x 6gb 3core
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.5", GitCommit:"32ac1c9073b132b8ba18aa830f46b77dcceb0723", GitTreeState:"clean", BuildDate:"2018-06-21T11:46:00Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.3", GitCommit:"a4529464e4629c21224b3d52edfe0ea91b072862", GitTreeState:"clean", BuildDate:"2018-09-09T17:53:03Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}

Docker (use docker info):
(I ran this on a worker node)

Containers: 42
 Running: 19
 Paused: 0
 Stopped: 23
Images: 32
Server Version: 18.09.0
Storage Driver: overlay2
 Backing Filesystem: extfs
 Supports d_type: true
 Native Overlay Diff: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
 Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: d6de12e2f362cb9dc49ad957911996d3de59b338
runc version: 4fc53a81fb7c994640722ac585fa9ca548971871
init version: fec3683
Security Options:
 apparmor
 seccomp
  Profile: default
Kernel Version: 4.15.0-20-generic
Operating System: Ubuntu 18.04 LTS
OSType: linux
Architecture: x86_64
CPUs: 3
Total Memory: 5.821GiB
Name: daave1.aaathats3as.com
ID: 4IIJ:H7ZV:J2YH:CEKX:TYRQ:S5DD:T7UI:VDHK:ZU3B:QRUX:RSRE:H372
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Labels:
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false
Product License: Community Engine

WARNING: No swap limit support

I actually do relate to this; this happens to me to and its rather annoying. fix when

This is the way k8s works and not a "rancher issue".  It's because the upgrade policy is rolling update instead of on delete.
Rolling update creates a new pod and then deletes an old one.  But because you (they) are using daemonset + node ports, there is nowhere a new pod can be successfully scheduled and bind the requested ports, since they are still all already in use on every node.

So changing the upgrade policy to ondelete should fix it, or have i misread

@vincent99 the update policy is ondelete not rolling update


The search box with title “Filter by title” has a white background

Describe the bug
The search box with title “Filter by title” has a white background. Please consider this an accessibility issue for people with sensitive eyes.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://aka.ms/rds'
Switch to Dark mode
Notice that the search box (top left of the page) with title “Filter by title” has a white background.

Expected behavior
The search box has a gray background. The color should preferably be same as the address bar on Microsoft Edge in dark mode (see attachment)
Screenshots

Desktop (please complete the following information):

OS: All
Browser: All
Version: All

Smartphone (please complete the following information):

Device: All
OS: All
Browser: All
Version: All

Additional context
None

@ShevaDas Can you follow up this suggestion?

@spatnaik Thanks for reporting this. I'll work with our engineering team to get it resolved.
#log-suggestion @ShevaDas

#log-suggestion @ShevaDas

🚀 ATTENTION: Internal request logged.

I believe this can be closed. Here's current screenshot from today:

Closing this issue as the box is now theme-aware.


Can you run an animation as an animation, then interrupt and scrub through it?

Wondering if there is a way to combine the scheduler and scrubber.

This is a really cool idea actually, it's not a use case that I'd thought about. I'm not sure that this is that simple to do out of the box with the current functionality, it's really either scheduled or scrubbed at the moment.
I'm not sure what the api for this would look like, but I'll have a think about it. It would be a cool feature to have, so if I have time I might try to include it. If you have any ideas for how you think it might work then let me know.
In the meantime, one work around is that you can setup your own CADisplayLink and use that to drive the ActionScrubber`. Than you can switch between using the display link / scrubbing to drive the animation.

Thanks. I'll think about it too. Maybe when an ActionScheduler is interrupted, its Actions get copied to an instance ActionScrubber?

Conversely, I can see a use for a "snap" behavior, where you animate to the start or end of a scrubber, like when the user stops dragging.

@GoldenJoe It's actually quite a tricky issue to resolve in a way that doesn't negatively impact the usage when you're not mixing scheduling / scrubbing.
When I have a bit of time, I might create another example in the example application that I can use to play around with some apis for this use case
It might be that the scrubber can have some sort of 'auto-scrub' feature, where it essentially drives it's animation with a scheduler. There's some additional complexity to handle there, because the scrubber infers the direction of the animations from the values it receives, and schedulers only run animations in a forwards direction. I think we could make it work though.
Sure, you're welcome to send me an email. My email address is public on my github profile, or you can use the contact form on my website website


Allow Collective to Re-Dispatch Proposal with an Account

polkadot companion: paritytech/polkadot#2811
This PR introduces a way for the collective origin to re-dispatch their proposal execution using a "signed" origin with a deterministic account they can control.

[x] tests
[x] benchmarks

The target scenario is to allow governance on a relay chain to control a parachain. The parachain can whitelist a set of accounts which can be generated from the relay chain governance, that has access to do things like runtime uprgrades, minting balances, etc...
The account generated by the collective origin is deterministic and consistent based on:

The Module ID of the collective pallet instance.
The number of target approvals (the "numerator")
The number of target members (the "denominator")

The target number approvals and members can be specified in order to match what the parachain will expect.
For example:
A parachain can have collective_account(1/2) registered to be sudo on the chain.
The relay chain may actually have an odd number of council members, lets say 13.
Council would need to pass the proposal with greater than half of the members, and then would be able to redispatch the call specifying they want the account representing 1/2.
The target members specified must be less than or equal to the actual number of members, and the ratio of the collective account must be less than or equal to the actual collective origin, so not to misrepresent the power of the account.

/benchmark runtime pallet pallet_collective

stale

re-opening this one last time as it is ready and I think we need it in the future.


Add NeuralNetwork to OM package manager

Add NeuralNetwork library to the OpenModelica package manger: https://github.com/OpenModelica/OMPackageManager

See https://github.com/OpenModelica/OMPackageManager/pull/23. It's a bit tricky since there already is a NeuralNetwork key in repos.json from the obsolete v1.0.0 library.


Add typed for mypy as third party library

Fixing issue 8 -> https://github.com/danielSanchezQ/rusty_results/issues/8

Could we have a test that proves that this works? I am thinking about the CI/CD use that file somehow when running mypy. Or is that, as my understanding, that just bypass mypy and forwards the typing to the one we have inlined?

I just double-check the py.typed works for a third party so when they use mypy this library won't throw any errors.


intellisense?

I tried out this library this morning and love how simple it is to use. Also like the position aware feature which is critical since I want to highlight/color text as the user types it in and certain patterns are found. However I am interested in providing some kind of "intellisense" feature. As the user types in some text I'll parse it and pop up a window showing what text could come next. I don't think there is an easy way to rig this up with Sprache. One thought is I could manipulate the text before parsing to include an explicit cursor character, like "123 45th Avenue NE, |", and then see if it matches a pattern called AwaitingCityStart and then if the input text was "123 45th Avenue NE, Sea|" it might match a pattern for AwaitingCityEnd. Anyway this is just a rough idea; don't know if it would work or how ugly it could get. Is Sprache the right tool for what I'm trying to accomplish or is there some other tool that plays nice with Visual Studio and C# that I could use?

I read more about this. Doing anything approaching what Visual Studio does is incredibly hard because I need to be able to recognize and deal with incomplete/broken patterns. Maybe I need to explicitly define parsers for all the incomplete patterns I care about. I think Roslyn might work for doing something sophisticated, but this is way beyond what I need.

It's a pretty challenging problem. Thanks for the follow-up, and good luck with the project!


Clarify

closes https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/3410
Suggestions provided by @E-Saggese
@JohanFreelancer9 For Copy review, thanks

Edited for structure.


Testing crashes randomly

When call runtime.GC() after every case, some case crashes randomly, currently workaround by disable automatic DecRef.
func setupTest(t *testing.T) {
	testMutex.Lock()
	Initialize()
	// TODO: Remove this once we solve random segfaults
	getGlobalData().disableDecRef = true
	t.Cleanup(func() {
		runtime.GC()
		Finalize()
		testMutex.Unlock()
	})
}

Need locate the problem.

Fix by https://github.com/cpunion/go-python/pull/22


Github Actions charts upload

Issue:

Problem

Solution

Testing

Engineering Testing
Manual Testing

Automated Testing

Test types added/modified:

Unit
Integration (Go Framework)
Integration (v2prov Framework)
Validation (Go Framework)
Other - Explain: EXPLAIN
None
REMOVE NOT APPLICABLE BULLET POINTS ABOVE

If "None" - Reason: EXPLAIN THE REASON

If "None" - GH Issue/PR: LINK TO GH ISSUE/PR TO ADD TESTS

Summary: TODO
QA Testing Considerations

Regressions Considerations

TODO
Existing / newly added automated tests that provide evidence there are no regressions:

TODO

@brooksn Yes, it won't work on rancher/rancher


Add template HPA

Ready for your review, thanks!

Note, here is a PR for a fixed HPA Virtual driver: https://github.com/humanitec/resource-registry/pull/128

Note, here is a PR for a fixed HPA Virtual driver: humanitec/resource-registry#128

Great to see, thanks!
And like discussed, we could also move forward with this PR, which will be still relevant even when this new virtual driver will be ready.
I'm merging this PR with these examples.


daemon: Allow DNS transparent mode to be turned off with encryption

DNS transparent mode was introduced to make sure that DNS traffic is always encrypted if the user is running with transparent encryption. If DNS proxy transparent mode is turned off, proxied DNS traffic will be leaked.
However, DNS transparent mode is suffering from various bugs, e.g.

https://github.com/cilium/cilium/issues/31535
https://github.com/cilium/cilium/issues/31197
https://github.com/cilium/cilium/issues/33144

While we are working on addressing these bugs, some users might be fine with proxied DNS traffic being leaked. Therefore, this commit introduces a hidden and undocumented flag which requires DNS proxy transparent mode to be enabled with IPSec.

/test

/test

/test

Discussed with @gandro. This seems OK with me as long as the user is aware of the compromises they make when using these features in tandem.


Rename 0168 Excel Sheet Column Title.cpp to 0168 excel sheet column t…

File(s) Created:  0168 excel sheet column title.cpp
Language(s) Used: cpp
Submission URL:  https://leetcode.com/problems/excel-sheet-column-title/submissions/1069538088/

The answer is perfectly correct.


Add alternative sound system

In progress. The core code is in place, it needs the sound generator.


Add coverage (like clover)

Coverage reports are something I thought about, too, but to be honest,
at this stage - as webp-convert is still a rather small library - it
adds overhead we don't really need.
But, as always, I'm open for discussion and of course, PRs are always
welcome!

Maybe on some shiny day... (or perhaps, rather, on a rainy day)... Closing, as it will not be in near future


Update README.md

Common flaw affecting multiple linux distros. It causes udev to hang on /dev/sr0 after inserting some video discs which causes udev to kill off /dev/sr0. this can result in docker hanging, crashing, and/or creating unkilllable makemkv zombie processes. Only certain discs appear to trigger it, but its been about 1 in 10 here lately.
additional details: https://forum.makemkv.com/forum/viewtopic.php?t=25357

@jcfrosty  Thanks for the insightfull addition to Readme. I've fixed a small typo and will gladly merge.


Add tooltip to dice on skill dice pool

Additional ideas (I'll edit more in):

Add a tooltip for dice in the skills page explaining what added that die (my terrible attempt at demonstrating this below)

Originally posted by @wrycu in https://github.com/StarWarsFoundryVTT/StarWarsFFG/issues/403#issuecomment-714178673

Not going to be able to do this per die, but maybe possible as a whole pool breakdown.

Sounds good to me.
That still does what I'm looking for - explains why certain dice are in the pool.

Added to dev branch

Just checked it out - dope, thank you.


Thermal Detonators (device.json) is missing a device node

"device" : {
"name" : "Thermal Detonators",
"type" : "Bomb",
"effect" : "At the end of the Activation Phase, this device detonates. When this device detonates, each ship and remote at range 0–1 rolls 1 attack die. Each ship gains 1 strain token for each [Focus] result, and each ship and remote suffers 1 [Hit]/[Crit] damage for each matching result."
}

Closed in #500


ensure that basic order type is within range

Ok.
ReferenceAssertions.sol could also be updated so both are on the same page.

Ok.
ReferenceAssertions.sol could also be updated so both are on the same page.


StackExchange Sites

StackExchange
Dataset URL - The data dump can be used to obtain all the dumps here
Does the dataset exist in a scraped format?

Description
Stack Exchange is a network of question-and-answer websites on topics in diverse fields, each site covering a specific topic, where questions, answers, and users are subject to a reputation award process. The reputation system allows the sites to be self-moderating.
Procedure

[ ]  Download all the dumps
[ ]  Formulate appropriate filters if at all we want to.
[ ]  Processing and formatting.
[ ]  lm_dataformat Processing

Hi, i have started working on this

Hi, i have started working on this

Awesome, please make sure to check the original Pile: https://github.com/EleutherAI/the-pile
To make sure we aren't doing duplicate work

@flowpoint, Look at this https://github.com/EleutherAI/stackexchange-dataset

i know about the similar scripts by collin and sdtblck
we surely can merge and improve their functions, i try to write modular code anyway
i've gone ahead and written less scripty code in #6
have a look and feel free to merge/comment
it strongly sounded, like we're assigned to the stackexchange data and still have to watch out with coordination and consolidation to prevent further duplicated work

Additional stack exchanges that don't fall under technology that may be useful:
https://cs.stackexchange.com/
https://cstheory.stackexchange.com/
https://opensource.stackexchange.com/
https://cseducators.stackexchange.com/

@reshinthadithyan Just some details about StackExchange archives based on my experience with the Code Review project.
The StackExchange archives are saved as XML files which can be hard to work with so we converted them to JSON files with this script.
The formatting of code is a little weird so I manually added <code> tags:
https://github.com/CarperAI/CodeReviewSE/blob/cd2af9c1df1e617c42f7865e21ec0746bffb2579/data/helper.py#L21
Creating an lm_dataformat file is pretty easy, it's just:
ar = Archive('dataset')
ar.add_data(text)
ar.commit(archive_name)

Other websites to potentially include:

Super User
Tex - LaTeX
Unix & Linux
Information Security
Database Administrators
Artificiall Intelligence
Data Science
Blender
Emacs
VI and Vim
Internet of Things
Quantum Computing
Computer Graphics
DevOps

There is this scraping script that might be useful: https://gist.github.com/craffel/a1e2aff893776d0ef2b0a95ed0fd7e7a

Looked at https://gist.github.com/craffel/a1e2aff893776d0ef2b0a95ed0fd7e7a. This script processes questions and answers into a stream of paras. Which is a good post-processing exercise for modeling. But I guess for us, retaining upvotes and other metadata makes a lot of sense for cataloging as an intermediate. I guess we can go ahead with @tmabraham's CodeReviewSE(https://github.com/CarperAI/CodeReviewSE) scripts, Which retain the complete metadata. Later we can post-process per our need.

The dump for each site has the following files, Badges.xml, PostHistory.xml, Posts.xml, Users.xml, Comments.xml, PostLinks.xml, Tags.xml, Votes.xml. We'll want to process the entire site dump into a file,
Preprocessed Format:
SitePost.json - Wherein the entire site's posts are dumped here. post_id is the key and all the related comments, answers, and user_infos of users interacting in the post are under them.
Note : This is a clean way to store all the metadata in a singular format.
PostProcessed Format:
For the post-processed version, we'll want to convert the pre-processed dataset into lm_dataformat with a conversation-like format ?
Need for Preprocessed and Postprocessed format
The reason that we would want to maintain these two formats is that we're still unclear about how we're going to prompt.
@ncoop57, thoughts ?

Yeah, we can go ahead and store in this format while we finalize the final lm_dataformat

I've come up with this intermediate document structure oriented around a stackexchange site/document/thread.
It erases quite some fields.
Some because they look too granular/ unmatchable with other datasets (like votes).
Other ones, like Edit-history are nontrivial and can be considered later.
Dates are kept for distribution analysis.
dump data schema for reference
intermediate structure as pseudocode/python/pydantic:
class Comment(BaseModel):
    user_id: str
    score: int
    creation_date: datetime
    text: str

class User(BaseModel):
    user_id: str
    reputation: int
    upvotes: int
    downvotes: int
    profileviews: int
    creationdate: datetime
    # this is not pure text
    aboutme: str

class Post(BaseModel):
    # ids
    post_id: int
    user_id: str

    # main post text
    title: str
    text: str
    comments: list[Comment]

    score: int
    tags: list[str]
    favourite_count: int
    view_count: int
    creation_date: datetime
    last_edit_date: datetime

# this should represent a single document/jsonl line
class StackExchangeDoc(BaseModel):
    question: Post
    answers: list[Post]
    accepted_answer_post_id: int
    users: list[Users]

I am working on denormalising the data.
Here is a denormalized sample with just the relations mentioned by @flowpoint above. I haven't excluded any columns for each table. Eventually, we would only want some columns to be included in the final output.
https://gist.github.com/vanga/b1f1a4a6d82accd69ba35c0f33b76435
Excluded relations: Votes, Badges, PostHistory, PostLinks
If we need to include all the relations, we could do that.

@reshinthadithyan  and @vanga  please make sure to add this information to the issue description:
Tests
Include a dummy_dataset.parquet file to test your code against. This dummy_dataset should include the columns for the data and metadata associated with the dataset, which will then be converted into the final format for language model consumption, along with an example row or rows that you can verify your code correctly collects. In addition to this file, include the unit test that evaluates your code against this dummy_dataset.
Give an example of the columns and data:

col1
col2
....

row1
row1
....

@ncoop57, https://github.com/CarperAI/Code-Pile/issues/1#issuecomment-1257333026 This should be a separate issue so that people are aware of it. I'm adding the same in the Issue template.

For anyone working on this, current version of the dumps seem to be corrupted
Old version can be found here https://archive.org/details/stackexchange_20220606


up(parser): gracefully handle illegal list separators

Continue parsing when an illegal list separator is found when parsing a SeparatedList. Illegal but handle-able separators are specified via illegal_separator(). This defaults to ;, but can be changed or disabled by returning None.
For example:
const x = {
  a: 1;
  b: 2
}

instead of panicking, the parser will produce a valid AST and still report the illegal ; in the object literal expression.

Will probably improve list parsing at some point: https://github.com/oxc-project/oxc/issues/3887

This PR bloats the code too much. Let's defer the change and rewrite the whole thing instead.


[release-2.11] Downgrade to UBI8

This matches the oc build for 4.15.
ref: https://issues.redhat.com/browse/ACM-10920

/cherry-pick release-2.10

/hold for:

https://github.com/stolostron/must-gather/pull/196

/unhold


search stuff

[ ] open keyboard when going to search page
[ ] search text is cut off on letters like f

Everything in this issue seems like it has been fixed. Should this be closed?


Improved Rename and Show Usage

At the moment, files that are to be included in rename and show usage needs to be opened and parsed by libclang. Sadly, libclang is too slow to do a full parse of an entire medium or larger sized project. However, there are other options, like using cscope, that might solve this adequately. Feel free to post your opinions here, or suggest other solutions.

Many IDEs I know (CLion, MSVS) schedule parsing in the background, with on-disk cache so the reparsing doesn't occur when reopening file or reopening program.

I do not think we can use libclang for this as it can take hours to parse a larger project. Additionally, every time one changes the compile options, a full reparse would be needed. CLion and MSVS solves this by using faster but far less complete parsers than libclang.

The new flag CXTranslationUnit_KeepGoing  (http://clang.llvm.org/doxygen/group__CINDEX__TRANSLATION__UNIT.html#ggab1e4965c1ebe8e41d71e90203a723fe9afda3272e16e2aa499fc265615991f968) might be of use, for instance, parsing sources without including system includes and maybe still get some cursor information. I'll investigate later.

Well i just naively tried to create a translation unit for every file that had corresponding information inside the compilation database of the Krita project (around 5k .cpp and .h, 2327 files effectively parsed).
Just a simple single threaded loop and it took 1 minute and 30 seconds.
Now i would be surprised if traversing the AST to find types, functions and function calls would take much more.
Probably the highest time would be needed to create the database with crossreferences or the like.

@Smjert This is much faster than expected. Are you sure the headers were found when parsing the source files? I seldom see parse times below one second on sources of more complex projects. However, if the header files are not found during parsing, it is much faster.

Nevermind ^^', it was a bit late in the day and i supposed that given that i was querying the TU without crashes (like, trying to have some informations from the cursors) it wasn't null.
Now i rechecked and indeed the translation unit returned is null (so error happened).
I'm sorry i did came to conclusions really too fast.

@eidheim: So i fixed my code and it takes like 2-3 seconds to parse each file, so yes it's really not doable like that.
Disclaimer: Long Text Ahead!
While having read a bit more about libclang doc, having looked around on what others do (QtCreator, YouCompleteMe, RTags) i think that there aren't many choices. Kind of 3 choices in general:

Use/write some text parser/tokenizer which quickly creates cross references. Obviously this will be imprecise due to the "impossibility" of dealing with defines/ifdefs (unless one writes a small preprocessor), and having issues with templates.

Manually create an AST, trying to speed it up by skipping non interesting things (which would basically mean doing point 1 plus dealing with an AST, preprocessor and stuff).

Use libclang with a "clever" use of precompiled headers.

The third choice is the way to go imho though the question is exactly how:

QtCreator compiles TU (.h and .cpp) with the preamble stored on disk, automatically created by the option CXTranslationUnit_PrecompiledPreamble, as soon as you open a file.
But it doesn't seems to reuse .pch coming from already parsed headers or to use Clang at all to Find All References, where it resorts to its own AST.
QtCreator so uses clang for syntax highlighting, go to declaration and diagnostic.
The issue here is that if it had to go with a clang only solution, it would do too much work and not reusing the pch it already precompiled.

RTags seems to do something very similar but given that it only uses clang it does it a bit more cleverly (though i'm not 100% sure of the details... the code is a bit hard to follow):
Basically for each source file found in the compilation database it walks all the include paths args on the command line and "i think" it compiles all the headers it finds there. It's not then clear to me if it's RTags itself that knows which pch a source file has to use and reuses them, also checking if the args that compiled that pch are the same or if it's up to the user to pass the "include-pch" command line.

That being said both, done like that, still have a big issue imho, that is space consumption of the pch headers (opening the source files of a project like RTags with QtCreator creates preambles of like 10 to 20MB each, and it can go worse https://github.com/Valloric/YouCompleteMe/issues/1419).
What could be done is to create smaller parts/pch which could be reused, depending also on the compile arguments needed (meaning that some of the pch must be duplicated if there are two sources that includes that same include but have different flags like defines or C/C++ version).
The question is where to stop walking the chain of includes.
My first idea is probably a bit too crazy: create the full dependency tree of includes and then compile the bottomest, then while going up the tree, chain all the pch. This way each branch is an include chain starting from a source file and the whole tree represents all the needed includes that are compiled with the same flags.
The second step would be to store these pch along with flags in some kind of hashmap i guess, where each key is the include path and the values is a list of pch, with each entry being a pch + flags.
Obviously the best situation would be where all the project is compiled with the same flags, so one would have exactly one pch in that list.
Now the dependency tree can be created by clang itself: if one compiles the TU with some special flags that skips function bodies (and maybe something else?), it could already have the include dependency tree on its hands.
Though this might be crazy because the dependency tree could be pretty big... especially if the project itself has a lot of includes and a complicated tree (not including system includes and library), so not only it could require a lot of memory to keep the entire hashmap in memory (ok it could be offloaded to disk..), but also when a change happen it could potentially trigger a very big reparse.
So maybe the less crazy choice is to do what people would already do by hand: precompile system/library headers only, though not doing one mega pch, but as i described earlier, doing once "for each" header and then reuse them when walking up the tree.
Then watch for any flag/source code change and reparse accordingly (for instance if the user adds a #define or #ifdef before an include, then reparse that specific branch of includes).
This could be maybe also sped up by a simple grepping of the header files to find which ones in the subtree uses that define/ifdef, to avoid unnecessary reparsing.
But without actually implementing it/measuring it first i really don't know what really are the time/space issues..
As a last thing: your solution of using KeepGoing, from what i can understand, would maybe speed it up a bit, but first of all it would only permit you to search for types defined by the project itself, no library/system library type or function, because you would need the full name (namespace, class and arguments); second it could be still quite slow if the project has a lot of its own headers.

Thank you for the write up, and for the interesting research you have done on this issue!
I tried CXTranslationUnit_KeepGoing, but it solves nothing regarding this issue, but the flag will be added next week or so for those who have libclang version >=3.9. However, I did some experimentation on parsing with some headers not included. I found that as long as you have included the type definition of an variable, and type definition and return type of a method, one could probably receive enough information to find these variables and methods. One thought then, is to create a PCH of the include files that have the definitions of the type (and return type) of the variable (or method/function) in question. Then, when parsing the rest of the sources in a project, remove includes that do not have the definitions of the type (and return type for methods). This would be fast enough for performing each time show usage or renaming is performed, though we would need a preference option for especially large projects. The problem with caching PCHs for a longer period is troublesome in quite a few occasions I think. Especially problematic is that one now quite often is switching branches, checkouts commits, or performs rebasing in git repositories.

After some additional thought, one would of course need the parameter type declarations too (their includes).
Regarding PCH's, they are only valid as long as you do not add or remove a header file, so that is why one would have to remove includes from the source you do not include in the temporary PCH. One must possibly just replace all includes in a source with the includes you have in the temporary PCH.

In the meanwhile i did some experimentation too and unfortunately just parsing the TUs with the bare minimum for walking the includes takes like 500ms per file.
For the whole Krita project (~3000 .cpp) it took 9 minutes. Other solutions, like QtCreator, takes like 30 seconds when starting up the project, then whatever search is almost instant.
Well the idea of using PCHs and creating them is that one has to go the deeper it can, without being unmanageable, so that it's unlikely they change and they can be composed.
So continuing with this idea i would try to parse all the includes present in a source file, opening them and doing the same until you hit something that is not part of the project (it goes out of the project root), then you start to transform these in PCHs (so namely all the external libraries and system libraries).
It's true that when switching things may change, but if you have "small enough" PCHs, you can save a lot of time because it's unlikely that many will change, at least for the library and system ones.
What you propose makes sense though, but then you would not be able to research types that are present in other locations of the source (types that appears in function body? class fields?)
I think it has to be an all or nothing.
I'll do some more experimentation...

The idea I proposed was only to solve rename and show usages across multiple files.
One could also exclude source files in the parsing that in no way would include the declaration of the variable or function that you want to rename or show usages of.

Nevertheless, I think this problem needs some additional thought, as the solution is not self evident at all. But it's great that we consider and try out different options!

The idea I proposed was only to solve rename and show usages across multiple files.
One could also exclude source files in the parsing that in no way would include the declaration of the variable or function that you want to rename or show usages of.

Well but you would have to retrieve the full name of the thing you have under the cursor to be sure to be searching the right thing, are you sure it would be enough to use the visible/keyboard typed type you see?

Nevertheless, I think this problem needs some additional thought, as the solution is not self evident at all. But it's great that we consider and try out different options!

Indeed!
And unfortunately the more i go on the more i think what i propose cannot fully done.. it needs big cuts and approximations.

Well but you would have to retrieve the full name of the thing you have under the cursor to be sure to be searching the right thing, are you sure it would be enough to use the visible/keyboard typed type you see?

The file from which you initiate the rename/show usage would have to be fully parsed so that you would know which declarations to look for. The parsing tricks are to find usages in unopened files that are not parsed.

The file from which you initiate the rename/show usage would have to be fully parsed so that you would know which declarations to look for. The parsing tricks are to find usages in unopened files that are not parsed.

Ah yes, i thought one thing and written another. The full name is needed in the unopened files.
I start getting much better what you're thinking and it can definitely work well.
I forgot that libclang can tell you where a type/function is defined.. in this way you could on the fly just do a quick grep of which files contains what you're searching and then parse those just including the single header + KeepGoing; or just parsing all the source files, whichever is the fastest.

I think you understand me even though my explanation was a bit bad, I'll add some examples Tomorrow morning just to be on the safe side. But I'm pretty sure it will not work in all cases, but then, no one has a rename refactoring tool in C++ that work in 100% of the cases, and I'm not sure if it is at all possible. I will be happy with a 95% working solution though!
Actually, you will not need the KeepGoing flag in this case, the flag helps if your source have an include that it does not find for instance. With or without the flag, libclang will create the cursors it can for the entire source that is being parsed. This is good news though, since the solution we are discussing will work on older libclang versions as well.

Well the idea was that the unopened files would artificially have all the includes not to be found except for the one that contains the type you're trying to find, otherwise it would still a no go if you had to entirely parse the source files where you think the type is referenced.
For instance i've tried again in QtCreator to search a type that is found in 490 files... it takes like 10 seconds, but if you have to do that fully parsing the TU of those 490 files it would take too much (and you can't do otherwise if you don't use KeepGoing because a missing include won't return a TU as you said).

There is a rather significant problem with my solution. Given the following files:
// test.hpp
#pragma once

#include <string>

class Test {
public:
  std::string testing() { return "testing"; }
};

Imagine that one wants to rename Test::testing to Test::testing2. One builds a PCH from the following source:
#include "test.hpp"

But in the following file, in order to use the same PCH, not all occurrences of Test::testing() can be renamed due to errors in the line containing std::cout:
// main.cpp

// #include <iostream> removed in order to use the same PCH
#include "test.hpp"

int main() {
  Test test;
  test.testing();              // test and testing can be renamed through libclang
  std::cout << test.testing(); // test and testing can NOT be renamed through libclang
  test.testing();              // test and testing can be renamed through libclang
}

I've tried various libclang flags trying to solve this, but no success yet.

I've tested what we were talking about yesterday:

Took a .cpp with a type i wanted to search (KoColor in Krita source)
Fully parsed the TU
Got the USR of the type
Took a source file where i knew there were some references
Cleaned up the command line removing all "-I and -isystem" arguments
Parsed the TU with KeepGoing
Visited (recursing) all the TU children

The parsing of this last TU takes like 20ms.
It does find part of what i'm searching (funnily it seems also to still have some pieces from the header it includes), though there's a couple of lines like these:
59     KoColor currentColor;
60     KoColor buddyColor;

It finds line 60 but not 59.
And this is not me borking something because even with clang-check -ast-print (without giving the compilation database) it doesn't show it, only line 60 is shown.
But there's also another issue still... the damn #defined and #ifdefs could hide from the AST a lot of things if we don't fully parse the TUs, so you would leave a lot of pieces behind..
And apparently you kind of have the same issue i've seen.

One solution that could work for up to medium sized projects is that one includes all the needed headers in the PCH. For instance, given the files:
#pragma once

#include <string>

// test.hpp

class Test {
public:
  std::string testing() { return "testing"; }
};

#include <iostream>
#include "test.hpp"

// main.cpp

int main() {
  Test test;
  std::cout << test.testing();
}

#include <vector>
#include "test.hpp"

// main2.cpp

int main() {
  std::vector<Test> test;
  test.emplace_back();
  auto testing=test.back().testing();
}

Then one would create a PCH from the following source:
#include <iostream>
#include <vector>
#include "test.hpp"

And in this source, given that one wants to rename Test::testing(), add the needed function bodies that include testing and rename these through libclang.
But again, this would potentially create a pretty large memory consuming PCH, but it would work for small and medium sized projects I think.

The issue with that though are compilation flags and defines (which you can't remove in any way): to be sure that everything will work (the TU will successfully parse or you don't get a weird result) you will have to have a PCH for each set of flags/defines used in source files.
Which means that even if between A.cpp and B.cpp you actually include the same headers, so you think the same PCH can be used, if to compile A.cpp you use different flags than B.cpp, which "changes the content" of the header (-D, some -f switch), you would need a PCH for A and one for B, which means doubling the time already to create it.
Moreover you would have to rebuild both PCH from scratch at each header change.
Also i didn't get if you meant to have a single PCH for the entire project or one for each TU.. the first case has the issue that is slower to create and it might not even be correct, again for different sets of flags between source file compilations... the second one suffer from poor reusability and it would be the same thing as just parsing all the TU the first time and caching them on disk.
I'm currently trying the third "crazy way", which though i don't know how much it will cost in terms of in memory management (so it might be unfeasible this too): i'm trying to play with the frontend API of clang which lets you insert hooks when the preprocessor process the source file.
It's the original idea: build a tree of includes, store all the defines for each header which could change what the header contains (so anything that is used in #if and #ifdef) and then listen for those changes when the user writes the code or changes compilation flags.
It would still need to duplicate headers depending on flag but on a smaller scale and with less space consumption, because by creating smaller PCHs that could be used as building blocks for the final source file needed PCH you can reuse much more.
The real unknown part is how it performs between marking all the possibly dirty includes (all the parents of the changed include, plus all the one that comes after which are on the same header file level) and chaining possibly hundreds of already built PCHs vs just rebuilding from scratch.

I meant 1 PCH for all the source files that maybe contain (after a grep or the like) the variable or function you a looking for. One can luckily find the flags used for each source file in compile_commands.json, so in cases where more than one PCH is needed, it is possible to take this into account.
I think your "crazy way" option is interesting, and we might have to look for such options! Looking forward to see how this is going.

I meant 1 PCH for all the source files that maybe contain (after a grep or the like) the variable or function you a looking for. One can luckily find the flags used for each source file in compile_commands.json, so in cases where more than one PCH is needed, it is possible to take this into account.

Well yes the flags are in the JSON, i'm just worried about the amount of combinations.
Though even with that, it's easy to include headers that conflicts in the PCH, unless you really mean to have a preprocessor that tells you which includes are really included for that source file and set of flags. And to know that you would have to parse all the chain of includes because the #if/#ifdef that control what it has to be included might depend on something that is defined way down the chain.
I mean, sure this can work with the new small project of some guy X, but if you're writing even a "small" library that though has to deal with platform differences, so "it needs" to use ifdefs.. then i fear it's not that viable.

It might be that one can do an initial parse of a source file with all the includes removed (this parse is very fast), and still be able to figure out which conditional inclusions (#if, #ifdef, ...) are valid. Might be worth investigating.

From experimenting I found that include guards (not #pragma once) forces PCH's to be reconstructed and greatly increase the parse time of header files! I'll add removal of these include guards before passing header files to libclang in the following week hopefully. Nevertheless, this discussion most likely lead to a vast performance boost when working on complex headers:)

From experimenting I found that include guards (not #pragma once) forces PCH's to be reconstructed and greatly increase the parse time of header files![...]

What you mean exactly that it forces PCH's to be reconstructed? How are you compiling them in the first place?
Seems really weird because i would actually expect them to make the PCH compile in the first place, due to the possible multiple inclusion issue.

When reusing a PCH, either internally through libclang, or an external PCH file, include guards forces a full reparse of all its header files when changes anywhere in the source is made.

When reusing a PCH, either internally through libclang, or an external PCH file, include guards forces a full reparse of all its header files when changes anywhere in the source is made.

I still don't get it or why it should happen... and why it shouldn't break when you remove the guards.
Something tells me that this is related to https://reviews.llvm.org/D15994.

Yes, the patch you link to seems to be related. Currently, we now remove these include guards when one is working on a header file in juCi++. The speed up is significant. My guess though is that people will be using #pragma once more and more as it has been supported quite a while by both gcc (since 3.4) and clang. Though it will take a while before the C++ literature is updated, even my own lectures and presentations for NTNU are outdated...

I'm slowly coming to think again that what i'm trying to do is too complicated with the tools i have.
I managed to simply have the clang preprocessor to pass on all files of a compilation database and execute some callbacks when it encounters ifs/includes, when it enters or exits from files and the like (http://clang.llvm.org/doxygen/classclang_1_1PPCallbacks.html).
In these callbacks i'm not currently doing anything and doing a single thread run on all the files takes ~300 seconds, which would mean ~50 seconds if i use my 6 cores, with 6 threads (again ~2800 files.. my target is still QtCreator which is ready in around 30 seconds), though all those seconds just to parse... and i would still have to actually store informations about defines i encounter.. build the include graph and compile/compose the PCHs.
So... i would gain some speedup if i was able with clang to avoid preprocessing headers that i've already preprocessed with the same identical set of defines... but there's no way to do that unless i reimplement some good parts of it, so that i can load from disk the internal status of the preprocessor as if it had already preprocessed those files.
Given that this means basically giving a good alternative to modules, and so partially solve the big issue in reparsing with normal compilations... it means modules aren't useful anymore and i'm a genius (sure...).
Joking aside and long story short i have no idea how to turn this into something simpler and speed up the whole thing enough.
I can already create PCH or even PTH if i want with the first source file (and all its dependency chain) i preprocess; the issue though is that for the next source file i know if a certain PCH/PTH X is good for the header Y i'm about to preprocess only after preprocessing it up to that header... and it would be already too late because i have to pass PCH/PTH to the preprocess invocation on the source file, so before starting to preprocess.

First, sorry for the confusion, I meant a precompiled preamble and not PCH in my last comments. They are basically the same, but libclang works with its own precompiled preamble, and you can combine this with a PCH that is passed to libclang when parsing a source. That is, when I said reparsing the PCH, I meant reparsing the precompiled preamble of a source file.
By the way, as I understand it, the precompiled preamble in libclang exists for faster reparsing of a source file, while PCH's exists for faster parsing of source files.
Thank you for investigating your solution further, I still think your idea is worth investigating. I have tested using PCH's passed to libclang and it's working quite ok. One possible issue with having many PCH's, however, is that parsing of all these PCH's might take some time.
I'll continue with my idea as well, that is working on a common precompiled preamble within libclang for the source files in question when finding function or variable usages. You would of course have the issue with conditional inclusions, but at least initially we can be working towards a 90-95% correctly working solution through simply adding the conditional inclusions when creating the precompiled preamble.
I would not worry too much about the processing time compared to for instance Qt Creator. They have an entirely different approach, which require a significant effort to make it work for newer C++ standards. And it will most likely never reach the quality of libclang processing.
And, of course, a libclang solution would work much faster when using C++17 modules:) Modules being basically a general approach to PCH's if I understand it correctly.


chore(release): update rbac-backend

Description
Update rbac-backend to the version 4.10.7
Which issue(s) does this PR fix

Fixes https://issues.redhat.com/browse/RHIDP-4069

PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

/retest

Works correctly 🧙♂

/retest

/test e2e-tests

/retest

/retest

/retest

/retest

/test e2e-tests

The image was broken, @ciiay fixed an issue with #1847, let see if this helps
/retest

/test e2e-tests

/test e2e-tests

/test e2e-tests

/retest


Need a privacy policy

As a user who didn't have time to examine the code, I think it's good to have statements about user data would be handled in webamp.org , since this web-based player can open user's files (music, equalizer presets or skins).

Is the music loaded into Webamp kept only in the browser's cache, or would it be uploaded to some servers?
Is the playlist handled the same way? (I.e. Will the server keep a copy of user's playlist?)
If foreign codecs are supported, will the codecs be run in the user's browser? Or will there be server-based decoding?

Not intended to be a long (and boring) legal document, but short statements in the web page that answers at least these questions for webamp.org server.

This is a fair point. I'll try to answer briefly here as a sketch of some more formal documentation:
Webamp is entirely a browser-based software project. There is no server component. That said, we do make use of some third-party services to gain insight into how the project is being used, and to ensure that it is working properly.

Your music/skin/eq presets never leave your browser.
Data about how users interact with the website, is logged to Google Analytics to help me understand how people are using it.
In the event that we encounter an error, a log report is sent to Sentry with a snapshot of the current state of the application. This includes metadata about the tracks in your playlist, but no actual media contents.

These are true of the demo site, which lives at webamp.org, but none of these third party integrations are embedded in the core library which is published to npm: https://www.npmjs.com/package/webamp

Update:
I've just removed the detailed analytics logging. We now only log when you visit and how long you stay (the default Google Analytics logging) rather than logging each interaction.
https://github.com/captbaritone/webamp/commit/967a4d4feac7ab1a910c03fb42e8ca1635fa8f26

Hey @durasj thanks for following up here.
I've added a commit to anonymize the GA ip, and found that in the Sentry UI I can turn off IP logging, so I've done that as well.

I can't seem to find any consensus as to what exactly they mean by "identifying information". I never intentionally establish any type of "identity" for a user. For now I feel like I've made a best effort to not log anything identifiable so until we as an industry learn more, I think this will do.
One thing that's not clear to me is what the privacy policy needs to include, and how prominent it needs to be, especially considering I don't ever intentionally identify the user in any way. Can I just include it in a context menu? I would really hate to add a prominent link to the extremely minimal UI that I currently have at webamp.org.

@captbaritone My suggestion was to put a small link titled "Privacy Info" or "Privacy Policy" on the bottom of the webamp.org page, similar to how Github and Twitter link have been done. When user clicks it, you can pop up a dialog which you can explain everything privacy related.
I don't know if you have a better idea.


Add monthly CWV Tech Report materialization query

The query to materialize the technologies table with the latest CWV Tech Report data should be run before the new pipeline, as those queries read from the table.

I've added a scheduled query with the date replaced the @run_date

product documentation
Scheduled query:  Core Web Vitals Technologies Report

Note: there is a bug with scheduled queries using @run_date with partitioned tables and required filters (issue tracker), so the scheduled query includes a hacky workaround not present in the original query.
This scheduled query is currently set to run on-demand in order to be triggered manually from some other source (e.g. API call) rather than on the first of the month, since the execution date may vary from the run date.

Testing as of 2024-03-10 after the most recent crawl completed for 2024-03-01 and using a run date of 2024-02-01. This produced no results since the new pipeline has not finished yet and upstream tables have not been updated in the all and summary_* datasets. This query may need adjustment if it needs to be run before the new pipeline.

Testing as of 2024-03-10 after the most recent crawl completed for 2024-03-01 and using a run date of 2024-02-01. This produced no results since the new pipeline has not finished yet and upstream tables have not been updated in the all and summary_* datasets. This query may need adjustment if it needs to be run before the new pipeline.

The crawl starts after the CrUX data is published on the second Tuesday of the month. So the all data is available for February, but March will only start this week. We also don't need the summary_* datasets for this report as it uses the all datasets (which is good as we're having trouble with that dataset as discussed!) The CrUX data is for the full month so won't show until April 9th (second Tuesday of the next month) and it's joined to the crawl data for this report.
I also went to run this manually this month (for February) but it says it'll process 337 TB, whereas the manual one say 18 TB so went with that for this month as quite a difference! It could be over estimating how much it'll take but 337 TB is a LOT so didn't want to take the chance. Thoughts?

The crawl starts after the CrUX data is published on the second Tuesday of the month.

Ok, perfect! The data isn't "delayed"; instead, I ran the query with dates too far into the future at that point in time.

We also don't need the summary_* datasets for this report as it uses the all datasets (which is good as we're having trouble with that dataset as discussed!)

That's good. The query mentioned in the issue description is written to use tables in the all dataset only. I mentioned the summary_* datasets as well just in case there may be a reason to use them as an alternative.

I also went to run this manually this month (for February) but it says it'll process 337 TB, whereas the manual one say 18 TB so went with that for this month as quite a difference! It could be over estimating how much it'll take but 337 TB is a LOT so didn't want to take the chance. Thoughts?

In my experience, the Google Cloud console's query validator is not very accurate, especially when working with scripts, partitioned tables, and filters. I ran this scheduled query during testing and it processed 17.27TB (rather than the estimated 337TB).
giancarlo_faranda@cloudshell:~ (httparchive)$ bq show --location=US --job httparchive:scheduled_query_65fd9c6e-0000-255a-8802-582429c39d0c
Job httparchive:scheduled_query_65fd9c6e-0000-255a-8802-582429c39d0c

  Job Type    State      Start Time         Duration              User Email            Bytes Processed    Bytes Billed    Billing Tier                       Labels
 ---------- --------- ----------------- ---------------- ----------------------------- ----------------- ---------------- -------------- -------------------------------------------------
  query      SUCCESS   10 Mar 22:41:23   0:07:59.888000<EMAIL_ADDRESS> 18990487562745<PHONE_NUMBER>4816                  dts_run_id:65fd9c6e-0000-255a-8802-582429c39d0c
                                                                                data_source_id:scheduled_query

OK cool. Will give it a go next month then. I presume it is NOT automated yet to kick off? Especially for this month since I've manually run it for this month.

@tunetheweb Correct, it is not automated yet but I think we trigger this from the same Pub/Sub subscription that initiates the crawl.
Is this the topic which signals the start of the crawl? projects/httparchive/topics/crux-updated (CC @pmeenan @rviscomi )

Closed in https://github.com/HTTPArchive/dataform/pull/3


feat: [CDS-91176]: add doc for delegate hotfix 82305

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
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 Screenshot.

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

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

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


Prevents MX & BR users from getting the onboarding flow

What's this PR do?
what the title says
How should this be reviewed?
do you get onboarding flow as US user?
what about MX/BR?
Any background context you want to provide?
nope
Relevant tickets
Fixes #6663
Checklist

[ ] Tested on staging.

@DFurnes updated, i think this should be ok

👍


Allow "Maybe" to be exported, so it can be imported into a implemented class

I am using  "implements" on the defined types, this is so I have better control over mapping using class-transformer (another library) which allows annotating over fields.
currently, it is
type Maybe<T> = T | undefined;

So when I implement a type, the fields using Maybe .. but I don't have access to Maybe as its not exported.
Would be nice to have a config setting to enable exporting or not, just as it has the types that it should use i.e. undefined | null . etc

Hi @appsolutegeek ,
Please use this configuration: https://graphql-code-generator.com/docs/plugins/typescript#maybevalue-string-default-value-t-null
generates:
   path/to/file.ts:
     plugins:
        - typescript
      config:
          maybeValue: T | undefined

Thanks @dotansimha, i am already using that config.. But the type isn't being exported i.e. its
type Maybe<T> = T | undefined;

and NOT
export type Maybe<T> = T | undefined;

All over types are being exported but Maybe is not..

@appsolutegeek you are right, I thought it does: https://github.com/dotansimha/graphql-code-generator/blob/master/packages/plugins/typescript/src/index.ts#L104

#1718

Fixed in 1.1.0 🚀


⚠ ClamAV has degraded performance

In d772dd9, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 56 ms

Resolved: ClamAV performance has improved in a275d0a.


quick test

Update arch display link
Upgrade hugo to v0.133.1
Fix a bunch of spellcheck errors
Add a push secret blog post
Clarify the explanation a bit
Clean up some syntax in Key Concepts
Fix multiple docs and testing problems
Align htmltest versions with homepage-container
Fix pattern filtering
docs: add EDB to the rag-llm-gitops validated pattern (#457)
Added Workshop to menu (#463)
TELCODOCS-1999 Documentation for VP OpenShift Framework
TELCODOCS-1999 Adding new section on frameworks
TELCODOCS-1999 Updating new section on frameworks
TELCODOCS-1999 UPdating validated pattern struture and moving from contribute to learn
TELCODOCS-1999 UPdating validated pattern struture and moving from contribute to learn 2
TELCODOCS-1999 UPdating validated pattern struture and moving from contribute to learn 3
TELCODOCS-1999 UPdating validated pattern struture and moving from contribute to learn 4
TELCODOCS-1999 UPdating validated pattern struture and moving from contribute to learn 5
TELCODOCS-1999 UPdating validated pattern struture and moving from contribute to learn 6
TELCODOCS-1999 UPdating validated pattern struture and moving from contribute to learn 7
TELCODOCS-1999 UPdating validated pattern struture and moving from contribute to learn 8
Updating based on peer review
adding line test to trigger update
updating based on peer review 2
Add /livereaload.js to htmltest ignore*
WIP quick test

This is an automated message:
You can preview this docs PR at http://467.docs-pr.validatedpatterns.io
Note that they get generated every five minutes, so please wait a bit.


chore: limit feature_flag method to ProjectSettingsService

Following up on  this discussion in #6485.
Currently the feature_flag decorator is a method defined on the SettingsService base class. However, since feature flags are top-level settings in a Meltano project and currently never defined in plugin settings, this has led to multiple cases of the method being used from a PluginSettingsService, where it will always show the given flag as not being set. To avoid this confusion, we should just move the method to ProjectSettingsService.
CC: @edgarrmondragon @kgpayne

FYI @tayloramurphy and @aaronsteers, for some reason I'm not seeing a weight field for this, but I'd weight this at a 2

@cjohnhanson - It sounds like you have a good grasp on this so I'm assigning back to you to pickup whenever you're able.
Do I understand the symptom correctly, that the feature flag works as expected, but debug prints will have incorrect/missing values and lead to incorrect assumptions about the feature flag's config'ed value?
cc @pnadolny13 as fyi

@aaronsteers Actually the symptom is more that the feature flag works as expected when used correctly in the code base, but the way it's written right now makes it easy to use "incorrectly". There's no impact to end users right now but any contributors that are using feature flags for a new feature are liable to introduce a bug by using the feature flag method "incorrectly". We had two regressions in 2.3.0 that were a result of such incorrect usage, so this is just a quick refactor to head off those types of mistake and enforce "correct" usage.

@cjohnhanson - It's been a while since we've touched or commented on this issue so just want to check in.
Is it okay if I remove the "Urgency=High" flag from this item on our Engineering board? If not, will talk with Taylor and try to prioritize in the next week or two.

@aaronsteers yeah I put this on the backburner since it's definitely a chore that doesn't have any immediate or short-term user impact. It should definitely have the Urgency=High flag removed.

This was actually resolved in https://github.com/meltano/meltano/pull/6971; all SettingsServices now have a project_settings_service attribute that's used for this. Closing.


Data numbers in Australia

•	Country/region affected: AU
•	Example number affected:<PHONE_NUMBER>
•	The phone number range(s) to which the issue applies: 0198x
•	The type of the number: Data number (fixed line)
•	The cost: Local rate
•	Supporting evidence: https://en.wikipedia.org/wiki/Telephone_numbers_in_Australia#Data_numbers_.280198.29
These numbers should be nationally formatted with a 0 on the front (NDP), as is usual in Australia, i.e. 0198x. Libphonenumber currently outputs national format without the 0 (198x), which is incorrect. The correct E.164 format is produced (+61198x). Also, the number<PHONE_NUMBER> is in use as the Dial-Up PoP number for iiNet, but is not recognised by libphonenumber as a valid number.

@beccabowen
Thanks for reporting this issue.
LibPhoneNumber API presently do not support data / internet access numbers.
The phonenumber library is currently aimed at cases where numbers can be dialed to reach a human endpoint.
We have no plans to support data numbers at the moment, but might in the future.


Feedback 4/5: Document supported parts of Cypher

Reference (partially done here)
Example queries

I am looking at the OpenCypher Reference and wanted to know what features are supported in CAPS right now.  Is there a document that talks about it ?
One particular feature is the support for multiple edge labels in the Match query - Is this supported ?

Hello @sgolecha and thanks for reaching out to us.
We do not yet have a comprehensive document on exactly what is supported, but once we get to that it will most likely end up in the Neo4j Morpheus User Guide. (Neo4j Morpheus is a commercial product built around CAPS.)
For your particular question: yes, we do support disjunctions on relationship types in pattern matches (although I can find no actual tests for this -- will add).

The supported level of Cypher 9 is now covered in this part of the Neo4j Morpheus User Guide: https://neo4j.com/docs/morpheus-user-guide/preview/caps/cypher/cypher9-features/
We will keep updating this as we proceed.


adding hparams as parameters during model initialization

WARNING: This is not backwards compatible with older model versions as it requires the model hyperparameters to be saved and loaded alongside the model.

Resolved comments. Also added functionality to save only the best model that is being trained. PTAL.


Move all Dotenv variables to Mix.Config

Resolves #111.
Features

There are now only two files that hold secrets,

config/secrets.exs, which holds secrets that are useful for development
config/prod.secrets.exs, which holds secrets that are useful only for production

Each of these files have a corresponding config/*.example file.
With this, the .env.example and Dotenv file and dependency has been removed.

The task digest has also been removed
The task server has been updated to remove its use of Dotenv

The interval for Cadet.Updater.Public has been decreased from 5 minutes to 1 minute.
The README and moduledocs has been updated accordingly.
Travis has been updated with an extra before_install step to avoid errors w.r.t. config/secrets.exs not existing for $ mix local.rebar.

Fixing tests, please hold.

Pull Request Test Coverage Report for Build 964

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 950:
0.0%

Covered Lines:
483

Relevant Lines:
483

💛  - Coveralls

Oh, and rebase on/merge from master, just added mix format --check-formatted to travis on master

Done and fixed.


Test with contracts deployed at fixed addressed.

Component
Forge
Describe the feature you would like
Hi team,
Are there any features that allow me to run the tests with contracts deployed at fixed addresses. So that my test files won't have those dependencies of the contract files. I do not want my test files re-compiled.
Thanks much.
Additional context
No response

I think you're looking for
https://book.getfoundry.sh/cheatcodes/etch

I think you're looking for
https://book.getfoundry.sh/cheatcodes/etch

Thanks but this is not my want. This is to read the contract's bytecode? For example, I want to deploy contract Foo at address(0x01), so that i can write IFoo interface in the test file. And I can write IFoo targetContract = IFoo(addres(0x01)); So the test contract might not need compiled again.

I want to deploy contract Foo at address(0x01)

maybe https://book.getfoundry.sh/reference/forge-std/deployCodeTo

/

I want to deploy contract Foo at address(0x01)

maybe https://book.getfoundry.sh/reference/forge-std/deployCodeTo

Yes, it's something like deployCodeTo. But the key is that the code of the contract is frequently changed, and i don't want the test file frequently compiled since as per the documentation, the test file has the dependencies of those codes.

Closing, ask unclear / seems compilation related in which we're limited on we can do, and issue is stale.


FrankerFaceZ (FFZ) emotes are not loading

nightly v1.0.717 built 2018-09-17T10:51:16.0000000Z

To clarify upon this issue: Global FFZ emotes work, but FFZ channel emotes only sometimes work. This could be due to rate-limiting, us consuming the API improperly. The API call could also be really delayed, so we might wanna look into setting a reasonable timeout on the request.

I did some analysis on this issue since it also affected me. I watched the network traffic in procmon to find out and behavioural differences between the case where it loads FFZ on startup successfully and the case where it fails to load.
A normal startup looks like this:

Note all the network requests are all within a short timespan of a few hundred milliseconds.
A startup which fails to load looks like this:

Note there is a break in the network requests, it seems like the second batch of requests is issued after something else happens in the application (logs are loaded maybe?)
Replicating this issue also seems to depend on your peer FFZ server to some degree. In my short testing I could only replicate the issue with IPv4 servers (IP is in the screenshot), whilst with the v6 servers my emotes always loaded fine. (Could just be a load/timing difference between them though, instead of an issue directly stemming from the difference between v4 and v6)
Raw Procmon captures are here:
https://cloud.zwb3.de/s/PKWLFgorRgqDEQK success.pml
https://cloud.zwb3.de/s/bPdgmwn6ioKpXSa failure.pml


Test erc 721

You can use the GitHub - starknet-edu/starknet-erc721: Learn how to deploy and customize an ERC721 token on StarkNet contracts

Codecov Report

Merging #204 (e57f7a3) into main (1ca883e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #204   +/-   ##
=======================================
  Coverage   73.64%   73.64%
=======================================
  Files          42       42
  Lines        2660     2660
=======================================
  Hits         1959     1959
  Misses        701      701

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Make plots for Figure 4

[ ] Series of multi-panel plots (time series plots)

[ ] Time series & absolute change (Fire-Base)?

[ ] Flow as a reference - Base case burned (Wenas fire scenario) and unburned

[ ] Climatic regimes, flow path, slope, land use - burned (Wenas fire scenario) and unburned

[ ] Nitrate, TSS, DOC: Burn severity (3 levels) and area (10 levels) → 10 plots with 3 curves each (or 3 plots with 10 curves each)

[ ] Summary metrics

[ ] Total export of NO3, DOC over the time period - integrate flow*concentration (mass flux) over time period

[ ] Flow as a reference - Base case burned (Wenas fire scenario) and unburned
[ ] Update means for each scenario
[ ] % change with base cases that change (update peter figure)
[ ] Difference in annual mean severity x area burnt
[ ] Scenarios that show thresholds/interesting patterns (that make sense environmentally)
[ ] conceptual figure relating meta analyses and modeling scenarios


Modify hook on verified status check

I was asked about what the verified column meant on the automations table, and I noticed a potential improvement where we wouldn't have to filter through all the sources every time we display the status, and instead just request the object we need directly (and only running the request if it's a Git repo or OCI repo in the first place). Is this a thing?

Hey Josh, this looks good to me! It's code moved from Backstage, should probably look into something similar there but we'll leave that for later 🤔 I think the VerifiableResource interface was initially built like that to be more extensible though you are correct atm we're looking at Git Repositories | OCI repositories only as having the potential to be verified so we can leave it in the new format.


Review grammar and spelling throughout the crate

Fixes grammar and spelling issues throughout the crate.

Thank you! I'm glad I could be of some help :)


Make lint failure warnings easier to understand

Problem

If the linting step fails in the CI, you get a message that "Lint step failed on line xx". This is very confusing, especially for new contributors.
Description

We should add a message saying what the user can do to fix the failure. Something like "Lint step failed. Please run just lint and commit the changes".
Alternatives

Add a comment to the PR with the same message, if the Linting step fails.
Additional context

I'm not sure if this message is produced by just script or something else. @dhruvkb, maybe you know what (and where) can be done to fix this issue?

If you are referring to the error message in log line 6302, i.e.
error: Recipe `lint` failed on line 92 with exit code 1

the message is reported by just because the actual lint recipe, defined at line 92,
https://github.com/WordPress/openverse/blob/361bc0d17d86cff9d8ce927cc0f7d62e6615afb5/justfile#L92
returned a non-zero exit code. To change the message into something helpful or more relevant, we can use the approach similar to the _loop recipe in the justfile
https://github.com/WordPress/openverse/blob/361bc0d17d86cff9d8ce927cc0f7d62e6615afb5/justfile#L38-L41
where we catch the exit code and echo an appropriate error message.

@dhruvkb Hello I'm a student in CMU and I have a project. I'm interested in working on this issue. Looks doable could do assign me?

Hey I'm interested in solving this issue. But as I'm new to open source contributions can you provide some more details to how to resolve this issue?


sql, kv: add sql.mutations.max_row_size guardrails

https://github.com/cockroachdb/cockroach/pull/67953 --- Release note (ops change): A new cluster setting, sql.mutations.max_row_size.err, was added, which limits the size of rows written to the database (or individual column families, if multiple column families are in use). Statements trying to write a row larger than this will fail with a code 54000 (program_limit_exceeded) error. (Internal queries writing a row larger than this will not fail, but will log a LargeRowInternal event to the SQL_INTERNAL_PERF channel.) This limit is enforced for INSERT, UPSERT, and UPDATE statements. CREATE TABLE AS, CREATE INDEX, ALTER TABLE, ALTER INDEX, IMPORT, and RESTORE will not fail with an error, but will log LargeRowInternal events to the SQL_INTERNAL_PERF channel. SELECT, DELETE, TRUNCATE, and DROP are not affected by this limit. Note that existing rows violating the limit cannot be updated, unless the update shrinks the size of the row below the limit, but can be selected, deleted, altered, backed-up, and restored. For this reason we recommend using the accompanying setting sql.mutations.max_row_size.log in conjunction with SELECT pg_column_size() queries to detect and fix any existing large rows before lowering sql.mutations.max_row_size.err.  Release justification: Low risk, high benefit change to existing functionality. This causes statements adding large rows to fail with an error. Default is 512 MiB, which was the maximum KV size in 20.2 as of https://github.com/cockroachdb/cockroach/pull/61818 and also the default range_max_bytes in 21.1, meaning rows larger than this were not possible in 20.2 and are not going to perform well in 21.1.

Note that this setting was renamed, as noted in #11164, and metrics were added, as noted in #11165.


Obsolete logging facility enum

That felt really good! Thanks for the leading PR ....

Whoops too soon ...

Not sure where this came from because I don't think we are ready for Windsor v5, however if you want to remove that much from the logging facility in v5 then we'll need other ways to actually configure the logger factory config file and log level because the logging facility currently creates the logger factory instance.
We'd have to change:
LogUsing<TLoggerFactory>(TLoggerFactory loggerFactory)

to:
LogUsing(ILoggerFactory loggerFactory)

Which means the logger factory can't automatically have dependencies resolved from the container. Obviously if this is the direction we'd want to take adding the new APIs now would be the way to go.
Would love to hear your thoughts here.


Create New Terminal in Editor Area not working on non-main windows

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.92.0, commit b1c0a14de1414fcdaa400695b4db1c0799bc3124
OS Version: Windows 10 Enterprise, 22H2 build 19045.4651

Error message:
The editor could not be opened due to an unexpected error: Not allowed to create elements in child window JavaScript context. Always use the main window so that "xyz instanceof HTMLElement" continues to work.
Steps to Reproduce:

Open a new VS code window

Create a new file and drag out to make another window

Run "Create New Terminal in Editor Area" on that new window

The error will show up

Clicking "Try Again" will reload the terminal, but it doesn't accept inputs, cannot type anything there

Creating terminal in main window and drag it to side windows does work, but I have to do that with every window reload.
Yesterday I could still use terminals in side windows like normal, this only occured after updating to 1.92 July update
Thanks for reading :)

coming from xterm here I think

I have the same problem on Fedora 40 Sway.
After I click Try Again, it will create a non-interoperable terminal.

I'm also having this issue, I thought it might be a problem with WSL but it started for me after updating to VSCode 1.92.0

QUICK FIX!!!
Just create a new In-window terminal and drag it to the new terminal window. 👍

Will this be hotfixed or should we downgrade?

@michaelschufi sorry this won't get a patch release as there's a few workarounds

Hi.

QUICK FIX!!!
Just create a new In-window terminal and drag it to the new terminal window. 👍

Didn't do the trick for me. Other hack: Ctrl+J (new term in main window) > Ctrl + Shift + P > Terminal: Move Into New Window > Drag & Drop into wanted window. Cumbersom many many times a day.