Add feature azurerm_kubernetes_cluster_trusted_access_role_binding

Description of your changes
Adds azurerm_kubernetes_cluster_trusted_access_role_binding to the authorization provider.

Fixes #
I have:

[X] Read and followed Crossplane's [contribution process].
[X] Run make reviewable to ensure this PR is ready for review.
[-] Added backport release-x.y labels to auto-backport this PR if necessary.

Notes
2 notable things about this PR:

external name configured quite standard, the referenced var exists in the generated init paramenets type file.
Added an additional resource configurator and it's dependency rconfig here so that the resource ends up in the authorization provider as opposed to the containerservice one.

How has this code been tested

I've built the provider as a docker image using make e2e
I've took the image to my company's test environment and deployed the following tempalte:

          ---
          apiVersion: authorization.azure.upbound.io/v1beta1
          kind: TrustedAccessRoleBinding
          metadata:
            annotations:
              gotemplating.fn.crossplane.io/composition-resource-name: trustedaccessrolebinding
            name: "{{ $.observed.composite.resource.metadata.name }}-ta"
          spec:
            forProvider:
              kubernetesClusterIdRef:
                name: {{ .observed.composite.resource.metadata.name }}
              roles:
                - "Microsoft.DataProtection/backupVaults/backup-operator"
              sourceResourceId: "{{ .observed.resources.backupvault.resource.status.atProvider.id }}"
            providerConfigRef:
              name: "{{ $.observed.composite.resource.metadata.name }}"

Then I took a look at the resources:

k get trustedaccessrolebindings.authorization.azure.upbound.io
NAME                    SYNCED   READY   EXTERNAL-NAME           AGE
aks-cluster-3-ta   True     True    aks-cluster-3-ta   13m
aks-cluster-4-ta  True     True    aks-cluster-3-ta  13m

Then I described the resources:

Status:
  At Provider:
    Id:                     /subscriptions/redactedname/resourceGroups/redactednameproviders/Microsoft.ContainerService/managedClusters/redactedname/trustedAccessRoleBindings/redactedname-ta
    Kubernetes Cluster Id:  /subscriptions/redactedname/resourceGroups/redactedname/providers/Microsoft.ContainerService/managedClusters/redactedname
    Roles:
      Microsoft.DataProtection/backupVaults/backup-operator
    Source Resource Id:  /subscriptions/redactedid/resourceGroups/redactedname/providers/Microsoft.DataProtection/backupVaults/redactedname
  Conditions:
    Last Transition Time:  2024-11-07T17:43:35Z
    Reason:                Available
    Status:                True
    Type:                  Ready
    Last Transition Time:  2024-11-07T17:43:34Z
    Reason:                ReconcileSuccess
    Status:                True
    Type:                  Synced
    Last Transition Time:  2024-11-07T17:43:35Z
    Reason:                Success
    Status:                True
    Type:                  LastAsyncOperation
Events:
  Type    Reason                   Age   From                                                                           Message
  ----    ------                   ----  ----                                                                           -------
  Normal  CreatedExternalResource  48s   managed/authorization.azure.upbound.io/v1beta1, kind=trustedaccessrolebinding  Successfully requested creation of external resource

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

As far as I understand, TrustedAccessRoleBinding and Workspace resources failed. Uptest output easily gets crowded, especially when there are multiple dependent resources and the tests are failing. I'm planning to work on a solution. Here's the relevant parts that I could find:
- apiVersion: authorization.azure.upbound.io/v1beta1
  kind: TrustedAccessRoleBinding
status:
    conditions:
    - lastTransitionTime: "2024-11-10T22:36:23Z"
      message: 'cannot resolve references: mg.Spec.ForProvider.SourceResourceID: referenced
        field was empty (referenced resource may not yet be ready)'
      reason: ReconcileError
      status: "False"
      type: Synced
- apiVersion: machinelearningservices.azure.upbound.io/v1beta2
  kind: Workspace
  status:
    conditions:
    - lastTransitionTime: "2024-11-10T22:35:42Z"
      message: |-
        create failed: async create failed: failed to create the resource: ["***"0 creating Workspace (Subscription: "2895a7df-ae9f-41b8-9e78-3ce4926df838"
        Resource Group Name: "example-tarb-rg"
        Workspace Name: "example-tarb-wspace"): polling after CreateOrUpdate: polling failed: the Azure API returned the following error:

        Status: "BadRequest"
        Code: ""
        Message: "Soft-deleted workspace exists. Please purge or recover it. https://aka.ms/wsoftdelete"
        Activity Id: ""

        ---

        API Response:

        ----[start]----
        "***"
          "status": "Failed",
          "error": "***"
            "code": "BadRequest",
            "message": "Soft-deleted workspace exists. Please purge or recover it. https://aka.ms/wsoftdelete"
          "***"
        "***"
        -----[end]-----
          []"***"]
      reason: ReconcileError
      status: "False"
      type: Synced
    - lastTransitionTime: "2024-11-10T22:35:27Z"
      reason: Creating
      status: "False"
      type: Ready
    - lastTransitionTime: "2024-11-10T22:35:42Z"
      message: |-
        async create failed: failed to create the resource: ["***"0 creating Workspace (Subscription: "2895a7df-ae9f-41b8-9e78-3ce4926df838"
        Resource Group Name: "example-tarb-rg"
        Workspace Name: "example-tarb-wspace"): polling after CreateOrUpdate: polling failed: the Azure API returned the following error:

        Status: "BadRequest"
        Code: ""
        Message: "Soft-deleted workspace exists. Please purge or recover it. https://aka.ms/wsoftdelete"
        Activity Id: ""

        ---

        API Response:

        ----[start]----
        "***"
          "status": "Failed",
          "error": "***"
            "code": "BadRequest",
            "message": "Soft-deleted workspace exists. Please purge or recover it. https://aka.ms/wsoftdelete"
          "***"
        "***"
        -----[end]-----
          []"***"]
      reason: AsyncCreateFailure
      status: "False"
      type: LastAsyncOperation

@mergenci Thanks for the efforts to help. That issue is clear, that's just because there is a soft delete retention setting on the vault which apparently can't be disabled and must be set between 7 and 90 days as per this doc.
The only way around this is changing the name until the deletion completes after 7 days.
The run I am curious about is this one where everything seems to have succeeded but the pipeline still does't proceed as it complains about the vault without an error message.

That one seems to be missing the Test status condition in kind: Vault, which usually means that there was an update loop. If so, you can examine pod logs to see if there are any “Diff detected” messages. As far as I know, the only indication of an update loop in Uptest logs is the timeout message:
case.go:363: command "$***KUBECTL*** wait vault.keyvault.azure.upbound.io/example-ai-v --for=condition=Test --timeout 10s" exceeded 7 sec timeout, context deadline exceeded

Hi @turkenf,
Thanks a lot for the response and your suggestions - i've added both of them to the example file.
The "resource exists with the same name" issue is present for all resources in the example file until the deletion topic is solved. I'm not sure if it would be possible to force delete the resource group in azure? In my opinion that would be a good approach since we could end up in this situation in different cases as well. I am not sure if the resources created by my pipeline are running indefinitely in Azure.
Unfortunately I can't test these specific examples manually as I could only test the AKS Trusted Access relation creation for the BackupInstanceKubernetesCluster at my workplace - where my user is only able to create some specific resources which I have to manage.

The "resource exists with the same name" issue is present for all resources in the example file until the deletion topic is solved.

Hi @drew0ps, in fact, all resources except the resource group in the example YAML file are deleted. Just randomizing the name of the workspace resource as you mentioned here is enough.
As I mentioned above, the main problem here is the creation of two extra resources Application Insights Smart Detection and Failure Anomalies. These are not in our YAML file and I think they are created by the API because something went wrong.

I'm not sure if it would be possible to force delete the resource group in azure?

I don't know if this is possible, but even if it was, I wouldn't choose it.

I am not sure if the resources created by my pipeline are running indefinitely in Azure.

I spent some time today cleaning these out of our test account 🙂

Unfortunately I can't test these specific examples manually as I could only test the AKS Trusted Access relation creation for the BackupInstanceKubernetesCluster at my workplace - where my user is only able to create some specific resources which I have to manage.

I understand and I really appreciate your interest, but I prefer not to proceed without understanding why the extra resources are being created and seeing if we can resolve this situation.

Hi @turkenf,
Thanks a lot for the additional insights - randomizing the workspace name makes sense with your explanation.

I spent some time today cleaning these out of our test account 🙂

Sorry about this one, I won't run the pipeline until I figure this out manually. I thought after 6 hours, when the pipeline ends (times out?), the resources are somehow force deleted.

... I prefer not to proceed without understanding why the extra resources are being created and seeing if we can resolve this situation.

Understood - I'll spend a bit more time testing manually on why this happens and how to prevent it. The screenshot you added already helps a lot in this.

👍

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

/examples/alertsmanagement/v1beta1/monitorsmartdetectoralertrule.yaml

/test-examples="examples/alertsmanagement/v1beta1/monitorsmartdetectoralertrule.yaml"

/test-examples="examples/authorization/v1beta1/trustedaccessrolebinding.yaml"

Thanks a lot for the hints - the problem was some missing resources for the Application Insights, which get created by default if not defined in crossplane. Of course when not defined crossplane can't request their deletion. It's fixed now, uptest pipeline is green.

Thank you for your contribution and pushing through with this one @drew0ps


Internal links with middleman not working

I'm trying to render some internal links (h2 & h1) inside the same .md file, but they are not working.
Is this a know issue of middleman or maybe i'm doing something wrong?

Hey shakisha! Do you think you could post a markdown snippet showing the error?

When I am using chinese character as the second level title, the link in the menu is translated.
Like so: 'http://localhost:4567/#TOC_7f24fb2df02b8d9af280e37188785532'
But internal links are not translated. like so: 'http://localhost:4567/#查询大盘列表'

[大盘列表](#查询大盘列表)

Hi Lord,
i tried with the following code creating an internal link for ‘database' , but still doesn’t work:
# Introduction

You can use our API to access Kittn API endpoints, in our [database](##Create a new message).

....

# Voice Messages

##Create a new message

We have language bindings in Shell, Ruby, and Python! You can view code examples in the dark area to the right, and you can switch the programming language of the examples with the tabs in the top right.

You should have the internal link point to the URL hash, which is different from the actual content of the header. So for instance, ## Get All Kittens be linked to like
[link text](#get-all-kittens)

and Susan, your chinese example would be linked to with
[link text](#TOC_7f24fb2df02b8d9af280e37188785532)

Does that solve your problem?

Thanks Lord, I thought slate would automatically convert inner links the same way as api title.

np! closing since it seems resolved for some people? LMK if anyone is still having issues!

Thank’s Lord, problem fixed :)


Not able to drag in all position

Hi Anseki,
I am trying to create something like this https://jsfiddle.net/saurabh97/mc3fe6az/5 but I am not able to drag the box in all direction,for some reason, it's fixed in a certain axis where it's placed. https://jsfiddle.net/saurabh97/zof7qtjw/32/.
And also,
Is it possible to restrict the area in which I can drag these lines?
Is it possible to have these lines how it's drawn in this fiddle https://jsfiddle.net/saurabh97/mc3fe6az/5?
Please help me out here!!
Thanks

Hi @emosrb, thank you for the comment.
Sorry, I couldn't understand what you want well.
Does this example help you?
https://jsfiddle.net/tfhugqvx/

Yes, this is what I wanted, but why this is needed?
.center {
height: 100vh;
}
and also is it possible to restrict the drag area?

I'm glad if I could help you. :smile:
That is PlainDraggable, and this is LeaderLine repository.
Please close this issue, and open new issue in PlainDraggable repository (not here) if you want.
https://anseki.github.io/plain-draggable/


atlassian-confluence: 7.8.0 -> 7.8.1

Motivation for this change
https://confluence.atlassian.com/doc/issues-resolved-in-7-8-1-1027114676.html
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

20.09: 012e55a4606


"Input string was not in a correct format" error

I'm trying to run the  'Offload Excel workbooks' example, but I get an "Input string was not in a correct format" error on the following line:
HPCExcelClient.OpenSession headNode:=HPC_ClusterScheduler, remoteWorkbookPath:=HPCWorkbookPath, UserName:="hpc\azureuser", Password:="<YourPassword>"
Any idea what the issue is?

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

ID: ddcb1d67-87f1-4c96-af24-5dcc57bde92e
Version Independent ID: a29c1cf8-d413-aca7-9190-6007513937b4
Content: HPC Pack cluster for Excel and SOA
Content Source: articles/virtual-machines/windows/excel-cluster-hpcpack.md
Service: virtual-machines-windows
GitHub Login: @dlepow
Microsoft Alias: danlep

Here is my configuration:

I created a cluster on Azure using the "HPC Pack cluster for Excel workloads" template.
I'm using Excel 2016.
I installed the HPC 2012 R2 Update 3 client utilities.
Contrary to what's written in the article, I had to manually browse for the "Microsoft.Hpc.Excel" reference in the VBA project.

Thanks for the feedback! We are currently investigating and will update you shortly.

I also tried with Excel 2013, but I got the same error.
On another computer (still with Excel 2016), I get another error (from the VBA engine): "Automation error. Exception occurred.", then Excel crashes.

@poulinjulien Did you get a chance to try the solution provided in the MSDN thread you posted here.

@poulinjulien Since you are engaged with the engineer on the MSDN thread, I am closing this issue here.
If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

I still have some questions/remarks.

I used "HPC Pack cluster for Excel workloads" from the Azure Marketplace to create the cluster and not the template referenced in the article. Is that still a correct way to instantiate the cluster?
When using that template, the article states that "The steps differ slightly from the following". What are the differences?
Do I need to install Excel (2013?) on the head node if I just want to be able to run a job from a client computer?
The steps about exporting and importing the head node certificate are not very clear. I'm not sure I did it correctly. Is there a way to test it?
I get an error when I try to connect to the head node (using the DNS name) with the 'HPC Cluster Manager' on my client machine. The error I get is "Failed to communicate with remote SDM store. Authentication failed. A call to SSPI failed, see inner exception", or sometimes simply "A call to SSPI failed, see inner exception". Is that normal?
In the repository MicrosoftHPC/HPC2016.SampleCode, the documentation says that I need to take care of uploading the workbook to the compute nodes. Do I really need to do that or is it supposed to be done automatically? Is that documentation up to date?

@Karishma-Tiwari-MSFT, @MicahMcKittrick-MSFT

@amat27 I saw that you were helping out customer on this issue on MSDN thread. Can you please share your expert insights and help answer customer's additional questions on the same?

@poulinjulien

We'll suggest you use ARM template. But both ways should work.
It means you'll go through different pages (ARM template pages vs market place pages).
As long as your head node is not also a compute node, no.
Just to check if the cert is in client machine's Cert:\CurrentUser\Root.
Yes. HPC Cluster Manager cannot connect to a remote Azure cluster. Use Job Manager instead pease.
If you are following this documentation, uploading is taken care of.

Let's put further discussion in the MSDN thread.

@poulinjulien Since you have been helped by our engineer on MSDN, I will close this issue now.
If you have additional questions let me know and we can reopen and continue.


Handle invalid user docid in changes_callback

Overview

Handle user document id without correct id. I simply catch any errors and log a debug message (perhaps we want a info or warn level message?)
Testing recommendations

Disable the VDU in _users database (I removed the validate function)
Set the log level to Debug
Add an valid user without an id (let Couch generate one)
Replicate the user database to another database to trigger the changes_callback function.
You should see the debug level saying that the id is invalid and the cache reader should not throw a function clause.

Related Issues or Pull Requests
#1821

Checklist

[x] Code is written and works correctly;
[ ] Changes are covered by tests;
[ ] Documentation reflects the changes;

@popojargo any plans to get this in? I am preparing to mass-close old PRs that never got merged.


I18n cleanup in User model

The model shouldn't know the full path to the locale error messages,
they should be derived upon validation.

Go! :checkered_flag:


onNodeExpanded and onNodeCollapsed props

Hi, thanks for this great library! I was wondering if you were accepting PRs. I have a need for a JSON viewer that allows persisting the collapse/expansion state and yours seems best suited for it since you have the super useful shouldExpandNode prop that other libraries are missing. Only problem is I don't have a way right now to keep track of those expansion changes as they happen. I have a branch on a fork that implements a basic setup for this using simple onNodeExpanded and onNodeCollapsed callback props. Is this something you would find useful to your other users as well?
Commit on my fork:
https://github.com/ryanteamflow/react-json-view-lite/commit/b913ed0ef404a00ce0141397f6a013c061bea559

oops forgot you can't share fork repos publicly, I'll just go ahead and open a PR but feel free to close if it's not something you're interested in. Thanks!

https://github.com/AnyRoad/react-json-view-lite/pull/18


Solves issue 157 (Updated index.html).Footer improved and originally intended footer implemented by removing syntax errors in code.

PR Description 📜
The html file had the footer code correctly but due to errors in code the footer was not properly visible.I have fixed the errors.
There were some issues in the index,html file due to which the footer was appearing in the unorganized and unclear format.I have fixed the issues and now the footer is appearing in the proper format as intended by the code.I have also changed the footer link colours as the other content was now already visible after fixing the syntax issues in the html code.
Fixes issue #157

[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] My changes generates no new warnings.

[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE

[x] I have added screenshot for review.

## Add your screenshots(Optional) 📸
Before removing code errors and changing some link colours:

After making required changes:

Kindly mark this issue as closed.#157

@ahinagangopadhyay done


Create 5.2 branch

We need to create a 5.2 branch/tag for this vmod. The last working commit is here:
58f0c6d7ce42dc2be5bf505c2afc3cb2d565ab74

58f0c6d7ce42dc2be5bf505c2afc3cb2d565ab74 is now branch 5.2


Explore and decide on a technology stack for UI

Add the suggestions here in the description:
Vinod’s Suggestions:
Frontend Framework: React with TypeScript
Why? - team familiarity, rendering, resuable componenents, static typing, …
HTTP Client: Axios
Why? - easy to use, promise-based, async operatuons
Routing: React Router
Why? - easy to integrate to react
Authentication: JWT
Why? - works well with Axios, standard for transmitting JSON objects
UI Styling: Material-UI
Why? - easy to integrate to react
Possibly:

State Management: React Context API or Redux

Why React Context API? - minimal complexity, lightweight and built into React
Why Redux? - Ideal for larger applications?
Allen's Suggestions:
Framework: React

familiarity
not too much overhead
fast development time

HTTP: Axios or React Query
Axios

easy to use
very lightweight
may need more libraries such as Redux for state management
React Query
good for state management
built in support for React
not as easy to use as Axios

Authentication: JWT (standalone or with a library)

easy to use
token is stored on the client (can be good or bad)

Styling: MUI or Chakra-UI

compatibility with React

Result:
React
Axios
Material UI

Result:
React
Axios
Material UI


Error Message: Class 'League\OAuth2\Client\Provider\GenericProvider' not found

An uncaught Exception was encountered
Type: Error
Message: Class 'League\OAuth2\Client\Provider\GenericProvider' not found
Filename: C:\wamp64\www\CIQB\application\controllers\Qb_oauth_endpoint.php
Line Number: 33
Backtrace:
File: C:\wamp64\www\CIQB\index.php
Line: 315
Function: require_once

I have loaded autoload.php as below.
$config['composer_autoload'] = '../vendor/autoload.php';
Below is my project structure.
I am working in localhost.

Can you please guide me?

Hello, @virenbpanchal :
Thanks for your patience. I'm not sure how Windows manages paths, but it might worth trying to specify the full path to the autoload.php file. If it works, then you would just need to indicate the correct relative path to it.
Also, I'll revisit the steps in order to get sure that Intuit didn't change anything in terms of class names, etc. That might also be a possible reason for the error.

I am still receiving the error listed above no matter what I do.. I have the same file structure as the user listed above but I'm still receiving this error after clicking the Connect Button

Hello, @timburks2392 : I just made a reassessment of the README file in order to get sure everything works as described. You might want to take a look at it in order to know if you missed some detail.

am still receiving the error listed above no matter what I do.. I have the same file structure as the user listed above but I'm still receiving this error after clicking the Connect Button


Pop up an error after copying a new created file share

Storage Explorer Version: 1.3.1
Platform/OS Version: Windows 10/MacOS High Sierra/ Linux Ubuntu 16.04
Architecture: ia32/x64
Build Number: 20180717.9
Commit: 633ce83b
Regression From: Previous release 1.3.1(20180711.1)
Steps to Reproduce:

Create a new file share.
Right click the new file share and click 'Copy File Share'.
Check whether the created file share can be copied successfully or not.

Expected Experience:
The new created file share can be copied successfully.
Actual Experience:
Pop up an error after copying a new created file share.
And the new created file share can be copied successfully after refreshing the 'File Shares' node.

@v-haiz , what was the error?

@MRayermannMSFT Below is the error message:

I can no longer repro this. @v-haiz , please confirm with the latest release candidate build.

@MRayermannMSFT Yes. This issue does not repro in the latest release candidate build(●'◡'●).


DBZ-4367 LogMiner start mining from oldest pending TX instead of snapshot SCN

https://issues.redhat.com/projects/DBZ/issues/DBZ-4367
Supersedes PR #3133

[x] LogMiner tests :heavy_check_mark:
[x] XStream tests :heavy_check_mark:

Applied to main (=1.9) and candidate-1.8.1.Final PR


Cannot inject Service in HostBuilder in .net core 3.0

The repository is available here: https://github.com/ranouf/TestingWithDotNetCore3_0/tree/WithDatabase
I'm looking for a way to upgrade from .Net Core 2.0 to 3.0 my Integration Tests without using WebApplicationFactory.
I really try my best to be able to inject a damned service, as you can see, I tried every where I could:
        var hostBuilder = new HostBuilder()
            .UseServiceProviderFactory(new AutofacServiceProviderFactory())
            .ConfigureLogging(logging =>
            {
                logging.ClearProviders();
                logging.AddXunit(Output);
            })
            .ConfigureServices(services =>
            {
                services.AddAutofac();
                services.AddSingleton<IMyService, MyService>(); //Here
            })
            .ConfigureContainer<ContainerBuilder>(builder =>
            {
                builder.RegisterType<MyService>().As<IMyService>().InstancePerLifetimeScope();//Here
            })
            .ConfigureWebHost(webHost =>
            {
                // Add TestServer
                webHost
                    .UseStartup<TestStartup>()
                    .UseTestServer()
                    .ConfigureServices(services =>
                    {
                        services.AddSingleton<IMyService, MyService>();//Here
                        services.AddAutofac();
                        services
                            .AddControllers()
                            .AddApplicationPart(typeof(TestStartup).Assembly);
                    })
                    .ConfigureTestServices(services =>
                    {
                        services.AddSingleton<IMyService, MyService>();//Here
                        services.AddAutofac();
                        services
                            .AddControllers()
                            .AddApplicationPart(typeof(TestStartup).Assembly);
                    })
                    .ConfigureTestContainer<ContainerBuilder>(builder =>
                    {
                        builder.RegisterType<MyService>().As<IMyService>().InstancePerLifetimeScope();//Here
                    });
            });

But guess what? It doesnt work at all ... I really don't know how I can make the HostBuilder understand ...
Of course I tried to alternativaly keep only one way to inject my service
So when I want to get the Service:
    Host = hostBuilder.Start();
        Server = Host.GetTestServer();
        Client = Host.GetTestClient();

        using (var scope = Host.Services.CreateScope())
        {
            var services = scope.ServiceProvider;
            try
            {
                var myService = services.GetRequiredService<MyService>();
            }
            catch (Exception ex)
            {
                Output.WriteLine("HOST: " + ex.Message);
            }
        }
        using (var scope = Server.Services.CreateScope())
        {
            var services = scope.ServiceProvider;
            try
            {
                var myService = services.GetRequiredService<MyService>();
            }
            catch (Exception ex)
            {
                Output.WriteLine("SERVER: " + ex.Message);
            }
        }

The error message is:

The requested service 'MyAPI.Services.MyService' has not been
registered. To avoid this exception, either register a component to
provide the service, check for service registration using
IsRegistered(), or use the ResolveOptional() method to resolve an
optional dependency.

How do you succeed to inject a service in HostBuilder?

The correct to get the service is:
services.GetRequiredService();
Not
services.GetRequiredService();


[Bug] Scroll placement in the creative inventory

The Empty Large Scroll is around the top in the creative invetory tab of Hex casting, while the other two are around the bottom.
Pretty sure thats a bug?

that's just a forge caching thing, don't worry bout it


Group Connect (Web - Chat app)

Title and Issue number

This PR resolves the issue #1785 . Please review it sir.

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

I have made the changes sir. Please review it

Sure sir , But the Change is Typo in PR and i have corrected it. Please review it sir

@U-DAY-333 anything to be a better review it's good to have a proof.

I have made the changes sir, Please review it


Doesn't run under Windows 7 due to ClientWebSocket dependency

DiscordWebSocket internally uses System.Net.WebSockets.ClientWebSocket. Microsoft openly admits sabotaging their own framework by not providing the necessary "websocket.dll" dependency on Windows 7, Vista and Server 2008, thus artificially limiting it to Windows 8 and Windows Server 2012:
(from https://msdn.microsoft.com/library/system.net.websockets.clientwebsocket(v=vs.110).aspx)
Remarks
[...] the only public implementations of client and server WebSockets are supported on Windows 8 and Windows Server 2012. The class elements in the System.Net.WebSockets namespace that are supported on Windows 7, Windows Vista SP2, and Windows Server 2008 are abstract class elements. [...]
Any chances of dropping this dependency in favor of one of the free WebSocket implementations like
http://sta.github.io/websocket-sharp/
or
http://websocket4net.codeplex.com/
(or rolling your own) and making this project available to a broader audience?

Oh sorry, completely missed this issue.
We recently switched to websocket-sharp anyway due to issues using ClientWebSocket in Mono.


Make JSON schema resilient to any JSON-compatible value as input

The JSON standard defines the following data types

number (WRT ruby both float and integer shall be supported)
string
boolean
array
object (represented as Hash in ruby)
null (= nil)

If any JSON schema receives any value of the types above it shouldn't raise an exception at least. Better yet it should provide a helpful error message.

We want this for 1.0.0?

I guess so, it's just about having a sanity check at the beginning of validation, no?

@flash-gordon yes, JSON is a Processor specialization, it supports adding arbitrary steps, it simply passes result-like objects from step to step, and can aggregate errors in this process. This means you should be able to add a special validation step to do your checks. I suspect we'll want to make step API nicer to work with though, but its core functionality is implemented already.

Can we make :float values in JSON schema to be coercible from integers?
Javascript from {a: 1.0} generates JSON {"a": 1}, and then ruby parses this 1 to integer value.
And json schema returns error that value must be float.
Looks like a bug

@aglushkov we can, but it looks like it would be a breaking change. I think it should be possible to use a custom type container to override json's float as a workaround for the time being.

I reckon we could have something like this @flash-gordon :
Dry::Schema.JSON(Array) do
   # do your thing
end

Dry::Schema.JSON do
   # do your thing
end

API-wise it's not a big deal, though I would expect .JSON and .JSON(Hash) to work the same way, just in case

I mean, no matter how we do this on the user side, dealing with arrays inside dry-schema will be non-trivial

I would propose to introduce the root keyword:
Dry::Schema.JSON do
  root.array(:integer)
end

To implement this I would:

add a special __root__ key to the schema
wrap input with hash before passing to the steps
filter __root__ out from the result
return an error on  "root" schema nesting
handle a special case on passing "root" schema as value

It's a bit hacky, but still, wdyt? @solnic @flash-gordon

Why not having something like .JSONArray instead? It would be easier to implement IMO

besides, the biggest problem I think is producing error messages, not the DSL part

besides, the biggest problem I think is producing error messages, not the DSL part

I think you can pass a special option to the message_compiler to skip the first node of the ast. Still don't know how this will play with dry-v though.

Here's some food for thought:
require 'dry/schema'

Dry::Schema::ProcessorSteps.prepend(Module.new {
  def call(result)
    catch(:halt) { super }
    result
  end
})

class Dry::Schema::MessageCompiler
  def visit_unexpected_input(node, opts)
    input, * = node

    Dry::Schema::Message.new(
      path: [nil],
      predicate: nil,
      text: "we did not expect this",
      input: input
    )
  end
end

schema = Dry::Schema.JSON do
  required(:name).filled(:string)

  before(:key_coercer) do |result|
    unless result.output.is_a?(Hash)
      result.add_error([:unexpected_input, [result.output]])
      throw :halt
    end
  end
end

puts schema.("oops").errors.to_h.inspect
# {nil=>["we did not expect this"]}
#
# `nil` represents input itself in error hash

For anyone in the future, before using catch/throw pls consult me :) Maybe we'll be able to find a better solution.

@flash-gordon we can have result.halt that just sets it internally to a failure state and further processing won't happen

@solnic yeahyeah sure, I just mean using catch/throw without a real need may have undesirable consequences. And even though this doesn't seem to the case here, I'd like to try other options if available


Drop support for cdecimal

From https://stackoverflow.com/a/8192918/284318:

As of Python 3.3, the cdecimal implementation is now the built-in
implementation of the decimal standard library module, so you don't
need to install anything. Just use decimal.

Since combining decimal and cdecimal can result in weird bugs, and
since I will drop Python 2 support in the next release anyways,
supporting cdecimal doesn't make sense anymore.

Coverage decreased (-1.3%) to 61.111% when pulling 69b247f8949d6b74a1e7d0ec4eb3e0f660b361bb on drop-cdecimal into 7f22f98e906d46bee487ce2d78fb995f74c1d568 on master.


branch copr builds are temporary

This is more of a question: I just got a question from @sopos about this copr project:
https://copr.fedorainfracloud.org/coprs/packit/beakerlib-beakerlib-master/
The ask was to remove the "temporary project" configuration so that the project persists as the team is using it in their automation. I logged in as packit to copr and did the configuration manually.
What should be a long-term answer here? Should people do this in their packit.yaml or should we change our defaults?

I think it is not necessary to make it forever. It might be enough to be able to configure some TTL value. E.g. for a live project with kind-of regular release cycle it might be enough to set TTL to a few month.
Specifically for beakerlib I would set rather a year as there are usually few 1-2 releases per year.
So generally speaking configurable setting from .packit would be nice.

ah oki, now I understand your ask
you can actually do that: https://packit.dev/docs/configuration/#supported-jobs
preserve_project – The project will not be removed after 60 days if set to True.
Defaults to False. The value is represented as delete_after_days in Copr project
settings (True is -1 and False is 60).

so this should do it:
- job: copr_build
  trigger: commit
  metadata:
    branch: master
    targets:
    - fedora-all
    - centos-stream-8
    - centos-stream-9
    - epel-6
    - epel-all
    preserve_project: 365

Just a slight correction -- it's True/False in our config as it is written in the docs.

Just a slight correction -- it's True/False in our config as it is written in the docs.

I'm confused a bit. Can I actually define a custom number or must I use True/False?

You must use True/False since we wanted to make it easy -- to preserve or not. (True  is converted to -1=forever and False to 60.)
But if you really need a custom value, we can think about supporting both boolean and integer.

sorry about the confusion, I read it wrong, will update


Problem in uploading photo

When I tried to edit a cafe to upload a photo It's not working and after some investigation I figured out it's related to PUT method and multipart/form-data so I added:
formData.append('_method', 'PUT')
to putEditCafe in api/cafe.js and changed axios method from put to post so the problem solved in this way.

Awesome thanks for finding that! I made the fix and pushed this into the repo!


first integration test version

This PR implements a first integration test first (#12)
$ make tests
Starting minikube testing setup ... please wait ...
1..12
ok 1 deploy csi-lvm-controller
ok 2 csi-lvm-controller running
ok 3 deploy linear pod
ok 4 deploy block pod
ok 5 linear pvc bound
ok 6 linear pod running
ok 7 block pvc bound
ok 8 block pod running
ok 9 delete linear pod
ok 10 delete block pod
ok 11 linear pvc deleted
ok 12 block pvc deleted
🔥  Deleting "minikube" in kvm2 ...
💀  Removed all traces of the "minikube" cluster.

Great, next step should be to make this runnable in the CI pipeline. Maybe with kind.


Event cleanup

Cleanup event listeners when they get re-added on update.

The performance improvement comes from multiple interactions with the drag-n-drop-items.
Using the current branch, we noticed the steadily increasing time of execution when clicking on the drag elements of multiple items using Chrome's performance tools:

Click Event Listener Time:
1: 30ms
2: 43ms
3: 81ms
4: 108ms
5: 209ms
6: 347ms
7: 662ms
8: 1180ms
However on the PR branch the same click events execution time change to :

Click Event Listener Time:
1: 18ms
2: 25ms
3: 23ms
4: 22ms
5: 21ms
6: 24ms
7: 25ms
8: 20ms
The root cause of this is that when componentDidUpdate gets called, new event listeners are getting added to the dragPointRef for the click event(as well as other events).  However with javascript, elements can have multiple listeners on elements for the same events.  This means that as a user drags and drops multiple items, they will get duplicate event listeners added to the dragPointRef, which all require memory to resolve.  This can eventually cause the browser to crash.

I got it now. Good findings!

Merged and npm package version bumped to<EMAIL_ADDRESS>

@iterativo , this PR may help improve DnD in Attendance as well.


Autocrafting: Crafted item not detected

Does autocraft 16k Storage Block, but does not detect when crafted. Tried to take it out of system and back in, still not detecting. Didn't seen problem on other items, but dint tested on other storage blocks.

Minecraft: 1.10.2
Forge: <IP_ADDRESS>1
Refined Storage: 1.2.6

@raoulvdberge related to some NBT weirdness when inserting RS items that can store items in the grid. Eg. when inserting a storage disk the protocol:1 tag will vanish. Or when inserting a storage block its tag will go from {Storage:{Items:[], stored:0}} to {Items:[], stored:0}. This is what breaks the insert checks.


pthread_cond: Use Task Notifications

Resolves #8
Description of changes:
Update the pthread_cond_t implementation to utilize Task Notifications. In the existing implementation, if multiple threads of different priority are blocked on the same condition, a higher priority thread, in most cases, will unblock multiple times due to taking the underlying semaphore multiple times instead of just once. Switching to Task notifications guarantees that all tasks are equally notified and unblocked.
pthread_cond_signal has also been updated to conform to the POSIX specification in that it will unblock the highest priority task waiting on the condition.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for your contribution, I'll forward this to team


Fix http test

This should fix the test

Hey Erik, I have rebased my feature/http-management-test branch against master which now contains the fixes required to avoid the OpenJDK issues we've seen.
I've also raised a test branch to check if CI on this branch (once rebased against miles/feature/http-management-test) looks good in CI.
Everything seems to check out, so if you could (1) clean up the commit history here and (2) rebase against my branch, I'll then merge this into mine, then mine into master, and this piece of work will be complete :)

@emerkle826 I'm merging this in to form a basis for the integration branch I'm creating.


chore: use cimg/postgres

Our CI build uses circleci/postgres:11.1-alpine-ram
This is a deprecated image, but we can't use cimg/postgres as it does not have the FR locale https://github.com/CircleCI-Public/cimg-postgres/issues/17

Just FYI, for now I have solved my problem using postgres:9-alpine which does have the swedish locale pre-installed. So you could try postgres:11-alpine and see if it works for FR.

Btw, I am now referring to the official postgres images, not the circle convenience images.
https://hub.docker.com/_/postgres

@hwsoderlund Thanks for letting us know! I'll give that a try 👍

Thanks @hwsoderlund : the official postgres image does the job


Create additional profile for Huawei nodes

On Huawei devices need to operate in two different modes.
Would be good to make proper recognation each of the modes.
it is related login prompt > or ].
for example:
screen-length 0  - works in > mode only
display interface description - works in boths modes
display current-users  - works in ] mode only
quit  -  logout from ] and > modes

fixed in main_dev


Installation has a broken link for Installation for Azure DevOps

https://fluxcd.io/docs/installation/ link to Azure DevOps should point to https://fluxcd.io/docs/use-cases/azure/#flux-installation-for-azure-devops it is pointing to https://fluxcd.io/docs/use-cases/azure.md#flux-installation-for-azure-devops instead which causes a broken link.

Thanks for the report! I found the docker-support stuff that is meant to make the website repo easy to maintain needs a bit of updating. I'll close both of these issues in a PR together.


Feature: Automatically update date differences

What feature or improvement do you think would benefit Files?
Currently, when a date is displayed by difference, its value is not updated unless the page is refreshed. Thus, creating a new file multiple times will all show "Now" or "1 second ago" even though they actually have different dates. If the date is displayed by difference, the value should be automatically updated.
Requirements
If the date is displayed by difference, the value should be automatically updated.
Files Version
<IP_ADDRESS>
Windows Version
10.0.19045.3208
Comments
No response

Thank you for the feedback, I added this to the project board.


LEGO_EMAIL value from the kube-lego chart is bogus and invalid

Per discussion on https://github.com/jetstack/kube-lego/issues/131#issuecomment-290692369, the default value for the LEGO_EMAIL will not allow a user of the kube-lego chart to receive certificate expiry emails. If the LEGO_EMAIL default is removed from the chart, the first time a user tries to use the chart, they will receive an early fail indicating they need to set a valid email address.
The specific line of code I am talking about is https://github.com/kubernetes/charts/blob/master/stable/kube-lego/values.yaml#L7

Resolved by #912


Enter key does not submit login form

🐛 Bug Report
When I press the enter key in the login form, it should submit a login request.
To Reproduce
Steps to reproduce the behavior:

Enter username
Enter password
Click enter
see that it doesn't submit a login form

Expected behavior
It should submit the login form

I can fix this if no one has claimed it yet.

@kilcorse-michael i've assigned this to you

closes via #2221


Can't upload file

I'm trying to upload a file (it might be a bit large, but it's not giant) and I get this error:
Traceback (most recent call last):
File "/home/pi/backup.py", line 35, in
returned_item = client.item(drive='me', id='root').children['backup'+str(time())+".zip"].upload('./backup.zip')
File "/home/pi/.local/lib/python3.7/site-packages/onedrivesdk/request/item_request_builder.py", line 117, in upload
return self.content.request().upload(local_path)
File "/home/pi/.local/lib/python3.7/site-packages/onedrivesdk/request/item_content_request.py", line 60, in upload
entity = Item(json.loads(entity_response.content))
File "/usr/lib/python3.7/json/init.py", line 348, in loads
return _default_decoder.decode(s)
File "/usr/lib/python3.7/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/lib/python3.7/json/decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
Do you know what the source of this is?

Looking at the stack trace a bit more, I see that I think it uploaded it but it wasn't able to parse the response.

I can manually upload it just fine. If I look at the files, I think it wasn't able to complete a finalization step or something, because the program uploaded version doesn't work.

The solution to #32 worked. Can you add a progress indicator please, though?

Found the progress indicator.


[Feat]Implement Dark Mode Theme switcher for Improved User Experience

Description
Many users prefer dark mode for better readability and reduced eye strain, especially in low-light environments. The current light theme may not be ideal for users who work at night or prefer a darker aesthetic. Adding dark mode will enhance user comfort and accessibility.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@RaiDevX8 Go for it

@Shiva-Bajpai can you give labels please

Hey @RaiDevX8, dark mode is a very good feature, but we have not decided to add the dark mode feature, we will let you know if it is needed in the future. Thank you for raising the issue.

Please don't forget to give our repository a star and let me know once you've done it.


Fixes device nil issue with pipelines

This fixes #6.

Made the default of torch device a cpu for pipelines to specify a device on every pipeline
Also added the ability to create a pipeline's device as either String class or Torch::Device class

Thanks @kaelonswanton!


Motion sensors connected via JA-116H sometimes yield DEVICE_PACKET_TYPE_BATTERY packets

My (non-bus) motion sensors that are connected via JA-116H yield packets of type DEVICE_PACKET_TYPE_BATTERY when motion is detected:
2022-07-05 08:54:29 ERROR (ThreadPoolExecutor-2_0) [custom_components.jablotron100] Unknown state packet (device 26): 5508a4d28006d0d6a838 022-07-05 08:54:38 ERROR (ThreadPoolExecutor-2_0) [custom_components.jablotron100] Unknown state packet (device 24): 5508a4ca000660d72839 2022-07-05 08:54:40 ERROR (ThreadPoolExecutor-2_0) [custom_components.jablotron100] Unknown state packet (device 25): 5508a4cc400680d74839
However, the state change is correctly registered by home assistant:

Analyzing the packets shows that the error is thrown here because the packet type is DEVICE_PACKET_TYPE_BATTERY:
/opt/homeassistant/bin/python3.9
>>> from jablotron100 import jablotron, const
>>> import binascii
>>> sh= '5508a4c8000640d70839'
>>> sb = binascii.unhexlify(sh)
>>> jablotron.Jablotron._convert_jablotron_device_state_to_state(sb, 24)
'on'
>>> jablotron.Jablotron._bytes_to_binary(sb[2:3])
'10100100'
>>> packet_state_binary = _
>>> jablotron.Jablotron.binary_to_int(packet_state_binary[5:])
4
>>> packet_type = _
>>> packet_type == const.DEVICE_PACKET_TYPE_BATTERY
True

I’m sorry I don’t understand.

I’m sorry I don’t understand.
Sorry. I accidently submitted the issue while still typing the description. :-)
I have now updated the text.

Do the sensors have battery or not?

No. They are old-style wired IR sensors with 12V power and NO/NC contacts. Power and signal lines are connected to this expansion card
https://www.jablotron.com/underwood/download/pdf/en/JA-116H.pdf

@leo-b Can you enable the debug and sabotage the device 24? You can try to open it or something similar. It should log packets that may help.

Should be resolved in https://github.com/kukulich/home-assistant-jablotron100/releases/tag/3.14.0

Should be resolved in https://github.com/kukulich/home-assistant-jablotron100/releases/tag/3.14.0

Confirmed.
Thanks!


[WIP] Fix opencv

Description:
Refactor OpenCV to follow the other image processors
Tested on docker, debian 8, ubuntu 16.04 server, ubuntu 16.04 desktop
Waiting on validation from @arsaboo - he is getting a weird error that I'm unable to reproduce
Related issue (if applicable): fixes #7798
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#2785
Example entry for configuration.yaml (if applicable):
image_processing:
  - platform: opencv
    name: OpenCV
    source:
      - entity_id: camera.front_door
    classifier:
      faces:
        file: /path/to/classifier
        name: Husband
        neighbors: 4
        min_size: (40, 40)
        scale: 1.1

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

def setup_platform(hass, config, add_devices, discovery_info=None):
    """Set up the OpenCV image processing platform."""
    try:
        import cv2
    except ImportError:
        _LOGGER.error("No opencv library found! http://url_to_our_docu_for_opencv")
        return

After that we add a note to documentation, that it Need to install opencv from source or on i386/amd64 it give a custom pip module with opencv binarys

Alright, I will add this when I get home this evening - I assumed it would be more complex than that lol.

Good work :+1:

Hi guys, i've used the last docker image to get the fixed code. Now when i use the config
https://github.com/home-assistant/home-assistant.github.io/blob/next/source/_components/image_processing.opencv.markdown i get some errors in the log.
Errors: 2017-06-13 16:58:42 ERROR (MainThread) [homeassistant.config] Invalid config for [image_processing.opencv]: [mom] is an invalid option for [image_processing.opencv]. Check: image_processing.opencv->classifier->0->mom. (See ?, line ?). Please check the docs at https://home-assistant.io/components/image_processing.opencv/
When i use this config i become this errors:
Config: `image_processing:

platform: opencv
name: Front Door Faces
entity_id:

camera.local_file
classifier:
file_path: /config/haarcascade_eye.xml`

Error: 2017-06-13 16:49:17 WARNING (<concurrent.futures.thread.ThreadPoolExecutor object at 0x7ffbb7438358>_5) [homeassistant.components.emulated_hue] Alexa type is deprecated and will be removed in afuture version 2017-06-13 16:49:17 WARNING (<concurrent.futures.thread.ThreadPoolExecutor object at 0x7ffbb7438358>_5) [homeassistant.components.emulated_hue] Listen IP address not specified, auto-detected address is <IP_ADDRESS> 2017-06-13 16:49:17 ERROR (MainThread) [homeassistant.setup] Error during setup of component opencv Traceback (most recent call last):   File "/usr/src/app/homeassistant/setup.py", line 190, in _async_setup_component     component.setup, hass, processed_config)   File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 55, in run     result = self.fn(*self.args, **self.kwargs)   File "/usr/src/app/homeassistant/components/opencv.py", line 170, in setup     for group in config[DOMAIN][CONF_GROUPS]: KeyError: 'opencv' 2017-06-13 16:49:17 ERROR (MainThread) [homeassistant.setup] Unable to setup dependencies of image_processing.opencv. Setup failed for dependencies: opencv 2017-06-13 16:49:17 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform image_processing.opencv: Could not setup all dependencies.
What am I doing wrong?

image_processing:
  - platform: opencv
    name: OpenCV
    source:
      - entity_id: camera.front_door
    classifier:
      faces:
        file: /path/to/classifier
        name: Husband
        neighbors: 4
        min_size: (40, 40)
        scale: 1.1

You're using very incorrect config. Your camera should be under source:entity_id, file_path is not a configuration option and it should be under classfier not entity_id

Thanks for your answer. Now i've tryed your config but it doesen't work
Error:
`2017-06-13 22:15:56 ERROR (MainThread) [homeassistant.config] Invalid config for [image_processing.opencv]: [faces] is an invalid option for [image_processing.opencv]. Check: image_processing.opencv->classifier->0->faces. (See ?, line ?). Please check the docs at https://home-assistant.io/components/image_processing.opencv/
Config:
`camera:

platform: local_file
file_path: /config/image.jpg

image_processing:

platform: opencv
name: OpenCV
source:

entity_id: camera.local_file
classifier:
faces:
file: /config/haarcascade_eye.xml
name: Husband
neighbors: 4
min_size: (40, 40)
scale: 1.1

For your info i use the last Docker build from today!
Thanks

@DroidC-3PO It may be better to file a bug and discuss this issue there so that others can also follow.

 This is not a big at all.
On Jun 13, 2017 2:28 PM, "DroidC-3PO"<EMAIL_ADDRESS>wrote:

Thanks for your answer. Now i've tryed your config but it doesen't work
Error:
`2017-06-13 22:15:56 ERROR (MainThread) [homeassistant.config] Invalid
config for [image_processing.opencv]: [faces] is an invalid option for
[image_processing.opencv]. Check: image_processing.opencv->classifier->0->faces.
(See ?, line ?). Please check the docs at https://home-assistant.io/
components/image_processing.opencv/
Config:
`camera:

platform: local_file
file_path: /config/image.jpg

image_processing:

platform: opencv
name: OpenCV
source:

entity_id: camera.local_file
classifier:
faces:
file: /config/haarcascade_eye.xml
name: Husband
neighbors: 4
min_size: (40, 40)
scale: 1.1

For your info i use the last Docker build from today!
Thanks
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/7864#issuecomment-308238458,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC2fZbBGTX6gCR-U8Vq6WllJoAHGuRB2ks5sDvDbgaJpZM4NtzAg
.

Classifier should be at the same indentation as source, faces indented,
then everyone's faces indented again.
On Jun 13, 2017 2:28 PM, "DroidC-3PO"<EMAIL_ADDRESS>wrote:

Thanks for your answer. Now i've tryed your config but it doesen't work
Error:
`2017-06-13 22:15:56 ERROR (MainThread) [homeassistant.config] Invalid
config for [image_processing.opencv]: [faces] is an invalid option for
[image_processing.opencv]. Check: image_processing.opencv->classifier->0->faces.
(See ?, line ?). Please check the docs at https://home-assistant.io/
components/image_processing.opencv/
Config:
`camera:

platform: local_file
file_path: /config/image.jpg

image_processing:

platform: opencv
name: OpenCV
source:

entity_id: camera.local_file
classifier:
faces:
file: /config/haarcascade_eye.xml
name: Husband
neighbors: 4
min_size: (40, 40)
scale: 1.1

For your info i use the last Docker build from today!
Thanks
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/7864#issuecomment-308238458,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC2fZbBGTX6gCR-U8Vq6WllJoAHGuRB2ks5sDvDbgaJpZM4NtzAg
.

That's odd, your error message is making it seem like it's converting the classifier (an object) into a list image_processing.opencv->classifier->0->faces (see the 0 in there). I can't fiddle with this right now, but I will look some time today or tomorrow. You can also try the simple configuration for now:
image_processing:
  - platform: opencv
    name: OpenCV
    source:
      - entity_id: camera.local_file
    classifier:
      faces: /config/haarcascade_eye.xml


Fix default builder leak and make a warning non fatal in criterion's test.

Catch parses expression to detect operators at build time by overloading operators. This leaded gcc to warn that some expression might be ambiguous after macro extension.
Port C++14's make_unique.
Manage default builders with unique_ptr to guard again leaks at compile time.

5c1a5cd
:-1:

:+1:


Initial EL (RHEL, CentOS, Fedora, Scientific Linux, Amazon Linux, etc) support

This change introduces support for RedHat, Fedora, CentOS, Scientific Linux,
Amazon Linux, etc. The support isn't complete, in that master/slave mode
isn't supported yet, however this is handled with a graceful error
message.
The biggest adjustment is moving all the params into a dedicated
params.pp file and adding the complete set of configuration files to the
modules.
It also fixes issue #25 as it ensures consistent behaviour regardless of
variations in package manager between different distributions.

I'd like to have master/slave working for EL as well, but it's likely to be a while before I have time to attack that and figure you might like to get standalone EL support merged sooner rather than later.
I've tested with Ubuntu 14.04, Fedora 22 and CentOS 7 in standalone modes, but NOT master/slave mode, so please conduct some additional tests there to make 100% sure it wasn't impacted in any way. There should be no change to Ubuntu systems other than Puppet taking ownership of all the default Smokeping configuration files, not just the ones we tweak.

Fedora ships with it's own Smokeping RPM, however CentOS lacks one - I tested with a copy of the Fedora one backported (available at https://repos.jethrocarr.com/) which worked fine.

Thanks a lot for your work. I'll give it a try soon merge your PR then for having a new module release soon.

Thanks @tobru, let me know if you run into any issues or need any tweaks.


MessagePackHubProtocol and unions

The current version of the method  MessagePackHubProtocol.WriteArgument only takes two arguments:
private void WriteArgument(object argument, Stream stream)
This is fine for regular serializations, but it is failing when an Interface marked with the Messagepack.UnionAttribute is used as an argument. This happens because the current implementation infer the type to pass to the serializer from a call to GetType(), while for the case of the union the interface type should be used.
A possible solution would be to turn the WriteArgument method into a 3 arguments method, providing the real type of the argument along with its value

Can you provide an example so that we can reproduce the issue?

Closing this as we haven't heard from you. Please feel free to comment if you're able to get the information we're looking for and we can reopen the issue if we're able to identify the problem.


How to enable multiple namespaces on linux AKS with AGIC without using helm chart ?

Problem
I have a single kubernetes cluster with 2 node pools and 2 namespaces (A,B) and 2 applications(C,D). I want to use a single Azure app gateway for both of the application's traffic. How can I configure on existing AGIC for  routing traffic to services running on multiple namespaces ?

This is exactly my current requirement. Is it possible to accomplish the stated question?


[WIP] Add send pacing

Is this ready for review? I haven't taken a look yet, as it still has WIP in the title ;)

No, I don't have the time.
I haven't implemented the comment from #355 if you're wondering.
But I do plan to pick it up again (probably in september).

Okay, no rush :)

Hi @twdkeule, what's the status of this PR? Do you need help anywhere?
I spent some time working on a decent benchmarking suite for quic-go (see the https://github.com/lucas-clemente/quic-go/tree/tcp-benchmark branch), which compares the transfer rate with a TLS/TCP transfer under a wide range of network conditions. I'm really curious to see how packet pacing will improve things here.

 I will see if I have pushed all my changes

Has there been any movement on this?

No. :disappointed:


EnShowPackage Throws Exception

the append_line command shortcut was mistakenly removed in a recent pull. This breaks the show package functionality.

Introduced by #230 , probably a typo.


Css prop With Typescript and transpileOnly: true fail compile

Hello,
using (on webpack config)
{
   loader: "ts-loader",
   options: {
      transpileOnly: true
   }
}

the prop css does not compiled to class, removing this part from webpack it works.
Is this a bug or am i missing something?

Hey there, we don't deal with TS in this repo so you're better off asking in our Spectrum community or on a help forum like Stack Overflow.


Update pull request template

The last PR #265 did not fix the template. Maybe this has to be on a different path .github/pull_request_template.md if we don't use it with the template query parameter, see: https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/creating-a-pull-request-template-for-your-repository

Yeah, now it works. I'll issue PRs for the other repos to fix this.


Allow for multiple endpoints and many-to-many mappings of endpoints to sinks

This is substantial change that allows to define multiple endpoints as event sources and route events through different mappings, using different schema's and sink them into different sinks (multiple HDFS locations, multiple Kafka topics).

Is there a need to change the MappingTestServer to incorporate the changes using multiple schema's?
I think it can be used as is since you can specify a schema on the cmd line, but i think it would be useful if it takes the schema location(s) from the config

Good morning,
I'm trying to send 2 custom events to kafka; the idea is send each event to its topic; but at the moment I receive both events in both topics.
I have configured 2 mappings with different schemas and 2 sinks (one for each mapping).
I don't know what is exactly happening, do you hace any idea?
Thanks !
Manu

Did you find any solution for that ? I'm also running in the same issue.


android:checked attribute ignored

Could you please support android:checked attribute to preset default state in XML?

Set checked attribute requires, by default a boolean value.
In order to preset a default state it means to set it with the index of the button to be checked.
Unfortunately this is not standard.
You can easily achieve this progtammatically.


Change http_parse_headers function in rest_proxy.php

Change "http_parse_headers" function to avoid deprecated errors in php5.5.
Also is faster as no regexp usage.

With current version of "http_parse_headers" I was getting errors via ajax on my local dev machine. This prevented jquery parsing the JSON.
Obviously on production errors are off but figured this is worth updating due to it being deprecated in php5.5 anyway.


Update README.md

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


Is TypeScript supported?

Our vueJs project is using typescript, so I was wondering if v-idle supports it and if so, how I can install it accordingly.
When installing according to readme i receive the following error, pointing to the import statement:
TS7016: Could not find a declaration file for module 'v-idle'. '/home/masus/workspace/ethz/ape-web-editor/frontend/ape-vue-app/node_modules/v-idle/build/vidle.min.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/v-idle` if it exists or add a new declaration (.d.ts) file containing `declare module 'v-idle';`

    10 |
  > 11 | import Vidle from 'v-idle'
       |                   ^^^^^^^^
    12 |

@masus04
Please check here:
https://github.com/eladcandroid/v-idle-3

Typescript definitions added


Remove ctypes include path hack

This enables using hdr_histogram_ocaml in a monorepo.
Thanks to https://github.com/ocaml/dune/pull/6883 (merged in dune 3.7), the ctypes rule is interpreted in the folder where it is defined. This means we can use relative paths and everything works as expected.

Great. Thanks!

Ah great to see that it works in the wild :)


How to compile Google filament for web in Windows?

Describe the bug
I would like to know how to compile the filament to wasm in Windows. The readme describes only for linux / macOs. Could anyone help with this?

Please open a discussion instead of a bug.


Typo in Issue templates

In the templates files Eddie Jaoude is spelt wrong on the last line https://github.com/EddieJaoudeCommunity/support/tree/main/.github/ISSUE_TEMPLATE
For example, it is
Feel free to check out other cool repositories of EddieJoaude Community [here](https://github.com/EddieJaoudeCommunity)

But should be
Feel free to check out other cool repositories of Eddie Jaoude Community [here](https://github.com/EddieJaoudeCommunity)

I added a space too between first name and last name for readability

ok thanks, i'll get to it now

Awesome @tory-brown , thank you 🤓 . You can raise a PR for each file or do it all in one PR, whichever you prefer. Any question just let us know :)

Hello, is this issue still open?

I just check the files in the link in the description and yes it looks like it is 👍

okay.
Can I work on it?

Sure, I think @tory-brown who it was initially assigned to 10 days has got busy and hopefully won't mind 🤓

Okay. Thanks, will gt to it immediately

@eddiejaoude I raised a PR kindly review. Thank you :slightly_smiling_face:

Thank you @VictoryWekwa , already reviewed and merged 🤓


Cannot add tags to post

Issue Summary
When creating a new post, or editing a draft/published post, I am unable to create new tags from the post settings menu, however adding existing tags works fine.

Steps to Reproduce

Create new post / Edit existing post
Expand the post settings menu
Begin typing a new tag and hit enter
Nothing happens, the tag is not assigned/created

Expected behaviour would be that hitting enter would create the new tag and assign it to the post as it had once worked before.
Technical details:

Ghost Version: v1.22.5
Browser/OS: Chrome (latest) - Mac OS v10.13.3

I am having the same issue.

Had same issue, found that forcing _findTagByName to return a promise would get past this.
https://github.com/TryGhost/Ghost-Admin/blob/cd563b66b7fd825968de4149d9f06c1e1d44e20f/app/components/gh-psm-tags-input.js#L90
changed:
    _findTagByName(name) {
        let withMatchingName = function (tag) {
            return tag.get('name').toLowerCase() === name.toLowerCase();
        };
        return this.get('availableTags').then(availableTags => availableTags.find(withMatchingName));
    }

to:
    _findTagByName(name) {
        let withMatchingName = function (tag) {
            return tag.get('name').toLowerCase() === name.toLowerCase();
        };
        return Promise.resolve(this.get('availableTags')).then(availableTags => availableTags.find(withMatchingName));
    }

wrapping in a try/catch worked too.
Disclaimer: I am definitely an ember noob, so apologies if this code scares you.

The fix for this has been released in Ghost 1.22.7


Release 3.0.0

Need to run a new release to pick up the ASM upgrade for Java 24.  Any other fixes we can include?

Fixed by 29746e774a4f38accd3065a0b4cc4802d11b7480.


Keep scrollbar when grid-layout on and no-results-message

Address scenario described in #490:

keep scrollbar when no results message is displayed in grid-layout mode with horizontal overflowing columns

Codecov Report

Merging #492 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #492   +/-   ##
=======================================
  Coverage   87.56%   87.56%
=======================================
  Files          38       38
  Lines        4392     4392
  Branches      333      333
=======================================
  Hits         3846     3846
  Misses        525      525
  Partials       21       21

Impacted Files
Coverage Δ

src/modules/noResults.js
92.85% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

:shipit:


⚠ Processors has degraded performance

In 3fe41ee, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 57 ms

Resolved: Processors performance has improved in b38a29a.


Update install.md

Fix for issue #346
Updated the complete setup process for MacOS inside install.md file, according to the latest updates.
Please review it and let me know if any change is required.

Nice. Thanks.


Bluetooth: Mesh: Use separate net_bufs for bt_mesh_app_decrypt in friend

unseg_app_sdu_decrypt decrypts messages in place using a single net_buf.
While this is safe in terms of data access, the buffer state is
manipulated with the assumption that they're two different buffers, and
the output buffer's length field is increased at the end. When
assertions are enabled and the pdu length is 11 or 12 bytes, this
triggers the net_buf length assert, as the decrypt function attempts to
add the pdu length to the out buffer, with the assumption that it was
reset before decryption was started.
Create a separate output buffer with len = 0 to avoid triggering the
assert. Improve readability of the unseg_app_sdu functions to highlight
the need for the additional buffer.
Add test coverage for this bug.

cc @ChasinSpin


Adaptor.evaluate() always finishes after a single batch

With transformers 4.17.0, 4.18.0, and later, calling Adaptor.evaluate() finishes after a single batch regardless of the dataset size:

You can reproduce this issue using the following code:
from pathlib import Path

from adaptor.lang_module import LangModule
from adaptor.objectives.MLM import MaskedLanguageModeling
from adaptor.schedules import SequentialSchedule
from adaptor.adapter import Adapter
from adaptor.utils import AdaptationArguments, StoppingStrategy
from ibo.lm.language_model import _get_checkpoint_number

def get_checkpoint_number(checkpoint: Path) -> int:
    *_, checkpoint_number = checkpoint.name.split('-')
    return int(checkpoint_number)

evaluations = dict()
checkpoints = sorted(
    Path('/nlp/projekty/intelligent-back-office/'
         '2022-04-25-lm.mlm-objective/').glob('checkpoint-*'),
    key=get_checkpoint_number,
)

for checkpoint in checkpoints:
    lang_module = LangModule(str(checkpoint / 'MaskedLanguageModeling'))
    objectives = MaskedLanguageModeling(lang_module, batch_size=24,
                                        texts_or_path='notebooks/texts.txt',
                                        val_texts_or_path='notebooks/texts.txt')
    training_arguments = AdaptationArguments(
        output_dir='.',
        stopping_strategy=StoppingStrategy.FIRST_OBJECTIVE_CONVERGED,
        evaluation_strategy='steps',
        do_train=True, do_eval=True,
        gradient_accumulation_steps=11,
        logging_steps=1000, eval_steps=1000, save_steps=500,
        num_train_epochs=100
    )
    schedule = SequentialSchedule([objectives], training_arguments)
    adapter = Adapter(lang_module, schedule, training_arguments)
    evaluations[checkpoint] = adapter.evaluate()

with open('evaluations.txt', 'wt') as f:
    for checkpoint, result in evaluations.items():
        print(result['eval_loss'], file=f)

I reproduced the issue by strengthening the test_evaluation_ner() unit test.

The issue is not present with Transformers 4.11.0 and present with Transformers 4.12.4. It must relate to changes in 4.11.0 < Transformers <= 4.12.4.

Hi Witiko, implementation of per-objective evaluation is in-progress as a new feature in feature/evaluations branch.
I will let you know once we're green.

Hi @Witiko, the pipeline in feature/evaluations branch is now green.
You can either evaluate you objective either in a standalone:
objective = MaskedLanguageModeling(lang_module, batch_size=24,
                                   texts_or_path='notebooks/texts.txt',
                                   val_texts_or_path='notebooks/texts.txt')

results: Dict[Union[Evaluator, str], float] = objective.evaluate()

where results contain 'loss' and val_evaluators given to MaskedLanguageModeling constructor as keys, and corresponding valuations.
or if you want to evaluate more objectives at once, you can call:
results: Dict[Objective, Dict[Union[str, EvaluatorBase], float]] = adapter.evaluate_all_objectives()

where results contain per-objective evaluations in the objective.evaluate() format.
If you need more context, also look into the updated tests.
Sadly, we can not override adapter.evaluate(), as this would now break things in the main train() routine that relies on it. Also, Trainer.evaluate() does not return the results but instead log them, so we would have to be inconsistent with the HF interface.
This was a big change in evaluation logic that I think should be tested in larger-scale training. We will merge this into master only once the test is done, so do not wait for it and if you need it, use it from the branch.

Thank you for the thorough explanation. I will test out the feature/evaluations branch in witiko/scm-at-arqmath3.

Your changes seem to have fixed https://github.com/gaussalgo/adaptor/issues/21#issue-1248944150 for me.

For some reason, calling objective.evaluate() runs on CPU, though.

By design, this should use the user-picked device of the evaluated lang_module.
You can choose the device of the lang_module like any other torch.nn.Module instance, e.g. by:
lang_module = lang_module.to("cuda" if torch.cuda.is_available() else "cpu")

Closing, feel free to reopen if something does not work as expected.


Why is the line commented?

https://github.com/acidicoala/dream-api/blob/68cca8c11ead17341002cdacce443d58bacf1004/src/setup/registry.py#L31
Just out of curiosity, is it for a further release?  Since the value for it in the set_reg 'Proxy_Enable' later on is 0.

Imagine DreamAPI has crashed or its process was killed. The result would be that the Proxy is left enabled. To fix that I suggest people to simply open the DreamAPI again and shut it down properly. Now, if we were to save & restore the enabled state, that would mean that DreamAPI would not disable the Proxy when shutting down DreamAPI in this case, thus leaving users with lost network connection once again.

Oh I see, is the line necessary in the code then?

Technically no, it's just a comment :) I left it just in case it might be useful in the future.
For example, what if someone was using some sort of proxy for other purposes? Turning proxy off instead of restoring previous proxy state might present an issue in this case.
So either strategy has its own drawbacks, so I chose the one which is appropriate for most users.

Gotcha :D


Professions

Need to add all the new profession bonuses, as well as 2 profession selectors in the UI.
We can have warnings when profession bonuses are used but the relevant profession is not selected.

Decent summary: https://www.wow-professions.com/news/wotlk-profession-bonuses


how to send message/notification from any controller

Hi again! :P
I've a question may be I don't undestand so much how this work, but is possible send notification from any controller to the client side ?
If the answer is yes, how I can do it ?
I did a little search for this, but i can't find just I think I need.
I mean I've this:
class ApiController extends Controller
{
public function testAction(Request $request)
{
$connectionTopic = $this->get("autogestion.students_connections");
$connectionTopic->onPublish(ConnectionInterface $connection, Topic $topic, WampRequest $request, $event, array $exclude, array $eligible)
}
  $response = new Response( json_encode(array( "result"=> 200, "message" => "success"  )) );
  $response->headers->set('Content-Type', 'application/json');
  return $response;

} // end controller
From a Controller, how I can get:

ConnectionInterface $connection
Topic $topic
WampRequest $request
$event
array $exclude
array $eligible

I hope you can help me! :)

everything is documented : https://github.com/GeniusesOfSymfony/WebSocketBundle/blob/master/Resources/docs/Pusher.md

great thanks! now i going to see

Ok, I reed all the document, and I've a question :P
Is need to have installed all of them (zmq, AMQP) ?
Because, I only have installed and configurated zmq and when i try to send a push message in the command console I get:

websocket.DEBUG: Matched route "connection_topic"
websocket.INFO: Pusher zmq has pushed {"provider":"zmq", "topic":"class/connection/channel"}
websocket.ERROR: AMQP handler failed to ack message {"exception_message:" "Topic autogestion.class.students.connection doesn't suppor push feature", "file":"........\gos......App\Dispatcher\TopicDispatcher.php", "line":152, .....}

This is my code:
controller class:
try {
$pusher = $this->container->get('gos_web_socket.zmq.pusher');
//push(data, route_name, route_arguments)
$pusher->push(['my_data' => 'data'], 'connections_topic', array());
        $result = array( "result" => 200, "msj"=> "success" );
    }catch(Exception $e) {
        $result = array( "result" => 400, "msj"=> "fail . \n" . $e->getMessage() );
    }

    $response = new Response(json_encode($result));
    $response->headers->set('Content-Type', 'application/json');
    return $response;

Thanks for help me! :)

Do you use ZMQ or AMQP ?
The "ConnectionInterface", "Topic",... objects aren't be in the Controller but in the Topic service !

Topic service receive data from client or controller.
Controller can push data to topic !

So if you want send notification from controller to client side, you must use this way : controller => topic service => client

Only I've installed ZMQ.
I thought that I can do this:
$pusher = $this->container->get('gos_web_socket.zmq.pusher');
$pusher->push($data, $route_name, $route_arguments);
When $route_name is the route where is my topic class (ConnectionTopic).
But when I do push, I get the error mentioned above! :S

Your ConnectionTopic class must implements TopicInterface AND PushableTopicInterface !
After that, use "onPush" method in your topic class to receive data from your controller and send it to client :
public function onPush(Topic $topic, WampRequest $request, $data, $provider)
{
    // Send to all clients in this topic
    $topic->broadcast($data);
}

xD
It's work!!! :D
Thanks a lot man!!!

ty @matthieuy :)


Integrate with financial company API's (Discover card transaction integration)

At this point the user has to manually enter each of their expenses into the app.  Currently, there do not seem to be sufficient API's available from banks / credit card companies to allow for complex integration, but if any are added, it would be very helpful to have the app check for charges on all of the user's accounts and then suggest expenses to add to sheets/spreadsheets based off of these charges.

I looked into what Discover offers for this. Discover does not offer a ready-to-go API for downloading transactions, but they do allow the user to download their transaction history. In the web interface, only downloading a PDF includes pending transactions, but the URL (found using Firefox's network analyzer, does offer a flag to include pending transactions). Using Postman, I can hit the URL to download transaction history, but it says that I am not authenticated. I will need to look into how to authenticate/login programmatically.
A good plan now would be to talk to some sort of Discover developer support / representative and see if I can get more information (or find a better way) without trying to reverse engineer everything.


Questions about the online desatomat tool, CTRE, and pcre.hpp

I am trying to recreate your desatomat tool in python so I can make a LL(1) ANTLR & EBNF grammer parser.
The following types are missing from pcre.hpp but were present in the output of the desatomat tool:

asserts
atom
atom_in_capture
atom_repeat
atom_repeat_in_capture
backslash_set
block_name
character
character_class
character_in_capture
class_named
mode_switch
preblock
set
setitem
setitem2
string
string_in_capture

I have some questions about the tool:

Were these optimized out of the pcre grammar?
Why not make a rule called "ignore" and whenever it is come across it you ignore it and don't place it on the stack?
If I write my grammar in Extended Desátomatu Syntax and replace the pcre.hpp with the output from the desatomat tool would it work?
Is the tool on the website less advanced than what is used to generate pcre.hpp?
What is Extended Desátomatu Syntax? I have never heard of it in any literature. Is it a variant of EBNF?
Why not use EBNF/PEG to describe the regex grammar? Is there something special about EDS?
Why are comments not supported by CTRE? You can split the regex by comments, join the regex_results, and then feed the resulting string back into CTRE

Were these optimized out of the pcre grammar?
yes

Why not make a rule called "ignore" and whenever it is come across it you ignore it and don't place it on the stack?
not sure if I follow you, why?

If I write my grammar in Extended Desátomatu Syntax and replace the pcre.hpp with the output from the desatomat tool would it work?
yes, if you use right generator

What is Extended Desátomatu Syntax? I have never heard of it in any literature. Is it a variant of EBNF?
it's my own, it's the syntax you can see in pcre.gram

Why not use EBNF/PEG to describe the regex grammar? Is there something special about EDS?
because long time ago I did my thing and start building it with it, and I'm not going to change that now, especially I have a lot of my own transformations which would be hard to replicate somewhere else with significant time investment

Why are comments not supported by CTRE? You can split the regex by comments, join the regex_results, and then feed the resulting string back into CTRE
Because you are in C++, you can put comment into normal code and won't pay price for its parsing.


Patches

Codecov Report
Attention: 15 lines in your changes are missing coverage. Please review.

Comparison is base (79828c5) 59.53% compared to head (ee2c14b) 59.51%.

Files
Patch %
Lines

mth5/timeseries/ts_filters.py
12.50%
14 Missing :warning:

mth5/timeseries/channel_ts.py
50.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #181      +/-   ##
==========================================
- Coverage   59.53%   59.51%   -0.03%
==========================================
  Files         138      138
  Lines       15244    15251       +7
==========================================
  Hits         9076     9076
- Misses       6168     6175       +7

Flag
Coverage Δ

tests
59.51% <16.66%> (-0.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Replace H256 with PoolId/DelegationId

Just please look into PoolId and DelegationId definitions as I mentioned in the comment

Confused what do you mean? I defined empty structs as you suggested

Just please look into PoolId and DelegationId definitions as I mentioned in the comment

Confused what do you mean? I defined empty structs as you suggested

All good. I submitted the comment late.


Something went wrong:  t.spec.chart is undefined

Capacitor appears to be throwing these messages on HelmRelease when chartRef is used in place of chart.  From the flux documentation, they imply that chart and chartRef are kind of mutually exclusive.
Flux is using the chartRef spec as expected, and the CLI tools show that it's deployed properly.  However, capacitor just displays the error about "chart is undefined" instead of the real status.
Here is the relevant documentation link:  https://fluxcd.io/flux/components/helm/helmreleases/#chart-reference
Here is our example that is deploying properly.  This was previously reported correctly inside capacitor.  This error seems to have started with of the recent releases of capacitor.
OCIRepository
apiVersion: source.toolkit.fluxcd.io/v1beta2
kind: OCIRepository
metadata:
  name: netbox
  namespace: netbox
spec:
  interval: 5m0s
  url: oci://ghcr.io/netbox-community/netbox-chart/netbox
  ref:
    tag: 5.0.0-beta.52

HelmRelease
apiVersion: helm.toolkit.fluxcd.io/v2
kind: HelmRelease
metadata:
  name: netbox
  namespace: netbox
spec:
  interval: 5m
  chartRef:
    kind: OCIRepository
    name: netbox
    namespace: netbox

Flux status
LAST SEEN                TYPE     REASON             OBJECT                 MESSAGE
23m (x1368 over 4d18h)   Normal   ArtifactUpToDate   OCIRepository/netbox   artifact up-to-date with remote revision: '5.0.0-beta.52@sha256:8c2e5f734841a1bc728be3184a80fc9dfc8e8aadb4040c365dd16bc9ca82fa99'

I was able to replicate this when the HelmRelease was not in fully reconciled state.
Pushed a fix for it. Let me know what you see with v0.4.7

Looking much better now, thank you.


Multiple file uploading

Hi guys already uploaded my code on ionic forums regarding this one but got nothing there hope here will get some response.. I just want to  upload multiple files in angular 2 and sent to node js server so here below is my code ..
    this.camera.getPicture(options).then((imagePath) => {

        this.tempPath = "data:image/jpeg;base64," + imagePath;
        this.selectedImagePath = imagePath;
        this.signaturePad.fromDataURL(this.tempPath);

this.storage.set('savedSignature', this.signature);
      // this.signaturePad.fromDataURL(this.image);
      this.storage.get('savedSignature').then((data) => {
          this.formData.append('uploadArray['+this.i+']', data);
       })
   }

Posting..
let headers: any = new Headers({ 'Content-Type': undefined  });
    headers.append('Accept', 'application/json');
    let options: any = new RequestOptions({ headers: headers }),
this.http.post(url,this.formData,options).map(res => res.json())
      .subscribe((data) => {})

on server side not able to get the files
just want to know the way I am sending and appending is correct or not

Hello! Thanks for opening an issue with us! As this seems like more of a support question I will urge that you ask this question on our forum or on our slack channel. Thanks for using Ionic!

Hey bro thanks for replying ..
On Jun 7, 2017 8:08 PM, "Justin Willis"<EMAIL_ADDRESS>wrote:

Closed #11955 https://github.com/ionic-team/ionic/issues/11955.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ionic-team/ionic/issues/11955#event-1113784023, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQleTCVIgVa33Kk83nLGCmaG1aA7GgImks5sBrXrgaJpZM4NyjZN
.

But I really need help in this ...plz
On Jun 7, 2017 8:08 PM, "Justin Willis"<EMAIL_ADDRESS>wrote:

Closed #11955 https://github.com/ionic-team/ionic/issues/11955.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ionic-team/ionic/issues/11955#event-1113784023, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQleTCVIgVa33Kk83nLGCmaG1aA7GgImks5sBrXrgaJpZM4NyjZN
.


Bump the branch to iree-org/iree@40258db6

I had an offline discussion with Alan, and I will go with "rebase + force-push" way.

You might also be able to
git checkout shared/gpu-data-tiling-materialize-encoding
git merge main
git push shared/gpu-data-tiling-materialize-encoding

(just like you would push a regular, non-merge, commit to the branch)

I think it could mess up the git log information because it inserts many commits in between. It's easier to track diffs and make changes accordingly in the "rebase + force-push" way.


Add x-forwarded-port if required

Description:
This patch adds x-forwarded-port with listener port number as its value if required. The append_x_forwarded_port is introduced in HttpConnectionManager.
Risk Level: Low
Testing: Unit
Docs Changes: Added
Release Notes: Added
Fixes #3732
Signed-off-by: Dhi Aurrahman<EMAIL_ADDRESS>

@ggreenway looks good to me - assigning to you for final review!

I'm not going to have time to review this before I leave for vacation.

I can look tomorrow.

Oops. I'm working on it.

Any progress on this? Is it possible to retrieve somehow the port from x-forwarded-for header?


test(e2e): allow flaky angular test for now

See #1749

@simondel Any idea why the grunt test is flaky? I've looked into it, but can't find a reason

I've set the wct tests timeout higher, as that also caused flaky tests in unit+integration.

Unfortunately not. Although they almost never seem to fail (compared to the Angular test)


Add a few variant datasets/tracks to JB2 manually

-- Add a few variant data:
Add manually the genotyping dataset tracks to JB2 config since Connor's script can't pick it up.

OK, I've added the three existing diversity sets for hypogaea:

aradu1_araip1.gnm1.div.Clevenger_Korani_2018: Genotyping of African germplasm collection
aradu1_araip1.gnm1.div.Otyama_Kulkarni_2020: Diversity data for the U.S. peanut core collection, genotyped relative to the diploid A. duranensis and A. ipaensis genome assemblies.
aradu1_araip1.gnm1.div.Otyama_Wilkey_2019: Genotype data for US Peanut Mini Core collection.

to the jbrowse2 config for both the aradu.gnm1 and araip.gnm1 against which the variants in those files are located. As is typical, I also discovered that the compression for these genomes/annotations was not bgzip, so have fixed those. In case we ever need to recreate these following an autogenerate of the config by Connor's scripts, the commands I used to add these tracks to the configs are in my home dir on the dev-pb-jekyll VM: ~adf/jbrowse2_config_cmds


Buttons vs links in Chaise apps

This continues from issue #782, where @robes said:

One other thing. I believe that karl and hongsuda felt that everything except "Permalink" should use a style that looks more like a button than simply a link. They felt that simply looking like a link didn't give the user the right indication of its affordances.

In the front-end standup today, we discussed whether we should be using buttons or links in our Chaise apps. In Record, we use links (e.g. Add, Edit, Delete, Show All...) while in RecordEdit we use buttons (e.g. Submit Data, Delete) to indicate the similar actions a user can take.
For the sake of consistency, we need to decide on a unified UX to expose these actions to Chaise users. At the end of the meeting, we decided to punt until next week when everybody will be back in office and can weigh in on this; this issue can track that discussion.

I am marking this as a wishlist. We all are kind of aware of the consistency issues, and hopefully, we can eventually merge on consistent style.. I don't see us spending a lot of time refactoring all apsp on this.


Stop altChapter from preventing move of Chapter to correct para

This seems to be the cause of a bug in the SOFRIA endpoint when a chapter is specified. The wrong paragraphs are selected because the start-chapter tag is an orphan at the end of the previous paragraph, because it is hiding behind an altChapter tag:
Object: null prototype] {
      dump: 'Block:\n' +
        '   title graft to OWIyMTkwOWEt\n' +
        '   Scope blockTag/q\n' +
        '   +chapter/8++verse/1++verses/1++altVerse/2++pubVerse/1 (2)+|Content| |before| |altChapter|.-pubVerse/1 (2)--altVerse/2--verses/1--verse/1--chapter/8-+chapter/9++altChapter/9a+'
    },
    [Object: null prototype] {
      dump: 'Block:\n' +
        '   Scope blockTag/q\n' +
        '   +verse/1++verse/2++verses/1-2++altVerse/2-3++pubVerse/1-2 (2-3)+|First| |verse| |after| |altChapter-pubVerse/1-2 (2-3)--altVerse/2-3--verses/1-2--verse/2--verse/1-'
    }

There's a specific exception for altChapter in the parsing code. Bad Things Happen if I remove that, which I think is because of some other scary reordering around pubChapters. So I've made a second, no exception orphan mover that is called after the pubChapters alchemy.
All tests pass.

The cp_vp test confirms that adding
seq.moveOrphanStartScopes2()
to parser/index.js does result in altChapter and chapter moving to the right place.

@DanielC-N I think this is now good to go (next week!) The package version also needs bumping, depending what else we do before next publishing to npm.

Review OK.
All tests passes from fresh install.


PDF document says that should be opened in Adobe Reader or higher

When I try to open PDF document from server Justice.cz I see PDF document with info that I should use Adobe Reader 9 or higher. Maybe because AcroForm/XFA support is missing?
PDF document: http://www.jasnapaka.com/bugs/aaa.pdf
OS: Windows 7 64 bit, HWA enabled
pdf.js: Firefox 30.0a1 (2014-02-11), PDF.js: 0.8.990
Screenshots
Firefox: http://www.jasnapaka.com/bugs/aaa-firefox.png
Adobe Reader: http://www.jasnapaka.com/bugs/aaa-adobe-reader.png
Web Console
"Warning: AcroForm/XFA is not supported" viewer.js:3193
"Warning: unimplemented annotation type: Widget signature" pdf.worker.js:190
"Warning: unimplemented annotation type: Widget" pdf.worker.js:190
"Warning: unimplemented annotation type: Widget signature" pdf.worker.js:190
"Warning: unimplemented annotation type: Widget"

#4329 implemented attachment support. When I visit the mentioned PDF link, the viewer suggests that the PDF contains forms (I'm not sure if this is correct here). The document displays as before, however when I open the sidebar and switch to attachments (see #6644 for making this more obvious), I see the attached PDF (though when clicking, it prompts me to download the file instead of trying to open it, see #6643).
Is it possible to implement some heuristics that instead of showing the "download Acrobat" message the attachments are shown?

How do I set it up to display a PDF the first time it loads, instead of displaying the downloaded Adobe Reader because the user doesn't want to click  the attachment  again

What's the reason it doesn't display PDF, instead displaying this for users to download Adobe Reader?Is the generated PDF version too high or is there some compatibility issues?And I found that if I downloaded the PDF with pdf.js and then opened the PDF with pdf.js, it would display as normal, rather than displaying the page that allowed the user to download Adobe Reader.

Immigration Canada (IRCC) XFA form, also as attachment for testing purposes. A friend of mine needed the form and they do not offer it on any other format apparently. I was investigating this for him.
imm5708e.pdf
I think I was able to fill a PDF form recently and I was surprised it worked. Maybe I imagined it or the file format was different :confused: . Still, awesome job folks, you rock.

There are a few things in the queue which will hopefully land soon and should improve the rendering of XFA forms.

@calixteman this is still occurring in latest Nightly.

I'm not sure if 4-form-xfa is an entirely appropriate label here, since the document uses the "Collections" feature, see https://www.adobe.com/content/dam/acom/en/devnet/pdf/pdfs/PDF32000_2008.pdf#G11.2076363


Laravel 5.3

Can't use the package with Laravel 5.3.
Problem 1
- Conclusion: remove propel/propel 2.0.0-alpha5
- propel/propel-laravel dev-develop requires propel/propel ~2.0.0-alpha5 -> satisfiable by propel/propel[2.0.0-alpha5].
- propel/propel-laravel dev-develop requires propel/propel ~2.0.0-alpha5 -> satisfiable by propel/propel[2.0.0-alpha5].
- Conclusion: don't install propel/propel 2.0.0-alpha5|keep propel/propel 2.0.0-alpha5
- Installation request for propel/propel-laravel dev-develop -> satisfiable by propel/propel-laravel[dev-develop].

@Snake231088 , ensure please that you have "minimum-stability": "alpha" row in your composer.json file.

I've   "minimum-stability": "alpha" in composer.json but it not work.

solved with minimum-stability": "dev