Revise archive cosmetics and bulk CSV import

Reviewer: @sidravi1
Estimate: ~20 minutes

Ticket
Fixes: N/A
Description
Goal

Update frontend cosmetics for archive icon and message.
Update bulk CSV import.

Changes

Changed ArchiveOutline button back to Delete button.
Changed "delete" message.
Allow users to re-upload a CSV file that contains entries that were previously archived.

Future Tasks (optional)
How has this been tested?

Example demonstration

To-do before merge (optional)
Checklist
Fill with x for completed.

[X] My code follows the style guidelines of this project
[X] I have reviewed my own code to ensure good quality
[X] I have tested the functionality of my code to ensure it works as intended
[X] I have resolved merge conflicts

(Delete any items below that are not relevant)

[X] I have updated the automated tests

Last commit adds a try/except when deleting contents with feedback. This change was suggested by Carlos as a way to inform users who want to use the backend Delete endpoint to permanently delete content.


Fix #276 Streaming subscription throws connection error

Fixes issues with subscriptions (e.g. streaming) and now uses a PoolingHttpClientConnectionManager instead of BasicHttpClientConnectionManager.
Specifically, it will create a separate PoolingHttpClientConnectionManager for subscriptions while still retaining the BasicHttpClientConnectionManager for all other connections.

Hi @avromf, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;

@avromf, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, AZPRBOT;

@avromf can you provide more information on the incident? Are there any existing issues corresponding to this (if not will you create one)?

Yes, it is in the title, issue #276.

In a nutshell, my fix here follows @MikeN123 's suggestion in the comments on issue #276 that a separate pooling manager should be created for subscriptions.

Please note that the PoolingHttpClientConnectionManager only creates 2 connections per route by default, so you can only have 2 subscriptions at the moment (I think). Maybe that should be fixed, other than that it seems like an ok workaround to me (but I did not look at the PR in detail).
A better solution would be to rewrite the complete connection handling, but that's would probably be quite some work.

@MikeN123 I went back and tested this by creating more than 2 subscriptions and you are correct. It hits the default limit of 2.
I also went back and took a look at the original code before the change to BasicHttpClientConnectionManager. It also had the same limitation.
Should we increase the limit and if so, what should we increase it to?

10 would probably be ok, with an option to change it, since that is also the default value for Exchange (HangingConnectionLimit setting)

@avromf are you planning any further changes on this?

just so you guys know I just verified this appears to have fixed this problem for us

i'm happy to wrap this up if @avromf is unable to at this time (he did all the hard work already).  is the only concern the connection pool?

I'm currently busy today and tomorrow, but I may be able to take care of it next week.
Setting the connection pool to increase the limit beyond 2 is not an issue, my only concern is @MikeN123 suggestion to have an option to change it. How is that supposed to be done? Via a method call?

A method call should be ok (and maybe an overloaded constructor for convenience?). I would also increase the defaults to 10 or something. Docs on how to set the limits can be found here: https://hc.apache.org/httpcomponents-client-4.4.x/tutorial/html/connmgmt.html#d5e393

Even though a number of 10 seems some kind of random to me this should be more usefull then the restricted value of 2. Once added we should also include an update to the readme/wiki to mention the possible change of the connection size.

10 was not really randomly chosen, it's the default max concurrency of EWS. But the streaming subscriptions seem to use another limit (HangingConnectionLimit), which seems to have a default of 3 according to https://msdn.microsoft.com/en-us/library/office/jj945066(v=exchg.150).aspx#bk_ThrottlingNotifications

@MikeN123 The only thing I was wondering about is that it was also suggested that the implementation be completely written. Assuming this is actually done some time in the future, that may possibly make any API changes (like the addition of a method to set the number of connections in the pool) obsolete.
Should we then simply leave it at max 10 since the chances that someone may need to change it are rare?

It's probably a small effort to add 1 method that gives the opportunity to change the maximum, so why not add it then?

@avromf any update on this? Are there any questions left or are you just busy?

No, no more questions, just currently busy.

@avromf maybe we should get this in shortly. If you dont have time @kentongray should support as offered.

@avromf this PR is now open for over 1 month. I would love to have this integrated in stable 2.0 release. So if you want your name on this change please have this PR updated and apply the wanted changes. If I dont notice any changes here within the next days I will address this issue on my own and close this PR due to lag of contribution.

Sorry for the delay, @serious6 . I've now set the default number of connections for the pooling manager to 10 and added a method allowing one to change it if needed.
I've also merged it with the latest. (I hope I did that correctly.)

@avromf thanks for the work. Unfortunately your commits also include some conflict resolution which we should not have on the upstream/master.Maybe you can reset these commits resolve the conflicts and only commit the changes made? If you are facing any trouble plz report back.

While in my attempt to fix this, this pull request ended up being closed thus I have created a new one.
#336


After setting hostNetwork to true, mpi does not work

I set hostNetwork to true for the mpi job, but the new ip is not used when the MPI is actually executed, resulting in no communication between the worker pods.
I found the mapping between the physical machine ip and the worker pod name in /opt/kube/hosts of the launcher pod, but there is no such mapping relationship in the worker pod. I doubt this is the reason why MPI is not using the new ip?
Can anybody help me？

Could you confirm the mapping file is located at /opt/kube/hosts? From
the mpi-controller in the master branch, I think the controller does not
mount such a file.
Given the circumstance that the real ip of a pod will only be assigned
after the pod is scheduled, it seems quite possible that the
kube-delivery image does the job. In this case, could you confirm the
image (version/tag) of the kube-delivery, which is the image used by the
init-container of the launcher pod.
On Mon, Sep 5, 2022 at 4:24 PM Yongmin Hu @.***> wrote:

I set hostNetwork to true for the mpi job, but the new ip is not used when
the MPI is actually executed, resulting in no communication between the
worker pods.
I found the mapping between the physical machine ip and the worker pod
name in /opt/kube/hosts of the launcher pod, but there is no such mapping
relationship in the worker pod. I doubt this is the reason why MPI is not
using the new ip?
Can anybody help me？
—
Reply to this email directly, view it on GitHub
https://github.com/kubeflow/training-operator/issues/1657, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AK7V6ISQYXP43ILWCABNKQTV4WU4ZANCNFSM6AAAAAAQEYDFCI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

The problem can be solved by removing "-mca pml ob1 -mca btl ^openib" parameter in the mpirun command.
Thanks!


Comments in begin of policy-file throws Exception

All xml policy-files, that have a comment at the beginning throws an excpetion:
<?xml version="1.0" encoding="UTF-8"?>
 <!--This file was generated by the ALFA Plugin for Eclipse from Axiomatics AB (http://www.axiomatics.com).
 Any modification to this file will be lost upon recompilation of the source ALFA file-->
<xacml3:PolicySet ...

com.att.research.xacml.std.dom.DOMStructureException: Exception loading Policy file "C:\...\src\main\resources\XACML\policy.xml": Exception parsing Policy: Unexpected element "null"
	at com.att.research.xacmlatt.pdp.policy.dom.DOMPolicyDef.load(DOMPolicyDef.java:125) ~[xacml-pdp-1.0.0.jar:na]
	at com.att.research.xacmlatt.pdp.std.StdPolicyFinderFactory.loadPolicyDef(StdPolicyFinderFactory.java:84) [xacml-pdp-1.0.0.jar:na]
	at com.att.research.xacmlatt.pdp.std.StdPolicyFinderFactory.getPolicyDefs(StdPolicyFinderFactory.java:148) [xacml-pdp-1.0.0.jar:na]
	at com.att.research.xacmlatt.pdp.std.StdPolicyFinderFactory.init(StdPolicyFinderFactory.java:195) [xacml-pdp-1.0.0.jar:na]
	at com.att.research.xacmlatt.pdp.std.StdPolicyFinderFactory.getPolicyFinder(StdPolicyFinderFactory.java:221) [xacml-pdp-1.0.0.jar:na]
	at com.att.research.xacmlatt.pdp.std.StdEvaluationContextFactory.getPolicyFinder(StdEvaluationContextFactory.java:78) [xacml-pdp-1.0.0.jar:na]
	at com.att.research.xacmlatt.pdp.std.StdEvaluationContextFactory.getEvaluationContext(StdEvaluationContextFactory.java:149) [xacml-pdp-1.0.0.jar:na]
	at com.att.research.xacmlatt.pdp.ATTPDPEngine.decide(ATTPDPEngine.java:188) [xacml-pdp-1.0.0.jar:na]

Caused by: com.att.research.xacml.std.dom.DOMStructureException: Exception parsing Policy: Unexpected element "null"
	at com.att.research.xacmlatt.pdp.policy.dom.DOMPolicyDef.newInstance(DOMPolicyDef.java:64) ~[xacml-pdp-1.0.0.jar:na]
	at com.att.research.xacmlatt.pdp.policy.dom.DOMPolicyDef.load(DOMPolicyDef.java:123) ~[xacml-pdp-1.0.0.jar:na]

Caused by: com.att.research.xacml.std.dom.DOMStructureException: Unexpected element "null"
	at com.att.research.xacml.std.dom.DOMUtil.newUnexpectedElementException(DOMUtil.java:151) ~[xacml-1.0.0.jar:na]
	at com.att.research.xacmlatt.pdp.policy.dom.DOMPolicyDef.newInstance(DOMPolicyDef.java:61) ~[xacml-pdp-1.0.0.jar:na]

Hi, thanks I will take a look at this. How are you testing this in order to generate the error?

properties props = new Properties();
props.setProperty(XACMLProperties.PROP_ROOTPOLICIES, "policy");
props.setProperty("policy.file", "src/main/resources/XACML/policy.xml");
PDPEngine engine = PDPEngineFactory.newInstance().newEngine(props);

Request request = //simple XAML request
Response response = engine.decide(request);

If I remove the comment from policy file the request will be responed without error.

Hi, any news on this? I am facing exactly the same problem.

I've seen this issue as well. I have submitted the following pull request https://github.com/att/XACML/pull/38

@benkuly @g-spot should be fixed now


Azure Repos that have a space in name don't work with infracost comment

Problem
In the Infracost Azure DevOps integration, when $(Build.Repository.Uri) has a space in the repo name, e.g. CloudSolutions Playground, Azure DevOps automatically encodes the space as %20 and outputs something like<EMAIL_ADDRESS>That then causes the following error:
$ infracost comment azure-repos --path=/tmp/infracost.json \
                                        --azure-access-token=$(System.AccessToken) \
                                        --pull-request=$(System.PullRequest.PullRequestId) \
                                        --repo-url=$(Build.Repository.Uri) \
                                        --behavior=update

Error: The pull request comment was generated successfully but could not be posted:
Error creating comment: 404 Not Found

Even when we use --repo-url="$(Build.Repository.Uri)" (notice the double quotes), the same issue happens.
Proposed solution
The infracost comment could detect that --repo-url has spaces that are encoded already and decode them.
Thanks to Armand Seukunian from the community chat for pairing with us to debug this tricky issue! 🚀

This is now fixed in the v0.10.9 release.


Add FTPS support

I already said UI created.
confirm your code, now I don't see differences.
it is not a complete FTPS is only waring a secure option.
how selected ssl and tls?


Fixed a few typos

Fixed typos in "Converting a PDF file to text" doc

After creating this PR, it looks like it's quite similar to #611 and #579

Closing because typos are indeed already solved in other MR's. But thanks for taking the effort!


Python wrapping

The time as come...
Following some documentation on the different way to wrap Python around C++ code, two approaches are possible: SWIG, Cython, cppyy
All have pros and cons, the issues is to discuss what is the best for us.

SWIG
I started to play a bit with SWIG here. I have extracted a minimal part of CLIc which include a GPU class (platform, device, context and CQ) and a Buffer class (OCL buffer and size, datatype info) and to test simple push pull and create command. No kernel execution yet.
For now only a push and create method exist and is compatible with numpy.
main first issues:

Overwrite of the same function (e.g. push) for 1d, 2d and 3d numpy array or type does not work. This force an extract python level function which call the correct C++ function depending on the numpy shape.
Generate a class interface cle with all method push, pull, etc VS cle module which containt a vast list of functions.
Numpy shape is (z, y, x)
Do we try to link the C++ side with the top layer of pyClesperanto (complexe but allows to switch to pyopencl back-end if wanted) or not?

Cython
to do ... open for advise on a nice starting tutorial / repo compatible with cmake

Hey @StRigaud !
Just a note that the effort to add ITK bindings for clEsperanto is now active :-). The code is here, and this will generate SWIG-based bindings and cross-platform, binary Python packages.
For more Python bindings, pybind11 is worth a look. After working with SWIG, Cython for a while it is the best option, in my humble opinion. There are tools / tutorials for bridging with CMake along with tools for CMake + Cython here: https://github.com/scikit-build

That actually pretty cool!!! Do not hesitate to poke me if I can be of any help! Or if you see any issue / possible improvement to do on my side that can be useful.
In the end I will test all wrapper I think. Mainly because I need to see which would fit the best to work with whats already in place in pyClesperanto. And also because it's pretty cool tech.

PyBind11
sandbox prototype here. Clearly not my prettiest nor cleanest coding BUT it works.
Globally pretty cool though requires more effort for integration.

CMake compatible
Numpy and stl compatible
The numpy interface is clean and easy to use
One .cpp file for one module for one .so output deal
Wrap around classes and functions
Need some extra c++ coding
The doc is complete but remain minimalist

It is a pretty cool wrapper lib, though I do not grasp all of the possibility, I found it very cool to learn and easier to understand (hence to do what you want to do).

I have dropped a first prototype of python wrapper using pybind11 and scikit-build.
Curious also to try with Cython before committing to one. So most likely a new branch coming up soon on this.
pybind11 is more C++ oriented. The interfacing is done in C++ though the manipulation of some python C object.

It is more simple for me in a sense that i understand a bit more what am i doing.
But because C++ maybe less people will be able to interact and help me improve it.

cython is more python oriented. The interface is done in python with pythonized C object.

More code to write and more effort on my side.
But python aspect make it maybe a bit easier for people to join and help.

In addition to these points my other criteria would be:

maintainability, which one has more chance to "survive" time and evolution.
easiness for using both C++ stl and python list, dict, numpy, etc.
speed


T820 unlocker linux support

Are you planning to add linux support for the t820 unlocker?
Or is there another way with wine or vm?
I know this doesnt belong here, but I couldnt find the issues tab in the t820 unlocker repo.

Nevermind I found a windows pc


FALCOSECURITY_LIBS update: sinsp_plugin::version to sinsp_version

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build
/area engine

/area rules

/area tests

/area proposals

What this PR does / why we need it:
Update falcosecurity/libs to https://github.com/falcosecurity/libs/pull/313 and make it compile
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

bump libs version to 4015ea7179263541d8e2448e8247f881667289a1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: deepskyblue86
To complete the pull request process, please assign jasondellaluce after the PR has been reviewed.
You can assign the PR to them by writing /assign @jasondellaluce in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hey @deepskyblue86 I think this has already been addressed in https://github.com/falcosecurity/falco/pull/1999!

@jasondellaluce :sweat_smile: :kissing_cat:


pip-style authentication when publishing

Pull Request Check List
This PR leverages the pip-style authentication for poetry publish.
This change has been tested with Google Cloud.
Feel free to reach out to me to discuss more.
Resolves: N/A

[ ] Added tests for changed code.
[ ] Updated documentation for changed code.

I like your more general solution. I'll close this now.


cant save configs?

when i try to save a config it crashed the game

Try to update the DirectX SDK before 2010 and again compile cheat

Be sure to be putting a name in the config line before pressing save. If this doesn't work follow these steps.
Open "Client.cpp" > Go to lines 29-32 > Replace the "\" with the directory of where you want to save configs and other output files (IE: "C:\Users\NAME\Desktop\Cheats" be sure it is "C:\" and not "C:") > Now go to line 66-68 and do the same, example below:
	[SPOILER]string ConfigDir = BaseDir + "C:\\Users\NAME\Desktop\Cheats" + "*.ini";
	GuiFile = BaseDir + "C:\\Users\NAME\Desktop\Cheats" + "gui.ini";
	CreateDirectoryW(L"C:\\Indigo", NULL); [/SPOILER]

Hopefully this helps out.

Be sure to be putting a name in the config line before pressing save. If this doesn't work follow these steps.
Open "Client.cpp" > Go to lines 29-32 > Replace the "\\" with the directory of where you want to save configs and other output files (IE: "C:\\Users\NAME\Desktop\Cheats" be sure it is C:\\ and not C:\ > Now go to line 66-68 and do the same, example below:
                string ConfigDir = BaseDir + "C:\\Users\NAME\Desktop\Cheats" + "*.ini";
		GuiFile = BaseDir + "C:\\Users\NAME\Desktop\Cheats" + "gui.ini";
		CreateDirectoryW(L"C:\\Indigo", NULL);

Hopefully this helps out.


Make fakes more real

We had some code to generate our Fake KtFile that just make them differ from the real ones without a good reason. This PR changes that. Now our tests are more similar to the real code that we executes.
The main reason to this change is that there was no way to have a KtFile with a basePath and a Path. The next PR (#6830) will use these changes to add missing tests on our code.

I'm seeing weird failures after this change @BraisGabin like the one on:

https://github.com/detekt/detekt/pull/6862
which is just a file renaming.

Do you know what might be happening here?

I filled #6869 to fix the reason that make this happen. Now I'll create a PR to fix the failing tests.


feat: add combobox multiple select with delete item icon

I added a new example to the combobox component in story, which displays selected items above the input and allows each item to be removed. I thought this example was necessary to better showcase other uses of this component

Thanks @no2ehi . However we have a better example here. It caters for the acccessibility as well

Thanks for reminding me of this.


Add Dockerfile

This enables building a Docker image with aptly-cli in it:
$ docker build -t aptly-cli .
...

$ alias aptly-cli='\
    docker run \
        -v ~/.config/aptly-cli/aptly-cli.conf:/etc/aptly-cli.conf \
        -it --rm --name=aptly-cli \
        aptly-cli'

$ aptly-cli repo_package_query --name develop --query 'Name (~ koala)'
Pall python-koala 0.0.41-1 7e42160a4f2f122f
Pall python-koala 0.0.50-1 4f03e421bcc6eaf5
Pall python-koala 0.0.42-1 aaa61514e74d89cf

You could even perhaps set up a Docker Hub automated build using this Dockerfile so that folks can them simply pull your sepulworld/aptly-cli image to get going.
$ alias aptly-cli='\
    docker run \
        -v ~/.config/aptly-cli/aptly-cli.conf:/etc/aptly-cli.conf \
        -it --rm --name=aptly-cli \
        seuplworld/aptly-cli'

Coverage remained the same at 92.403% when pulling d8df8fcfd3e48bda326646a5b8e4513749f0d1b2 on msabramo:Dockerfile_for_aptly-cli into 91a1bf42e49cf84c7853bae8b2e89fd13cef3db6 on sepulworld:master.

This is great! Ok, I am checking out the Docker Hub automated build. Thanks for your help!

I added the auto build settings to sepulworld/aptly_api in Docker Hub.  It is building on 'latest' now.

Glad you like it! Thanks for sharing aptly_cli in the first place and for creating the image on Docker Hub!


Rogue Sod Sim Crusader bug

Crusader on a recent update went from tracking offhand and mainhand as seperate buffs to for some reason tracking as a single buff. Thus giving people the wrong impression on enchants etc

64%ish uptime with both 51%ish uptime main 31%ish offhand  its just not tracking separately in a recent update

This should be fixed. Thanks for reporting it!


Reword the Sentence on the Crisis and Grant Request Page

Re-word it to “Document supporting the remaining amount if total owed is more than total requested, (ex: Total owed is $3,000, HIF request is $2,500, how will remaining $500 be covered? SOS, HTSV, applicant?)

Duplicate.


Search bar visuals

This pr is just for the visuals of the search 'dummy button' and my next pr will focus on the wiring between emission & eigen.

          1 Error

:no_entry_sign:
No CHANGELOG added.

          1 Warning

:warning:
The following files do not have tests: lib/components/home/search_bar.js, lib/containers/home.js & lib/native_modules/switch_board.js

  Generated by :no_entry_sign: danger

@mennenia ready for review!

I’m taking this :+1:
@sarahscott The search icon assets for 1x and 3x are missing, please don’t forget to add those!


draggable image

Hello,
I put an image inside the draggable, it doesn't move according to the edges when I enlarge it by holding the edges. how can i handle photos or how can i solve it?

Try to set width and height of image the same as width and height of the draggable


Citeform: display bugs for references

Originally on 2012-04-30
Reported by Michael Peskin.  Example: document 884003.
Note that references 1c, 8, 10, 23 and 24 look fine on the detailed record references tab.
Click on the link to go to citeform.

Notice that in citeform mode to display things, all of the lines mentioned above (the 3rd '1', 8, 10, 23 and 24) display incorrectly.  Michael was confused when trying to update his citations about why they looked wrong.

It wasn't obvious that updating the eprint number in the edit box on the left, immediately updated the display of the title on the right. But it probably should.

more mouseover help.

see https://github.com/inveniosoftware/invenio-csl-rest


unsupported operand type(s) for +: 'int' and 'str'

Sometimes I get this error after using an Azure CLI extension on an on-premise environment:
TypeError: unsupported operand type(s) for +: 'int' and 'str'
This is the extension if it is required, but the error is not explicitly caused by the extension:
https://github.com/Azure/azure-cli-extensions/tree/main/src/connectedk8s
The stacktrace goes into an Azure CLI code here:
https://github.com/Azure/azure-cli/blob/d005c8fcad763d56d8dbf3da90d8e63bcba234b5/src/azure-cli-core/azure/cli/core/auth/util.py#L161
The fix for the above is to check that a string is not being added to an int for request_time + token_entry["expires_in"]. Could token_entry["expires_in"] be an empty string? This error happens once after a while, so I would imagine it is caused by token or context being refreshed. After the error happens, if I submit the command again, it moves past the error.
Describe the bug
The error can be caused by any command that calls util.py line 161 linked above. Again, it is not related to the command I am showing as an example!
Command Name
az ad sp create-for-rbac -n "my-spn" --role owner --scopes "/subscriptions/<subscription ID>" --create-cert
az connectedk8s delete

But the error happens with az connectedk8s connect, and I'd imagine other extensions as well. Again, as stated above, I think this error is caused by the code in Azure CLI, not the extension. I am only putting this here because GitHub issue asked for it.
Errors:
The command failed with an unexpected error. Here is the traceback:
unsupported operand type(s) for +: 'int' and 'str'
Traceback (most recent call last):
  File "/opt/az/lib/python3.10/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 718, in _run_job
    return cmd_copy.exception_handler(ex)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/role/commands.py", line 51, in graph_err_handler
    raise ex
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/role/custom.py", line 1173, in create_service_principal_for_rbac
    existing_sps = list(graph_client.service_principal_list(filter=query_exp))
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/role/_msgrpah/_graph_client.py", line 179, in service_principal_list
    result = self._send("GET", "/servicePrincipals" + _filter_to_query(filter))
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/role/_msgrpah/_graph_client.py", line 52, in _send
    r = send_raw_request(self._cli_ctx, method, url, resource=self._resource, uri_parameters=param,
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/util.py", line 987, in send_raw_request
    token_info, _, _ = profile.get_raw_token(resource)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/_profile.py", line 383, in get_raw_token
    sdk_token = credential.get_token(*scopes)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/auth/msal_authentication.py", line 99, in get_token
    return build_sdk_access_token(result)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/auth/util.py", line 161, in build_sdk_access_token
    return AccessToken(token_entry["access_token"], request_time + token_entry["expires_in"])
TypeError: unsupported operand type(s) for +: 'int' and 'str'
To check existing issues, please visit: https://github.com/Azure/azure-cli/issues
To open a new issue, please run `az feedback`

To Reproduce:
I used the connectedk8s delete command, but it happens with connect command as well. The error only happens once in a while. If it does happen, resubmitting the command again causes it to work.

az connectedk8s delete --name {} --resource-group {}
az ad sp create-for-rbac -n "my-spn" --role owner --scopes "/subscriptions/<subscription ID>" --create-cert

Expected Behavior
Should not throw unsupported operand types error.
Environment Summary
Windows-10-10.0.17763-SP0
Python 3.10.8
Installer: MSI

azure-cli 2.44.0 *

Extensions:
account 0.2.5
connectedk8s 1.3.4

Dependencies:
msal 1.20.0
azure-mgmt-resource 21.1.0b1

Additional Context
I am using an on-premise environment.

route to CXP team

Adding Service team to look into this.
@akashkeshari Could you please look into this once you get a chance ?

@navba-MSFT The error is likely not be related to Azure CLI extension connectedk8s since the error happened within the context and token code inside Azure CLI. The extension may be a red-herring because that's just the command I used that caused the error. Regardless, it might be a good idea to make sure you're adding integers and not integers with strings at the point where the addition operation happens.

request_time must be an int:
https://github.com/Azure/azure-cli/blob/8474a10fc16b5a168f96dbbe0fd94afc8eecdf4e/src/azure-cli-core/azure/cli/core/auth/util.py#L149
It should never be cast to str, so https://github.com/Azure/azure-cli/pull/25566 is not correct.
When refreshing the access token, AAD returns expires_in as an int.
https://github.com/AzureAD/microsoft-authentication-library-for-python/blob/38001a4d087e5d3ec9b77f4096d52d76ab359c46/msal/oauth2cli/oauth2.py#L244

Then the access token entry bubbles all the way up to
https://github.com/Azure/azure-cli/blob/21266e02457263420b62fba1c529f69e28852414/src/azure-cli-core/azure/cli/core/auth/msal_authentication.py#L69
There is no logic in MSAL or Azure CLI to convert expires_in to str.
@TheOnlyWei, are you authenticating against public Azure? The most possible reason I guess is your authentication endpoint returned expires_in as str.

It was decided to fix this issue on the service side. This was caused by an edge case where we used "/adfs" ending for the login endpoint for an environment with a custom authority that is not well known in order to avoid validating instance discovery or validate authority.


UBM training guidelines

Can you explain to me how to train the UBM on my dataset?
I know we can use testbench/train-ubm.py to train the ubm model.
I want to know:

what's the use of adapt-ubm.py?
how to setup the directory structure of the data set for training.
why does read_raw_data() in src/testbench/datautil.py uses numpy.loadtxt?

#32


refactor: improve readability of bumpAttachCount

Fixes #1392
Improve code readability of function device.bumpAttachCount
Signed-off-by: Wei Zhang<EMAIL_ADDRESS>

/test

ping @jodh-intel @sboeuf

Thanks @WeiZhang555


Add Kotlin DSL example

I just added a second exmaple in Kotlin DSL and made it visible by "default".

Thanks @StefMa! Could you please sign your commit ?

Isn't it signed?

Or do you mean commit signing?

Yes I meant commit signing (with GPG or SSH), there's a check in place that requires commits to be signed before merging the PR


Update release.yml

@cvanaret
I tested the script for the release on my fork and added some fixes.
I have now an example of release automatically generated here:
https://github.com/amontoison/Uno/releases/tag/v1.2.0

Neat stuff! Thank you!

@amontoison do you know if there's a way to automatically create a new discussion for the release in Announcements, like https://github.com/cvanaret/Uno/discussions/15?

@cvanaret
I never implemented a workflow for that but somethink like this should work:
name: Post Release Announcement

on:
  release:
    types: [published]  # Se déclenche lorsqu'une release est publiée

jobs:
  delay_and_announce:
    runs-on: ubuntu-latest

    steps:
    - name: Attendre 24 heures
      run: sleep $((24 * 60 * 60))  # Attend 24 heures (86400 secondes)

    - name: Obtenir les notes de release
      id: get_release_notes
      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
      run: |
        # Récupère les informations de la dernière release
        response=$(curl -s -H "Authorization: token $GITHUB_TOKEN" \
          -H "Accept: application/vnd.github.v3+json" \
          https://api.github.com/repos/${{ github.repository }}/releases/latest)

        # Extraire le titre et le corps de la release
        echo "::set-output name=title::$(echo "$response" | jq -r '.name')"
        echo "::set-output name=body::$(echo "$response" | jq -r '.body')"

    - name: Poster l'annonce dans les Discussions
      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
      run: |
        # Remplacez '123456' par l'ID de la catégorie où l'annonce doit être postée
        curl -X POST -H "Authorization: token $GITHUB_TOKEN" \
          -H "Accept: application/vnd.github.v3+json" \
          https://api.github.com/repos/${{ github.repository }}/discussions \
          -d "{\"title\": \"Nouvelle Release : ${{ steps.get_release_notes.outputs.title }}\", \"body\": \"${{ steps.get_release_notes.outputs.body }}\", \"category_id\": 123456}"

The idea is to wait a few hours or one day after that a release is created to do a new announcement.
It will copy-paste the content of the release note and use it for the announcement.
I added a "sleep" because the workflow that cross-compiles binaries and create the release doesn't contain the changelog / release note.
So it will give you one day to add it before that the announcement is created.

@amontoison thank you very much!

Do you want that I open a PR to add a workflow?
I can do next week and try it on my fork.

If you find time, sure. It's not urgent though.
Thanks!


fix(src/index.js): fixed dispatch success type action if there is no …

fixed dispatch success type action if there is no success type

awesome!


[#cleanup] remove disabled setting

Codecov Report

Merging #6242 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6242      +/-   ##
==========================================
+ Coverage   39.63%   39.63%   +<.01%
==========================================
  Files        3051     3051
  Lines       91215    91215
  Branches    11826    11826
==========================================
+ Hits        36151    36153       +2
- Misses      52085    52088       +3
+ Partials     2979     2974       -5

Impacted Files
Coverage Δ

...rp/pinpoint/rpc/control/ControlMessageDecoder.java
64.28% <0%> (-14.29%)
:arrow_down:

...p/pinpoint/rpc/stream/StreamChannelRepository.java
75% <0%> (-10%)
:arrow_down:

.../pinpoint/rpc/client/PinpointClientHandshaker.java
60% <0%> (-6.43%)
:arrow_down:

...rc/main/java/com/navercorp/pinpoint/test/Item.java
70% <0%> (ø)
:arrow_up:

...point/rpc/client/DefaultPinpointClientHandler.java
72.07% <0%> (ø)
:arrow_up:

...ollector/cluster/zookeeper/ZookeeperJobWorker.java
80.93% <0%> (+1.39%)
:arrow_up:

...ercorp/pinpoint/profiler/util/ThreadDumpUtils.java
62.12% <0%> (+1.51%)
:arrow_up:

...m/navercorp/pinpoint/web/util/ThreadDumpUtils.java
35.23% <0%> (+1.9%)
:arrow_up:

...cluster/zookeeper/ZookeeperClusterDataManager.java
54.83% <0%> (+1.93%)
:arrow_up:

...om/navercorp/pinpoint/rpc/codec/PacketDecoder.java
59.57% <0%> (+2.12%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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


Error with https and Python's urllib.urlretrieve: 400 Couldn't parse URL

I've noticed that Polipo fails to parse the URL from Python's urllib.urlretrieve (Python 2.7.9):
% python2 -c 'import urllib; r = urllib.urlretrieve("https://www.example.com/"); print(file(r[0]).read())'<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">

<title>Proxy error: 400 Couldn't parse URL.</title>

400 Couldn't parse URL

The following error occurred while trying to access https://www.example.com/:

400 Couldn't parse URL

Generated Tue, 03 Mar 2015 13:11:53 CET by Polipo on localhost:3128.

This appears to happen for any https URLs, and is probably caused by an improper / no use of CONNECT from Python's side?!
The relevant bug for Python is http://bugs.python.org/issue1424152; there's a patch for 2.7, which hasn't been applied yet.

Their bug :-)


fix parsing generic function parameter

The Problem:
The following code is reported having wrong completion.
Both bar1 and bar2 should appear but only bar1 shows
struct Foo;

impl Foo {
  fn bar1(&self, x:i32) {}
  fn bar2<T>(&self, x:T) {}
}

fn main() {
  let x = Foo {};
  //  x.   only shows bar1
}

The Cause:
In the function nameres.rs:search_scope_for_methods, it will call
typeinf.rs:first_param_is_self to check the function belong to Foo. The
function deals with generic parameter like this:
If we find the position of '(' and there is '<' before it, we will skip
this '(' and find next one. The thought is avoiding parenthesis inside
generic parameter, like following case:
pub fn map<U, F: FnOnce(T) -> U>(self, f: F)
However this will skip the parenthesis of arguments if there is no
parenthesis in generic arguments. There is no parenthesis in the remaining
string "&self, x:T)", so function first_param_is_self return false
The Solution:
Instead of just skip the parenthesis, we firstly skip the generic arguments
list by skiping the <> pair.
The method is simple: a stack keep record of '<' '> we met. The only
special case is that there may have '->' appear in generic arguments
list.
TODO:
The function first_param_is_self now become a little stupid, and the
way it skip generic arguments list may have room for improvement.
Close #564

Just an thought: why can't we replace function first_param_is_self with something like this:
blob.find("self")

Since normal programmer will write self as first argument, and there shouldn't have self appear in other place.

Thanks @yodalee. You're right, just finding 'self' as an exact ident should do it.


Non-zero padded exponent in float string representation after Ryu implementation

As discussed in the forum post of the same name, there was an unintended change in the scientific notation for single-digit exponents, where they are no longer zero-padded after PR #14084 included in v1.11:
# before
printf("%E", 123.45) # => 1.234500E+02
printf("%E", 123.45e15) # => 1.234500E+17
# after
printf("%E", 123.45) # => 1.234500E+2 (note the missing leading zero in the exponent)
printf("%E", 123.45e15) # => 1.234500E+17

This no longer follows the C99 standard, which most languages adhere to. Furthermore, the official and other Ryu implementations (used in #14084) also print zero-padded exponents. It allows nicely aligned numbers, which is useful when writing files with hundreds/thousands lines of floating numbers such as those used in computational chemistry; my field of work.
I ask to revert the format back to include the leading zero.

I mildly agree with reverting this change in formatting:
As mentioned in the forum, the original reason to drop the leading zero was to make it consistent with normal printing (e.g., 1e-6.to_s returns "1.0e-6").
So reverting it means breaking again the internal consistency, in favor of some external consistency and backward consistency.
I would like to point out, though, that the argument drawn was to ensure numbers will be parsed correctly. This sounds a bit sketchy. Because while indeed Python, Ruby, OCaml, and, of course, C, seems to agree on this formatting, .NET and Haskell doesn't. .NET pads the exponent with three digits. The C99 std says (bold is mine):

The exponent always contains at least two digits, and only as many more digits as necessary to represent the exponent.

Technically speaking, if a parser fails to parse a number without the "%2d" format of the exponent, it might as well break it .NET's "%3d".
Haskell does what Crystal 1.11 does.
ghci> Text.Printf.printf "%e\n" 1e-6
1.0e-6

@Sija they're not wrong, they're free to not follow an external spec perfectly. Lang X printf doesn't have to be an exact C printf implementation.

Hey everyone, thank you for your input in this minor issue. I think the main problem is that #14084 introduced the change without documenting it, as @straight-shoota suggested in the forum post. From your discussion, it's not agreed which format should be used, so I think is better to revert it to the previous behavior for now. If it's later decided to drop the leading zero, it should be clearly documented on the PR/release notes.
IMHO, being consistent with the C99 standard and all most other languages is important to avoid surprises in a heterogeneous environment (multiple languages), especially in scientific software (I believe Crystal is an excellent language for this use case). I think there should a very strong reason to go against a widely-used spec. The issue of internal consistency may be resolved by following this standard instead when using scientific notation in the normal printing of floats.


[Bazel CI] Error toolchain_config_utils.bzl' is invalid because 'cc/toolchains' is not a package

CI: https://buildkite.com/bazel/bazel-at-head-plus-downstream/builds/4205#019298aa-c29a-4322-9cff-f0d54e9f932e
Platform: Ubuntu
Logs:
ERROR: Error computing the main repository mapping: at /var/lib/buildkite-agent/.cache/bazel/_bazel_buildkite-agent/d7f9c650ce4289e103215f92cf61272f/external/bazel_tools/tools/cpp/cc_configure.bzl:18:6: Label '@@rules_cc//cc/toolchains:toolchain_config_utils.bzl' is invalid because 'cc/toolchains' is not a package; perhaps you meant to put the colon here: '@@rules_cc//cc:toolchains/toolchain_config_utils.bzl'?
Error: The command exited with status 1
error: The plugin docker command hook exited with status 1

Culprit: https://github.com/bazelbuild/bazel/commit/1c4e78a9ceab9794140324d64833c628e382a3da
Steps:
git clone https://github.com/bazelbuild/rules_testing
git reset 83377311aa9d389d87a1bbba4fd71800a006493e  --hard
export USE_BAZEL_VERSION=1d022f403d26505d6edda73a50e08da53109dd3b
bazel build --noenable_bzlmod --enable_workspace ...

CC Greenteam @meteorcloudy

This errors sounds very familiar. I think we saw this in rules_python and the fix was to upgrade the rules_cc version being using.


Fix for moving lines with Alt+Up/Down

Fixed moving lines with Alt+Up/Down.
PR Type
What kind of change does this PR introduce?
Bugfix

Another bug I found:

Now, do alt+down twice and then alt+up

Also found an issue in indentation:
Before:

After:

(last line is gone)

@JasonStein fixed all the issues reported so far.


fix: bugs in organization screen and repository issues list

Question
Response

Version?
v1.6.0

Devices tested?
iPhone 6 Simulator

Bug fix?
yes

New feature?
no

Includes tests?
no

All Tests pass?
yes

Related ticket?
#826

Screenshots

Before
After

Description

Fixed 2 bugs,

https://github.com/gitpoint/git-point/issues/826#issuecomment-427588684
in any repository, i.e. gitpoint/git-point, click view all of preview issues, click any issue of the opened issue list

Coverage remained the same at 51.029% when pulling 8acd0b8984db2519d225cfbd6418b3b905d926dc on fix_826 into e668b045ae2e8b5bf1c3460f78e0d0fecc6c9b61 on master.

Added translation of the new phrase into Russian and Ukrainian.

@lex111 Fixed. It is due to the new Catalan translation doesn't contain a key.


[Webex Adapter] Add Webex Functional Test and YAML File

Description
This pull request adds a YAML file to set up a build pipeline for Webex Functional Test.
Specific Changes
We added the file botbuilder-dotnet-ci-webextest.yml inside the build/yaml folder, this file can be used to set up a pipeline to run functional tests for Webex.
By default, the file is configured to run the pipeline for each commit or pull request in the master branch. This can be changed in the file or in the UI when setting the pipeline.
How to use
The next steps will guide you through the configuration of a Build pipeline based on the YAML files.
Prerequisites

Azure DevOps organization.
Azure subscription. Required to create and delete Azure resources.
One app registration in Azure. More info
Configure an Azure Connection Service.
Create a Webex Application and a Webex webhook.

Creating an account & a Bot in Webex
Sign up for a developer account in the Webex Developer portal and create a new Webex Bot app. The only mandatory fields you'll need to set are the bot's name, user (email) and icon:

Once created, you will receive the bot's Access token and ID. The Access token will be displayed only once when you create your bot and never again, so don't forget it or you will have to generate a new one.

You can find further reading about Bots and how to create one on Webex's Documentation Page.
Creating and configuring an Azure pipeline
We will be using the file at \build\yaml\botbuilder-dotnet-ci-webextest.yaml for setting up a pipeline in Azure with the following steps:
•	Use .Net Core sdk 2.1.x [Use .NET Core]
•	Use .Net Core sdk 3.0.x [Use .NET Core]
•	Use NuGet 4.9.1 [NuGet tool installer]
•	NuGet restore [NuGet]
•	Build solution Microsoft.BotBuilder.sln [Visual Studio Build]
•	Dotnet Publish TestBot [.NET Core]
•	Create Azure Resources [Azure CLI]
•	Zip Deploy Bot [Azure CLI]
•	Set Environment Variables [PowerShell]
•	dotnet test [.NET Core]
•	Delete Resources [Azure CLI]
To create a pipeline based on a yaml file, select this option when creating a pipeline

Configure the repository and branch.

In the Configuration as code, select the YAML option

In the YAML section, write the build name and select the build YAML file inside the build folder in the root of the directory.

Setup variables
The following are the variables we need to add to the pipeline. It is a good idea to prefix their names with "Webex" or something to make them more relatable to Webex, since more than one set of functional tests may be running in the same pipeline.

Name
Value

BuildConfiguration
Set it to debug

BuildPlatform
Set it to any cpu

AppId*
An App Id registered in Azure

AppSecret*
The App Secret from said App   registration

RefreshToken*
(see Create an Integration below)

WebexIntegrationClientId
(see Create an Integration below)

WebexIntegrationClientSecret*
(see Create an Integration below)

WebexBotAccessToken*
The bot's access token obtained when it   was created

WebexRoomId*
(see Obtaining the room Id below)

WebexBotUserName
This is the username (an email   @webex.bot) that identifies the Webex bot you created in the previous step

AzureSubscriptionName
The subscription connection service to   authorize the pipeline with Azure

AzureBotPublicAddress
The public, messaging endpoint address   the bot will have when deployed to Azure

WebexWebhookName
The name you want for the webhook. Make   up one and write it down for later

WebexWebhookSecret
An optional secret string to secure the   communications. Make up one and write it down for later

*Set these variables as private.

Create a Webhook
The Webex .NET Echobot requires webhooks to be created manually. You can find the Webex webhook reference and a platform to test them at https://developer.webex.com/docs/api/v1/webhooks
In there you'll have a list of what the API offers in relation to webhooks (list, create, update…)

Either through the corresponding link, or using curl/Postman, you'll have to create a webhook by making a POST request to https://api.ciscospark.com/v1/webhooks
You will be given a 12-hours-expiring personal access token for this operation. You can leave the 'Use personal access token' option on if you make the POST from the web interface, otherwise you'll need to copy this value if you are going to make the POST from curl or Postman.

The remaining parameters you need are the webhook name, targetUrl, resource, event, filter, and secret. See the Create a webhook documentation for further reference on each.
For the targetUrl we will need the messaging endpoint our bot will have once deployed in Azure. Take the name we have set for our bot in the pipeline variables to compose this URL. It will go like this:
https://.azurewebsites.net/api/messages
Remember,  is the value you set for the pipeline at the previous step of this tutorial.
for resource and event, use messages, and created, as suggested by the example.
Obtaining the room Id
One parameter we're going to need to talk to the bot is the room Id. Every chat in Webex happens in a room so the functional test uses this value to retrieve messages by room.
You can get the room Id in the response for creating a message https://developer.webex.com/docs/api/v1/messages/create-a-message. You don't need to have the bot deployed to Azure to get this, only the bot created in Webex.
Send any test message to it and target the bot through its user (email). The response will bring you the room Id:

Create an Integration
The personal access token you are given in the previous step is only for developing and should not be used in a production environment, since it expires every 12 hours. So for having the Azure pipeline running we need to get a better token. We will get this token from a Webex Integration we will create.
We should be able to create an Integration from the same menu where we created our bot in Webex

The process is pretty straightforward and will require name, email, icon, description, redirectUri(s), and scopes.
For scopes, tagging spark:all should be enough. For the redirectUri(s) add any https:// you wish to be redirected to.
Get the RefreshToken value
Once you finish creating your Integration, you will be given a Client ID and a Client Secret. Copy and store both.
You will also be presented with something like this:

Copy and paste that URL into a browser (you can set any state for the set state here value). That will take you to the URL you set as a redirection URI where at the address bar you will see
http://[YOUR_REDIRECT_URL]/?code=[CODE]. Copy the value at [CODE]. This is your authorization code.
You'll need now to make a POST request to _https://api.ciscospark.com/v1/access_token_ with the content type as x-www-form-urlencoded with the following parameters:

The Webex REST API will then return a payload like the following (picture taken from Webex's site):

This includes an access_token. That's the kind of token we need to use for the Azure pipeline, since it will allow us to send messages to the bot. It has a validity of 90 days.
The payload will also include a refresh_token that can be used to get a new token once the 90 days time passed (or earlier).
While the access_token will be valid for 90 days, you don't need to wait that long to request a new access token. As a matter of fact, in order for the access token not to have to be manually re requested after 90 days, the functional test requests a new access token (using the provided refresh_token) every time it runs.
Fill the RefreshToken, WebexIntegrationClientId, and WebexIntegrationClientSecret in the pipeline with the refresh_token, Client ID, and Client Secret values obtained in this step.

Pull Request Test Coverage Report for Build 99958

0 of 0   changed or added relevant lines in 0 files are covered.
23 unchanged lines in 19 files lost coverage.
Overall coverage remained the same at 78.369%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Builder/Adapters/TestFlow.cs
1
79.67%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/EmitEvent.cs
1
55.56%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/GetActivityMembers.cs
1
30.77%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/IfCondition.cs
1
76.47%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/SignOutUser.cs
1
63.64%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/SwitchCondition.cs
1
80.49%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/DialogManager.cs
1
81.03%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Generators/ActivityChecker.cs
1
86.62%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Testing/TestActions/AssertReplyActivity.cs
1
80.77%

/libraries/Microsoft.Bot.Builder.Dialogs/ObjectPath.cs
1
86.03%

Totals

Change from base Build 99879:
0.0%

Covered Lines:
14354

Relevant Lines:
18316

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.ApplicationInsights.Core.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.AspNet.Core.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Testing.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Streaming.dll compared against version 4.6.3


wsl.exe --update failed with error: 0x80070422

Windows Version
Microsoft Windows [Version 10.0.19044.2604]
WSL Version
I cannot check because wsl.exe fails with an error.
Are you using WSL 1 or WSL 2?

[X] WSL 2
[X] WSL 1

Kernel Version
I cannot check.
Distro Version
No response
Other Software
No response
Repro Steps

I ran wsl.exe --update with both WSL1 and WSL2 running.
wsl.exe --update failed with error code 0x80070422. (I'm sorry, but I missed the error message.)
After the update failed, WSL failed to boot with the following message.

Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0x80070422
Error: 0x80070422 ?????????????????????????????????????????????????

Press any key to continue...

Expected Behavior

wsl.exe --update should succeed.
WSL should boot without errors.

Actual Behavior
I got an error 0x80070422, and WSL failed to boot.
Is there any ways to recover WSL?
Diagnostic Logs
No response

/logs

WslLogs-2023-03-14_07-19-46.zip

Thanks for sharing the logs. It looks like you're on WSL 1.0.3. Could you try removing the WSL app and re-installing?
While looking through the code I did notice a change that can be made that would allow you to update your package even if the class registration is broken. I'll go ahead and make that now.

Thank you for your response.
I removed the Ubuntu-20.04 App (which is WSL2).
Then I went to Settings > Installed Apps > Windows Subsystem for Linux, and clicked "Repair", which seems to have repaired the issue. Now I can run wsl.exe.

Good to hear, thanks.

https://github.com/microsoft/WSL/releases/tag/1.1.6

I have the same problem, but the solution is not working. my computer has vmware installed.
C:\Users\Cuser>wsl
无法启动服务，原因可能是已被禁用或与其相关联的设备没有启动。
Error code: Wsl/0x80070422

I have the same problem, but the solution is not working. my computer has vmware installed. C:\Users\Cuser>wsl 无法启动服务，原因可能是已被禁用或与其相关联的设备没有启动。 Error code: Wsl/0x80070422

Same. What happened?

I have the same problem, but the solution is not working. my computer has vmware installed. C:\Users\Cuser>wsl 无法启动服务，原因可能是已被禁用或与其相关联的设备没有启动。 Error code: Wsl/0x80070422

我发现我是服务里面那个 wsl service 给禁用了，跟那个最新的 issue 里面说的一样，我打开之后就没事了

WslLogs-2024-10-27_07-23-29.zip
hi please check my logs


Ignoring sourcemaps

Hello,
I am using TypeScript, and the compiler generates sourcemap files alongside all javascript files.
Taking into consideration this commit: https://github.com/sequelize/umzug/commit/890e5e16983fa52a258740d0c5246bc97a58d432
This causes warnings to be logged for each sourcemap file:
File: ....js.map does not match pattern: ...

The commit description says:

If there is other files, the pattern is probably filtering out some migrations.

However, in my opinion, sourcemaps are very common and should be allowed to exist alongside the javascript files. Come to think of it, the reason I use a filter is because I have other files and want to ignore them.
I would like to come up with a solution for this.
Possible solutions:

Ignoring sourcemap files before trying testing the pattern matching
Adding a suppressWarnings flag to the configuration so that it will not be logged

What do you think is the best solution?
Thanks!

I'll be working on improving this verbosity level, I'm thinking of adding the new option, and probably a companion PR to sequelize-cli as well.

With https://github.com/sequelize/umzug/commit/cdfd53ee52bd7d9a64a6d0f11cb4c76f2ad3c6d3, non-migration files are no longer logged at all, so I think this issue can be closed.

This should be fixed with 2.3.0 release


Semver implications of #[non_exhaustive] behavior on tuple/unit enum variants

Consider the following enum as the public API of a crate.
pub enum Foo {
    #[non_exhaustive]
    Unit,

    #[non_exhaustive]
    Tuple(i64),
}

Per the reference, other crates may not construct nor pattern-match on Foo::Unit or Foo::Tuple. For struct variants, only constructing them is not allowed; pattern-matching on them is allowed by including .. in the match pattern.
This appears to have some surprising implications for use of this type from outside its defining crate:

Converting a non-exhaustive unit variant to a non-exhaustive tuple variant, or vice versa, is completely invisible i.e. semver-patch. (Optionally, it can be opt-in semver-major if serialization is involved. A good use case for #58.)
There's no way to access the fields of the tuple variant. This means that both adding and removing fields is non-breaking — in fact it's semver-patch because it's completely invisible. (Same caveat about serialization as above.)
As written, there's no reasonable way to even determine which of the two variants is inside. Sure, std::mem::discriminant() can return a value, but in isolation that value means nothing — its ordering is specified but the exact value is not. As written, deleting a non-exhaustive unit or tuple variant is invisible and therefore semver-patch.

The only way I've found to externally distinguish between Foo::Unit and Foo::Tuple requires explicit discriminants so that std::mem::discriminant()'s return value can be defined exactly rather than just in terms of its ordering:
#[repr(C)]  // or any explicit int repr
pub enum Foo {
    #[non_exhaustive]
    Unit = 1,

    #[non_exhaustive]
    Tuple(i64) = 2,
}

Another question: is reordering enum variants considered semver-major, minor, or patch? It definitely could change the outcomes of use of  std::mem::discriminant(), but some rare technically-might-be-breaking changes are not considered semver-major (e.g. adding a method) so I feel this isn't a clear-cut case. The API evolution RFC does not address this either.

The following syntax is able to match on the variants and access the tuple variant's fields:
match foo {
    Foo::Unit { .. } => {},
    Foo::Tuple { 0: val, .. } => {},
}

h/t to https://old.reddit.com/r/rust/comments/10h9my4/semver_implications_of_non_exhaustive_behavior_on/j57d5cf/

You can also use .. in a tuple variant pattern:
match foo {
    Foo::Tuple { 0: val, .. } => {},
    // is equivalent to
    Foo::Tuple(val, ..) => {},
}

That's what I thought at first too, but it didn't seem to work: on rustc 1.68 (nightly) I got an error[E0603]: tuple variant Tuple is private error.

Opened a diagnostic issue on rustc: https://github.com/rust-lang/rust/issues/107165


AttributeError

Hello, I encountered such an error when running the file deal_data.py to generate the train.json file. How should you solve it? Looking forward to your answer！

More details?


FW-850 Adapt timing for openmote-b-subGHz board

@minarady1  for the future work, when you commit codes, use the issue number at beginning so JIRA can hook up the changes. Such FW-850. 160 ms timing.  The branch name should be named as develop_FW-50.

@minarady1  Could you write a summary about how you test the code? When I check the radio_settings of subghz port, only one frequency is used. When using 11 length slotframe with 10 EB portion. The mote should be sync'ed within 1 second, which is not what I saw.

For this PR to work correctly, the following needs to be configured. With this configuration, it should synchronise within 2 minutes and you should see the first RPL DAO within 10 mins.
Configuration > Location:
fix_channel = <channel_id> scons compile option fix_channel = 11 (for example)
EB_PORTION 4 > IEEE802154E.h
SLOTFRAME_LENGTH11 > schedule.h
LENGTH_CRC = 2 > radio.h

@minarady1 I had tested your code with the changes you mentioned,  it works on my side perfectly, with little changes. Could I ask to do the following before I merge your code?

create a README.md file under project->openmote-b-subghz to record the changes to make to see the network starts up quickly.
change Port_delayTx 100 to Port_delayTx 68
change the timing in IEEE802154E.h file under slot duration 160 from ticks to micro-second/PORT_US_PER_TICK.


Readme: install - via console last stable version

Thank you. :) I was a busy these days.

:+1: No prob, thanks


SKTypeface.FromFamilyName returns null

My app uses the Avalonia library to show UIs on ARM devices. Avalonia internally uses SkiaSharp. I have a strange behavior on the Raspian system where SKTypeface.FromFamilyName returns null if one passes a font family name that does not exist.
The authors of Avalonia say it is expected to return the default font of the system. Unfortunately this unexpected null crashes the application during the application startup.
Any idea what is going wrong here?
Is this an issue with SkiaSharp, Skia or the specific operating system?
I use a self built libSkiaSharp 1.60.3 with .NET Core 2.1 on Raspian 9.4

I should add the following:
the Avalonia guys asked me to run the following line on Raspian
var typeface = SKTypeface.FromFamilyName(FontFamily.Default.Name);

FontFamily.Default.Name is hardcoded in Avalonia as Courier New.
On Raspian this call returns the Liberation Mono font. But the same
call with "Ubuntu" as the name returns null (The Ubuntu font does not
exist on Raspian).

Hmm. I believe the Rasbian build should be using the fontconfig font manager, so I had a look and noticed this:
https://github.com/google/skia/blob/chrome/m68/src/ports/SkFontMgr_fontconfig.cpp#L894
Basically, if there is a problem matching the style it will return nullptr. This is not good, but appears to be the case.
However, there is hope: it appears that if you pass null as the name, then it will find something:
https://github.com/google/skia/blob/chrome/m60/src/core/SkTypeface.cpp#L119
So, as a fix, you can do something like this:
var typeface = SKTypeface.FromFamilyName(FontFamily.Default.Name) ?? SKTypeface.FromFamilyName(null);

In the coming release, I will expose the SKTypeface.CreateDefault() method that will never return null.

Thanks for the analysis!

I am closing this as I believe there was never a promise that these methods would not return null.


A social profile is not a discovery mechanism

The WebID-Profile draft contains a number of recommendations which seem to privilige some (unspecified) pieces of data, lifting them out of the normal mechanisms of discovery. In this issue, I would like to discuss the reasons and problems of such an approach.
Concretely, the draft contains the concept of a Preferences File, pointed to from the Profile Document, as well Extended Profile Documents, pointed to from the Profile Document, the Preferences File, or other Extended Profile Documents. The idea is to have these documents contain "all statements with the WebID as subject," and use the proliferating structure of documents to regulate access (in quite an untenable way, as @RubenVerborgh has already pointed out). For example, consider the following paragraph from the introduction.

Keisha's Solid profile might list her name publicly, her phone number only for friends, and the configuration settings for her "notes to self" only for apps operating on her own behalf. Solid supports these options by splitting the profile into documents with separate access restrictions. So the first task of an application may be to load all of the profile documents it has access to or load them on a need to basis.

A first question that bothers me is: why are those triples so special? If I store photos in my pod, sorted into albums using metadata, these are photo albums. Whether or not I decide to add a me: photo:hasAlbum <./albums/xyz#this> triple to those albums should i.m.o. not influence the discoverability of my album from my Extended Profile. The other way around, if I use me: vcard:n <./info#name> instead of me: vcard:fn 'Wouter Termont', suddently my name is no longer part of my Extended Profile. I fail to see how this can be desired behavior.
If not the special status of these triples with the WebID as subject, then what makes some data deserve to be in the profile and other data not? The fact that they identify the referent of the WebID? But then my entire gene sequence might be a better candidate for the Profile than my name. Moreover, "configuration settings for [someones] 'notes to self'" are hardly identification.

The conclusion must be that the WebID-Profile proposal recommends a rather random selection of data to be present in the profile. This, in its own, leads to even more problems. How, for example, will an app know where to look for a specific piece of data? Shall it find it by crawling the possibly complex web of Extended Profile Documents? Or should it rather look for it via an indexing/registry system? What if it finds the data via both ways, but it conflicts? What if it does not find it via either way; where should it then store its initial value? What if one app stores data of type X using one mechanism, yet afterwards you'd like to use another app to process that data, but this app can only use the other mechanism?
I hope to have pointed out clearly a number of issues with the Preferences File and Extended Profile Documents proped by WebID-Profile. My initial suggestions based on them would be:

[ ] to rethink the idea of having "profile data" that is separate from other data;
[ ] to brainstorm with the people from SAI about how we can use interfaces which make discovery mechanisms really compatible/interchangeable;
[ ] to try to build a social media style profile on top of a (separate) discovery specification (if that is what this specification wants to do).

rethink the idea of having "profile data" that is separate from other data

A noble goal, but not one of this specification.  Our task is to document existing practices and make recommendations related to those existing practices.  All existing apps I am aware of that seek to find out about a Social Agent use this concept of a profile that consists of a graph of statements with the WebID as subject.  That is the situation we will document.

Or should it rather look for it via an indexing/registry system?

That will be nice when it  is possible.  We are documenting what exists today, that doesn't.

I hope to have pointed out clearly a number of issues with the Preferences File and Extended Profile Documents

Again, it might be nice to live in a world where those things didn't exist, but the current world isn't that one.

How, for example, will an app know where to look for a specific piece of data?

We will be covering that in detail with multiple use cases.

What if one app stores data of type X using one mechanism, yet afterwards you'd like to use another app to process that data, but this app can only use the other mechanism?

Our document is about things that exist.  What other existing mechanism are you referring to?

A social profile is not a discovery mechanism

Perhaps, but discovery of a social profile is.  This spec will only indirectly cover most social profile predicates, leaving those descriptions for related documents for Persons and Organizations in the future.  However this spec is aimed at developers and therefore needs to be a framework for the start of the discovery process.  Discovering where an Agent puts their social profile information is a key part of that process and will be covered in this spec.

Our task is to document existing practices and make recommendations related to those existing practices.

Please read my argumentation in #68. The document being drafted here can either be a descriptive one or a normative one, not both. The arguments I give are based on a normative reading (which I infer from the wording used), so rebuttals of the form "this is the way it is being done" are not valid (unless accompanied by a clear need for compatibility and a clear argument that this can't be achieved by my counterproposal).

A social profile is not a discovery mechanism

Perhaps, but discovery of a social profile is. [T]his spec is aimed at developers [...] Discovering where an Agent puts their social profile information is a key part of that process [...]

If this spec is aimed at developers, it would take into account interoperability, performance, number of libraries etc. (on all of which this spec scores low), not the fact that an insignificant amount of early adopters will have to update their profile a bit.

Or should it rather look for it via an indexing/registry system?

[...] [I]n the general case this is possible now and we will point to other specs about them

My point was not that an indexing system itself was a problem, but rather that when we have two or three indexing mechanisms (e.g. extended profiles, type indexes, and SAI registries), the client will not know where to discover existing data, where to write non-existing data, and definitely not how to interact with other apps using the same data.

What if one app stores data of type X using one mechanism, yet afterwards you'd like to use another app to process that data, but this app can only use the other mechanism?

Our document is about things that exist. What other existing mechanism are you referring to?

I'm talking about extended profiles, type indexes, and SAI registries; three discovery mechanisms that exist. One piece of data could be stored and retrieved via any of those mechanisms, and work perfectly with one app, but as soon as a second app comes into play, it's a free for all; no interoperability at all.


Provide Sentry service intergation

Provide Sentry service to work out-of-the-box (Create script)

Need a version control built in template to be integratable with sentry


zigbee.zip

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
Python release (python3 --version):
Component/platform:
Description of problem:
Expected:
Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):

Additional info:

Please open a new issue if there really is one and fill in the issue template.


Error in _zephir_call_user_function in ZendEngine3 fcall.c

Environment:

php 7.0 x86 nts
1.1) Trying with TS too
VS 2015 (version 14.0)
Zephir v0.8
3.1) Trying with master branch too
Windows 7 x64

Code:
namespace App;

class Version
{
    public static function get()
    {
        return "0.0.1 super beta";
    }
}

Then run zephir compile:
...>zephir compile
Copying new kernel files...
Preparing for PHP compilation...
Preparing configuration file...
Compiling...

No errors but has compile.log:
"Cl.exe" / FI main blah blah blah...
...
object.c
operators.c
require.c
string.c
D:\Projects\Morph\bin\ext\kernel\object.c (65): warning C4090: dgЄzhЁp: a §lҐ Єў "ЁdЁЄ v®al" const "
time.c
variables.c
"Cl.exe" /... blah blah blah ...

Microsoft (R) Windows (R) Resource Compiler Version 10.0.10011.16384
Copyright (C) Microsoft Corporation. All rights reserved.

D:\Projects\Morph\bin\ext\Release\array.obj: warning LNK4042: Set the object more than once; excess are not counted
D:\Projects\Morph\bin\ext\Release\fcall.obj: warning LNK4042: Set the object more than once; excess are not counted
   Create a library D:\Projects\Morph\bin\ext\Release\php_app.lib and object D:\Projects\Morph\bin\ext\Release\php_app.exp
fcall.obj: error LNK2019: unresolved external reference to the symbol in _zephir_call_function_opt _zephir_call_user_function function
D:\Projects\Morph\bin\ext\Release\php_app.dll: fatal error LNK1120: unresolved external elements: 1

And only php_app.dll.manifest and php_app.dll.res in Release dir, no binaries
='(

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/31660594-error-in-_zephir_call_user_function-in-zendengine3-fcall-c?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github).

I trying to build extension over clear virtual machine:

Windows 10 x64 Enterprise 2015 LTSB
PHP 7.0.4 Non Thread Safe x86
PHP 7 (x86) develpack from http://windows.php.net/downloads/releases and php 7 sdk from http://windows.php.net/downloads/php-sdk/ (with binary tools copied inside sdk path)
VisualStudio 2015 Express with C++ support and Extensibility tools update 1

And i have this error again:

If i remove this to lines:

Building are finished successfully! :dancers: I dont know what this means (because i don know C++ like a pro)? But this works. Jedi Driven Development...

I'm not really having an idea what's causing that failure on your end.
From the symptoms it seems like extended/fcall.c is not compiled.
(Your file changes above are not recommended, that will simply prevent most function calls from working)
Does the file fcall.c in ext/kernel/ exist?
Is there a line like ADD_SOURCES(... + "/kernel/extended", "array.c fcall.c",") in config.w32?

Sholdn't we use:
zephir build --backend=ZendEngine3
for php 7 ? Or zephir already figure it out ? Beacause in phalcon you have to use it to compile 2.1.x for php7.

You only need to specify a backend if it differs from the PHP version running zephir.

And use master branch :)

Trying with master branch too

This error becouse sources have duplicates: zephir-master\kernels\ZendEngine3\fcall.c and zephir-master\kernels\ZendEngine3\extended\fcall.c. Its creating one .obj file named as fcall.obj

NMAKE : warning U4004: too many rules for target 'ext\Release_TS\fcall.obj'
NMAKE : fatal error U1077...

I'm rename _zephir-master\kernels\ZendEngine3\extended_fcall.c and _zephir-master\kernels\ZendEngine3\extended_array.c to
_zephir-master\kernels\ZendEngine3\extended_efcall.c and _zephir-master\kernels\ZendEngine3\extended_earray.c
and has made changes to a two files:
templates\ZendEngine3\config.m4 at line 12
and templates\ZendEngine3\config.w32 at line 6
(rename extended\fcall.c to extended\efcall.c etc.)

cannot reproduce that now.
An x64 extension for PHP7 builds flawlessy here.


Video player

It seems the video player doesn't trigger a "click"-event on the "play"-button.

Thanks, this was an issue with the icon button which has been fixed in version 0.3.6


Tasks

hello... I'll give you some improvements soon... I'm going to fork and then make a PR. thanks

I have evolved the project a lot.
I made some improvements that the project needed.
As Apache Hop was developed to work in a decoupled way, I made some adjustments to make the most of the Plugin.
At the moment I am creating a routine to create a json with the metadata.json that the plugin needs, this way, it will no longer be necessary to export through the graphical interface.
As soon as possible, I will create a PR and hope it is reviewed and approved.
Thanks.

TKS.
Something interesting is that the Plugin reads the variables used in the project.
And one of the important variables for the Project is PROJECT_HOME, which needs to be defined in the Apache Hop interface for perfect execution.

Example project configuration.
It is necessary to create this environment variable, pointing to the exact location of the project.
This way the plugin works in an exemplary way.

You could also find a way to make this variable and add it to the Payload XML.
It is found in the hop configuration file.
"projectConfigurations" : [ {
"projectName" : "hop_repo",
"projectHome" : "/opt/projetos/hop_repo",
"configFilename" : "project-config.json"
}
I didn't implement this part, but I believe it is possible.
This way the plugin is completely independent
TKS

I did a test running only the Pipeline, and the process ran perfectly.
Now, when a Workflow calls a pipeline, it cannot execute it, because it cannot find the PROJECT_HOME path, and therefore cannot find the pipeline path.
I tried to find it in the code, but I couldn't

TKS.
Something interesting is that the Plugin reads the variables used in the project.
And one of the important variables for the Project is PROJECT_HOME, which needs to be defined in the Apache Hop interface for perfect execution.

Example project configuration.
It is necessary to create this environment variable, pointing to the exact location of the project.
This way the plugin works in an exemplary way.
You could also find a way to make this variable and add it to the Payload XML.
It is found in the hop configuration file.
"projectConfigurations" : [ { "projectName" : "hop_repo", "projectHome" : "/opt/projetos/hop_repo", "configFilename" : "project-config.json" }
I didn't implement this part, but I believe it is possible.
This way the plugin is completely independent
TKS

I encountered the problem with the PROJECT HOME variable.
I needed to add a variable to root: root.append(project_home).
PR Created.

hello... Did you test the plugin?

hello... Did you test the plugin?


Add round e-ink meter with external wired outdoor sensor

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.607%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6819

Relevant Lines:
8156

💛  - Coveralls

Codecov Report
Base: 83.60% // Head: 83.60% // No change to project coverage :thumbsup:

Coverage data is based on head (634b7be) compared to base (a16c153).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #2119   +/-   ##
=======================================
  Coverage   83.60%   83.60%
=======================================
  Files         254      254
  Lines        8156     8156
=======================================
  Hits         6819     6819
  Misses       1337     1337

Impacted Files
Coverage Δ

zhaquirks/tuya/ts0601_sensor.py
100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

Duplicate of #2117

Thanks for your contribution!
It's rare but apparently someone was a bit faster and also added the same sensor in this PR: https://github.com/zigpy/zha-device-handlers/pull/2117


Proxy Authentication Required (407) status code implementation, tests, docs (#1)

Merged! Thanks! 👍


Flutter Web - Enable option to give semantics role combobox

Use case
Tried all the semantic properties but can't set a role combobox. Give a proper way to filled a dropdown button with the right accessibility role. So the flt-semantics to have role: combobox.
Proposal
It looks like expanded property is not changing anything on Flutter web. So as a suggestion when it is set to expanded: false it could change the role of flt-semantics to combobox.
Example:
<flt-semantics id="flt-semantic-node-68" role="combobox" tabindex="0" aria-label="Time
Now" flt-tappable="" style="position: absolute; width: 392px; height: 74px; transform-origin: 0px 0px 0px; transform: matrix(1, 0, 0, 1, 16, 62); pointer-events: all; z-index: 3;"></flt-semantics>

This will need work on the framework too.

Is this still being worked on?
I'm trying to create a simple table that works how normal html tables do on the web.
I'd like to specify the WAI-ARIA Roles for table and column but can't find an obvious way to do it.

the topic seems to be addressed this document
as mentioned in this issue
https://github.com/flutter/flutter/issues/107861#issuecomment-2066989390


Correct some spelling mistakes and reword some sentences

Just a few small changes to the README, I may have gone a little too far with some of the rewording in places where it wasn't entirely necessary but hopefully this improves the quality of the readme a little.
If you'd like me to separate the spelling changes from the rewording so you can accept it separately I can do that.
I'm up for writing some more documentation in the future once I've put in some more time with the project.

Thanks


#85645 display Variables fetch failure message

This PR fixes #85645.
A modified vscode-mock-debug is available at https://github.com/gjsjohnmurray/vscode-mock-debug/tree/show-85645 in which the Variables request always returns a failure status and a message.
Prior to this PR the message was not displayed in the Variables panel:

After the fix:

Thanks for this PR. Even though PR indeed fixes the issue the actual probel is the following:
Variables that have available = false are not properly rendered in the VARIABLES view. I am not sure why is that. The idea behind available = false is that we can render the variable like an error.
So ideally we would leave for that variable availble = false and we would fix the rendering
If that is not possible, than I am fine that we remove the available attribute altogether (might be a cleaner solution and I do not mind that we loose the special rendering)
Let me know what you think and thanks a lot!

@isidorn here's what it now gives. Is that what you meant?

Looks good, tahnks a lot for the PR!


Route Command (SuperStream)

A super stream is made of several individual streams, so it can be considered a logical entity rather than an actual physical entity.

https://github.com/rabbitmq/rabbitmq-server/blob/1ee8454129b7769fda6dfc1a528e886b290aa258/deps/rabbitmq_stream/docs/PROTOCOL.adoc#route
RouteQuery, is initiated by the client, and the server returns an array of stream names associated with a Superstream.
RouteQuery => Key Version CorrelationId RoutingKey SuperStream
  Key => uint16 // 0x0018
  Version => uint16
  CorrelationId => uint32
  RoutingKey => string
  SuperStream => string

RouteResponse => Key Version CorrelationId ResponseCode [Stream]
  Key => uint16 // 0x8018
  Version => uint16
  CorrelationId => uint32
  ResponseCode => uint16
  Stream => string

Pr opened. https://github.com/coders51/rabbitmq-stream-js-client/pull/97


Added instructions and schema to use or migrate to SQLite

- makes SQLite 3 the default database engine (contentious, so i separated from this PR)

adds SQLite schema in addition to MySQL one
readme now includes instructions on how to setup SQLite DB
readme now includes migration instructions

See discussion at #11

what i would do for the migration is to just create a bash script for it, and perhaps we can have make generate a sqlite db as well? albeit it can leave an unwanted file

@alucard0134 that's what i had initially (bash script), but thought it made the process look too complicated :P
The make thing is a good idea actually, I'll add that.
MYSQL_USER=user
MYSQL_PASS=pass
WEB_USER=nginx
# below options are unlikely to need to be changed
MYSQL_DB=pomf
SQLITE_DIR=/var/db/pomf
SQLITE_FILE=pomf.sq3
WORKING_DIR=/tmp

mkdir $SQLITE_DIR
wget -O ${WORKING_DIR}/m2s https://github.com/dumblob/mysql2sqlite/raw/master/mysql2sqlite.sh
mysqldump -u $MYSQL_USER -p $MYSQL_PASS $MYSQL_DB | sh /tmp/m2s > ${SQLITE_DIR}/initial_dump.sql
 sqlite3 ${SQLITE_DIR}/${SQLITE_FILE} -init ${SQLITE_DIR}/initial_dump.sql

rm ${WORKING_DIR}/m2s
chown -R ${WEB_USER}:${WEB_USER} $SQLITE_DIR
chmod 0750 $SQLITE_DIR
chmod 0640 ${SQLITE_DIR}/${SQLITE_FILE}

echo == SQLite DB at ${SQLITE_DIR}/${SQLITE_FILE} prepared from MySQL dump ==
echo == The dump used has been saved at ${SQLITE_DIR}/initial_dump.sql ==

have it be curl/wget from ghostbin/raw and then tell them to edit the variables, shouldn't be rocket science

also i would prefer instead of using nil variables is to have an if else statement in the database connecting using POMF_DB_TYPE
thoughts @ewhal

@alucard0134 pls read relevant PDO docs before making comments like that
http://php.net/manual/en/pdo.construct.php
the parameter is ignored for SQLite, having a value in it make no difference, so an if else statement is silly IMO
The null thing, btw, was so that there aren't random useless credentials in the configuration file, rather than some necessity to use SQLite through PDO.

I think it's fine to merge. We will be relocating the instructions to the documentation in the future.


Add command run-custom-build.

Add command run-custom-build for just run Cargo's build script.
See some context from #7178.
I left the doc/man changes for now.
Post the PR here hoping to receiving some feedback.

Thanks for the PR! Considering that RFC 3344 has not yet been accepted, it may be premature to open a PR for the implementation. If there is some feedback you'd like to leave on the RFC, it would be helpful to leave a comment over there.

Thanks for the PR! Considering that RFC 3344 has not yet been accepted,

Right, looking forward to that rfc.
Until then, I am working on this cargo-ninja. Generating build.ninja using data from Cargo build-plan.
Made it work for just run run-custom-build target for my project. So it's OK to wait a bit longer.

Closing until we have an agreed-to-design.


Controlsd should verify safetyParam

Controlsd should check if the safetyMode and safetyParam currently set by the panda match those from carParams. This also needs a change on the panda side to put the safetyParam in health/pandaState.

Safety model is already checked. https://github.com/commaai/openpilot/blob/master/selfdrive/controls/controlsd.py#L210


Support Helm 3

What problem are you facing?
Helm 3 brought some breaking changes on how Helm works and what flags its CLI accepts.
When you fix your CLI flags to the new ones a few problems arise:

Our helm chart does not have crossplane-system namespace, hence we're not able to do helm3 install crossplane -n crossplane-system because that behavior is identical to how kubectl namespace works now, i.e. you get namespaca is not found.
Just raw helm3 install crossplane crossplane-alpha/crossplane --version 0.3.0 doesn't work, it returns Error: customresourcedefinitions.apiextensions.k8s.io "kubernetesapplicationresources.workload.crossplane.io" already exists on a fresh cluster. The actual resource changes randomly.

How could Crossplane help solve your problem?
I haven't invested so much time into this, these are my initial observations. Helm3 is still in beta phase, so, not sure when we want to support it. Wanted to track this here.

I also haven't invested time into this, but I am wondering whether creating the namespace ahead of the installation command would be sufficient to make things work. Did you happen to already try that @muvaf ?

@suskin If the namespace doesn't exist, it says the namespace is not found just like kubectl does. I cleaned the cluster, created namespace beforehand and got the crd already exists error.

Dang, ok. Thanks for clarifying @muvaf !


⚠ ClamAV has degraded performance

In 2f02f10, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 122 ms

Resolved: ClamAV performance has improved in 25f9c45 after 5 minutes.