Skill Bot not working with Alexa channel

JS sample 80.skills-simple-bot-to-bot Skill Bot does not work with Alexa channel.
[PROBLEM DESCRIPTION]
Customer deployed to Azure the sample https://github.com/microsoft/BotBuilder-Samples/tree/main/samples/javascript_nodejs/80.skills-simple-bot-to-bot.
Bots work fine with Webchat channel but fail with Alexa one.
Root bot replies correctly to questions from Alexa channel, but when Skill bot is called and a question is sent from Alexa channel, no reply is received.
[TROUBLESHOOTING STEPS]
I deployed the same sample and was able to reproduce the issue on Alexa channel. Apart from that, I added Webchat and Teams channels and issue does not happen on them.
Per my troubleshooting, Alexa only receive the first welcome message when connecting to Skill bot, after that whatever the question is sent from Alexa, no reply is received and Alexa close the connection.
I could see Skill bot receives messages from Alexa instead.
Below, I am appending image with working and non-working scenarios.
Teams channel:
Connecting to Root bot

Connecting to Skill bot
You can see Skill bot sends two welcome messages, then it echoes every message I send.

Alexa channel:
Connection to Root bot.
It replies fine.

Connecting to Skill bot.
Here, only first welcome message is received, then it does not echo messages I send and on debugging console I can see Alexa disconnects from Skill bot after sending the second message. I guess it is because Alexa does not receive the reply to the first message I sent.

And on Alexa debugging information, connection is closed.
{
"header": {
"namespace": "SkillDebugger",
"name": "CaptureDebuggingInfo",
"messageId": "ded9bc2d-67ff-43e8-8801-80cd0e04fc2a"
},
"payload": {
"skillId": "amzn1.ask.skill.d10705f7-1027-4841-9699-dc731ce140f5",
"timestamp": "2022-04-22T12:58:07.704Z",
"dialogRequestId": "48999c37-8b95-4364-9d7c-e20e8a22c6e8",
"skillRequestId": "amzn1.echo-api.request.21f74d56-1118-495b-a0e6-1d0e2652bb3d",
"type": "SkillExecutionInfo",
"content": {
"invocationRequest": {
"endpoint": "https://alexa.botframework.com/api/v1/bots/DfvBOSQBoYf/simple-root-bot-80-js",
"body": {
"version": "1.0",
"session": {
"new": false,
"sessionId": "amzn1.echo-api.session.c1fb3a06-df9c-47e5-9ede-60ea8dfc3faf",
"application": {
"applicationId": "amzn1.ask.skill.d10705f7-1027-4841-9699-dc731ce140f5"
},
"attributes": {},
"user": {
"userId": "amzn1.ask.account.AESB36F2J4RID2B46HL5EUANMAAVEG2LV37G33NDOJL7R76U3XOUN2QFR6PCC7EWHTBDFMWDEVTLX6AUI64SQNHAC5D2DTWNMYM3MUGLKMYGJPEK2SKNTPNN3QMM3ZI67OGO7SHZ4W2UXPQIIKSPJICD4QJIJHN4AZMQB6Z6RUBXCJ675DMKZOZPNLRAAFUEIX2UVPARXGJ3LOA"
}
},
"context": {
"Viewports": [
{
....
},
"System": {
"application": {
"applicationId": "amzn1.ask.skill.d10705f7-1027-4841-9699-dc731ce140f5"
},
"user": {
"userId": "amzn1.ask.account.AESB36F2J4RID2B46HL5EUANMAAVEG2LV37G33NDOJL7R76U3XOUN2QFR6PCC7EWHTBDFMWDEVTLX6AUI64SQNHAC5D2DTWNMYM3MUGLKMYGJPEK2SKNTPNN3QMM3ZI67OGO7SHZ4W2UXPQIIKSPJICD4QJIJHN4AZMQB6Z6RUBXCJ675DMKZOZPNLRAAFUEIX2UVPARXGJ3LOA"
},
"device": {
"deviceId": "amzn1.ask.device.AGTYF2U3G3BJP4LGZ4SQOVZ5F5JF3R2JBENJRLUXAM3VPIGUJ4JUEGWOU2MYJERWNYB3VEJ633P4YPU347O7FG45AAIEYRRG2E4QAS2X7TP7LJMPZZKMXBCSKFIZAMEUV7RRKYHHL6NW7D6JSTKQTTK3U5GJXKSHF6I4FAMIPWQ2MMTULOGXY",
"supportedInterfaces": {}
},
"apiEndpoint": "https://api.amazonalexa.com",
"apiAccessToken": "ey........"
}
},
"request": {
"type": "IntentRequest",
"requestId": "amzn1.echo-api.request.21f74d56-1118-495b-a0e6-1d0e2652bb3d",
"locale": "en-US",
"timestamp": "2022-04-22T12:58:07Z",
"intent": {
"name": "GetUserIntent",
"confirmationStatus": "NONE",
"slots": {
"phrase": {
"name": "phrase",
"value": "pizza",
"resolutions": {
"resolutionsPerAuthority": [
{
"authority": "amzn1.er-authority.echo-sdk.amzn1.ask.skill.d10705f7-1027-4841-9699-dc731ce140f5.phrase",
"status": {
"code": "ER_SUCCESS_NO_MATCH"
}
}
]
},
"confirmationStatus": "NONE",
"source": "USER",
"slotValue": {
"type": "Simple",
"value": "pizza",
"resolutions": {
"resolutionsPerAuthority": [
{
"authority": "amzn1.er-authority.echo-sdk.amzn1.ask.skill.d10705f7-1027-4841-9699-dc731ce140f5.phrase",
"status": {
"code": "ER_SUCCESS_NO_MATCH"
}
.....
},
"invocationResponse": {
"body": {
"version": "1.0",
"response": {
"outputSpeech": {
"type": "PlainText",
"text": ""
},
"shouldEndSession": true,
"type": "_DEFAULT_RESPONSE"
}
}
},
"metrics": {
"skillExecutionTimeInMilliseconds": 469
}

Hi @JuanAr.
Same behavior occurs for C# sample 80.skills-simple-bot-to-bot as well.
The below code only resolves for DotNet C# sample.
In RootBot.cs file, update the SendToSkill method with the following code:
private async Task SendToSkill(ITurnContext turnContext, BotFrameworkSkill targetSkill, CancellationToken cancellationToken)
{
    // NOTE: Always SaveChanges() before calling a skill so that any activity generated by the skill
    // will have access to current accurate state.
    await _conversationState.SaveChangesAsync(turnContext, force: true, cancellationToken: cancellationToken);

    // Clone activity and update its delivery mode.
    var activity = JsonConvert.DeserializeObject<Activity>(JsonConvert.SerializeObject(turnContext.Activity));
    activity.DeliveryMode = DeliveryModes.ExpectReplies;

    // Create a conversationId to interact with the skill and send the activity
    var options = new SkillConversationIdFactoryOptions
    {
        FromBotOAuthScope = turnContext.TurnState.Get<string>(BotAdapter.OAuthScopeKey),
        FromBotId = _botId,
        Activity = turnContext.Activity,
        BotFrameworkSkill = targetSkill
    };
    var skillConversationId = await _conversationIdFactory.CreateSkillConversationIdAsync(options, cancellationToken);

    using var client = _auth.CreateBotFrameworkClient();

    // Route the activity to the skill.
    var expectRepliesResponse = await client.PostActivityAsync<ExpectedReplies>(_botId, targetSkill.AppId, targetSkill.SkillEndpoint, _skillsConfig.SkillHostEndpoint, skillConversationId, activity, cancellationToken);

    // Check response status.
    if (!expectRepliesResponse.IsSuccessStatusCode())
    {
        throw new HttpRequestException($"Error invoking the skill id: \"{targetSkill.Id}\" at \"{targetSkill.SkillEndpoint}\" (status is {expectRepliesResponse.Status}). \r\n {expectRepliesResponse.Body}");
    }

    // Route response activities back to the channel.
    var responseActivities = expectRepliesResponse.Body?.Activities;

    foreach (var responseActivity in responseActivities)
    {
        if (responseActivity.Type == ActivityTypes.EndOfConversation)
        {
            await EndConversationAsync(responseActivity, turnContext, cancellationToken);
        }
        else
        {
            await turnContext.SendActivityAsync(responseActivity, cancellationToken);
        }
    }
}

/// <summary>
/// Clears storage variables and sends the end of conversation activities.
/// </summary>
/// <param name="activity">End of conversation activity.</param>
/// <param name="turnContext">Context for the current turn of conversation.</param>
/// <param name="cancellationToken">CancellationToken propagates notifications that operations should be cancelled.</param>
/// <returns>A <see cref="Task{TResult}"/> representing the result of the asynchronous operation.</returns>
private async Task EndConversationAsync(Activity activity, ITurnContext turnContext, CancellationToken cancellationToken)
{
    // Forget delivery mode and skill invocation.
    await _activeSkillProperty.DeleteAsync(turnContext, cancellationToken);

    // Show status message, text and value returned by the skill
    var eocActivityMessage = $"Received {ActivityTypes.EndOfConversation}.\n\nCode: {activity.Code}.";
    if (!string.IsNullOrWhiteSpace(activity.Text))
    {
        eocActivityMessage += $"\n\nText: {activity.Text}";
    }

    if (activity.Value != null)
    {
        eocActivityMessage += $"\n\nValue: {JsonConvert.SerializeObject(activity.Value)}";
    }

    await turnContext.SendActivityAsync(MessageFactory.Text(eocActivityMessage), cancellationToken);

    // We are back at the host.
    await turnContext.SendActivityAsync(MessageFactory.Text("Back in the host bot."), cancellationToken);

    await _conversationState.SaveChangesAsync(turnContext, false, cancellationToken);
}

Perhaps translating the above code to Javascript might fix JS sample 80.skills-simple-bot-to-bot in respect with Alexa channel?

Hi, @ramfattah .
We were able to reproduce the problem. While testing, we noticed that the first activity the bot receives from Alexa has the delivery mode set to expect replies. So the channel is waiting for this mode of communication, that's why the normal delivery mode, with asynchronous responses, was not working.
We tested the proposed changes in the JS samples, setting the delivery mode to "expect replies" and sending all the activities we receive from the skill in a single response to the channel and it worked.
However, these samples don't support expect replies and probably they don't need to do it. (issue#2313).
You can use the bots from the BotFramework-FunctionalTests repository as samples as they support both delivery modes and we confirmed that they are working with Alexa.
Let us know if you have any questions.
Regards.

Hi @erquirogasw,
Looks like the skill is still not working in Alexa channel using SimpleHostBot as rootbot and EchoSkillBot as skill bot.
Also, in Alexa, I noticed the suggestion choices were not showing up. Used numbers to select choices.
Alexa (skill not working):

Screenshot in WebChat (working):

any thoughts? looking forward to hearing from you soon.

Hi @ramfattah .
We tested the SimpleHostBot and EchoSkillBot and worked as expected in Alexa.
Could you provide us with information on how you have done the tests? Have you made any modifications to the bots?
We were able to visualize through the ngrok web ui that in our test the deliveryMode is correct. Can you check using ngrok if the message sent to the skill is using expectReplies?
This screenshot shows the bots working in Alexa.

Regards.

Hi @erquirogasw, Thank you so much.
Indeed I made code changes I did not realize.
Using a fresh SimpleHostBot as rootbot and EchoSkillBot as skill bot works as expected with Alexa channel.

Thanks for the help @erquirogasw :)


Support WITH clause in CREATE TABLE Statements

Is your feature request related to a problem or challenge?
In https://github.com/ArroyoSystems/arroyo we use DataFusion to parse and plan SQL queries, before mapping the logical plan into a streaming dataflow. As such, our sources will tend to be datastores such as Kafka. The typical way to define a kafka source in the streaming world is with the WITH statement, e.g.
CREATE TABLE people (
  id INT not null,
  name STRING not null,
) with (connection='kafka', topic='output');

SELECT * FROM people;

Describe the solution you'd like
The logical plan should accept Statement::CreateTable with a non-empty with_options, so that systems that use DataFusion for planning don't need to fall back on the raw AST.
Describe alternatives you've considered
We're currently checking the raw AST to see if it has with_options and manually translating that.
Additional context
No response

👋  welcome!
This seems reasonable to me -- CREATE TABLE already has some version of this:
https://docs.rs/datafusion-expr/23.0.0/datafusion_expr/logical_plan/struct.CreateExternalTable.html
    pub options: HashMap<String, String>,

Are you thinking to make a new CreateTable statement with the various parsed bits?


updated patch

Hi Ferdinand! Thank you for the contribution. Unfortunately, the ultimate problem that Command wanted to solve cannot actually be solved. :( We're going to deprecate this gem and remove it completely soon. I'll close this issue, but thank you for your contribution!


Trace Exemplars to just "Exemplars"

Rename the header to say "Exemplars". Was there any reason why it was called "Trace Exemplars" originally? After all, exemplars is a Metrics feature, and one use-case of it is to allow correlating to trace, so just "Exemplars" or "Metric Exemplars" is more suited here.

I assume it's that because the only current implementation of exemplars is for metrics -> traces, but yeah, I'd agree that we should just say 'exemplars'

/fix:format


eslint-config-prettier is in dependencies, not devDependencies

It looks like eslint-config-prettier is in the dependencies section of package.json - judging by where all the other eslint packages are, I'm guessing it should be in devDependencies.

Thanks for reporting this, I've fixed it in #6

Fix released in https://github.com/node-loader/node-loader-core/releases/tag/v1.0.5


Create PostgreSQL Cheat Sheet

Make PostgreSQL Cheat Sheet

Hello @Ruhul12, can you add the cheatsheet to the readme also?

@LeCoupa Yes, I can. I will do it.

added Postgresql cheat sheet on readme and typo mistakes


NVDA Repeats Controls Twice On Activity View Switcher Control

Issue Type: Bug
Switch to this control and navigate through it. We can eliminate the title attribute if that is what is duplicating the information.
VS Code version: Code - Insiders 1.48.0-insider (731f9c25632dbbf01ee3a7892ad9d2791fe0260c, 2020-07-24T05:57:10.466Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-5500U CPU @ 2.40GHz (4 x 2397)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.87GB (3.37GB free)

Process Argv
--crash-reporter-id ab8e342c-4a51-45aa-9ade-8ae02d47f2e7

Screen Reader
yes

VM
0%

Extensions (14)

Extension
Author (truncated)
Version

author-stats
ada
1.0.4

vscode-languagetool
ada
3.8.0

Bookmarks
ale
11.3.1

spellright
ban
3.0.50

json-tree-view
Cha
2.6.0

vscode-markdownlint
Dav
0.36.1

vscode-pandoc
Dou
0.0.8

vscode-hugo-snippets
fiv
0.4.1

readabilitycheck
jem
1.3.0

markdown-shortcuts
mdi
0.12.0

python
ms-
2020.7.96456

vscode-hugo
rus
0.24.0

rewrap
stk
1.13.0

markdown-all-in-one
yzh
3.1.0

@jhomme root cause is https://github.com/nvaccess/nvda/issues/7841
Can you please comment on that issue?
We have to set the title attribute for users reallying on the mouse hover information to discover keybindings. All other screen readers handle this correctly, so I believe this should be fixed in NVDA.

Hi,
I commented on the issue, but I still feel that the ARIA label should be removed if it duplicates the on-screen label. I also still feel that the title attribute should be removed if it is the same as either the on-screen label or the ARIA label. If the title attribute is different, it should not be removed. There is never a need to make the ARIA label duplicate an on-screen label, especially if the on-screen label is inside a label tag and coded correctly.
==========
Jim Homme
Skype: jim.homme
FreeChess: jhomme
Twitter: jimhomme https://twitter.com/jimhome
Facebook: jimhomme
Website: jimhomme.com https://www.jimhomme.com/
From: Isidor Nikolic<EMAIL_ADDRESS>Sent: Tuesday, July 28, 2020 5:18 AM
To: microsoft/vscode<EMAIL_ADDRESS>Cc: Jim Homme<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/vscode] NVDA Repeats Controls Twice On Activity View Switcher Control (#103247)
@jhomme https://github.com/jhomme  root cause is nvaccess/nvda#7841 https://github.com/nvaccess/nvda/issues/7841
Can you please comment on that issue?
We have to set the title attribute for users reallying on the mouse hover information to discover keybindings. All other screen readers handle this correctly, so I believe this should be fixed in NVDA.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/103247#issuecomment-664911384 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRUZJKG327AEXUZK3CARZLR52JVLANCNFSM4PGWBWBA .

Same root cause as https://github.com/microsoft/vscode/issues/96077
I prefer to only have one issue around for that, since the bottom issue is on the nvda side, thus closing this.


Magnet TriggerBot/Fix Viewmodel on sg/aug

Aimlock on people when triggerbotting
settings base on your aimbot settings
basically toggle aimlock when triggerbotting lol
ez and fast xD
傲天认证

epic

@thomas0aotian can u make a fix for the viewmodel fov so it looks like the same viewmodel fov when i spectate people instead of it being zoomed in and looks extremely ugly, since i've changed something that makes my viewmodel go more out when i zoom

@thomas0aotian it doesnt shoot for me even with max fov

Useless PR, this is already in Osiris.

Set the aimbot on key to be the same as the trigger bot on key (this causes it to turn on when the trigger bot turns on)
Disable silent aim on aimbot (this causes you to see the aimbot)
Enable aimlock on aimbot (this causes it to lock on even when not shooting)
Set the aimbot FOV to be the magnet range

@xAkiraMiura I had a complete magnet triggerbot on my paste but I'm posting it out atm

if you havent noticed you have an open pull still. anyone watching the repo gets notifications for this.
nice ego you got there :)


Fix E2E tests in WordPress 5.8

Context

Summary
TLDR, in WordPress 5.8+, the block needs to be focused, before this is out was automatically focused and the inserter closed.
Much of these changes are copied from upstream.
Relevant Technical Choices

To-do

User-facing changes

Testing Instructions

QA

[x] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

UAT

[x] UAT should use the same steps as above.

This PR can be tested by following these steps:

Reviews
Does this PR have a security-related impact?

Does this PR change what data or activity we track or use?

Does this PR have a legal-related impact?

Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #8160

Closing in favour https://github.com/google/web-stories-wp/pull/8223


NullPointerException in TestTool.checkpoint

After using the latest ladybug version I'm getting occasional NullPointerExceptions.
It happend when running the larva scenarios in IAF-test on one of the /cache/sender scenarios.
NPE.txt

NullPointerException.txt
Ook in ibis4titan op testomgeving dezelfde fout, FF! 7.7-20211101.091958.

Should be fixed with https://github.com/ibissource/ibis-ladybug/commit/ac8cecc87681d2957f2d846f22c011e4f1f97404


Permission Issue in android 13

Update the android 13 permission changes..

Hi @sharawanan  , Can you please ensure you are using the latest version of this library? it's the newest version I already fixed it. please confirm me.


Extending Bridge namespace is causing compiler error

Expected
meaningfull error message instead of exit(1)
Actual
Executing: mono /opt/app/other/Bridge/Bridge.Builder.exe -lib /tmp/Foo/bin/Debug/Test.dll
Command line arguments:
        -lib /tmp/Foo/bin/Debug/Test.dll
Reading configuration file bridge.json at /tmp/Foo for configuration  ...
Getting configuration file path bridge.json at /tmp/Foo ...
Getting configuration file path done - /tmp/Foo/Bridge/bridge.json
Reading base configuration at /tmp/Foo/Bridge/bridge.json ...
Setting logger configuration parameters...
ERROR: Custom command failed (exit code: 1)

Steps To Reproduce
namespace Bridge
{
    [External]
    public static class Console
    {
        public static extern void Warn(params object[] args);
    }
}

namespace Test
{
    using Bridge;
    using Bridge.Html5;

    [Namespace(false)]
    public class MainClass
    {
        [Ready]
        public static void Main()
        {
            Bridge.Console.Warn("a");
        }
    }
}

mdtool build Test.csproj

Ahh!! Forgot to set logging level
"logging": {
         "level": "Trace"
     }

Now works as expected ;)

I will check whether an error message should be shown even if  level is None

Moving to 16.0 milestone.

Hi @kcudnik,
We did decide to include this into 15.0.
Console loggers enabled to log errors even if current loggerLevel is set to LoggerLevel.None.
For console Bridge Compiler it would look like this.


Increase latest feed articles number

Tasks

[ ] Increase latest feed article number (from cc)

Issue requires review and rearrangement. Close it.


Vulnerability of dependency "github.com/gorilla/websocket"

Hello, we are a team researching the dependency management mechanism of Golang. During our analysis, we came across your project and noticed that it contains a vulnerability (CVE-2020-27813). In your project, the github.com/gorilla/websocket package is being used at version github.com/gorilla/websocket v1.4.0, but the patched version is v1.4.1. To fix the vulnerability, we recommend modifying the go.mod file to update the version to v1.4.1 or higher. Thank you for your attention to this matter.

see also https://github.com/joewalnes/websocketd/pull/446


System V 'matherr' support removed from 'glibc' as of version 2.27

Support for the deprecated System V (SVID) matherr exception handling mechanism has been removed from_glibc_ as of version 2.27.  With that version now standard in at least Ubuntu 18.04 LTS, an upgrade to the machinery of math_error(7) is now required for least for our Linux distributions.

Addresses by pull request #49


Failed to send web response body

Hi,
i've created a Web User Response and linked it to my assets.
Unfortunately when triggering a request that gets blocked, just a blank page is visible in the browser window.
Browsers dev tool shows a 403 which is expected.
Attached you can see a screenshot of my Web User Response config.
The nginx-attachment is logging the following when triggering a request:
ngx_http_cp_finalize_rejected_request@ngx_cp_custom_response.c:323 [uid 1 | pid 22] <session id 7> | Failed to send web response body
Thanks,
Christian

Hi @cm91,
Thank you for reporting this, we have successfully recreated the issue, and are looking into it.
I'll update here on any progress we make.
Best,
open-appsec team

Hi,
The issue was fixed as part of our 1.1.10 release, thank you for your patience!


fix: readme invalid header

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[DOC] Material UI Tutorial

Documentation issue
The Material UI Tutorial has incorrect directory references for the boilerplate project.
At "Create a List Page", the instructions say to create a directory "interfaces" under "/src". However, the "list.tsx" file calls the interface like it's a library import { IPost } from "interfaces" not a relative directory.
Later in the step, when adding import { PostList } from "pages/posts" it's the same issue - that's not a relative file or directory. It's calling it like a library.
Describe the thing to improve
Inside "src/pages/posts/list.tsx" revise from import { IPost } from "interfaces" to import { IPost } from "../../interfaces".
Inside "src/App.tsx" revise from import { PostList } from "pages/posts" to import { PostList } from "./pages/posts/list".
Describe the solution (optional)
No response

Hey @cboydstun,
At "Setting Up", you can see the following warning in the use create-react-app section:

Warning: This tutorial assumes your project is configured for absolute imports. Since CRA does not configure it by default, you should configure it yourself to be able to use absolute imports. You should configure a tsconfig.json with baseUrl setting pointing to the /src directory in your project.

We will update the tutorials to be more descriptive for all the steps. Thanks for the feedback! 🙏🏻


Matching cannot be initialized for parity check matrix for 3D codes cannot

When the parity check matrix for the X stabilizers of the 2x2 2D toric code is used to initialise a Matching object, the following error is returned in version  v0.7.0. It had previously worked in version v0.2.4 without any error.
import numpy as np
from pymatching import Matching
H = np.array([
    [1, 0, 1, 0, 1, 1, 0, 0],
    [0, 1, 0, 1, 1, 1, 0, 0],
    [1, 0, 1, 0, 0, 0, 1, 1],
    [0, 1, 0, 1, 0, 0, 1, 1]
], dtype=np.uint)
syndrome = np.array([
    0, 0, 0, 0
], dtype=np.uint)
matcher = Matching(H)
matcher.decode(syndrome)

The following error message is produced
File ~/Documents/bn3d/panqec/venv/lib/python3.8/site-packages/pymatching/matching.py:522, in Matching.load_from_check_matrix(self, H, spacelike_weights, error_probabilities, repetitions, timelike_weights, measurement_error_probabilities, **kwargs)
    520         v1 = H.indices[s] + H.shape[0] * t
    521         v2 = H.indices[e - 1] + H.shape[0] * t if e - s == 2 else next(iter(boundary))
--> 522         self.matching_graph.add_edge(v1, v2, {i}, weights[i],
    523                                      error_probabilities[i], error_probabilities[i] >= 0)
    524 for t in range(repetitions - 1):
    525     for i in range(H.shape[0]):

ValueError: Edge (0,2) already exists in the matching graph, but parallel edges are not supported. Consider merging parallel edges into a single edge instead.

Perhaps is it possible to have backwards compatibility?

Having what I think is the same problem trying to initialize a Matching object for the 17-qubit surface code.
import numpy as np
from pymatching import Matching
Hz = np.array([[0, 1, 1, 0, 1, 1, 0, 0, 0],
       [0, 0, 0, 1, 1, 0, 1, 1, 0],
       [1, 0, 0, 1, 0, 0, 0, 0, 0],
       [0, 0, 0, 0, 0, 1, 0, 0, 1]])
Matching(Hz)

And this produces exactly the same error message (but with edge (2,4)).

The reason for this issue is that parallel edges aren't currently supported in pymatching (for the check matrices you have both given, there are duplicated columns) although I will add support for parallel edges in a future version. You can fix the issue with the current version by either (1) keeping only the parallel edge with the smallest weight, or (2) merging parallel edges. (2) is probably the better option since the error mechanisms of parallel edges are indistinguishable. Suppose edges e1 and e2 are parallel and have probabilities p1 and p2. Assuming they represent independent error mechanisms, they can be merged into a single edge e3 with probability p3=p1*(1-p2)+p2*(1-p1) and corresponding edge weight w3=log((1-p3)/p3). More generally, if you have many parallel edges with probabilities given in some list probabilities, then you can use the following function to find the probability of the merged edge (the probability that an odd number of the independent parallel edge error mechanisms occurred):
import functools

def prob_odd_number_of_errors(probabilities: List[float]) -> float:
    return functools.reduce(lambda x, y: x * (1 - y) + y * (1 - x), probabilities, 0.0)

In a future pymatching version, I'll probably just use the smallest-weight parallel edge by default, but provide an option to merge edges. I'll leave this issue open until then.

This is now fixed and will be in the next release. There will be a merge_strategies argument when loading from check matrix or adding an edge, which can be used to determine how pymatching merges parallel edges (with options "disallow", "independent", "smallest-weight", "keep-original" and "replace").


Remove route configuration for instances

Now that we have Virtual Private Gateways and NAT gateways provided by AWS, we no longer need to be able to set up sink instances for VPN and NAT. Also, we haven't been using it anyway.

Looks Good.


How to create a vmbuilder for debian

Wanted to try a debian using their https://cloud.debian.org/images/cloud/, but the ubuntu script download vmlinuz/initrd from unpacked cloud image, how to get them in order to create (and PR) a script for debian ?

I have created a script like the ubuntu vmbuilder script for creation of a debian VM:
https://gist.github.com/remyvv/62954ecb033fade09e4407fd9a80365e
References used:

#25
https://gist.github.com/alexsarmiento/c52874738c3bdb96424c57b297b2678d
https://github.com/evansm7/vftool/issues/17
https://github.com/evansm7/vftool/pull/20/commits/33539a4229a0dc332a38d2685a760d8a8d410bbf


Allow committing / adding empty files using the web ui (#8420)

Hey,
this commit tries to fix the issue #8420.
The Problem
It wasn't possible to commit empty files using the online editor. There was also no error when the content of the file was empty, it just did not work.
Empty files are needed to force Git to clone empty folders, for example .gitkeep.
My solution
I removed the required tag for the edit_area and now it's possible to commit empty files.

Would it be reasonable to add a modal block to ask if you're sure you want to commit an empty file?

Yeah that's sounds like a good idea.

I created a modal to confirm the commit of an empty file. For the modal I needed to create two new translations, I don't know whether the place, I put them, is the best one.
And here's a screenshot of the modal:

Codecov Report

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

@@            Coverage Diff             @@
##           master    #8532      +/-   ##
==========================================
- Coverage   41.72%   41.72%   -0.01%
==========================================
  Files         526      526
  Lines       68403    68403
==========================================
- Hits        28542    28541       -1
- Misses      36213    36217       +4
+ Partials     3648     3645       -3

Impacted Files
Coverage Δ

modules/auth/repo_form.go
41.73% <ø> (ø)
:arrow_up:

modules/sync/unique_queue.go
78.57% <0%> (-10.72%)
:arrow_down:

modules/task/migrate.go
24.35% <0%> (-3.85%)
:arrow_down:

models/error.go
32.08% <0%> (-1.25%)
:arrow_down:

models/repo.go
48.94% <0%> (+0.05%)
:arrow_up:

modules/migrations/gitea.go
11.75% <0%> (+1.49%)
:arrow_up:

modules/migrations/migrate.go
22.9% <0%> (+1.67%)
:arrow_up:

Continue to review full report at Codecov.

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

One thing the current check also prevents against is accidentally committing an existing file with no changes. It seems this would loose that feature and if you click edit and then accidentally hit enter it will create a new commit with no content and default message like "Update README.md".
Perhaps this can check the difference between adding a new file and editing an existing one?

Edit: Actually I'm wrong and the issue below already can happen so this PR doesn't change anything with regards to that (though if it were easy to show a modal on that as well I think its in the same scope of committing empty files -- nbd if not).

Your idea sounds good, but I guess now I have to create a new pull request for this feature.

Not sure how you'd do this. Monitor change events on the textarea?

I thought a little bit about this and came upon a condition when not to show a modal: the content of file could be equal, if the user just wants to change the path of the file.
So first I'll monitor the hidden input #tree_path for changes. An easy way to do this, is to add a new data option to the template and set it to the .TreePath value at render time.
<input id="tree_path" value="{{.TreePath}}" data-initial="{{.TreePath}}" ... />
When the user wants to commit, just grab this data option and compare it.
That's it for the check of the filename.
I'm thinking about using the same method to compare the content of the file with the initial content, but I don't know how this affects the performance especially for large files (this would be easier).
My other idea for the content would be to use the api endpoint for the diff (see initCommentPreviewTab) and check if nothing has changed.


xclim "time difference between dates with different calendars" on euro-cordex daily data

Cordex Python Package version: '0.2.0a1'
Python version:
Operating System:

Description
Trying to use xclim atmos indices, such as "growing degree days" (tas) and "cumulative Dry Days" (pr), on daily euro-cordex ensemble.
What I Did
Using:  https://github.com/euro-cordex/py-cordex/blob/master/notebooks/euro-cordex-ensemble.ipynb
Changed: 'month' data to 'day' data.
Tested xclim index on single file, it works. But using it on the data set, even when I have a single model "GCM-RCM.historical" it crashes because 'the calendars' are different. It may be related to "use_cftime": True vs  "only_use_cftime_datetimes": True   when we use ".to_dataset_dict" See issue: https://github.com/Unidata/cftime/issues/128
But I could not figure it out.
Growing degree days Index
Sum of average temperatures above 5.6 °C (°C-days); Rivington et al. (2013)
with dask.config.set(**{'array.slicing.split_large_chuncks': False}):
    for exp in dset_split.keys():
        for name, ds in dset_split[exp].items():
            gdd[name] = xclim.atmos.growing_degree_days(tas=ds, thresh='10.0 degC', freq='YS')
gdd

traceback error: ...
src/cftime/_cftime.pyx in cftime._cftime.datetime.__sub__()
TypeError: cannot compute the time difference between dates with different calendars

related to https://github.com/Unidata/cftime/issues/128


[WORKFLOWS-447] Create Dockerfile for sagetasks

I'm re-using Pipfile.lock to create a containerized Python environment with sagetasks installed. Because the package version is inferred from the Git metadata, I had to fiddle with the pipenv install command based on these instructions.

Codecov Report

Merging #7 (4d2c529) into bgrande/WORKFLOWS-447/packaging (a38b3b3) will not change coverage.
The diff coverage is n/a.

@@                       Coverage Diff                       @@
##           bgrande/WORKFLOWS-447/packaging      #7   +/-   ##
===============================================================
  Coverage                             1.16%   1.16%
===============================================================
  Files                                    8       8
  Lines                                  342     342
  Branches                                37      37
===============================================================
  Hits                                     4       4
  Misses                                 338     338

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

@thomasyu888 I haven't tested that yet, but I don't see why not. I can't think of any additional software requirements other than Prefect and Sagetasks (with its dependencies).


Updating French/Spanish/German for translation

Description

Update of the French, Spanish (Castellano) and German translation.
These are only the ts files, not the qm

Type of change

[ ] Bug fix~~
[ ] New feature~~
[ ] Documentation update~~
[x] Other~~
Translation

How Has This Been Tested?

Reminders:

Please follow the style guidelines of this project.
Please perform a self-review of your code before submitting a PR and asking others to review it.
Please assign reviewers (see VisIt's PR procedures for more information).

Checklist:

[ ] I have commented my code where applicable.~~
[ ] I have updated the release notes.~~
[ ] I have made corresponding changes to the documentation.~~
[ ] I have added debugging support to my changes.~~
[ ] I have added tests that prove my fix is effective or that my feature works.~~
[ ] I have confirmed new and existing unit tests pass locally with my changes.~~
[ ] I have added new baselines for any new tests to the repo.~~
[x] I have NOT made any changes to protocol or public interfaces in an RC branch.~~

I failed/neglected to ask for a reviewer.
Maybe @markcmiller86 would you ?

@cessenat please remind me...what is the difference between ts and qm files? Are qm generated from ts?

@cessenat please remind me...what is the difference between ts and qm files? Are qm generated from ts?

Exactly: ts files are ASCII, qm files are those used by VisIt at runtime when 'visit -lang fr...' and obtained by the command
lrelease visit_fr.ts -qm visit_fr.qm


Alternative number bases

Like in other languages! 0b10110 for binary, 0x4f for hex, and so on.

Done.


Update leader election settings

This PR updates the leader election settings and renames the leader election options from
--renew-deadline ==> leader-election-renew-deadline
--retry-period ==> leader-election-retry-period

/lgtm

/lgtm


[FancyZones] Do not try to zone minimized windows

Summary of the Pull Request
We shouldn't try to move minimized windows into zones, since that interferes with applications' ability to start itself minimized.
What is this about:
What is include in the PR:
How does someone test / validate:
Quality Checklist

[x] Linked issue: #11010
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[x] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

This PR is also fixes https://github.com/microsoft/PowerToys/issues/4176

Can we fix the conflict and merge in?


Error with patch file in v0.18.1

Hello Sir,
I have tried the qbactivator for V19. I still get the same messages, patcher is not found and if I run the script with powershell, I get the message "error patch file is corrupted" please contact...
There seems to be something wrong..

Originally posted by @861861 in https://github.com/neuralpain/qbactivator/issues/1#issuecomment-1396882202 (for qbzctivator v0.16.1)

Have you tried re-downloading the activator?
Alternatively you can download the patch file from the /src folder for v0.18.1 and test the activator with that one.

Your issue is different from #1 since the error is with a corrupt patch file but you shouldn't be having an issue if you downloaded the archive directly from the releases page. Have you tried re-downloading the activator?
Alternatively you can download the patch file from the /src folder for v0.18.1 and test the activator with that one.


Create Book Appointments Front-end

This branch includes the following features:

[x] Created front-end for Book Appointments application.
[x] Added Redux store.
[x] Added Redux reducers to handle authentication and API data.
[x] Created Sign up and login forms.
[x] Created navigations.
[x] Created components for all pages.
[x] Added API-docs links.
[x] Applied styles
[x] Added responsiveness.

Link to the backend: Book Apponitments back-end

@mcrd25 Hi, Thank you for your time and feedback 😄

@mcrd25 Hi, Thank you for your time and feedback 😄

Np!
If you liked my review, feel free to connect on LinkedIn and leave a skill endorsement: https://www.linkedin.com/in/mayadouglas/
You are also free to follow me on GitHub


Publish to AppSource workflow

I tried publishing an app using the workflow and am getting the following error message:
WARNING: AuthContext.Scopes is 'https://api.businesscentral.dynamics.com/', should have been 'https://api.partner.microsoft.com/.default'
When trying "New-BcAuthContext" with the proper scope, it works, but it seems that the scope isn't set correctly when running the workflow? Am I missing something in the settings? I do not have it set to automatic delivery during build.

When you say the scope isn't set correctly when running the workflow - what do you mean?
The scope is part of the authcontext and not set in the workflow (or maybe i misunderstood something) - it is just validated in the workflow.

You answered my question basically... The scope was missing in the AppSourceContext.
Second question in the process: Is there a way to define an external repo as the library app (or "apps") when publishing to AppSource? I can only see the "appSourceMainAppFolder" and it seems that the "appDependencyProbingPaths" is not used in the process. Is there any other way to define this?

Right now, the remaining apps from the same repository are attached as library apps.
Apps from other repositories are expected to be submitted from there.

Thank you!

BTW - there is a function called
New-ALGoAppSourceContext -authContext $authcontext

which will test and format your authcontext correctly.
you can add | Set-Clipboard to put the value into the clipboard.


[Help Needed] - Is a on-prem supabase install supported ?

Expose the next_auth schema
Expose the next_auth schema in the API settings by adding next_auth to the "Exposed schemas" list.
More information here.
Then copy the output and save it for the next step.
API settings don't exists in the UI in a self hosted instance.

I just created a PR #83 where you can find the settings for your local Supabase instance. In the config.toml you can define the API schemas to expose:
https://github.com/jorge-menjivar/unSAGED/pull/83/files#diff-56c3d03bfaddf3efa07a4efe3dd767c0ecc36441ffe865ee5865794b066ee2faR10

@sebiweise Where is the [supabase/config.toml] file in the docker setup ? Which container does host that ? Any docker env. variables to set this up ?

The config.toml I´ve linked in the privous comment is just for the supabse-cli to get your development env up and running.

Hmm I'm using the supabase docker self-hosted setup.
https://supabase.com/docs/guides/self-hosting/docker


New Filter, Plot, and Explore Tutorial Using Seurat Tools!

New filter, plot, and explore tutorial (mimicking currently available Scanpy and Seurat RStudio ones) using Seurat Tools by Pablo!

I pushed some changes to make the CYOA work (you needed hyphens in the class names) and make the branches more consistent but now it looks good formatting-wise. @pavanvidem is the technical side ok as well?

If I understood it correctly, there are two ways of generating Seurat objects based on the input data:

EBI-Data-Retrieval and Importing-from-a-history should use Read10X tool
Uploading-from-Zenodo should Filter Cells tool

@Camila-goclowski please correct me if I'm wrong.
In the last commit, I created a new div with both the EBI-Data-Retrieval and Importing-from-a-history classes. Hope that works as needed.

@hexylena how to know which div is visible by default on page load? When I served on my laptop, content from all the divs is shown by default.

It would be nice to provide a link to a workflow or a history to compare the results. We have them in most of the single-cell tutorials.


start branch

I have made changes.

Pull all changes from dev branch. It's not relevant fixes.

Please change enpoint to sign-in so GitHub CodeSpace will work properly


Not working

The error occurs when starting
[16:51:53 INFO]: Enabling ViaPotions v0.0.3
[16:51:53 ERROR]: No compatible plugins have been detected, disabling the plugin.
[16:51:53 ERROR]: In order to make ViaPotions functional, ViaRewind must be installed.
[16:51:53 INFO]: Disabling ViaPotions v0.0.3

I have ProtocolSupport (1.16.5-3) and ProtocolLib (4.7.0) installed.

We don't support ProtocolSupport anymore. We did this because the latest versions wasn't compiling.

We don't support ProtocolSupport anymore, only ViaRewind. We did this because the latest versions wasn't compiling.

What was the version supported in e5c95fe?

We don't support ProtocolSupport anymore, only ViaRewind. We did this because the latest versions wasn't compiling.

What was the version supported in e5c95fe?

I don't know but it requires Java 16, doesn't have official maven repo and it's slow. There is no reason to run ProtocolSupport anymore as ViaRewind/ViaBackwards does a better job.


Dead bodies can be revived without cloning, can not control body.

Description
CMO continued to try and revive me with hypo and im already dead but he can't tell because theirs not dead text, i think if you hit a dead body enough with meds it shows alive.
Reproduction
no idea but maybe if you hit a dead body enough with meds it shows alive.
Screenshots

Additional context

Dead body stands up and can not re enter round because of this bug.

disregard i guess if you press return to body you reenter.

What is the expected/desired behavior in this case?
If a patient is Dead (with or without a ghost observer spawned), should medicine still work on them?
If we allow this, would the ghost be automatically brought back to the body if it is returned to a Critical/Normal state?

just make beds not able to heal if someone is dead?
the same thing that happend to bruise packs and ointment, cant use used on dead targets


Proposal: Enum Literals Support

See Discussion in #56
I actually think we're able to get best of both worlds here by using the mypy Literal type. This way, the typing will be indeed an int, but the int is constrained to only the legitimate values.
https://mypy.readthedocs.io/en/latest/literal_types.html
First attempt at a diff was #79
Discussion after landing:
@Kami

First thanks for adding enum literal support.
This indeed seems to fix direct enum literal notation, but it breaks the Value() method scenario which takes a string (enum name) and returns an enum value.
For example, with the following definition:
enum ExecutionStatus {
    REQUESTED = 0;
    SCHEDULED = 1;
    RUNNING = 2;
}

This used to work:
from messages_pb2 import ExecutionStatus

status_str = 'requested'
obj.status = ExecutionStatus.Value(status_str.upper())  # this should return enum value (aka Enum type / int)

Now it returns the following error:
error: Incompatible return value type (got "Union[Literal[0], Literal[1], Literal[2]]", expected "ExecutionStatus")

It works fine with just the change from #81, but not with this change.
It may also brake Name() and other enum class methods, but I didn't have time to dig in yet.

@nipunn1313

I poked around a bit and I believe that the issue here is that obj.status is typed to ExecutionStatus, but it would now need to be typed to ExecutionStatus.ClosedValueType
since ExecutionStatus is the enum itself and ExecutionStatus.ClosedValueType is the type of the Values in the enum (and ExecutionStatus.ClosedKeyType is the type of the Keys in the enum)
The merged PR was non-backward-compatible in that it changed the return value of Typing to literals, and move the int-y-ness of the enum to the ClosedValueType member.
This is more accurate based on this example I found
enum UserFSWAction {
   UNKNOWN = 0;
   ALLOW = 1;
}

>>> type(ALLOW)
<class 'int'>
>>> ALLOW
1
>>> UserFSWAction.Name(1)
'ALLOW'
>>> UserFSWAction.Value('ALLOW')
1
>>> type(UserFSWAction.Value('ALLOW'))
<class 'int'>

@Kami - does that make sense directionally?
I'll hold off on publishing to pypi until we have some alignment here. I'm ok with making a non-backward compatible change, but I'll want to have some clear changelist and bump the big version number.
Couple more considerations for proposal
change typing of the constants to be the ClosedValueType for better error messages
- ALLOW: typing_extensions___Literal[1]
+ ALLOW: UserFSWAction.ClosedValueType

change ClosedValueType to just ValueType to simplify notation required by callers.

For the past two weeks of using mypy-protobuf, I was always thinking about this issue and whether its good or not to use Literal for enum values, now I have some conclusions, it may not be the best or the most suitable (for your situation), but I hope it can provide some ideas to improve.
I'll illustrate my ideas by compare the assuming changes to the current code of test_pb2.py:OuterEnum:
class OuterEnum(builtin___int):
    DESCRIPTOR: google___protobuf___descriptor___EnumDescriptor = ...
    @classmethod
    def Name(cls, number: builtin___int) -> builtin___str: ...
    @classmethod
    def Value(cls, name: builtin___str) -> 'OuterEnum': ...
    @classmethod
    def keys(cls) -> typing___List[builtin___str]: ...
    @classmethod
    def values(cls) -> typing___List['OuterEnum']: ...
    @classmethod
    def items(cls) -> typing___List[typing___Tuple[builtin___str, 'OuterEnum']]: ...
    FOO = typing___cast('OuterEnum', 1)
    BAR = typing___cast('OuterEnum', 2)
FOO = typing___cast('OuterEnum', 1)
BAR = typing___cast('OuterEnum', 2)

Firstly, I think def Name should accept any int value, not just restricted to the defined enum values. Assuming we are writing some code to convert int to enum name, the int may come from anywhere like json, database, user input etc, so it's possible to be an invalid enum value:
def int_to_enum(i: int) -> Optional[str]:
    try:
        return OuterEnum.Name(i)
    except ValueError:
         return

looking at this code, i is totally arbitrary but as expected, if def Name is annotated as Union[Literal[0], Literal[1], ...], it would be very weird and unnecessary. So I think using Union[int, OuterEnum] for Name is enough:
--- current.py	2019-08-06 21:28:13.000000000 +0800
+++ one.py	2019-08-06 21:22:50.000000000 +0800
@@ -1,7 +1,7 @@
-class OuterEnum(builtin___int):
+class OuterEnum:
     DESCRIPTOR: google___protobuf___descriptor___EnumDescriptor = ...
     @classmethod
-    def Name(cls, value: builtin___int) -> builtin___str: ...
+    def Name(cls, value: Union[builtin___int, 'OuterEnum']) -> builtin___str: ...
     @classmethod
     def Value(cls, name: builtin___str) -> 'OuterEnum': ...
     @classmethod

Furthermore, I think the most important thing we want from pyi is the information and suggestions about type, not value, define all enum names and values in Literal is just too bloated, I can't see much benefits from it.
Secondly, we know the truths are, OuterEnum is an object of EnumTypeWrapper, and enum values are nothing but builtin int values, why not we just annotate enum exactly as the implementation of pb2 files, rather than pretend OuterEnum is a subtype of int? There may be historical reasons or special needs in Dropbox's code for that, but from my perspective the idea of just using int as enum value type will make the understanding much simpler, Literal is also not considered in this situation:
--- current.py	2019-08-06 21:28:13.000000000 +0800
+++ two.py	2019-08-06 21:29:03.000000000 +0800
@@ -1,16 +1,16 @@
-class OuterEnum(builtin___int):
+class OuterEnum:
     DESCRIPTOR: google___protobuf___descriptor___EnumDescriptor = ...
     @classmethod
     def Name(cls, value: builtin___int) -> builtin___str: ...
     @classmethod
-    def Value(cls, name: builtin___str) -> 'OuterEnum': ...
+    def Value(cls, name: builtin___str) -> builtin___int: ...
     @classmethod
     def keys(cls) -> typing___List[builtin___str]: ...
     @classmethod
-    def values(cls) -> typing___List['OuterEnum']: ...
+    def values(cls) -> typing___List[builtin___int]: ...
     @classmethod
-    def items(cls) -> typing___List[typing___Tuple[builtin___str, 'OuterEnum']]: ...
-    FOO = typing___cast('OuterEnum', 1)
-    BAR = typing___cast('OuterEnum', 2)
-FOO = typing___cast('OuterEnum', 1)
-BAR = typing___cast('OuterEnum', 2)
+    def items(cls) -> typing___List[typing___Tuple[builtin___str, builtin___int]]: ...
+    FOO: builtin___int = 1
+    BAR: builtin___int = 2
+FOO: builtin___int = 1
+BAR: builtin___int = 2

That's it, what do you think?

yep - that sounds reasonable - I think the typing as a subclass of int hasn't added value.
As for using Literal, see https://github.com/dropbox/mypy-protobuf/issues/56#issuecomment-445752916 for the motivation. Without a significantly compelling reason, it's simpler not to add Literal - but will leave this task open for discussion.
Removing the subclassing from int feels independently valuable based on this discussion.
Thanks!

Closing issue now that we have an enum value type!


Merge Overlapping Sub-Intervals in C++

I want to implement Merge Overlapping Sub-Intervals in C++.
@HarshwardhanPatil07 Please assign this issue to me under Hacktoberfest-2024. Thank You!

@HarshwardhanPatil07 Please assign this issue to me under Hacktoberfest-2024.

@HarshwardhanPatil07 Please assign this issue to me under Hacktoberfest-2024.

Please assign this issue to me under Hacktoberfest-2024, and add label Hacktoberfest in it .


Update default.nix

libinput-gestures: 2.76 -> 2.77

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Please follow the commit conventions.

Please follow the commit conventions.

Ok, updated.

No, you've edited the GitHub PR description but not the underlying Git commit. I think you need to use the Git CLI for this; type git commit --amend and then push with --force-with-lease.

libinput-gestures: 2.76 -> 2.77

I was unable to make the updates with the commands you suggested, so I made a new pull request: https://github.com/NixOS/nixpkgs/pull/346289

No, you've edited the GitHub PR description but not the underlying Git commit. I think you need to use the Git CLI for this; type git commit --amend and then push with --force-with-lease.

I was unable to make the updates with the commands you suggested, so I made a new pull request: https://github.com/NixOS/nixpkgs/pull/346289

Recreated here: https://github.com/NixOS/nixpkgs/pull/346289


Hi, I'm getting this error

While trying to import MUXSDKKaltura I get this error.
I'm installing it with cocoa pods

There should be an update for this issue going out today. Thank you for your report!

Hello! Version 2.0.0 was just release with a fix for this issue. Thanks for your patience and again for your report!


StorIOSQLite: add support for notification tags

Add the same thing like [affectsTables] ( https://github.com/pushtorefresh/storio/blob/c8c4f8a58888c97594acef011722f86fed9a0d11/storio-sqlite/src/main/java/com/pushtorefresh/storio/sqlite/queries/RawQuery.java) in all query classes, to make it possible to emit notifications of some 'uris', real tables for which aren't exist.
storIOSQLite.observeChangesInTable(TAG)
InsertQuery.builder().table(TABLE).tags(TAG).build()
@artem-zinnatullin, @thevery will it make sense or variant with [manual notifyAboutChanges call]
(https://github.com/pushtorefresh/storio/compare/dummy-table-changes?expand=1) is also applicable?

I'd start without API for manual notifyAboutChanges because DB operations
should be single source of true for such system. Let's see how users will
use this feature and if somebody will request manual API.
On Thu, 16 Jun 2016, 01:55 Dmitrii Nikitin<EMAIL_ADDRESS>wrote:

Add the same thing like affectsTables
https://github.com/pushtorefresh/storio/blob/c8c4f8a58888c97594acef011722f86fed9a0d11/storio-sqlite/src/main/java/com/pushtorefresh/storio/sqlite/queries/RawQuery.java
in all query classes, to make it possible to emit notifications of some
'uris', real tables for which aren't exist.
storIOSQLite.observeChangesInTable(TAG)
InsertQuery.builder().table(TABLE).tags(TAG).build()
@artem-zinnatullin https://github.com/artem-zinnatullin, @thevery
https://github.com/thevery will it make sense or variant with manual
notifyAboutChanges call
https://github.com/pushtorefresh/storio/compare/dummy-table-changes?expand=1
is also applicable?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pushtorefresh/storio/issues/663, or mute the thread
https://github.com/notifications/unsubscribe/AA7B3Ar5Lg_XlILfrsCfQAY2d2iDYrM-ks5qMILzgaJpZM4I2392
.

I don't know if I'm right here....
I have a bunch of raw-sqls which affect different tables (I really don't know which exactly).
After executing these sqls I want to tell manually a defined set of "affected tables".
Is this now possible?

Can't you call something like this?
storIOSQLite
   .executeSQL()
   .withQuery(RawQuery.builder()
      .query("delete from users")
      .affectsTables(users) // table names here
      .build())
   .prepare()
   .executeAsBlocking();

Or what do you mean by don't know which exactly?

Sorry, I should have provided more information.
 I really don't now which tables are affected exactly.
I execute these sqls via executeSQL().
After processing all these sqls I want to to make a "manually affectedTables(syncTables)" at the end (not after every executeSQL()).
worklflow

get sqls from server
executeSQL() every sql from 1)
at the end of 2): manual affectedTables(syncTables)

additional question:
Is it possible to give StorIO not only one sql, but a list of sqls to execute?

When you call executeSQL() DefaultStorIOSQLite uses sqlite#execSQL and according to documentation for it:  Multiple statements separated by semicolons are not supported. =(
But you can generate the list of PreparedOperation and execute it inside transaction.
class SqlFromServer {
   String statement;
   List<String> affectedTables;
}
...
storIOSQLite.lowLevel().beginTransaction();
try {
   for (SqlFromServer sqlFromServer : sqlsFromServer) {
      storIOSQLite
         .executeSQL()
         .withQuery(RawQuery.builder()
            .query(sqlFromServer.statement)
            .affectsTables(sqlFromServer.affectedTables)
            .build())
         .prepare()
         .executeAsBlocking();
   }
   storIOSQLite.lowLevel().setTransactionSuccessful();
} finally {
   storIOSQLite.lowLevel().endTransaction();
}

All notifications you'll receive after transaction end.
But anyway you have to know what tables you are updating, to prevent parsing sql statements manually. Can't server provide you with it?

SyncTables: I know these tables.
Sqls: I know not exactly which tables they are affecting, but they are inside the "SyncTables".
@nikitin-da I like your example with transaction. I'll give it a try. Thanks a lot.

What about notifyAboutChanges() after executing all sqls?

If "SyncTables" - it is the set of affected tables by all statements, I guess it make sense to call manually notifyAboutChanges() in the end of transaction.

@nikitin-da notifyAboutChanges() is working like a charm! :+1:
And thanks for StorIO!
(Sorry, if this issue wasn't the right one to ask.)


South Africa countrywide

Are these data open ? No source is mentioned.
https://revolutionarygis.wordpress.com/2018/08/09/every-address-in-south-africa

https://github.com/openaddresses/openaddresses/issues/4106 is the issue; they pulled the data and it should be implemented soon


How to open or focus the NativeBase Select component

NativeBase Version
"native-base": "^3.1.0",
Bare "react-native": "0.64.2",
Issue Description
How can I trigger an onPress or open a Select Component
I want open or focus the select component from a function.
It doesn't have an isOpen state I can access.
I tried using the getting the ref and calling the onPress function
return (
      <Select
       wrappedRef={selectRef => console.log(selectRef)}
        selectedValue={language}
        onValueChange={(itemValue) => setLanguage(itemValue)}
      >
        <Select.Item label="JavaScript" value="js" />
        <Select.Item label="TypeScript" value="ts" />
     </Select>
)

but I get this object which doesn't have the onPress function
{
  "_children": [...],
  "_internalFiberInstanceHandleDEV": {
    "_debugHookTypes": null,
    "_debugID": 16533,
   ...
  },
  "_nativeTag": 865,
  "viewConfig": {
    "Commands": {},
    "bubblingEventTypes": {...},
    "directEventTypes": {...},
 ...
  }
}

Me Also Facing Same Issues Like that Please try to resolve this query

bump for this thread. Also experiencing this issue.
It looks like the components build in the library are not build with inversion of control in mind.
The select component is using an internal Pressable which is hooked to an internal state called [open, setOpen].
This makes hooking into this programmatically impossible.
To the makers: could you please gives us an api to open selects, inputs etc programmatically?

I have same issue, any solve?

Hi Everyone, We don't have support to open select programmatically but We might add it in the future versions. Thanks for your suggestions. Closing the issue for now. Feel free to open it if you have any queries.


Update devops-command-center.rst

Editor review

Question: Should https://github.com/mattermost/docs/blob/19d45d0826d3026b3e37961822e9469f91cba2bc/source/administration/devops-command-center.rst#telemetry be added to https://docs.mattermost.com/administration/telemetry.html?

No, the beta telemetry should not be added to the main docs. It is temporarily given to select customers, and will likely be completely rethought as we prepare for Cloud first.

Thanks for the improvements, @amyblais. No, the beta telemetry should not be added to the main docs. It is temporarily given to select customers, and will likely be completely rethought as we prepare for Cloud first.

Thanks so much, @amyblais, much appreciated! :)


Fix: String::format Variant to String conversion

Previosly String::format used variant.get_construct_string() to convert the passed Variants to Strings. This however did not match the expected printing behavior of some datatypes: Strings for example turned out escaped.
This fix replaces the call to get_construct_string() with a simple conversion. This makes String::format consistent with "%s" % "sth" and formated Variants turn out like printed as expected.
Closes #23129

Thanks!


Does not work with Berlin

Stating it works from Xe7+ but it will not compile in Berlin (the only Version I have installed now)
there is no TryAdd in Dictionary

Fixed, thanks!

Fixed, thanks!


Occlusion culling causing unrefining of visible tiles (pretty rare)

I've seen a very rare problem where initially refining an area sometimes causes a few previously visible tiles to unrefine to a parent (or maybe further up ancestor) for a few frames, before it then refines completely to the correct detail. I can only seem to run into the issue after minutes of flying around and carefully looking for it. It doesn't seem to appear when occlusion is disabled (but disabling the "wait for occlusion results" option doesn't seem to fix it though). This might be related to how occlusion culling and external tilesets work together or how occlusion and the kicking mechanism interact.

Got a screen-capture of the issue happening with the output window up, here are a couple of screenshots from it:
Before the glitch:

The glitch (tile dubiously unrefines):

After the glitch:


Getting some visual information in the Readme

Hi I think we should be posting some visual information in the Readme so people have an idea of what it looks now, some screenshots or gifs would work.
Let me know your thoughts on this @argenisleon @JorgeParraAndrade

Yes,
We are working on some custom histogram that will be ready tomorrow and we will put it on the readme.

👍


MAP_READY doesn't load on second Instance

I'm submitting a ...  (check one with "x")
[ x ] question
[ ] any problem or bug report
[ ] feature request
The plugin version:  (check one with "x")
[ ] 1.4.x
[ x ] 2.0.0-beta3
If you choose 'problem or bug report', please select OS:  (check one with "x")
[ ] Android
[ ] iOS
cordova information:  (run $> cordova plugin list)
cordova-plugin-googlemaps 2.0.0-beta3-20170903-1743 "cordova-plugin-googlemaps"

Current behavior:

The map logic is inside the MapsPage, when I enter to the MapsPage the map loads correctly, from there I go to another page, and from that page I go again to the MapsPage (pushing the page, not popping it from the navigation stack), now the map never hits the MAP_READY event.
I don't want to destroy the first map, because I might go back to it later.
What am I missing to see the expected behavior?
Environment info:
local packages:
ionic/app-scripts : 2.1.3
Cordova Platforms  : android 6.2.3
Ionic Framework    : ionic-angular 3.6.0
System:
Android SDK Tools : 25.1.3
Node              : v6.11.2
npm               : 5.0.4
OS                : Windows 10
Expected behavior:

The map should hit the MAP_READY event again (the second time I enter to the MapsPage), with a new id for the map.
Steps to reproduce:

Add maps plugin
Go to the Maps Page
Go from the MapsPage to a second page
Go from that second page to the MapsPage (pushing the MapsPage, not popping it)
The MAP_READY event doesn't fire again

Screen capture or video record:

Related code, data or error log (please format your code or data):
Code for the Maps Page:
.ts file:
  map: any;
  selectedMarker: {
    title: string,
    snippet: string
  }
  constructor(
    private navCtrl: NavController,
    private platform: Platform) {
    this.setSelectedMarker("Hello Friend...", "¡¡¡Click Me!!!");
    let mapCenter: LatLng = new LatLng(-17.3829278, -66.1596725);
    setTimeout(() => {
      this.loadMap(mapCenter).then(success => {
        this.addMarker(mapCenter);
      });
    }, 1)
  }

  private setSelectedMarker(title: string, snippet: string) {
    this.selectedMarker = {
      title: title,
      snippet: snippet
    }
  }

  private loadMap(mapCenter: LatLng): Promise<boolean> {
    var promise: Promise<boolean> = new Promise<boolean>(resolve => {
      this.platform.ready().then(() => {
        console.log("Platform ready!");
        let element = document.getElementById('map');
        this.map = (<any>window).plugin.google.maps.Map.getMap(element, this.getMapConfigurations(mapCenter));
        this.map.one((<any>window).plugin.google.maps.event.MAP_READY, map => {
          console.log("Map Ready!");
          resolve(true);
        });
      });
    });
    return promise;
  }

  private addMarker(pinPos: LatLng) {
    let markerOptions: MarkerOptions = {
      position: pinPos,
      title: "Hello",
      snippet: "World!"
    }
    this.map.addMarker(markerOptions);
  }

  private goToSecondMap() {
    this.navCtrl.push(BlackPage);
  }

  private getMapConfigurations(mapCenter: LatLng) {
    return {
      'controls': {
        'compass': true,
        'myLocationButton': true,
        'indoorPicker': true,
        'zoom': false
      },
      'gestures': {
        'scroll': true,
        'tilt': true,
        'rotate': true,
        'zoom': true
      },
      'camera': {
        'latLng': mapCenter,
        'tilt': 10,
        'zoom': 15,
        'bearing': 15
      }
    }
  }

.html file:
<ion-header>

  <ion-navbar>
    <ion-title>Maps</ion-title>
  </ion-navbar>
</ion-header>

<ion-content>

{{selectedMarker.title}}

{{selectedMarker.snippet}}

</ion-content>

.scss file:
ion-app._gmaps_cdv_ .nav-decor {
    background-color: transparent;
    display: none !important;
  }

  page-maps {
    .mapsSpace{
      height: 100%;
    }

    .map {
      height: 100%;
    }

    .noMapSpace {
      z-index: 1000;
      opacity: 0.8;
      border-radius: 10px;
      display: flex;
      justify-content: center;
      position: absolute;
      bottom: 12vh;
      left: 5vw;
      height: 15vh;
      width: 90vw;
      padding: 5px 0;
      background-color: #1C1C1C; //border: solid 2px color($colors, darkGray);
    }

    .hostSpace {
      margin: 0;
      display: flex;
      align-items: center;
      justify-content: center; //background-color: $cardBackgroundColor;
      border-radius: 10px;
    }

    .hostInfo {
      width: 65vw;
      padding: 0 10px;
    }

    .hostAddress {
      color: white;
      white-space: nowrap;
      overflow: hidden;
      text-overflow: ellipsis;
    }
}

If your problem is solved, please consider small amount donation to this project.
Appreciate for your kindness.
https://github.com/mapsplugin/cordova-plugin-googlemaps-doc/blob/master/README.md#buy-me-a-beer

Please share your project file on github or others.

@wf9a5m75 I invited you to the repo

Thank you. I noticed one thing you use old version of @ionic-native/core.
https://github.com/Guchster/googleMaps-plugin-demo-app/blob/master/package.json#L23-L24
You need to use @ionic-native/core version 4.2.1, or higher

@wf9a5m75 you are right, I updated @ionic-native/core to version ^4.2.1, however the problem remains

Ok, I'm building your app currently. Please wait for a while.

I noticed you don't use the @ionic-native/google-maps ...

I see. This is not bug.
Since you display the same map page twice times, you are trying to create map views for the same map div.
This plugin does not allow it.

If you want to do that,

you need to map.remove() / map.getMap() or   // regenerate map instance every times
map.setDiv(null) / map.setDiv(mapDiv)   // keep the one instance

That's the problem, as you said I am trying to create two map views for the same map div.
I don't understand though how to keep one instance of the map.
I am trying to save the map instance in a provider (dataManager), and if it's not empty when I enter the maps page, i assign to it the current mapDiv.
Like this:
  private loadMap(mapCenter: LatLng): Promise<boolean> {
    var promise: Promise<boolean> = new Promise<boolean>(resolve => {
      this.platform.ready().then(() => {
        let mapDiv = document.getElementById('map');
        let map = this.dataManager.getMap();
        if (map == null) {
          this.map = (<any>window).plugin.google.maps.Map.getMap(mapDiv, this.getMapConfigurations(mapCenter));
          this.map.one((<any>window).plugin.google.maps.event.MAP_READY, map => {
            console.log("Map Ready!");
            this.dataManager.setMap(map);
            resolve(true);
          });
        } else {
          this.map = map;
          this.map.setDiv(mapDiv);
          resolve(true);
        }
      });
    });
    return promise;
  }

However is not working, what would be the correct way to use the map.setDiv(mapDiv) in this case?
Is this logic even going on the right path?

Check this project.
https://github.com/carvemerson/map

@wf9a5m75 Could you recommend a project for me to check out, if I wanted to regenerate the map instance every time, instead of keeping one instance?
I would be very helpfull if I could see a project that does this with map.remove() / map.getMap() as you suggested.
Thanks a lot!

Maybe this?
https://github.com/mapsplugin/v2.0-demo

I have the same issue with my application. The map only load once. At the second time the div is empty in my screen but something I saw debugging my app is inside the div appears this:
__pluginmapid="map_0_XXXXXXXXX"
Every time i load the same page the value change to "map_1_XXXX", "map_2_XXX"..
How I can fix this issue to get always the map? I tried with those examples and is still not working.
Thanks in advance.


Check for existance of homebrew on Mac and return lib path array

Not exactly sure why I ran into so many issues (couldn't find dylibs), but following the standard install instructions did not work for me. I can back fill more info later on, but wanted to start a PR in case it's helpful to others and/or I can figure out what I was missing.

This PR checks for the existence of homebrew on Mac platforms, then returns an array of libraries with the fallback being a full path to where homebrew would have placed a symlinked dylib. It continues to return the library name if the brew command is not found or returns a failure code.
I'm on macOS Catalina (10.15.7) with Ruby 2.6.6 installed using asdf using homebrew installed to a custom directory (~/homebrew).
If this does end up being a solution that is needed for others I can add tests around the changes.

Hi @jonknapp, thanks for looking at this.
This is a really thorny problem on macOS and you'll probably be unsurprised to learn that this has been discussed many times before.
In my opinion, the right place to fix this is in the ffi gem.  Moreover, fixing the issue in ffi ought to fix it for all Ruby gems that use ffi to call native code. ruby-vips used to have more complex lib finding rules, but they've slowly been copy-pasted into the ffi gem.
However! You have a lot of reading ahead of you, unfortunately. Here are some of the ffi issues that have worked on this:
https://github.com/ffi/ffi/pulls?q=is%3Apr+library+search
This whole mess crosses over into security too -- there's been at least 1 CVE in this area where programs could be tricked into loading the wrong library, so you have to be extremely careful.
Good luck!

Thanks for the info. I figured this was a conversation that's occurred over the years with many projects and maintainers. Didn't think a silver bullet was likely but figured I'd hear about it if there was one.
For this project specifically, would this topic be better framed as an open issue with a desired PR opened in ffi? If that's the case, I can create an issue which links to this PR which could be closed.
I'm not sure what a timeline (if any) would look like for a ffi pull request but want to be respectful of your project. I'm also in no hurry for a resolution either if you'd rather leave things as they are for a bit.

Yes, that's a good idea. Make an issue here setting out the problems you are having as clearly as you can, and it'll be a useful reference to a tricky use-case.
I would then quickly read the old ffi issues on this and (if you still want to) open an issue there, perhaps even with a brief proposal. Lars is very helpful and I'm sure he'd think about your ideas carefully.

I'll close. Things should be better now.


Improve documentation including links to docs or modules for further details

The PR description offers insights that are not present in the ecx_l2_connection markdown file.  Perhaps this could be used to improve the Secondary Connection heading in the connection doc.
I noticed that we don't link out to any of the product documentation for the Fabric features. I find links to the backing API and product guides helpful when reviewing Terraform resource documentation.
A guide on Interconnection could discuss this topic in greater depth and could link to the various modules that you've made available on Equinix-labs and Equinix repos.
Originally posted by @displague in https://github.com/equinix/terraform-provider-equinix/pull/111#pullrequestreview-920222471

Spelling errors should also be checked


[Sonar] Remove this expression which always evaluates to "true" in src/main/java/org/trebol/jpa/services/conversion/BillingTypesConverterJpaServiceImpl.java

Summary

Source Sonar issue URL
Issue type: Code Smell
Severity: Major
Technical debt: 10min effort

hey @bglamadrid I can take on this issue if you like .

@shivam-Purohit Correct, it is pretty much that simple! The Sonar issue itself explains it pretty well I think 🙂 the comparison to null is redundant because name has a non-null constraint


without "/* @flow */"

hi, I wonder how i can scan js project without the comment "/* @flow */"

https://flow.org/en/docs/config/options/#toc-all-boolean

This probably can be closed.
/cc @vkurchatkin


Update README.md adding the SonarQube Quality gate badge

This pull request includes a small change to the README.md file. The change adds a SonarCloud quality gate status badge to the documentation.

README.md: Added a SonarCloud quality gate status badge to the top of the file.


Add ability to load different config file types

Now, it's not possible to use svelte.config.cjs or svelte.config.mjs files, the config won't be used at all. Svelte kit doesn't show any error, that the config doesn't exist, so we can face situations, when something doesn't work.
For example, I had problem with typescript (unexpected token error on any specific typescript token) and spent much time to figure out, what's going wrong. But, I just had svelte.config.mjs, not .js and kit was using empty config.
The PR allows using any config file extension to prevent similar unobvious problems.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Please provide more details why you need this and how this addition solves your use case

Please provide more details why you need this and how this addition solves your use case

Done

Perhaps we should improve the warning/error messaging when there is no svelte.config.js but the user has specified the sveltekit plugin in their vite.config.js. Supporting additional filenames just because the user might have accidentally named it something else does not sound like the right solution.
Perhaps we should also be clearer in the documentation that only svelte.config.js is supported, not other file extensions.
Even the skeleton app created by create-svelte includes  svelte.config.js, so I don't know how common it would be for someone to end up with another name for that file.

I'm not sure it should be an error not to have a svelte.config.js. You don't need it in dev mode for a lot of apps. The most common need for it is to specify an adapter. But maybe if we find a svelte.config.mjs or svelte.config.cjs we should throw an error

Even the skeleton app created by create-svelte includes  svelte.config.js, so I don't know how common it would be for someone to end up with another name for that file.

Actually, I got in the situation. postcss throws an error and forces using .cjs or .mjs extension, so I updated all the configs with the extension.
In my vision, the tool which imply using modern standard should support modern files extensions.

postcss throws an error and forces using .cjs or .mjs extension

I've used postcss in lots of projects and never had this issue. I suspect you set something up incorrectly. Try using the Svelte adder rather than setting it up yourself: https://github.com/svelte-add/postcss

so I updated all the configs with the extension

You really shouldn't do that. Just because one tool is outdated and broken you shouldn't go backwards with the rest

In my vision, the tool which imply using modern standard should support modern files extensions.

SvelteKit uses the most modern support which is "type": "module" with .js extension. The .mjs and .cjs extensions are just to be used in a transition period until projects reach the end state that SvelteKit has started at from day 1


[Band of Blades] Critical roll template hotfix

Temporarily replace the critical background by the normal background until the image proxy is no longer broken.
Fixes issue #6541.

If you change the wording to to fixes #6541, this PR would automatically close the issue.
My recent #6551 is an example of how this can be used.

Thanks!


Music Player doesn't load mp3 data

The music player doesn't seem to play music nor load the mp3 data. The server returns a 'internal error'. The files are served from localhost:8000 which gives all access permissions but some conflict causes the files not to load.

Hello..How did you solve the loading of mp3 files? I have downloaded your code and out of curiosity tried to execute it without making any changes..Could you please guide me how you added your own songs in PyMusicMood because in the CSV files I cannot add my own songs..Could you guide me how to do it or link me to some guide? How should I add my own songs according to my mood and add directory to the mp3 files? I am using Windows 10.


⚠ Bofrost.lu has degraded performance

In 1d14106, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2498 ms

Resolved: Bofrost.lu performance has improved in 90e3b5c after 6 minutes.


ffmpeg-full crashes due to illegal instruction on x86_64 CPU without avx

Describe the bug
ffmpeg-full no longer works at all after upgrading to nixos-24.11. ffmpeg still works fine.
Steps To Reproduce
$ nix run github:nixos/nixpkgs/de1864217bfa9b5845f465e771e0ecb48b30e02d#ffmpeg-full -- -version
Illegal instruction (core dumped)

$ nix run github:nixos/nixpkgs/de1864217bfa9b5845f465e771e0ecb48b30e02d#ffmpeg -- -version
ffmpeg version 7.1 Copyright (c) 2000-2024 the FFmpeg developer

de1864217bfa9b5845f465e771e0ecb48b30e02d is the current commit for nixpkgs-unstable on https://status.nixos.org/
Expected behavior
ffmpeg-full works as well as ffmpeg.

Notify maintainers
Ping @Atemu, @jopejoe1, @emily

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

Here are the AVX/AVX2 instructions (vpxor or vmovdqu) causing the SIGILL, they're loaded by liblcevc_dec_api.so:
$ gdb --args nix run github:nixos/nixpkgs/de1864217bfa9b5845f465e771e0ecb48b30e02d#ffmpeg-full -- -version
Thread 1 "ffmpeg" received signal SIGILL, Illegal instruction.
0x00007ffff2ededdd in lcevc_dec::decoder::DecoderPool::DecoderPool(unsigned long) () from /nix/store/9ng1mirk4xsmc293ck41fgbbs93ssn8g-lcevcdec-3.2.1-lib/lib/liblcevc_dec_api.so
(gdb) disass
Dump of assembler code for function _ZN9lcevc_dec7decoder11DecoderPoolC2Em:
   0x00007ffff2edeb40 <+0>:     push   %rbp
   0x00007ffff2edeb41 <+1>:     mov    %rsp,%rbp
   0x00007ffff2edeb44 <+4>:     push   %r15
   0x00007ffff2edeb46 <+6>:     push   %r14
   0x00007ffff2edeb48 <+8>:     push   %r13
   0x00007ffff2edeb4a <+10>:    push   %r12
   0x00007ffff2edeb4c <+12>:    push   %rbx
   0x00007ffff2edeb4d <+13>:    mov    %rdi,%rbx
   0x00007ffff2edeb50 <+16>:    sub    $0x18,%rsp
   0x00007ffff2edeb54 <+20>:    mov    0x1d315(%rip),%rax        # 0x7ffff2efbe70
   0x00007ffff2edeb5b <+27>:    movq   $0x0,0x8(%rdi)
   0x00007ffff2edeb63 <+35>:    movq   $0x0,0x10(%rdi)
   0x00007ffff2edeb6b <+43>:    add    $0x10,%rax
   0x00007ffff2edeb6f <+47>:    movq   $0x0,0x18(%rdi)
   0x00007ffff2edeb77 <+55>:    mov    %rax,(%rdi)
   0x00007ffff2edeb7a <+58>:    movq   $0x0,0x20(%rdi)
   0x00007ffff2edeb82 <+66>:    movq   $0x0,0x28(%rdi)
   0x00007ffff2edeb8a <+74>:    movq   $0x0,0x30(%rdi)
   0x00007ffff2edeb92 <+82>:    movq   $0x0,0x38(%rdi)
   0x00007ffff2edeb9a <+90>:    movq   $0x0,0x40(%rdi)
   0x00007ffff2edeba2 <+98>:    movq   $0x0,0x48(%rdi)
   0x00007ffff2edebaa <+106>:   test   %rsi,%rsi
   0x00007ffff2edebad <+109>:   jne    0x7ffff2edec10 <_ZN9lcevc_dec7decoder11DecoderPoolC2Em+208>
   0x00007ffff2edebaf <+111>:   mov    0x1d362(%rip),%rax        # 0x7ffff2efbf18
   0x00007ffff2edebb6 <+118>:   movq   $0x0,0x50(%rdi)
   0x00007ffff2edebbe <+126>:   xor    %r13d,%r13d
   0x00007ffff2edebc1 <+129>:   movq   $0x0,0x60(%rdi)
   0x00007ffff2edebc9 <+137>:   add    $0x10,%rax
   0x00007ffff2edebcd <+141>:   mov    %rax,(%rdi)
   0x00007ffff2edebd0 <+144>:   mov    %r13,0x58(%rbx)
   0x00007ffff2edebd4 <+148>:   vpxor  %xmm0,%xmm0,%xmm0
   0x00007ffff2edebd8 <+152>:   movq   $0x0,0x88(%rbx)
   0x00007ffff2edebe3 <+163>:   vmovdqu %ymm0,0x68(%rbx)
   0x00007ffff2edebe8 <+168>:   vzeroupper

Disable withVmaf:
https://github.com/NixOS/nixpkgs/blob/162b4bf7b304b9a655ee01da216924946293f760/pkgs/development/libraries/ffmpeg/generic.nix#L125
Or override the flags it's built with, via an overlay (since you want to modify all instances of the package in your closure):
https://github.com/NixOS/nixpkgs/blob/162b4bf7b304b9a655ee01da216924946293f760/pkgs/by-name/li/libvmaf/package.nix#L34

The problem seems to be that avx instructions are required in the library ( https://github.com/v-novaltd/LCEVCdec ) constructor, so just linking with it makes ffmpeg fail even if he's not using that library.
It doesn't seem related to Vmaf ?
That also matches that this is a recent regression, because lcevcdec was added to nixpkgs only recently.
I think requiring avx for some ffmpeg modules makes sense, but these libraries really shouldn't be using a constructor function at link time and rely on explicit init by ffmpeg only if they are used...

Missed the last comment as I was typing my own, though I believe my comment highlights another issue in its own right. Again, IMO requiring AVX is fine, AVX predates AVX-512 by at least 6 years. AVX-512 is IMO unreasonable, if it's unrelated then a new issue should be made to fix that.
Though back to this issue, lcevcdec seems to have some code that attempts to check cpuinfo:
https://github.com/v-novaltd/LCEVCdec/blob/364ae4f11bc6573809b749056104b32109c517d7/src/core/decoder/src/common/simd.c#L53
Surprised it's running that codepath in this case, unless I'm missing something?

From the trace above (lcevc_dec::decoder::DecoderPool::DecoderPool(unsigned long)) that seems to be just initialization code because they're allocating a static pool in a global variable, so the code runs at link time.
And any code running from this lib will fail on machines that do not have avx because it's built with avx, even if it's trivial:
cmake/modules/Compiler/ClangGNU.cmake
13:    target_compile_options(lcevc_dec::compiler INTERFACE -mavx)
17:    target_compile_options(lcevc_dec::compiler INTERFACE -pthread -msimd128 -mavx)

So just running ffmpeg -version or any noop will fail as soon as it's linked with that lib.
As said above I see no problem requiring avx or even later for encoding and rare things, but basic decoding should still work reasonably well on older machines if performances used to be good enough; perhaps we'd better just build yet another ffmpeg-noavx build...

Depending on AVX of any kind is not fine as we generally target the generic x86_64 CPU target. Applications should opt to use AVX codepaths at runtime on CPUs that support them if they benefit from those but it should not be the fallback code path.
We will not have yet another ffmpeg variant.
@jopejoe1 could you patch out the dependency on AVX in lcevcdec for the time being?
Otherwise we'll have to disable lcevcdec support by default in -full. Given that I've never heard of it, it's probably not very critical to have it by default and having -full function on all CPUs is much more important.

Just to confirm the previous analysis, withLcevcdec=false does indeed workaround the SIGILL:
$ nix -L shell --impure --expr "let pkgs = import (fetchTarball "https://github.com/nixos/nixpkgs/archive/de1864217bfa9b5845f465e771e0ecb48b30e02d.tar.gz") { }; in (pkgs.ffmpeg-full.override {withLcevcdec=true;}).overrideAttrs {doCheck = false;}" -c ffmpeg -version
Illegal instruction (core dumped)

$ nix -L shell --impure --expr "let pkgs = import (fetchTarball "https://github.com/nixos/nixpkgs/archive/de1864217bfa9b5845f465e771e0ecb48b30e02d.tar.gz") { }; in (pkgs.ffmpeg-full.override {withLcevcdec=false;}).overrideAttrs {doCheck = false;}" -c ffmpeg -version
ffmpeg version 7.1 Copyright (c) 2000-2024 the FFmpeg developers

Opened https://github.com/NixOS/nixpkgs/pull/369159.


Error when reloading pythonnet code in Spyder IDE

Problem Description
Hello,
I have used Spyder for years without issues. However, after getting a new laptop and reinstalling Spyder, I encountered problems with pythonnet.
Code that imports pythonnet modules like clr works on first run, but fails on reload with the error "RuntimeError: Failed to initialize Python.Runtime.dll".
As suggested in https://github.com/pythonnet/pythonnet/pull/2079, I adjusted Pythhonnet code when importing clr which fixed the reload issue.
However, this then surfaces another error on reload - "AttributeError: module 'clr' has no attribute 'AddReference'".
I have tried this with multiple versions of Anaconda, Miniconda, and Spyder but observe the same reloading issue.
Interestingly, the code works flawlessly when run in other IDEs like PyCharm, VSCode, and Visual Studio.
In Spyder, I have to restart the kernels everytime to avoid this error.
A similar error has been also reported here for Spyder.
It seems to be an isolated problem with reusing pythonnet in Spyder. I would appreciate any suggestions on how to resolve the reload errors when importing clr. Let me know if any other details would be helpful.
Paste Traceback/Error Below (if applicable)
Traceback (most recent call last):

  File ~\AppData\Local\miniconda3\Lib\site-packages\spyder_kernels\py3compat.py:356 in compat_exec
    exec(code, globals, locals)

  File e:\TEST\python test\TEST_2023.py:15
    clr.AddReference('OSIsoft.AFSDK')

AttributeError: module 'clr' has no attribute 'AddReference'

Versions

Spyder version: 5.4.4
Python version: 3.11.4
Qt version (qt-main):  5.15.2
PyQt version: 5.15.7
Operating System name/version: Windows 10 Pro

Hi @ARa2of thank you for the feedback! Could it be that disabling the UMR functionality could help here? For that you can go to Preferences > Python interpreter > User Module Reloades (UMR) and uncheck the Enable UMR checkbox or a pythonnet to the exclusion list (Set UMR excluded (not reloaded) modules). Also, is there an example script that you could share with us to reproduce this locally?
Any other info in order to better understand what could be happening here is greatly appreciated. Let us know!

Hi @ARa2of thank you for the feedback! Maybe disabling the UMR functionality could help here? For that you can go to Preferences > Python interpreter > User Module Reloades (UMR) and uncheck the Enable UMR checkbox or add pythonnet to the exclusion list (Set UMR excluded (not reloaded) modules). Also, is there an example script that you could share with us to reproduce this locally?
Any other info in order to better understand what could be happening here is greatly appreciated. Let us know!

Disabling UMR has actually solved the problem!  Thanks a lot, I have been trying for a week!
I assume it is worth mentioning this in the documents (or it might be mentioned but I missed it).
Thanks for the great work and for Spyder!

Awesome, glad to know that things are working for you now! Just in case, I think the relevant doc entry related to the UMR functionality is available at https://docs.spyder-ide.org/current/panes/ipythonconsole.html?highlight=umr#reload-changed-modules


Fix V3 Get-AzMigDependenciesAgentless

Description
Fixes Get-AzMigDependenciesAgentless now that API returns applianceNameToSiteIdMapV3 rather than applianceNameToSiteIdMapV2
Checklist

[x] This pull request was tested on both of:

[x] PowerShell 5.1 (Windows)
[x] PowerShell 6.x
[x] PowerShell 7.x (Latest PowerShell)

[x] Scripts do not contain static passwords or other secret tokens.
[x] All Azure resource identifiers which must be universally unique are guaranteed to be so.

Testing information

Platform and PowerShell version: Darwin 21.1.0 Darwin Kernel Version 21.1.0: Thu Sep 30 06:42:03 PDT 2021; root:xnu-8<IP_ADDRESS>.2~12/RELEASE_ARM64_T8101 7.1.4
Az version: 6.4.0

See my comment about selecting all properties with the ExpandProperty parameter.

Addressed with 96f35f735561d5788314f5196983c9fac969ca5f


Extend QueryTrajectoryState to allow to report errors

Afaik there's no way of reporting an error on a service call in ROS2 other than adding fields to the service response.
In order to port the query_state service to ROS2 I need to be able to report some errors
If there's a different way of doing this, I'll be happy to change it.

@destogl @Karsten1987 FYI

@destogl @Karsten1987 FYI

It looks also good to me. It makes sense.

It looks also good to me. It makes sense.

Thanks guys, going forward with this then!

Thanks guys, going forward with this then!


Deserialize circular referenced classes

Suppose there are two classes: Post and Comment.
class Post {
  @autoserialize
  id: string;

  @deserializeAs(Comment)
  comments: Comment[];
}

class Comment {
  @autoserialize
  id: string;

  @deserializeAs(Post)
  post: Post;
}

When deserialize a post with comments, it failed to deserialize the comments:
Input:
{
  "id": "foo",
  "comments": [
    {
      "id": "comment-1"
    }
  ]
}

Output:
let post = Deserialize(json, Post);
expect(post.comments).not.toBeUndefined() // failed!

This implementation might seems odd, but I suppose it's a valid use case to have bidirectional association that I can lookup the relevant objects from either side.

Is there any limitation on the deserializer?
Is there a way to implement the association?

Thanks!

Two issues here. The first has to do with Javascript not having a lookahead compiler. At parse time, either Post or Comment won't have been parsed yet and will be undefined, whichever is parsed first, the other won't exist. The second part is a circular reference problem. I certainly see the benefit of having bidirectional pointers here but if you step back and think about how the data would be serialized you'll realize that it can't really work that way. What you can do is sticht up the references post deserialization. Here is an example:
class Post {
  @autoserialize
  id: string;

  @deserializeAs(Comment)
  comments: Comment[];

  public static OnDeserialized(instance : Post, json : any) : any {
      instance.comments.forEach(function(comment : Comment) {
      comment.post = instance;
  });
}

class Comment {
  @autoserialize
  id: string;
//note: Not deserialized through an annotation!
  post: Post;
}

Does that make sense?

That makes sense! Regarding the second part, suppose the json objects are fetched from two different endpoints:
GET /posts/1

// Response
{
   id: 1,
   comments: [
     {
        id: 42
     }
   ]
}

GET /comments/42

// Response
{
   id: 42,
  post: {
    id: 1
  }
}

The API response has made sure that there is not circular reference in one object, though I might need to deserialize the post and comment in different places. Put it in another way: I might sometimes need to deserialize comment, sometimes need to deserialize post, with the API response properly populates the information according to the class definitions. How about this kind of scenario?

This isn't something i think this library should handle but I can see why might ask about it. Probably the right move here is to use client side caching and do a second http call to get whichever one you dont have. You can keep a local index by id. If you have two sources of data and need to aggregate them its a good idea to leave that to an abstraction and let the library do what it is built to do.

Ok, thanks anyway.
BTW, thank you for the great library. It has been the foundation of a large Angular2 app I am working on and serves the purpose of serialization well.

I'm glad you're getting some good use out of it, thats great to hear thanks!


STONEINTG-555: Manual dry-runs of integration tests

TODO

[ ] How to handle when dry-run test is being evaluated for snapshot pass/failure (probably just marked is failed with message about dry-run test)
[ ] Labeling SEB and PLR with label about dry-run (for easier debugging)
[ ] Unittests
[ ] Manual testing

Maintainers will complete the following section

[ ] Commit messages are descriptive enough (hints)
[ ] Code coverage from testing does not decrease and new code is covered (check the PR coverage on codecov)
[ ] Controllers diagrams are updated when PR changes controllers code  (if applicable)

This feature is dropped, closing PR


add beautifier rule for space after close paren and destructure bracket

Fixes #21675
Adds a new rule to the javascript beautifier to add a space after a closing parenthesis if the next token is an open bracket and the context is a control declaration.
Tests added as well.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: grubeeric sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks!

Some more formatting issues if you are still interested in the area


Shouldn't this url be invalid?

isHttpUri and isHttpsUri methods both report 'http://http://blah' as valid.

Here others example:
http://www,example,com
http://www,example.com
http://www.example,com
<EMAIL_ADDRESS>

Hi @cbshakumar @jodacame,
Same issues with valid-url but also validator, just built this strictly based on RFC-3986: https://github.com/adrienv1520/node-uri
The main features of the project are:

parse any URI (URNs, URLs, URIs with IDNs support, etc.);
get the safe Punycode ASCII or Unicode serialization of a domain;
check an URI, HTTP/HTTPS/Sitemap URL, IP, domain is valid with clear checking errors;
encode/decode an URI, HTTP/HTTPS/Sitemap URL.

I hope it could help you guys ;)