Pluggable components injection

Description
This PR adds support for automatically injecting pluggable components based upon the declared components and its annotations. To see it working:

Install dapr

helm upgrade --install dapr dapr/dapr \
--namespace dapr-system \
--create-namespace \
--set global.registry=ghcr.io/mcandeia \
--set global.tag=1.10.0-components-injection-linux-amd64 \
--wait

Apply the following yaml

apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: jsonlogic
  annotations:
    dapr.io/component-container-image: "ghcr.io/mcandeia/dapr-components:latest"
    dapr.io/component-container-env: "NAME=VALUE"
    dapr.io/component-container-volume-mounts-rw: "mymount:/tmp"
spec:
  type: bindings.jsonlogic
  version: v1
  metadata: []
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: app
  labels:
    app: app
spec:
  replicas: 1
  selector:
    matchLabels:
      app: app
  template:
    metadata:
      labels:
        app: app
      annotations:
        dapr.io/enabled: "true" # required
        dapr.io/app-id: "my-app"
        dapr.io/inject-pluggable-components: "true"
    spec:
      containers:
        - name: "httpbin"
          image: "kennethreitz/httpbin"

Notice that three containers were created within the same pod.
Issue reference

Please reference the issue this PR will close: #5492
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Unit tests passing
[x] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/ok-to-test

/ok-to-test

@yaron2 @artursouza @tmacam @philliphoff

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

Do not merge until proposal is accepted https://github.com/dapr/proposals/pull/9.

Please, fix lint.


math:sin, cos, sincos: Inconsistencies between documentation and implementation

According to Questions and functions' docs, there's a clearly defined input value range of 0:2Pi. Since pal is meant to be super fast, one should not check nor normalize input which is out of the range (GIGO rule). My questions arises:

Why there is a M_NORMALIZE_RADIANS macro used?

is it even working properly? (see #118)
is it whatsoever necessary in pal?

Why test cases for these functions are using values from range -Pi:Pi?

I would argue this is a case where the documentation should be changed, and M_NORMALIZE_RADIANS should be cleaned/fixed. Having a function called sin(or cos) that has a different input range than the normal mathematical function is just asking for trouble. I violates the "principle of least surprise" and is likely to lead to problems for users of the library.
Having a faster version of sin(or cos) that has a distinct name to be used in those cases where the input is known to be constrained might be a good idea, If we did have such a function I think either +/-Pi or +/-2Pi would be more reasonable restrictions than 0:2Pi. Off the top of my head, here are some possible names:

fastsin
sin_restricted
sin_PiOrLess
sin_???
sin_profit

P.S.
Should M_NORMALIZE_RADIANS a part of the public API of PAL? Currently it lives in one of the public headers and as a result could be used by users of PAL. I suspect M_NORMALIZE_RADIANS is something that should be internal. If you look at the implementations of trig function in the msun code it takes a different approach to "normalizing." We probably we want the flexibility to tweak this macro in the future for performance reasons without risking breaking user code.

Given the whole idea of the library, wouldn't it make sense to define a range---be it [0,2pi) or [-pi,pi)---and leave the generation/normalization of the range to the user. In other words, just p_sin and p_cos be the "fast" functions. That would seem to be more in line with the "no belt/suspender" and GIGO ideas. Specially given the benchmarks @kfitch reported in #117. Otherwise, +1 for making M_NORMALIZE_RADIANS internal.

@kfitch Yes, agree that we may be breaking one principle here for the sake of another. There are many embedded libraries that have used this before. I will add links to them in the README.md to clarify. We seem to be having a culture clash here between resource embedded and desktop/server developer viewpoints?:-)

I agree that the periodic trigonometric functions should assume their argument is in the correct range. But in this case it would be nice to provide an fmod function which developers can use to ensure that it is.
For example:
p_fmod_f32(x, n, 2.f * M_PI); // optional
p_cos_f32(x, y, n);

@eliteraspberries That is definitely one reasonable approach, but it brings up a few more questions:
For the case where the library user does not know for sure the range of his inputs and would use the example code you presented, could we do better performance wise? If the "fmod" operation was inside the cos function then we would have a chance to have better cache usage (or even avoid the cache by using registers). Also, the compiler might have more wiggle room to schedule things when unrolling loops and looping to fill in potential pipeline holes. And finally there are potential efficiencies to be had when combining the fmod with the cos (or other trig function). Look at the implementation under the msun directory. They actually wrap into +/-pi (instead of +/-2pi), and keep track of the multiple of pi needed. This lets them use various symmetries when evaluating the function. By limiting to a range closer to 0 it might be possible to use fewer terms in the Taylor series expansion approximation, which again might improve performance. Of course, the proof is in the pudding. The only way to know how much impact these potential optimizations would have would be to try it out. And, probably try it out on a few different vector lengths on a few different platforms.
For the case where the library user does know the input range to be limited to +/- 2Pi we can be very fast.
I would argue that having two versions of each trig function would be the ideal case; one with the full input range, and another limited to +/- 2Pi. This lets the library user express their intent/expectations clearly. Of course a larger API means a larger maintenance burden.

@kfitch That sounds like it would please everyone, with a little more work.
In that case, I would humbly suggest to implement a static inline version of the fmod function in a header file, so it could be used inline by several functions. So the compiler does all the work.
Something like this for example:
fmod.h:
static inline void _fmod(const float *a, float *c, size_t n, float x)
{
    size_t i;
    for (i = 0; i < n; i++) {
        ...
    }
}
static inline void _fmod_2pi(const float *a, float *c, size_t n)
{
    _fmod(a, c, n, 2.f * M_PI);
}
void fmod(const float *, float *, size_t, float);
void fmod_2pi(const float *, float *, size_t);

fmod.c:
#include "fmod.h"
void fmod(const float *a, float *c, size_t n, float x)
{
    _fmod(a, c, n, x);
}
void fmod_2pi(const float *a, float *c, size_t n)
{
    _fmod_2pi(a, c, n);
}

cos.c:
#include "fmod.h"
static inline _cos(const float *a, float *c, size_t n)
{
    size_t i;
    for (i = 0; i < n; i++) {
        ...
    }
}
void cos(const float *a, float *c, size_t n)
{
    _cos(a, c, n);
}
void cos_norm(const float *a, float *c, size_t n)
{
    _fmod_2pi(a, c, n);
    _cos(a, c, n);
}

The compiler would inline _fmod and _cos into the cos_norm function, perhaps combine the two for loops, and do whatever instruction scheduling.
The library would end up being slightly bigger though. If everyone likes the idea, I can work on it tomorrow. In the end, I'm happy with whatever the project decides.

@eliteraspberries :+1: LGTM

Is implementing the fmods in a header file a good idea (it's effectively what's happening)? Also, if you're providing cos_norm, do you need a public fmod_2pi (@kfitch raised that issue earlier)? Some implementations of trigonometric functions might use different ranges in the future.

I think there are two separate issues here. First is the unit testing data. It should have input which is bound to [0, 2pi]. That's easy to fix, just generate new data (gold).
The second is that the implementations may not properly handle input on [0, 2pi] -- perhaps only on [-pi, pi] or [0, pi/2]. This issue has to be handled by the individual function implementation. For example the tangent implementation uses a core algorithm for input in [0, pi/4] and then uses identities and symmetries to extend that to [0, 2pi].


Local images not rendering

Link to notebook:
https://github.com/jordwil/ds_blog/blob/master/_notebooks/2020-07-30-int-project-gold.ipynb
Corresponding blog post will not display any local image that I provide a path to with the  notation.

Please follow the troublehsooting guide carefully to try to debug.
I was not able to reproduce your error.  My site: https://hamelsmu.github.io/image-test/machine learning/python/eda/cleaning/multi-target/regression/2020/07/30/int-project-gold.html
and minimal repo with your same files for reference: https://github.com/hamelsmu/image-test


Converting raw bytes to offset fails

Having function or ascii pointers (vtable) outside of the analysed area (raw bytes) and pressing 'o' will not convert the bytes to a pointer to a function or ascii. Pressing '4' will just change the type to int_t but also doesn't convert.

Expected result would be to press 'o' and it would be converted to a pointer to a string (ascii@08019420).

This only happens in the latest demo version. Personal version however seems to have the correct behaviour.

Do you know which versions the demo was and the personal version were that you were testing with?

We're going to assume this issue is fix, as we've not heard back.


In WSLg, Alt + F4 doesn't work

Windows Build Number
Microsoft Windows [Version 10.0.21376.1]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
Kernel version: 5.10.16
Distro Version
Kali 2021.1
Other Software
gedit
Repro Steps
(If you doesn't installed gedit)
sudo apt install gedit
gedit
Press Alt + F4 in gedit WSLg window
Expected Behavior
Nothing to insert here
Actual Behavior
Nothing to insert here
Diagnostic Logs
No response

This is pretty annoying since the combination is deeply ingrained in muscle memory.


fix stats-internal with option -m cause active ledger recover then close

Fixes #16661
Motivation
fix stats-internal with option -m can cause active ledger recover then close
Modifications
ManagedLedgerImpl.getLedgerMetadata doesn't call getLedgerHandle from ledgerCache directly, it will use ManagedLedgerImpl.currentLedger if the ledger id is ManagedLedgerImpl.currentLedger
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change added tests and can be verified as follows:
get newest ledgerid of ManagedLedgerImpl and call getLedgerMetadata with this ledgerid. Then validate ledgerCache donot contain this ledgerid.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(It is a minor bug fix, no api is affected)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

@yapxue Good catch!
It seems that there could be a similar problem with getEnsemblesAsync method in ManagedLedgerImpl. Can you please check that too?

@yapxue Good catch! It seems that there could be a similar problem with getEnsemblesAsync method in ManagedLedgerImpl. Can you please check that too?

Thanks for suggestion. I have checked and added similiar fix.


how to auto add (auto generate) video title in fluid-player ?

Hello Anyone,
can you tell me how to auto add (auto generate from video) video thumbnail in fluid-player ?

Hi @rsk1987, Fluid Player doesn't support this right now. As for a custom implementation, I imagine that you can setup a server that can return an image based on your video source.

@rsk1987 is this still an issue? Can this ticket be closed? Thank you!

Closing the ticket due to a lack of feedback over an extended period. Please feel free to reopen if you have further input or require additional assistance. Thank you.


Audio Encryption

The files should not be able to play using local media player.
The recordings should work in-app.

After team discussion, we decided that local encryption is not necessary but keep the cloud encryption.


Improve surfacing UI alerts

Fellow Kudu'ers,
So, i've implemented the long awaited (or was it?) New File "feature" in the UI - that was trivial - i'll send a PR for that. Of course you could do touch newfile.txt but how discoverable is that?
Now for the more interesting part, i'm looking at improving the way errors are being surfaced to the UI.
Here's my first attempt:
UI alert on backend error -- Before:

UI alert on backend error -- After:

Code addition is minimal, just a few lines of jQuery for the toast and a few lines of .css, no plugins.
Needs a bit more work on the styling (perhaps a broken Kudu swiss army knife replacing the current (!) glyph in the toast...). I'm not sold on the bright red color either, maybe it's too much.
Feedback is appreciated.

@snobu i think it is a great idea. I love demo.
some suggestions and questions:

For styling, Kudu has reference to bootstrap css, maybe can take a look at their Contextual classes ?

For the toast, other than jQuery, there is also also Bootstrap Alert messages? Maybe take a quick look and see which one works better for you in this case as well as on different browsers.

For the toast animation, will it auto disappear? I would suggest to NOT "auto disappear" and have a way (a x button maybe) to let user close it themselves. Since error message sometime would be long and people might not able to finish reading it before message disappear.

Will be good to have a easy way to copy the error message :)   (not an important improvement i think, since we can always select the text and do copy from right click menu)

Thanks! All very good suggestions! I'll do more research on all points.

Tried Bootstrap's alert alert-danger and alert-warning, they're... very washed out for the purpose of showing a fatal error.

I've also looked at Bootstrap Alert but it's mainly modal and i'd have to work against it (deconstruct the modal, remove what's not needed and make a toast out of it), i think that just building from scratch around JQuery's .animate() and keeping it simple is good recipe for cross-browser success. It already works in Edge, Chrome, IE and Firefox since it's just jQuery. Using my killer smooth skills i'll ask my girlfriend for her Macbook to test Safari and not get into trouble!
Here's the new improved build with manually disposable toasts (no auto hide)

// Toast notifications for backend errors
// TODO: Move as much scaffolding as possible to WindowsConsole.cshtml

function toast(errorMsg) {
    var scaffold = `

` + errorMsg + '
' +
        '
';
    var item = $(scaffold);
    $('#toastem').append($(item));
    $(item).animate({'right': '12px'}, 'fast');

    $('#toastem').on('click','.notification', function(){
  	    $(this).animate({'right': '-400px'}, function(){
            $(this).remove();
  	    });
  	});
}

@snobu thanks for the update, I think it looks great! Feel free to send out PR.

@snobu just notice below statement from you, if user want to copy the error message, they will have to click on the toast, will be good to only dismiss when click on "X" ?  Unless there is a way for user to copy error message.
BTW: You can click anywhere on the toast to dismiss, not just the X

Right. The close div should close it and the right part should allow you to copy.. Although I wonder if that's something people will really use since these are pretty much errors you deal with on the spot not something you'd send over to someone.. Maybe we should just console.error() the message.. But then again I wonder how to make that discoverable.. Let me do some more thinking around this.

Here's what i believe to be the final form.
I've shown this to my girlfriend (who is a DTP artist) and the color combo seems to be the best match for the Bootstrap theme and communicating a fatal error.
Of course i need this to also be Project Kudu approved not just girlfriend approved.

I only need to iron out a few quirks with passing JSON from the $.axaj calls into the toast a bit more cleanly and i'll send the PR. But after my holiday. Which starts now. :airplane:

Looks good to me, so let's call it Project Kudu approved!

PR sent - https://github.com/projectkudu/kudu/pull/2124


added probe_spawn_tile_count_buffer_ & removed CompactScreenProbes …

added probe_spawn_tile_count_buffer_ to hold spawned tile count
and mark as overrideable at the end of the SpawnScreenProbes pass
so that we can remove CompactScreenProbes , which is , from my understand, not necessary for the gi pipeline
pipeline screenshot before and after this change:

Thanks for the PR!
What you've implemented is essentially compaction using atomic operations, which is totally valid here.
What we have instead is a "stable" compaction (i.e., preserves the ordering of the input elements) using a regular sum scan (also known as prefix sum).
I expect both approaches are valid and performance should be similar (seems to be confirmed with the performance numbers visible on your screenshots) so feel free to use this approach although we probably won't merge the changes as it is functionally equivalent to what's already in place.

ok, got it. thanks for your reply


Flow resolvers signature shouldn't use optional

Related:
https://github.com/dotansimha/graphql-code-generator/issues/489

Fixed in 1.1.0 🚀


What happened to Dat?

@wcwitt  -- I'm currently trying to implement a fitting script for a new project and noticed for the first time how much the structure of ACEfit has changed. The new AtomsData is now very restrictive and moreover seems to require far more code overhead that the old code that was inspired by IPFitting. I'm guess there were good reasons for those changes though but I don't remember the discussion. Can you remind me please?
Depending on this, I may bring the old datastructures back. As far as I can tell they can easily live side-by-side with your new framework.

... so my first impression is that the current model is very general and would be very suitable for a package such as ACE1.jl + ACE1pack.jl where it is worthwhile writing out in detail what kind of observations we like. But it is not flexible. If I want to introduce a new kind of observation I cannot do this easily e.g. by just defining a new local observation type or even monkey-patching ACE1pack.jl. I am missing that possibility.

There were several reasons for moving things around ... I'm trying to remember.
One was this request: https://github.com/ACEsuit/ACEfit.jl/issues/31. That led to me defining a very general AbstractData in ACEfit, which is fleshed out in ACE1pack. (And could be fleshed out in other ways for other fitting purposes.)
I think the restructuring was also better for distributed fitting somehow, but don't recall exactly.

And I can restore some of the ObsEnergy, ObsForce, and ObsVirial stuff if you prefer ... if I remember correctly I found that part of IPFitting challenging to parse, so my compromise was to make a very abstract AbstractData in ACEfit and a very concrete AtomsData in ACE1pack.

Maybe the best way forward is if I first try to make the ACE1pack datastructure as flexible as I would like it to be and then we can see how much of that can be moved back into ACEfit.

as a hook: immagine you want to add new observations about charges or magnetic moments.


Use a valid canceled task

Task.FromCanceled throws if the token isn't cancelled. From the docs:

Exceptions
ArgumentOutOfRangeException
Cancellation has not been requested for cancellationToken; its IsCancellationRequested property is false.

@aspnet-hello triage https://dev.azure.com/dnceng/public/_build/results?buildId=224856&view=logs&jobId=bfccef3f-45d4-504d-9a7f-a8a74922e6bf

This comment was made automatically. If there is a problem contact<EMAIL_ADDRESS>I've triaged the above build. I've created/commented on the following issue(s)
https://github.com/aspnet/AspNetCore-Internal/issues/2674


VisIt won't plot cell centered vector data from xdmf files

When trying to import XDMF vectors VisIt does not behave according to the spec. Attached are two XDMF files. One works with VisIt (a 1x1x2 grid with two 3d vector elements stored in a 1x6 array) and the second file does not work with VisIt although it is the same except the grid is expanded out by one cell and one more vector is added on. Both files display properly in Paraview. If one tries to display the vector in the second file the following error appears: Vector: () viewer: An invalid variable (TESTvariable) was specified. When displaying properly as in Paraview it should appear as below.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 2623
Status: Resolved
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: VisIt won't plot cell centered vector data from xdmf files
Assigned to: Kathleen Biagas
Category: -
Target version: 2.12.3
Author: Amos Manneschmidt
Start: 05/31/2016
Due date:
% Done: 100%
Estimated time: 1.00 hour
Created: 05/31/2016 02:02 pm
Updated: 06/13/2017 02:10 pm
Likelihood: 2 - Rare
Severity: 4 - Crash / Wrong Results
Found in version: 2.10.0
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
When trying to import XDMF vectors VisIt does not behave according to the spec. Attached are two XDMF files. One works with VisIt (a 1x1x2 grid with two 3d vector elements stored in a 1x6 array) and the second file does not work with VisIt although it is the same except the grid is expanded out by one cell and one more vector is added on. Both files display properly in Paraview. If one tries to display the vector in the second file the following error appears: Vector: () viewer: An invalid variable (TESTvariable) was specified. When displaying properly as in Paraview it should appear as below.
Comments:
IMHO, the xdmf data files here are broken. They use '2DRectMesh' but then go on to define 3 coordinate axes for them.I fidddled with the files a bit and got VisIt to plot some things. I've attached those. But, I was never able to get VisIt to plot cell centered vector data. It could be because I don't understand xdmf files. I tried it with 3DRectMesh but the behavior is exactly the same in either case. It appears that because the XML is interpretive, 2DRectMesh that is specified to have 3 elements is identical to a 3DRectMesh. Considering that Paraview plots it correctly I strongly suspect that it is a problem with VisIt's interpretation. I will see if I can get a 3rd opinion from EnSight later today. In my opinion, if 2/3 of the major visualization packages handle it the way the spec would seem to suggest it should be handled then the deviant interpretation is probably wrong. One problem is this spec page doesn't seem to be terribly precise. If I was trying to write my own software that implements the standard in a few areas I would be left with some ambiguity. Is there a more technical spec available somewhere akin to the HTML and CSS spec documents? I verified that the modifications to resolve #2355 also fixed this issue.I opened simplified_broken.xmf and was able to created a Vector plot of 'TESTvariable'.

2623_bar.xdmf.zip
2623_foo.xdmf.zip


Adding resources page

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/Resources.md
:white_check_mark:Succeeded

windows-iotcore/TOC.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fields part of nodegroup are not disabled on an edit of an EKS v2 cluster

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Deploy a Hosted eks cluster
When the cluster comes up Active in Rancher --> Edit on the cluster.
In the nodegroup section, currently only fields - min size, max size, desired size, labels, tags are editable.
The other fields - instanceType, gpu, diskSize and ec2SshKey --> cannot be updated.

Expected Result:
UI should disabled edit on these fields - instanceType, gpu, diskSize and ec2SshKey
Other details that may be helpful:

Related issue -- https://github.com/rancher/rancher/issues/28126
Currently when  instanceType, gpu, diskSize and ec2SshKey, there is no update done on the cluster for those fields.

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-head - commit id: 89dd0fe
Installation option (single install/HA): single/HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): hosted/imported EKS

On master-head commit id: c983bb ui tag: latest2

Deploy a hosted EKS cluster.
When the cluster has ocme up active, Edit cluster.
Only Desired size, Max/Min size, labels and tags are editable fields.


Individual containers within container set do not count towards progress

This may be a minor issue, creating for reference.
The progress of a workflow could a whole container set a 1 toward progress. This is because we count pods, not containers, towards progress.

Should we distinguish them and have a mechanism for container set?


Sonar analysis shows 0 bugs but reports bugs when output file is used

I installed v1.1.0 SonarTsPlugin plugin in Sonarqube 5.6.6 and my project had the following configuration
Typescript: 2.2.0
Sonar-Scanner: 2.8
Tslint: 2.4.5
When I run Sonar-scanner -X, I am able to see that it is executing node command with ts files and outputting to a tmp file. While I cat it, it is always an empty file and then it gets deleted. And no bugs are reported to sonarqube. I copied the node command and ran without --out parameter and I can see the output.
When I run with sonar.ts.tslint.outputPath=opfile.json, then I can see bugs in sonarqube. I am running with administrator permissions. Am I missing something?

The only time I've seen that happening has been when the user account running the analysis doesn't have write access to the folder being analysed - i.e. a build agent in a CI environment running as a limited user account. But if you're running as administrator that's that out the window!
If you do use the --out parameter (pointing it to some random folder), presumably you're getting output in that situation? Is that how you generated opfile.json which subsequently went on to work, or did you just pipe the output of the manual command run into a file?

Yes, you are right. I gave --out opfile.json to the node command and I used that in sonar.ts.tslint.outputPath parameter.

The only time I've seen that happening has been when the user account running the analysis doesn't have write access to the folder being analysed - i.e. a build agent in a CI environment running as a limited user account. But if you're running as administrator that's that out the window!
This is exactly my config and I'm getting the same behavior. I'm happy to get my bugs now :D
I don't really need it, but I'm not able to see the opfile.json, so my config looks pretty weird... Is there a way to make things cleaner?

Not sure what you mean - you can simulate what the plugin is doing by:

Run the scanner with the -X flag (to get diagnostic info)
Copy the command line the plugin reports will be used to run the tslint analysis (you will need to change the target of the --out parameter since it'll refer to a folder that, by the time the analysis has finished, won't exist0
Run that - the output you get could then be passed via the sonar.ts.tslint.outputPath parameter if desired

Alright, I did this. I was able to access the opfile.json running this command.
So does it mean than, when we use the sonar.ts.tslint.outputPath parameter un Jenkins, the opfile.json is just a temporary file? Is it normal that it's deleted in the end of the build?


Switch from CounterSimulator.GetMaxQubitCount to AllowAtMostNQubits

Context
Currently we implement the tests for the tasks that restrict the number of qubits the solution can allocate using a custom simulator CounterSimulator, which offers several counting operations, in particular ResetQubitCount and GetMaxQubitCount.
Improvement
We should be able to use the recently introduced library operation AllowAtMostNQubits instead, and use the regular QuantumSimulator instead. The affected katas are:

[ ] GraphColoring, task 1.5
[ ] RippleCarryAdder, task 3.5
[ ] PhaseEstimation, tasks 2.1 and 2.2
(In task 2.1 the test also uses GetOracleCallsCount operation, which should be possible to replace with AllowAtMostNCallsCA - needs some experimentation.)

This needs some careful testing to make sure the correct solutions passe and allocating extra qubits fails.
Example
In GraphColoring task, for example, task 1.5 check should look something like this:
within {
    AllowAtMostNQubits(2*N+1, "You are not allowed to allocate extra qubits.");
} apply {
    CheckColorEqualityOracle(N, ColorEqualityOracle_Nbit);
}

Feel free to target one test at a time!

I Would like to work on these!
Also, while testing GraphColoring, task 1.5 using the reference implementation i got errors. It looks like either the solution allocated twice the qubits, or the current simulator counts double the qubits.
Not sure if im doing something wrong but im copying :
within {
for ((q0, q1) in Zip(c0, c1)) {
// compute XOR of q0 and q1 in place (storing it in q1)
CNOT(q0, q1);
}
} apply {
// if all XORs are 0, the bit strings are equal
(ControlledOnInt(0, X))(c1, target);
}
Inisde   operation ColorEqualityOracle_Nbit located in Task.qs
and i get the following failed test doing dotnet test in the GrapColoring folder:
X T15_ColorEqualityOracle_Nbit_Test [80ms]
Error Message:
Microsoft.Quantum.Simulation.Core.ExecutionFailException : You are not allowed to allocate extra qubits maxcout 6 jsais paskoi 3. You allocated 3
Expected:       3
Actual: 6
I actually can only run the test by doing this in the test :
EqualityFactI(nq**/2**, 2N+1, $"You are not allowed to allocate extra qubits maxcout {nq} jsais paskoi {2N+1}. You allocated {nq - (2*N+1)}");
Any idea?
I'll try to bypass this problem while testing my changes for now !

I was literally just typing the message to ask whether you got the latest version :-)

haha yeah, my fault for writting comment at midnight :-) thanks for the quick answer tho! There is some tests that used the QuantumCounter to print the number of extra qubits that were used. Is that something that we want to loose using these new calls? or is there a way to fetch that information using AllowAtMostNQubits and  AllowAtMostNCallsCA?

Good catch, I missed the second task in PhaseEstimation - yes, please!

on it!
A little note on what im doing right now: the first 2 were pretty straightfoward, howerver im having issues with:  AllowAtMostNCallsCA. Right now im debuging without even considering the maxqubit constraint.
My test are failing at : Test1BitPEOnOnePair(X, BoundCA([X, H]), -1);
The framework output saysthat X is called more than once. It seems that for AllowAtMostNCallsCA(1, Controlled U, "You are allowed to call Controlled U exactly once"), ---- AllowAtMostNCallsCA does not see a difference between U and Controlled U. I might be doing something wrong on my side so i'll keep digging, but this is my conclusion as of right now.

@cgranade Do you know whether AllowAtMostNCallsCA allows to differentiate between controlled and not-controlled variants of a unitary in this scenario?

@JimmyHamel If AllowAtMostNCallsCA doesn't work in this scenario, I would switch just the qubit counting to AllowAtMostNQubits and leave operation counting as is, and probably file an issue on the libraries repo describing the behavior, similar to https://github.com/microsoft/QuantumLibraries/issues/343. We won't be doing a mass switch to AllowAtMostNCallsCA until that one is resolved, so it's ok to keep CounterSimulator here for now.

Sounds good! I'll keep the CounterSimulator for PhaseEstimation, tasks 2.1 only and change only the qubit counter part.
I created an issue in the QuantumLibraries repo, you can find it here: https://github.com/microsoft/QuantumLibraries/issues/367
thank you!

Related merge request: https://github.com/microsoft/QuantumKatas/pull/560


[Draft]Add support for dynamic doc generation for Meshmodels

Description
This PR fixes #
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

Here's an example of what this capability can offer.
https://github.com/meshery/meshkit/assets/7570704/d5dd9f77-3750-4506-b205-5a8c41d81040

Proposal for the doc format (in JSON) for each component:
{[
{"title": "....",
"description":  "....",
"url":  "....",
},
{"title": "....",
"description" "....",
"url" "....",
},
{...},
{...},
{"title": "....",
"description" "....",
"url" "....",
}
]}

Here number of array elements determines whether navigation button is to be shown or not.
// @leecalcote @MUzairS15

@theBeginner86 is the idea that "description" is the same as "caption"?

@theBeginner86 is the idea that "description" is the same as "caption"?

yes, caption and description are the same thing. We can use either of them. 👍

{
"doc": [
{"title": "....",
"description":  "....",
"url":  "....",
"formatted": false
},
{"title": "....",
"description" "....",
"url" "....",
"formatted": false
},
{
"html": "..."
"formatted": true
},
{...},
{...},
{"title": "....",
"description" "....",
"url" "....",
}
]}

I suppose that some of the implicit possible attributes are excluded from this "internal" consideration; e.g. registry deep link, Meshery docs link, Model website link


product_version regex matching for the version is too relaxed

When we add 5\..*, not only 5.0.0 will be matched, but also anything with 5. in it will be matched, e.g. 6.5.0 will also be matched. This caused some unexpected behavior when we are using it.
We saw the reference at https://github.com/pivotal-cf/pivnet-resource/blob/master/filter/filter.go#L40, where regexp.MatchString(version, release.Version) will be able to match the string in any position.
Maybe update the product_version description to highlight the usage, e.g. adding anchor ^5\..* in order to match any 5.* releases. Or, make the regexp.MatchString more strict to always add a ^ anchor automatically in front the version.

Thanks for the feedback!
Does this section in the documentation capture what you are describing? If so, we can update the product_version description, like you suggested, with a link to that as well to clarify to users more explicitly.
What are your thoughts on this?


Allow user to remove narrative from (some) elements

Is your feature request related to a problem? Please describe.
When exporting a query output to csv, I would like to have the option to remove the narrative attribute for most/all codelist-based elements, keeping only the codes. For instance, if I request the sector codes for a number of activities, I would prefer to only have the codes and percentages, not the name of each sector.
I have not checked systematically all codelist-based elements. I see that activity status returns only codes (no narrative), but sector codes include narratives.
Describe the solution you'd like
Either when selecting a filter or in the Output Format tab, I would like to have a checkbox to include narrative attributes. Default would be not to include them.
Describe alternatives you've considered
None. The narrative is mixed in with the sector codes in the csv output, I don't see how to remove it.
Additional context
Add any other context or screenshots about the feature request here.

Hi @YohannaLoucheur
Please see #1856 whereby the team are working on flattening the columns which are currently being output with a JSON string within a single cell. Although Sectors is not on the list on that issue, I have asked Z&Z to include it in the delivery

This has been fixed as part of work some time ago now. Closing it.


Allow 0-D interface MemRef in FlattenMemRefSubspan

This patch allows 0-D MemRef to appear in interface IREE subspan ops.
This it to support cases where we want to use 0-D MemRef to represent
raw pointers for micro-kernel ABI purposes.

Ha, actually there is a VMVX bug I think. Let me push out a fix.


Unicode is not rendered correctly

test('Unicode', () {
  var output = parse('\u{1F634}\n').renderString({});
  expect(output, equals('\u{1F634}\n'));
});

The above test fails with the following error:
Expected: '😴\n'
            ''
  Actual: '😴\n'
            '😴\n'
            ''
   Which: is different. Both strings start the same, but the actual value also has the following trailing characters: 😴\n

package:test_api              expect
test/mustache_test.dart 41:7  main.<fn>.<fn>

@xxgreg @jonahwilliams this issue is a blocker for me. I believe I have a fix but this repo and https://github.com/jonahwilliams/mustache don't appear to be very active. I really don't want to fork and republish yet again so can either of help get a fix for this merged and published? Thanks! 🙏

Pinged you privately with a possible solution.

@xxgreg @felangel I'm facing a similar issue I believe and wonder if a similar solution could help.
test('Less than', () {
    var output = Template('{{val}}').renderString({'val': '<'});

    expect(output, '<');
  });

fails with the error:
Output for Less than
Expected: '<'
  Actual: '<'
   Which: is different.
          Expected: <
            Actual: <
                    ^
           Differ at offset 0

package:test_api                                   expect
test/src/generators/dart_generator_test.dart 91:5  main.<fn>

Thanks! 🙏

@mtwichel yeah I think it's the same root cause 👍
@xxgreg can you please add me as a publisher/contributor? I'd love to open a PR with a fix and publish a new release 👍

@mtwichel yeah I think it's the same root cause 👍
@xxgreg can you please add me as a publisher/contributor? I'd love to open a PR with a fix and publish a new release 👍

would be great to push this along with null safety

@xxgreg @jonahwilliams this issue is a blocker for me. I believe I have a fix but this repo and https://github.com/jonahwilliams/mustache don't appear to be very active. I really don't want to fork and republish yet again so can either of you help get a fix for this merged and published? Thanks! 🙏

this one has at least null safety now 👍


Fix GitHub Chrome Extension after GitHub introduced a new design for viewing/editing files

GitHub introduced several changes in the code navigation and editing that broke the GitHub Extension for DMN and BPMN.

GitHub is introducing incrementally a new code navigation, and the current Chrome Extension doesn't work for this new navigation as you can see in the attached images.
Code View:

Code editing:

@ljmotta done :)


Submit button is unresponsive

When I try to create a new env, I am unable to because the Submit button doesn't seem to be responsive. This is the same for the Token button. I am logged in and I have attempted this on two different Nebari deployments. I have attached console logs.
Known affected versions: v0.4.9 and v0.4.11 (presumably v0.4.10 as well)
livedemo.qhub.dev-1660781750506.log

Hi @iameskild  :)
Can you share the YAML of the env you were trying to build ?
The current UI has this behavior, that when the YAML doesn't validate, the build isn't triggered but no feedback is shown to the user. We're working on it.

MIssing c.CondaStoreServer.behind_proxy = True in the Qhub server configutation. See ref.
conda-store:
   extra-settings:
       CondaStoreServer:
          behind_proxy: true

Apologies this issue should have been opened on the QHub side. As @viniciusdc mentioned, the resolution was to add c.CondaStoreServer.behind_proxy = True to the conda_store_config.py.


php81Extensions.datadog_trace: 0.96.0 -> 0.97.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for php81Extensions.datadog_trace is: Datadog Tracing PHP Client
meta.homepage for php81Extensions.datadog_trace is: https://github.com/DataDog/dd-trace-php
meta.changelog for php81Extensions.datadog_trace is: https://github.com/DataDog/dd-trace-php/blob/0.97.0/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.97.0 with grep in /nix/store/isx3kzspr22ggvaz19pl93fjka8mkqip-php-ddtrace-0.97.0
found 0.97.0 in filename of file in /nix/store/isx3kzspr22ggvaz19pl93fjka8mkqip-php-ddtrace-0.97.0

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
php81Extensions.datadog_trace
php82Extensions.datadog_trace
php83Extensions.datadog_trace

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/isx3kzspr22ggvaz19pl93fjka8mkqip-php-ddtrace-0.97.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A php81Extensions.datadog_trace https://github.com/r-ryantm/nixpkgs/archive/890e33651f200ad21956790f15b89568ef4802a9.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/890e33651f200ad21956790f15b89568ef4802a9#php81Extensions.datadog_trace

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/isx3kzspr22ggvaz19pl93fjka8mkqip-php-ddtrace-0.97.0
ls -la /nix/store/isx3kzspr22ggvaz19pl93fjka8mkqip-php-ddtrace-0.97.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

php81Extensions.datadog_trace
php82Extensions.datadog_trace
php83Extensions.datadog_trace

Maintainer pings
cc @aanderse @drupol @etu @Ma27 @talyz for testing.

Broken on Darwin because of Libxml 2.12

Broken on Darwin because of LibXML.

@ofborg build php81Extensions.datadog_trace

Let me rebase it since https://github.com/NixOS/nixpkgs/pull/283175 has been merged.

Result of nixpkgs-review pr 283296 run on aarch64-darwin 1


Error "...Scans not sorted in retention time dimension..."

Basic information

My MZmine version: 3.0.54-beta
Input file: .mzML from MSV000089819

Issue
Clicks on rows in raw data overview or attempts to build a chromatogram give errors "...Scans not sorted in retention time dimension...". So, the file cannot be processed. Probably, it would be better if MZmine could automatically sort spectra by RT values and give a warning about it.

bloody waters files... :D
i thought the mzml import already does the sorting. we definitely had a method to renumber the scans and sort them by rt, maybe the pattern is not recognised correctly.

I think we are sorting the mz values of the scans but not the RT are we

closing, duplicate

Your Waters file was improperly converted. See fix in #1277


Turn VR2G-nativeLibraries into a package

Prerequisite for #142 because it will get rid of all submodules within the upcoming VR2Gather package.

Fixed in e1553bf6


Enhancements That I Would Love To See

Hey! I love this plugin for letting me sketch and handwrite directly in notes, it's that last 'piece' obsidian is missing.
Currently stopping this from being an S-tier plug-in are:

The lack of an eraser
The pen size is way too phat
The UI is a bit chunky and ugly

Cheers!

Thanks for the feedback! The UI definitely needs some reworking, which I'm planning on doing in the near future. Unfortunately, I don't have much control over the pen size since it is an excalidraw feature, but I will try my best to add as much variability as possible. There is an eraser in excalidraw, so I will add it as an option to enable it in the settings. If you have any specific recommendations for fixing the UI, I'd love to hear them. Thanks!


dist/naf-easyrtc-adapter.js 404 (Not Found)

npm run start result in 404 on dist/naf-easyrtc-adapter.js

Fix by aed2ce6f02519fc97b87b442782de072d5715e77 in #1

The easyrtc adapter moved again to the networked-aframe repository. I'm closing all issues in this repository.


how can i change  Language

Hello,
depending on what you actually need, with this image, it would probably require some modifications in the Dockerfile. Check the stage-xfce part please. It's not necessarily the case with the newer images accetto/xubuntu-vnc or accetto/xubuntu-vnc-novnc. They both support sudo, so you're able to install new stuff also in running containers. It would help, if you'd provide more information.
Regards, accetto

I'm closing this question because of the lack of cooperation.


Fixes #390.
Fixes #403.
Fixes #404.
Fixes #405.
Fixes #407.
Fixes #408.
Fixes #409.
Fixes #410.

One small change to help, then :+1: to merge.

Rebased against master, added comment to the class Media, shortened iata.help_text, added a merge migration.


New package: GetLine v0.1.0

Registering package: GetLine
Repository: https://github.com/gfr1337/GetLine.jl
Created by: @gfr1337
Version: v0.1.0
Commit: 5f2f22f39cd9d87dddadad870b1cf7e49a092be8
Git reference: HEAD

A small wrapper around the readline function to better detect eof.

[noblock]
Hi there! Is this a training exercise or a test? It doesn't look like it has enough functionality to be added to the General Registry... 🤔

It's just a convenience wrapper around readline. If i create a set of more useful packages that use it as a dependency then I probably resubmit.

[noblock]
No worries! Remember, as per https://github.com/JuliaRegistries/General#faq :

The General registry is not a place for "personal packages" that consist of collections of "utility functions"


Format 'pending request" btn in committeeEdit page

pending request button is not formatted correctly. Give it proper spacing

this will be part of a future issue revamping the committee screens


kube3d missing arm platforms

Project description
k3d creates containerized k3s clusters. This means, that you can spin up a multi-node k3s cluster on a single machine using docker.
This package already exists in nix but is missing arm and darwin platforms.
Rancher does support darwin and arm as a target: https://github.com/rancher/k3d/releases/tag/v5.0.3

k3d-darwin-amd64
k3d-linux-arm64

https://github.com/NixOS/nixpkgs/blob/a0dbe47318bbab7559ffbfa7c4872a517833409f/pkgs/applications/networking/cluster/kube3d/default.nix
Missing:

[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

the assets are binaries not source code
aarch64-linux already works
for darwin we need to change the platforms.linux to platforms.unix

If someone is up for testing it on aarch64-darwin semi-regularly I'm happy to swap the platform

I can test semi-regularly. M1 is my main development env and this is a primary dependency for my work.

Cool, thanks. Ill get that swapped out


Spurious (and Duplicated) Entry on Mismatching Filter

When filtering for words (at the top) but not finding anything, then the list is not empty. Instead, a spurious entry shows up instead:

Upon a second mismatch, this entry now shows up twice:

This can go on ad infinitum. This mismatching entry (and all its duplicates) also suddenly shows up at the beginning.

Turned out to be quite simple: fixed in dc5829d .


Is 'server' injected or should I import it?

The docs seem to indicate that 'server' will be available and that I should use it and tell jshint about it.  However it doesn't seem to be set on my tests??
If I run import Server from 'ember-cli-mirage/server'; that works, is that the correct way? Or is there some kind of global 'server' variable getting created that I should use?


Android Notification Icons - Won't Build

Using the example from "Android Notification Icons":
<platform name="android">     <resource-file src="res/android/drawable-mdpi/notification_icon.png" target="app/src/main/res/drawable-mdpi/notification_icon.png" />     <resource-file src="res/android/drawable-hdpi/notification_icon.png" target="app/src/main/res/drawable-hdpi/notification_icon.png" />     <resource-file src="res/android/drawable-xhdpi/notification_icon.png" target="app/src/main/res/drawable-xhdpi/notification_icon.png" />     <resource-file src="res/android/drawable-xxhdpi/notification_icon.png" target="app/src/main/res/drawable-xxhdpi/notification_icon.png" />     <resource-file src="res/android/drawable-xxxhdpi/notification_icon.png" target="app/src/main/res/drawable-xxxhdpi/notification_icon.png" /> </platform>
No longer builds in the latest Cordova, I think because all the icon paths have changed since the introduction of adaptive icons. I've tried specifying the new paths with the icon file in there, etc - but no build, no bueno,
Does someone have a working example with the current Cordova or workaround for this they can post?

Ok, it's choking on this from the docs:

Execution failed for task ':app:mergeReleaseResources'.

A failure occurred while executing com.android.build.gradle.internal.res.ResourceCompilerRunnable
Resource compilation failed (Failed to compile resource file: C:\Users\myapp_path\platforms\android\app\src\main\res\xml\config.xml: . Cause: javax.xml.stream.XMLStreamException: ParseError at [row,col]:[35,140]
Message: http://www.w3.org/TR/1999/REC-xml-names-19990114#AttributePrefixUnbound?meta-data&android:name&android). Check logs for more details.

So this must not be the correct XML anymore - does anyone have a working replacement?

Bump
Did u resolve problem?

No, but I haven't come back to this plugin while I worked on other stuff - it seems like it has numerous issues since the new Firebase SDK came out, hopefully those get ironed out.
If someone has a working icon notifications setup, I'd sure appreciate an example.

Fixed by https://github.com/dpa99c/cordova-plugin-firebasex/pull/836
Please see the example project for a working illustration of Android notification icons


Slow USB3.0 HDD speed

Hello, I have a rock64 in which I have installed stable OMV image (https://github.com/ayufan-rock64/linux-build/releases/download/0.9.14/stretch-openmediavault-rock64-0.9.14-1159-armhf.img.xz.
I am willing to use my Seagate Expansion Desk drive (8TB) for samba shares, when I connect my drive directly to my desktop PC, I can read/write at 100MBps or more, but in rock64 it drops to 20-30MBps write and 30-40 read speeds.
I tried installing armbian and blacklisting UAS driver, which made me able to have 120MBps read speeds. Maybe there are problems with UAS driver in OMV image?
To reproduce, hook up a USB3.0 Seagate external HDD and test read/write speeds of OMV images

External Seagate drives have been known to be buggy for years. If it's a fairly new one, I would assume that there might be workarounds missing from the Kernel. https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/drivers/usb/storage/unusual_uas.h?h=v5.10.1
Personally I have not tested USB3.0 storage on the Rock64, so I cannot say if there is any devices that run faster.

I found the reason, it was that the disk has NTFS partitions, and you must use "-o big_writes" option when mounting them.
Using it enables the rock64 to get all the performance out of it's USB3 port :+1:


Sockscode-VSCode section does not appear in Explorer pane

I am on VSCode 1.25.1 on Windows, using version 1.0.1 of the extension.
Neither creating a session nor joining one shows me the "SOCKSCODE-VSCODE" GUI element I see in the extension homepage.
The extension may be working, but I can't see it!

Same here. Joining the room does not share the code. So we don't see each others code.


feat(tutorial): complete step 5

Completed tutorial step 5 from https://www.carbondesignsystem.com/tutorial/react/step-5/
Note: Things run much faster if .cfignore is this:
# .cfignore

*
!Staticfile
!build

I apologize, @alisonjoseph !
The URL is https://carbon-tutorial-docwhat.mybluemix.net/


Fix null manufacturer data

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 5 committers have signed the CLA.:white_check_mark: ened:x: Adam Burnett:x: thermoChas:x: tw-tjd:x: adamburnetAdam Burnett seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add grouped Optuna trials as runs when using WandbLogger

Before submitting

[x] Was this discussed/approved via a Github issue? (Discussed in Slack with Thomas)
[x] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?

adds the ability to pass additional kwargs to the Sweep's logger.
adds ability to group runs when using wandb

these groups are the optuna trials. each new sweep in a given project will be grouped according to some name, so that a common trial naming convention can be used i.e. wandb_project/{ some group name }/trial-n
wandb project with grouping: lightning-examples-optuna
lightning + optuna + wandb that implements grouping: lightning_pod/flows/optuna_flow
Fixes # (issue).
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

changed branch name. will reopen new draft.

see #269


Add external admin account

Summary
This commit adds an admin account which is intended to be exposed to the user. This enables the user to change the password of the admin user without breaking the internal scripts of the image.
However, there are no safeguards available to prevent the user from changing the password of the internal admin as well.
Checklist

[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog
[x] Update tests.
[x] Link this PR to related issues.

There are also two commits that change the image tags, as Inventage now provides shorter tags as well.


Test freezes when aws credentials are invalid (probably deadlock in Aws::Utils::Stream::ConcurrentStreamBuf::overflow)

https://github.com/aws/aws-sdk-cpp/blob/4dfce9c92c7f9f95fcd13c8973df7bfc059e419d/aws-cpp-sdk-transcribestreaming-integration-tests/TranscribeTests.cpp#L89
Steps to reproduce:

comment out FAIL() in OnErrorCallback handler
set aws credentials through env variables
build & run

Is it expected to kill the thread which executes TranscriptEventCallback when OnErrorCallback is called? What is a proper way of graceful invalid credentials error handling?
Added usleep between writing events to AudioStream, finishes fine.
Freeze stack strace:
Thread 2 (Thread 0x7fae6420b700 (LWP 29896)):
#0  0x00007fae651254d5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib64/libpthread.so.0
#1  0x00007fae64fee6d0 in std::condition_variable::wait(std::unique_lock<std::mutex>&) () from /lib64/libstdc++.so.6
#2  0x00007fae6528b7de in LogThread(Aws::Utils::Logging::DefaultLogSystem::LogSynchronizationData*, std::shared_ptr<std::ostream> const&, std::__cxx11::basic_string<char, std::char_traits<char>, Aws::Allocator<char> > const&, bool) () from aws-sdk-cpp/build/aws-cpp-sdk-core/libaws-cpp-sdk-core.so
#3  0x00007fae6528c31c in std::thread::_State_impl<std::thread::_Invoker<std::tuple<void (*)(Aws::Utils::Logging::DefaultLogSystem::LogSynchronizationData*, std::shared_ptr<std::ostream> const&, std::__cxx11::basic_string<char, std::char_traits<char>, Aws::Allocator<char> > const&, bool), Aws::Utils::Logging::DefaultLogSystem::LogSynchronizationData*, std::shared_ptr<std::basic_ofstream<char, std::char_traits<char> > >, std::__cxx11::basic_string<char, std::char_traits<char>, Aws::Allocator<char> >, bool> > >::_M_run() () from aws-sdk-cpp/build/aws-cpp-sdk-core/libaws-cpp-sdk-core.so
#4  0x00007fae64ff4474 in ?? () from /lib64/libstdc++.so.6
#5  0x00007fae6511f5a2 in start_thread () from /lib64/libpthread.so.0
#6  0x00007fae64cf1303 in clone () from /lib64/libc.so.6
Thread 1 (Thread 0x7fae6420c880 (LWP 29895)):
#0  0x00007fae651254d5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib64/libpthread.so.0
#1  0x00007fae64fee6d0 in std::condition_variable::wait(std::unique_lock<std::mutex>&) () from /lib64/libstdc++.so.6
#2  0x00007fae6528ecab in Aws::Utils::Stream::ConcurrentStreamBuf::FlushPutArea() () from aws-sdk-cpp/build/aws-cpp-sdk-core/libaws-cpp-sdk-core.so
#3  0x00007fae6528f047 in Aws::Utils::Stream::ConcurrentStreamBuf::overflow(int) () from aws-sdk-cpp/build/aws-cpp-sdk-core/libaws-cpp-sdk-core.so
#4  0x00007fae6506437a in std::basic_streambuf<char, std::char_traits<char> >::xsputn(char const*, long) () from /lib64/libstdc++.so.6
#5  0x00007fae65052ae7 in std::ostream::write(char const*, long) () from /lib64/libstdc++.so.6
#6  0x00007fae652841a2 in Aws::Utils::Event::EventEncoderStream::WriteEvent(Aws::Utils::Event::Message const&) () from aws-sdk-cpp/build/aws-cpp-sdk-core/libaws-cpp-sdk-core.so
#7  0x000000000040a2e7 in Aws::TranscribeStreamingService::Model::AudioStream::WriteAudioEvent(Aws::TranscribeStreamingService::Model::AudioEvent const&) ()
#8  0x0000000000408469 in TranscribeStreamingTests_TranscribeAudioFile_Test::TestBody()::{lambda(Aws::TranscribeStreamingService::Model::AudioStream&)#3}::operator()(Aws::TranscribeStreamingService::Model::AudioStream&) const [clone .isra.0] ()
#9  0x00007fae6531d2b9 in Aws::TranscribeStreamingService::TranscribeStreamingServiceClient::StartStreamTranscriptionAsync(Aws::TranscribeStreamingService::Model::StartStreamTranscriptionRequest&, std::function<void (Aws::TranscribeStreamingService::Model::AudioStream&)> const&, std::function<void (Aws::TranscribeStreamingService::TranscribeStreamingServiceClient const*, Aws::TranscribeStreamingService::Model::StartStreamTranscriptionRequest const&, Aws::Utils::Outcome<Aws::NoResult, Aws::Client::AWSError<Aws::TranscribeStreamingService::TranscribeStreamingServiceErrors> > const&, std::shared_ptr<Aws::Client::AsyncCallerContext const> const&)> const&, std::shared_ptr<Aws::Client::AsyncCallerContext const> const&) const () from aws-sdk-cpp/build/aws-cpp-sdk-transcribestreaming/libaws-cpp-sdk-transcribestreaming.so
#10 0x0000000000406d46 in TranscribeStreamingTests_TranscribeAudioFile_Test::TestBody() ()
#11 0x00007fae652f1afe in testing::Test::Run() () from aws-sdk-cpp/build/testing-resources/libtesting-resources.so
#12 0x00007fae652f1c30 in testing::TestInfo::Run() () from aws-sdk-cpp/build/testing-resources/libtesting-resources.so
#13 0x00007fae652f1cf5 in testing::TestCase::Run() () from aws-sdk-cpp/build/testing-resources/libtesting-resources.so
#14 0x00007fae652f21ec in testing::internal::UnitTestImpl::RunAllTests() () from aws-sdk-cpp/build/testing-resources/libtesting-resources.so
#15 0x00007fae652f22d7 in testing::UnitTest::Run() () from aws-sdk-cpp/build/testing-resources/libtesting-resources.so
#16 0x0000000000405691 in main ()

Hi, there.
In this test, the OnErrorCallback will always be triggered because of the reason specified above. And by transcribedResult.empty() we know if it's a true error.
If it's a true error, transcribedResult is empty, you can put your logic inside this block to handle the error.

Hi, thanks for reply, but the problem it's not about detecting the error, it's about freezing main thread. I mean, what kind of logic is expected in OnErrorCallback block? killing main thread?

Just re-produced your case, and there is actually no error event response from service when transferring data with wrong credentials/signature (probably a service bug). The stream is still alive, service didn't terminate/close it as well. That's why the program hang there do nothing.

Thanks, what is the next step to resolve this issue? Are you going to open some inner aws issue, or public one (which can be tracked) ? Or I have to open another one?

I have submitted an internal issue to transcribe-streaming team, we'll see what they would response.

Root cause:
When event stream request encountered error and returned from MakeRequest, http backend may stop reading from event stream buffer.
If the stream is still active, application could keep writing data to stream buffer. When there is not enough space in buffer, writing will wait, but if there is no thread reading from the buffer, then the application will wait forever (we need to wake up from waiting and return when the buffer's eof flag is set).
A fixing PR is in code review.

We submitted a commit to fix this issue: https://github.com/aws/aws-sdk-cpp/commit/57c88fb8baa6da2b14f79aacf05b8ec41590362d
Can you add these lines to see if it works for you?
https://github.com/aws/aws-sdk-cpp/blob/57c88fb8baa6da2b14f79aacf05b8ec41590362d/aws-cpp-sdk-transcribestreaming-integration-tests/TranscribeTests.cpp#L120


Different color schemes for different users

How can I most easily implement different color schemes for different users?
Use Case
Lets say I have an application with user login. Depending on a flag of the user (i.e. company A) I want to use a different color scheme than the default.

text colors
bg colors
border colors

The app can store the colors in a database, that's no problem. But what would be the best way to implement with tailwind?
Any thoughts?

Well I found this

https://github.com/adamwathan/theming-tailwind-demo


add airports

in the previous terrain icons for airports were added as follows:
https://github.com/Citytracking/Terrain/blob/master/mapnik/labels.mss#L25-L151

fixed.


Updated raylib to fix deprecated addIncludeDir

calling addIncludeDir is deprecated, and was replaced with addIncludePath. Raylib was updated but the submodule was not updated in these bindings.
This pull request updates the submodule.

This binding will only keep up with raylib releases, not with raylib's HEAD.


(refactor) some opportunities to use lambda functions

From the README, it seems that Hystrix still wants to support Java 6 which does not have lambda functions yet.


Add support on web editor to merge a file if it changed since started editing

Gogs version (or commit ref): <IP_ADDRESS>9
Git version: 2.6.6
Operating system: CoreOS 1068.10.0
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] SQLite

Can you reproduce the bug at https://try.gogs.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
Steps:

Browse to any gogs repository
Pick a file and edit it in the web UI
Open another tab and edit the same file (while keeping the first tab/file in edit-mode)
Add a line to the file in the second tab and commit it
Now edit the file in the first tab and also commit it

It will show the red warning with text "File content has been changed since you started editing. Click here to see what have been changed or press commit again to overwrite those changes."

Thanks, but I think this has quite low priority, Gogs UI is not real time at all in current stage.


Render composition without asset substitution

Is your feature request related to a problem? Please describe.
As part of some pipeline, I'd like to be able to render different compositions from a project within the same job. Some composition might have asset replacement, some not. Right now, if no asset is provided, the Job fails with the following error :
Error veryifing job: TypeError: Cannot read property 'map' of null
If an empty Array of empty object is provided , the following error occurs:
Error veryifing job: AssertionError [ERR_ASSERTION]: job asset must have type defined
Describe the solution you'd like
adding a new flag in the cli --no-asset that would bypass the job.assets validation id the "assets" field is empty.
Maybe it's a new asset type "no-data". That would be easier to  implement in the job validation and still ask for the user to be explicit about it to avoid errors with empty "assets" field.
Describe alternatives you've considered

Using aerender directly but we loose the benefits of NexRender binary with ffmpeg.
Having a dummy asset in the json with some property values that are actually not changing the layer.

Additional context
I'm writing a program that automatically creates multiple Json files for the same After Effect project, rendering  different compositions. Sometimes with new data , sometimes not. It would be nice to be able to use nexRender, which by the way is Awesome, to have all the renders in the same pipeline, and also benefits of nexRender packages like encoding, copy,upload...
Thank you so much for the awesome work. And the help.
Ben

Hello,
Have you tried providing an empty array for the assets field in the job?
Like this: "assets": []

Boom , that works. Nevermind.
Great great work.
Thank you

If I would like to replace a json file in the project is that also possible?
Error veryifing job: AssertionError [ERR_ASSERTION]: unknown job asset type: Json


Due to the type definition, UncontrolledAccordion cannot be used without toggle

components: UncontrolledAccordion
reactstrap version #9.2.1
import method es
react version #18.2.0
bootstrap version #5.3.2

What is happening?

A type error occurs.

What should be happening?
I want to make it possible to use it without setting up props, like in the following URL.
https://reactstrap.github.io/?path=/docs/components-accordion--uncontrolled#uncontrolled

Steps to reproduce issue

Render  component without setting any props, like the following URL.

https://reactstrap.github.io/?path=/docs/components-accordion--uncontrolled#uncontrolled

Error message in console
No overload matches this call.
  Overload 1 of 2, '(props: UncontrolledAccordionProps | Readonly<UncontrolledAccordionProps>): UncontrolledAccordion', gave the following error.
    Property 'toggle' is missing in type '{ children: Element; }' but required in type 'Readonly<UncontrolledAccordionProps>'.
  Overload 2 of 2, '(props: UncontrolledAccordionProps, context: any): UncontrolledAccordion', gave the following error.
    Property 'toggle' is missing in type '{ children: Element; }' but required in type 'Readonly<UncontrolledAccordionProps>'.ts(2769)
Accordion.d.ts(10, 3): 'toggle' is declared here.
Accordion.d.ts(10, 3): 'toggle' is declared here.

Code

https://reactstrap.github.io/?path=/docs/components-accordion--uncontrolled#uncontrolled

Proposed Solution

https://github.com/reactstrap/reactstrap/blob/master/types/lib/Accordion.d.ts#L13

export interface UncontrolledAccordionProps extends Omit<AccordionProps, 'open' | 'toggle'> {

Hi, thank you for the issue. Would you like to make a PR for the same?


Gii extension generator enhancements

continued from https://github.com/yiisoft/yii2/pull/2411#issuecomment-35986454

[ ] LICENSE file creation with license content
[ ] Optional "root" directory. Now in composer.json namespace is relates to "". It is not always convenient, especially when your have many files like CHANGELOG.md, README.md, LICENSE, composer.json, composer.lock, .gitignore, .travis.yml, .scrutinizer.yml, helpers for IDE, test environment, etc. In NodeJS usually used "lib" directory, in C++ "src". Yii2 extensions could use something like that.
[ ] Optional tests directory structure: phpunit or codeception
[ ] Additional types (yii2-application, yii2-theme, freetext?)
[ ] Custom licenses (freetext?)

shouldn't @yii-bot close this and pass it to gii?

I think there's nothing to be done about it.


Can webrtc be compiled from source code on centos7 or centos8?

Hello, I want to know if webrtc can be compiled from source code on centos7 or centos8? There is little information available online, and compiling Webrtc from source can be complex. I want to know if it's worth attempting.
According to https://webrtc.github.io/webrtc-org/native-code/development/prerequisite-sw, seems it just support ubuntu or debian.
Thanks!

@zxx8808 We did not try compiling from centos. I would stick with Google's documentation for this. However, you could try using the pre-built binaries included in this project and try using them on centos.  Good luck!


Small fixes

Ignore build
Made the npm script dev windows/ps-friendly
Changed import statements and changed class to stateless function

Thanks ;)


Drop unused service-related test helpers

They appear to be unused since bc927c0088f1 ("test: Remove RuntimeDatapathLB").

/test


how to use coverlet as project in a solution

Hello,
we would like to use this source code and create coverlet .net core projects in our application (sln) which can be used on other projects to check coverage.
We have taken the code from this repo and added it in our solution. However, no coverage is generated.

Hi Nitin,
it's not clear to me your question...would you like to have a coverage of your test projects?
You need to follow the guide https://github.com/tonerdo/coverlet#installation
Why you download sources?

Hello, we would not like to use coverlet as package. We want to bring source code into our solution, build the same and use that as dll to check the coverage on other projects.

As I've said above you can copy .net tool usage https://github.com/tonerdo/coverlet/blob/master/src/coverlet.console/Program.cs#L64 we don't have an example on codebase
Maybe @Shargon can help here, they used coverlet by code.

At the moment we're blocked on vstest bug, after that fix we can go on with hiding all class and design a stable api to expose internal features https://github.com/microsoft/vstest/pull/2221

You have an example in github.com/red4sec/TuringMachine

@Nitin5acc feel free to close if solved, thanks @shargon for the sample!

@Nitin5acc related PR https://github.com/tonerdo/coverlet/pull/628


How to use nvdec/nvenc

Issue guidelines

[X] I've read the Issue Guidelines and wholeheartedly agree.

Issue Checklist

[X] I have searched open or closed issues for my problem and found nothing related or helpful.
[X] I have read the Documentation and found nothing related to my problem.
[X] I have gone through the Bonus Examples and FAQs and found nothing related or helpful.

Describe your Question
I am trying to make a yolo detector using mp4 video or real time camera. I am trying to speed up my pipeline because the fps of the model is much higher than the video and the fps marked. How can I use hardware e.g. nvenc and nvenc to encode and decode the video?
I have opencv and ffmpeg with cuda. The device is a jetson agx orin
Terminal log output(Optional)
No response
Python Code(Optional)
No response
VidGear Version
0.3.2
Python version
3.11
Operating System version
Ubuntu 22.04
Any other Relevant Information?
No response

I am trying to make a yolo detector using mp4 video or real time camera. I am trying to speed up my pipeline because the fps of the model is much higher than the video and the fps marked. How can I use hardware e.g. nvenc and nvenc to encode and decode the video?
I have opencv and ffmpeg with cuda. The device is a jetson agx orin

@johnnynunez Don't use OpenCV for video IO, but FFmpeg directly which supports both nvenc and nvenc to encode and decode the video. For this purpose, You could use my deffcode library instead which will be integrated soon in https://github.com/abhiTronix/vidgear/issues/148. The complete HW transcoding example is given in this docs:  https://abhitronix.github.io/deffcode/latest/recipes/advanced/transcode-hw-acceleration/#hardware-accelerated-video-transcoding
Goodluck!

I am trying to make a yolo detector using mp4 video or real time camera. I am trying to speed up my pipeline because the fps of the model is much higher than the video and the fps marked. How can I use hardware e.g. nvenc and nvenc to encode and decode the video?
I have opencv and ffmpeg with cuda. The device is a jetson agx orin

@johnnynunez Don't use OpenCV for video IO, but FFmpeg directly which supports both nvenc and nvenc to encode and decode the video. For this purpose, You could use my deffcode library instead which will be integrated soon in #148. The complete HW transcoding example is given in this docs: https://abhitronix.github.io/deffcode/latest/recipes/advanced/transcode-hw-acceleration/#hardware-accelerated-video-transcoding
Goodluck!

but Does it have threaded-queue-mode too?? https://abhitronix.github.io/vidgear/v0.3.2-stable/bonus/TQM/#threaded-queue-mode

You don't need multi-threading here.

You don't need multi-threading here.

and deffcode uses queue? Or only decoding?

and deffcode uses queue? Or only decoding?

@johnnynunez No queue, just decoding. You could learn more about it here: https://abhitronix.github.io/deffcode/latest/recipes/advanced/decode-hw-acceleration/#hardware-accelerated-video-decoding

and deffcode uses queue? Or only decoding?

@johnnynunez No queue, just decoding. You could learn more about it here: https://abhitronix.github.io/deffcode/latest/recipes/advanced/decode-hw-acceleration/#hardware-accelerated-video-decoding

Yeah it is working for me


Ability to bind system keys (Ctrl+Alt+Tab)

Discussed in https://github.com/Schneegans/Fly-Pie/discussions/135

Originally posted by kreejzak September 28, 2021
Hi,
it would be really awesome to be able to use system keys as shortcut triggers.
For example MX Master mouse has a thumb button that sends Ctrl+Alt+Tab key combination.
This extension would be really awesome to overcome the lack of drivers for this mouse for gestures like swiping to change desktop or quickly maximize or minimize window.
Have a great day!

As said in the discussion, Fly-Pie does not let you choose shortcuts for menus which are already used by the system. You won't be able to bind a menu to print for example, if this is used to capture screenshots, for example. However, you can make the shortcut actions simulate these shortcuts. So you cannot select print as a trigger for a menu, but you can select print for shortcut actions.
Nevertheless, I can reproduce this issue. If I disable Ctrl+Alt+Tab in the system-wide shortcuts, I can still not make it open a menu. After playing around a bit, it seems that the Tab key is somewhat special. I cannot make it open a menu with any modifier key. Maybe that's a bug, or it's a limitation as this key may get consumed by GNOME Shell.
I'll have to investigate :)

The tab key is for some reason not considered to be a valid key for accelerators by Gdk. I just add a check to accept it, now it works :smile:
You can test this in the current develop branch! How to install Fly-Pie from Git is explained here.


sourceMappingURL triggers access to non-existent file

Checklist

[X] The bug is reproducible against the latest release or master.
[X] There are no similar issues or pull requests to fix it yet.

Describe the bug
The .js files specify sourceMappingURL but the .map files are not included in the package. The browser tries to fetch it and gets a 404 error. We should add the source map files or remove the following lines:
https://github.com/aminalaee/sqladmin/blob/daea8c9c385794605ddeb02b7105d71ca9a9823f/sqladmin/statics/js/bootstrap.min.js#L7
Steps to reproduce the bug
No response
Expected behavior
No response
Actual behavior
No response
Debugging material
INFO:     <IP_ADDRESS>:8000 - "GET /js/bootstrap.min.js.map HTTP/1.1" 404 Not Found

Environment

Ubuntu 20.04.4 LTS
Python 3.9.12
SQLAdmin 0.1.11

Additional context
No response

Or just use the CDN link 😀

I think using the CDN has some downsides:

The site will be unavailable if the CDN is down
Security consideration


seems to have issue with _

For some reason when I specify a name with _ in it, it converts it to -
object controlScript : PropertyGroup() {
val APPDYNAMICS_AGENT_UNIQUE_HOST_ID by stringType
}
println(config[controlScript.APPDYNAMICS_AGENT_UNIQUE_HOST_ID] )
Exception in thread "main" com.natpryce.konfig.Misconfiguration: controlScript.APPDYNAMICS-AGENT-UNIQUE-HOST-ID property not found; searched:

controlScript.APPDYNAMICS-AGENT-UNIQUE-HOST-ID in /Users/laipt/Desktop/petels/POC.cicd/cicd/src/main/resources/application-dev.properties

The reflectomagic property group classes represent nesting as nested objects, not in the name of properties.
Konfig expects property names in code to follow Java property name conventions (dotted separators for hierarchy, lower case with hyphens to separate words in identifiers within that hierarchy).  The Configuration implementations map from property-names-in-code to property names in a configuration store.  E.g. the EnvironmentVariables implementation maps Java property name convention to the upper-case-and-underscores convention used for Unix environment variables.  So if your property names are stored with a different naming convention, you can write a Configuration implementation to map to the naming convention of your underlying store.


[stable/nginx-ingress] enhance --publish-service use

Signed-off-by: Shane Nayler<EMAIL_ADDRESS>@jackzampolin
@mgoodness
@chancez
What this PR does / why we need it:
The default deployment needs to have the publish-service option enabled to work, otherwise new ingress resources get assigned a node IP instead of an External IP.
This PR will automatically assign the most likely to be correct publishServicePath automatically, without removing the option to override it if somebody needs to do that.
It may be worth also changing the default in values.yaml to set .Values.controller.publishService.enabled to be true by default. I am happy to add that if it seems necessary to others.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #7595
Special notes for your reviewer:
This has been tested in a GCE cluster, but I do not have other environments to test the changes in. If somebody is able to check it in other non-GCE environments for the unlikely case that there is something quirky about the specific environment that could be causing trouble it may be worth it for peace of mind. See the steps to replicate the issue this PR is attempting to fix in #7595

/assign @linki


Update Icons

there some fixes and updates require in the BitIconName enum and also the Fabric mdl2 icons

not valid anymore


Handoff on Sierra no longer working after update on iMac(17,1) Hackintosh.

OS Version: macOS Sierra 10.12
Mac model: Hackintosh iMac(17,1)
Dongle or replaced card?: Added a PCI-E card
Used dongle/card: Wifi: Broadcom BCM43xx, Bluetooth: Broadcom 20702B0 Chipset

Your Issue:
Handoff is not working, together with Universal Clipboard.
Airdrop and Continuity is working well.
Handoff worked on El Capitan after running the script with -f(because of " Entry, ":Mac-65CE76090165799A:ContinuitySupport", Does Not Exist NOT OK. Unknown state. Your Mac might not be compatible." issue. btw any possibilities to fix this?)
Steps done to try to fix the issue:

Run the beta branch script, not working.
Uninstall and reinstall CAT, not working.
Reboot both devices. Toggle bluetooth and Handoff, on both devices. Log out and log in iCloud. Toggle iCloud Drive.

System Diagnostic Output
--- Initiating system compatiblity check ---

--- Hardware/OS checks ---
Verifying Continuity status...          OK. OS X reports Continuity as active
Verifying Mac model reference...        WARNING. The compatibility of this Mac Model (iMac17,1) with this mod is unknown and may have unpredictable results
Verifying Mac board-id...               OK. Long board id detected: Mac-65CE76090165799A
Verifying OS X version...               Warning: This tool wasn't tested with OS X versions higher than 10.10. Detected OS version: 10.12
Verifying Wi-Fi hardware...             OK. A Broadcom AirPort card is active, and is using the Continuity compatible Brcm4360 kext
Verifying AWDL status...                OK. An AWDL interface is up, Wi-Fi is ready for Continuity
Verifying Bluetooth hardware...         OK. The internal Bluetooth card is active
Verifying Bluetooth version...          OK. Bluetooth 4.0 detected
Verifying Bluetooth features...         OK. Bluetooth features are Continuity compliant
Verifying Bluetooth firmware...         OK. Bluetooth firmware version: v126 c9196
Verifying Login Item...                 OK. Login item for Auto Continuity Check is not set.

--- Modifications check ---
Verifying OS kext protection...         OK. Kext developer mode is not active. This tool can fix this.
Verifying SIP...                        Ok. System Integrity Protection is already disabled
Verifying ContinuitySupport...          Print: Entry, ":Mac-65CE76090165799A:ContinuitySupport", Does Not Exist
NOT OK. Unknown state. Your Mac might not be compatible.
Verifying kexts readability...          OK. Wi-Fi and Bluetooth kexts were found and could be read
Verifying Wi-Fi whitelist status...     OK. The whitelist is correctly patched with your board-id
Verifying BT4 dongles compatibility...  OK. Compatibility with BT4 USB dongles is not enabled, this tool can fix this if a dongle is plugged in
Verifying old Wi-Fi kext presence...    OK. Legacy Wi-Fi driver Brcm4331 was already removed
Verifying legacy Wi-Fi card patch...    OK. The patch is already done. Old Broadcom Wi-Fi cards may work.
--- Modifications check ---

Press any key to go back to the main menu...

Thanks for all these works you guys have done!

I am having the same issue with 17,1 on Sierra. BCM94360CD is used.
iCloud: All syncing between devices nicely.
Airdrop: Works
Personal Hotspot: Works
Apple Watch Unlock: The option was available when the WIFI card was initially installed, but I couldn't get it to work. After playing with it, and reinstall OS etc, the option is no longer shown up.
Handoff: Not working
Universal Clipboard: Not working
Wondering if disabling SIP is necessary.
--- Hardware/OS checks ---
Verifying Continuity status...          OK. OS X reports Continuity as active
Verifying Mac model reference...        WARNING. The compatibility of this Mac Model (iMac17,1) with this mod is unknown and may have unpredictable results
Verifying Mac board-id...               OK. Long board id detected: Mac-65CE76090165799A
Verifying OS X version...               Warning: This tool wasn't tested with OS X versions higher than 10.10. Detected OS version: 10.12
Verifying Wi-Fi hardware...             OK. A Broadcom AirPort card is active, and is using the Continuity compatible Brcm4360 kext
Verifying AWDL status...                OK. An AWDL interface is up, Wi-Fi is ready for Continuity
Verifying Bluetooth hardware...         OK. The internal Bluetooth card is active
Verifying Bluetooth version...          OK. Bluetooth 4.0 detected
Verifying Bluetooth features...         OK. Bluetooth features are Continuity compliant
Verifying Bluetooth firmware...         OK. Bluetooth firmware version: v126 c9198
Verifying Login Item...                 OK. Login item for Auto Continuity Check is not set.
--- Modifications check ---
Verifying OS kext protection...         OK. Kext developer mode is not active. This tool can fix this.
Verifying SIP...                        NOT OK. System Integrity Protection is still enabled
Verifying ContinuitySupport...          Print: Entry, ":Mac-65CE76090165799A:ContinuitySupport", Does Not Exist
NOT OK. Unknown state. Your Mac might not be compatible.
Verifying kexts readability...          OK. Wi-Fi and Bluetooth kexts were found and could be read
Verifying Wi-Fi whitelist status...     OK. The whitelist is incorrectly patched, your board-id wasn't found at the right place. This tool can try to fix this.
Verifying BT4 dongles compatibility...  OK. Compatibility with BT4 USB dongles is not enabled, this tool can fix this if a dongle is plugged in
Verifying old Wi-Fi kext presence...    OK. Legacy Brcm4331 Wi-Fi driver is present. This tool can fix this.
Verifying legacy Wi-Fi card patch...    OK. The legacy Wi-Fi patch is not present. This tool can fix this.
--- Modifications check ---

My iMessage/Facetime is working normally..
@mu2004 Would you please tell me how did you fix your issue? It still worth a try.
Thanks!

Is your WIFI BCM94360? If it's not, it may not be natively supported and some hacking is needed.

Can you check if the latest beta solves this issue?

I'm afraid it is still not working.
I tried the latest beta branch and getting Patching ContinuitySupport...           Usage: PlistBuddy [-cxh] <file.plist>, so I decided to force install the script. After rebooting, log out and log in iCloud and toggling iCloud Drive, Handoff and Instant Hotspot are still not working.
This is all command I have run:
➜  Resources git:(beta) ✗ sudo ./contitool.sh
Password:
--- OS X Continuity Activation Tool 2.4b3 ---
                 by sysfloat
           original by dokterdok

Select an option:

1) Activate Continuity	    4) Uninstall with Recovery
2) System Diagnostic	    5) Disable Auto Check App
3) Uninstall		    6) Quit
#? 1
--- OS X Continuity Activation Tool 2.4b3 ---
                 by sysfloat
           original by dokterdok

OS X reports Continuity as active.
Patching ContinuitySupport...           Usage: PlistBuddy [-cxh] <file.plist>
    -c "<command>" execute command, otherwise run in interactive mode
    -x output will be in the form of an xml plist where appropriate
    -h print the complete help info, with command guide

Usage: PlistBuddy [-cxh] <file.plist>
    -c "<command>" execute command, otherwise run in interactive mode
    -x output will be in the form of an xml plist where appropriate
    -h print the complete help info, with command guide

OK.

Press any key to reboot or CTRL-C to cancel...^C
➜  Resources git:(beta) ✗ sudo ./contitool.sh -f

--- Initiating Continuity mod ---

Disabling OS kext protection...         OK
Fixing disk permissions...              OK
Backing up original drivers...          Would you like to overwrite the existing backup found in /Users/JMSuen/KextsBackupBeforePatch?
1) Yes, overwrite
2) No, skip this backup
#? 2
Skipping backup...                      OK
Skipping blacklist patch...             OK
Verifying BT4 dongle patch status...    OK
Setting HCI switch behavior...          OK
Patching Bluetooth feature flags...     OK
Applying legacy Wi-Fi card patch...     OK
Skipping whitelist patch...             OK
Skipping old Wi-Fi driver clean up...   OK
Applying legacy Wi-Fi card patch...     OK
Patching ContinuitySupport...           Set: Entry, ":Mac-65CE76090165799A:ContinuitySupport", Does Not Exist
OK.
Updating kext caches...                 OK
Updating system caches...               OK
Backing up patched drivers...           Would you like to overwrite the existing backup found in /Users/JMSuen/KextsBackupAfterPatch?
1) Yes, overwrite
2) No, skip this backup
#? 2
Skipping backup...                      OK
Do you want to enable a Automatic check for Continuity each boot?
1) Yes
2) No
#? 2
OK.

ALMOST DONE! After rebooting:
1) Make sure that both your Mac and iOS device have Bluetooth turned on, and are on the same Wi-Fi network.
2) On OS X go to SYSTEM PREFERENCES> GENERAL> and ENABLE HANDOFF.
3) On iOS go to SETTINGS> GENERAL> HANDOFF & SUGGESTED APPS> and ENABLE HANDOFF.
4) On OS X, sign out and then sign in again to your iCloud account.
Troubleshooting: support.apple.com/kb/TS5458
After verifying that Continuity works, you can reenable SIP via the Recovery OS

Thanks to Lem3ssie, UncleSchnitty, Skvo, toleda, TealShark, Manic Harmonic, rob3r7o, RehabMan, kramsee and the many beta testers for their support.
Updated by sysfloat

Press any key to reboot or CTRL-C to cancel...

Thanks again.

@JiamingSuen thanks for testing. Can you please try one more time? I just pushed beta 2.4b4.

Yes I believe this time the patching is done, but after rebooting and other toggling stuff it is still not working.. I even tried with a different Wifi router.
➜  Resources git:(beta) ✗ git pull
remote: Counting objects: 6, done.
remote: Compressing objects: 100% (5/5), done.
remote: Total 6 (delta 0), reused 0 (delta 0), pack-reused 0
Unpacking objects: 100% (6/6), done.
From https://github.com/dokterdok/Continuity-Activation-Tool
   67c0ef1..0c72056  beta       -> origin/beta
Updating 67c0ef1..0c72056
Fast-forward
 Continuity Activation Tool.app/Contents/Resources/contitool.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
➜  Resources git:(beta) ✗ sudo ./contitool.sh
Password:
--- OS X Continuity Activation Tool 2.4b4 ---
                 by sysfloat
           original by dokterdok

Select an option:

1) Activate Continuity	    4) Uninstall with Recovery
2) System Diagnostic	    5) Disable Auto Check App
3) Uninstall		    6) Quit
#? 1
--- OS X Continuity Activation Tool 2.4b4 ---
                 by sysfloat
           original by dokterdok

OS X reports Continuity as active.
Patching ContinuitySupport...           OK.

Press any key to reboot or CTRL-C to cancel...

Could you check SystemParameters.plist located in /System/Library/Frameworks/IOBluetooth.framework/Versions/A/Resources/ ? Your Mac board-id should be in there with a bool for "ContinuitySupport" set to Yes.

Is this the bool you mentioned?
<key>Mac-65CE76090165799A</key>
	<dict>
		<key>ContinuitySupport</key>
		<true/>
	</dict>

Well there's no "Yes" in this file. I attached the file to this gist in case you need to have a look.
Thanks.

Yes that's the one. It gets shown as "Yes" in xcode. So all patches should be applied then.. I might have to set up a hackintosh myself and test some things out. Will keep you updated.

OK thanks a lot!
It's quite weird that it worked before with the same hardware and OS configuration.. And other people's Sierra is working fine..

Weird. You could try to remove your iCloud account from your devices and try again. This usually fixes the issue for me.

Well, something more weird happened..
I did nothing since my last attempt with the latest beta script and everything suddenly got working this afternoon. Sorry I cannot provide more information about how exactly it got working but thanks a lot for your patience @sysfloat .

Yeah most likely just the iCloud sync taking some time, you can force this by removing and re-adding the account on your Mac.
Good to hear it's finally working, so it was the ContinuiySupport not being patched after all :)


Adding functionality for validator hub passthrough of hosted ML endpoints

Copied some hub token auth code from the CLI as @CalebCourier mentioned this will be removed at some point. I did this because the imports need to be reformatted, they can't be called from cli due to it causing circular imports.
Figured adding a copy was better than refactoring just to refactor again later. Can revert to reformatting all of them if needed

Yeah that's fine for now.  I think long term we need to split the open source core package and the cli anyway so this would either need to be duplicated or the cli would import this new file you created instead.


Chip: Styling issues with SlottedSVG

Summary of the bug
When trying to add SlottedSVG to a React example of  neighbouring components are pushed away due to the internal  being 300x150px

Or if you use width, height and viewbox, the icon gets cut off:

[ ] Fix SlottedSVG display on Chips, add a React Story for it.
[ ] Update Chip page on Guidance site to show an example with SlottedSVG

🪜 How to reproduce
In the react/ic-chip.stories.mdx add some Chips with SlottedSVG components:
import { SlottedSVG } from "../react-component-lib/slottedSVG";

import { mdiAccountCircle } from "@mdi/js";
<Canvas>
  <Story name="Static">
      <IcChip label="Small" size="small">
      <SlottedSVG
        slot="icon"
        path={mdiAccountCircle}
        aria-label="account"
      />
    </IcChip>
    <IcChip label="Default">
      <SlottedSVG
        slot="icon"
        path={mdiAccountCircle}
        aria-label="account"
      />
    </IcChip>
    <IcChip label="Large" size="large">
      <SlottedSVG
        slot="icon"
        path={mdiAccountCircle}
        aria-label="account"
      />
    </IcChip>
    ... (the rest of the chips in the static story)

Adding width, height and viewbox props will result in the other display issue:
<IcChip label="Small" size="small">
      <SlottedSVG
        slot="icon"
        path={mdiAccountCircle}
        aria-label="account"
        height="20px"
        viewBox="0 0 20 20"
        width="20px"
      />
    </IcChip>

🧐 Expected behaviour
SlottedSVG should behave the same as regular SVG icons.

Not sure this is really an issue. The  image can be displayed by setting a viewBox of "0 0 24 24" on the SlottedSVG:

note: the icon is not exactly the same size as non-slotted, as, despite being visually the same icon, the svg content is different:
mdi icon:
<path d="M12,19.2C9.5,19.2 7.29,17.92 6,16C6.03,14 10,12.9 12,12.9C14,12.9 17.97,14 18,16C16.71,17.92 14.5,19.2 12,19.2M12,5A3,3 0 0,1 15,8A3,3 0 0,1 12,11A3,3 0 0,1 9,8A3,3 0 0,1 12,5M12,2A10,10 0 0,0 2,12A10,10 0 0,0 12,22A10,10 0 0,0 22,12C22,6.47 17.5,2 12,2Z"></path>
inline svg in story:
<path d="M10 0C4.48 0 0 4.48 0 10C0 15.52 4.48 20 10 20C15.52 20 20 15.52 20 10C20 4.48 15.52 0 10 0ZM10 3C11.66 3 13 4.34 13 6C13 7.66 11.66 9 10 9C8.34 9 7 7.66 7 6C7 4.34 8.34 3 10 3ZM10 17.2C7.5 17.2 5.29 15.92 4 13.98C4.03 11.99 8 10.9 10 10.9C11.99 10.9 15.97 11.99 16 13.98C14.71 15.92 12.5 17.2 10 17.2Z"></path>
In all our other use cases\examples of SlottedSVG there seems to be viewBox value set.
An enhancement could be made to default SlottedSVG to apply a viewbox of "0 0 24 24", but this could constitute a breaking change for users, if they are not currently setting a viewBox value


When is the repeater going to be available?

Describe the feature
When is the repeater going to be available?

apparently next moth or so

Do you have a recommendation in advance of the repeater field being ready? Use case is the ability for a user to add n staff members with various property attributes such as Name, Title, Role.

Hi @edwardsf — our roadmap is here: https://formkit.com/about. The repeater will be released with the first batch of Pro Inputs, which we're hoping to release by the end of Q3, 2022 (Sep 30).
In the meantime, you can build your own repeater with the existing primitives! Here's an example: https://formkit.link/b7cc2c4ae2b02807065d6617ead62783 which you can also find on the examples page.

@luan-nk-nguyen Thanks for the example, was super helpful.


sdk automation, support breakingChangeItems

Description

design: https://github.com/Azure/azure-sdk-tools/issues/9222#issuecomment-2443277032
test

mgmt swagger: https://github.com/test-repo-billy/azure-rest-api-specs/pull/3382/checks?check_run_id=32263329870
mgmt typespec: https://github.com/test-repo-billy/azure-rest-api-specs/pull/3381/checks?check_run_id=32263350223
dpg typespec: https://github.com/test-repo-billy/azure-rest-api-specs/pull/3383/checks?check_run_id=32263357110

Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

I'm trying to understand below part's intension.

Are we making it easier for users to add suppression? Do we recommend users to add suppresion?

https://github.com/test-repo-billy/azure-rest-api-specs/pull/3382/checks?check_run_id=32263329870

Yeah, user should add suppression to the breaking change items:

Suppression doc:
https://github.com/Azure/azure-rest-api-specs/blob/main/documentation/sdkautomation/sdk-suppressions.md#how-to-suppress-the-sdk-breaking-changes

For changelog format, please follow https://azure.github.io/azure-sdk/policies_releases.html#change-logs
According to the policy, In order for consistency across our SDKs and to enable automation to validate and parse these changelogs the files must use the following format.
This is the only format allowed
### Breaking Changes
- <for changes to be called out in release notes including>

But your changelog in another formats
info	[Changelog] ### Breaking Changes
info	[Changelog] #### `models.VirtualMachineInstances` was modified
info	[Changelog] * `stop(java.lang.String,models.StopVirtualMachineOptions)` was removed
info	[Changelog] #### `models.Checkpoint` was modified
info	[Changelog] * `withName(java.lang.String)` was removed
info	[Changelog] * `withDescription(java.lang.String)` was removed
info	[Changelog] * `withCheckpointId(java.lang.String)` was removed
info	[Changelog] * `withParentCheckpointId(java.lang.String)` was removed
info	[Changelog] #### `models.StorageQosPolicy` was modified
info	[Changelog] * `withBandwidthLimit(java.lang.Long)` was removed
info	[Changelog] * `withIopsMinimum(java.lang.Long)` was removed
info	[Changelog] * `withIopsMaximum(java.lang.Long)` was removed
info	[Changelog] * `withId(java.lang.String)` was removed

LGTM

add a comment for changelog format for test result: https://github.com/azure-sdk/azure-sdk-for-java/pull/13054/files#r1827510978

Ah, it's an older run and is fixed in the latest. See https://github.com/azure-sdk/azure-sdk-for-java/pull/13130/files

/check-enforcer override