Fix the Hello.mp3 file

Modified the rate to 24kHz and changed to Mono to match other alexa sound files,
Small fix but the hello.mp3 we were using which I think dates back to my alexaweb project! was 16Khz stereo MP3, the other files along with the responses from AVS are 24Khz Mono, this file wasn't playing right on my Jabra speaker it was too fast, have also tested this on my regular Pi

@sammachin Works fine (not that I was expecting anything else).
Looking at the file, it seems like there's almost two seconds ... of silence after the hello itself. I'm thinking, should we crop it? Not that it really matters, since almost all parts of the initialization happen before playing this and in threads, so the only thing it blocks is platform indication of success (blinking green LEDs and shit), which no one cares about, but if it's there like to keep the audio output open, so there is no cut/mess after the sound, I believe one second should be enough. Usually this is needed before the sound, not after it, before the sound card wakes up, so I'm not sure if this actually makes sense ... Thoughts?

Yeah I did wonder about trimming the audio down but then it's not really a
log file and it's used at startup rather than as part of an interaction so
I figured it was bette left slightly longer than risk clipping it on
unknown hardware, particularly thinking of things like Bluetooth speakers
where the connection may need to be setup.
I take your point the we could trim the end down if we wanted to but it's
minimal gain.
On Fri, 10 Mar 2017 at 08:02, René Kliment<EMAIL_ADDRESS>wrote:

@sammachin https://github.com/sammachin Works fine (not that I was
expecting anything else).
Looking at the file, it seems like there's almost two seconds ... of
silence https://www.youtube.com/watch?v=4zLfCnGVeL4 after the hello
https://www.youtube.com/watch?v=YQHsXMglC9A itself. I'm thinking,
should we crop it? Not that it really matters, since almost all parts of
the initialization happen before playing this and in threads, so the only
thing it blocks is platform indication of success (blinking green LEDs and
shit), which no one cares about, but if it's there like to keep the audio
output open, so there is no cut/mess after the sound, I believe one second
should be enough. Usually this is needed before the sound, not after it,
before the sound card wakes up, so I'm not sure if this actually makes
sense ... Thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alexa-pi/AlexaPi/pull/193#issuecomment-285603402, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AALc_RM3vvcRmGsEB_Ts5BgzCfvYEmrDks5rkQOHgaJpZM4MXUu5
.

Haha, okay :smile: Merging and I'll do a release during the day.


Please add a license

Without a license, this project isn't open source and no one can use the code.
Please use something like http://choosealicense.com/ to decide what license to use. I recommend MIT or GPL.

A novel effort by Add a License Please

I'm checking the link and I will add license
thanks

Thanks @Ngorror


support for connect with connection string

Add support for connect with connection string and enable it for SQL Server.
Scenario 1: happy path, connect and view it in recent connections list and then select it, observe that the advanced options are also saved.

Scenario 2: provider with connection string option disabled

Scenario 3: provider with connection string enabled but not set as default option

scenario 4: provider with connection string enabled and set as default option

scenario 5: invalid connection string

scenario 6: incorrect information in connection string

scenario 7: use parameters to connect. (existing feature)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 137 (5.84%)  changed or added relevant lines in 5 files are covered.
10 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.04%) to 42.025%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/services/connection/browser/cmsConnectionWidget.ts
0
2
0.0%

src/sql/workbench/services/connection/browser/connectionWidget.ts
3
130
2.31%

Files with Coverage Reduction
New Missed Lines
%

src/sql/workbench/services/connection/browser/cmsConnectionWidget.ts
1
1.2%

src/sql/workbench/services/connection/browser/connectionWidget.ts
9
4.6%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
27928

Relevant Lines:
62138

💛  - Coveralls


add/cleanup documentation on simulation functions

This PR updates the user guide and docstrings to provide more information on simulation functions, addressing issues #890 and #893.
I also added the missing OptimalEstimationProblem and OptimalEstimationResult classes to the documentation.

coverage: 94.623%. remained the same when pulling f0a34dff88f0ea9e43cd54a306ce3f828254a718 on murrayrm:simdocs-04Jun2023 into 184d83d2be20c7a1a34560b76a6eacb812097d0d on python-control:main.

It should be possible to make all of the systems work for a NonlinearIOSystem.
For the impulse response, we could take the linearization of the system at the initial condition and then use that to reset the initial condition, though I'd want to think that through make sure that is the right thing to do mathematically.

@sawyerbfuller,
Here are two comments for your questions.
The terms zero state (forced response) and zero input response (initial condition response) are really only meaningful in a linear system, because, in a linear system, they may be added to get the full response.  If the system is nonlinear, then they make no sense.  So my advice is do not do anything with them in the non-linear domain.
Likewise, there is no reason to handle an impulse response in a nonlinear way, because its importance comes through superposition, where you can use the impulse response to obtain the total response (the convolution integral).  So I don't think you should think about computing it for a nonlinear system.  You will just be leading folk astray in their understanding of linear systems.
Rob

hi @frohro -
These four simulation functions derive their names from the matlab equivalents, initial, impulse, step, and lsim, updated somewhat to have more pythonic or descriptive names, especially for lsim. My sense is that their meaning is widely understood to be what you describe - no input or zero state respectively. Is your sense that they are mis-named in Matlab as well? I don't think I have encountered the precise terminology "zero state" and "zero input", but I see what they mean.
At the very least, there should probably be a warning in initial if the user supplies an input that is ignored.
As for nonlinear simulation, it seems like it would be useful to have functionality available in the library to do the same for nonlinear systems, even if superposition doesn't hold for them. An example that comes to mind is simulating a nonlinear pendulum system from an arbitrary initial state.
Granted, complexities may arise in the extensions of such functions to nonlinear systems because in many cases we will not be able to auto-compute a reasonable time window, but I will assume they can be handled somehow.

Hi @sawyerbfuller,
My beef is with the naming of these simulation functions.  It would be nice if it was as you claim, but it does not match MATLAB.
I'm sorry.  I must not have made my point of view clear.  Let me try again:

control.forced_response() should be named in a way that it is clear it is a general linear simulation, like MATLAB, which calls it lsim().  The term "forced response" of a linear system is also known as the zero-state response, which means X0 = 0.0, so the initial condition option for the user should not be an option in a function called forced_response!  I am unaware of a MATLAB simulation that computes only the forced or zero-state response.  Maybe we should call it lsim().
control.input_output_response() already computes the response in a way that works for linear and nonlinear systems.  Its name is not quite so poorly chosen, though I would prefer the name to emphasize that it works for both linear and nonlinear systems.  In my opinion, we should leave the linear simulation alone, and not add the nonlinear feature to it, because we have a nonlinear one already.  It should perhaps be more clear that it is a more general simulation.
control.initial_response() has a parameter, input, which the docs claim is for compatibility with the other simulations, but it is ignored.  From my perspective, the confusion of having "input" as a parameter that is ignored is worse than any benefit coming from having the same arguments as other simulations.  MATLAB certainly does not do anything like this in its initial function.  They have no problem having functions with different arguments if it cuts the confusion.  When a casual reader sees a parameter in a function, they will assume it is used in the function.  Why would you want two functions to have the same arguments if they perform different tasks, especially when the parameter is ignored in one?  Every function would be harder to use, understand, etc.!  It just makes the user think you have to take into account an input, even if it is zero in a linear system, and that is false.
I use the Python control package as a teaching tool.  I used to use MATLAB/octave, but I'm switching to Python, and I don't want the control package to give my students the wrong ideas.  :-)

I understand that changing function names and parameters are painful, but Python is famous for changing those things, and it can probably be done in a less painful way if careful thought is put into it.  Hopefully, it is more clear what bothers me about these things.  Thanks for thinking about this.
Rob

Dealing with matlab legacy almost daily on scipy, I think lsim is the worst possible name that they can choose when they were dealing with 8.3 fortran name silliness.
forced response is exactly what is happening. Zero state response has nothing to do with it since initial condition is an impulse response super imposed to the input function. So I don't quite get the argument there. Initial condition is a forcing if you let a mass spring damper system from a nonzero condition. It only hides the fact that somebody/something pumped energy to the system prior to t=0. So in that sense I disagree that initial conditions are independent from the forcing signal that is u. I think this should have been the case for matlab but they did poor naming themselves decades ago.
linear or not input output response tells you that there is this input and and there will be an output. I am not a big fan but any other complexity will make things more magical than what it seems. Again, I don't quite get the distinction between linear and nonlinear it's an operator and if you disagree with that signal processing view than you should have bigger problems that control theory treats systems not this naming.
I believe, somewhat strongly that all this is coming from terrible matlab naming schemes and it's not the fault of python-control. Folks should stop treating matlab as the canon instead and move on.

@ilayn,
It seems to me that the initial condition can be caused by an impulse or by a lot of other functions driving it before the simulation is started. I like that we have a set of simulations that are strictly linear.  The great thing about linear systems is you can use superposition and proportionality.  This allows things like convolution, Laplace Transforms, and Fourier series.  You can use ideas from linear algebra, which are so helpful in solving real-world problems.  The EE students here get several courses in linear systems, Circuit Analysis, Linear Systems Analysis, Signals and Systems, and Linear Algebra just as an undergraduate.  If a system isn't linear, we often linearize it to make a solution easier.  So I think understanding linear systems is important, and the terms we use to describe them are also important because they help us all communicate better and more easily.  I am not pushing this language, because MATLAB does, but because it is the language EE's have been using, at least since I was an undergrad 40 years ago.
Rob

Yes I agree linear systems is the key and they play important role but your argument is basically your last sentence, and if I am understanding correctly, just keep things the same don't change them which I cannot participate in sharing your enthusiasm about it. What is it that goes against the grain here? The initial conditions or matlab incompatibility. I don't quite understand the argument here.
The matlab syntax is/was clearly wrong for many decades why should we continue that tradition?

@ilayn, I guess I don't understand why MATLAB was wrong and the present control package is right.  Can you help me?  Maybe it would help to answer my three points, with proofs or citations why they and MATLAB or wrong?  Sorry I'm so dense.  :-)

Just to weigh in a bit:

We have a way to handle compatibility with MATLAB: the control.matlab module implements things in the same way as MATLAB for those who prefer that interface (including lsim).  If there are places in control.matlab that are not compatible with MATLAB, we should fix those.
Some of the problems we are arguing about relate to the history of how these functions came about.  In particular, input_output_response was added when we put in nonlinear models and (at that time) state space system were a separate class (now they are effectively a subclass) => we needed both a method for linear systems (forced_response) and nonlinear systems (input_output_response).
Related to that, part of the reason that forced_response allows the initial condition to be given is that prior to input_output_response, there was no other function that allowed you to specify both an input function and a non-zero initial condition.  I personally agree that lsim is not a great name and that the control module (versus the control.matlab module) should be Pythonic rather that "MATLABic" and the convention we have been using is to write things out (bode_plot, initial_response, etc).

Having said all of that, as we move into 0.10 (see #904), I think we can do the following:

Allow initial_response and step_response to work for either linear or nonlinear systems.
Allow forced_response to take a nonzero X0 as an optional argument, but perhaps document it as mainly being there for the "pure" forced response, which at least one textbook defines as corresponding to X0=0 (FBS, page 6-3).
Use input_output_response as the general function for simulating a linear or nonlinear system for a specified input and initial condition (suggestions welcome for a better name).
Allow impulse_response only for linear systems (mainly because I think it is mainly useful for educational purposes when teaching linear control theory, convolution, etc).

Sorry I'm so dense. :-)
I did not imply anything personal with my remarks, and I don't want the discussion to derail to such direction.

Maybe it would help to answer my three points, with proofs or citations why they and MATLAB or wrong?

If you bother to check matlab's documentation on lsim it accepts both linear and nonlinear models, LPV and identified models of every kind. Thus if we were to be hygenic about linear nonlinear distinction, matlab itself is contradicting its rule. That is what I mean by wrong naming/syntax.
What you probably have in mind for "forced response" is the differential equation solutions; homogeneous solution when you set input term to zero (complementary equation) and non-homogeneous  solutions and so on. That is not the right terminology to use if you call nonhomogeneous a forced response. Because like I mentioned, initial conditions are also forcing terms or say inputs happened before t<0. "Forced response" term for zero RHS is a sloppy usage in that respect.
The common terminology for such things are, that I am sure you know vrey well, typically zero-input response (ZIR) (which is called initial in matlab historically) and Zero state response ZSR which initial condition is zero and only input is driving to denote exactly what you are taking as 0.
This in fact opens up for nonlinear systems since we have concepts like zero dynamics and so on.


PostgreSQL HammerDB Redesign

Bump the version if you want to produce a package, also please update the document when you have the time


"Firebase settings" section to store OAuth access token?

I see the tags of Google, Microsoft & Reddit have this section called "Firebase settings" to store the OAuth access token in Firestore.
Why the Twitter tag doesn't have such section (Firebase settings) and it includes the Oauth token & Oauth token secret directly in the general settings?

Twitter do not use oAuth 2 but different version more info https://developer.x.com/en/docs/twitter-ads-api/making-authenticated-requests


Docker installation on Mac part need to be updated

https://github.com/scrapinghub/splash/blob/master/docs/install.rst
The default and recommended way to install docker on Mac is using 'Docker for Mac' rather than the Docker toolbox.
As mentioned in https://docs.docker.com/docker-for-mac/docker-toolbox/#/the-docker-toolbox-environment. Docker for Mac doesn't need to create the docker-machine.

The Docker for Mac application does not use docker-machine to provision that VM; but rather creates and manages it directly.

The instructions on the doc is confused.
`➜  00 docker-machine create --driver virtualbox default
Running pre-create checks...
Error with pre-create check: "VBoxManage not found. Make sure VirtualBox is installed and VBoxManage is in the path"
➜  00 docker-machine create  default
Usage: docker-machine create [OPTIONS] [arg...]
Create a machine
Description:
Run 'docker-machine create --driver name' to include the create flags for that driver in the help text.
Options:
--driver, -d "none"											Driver to create machine with. [$MACHINE_DRIVER]
--engine-install-url "https://get.docker.com"							Custom URL to use for engine installation [$MACHINE_DOCKER_INSTALL_URL]
--engine-opt [--engine-opt option --engine-opt option]						Specify arbitrary flags to include with the created engine in the form flag=value
--engine-insecure-registry [--engine-insecure-registry option --engine-insecure-registry option]	Specify insecure registries to allow with the created engine
--engine-registry-mirror [--engine-registry-mirror option --engine-registry-mirror option]		Specify registry mirrors to use [$ENGINE_REGISTRY_MIRROR]
--engine-label [--engine-label option --engine-label option]						Specify labels for the created engine
--engine-storage-driver 										Specify a storage driver to use with the engine
--engine-env [--engine-env option --engine-env option]						Specify environment variables to set in the engine
--swarm												Configure Machine to join a Swarm cluster
--swarm-image "swarm:latest"										Specify Docker image to use for Swarm [$MACHINE_SWARM_IMAGE]
--swarm-master											Configure Machine to be a Swarm master
--swarm-discovery 											Discovery service to use with Swarm
--swarm-strategy "spread"										Define a default scheduling strategy for Swarm
--swarm-opt [--swarm-opt option --swarm-opt option]							Define arbitrary flags for Swarm master
--swarm-join-opt [--swarm-join-opt option --swarm-join-opt option]					Define arbitrary flags for Swarm join
--swarm-host "tcp://<IP_ADDRESS>:3376"									ip/socket to listen on for Swarm master
--swarm-addr 											addr to advertise for Swarm (default: detect and use the machine IP)
--swarm-experimental											Enable Swarm experimental features
--tls-san [--tls-san option --tls-san option]`

Instructions are updated in https://github.com/scrapinghub/splash/pull/614.


How to use stubs on mobile development (Android / Swift)?

Hi,
In some scenarios, native developments like Android/IOS could be considered like other Consumers from Contract testing perspective for a REST API.
Using Spring cloud contract is possible to run a Docker image connecting the image with some Stubs published in Artifactory/Nexus but it is necessary to create some Ops operations in order to run/maintain that images.
https://github.com/spring-cloud/spring-cloud-contract/tree/master/docker
Do you know some toolset to run docker images for Android & IOS in the Dev environment?
Many thanks
Juan Antonio

You've pointed to a wrong repo https://github.com/spring-cloud/spring-cloud-contract/tree/master/docker

Upgraded the link

a Docker image connecting the image with some Stubs published in Artifactory/Nexus
Sorry, that doesn't make any sense for me, could you rephrase?

So, what problems do you encounter running stubs for Android/IOS? I am also planning to do the exact same thing but I haven't tried yet. I imagine that mobile devs can just use the docker stub runner to run the stubs during their testing and I don't really see anything else to be done.

@Sam-Kruglov
Yes, I was reviewing the project and documentation maybe in Documentation and in samples area, the project should a section about How to for mobile applications.
Why: En mobile applications, you could consider a native development as another consumer of API stub to guarantee that contracts doesn't change in the time. The lack of JSON schema is not only affected for others Microservices, the mobile development has the same impact. This is the reason.
In an Android, I am going to test with testcontainers-java  in order to load the docker image to load the stubs:
https://github.com/spring-cloud/spring-cloud-contract/tree/master/docker
But in IOS, I am not sure if exist exist something to load images.
In the other hand, the documentation in this area is not extensive:
https://cloud.spring.io/spring-cloud-contract/reference/html/using.html#flows-provider-non-jvm
This is the reason for this issue.
Juan Antonio

Thanks for the insight @jabrena! I have engaged our IOS team to investigate that. We're going to do it next week. I will post here what we find in the end.

Hey, just had a chat with IOS. They decided against WireMock for these reasons:

I have not researched what WireMock does. It’s nothing to do with being good enough. It does not fit into our CI as we are not running unit tests against HTTP server. And I don’t see the need to run contract tests against HTTP stack.
I guess it could be useful for mocking API responses during development, but as I said, different teams/developers already use different tools for that. Some have their own mock server, some use CharlesProxy with local mapping, and iOS used to have something written in NodeJS also.

But they also were familiar with this presentation on Pact and we noticed that Spring Cloud Contract actually has some integration with it. So, that is what we are going to investigate next. The first thing I've read is that using Pact, Spring Cloud Contract can generate the tests and stubs, so should be fine with backend.

I have not researched what WireMock does. It’s nothing to do with being good enough. It does not fit into our CI as we are not running unit tests against HTTP server. And I don’t see the need to run contract tests against HTTP stack.

That's an interesting response since Pact does exactly the same thing AFAIR (but maybe I'm wrong).

Thanks, @marcingrzejszczak, I've stressed your point.

Tomorrow, I will send the proposal for SpringIO including this topic.


Mark project as not Native AoT ready

Describe the bug
With #168 there was bump for AWSSDK.Extensions.NETCore.Setup from 3.7.1 (2021) to 3.7.300 (2023).
In the interim AWSSDK.Extensions.NETCore.Setup has been marked not trimming safe for Native AoT (https://github.com/aws/aws-sdk-net/pull/3044) with .NET 8.
I think that means, Amazon.Extensions.Configuration.SystemsManager should also be marked not trimming friendly for Native AoT now? Since with .NET 7.0 on-wards IIUC libraries need to opt out of trimming support.
Since there are now trimming warnings when including this package in a NativeAoT application + runtime errors.
Expected Behavior
Amazon.Extensions.Configuration.SystemsManager should not be trimmed since it doesn't support trimming
Current Behavior
Build succeeds with trim warnings,
With package 6.1.0+:
    <PackageReference Include="Amazon.Extensions.Configuration.SystemsManager" Version="6.2.0" />

we now get trim warnings on a successful build:
warning IL2104: Assembly 'Amazon.Extensions.Configuration.SystemsManager' produced trim warnings.

here is the same warning with <TrimmerSingleWarn>false</TrimmerSingleWarn>:
ILC : Trim analysis warning IL2026: Amazon.Extensions.Configuration.SystemsManager.Internal.AwsOptionsProvider.GetAwsOptions(IConfigurationBuilder): Using member 'Microsoft.Extensions.Configuration.ConfigurationExtensions.GetAWSOptions(IConfiguration)' which has 'RequiresUnrefer
encedCodeAttribute' can break functionality when trimming application code. The AWSSDK.Extensions.NETCore.Setup package has not been updated to support Native AOT compilations.

There is runtime errors in the application as a result ex:
System.NullReferenceException: Object reference not set to an instance of an object.\n at Amazon.Extensions.NETCore.Setup.ClientFactory.CreateConfig(Type, AWSOptions) + 0x8a\n at Amazon.Extensions.NETCore.Setup.ClientFactory.CreateServiceClient(ILogger, Type, AWSOptions) + 0x8f\n at Amazon.Extensions.NETCore.Setup.AWSOptions.CreateServiceClient[T]() + 0x2c\n at Amazon.Extensions.Configuration.SystemsManager.Internal.SystemsManagerProcessor.<GetParametersByPathAsync>d__6.MoveNex

Reproduction Steps
Sorry, setting a Native AoT project is a little involved, I can cook up a PoC if its really needed.

Include Amazon.Extensions.Configuration.SystemsManager > 6.1.0 in project
Setup project for Native AoT build
Notice null reference exceptions w/ the package

Possible Solution
I believe we can mark the project with:
<IsAotCompatible>false</IsAotCompatible>

https://learn.microsoft.com/en-us/dotnet/core/deploying/native-aot/?tabs=net8plus#aot-compatibility-analyzers
Additional Information/Context
No response
AWS .NET SDK and/or Package version used
Amazon.Extensions.Configuration.SystemsManager 6.1.0+
Targeted .NET Platform
.NET 8
Operating System and version
AmazonLinux, scratch

Needs review with the team.

Did some more testing excluding various assemblies (that are transitive deps from this) from trimming,
Was able to eliminate runtime errors with:
  <ItemGroup>
    <TrimmerRootAssembly Include="AWSSDK.SimpleSystemsManagement" />
  </ItemGroup>

I believe that is the actual culprit here (at least with the runtime error I see).
There appears to already be an issue in the core SDK re: trimming problems that reports this: https://github.com/aws/aws-sdk-net/issues/3173

@boarnoah I just pushed out a PR AWSSDK.Extensions.NETCore.Setup to make it AOT compatible. That is part of in development V4 of the SDK. https://github.com/aws/aws-sdk-net/pull/3353


docs: add within and intersects operators documentation

Adds documentation for within and intersects operators.
Querying - within
In order to do query based on whether points are within a specific area defined in GeoJSON, you can use the within operator.
Example:
const polygon: Point[] = [
  [9.0, 19.0], // bottom-left
  [9.0, 21.0], // top-left
  [11.0, 21.0], // top-right
  [11.0, 19.0], // bottom-right
  [9.0, 19.0], // back to starting point to close the polygon
]

payload.find({
  collection: "points",
  where: {
    point: {
      within: {
        type: 'Polygon',
        coordinates: [polygon],
      },
    },
  },
})

Querying - intersects
In order to do query based on whether points intersect a specific area defined in GeoJSON, you can use the intersects operator.
Example:
const polygon: Point[] = [
  [9.0, 19.0], // bottom-left
  [9.0, 21.0], // top-left
  [11.0, 21.0], // top-right
  [11.0, 19.0], // bottom-right
  [9.0, 19.0], // back to starting point to close the polygon
]

payload.find({
  collection: "points",
  where: {
    point: {
      intersects: {
        type: 'Polygon',
        coordinates: [polygon],
      },
    },
  },
})

Hey @r1tsuu thanks for this 🤜
I know the answer is probably a yes, but presumably these queries are supported in Postgis?

Yes, they are supported in both postgres and mongodb.


"Ignore TLS error" do not recover from backup

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
i created a monitor with tag then created a status page then when check the status page not showing the tag
ticked all the options of status page

ignore TLS error in monitor from backup
When loading a backup from the other website version 1.16.0.beta.0 i ticked ignore TLS Error but when i load it shows i ticked but is showing but when the website is down is showing Client network socket disconnected before secure TLS connection was established

👟 Reproduction steps
Tag Problem in status

I created a monitor
added tag
created a status page
tick everything then save

ignore TLS error in monitor from backup

i created a backup from 1.16.0-beta.0
uploading to 1.16.0

👀 Expected behavior
Tag Problem in status
to show tag in status page
ignore TLS error in monitor from backup
ignore TLS error from my website
😓 Actual Behavior
Tag Problem in status
doesn’t show tag in status page
ignore TLS error in monitor from backup
not Ignoring TLS  error from the website
🐻 Uptime-Kuma Version
1.16.0
💻 Operating System and Arch
Linix
🌐 Browser
Chrome Latest
🐋 Docker Version
No response
🟩 NodeJS Version
Latest
📝 Relevant log output
No response

I can confirm issue about tag

Same about tags, not showing

I cannot reproduce, may need a video or detailed steps,

I cannot reproduce, may need a video or detailed steps,

My laptop so slow. I cant record video, maybe AnyDesk?

Oh, I forgot to update to 1.16.0, now I can reproduce.
1.16.0-beta.0 is ok in this case.

It will be released in the next version.

It will be released in the next version.

you mean that ignore TLS error doesn’t recover from backup

It will be released in the next version.

what about ignore TLS error recover from backup


[stable/prometheus] harmonization of regex references without braces

Signed-off-by: Pierre DEMAGNY<EMAIL_ADDRESS>What this PR does / why we need it:
This PR harmonizes regex references in values.yaml. There were some references with braces, some without. Removed all occurences of reference with braces to avoid breaking external tools like Terraform Template Provider, for example:
data "template_file" "prometheus_tmpl" {
  template = "${file("templates/values-prometheus.yaml.tmpl")}"

  vars {
    PROMETHEUS_INGRESS_HOST   = "${local.workspace["prometheus_ingress_host"]}"
    PROMETHEUS_INGRESS_PATH   = "${local.workspace["prometheus_ingress_path"]}"
  }
}

server:
...
  ingress:
...
    ## Prometheus server Ingress hostnames with optional path
    ## Must be provided if Ingress is enabled
    ##
    hosts:
      - ${PROMETHEUS_INGRESS_HOST}${PROMETHEUS_INGRESS_PATH}
    #   - domain.com/prometheus

Regex references of the form ${n} are interpolated by Terraform's Template Provider, so for example in:
      - job_name: 'kubernetes-nodes'
...
        kubernetes_sd_configs:
          - role: node

        relabel_configs:
          - action: labelmap
            regex: __meta_kubernetes_node_label_(.+)
          - target_label: __address__
            replacement: kubernetes.default.svc:443
          - source_labels: [__meta_kubernetes_node_name]
            regex: (.+)
            target_label: __metrics_path__
            replacement: /api/v1/nodes/${1}/proxy/metrics

My __metrics_path__ end up with /api/v1/nodes/1/proxy/metrics and the 'kubernetes-nodes' job does not work.
When using regex references of the form $n, they are not interpolated and labelling magic works:
      - job_name: 'kubernetes-nodes'
...
        kubernetes_sd_configs:
          - role: node

        relabel_configs:
          - action: labelmap
            regex: __meta_kubernetes_node_label_(.+)
          - target_label: __address__
            replacement: kubernetes.default.svc:443
          - source_labels: [__meta_kubernetes_node_name]
            regex: (.+)
            target_label: __metrics_path__
            replacement: /api/v1/nodes/$1/proxy/metrics

My __metrics_path__ end up with /api/v1/nodes/<node_name>/proxy/metrics where <node_name> is the name of each of my nodes and the 'kubernetes-nodes' job works.
Which issue this PR fixes
There were no open issues for this.
Checklist

[x] DCO signed
[x] Chart Version bumped

/ok-to-test
/lgtm


[Request] Imporove bash's colours.

Is your request related to a specific problem you're having?
I'm always frustrated when I write a bash code block but it does not support the gnu coreutils so some of the commands are not highlighted like grep and cat for example.
it also doesn't recognize executables that i installed like nmap or any other program.
The solution you'd prefer / feature you'd like to see added...
A clear and concise description of how you imagine we might solve this (if you have any ideas).
add it to the built_in of bash.js
for the other executables that are not in the gnu coreutils or sh buildins i don't know a easy fix, you could make everything coloured that starts at the beginning of the line also include ./<executable name> or if it comes after the pipe echo "cGxlYXNlIGZpeAo=" | base64 -d so echo and base64 are yellow because currently base64 is not.
gnu core utils
Any alternative solutions you considered...
I don't have an alternative solution.
Additional context...
./test "3"
echo "test"
cat test.txt | grep "lol"

only test and echo are coloured, im missing the cat and grep.
also it would be nice to see the ./ also getting a colour.

does not support the gnu coreutils

How long would this list be?

you could make everything coloured that starts at the beginning of the line

We could.  Are there no cases where that would lead to false positives?

or if it comes after the pipe

Neat idea, I think the new multi-matcher stuff could help here since we'd want to highlight | as an operator too probably.

How long would this list be?

the gnu core utils would be 104 items as listed on the wikipedia page, there might be more.
list of unix commands are more that might be missing.

We could. Are there no cases where that would lead to false positives?

There is a # which is a comment, there is also assigning variables example: hello='World!'
I am including some images from my .bashrc that might show some weird edge cases or false positives.
https://imgur.com/a/OLhBWO9

So we'd need to exclude:

variable assignment
statements (if, case, etc)

I imagine the rule would be (roughly) something like ^[a-z][a-z0-9]+, so we wouldn't need to worry about # at all...
I'd welcome a PR exploring some of these ideas, perhaps starting with just the GNU core utils list...

my first PR is going better then expected, running into one issue with the tests, i fixed the bash tests to reflect the changes i made but im getting some issues in the dockerfile test and shell test because both have  in the expected output and have a new built_in command in that span.
I counted a total of 3 keywords where only 1 of them should be updated to reflect the changes the other 2 should not be included in the parsing with bash.
for shell:
command-continuation.txt:10
it recognizes in /bin/cat the cat part as a command which it should exclude or capture the whole part maybe.
for dockerfile:
default.expect.txt:10
it recognizes in && apt-get install -y install as a command which it should exclude because its an argument for apt.
default.expect.txt:11
it recognizes in  mkdir /tmp/sessions that mkdir is a command and thus this could be highlighted but i don't know if this is needed.

but im getting some issues in the dockerfile test and shell test

If those embed bash then they would need to be updated also to reflect the new changes to bash.

it recognizes in /bin/cat the cat part as a command which it should exclude or capture the whole part maybe.

Yes, we'd need a mode to capture full paths... likely just a variant of whatever rule we use to capture commands at the beginning...

it recognizes in && apt-get install -y install as a command

There is no way to not recognized keywords, so if the commands aren't unambiguous then we may not be able to use them with the keywords system.

that mkdir is a command

Not sure why this is an issue, it is effectively a command in this case, no?

Would it be enough to change the test so the mkdir is reflecting the new changes and leave the other 2 issues for a future change and make send the pull request of what i have currently?

Just submit what you having passing or failing and let me have a look.


Godot doesn't read FBX exported from Blender correctly

Godot version
3.3.stable
System information
Linux, GLES3
Issue description
Godot has some problems with FBX files.
Steps to reproduce
Please have a look at the Quad Bike model (https://www.dropbox.com/s/63ismbltoa496aq/PolygonQuadBike.zip?dl=1) from here: https://syntystore.com/pages/free-assets
Importing into Godot works good, but when you import it into Blender and then export it again as FBX, the Quad Bike payloads are not correctly transformed.
Minimal reproduction project
See above.

Because autodesk's fbx is not open source and free.So godot as a open source software can't use it's sdk.
Engine developers use Reverse engineering technology to Implement the import FBX function ，so the function may not be perfect.
Just like linux use  Reverse engineering technolog to Implement the import NVIDIA driver function。so it may not be perfect and have some problems and bugs

I know that, but other open source projects are also able to import FBX files correctly. Beside that, Blender is open source and it's FBX exporter too, so it should be possible to fix this problem somehow.

That's expected currently, see #44402.

This is probably fixed with the new FBX importer using FBX2glTF.
You can test it in Godot 4.0 alpha 6: https://godotengine.org/article/dev-snapshot-godot-4-0-alpha-6
This new importer might also be backported for the 3.x branch eventually.
If you still find issues with these models using the new importer, we suggest opening a new issue with details about the problems in Godot 4.0 alpha 6 or later.


Add new sample yaml file for setting up CI/CD with CodeMagic

Why
Original RFC
As of now, we are using CodeMagic to execute CI/CD pipelines on some projects, there is a lack of a sample yaml file for quick setup when initializing a new project (.cicdtemplate)
Solution

Create a new .codemagic folder
Add new sample yaml file codemagic.yml and README.md for instruction

Who Benefits?
Developers
Acceptance Criteria

[ ] Create a new .codemagic folder
[ ] Add new sample yaml file codemagic.yml and README.md for instruction

Sample should include running unit tests, detekt and deploying to Firebase App Distribution (similar to .cicdtemplate/.github/workflows/ folder)

@ryan-conway Currently, our template does not have the Firebase App Distribution deploy script. So we might skip it on Codemagic too. 🙏

@lydiasama There is a Firebase deployment workflow in .cicdtemplate/.github/workflows/ 😉 I suppose the issue description wasn't too clear, will update it too 😇


Implement Nix build for Sovereign Rollups

Background
The goal here is to create a Nix build for Sovereign Rollups, streamlining the build process, enhancing reproducibility and get it usable within our CI.
This is based on sov-rollup-starter crate.
Relevant works is in investigation and in progress here: https://github.com/informalsystems/sov-rollup-starter

Accomplished in the rollup-ibc branch of https://github.com/informalsystems/sov-rollup-starter


Two player instances doesn't work on Samsung or LG TV

Have you read the FAQ and checked for duplicate open issues?
YES
What version of Shaka Player are you using?
2.5.0
Can you reproduce the issue with our latest release version?
YES
Can you reproduce the issue with the latest code from master?
YES
Are you using the demo app or your own custom app?
custom app
If custom app, can you reproduce the issue using our demo app?
Didn't try
What browser and OS are you using?
Samsung Tizen, LG webos - doesn't work
Version 79.0.3945.88 (Official Build) (64-bit) - works
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
Samsung UE43RU7092UXXH, version T-MSLDEUC-1335.0
What are the manifest and license server URIs?
will send it via email
What did you do?
When I run the code from https://github.com/janliska/shaka-issue at browser, it works correctly and I see two player instances playing correctly the stream.
When I run the code on TV, only second player is running. First one is doing nothing. No errors or warnings in console
What did you expect to happen?
I would expect same behavior on TV and PC
What actually happened?
There is only one player streaming video on TV, but there should be two.

Some embedded platforms like smart TVs set restrictions on the number of concurrent video playbacks.  Not all of them allow having two video elements playing at the same time.  So this is probably a platform restriction.

Tizen and WebOS only allow one video concurrent at 2019. If I remember correctly, in the Tizen 5.5 code I have seen something that allows two players, but I have not tried it yet.

If the platform only allows one video element at a time, we can't do anything about that.  Sorry!


The underlying GCP instance doesn't install gpu drivers

Is your feature request related to a problem? Please describe.
We are working on GHA self-hosted runners to provide A100 instances from GCP.
After deploy actions-runner-controller, we find the A100 GCP instances created doesn't have gpu driver installed as described in https://cloud.google.com/container-optimized-os/docs/how-to/run-gpus.
We also need the nvidia driver to be > 500 to perform our tasks, so the stable release won't work.
Also, the runner container created needs to have access to the Nvidia utilities, therefore they need to be run with options
docker run \
  --volume /var/lib/nvidia/lib64:/usr/local/nvidia/lib64 \
  --volume /var/lib/nvidia/bin:/usr/local/nvidia/bin \
  --device /dev/nvidia0:/dev/nvidia0 \
  --device /dev/nvidia-uvm:/dev/nvidia-uvm \
  --device /dev/nvidiactl:/dev/nvidiactl \
  <DOCKER-IMAGE>

Describe the solution you'd like
I would like to the GCP compute instances created with Nvidia gpu driver installed, and the GHA runner container can use Nvidia GPU tools such as nvidia-smi.

Please forgive me to move this to Discussions as this isn't a feature request or bug report to ARC itself


There should be an Async / Parallelized version of PlaylistManager.GetAllPlaylists()

With 7 small playlists, that method takes 100ms to execute - Most of the time is probably spent on JSON parsing, so the best optimization that could be done there is probably parallelizing it :D
With how LoadAllPlaylists() is implemented, it should be easy enough to paralellize it

It is now parallelized


Left layer menu

The new  draggable layer menu now initially appears on the left side.

FYI @ruchimotwaniBC


Find transition point

💥 Proposal
Given a sorted array containing only 0s and 1s, find the transition point.
Example 1:
Input:
N = 5
arr[] = {0,0,0,1,1}
Output: 3
Explanation: index 3 is the transition
point where 1 begins.
Have you read the Contributing Guidelines on Pull Requests?
yes

I have already closed an issue

so please open it and assign me


comment line maybe crlf

My project has the problem that when users copy the sql from windows platform, formatter cannot match the sql of comment，instead of thinking them as the operators.
before

after

so I add the reg \r\n

Coverage remained the same at 100.0% when pulling 0e7e909547c7f78c8d5a9e047b6adbd97e7f9398 on sally2015:master into 15e1188371e684e814e05a250bb1a1c75206f82b on zeroturnaround:master.

but I installed version ^2.3.2 by npm,it still has this problem.and to solve it ,i use an display-none textarea.every time before i try to format the text ,i assign the text to textarea and then get value from it.

It's not really a proper fix, but for now a similar partial fix is included in #112

It's not really a proper fix, but for now a similar partial fix is included in #112

Now part of 3.1.0 release

Now part of 3.1.0 release


Implement security headers in HTTP response

We should add the most important headers recommended on this page to the HTTP response to improve the security of the site:
https://snyk.io/test/website-scanner/?test=210426_AiDc9P_d22dc7cfdf653e3199b8415bcc3c7a26

The following security headers are missing from the website:
HIGH SEVERITY
Strict Transport Security
A HSTS Policy informing the HTTP client how long to cache the HTTPS only policy and whether this applies to subdomains.
Strict Transport Security documentation
LOW SEVERITY
X Content Type Options
The only defined value, "nosniff", prevents Internet Explorer from MIME-sniffing a response away from the declared content-type. This also applies to Google Chrome, when downloading extensions
X Content Type Options documentation
MEDIUM SEVERITY
X Frame Options
Clickjacking protection: deny - no rendering within a frame, sameorigin - no rendering if origin mismatch, allow-from - allow from specified location, allowall - non-standard, allow from any location
X Frame Options documentation
HIGH SEVERITY
Content Security Policy
A computer security standard introduced to prevent cross-site scripting (XSS), clickjacking and other code injection attacks resulting from execution of malicious content in the trusted web page context
Content Security Policy documentation
LOW SEVERITY
X XSS Protection
A Cross-site scripting filter

There is no user generated content on the website. The website does not take user input. These are not particularly relevant, and instead you end up creating a more difficult to maintain software in some cases.
HSTS is just something that should be enabled for all domains, yes, it's super easy to do in most cases, use it with subdomains and preload flags.
X-Content-Type-Options is pointless with no user content
X-Frame-Options is pointless with no user input
Content-Security-Policy is pointless with no user content and it not being a high security system with sensitive information etc. - and this worst of all requires very strict control of the build process and libraries loaded externally and manually listing every allowed source in the CSP for every different type of content or it's doing nothing useful at all, and many 3rd parties do not actually document how to use their scripts with CSP so you need to know it pretty well to be able to use it
X-XSS-Protection is again pointless with no user input


Begin to update documentation to version 2.0.0

I started to update the documentation with the version 2.0.0 of the directive to help people to understand what I understand.
Not sur if it's the best way and if it's finished

$scope.video undefined

Thank you for your contribution, but it got outdated after I updated the gh-pages branch to v3.0.0
See commit b3d08d2


Update AdapterVersion log message

What are you changing in this pull request and why?
Updating the AdapterVersion log message in this section:
https://docs.getdbt.com/docs/dbt-versions/core#how-we-version-adapter-plugins
Resolves #3726
Checklist

[x] Review the Content style guide and About versioning so my content adheres to these guidelines.
[ ] Needs review from product team

@dbeatty10 : folded in your review comment. can i get a re-review? tysm!

thanks for flagging, @dbeatty10 !
added content for the "registered adapter version". not sure if i did it right tho. can you take a look? tysm!


Star certificate generation error

If you want to create a "star" certificate which is valid in Chrome, you have to fill the alternative names.
For example, I would like to create a certificate *.subdomain.domain.intra. In that case, I have to fill :

CN : *.subdomain.domain.intra
Alternative Names

DNS : *.subdomain.domain.intra

If the Alternative name is omitted Chrome produce an error NET::ERR_CERT_COMMON_NAME_INVALID  with a "Subject Alternative Names" missing
After looking at the code, the problem seems to be here :
https://github.com/opnsense/core/blob/eb95e8f53e56e25ce8f0898ee6936d8201bd9a29/src/www/system_certmanager.php#L343
because the is_hostname(...) function does not handle the *. You can see it here
https://github.com/opnsense/core/blob/352428ba5c4b8a03151afe1089331bd452469542/src/etc/inc/util.inc#L565
It seems pfSense fixed it by adding a "wildcard" option. You can see it here
https://github.com/pfsense/pfsense/blob/8be7aff9d53954f4a952789c6bf79cc628ac5746/src/etc/inc/util.inc#L981

@pgaultier can you try https://github.com/opnsense/core/commit/d573662f5e3ecd0c533aecd5662c2150ac1d0b3e ?

@AdSchellevis Thank you !!, applied the patch and it works like a charm.

@pgaultier thanks for confirming!


Large file

Hello,
I've a big mailbox...
Is it possible that you implement a cache function bzw. local mail-save?

I would impletment this by yourself. As the data would be cached on your file system and adding this would add class. Breaking the whole "As simple as one class" idea


ERROR in<EMAIL_ADDRESS>

ERROR in<EMAIL_ADDRESS>4946:12-14
export 'default' (imported as 'Bo') was not found in '@stablelib/random' (possible exports: __esModule, defaultRandomSource, randomBytes, randomString, randomStringForEntropy, randomUint32)
For this issue, I couldn't even build my react project.
What I am using:
Node version: 20.0.0
Thirdweb version: latest (5.48.1)  -also tried in v5.47.1 not working

Hello @developernajib, could you please provide a minimal reproduction of the issue? We haven't seen this issue on our internal testing projects so we'd need the exact setup you're using to isolate the issue.


Positional Parameters use 0-based positioning instead of 1-based positioning in generated markdown files

Steps to reproduce

Create a PowerShell cmdlet with at least two positional parameters

According to about_Functions_Advanced_Parameters, a "position value of '0' represents the first position in a command..."

Issue a command for platyPS to either generate a new Markdown help file or update an existing Markdown help file. (I.e. both New-MarkdownHelp and Update-MarkdownHelp are affected.)

Expected behavior
I expected the YAML generated for parameter metadata would result in 1-based positioning for positional parameters in keeping in line with how Microsoft displays parameter help in its own PowerShell cmdlets.
Actual behavior
The generated parameter YAML metadata is using 0-based positioning for positional parameters. Mind you, I haven't actually generated the Help file yet—so I'm unsure as to whether or not the positions are "adjusted" during that creation process. However, please see below under Environment Data for more information regarding this current behavior.
Environment data
I'm using PSDepend to keep my modules up-to-date with those published to the PowerShell Gallery on every build of my module. The latest installed version on my machine is 0.12.0.
Now, running Get-Module -ListAvailable platyPS reveals that I also have v. 0.11.0 installed. Furthermore, looking at a project I haven't touched in a while, for which I also generated help using platyPS (presumably the 0.11.0 version), the YAML output for positional parameters was 1-based for those generated makdown help files.

Looking at the schema document for v2.0 of the schema, this behavior also seems to contradict the schema which indicates that the generated Position attribute will be {1..n} | Named.

Oh good call! That should be an easy fix. Would you consider sending a pull request for it? I'd be happy to help you navigate the code, if you have any questions.

Sure thing.
Sent from my Windows 10 phone
From: Sergei Vorobev
Sent: Sunday, February 10, 2019 12:49
To: PowerShell/platyPS
Cc: Craig E. Shea; Author
Subject: Re: [PowerShell/platyPS] Positional Parameters use 0-basedpositioning instead of 1-based positioning in generated markdown files (#440)
Oh good call! That should be an easy fix. Would you consider sending a pull request for it? I'd be happy to help you navigate the code, if you have any questions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Lol, I thought this would be "easy". But I must admit, I'm having a hard time finding my way around the codebase. I'll keep looking, but any pointers would be appreciated. 😉

I think I found my way around. 😄

Alright!

If you haven't seen already https://github.com/PowerShell/platyPS/blob/master/CONTRIBUTING.md#contributing-to-platyps has some good starting info about the code layout.

I'm beginning to think this is only an issue if the cmdlet does not have comment-based help when documentation is generated. Not sure, need to do more tests to confirm.

No, platyPs is not handling numbering positional parameters correctly in other instances. I've modified existing tests and added additional assertions which make the tests fail. Now I just need to figure out how to make them pass. 😉

TBD! Love that.

OK, so, I was wrong about platyPS not doing the right thing in other places. I made a change to the code and forgot about it. Anyway, long story short, there is an issue, but I don't think it resides with platyPS, but resides with PowerShell itself.
In test/Pester/PlatyPs.Tests.ps1, there are two contexts under the describe New-MarkdownHelp:

Generated markdown features: comment-based help
Generated markdown features: no comment-based help

Under both of these contexts, I updated the test function parameters as follows:
function Test-PlatyPsFunction {
    param (
        [Switch]$Common,
        [Parameter(Position = 0, ParameterSetName = 'First', HelpMessage = 'First parameter help description')]
        [string]$First
        # Changed parameter type because in the real world, PS wouldn't be able to
        # differentiate the parameter sets and didn't know if maybe this was causing the issue...
        [Parameter(Position = 0, ParameterSetName = 'Second')]
        [int]$Second,
        [Parameter(Position = 1)]
        [int]$Third
    )

    Write-Host "`$Common: $Common"
    Write-Host "`$First:  $First"
    Write-Host "`$Second: $Second"
    Write-Host "`$Third:  $Third"
}

I then modified and added the following It tests for both contexts:
It 'generates markdown with correct parameter set names' {
    ($content | Where-Object {$_ -eq 'Parameter Sets: (All)'} | Measure-Object).Count | Should Be 2
    ($content | Where-Object {$_ -eq 'Parameter Sets: First'} | Measure-Object).Count | Should Be 1
    ($content | Where-Object {$_ -eq 'Parameter Sets: Second'} | Measure-Object).Count | Should Be 1
}

It 'generates markdown with correct parameter positions' {
    ($content | Where-Object {$_ -eq 'Position: 0'} | Measure-Object).Count | Should be 0
    ($content | Where-Object {$_ -eq 'Position: 1'} | Measure-Object).Count | Should Be 2
    ($content | Where-Object {$_ -eq 'Position: 2'} | Measure-Object).Count | Should Be 1
    ($content | Where-Object {$_ -eq 'Position: 3'} | Measure-Object).Count | Should Be 0
}

The tests executed exactly as I thought they would, though, I still didn't know why. They all passed for the context Generated markdown features: comment-based help. But the second It test shown above failed only for the context Generated markdown features: no comment-based help.
By this time, I've stepped through the platyPs code many times and have become familiar with how it's generating PowerShell help. Basically, the tests use Get-Help to generate, in this case, an ExtendedCmdletHelpInfo object, which is then used to generate a Maml document object graph, which is then transformed to markdown. So I decided to start with Get-Help since this is where it all begins. And this is where it gets interesting.
$testFnWithHelp = {
  <#
    .SYNOPSIS
    A function with comment-based help

    .DESCRIPTION
    This function has comment-based help. Its help, however, does not cover all
    of the function parameters. But, it still results in correct auto-generated help
    by PowerShell.

    .PARAMETER Second
    This is the only documented parameter

    .EXAMPLE
    PS C:\> & $testFn 2
    $Common: False
    $First:
    $Second: 2
    $Third:  0

  #>
  param (
    [Switch]$Common,
    [Parameter(Position = 0, ParameterSetName = 'First', HelpMessage = 'First parameter help description')]
    [string]$First,
    [Parameter(Position = 0, ParameterSetName = 'Second')]
    [int]$Second,
    [Parameter(Position = 1)]
    [int]$Third
  )
  Write-Host "`$Common: $Common"
  Write-Host "`$First:  $First"
  Write-Host "`$Second: $Second"
  Write-Host "`$Third:  $Third"
}

$testFnWithNoHelp = {
  param (
    [Switch]$Common,
    [Parameter(Position = 0, ParameterSetName = 'First', HelpMessage = 'First parameter help description')]
    [string]$First,
    [Parameter(Position = 0, ParameterSetName = 'Second')]
    [int]$Second,
    [Parameter(Position = 1)]
    [int]$Third
  )
  Write-Host "`$Common: $Common"
  Write-Host "`$First:  $First"
  Write-Host "`$Second: $Second"
  Write-Host "`$Third:  $Third"
}

New-Item -Path Function:\TestFnWithHelp -Value $testFnWithHelp
New-Item -Path Function:\TestFnWithNoHelp -Value $testFnWithNoHelp

Write-Host "No comment-based help parameter positions:"
(Get-Help -Full TestFnWithNoHelp).Parameters.parameter |
Select-Object -Property Name,Position |
Format-Table

Write-Host "-"*60

Write-Host "Comment-based help parameter positions:"
(Get-Help -Full TestFnWithHelp).Parameters.parameter |
Select-Object -Property Name,Position |
Format-Table

Remove-Item Function:\TestFnWithHelp
Remove-Item Function:\TestFnWithNoHelp
Remove-Variable testFnWithHelp
Remove-Variable testFnWithNoHelp

Running the code above results in the following output:
No comment-based help parameter positions:

name   position
----   --------
Common Named
First  0
Second 0
Third  1

------------------------------------------------------------
Comment-based help parameter positions:

name   position
----   --------
Common named
First  1
Second 1
Third  2

Therefore, this is not a bug in platyPS.

I did two more tests just to fully flesh out my understanding. I was pretty sure I knew what the results would be, but I had to try. Here's the code and output:
$testFnWithNoHelp2 = {
  param (
    [Switch]$Common,
    [Parameter(Position = 1, ParameterSetName = 'First', HelpMessage = 'First parameter help description')]
    [string]$First,
    [Parameter(Position = 1, ParameterSetName = 'Second')]
    [int]$Second,
    [Parameter(Position = 2)]
    [int]$Third
  )
}

$testFnWithHelp2 = {
  <#
  .SYNOPSIS
  This much help is enough to affect the outcome.
  #>

  param (
    [Switch]$Common,
    [Parameter(Position = 1, ParameterSetName = 'First', HelpMessage = 'First parameter help description')]
    [string]$First,
    [Parameter(Position = 1, ParameterSetName = 'Second')]
    [int]$Second,
    [Parameter(Position = 2)]
    [int]$Third
  )
}

New-Item -Path Function:\TestFnWithHelp2 -Value $testFnWithHelp2
New-Item -Path Function:\TestFnWithNoHelp2 -Value $testFnWithNoHelp2

Write-Host "No comment-based help parameter positions:"
(Get-Help -Full TestFnWithNoHelp2).Parameters.parameter |
Select-Object -Property Name,Position |
Format-Table

Write-Host "-"*60

Write-Host "Comment-based help parameter positions:"
(Get-Help -Full TestFnWithHelp2).Parameters.parameter |
Select-Object -Property Name,Position |
Format-Table

Remove-Item Function:\TestFnWithHelp2
Remove-Item Function:\TestFnWithNoHelp2
Remove-Variable testFnWithHelp2
Remove-Variable testFnWithNoHelp2

And here are the results:
Comment-based help parameter positions:

name   position
----   --------
Common Named
First  1
Second 1
Third  2

Comment-based help parameter positions:

name   position
----   --------
Common named
First  2
Second 2
Third  3

So, any comment-based help is enough to affect how the PowerShell help system indexes function parameter positions for help "generated on the fly".

Thank you digging into that! I think it's worth to open an issue in https://github.com/powershell/powershell about this inconsistency.
Sometimes we workaround such problems in PlatyPS, I'm unsure is it worth doing in this case. If you are interested in pushing this forward, I'd be happy to merge a workaround fix.

I tried working around it, but I think you’re right: We’re likely to cause more problems than solve by attempting to work around it.
Sent from my Windows 10 phone
From: Sergei Vorobev
Sent: Saturday, February 23, 2019 12:45
To: PowerShell/platyPS
Cc: Craig E. Shea; Author
Subject: Re: [PowerShell/platyPS] Positional Parameters use 0-basedpositioning instead of 1-based positioning in generated markdown files (#440)
Thank you digging into that! I think it's worth to open an issue in https://github.com/powershell/powershell about this inconsistency.
Sometimes we workaround such problems in PlatyPS, I'm unsure is it worth doing in this case. If you are interested in pushing this forward, I'd be happy to merge a workaround fix.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

In the current version 0.14.1, parameters are still 0-based. This is by design and preferred. Changing the documentation to be 1-based will cause confusion because PowerShell (Get-Command) shows a 0-based value. We should not make the documentation be different.


Support $rand query operator

Refs #258.
$rand operator has been implemented for update command.
For the find command, the $expr operator must first be implemented.

Thank you! can you please add tests if one wants to update something like update := bson.A{bson.D{{"$set", bson.D{{"foo", bson.D{{"bar": bson.D{{"baz", bson.M{"$rand": bson.D{}} }} }} }} } }}

This is the second time I have met the need for deepEqual in a document :smile:

Thank you! can you please add tests if one wants to update something like update := bson.A{bson.D{{"$set", bson.D{{"foo", bson.D{{"bar": bson.D{{"baz", bson.M{"$rand": bson.D{}} }} }} }} } }}

This is the second time I have met the need for deepEqual in a document smile Maybe this time is not exactly the same case as the first one, but something like that might help.
Added the test, it's failing. Working on fixing, current implementation is not flexible.

Use of reflect.Deep is unlikely, preferably write a recursive function.

This ticket was specifically about $rand query operator, not about the aggregation pipeline operator. They are very separate.

This ticket was specifically about $rand query operator, not about the aggregation pipeline operator. They are very separate.

This comment relates to me? Bcs honestly, I do not understand what do you mean. I follow this https://www.mongodb.com/docs/manual/reference/operator/query/rand/

I guess there is no way to implement this issue now.
For queries, $rand can be used only inside $expr, and $expr requires a full aggregation pipeline support that we don't want to implement now (that's item 21 on our roadmap, and we are on item 2 now).
When used in updates, the array there is the same aggregation pipeline.
Sorry for confusion.

No worries. Thanks for answers :+1:


Latest version, if different from the one listed, creates double entry

I noticed that the source code comes with pytest==6.0.2 listed in requirements.txt. However, if I do install pytest again, the latest version is different and we end up having two entries in the requirements.txt. It feels like we should be doing more than comparing the entire string mentioning a dependency, but I don't know the stuff very well so am not 100% sure. 😐

That's a really good catch. We definitely should definitely be taking version number into account when we write the requirements file. Did you want to take this on? If not, no worries, I'll look into it.

No, I want to do this. Or at least give it a good try! 😅 Let me think about this and come back with questions and/or an approach.

Did we ever fix this, or is it still an issue?

Hey @zachtylr21, sorry I couldn't do anything at all these past few weeks. Life hasn't been great for me, is all I can say in brief. 🙂 Please feel free to assign this to someone else; if I can get around to finishing this, I'll let you know (maybe I'll leave a comment on this very thread?). Thanks and apologies again.


Revert "Upgrade ZooKeeper server to 3.5.8 "

Reverts vespa-engine/vespa#13254
In case there is issues with the upgrade. Please review only.
@aressem FYI

Fails with "No route to host" in AWS zones

😭


Update nightly.

Includes this TAIT breaking change. https://github.com/rust-lang/rust/pull/110237

ICE is maybe https://github.com/rust-lang/rust/issues/110206


bot is offline

Hi after following all the steps the bot is offline and is not working how do i rectify this issue

The bot will always be offline, it does not require an active connection to work. It will still be able to send the ban appeals messages.


Tpm support

Result with:
./scripts/ci/qemu-run.sh graphic os

robot -L TRACE -v config:qemu -v rte_ip:<IP_ADDRESS> -v snipeit:no dasharo-security/tpm2-commands.robot
==============================================================================
Tpm2-Commands
==============================================================================

Checking if tpm2-tools is installed...

Package tpm2-tools is installed
TPMCMD001.001 Check if both SHA1 and SHA256 PCRs are enabled (Ubun... | PASS |
------------------------------------------------------------------------------
TPMCMD002.001 PCRREAD Function Verification (Ubuntu 22.04) :: This... | PASS |
------------------------------------------------------------------------------
TPMCMD003.001 PCREXTEND And PCRRESET Functions (Ubuntu 22.04) :: T... | PASS |
------------------------------------------------------------------------------
TPMCMD003.002 PCREXTEND And PCRRESET Functions - locality protecti... | PASS |
------------------------------------------------------------------------------
TPMCMD004.001 PCREVENT Function (Ubuntu 22.04) :: This test aims t... | PASS |
------------------------------------------------------------------------------
TPMCMD005.001 CREATEPRIMARY Function Verification (Ubuntu 22.04) :... | PASS |
------------------------------------------------------------------------------
TPMCMD006.001 NVDEFINE and NVUNDEFINE Functions Verification (Ubun... | PASS |
------------------------------------------------------------------------------
TPMCMD007.001 CREATE Function (Ubuntu 22.04) :: This test aims to ... | PASS |
------------------------------------------------------------------------------
TPMCMD007.002 CREATELOADED Function (Ubuntu 22.04) :: This test ai... | PASS |
------------------------------------------------------------------------------
TPMCMD008.001 Signing the file (Ubuntu 22.04) :: Check whether the... | PASS |
------------------------------------------------------------------------------
TPMCMD009.001 Encryption and Decryption of the file (Ubuntu 22.04)... | SKIP |
TPM doesn't supports TPM2_EncryptDecrypt nor TPM2_EncryptDecrypt2
------------------------------------------------------------------------------
TPMCMD010.001 Hashing the file (Ubuntu 22.04) :: Check whether the... | PASS |
------------------------------------------------------------------------------
TPMCMD011.001 Performing HMAC operation on the file (Ubuntu 22.04)... | PASS |
------------------------------------------------------------------------------
Tpm2-Commands                                                         | PASS |
13 tests, 12 passed, 0 failed, 1 skipped
==============================================================================
Output:  /home/macpijan/projects/github/dasharo/open-source-firmware-validation/output.xml
Log:     /home/macpijan/projects/github/dasharo/open-source-firmware-validation/log.html
Report:  /home/macpijan/projects/github/dasharo/open-source-firmware-validation/report.html

The swtpm pacakge is also available in ubuntu, so it is the easiest one to go for in the CI as well.
The CI with QEMU still starts: https://github.com/Dasharo/open-source-firmware-validation/actions/runs/7221463998/job/19676481422 but we have no firmware-level TPM tests as of now AFAIK, so we do not really check the TPM feature in CI as of know.
The TPM details are visible in setup menu under Device Manager -> TCG2 Configuration, so we could design some tests around that I suppose @TomaszAIR .

@TomaszAIR Can you confirm you are using it, so we can merge it already?

@macpijan yes, the tpm is visible in firmware and OS

I can merge this, but should I do it from cmdline or I can do it using GH GUI?

@macpijan merged into main from terminal


TypeError: Cannot read property 'ne' of null

Why I get this error, make no sense?
TypeError: Cannot read property 'ne' of null
(anonymous function)
src/App.js:23
20 | }, [])
21 |
22 | useEffect(() => {

23 |     console.log(bounds.ne)
| ^  24 |     getPlacesData()
25 |         .then((data) => {
26 |             setPlaces(data);

How did you solve this?


Add SonarCloud to GitHub Actions

Addresses:

https://github.com/stolostron/backlog/issues/10735

Dependency on:

https://github.com/openshift/release/pull/26108

/hold for review and Prow PR

/retest

/retest

/unhold


add a base class without skeletalMesh

Using a skeletalMesh as a root component results in problems when using collisions.
The collision needs to be set on the skeletal mesh, and thus the skeletal mesh needs to be the moving part that pick payloads. But then, the robot position appears to move in /odom
So I added the class ARobotEmptyVehicle which is like ARobotVehicle but without the skeletalMesh component. And the ARobotVehicle class is derived from ARobotEmptyVehicle, adding the skeletal mesh like before.

Thanks for the reasonable changes, since ARobotVehicle is quite a sensitive class due to parenting various BP actor class, some check might be required for io_amr_UE & tb3-UE also I guess.

Cool. Please

check io_amr_UE and turtlebot3-UE build pass
update documentation.

ok, I'm still doing some tests for obayashi_ue but I'll test io_amr_UE and Turtlebot3 also

Compilation and run tests ok for io_amr_UE and turtlebot3-UE


cuQuantum v22.03.0

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Blocked by a fix in the repodata: https://github.com/conda-forge/admin-requests/pull/411#issuecomment-1075827855

Blocked by a fix in the repodata: conda-forge/admin-requests#411 (comment)

The fix is just in (https://github.com/conda-forge/conda-forge-repodata-patches-feedstock/pull/240), I guess we still need to wait for the CDN to sync but let me give it a shot...
@conda-forge-admin, please restart ci

The fix is just in (conda-forge/conda-forge-repodata-patches-feedstock#240), I guess we still need to wait for the CDN to sync but let me give it a shot...

Hopefully fixed by https://github.com/conda-forge/cutensor-feedstock/pull/28 this time... Need to wait 1~2 hrs for it to propagate... 😞

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The build section contained an unexpected subsection name. missing_dso_list is not a valid subsection name.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


⚠ API - Lens has degraded performance

In 47c70a0, API - Lens ($LENS_API_URL) experienced degraded performance:

HTTP code: 503
Response time: 583 ms

Resolved: API - Lens performance has improved in 3e350b3 after 8 minutes.


Zf3 item model

Coverage increased (+4.0%) to 72.626% when pulling f165e54da6da481dcc7a6fc448d615cf4fbc93d5 on zf3-item-model into 047d1835935ecc35ed9b9343f037b21d42533a91 on master.

Coverage increased (+4.0%) to 72.626% when pulling b7c7969d518bc39a7ede7bf4046d82b61e1e8f76 on zf3-item-model into 047d1835935ecc35ed9b9343f037b21d42533a91 on master.


#1623 Fixed equals() and hashcode()

Signed-off-by: Udit Sharma<EMAIL_ADDRESS>GitHub issue resolved: #1623
Briefly describe the changes proposed in this PR:

Fixed equals and hashcode in BindingSetAssignment

@rdf4j-bot run tests

Thanks for this @uditsharma7! The build currently fails with a test failure in the SpinSail - from a glance it looks as if it is the test itself that is the issue here but it will need some eyeballs. I would also like to run this through a full compliance test run before merging as there may be side effects in the query planner that won't show up until we go through full end-to-end testing.

@jeenbroekstra I think Test5 is correct. It is also present here.
In the Unit Test the text query is parsed as
Projection
   ProjectionElemList
      ProjectionElem "country"
      ProjectionElem "code"
   Extension
      ExtensionElem (country)
         Var (name=country)
      ExtensionElem (code)
         Var (name=code)
      BindingSetAssignment ([[country=ex:Germany;code="de"], [country=ex:Norway;code="no"], [country=ex:Endor]])

and the spin query is parsed as
Projection
   ProjectionElemList
      ProjectionElem "country"
      ProjectionElem "code"
   Extension
      ExtensionElem (country)
         Var (name=country)
      ExtensionElem (code)
         Var (name=code)
      BindingSetAssignment ([[country=ex:Germany;code="de"], [country=ex:Norway;code="no"], [country=ex:Endor;code=http://spinrdf.org/sp#undef]])

It seems there is another bug in SpinParser.java due to which it is treating sp:undef as an IRI(http://spinrdf.org/sp#undef) while it should be treated as missing binding.

It seems there is another bug in SpinParser.java due to which it is treating sp:undef as an IRI(http://spinrdf.org/sp#undef) while it should be treated as missing binding.

You're quite right @uditsharma7 . I am running some local tests on your issue branch and will push a fix for this directly to your branch (it's trivial enough that I don't want to bother with a separate bug report).

add to whitelist


Feature: Stalled stream protection

See the upgrade guide for this feature to learn more.
The breaking change is to the MinimumThroughputBody::new method.
Motivation and Context

#1562
Description

https://github.com/awslabs/aws-sdk-rust/discussions/956
Testing

I added several tests
Checklist

[x] I have updated CHANGELOG.next.toml if I made changes to the smithy-rs codegen or runtime crates
[x] I have updated CHANGELOG.next.toml if I made changes to the AWS SDK, generated SDK code, or SDK runtime crates

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I need to fix some broken tests before merging this. Namely, turning off the protection for some old tests.


Initial integration tests for FTP utilities.

Includes:

travis configuration
connection retry logic in IT fixtures for working around a potential bug in the Apache FTP client (TODO consider incorporating into main submission codebase)

Closing, and will re-open with Travis enabled.


Quicktime videos not working in QFG5

Hello,
I tried the wrapper with Quest for Glory 5 and it works great with the exception of Quicktime videos (intro etc.). I'm using gog version, the game runs with Quicktime 3.0 with ddraw acceleration on QT turned on.
With ddraw turned on, the videos start, but there's only audio, no video. If I turn the acceleration off to a pure GDI mode (via Quicktime.cpl control panel that's accompanied with a QT3 install), the videos are shown in windowed/borderless windowed mode, but without scaling.
I suppose ddraw mode would be preferable. Would it be possible to hook the ddraw mode properly with cnc-ddraw? While I'm having this issue in QFG5, this would benefit many other adventure games that use Quicktime.
upload of the gdi quicktime setting -> https://icedrive.net/s/bj96NATGfhhxGGZ5xgQuPhNFuFtT

Not sure yet why it happens, it should actually work fine. I added a workaround for now since I don't think any other game is affected by this bug, so it might be fine:
cnc-ddraw.zip

Thanks, it now works correctly.


ERROR:  invalid byte sequence for encoding "UTF8"

I'm running nodeos v.1.8.5 and meet the error of invalid byte sequence.

History tools work with nodeos v.1.8.1


Supporting beta versions of Packages

Looking through the differences between the existing dataset from swiftpm.co, and the new dataset from this project there are a few differences. One that's important is that swiftpm.co correctly parses semantic versions that have a pre part, more commonly known as beta versions 😂
For example, for one package on swiftpm.co:

and for the same package in this project:

Is there a spec for what the pattern should look like? I.e. is there always a - and is anything after the - allowed?

Right this has got the details and there's a regex we can use:
https://regex101.com/r/Ly7O1x/3/
(and filter down)

As you might imagine, the spec is more complex than you might imagine 😬 https://semver.org
I used this gem originally and then explicitly only take the MAJOR/MINOR/PATCH and PRE parts.
There's another little gotcha that we'll need to cope with. SPM supports semantic versions that start with a v character even though anything starting with a v is not a semantic version. SO I manually cope with that too. I just trim any leading v characters before parsing the semantic version.

There's also this if a regex doesn't work out:
https://github.com/AlexanderNey/SemanticVersioning
Seems like it's well maintained, and it supports SPM.


DSCP Tos

This tool is very helpful. How to enable DSCP or TOS mark?

Any progress on this request?

@thiezn I have code for adding TOS support if you will consider a pull request?

@eyehearttech Would it be possible for you to make a pull request or make your code available as a fork? Thank you!


fix Recycle progress before removing bulk load job

Proposed changes
Issue Number: close #xxx

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

run buildall

run buildall


Deprecated template warning when used in NG4

Template parse warnings:
The
  [WARNING ->]<template [ngIf]="tree.isNodeExpanded()">
    <tree-internal *ngFor="let child of tree.childrenAs"): ng:///TreeModule/TreeInternalComponent.html@25:6

Is there a way to use or add custom icons with tree nodes and leaves ?
Also is there way to show tree lines and branches and selectable check boxes instead of expanding triangle nodes , similar to jsTree ?

@pdhar-tibco

Is there a way to use or add custom icons with tree nodes and leaves ?

There is a PR for this - so this will be available in a future version.

Also is there way to show tree lines and branches and selectable check boxes instead of expanding triangle nodes , similar to jsTree ?

No this is not available yet

The element is deprecated. Use instead ("node-menu *ngIf="isMenuVisible" (menuItemSelected)="onMenuItemSelected($event)">

Thanks for this!

@pdhar-tibco Fixed


Implement 1ES PT for production pipelines

Add 1ES PT to the production pipelines. These were pretty straightforward.
Test runs:

release-go-start: https://dev.azure.com/dnceng/internal/_build/results?buildId=2405183&view=results

Filed https://github.com/dagood/go-release-test/issues/67 (private repo) and updated it correctly.
This kicked off release-build: https://dev.azure.com/dnceng/internal/_build/results?buildId=2405185&view=results, which (correctly) waited around because the version I gave it (1.24.0) doesn't exist.

release-build https://dev.azure.com/dnceng/internal/_build/results?buildId=2405187&view=results

Failed to publish new aka.ms links for 1.21.8-1 with a dev prefix. This is almost definitely due to secret rotation, not this PR. https://github.com/microsoft/go-lab/issues/61

release-go-images: https://dev.azure.com/dnceng/internal/_build/results?buildId=2405186&view=results

Correctly waited around for my private testing repo to get a PR to update to 1.24.0-1, which never happened.

These tests aren't all that extensive, but given what I expect about the kind of operations that 1ES PT affects, I'd be satisfied merging this and working on the aka.ms link issue afterwards.
The aka.ms link generation test does cover some good ground. It demonstrates successfully downloading the build asset JSON from an official build, which was something I was concerned might be limited in some way or need more work to migrate.
I noticed that https://github.com/microsoft/go/pull/1156 is missing the BuildAssets build asset JSON pipeline artifact, so I didn't run tests against that yet. I probably missed it during the switch, because artifact publishing had to change for 1ES PT.
Reviewing with "ignore whitespace" highly recommended.

New test builds:

https://dev.azure.com/dnceng/internal/_build/results?buildId=2405745&view=results
https://dev.azure.com/dnceng/internal/_build/results?buildId=2405736&view=results
https://dev.azure.com/dnceng/internal/_build/results?buildId=2405746&view=results

Failed to publish new aka.ms links

This worked in my new test build! I see that .NET Engineering fixed it this morning.


has no initializer and is not definitely assigned in the constructor

i want to know why ?

The error is that you haven't initialized name.
It looks like this is a question rather than a bug report. This issue tracker is for tracking bugs and active work on TypeScript itself, rather than a general forum for programmers using TypeScript to get help or ask questions.
You can ask questions on sites like Stack Overflow. We are not able to provide one-on-one support on the issue tracker. Please read the issue template carefully - it has important information on what kinds of reports can be acted on here, as well as links to useful TypeScript resources. Thanks!

The error is that you haven't initialized name.
It looks like this is a question rather than a bug report. This issue tracker is for tracking bugs and active work on TypeScript itself, rather than a general forum for programmers using TypeScript to get help or ask questions.
You can ask questions on sites like Stack Overflow. We are not able to provide one-on-one support on the issue tracker. Please read the issue template carefully - it has important information on what kinds of reports can be acted on here, as well as links to useful TypeScript resources. Thanks!

OK，thks


How could it be a lock-free ring buffer?

There is no atomic variables, member barriers. How could it be a lock-free ring buffer?

Yes, there are some additional assumptions about the "atomicity" of the RingBufCtr type, which need to be spelled out explicitly. The ring_buf.h header file has been updated with a comment that describes these additional assumptions. Here is the comment:
/*
* ASSUMPTION:
* The following RingBufCtr type is assumed to be "atomic" in a target CPU,
* meaning that the CPU needs to write the whole RingBufCtr in one machine
* instruction. An example of violating this assumption would be an 8-bit
* CPU, which writes uint16_t (2-bytes) in 2 machine instructions. For such
* 8-bit CPU, the maximum size of RingBufCtr would be uint8_t (1-byte).
*
* Another case of violating the "atomic" writes to RingBufCtr type would
* be misalignment of a RingBufCtr variable in memory, which could cause
* the compiler to generate multiple instructions to write a RingBufCtr value.
* Therefore, it is further assumed that all RingBufCtr variables in the
* following RingBuf struct *are* correctly aligned for "atomic" access.
* In practice, most of C compilers should provide such natural alignment
* (by inserting some padding into the struct, if necessary).
*/

--MMS


Add non binding UDP sink support

Signed-off-by: Heinz N. Gies<EMAIL_ADDRESS>Pull request
Description
Introduce the bound option to the UDP sink which can be set to false to let the sink re-acquire the connection to a target for every package
Related

Related Issues: fixes #900
Related docs PR

Checklist

[x] The RFC, if required, has been submitted and approved
[x] Any user-facing impact of the changes is reflected in docs.tremor.rs
[x] The code is tested
[x] Use of unsafe code is reasoned about in a comment
[x] Update CHANGELOG.md appropriately, recording any changes, bug fixes, or other observable changes in behavior
[x] The performance impact of the change is measured (see below)

Performance
only sink changes

Coverage remained the same at 83.185% when pulling d1294ec4851724e832d702e1f8581d0c7b35953c on udp-allow-round-robin into c50eb02818e2ce23ae8d32fc87a8890c67e3d869 on main.


⚠ Fortnite has degraded performance

In c8903f8, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1533 ms

Resolved: Fortnite performance has improved in 0bb2dae.


Proof of concept provenance reporting task

@mattyb149 This is the refactoring I made to allow reuse of the same properties in my ReportingTask as in the PutElasticSearch processor.
The code most definitely isn't suitable for merging and there is still a lot of copy/paste in AbstractElasticSearchReportingTask, but I thought seeing the sort of changes I'd had to make may inform your thoughts.

Closing this PR for now.
The refactor to use a ControllerService doesn't look too bad so I'll look at implementing that once this code has made it through to master.


speakingurl TypeScript type definitions

Hello,
You might want to add the speakingurl (https://github.com/pid/speakingurl) type definition to your repository. The project is pretty popular but the definitions I did are not very complex so maybe they are not that useful.
Btw thank a lot for this great repository, it saved me hundreds of hours of searching and work :)

I did a cleaner version with interfaces

closed because I noticed that my definitions are wrong, will recommit when ready


Remove default fallback on page load?

I noticed that even if I don't specify a fallback as a config parameter for withRelay, there's still a default fallback that is showing which is causing a brief flash when I navigate between pages.
I noticed this also on the example in the repo.
Is there a config parameter or recommendation to avoid this?

This should now be possible with React 18 (which is still in beta). It doesn't seem to require any changes to this library, simply having your own suspenses at the correct places means you can render a skeleton page before the data is loaded. If your query only includes fragments, it's enough to have suspense wrapping all the components that call useFragment.
Unless I'm missing something, the only issue preventing this earlier was that suspense wasn't supported server side in React 17.


Send all button to populate amount with all available funds

Fixes #74.

🎉

👍

Hi can you merge with last version, since i merged a PR with a fix on amount.

Merged and updated.

+5


WS Issues

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
I setup Uptime Kuma with the .sh installer script and chose the local version. When I go to the site, it says Could not connect to socket server. So I checked browser console and it says

What do I do?
My Apache Config:
<VirtualHost IP REDACTED:80>
        ServerName status.pyrohost.cloud
        ServerAdmin<EMAIL_ADDRESS>        ServerAlias www.status.pyrohost.cloud
        Redirect permanent / https://status.pyrohost.cloud/

ProxyPass / http://localhost:3001/
  RewriteEngine on
  RewriteCond %{HTTP:Upgrade} websocket [NC]
  RewriteCond %{HTTP:Connection} upgrade [NC]
  RewriteRule ^/?(.*) "ws://localhost:3001/$1" [P,L]
</VirtualHost>

<VirtualHost IP REDACTED:443>
        ServerName status.pyrohost.cloud
        ServerAdmin<EMAIL_ADDRESS>        ServerAlias www.status.pyrohost.cloud
        SSLEngine on
        SSLCertificateFile /etc/apache2/sites-ssl/pyrohost.cloud.crt
        SSLCertificateKeyFile /etc/apache2/sites-ssl/pyrohost.cloud.key

Protocols h2 http/1.1

  ProxyPass / http://localhost:3001/
  RewriteEngine on
  RewriteCond %{HTTP:Upgrade} websocket [NC]
  RewriteCond %{HTTP:Connection} upgrade [NC]
  RewriteRule ^/?(.*) "ws://localhost:3001/$1" [P,L]
</VirtualHost>

🐻 Uptime-Kuma Version
1.12.1
💻 Operating System and Arch
Ubuntu 20.04
🌐 Browser
Safari
🐋 Docker Version
No response
🟩 NodeJS Version
14.19.0

Please read wiki:
https://github.com/louislam/uptime-kuma/wiki/Reverse-Proxy

suggest sudo a2enmod proxy_wstunnel is added to the wiki page under the apache section. I also had a similar problem.


Warnings not directly created in init() are resolved on LXD start (even if triggered during startup)

Warnings that are triggered during LXD start (such as network failures) are immediately marked as resolved as LXD finishes starting up due to https://github.com/lxc/lxd/blob/0754c9fe9d35e7b76a618263e0cc15682525be3b/lxd/daemon.go#L1322-L1342
This means that these warnings that are triggered during LXD start but are not directly created by the init() function are never visible to the user.
Perhaps we should switch to resolving any warning with a date that is older than the LXD daemon start up time, allowing ones created after the start time to remain.

Perhaps we should switch to resolving any warning with a date that is older than the LXD daemon start up time, allowing ones created after the start time to remain.

@tomponline That won't work because startTime is set at the end of init (in the Ready function) which would cause all warnings to be resolved.

Could we not set it earlier or take another reading earlier?

Could we not set it earlier or take another reading earlier for use with resolving older warnings?

We could set it earlier, e.g. in Init() (which calls init()). Looks like startTime is only used for image updates.


In some cases I got 2 Left hands (or Right)

I use mediapipe version 0.8.11 and Hands detection with settings:
hands = mpHands.Hands(
                static_image_mode= False,
                max_num_hands = 2,
                min_detection_confidence= 0.8,
                min_tracking_confidence = 0.8
        )

.....
    if results.multi_hand_landmarks:
        for handType, handLms in zip(results.multi_handedness, results.multi_hand_landmarks):
            mpDraw.draw_landmarks(img, handLms, mpHands.HAND_CONNECTIONS)
            print(handType)
        hand1_new = results.multi_hand_landmarks[0].landmark
        if len(results.multi_hand_landmarks) == 2:
            hand2_new = results.multi_hand_landmarks[1].landmark

Python, Windows, JupiterNotebook.
Describe the current behavior:
In some cases (when hand is visible as fist) I have 2 Left hands with good probabilities.
With Right hand it's also possible.
Describe the expected behavior:
I guess it should be only one Left hand if it's the same coords.

Hi @PavelAgurov,
Please allow us some time to share the any possible solution to the observation mentioned.

Hi @PavelAgurov,
Please allow us some time to share the any possible solution to the observation mentioned.

I can provide example(s) of input if needed.

@PavelAgurov,
Thanks for the reply!! Providing the example will be more helpful to understand the issue better from our end.

Please find data that was recognized as 2 Left hands with the same coords:
[landmark {
  x: 0.2769083
  y: 0.76771235
  z: -3.239203e-07
}
landmark {
  x: 0.2533634
  y: 0.802271
  z: -0.050840996
}
landmark {
  x: 0.23432064
  y: 0.8256263
  z: -0.09188586
}
landmark {
  x: 0.23488261
  y: 0.8416432
  z: -0.124252416
}
landmark {
  x: 0.23797552
  y: 0.84089196
  z: -0.15588646
}
landmark {
  x: 0.26496348
  y: 0.75405574
  z: -0.113334954
}
landmark {
  x: 0.25849894
  y: 0.75561446
  z: -0.1566533
}
landmark {
  x: 0.23374778
  y: 0.7916034
  z: -0.17892459
}
landmark {
  x: 0.20937555
  y: 0.8204909
  z: -0.19189456
}
landmark {
  x: 0.27643466
  y: 0.7087383
  z: -0.10346897
}
landmark {
  x: 0.24764231
  y: 0.695384
  z: -0.14117454
}
landmark {
  x: 0.20555642
  y: 0.71918327
  z: -0.1518512
}
landmark {
  x: 0.17359903
  y: 0.7420903
  z: -0.16102219
}
landmark {
  x: 0.2775539
  y: 0.6780304
  z: -0.09494146
}
landmark {
  x: 0.24329485
  y: 0.66316193
  z: -0.12708178
}
landmark {
  x: 0.19884942
  y: 0.6780439
  z: -0.13595977
}
landmark {
  x: 0.16818902
  y: 0.69536126
  z: -0.14275958
}
landmark {
  x: 0.27434954
  y: 0.6598588
  z: -0.0889524
}
landmark {
  x: 0.23745586
  y: 0.6539933
  z: -0.114237
}
landmark {
  x: 0.19944549
  y: 0.6640084
  z: -0.123690054
}
landmark {
  x: 0.17279452
  y: 0.67714
  z: -0.13072725
}
, landmark {
  x: 0.1782154
  y: 0.93768847
  z: -4.0703956e-08
}
landmark {
  x: 0.23144819
  y: 0.8727762
  z: -0.0033394776
}
landmark {
  x: 0.2520983
  y: 0.8018973
  z: -0.017812446
}
landmark {
  x: 0.2684797
  y: 0.7497646
  z: -0.034932632
}
landmark {
  x: 0.28295743
  y: 0.7001501
  z: -0.047864318
}
landmark {
  x: 0.16477507
  y: 0.6853812
  z: -0.024858348
}
landmark {
  x: 0.26766855
  y: 0.6634499
  z: -0.058381677
}
landmark {
  x: 0.29581043
  y: 0.7405415
  z: -0.07409343
}
landmark {
  x: 0.28071713
  y: 0.79440665
  z: -0.07967225
}
landmark {
  x: 0.14938074
  y: 0.6872054
  z: -0.036880527
}
landmark {
  x: 0.2723422
  y: 0.67256826
  z: -0.06744876
}
landmark {
  x: 0.29827994
  y: 0.75906986
  z: -0.0688904
}
landmark {
  x: 0.27903694
  y: 0.80650365
  z: -0.06530807
}
landmark {
  x: 0.14578941
  y: 0.71389145
  z: -0.05016167
}
landmark {
  x: 0.26555118
  y: 0.6935424
  z: -0.08037126
}
landmark {
  x: 0.2860596
  y: 0.779207
  z: -0.06723591
}
landmark {
  x: 0.26587772
  y: 0.8275005
  z: -0.052122686
}
landmark {
  x: 0.14938967
  y: 0.75642943
  z: -0.06341711
}
landmark {
  x: 0.24713139
  y: 0.7471509
  z: -0.08149708
}
landmark {
  x: 0.26482272
  y: 0.8056644
  z: -0.06949914
}
landmark {
  x: 0.24952382
  y: 0.8439414
  z: -0.05640859
}
]

Hi @PavelAgurov,
We need your help to debug the issue more this is a stray hand detection from some previous frame?
Do you have insight about where the additional left hand comes from?

I think it's issue of model - in some cases model decided that it's 2 hands instead of one.
I can try to check source code of library and check if I can trace it.
From other side I think if you will get coords and run model with it you can receive 2 hands.

@PavelAgurov,
Thanks for reaching us back. If any observation you can find out that will really helpful for us to fix this issue and improve the solution for the other community members. Meanwhile, We are assigning this issue to right owner. Thanks!!

Not sure if issue above is related to my.
Sorry, I will try to find more details this week, was very busy.

Hi, sorry for long delay. So, please find short test and test image:
import cv2
import mediapipe as mp

file= "hand_2.jpg"
imageFromFile = cv2.imread(file)

mpHands = mp.solutions.hands
hands = mpHands.Hands(
                static_image_mode= False,
                max_num_hands = 2,
                min_detection_confidence= 0.8,
                min_tracking_confidence = 0.8
        )

for i in range(100):
    # pre-processing
    img = imageFromFile
    img = cv2.flip(img, 1)
    imgRGB = cv2.cvtColor(img, cv2.COLOR_BGR2RGB)

    # recognition
    results = hands.process(imgRGB)

    # result
    if results.multi_hand_landmarks:

        if len(results.multi_hand_landmarks) == 2:
            print('2 hands')
            print(results.multi_handedness[0], results.multi_handedness[1])
        else:
            print('1 hand')
            print(results.multi_handedness[0])

Output:
1 hand
classification {
  index: 1
  score: 0.9981314
  label: "Right"
}

2 hands
classification {
  index: 1
  score: 0.9894802
  label: "Right"
}
 classification {
  index: 1
  score: 0.99468213
  label: "Right"
}

1 hand
classification {
  index: 1
  score: 0.9986132
  label: "Right"
}

1 hand
classification {
  index: 1
  score: 0.9985285
  label: "Right"
}

1 hand
classification {
  index: 1
  score: 0.9981314
  label: "Right"
}

2 hands
classification {
  index: 1
  score: 0.9894802
  label: "Right"
}
 classification {
  index: 1
  score: 0.99468213
  label: "Right"
}

1 hand
classification {
  index: 1
  score: 0.9986132
  label: "Right"
}

1 hand
classification {
  index: 1
  score: 0.9985285
  label: "Right"
}

1 hand
classification {
  index: 1
  score: 0.9981314
  label: "Right"
}

2 hands
classification {
  index: 1
  score: 0.9894802
  label: "Right"
}
 classification {
  index: 1
  score: 0.99468213
  label: "Right"
}

As you can see the same image can be recognized as 2 hands or only one.
Image example attached.

Please note - if I add coordinates output I will have very close values:
print(results.multi_hand_landmarks[0].landmark[0], results.multi_hand_landmarks[1].landmark[0])
hand1:
x: 0.81331605
y: 0.7915341
z: -5.4251996e-07

hand2:
x: 0.8017197
y: 0.7396451
z: -2.7306643e-07

Hello @PavelAgurov,
We are upgrading the MediaPipe Legacy Solutions to new MediaPipe solutions However, the libraries, documentation, and source code for all the MediapPipe Legacy Solutions will continue to be available in our GitHub repository and through library distribution services, such as Maven and NPM.
You can continue to use those legacy solutions in your applications if you choose. Though, we would request you to check new MediaPipe solutions which can help you more easily build and customize ML solutions for your applications. These new solutions will provide a superset of capabilities available in the legacy solutions. Thank you

Thanks, of course I will check new one. But my problem was not to easily build solution, but have output without bug :)

@PavelAgurov,
Please explore our new upgraded hand solutions and if you encounter the similar behaviour, feel free raise a new issue with complete details to reproduce the issue from our end. Further, We need your confirmation to move the status to close if now this is no longer issue from your end. Thank you