Adds more tests for untested services

Related Issue
#41 @joshwcomeau requested some tests to be written for the more important files/functions
Summary
Adding tests for some of the services/modules:

[x] Project Name Service
[x] Project Type Specifics

For the Project Name service, I had to export the prefixes and suffixes array to test against this list given the format we output. Let me know if you guys think this is an appropriate way to test this.
I did also add some tests for the create-project.service.js, but it doesn't test all of the contained functions (just the easy ones 😝). I did have to mock a few things, but maybe this mocking should live somewhere else, instead. WDYT?
There was a test that was failing for me locally in the dependencies.reducer.test.js, so I updated it. If that's not okay, I can revert that change.

This looks great! Thanks for adding these.
I'm requesting changes for a couple small things, described in the inline comments.

Thanks for the review! All of this is fixed 👍

Also, it looks like the file project-type-specifics.test.js isn't being transformed by Prettier - I see different quotes used, lines that are too long, etc. There's a pre-commit hook that should run Prettier, so it should be impossible for PRs to not be formatted :/ do you have any clue why it wasn't formatted?

It is possible to bypass them commit hooks by running git commit --no-verify, but IDK if that's why was that file didn't have Prettier run over it. I have made sure Prettier has been run over the file now though.

Hey @scottwarren, I just sent an invite to become a collaborator on Guppy :D I really appreciate your contributions to increasing our test coverage.
You can learn more about how we view collaborators in the Contributing docs. Please give it a quick read, and let me know if you have any questions :D
Thanks again for your contributions!

Thanks @joshwcomeau, I am glad to help!


Drops of events in windows

Iohook stopped in windows platform.
The scenario in detail:
Iohook is supported with electron js environment. But somehow It's not working in windows platform with the use of electron js. It's dropping events sometimes with random behavior. I tried to install VC_redistx64.exe. But, It'll fix the issue for a temporary basis.
Here's my environment detail:

nodejs version: 10.16.3
electron js version: 5.0.2
Iohook version: 0.6.5

Hey, thanks for opening an issue. Could you provide a minimum reproducible example ?

Hey @Djiit, I can share a demo. But, there is a random behavior. and major of this issue we're facing is in windows system.

@Djiit  I also found this problem. When I listen to keydown/keyup, sometimes I can't get events.
Only in windows os.

In my case,  I saw some users's windows will drop events or trigger wrong events.   I think all due to the protect software which  in user's pc.

Maybe it could help. If electron run on user right, but you make active an application's window with admin right (right mouse click run as administrator). You couldn't get events.

Maybe it could help. If electron run on user right, but you make active an application's window with admin right (right mouse click run as administrator). You couldn't get events.

I think i had found the reason! libuiohook listen a event names EVENT_OBJECT_NAMECHANGE on windows. when the event is triggered, libuiohook will unhook all listener and rehook it.
when the event emited after unhook and before rehook, it will lost!
I am try to remove the code of SetWinEventHook on EVENT_OBJECT_NAMECHANGE. it looks good!
I found the commit of this changed on it: https://github.com/kwhat/libuiohook/pull/14

I think i had found the reason! libuiohook listen a event names EVENT_OBJECT_NAMECHANGE on windows. when the event is triggered, libuiohook will unhook all listener and rehook it.
when the event emited after unhook and before rehook, it will lost!
I am try to remove the code of SetWinEventHook on EVENT_OBJECT_NAMECHANGE. it looks good!
I found the commit of this changed on it: https://github.com/kwhat/libuiohook/pull/14

Is it currently an issue or was it fixed? Is it something that was resolved upstream?

@ash0x0 Yes, It's still coming up in the windows system.

@ash0x0 Yes, It's still coming up in the windows system.

Is there a pattern to this that you noticed or is it completely random? I haven't been able to repro, does it happen after some time or right away? With certain applications of after certain events?

@ash0x0 It's completely random. It's most probably occurs when the app is freshly installed into the system.

I had fixed this bug on my repo. But it is a temporary modification because only remove the logic of SetWinEventHook. I think it's a better way to do a configuration. But I am not familiar with this. If you know how to do, please tell me.
https://github.com/TLeaves/iohook/commit/226ce6c27243337075050002b641c13232077d96


Fixed the name of the util function

Description of change
The documentation did not much the code.
Pre-Merge Checklist

[x] Documentation is changed or added

Codecov Report

Merging #6019 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #6019   +/-   ##
=======================================
  Coverage   70.54%   70.54%
=======================================
  Files         208      208
  Lines       10533    10533
=======================================
  Hits         7430     7430
  Misses       3103     3103

Impacted Files
Coverage Δ

packages/utils/src/data/pow2.js
5.55% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Add version bytes consts

BIP-32 defines 4 4-byte consts used as version bytes; currently we are hardcoding the version bytes in multiple places.
Add BIP-32 version bytes consts and use them throughout the module.

LGTM


Release 1.1.0

Updating to use versions 10.3.0 of the iOS and android SDKs, and removed iOS framework from inside project

@kbrdthenerd, Will this release include a fix for https://forum.intercom.com/s/question/0D52G00005ByNX7SAN/react-native-android-crash by any chance?


feature: allow disabling update checks in UI

Feature scope
Web UI
Description
See https://github.com/meltano/meltano/issues/6750#issuecomment-1247116799
Figured I'd file a separate issue for it.
Disabling the update check in the UI while still displaying the current version.
PR to follow shortly.

Thank you for the issue @dingobar. As you've hopefully heard by now, the Meltano UI has been deprecated, and is scheduled for removal in Meltano 3.0. Since no additional work/improvements for the UI are planned leading up to its removal, I'll be closing this issue.


svtplay.se issues

Since a couple of days, downloads from svtplay.se (for example https://www.svtplay.se/video/27050573/mississippi-grind?start=auto) fails with the error message "Unable to extract video id".

Duplicate of #26428


About us page added

Description
About us page added with linkedin icon in footer
Fixes #41
Type of Change:

Code
Quality Assurance
User Interface

Checklist:

My PR follows the style guidelines of this project

I have performed a self-review of my own code or materials

I have made corresponding changes to the documentation

@aashimgarg please have a look.

@Bindusar1212 you can do the following changes:-

reduce the space between the op of the age and the heading.
change the font.
(take reference from the month page).

@aashimgarg check this...Is it gud??

@aashimgarg check this...Is it gud??

its good but try to to keep the font uniform throughout the page!

@aashimgarg check this


Implement explain for aggregate command

What should be done?

https://www.mongodb.com/docs/v6.0/reference/command/aggregate/#command-fields
https://www.mongodb.com/docs/v6.0/reference/explain-results/#mongodb-data-explain.queryPlanner.optimizedPipeline

I think we did that; let's check

No we didn't I just checked on main branch.

Interesting – at least a part of the code for that is already there, but the command's parameter is unhandled, yeah.


Execute Ansible playbook

This PR add the possibility to execute an Ansible playbook in the k4e-worker-device.
The goal is to reproduce the behavior of rhc-worker-playbook (https://github.com/RedHatInsights/rhc-worker-playbook).

We need to make sure ansible is part of our blueprint.

@pkliczewski

It would be great if you could think about end to end tests we can implement once CI is done.

Please add end to end test in separate PR. @machacekondra can help with it

I created a new jira story to handle that. Is it ok?

We need to make sure ansible is part of our blueprint.

Created a new Jira story for it.

/lgtm

pls update PR message

This PR add the possibility to execute an Ansible playbook in the k4e-worker-device.

to

This PR add the possibility to execute an Ansible playbook in the flotta-worker-device.

@gciavarrini please rebase

LGTM, let's wait on @jakub-dzon  feedback on this one, but I'm happy to merge it as it is:
https://github.com/project-flotta/flotta-device-worker/pull/69#discussion_r830130274

:eyes:

:tada:


Add missing UCI go options

Codecov Report

Merging #92 (235e852) into main (c73a128) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #92      +/-   ##
==========================================
+ Coverage   96.49%   96.50%   +0.01%
==========================================
  Files          66       66
  Lines        6538     6558      +20
==========================================
+ Hits         6309     6329      +20
  Misses        229      229

Impacted Files
Coverage Δ

uci/src/uci_in/go.rs
100.00% <100.00%> (ø)

uci/tests/uci.rs
99.03% <100.00%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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


Certificate Checking Recently Broke

A recent update to FreeRDP broke the checking of SSL certificates used by RDP (at least those signed by Starfield Technologies, Inc.). With the new version, connecting to these servers returns:
[19:49:36:178] [32096:32097] [WARN][com.freerdp.crypto] - Certificate verification failure 'certificate rejected (28)' at stack position 2
[19:49:36:178] [32096:32097] [WARN][com.freerdp.crypto] - C = US, ST = Arizona, L = Scottsdale, O = "Starfield Technologies, Inc.", CN = Starfield Root Certificate Authority - G2
Certificate details for ..com:3389 (RDP-Server):
Common Name: ..com
Subject:     OU = Domain Control Validated, CN = ..com
Issuer:      C = US, ST = Arizona, L = Scottsdale, O = "Starfield Technologies, Inc.", OU = http://certs.starfieldtech.com/repository/, CN = Starfield Secure Certificate Authority - G2
Thumbprint:  68:3a::39:0c
The above X.509 certificate could not be verified, possibly because you do not have
the CA certificate in your certificate store, or the certificate has expired.
Please look at the OpenSSL documentation on how to add a private CA to the store.
Up until a few of days ago, this was working. This computer is openSUSE Leap 15.1, running FreeRDP version 3.0.0-dev (27f2eb13c). I have another nearly identical computer that I have not updated FreeRDP on recently - it is still running FreeRDP version 2.0.0-dev5 (07605b028) - and it connects without any certificate warning.

@KenHolloway the only change that happened is X509_PURPOSE_SSL_SERVER to X509_PURPOSE_ANY in X509_STORE_CTX_set_purpose file libfreerdp/crypto/crypto.c
Could you verify this is the cause?

could you also add the X509v3 elements in the output of openssl x509 -in yourcert.pem -text?

I changed X509_PURPOSE_ANY back to X509_PURPOSE_SSL_SERVER, recompiled, and it is happy with the certificates again. Thank you for your quick response, and all you do with FreeRDP!
The X509x3 extensions on the certificate are:
X509v3 extensions:
X509v3 Basic Constraints: critical
CA:FALSE
X509v3 Extended Key Usage:
TLS Web Server Authentication, TLS Web Client Authentication
X509v3 Key Usage: critical
Digital Signature, Key Encipherment
X509v3 CRL Distribution Points:
            Full Name:
              URI:http://crl.starfieldtech.com/sfig2s1-141.crl

        X509v3 Certificate Policies:
            Policy: 2.16.840.1.1144<IP_ADDRESS>.1
              CPS: http://certificates.starfieldtech.com/repository/
            Policy: <IP_ADDRESS>.2.1

        Authority Information Access:
            OCSP - URI:http://ocsp.starfieldtech.com/
            CA Issuers - URI:http://certificates.starfieldtech.com/repository/sfig2.crt

        X509v3 Authority Key Identifier:
            keyid:25:45:81:68:50:26:38:3D:3B:2D:2C:BE:CD:6A:D9:B6:3D:B3:66:63

Let me know if there is any more information that would be helpful.

@KenHolloway If you remove the function altogether, does it work then? (aka is the check for purpose skipped?)
(the openssl docs are not entirely clear or I couldn´t find the location for that)

@KenHolloway also what does the verification with openssl binary tell you?

When I comment out "X509_STORE_CTX_set_purpose(csc, X509_PURPOSE_SSL_SERVER);", it goes back to not trusting the certificate.
"openssl s_client -connect RDS..com:443" seems to be perfectly happy with the certificate:
CONNECTED(00000003)
depth=2 C = US, ST = Arizona, L = Scottsdale, O = "Starfield Technologies, Inc.", CN = Starfield Root Certificate Authority - G2
verify return:1
depth=1 C = US, ST = Arizona, L = Scottsdale, O = "Starfield Technologies, Inc.", OU = http://certs.starfieldtech.com/repository/, CN = Starfield Secure Certificate Authority - G2
verify return:1
depth=0 OU = Domain Control Validated, CN = *..com
verify return:1
Certificate chain
0 s:/OU=Domain Control Validated/CN=*..com
i:/C=US/ST=Arizona/L=Scottsdale/O=Starfield Technologies, Inc./OU=http://certs.starfieldtech.com/repository//CN=Starfield Secure Certificate Authority - G2
1 s:/C=US/ST=Arizona/L=Scottsdale/O=Starfield Technologies, Inc./OU=http://certs.starfieldtech.com/repository//CN=Starfield Secure Certificate Authority - G2
i:/C=US/ST=Arizona/L=Scottsdale/O=Starfield Technologies, Inc./CN=Starfield Root Certificate Authority - G2


Fixed temporary file bug in Google Cloud api function

There was an IOError reading the temporary file. Fixed the bug using that "delete" argument. Please check.

Hi @asshatter,
Sorry, missed this PR initially. It seems like this patch uses a delete=False kwarg, which doesn't actually exist. What is the IOError issue you were experiencing?

Seems like this issue is no longer present, at least in my testing. Closing, but feel free to reopen if the issue persists!


Enhance README.md with Emojis for Clarity

Issue Description:
I would like to contribute to this repository by enhancing the README.md for improved clarity and aesthetics. My suggestion is to add emojis to the existing headings in the README to make it more visually appealing and user-friendly.
Proposed Emojis for Headings:

Development Process 🚧
Setting up dev environment 🛠
Why Vanilla Prettier ? 🤔
Repository Structure 📁
Development workflow 🔄
Dependencies 📦

I believe that adding emojis to the headings will not only make the documentation more engaging but also help readers quickly identify and understand the content of each section.
Please assign this issue to me for the hacktoberfest. :)

:)

Can I be assigned for this issue

FYI: I /used to/ work at Coil (now closed, but where this originated)
I forwarded this on the interledger folk for increased visibility
Thanks.

@SOBANEJAZ We aren't looking to improve this README at the moment. You can pick open issues in other repositories.


Dart2's ResizeObserver being broken by Dart minifier

When using ResizeObserver in Dart2 and compiling for release (i'm assuming it uses dart2js), it stops working. Have reproduced this issue in Dart 2.7.
ResizeObserver works when doing a regular webdev serve:
pub global run webdev serve web:9999

ResizeObserver stops working when building for release:
pub global run webdev build --output=web:build --release

Test HTML file to reproduce issue:
<!DOCTYPE html>

    <meta charset="UTF-8">
    <title>Title</title>

        <textarea style="padding: 10px; margin: 10px;">
            TESTING
        </textarea>

        .

Test Dart file to reproduce issue:
import "dart:html";

void main() {
  final DivElement div1 = querySelector("div.div1");
  final DivElement div2 = querySelector("div.div2");

  final ResizeObserver divResizeObserver = ResizeObserver((List<ResizeObserverEntry> blah1, ResizeObserver blah2) {
    print("inside divResizeObserverCallback");
    for (ResizeObserverEntry entry in blah1) {
        div2.style.height = "${entry.target.clientHeight}px";
    }
  });
  divResizeObserver.observe(div1);
}

When doing a regular webdev serve, it works fine, when you do a build with release mode, the ResizeObserver stops working. It would seem that minification is breaking it.
DevMode working (resizing text area and the blue div resizes as well):

Release Mode not working (resizing text area and the blue div does nothing and errors appear in the console):

I've documented in a lot of detail the problem here and all the workarounds i've tried without success:
https://stackoverflow.com/questions/60620940/dart2s-resizeobserver-being-broken-by-dart-minifier

Thanks for the detailed bug report!
I believe this is the same issue as https://github.com/dart-lang/sdk/issues/36798, which we were able to fix with 16bc1806ea921ca5f6fdbab22fe4ee437961642f
You should expect it on the next stable release of the SDK (it is currenlty available in the dev channel version 2.8.0-dev.1.0).

I've just tested this with 2.8.0-dev.13.0 and it's still giving the same error.
Had to change the Dart code though seeing as the list is now List<dynamic>
import "dart:html";

void main() {
  final DivElement div1 = querySelector("div.div1");
  final DivElement div2 = querySelector("div.div2");

  final void Function(List blah1, ResizeObserver blah2) callback = (blah1, blah2) {
    print("inside divResizeObserverCallback");
    for (ResizeObserverEntry entry in blah1) {
      div2.style.height = "${entry.target.clientHeight}px";
    }
  };
  final ResizeObserver divResizeObserver = ResizeObserver(callback);
  divResizeObserver.observe(div1);
}

It seems the error is now with the ResizeObserver inside the loop and no longer the callback itself
import "dart:html";
import "dart:js";

void main() {
  final DivElement div1 = querySelector("div.div1");
  final DivElement div2 = querySelector("div.div2");

  final void Function(List blah1, ResizeObserver blah2) callback = (blah1, blah2) {
    print("inside divResizeObserverCallback");
    blah1.forEach((entry) {
      context["console"].callMethod("log", [entry]);
      print((entry as ResizeObserverEntry).target);
      div2.style.height = "${entry.target.clientHeight}px";
    });
  };
  final ResizeObserver divResizeObserver = ResizeObserver(callback);
  divResizeObserver.observe(div1);
}

it fails at      print((entry as ResizeObserverEntry).target); with a casting error and if i don't cast it, it fails as well, although it works in Dev mode.

Logged a separate issue for ResizeObserverEntry:
https://github.com/dart-lang/sdk/issues/40975


Version catalog accessors for plugin aliases shown as errors in IDE kotlin script editor

Expected Behavior
No error should be shown in the IDE kotlin script editor.
Current Behavior

Workaround
You can install the Gradle Libs Error Suppressor IntelliJ Plugin or add the @Suppress("DSL_SCOPE_VIOLATION") annotation as follows:

Context
This was reported on the Kotlin IntelliJ plugin bug tracker first:

KTIJ-19369 - False-positive "can't be called in this context by implicit receiver" with plugins in Gradle version catalogs as a TOML file

It is in fact a Gradle issue.
Version catalog accessors are available as a Project extension, e.g. libs, outside of the scope of the plugins {} block. This is exactly what the IDE correctly reports.
It works in the Groovy DSL because there the version catalogs are explicitly made available via ScriptTarget.getPluginsBlockPermits(). This was not done for the Kotlin DSL.
This was not spotted when adding support for version catalog plugin aliases in the plugins {} block because the template class used for the evaluation of the Kotlin DSL plugins block does not restrict access to the Project in CompiledKotlinBuildscriptAndPluginsBlock (not using the PluginDependenciesSpecScope and has @ImplicitReceiver(Project::class)).
This is asymetrical with CompiledKotlinSettingsPluginManagementBlock that correctly applies the limitation using PluginDependenciesSpecScope which is annotated with @GradleDsl but still it has @ImplicitReceiver(Settings::class). This is an old mistake. This should be fixed. IOW version catalogs in the Kotlin DSL plugins block work by accident.
Once fixed it will break usage of version catalog plugin aliases from the Project extension in the plugins {} block for the Kotlin DSL. In order to make it work again, version catalog accessors should be made available in the plugins {} block.
There are of course complications because, although the Groovy DSL applies limitations so that only <catalogName>.plugins.* is allowed, there's no such limitation on the Kotlin DSL. The Groovy DSL limitations capture the original intent for the feature but adding the same limitation on the Kotlin DSL to a stable feature is a breaking change. That can only happen on a major Gradle version. It could be introduced behind a feature flag in a minor version though.
Steps to Reproduce

clone https://github.com/eskatos/gradle-kotlin-dsl-version-catalog-plugins-issue
open in IDE
open the build.gradle.kts file

The issue is in the backlog of the relevant team and prioritized by them.

For SEO (images are not searchable).
The error message is:

'val Project.libs: LibrariesForLibs' can't be called in this context by implicit receiver. Use the explicit one if necessary

for this code:
plugins {
    alias(libs.plugins.kotlin.jvm)
}

Workaround:
@Suppress("DSL_SCOPE_VIOLATION")
plugins {
    alias(libs.plugins.kotlin.jvm)
}

Any updates here?

@liutikas, this issue is assigned to 8.1 RC1 and associated with a PR already :)

This has been fixed by

https://github.com/gradle/gradle/pull/23639

This will ship in Gradle 8.1.
You can give it a try starting with 8.1-20230208002420+0000. You might need to clear IDEA caches and sync your build to see the full effect of the fix.

@eskatos would you mind summarizing the problem's root cause and the fix in 2-3 sentences? I'm really curious how it was working in command line, but not in IDE, yet the fix had to be done in Gradle.

It was working in both scenarios, this error did not prevent you from using your project in the IDE, it just made your plugins block show errors which did not prevent budling.

@TWiStErRob please see this issue description and the PR description and commit messages

Thanks Paul, some gold in the commit messages, sorry I didn't notice that.
@Renari in an IDE "red, that otherwise executes just fine" is not considered "working". False positives problem reports really kill confidence in any tool.

@TWiStErRob Saying that something isn't working implies that it's dysfunctional, in this scenario. From Gradles perspective everything was functional, while that was a technicality the end result was the same so it was "working", that is producing the desired result despite the raised issue. Usually when someone asks if something is working they're looking for the big picture, you don't tell someone the water plant isn't working because of issues with a single valve.

@Renari I believe @TWiStErRob is criticizing the priority given to this bug not questioning if it was blocking or not.
And I've to agree with them.
Having an error reported when there isn't one is not a minor issue. It isn't worst than something breaking, but it is right next in priority. And this one had an huge visibility / impact on users.
I lost half day chasing down this bug to than know it was nothing to worry about and suppressing it, and this was 1 year ago when I migrated to version catalog. You can imagine how much time people lost with this in the last year. And it surely hurt really hard the adoption of that feature. It also scare away early adopters for trying new features when they come.
That said, thanks for fixing it. I just hope you'll have an internal discussion on how you chose priority bugs.

Let's just step back and appreciate that we are able to make a living off of a free build system which Gradle provides to us.

I think it also was not clear from the beginning that this had to be fixed in Gradle. As it is building fine, assumption probably was, and imho rightfully, that the needs to be fixed in IntelliJ. Until JetBrains investigated the issue and told, that it needs a fix inside Gradle.

Yeah, I followed it through the lifecycle as @Vampire said, there were a 2 or 3 issues with Version Catalogs, and some of them were actually fixed by JetBrains, this wasn't really clear where it fits. I wasn't criticizing the priority in Gradle, in the end they fixed in in 3 months after realizing it belongs here (and that includes the holiday season, so 👏).
I was trying to explain my view that "red" in IDE is synonymous to breaking, because it's stops us from doing our job, while trying to understand an error that doesn't exists. Luckily only temporarily. One of the IDE's job is to show the problems that exists in our code at the right place at the right time (hence "integrated"). There was no issue in our code here, as it's clear from the OP: Gradle actually allowed more than it should have, but IDEA was flagging it as "wrong".
This has all kinds of psychological effects, and I definitely agree it hurt the adoption of Version Catalogs. Build systems in general don't have a good reputation, few quotes from yesterday from a colleague: "Sick and tired of gradle", "What is the hell??? How come???", "Why is it so unstable and unpredictable? Can't Gradle handle such a simple task?"; not related to this issue in particular, but this just emphasizes that adding false positives on top of the already complex build system is just going to hurt more.
@eskatos thank you very much for the fix again! Could you please make sure to put a section in the 8.1 upgrade guide to let people know that they can remove "the suppression" once their wrapper is set to 8.1+?

@TWiStErRob, good point
Please take a look

https://github.com/gradle/gradle/pull/23832

Yes, I'm sorry my comment come out harsher than I intended. I appreciate Gradle and that this was fixed.

@eskatos I'm still experiencing errors like this when using the Tooling API even with version 8.1-20230208002420+0000. Is that a separate issue or should it be fixed at that version? Does the consumer Gradle version need to be updated as well?
I have a reproducer here. The error can be replicated via ./gradlew :server:run.
        ProjectConnection gradleConnection = GradleConnector.newConnector()
            .forProjectDirectory(Paths.get(".").toFile())
            .useGradleVersion("8.1-20230208002420+0000")
            .connect();

        BasicGradleProject projectModel = gradleConnection.getModel(BasicGradleProject.class);

===>
org.gradle.tooling.BuildException: Could not fetch model of type 'BasicGradleProject' using connection to Gradle distribution 'https://services.gradle.org/distributions-snapshots/gradle-8.1-20230208002420+0000-bin.zip'.

Hi @DavidGregory084,
This issue is unrelated to the problem you are experiencing.
Thank you for opening https://github.com/gradle/gradle/issues/24046

Just adding in case someone comes across an expression expected error after implementing this fix. Change suppress annotation to @file:Suppress("DSL_SCOPE_VIOLATION") instead of @Suppress("DSL_SCOPE_VIOLATION")

@Shayne3000 just update your Gradle version and remove the suppression, it is fixed now. ;-)

@Shayne3000 just update your Gradle version and remove the suppression, it is fixed now. ;-)

Not for me with latest version

Did you refresh the Gradle project?

Yes, I actually use it in android studio

Did you refresh the Gradle project?

Of course, the problem is not for alias, but the same problem still exists.

If that's valid Gradle, it's probably worth a separate issue.

You said you updated AGP to 8.1.1, but that is pretty irrelevant.
The important question is, did you update Gradle to 8.1+ in which this issue is fixed?

Thank you, indeed I mixed them up. After upgrading to Gradle 8.3 the errors are gone 👍 (Android Studio : File > Project Structure > Project > Gradle Version)

@Monabr You should use the latest Gradle release version (8.6) when testing

That has nothing to do with this issue though.
This was about a similar error you got when legally using the libs.plugins... in the plugins { ... } block.
You get it because you illegally use rootProject within the plugins { ... } block.


Support for custom time metrics

Hi, recently I needed to load test an elasticsearch cluster and for what it's worth, I wasn't interested in connection time, only in the work time spent by the ES server. Elasticsearch returns that data as part of a JSON response, under a took key. I wanted to measure that, so I hacked this:
if response.status_code not in [200, 300, 301, 302, 303]:
    response.failure("Status different from 200")
else:
    resp = json.loads(response.content)
    events.request_success.fire(
        request_type=response.locust_request_meta["method"],
        name=response.locust_request_meta["name"],
        response_time=resp['took'],
        response_length=response.locust_request_meta["content_size"],
    )
    response._is_reported = True

I think it could be wrapped in something much nicer, e.g.:
if response.status_code not in [200, 300, 301, 302, 303]:
    reporting.record_failure(response)
else:
    reporting.record_success(response, took=10)

I was interested if anyone else is interested in being added to locust

Hi!
Rather than modifying Locust, I would probably write a small custom client (which wraps python-requests or some other HTTP client) that fires the request_success & request_failure events.
I'm not sure I understand what you're proposing. Is it the API for fire:ing request_success/request_failure that you think could be simplified?

I think I understand your approach better now. I think you lack good docs on how one should approach doing such custom stuff, though I've got the idea now how to port my code. Can I contribute it as an example?

@mdomans yes, definitely!


Enable or Disable Ldap based in configuration

¿It is possible to disable adldap based on configuration?
My app is on-premise and each client differs in its requirements. Some of them does not use ldap and I will like to "disable" it from autoregistering-connect based in configuration.
I did try but since it is in some way "merged" with the auth system I couldn't find a way.

Hi @Surt,
I'm not sure I fully understand your question - but I'll do my best to help.
You can disable the auto_connect option for your configured LDAP connection via run-time if you call Config::set('ldap.connections.default.auto_connect', false).
Or you can simply create another authentication guard in your config/auth.php file named ldap and then call upon it if it is enabled in your application:
'guards' => [
    'web' => [
        'driver' => 'session',
        'provider' => 'users',
    ],

    'ldap' => [
        'driver' => 'session',
        'provider' => 'ldap',
    ],

    'api' => [
        'driver' => 'token',
        'provider' => 'users',
    ],
],

'providers' => [
    'users' => [
        'driver' => 'database',
        'model' => App\User::class,
    ],

    'ldap' => [
        'driver' => 'ldap',
        'model' => App\User::class,
    ],
],

Then when authenticating:
// Somehow check if LDAP authentication is enabled in your application
// This is simply an example.
if (Config::get('settings.ldap_enabled', false)) {
    if (Auth::guard('ldap')->attempt($credentials)) {
        // Authenticated via LDAP.
    }
} else {
        if (Auth::guard('web')->attempt($credentials)) {
        // Authenticated via regular local database account.
    }
}

Let me know if you need any further help.


Finished ticket

Only selected roles may not be shoved.

Noticed side bug which I wont fix, when an creep is in the way it cant be moved so creates more issues then it fixes


fix(segment): scrollable segments only show scrollbar if they overflow

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] Build (npm run build) was run locally and any changes were pushed
[ ] Lint (npm run lint) has passed locally and any fixes were made for failures

Pull request type
Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?
A scrollable segment shows a horizontal scrollbar (e. g. on Windows) even if there is no overflow.
Issue Number: #20758
What is the new behavior?
A scrollable segment only shows a scrollbar if it overflows horizontally.
Does this introduce a breaking change?

[ ] Yes
[x] No

Other information
Closes #20758.

Merged. Thank you!


Manage staff - Bookable yes/no

How to change bookable attribute under Manage staff?
Please add this question to FAQ.
According to answer of Arun Satyarth<EMAIL_ADDRESS>When the staff was added, he would have received an email for accepting or rejecting the request. They can open the same email and select "Reject" . This will make him non bookable. We currently do not have a way to toggle this from inside the app.

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

ID: f7484dd4-5d94-dd6c-0dbe-551c852677c1
Version Independent ID: 257d60e2-9313-e08b-dccf-120cb4b30f5b
Content: Microsoft Bookings Frequently Asked Questions
Content Source: microsoft-365/bookings/bookings-faq.md
Service: bookings
GitHub Login: @kwekuako
Microsoft Alias: kwekua

@trudolf-msft Thank you for the feedback. We have added this to the FAW section via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/3688
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @trudolf-msft Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@msbemba Thank you very much for the contribution and updating the documentation with PR. @trudolf-msft Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


API is ingoring limiter parameter

All routes are limited to 50 results regardless of what you make the limiter(provided the limiter is greater than 50), so some results are missing from the All items, characters, dungeons etc route, but can be found by searching for them directly by name.
I think the issue is in the utils file, lines 1-12 https://github.com/deliton/zelda-api/tree/master/utils
https://zelda.fanapis.com/api/items?limit=100 > does nothing always shows only 50 results. It's capped.
https://zelda.fanapis.com/api/items?limit=20 > shows the correct number of results.

Turns out I'm still learning a lot about APIs. Apologies.


Command quality of life

As of 1.12.2-6.1.BETA26, when making a modpack in single player, I use the /thaumcraft command to unlock all researches. When this happens:
1- I'm getting spammed with unlock notifications in the top right corner
2- recipes won't show up in JEI until I reload the world
On a related suggestion, when typing just /thaumcraft:
3- we're getting a message to type /thaumcraft help.
I suggest to:
1- only show the first few notifications, then drop it
2- trigger recipes without having to reconnect
3- directly give the help message instead of asking player to type the command again.

That timing, I just ran into this too. I would suggest that a single special notification be given.


Added a "maskedInput" function

Added a function+test to apply a formatting mask to a given string (similar to http://digitalbush.com/projects/masked-input-plugin)

Coverage increased (+0.02%) to 97.527% when pulling 07667fc393596f7b1e4ca23781c9683fb461413a on e59:mask-string into 8d4e335434724e699469350e3cf49017992511c8 on brandonwamboldt:master.


How to start

Brother how to start Jarvis in termux

soo how do you start it???

Solution.txt
I think this will help you

Brother how to start Jarvis in termux

@Cybercrimi


make task  tar all needed files for works on nointernet pc

make quickstart is simple but convenient, I think somebody need another task
that  tar all config and  package files for move doom to a PC without internet connect.

Seems to me that you could just run git archive -o backup.tar and copy the tarball to wherever, not sure it really needs to be a make task provided by doom. If you want it to backup private configs or customizations too, that's going to be too prone to failure, since people may symlink things into their config directory, or reference external files, etc. I would recommend building something based on your own needs, whether that's a custom backup script or maybe something  built on GNU Stow, or something else entirely.

This is currently beyond the scope of Doom's makefile. This may change around 2.1.6, but for the time being there are no plans to do this, and I'd rather not crowd Doom's makefile.
If it's something you expect to do often, I suggest creating a ~/.doom.d/bin directory, putting it into your PATH and adding a backup script there.


[YUNIKORN-2002] Adding Chinese translation of K8s Scheduler Plugin

What is this PR for?
Adding Chinese translation of K8s Scheduler Plugin
What type of PR is it?

[ ] - Bug Fix
[ ] - Improvement
[ ] - Feature
[x] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

What is the Jira issue?
https://issues.apache.org/jira/projects/YUNIKORN/issues/YUNIKORN-2002

@wusamzong Thanks for the review and suggestions


[GHSA-hxwp-5hw8-g4xg] Vulnerability in OpenGrok (component: Web App). Versions...

Updates

Affected products
CVSS
References
Summary

Comments
Add three patch commit links: https://github.com/oracle/opengrok/pull/3528/commits/ff2b8efab6a844933bd963f4121997afd4009545
https://github.com/oracle/opengrok/pull/3528/commits/e0f6c20ca12f4a1f5a80095a4cb7fa7b85d1602c
https://github.com/oracle/opengrok/pull/3528/commits/601c283179d97542df011f92509fdc204a90b04b
And the corresponding pr: https://github.com/oracle/opengrok/pull/3528
The rationale is that RedHat has mentioned it in the reference link: https://www.oracle.com/security-alerts/oracle-open-source-cves-outside-other-oracle-public-documents.html.

Hi @MarkLee131, OpenGrok appears to not be in a supported ecosystem. I checked the modern Maven repository as well as an archived version from 2021 and, despite the GitHub repo having a pom.xml file, could not find opengrok anywhere.


58 3.16 personal email refactoring

Clean Up Email Controller

Renamed template_status to unify it with the application_status
Only use one specific status
Move EventUpdates into own method to clean up Send method
Move ICAL Attachement generation from the email to the event

Coverage remained the same at 100.0% when pulling ee8b76d381d233b3a75ce2029b4ba55031b81df7 on 58_3.16_PersonalEmail_Refactoring into 07d273b8199cad45af9f12780a2a3f3a4055af85 on dev.

@TimBasel can you explain why you closed this?

It was a duplicate, because at that time I couldn't fix the merge conflicts. https://github.com/hpi-swt2/workshop-portal/pull/622 is the now updated version.


🚸 Do not show notebook package version information if no uid is present

First line in logging is not necessary

Also it's too much text in general and it'd be nice if it was spread across two lines for easier copy & paste


Release v0.11

with the release of v0.10.1 we added nice features but we also break essential ones mainly related to links generation. I think we could create a new patch release to fix people documentation.
tagging @choldgraf and @jarrodmillman as I never been trough the release process.
Checklist below
Double check for quality-control

[x] There are no more open issues in the release milestone
[x] There are no open issues with a block-release label

Release candidate

[ ] PR
[ ] Wait for feedback for a few days...
[ ] new tag
[ ] Wait for a few more days...

Prepare the codebase for a new version

[ ] Bump __version__ in __init__.py[^semantic-version]
[ ] Update our version switcher .json file with the new version
[ ] Make a release commit: git commit -m 'RLS: v0.10.2'
[ ] Push the RLS commit git push upstream master

Make the release

[ ] Start a new GitHub release
[ ] Call the release the current version, e.g. v0.2.0
[ ] In the Choose a Tag: dropdown, type in the release name (e.g., v0.2.0) and click "Create new tag"
[ ] In the Target: dropdown, pin it to the release commit that you've just pushed.
[ ] Add release notes in the field below (if it's a minor/major version bump)[^github-activity]
[ ] Confirm that the publish github action job has completed successfully in the actions tab
[ ] Confirm that the PyPI version is updated
[ ] Celebrate, you're done!

[^semantic-version]: Use semantic versioning to decide whether it's a major, minor, or patch bump. Before we have released 1.0, treat minor versions as breaking releases, and patch versions as feature / patch releases.
[^github-activity]: If you wish, use github-activity to generate a changelog, eg github-activity pydata/pydata-sphinx-theme --since v0.2.2 --until v0.3.0.

If there is a bugfix release, it would be nice, if #908 would be backported to that as well.

I am all for making a new release, happy to help answer any questions about the process @12rambau .
I'm in favor of backporting fixes, though personally I don't have the bandwidth to backport things.

I also would just release 0.11. I don't think there is a need to add the extra work of backporting unless the main branch is broken or in an experimental state.

As one of the people that originally requested a 0.10.2 release, I'd be OK with a 0.11 release as a fix for my problems (assuming no new 0.11 features break my workflow :wink: ).

@jarrodmillman, I'm fine with releasing 0.11 as well , let me just solved the "self" issue in toctree first

I updated the milestone information (moved the 0.10.2 items to 0.11 and closed 0.10.2). I also marked milestone 0.11 as due Oct. 4th. I thought that would be plenty of time to test the release candidate.

Note that issue #932 is marked as a release blocker.

It would be nice to get a release candidate out soon. There are a lot of fixes in the new release already and I would be tempted to move all the remaining issues and PRs to 0.12.  We could release 0.11 soon and then 0.12 in a few weeks, if necessary or desired. I am happy to make the 0.11 and / or 0.12 releases (or let someone else).
I know there are two release blockers, but I think they can potentially be punted (especially if we make a 0.12 release soon).

@12rambau @choldgraf Should we just bump the remaining issues to 0.12 and release 0.11rc1 today or tomorrow? I would prefer more frequent releases. We could plan to release 0.12 in a month or so.

Yep sounds good to me. I'm also a fan of more frequent releases as long as we are not knowingly introducing bugs

I was all in for a 0.10.2 version so I'm in as well to release 0.11rc1 as soon as possible.

@12rambau I've moved all the remaining issues (other than this issue) to 0.12. Please go ahead and make a 0.11rc1 release (or let me know if you would prefer I do it). Thanks for all your work on this release!!

@jarrodmillman if you have a PR example I'm happy to make the PR

@choldgraf I thought I understood how the automated publishing system worked, but I ran into problems again. I can manually make the release, but I don't have permissions on PyPI anymore.
I also can't replicate the failure on GH actions on my computer, so I don't know how to fix it.
I am not sure what the process for creating a rc1 is now. The wiki instructions say to make a PR, but I don't know what should be in the PR. So I just followed the instructions for making a release. Now I am not sure how to proceed. Someone with permission to upload to PyPI could upload the new release or someone who understands why the automated release process failed could fix it (and we could release 0.11rc2).

@jarrodmillman discussed a bit on discord, quick update: we think this is a real error uncovered by the test suite, here's an issue:

https://github.com/pydata/pydata-sphinx-theme/issues/973

@12rambau @choldgraf Should we release a 0.11.0rc4 or 0.11.0 next? I am slightly tempted to release 0.11.0 tomorrow evening, but would be happy to make a final release candidate.

I'm happy with the 0.11.0 (I suspect we are not on the same timezone so night for you will be morning for me). If anything breaks in someone's build we can still make a patch (I doubt everyone is building against rc releases in its CD/CI).

Go for it! Unless we know there's an issue that should block, I think we should just release.

If anything breaks in someone's build we can still make a patch

💯

I got up early, so I decide to go ahead and make the release.


[Feature Request]

Dimension name wildcards so all dimensions starting with DIM have a set border size and any new dimensions also adhere to that border size as well.

I like this idea, your Issue title is crap and you should really reconsider your life choices but the idea is really great.  I feel like this and #21 are basically the same ticket or at least an extension of each-other.


After executing  enduro culture add en

After executing
enduro culture add en

Thank you for reporting this. I removed the config file altogether and the cultures are being stored at enduro.json (this stores all the other enduro settings)


Add more illegal name replacement in MSL.

Fix #517.

@billhollings Please make a PR if you want to refactor this. Ideally, there should be no overrides for replace_illegal_names.


Compacted User Gains

Offer an alternate table view that shows "compacted" user gains, or update the existing table to display compacted gains by default.

I'm assuming this is for the overview table for the updated one? If so, the compacted values would be just pulled from the gained details?

I'm assuming this is for the overview table for the updated one? If so, the compacted values would be just pulled from the gained details?

It would be on whichever table @psikoi determines it should go on. Personally I suggest removing the need for both a "deltas" and "details" table and combine it all into 1 table, that would clean things up and be a bit more on par with other trackers.

I did some experiments with this and it really doesn't fit with the current table width. We'd have to change the layout of the overview page, and that doesn't seem to be happening anytime soon. Closing for now.


New Developer

Hi, I'm a front-end developer, and I like to contribute to the modern website of this project. My team and I are working on this project, and we improved the project in so many ways. And I would like to append this improvement to this project and have some contributions.
Can I start with refactoring?

Depends on what the changes are. Happy to discuss one by one.

One of the improvements was that I migrated to the parcel-bundler, and after that, I worked on lazy loading, Suspense, and Error boundaries, and the result was so good. I reduced the website size from 4-6MB to only 700kb and added the loading animation and error handler to each component, making it so better.

OK, let's see some changes, but please make sure you submit separate pull requests for each separate thing.

Thanks, I will start with migrating to the parcel and after that, I will begin to add lazy loading and some other features

Thanks for the time you spent. I reached a point I think you will not understand what I want to do.
I will continue my development in a new repo that forked from this.
I wish the bests for you and your team.


Format full values for fields

Previously we would just use the equivelent of a
%s but this wouldn't show structs that were passed
in as key values very nicely. If we change to process
them as %+v the output is much more helpful, and
saves from having to implement stringers for every
single struct that might be logged.

Would love to see this merged!

Thank you, but in order to clean up older PRs and issues (this PR is > 2 years old), I'm going to close this PR. Feel free to re-open with an updated branch if this is still of interest. PS: I'm sorry this sat for so long.


React extension - no className on div withAiTracking function

Hi, my react web apps layout is built using flexbox but when I use the react extension it breaks the layout. This is because the extension wraps the components in a div. I understand why it does this but I need to target this div to apply flexbox properties to it or alternatively pass in my own component.
A fix for this issue is in place here https://github.com/microsoft/ApplicationInsights-JS/commit/19ad5784240c191c955645d824ccde164e23fd9c but it has not been published.
Any idea when this might be published? @florianchevallier @markwolff

This has now been deployed as v2.1.0, closing this issue.
Cheers.


[WIP][SQL] Fix rebasing of 29 February of Julian leap years

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Test build #120126 has started for PR 27974 at commit 2cf2e8d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24840/
Test PASSed.

Test build #120126 has finished for PR 27974 at commit 2cf2e8d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/120126/
Test PASSed.

@cloud-fan @HyukjinKwon Please, review the bug fix.

This makes me realize that we can't be fully compatible, and picking the closest valid date in Proleptic Gregorian calendar makes sense to me.
Thanks, merging to master/3.0!

+1


[ENHANCEMENT] - Blame commit selection

Proposal
In the blame view, it could be nice if we could select a commit in the left side bar and highlight all the involved lines for this particular commit instead of only relying on SHA and author date.
It could be easier to quickly identify one commit in the file, also having selection that stick would allow scrolling through the file without losing focus.
I know we can click on the commit hash and see the changes in the diff view but I am not talking about viewing diff and since the blame view exists, it would be very useful to improve it by highlighting this information right there.
Mockup

added:

You can download the latest build from Github Action

Thank you for tackle this feature so rapidly @love-linger !
However after testing the action #1238 (commit 2430175), the selection doesn't seems to properly hold (slide) and I have difficulties to select back the commit after scrolling
(see joined gif)

This issue is closed now, tell me if it's better to open a new one for this.

You should change selection by click the text editor not the commit info.

What OS do you using?

Fixed. You can try the latest build.

It is because that the highlighted background did not update when the text area lost focus.

Oh sorry, nvm @love-linger.. I've still have the same issue but only if I scroll to the very top of the file.
I'm on Windows 10.

I can see when you are scrolling, the highlight background is not updated (following with the scroll)

The expected result is
https://github.com/user-attachments/assets/6b7c91d9-51fb-4185-a882-c98cd2af41a4

I can see when you are scrolling, the highlight background is not updated (following with the scroll)

Yes, it does work well if I select a commit further down the file and scroll around though. The issue seems to be trigger when I reach the very top.

I can not reproduce this issue. The OnTextAreaLayoutUpdated event works fine on my PC & mac

You can try the latest build.

Seems to be fixed on my end ! 😄
Thanks !


[BOUNTY] Manta Network by p0xeidon labs - On-chain Privacy use Cases

Bringing Privacy to Web3
For our bounty, we're looking for some amazing privacy-enabled use cases! No, the core value of your product doesn't have to be privacy, but you should think about creatively and practically implementing privacy into it in some manner. For example, a private lending protocol, private NFTs, private on-chain voting systems, etc.
How to qualify? ☑
To qualify:

Please make sure that your application leverages a privacy integration
Submit your entry through our application
Post a link to your devposst submission as a comment to this issue before the hackathon deadline

Additional Resources 🌐
If you have any questions or want to talk about some ideas, you can come find us at our booth, right next to the entrance of the main stage!
Judging Criteria ⚖
Outline the criteria which you will be using to determine the winner of your bounty here.

The use case should be practical and creative with privacy integrated
Please make sure to focus on user experience so that privacy feels seamless

Prize 🏆

First Prize: $2,500
Second Prize: $1,500
Third Prize: $1,000

Not sure if we had to use the Manta network, but if it's overall privacy, I think mine can compete: https://github.com/Dev43/harpocrates
https://devfolio.co/projects/harpocrates-522c

Hi there,
we had technical issues with devfolio so we thought we should post our project it here as well.
https://github.com/hasselalcala/proofID
https://www.loom.com/share/103bd48e5a764a37843754ff785920b6
https://proof-id-front-end.vercel.app/

Howdy,
Seeing as others posted their submissions here - also thought I'd go ahead as well!
https://github.com/mwyatt896/ethberlin-cryptohub
https://www.loom.com/share/b98deae23b664606b372a815d0b855bb
https://devfolio.co/projects/cryptohub-cd20

Hey, I think my team may have forgotten to post here, so here it is:
https://devfolio.co/projects/the-athens-protocol-cd41
https://github.com/cheethas/governor-of-athens
https://github.com/cheethas/aztec-connect-bridges/blob/cheethas/compoundGov/src/bridges/athens_governor_bravo/AthensGovernorBravoBridgeContract.sol
https://github.com/cheethas/athens-interface
https://docs.google.com/document/d/1zdUz0KbyfOkg6A3Ev0rIr9Q4sBkd3mBTrJBsjBaNbYQ/edit

Our Team forgot to post the links, I hope we ae still eligible:
https://devfolio.co/projects/private-poap-7418

Hey Kenny, any news on this? Curious to find out, when are you announcing the winner (and what is your judging process)? Looking forward to finding out more! :)

Hey @kennymuli and Manta team - just seconding what @RizzeKnight said! Also - will the winners be posted here on this thread as well?


Incorrect function call reconstruction

Version and Platform (required):

Binary Ninja Version: 3.6.4561-dev, 2d2336f2
OS: macos
OS Version: 13.6
CPU Architecture: arm64

Bug Description:
For a Decree binary LUNGE_00005_6, a function call is incorrectly reconstructed.
LUNGE_00005_6.gz
Steps To Reproduce:
Open LUNGE_00005_6, then show read_until function (with Medium Level IL view).
Expected Behavior:
Function call of receive at 0x080490aa, arg1(eax_1) and arg4(esi_1) should be transposed.
Screenshots:

Additional Information:
Binary Ninja Version: 3.4.4271 worked correctly.

Changing the signature of receive to int receive(int, void*, size_t, size_t*) seems to work.

The problem is BN does not understand the calling convention for x86 system calls. From the disassembly of the function receive, we can see that BN thinks the syscall takes parameters from the stack, rather than the registers:
receive:
0804a849  mov     eax, 0x3
0804a84e  push    ebx {__saved_ebx}
0804a84f  push    ecx {__saved_ecx}
0804a850  push    edx {__saved_edx}
0804a851  push    esi {__saved_esi}
0804a852  mov     ebx, dword [esp+0x14 {arg_4}]
0804a856  mov     ecx, dword [esp+0x18 {arg_8}]
0804a85a  mov     edx, dword [esp+0x1c {arg_c}]
0804a85e  mov     esi, dword [esp+0x20 {arg_10}]
0804a862  int     0x80
0804a864  pop     esi
0804a865  pop     edx
0804a866  pop     ecx
0804a867  pop     ebx
0804a868  retn     {__return_addr}

This results in the following MLIL, in which we can see the parameters are incorrectly shuffled as a result:
receive:
   0 @ 0804a84e  int32_t __saved_ebx = ebx
   1 @ 0804a84f  int32_t __saved_ecx = arg3
   2 @ 0804a850  int32_t __saved_edx = arg2
   3 @ 0804a851  int32_t __saved_esi = arg4
   4 @ 0804a862  eax = syscall(__saved_esi, __saved_edx, __saved_ecx, __saved_ebx, __return_addr)
   5 @ 0804a868  return eax

Btw, the issue could have been workaround if we can override the call type of the system call instruction to:
int32_t calltarget(int32_t arg1 @ ebx, int32_t arg2 @ ecx, int32_t arg3 @ edx, int32_t arg4 @ esi)

However, this is not possible since it is blocked on https://github.com/Vector35/binaryninja-api/issues/4674


Listaddressgroupings crashes wallet

Actual behaviour
typing listaddressgroupings in console crashes the wallet
13.8.4 Windows
Desired behaviour
Doesn't crash

Can confirm this issue.


Font::measure seems to return wrong values

I found this issue when working on #35.
Right now, the UI renderer applies a correction when measuring Text widgets. This is a dirtyfix and it probably doesn't work well with different fonts.
Here is what happens if we do not apply the correction with Inconsolata Regular:

I think GlyphBrush::pixel_bounds is not returning a correct value. We should probably investigate why and file an issue/PR in the glyph-brush repo.

Issue reproduced and reported: https://github.com/alexheretic/glyph-brush/issues/68


Improve the documentation

What
Need to improve the documentation.
How

[x] Add "Architecture" section to the Documentation
[x] Add "Architecture > Argument properties" to explain the usage of Object-as-an-argument callback pattern
[x] Move "Referencing" to "Architecture" section
[x] Architecture. Explain Field's lifecycle, internal state management, how RAF works under the hood.
[ ] Improve "Concept". Think about more clear way how to present the problems RAF solves and its benefits.
[x] Update "Getting started" section for Validation
[x] Update "Validation rules" to correspond to the 0.18.0 changes
[x] Update "Validation messages" to correspond to the 0.18.0 changes
[x] Add createField documentation

I will update the "Concept" section once we are close or even once we surpassed the 1.0. "Concept" doesn't contribute much to overall DX of using the library, but is rather an FYI.


Update hash ring to use worker identity instead of worker net address

Update hash ring to use worker identity instead of worker net address

@jja725  i will test out and make sure ETCD works first, and then we can see how to support local testing


Coinbase Wallet configuration produces weird behavior on browser's where Coinbase Wallet extension is installed.

Web3Modal's suggested coinbase config
https://github.com/Web3Modal/web3modal/blob/master/docs/providers/coinbasewallet.md
has some weird behavior.
On a browser with Metamask extension but NOT coinbase extension I get this normal looking web3modal, showing Metamask, wallet connect & coinbase wallet.

On a browser with Coinbase extension & NOT Metamask extension I got this double'd up coinbase web3modal.

Here's the configuration I used to create this issue:
const initModal = new Web3Modal({
  cacheProvider: true, // optional
  providerOptions: {
    coinbasewallet: {
      package: CoinbaseWalletSDK, // Required
      options: {
        appName: "Whatever",
        rpc: {
          1: NETWORKS[1].uri(),
          4: NETWORKS[4].uri(),
          42161: NETWORKS[42161].uri(),
          421611: NETWORKS[421611].uri(),
          43113: NETWORKS[43113].uri(),
          43114: NETWORKS[43114].uri(),
        },
      },
    },
    walletconnect: {
      package: WalletConnectProvider,
      options: {
        rpc: {
          1: NETWORKS[1].uri(),
          4: NETWORKS[4].uri(),
          42161: NETWORKS[42161].uri(),
          421611: NETWORKS[421611].uri(),
          43113: NETWORKS[43113].uri(),
          43114: NETWORKS[43114].uri(),
        },
      },
    },
  },
});

Closing this as I realize now I was looking at version 1.9.7's config, but was on version 1.9.6 of web3Modal


Make --quiet more quiet

Would it be possible to make the output more quiet when using the --quiet option?
Right now ruff reports

That there are diagnostics
That it's possible to fix some of them automatically.

But that means I have to use grep -v to filter out the diagnostics.

Good idea.

Will fix this today. I want to do a small refactor around those settings.

This is going out in v0.0.98 (building now).


fix(guides): Add DV HDR10Plus to Guides

Pull Request
Purpose
Fix missing DV HDR10Plus custom format on guide pages
Approach

Add new DV HDR10Plus custom format to relevant guide page HDR sections

Open Questions and Pre-Merge TODOs
None
If you're adding a new Custom Format, make sure you follow the Radarr/Sonarr Custom Format (JSON) Guidelines. -->
Requirements

[x] These changes meet the standards for contributing.
[x] I have read the code of conduct.

same

Looks like we're all good to go here. Thanks all.


branch: aria-labelledby

To track changes for #701.
@jessegreenberg I just merged this into master. I don't think this is needed anymore. I wanted to check with you first though.

Thanks for documenting @zepumph, but I agree the branch can be deleted and this issue closed. I just deleted it.

Should be re-deleted as part of https://github.com/phetsims/special-ops/issues/198

Deleted, closing.


shim oe_runmake fails

I am encountering an issue using with UEFI secure boot without MOK secure boot.
When I set the SIGNING_MODEL to "user", the shim recipe fails to build with this error message:
Error: file not found: /opt/yocto/build/tmp/work/corei7-64-poky-linux/shim/12+gitAUTOINC+5202f80c32-r0/vendor_dbx.esl
Looking through the code, the vendor_dbx.esl file is only created if MOK_SB is active (meta-signing-key/classes/user-key-store.bbclass:303):
def create_mok_vendor_dbx(d):
    if d.getVar('MOK_SB', True) != '1' or d.getVar('SIGNING_MODEL', True) != 'user':
        return None

But the EXTRA_OEMAKE variable in the shim recipe is initialized like this(meta-efi-secure-boot/recipes-bsp/shim/shim_15.8.bb:41):
<EMAIL_ADDRESS>\
        if d.getVar('MOK_SB') == '1' else ''} \
<EMAIL_ADDRESS>\
        if uks_signing_model(d) == 'user' else ''} \

From what i can see, the latter block should be guarded by MOK_SB==1 and SIGNING_MODEL=='user', not just MOK_SB.
I think a possible fix would this be as simple as:
<EMAIL_ADDRESS>\
        if d.getVar('MOK_SB') == '1' else ''} \
<EMAIL_ADDRESS>\
        if d.getVar('MOK_SB') == '1' and uks_signing_model(d) == 'user' else ''} \

I originally ran into this on the kirkstone branch, but looking at the current master branch, the same issue seems to exist there.
If I'm operating under some massive misconception here please let me know, but I do think this is a bug. I'd would be willing to whip up a PR for each branch, if my understanding of this issue is indeed correct

I can confirm that this is a bug and your fix is correct. If you could submit a PR it would be much appreciated.

All right, thank you!
That was a very fast response :D
I have already set up a fork with the fix, would you like me submit it only against master and you backport it yourself, or against master, kirkstone and any others that have the bug?

All right, thank you! That was a very fast response :D
I have already set up a fork with the fix, would you like me submit it only against master and you backport it yourself, or against master, kirkstone and any others that have the bug?

Only against master. I will backport it to other branches.

Alright, done :)


Can't train

kaan@ALTAR:ssd.pytorch$ python3 train.py
Loading base network...
Initializing weights...
Loading Dataset...
Training SSD on VOC0712
Traceback (most recent call last):
File "train.py", line 231, in
train()
File "train.py", line 170, in train
images, targets = next(batch_iterator)
File "/usr/local/lib/python3.5/dist-packages/torch/utils/data/dataloader.py", line 201, in next
return self._process_next_batch(batch)
File "/usr/local/lib/python3.5/dist-packages/torch/utils/data/dataloader.py", line 221, in _process_next_batch
raise batch.exc_type(batch.exc_msg)
AttributeError: Traceback (most recent call last):
File "/usr/local/lib/python3.5/dist-packages/torch/utils/data/dataloader.py", line 40, in _worker_loop
samples = collate_fn([dataset[i] for i in batch_indices])
File "/usr/local/lib/python3.5/dist-packages/torch/utils/data/dataloader.py", line 40, in
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/kaan/ssd.pytorch/data/voc0712.py", line 117, in getitem
im, gt, h, w = self.pull_item(index)
File "/home/kaan/ssd.pytorch/data/voc0712.py", line 129, in pull_item
height, width, channels = img.shape
AttributeError: 'NoneType' object has no attribute 'shape'

Hi. I got the same problem as you.
Why did you close this issue? Did you solve?
If so, can you share your solution?

can you share your solution? I get the same problem with you. Thx.

@kunguz Did you solve it?
If yes, what was the problem and how did you solve it??

If yes, what was the problem and how did you solve it??

i have met the same trouble，did you deal with it？thank your watching！

Maybe there is no that file. Please check does the pic file exsist. I tried to train kitti. First I converted kitti to VOC format. Then I met this problem. It turned out that my picture is in png format. After I change those pictures to jpg format. The problem is solved. BTW, if this is your problem you can use this command to convert the picutres.ls -1 *.png | xargs -n 1 bash -c 'convert "$0" "${0%.png}.jpg"'


Don't report UnusedImport when binding global name

When an import binds to a name declared global by the "global"
statment, don't report it as unused.

Travis is broken. This is incompatible with #26 so I'd happily rebase one if the other is merged.

I'm not sure why, but Travis CI was toggled off. I've turned it on again. You can check the results now.

Can this please get rebased onto master? That should pick up the fix from #29 and give us success on Travis.

@bitglue Done.

Test looks good, and the goal seems sound, but I can't figure out how this works. Sometimes I just want to rewrite the whole thing :confounded:
Anyone else?

I had looked at it. I kind of wish there was a comment explaining that the ordering here was important, but I don't know it well enough to know why this works either.

I can't say I understand the reasoning behind these methods, but it seems like handleNodeStore is only called by the NAME method (which I'm assuming is only called for binding names with foo = bar syntax, whereas addBinding is called when a name is bound in another way too (imports, defining a function, defining a class, etc.).

I kind of wish there was a comment explaining that the ordering here was important, but I don't know it well enough to know why this works either.

Rereading this comment I think I understand it today. The change doesn't have to do with the order. It is simply moving it into a more general method that is called for more statements, including import.

OK, let's go with it. Thanks for the patch.


[RFC] Generic GPIO Shell and a command to list devices

This introduces a shell command to toggle GPIO pins for basic testing. To get the right GPIO port name, also added a shell command to list all configured devices in the system and their priorities.
The idea here is to such a shell for each driver to all dynamic testing of peripherals and other subsystems instead of hardcoding for each board.

@spoorthik @nniranjhana fyi, that is the shell command from peripherals I was talking about in our meeting a few weeks ago.

General remark is that you can use only \n instead of \r\n

It seems you have a spelling mistake in one place. There is: shell_printf, there shall be: shell_fprintf


tmp_trainer directory getting created after running the training

tmp_trainer directory is automatically getting created while running training. The directory is created in the same path in which the training script is being invoked.

hey, @debanjanbhucs this is expected behaviour from hugging face, pls refer to this. This folder is meant to save statistics during inference. In future, we can use it to store queries and their keyphrases.

Ok! In that case if we change the output_dir this should not get created. Will try out and then close this.


DbalConsoleExtension: accept %consoleMode% in constructor

The extension in contributte/console requires %consoleMode% in constructor, and skips any configuration if it evaluates to false. However, the code here depends on PHP_SAPI, resulting in an exception in environments where PHP_SAPI === 'cli', but consoleMode === false (namely integration tests).
This PR unifies this behaviour and allows users to provide %consoleMode% in constructor as well, in a backward-compatible way.

Thanks.


Bot won't be dead!

We have all heard the news about the bot being banned and people using it will be banned. Who in what world thought that a bot will always be, a bot..?
It's pretty-ish easy to fix this. The apos bot is quite close to playing as a real player, and with a mix of not using the bot for 24 hours and having longer reset wait times, it might just work.
But this is where the issue comes in. If Moneyclip does jump in and have some sort of way to check, we are all screwed. If this update ever get's released, DO NOT use it on your main account. There will always be a chance of getting banned!
Of course this will be much less efficient, but we have no choice anymore.

It may just be the instant respawn which is not possible in vanilla.

@Cr4xy You might be right. But the issue is that we don't know if they have some sort of checker to check if the inbound connection is coming from agario-client. I don't know how they do it, but if that's the case we might have some issues.
I'll do some testing later this week with timed pauses and without instant spawn to check if it works. People have been saying that the fast w ejecting tools screw it up too. I wonder if they are checking if it's ANY mod. If it is, AgarPlus is next to be screwed :(


Dist Html file asset path is not updates

Thanks for quick boilerplate,
When we run gulp build task index html get copied into dist folder but it shows 404 error for js file as in dist asset folder js file have min extension but same is not getting updated in index html

gulp build  will only copy the assets into dist folder. You need to run only gulp command. It will run gulp task clean, build and replaceHtml (In index.html it will replace assets with minified one)

gulp build  will only copy the assets into dist folder. You need to run only gulp command. It will run gulp task clean, build and replaceHtml (In index.html it will replace assets with minified one)


[Bug]: csv_file_name outdated variable

Describe the bug
In LabelingAgent.run, csv_file_name is only used to define and cache the task but now that there is support for more than just csvs, we need a more robust solution. This is more of a problem of technical debt than a bug.
Additional context
We could make an automatically generated name for the dataset in DatasetLoader's constructor but open to other suggestions ofc.

As discussed here, we can leave the default csv output as-is (no need to add more support for different output file formats). Cleaning up csv_file_name and the code surrounding it should be more of the focus for this issue.


Non consistent behavior with Dataframe.sum

Hello,
I noticed a strange behavior involving important errors in my scripts.
a = pd.DataFrame(np.nan, index=[1,2], columns=[3,4]).astype(np.float64)
b = pd.DataFrame(np.nan, index=[1,2], columns=[3,4]).astype(np.float32)
c = pd.DataFrame(np.nan, index=[1,2], columns=[3,4])
d = pd.DataFrame(index=[1,2], columns=[3,4])

print(a.sum())
3    0
4    0
dtype: float64
print(b.sum())
3   NaN
4   NaN
dtype: float64
print(c.sum())
3    0
4    0
dtype: float64
print(d.sum())
3   NaN
4   NaN
dtype: float64

According to your doc, the right result should be the second and fourth output.
I guess it is due to Numpy changes 0.18. However I want to point out that numpy's choice is not good. A sum of NaN must not be zero.
First argument is about logic : Nothing + Nothing is not something but Nothing + object could be something. Two things which are not number can not be a number by arithmetic operation
Second argument : it's easy to fill the result with zeros if you don't want NaN values.
a = pd.DataFrame(np.nan, index=[1,2], columns=[3,4]).sum()
a.fillna(0, inplace=0)

Whereas you are not able to fill the result with NaN instead of zeros into NaN rows (or columns whatever..) if they have been filled with zeros before (easily). I hope you see
what I mean...
Thanks for your work ;)

this is related to #9422
pandas defers to bottleneck for these reductions. if you have bottlneck >= 1.0 then the results will be as above. if you have an older version that you would get all NaN results.
This is the numpy idea of the difference between nansum and sum.
I happen to agree with you as you are effectively losing information, though @shoyer thinks we should keep in sync with numpy.
you can control this via the skipna parameter.
In [17]: a.sum(skipna=False)
Out[17]:
3   NaN
4   NaN
dtype: float64

In [18]: a.sum(skipna=True)
Out[18]:
3    0
4    0
dtype: float64

this is a dupe of #9422  in any event

The difference of behaviour between float32 and float64 is normal as well ? This is from bottleneck as well ?

The difference of behaviour between float32 and float64 is also due to bottleneck ?

I think so you would have to trace this thru to be sure though


BUG In Azure Function Trigger with Managed Identity

I think there is a bug in the use of Azure Function Trigger Bindings on the Connection value when a Managed Identity is used.
We are using Managed Identity with Consumption based Function App to receive events from the EventHub. As per the Microsoft documentation: https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-event-hubs-trigger?tabs=csharp#identity-based-connections
we are using the following connection string in the EventHub trigger:
EhubSaaSSecConnection__fullyQualifiedNamespace: [[ REDACTED ]].servicebus.windows.net
Here’s the Function App signature:
[FunctionName("EventHub-To-EventHubFunction")]
public static async Task Run(
// SaaS EH Trigger - Receives events from source EH
[EventHubTrigger(Constants.SaasEHName, Connection = Constants.SaasEHConnectionName)] EventData[] events,
// QRadar EH Output binding - Sends events to dest EH
[EventHub(Constants.QRadarEHName, Connection = Constants.QRadarEHConnectionName)] IAsyncCollector<string> outputEvents, ILogger log)

Here are the constants:
public const string SaasEHConnectionName = "EhubSaaSSecConnection";
public const string QRadarEHConnectionName = "EhubQRadarConnection";

The function EventHub-To-EventHubFunction is getting triggered without any exception when the Azure portal is open, but the trigger gets hibernated afterward and does not activate even if events queued up in the Event Hub. Upon the troubleshooting, we found the following errors in Scale Controller which is preventing the function to scale up when there’s an event.
After you enable the logging of the Scale Controller Logs you can use the Kusto query:
traces
| extend CustomDimensions = todynamic(tostring(customDimensions))
| where CustomDimensions.Category == "ScaleControllerLogs"
| order by timestamp desc
You'll see logs as the following:
Specified  connection string is null or empty for connection: EhubSaaSSecConnection.   Invalid connection.
More Information below from a different thread:
https://github.com/Azure/azure-sdk-for-net/issues/12657#issuecomment-1011820099
All credits to @Swing0601
https://github.com/Swing0601:
I got the same error message, and looked into the source code, finally figured it out. Hope it will be helpful.
here is the source code, there is condition whether get the connection with Identity or Shared Access
https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/eventhub/Microsoft.Azure.WebJobs.Extensions.EventHubs/src/Config/EventHubClientFactory.cs

Identity-based connections require "<CONNECTION_NAME_PREFIX>__fullyQualifiedNamespace" property.
https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-event-hubs-trigger?tabs=csharp#identity-based-connections
My issue is caused by CONNECTION_NAME_PREFIX is not correct, the format is AzureWebJobs + connectionName, here is an example, CONNECTION_NAME_PREFIX is "AzureWebJobsmyEventHubConnectionString", then "<CONNECTION_NAME_PREFIX>__fullyQualifiedNamespace" value is "AzureWebJobsmyEventHubConnectionString___fullyQualifiedNamespace"
function.json:

BTW, my develop language is Python,
CONNECTION_NAME_PREFIX source code:
https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/extensions/Microsoft.Azure.WebJobs.Extensions.Clients/src/Shared/WebJobsConfigurationExtensions.cs

Good Luck!!!
Originally posted by @Swing0601 in https://github.com/Azure/azure-sdk-for-net/issues/12657#issuecomment-1011820099

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Functions:0.2916388,Event Hubs:0.2840439,Azure.Identity:0.23265213'

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Just providing an update to this and yes there is a bug and to resolve it, you need to create the AppSettings for the Connection value with: "AzureWebJobs" +
In my example, adding this to the AppSettings in the Azure Function fixed the problem:
OLD AppSettings Name:
EhubSaaSSecConnection__fullyQualifiedNamespace: [[ REDACTED ]].servicebus.windows.net
New AppSettings Name that works:
AzureWebJobsEhubSaaSSecConnection__fullyQualifiedNamespace: [[ REDACTED ]].servicebus.windows.net
Just had to restart the function app to force it to pick up the changes and the error in the Scale Controller logs are gone and everything is working.

Just reviewing the Unit Tests for Event Hubs here:
https://github.com/Azure/azure-functions-eventhubs-extension/blob/2e30273447f7d845328bbedca1180af86cf2131c/test/Microsoft.Azure.WebJobs.Extensions.EventHubs.Tests/EventHubEndToEndTests.cs
You can see that the test case is the Happy Happy Path: https://github.com/Azure/azure-functions-eventhubs-extension/blob/2e30273447f7d845328bbedca1180af86cf2131c/test/Microsoft.Azure.WebJobs.Extensions.EventHubs.Tests/EventHubEndToEndTests.cs#L21
E.g.
private const string TestHub2Connection = "AzureWebJobsTestHubConnection2";
and Includes the prefix constant. Suggest a new unit test to remove this requirement and show it works, and/or update all the documentation to ensure that this constant value is required for these scenario's.
Thank you

Thanks for filing this issue @LRomano72. I will try to repro it on my end and get back to you.

Just to update everyone on this. The fix of adding "AzureWebJobs" to the connection variable name in AppSettings did not fix the issue. MSFT is reaching out to the Scale Controller team as they must be an issue there when working with Managed Identity.
As a work around that removes the Scale Controller but will cost you more, is to create an App Service Plan - Standard (min) and then deploy your Function App into that and enable the "Always On" setting so you don't have cold starts issues and the FApp is warm.
Someone else was doing the IaC and we had thing up and running in less then an hours, no changes to any resource names and we had to whitelist all the new IPs to the Event Hub Namespace for security reasons. In general, this scenario will cost you CAD $100/month if you don't have a spare App Service Plan.
MSFT will continue to work on the issue as FApp Consumption plan tier was designed for this scenario so they will make it work.

I believe the bug here has been tracked down to an issue in the scale controller that the service team is aware of and the workaround is to prepend "APPSETTING_" to your connection setting.
/cc @karshinlin

Just leaving this here for anyone else experiencing this and not able to see in their logs where the exception is coming from. It requires a special app setting to enable these logs
"name": "SCALE_CONTROLLER_LOGGING_ENABLED",
"value": "AppInsights:Verbose"

Scale Controller is definitely the problem.
An error occurred while setting up blob references. Exception: 'System.ArgumentException: ServiceBus Namespace not found.
   at Microsoft.Azure.AppService.ScaleController.Extension.ActivationTasks.EventHubActivationTask.GetServiceBusNamespace(SiteMetadata siteInfo, String connectionName, Guid activationTaskId)
   at Microsoft.Azure.AppService.ScaleController.Extension.ActivationTasks.EventHubActivationTask.TryCreate(ScaleControllerConfiguration config, SiteMetadata siteInfo, TriggerConfiguration trigger, Guid activationTaskId, ActivationTask& eventHubActivationTask, List`1& activationErrors)'

any updates on this?
prefixing with "AzureWebJobs" does not solve this issue. Basically, consumption plan triggers using MID-authentication basically doesnt work - i'll have to rollback to connection strings before this is solved.. or are there any other workarounds?

@snekkermoen, there is not fix at this time and the only option is to deploy your function app into an App Service Plan which will then allow the FApp to use the MI correctly. See comments above. MSFT still working on this from email we got from Microsoft reps in Canada.

@LRomano72 thanks.
fwiw, this issue looks similar or equal: https://github.com/Azure/azure-functions-host/issues/8241

Bumping.

Any update?

Any update?

This should be resolved now. Please let us know if you are still observing any issues.

Right, thanks! I set up the Azure Function app configuration like so, I'll monitor it and see if it works correctly!


Invoke reports 500 when called service returns 200 for results > 4mb

Expected Behavior
When calling InvokeMethodAsync, I expect the status code from the called service to be the same statuscode as the calling service is.
Actual Behavior
InvokeMethod Async returns a 500 error, while the calling service returns a 200 (OK) from the controller when the results exceeds the default 4MB limit. Changing the limit to 40mb or 500mb does not change the result.
When having the called service return less data, the 500 error is not thrown.
Steps to Reproduce the Problem
Nuget package:
Dapr.AspNetCore 1.4.0
Dapr Image:
1.4.2
Call a service using await client.InvokeMethodAsync<TResponse> (invokeRequest); while having the called service return a List of objects that have more than 4mb in size.
these are the client settings and method preparation:
using var client = new DaprClientBuilder()
        .UseGrpcChannelOptions(new Grpc.Net.Client.GrpcChannelOptions
        {
            MaxReceiveMessageSize = 40000000, // about 40 mb, does not seem to do anything
        })
        .Build();

            var invokeRequest = client.CreateInvokeMethodRequest(service, method, request);
            var result = await client.InvokeMethodAsync<TResponse> (invokeRequest);

Exception:
Dapr.Client.InvocationException
  HResult=0x80131500
  Message=An exception occurred while invoking method: 'GetLocations' on app-id: 'locationmanagement'
  Source=Dapr.Client
  StackTrace:
   at Dapr.Client.DaprClientGrpc.<InvokeMethodAsync>d__27`1.MoveNext()
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at ChainCargo.Messaging.Invoke.DaprInvokeClient.<InvokeMethodAsync>d__0`2.MoveNext() in C:\Users\Brian\Source\Repos\ChainCargo\Libraries\ChainCargo.Messaging\Invoke\DaprInvokeClient.cs:line 19

  This exception was originally thrown at this call stack:
    System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
    Dapr.Client.DaprClientGrpc.InvokeMethodAsync<TResponse>(System.Net.Http.HttpRequestMessage, System.Threading.CancellationToken)

Inner Exception 1:
HttpRequestException: Response status code does not indicate success: 500 (Internal Server Error).

Actual result by locationmanagement:

HI @Immorality21 - this is actually a Dapr runtime setting you are running into, not an SDK feature.
By default Dapr has a 4mb limit on request/response size, but this is configurable. See: https://docs.dapr.io/operations/configuration/increase-request-size/


Bootstrap stage fails when using prefixe list in VPC routes

Describe the bug
The Bootstrap stage fails when using prefix lists in VPC route table routes in the network-config file.
Error bellow:
[2023-02-24 20:26:37] - info:       [network-vpc-stack] Adding VPC xxxxx
[2023-02-24 20:26:37] - info:       [network-vpc-stack] Adding subnet xxxxx
[2023-02-24 20:26:37] - info:       [network-vpc-stack] Adding subnet xxxxxx
[2023-02-24 20:26:37] - info:       [network-vpc-stack] Adding subnet xxxxxx
[2023-02-24 20:26:37] - info:       [network-vpc-stack] Adding subnet xxxxxx
[2023-02-24 20:26:37] - info:       [network-vpc-stack] Adding subnet xxxxxxx
[2023-02-24 20:26:37] - info:       [network-vpc-stack] Adding subnet xxxxxxx
[2023-02-24 20:26:37] - info:       [network-vpc-stack] Adding Transit Gateway Attachment for xxxxx
[2023-02-24 20:26:37] - error:  undefined
[2023-02-24 20:26:37] - error:  undefined
Subprocess exited with error 1
error Command failed with exit code 1.
To Reproduce
Create a prefix list like below:
prefixLists:
  - name: prefix-list-01
    accounts:
      - Network
    regions:
      - *HOME_REGION
    addressFamily: "IPv4"
    maxEntries: 5
    entries:
      - <IP_ADDRESS>/16
      - <IP_ADDRESS>/16

Reference the prefix list in a route table for a vpc like below:
- name: vpc-test
    account: Network
    region: *HOME_REGION
    cidrs:
      - <IP_ADDRESS>/24
    internetGateway: false
    routeTables:
      - name: route-subnetnfw-aza
        routes:
          - name: TgwRoute
            destinationPrefixList: prefix-list-01
            type: transitGateway
            target: tgw-principal
        tags: []

Expected behavior
The route table should be created with the referenced prefix list.
Please complete the following information about the solution:

[ ] Version: 1.3.0

[ ] Region: ca-central-1

[ ] Was the solution modified from the version published on this repository? No

[ ] If the answer to the previous question was yes, are the changes available on GitHub?

[ ] Have you checked your service quotas for the sevices this solution uses? Yes

[ ] Were there any errors in the CloudWatch Logs? Yes

Additional context
I added a separate route for all the CIDRs in the prefix list as a workaround. This solved the issue.

Hello @hpfpv, and thanks for reaching out to the LZA team!
I have tried to reproduce this issue on my end, however I am able to create the route successfully. Syntactically the provided configuration looks correct, however error: undefined is generally caused by a configuration-related error that is not caught until application runtime. Based on where your application failed, it seems as through there may have been an issue creating a TGW attachment to a VPC in your config file. Did you add any additional configurations beyond the prefix list route you've referenced? There are a few items I'd like to call out that may also help troubleshoot this issue further:

Our most recent release branch release/v1.3.2 includes several enhancements to logging and configuration validation, including removing the error: undefined log messages and providing more helpful application runtime errors. I'd highly recommend upgrading your environment if possible and seeing if any configuration error is caught during the Build stage of the pipeline, or if any additional context is provided during the runtime failure during the Bootstrap stage.

If you are unable to upgrade your environment, you may run the latest config validation code against your configuration files locally by pulling the release/v1.3.2 branch to a local workstation and following the steps listed in DEVELOPING.md.

If these steps do not resolve your issue, please provide us with your network-config.yaml, and we will be happy to try reproducing on our end with your config. If choosing this option, please ensure there is no sensitive data in the configuration file prior to upload.

Thanks, and we look forward to hearing back from you!


[Enhancements] Add Debugging section in README.md

Adding debugging section into the documentation to make the SDK more user friendly.

Thanks. This has been integrated into the 1.2 update. This PR can be deleted.


Queue abstraction for the transport of events

Standard implementation simply uses buffered channels but later other types may be interesting for distributed cells.

This would be really useful.
Http seems the obvious one.
Quic would be an interesting one because it is UDP based and so can inherently do multicast which might be useful , but i am sure you have your own thoughts on this too.


How to get the layer which drawn with pm.enableDraw and how to add event code on that layer?

Hello there,
I add a marker with the following code.
(this code is called when a button clicks.)
function add_marker(type){
  var icon = null;
  if(type == 1){
      icon = "/map/images/aaa.png";
  }else if(type==2){
      icon = "/map/images/bbb.png";
  }else if(type==3){
      icon = "/map/images/ccc.png";
  }

  var stationIcon = new L.icon({
      iconUrl: icon,
      iconSize:[30,30]
   });

  var options = {
      markerStyle: {
          opacity: 1,
          icon:stationIcon
      },
      continueDrawing:false
  };
  that.mymap.pm.enableDraw('Marker',options);
}

It can be called several times with different types.
I want to get some events for changed coordinates and removed by each marker.
However, I don't know how to get each layer and how to set the event code.

You can use the event pm:create to listen for new created layers:
that.mymap.on('pm:create',(e)=>{
    var layer = e.layer
    // now you can add listeners
   layer.on('pm:dragend',(x)=>{console.log('Layer dragged');});
});

Or you can get all Layers with that.mymap.pm.getGeomanDrawLayers()

Oh! Thanks for your reply.
It was so simple.


Task/UOT-126352 - Fixing issues with finetuning sentence transformer

Description:
Training/Validation Data:

Added string matching functions (text_utils)
Add arg to filter queries (checks queries that look like titles or mostly numbers and removes them) (update_eval_data)
Updated step to remove excluded searches to use filtered queries (validation_data)
Using space similarity model to re-rank/filter paragraphs for expected docs (make_training_data)
Retrieving paragraphs directly from corpus json directory instead of data.csv from previous sentence model (could be outdated) (make_training_data)
Retrieving up to 50 paragraphs per matching doc (make_training_data)
Updated step to get negative samples to match changes to result formats for sent index (make_training_data)

Finetuning:

Fixed issue with saving model/metadata

API:

Added in more args to pass into API endpoint for updating metadata (controls.py)
Updated default args for finetuning/updating metadata (pipeline.py)
Set batch size = 8 to avoid memory error
Convert args to ints for finetuning trainModel endpoint
Logging 0/100% progress for finetuning

Manual Testing:
To test finetuning on mlapi from the backend, hit endpoint /trainModel with the app running with params:
{"build_type": "sent_finetune", "epochs": 1, "warmup_steps":100, "testing_only":true}
To test from the front end (ML Dashboard) on models tab, enter params (**batch_size = 8, epochs = 1, warmup_steps=100) and click the button to finetune.
To test making the training data from the backend, hit endpoint /trainModel with the app running with params:
{"build_type":"meta","meta_steps":"update_sent_data","update_eval_data":false} (currently there isn't a test option for this so the whole process takes ~1hr+ if you don't stop it)
Unittests (updated 02/09/22):
platform darwin -- Python 3.8.5, pytest-6.1.1, py-1.9.0, pluggy-0.13.1 -- /Users/katherinedowdy/anaconda3/anaconda3/bin/python
cachedir: .pytest_cache
rootdir: /Users/katherinedowdy/Documents/GameChanger/gamechanger-ml
collected 18 items

gamechangerml/api/tests/api_tests.py::test_conn PASSED                                                                                [  5%]
gamechangerml/api/tests/api_tests.py::test_expandTerms FAILED                                                                                [ 11%]
gamechangerml/api/tests/api_tests.py::test_remove_kw_1 PASSED                                                                                [ 16%]
gamechangerml/api/tests/api_tests.py::test_remove_kw_2 PASSED                                                                                [ 22%]
gamechangerml/api/tests/api_tests.py::test_remove_kw_3 PASSED                                                                                [ 27%]
gamechangerml/api/tests/api_tests.py::test_remove_kw_4 PASSED                                                                                [ 33%]
gamechangerml/api/tests/api_tests.py::test_getTransformerList PASSED                                                                                [ 38%]
gamechangerml/api/tests/api_tests.py::test_postSentSearch PASSED                                                                                [ 44%]
gamechangerml/api/tests/api_tests.py::test_recommender PASSED                                                                                [ 50%]
gamechangerml/api/tests/api_tests.py::test_qa_regular PASSED                                                                                [ 55%]
gamechangerml/api/tests/api_tests.py::test_qa_one_question PASSED                                                                                [ 61%]
gamechangerml/api/tests/api_tests.py::test_qa_multiple_question PASSED                                                                                [ 66%]
gamechangerml/api/tests/api_tests.py::test_qa_allcaps PASSED                                                                                [ 72%]
gamechangerml/api/tests/api_tests.py::test_qa_apostrophe PASSED                                                                                [ 77%]
gamechangerml/api/tests/api_tests.py::test_qa_past_tense PASSED                                                                                [ 83%]
gamechangerml/api/tests/api_tests.py::test_qa_future_tense PASSED                                                                                [ 88%]
gamechangerml/api/tests/api_tests.py::test_qa_specific PASSED                                                                                [ 94%]
gamechangerml/api/tests/api_tests.py::test_qa_outside_scope PASSED                                                                                [100%]

INFO:root:Starting to load data for finetuning... 2022-01-25 20:19:12,639 [root        ] INFO     Starting to load data for finetuning... WARNING:root:Could not complete finetuning 2022-01-25 20:19:12,639 [root        ] WARNING  Could not complete finetuning ERROR:root:batch_size should be a positive integer value, but got batch_size=32 2022-01-25 20:19:12,640 [root        ] ERROR    batch_size should be a positive integer value, but got batch_size=32
getting this when finetuning

The old issues with this have been resolved: finetuning now works with the default batch size, other fixes in the description of the PR.


Update README.md

Reorganize the order of the sections (explain features at first)
Rewrite "Features" section to clarify how this bridge should work

Feel free to add edits to correct wrong changes if any.

no activity is expected on this side so closing


Facebook handover

This pull request contains the code for the upcoming Facebook handover blog post

I am not sure why I am added as a reviewer here. Removing myself.


Update Cloudflare Tunnel Version to v2024.1.1

https://github.com/cloudflare/cloudflared/blob/master/RELEASE_NOTES
Proposed Changes

This PR will update the Cloudflared Tunnel to v2024.1.1, to get the latest imporvemnts and fixes.

Related Issues

([Github link]autolink-references to related issues or pull requests)

We have automated the updates, but thanks for creating the PR. :)
#568


Radix sort using python

Again it's not a spam, it's my 4th contribution to 1st hacktoberfest . please don't close it thinking of it as a spam.

Hey @killer0921 this is not the way for contributing in this repository. Please read our Contributing guide
You can make this pull requests to that repository: https://github.com/ReciHub/FunnyAlgorithms


Update docs

Big update with documenation fixes.

Codecov Report

Merging #66 into master will decrease coverage by 1.02%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master      #66      +/-   ##
==========================================
- Coverage      86%   84.98%   -1.03%
==========================================
  Files           3        3
  Lines         243      253      +10
  Branches       50       53       +3
==========================================
+ Hits          209      215       +6
- Misses         21       24       +3
- Partials       13       14       +1

Impacted Files
Coverage Δ

xgcm/grid.py
90.5% <88.88%> (-1.81%)
:arrow_down:

Continue to review full report at Codecov.

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