route awareness tracing

Title: One line description
tracing configuration per route (virtual host)
Description:

Describe the desired behavior, what scenario it enables and how it
would be used.

According to http.tracer and skywalking example, I followed the docs, and http tracing can be achieved.

However it's a filter-level setting, a single v3.HttpConnectionManager may route traffic to various different backend services. Such services may want different tracing configs like sampling rate, etc.
So is it possible to specific different tracing settings within a single listener - filter - connection manager in current envoy implementation? Or am I the only one who seeking tracing at route level.
[optional Relevant Links:]

Any extra documentation required to understand the issue.

cc @wbpcode @dio @lizan @Shikugawa as owners of the skywalking tracer who may know if this is possible/plausible

@tedli AFAIK I think we can't do such a behavior. We may have the following requirements with a route specific config, such as per_filter_config. But I'm not sure the strong usecases to have.

Hi @Shikugawa ,
Thanks for reply. Our story is that, we want to using istio EnvoyFilter CRD to turn on/off sidecar tracing at service level (these service behind sidecar runs with APM agent). So when any service runs into terrible latency, we can easily tell it's sidecar caused this or it's just  the backend service's own fault.
And nowadays, there still some developers refuse to use service mesh, because they thought it may introduce extra uncertainty which is not necessary.
So my point of view is that this would be a strong use case.
P.S.
# the EnvoyFilter I used
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: demo-a
  namespace: sky
spec:
  workloadSelector:
    labels:
      name: demo-a
  configPatches:
  - applyTo: NETWORK_FILTER
    match:
      context: ANY
      listener:
        filterChain:
          filter:
            name: envoy.filters.network.http_connection_manager
    patch:
      operation: MERGE
      value:
        name: envoy.filters.network.http_connection_manager
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
          generate_request_id: true
          tracing:
            provider:
              name: envoy.tracers.skywalking
              typed_config:
                "@type": type.googleapis.com/envoy.config.trace.v3.SkyWalkingConfig
                grpc_service:
                  envoy_grpc:
                    cluster_name: skywalking-oap
                  timeout: 0.250s
                client_config:
                  service_name: demo-a-sidecar
                  instance_name: demo-a-sidecar
  - applyTo: CLUSTER
    match:
      context: SIDECAR_OUTBOUND
    patch:
      operation: ADD
      value:
        name: skywalking-oap
        type: STRICT_DNS
        connect_timeout: 0.5s
        lb_policy: ROUND_ROBIN
        typed_extension_protocol_options:
          envoy.extensions.upstreams.http.v3.HttpProtocolOptions:
            "@type": type.googleapis.com/envoy.extensions.upstreams.http.v3.HttpProtocolOptions
            explicit_http_config:
              http2_protocol_options: {}
        load_assignment:
          cluster_name: skywalking
          endpoints:
          - lb_endpoints:
            - endpoint:
                address:
                  socket_address:
                    address: tracing-analysis-server.system-mf
                    port_value: 11800

@tedli I am not sure, but can route level Tracing helps? That config can be used to add some custom tags and also override the sampling rate.
https://www.envoyproxy.io/docs/envoy/latest/api-v3/config/route/v3/route_components.proto#envoy-v3-api-msg-config-route-v3-tracing

Hi @wbpcode ,
Thanks for pointing out. I didn't express clear above, What I actually want is to turn on/off tracing for a specific service at runtime.
I'm going to try with sampling rate -> 0 in route_components. Or may be this really not a general needs. I may try reusing most of envoy current tracing code, to build a wasm filter like istio/proxy stats do (no enough confidence :sob:).

Here is config I tried:
node:
  id: envoy
  cluster: envoy
admin:
  access_log:
  - name: envoy.access_loggers.stdout
    typed_config:
      "@type": type.googleapis.com/envoy.extensions.access_loggers.stream.v3.StdoutAccessLog
  address:
    socket_address:
      protocol: TCP
      address: <IP_ADDRESS>
      port_value: 15000
static_resources:
  listeners:
  - name: http
    address:
      socket_address:
        address: <IP_ADDRESS>
        port_value: 8887
    access_log:
    - name: envoy.access_loggers.stdout
      typed_config:
        "@type": type.googleapis.com/envoy.extensions.access_loggers.stream.v3.StdoutAccessLog
    filter_chains:
    - filters:
      - name: envoy.filters.network.http_connection_manager
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
          stat_prefix: http
          access_log:
          - name: envoy.access_loggers.stdout
            typed_config:
              "@type": type.googleapis.com/envoy.extensions.access_loggers.stream.v3.StdoutAccessLog
          route_config:
            name: http
            virtual_hosts:
            - name: http
              domains:
              - "*"
              routes:
              - match:
                  prefix: /
                route:
                  cluster: http
                tracing:
                  client_sampling:
                    denominator: HUNDRED
                    numerator: 0
                  random_sampling:
                    denominator: HUNDRED
                    numerator: 0
                  overall_sampling:
                    denominator: HUNDRED
                    numerator: 0
          http_filters:
          - name: envoy.filters.http.router
          generate_request_id: true
          tracing:
            # client_sampling:
            #   value: 0
            # random_sampling:
            #   value: 0
            # overall_sampling:
            #   value: 0
            provider:
              name: envoy.tracers.skywalking
              typed_config:
                "@type": type.googleapis.com/envoy.config.trace.v3.SkyWalkingConfig
                grpc_service:
                  envoy_grpc:
                    cluster_name: skywalking
                  timeout: 0.250s
                client_config:
                  service_name: sidecar
                  instance_name: sidecar
  clusters:
  - name: http
    connect_timeout: 0.25s
    type: STATIC
    load_assignment:
      cluster_name: http
      endpoints:
      - lb_endpoints:
        - endpoint:
            address:
              socket_address:
                address: <IP_ADDRESS>
                port_value: 8888
  - name: skywalking
    type: STRICT_DNS
    connect_timeout: 0.25s
    typed_extension_protocol_options:
      envoy.extensions.upstreams.http.v3.HttpProtocolOptions:
        "@type": type.googleapis.com/envoy.extensions.upstreams.http.v3.HttpProtocolOptions
        explicit_http_config:
          http2_protocol_options: {}
    load_assignment:
      cluster_name: skywalking
      endpoints:
      - lb_endpoints:
        - endpoint:
            address:
              socket_address:
                address: <IP_ADDRESS>
                port_value: 11800

I want to set sampling rate to 0% at connection manager level, 100% at route level, also I tried 100% at connection manager, and 0% at route. None of those work.
Result is if tracing of connection manager is not null, then 0% won't affect.


Fix group by in compare_column_values

Description & motivation
Grouping by aliases does not seem to be permitted in SQL standard
Fix https://github.com/dbt-labs/dbt-audit-helper/issues/87
Checklist

[x] I have verified that these changes work locally
I have updated the README.md (if applicable)
I have added tests & descriptions to my models (and macros if applicable)

Looks like the fix for this was in place but couldn't be reviewed/merged? I am wondering what the process is to revive this PR. thoughts?


Investigate partial hits in code coverage, and add additional tests to cover more paths

Our code coverage is pretty high (95% or so), but we still have about 100 partial hits. Some of those are false positives, but others do in fact hint at missed paths through the code. We should investigate this a bit, and write additional tests for the uncovered paths.

Should probably also look into the asserts, and figure out why they're not being hit (and if that is something I care about or not).

I had a look at this just yet, and I think the current behaviour is OK.


Checked for duplicates
Yes - I've already checked
Describe the needs
We need a guide to document institutionally authorized ways to work with popular licenses such as Apache V2 and MIT while respecting institutional copyright / markup language.

Some possible paths forward:

Provide examples of licenses that include institutional markup in a place within the license that doesn't break GitHub's license validator
Recommend separation of LICENSE and COPYRIGHT information into separate files to hold the info separately (but figure out how to deal with copyright language within the license that ships with it)

Thank you @riverma! I don't know why we didn't simply make this a ticket previously.


Add BIDS participant_id to all TSVs and JSONs

The rule should just be: "Whatever the study's unique subject ID is for each subject, then THAT turned into an alphanumeric-only becomes the alphanumeric after sub- in the participant_id column."

[ ] ABCD_4
[ ] ABIDE_I
[ ] ABIDE_II
[ ] HBN
[ ] HCP
[ ] NKI
[ ] PNC
[ ] UKB


Handler customisation

Options for customising handlers.
Mappings
Mappings are the simplest option for most behaviour changes. If you want to change how a property affects a native control (or even ignore the property), modifying the mapping for the property is the choice. Existing mappings can be modified/replaced, and you can add new mappings.
Custom factory method
Each handler type provides a factory method (public static Func<T>) which it uses when creating the native controls. If you wish to use a custom subclass of the native control type in your handler, you can modify the factory method to return the desired type. This should only be used if the customisations you require are not achievable via a mapping - for example, if your custom type requires particular constructor parameters.
Subclassing the native control type
Some native control customisations may not be achievable via public property setters and methods. In these cases, you may need to subclass the native control type. You'll also need to modify the factory method to return your custom type.
Example:
#if IOS
public class WorkaroundMauiTextField : Microsoft.Maui.Platform.MauiTextField
{
    public WorkaroundMauiTextField()
        : base()
    {
        BorderStyle = UIKit.UITextBorderStyle.RoundedRect;
        ClipsToBounds = true;
    }
    [Foundation.Export("selectedTextRange")]
    public new UIKit.UITextRange SelectedTextRange {
        get => base.SelectedTextRange;
        set => base.SelectedTextRange = value;
    }
}
#endif

Then, in MauiProgram.cs:
#if IOS
Microsoft.Maui.Handlers.EntryHandler.PlatformViewFactory =
    handler => new WorkaroundMauiTextField();
#endif

Create your own handler
Customising an existing handler may not be sufficient to achieve your goals. Instead, you may need to create your own handler. This can be necessary when you need a lot of mappings that aren't available on the default handler, or you want completely different behaviour.
Subclass a handler
Don't do this. Instead, log a bug to make an earlier approach possible.

Also, https://github.com/dotnet/docs-maui/issues/2111 for some specifics.


Count/ScannedCount Result Keys for DynamoDB Scan not Iterable (Pagination)

When attempting to use result_key_iters for page iteration, the following error is thrown for DynamoDB scan operations: TypeError: 'int' object is not iterable L676 in botocore/paginate.py
The error is caused by the result_keys for the DDB Scan operation containing Count and ScannedCount in the DDB pagination model. The paginator expects all the value of a result key to be an iterable object, whereas Count/ScannedCount are of type int.
Happy to raise a PR for this to remove these keys if that's the correct fix, however I want to be sure it's not a usage error on my part first!
Here is a little script with which you can reproduce the issue:
import boto3

def paginate(client, method, **kwargs):
    paginator = client.get_paginator(method.__name__)
    for page in paginator.paginate(**kwargs).result_key_iters():
        for result in page:
            yield result

if __name__ == '__main__':
    client = boto3.client("dynamodb")
    results = paginate(client, client.scan, **{
        'TableName': "YOUR_DDB_TABLE_NAME"
    })

    for result in results:
        print(result)

@HieronymusLex - Thank you for your post. The result_key_iters operation is specifically used for aws cli and this can't be used the way you are using. That's why you are getting error as it will be trying to iterate through every value including integer.
This is the documentation for how to use paginator with boto3:
https://boto3.amazonaws.com/v1/documentation/api/latest/guide/paginators.html
you can use something like this:
import boto3

client = boto3.client("dynamodb")
paginator = client.get_paginator('scan')
page_iterator = paginator.paginate(TableName='Music')
for page in page_iterator:
     print(page['Items'])  #to list only the item values otherwise you can do print(page)

Hope it helps and please let me know if you have any questions.

@swetashre I still think there is an issue here which is missing a unit test. If you look at the tests for result_key_iters it only caters for a scenario where 1 result_key is present in the paginator definition, and where that result_key is of type list.
https://github.com/boto/botocore/blob/7ab300620eff73ff82bdade4115ccbe94322cfc4/tests/unit/test_paginate.py#L693
If you added a test to cover multiple result keys where one of the result keys is of type int (as is the case in the Dynamo paginator), then it test fails e.g.
class TestKeyIterators(unittest.TestCase):
    def setUp(self):
        self.method = mock.Mock()
        self.model = mock.Mock()
        # This is something we'd see in DDB pagination.
        self.paginate_config = {
            "output_token": "Marker",
            "input_token": "Marker",
            "result_key": ["Users", "Count"] # added multiple keys as with DDB Scan
        }
        self.paginator = Paginator(self.method, self.paginate_config, self.model)

    def test_non_list_result_key_iters(self):
        responses = [
            {"Items": ["User1"], "Marker": "m1", "Count": 1},
            {"Items": ["User2"], "Marker": "m2", "Count": 1},
            {"Items": ["User3"], "Count": 1},
        ]
        self.method.side_effect = responses
        pages = self.paginator.paginate()
        iterators = pages.result_key_iters()
        self.assertEqual(len(iterators), 2) # updated as there are 2 result keys now
        self.assertEqual(list(iterators[0]),
                         ["Item1", "Item2", "Item3"])
        self.assertEqual(list(iterators[1]), [1, 1, 1]) # throws error

The same TypeError: 'int' object is not iterable is thrown when running this test. The existing test for result_key_iters only covers 1 result key with type list
Also, I couldn't find any instances of this method being used in boto3 or the CLI?

Hi @HieronymusLex,
Really sorry for the delayed response on this!  I don't think anything is missing in terms of the paginator model or a missing unit test.  These classes/methods were added in this commit to add support for providing multiple input tokens, which (as far as I know), only apply to certain s3 operations.


ActionCable connection not reestablished after authentication with Turbo

So I recognize that there are other issues similar or related to this one like https://github.com/hotwired/turbo/issues/45 and if this has been covered elsewhere I apologize.
I am noticing that if I have a channel identified by current_user then if I submit a Turbo enabled authentication form (e.g. with Devise) then I don't have access to the current user through the ActionCable channel. A full refresh of the page establishes connection immediately. This behavior is also fixed by adding a turbo-false data element to the form.  A final option that sort of works is to do something like this (note that this is with an ApplicationCable::Connection identified by both session id and current_user):
class Users::SessionsController < Devise::SessionsController
  def create
     super do
        ActionCable.server.remote_connections.where(session_id: session.id, current_user: nil).disconnect
     end
end

The problem with this last solution is that it takes several seconds for ActionCable to detect the stale connection which can lead to confusing UX for the user. The initial connection on page load is quite snappy in comparison.
My conclusion is that when Turbo handles the redirect the connection to ActionCable does not get refreshed.
Maybe just disabling turbo in the form is good enough but I am curious if there is a way and whether it would make sense for Turbo to trigger ActionCable to reconnect in such cases.

In case someone else bumps into this here is one of probably many solutions to deal with this problem if you are using CableReady
first you need multiple identifiers for your connection to deal with unauthenticated users:
module ApplicationCable
  class Connection < ActionCable::Connection::Base
    identified_by :current_user
    identified_by :session_id

    def connect
      self.current_user = env["warden"].user(:user)
      self.session_id = request.session.id
      reject_unauthorized_connection unless self.current_user || self.session_id
    end
  end
end

then you will want a SessionChannel for when users are not yet authenticated:
class SessionChannel < ApplicationCable::Channel
  def subscribed
    stream_for session_id
  end
end

import CableReady from 'cable_ready'
import consumer from './consumer'

consumer.subscriptions.create('SessionChannel', {
  received (data) {
    if (data.cableReady) CableReady.perform(data.operations)
  },

  connected () {
    document.addEventListener('reconnect', this.reconnect)
  },

  reconnect() {
    consumer.disconnect()
    consumer.connect()
  }

})

Then finally in your session_controller:
class Users::SessionsController < Devise::SessionsController

  # POST /resource/sign_in
   def create
     super do
       cable_ready[SessionChannel].dispatch_event(name: 'reconnect')
         .broadcast_to(request.session.id)
     end
   end

 end

I am sure there are ways to do this with just Turbo and Stimulus but perhaps this can be an inspiration.


Fix ADDR_Contact_TDTM chunking error

Critical Changes
Fixes a bug that was causing a too many chunks exception.
Changes
Issues Closed
Fixes #447

Great work. I changed the base branch so this would run through our build system after merging to a new feature branch.


Allow users to explicitly set partialPath and helpersPath

This small change adds greater flexibility by allowing the user to explicitly set paths for partials and helpers in the task config section in the grunt file.

Could somebody merge this please? Looks good 👍


Changed 'tmp_name' to 'name'

Coverage remained the same at 89.88% when pulling b7fbe406dbf9d29f34a262125a7c5e8753db489d on damianoporta:develop into ec0658bd61ec9026021c6c636516eae22ca77d97 on davidyell:develop.

Great, thanks!


Add action feed for new release

As title

Maybe omit the downloads? It seems a bit too much information.

Maybe omit the downloads? It seems a bit too much information.

I have limited the number of attachments to show here to 5, I think it's enough.

Then I'd suggest removing at least the "Downloads" text and unintending the file list to save some space.

Why not make attachments a details list, like it is on the actual release page?

I also think that no downloads should be shown here. If you need downloads you could click on release and go to release page and download from there.

Click 1.0 will jump to the release page?

Click 1.0 will jump to the release page?

yes

If you create a release to an existing tag nothing will show up

If you create a release to an existing tag nothing will show up

I think 32dffe7 has fixed it.

If you create a release to an existing tag nothing will show up

Maybe, I will check it later.

@6543 fixed

Wow that was fast!

Release title would be usually same as tag imho so that should be shown in the link text instead of tag name

Maybe this pr should have bug label because of 6622464

-> simplify of the template didnt worked as expected ...

🚀

🚀

@a1012112796 can you backport the bugfix part?


Feature request: reverse selection

The title says it all. I'd like to be able to select some text (a word, a line, a block, whatever) and reverse it.
Just found this plugin today, and I'm definitely liking it so far. Thank you!

You mean abc -> cba ?

Exactly that, yep!


RKE1 - Add a Button Force Provision to RKE1 cluster

Detailed Description

RKE1 force provision is a new feature that allows users to trigger a force provision of the RKE1 cluster when the cluster is stuck in an error state because Rancher exhausts retries.
We would like to have a button in the RKE1 cluster for users to trigger the force provision.
When the button is clicked, the UI should set the annotation io.cattle.rke.force.provision=true on the cluster.management.cattle.io v3 object, send a PUT request to the backend which will trigger the backend to upgrade the cluster.
Context

Please check https://github.com/rancher/rancher/issues/37234 for more details.

Due to a shift in priorities, this is no longer in the v2.6.6 release plans, so moving it to Unscheduled for now.


DefaultAzureCredential: How to know which credential was used internally?

Library name and version
Azure.Identity 1.6.0-beta.1
Query/Question
I love the flexibility of DefaultAzureCredential. I use it in many applications, for instance to support different credentials for my local debugging, during workflows or in production without the need for code changes or a factory pattern.
So i know that the DefaultAzureCredential sequentially calls GetToken() on all the included credentials. The order is clearly specified in the documentation.
My use case was that is got different behavior for my xUnit test running from the command line or from VS Test Explorer. It turned out the command line used the AzureCliCredential and the Test Explorer used the VisualStudioCredential.
It took me a while to figure this out and if i could log this in my code that would have helped a lot.
So I was wondering how i can verify which credential was actually used for the token by DefaultAzureCredential. This could be informative and for instance used in debug logging.
Regards,
Martijn
Environment
.NET SDK (reflecting any global.json):
Version:   6.0.201
Commit:    ef40e6aa06
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.22000
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\6.0.201\
Host (useful for support):
Version: 6.0.3
Commit:  c24d9a9c91
.NET SDKs installed:
3.1.417 [C:\Program Files\dotnet\sdk]
5.0.406 [C:\Program Files\dotnet\sdk]
6.0.100 [C:\Program Files\dotnet\sdk]
6.0.101 [C:\Program Files\dotnet\sdk]
6.0.103 [C:\Program Files\dotnet\sdk]
6.0.201 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 3.1.22 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.23 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.13 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.15 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 3.1.22 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.23 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.13 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.15 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.1 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.22 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 3.1.23 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.13 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.15 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.1 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.3 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Extensions:0.1839506,Azure.Identity:0.1664758,Storage:0.08103299'

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

Thanx @christothes ... those settings should do the job...
And I learned about ChainedTokenCredential which could be handy 😀

So i tried the logging today and i does print out the authentication flow in order so this is informative. Although it is also very excessive information so i would not use this besides debugging.
I was also looking onto below option as reported on the referred page but i cannot find the IsAccountIdentifierLoggingEnabled on the Diagnostics property, and also not in the documentation.
DefaultAzureCredentialOptions options = new DefaultAzureCredentialOptions
{
    Diagnostics =
    {
        LoggedHeaderNames = { "x-ms-request-id" },
        LoggedQueryParameters = { "api-version" },
        IsAccountIdentifierLoggingEnabled = true
    }
};

/unresolved

Oke, did some more reading and found a customization option for logging. I tried this and that seems to be a viable option for me :). This cleans up the logs a bit so i get only the relevant information from the identity logging.
Below is an example if someone finds this useful.
using AzureEventSourceListener listener = new AzureEventSourceListener((e, message) =>
{
    if (e.EventSource.Name == "Azure-Identity")
    {
        logger.LogInformation(message);
    }
},
System.Diagnostics.Tracing.EventLevel.LogAlways);

Source:
https://docs.microsoft.com/en-us/dotnet/azure/sdk/logging#configure-custom-logging

Oke, thanks for informing me. I'll check out the next release when it comes out.
Closing this issue as my questions have been addressed.


vmap with negative in_dims ?

I'm wondering if it would make sense to specify and cover in_dims as negative value in vmap ? For example:
from functorch import vmap

def f(x):
    return torch.sin(x).sum()

t_in = torch.randn(4, 4)
t_out = vmap(f, in_dims=-1)(t_in)

Currently, this is not supported
ValueError: vmap(f, in_dims=-1, ...)(<inputs>): Got in_dim=-1 for some input, but that input is a Tensor of dimensionality 2 so expected in_dim to satisfy 0 <= in_dim < 2.

Yeah, I don't see any reason why we can't support this :P


Task 3

[x] write function to generate shortned URL
[x] create URL storing view
[x] write function to check if shortned URL already exists
[x] write function to redirect

Okay, I am so glad to announce that I have reached my first Milestone: Getting the MVP ready!


ModuleNotFoundError: No module named 'skipgram'

When I try to import gmatch4py, the below error happens:
ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-1-789a2ae4f894> in <module>
----> 1 import gmatch4py as gm

c:\users\XXX\appdata\local\programs\python\python39\lib\site-packages\gmatch4py\__init__.py in <module>
     13 # Graph Embedding import
     14 from .embedding.graph2vec import *
---> 15 from .embedding.deepwalk import *
     16 from .embedding.node2vec import *
     17 # Helpers import

gmatch4py\embedding\deepwalk.pyx in init gmatch4py.embedding.deepwalk()

ModuleNotFoundError: No module named 'skipgram'

Same here, but I commented this import out since I'm only interested in GED and that worked.

Also the same here. Tested on Windows and Linux.

I fix
https://github.com/Jacobe2169/GMatch4py/blob/aec78a2ca76f4098d867dc30384ce843e7cf027a/gmatch4py/embedding/deepwalk.pyx#L31
to
from .skipgram import Skipgram

then reinstall and it works !!

Thanks @doralune. I had to install it twice (first time got an error) but then it worked like a charm. Thanks alot.

Hi @kodama-codes
The error came from that the latest gensim library no longer has Vocab.
I downgraded to gensim==3.8.3 and then it worked.

Ahoi @doralune
Perfect, now it workes like a charm. Thx alot!

gensim 3.8.3 can not easily be installed on newer systems. Modern gcc won't compile it. Please don't expect me to downgrade gcc as well^^. Any easy way to fix this, or is this library dead?

I fix
https://github.com/Jacobe2169/GMatch4py/blob/aec78a2ca76f4098d867dc30384ce843e7cf027a/gmatch4py/embedding/deepwalk.pyx#L31
to
from .skipgram import Skipgram

then reinstall and it works !!

I did that, but still having the very first Error, with (ModuleNotFoundError: No module named 'skipgram')
Starting to lose faith
Please Help!

@Anmario It should be different error after fixing something. Please give more details for reproducing the error.

gensim 3.8.3 can not easily be installed on newer systems. Modern gcc won't compile it. Please don't expect me to downgrade gcc as well^^. Any easy way to fix this, or is this library dead?

GMatch4py is not updated for 3 years ? and still uses the old gensim, so we don't have much choice. For me, I use the docker container for running old stuff.

@Anmario It should be different error after fixing something. Please give more details for reproducing the error.

That's logic, but I get in the both cases the same below.
:228: UserWarning: To obtain optimal results install the Cython 'munkres' module at  https://github.com/jfrelinger/cython-munkres-wrapper
ModuleNotFoundError                       Traceback (most recent call last)
~\AppData\Local\Temp/ipykernel_27028/812984187.py in
1 import networkx as nx
----> 2 import gmatch4py as gm
3 1+1
~\anaconda3\lib\site-packages\gmatch4py_init_.py in
13 # Graph Embedding import
14 from .embedding.graph2vec import *
---> 15 from .embedding.deepwalk import *
16 from .embedding.node2vec import *
17 # Helpers import
gmatch4py\embedding\deepwalk.pyx in init gmatch4py.embedding.deepwalk()
ModuleNotFoundError: No module named 'skipgram'

Hello all, I took the time to make a fork of this library and remove all code that is not related to graph edit distance and cleaned it up, so it has no legacy dependencies.
If you want to use only graph edit distances and no other features feel free to use my fork. It should easier to maintain, since it has much less dependencies.
Check it out at https://github.com/chilligerchief/GED4py

Hi all !
Sorry for being silent for many years...I've been busy and did not have much time for GMatch4py. Thanks to you all for helping peoples in need when it comes to make GMatch4py work !
I push few modifications that integrates solutions proposed in this feed.
Bests,
Jacques


style(sponsors): Fix not showing svg image

Update max-height: 125px to height: 125px

(Thanks for sending a pull request! Please fill in the following content to let us know better about this change.)
Types of changes
Please put an x in the box that applies

[x] Bugfix
[ ] New feature
[ ] Refactoring
[ ] Breaking change (any change that would cause existing functionality to not work as expected)
[ ] Documentation Update
[ ] Other (please describe)

Description
Describe what the change is
You don't need to explain HOW you change the code. Your code, including comments in the code should explicitly
self-describe enough about how you change the code.
Steps to Test This Pull Request
Steps to reproduce the behavior:

Running command python3 manange.py runserver
Open your preferred browser and go to localhost:8000
Scroll down in this page

Expected behavior
A clear and concise description of what you expected to happen.
The SVG image could show normally
Related Issue
If applicable, refernce to the issue related to this pull request.
#835
More Information
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here. You may also want to refer
to how to wirte the perfect pull request

Codecov Report

Merging #852 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #852   +/-   ##
=======================================
  Coverage   76.37%   76.37%
=======================================
  Files          70       70
  Lines        2688     2688
=======================================
  Hits         2053     2053
  Misses        635      635

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 a452e11...2805ec9. Read the comment docs.

怪哉，我這邊在還沒 Fix 之前就看得到 logo 了... 我先 check 一下是不是某一張特定的 svg 會有這個問題。

Sad，但起來又會有比例問題 orz

我先請贊助組 / 設計組把目前有的 logo 圖檔都先確認一下，然後再來測可能會比較準。

好的，這邊更新一下 svg 圖片沒有顯示的原因是  放在  中但 svg 本身沒有設定 height, width 屬性導致  的 width 為零不會顯示


Move support to more modern versions of ruby and rails

This deprecates support for Rails 3.x and 4.x, while adding support for Rails 6.x. The rationale is that:

Rails 3 is now more than three versions old.
Rails 4 doesn't play nicely with modern bundler.

This also deprecates support for Ruby 2.3, 2.4 and 2.5, which have either been end-of-lifed, or will be in March 2021, and adds support for Ruby 3.0
Overall, this allows for a simpler travis file, and Gemfile.
Ruby 3.0 deprecated allowing the last parameter of a method call being a hash, when using named arguments. This only occurred in one place, and was fixed.
However Rails <IP_ADDRESS> doesn't currently install properly with Ruby 3.0, while Rails 6 does work, so it's been excluded from the travis configuration.
Given this PR removes support for a couple of versions of ruby and rails, it's not really backwards compatible, so it's probably worth bumping the major version.

Nice! Makes sense to me.

Nice! Makes sense to me.


Customize activityBar.activeBorder thickness

Is it possible to make this (active border) customizable? Because creating a new theme just for the sake of it seems 'expensive'.

That is controlled by tab.activeBorder
You can override the color using this technique but I don't think the thickness is configurable, not even by a complete new theme.


HTTP::Server specs fail on OpenSSL 1.1

Error
Steps to reproduce
make libcrystal
bin/crystal spec/std/http/server/server_spec.cr

Expected
Success
But got
................................F.....

Failures:

  1) HTTP::Server handles exception during SSL handshake (#6577)
     Failure/Error: expect_raises(OpenSSL::SSL::Error) do

       Expected OpenSSL::SSL::Error, got #<Errno: SSL_connect: Success> with backtrace:
         # src/openssl.cr:115:15 in 'initialize'
         # src/openssl.cr:100:7 in 'new'
         # src/openssl/ssl/socket.cr:34:9 in 'initialize'
         # src/openssl/ssl/socket.cr:3:5 in 'new'
         # spec/std/http/server/server_spec.cr:517:13 in '->'
         # src/spec/methods.cr:255:3 in 'it'
         # spec/std/http/server/server_spec.cr:494:5 in '->'
         # src/spec/context.cr:255:3 in 'describe'
         # src/spec/methods.cr:16:5 in 'describe'
         # spec/std/http/server/server_spec.cr:234:3 in '__crystal_main'
         # src/crystal/main.cr:97:5 in 'main_user_code'
         # src/crystal/main.cr:86:7 in 'main'
         # src/crystal/main.cr:106:3 in 'main'
         # __libc_start_main
         # _start
         # ???

     # spec/std/http/server/server_spec.cr:516

Finished in 422.48 milliseconds
38 examples, 1 failures, 0 errors, 0 pending

Failed examples:

crystal spec spec/std/http/server/server_spec.cr:494 # HTTP::Server handles exception during SSL handshake (#6577)

Environment
git branch: master
commit fc52166cad58f224e9d54a2d83195702401db75c (currently the last on)
Crystal Version
crystal -v
Crystal 0.26.1 (2018-09-27)

LLVM: 6.0.1
Default target: x86_64-pc-linux-gnu

bin/crystal -v
Crystal 0.26.1 (2018-09-27)

LLVM: 6.0.1
Default target: x86_64-pc-linux-gnu

OpenSSL
openssl version
OpenSSL 1.1.1  11 Sep 2018
OS
uname -a
Linux MyHostname 4.18.9-arch1-1-ARCH #1 SMP PREEMPT Wed Sep 19 21:19:17 UTC 2018 x86_64 GNU/Linux

Additional notes
The error happened originally by running make std_spec. I'm running the spec directly and the error is the same.

I'm interested - can anyone print the error returned by expect_raises(OpenSSL::SSL::Error) on a platform where the spec passes?

Fixed by #7068?

@RX14 Shouldn't this be closed?


imglab: Do not build WebP features if WebP is not available

Approach 1: fix build of imglab when WebP is not available (using ifdef guards around WebP-specific code)
I understand it's not very clean, I can remove the WebP-writing features from imglab in a new commit and just keep the extensions in the match_endings object, as mentioned in #2580.
Anyway, now Travis CI should pass.

Cool, yeah, didn't realize github actions wasn't building this stuff for windows.  This PR looks good.  Let me put up a change to have github actions build this stuff on windows to see that it really builds it and fails too.

Just remembered we run out of ram on github actions when building the examples there :|

Yeah was thinking exactly the same thing 🤞😁

Maybe we can build the tools on Windows only, without the examples, I think that should do it...


Exclude metrics feature not working as intended

I'm trying to exclude logging and some metrics to save money, by limiting the numbers of metrics ingested. See below my config:
logging:
  service:
    pipelines:
      default_pipeline:
        receivers: []
metrics:
  receivers:
    hostmetrics:
      type: hostmetrics
      collection_interval: 60s
  processors:
    metrics_filter:
      type: exclude_metrics
      metrics_pattern:
        - agent.googleapis.com/agent/*
        - agent.googleapis.com/iis/*
        - agent.googleapis.com/interface/*
        - agent.googleapis.com/mssql/*
        - agent.googleapis.com/pagefile/*
        - agent.googleapis.com/swap/*
  service:
    pipelines:
      default_pipeline:
        receivers: [hostmetrics]
        processors: [metrics_filter]

Like this, nothing is excluded correctly, I can see everything in the metrics explorer.
When I keep only " - agent.googleapis.com/agent/* ", it's working, I don't see agent metrics anymore, but if I replace with " - agent.googleapis.com/swap/* " I can still see swap metrics... I am restarting successfully the service every time.
Are there metrics not deactivatable? Or am I not writing the metrics correctly?
Windows Server 2019 Datacenter - using google-cloud-ops-agent.x86_<IP_ADDRESS>@1

This feature was broken in recent agent version and was newly fixed in 2.7.1. https://github.com/GoogleCloudPlatform/ops-agent/pull/264 which was released last week.


mysql run failed

Mysql container can NOT run with error " Fatal glibc error: CPU does not support x86-64-v2".
Related to docker-library/mysql#1055 .
ref: #https://unix.stackexchange.com/questions/631217/how-do-i-check-if-my-cpu-supports-x86-64-v2
/lib64/ld-linux-x86-64.so.2 --help

Subdirectories of glibc-hwcaps directories, in priority order:
x86-64-v4
x86-64-v3
x86-64-v2
Legacy HWCAP subdirectories under library search path directories:
x86_64 (AT_PLATFORM; supported, searched)
tls (supported, searched)
avx512_1
x86_64 (supported, searched)

I would like to propose to update mysql to a broader hardware support version.

The effective specific version tag is mysql:8.0.37-oraclelinux9.

The tag mysql:8.0.37-bookworm can be used for replacement, but it only amd64.
The tag mysql:8.0.37-oraclelinux8 could be used to fix the error.


Rename project to projectId for listing buckets

Fixes issue #48

LGTM, which I believe I means this ":shipit:"


Storage Load N/A

I have been reading through the posts regarding the Storage Load query but I am not smart enough to glean how to fix it.  I am sure that N/A is the value because I am using mount points.  When I use the queries I found as a way to resolve the issue I get errors.
Currently this is the query:
(node_filesystem_size{fstype="aufs",mountpoint="/"} - node_filesystem_free{fstype="aufs",mountpoint="/"}) / node_filesystem_size{fstype="aufs",mountpoint="/"} * 100
Can anyone help a brother out with some pointers as to how I can fix this?

I ran into the same issue, my fstype is ext4 (check yours by running the query node_filesystem_free_bytes alone to see results)
so I adapted my query like this
(node_filesystem_size_bytes{fstype="ext4"} - node_filesystem_free_bytes{fstype="ext4"}) / node_filesystem_size_bytes{fstype="ext4"}  * 100

Ok its getting there.  When I plug in your query I get this

any ideas?  I know this is coming down to coding and formatting but again, not that smart!

Means the query returned two items, you may use filter in Tranformation Data tab to select the one you want to display.
Use "Filter By Name".

COOOOLLLL.  Thanks man!


Reorganization of array-like objects [array-dev]

This branch first proposes some const-correctness "fixes", then replaces the underlying POD data pointer in Vector, DenseMatrix, and DenseTensor with Array objects. By doing so, the logic in these classes is simplified (i.e. who owns data, when to reallocate, etc.) overall simplifying and streamlining the code.
This comes with a caveat: Although the tests I've run do pass, I'm not sure that I mapped over all of the logic from the original POD class versions to these. We'll need to do a thorough review of all the methods before merging, if that happens.
If this is accepted, a follow-on step would be to propose the uvm-dev branch that adds an option to MFEM to use unified memory allocations in all Array<> allocations. This would allow users to quickly port parts of the code to GPUs and could be used in other device ports as well.

Closing for now, as the ideas here have been included in other more recent PRs. We can re-open later if necessary.


Will not change "case" of file names

Microsoft PowerToys version
0.76.2
Installation method
Microsoft Store
Running as admin
None
Area(s) with issue?
PowerRename
Steps to reproduce
select a file or files with capital letters, for example ABCD.pdf
then lets say you click powerrename and then simply click on the lowercase letter formatting, the display preview on the right of the app will show the files to be renamed as "abcd.pdf", but then run and the files are still ABCD.pdf
✔ Expected Behavior
file names should be "abcd.pdf" as shown in the preview on the app
❌ Actual Behavior
file remains as "ABCD.pdf"
Other Software
No response

I just did this on my desktop, while i did have to refresh the desktop (just simple F5), it did rename the file.

What did you maybe do that i didn't?  /needinfo

Attempted to reproduce on Windows 10 with PowerToys 0.76.2 and did not see this issue.
I attempted from desktop and from nested directories.
Is it possible that another application is preventing you from renaming the files?

I finally figured out what it is. It doesn’t work if youre using an external hard drive using the exFAT file system, but does work if using an NFTS file system. Thank you.
From: Ethan Nagano @.>
Sent: Thursday, January 11, 2024 1:24 AM
To: microsoft/PowerToys @.>
Cc: Tripp @.>; Author @.>
Subject: Re: [microsoft/PowerToys] Will not change "case" of file names (Issue #30799)
Attempted to reproduce on Windows 10 with PowerToys 0.76.2 and did not see this issue.
I attempted from desktop and from nested directories.
Is it possible that another application is preventing you from renaming the files?
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/30799#issuecomment-1886371392, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALYVYHUYZLYTFM3OEH2MIRDYN6AOZAVCNFSM6AAAAABBQRKVIWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBWGM3TCMZZGI.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


feat(barcode-scanning): support all barcode types

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

[ ] The changes have been tested successfully.
[ ] A changeset has been created (npm run changeset).
[ ] I have read and followed the pull request guidelines.

Close #105

Android looks good so far. Feel free to continue with iOS.

@mertyldrr Thank you for your first contribution! 💜


San Lorenzo College of Davao, Inc.

@chesteralan Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Gherkin phrases are being nested

Describe the bug
After upgrade to 2.6.1, my allure report shows with nested Gherkin phrases. I had to revert all the way to 2.2.2 before phrases are listed as expected.
To Reproduce
Steps to reproduce the behaviour:

Run a Cypress Cucumber test with @badeball/cypress-cucumber-preprocessor
npx allure serve

Expected behaviour
Report containing Gherkin execution steps in a list
Actual behaviour
Report containing Gherkin execution steps in a list, but nested underneath each other. see screenshot
Screenshots

Desktop (please complete the following information):

language typescript
Browser Chrome headless
@mmisty/cypres-allure-adapter version 2.6.1
cypress version 13.13.2
OS: GitLab CI/CD Pipeline

Attachments
Workaround
Does the issue have workaround? [ yes / no ]
Yes, revert cypress-allure-adapter to version 2.2.2
Additional context

@Alvis-pixel thank you for reporting this, in the progress of fixing.

Could you please check 2.6.3 version, should be fixed there

It looks good in 2.6.3, thank you!


Fix tests namespace

@pranavkm can you do the needful?

We'll use https://github.com/aspnet/Localization/pull/400 to cover this. There's a bunch of incorrect test namespaces in this repo.

Ok


distinct on decimal / interval

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
In PostgreSQL:
test=# select distinct * from (values (1.0), (1.00)) as t;
 column1
---------
     1.0
(1 row)

test=# select distinct * from (values (1.00), (1.0)) as t;
 column1
---------
    1.00
(1 row)

test=# select distinct * from (values (interval '1' month), (interval '30' day)) as t;
 column1
---------
 1 mon
(1 row)

test=# select distinct * from (values (interval '30' day), (interval '1' month)) as t;
 column1
---------
 30 days
(1 row)

In RisingWave:
dev=> select distinct * from (values (1.0), (1.00)) as t;

---
 1
(1 row)

dev=> select distinct * from (values (1.00), (1.0)) as t;

---
 1
(1 row)

dev=> select distinct * from (values (interval '1' month), (interval '30' day)) as t;

------------------
 1 mon 00:00:00
 30 days 00:00:00
(2 rows)

dev=> select distinct * from (values (interval '30' day), (interval '1' month)) as t;

------------------
 30 days 00:00:00
 1 mon 00:00:00
(2 rows)

Expected behavior
Unsure. The PostgreSQL behavior is not preferable either, as it depends on the input order.
Additional context
Add any other context about the problem here.

I think we are right for the interval case, and pg is wrong.
For decimal, I think we our answer is good enough.

Although the difference between 1 month and 30 day feels more significant than the difference between 1.0 and 1.00, it seems we still need to considered them as equal, even in the context of distinct:
We have supported ordering and comparison for the interval type under PostgreSQL semantic, which means all interval values forms a total order, where 1 month equals to 30 day. We even wrapped real/float32 and double/float64 to assign total order for them. It is not an optional to say 1 month and 30 day are incomparable. And as a result of being equal, they should have the same hash value as well.
In #4575, we decided to output a single value for such distinct queries. But to avoid indeterministic due to input order, we will always output the canonical value from the equivalent class, which may be neither of the input value. This issue will be closed after #4600

This also has an impact on correlated subquery, where the domain-calculation step leverages distinct but later on decimal/interval breaks x = y -> f(x) = f(y). This can lead to wrong (unexpected) query results. For example:
PostgreSQL:
test=# select v, (select scale(v)) from (values (1.0), (1.00)) as t(v);
  v   | scale
------+-------
  1.0 |     1
 1.00 |     2
(2 rows)

test=# select v, (select v + date '2024-01-01') from (values (interval '30' day), (interval '1' month)) as t(v);
    v    |      ?column?
---------+---------------------
 30 days | 2024-01-31 00:00:00
 1 mon   | 2024-02-01 00:00:00
(2 rows)

RisingWave:
dev=> select v, (select scale(v)) from (values (1.0), (1.00)) as t(v);
  v   | ?column?
------+----------
  1.0 |        0
 1.00 |        0
(2 rows)

dev=> select v, (select v + date '2024-01-01') from (values (interval '30' day), (interval '1' month)) as t(v);
    v    |      ?column?
---------+---------------------
 30 days | 2024-02-01 00:00:00
 1 mon   | 2024-02-01 00:00:00
(2 rows)

Just acknowledging this as a known limitation. Not proposing any fix.


docs: Add user guide for change your name

Automated message from Dropbox CLA bot
@Alita2, it looks like you've already signed the Dropbox CLA. Thanks!

@Alita2  Just wanted to make sure if you intentionally closed it or it was by mistake.


Fixed project.version conflict in  spring data cosmos

Description

Because of multiple project.properties files in META-INF/project.properties, this tries to get the azure spring data cosmos specific version.
Closes https://github.com/Azure/azure-sdk-for-java/issues/27286

@backwind1233 / @saragluna  - please help review this PR - not sure if my method is correct to fix the above issue.

@backwind1233 / @saragluna - please help review this PR - not sure if my method is correct to fix the above issue.

@kushagraThapar I have tested the code just now, but it seems doesn't resolve the issue, it will return null.

because the code here will load files from
spring-cloud-azure-autoconfigure-4.0.0-beta.3.jar!/META-INF/project.properties or other folders
not azure-spring-data-cosmos-4.0.0-3.19.0-beta.1.jar!/META-INF/project.properties

I think we can specify the folder from META-INF/project.properties to META-INF/cosmos***/project.properties
or we can change the name from project.properties to cosmos***-project.properties;
but I am not sure whether it will cause any other issue?

@kushagraThapar Thank you so much, it looks great to me, I guess you will release this in March, right?

@backwind1233 - yes, this will be shipped next week, given the PR is merged, which is currently blocked on another PR here - https://github.com/Azure/azure-sdk-for-java/pull/27410 or this one - https://github.com/Azure/azure-sdk-for-java/pull/27416
Cosmos CI broke after we upgraded spring framework versions, so these above 2 PRs fix that issue.


Certificate issues running Keycloak in Docker

Describe the bug
I'm running KeyCloak as a docker container and have issues getting the plugin to work.
It loads the plugin as expected, but gets SSL certificate errors trying to communicate with BankID.
Environment:

OS: Windows
Browser Chrome
Keycloak version: 26.0.0
Version of BankID 4 Keycloak: 25.0.0
BankId Client: Desktop

Additional context
Here is the sample docker image:
FROM quay.io/keycloak/keycloak:26.0.0 AS builder
ENV KC_DB=mssql

COPY bankid4keycloak.jar /opt/keycloak/providers
COPY FPTestcert5_20240610.p12 /opt/keycloak/tls/bankid.p12

COPY FPTestcert5_20240610.pem .
RUN keytool -importcert -file FPTestcert5_20240610.pem -storepass qwerty123 -alias "BankID Test CA" -trustcacerts -storetype pkcs12 -keystore /opt/keycloak/conf/truststores/truststore.p12 -noprompt -v

WORKDIR /opt/keycloak
RUN /opt/keycloak/bin/kc.sh build

FROM quay.io/keycloak/keycloak:26.0.0
COPY --from=builder /opt/keycloak/ /opt/keycloak/
COPY /mytheme /opt/keycloak/themes/mytheme

ENTRYPOINT ["/opt/keycloak/bin/kc.sh"]

I initially ran the keytool command without -storepass as describeb in README. This didn't work as the file was protected by password.
I can see that that the files are created and copied to the right locations in the container image.
I've also configured in admin with the updated paths and written as as described in README. I've verified that it can find the certificates.
I'm running the test environment in Azure on an Linux App Service, so the https is managed in Azure. For the docker image the --http-enabled=true is used.
Exception shown in the container:
2024-10-18T10:04:59.1411524Z 2024-10-18 10:04:59,140 WARN  [org.keycloak.cookie.DefaultCookieProvider] (executor-thread-1) Non-secure context detected; cookies are not secured, and will not be available in cross-origin POST requests
2024-10-18T10:05:21.5191581Z 2024-10-18 10:05:21,518 WARN  [org.keycloak.cookie.DefaultCookieProvider] (executor-thread-2) Non-secure context detected; cookies are not secured, and will not be available in cross-origin POST requests
2024-10-18T10:05:25.0401928Z 2024-10-18 10:05:25,039 WARN  [org.keycloak.cookie.DefaultCookieProvider] (executor-thread-2) Non-secure context detected; cookies are not secured, and will not be available in cross-origin POST requests
2024-10-18T10:05:26.0354498Z 2024-10-18 10:05:26,035 WARN  [org.keycloak.cookie.DefaultCookieProvider] (executor-thread-2) Non-secure context detected; cookies are not secured, and will not be available in cross-origin POST requests
2024-10-18T10:05:26.6266176Z 2024-10-18 10:05:26,625 ERROR [org.keycloak.broker.bankid.client.SimpleBankidClient] (executor-thread-2) Failed to send request to BankID: javax.net.ssl.SSLHandshakeException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
2024-10-18T10:05:26.6266791Z 	at java.base/sun.security.ssl.Alert.createSSLException(Alert.java:130)
2024-10-18T10:05:26.6266838Z 	at java.base/sun.security.ssl.TransportContext.fatal(TransportContext.java:378)
2024-10-18T10:05:26.6266920Z 	at java.base/sun.security.ssl.TransportContext.fatal(TransportContext.java:321)
2024-10-18T10:05:26.6266949Z 	at java.base/sun.security.ssl.TransportContext.fatal(TransportContext.java:316)
2024-10-18T10:05:26.6266978Z 	at java.base/sun.security.ssl.CertificateMessage$T13CertificateConsumer.checkServerCerts(CertificateMessage.java:1318)
2024-10-18T10:05:26.6267007Z 	at java.base/sun.security.ssl.CertificateMessage$T13CertificateConsumer.onConsumeCertificate(CertificateMessage.java:1195)
2024-10-18T10:05:26.6267037Z 	at java.base/sun.security.ssl.CertificateMessage$T13CertificateConsumer.consume(CertificateMessage.java:1138)
2024-10-18T10:05:26.6267065Z 	at java.base/sun.security.ssl.SSLHandshake.consume(SSLHandshake.java:393)
2024-10-18T10:05:26.6267092Z 	at java.base/sun.security.ssl.HandshakeContext.dispatch(HandshakeContext.java:476)
2024-10-18T10:05:26.6267118Z 	at java.base/sun.security.ssl.HandshakeContext.dispatch(HandshakeContext.java:447)
2024-10-18T10:05:26.6267160Z 	at java.base/sun.security.ssl.TransportContext.dispatch(TransportContext.java:201)
2024-10-18T10:05:26.6267189Z 	at java.base/sun.security.ssl.SSLTransport.decode(SSLTransport.java:172)
2024-10-18T10:05:26.6267215Z 	at java.base/sun.security.ssl.SSLSocketImpl.decode(SSLSocketImpl.java:1506)
2024-10-18T10:05:26.6267242Z 	at java.base/sun.security.ssl.SSLSocketImpl.readHandshakeRecord(SSLSocketImpl.java:1421)
2024-10-18T10:05:26.6267269Z 	at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:455)
2024-10-18T10:05:26.6267296Z 	at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:426)
2024-10-18T10:05:26.6267324Z 	at org.apache.http.conn.ssl.SSLConnectionSocketFactory.createLayeredSocket(SSLConnectionSocketFactory.java:436)
2024-10-18T10:05:26.6267351Z 	at org.apache.http.conn.ssl.SSLConnectionSocketFactory.connectSocket(SSLConnectionSocketFactory.java:384)
2024-10-18T10:05:26.6267398Z 	at org.apache.http.impl.conn.DefaultHttpClientConnectionOperator.connect(DefaultHttpClientConnectionOperator.java:142)
2024-10-18T10:05:26.6267429Z 	at org.apache.http.impl.conn.PoolingHttpClientConnectionManager.connect(PoolingHttpClientConnectionManager.java:376)
2024-10-18T10:05:26.6267455Z 	at org.apache.http.impl.execchain.MainClientExec.establishRoute(MainClientExec.java:393)
2024-10-18T10:05:26.6267480Z 	at org.apache.http.impl.execchain.MainClientExec.execute(MainClientExec.java:236)
2024-10-18T10:05:26.6267505Z 	at org.apache.http.impl.execchain.ProtocolExec.execute(ProtocolExec.java:186)
2024-10-18T10:05:26.6267530Z 	at org.apache.http.impl.execchain.RetryExec.execute(RetryExec.java:89)
2024-10-18T10:05:26.6267557Z 	at org.apache.http.impl.execchain.RedirectExec.execute(RedirectExec.java:110)
2024-10-18T10:05:26.6267583Z 	at org.apache.http.impl.client.InternalHttpClient.doExecute(InternalHttpClient.java:185)
2024-10-18T10:05:26.6267626Z 	at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:83)
2024-10-18T10:05:26.6267654Z 	at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:108)
2024-10-18T10:05:26.6267680Z 	at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:56)
2024-10-18T10:05:26.6267707Z 	at org.keycloak.broker.bankid.client.SimpleBankidClient.sendRequest(SimpleBankidClient.java:91)
2024-10-18T10:05:26.6267733Z 	at org.keycloak.broker.bankid.client.SimpleBankidClient.sendAuth(SimpleBankidClient.java:47)
2024-10-18T10:05:26.6267758Z 	at org.keycloak.broker.bankid.BankidEndpoint.doLogin(BankidEndpoint.java:109)
2024-10-18T10:05:26.6267815Z 	at org.keycloak.broker.bankid.BankidEndpoint.start(BankidEndpoint.java:88)
2024-10-18T10:05:26.6267863Z 	at org.keycloak.broker.bankid.BankidEndpoint$quarkusrestinvoker$start_e0e4c6a15929b10e63afcdbe4483370874bdf21a.invoke(Unknown Source)
2024-10-18T10:05:26.6267890Z 	at org.jboss.resteasy.reactive.server.handlers.InvocationHandler.handle(InvocationHandler.java:29)
2024-10-18T10:05:26.6267921Z 	at io.quarkus.resteasy.reactive.server.runtime.QuarkusResteasyReactiveRequestContext.invokeHandler(QuarkusResteasyReactiveRequestContext.java:141)
2024-10-18T10:05:26.6267949Z 	at org.jboss.resteasy.reactive.common.core.AbstractResteasyReactiveContext.run(AbstractResteasyReactiveContext.java:147)
2024-10-18T10:05:26.6267975Z 	at io.quarkus.vertx.core.runtime.VertxCoreRecorder$14.runWith(VertxCoreRecorder.java:635)
2024-10-18T10:05:26.6268020Z 	at org.jboss.threads.EnhancedQueueExecutor$Task.doRunWith(EnhancedQueueExecutor.java:2516)
2024-10-18T10:05:26.6268049Z 	at org.jboss.threads.EnhancedQueueExecutor$Task.run(EnhancedQueueExecutor.java:2495)
2024-10-18T10:05:26.6268258Z 	at org.jboss.threads.EnhancedQueueExecutor$ThreadBody.run(EnhancedQueueExecutor.java:1521)
2024-10-18T10:05:26.6268290Z 	at org.jboss.threads.DelegatingRunnable.run(DelegatingRunnable.java:11)
2024-10-18T10:05:26.6268317Z 	at org.jboss.threads.ThreadLocalResettingRunnable.run(ThreadLocalResettingRunnable.java:11)
2024-10-18T10:05:26.6268344Z 	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
2024-10-18T10:05:26.6268369Z 	at java.base/java.lang.Thread.run(Thread.java:1583)
2024-10-18T10:05:26.6268403Z Caused by: sun.security.validator.ValidatorException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
2024-10-18T10:05:26.6268448Z 	at java.base/sun.security.validator.PKIXValidator.doBuild(PKIXValidator.java:388)
2024-10-18T10:05:26.6268475Z 	at java.base/sun.security.validator.PKIXValidator.engineValidate(PKIXValidator.java:271)
2024-10-18T10:05:26.6268500Z 	at java.base/sun.security.validator.Validator.validate(Validator.java:256)
2024-10-18T10:05:26.6268527Z 	at java.base/sun.security.ssl.X509TrustManagerImpl.checkTrusted(X509TrustManagerImpl.java:230)
2024-10-18T10:05:26.6268555Z 	at java.base/sun.security.ssl.X509TrustManagerImpl.checkServerTrusted(X509TrustManagerImpl.java:132)
2024-10-18T10:05:26.6268582Z 	at java.base/sun.security.ssl.CertificateMessage$T13CertificateConsumer.checkServerCerts(CertificateMessage.java:1302)
2024-10-18T10:05:26.6268612Z 	... 40 more
2024-10-18T10:05:26.6268640Z Caused by: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
2024-10-18T10:05:26.6268685Z 	at java.base/sun.security.provider.certpath.SunCertPathBuilder.build(SunCertPathBuilder.java:148)
2024-10-18T10:05:26.6268714Z 	at java.base/sun.security.provider.certpath.SunCertPathBuilder.engineBuild(SunCertPathBuilder.java:129)
2024-10-18T10:05:26.6268740Z 	at java.base/java.security.cert.CertPathBuilder.build(CertPathBuilder.java:297)
2024-10-18T10:05:26.6268798Z 	at java.base/sun.security.validator.PKIXValidator.doBuild(PKIXValidator.java:383)
2024-10-18T10:05:26.6268823Z 	... 45 more

This is what's described in the official docs. I assume it may has issues trusting the certificate because it's password protected?

The existing Java default truststore certs will always be trusted. If you need additional certificates, which will be the case if you have self-signed or internal certificate authorities that are not recognized by the JRE, they can be included in the conf/truststores directory or subdirectories. The certs may be in PEM files, or PKCS12 files with extension .p12 or .pfx. If in PKCS12, the certs must be unencrypted - meaning no password is expected
https://www.keycloak.org/server/keycloak-truststore

Update, think I have managed to get it working now. I will update with details later if anyone runs into the same problem.
However, now I ran right into the next exception.
org.infinispan.commons.marshall.MarshallingException: No marshaller registered for object of Java type org.keycloak.broker.bankid.model.AuthRespons

Got it working by using version 25.0.0. Is there any plan to upgrade and support 26.0.0?

@Ronnehag  Thanks to @mikaelkaron there is a 26.0.x version build.
The http client used when communicating with BankID is not using the default java trustore, instead it is using the one configured in the IDP settings when you setup the IDP in keycloak.

Will close this issue. Please reopen if there is anything else we can help you with.


rosdep for python-httplib on new distributions

Can you please add links to the packages as described in https://github.com/ros/rosdistro/blob/master/CONTRIBUTING.md#rosdep-rules-contributions ?  Thanks.

Sorry, didn't see that.
https://packages.ubuntu.com/focal/python-httplib2
https://packages.debian.org/buster/python-httplib2
https://packages.ubuntu.com/focal/python3-httplib2
https://packages.debian.org/buster/python3-httplib2
https://packages.gentoo.org/packages/dev-python/httplib2
Fedora App server is currently down, maybe this is enough: https://src.fedoraproject.org/rpms/python-httplib2


Support long press on E1524

If you hold the power button for 3 seconds it sends a cmdMoveToLevelWithOnOff with level: 254.

Actually this is useless, because you can't avoid the initial toggle command when holding down the button, which means that you'd always get automation conflicts unless you skipped the toggle button, which would be pointless.


Fix OpenMM Precision Bug

There was a bug (presumeably) by some merge faults so that the openmm_platform_properties were saved with an engine, but not used in the construction of the actual OpenMM Instance.
This will fix this. On my machine now OpenCL does not work with mixed precision. This seems to be the cause of all the NaNs.
@jchodera This is an important fix for you, too, if you use OpenCL!!!!

Took a couple re-runs to pass tests, but merging now! @jhprinz, thanks for figuring this out last night, after hours of text messages as we tried to figure out what the problem might be!

Fingers crossed that this fixes everything!


Micro ROS agent stopping and restarting when driving stepper motor. Dual core

Issue template

Hardware description: ESP32 and NEMA 17 driven with A4988 on 1/16th step mode
Installation type: humble

Steps to reproduce the issue
I found that i cannot drive the stepper motor within the main loop where 'RCCHECK(rclc_executor_spin_some(&executor, RCL_MS_TO_NS(100)))' is being spun. Since the steppers are extremely time sensitive, a delay of a few microseconds within the loop driving the steppers can affect the speed. I decided to use the ESP32 second core to handle stepper control and the main core for micro-ros. I have a gamepad connected where the y axis of a joystick on the gamepad is inclination and the x axis is steer_vel (these values only make sense since i am making a 2 wheel balancing robot)
Here is my code that causes the steppers to spin using the joystick for about a few seconds and then the micro-ros-agent restarts again and again.
#include <micro_ros_arduino.h>

#include <stdio.h>
#include <rcl/rcl.h>
#include <rcl/error_handling.h>
#include <rclc/rclc.h>
#include <rclc/executor.h>

#include <geometry_msgs/msg/twist.h>

#include <Arduino.h>

rcl_subscription_t subscriber;
geometry_msgs__msg__Twist msg;
rclc_executor_t executor;
rcl_allocator_t allocator;
rclc_support_t support;
rcl_node_t node;

#define LED_PIN 13

#define RCCHECK(fn) { rcl_ret_t temp_rc = fn; if((temp_rc != RCL_RET_OK)){error_loop();}}
#define RCSOFTCHECK(fn) { rcl_ret_t temp_rc = fn; if((temp_rc != RCL_RET_OK)){error_loop();}}

#include <AccelStepper.h>

// Define the number of steps per revolution for your stepper motor
const int step_resolution = 16;  // One 16th of a full step. Full step is 1.8 degrees
//const int STEPS_PER_REVOLUTION = 200.0 / step_size;

// Define the pins connected to the A4988 driverS
const int L_STEP_PIN = 33;
const int L_DIR_PIN = 25;
const int R_STEP_PIN = 12;
const int R_DIR_PIN = 13;
const int L_MS1 = 32;
const int L_MS2 = 19;
const int L_MS3 = 18;
const int R_MS1 = 26;
const int R_MS2 = 27;
const int R_MS3 = 14;

AccelStepper L_stepper(1, L_STEP_PIN, L_DIR_PIN); // Define left stepper motor object
AccelStepper R_stepper(1, R_STEP_PIN, R_DIR_PIN); // Define right stepper motor object

TaskHandle_t Core1Task; // Task handle for Core 1

float inclination;
float steer_vel;

void error_loop(){
  while(1){
    digitalWrite(LED_PIN, !digitalRead(LED_PIN));
    delay(100);
  }
}

void driveStepper(void *param){
  pinMode(L_MS1, OUTPUT);
  pinMode(L_MS2, OUTPUT);
  pinMode(L_MS3, OUTPUT);
  pinMode(R_MS1, OUTPUT);
  pinMode(R_MS2, OUTPUT);
  pinMode(R_MS3, OUTPUT);

  // Set microstep pins to HIGH for 16 microsteps
  digitalWrite(L_MS1, HIGH);
  digitalWrite(L_MS2, HIGH);
  digitalWrite(L_MS3, HIGH);
  digitalWrite(R_MS1, HIGH);
  digitalWrite(R_MS2, HIGH);
  digitalWrite(R_MS3, HIGH);

  L_stepper.setMaxSpeed(8000);
  L_stepper.setAcceleration(3000);
  R_stepper.setMaxSpeed(8000);
  R_stepper.setAcceleration(3000);

  int count = 0;
  int iterations = 1000;
  int steps_per_cycle = 200;

  while (true){
    R_stepper.setSpeed(inclination*steps_per_cycle);
    R_stepper.runSpeed();

  }
}
//twist message cb
void subscription_callback(const void *msgin) {
  const geometry_msgs__msg__Twist * msg = (const geometry_msgs__msg__Twist *)msgin;
  // if velocity in x direction is 0 turn off LED, if 1 turn on LED
  digitalWrite(LED_PIN, (msg->linear.x == 0) ? LOW : HIGH);
  inclination = msg->linear.x;
  steer_vel = msg->linear.y;
}

void setup() {

  xTaskCreatePinnedToCore(
    driveStepper,          // Function to implement the task
    "Core 1 Task",      // Name of the task
    10000,              // Stack size in words
    NULL,               // Task input parameter
    1,                  // Priority of the task
    &Core1Task,         // Task handle
    0                   // Run task on Core 1
  );

  set_microros_transports();
  pinMode(LED_PIN, OUTPUT);
  digitalWrite(LED_PIN, HIGH);

//  delay(2000);

  allocator = rcl_get_default_allocator();

   //create init_options
  RCCHECK(rclc_support_init(&support, 0, NULL, &allocator));

  // create node
  RCCHECK(rclc_node_init_default(&node, "micro_ros_arduino_node", "", &support));

  // create subscriber
  RCCHECK(rclc_subscription_init_default(
    &subscriber,
    &node,
    ROSIDL_GET_MSG_TYPE_SUPPORT(geometry_msgs, msg, Twist),
    "/combined_data"));

  // create executor
  RCCHECK(rclc_executor_init(&executor, &support.context, 1, &allocator));
  RCCHECK(rclc_executor_add_subscription(&executor, &subscriber, &msg, &subscription_callback, ON_NEW_DATA));

}

void loop() {
//  delay(100);
  RCCHECK(rclc_executor_spin_some(&executor, RCL_MS_TO_NS(100)));
}

Expected behavior
The stepper should always spin when the joystick feeds it commands instead of stopping the micro ros- agent and starting again

Actual behavior
stepper motor spins for a bit then stops then spins again once micro-ros-agent restablishes connection
Additional information

Maybe you can increase the priority of the stepper task to a greater number than the micro-ROS task

Hi Pablo. I again ran into this issue in another project involving ESP NOW.
It seems that when doing tasks on another core it seems that playing with the priority level is the way to prevent the micro ros agent from deleting the session and re establishing it.
For the stepper a priority of 0 worked but for the ESP NOW I had to use 3
Not sure what is going on


Conditional diagram changes causes weird bugs

Describe the bug
Conditional diagram changes causes weird bugs. If changing the diagram is set up to be changed conditionally, based on some flag, it will result in inconsistent diagram between toggles. Try to lock the diagram by preventing the calls to the onChange event, do some changes to the diagram visually, nothing should change (which is normal), unlock the diagram, and you will notice your changes (made while the diagram was locked) to be applied instantly on the diagram.
To Reproduce
Reproduction
Steps to reproduce the behavior:

Go to https://c5y6s.sse.codesandbox.io/.
Try to move any node, nothing moves, which is normal giving that the diagram is locked.
Press the "unlock" button below, and you will notice that your changes (made while the diagram was locked) are applied.

Expected behavior
For changes made while the diagram is locked not to be applied once the diagram is unlocked.
Desktop (please complete the following information):

OS: Win64
Browser: Chrome
Version: 91

I've looked into the source code, and it turned out it's an easy fix (assuming I'm not breaking anything):
https://github.com/beautifulinteractions/beautiful-react-diagrams/blob/c18805eb503b810ec8ec53fc3923d8d6d1ae0d7e/src/Diagram/NodesCanvas/NodesCanvas.js#L16-L21
In the above code snippet, you are updating the nodes coordinates only if the onChange event prop is true, which is always the case given that it's always passed the reference of the function expression onNodesChange:
https://github.com/beautifulinteractions/beautiful-react-diagrams/blob/c18805eb503b810ec8ec53fc3923d8d6d1ae0d7e/src/Diagram/Diagram.js#L23-L27
https://github.com/beautifulinteractions/beautiful-react-diagrams/blob/c18805eb503b810ec8ec53fc3923d8d6d1ae0d7e/src/Diagram/Diagram.js#L74-L86
Fixing it should be as easy as passing the onChange prop provided by the user to the onChange prop of the NodesCanvas component, instead of passing it the always-true onNodesChange function expression.


DeepSort algorithm的链接找不到了

你好，你做的很棒。想问一下，DeepSort algorithm是哪个算法？对应的链接找不到内容了。是TianzongSong的Real-Time-Action-Recognition项目中用到的sort算法吗?

@sunfeng90 Deep sort is a multi object multi instance tracking algorithm .  https://github.com/nwojke/deep_sort https://arxiv.org/abs/1703.07402

@ajithvallabai Thanks.Do you know how to see the details of   the  VGG_origin model and mobilenet_thin?

@sunfeng90 What details ? do u want to set the details  ?

@sunfeng90 What details ? do u want to set the details ?

@ajithvallabai the code of   models. Do  you  know it?

@sunfeng90  yes models are present here Online-Realtime-Action-Recognition-based-on-OpenPose\Pose\graph_models
and its loaded in main.py

您好，我想知道还有下载 ./download.sh的路径吗，有的话能给我发一下吗


Add defineLocalSgpr

Added a new method to manage SGPR usage: defineLocalSgpr. This allows in-place defining/undefing of SGPRs.
Also added new parameter PrintRegisterDebug that informs the invoker if SGPRs have not been explicitly undefined, as a optimization hint.
The old defineSgpr function still works as expected and is in place elsewhere in the code.

CI tests failed due to incorrect asm code.
Please fix them.

@awhittle3 Some CI tests failed, do we still need this change?

This PR will conflict with PR #1843, which probably has higher priority. I'll address the issues here once that PR is merged into develop.
@AlexBrownAMD believes this change will be helpful in managing SGPR usage.

@awhittle3 Some CI tests failed, do we still need this change?

This PR will conflict with PR #1843, which probably has higher priority. I'll address the issues here once that PR is merged into develop.
@AlexBrownAMD believes this change will be helpful in managing SGPR usage.

#1843 is merged today. Would you please restart this PR?

@awhittle3 we are approaching FC 6.2 and we want to clear all the requests. Please let me know if you are planning to proceed with this PR. Thanks.

Out of date and would take a while to spin back up. Closing work on this for now.


Use warnings instead of print statements

Reference issue
Partially addresses: https://github.com/scipy/scipy/issues/1953
What does this implement/fix?
Use warnings.warn instead of print statements in scipy/optimize/optimize.py, specifically within def _minimize_bfgs and def _minimize_powell
Additional information
Related PR with partial fixes for another function: https://github.com/scipy/scipy/pull/14050
As discussed with @WarrenWeckesser, it might be worth having a look at changes in scipy/optimize/tests/test_optimize.py, to understand if those tests always generate a warning and, if that's the case, use something like with pytest.warns(OptimizeWarning):, instead of just filtering the warnings.

I have tried the approach suggested by @mdhaber, i.e. creating a separate test with an expected failure, but passing a maxiter with a low value does not raise any warning. I run the tests a couple times, and I always get Failed: DID NOT WARN. No warnings of type (<class 'scipy.optimize.optimize.OptimizeWarning'>,) was emitted, using a maxiter = 3, or less.

Interesting. Can you tell how many iterations are occurring when you give it maxiter=3? It's possible that the function is not obeying the iteration limit, and that would be another issue!

@StefanoFrazzetto would you be interested in continuing this if I were to help fix merge conflicts?

@mdhaber it's been a while! If it adds any value to the project, I'll try to find some time and help you getting this merged. By the way, while setting up my env, I took the chance to resolve the conflicts, so hopefully this won't take too much effort. Let me know how I can help.

Hmm oops I didn't notice that gh-1953 was already resolved by gh-15259. This may not be needed anymore.
The refactoring of the minimize_powell work is nice, but typically I wouldn't encourage refactoring unless it has some other added benefit. Perhaps we should have raised an OptimizeWarning from the start, but I'm not sure that we can now, since they are RuntimeWarnings, whereas OptimizeWarning is a subclass of UserWarning. If people are filtering them, their filters will break.
It looks like gh-15259 didn't change bfgs.
@AtsushiSakai can I get your opinion here, since you worked on this before? What should this PR do now that the main issue (gh-1953) is resolved?

I realized now that https://github.com/scipy/scipy/pull/15259 fixed almost bare print statements, but the _minimize_bfgs ones are missed:
https://github.com/scipy/scipy/blob/2f36c582363b6e75ea1fca68bd87043f4d0b983c/scipy/optimize/_optimize.py#L1441-L1445
So, I think it would be great that this PR fixs these.
I reopened https://github.com/scipy/scipy/issues/1953.

@AtsushiSakai what warning class should be used? You used RuntimeWarning for the others, but OptimizeWarning would be more appropriate.

but OptimizeWarning would be more appropriate.
Yeah. I didn't know the OptimizeWarning, but I agree it is more appropriate.

In that case, do you think we should replace the RuntimeWarnings with OptimizeWarnnings, then? It's slightly backward incompatible, but I think if an email were sent to the mailing list suggesting the change, it probably wouldn't bother people.
If it does, then this PR could keep using them, and afterwards we could go back and change the others.
What do you think @StefanoFrazzetto @AtsushiSakai? Send an email?

@AtsushiSakai I was hoping to get your opinion on https://github.com/scipy/scipy/pull/14069#issuecomment-1273444118 when you have a moment.

Let's go ahead and leave these as RuntimeWarnings so we can merge this. @AtsushiSakai does this look good to you?
You can test locally with:
import numpy as np
from scipy import optimize

def f(x):
    return np.sum(x**2)

method = 'newton-cg'  # PR changed bfgs, cg, newton-cg, and powell
maxiter = 1000  # if 10, warning will be raised

options={'disp': True, 'maxiter': maxiter}
optimize.minimize(lambda x: optimize.rosen(x), [0, 0], method=method,
                  jac=optimize.rosen_der, options=options)

@StefanoFrazzetto would you be willing to add unit tests that try to raise the warning and check that it occurs?

OK, I fixed two little bugs and added tests. Since it was approved already, I'll go ahead and merge. Thanks, all!


C/C++ plugin, includePaths should not search recursively

VSCode Version: 1.10.2
OS Version: OS/X 10.11.2

#492 was landed, which helps a little with #374 but still doesn't solve our problem completely.
Let's say I have a directory structure like this:
mbed-os
    - mbed.h
    - folder1
        - rtos.h
    - folder2
        - unrelated-file.h

I create my c_cpp_properties.json like this:
        {
            "includePath": [
                "${workspaceRoot}/mbed-os",
                "${workspaceRoot}/mbed-os/folder1"
            ],
            "name": "Mac",
            "browse": {
                "databaseFilename": "",
                "limitSymbolsToIncludedHeaders": false
            }
        }

However, the includePaths are searched through recursively and thus folder2 is still added to intellisense.
We need a setting to not search through includePaths recursively.
Related #475.
/cc @sg-

Move it to the c/c++ plugin.


[feature] Hide *.js files from sidebar if it's *.coffee source exists

I'm using coffee-compile plugin to compile *.coffee files on save. Unfortunately, it does not provide options to choose where to store compiled files. It just creates *.js file near the original coffee one. If directory has over 10 coffee files, sidebar will be messed with 20 files (10 pairs, 1 coffee and 1 js for each pair), that may disturb the developer.
It would be nice if Atom has option to hide *.js file if there's *.coffee file with the same name. It may look like coffee file with dropdown arrow, js file is shown as subtree by click. It works the same way in JetBrains WebStorm.

Another approach is to use coffee-script tool directly, compiling app to separate directory ./build, but it has two major issues:

you always need to have another terminal session started
this ruins one-window Atom workflow

So, is there any exiting plugin that may do the trick?
Thank you!

This issue was moved to atom/tree-view#431


feat: chapter 17 - taking pictures with intent

Chapter 17: Taking Pictures with Intents
This pull request adds a new feature to the application.
Topics Covered

A Place for Your Photo
File Storage
Using a Camera Intent
Scaling and Displaying Bitmaps
Declaring Features

How to Test
To test the new feature, you can do the following:

Clone the repository.
Install the dependencies.
Run the application.
The new feature should be available.

Screenshots

Known Issues
There are no known issues with this pull request.
References
Reviewers
Thanks!

Looks good to me.


SQL Server 2017 SP1

With the release of SP1 for SQL Server 2017, I assume the :latest tag will be updated with this service pack.  How do I get notified when this is available both for linux and Windows versions?  What do I watch?

There will not be "service packs" going forward.  There will only be Cumulative Updates (CUs).  Please see this blog post for more details.  https://blogs.msdn.microsoft.com/sqlreleaseservices/announcing-the-modern-servicing-model-for-sql-server/
Part of this change to modern servicing is more predictable releases. You can expect a new CU every month in the week of the 3rd Monday for the first year after a major release.  For years 2-5, there will be a quarterly CU that is also released in the week of the 3rd Monday.
You can use something like IFTTT to monitor the SQL Server Release blog RSS feed for release announcements:
https://blogs.msdn.microsoft.com/sqlreleaseservices/

I meant CU#1 in my original question.  My question is how will I know when the docker images have been updated?  As of yesterday, at least, the Windows microsoft/mssql-server-windows-developer:latest was still RTM, not CU1.  What is the communicated channel for updates to the docker hub images?

The "communication mechanism" is the blog mentioned above.  When you see that happen you can look at the Tags tab on Docker Hub to see what the current tags are.


Exit gracefully if API token is invalid

Currently, the Ring object is not captured in a try/catch so if the API token is invalid, the script exits with a large block of error text.

Fixed with 2ebe8dd9463204cd16aff3b26113ae1c2c29a9d1.


Feature/add mocha tests tweaks

Related issue: #9

Add frictionless and ckan-client-js libs into the project for the testing
Add giftless url opton in cypress config - Once this gets merged make sure you update cypress.json accordingly.
Create a basic test for pushing a blob with Mocha

Closes #10

Great, thank you guys! I see the other commits here, thanks for the fixes @cotts 👍

I would prefer to have CKAN client and Fricitionless as dependencies (rather than development dependencies) because the aim of this repo is to be NPM-installed by other projects. Keeping these dependencies in dependencies instead of devDependencies avoids missing packages in this third parties, NPM-installed scenario.
For example: let's say the Government of Foo Bar Republic is my client and I install a CKAN instance for them. I can create a ckan-foo-bar-uat in which I would npm install ckan-integration-tests — keeping the CKAN client and Frictionless in devDependencies would not install these packages in my newly created repo, right?

@cuducos OK, I got you, actually was not clear on this and put there in order to bring attention. So my point is that how we can make sure that the versions we use are the right ones? I mean if we have a client X who uses some CKAN app based on let's say React (e.g. datapub), which uses some specific version of ckan-client-js and frictionless-js then shouldn't we provide those libs from that app to our tests? Or alternatively those libs will be installed by that app. For example if we try to install datapub it will automatically install those libs, as those libs are already listed in the dependencies.
I think maybe instead of devDependencies we should use peerDependencies?

Wow… didn't known about peerDependencies — sounds like a good use case for that!


Remove BC break (return value type hinting)

Return value type hinting is a feature introduced in PHP 7 but is used in release 1.0.5.
http://php.net/manual/en/functions.returning-values.php#functions.returning-values.type-declaration
This library supports PHP >= 5.3 as stated in composer.json.
Maybe think about some simple travis test instances for some PHP versions?

@javiereguiluz @theofidry Any updates on this? Every project using php<7.0 performing a composer update will be broken (or needs to specify a specific version of easy-log-handler as a workaround).

I’m sure that if someone do a pull request to fix it Javier will be quick
to merge and release a new tag.
On Sat 16 Jun 2018 at 09:56, Robert Freigang<EMAIL_ADDRESS>wrote:

@javiereguiluz https://github.com/javiereguiluz @theofidry
https://github.com/theofidry Any updates on this? Every project using
php<7.0 performing a composer update will be broken (or needs to specify a
specific version of easy-log-handler as a workaround).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/EasyCorp/easy-log-handler/pull/13#issuecomment-397798746,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE76gTZzuD7SbtSjvtE3yXS8kwHu-Ndsks5t9MgmgaJpZM4UlNrp
.

Thanks for fixing this. I tried to merge it with the merging tool we use in the Symfony project ... but I couldn't do that. It returned merging errors I couldn't fix. So I made the change myself and included it in the latest 1.0.7 version (https://github.com/EasyCorp/easy-log-handler/releases/tag/v1.0.7). I'm sorry!


Updates to existing multi-cluster configuration are not picked up by Pilot

Bug description
Currently, when an existing multi-cluster Secret is updated, no action is taken by the secret controller.
This causes problems in situations where the configuration is modified to update, for example, the API endpoint for remote K8s masters that Pilot calls to infer workload information. Without a restart of Pilot pods, this configuration is not reflected. Principle of least surprise would lead us to believe there should be an update of the multi-cluster clients in this case.
A real world consequence of this in our deployment was that the API endpoints for the remote clusters was updated, Pilot did not pick up the configuration and could not fetch workload information from the remote clusters (the endpoints had been removed, but Pilot continued to attempt to scrape them). This resulted in stale endpoint configuration in each of the data plane proxies.
Expected behavior
Updating an existing multi-cluster secret with a new K8s API endpoint should result in existing clients being shut down and new clients being instantiated with the new endpoint configuration.
This could just be missing documentation (I took a look through the multi-cluster docs, but didn't see anything), however, I believe not honoring an update to an existing secret is something that should be addressed.
Steps to reproduce the bug
Updates are currently not handled in the secret controller:
https://github.com/istio/istio/blob/2a54b0a217b3520688c97a34605aaec2cfe90664/pkg/kube/secretcontroller/secretcontroller.go#L122-L137
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.2", GitCommit:"c97fe5036ef3df2967d086711e6c0c405941e14b", GitTreeState:"clean", BuildDate:"2019-10-15T23:41:55Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"13+", GitVersion:"v1.13.11-gke.14", GitCommit:"56d89863d1033f9668ddd6e1c1aea81cd846ef88", GitTreeState:"clean", BuildDate:"2019-11-07T19:12:22Z", GoVersion:"go1.12.11b4", Compiler:"gc", Platform:"linux/amd64"}

$ istioctl version --remote
client version: unknown
control plane version: 1.4.2

NOTE: Despite running 1.4.x, the problem still exists on master.
How was Istio installed?
Helm.
Environment where bug was observed (cloud vendor, OS, etc)
GKE.

I remember we do support update in pilot before use the common secret controller

FYI https://github.com/istio/istio/pull/8479/files


cmd: move snapshots utility main and options

ASAN failure is not related to this PR, it was broken on master since https://github.com/erigontech/silkworm/pull/2561


Investigate test_basebackup_with_high_slru_count failure

Just now: https://neon-github-public-dev.s3.amazonaws.com/reports/main/8937443657/index.html#suites/c62b105f3a4f00dd6be4ad88810e0e02/7ddb54b21e1dcce3

I looked at the failure Joonas linked. It's a failure to start-up after a SIGQUIT stop for both the ps and the sk (we do that as part of the test setup).
Upon starting both the ps and sk seem to be stuck waiting for their PID file flocks (based on logs). The fd for the pidfile should have been dropped and, in any case, we shouldn't block on the lock since we use LOCK_NB. Must be missing something.

Also seen in https://neon-github-public-dev.s3.amazonaws.com/reports/main/8938399293/index.html#suites/c62b105f3a4f00dd6be4ad88810e0e02/a0635cfcff8697b9/.

This week:

eyeball code again
add more logs

Seen in this benchmark run: https://neon-github-public-dev.s3.amazonaws.com/reports/main/9204710983/index.html
There are two possibly-related failures:

test_basebackup_with_high_slru_count is timing out on pageserver startup, where the pageserver log prints the version and then nothing else: this may mean it is stuck on its flock call for a pidfile.
test_download_churn is timing out on pageserver shutdown, where the pageserver prints that it has handled the signal, but neon_local does not see the process disappear.

also hit this
test_basebackup_with_high_slru_count

Just checked a contemporary failure of this: https://neon-github-public-dev.s3.amazonaws.com/reports/main/10058440605/index.html#suites/c62b105f3a4f00dd6be4ad88810e0e02/8761d88f34fef590/
It appears to still be something like this:

Upon starting both the ps and sk seem to be stuck waiting for their PID file flocks (based on logs).

(i.e. we just see an INFO version line and then nothing)


Undo History Glitch

Which @ngneat/elf-* package(s) are the source of the bug?
state-history
Is this a regression?
No
Description
When stateHistory is paused, the previous value is partially retained when updating.
https://github.com/ngneat/elf/blob/master/packages/state-history/src/lib/state-history.ts#L62
      .pipe(pairwise()) // ← the previous value is stored in this operator
      .subscribe(([past, present]) => {
        if (this.skipUpdate || this.paused) {
          return;
        }
        …

Need to change to
      .pipe(
        filter(() => !(this.skipUpdate || this.paused)), // ← Need to skip the value before it saves
        pairwise(),
      ).subscribe(([past, present]) => {
      …

Exactly the same problem exists in Akito
https://github.com/datorama/akita/blob/master/packages/akita/src/lib/plugins/stateHistory/stateHistoryPlugin.ts#L94
Please provide a link to a minimal reproduction of the bug
https://stackblitz.com/edit/typescript-bjbno4?file=index.ts
Please provide the exception or error you saw
No response
Please provide the environment you discovered this bug in
No response
Anything else?
No response
Do you want to create a pull request?
Yes

You're welcome to create a PR


Fix result when no existing subnets are created

It's returning 256 ip addresses while only 10 where requested.

Fixed with PR: https://github.com/stefanstranger/AzureSubnetCopilot/pull/3


Resource consumption during construction

I would like to know from the author of the mod. No resources are spent during construction, is that how it should be? Or is work still in progress?

It should consume resources for what you build, just provide a convenient test for now.
I think it will be update before next major feature, maybe after the current todo list or earlier.
It will become an independent system to store your resource to avoid the case of the huge resource much more than your bag.
And, maybe need a little development work.


Fix broken link. Closes #87

The change should be done in Lokalise to allow it to survive site updates. In addition, there's a specific format it has to have.
Thanks for the effort. I'll close this PR in favour of the permanent change 👍


Put the project up for adoption

Not sure who takers would be, but this project is so useful. It would be a shame to see it die from low activity from the current maintainers. Not sure if anyone has looked at getting it included in the Apache umbrella or similar ones, but it may be worth while to see if there are any takers.

https://github.com/querydsl/querydsl/issues/2459

Thanks @kartoffelsup ! I didn't see that issue when searching. used the wrong keyword.


Limitation of possiblecells

added a few lines of code in last commit, which solve the bug with some incorrect moves.

Emil, please resolve the conflicts


Expand collapse IE11

https://open.rocket.chat/channel/support?msg=uM7f27Hn6Gye3FjcN

hi, we arwe observing in IE 11, the administation menu, expand/collapse icon is not working

Stale.


error building vmware image for centos7.1

Hi,
I received the following error building the VMware image.
Setup is unable to find the "ifconfig" program on your machine. Please make sure it is installed.
I found this VMware KB which specifies that the net-tools package must be installed.
http://kb.vmware.com/selfservice/microsites/search.do?language=en_US&cmd=displayKC&externalId=2075519
I added the package to the script vmware.sh.

Hi, @mmarseglia
Thanks for reporting this issue.
I'm the new mainainer of this project, but I use virtualbox only, not vmware.
Is the diff below what you meant?
diff --git a/centos-7.1/scripts/vmware.sh b/centos-7.1/scripts/vmware.sh
index 6b48740..0c7b391 100644
--- a/centos-7.1/scripts/vmware.sh
+++ b/centos-7.1/scripts/vmware.sh
@@ -1,4 +1,4 @@
-yum install -y fuse-libs
+yum install -y fuse-libs net-tools
 mkdir -p /mnt/vmware
 mount -o loop /home/vagrant/linux.iso /mnt/vmware

Could you send us a pull request so that you will be listed in https://github.com/shiguredo/packer-templates/graphs/contributors once the pull request is merged?
Thanks!

I'm closing this issue since there's no update for a long time.
Feel free to reopen this issue when you have a progress.


The source code of some javaFx classes is no longer accessible at debugging in Eclipse

Expected Behavior
When in debug mode the program stops on a line of code, the code of the class is visible.
Current Behavior
Only on part of the JavaFx classes, during the debug, instead of displaying the source code of the class, the editor displays "Source not found" with a button entitled "Edit source lookup path ...".
Context
This issue manifests in Eclipse 4.26.0 while it does not in Eclipse 4.25.0. I tried combinations with different versions of Gradle (7.6.1 and 8.0), different versions of JavaFX (16 and <IP_ADDRESS>), different versions of JavaFX Gradle plugin (0.0.10 and 0.0.13) and different versions of JDK (15.0.4 and 19.0.2) but the result is always the same: Eclipse 4.25.0 = success and Eclipse 4.26.0 = failure.
Because of this, debugging becomes much less comfortable than before when certain JavaFX classes come into play.
Steps to Reproduce
In Eclipse 4.26.0 put a breakpoint on the javafx.stage.Window class (on the class declaration line) then launch in debug mode a JavaFX example which opens a window.
The source code of the class will not be accessible.
Your Environment
MacOS 12.2
Eclipse 4.26.0
JDK 19.0.2
JavaFX <IP_ADDRESS>
Gradle 7.6.1
JavaFX Gradle plugin 0.0.13

Thank you for your interest in Gradle!
This is an issue with Eclipse, which cannot be addressed in the Gradle Build Tool project.
Please open such Eclipse-related issues on their Issue Tracker at https://bugs.eclipse.org/bugs/.


app_installations appears to be broken

I'm not entirely sure here why a set would be preferred over a numbered list, but with this usage:
app_installations   = [ var.some_number ]
Produces:
The given "for_each" argument value is unsuitable: "for_each" supports maps
and sets of strings, but you have provided a set containing type number.

Error: Invalid for_each set argument

  on .terraform/modules/terraform_github_modules/main.tf line 505, in resource "github_app_installation_repository" "app_installation_repository":
 505:   for_each = var.app_installations
    ├────────────────
    │ var.app_installations is set of number with 1 element

Related: https://github.com/mineiros-io/terraform-github-repository/pull/120

Thanks for reporting this. Yes - actually this should be a set of strings as an installation ID can be eg. 01234. I am gonna provide a fix for this later today.

We just released v0.16.2 which contains the fix for this issue! Again, thanks for bringing this to our attention!