Update eventhubs unit test CMakeLists.txt to remove dependency on attestation

Similar to https://github.com/Azure/azure-sdk-for-cpp/pull/4727
Service SDKs should not be depending on each other. Likely just a copy/paste error from KeyVault tests which also had this issue.

Are you also going to make the same change into the feature/amqp branch and in George's private branch?
I'm worried that this change is going to be lost if it's not present in all of the relevant places.
And why does this need to be made in main, rather than the amqp feature branch?

And why does this need to be made in main, rather than the amqp feature branch?

Because the issue exists in main already (and I would like components in main to be buildable if I exclude attestation). Is there a different workflow here that I should follow, where the fix first needs to be made in a feature branch and then moved over?

And why does this need to be made in main, rather than the amqp feature branch?

Because the issue exists in main already (and I would like components in main to be buildable if I exclude attestation). Is there a different workflow here that I should follow, where the fix first needs to be made in a feature branch and then moved over?

The eventhubs service is being developed in the feature/amqp branch, and by making this change in main, it runs a high risk of being overwritten when the feature/amqp branch comes in. It is safer to make the change everywhere the feature is being developed to avoid the potential of merge conflicts. If eventhubs was being built out of main, this wouldn't be a problem, but it is what it is.
 Also, it's not unreasonable for live service tests to require interacting between different client SDKs - we already see that with samples and the identity SDK.
In addition, the MHSM service has a strong dependency on the output of the attestation service, so having MHSM live tests depending on the attestation SDK is not at all unreasonable.

Why would you want to build everything in the azure-sdk-for-cpp repo without also building the attestation service?

It isn't just about attestation, I wanted to build keyvault keys and run a test. I wanted the smallest build set from the repo. Similarly, if I want to build just attestation, it would be surprising to me that I would need to include the eventhubs or storage folders in the build, especially since these SDKs were built independently (without reliance on other SDKs being present).
To me, having tight integration between the SDKs is a sign of introduced complexity and tight coupling.

Also, it's not unreasonable for live service tests to require interacting between different client SDKs - we already see that with samples and the identity SDK.

I don't think that's a scenario we support in other language, or should support here. Unlike samples, live tests are meant to test the service client and that alone, they aren't integration tests for applications built using multiple SDKs together. The SDKs should be buildable and testable with only a dependency on Core, even if the samples have an exception and need identity (or higher-level samples that highlight multiple SDK clients working together).
In this case, it was purely incidental (copy/paste error), and not enabling some scenario we need.

In addition, the MHSM service has a strong dependency on the output of the attestation service, so having MHSM live tests depending on the attestation SDK is not at all unreasonable.

Maybe, but that would be surprising. We don't have that today, there's no need for linking such a dependency.

The eventhubs service is being developed in the feature/amqp branch, and by making this change in main, it runs a high risk of being overwritten when the feature/amqp branch comes in. It is safer to make the change everywhere the feature is being developed to avoid the potential of merge conflicts.

Sure, I can make this change in the feature branch.

If eventhubs was being built out of main, this wouldn't be a problem, but it is what it is.

I thought we merged everything into main and the plan was to build out of that. Why do we have the eventhubs folder in main then if we are still using the feature branch? Should we remove it from main?

The eventhubs service is being developed in the feature/amqp branch, and by making this change in main, it runs a high risk of being overwritten when the feature/amqp branch comes in. It is safer to make the change everywhere the feature is being developed to avoid the potential of merge conflicts.

Sure, I can make this change in the feature branch. See #4732

If eventhubs was being built out of main, this wouldn't be a problem, but it is what it is.

I thought we merged everything into main and the plan was to build out of that. Why do we have the eventhubs folder in main then if we are still using the feature branch? Should we remove it from main?

I believe George started working in feature/amqp and has continued to work there. It is what it is, the reality is that feature/amqp is still alive and kicking. This will hopefully only be for a few weeks more.

Why would you want to build everything in the azure-sdk-for-cpp repo without also building the attestation service?

It isn't just about attestation, I wanted to build keyvault keys and run a test. I wanted the smallest build set from the repo. Similarly, if I want to build just attestation, it would be surprising to me that I would need to include the eventhubs or storage folders in the build, especially since these SDKs were built independently (without reliance on other SDKs being present).
To me, having tight integration between the SDKs is a sign of introduced complexity and tight coupling.

Also, it's not unreasonable for live service tests to require interacting between different client SDKs - we already see that with samples and the identity SDK.

I don't think that's a scenario we support in other language, or should support here. Unlike samples, live tests are meant to test the service client and that alone, they aren't integration tests for applications built using multiple SDKs together. The SDKs should be buildable and testable with only a dependency on Core, even if the samples have an exception and need identity (or higher-level samples that highlight multiple SDK clients working together).
In this case, it was purely incidental (copy/paste error), and not enabling some scenario we need.

In addition, the MHSM service has a strong dependency on the output of the attestation service, so having MHSM live tests depending on the attestation SDK is not at all unreasonable.

Maybe, but that would be surprising. We don't have that today, there's no need for linking such a dependency.

Today, we cannot build only one service, we only support building the entire tree. It stinks and I REALLY want to fix that during gallium.
And while it is a surprising result that one service would have a dependency on the output of another service, it's a reality in an interconnected world. There's really no difference between attestation and identity in this regard (attestation is an authorization service for MHSM and confidential ledger).

Closing in favor of https://github.com/Azure/azure-sdk-for-cpp/pull/4732


[T2A1][1]Richard Goh Chang Rui

Ready for review

@richardgohcr Some comments added. Please close the PR after reading comments.


[FEATURE] Add a feature to update Drifty

Is your feature request related to a problem? Please describe.
Currently, Drifty can not detect if the user is running an older version or not. As a result of which, the already fixed errors and new features do not appear in the end user's application.
Describe the solution you'd like.
A feature needs to be introduced such that Drifty can check whether the user is running the latest version of Drifty or not. If the user is running an outdated version, then Drifty should update itself after taking the user's confirmation.

If anyone wants to work on this issue, please comment your name here and I shall assign you.

i want to work in this issue

@Sakshikumari28 Have you started working on this issue? Are you facing any problems? Let me know.

I'm setting of the project in my local environment
On Wed, 5 Jul, 2023, 1:51 pm Saptarshi Sarkar, @.***>
wrote:

@Sakshikumari28 https://github.com/Sakshikumari28 Have you started
working on this issue? Are you facing any problems? Let me know.
—
Reply to this email directly, view it on GitHub
https://github.com/SaptarshiSarkar12/Drifty/issues/217#issuecomment-1621272418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZWYO2XFHEWICP6BPH4PVG3XOUP2DANCNFSM6AAAAAAZSRSZUA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'm setting of the project in my local environment
…
On Wed, 5 Jul, 2023, 1:51 pm Saptarshi Sarkar, @.> wrote: @Sakshikumari28 https://github.com/Sakshikumari28 Have you started working on this issue? Are you facing any problems? Let me know. — Reply to this email directly, view it on GitHub <#217 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZWYO2XFHEWICP6BPH4PVG3XOUP2DANCNFSM6AAAAAAZSRSZUA . You are receiving this because you were mentioned.Message ID: @.>

Great 👍 @Sakshikumari28

I'm setting of the project in my local environment
…
On Wed, 5 Jul, 2023, 1:51 pm Saptarshi Sarkar, @.> wrote: @Sakshikumari28 https://github.com/Sakshikumari28 Have you started working on this issue? Are you facing any problems? Let me know. — Reply to this email directly, view it on GitHub <#217 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZWYO2XFHEWICP6BPH4PVG3XOUP2DANCNFSM6AAAAAAZSRSZUA .

I'm setting of the project in my local environment
…
On Wed, 5 Jul, 2023, 1:51 pm Saptarshi Sarkar, @.> wrote: @Sakshikumari28 https://github.com/Sakshikumari28 Have you started working on this issue? Are you facing any problems? Let me know. — Reply to this email directly, view it on GitHub <#217 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZWYO2XFHEWICP6BPH4PVG3XOUP2DANCNFSM6AAAAAAZSRSZUA .

@Sakshikumari28 Please respond.

@sayande717 The update flag should be
- -u/-update
+ -u/--update

and how are you going to implement the new code, that is, how is the new version going to replace the old one?

@sayande717 After we get the tag_name from the api, we can check if that matches or is greater or smaller with respect to the user's Drifty version. There is a VERSION constant in DriftyConstants.java file, which will provide the version the user is using.
The API call can be made using the HttpsUrlConnection class of Java. After we receive the API response, we can parse that by a JSON parser for java and get the latest version in string format.
References for API calling -

https://www.baeldung.com/java-http-request
https://www.twilio.com/blog/5-ways-to-make-http-requests-in-java#:~:text=Java 1.1 HttpURLConnection

Sorry for the late reply sir  I tried but I wasn't able to do this task
assigned to me
I apologize for that .you can assign to some other
On Mon, 10 Jul, 2023, 12:51 pm Saptarshi Sarkar, @.***>
wrote:

I'm setting of the project in my local environment
… <#m_1817591701436097688_m_-1662355819554228259_m_7150549713078310479_>
On Wed, 5 Jul, 2023, 1:51 pm Saptarshi Sarkar, @.> wrote:
@Sakshikumari28 https://github.com/Sakshikumari28
https://github.com/Sakshikumari28 https://github.com/Sakshikumari28 Have
you started working on this issue? Are you facing any problems? Let me
know. — Reply to this email directly, view it on GitHub <#217 (comment)
https://github.com/SaptarshiSarkar12/Drifty/issues/217#issuecomment-1621272418>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZWYO2XFHEWICP6BPH4PVG3XOUP2DANCNFSM6AAAAAAZSRSZUA
https://github.com/notifications/unsubscribe-auth/AZWYO2XFHEWICP6BPH4PVG3XOUP2DANCNFSM6AAAAAAZSRSZUA
.
@Sakshikumari28 https://github.com/Sakshikumari28 Please respond.
—
Reply to this email directly, view it on GitHub
https://github.com/SaptarshiSarkar12/Drifty/issues/217#issuecomment-1628383591,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZWYO2QUQPFDVTWAX5CJZT3XPOUQZANCNFSM6AAAAAAZSRSZUA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sorry for the late reply sir I tried but I wasn't able to do this task assigned to me I apologize for that .you can assign to some other
It's okay, no problem @Sakshikumari28

@sayande717 What is the progress on this issue?

@SaptarshiSarkar12
Actually I'm a bit busy this week because of family issues, I'll check everything out next week. Till then I'll try setting the API and stuff, because I need to learn that before...I'll notify you whenever I can get back on it, hopefully I'll be free next week.

@SaptarshiSarkar12 Actually I'm a bit busy this week because of family issues. I'll try setting the API and stuff, because I need to learn that before...I'll notify you whenever I can get back on it, hopefully I'll be free next week.

Ok, no problem @sayande717

If this issue is still open I would like to help. @SaptarshiSarkar12

If this issue is still open I would like to help. @SaptarshiSarkar12

Yes, it is still open. If @sayande717 no longer wants to work, then you (@shivanjaywagh) can be assigned. You (@shivanjaywagh) can also collaborate with @sayande717.

If this issue is still open I would like to help. @SaptarshiSarkar12

Yes, it is still open. If @sayande717 no longer wants to work, then you (@shivanjaywagh) can be assigned. You (@shivanjaywagh) can also collaborate with @sayande717.

I don't think I'll be able to actively work on this right now, I have some important college projects coming up so I need to focus on that. So, @SaptarshiSarkar12 , you can assign it to @shivanjaywagh if you want to.

I'm curious about how code like this works in the first place ... I understand that it checks a URL of some kind to compare versions, but how does the code actually replace itself with the newer package?

I'm curious about how code like this works in the first place ... I understand that it checks a URL of some kind to compare versions, but how does the code actually replace itself with the newer package?

I have got a resource on this. It is called as update4j framework. @shivanjaywagh You can check if the code of this framework might help.

@shivanjaywagh I have a got an idea 💡 for this issue.
When the user passes -u or --update flag to Drifty CLI or ask the Drifty GUI to update, it will fetch the latest version number and match them. If a new version is available, then it would download an updater.
I am thinking to make an updater JavaFx app, which will pull the latest version of Drifty and store it in temp directory. Then, it will replace the new one with the old one and start the app. When the new Drifty version opens, it will delete the updater app to clean up resources.

@SaptarshiSarkar12 @shivanjaywagh - I actually wrote a class a couple of months ago that finds the full path to the currently running executable then it spawns a thread to launch that path, waits one second then exits out, leaving the newly updated package as the one that is running.
If it would be convenient to have that path for the download pull, I can work it into one of the enums easily enough.
I also have this class published as a test project right here It is important to note that the class won't work when you are running the project from your IDE because it finds the exact file that launched the program that is currently running and that program changes (theoretically) with every re-run of the IDE version so you need to compile it to test it. and those instructions are in the readme.
Here is the class for Drifty:
import CLI.Drifty_CLI;
import Enums.Mode;
import Enums.OS;
import GUI.Launcher;

import java.io.IOException;
import java.nio.file.Paths;
import java.security.CodeSource;
import java.security.ProtectionDomain;
import java.util.concurrent.TimeUnit;

public class Relauncher {

    private static final String macAppName = "TestRelaunch.app";
    private static final String linuxAppName = "TestRelaunch";
    private static final String winAppName = "TestRelaunch.exe";
    private static final ProtectionDomain protectionDomain = Mode.CLI() ? Drifty_CLI.class.getProtectionDomain() : Launcher.class.getProtectionDomain();

    public static void relaunch() {
        String[] args;
        if (OS.isWindows() || OS.isLinux()) {
            args = new String[]{getPath()};
        } else {
            args = new String[]{getPath()};
        }
        try {
            new ProcessBuilder(args).start();
            new Thread(() -> {
                sleep(1000);
                System.exit(0);
            }).start();
        } catch (IOException e) {
            throw new RuntimeException(e);
        }
    }

    private static void sleep(long time) {
        try {
            TimeUnit.MILLISECONDS.sleep(time);
        } catch (InterruptedException e) {
            throw new RuntimeException(e);
        }
    }

    private static String getPath() {
        CodeSource packageSource = protectionDomain.getCodeSource();
        String path;
        try {
            path = Paths.get(packageSource.getLocation().toURI().getPath()).toAbsolutePath().toString();
        } catch (Exception e) {
            path = Paths.get(protectionDomain.getCodeSource().getLocation().getPath()).toAbsolutePath().toString();
        }
        switch(OS.thisOS()) {
            case MAC -> {
                return path.replaceFirst(macAppName + ".+", macAppName+"/Contents/MacOS/universalJavaApplicationStub");
            }
            case WIN -> {
                return path.replaceFirst(winAppName + ".+", winAppName);
            }
            default -> {
                return path.replaceFirst(linuxAppName + ".+", linuxAppName);
            }
        }
    }
}

@shivanjaywagh You can take a look into the class created by @EasyG0ing1 although the package Names will change after #233 gets merged.
I think having an external updater program which would replace the old program and restart the new one, would be much better.

@shivanjaywagh @SaptarshiSarkar12 As long as these three lines just after the class declaration look like this:
    private static final String macAppName = "Drifty.app";
    private static final String linuxAppName = "Drifty";
    private static final String winAppName = "Drifty.exe";

It should work fine

@shivanjaywagh @SaptarshiSarkar12
I updated the class so that the name of the executable is no longer needed for static String assignment, it will find the name automatically. Tested in Mac and Windows but not Linux. I also updated the repository TestRelaunch so it should be easy to pull that down and do a mvn clean package on Linux and test it.
import java.io.File;
import java.io.IOException;
import java.nio.file.Paths;
import java.security.CodeSource;
import java.security.ProtectionDomain;
import java.util.concurrent.TimeUnit;

public class Relauncher {

    private static final ProtectionDomain protectionDomain = Launcher.class.getProtectionDomain();

    public static void relaunch() {
        String[] args = new String[]{getPath()};
        try {
            new ProcessBuilder(args).start();
            new Thread(() -> {
                sleep(1000);
                System.exit(0);
            }).start();
        } catch (IOException e) {
            throw new RuntimeException(e);
        }
    }

    private static String getPath() {
        CodeSource codeSource = protectionDomain.getCodeSource();
        String path = "";
        try {
            String codePath = codeSource.getLocation().toURI().getPath().replace("/C:", "C:");
            System.out.println(codePath);
            path = Paths.get(codePath).toAbsolutePath().toString();
        } catch (Exception e) {
            e.printStackTrace();
        }
        switch (OS.thisOS()) {
            case MAC -> {
                String macAppName = findProgramName(path);
                return path.replaceFirst(macAppName + ".+", macAppName + "/Contents/MacOS/universalJavaApplicationStub");
            }
            default -> {
                String appName = findProgramName(path);
                return path.replaceFirst(appName + ".+", appName);
            }
        }
    }

    private static String findProgramName(String filePath) {
        String extension = switch (OS.thisOS()) {
            case WIN -> ".exe";
            case MAC -> ".app";
            default -> "";
        };
        if (extension.isEmpty()) {
            return new File(filePath).getName();
        }
        String filename = new File(filePath).getName();
        if ((filename.endsWith(extension))) {
            return filename;
        } else {
            filePath = new File(filePath).getParent();
        }
        return findProgramName(filePath);
    }

    private static void sleep(long time) {
        try {
            TimeUnit.MILLISECONDS.sleep(time);
        } catch (InterruptedException e) {
            throw new RuntimeException(e);
        }
    }
}

@shivanjaywagh @SaptarshiSarkar12 As long as these three lines just after the class declaration look like this:
    private static final String macAppName = "Drifty.app";
    private static final String linuxAppName = "Drifty";
    private static final String winAppName = "Drifty.exe";

It should work fine

Ok

@shivanjaywagh @SaptarshiSarkar12
I updated the class so that the name of the executable is no longer needed for static String assignment, it will find the name automatically. Tested in Mac and Windows but not Linux. I also updated the repository TestRelaunch so it should be easy to pull that down and do a mvn clean package on Linux and test it.
import java.io.File;
import java.io.IOException;
import java.nio.file.Paths;
import java.security.CodeSource;
import java.security.ProtectionDomain;
import java.util.concurrent.TimeUnit;

public class Relauncher {

    private static final ProtectionDomain protectionDomain = Launcher.class.getProtectionDomain();

    public static void relaunch() {
        String[] args = new String[]{getPath()};
        try {
            new ProcessBuilder(args).start();
            new Thread(() -> {
                sleep(1000);
                System.exit(0);
            }).start();
        } catch (IOException e) {
            throw new RuntimeException(e);
        }
    }

    private static String getPath() {
        CodeSource codeSource = protectionDomain.getCodeSource();
        String path = "";
        try {
            String codePath = codeSource.getLocation().toURI().getPath().replace("/C:", "C:");
            System.out.println(codePath);
            path = Paths.get(codePath).toAbsolutePath().toString();
        } catch (Exception e) {
            e.printStackTrace();
        }
        switch (OS.thisOS()) {
            case MAC -> {
                String macAppName = findProgramName(path);
                return path.replaceFirst(macAppName + ".+", macAppName + "/Contents/MacOS/universalJavaApplicationStub");
            }
            default -> {
                String appName = findProgramName(path);
                return path.replaceFirst(appName + ".+", appName);
            }
        }
    }

    private static String findProgramName(String filePath) {
        String extension = switch (OS.thisOS()) {
            case WIN -> ".exe";
            case MAC -> ".app";
            default -> "";
        };
        if (extension.isEmpty()) {
            return new File(filePath).getName();
        }
        String filename = new File(filePath).getName();
        if ((filename.endsWith(extension))) {
            return filename;
        } else {
            filePath = new File(filePath).getParent();
        }
        return findProgramName(filePath);
    }

    private static void sleep(long time) {
        try {
            TimeUnit.MILLISECONDS.sleep(time);
        } catch (InterruptedException e) {
            throw new RuntimeException(e);
        }
    }
}

Ok, shall be testing it out after mergin #233 . TestRelaunch is probably a private repository as GitHub throws a 404 Error.

@SaptarshiSarkar12  Is it still open to work?

@SaptarshiSarkar12 Is it still open to work?

Yes @keshav08, it is. The issue is waiting for #233 to get merged, and then it can proceed as #233 is going to make some massive changes in the way Drifty is distributed to users.
If you (@keshav08) would like to work, then, I may assign you as @shivanjaywagh has been inactive on this issue for a long time.

@keshav08 The discussions on #233 has been continued in our Discord server. Join our Discord Server to get updates on that PR.

@SaptarshiSarkar12 Yes, I would like to explore more and will contribute this feature. When will be the #233 merged? Till then I will look into the repo

@SaptarshiSarkar12 I will join the discord for the further discussion.

Hi @SaptarshiSarkar12 ,
I'd like to contribute to this feature as well. I think @keshav08 and I can work together to explore ways to implement it.

@SaptarshiSarkar12 Yes, I would like to explore more and will contribute this feature. When will be the #233 merged? Till then I will look into the repo

@keshav08 It depends on the speed of development of the feature by the pull request author. I think it should not take more than 2 weeks.

@SaptarshiSarkar12 I will join the discord for the further discussion.

Sure :+1:! @keshav08 That's a good idea as you will get to know when #233 is going to get merged via discord.

Hi @SaptarshiSarkar12 ,
I'd like to contribute to this feature as well. I think @keshav08 and I can work together to explore ways to implement it.

Yes, you both (@keshav08 and @kmr-rohit) can discuss and contribute together on this issue. I am assigning to both of you.

@SaptarshiSarkar12 So, should I go through the repo for now? And do I have to wait to start coding until the merge?

@SaptarshiSarkar12 So, should I go through the repo for now? And do I have to wait to start coding until the merge?

Yes, you can go through it. Only the GUI package is going to change, which has no effect on this issue.
Remember, the executable filenames will get changed as below.

Linux
Windows
macOS

Drifty CLI
Drifty CLI.exe / msi  [To be decided later on]
Drifty CLI.pkg / dmg  [To be decided later on]

Drifty GUI
Drifty GUI.exe / msi  [To be decided later on]
Drifty GUI.pkg / dmg  [To be decided later on]

@keshav08 @kmr-rohit I have an idea/suggestion on the way this issue can be solved the most reliably.
You need to create a JavaFX app named Drifty Updater which will be under the Updater directory. It will be built as a separate executable.
Drifty will check if any update is available by getting the tag_name json field from the JSON response provided by the GitHub API call (https://api.github.com/repos/SaptarshiSarkar12/Drifty/releases/latest?tag_name). The tag_name contains the major, minor and patch version number. Drifty contains a version variable inside the Drifty backend. It will match the fetched version with its current version and if it is outdated, then, it will download and run the updater JavaFX app to which Drifty's current location and type of application (CLI or GUI) will be passed as command line arguments.
The updater will download the latest version and keep it in the temp directory. It will then replace the outdated one with the new one. Finally, it is going to run the new Drifty CLI or GUI.
Hence, Drifty gets updated :tada:!

@SaptarshiSarkar12 So, should I go through the repo for now? And do I have to wait to start coding until the merge?

You both can start working, no issue!

@SaptarshiSarkar12 Thank you for your suggestion , me along with @keshav08 will explore the codebase , and try to work in your suggested direction. Will catch you up whenever required.

@SaptarshiSarkar12 Just to make sure , should i fork Drifty repo by @EasyG0ing1 as that is 78 commits ahead main branch to understand codebase.? As #233 is going to be merged.

@SaptarshiSarkar12 Thank you for your suggestion , me along with @keshav08 will explore the codebase , and try to work in your suggested direction. Will catch you up whenever required.

Sure :+1:!

@keshav08 @kmr-rohit Is anyone of you interested in working on this issue? If not, I'm going to unassign you.

As anyone of you (@keshav08 and @kmr-rohit) are no longer interested in working on this issue, so, I'll work on it.
Thank you for your support!


Add css filters through API

Version and OS
0.42 on linux/docker
Is your feature request related to a problem? Please describe.
I tried setting additional parameters through the API but couldn't.
Describe the solution you'd like
Adding some parameters through the API, like the css filters, would be a great solution.

Hmm pretty sure you can just update the filters directly in the API
but without seeing an example of the API call you tried i cant do anything :'(
please add some more examples


Get on mainnet

I think we can close this as #47 will allow us to mint on potentially xDAI mainnet


Update Num.log builtin to support more implementations

Zulip discussion

Instead of a single Num.log we should have Num.ln, Num.log10 and Num.log2. They would have some sort of custom support and then a general logBase.

@folkertdev adding here so we can track this idea.


Arrow keys and terminal sizing do not work in admin shells

Version: 1.0.113
Platform: win32 10.0.19041
Plugins: clickable-links, relaxed-theme, scrollbar, theme-hype, theme-windows10, title-control
See picture. This happens in all admin shells. No issue in non-admin shells. Arrow keys come up as [D[A[C[B depending on which key you press, powershell core acts even weirder. xterm or hterm does not matter.

You can see in this one that nano does not expand to the entire size of the window

This one pressing enter will not go down to the bottom of the window

Thanks again

+1
The admin tab is usable, if I think about it carefully, but when I'm using admin it tends to be for more dangerous tasks.

FWIW, I found that this doesn't happen when launching the entire Terminus application as Admin. Of course, it isn't really good practice to be running in admin all the time, but it is a workaround if needed.

This is a duplicate of #691 which appears to be closed on inactivity. FWIW, it is still an issue on 120


Fix #778 Enhancement/switch between examples

Fix #778 [Enhancement] Switch between examples.

Added an entry point with a list of examples to choose from
Hash Router to all the existing examples

You can use examples.js to easily add more examples in the future, just add the label and the component to render.

Coverage remained the same at 75.223% when pulling 2b2ebe9e9e7978d80cfe7264d36a0bb61d5bd560 on alielkhateeb:enhancement/switch-between-examples into d1914e4156aea4ff7d6d8ad9794da3ca4b156758 on gregnb:master.

And I also think we could improve the documentation by adding live demo links to each option we have an example for, it would be much easier for the users to have a look on what this option actually does visually.
For example, in the description of customFilterListRender, we could have Demo along with the included Example
Just a thought.

@alielkhateeb Live demo links would be great, but probably make more sense in a full documentation website, like what material ui has. As the library matures we could definitely use something like that, but I think we should hold off for now. I know the repo owner attempted this a while ago, so he may have thoughts on it first.

@alielkhateeb Looks like some of the other PRs were touching the webpack.config.js to switch examples, so that created a conflict for you. After your changes are merged, we'll never need to deal with that again!

@gabrielliwerant Merged.
Yup, no one will have to push this file again 🍾


socket.io not working

Hi,
i am running LightTable 0.4.9 and have a node.js app, that runs fine when started from terminal and used with my default browsers (Firefox, Chromium).
If i open LightTable and add a node.js connection for the app.js, i get the following output when accessing the site (no matter if accessed from LightTable's own browser or my external one):
app.js[stdout]:
info: socket.io started

app.js[stdout]:
Express server listening on port 11907

app.js[stdout]:
debug: served static content /socket.io.js

app.js[stdout]:
info: client protocol version unsupported

The output in the internal browser is:
Failed to load resource: the server responded with a status of 500 (Internal Server Error): http://localhost:11907/socket.io/1/?t=1368688328748

In Firefox with Firebug i get:
socket.io.js (line 1659): Server Response: Protocol version not supported.

If you need further information, please tell me.
I really like LightTable and would like to use it over my current solution, but this, as you might assume, is a blocker for me.
Greetings,
David

@zencodism I don't know of any reason why this would be fixed, but would you mind confirming whether you can reproduce this with a build of the latest source version of the master branch?

I was just able to reproduce this using the latest source version of master (the same version as the latest alpha release).

I wasn't able to reproduce this just now. [I wasn't connecting to the right app.js file!]
In the console I have a new error logged:
Uncaught Error: prompt() is and will not be supported.
override.js [161]	window.prompt
index.html [9]	anonymous
socket.io.js [1335]	Emitter.emit
socket.io.js [754]	Socket.emit
socket.io.js [933]	Socket.onconnect
socket.io.js [833]	Socket.onpacket
socket.io.js [1197]	anonymous
socket.io.js [1335]	Emitter.emit
socket.io.js [395]	Manager.ondecoded
socket.io.js [1197]	anonymous
socket.io.js [1335]	Emitter.emit
socket.io.js [5934]	Decoder.add
socket.io.js [385]	Manager.ondata
socket.io.js [1197]	anonymous
socket.io.js [1335]	Emitter.emit
socket.io.js [1953]	Socket.onPacket
socket.io.js [1771]	anonymous
socket.io.js [1335]	Emitter.emit
socket.io.js [2378]	Transport.onPacket
socket.io.js [3218]	callback
socket.io.js [4661]	anonymous
socket.io.js [4660]	exports.decodePayloadAsBinary
socket.io.js [4428]	exports.decodePayload
socket.io.js [3222]	Polling.onData
socket.io.js [2806]	anonymous
socket.io.js [1335]	Emitter.emit
socket.io.js [2954]	Request.onData
socket.io.js [3025]	Request.onLoad
socket.io.js [2907]	xhr.onreadystatechange

The new error seems like good news. Apparently Electron just doesn't support prompt.

If I replace 'prompt with alert then it works – I can't however enter a name with the alert dialog box – but I can send messages and see them on the page in LT.
@dctr @zencodism If you would, please install the latest alpha 0.8.0 release and confirm whether this has been resolved.

Anyone – comment if this is still a problem.

A permalink for the "Apparently Electron just doesn't support prompt link above":

electron/override.coffee at 30ec7fdeed715f4a8e234bb2557497081f8d1171 · electron/electron


document the vector types and their methods

The various vector types are largely undocumented at this moment in time. Each type should get documentation along with each method. My hope is that we can find a way to write the docs for one type/set of methods, and have that apply to the rest via macros.

All vector types and their methods are already documented.


migrate README to github pages fully

Codecov Report
Patch and project coverage have no change.

Comparison is base (604e47f) 45.60% compared to head (773199f) 45.60%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #106   +/-   ##
=======================================
  Coverage   45.60%   45.60%
=======================================
  Files           8        8
  Lines         842      842
=======================================
  Hits          384      384
  Misses        429      429
  Partials       29       29

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


There is no development team setup by default in Xcode

When trying to setup the Developer Team team in xcode, it's confusing for a new comer. You have to setup a developer team to start with, then you do some Googling how do I do that. Instead, I would recommend adjusting make sure development team is selected is not explicit and could be your user account as well. But this means, in order to do that, you would have to enroll in the Apple Developer program which costs $99. So then the question is, do I have to pay $99 to be able to complete the settings to debug on iOS. I suspect not, but this is the process for either creating then selecting a development team. I would prefer to select my self, and provide alternate directions for teams, or delegate to the Apple guides.
Steps to Reproduce

create plugin
cd into example
run flutter build ios
Follow directions at the end of the execution.

Logs
[✓] Flutter (on Mac OS X 10.12.4 16E195, channel master)
    • Flutter at /Users/branflake2267/git/flutter
    • Framework revision d94943a9ac (3 hours ago), 2017-05-18 18:47:57 -0700
    • Engine revision 82815836eb
    • Tools Dart version 1.23.0-dev.11.11

[✓] Android toolchain - develop for Android devices (Android SDK 25.0.3)
    • Android SDK at /Users/branflake2267/Library/Android/sdk
    • Platform android-25, build-tools 25.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version: OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] iOS toolchain - develop for iOS devices (Xcode 8.3.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.3.2, Build version 8E2002
    • ios-deploy 1.9.1
    • CocoaPods version 1.2.1

[✓] Android Studio (version 2.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Gradle version 3.2
    • Java version: OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] IntelliJ IDEA Ultimate Edition (version 2017.1.2)
    • Dart plugin version 171.4424
    • Flutter plugin version 13.1

[✓] Connected devices
    • None

Here is the error in XCode, related to selecting my account. I'm not enrolled yet.

@xster has been working on auto-provisioning recently.

Hi,
There's indeed a non-trivial amount of initial busywork required for developers who has never developed any iOS software and/or doesn't have an Apple developer account and/or is using a physical device never used for testing. That's the area our current automated flows and documentation is lacking the most as well (#10135, #7839).
The semi-good news is that all of these requirements from Apple only applies to running physical phones so you can test now without doing any of this on a simulator (see the simulator section in https://flutter.io/setup-macos/#ios-setup).
For testing on a physical device, you do unfortunately need to go through the steps of creating an Apple developer account, register the device you want to use for testing's UDID with the account, download the certificates that would be generated for you via Xcode first. We'll do a better job of documenting it with less jargon on the website and in the command line tools.
Regarding your membership program question, you do need to purchase the Apple membership to be able to ship anything either to the App Store or in any packaged way. Though since late 2015, Apple did allow testing on a single physical device for development purposes (looking at your screenshot, you're halfway through that process. You're missing the part where you need to go to https://developer.apple.com/account/ios/certificate/ and register the device you want to use. We'll document that better). Although I haven't tested that mechanism myself yet.

Thanks for the info. That helps.

Going to dupe this into #10446 for concrete additions to the documentation.
In your case, the good news is that you're already 99% of the way there. The free account is totally a thing and all you got to do is plug in a phone and click try again in your screenshot.
You'll have to trust the new certificate on your phone via Settings > General > Device Management. Also you might get an error if someone already registered an app with the ID 'com.yourcompany.myplugin14Example'


Merging into ME2

Hi, are there any particular hurdles that would block this from being added to ME2 proper? Does it interfere with non-Proton/Steam Deck functionality in some way?
Also, ME2 has its own DLL-loading mechanism. Wouldn't it make sense to patch that up to work with Proton instead of EML? Or, perhaps it does already?

Nothing really, the whole feature is compile time option. It does switch the memory scanner to alternative implementation It's 2 commits: https://github.com/Cloudef/ModEngine2/commits/main


Current roadblocks for experimental SDK

First priority issues:

[x] resolve of lbry://@healthdear#e/right-attitude-to-success#a results in invalid channel signature

PR open https://github.com/lbryio/lbry-sdk/pull/3039

[x] Error 500 on requests with unknown or invalid parameters
[x] Reverse compatibility mechanisms (i.e. column effective_amount is not currently recognized)
[ ] Resolve discrepancies

Non-blocking issues:

[x] Proper exit procedure: while the main process quits, forked procedures are still staying in place (Ctrl+C doesn’t work as well)
[x] ZMQ readiness detection (currently just quits if lbrycrd isn't ready for ZMQ requests)

PR open: https://github.com/lbryio/lbry-sdk/pull/3040

[x] Traces are being logged in full instead of one-line entries (ideally they should be sent to Sentry for easier analysis)
[ ] Cryptic logging that doesn't indicate where SDK is at currently (example: lbrynet INFO:lbry.blockchain.lbrycrd:b'93efe9e752bfcc2c3721e51675224cd915b9b7bef58c09013a5cedb49243619e' 36)

@andybeletsky this is not expected to be completed:

Cryptic logging that doesn't indicate where SDK is at currently (example: lbrynet INFO:lbry.blockchain.lbrycrd:b'93efe9e752bfcc2c3721e51675224cd915b9b7bef58c09013a5cedb49243619e' 36)

Please speak up if this is still a priority issue for you.

Closing this with the understanding claim_search must still be fixed. Some URL discrepancies remain but they are trivial and acceptable.

continuing discussion here https://github.com/lbryio/lbrytv/issues/294


As a registered user, I want to be able to edit my publicly visible information, so that I can keep my information up to date.

Duplicate of #42


ensure duplicates (by name+version) are removed

There are situations where the same dependency is reported as both Transient and Direct dependency, in the source.csv file.
Currently the tool does not remove the (name+version) duplicates; please add this feature

Closed by pull request https://github.com/philips-software/blackduck-dependencies-to-reference-format/pulls?q=is%3Apr+is%3Aclosed


Remove free/paid matrix from FAQ; add (out-of-band) promo for support from Tetrate

Per Feb 2, 2021 with Prasad:

[x] remove the free/paid matrix from FAQ
[x] everything on GetIstio is always free
[ ] Need to add a promo for support from Tetrate to the FAQ and the homepage that links to https://tetrate.io/getistio

@oletizi can we close this down?


Don't generate genesis proof at startup

This PR

removes genesis proof generation from the startup code
adds support for generating genesis proofs in the prover process
adds logic to generate a genesis proof when a block producer will need one to be able to produce their block

if the next block for the producer is n slots in the future, the genesis proof will be produced at slot n-1, if it is still needed
if it's needed immediately, the genesis proof will be produced as soon as possible and then used to generate the block

Checklist:

[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them:

how does this interact with the --generate-genesis-proof true flag?

how does this interact with the --generate-genesis-proof true flag?

That flag will still crash the daemon if it's passed as false, but it should already be defaulting to true. Should we disable it completely?

Should we disable it completely?

I vote "yes".

I vote "yes".

Done :)

Done :)

The instructions for running a node will need to reflect this change, when it's released.

The instructions for running a node will need to reflect this change, when it's released.

I've deprecated the flag rather than removing it, so it's now a no-op, in the interest of not breaking existing setups on upgrade. We should still update the docs, but I don't think it's a necessity.

Is the standalone key-downloader tool in a usable state @mrmr1993 ?

@lk86 It's usable, but not very pretty or user friendly. Should we be thinking about a post-inst script to pull the keys automatically when the deb is installed? Or is it worth improving the tool and making that a required step for slow-connection?

Is the standalone key-downloader tool in a usable state @mrmr1993 ?

@lk86 It's usable, but not very pretty or user friendly. Should we be thinking about a post-inst script to pull the keys automatically when the deb is installed? Or is it worth improving the tool and making that a required step for slow connections?

it doesn't have to be pretty or user friendly to replace the functionality in the baked dockerfile and in CI, so that's good enough for me, we can integrate it nicely with the debian package later given that debian users are already dealing with the s3 download logic.


Testing Platform overwriting/removing user console output

Using TUnit, I created this simple test suite:
public class Tests
{
    [Before(TestSession)]
    public static async Task BeforeTestSession()
    {
        Console.WriteLine($"BeforeTestSession started at {DateTime.Now}");
        await Task.Delay(TimeSpan.FromSeconds(5));
        Console.WriteLine($"BeforeTestSession ended at {DateTime.Now}");
    }

    [Before(Assembly)]
    public static async Task BeforeAssembly()
    {
        Console.WriteLine($"BeforeAssembly started at {DateTime.Now}");
        await Task.Delay(TimeSpan.FromSeconds(5));
        Console.WriteLine($"BeforeAssembly ended at {DateTime.Now}");
    }

    [Test]
    public void Test1()
    {
        Console.WriteLine($"Executing Test at {DateTime.Now}");
    }
}

If you just do a dotnet run on the CLI, you'll have to be quick, but when it starts to show "BeforeAssembly started", you'll see it render really quickly, but then it'll be removed and overwritten by the rendering of the [✓0/x0/↓0] ConsoleApp5.dll (net9.0|x64).
I guess this is related to the new interactive output and showing running tests etc. But overwriting user output means they may lose visibility of important information.

I'm off for now so just dumping my thoughts for when I return. We need to try if the behavior is the same when using IOutput to display messages.


chore: add protocol to cozy client js init

If you want to use pouch, you need to specify a protocol for the URL.

Thanks @y-lohse 👍


[SVLS-4918] pin the dd-trace version for node to current

Verified with the selfmonitor apps to confirm that they still deploy correctly. Will follow up with the auto-pull-request script in a separate pull request.

Verified with the selfmonitor apps to confirm that they still deploy correctly. Will follow up with the auto-pull-request script in a separate pull request.

confirmed that this is still working as expected with the new local version variable formats.


[chore] migrating circle ci from 1.0 to 2.0

Title of change

Migrate circle ci to 2.0

Checklist

[x] Manually tested

Screenshot

Deployed to: http://staging.quran.com:32988

@Dhul-Husni can you send an empty commit? Seems like circleci did not pick this up

Deployed to: http://staging.quran.com:32989

@mmahalwy  it still thinks we are on ci 1.0
it doesn't share the same config as master
I have tested it manually both locally and on my forked branch
You could also create a branch on this repo, add this changes and test whether it builds (here)[https://support.circleci.com/hc/en-us/articles/115013849707-Can-I-use-CircleCI-1-0-and-2-0-at-the-same-time-on-a-project-]
other than that it has to be merged to master before builds start running.

I will look into this iA.

I will merge this into master and we can try it, there is no clear way to upgrade C-CI to use version 2 from the web UI.

for visibility, the configuration change seems to work: https://circleci.com/gh/quran/quran.com-frontend/64
@Dhul-Husni Thank you for your contribution. Would be amazing if we can do the same for https://github.com/quran/common-components
jzk

@sabeurthabti  inshaAllah i will do the same

Awesome! Thanks @sabeurthabti and @Dhul-Husni


Update device info in offline API and in some tests

Brings the backend info up to date with current reality.
Also remove a test for constructing backends using a device family (like "H1"), which doesn't work any more.

CI tests failing because of an unrelated credentials issue.


Making the Navbar Responsive

Describe the Bug 🐛
A clear and concise description of what the bug is
Currently, the Navbar isn't responsive properly. If you open it on small screens, you will find some issues with the toggle buttons.
Expected behavior
The Navbar and Toggle Buttons should be responsive for the screens.
Screenshots

Smartphone (please complete the following information):

Device: any
OS: any
Browser: any
Version: any

Hey @JanaTod, would you like to connect for this issue or should I start working on it? Let me know your discord username as well if wanted to connect. Thanks


Best Projects workflow for y'all?

Hey everyone!
We just launched Projects on GitHub—and code review, new profiles, Integrations, GraphQL API, and more :D—and I'd love to know how the Bootstrap community would like us to use the feature here. We haven't been the best at communicating larger changes or what we're actively focused on, and I'd love to give folks better information so they can contribute more easily.

What kind of columns would you like to see?
What's missing from our current milestones setup?
Do our ship lists work well enough for y'all?
Any new labels we should look into (e.g., has-pull-request)?

Holler with any and all feedback. I started putting together one for our next v4 release, Alpha 5, but wanted to pause that until more folks had the opportunity to help direct it's usage :).
<3

The label has-pull-request is a really good idea

Love the has-pull-request or attached-pr labels. It works great for the Rails contributor community. They make great use of this label.
X-ref #20399

Columns in Projects:

Backlog
Help wanted/Approved
Priority/Important/Bug
Pending PR
Test/Review
Good-to-merge
Done/Merged (Milestone/Release)

Ship list:

It would be nice to have kind of an "optical indicator" that show the progress on the current release/milestone. This could encourage people.
It would also be nice to see how (estimated) complex (no-brainer to unsolvable) an issue is. Could be solved with labels.

Add state of PRs via labels or labels within the title
Ex: If a PR is WIP, mark it with a WIP label.
Use the RFC label to request comments
Use the RDY label to let other people know that the PR is ready to merge

The high issues/spare-time ratio of this project means that a lot of issues and PRs to go without any core team response for months, so any mechanism to prioritise and organize work is very welcome. The issue that remains is that there is no mechanism for the community to help triage issues/PRs. There are a lot of, at least from my point of view, "easy-merge" PRs that have stalled, and short of pinging the core team (which I'm very reticent to do given your high workload), there is no way for me to ask for the to be prioritised.

Going with the hybrid of things suggested here and elsewhere. I'll look into the label for things that have a PR attached (usually I just get those merged if they're ready to go in our view).
Thanks y'all!


Adjustment to otpcode parameter on New-TssSession

Summarize Functionality
The current iteration of the otpcode parameter for new-TssSession works for standard OTP scenarios, but I've seen cases where companies add additional codes onto their OTP system making the int value they represent exceeding the upper limit of an Int class.
It would be nice to see the parameter allow larger number values to be used. 6 digit OTP codes with say, 4 additional numbers added at the end puts it in the billions value.
Is there a command that is similiar or close to what you are looking for?
Yes
Technical Details
No response

Closing, this will be worked in #316


#improve docs

Coverage remained the same at 91.72% when pulling 5d3e6dff0a4ce018fe49db90c5f443099d7d5f7f on amolkapadi:patch-1 into f911286ad361c9fba1b422c07f040852c0c193a3 on google:master.

Coverage remained the same at 91.72% when pulling 5d3e6dff0a4ce018fe49db90c5f443099d7d5f7f on amolkapadi:patch-1 into f911286ad361c9fba1b422c07f040852c0c193a3 on google:master.


Deprecate tuple-like access to CircuitInstruction

Summary
This has been the legacy path since CircuitInstruction was added in gh-8093.  It's more performant to use the attribute-access patterns, and with more of the internals moving to Rust and potentially needing more use of additional class methods and attributes, we need to start shifting people away from the old form.

Details and comments
Fix #12631.
The sister PR to Aer that fixes its remaining uses of this is Qiskit/qiskit-aer#2179.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

23 of 24 (95.83%)  changed or added relevant lines in 1 file are covered.
18 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.01%) to 89.764%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

crates/circuit/src/circuit_instruction.rs
23
24
95.83%

Files with Coverage Reduction
New Missed Lines
%

qiskit/transpiler/passes/synthesis/unitary_synthesis.py
2
88.2%

crates/qasm2/src/lex.rs
4
91.6%

crates/qasm2/src/parse.rs
12
97.15%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
63667

Relevant Lines:
70927

💛  - Coveralls

I think, randomized tests is still using this deprecated behavior https://github.com/Qiskit/qiskit/actions/runs/9656096284/job/26633022278


[5.7] Notifications can choose locale

Follow-up to https://github.com/laravel/framework/pull/23178 and https://github.com/laravel/framework/pull/24451, this brings Notification localization behavior in line with Mailable. This allows notifications to choose their locale so when they're queued the subject line, greeting, body, etc. are always running as that locale instead of the app's default locale.
Notification::locale('es')->send(
    $users, new OrderConfirmation($order)
);

Auth::user()->notify(
    (new OrderConfirmation($order))->locale('es')
);

Multiple recipients for more than one language
I have a follow-up pull request to add a HasLocalePreference contract to users/notifiables so you can easily store user settings in the database and have Notification::send($users, new OrderConfirmation($order)); magically send the correct translation to each account.
The mailable-only preview is here: https://github.com/derekmd/framework/commits/mailable-localized-recipients

@derekmd thanks. could you send a PR to docs (5.7 branch) to add this to the release notes with an explanation of how to use it?

Sure, if not today I'll have something in by mid-week.


Remove totallyTyped in favour of errorLevel=1

totallyTyped was deprecated in Psalm 4 and removed in Psalm 5

wrong repo


Fix Segmentation fault / aborted error

Moved the socket implementation over from sslsockets to asyncio streams, as the low-level socket interface was a pain to keep threadsafe.

Tested and works in dev


I don't get this error

I'm not working with JS from some years ago, but I don't get the error, it's taken as in the API docs

I think you need to use a for...of loop instead of a for...in loop.

Seems it's to be var floor of floors

thanks!!


Debug scorecard tests using a debugger

Feature Request
Describe the problem you need a feature to resolve.
A user requested the ability to debug scorecard test.

i'd like to use vscode to debug a scorecard test... can anyone share how this is done? is operator-sdk calling another executable for
scorecard that i need to run in debugger instead of operator-sdk itself?
with some hacking i got scorecard-test/main.go to run

Describe the solution you'd like.

Document how to debug scorecard tests
If needed, change scorecard to make it easier to debug

/language go
/language ansible
/language helm

/cc @thomasmckay

@thomasmckay could you please mention more about the workaround you did to get the debugging on scorecard pods?


Sanity video player play button is now justifying to the left instead of the center of the player

Example of issue can be seen here:
https://www.equinor.com/energy/techstars

@NickHaggerty1 This is fixed and ready to be tested in staging.

@padms ahh ignore that, Its because I had it under the scroll component (which raises another issue but not contected to the video player!


Add support for f32 scalar input in Python bindings

Request description
When calling a VM module function that accepts f32 as input it should support numpy.float32 and zero-dimensional numpy.array. The rational is that in the other cases IREE accepts f32 numpy arrays.
Other data types may be missing this support as well.
For example
func.func @f(%arg0: f32) -> f32 {
  %0 = math.exp2 %arg0 : f32
  return %0 : f32
}

You should be able to do
vm_module.f(numpy.float32(1))
vm_module.f(numpy.array(1, dtype=numpy.float32))

Doing this right now returns
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/petkantchin/ws/iree/build/ninja/RelWithDebInfo/runtime/bindings/python/iree/runtime/function.py", line 130, in __call__
    self._invoke(arg_list, ret_list)
  File "/home/petkantchin/ws/iree/build/ninja/RelWithDebInfo/runtime/bindings/python/iree/runtime/function.py", line 154, in _invoke
    self._vm_context.invoke(self._vm_function, arg_list, ret_list)
RuntimeError: Error invoking function: repo/runtime/src/iree/vm/list.c:710: FAILED_PRECONDITION; variant at index 0 is not a value type

What component(s) does this issue relate to?
Python
Additional context
No response

VM is very low level, it should probably be stricter (e.g., #12967 ) rather than looser.


fix small typo

Just read the post, I think that's not intentional 😅

Okay just re -read the title of the section and I guess it so intentional lol


HBASE-24325: TestJMXConnectorServer can fail to start the minicluster…

… due to it's port already having been chosen by another test.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 21s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
4m 18s
branch-2 passed

+1 :green_heart:
checkstyle
1m 22s
branch-2 passed

+1 :green_heart:
spotbugs
2m 22s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 48s
the patch passed

+1 :green_heart:
checkstyle
1m 17s
hbase-server: The patch generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
13m 24s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
2m 30s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 14s
The patch does not generate ASF License warnings.

38m 19s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1652/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1652

JIRA Issue
HBASE-24325

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle

uname
Linux 76e65d2ebb0a 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 3a375e668c

Max. process+thread count
84 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1652/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 51s
Docker mode activated.

-0 :warning:
yetus
0m  6s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
4m 13s
branch-2 passed

+1 :green_heart:
compile
1m  3s
branch-2 passed

+1 :green_heart:
shadedjars
5m 16s
branch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 42s
hbase-server in branch-2 failed.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 56s
the patch passed

+1 :green_heart:
compile
1m  2s
the patch passed

+1 :green_heart:
javac
1m  2s
the patch passed

+1 :green_heart:
shadedjars
5m 10s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 40s
hbase-server in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
120m 41s
hbase-server in the patch passed.

146m 49s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1652/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1652

JIRA Issue
HBASE-24325

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux cada94e56caa 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 3a375e668c

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1652/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1652/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1652/1/testReport/

Max. process+thread count
3644 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1652/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 33s
Docker mode activated.

-0 :warning:
yetus
0m  6s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 48s
branch-2 passed

+1 :green_heart:
compile
0m 56s
branch-2 passed

+1 :green_heart:
shadedjars
4m 49s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 37s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 27s
the patch passed

+1 :green_heart:
compile
0m 54s
the patch passed

+1 :green_heart:
javac
0m 54s
the patch passed

+1 :green_heart:
shadedjars
4m 46s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 35s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
183m 23s
hbase-server in the patch passed.

205m 35s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1652/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1652

JIRA Issue
HBASE-24325

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 9a910a1ac651 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 3a375e668c

Default Java
1.8.0_232

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1652/1/testReport/

Max. process+thread count
2708 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1652/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

This approach won't work? First it is making its own freeRandomPort when we have one in testing lib and it is possible for the port to be snatched between selection and use. Closing this PR. Put up a new one.


Update spring-cloud-cloudfoundry-connector Jackson dependencies from 2.9.1 to latest version 2.10.0

spring-cloud-cloudfoundry-connector-2.0.6.RELEASE.jar appears to be pulling in older versions of jackson-annotations, jackson-core, and jackson-databind resulting in red flags for tools like the org.owasp:dependency-check-maven plugin.
jackson-annotations, jackson-core, and jackson-databind would benefit from an upgrade to v2.10.0 or later.

Fixed in #305


added the search bar with improved ui

PR Description 📜
I have added the search bar  with animations to make it look good . I was assigned this issue which was raised by me and issue no. is <#116>
Fixes # <your_issue_number>

Mark the task you have completed ✅

[ x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[ x] I have performed a self-review of my own code or work.
[ x] I have commented my code, particularly in hard-to-understand areas.
[ x] My changes generates no new warnings.
[ x

] I have added screenshot for website preview in assets/images

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @gaur15 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Hey @gaur15 also write logic to search the game and then you come up with it !!

Hey @gaur15 we are having search bar PR with logic so we are having your UI of site and also we are confirming your contribution 😃


Unable to desearilize object  Validation error

I am trying to do something new here. So I have the yolov2 model in frozen_pb format from tensorflow. I have successfully converted it to mlmodel and got it working using this repo.
Now, I have a another model where I have quantized the weights in the frozen_pb model from float32 to 8-bits (but the numbers are still in float32 format just the different levels of unique values are now just 8-bits i.e 255 unique float values only). This kinda compresses the model.
I was able to successfully convert the model using the tf-coreml repo. Same as the float32 model.
Now on adding this model to the Xcode.proj , it gives this error.
HERE
There was a problem decoding this CoreML document
validation error : Unable to deserialize object

The model is still float32 data-type (and same size) . Any ideas where it might be going wrong ? Is there a way that the mlmodel protobuf stores differently for lesser unique values ?

That means the Core ML file is not a correct protobuf file, or that the contents in the file are not correct. Most likely something is wrong with the weights for the layers (since that's what you changed) but I can't see that from here. ;-)

I believe the weights of the layers are correct. After quantizing, I do get the expected predictions on the tensorflow platform. Do think the frozen_pb itself is not created correct after the quantization ?
            input_node_names = ["Input/Placeholder"]
            output_node_names = ["Convolutional_9/add"]
            print(input_node_names,"\n",output_node_names)

            gdef = tf.graph_util.convert_variables_to_constants(
                    sess,
                    sess.graph_def,
                    output_node_names)
            gdef = strip_unused_lib.strip_unused(
                    input_graph_def = gdef,
                    input_node_names = input_node_names,
                    output_node_names = output_node_names,
                    placeholder_type_enum = dtypes.float32.as_datatype_enum)
            # Save it to an output file
            frozen_model_file = self.base_dir_out+'/frozen_model.pb'
            with gfile.GFile(frozen_model_file, "wb") as f:
                f.write(gdef.SerializeToString())

This is the same snippet I use to create the model before and after quantizing. Do you think the model frozen_pb from the TF itself is wrong before I bring it to mlmodel ?


Dispatch model loses prediction accuracy because patterns are excluded

Tool
Name: Dispatch
Version: 1.1.2
OS: Windows 10
Describe the bug
Love the dispatch tool!
We have started using the Pattern feature in LUIS, which has proved to be very useful. However, when combining LUIS models with dispatch, it seems the benefits of patterns are lost because there are no entities/patterns transferred to the dispatch model. This means dispatching fails for cases where we have a very high score in the original LUIS model due to a pattern.
To Reproduce

Create an entity 'Person' and populate with a few names ('Dave', 'Sarah', 'Jane' and 'Darth Vader'
Create an intent 'LookupPerson'
Add a few utterances 'Lookup dave's number', tell me 'find me the email of Darth Vader.
Create a pattern entity {Person}
Train and publish the model
Test the model, entering the utterance 'Dave' should get the 'LookupPerson' intent with a very high score
Create a second LUIS model to do something simple, eg tell the time
Use dispatch to combine
Test the dispatch model for utterance 'Dave'. It will not identify this as a 'LookupPerson' model intent

Expected behavior
I would expect patterns to apply to the dispatch model
[bug]

@dividor looking into the matter!
To clarify so I could try to repro most precisely as possible, the Person entity that you're creating is a simple entity, right?
Additionally what is the pattern that you are using? And are you actually testing other utterances besides just a one-worded 'Dave' for example?
It may be more helpful if you could share the exported JSON of your two LUIS apps that Dispatch calls with us.
I will attempt to repro  your results w/o it in the meantime, however given the potential variables, it's possible that the investigation would yield different results

@dividor we will need the JSON of your LUIS apps if you'd like us to investigate further.
And you are correct in that the model created with dispatch does not bring in the entities and patterns from the LUIS models that get added to the dispatch model, and currently this is as intended, correct @tsuwandy?
--however, just speculating from the information given and not knowing for sure without your models, the LUIS app that you created that has the pattern probably should be built in a different way to detect the LookupPerson intent.

--however, just speculating from the information given and not knowing for sure without your models, the LUIS app that you created that has the pattern probably should be built in a different way to detect the LookupPerson intent.

I say this because, if I am understanding you correctly, you are using only "{Person}" as the entire pattern itself like so:

which is actually not the best use of the LUIS Pattern feature, because as stated in the LUIS best practices docs, Patterns should be used to help with intent prediction accuracy by using the combination of word choice and placement of the words. If your pattern is actually just "{Person}", then there's nothing surrounding the placement of your entity to help detect where it should be placed within an utterance.

See repro below and two tests to illustrate:

Created HumanResource app (can find model in article How to add Patterns to improve prediction accuracy)

Created Person entity, LookupPerson intent, and added your example utterances provided in your original post "Lookup dave's number" and  "find me the email of Darth Vader".

Testing

Test results without adding pattern {Person} to model

Notice how when we enter an utterance ("dave") that does not match any example utterances in the LookupPerson intent, LUIS does not detect the utterance to mean the LookupPerson intent in our  model.
Test results with pattern {Person} added to model

Notice how only the utterance "lookup dave's number" is the only utterance that LUIS is more confident about after adding the pattern {Person} to the app, because it

shows up matching an example utterance in the LookupPerson intent following the context of lookup {Person}'s number
"dave" is actually labeled directly on the example utterance, so LUIS can detect that "dave" is a {Person} entity with the 2nd signal of the LUIS pattern

It can't, however, detect that "caroline" is an entity within the pattern itself if the pattern is just {Person}, because there are words surrounding around "caroline" in the utterance "lookup caroline's number", so the utterance does not get an extra signal boost from pattern added to its confidence score.
Test results at the dispatch  model level

Not so surprisingly, the utterance "dave" fails at the dispatch app level as well, just like in the previously tested sub LUIS app.
the other two utterances ("lookup dave's number", "lookup caroline's number") are distinct enough between the two LUIS apps I added to dispatch model (HumanResource app and WeatherV4 app). Just like in our sub-LUIS app, dispatch correctly predicts that they are utterances from an intent within the I_HumanResource app, even without use of patterns in the dispatch model.

Summary
So, you can see that although dispatch models do not import patterns or entities automatically, you can design your model

design your LUIS models to accurately predict intents with good variations in your example utterances (recommended)
manually add entities and patterns in your dispatch model to help point it to the proper sub LUIS model

This issue is probably more suited for StackOverflow.
If you have further how-to-design-your-luis-models questions, please do so there, with a "luis" tag.
Pending confirmation from dev team who works on dispatch to confirm whether or not the lack of entities and patterns in the dispatch model is:

a bug or is actually intended to be built this way currently (what I am not 100% sure of)
should be tagged as feature request or not

Hi Zarryth,
Thanks very much for taking the time to respond.
Actually patterns have a dramatic effect on our production models. We are using them exactly as intended, to boost an intent for looking up a person’s details. We have utterances of “lookup {person}”, but a very common use case is somebody just entering the person’s name “{Person}”.
Previously we used a custom Recognizer to intercept names (6,000 of them in the directory), but with patterns we are able to totally remove that and simply have a single “{Person}” pattern. It works really well for any of the names (and their synonyms) in the directory.
I think dispatcher is a fantastic tool and we are already using it. My point here is that there is a fairly new powerful feature (Patterns) which if excluded means the dispatcher works very different to the sub-model, and so will be incorrect. Our use case might be unusual, but any use of patterns will male dispatcher impossible to use.
Hope that help give more of a picture.
Thanks

@dividor, your point is valid.  I agree if child model makes use of the pattern, then the same should be done on the Dispatch level.

Latest Dispatch with the requested feature is published to npmjs.

Woah! Fantastic! Thanks.

Following up on issue
##Tool
Name: Dispatch
Version: 1.1.5
OS: Windows 10
##Describe the bug
Downloaded new version of botdispatch package (1.1.5) to try to incorporate patterns and entities, yet I continue to get the following error when attempting to import the dispatch model to LUIS:
ERROR: Failed importing  app
##To Reproduce
This came up both when trying to use 'dispatch refresh --dispatch .dispatch' on an existing model, as well as trying to create a brand new model following the documentation steps:
dispatch init -n  ...
dispatch add (models)
dispatch create --dispatch .dispatch...
Following output appears every time before attempting to export dispatch to LUIS:
'''
Exporting services for dispatch...
Exporting  LUIS application...
Creating dispatch LUIS model json...
Creating training data...
Updating  model...
Importing  model...
ERROR
Failed importing  app
'''
Lastly,
I deleted the botdispatch package globally with:
npm -g uninstall botdispatch --save
And proceeded to reinstall old versions, beginning w/ our original version 1.1.2 with command:
npm install<EMAIL_ADDRESS>Attempted to export dispatch model to LUIS again with command
dispatch create --dispatch .dispatch
This worked! (No error)
I continued to upgrade my package to version 1.1.4, and no issues with exporting model to LUIS (while entities/patterns still did not make it to the dispatch from respective child models).
Lastly, tried with version 1.1.5 once again, and ERROR persisted.
Any help regarding the error would be appreciated - Thank you!
[bug]

@tsuwandy
Following up on issue
Tool
Name: Dispatch
Version: 1.1.5
OS: Windows 10
Describe the bug
Downloaded new version of botdispatch package (1.1.5) to try to incorporate patterns and entities, yet I continue to get the following error when attempting to import the dispatch model to LUIS:
ERROR: Failed importing (modelname) app
To Reproduce
This came up both when trying to use 'dispatch refresh --dispatch (modelname).dispatch' on an existing model, as well as trying to create a brand new model following the documentation steps:
dispatch init -n (modelname) ...
dispatch add (models)
dispatch create --dispatch (dispatchname).dispatch...

Following output appears every time before attempting to export dispatch to LUIS:
Exporting services for dispatch...
Exporting (modelname) LUIS application...
Creating dispatch LUIS model json...
Creating training data...
Updating (dispatchname) model...
Importing (dispatchname) model...

ERROR
Failed importing (dispatch) app

Lastly,
I deleted the botdispatch package globally with:
npm -g uninstall botdispatch --save
And proceeded to reinstall old versions, beginning w/ our original version 1.1.2 with command:
npm install<EMAIL_ADDRESS>Attempted to export dispatch model to LUIS again with command
dispatch create --dispatch (modelname).dispatch
This worked! (No error)
I continued to upgrade my package to version 1.1.4, and no issues with exporting model to LUIS (while entities/patterns still did not make it to the dispatch from respective child models).
Lastly, tried with version 1.1.5 once again, and ERROR persisted.
Any help regarding the error would be appreciated - Thank you!
[bug]

@gnoto96, I'm guessing the patterns in one of the child models use ML entities? i'm working to merge the labeling at the utterance level at the moment.  If you could share your .dispatch file (email to<EMAIL_ADDRESS>or the exported child models, I could make sure that the new fix will handle your sub models.  Thanks very much!

@tsuwandy Hello,
Any updates on this? Haven't attempted to refresh the dispatcher w/ the new package since the error. We would still very much like to import our entities and patterns from the child models as our dispatch models are not as strong without these.
Thanks for the help!

@gnoto96, the latest Dispatch is published to npmjs.  Please try it out and let me know if you are still seeing issues.  Thank you!

@tsuwandy
Hey! I updated the models with the new package (1.1.8) and all the entities and patterns from the child-models transferred flawlessly! Thank you so much for the help :D

@tsuwandy
Hello
I installed the new version of botdispatch and got this error while trying to create the LUIS json model:

ERROR
Invalid pattern hello [,] [.] [!] [dear] [!] [.]

Any ideas why my pattern is invalid?
Thanks :)


[SPARK-37402][PYTHON][MLLIB] Inline typehints for pyspark.mllib.clustering

What changes were proposed in this pull request?

This PR migrates type pyspark.mllib.clustering annotations from stub file to inline type hints.
Why are the changes needed?

Part of ongoing migration of type hints.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Existing tests.

Merged into master and branch-3.3.


[Category] Update/category

Add category-controller
Add category-mapper
Add category-service
Add category-repository

The code appears to be quite good; however, I have some suggestions for improvement. Please consider them and proceed with refactoring.


Fix can_remove_subscribers_group_widget dropdown placement.

When creating a new stream, the dropdown is not fully visible.
CZO.

Before
After

Fixes:

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Discussion on CZO ongoing; @amanagr is planning to take a look at this issue.

Closing as the approach is wrong.


Heliport error: itlwm not running

First of all, thanks for your work.
I just upgraded from Ventura to Sequoia using your Heliport EFI. I applied the OCLP patches to get my Broadcom Wifi card (bcm 43xx Edit: BCM94360NG) to work. When I try to start the Heliport app, it says itlwm isn't running. Running $ kextstat | grep -i "itlwm" as described here https://openintelwireless.github.io/itlwm/Troubleshooting.html shows me that itlwm is indeed running. Do you have any ideas what to try or where to look for a solution?

I found this guide (in German...). Turns out I don't need Heliport nor itlwm. FWIW, here's the guide: https://www.hackintosh-forum.de/forum/thread/58656-anleitung-für-broadcom-wlan-patch-unter-macos-sonoma/?postID=796267#post796267


Fixing broken link for MySQL Chrome

Hey @jaywcjlove
just fixing a link.
Cheers!
NOTE: A similar PR may already be submitted! Please search among the Pull request before creating one.
Types of Changes
What types of changes does your PR introduce? Put an x in all boxes that apply

[ ] New addition to list
[x] Fixing bug in existing item in list
[ ] Removing item from list
[ ] Changing structure (organization) of list

Proposed Changes
Describe each of your changes here to communicate to the maintainers why we should accept this pull request.
Broken link fixed
PR Checklist
Put an x in the boxes once you've completed each step. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING guide
[ ] I have explained why this PR is important
[ ] I have added necessary documentation (if appropriate)

Further Comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Thank you @spekulatius

No worries, happy to help out!


TFLite Hexagon Delegate accuracy with Depthwise conv 7x7 filter

Describe the problem
We have a model that has accuracy issue with depthwiseconv, I attached two models, one is right before the depthwise conv node, the result of DSP and CPU has small difference (~0.2), another model has a depthwise conv (7x7 filter) and the result difference is huge (~18.).
I see that there was an issue before with 5x5 filter that got a workaround last year. I am guessing it has the same issue with 7x7 filter: https://github.com/tensorflow/tensorflow/issues/54481
We have qualcomm NDA and we have the source code of nnlib (they don't release update now), wondering if you know where the bug is in nnlib? If so it's best to modify in nnlib side, otherwise maybe you can modify your workaround toward the 7x7 filter?
Source code / logs
dwconv_debug.zip

@tensorbuffer Could you use Post-training quantization techniques instead of quantization-aware training. Also using Integer Quantization (INT8) instead of Float16 (FP16) quantization could help. Please let us know the outcome. Thank you!

Yes that's how this model is generated (post training quantization, and int8):
converter = tf.lite.TFLiteConverter.from_keras_model(model)
converter.optimizations = [tf.lite.Optimize.DEFAULT]
converter.representative_dataset = representative_dataset_gen_rand
converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS_INT8]
tflite_quant_model = converter.convert()
You can just take the model and run on CPU and DSP to compare the result, feed it with a random input. You can see the model before depthwiseconv node has similar outputs, and the model with the depthwiseconv node has quite different results.

Hi @pkgoogle,
Please look into the issue.
Thank You

Hi @sirakiin, can you please take a look? Thanks.

Hi @tensorbuffer,
The NNAPI and Hexagon delegates are deprecated and no longer supported by TensorFlow Lite. For more information, see the NNAPI Migration Guide and TF Lite delegates documentation.
Thank You


spring.cloud.function.rsocket.enabled Boolean property in java.utils.Properties

Describe the bug
Hello after migrating to spring boot 2.7 and spring cloud 3.2.5 there is a "system property" when calling System.getProperties() called spring.cloud.function.rsocket.enabled with Boolean value false. According to the java.utils.Properties docs the values should all be String - "The Properties class represents a persistent set of properties. The Properties can be saved to a stream or loaded from a stream. Each key and its corresponding value in the property list is a string."
Our problem is that we are using hazelcast library that goes through all the properties and tries to cast them to string as follows:
return properties.entrySet()
          .stream()
          .map(e -> new AbstractMap.SimpleEntry<>((String) e.getKey(), (String) e.getValue()))
          .filter(e -> e.getKey().startsWith(prefix))
          .collect(toMap(this::processKey, Map.Entry::getValue));

and it fails with ClassCastException thanks to it. I wasnt able to find to the property being set anywhere but according to the name i came to conclusion that it comes from spring-cloud-function.
I can workaround the problem by setting the property using -Dspring.cloud.function.rsocket.enabled=false but its kinda annoying.
Sample
Can be tested with:
@Test
  void testBooleanValue() {
    assertTrue(System.getProperties().containsKey("spring.cloud.function.rsocket.enabled"));
    assertTrue(!((Boolean) System.getProperties()
        .get("spring.cloud.function.rsocket.enabled")));
    assertThrows(
        ClassCastException.class,
        () -> {
          String test = (String) System.getProperties().get("spring.cloud.function.rsocket.enabled");
        });
  }

It is actually fixed with https://github.com/spring-cloud/spring-cloud-stream/commit/38ce1f79d91b3d94e76b095e2014f9880fca8ddf in spring-cloud-stream which is the only place where this property is set as boolean. It is also back-ported to 4.0.x and 3.2.x


Add auto-generated acapy-client for use in integration tests

This is dependent on some fixes for the integration tests in #20. Marking as draft until we get those fixes merged.

We can probably prune a lot of the swagger.yaml file I used to generate the client. We're only really using the connections, resolver, and didcomm resolver portions. I imagine such a pruning would probably reduce the size of this PR a few tens of thousands of lines.


Not returning A records when queried

Describe the bug
When querying for an A record from a forwarding server, unbound is not returning retrieved A records.
To reproduce
Steps to reproduce the behavior:

Configure a forwarding domain/server
Query for an A record
Observe that while unbound gets A records in it's response from the upstream server, it fails to provide them to the client

Expected behavior
Retrieved A records should be returned.
System:

Unbound version: 1.17
OS: OpenWRT 21.02.3
unbound -V output:

Configure line: --target=mips-openwrt-linux --host=mips-openwrt-linux --build=x86_64-pc-linux-gnu --program-prefix= --program-suffix= --prefix=/usr --exec-prefix=/usr --bindir=/usr/bin --sbindir=/usr/sbin --libexecdir=/usr/lib --sysconfdir=/etc --datadir=/usr/share --localstatedir=/var --mandir=/usr/man --infodir=/usr/info --disable-nls --disable-dsa --disable-gost --enable-allsymbols --enable-ecdsa --enable-tfo-client --enable-tfo-server --with-libexpat=/builder/shared-workdir/build/sdk/staging_dir/target-mips_24kc_musl/usr --with-ssl=/builder/shared-workdir/build/sdk/staging_dir/target-mips_24kc_musl/usr --with-user=unbound --with-run-dir=/var/lib/unbound --with-conf-file=/var/lib/unbound/unbound.conf --with-pidfile=/var/run/unbound.pid --enable-ipset --with-libmnl=/builder/shared-workdir/build/sdk/staging_dir/target-mips_24kc_musl/usr --enable-event-api --with-libevent=/builder/shared-workdir/build/sdk/staging_dir/target-mips_24kc_musl/usr --with-pthreads
Linked libs: pluggable-libevent 2.1.12-stable (it uses epoll), OpenSSL 1.1.1s  1 Nov 2022
Linked modules: dns64 ipset respip validator iterator
TCP Fastopen feature available

BSD licensed, see LICENSE in source package for details.
Report bugs to<EMAIL_ADDRESS>or https://github.com/NLnetLabs/unbound/issues

Additional information
unbound forward configuration:
# unbound-control list_forwards
example.com. IN forward <IP_ADDRESS>
22.75.10.in-addr.arpa. IN forward <IP_ADDRESS>

Query from client:
# nslookup -q=A server.example.com
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53

Non-authoritative answer:
*** Can't find server.example.com: No answer

Log from unbound
Dec 22 11:24:10 ilinx-roam-test unbound: [19223:0] info: 0RDd mod0 rep server.example.com. A IN
Dec 22 11:24:10 ilinx-roam-test unbound: [19223:0] info: iterator operate: query server.example.com. A IN
Dec 22 11:24:10 ilinx-roam-test unbound: [19223:0] info: scrub for example.com. NS IN
Dec 22 11:24:10 ilinx-roam-test unbound: [19223:0] info: response for server.example.com. A IN
Dec 22 11:24:10 ilinx-roam-test unbound: [19223:0] info: reply from <example.com.> <IP_ADDRESS>#53
Dec 22 11:24:10 ilinx-roam-test unbound: [19223:0] info: incoming scrubbed packet: ;; ->>HEADER<<- opcode: QUERY, rcode: NOERROR, id: 0 ;; flags: qr aa rd ra ; QUERY: 1, ANSWER: 0, AUTHORITY: 2, ADDITIONAL: 2  ;; QUESTION SECTION: server.example.com.#011IN#011A  ;; ANSWER SECTION:  ;; AUTHORITY SECTION: example.com.#01172000#011IN#011NS#011server-staging.example.com. example.com.#01172000#011IN#011NS#011server.example.com.  ;; ADDITIONAL SECTION: server-staging.example.com.#0111200#011IN#011AAAA#0112001:123:4567:0:db58:c913:8379:eedf server.example.com.#0111200#011IN#011AAAA#0112001:123:4567::2 ;; MSG SIZE  rcvd: 139
Dec 22 11:24:10 ilinx-roam-test unbound: [19223:0] info: query response was nodata ANSWER
Dec 22 11:24:10 ilinx-roam-test unbound: [19223:0] info: finishing processing for server.example.com. A IN

tcpdump of query and response to forwarding server:
11:24:10.224707 IP (tos 0x0, ttl 64, id 63824, offset 0, flags [none], proto UDP (17), length 79)
    <IP_ADDRESS>.27337 > <IP_ADDRESS>.53: [udp sum ok] 38290+ [1au] A? server.example.com. ar: . OPT UDPsize=1232 DO (51)
11:24:10.225229 IP (tos 0x0, ttl 63, id 62355, offset 0, flags [none], proto UDP (17), length 314)
    <IP_ADDRESS>.53 > <IP_ADDRESS>.27337: [udp sum ok] 38290* q: A? server.example.com. 4/2/6 server.example.com. A <IP_ADDRESS>, server.example.com. A <IP_ADDRESS>, server.example.com. A <IP_ADDRESS>, server.example.com. A <IP_ADDRESS> ns: example.com. NS server-staging.example.com., example.com. NS server.example.com. ar: server-staging.example.com. A <IP_ADDRESS>, server-staging.example.com. AAAA 2001:123:4567:0:db58:c913:8379:eedf, server-staging.example.com. AAAA fd31:aeb1:48df:0:1401:f723:ad21:e146, server.example.com. AAAA 2001:123:4567::2, server.example.com. AAAA fd31:aeb1:48df::2, . OPT UDPsize=1232 DO (286)

Query to forwarding server directly:
# nslookup -q=A server.example.com <IP_ADDRESS>
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53

Name:	server.example.com
Address: <IP_ADDRESS>
Name:	server.example.com
Address: <IP_ADDRESS>
Name:	server.example.com
Address: <IP_ADDRESS>
Name:	server.example.com
Address: <IP_ADDRESS>

I was missing private-domain: example.com


Error: react/rename-unsafe-lifecycles command could not be recognized

While running the command npx codemod react/rename-unsafe-lifecycles --target <path> I'm getting the following response

Even though this solution Originally posted by @parthakshay in https://github.com/reactjs/react-codemod/issues/254#issuecomment-1099067963 works, either the command given in the documentation needs to be fixed or the documentation should be updated.
Happy to take this issue up and raise a PR for this

Hi @shubhraagarwal. Thanks for catching and reporting this! If we can adapt rename-unsafe-lifecycles to codemod CLI and publish it to Codemod Registry so that the command in the readme works, that would be great. Otherwise, we can raise a PR to update the doc.


Lookaround regex not working anymore

Describe the bug
A lookaround regex does not work after updating to last version.
Expected behavior
Using the regex /^Temperatura(?!.*(Gás|Evaporadora))/
I should receive all the temperature sensors data except from "Gás" and "Evaporadora", but none is shown.
Screenshots

Network data
This is the error that shows up in the query.
error parsing regexp: invalid or unsupported Perl syntax: `(?!`

It seems like Go (or Perl) doesn't accept this type of regex, what is weird because this used to work fine in older versions of grafana-zabbix.
Software versions

Grafana
Zabbix
Grafana-Zabbix Plugin

8.5.5
5.4.12
4.2.8

Duplicates #1264.

@tiagohconte This should now be fixed in 4.2.10, so this issue can be closed.


[Feature] Support for Fleet

Is your feature request related to a problem? Please describe.
At least right now, setting up Agent Policies and integration is a point and click operation. This gets especially difficult when setup up multiple different integrations across multiple policies, often with not too much variance.
Describe the resource you would like to have implemented.
https://www.elastic.co/guide/en/fleet/current/fleet-api-docs.html

Definitely would be a fan of this, currently using a standalone agent with terraform yaml template because there isn't a good way to add 500+ integrations to a single policy with many small changes.

I'm super ++ on this.
I would definitely love to be able to set up policies for the agent using Terraform
For now, I assume the only way of doing it is to access the API from terraform as described in https://github.com/elastic/kibana/issues/132263?

ICYMI Fleet support was released with 0.6.0 last week

Support was added in 0.6.0 for some resource types. We are waiting on changes to the Fleet API as tracked in https://github.com/elastic/kibana/issues/155044 before adding a resource type to manage package policies (this is what lets you add an integration package to an agent policy).

elastic/kibana#155044 is closed by elastic/kibana#160921, since some months. Are there any plans to proceed with this one?

I looked a little bit deeper into this topic and I wonder, if waiting for #elastic/kibana/155044 as stated in https://github.com/elastic/terraform-provider-elasticstack/issues/89#issuecomment-1571913779 by @andrewkroh basically means, that the terraform provider will only be able to work with package_policy resources with the new (yet to be released) version of Kibana, since the new API capabilities with the different format setting (simplified) is planned to be used. This would be a major draw-back for me, since it boils down to us not being able to use this terraform provider to manager package_policies until we have updated to the respective version (which might take a while in a corporate environment). This is would be very unfortunate.

For all those watching this issue, looks like the resource side of this functionality was added in v0.10.0 via https://github.com/elastic/terraform-provider-elasticstack/pull/454


Added MATRIX33 aggregate to TypeDesc. Used in OpenEXR input/output

Added a new MATRIX33 to TypeDesc, and made the existing TypeMatrix an alias for TypeMatrix44.
Also changed the c_str representation of matrix to always return the explicit matrix33 or matrix44, though left the fromstring behaviour the same.

This all LGTM except for my reservations about apps that assume that TypeMatrix.c_str() returns "matrix". I need to see how much needs to be fixed on the OSL side, and consider if anybody else would be affected.

See #1267 for my alternate proposal, which is mostly @Shootfast's work, with just a couple minor additions and reverting the "matrix" vs "matrix44" behavior change.

Mark, do you have opinions on #1267? Is that ok with you?

@Shootfast

Subsumed by #1267


Make the debug menu bar to move anywhere. wonder why this is not corrected yet, very annoying 99% of the time.

Found similar issues, but all those are closed, it's super annoying. please find a way, otherwise, this is a superb platform.

AFAIK this is behaving as designed. Why do you find it 'very annoying 99% of the time'?
If it's because you find it gets it the way of your tabs and breadcrumb, have you seen how the debug.toolBarLocation setting allows you to dock it?


where did v0.2.0 go?

I seem to have some changesets relating to this containing 0765d033cf5514107b45a192fead7a9a54a76e0d and 1a3abaa7ea9c07899604883645edf452fdc0e88c but I can't find them.

This is now irrelevant, updated version in the meantime.
I commented on your PR, so we can track this in there.