Issue with Company Communicator V5.0 Upgrade

We are facing issue with the ARM template deployment for Tech News/Company Communicator V5.0 upgrade in Production where we are getting the below error and the deployment is failed. We tried with resync the appservice and all 3 azure function after point to External Git Repos v5.x branch and then tried the ARM template redeployment again but it still failed.
{
"code": "DeploymentFailed",
"message": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.",
"details": [
{
"message": "There was a conflict. Conflict with existing ScmType: ExternalGit"
This issue seems to be the same as #695

Hi @EYSue ,
We will be releasing a separate MSBuild fix today. We will keep you updated once we release the fix.

Hi MSFT
This is great news, please let me know as soon as this is available.
Regards
Sue
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Sai Venkat @.>
Sent: Thursday, April 21, 2022 3:25:41 PM
To: OfficeDev/microsoft-teams-apps-company-communicator @.>
Cc: Susan McInulty @.>; Mention @.>
Subject: Re: [OfficeDev/microsoft-teams-apps-company-communicator] Issue with Company Communicator V5.0 Upgrade (Issue #702)
Hi @EYSuehttps://github.com/EYSue ,
We will be releasing a separate MSBuild fix today. We will keep you updated once we release the fix.
—
Reply to this email directly, view it on GitHubhttps://github.com/OfficeDev/microsoft-teams-apps-company-communicator/issues/702#issuecomment-1105288136, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AV477NGXAGINHL5DEL4BB2TVGFQOLANCNFSM5T4AJI5A.
You are receiving this because you were mentioned.Message ID: @.***>
This e-mail and any attachment are confidential and contain proprietary information, some or all of which may be legally privileged. It is intended solely for the use of the individual or entity to which it is addressed. If you are not the intended recipient, please notify the author immediately by telephone or by replying to this e-mail, and then delete all copies of the e-mail on your system. If you are not the intended recipient, you must not use, disclose, distribute, copy, print or rely on this e- mail. Whilst we have taken reasonable precautions to ensure that this e-mail and any attachment has been checked for viruses, we cannot guarantee that they are virus free and we cannot accept liability for any damage sustained as a result of software viruses.  EY refers to the global organization, and may refer to one or more, of the member firms of Ernst & Young Global Limited, each of which is a separate legal entity. Ernst & Young Global Limited, a UK company limited by guarantee, does not provide services to clients. EY Global Services Limited is a company registered in England and Wales with registered number 5483856. Its registered office is at 6 More London Place, London, United Kingdom, SE1 2DA. EY Global Services Limited's business is confined to the supply of services to member firms of Ernst & Young Global Limited and related entities

Hi
We tried to resync all the 3 azure function again by pointing to Company Communicator GitHub V5.x branch but it is still failing with the below message.

When I checked the Github branch the yesterday code changes are pushed to the master branch and not to the V5.x branch from where we are syncing the code to the azure function and app service for V5.0 upgrade. Would you please guide after discussion with the developers whether we should use master branch itself for syncing the latest code or the yesterday code changes will be pushed to V5.x branch also for us to use to for V5.0 upgrade.

Hi
We tried to resync all the 3 azure function again by pointing to Company Communicator GitHub V5.x branch but it is still failing, I added full details and screenshots to Github ticket #702
Regards
Sue
From: Porkai @.>
Sent: Thursday, April 21, 2022 5:23 PM
To: OfficeDev/microsoft-teams-apps-company-communicator @.>
Cc: Susan McInulty @.>; Mention @.>
Subject: Re: [OfficeDev/microsoft-teams-apps-company-communicator] Issue with Company Communicator V5.0 Upgrade (Issue #702)
Hi @EYSuehttps://github.com/EYSue, fix is merged, can you please try to resync the Azure functions and app services during your upgradation.
—
Reply to this email directly, view it on GitHubhttps://github.com/OfficeDev/microsoft-teams-apps-company-communicator/issues/702#issuecomment-1105438683, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AV477NCASJJMAL3EZPYICSLVGF6GZANCNFSM5T4AJI5A.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>This e-mail and any attachment are confidential and contain proprietary information, some or all of which may be legally privileged. It is intended solely for the use of the individual or entity to which it is addressed. If you are not the intended recipient, please notify the author immediately by telephone or by replying to this e-mail, and then delete all copies of the e-mail on your system. If you are not the intended recipient, you must not use, disclose, distribute, copy, print or rely on this e- mail. Whilst we have taken reasonable precautions to ensure that this e-mail and any attachment has been checked for viruses, we cannot guarantee that they are virus free and we cannot accept liability for any damage sustained as a result of software viruses.  EY refers to the global organization, and may refer to one or more, of the member firms of Ernst & Young Global Limited, each of which is a separate legal entity. Ernst & Young Global Limited, a UK company limited by guarantee, does not provide services to clients. EY Global Services Limited is a company registered in England and Wales with registered number 5483856. Its registered office is at 6 More London Place, London, United Kingdom, SE1 2DA. EY Global Services Limited's business is confined to the supply of services to member firms of Ernst & Young Global Limited and related entities

Hi @EYSue ,
We request you to use the master branch to upgrade the app to 5.0.
V5.x is a backup branch created that does not have the latest code as in master.

We tried to run arm template deployment using master branch as per below path but still get these below errors.
https://portal.azure.com/#create/Microsoft.Template/uri/https%3A%2F%2Fraw.githubusercontent.com%2FOfficeDev%2Fmicrosoft-teams-company-communicator-app%2Fmaster%2FDeployment%2Fazuredeploy.json
Error Logs:
{
"code": "DeploymentFailed",
"message": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.",
"details": [
{
"code": "RoleAssignmentUpdateNotPermitted",
"message": "Tenant ID, application ID, principal ID, and scope are not allowed to be updated."
},
{
"code": "RoleAssignmentUpdateNotPermitted",
"message": "Tenant ID, application ID, principal ID, and scope are not allowed to be updated."
},
{
"code": "RoleAssignmentUpdateNotPermitted",
"message": "Tenant ID, application ID, principal ID, and scope are not allowed to be updated."
},
{
"code": "RoleAssignmentUpdateNotPermitted",
"message": "Tenant ID, application ID, principal ID, and scope are not allowed to be updated."
},
{
"code": "RoleAssignmentUpdateNotPermitted",
"message": "Tenant ID, application ID, principal ID, and scope are not allowed to be updated."
},
{
"code": "RoleAssignmentUpdateNotPermitted",
"message": "Tenant ID, application ID, principal ID, and scope are not allowed to be updated."
},
{
"code": "RoleAssignmentUpdateNotPermitted",
"message": "Tenant ID, application ID, principal ID, and scope are not allowed to be updated."
}
]
}
When we tried to resync the app service with master branch this time it failed with the below errors. But rest all the 3 azure functions got synced with master branch successfully.
Error details:
Command: deploy.cmd
Handling ASP.NET Core Web Application deployment.
Restoring NuGet packages
Determining projects to restore...
All projects are up-to-date for restore.
Restoring npm packages (this can take several minutes)
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| 1.x but none is installed. You must install peer dependencies yourself.
up to date in 80.397s
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^5.0.0 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of fibers@>= 3.1.0 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"ia32"})
Building the client app (this can take several minutes)
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\webpack-dev-server\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"ia32"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\watchpack-chokidar2\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"ia32"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\react-scripts\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"ia32"})

<EMAIL_ADDRESS>build D:\home\site\repository\Source\CompanyCommunicator\ClientApp
react-scripts build

'react-scripts' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: react-scripts build
Failed exitCode=1, command=npm run build
npm ERR! Exit status 1
npm ERR!
An error has occurred during web site deployment.
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      D:\local\AppData\npm-cache_logs\2022-04-22T15_07_45_218Z-debug.log
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.\r\nnpm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.\r\nnpm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.\r\nnpm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| 1.x but none is installed. You must install peer dependencies yourself.\r\nnpm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.\r\nnpm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.\r\nnpm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^5.0.0 but none is installed. You must install peer dependencies yourself.\r\nnpm WARN<EMAIL_ADDRESS>requires a peer of fibers@>= 3.1.0 but none is installed. You must install peer dependencies yourself.\r\nnpm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.\r\nnpm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.\r\nnpm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):\r\nnpm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"ia32"})\r\nnpm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\webpack-dev-server\node_modules\fsevents):\r\nnpm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"ia32"})\r\nnpm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\watchpack-chokidar2\node_modules\fsevents):\r\nnpm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"ia32"})\r\nnpm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\react-scripts\node_modules\fsevents):\r\nnpm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"ia32"})\r\n\r\n'react-scripts' is not recognized as an internal or external command,\r\noperable program or batch file.\r\nnpm ERR! code ELIFECYCLE\r\nnpm ERR! errno 1\r\nnpm ERR<EMAIL_ADDRESS>build: react-scripts build\r\nnpm ERR! Exit status 1\r\nnpm ERR! \r\nnpm ERR! Failed at the<EMAIL_ADDRESS>build script.\r\nnpm ERR! This is probably not a problem with npm. There is likely additional logging output above.\r\n\r\nnpm ERR!      D:\local\AppData\npm-cache_logs\2022-04-22T15_07_45_218Z-debug.log\r\nD:\Program Files (x86)\SiteExtensions\Kudu\96.40113.5578\bin\Scripts\starter.cmd deploy.cmd

Hi @EYSue,

It seems that the deployment got failed. So, the existing resources needs to be deleted. Re-deployment has to be done from the start and during this please update GUID's as mentioned below.

. Role assignment issue:-
The mentioned issue can occur if customer might have attempted multiple deployment with same GUID, it is recommended to have new GUID’s generated for each deployment if it is performed in same subscription.

Please ask customer to modify the highlighted fields with newly generated GUID’s and continue with deployment.

Go to GUID Generator. Copy unique GUID and paste it in the parameters below.

After the updation, if the app service got failed then go for re-sync option.

We tried to run arm template deployment using master branch again and this time it is completed successfully without any error.
But when tried to resync the app service with master branch this time it failed again with the below errors. But rest all the 3 azure functions got synced with master branch successfully.

We cross verified at our end of the app service re-deployment is working as expected or not. It seems to be working as expected.
Hence the issue customer is facing is specific to the CC instance of their deployment. Could you please confirm what’s the instance of the App service plan that has opted during deployment (S1 or S2)?

Also, allow us some time to investigate the probable cause of the error.

Hi @EYSue ,
We are closing this issue as it was cleared over the call. You may also always open a new issue for further assistance on issues related to Company Communicator.


UWP crashes with XF <IP_ADDRESS>2540 when Sliding

With the latest XForms, something crashes when sliding with the next exception
System.AccessViolationException: 'Attempted to read or write protected memory. This is often an indication that other memory is corrupt.'

@JTOne123 hi) I have no idea, how can we solve it)
I'm sure, we should create an issue for Xamarin.Forms team.

@JTOne123 could you please create this issue for XF team (attach sample)
Because it doesn't look like CardsView bug

Exception is here

It looks like an issue with corner radius that was fixed at 3.3 for UWP
https://github.com/xamarin/Xamarin.Forms/issues/2838
https://github.com/xamarin/Xamarin.Forms/commit/6031bbfb691033680c877bb7da79fc5d6c717299

@JTOne123 amazing! Good job.
What do you think regarding to this issue?
Should we do anything?

I'm fixed it with the custom frame renderer
public class FixFrameRenderer : FrameRenderer
    {
        protected override void UpdateBackgroundColor()
        {
            var backgroundColor = Element.BackgroundColor;
            if (Control != null)
            {
                if (!backgroundColor.IsDefault)
                {
                    Control.Background = backgroundColor.ToBrush();
                }
                else
                {
                     //////Control.ClearValue(BackgroundProperty); ISSUE HERE
                    Control.Background = new Color(0, 0, 0, 0).ToBrush();
                }
            }
        }
    }

The issue in Control.ClearValue(BackgroundProperty);
It happens because do not set UnselectedIndicatorStyle and without explicit value background color, the nex XF frame renderer tries to Control.ClearValue and that cause an exception.
Workaround:
Use fixrenderer or set UnselectedIndicatorStyle

Cool
Let's leave it as is.


Reshim doesn't seem to work

$ pip3 list | grep gimme
gimme-aws-creds 2.3.4

$ gimme-aws-creds
zsh: command not found: gimme-aws-creds

$ asdf reshim python
$ gimme-aws-creds
zsh: command not found: gimme-aws-creds

Running this on python -V => Python 3.8.3
Can someone help me out?

try asdf global python system

asdf global python system will use the Python system installation, which might not be desirable.
Reshim does not happen automatically so please run
asdf reshim python

after running pip install and it should work.
Please comment if this does not fix the issue.


Optimizing loading series to different yAxis's?

Hi there,
Loving the package and its doing almost all that I want.
However I am running into a performance issue that I cannot seem to overcome.
The goal of what I am doijg is to have as many items on the page as possible.
Each item has 12 series attached to it spread across 4 yAxis.
First yAxis loads up near instantly, with the following 3 far far slower.
I have tried adding the yaxis as props and the series also as props to see if that would help, however even doing it that way kept giving me Highcharts Error 18, axis not found.
Example in action: https://gw2.silveress.ie/history?dates=2012-08-26,2018-08-05&series=b,bvol,bval,bvalp&ids=19721,19697,43773,19699,19703,19698,19702,19700,19701
Source code: https://gitlab.com/Silveress_Gw2/Stats_Frontend/blob/5bf031f81101daa94da57704900c7c906b5a363c/src/history.js#L812
What advice can you give as regards the series and the axis?
With thanks.

Hi @Silveress looks like there is a high degree of latency whilst awaiting the data from the server - in excess of 5 seconds on the example you linked (see below)

I also think the functions you're using to format the data before passing to the chart could do with optimisation, there are lots of nested for loops, and merging of data. I would suggest using array methods instead.
Finally I think your onSetExtremes on the XAxis in causing some sort of infinite loop, looks to me as if Highcharts Boost is repeatedly loading image blobs, which suggests to me something is causing the chart to constantly re-render.
Here is an example chart, it is loading the same amount of data points, across 4 axes and 12 series, and it seems to load without any issues.
With mock 5 second API latency: Demo, Code
With no mock API latency: Demo, Code

First of all thank you for the time.
I played around a lot with it last night and although yer examples didn't solve it for me they did point me in the right direction, namely that you only loaded 12 series.
In the example above I was loading 9 ID's  for a total of 112 series (4+(12*n)), turns out highcharts dosent like anything over 50 series or so.
The solution I went with was to only load the subset of series requested to teh hart.
As regards the latency, the api it is calling is my own one that I host on a  2.5 eur VPS, not overly powerful but gets the job done.
Again thank you for the help.
TLDR for anyone else: Highcharts can deal with a heck of a lot of data in a series but it dosent like 100+ series.


Added several functions to represent data

Added 3 fully functional flexible functions.
plot_results:

Visualizes glacier-related  quantities (or any other data with the same shape) using heatmaps.
Takes results dataset (for example that of the forward model of Huginn.jl) and list of variables to plot.
Arranges plots in a grid layout.
Mirrors data around both the X and Y axes.

plot_results_temporal:

Visualizes temporal evolution of a single glacier-related quantity.
Takes results dataset, one variable, a time span tspan, and list of metrics.
Generates line plots or heatmaps based on metrics (e.g., average, median, difference).

plot_results_IV:

Visualizes temporal evolution of integrated ice volume of a glacier.
Computes integrated ice volume by summing up ice volume and multiplying by glacier area.
Takes results dataset, one variable, a time span tspan, and an optional area parameter.

Features to be added: geometrical coordinates and/or scale on the axis of heatmaps.

Hi @vivekag7, thanks for the PR!
As I mentioned during the discussion, I think you should place your code in a separate file. The way we structure code in ODINN is by separating files for structs (e.g. Results.jl) and functions/utilities (e.g. results_utils.jl). Since you are contributing with functions, you should place this code in a functions/utils file. I suggest that you create a new file results_plotting_utils.jl, to contain everything related to plotting. You can follow the same structure as in results_utils.jl.
Also, before accepting this PR, you will also have to add tests to make sure you didn't break anything. We can discuss testing strategies for plots.

Hi @vivekag7, thanks for the update. In order to understand how the new plotting utils work, you should provide some tests to exemplify its usage. On the meantime, I will also give you individual comments in parts of the code which I think could be improved.

Hey Jordi, thanks for the valuable feedback! Hereby I would like you to show some plots, note that I've already implemented the function that acts as a hub. I include the prompt that is used to generate the plot, to provide more clarity:
note: results[2] is a object containing the following fields (:rgi_id, :H, :S, :B, :V, :Vx, :Vy)
plot_glacier(results[2], "heatmaps", [:H, :V, :B])

plot_glacier(results[2], "evolution_statistics", [:H], tspan=(2010.0,2015.0), metrics=["average","std","max","median","min"])

plot_glacier(results[2], "evolution_difference", [:H], tspan=(2010.0,2015.0), metrics=["difference","hist"])

plot_glacier(results[2], "integrated_volume", [:H], tspan=(2010.0,2015.0))

OK, I see you're still having the issue of reversed heatmaps. I don't know why the matrices that we get from OGGM are reversed. You should apply this trick in order to have them in the right order:
H = reverse(H', dims=2)

And then, as discussed, I would also add the coordinates in the axis, and add a scalebar inside the heatmap to get a reference of the glacier size.

Since I'm leaving soon on holidays, @facusapienza21 could you please take over the review of this PR? Thanks!

Hi @facusapienza21, I finished up the plotting functions with geospacial coordinates, scale division and fully functional tests. See for example the following image:

Please let me know if you have any more feedback and/or comments.

@vivekag7 The tests triggered some warnings and errors.
For the warning, please check:
┌ CairoMakie [13f3f980-e62b-5c42-98c6-ff1f3baf88f0]
│  WARNING: Makie.Combined is deprecated, use Plot instead.
│    likely near none:1
│  WARNING: importing deprecated binding Makie.Combined into CairoMakie.
│  WARNING: Makie.ContinuousSurface is deprecated, use VertexGrid instead.
│    likely near none:1
│  WARNING: importing deprecated binding Makie.ContinuousSurface into CairoMakie.
│  WARNING: Makie.DiscreteSurface is deprecated, use CellGrid instead.
│    likely near none:1
│  WARNING: importing deprecated binding Makie.DiscreteSurface into CairoMakie.

The error seems to come from the SSL certificate that is used by Python to use PyCall. This was fixed in the past, but maybe the error is back again. @facusapienza21 we should investigate this, because this can potentially break all our testing chain.

Hi @vivekag7, could you please open a separate PR for the Glathida stuff? Thanks!

Codecov Report
Attention: 27 lines in your changes are missing coverage. Please review.

Comparison is base (ad8ca44) 59.04% compared to head (dad101f) 67.50%.

Files
Patch %
Lines

src/simulations/results/results_plotting_utils.jl
85.32%
27 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
+ Coverage   59.04%   67.50%   +8.45%
==========================================
  Files          14       15       +1
  Lines         376      560     +184
==========================================
+ Hits          222      378     +156
- Misses        154      182      +28

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Vim hangs by a user's mistake

I have mistyped vim file_a.txt > file_b.txt in place of cat command and have managed to hang Vim while in terminal.
Is there any command that could terminate this operation without being forced to close or kill my terminal in order to return back to normal?
Please advice, thank you.

On Tue, Dec 26, 2017 at 03:30:44AM -0800, Στέφανος wrote:

I have mistyped vim file_a.txt > file_b.txt in place of cat command and have
managed to hang Vim while in terminal.
Is there any command that could terminate this operation without being forced
to close or kill my terminal in order to return back to normal?

Type :q! and press Enter.
HTH,
Marius Gedminas
We'll show a small example here with one user calling another. (Under
international treaties describing technical papers these users must be called
"Alice" and "Bob".)
-- Anthony Baxter

Even though Vim is not visible in terminal, your suggestion works as expected @vim-ml, cheers!
Here's a screenshot of what I see:


riot tag based coverage report incorrectness

LCOV reporting generates wrong coverage report on source code lines

While, HTML reporting works perfectly fine

You can see the difference in below screenshots.
How is it possible that LCOV reporting gets its data from unprocessed src/hello.tag file while HTML reporting gets its data from processed src/hello.tag file? What is the difference? I've read their source codes but I couldn't find anything there.
Reproduce:
Example project:
git clone https://github.com/vitogit/tdd-mocha-chai-riot
Karma Config (updated):
module.exports = function(config) {
  config.set({
    basePath: '',
    frameworks: ['mocha','chai','riot'],
    plugins: [
      'karma-mocha',
      'karma-mocha-reporter',
      'karma-phantomjs-launcher',
      'karma-chai',
      'karma-riot',
      'karma-coverage'
    ],
    files: [
      'node_modules/chai/chai.js',
      'src/**/*.tag',
      'test/**/*.js'
    ],
    preprocessors: {
      '**/*.tag': ['riot', 'coverage']
    },
    coverageReporter: {
      dir: 'coverage/',
      reporters: [
        // works correctly
        { type: 'html', subdir: 'report-html' },

        // does not work correctly -- gets its input from unprocessed riot tag file
        { type: 'lcovonly', subdir: '.', file: 'lcov.info' }
      ]
    },
    browsers: ['PhantomJS'],
    reporters: ['mocha', 'coverage'],
    failOnEmptyTestSuite: false,
    autoWatch: true
  })
}

How to run?
npm i && npm i karma-coverage --save-dev
karma start
cat coverage/lcov.info

You can also see html based coverage reports inside coverage directory's html coverage reports.
LCOV is incorrect:

HTML is correct:

By the way, actually, what I really wonder is why html coverage output is true while lcov coverage output is not?

I have the same problem. Lcov-Reporter shows incorrect line numbers, which makes me guess it really works on the already transpiled files (in this case react, but I do not think that this matters).
Output for text, html and cobertura seem to be fine... The main problem is that I cannot get other plugins to work in SonarQube, so I have to use lcov...
Is there any news on this problem?

@weblogixx Karma made everything complex. So, I've switched to more basic approach: Run mocha and istanbul directly from CLI through npm run. This way, what happens is clear and LCOV also works correctly, because I don't need to deal with karma plugins etc. I'm directly dealing with the core components directly.

@deeperx Thank you for your answer. This is interesting, it seems to be more like a karma coverage reporter problem then? Unfortunately, I am not able to change my setup from karma to pure mocha (I know it technically is possible, I already did it in one or two projects I am currently working on, but I have some projects that really depend on karma :().

Yes, it seems like that. It made so mad that I'd completely reconfigured to pure CLI based tools :)
Inanc Gumus
21 Tem 2016 tarihinde 08:23 saatinde, Chris<EMAIL_ADDRESS>şunları yazdı:

@deeperx Thank you for your answer. This is interesting, it seems to be more like a karma coverage reporter problem then? Unfortunately, I am not able to change my setup from karma to pure mocha (I know it technically is possible, I already did it in one or two projects I am currently working on, but I have some projects that really depend on karma :()�.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Link-check CI: only list errors

The Check Markdown links CI job currently lists all checked links, even successful checks. Only listing errors makes it easier to find mistakes.

/cc @kevinjqliu

is this correct? According to the docs use-quiet-mode should be yes/no

Seems to work


Add count of staking ledger accounts to GQL

Loosely relates to #1130

Required for "pagination" number (x of y of z - this issue is about z).

Blocks: https://github.com/Granola-Team/mina-block-explorer/issues/911

The test gives false feedback. I have reproduced this test and found a different result.
Here is how to reproduce:
Query:
query StakingLedgersQuery($limit: Int = 10, $query: StakeQueryInput!) {
  stakes(limit: $limit, query: $query) {
    epoch_num_accounts
    pk_epoch_num_blocks
    epoch_num_blocks
  }
}

Variables:
{
  "limit": 1,
  "query": {
    "epoch": 1,
    "ledgerHash": "jx7buQVWFLsXTtzRgSxbYcT8EYLS8KCZbLrfDcJxMtyy4thw2Ee"
  }
}

response:
{
  "data": {
    "stakes": [
      {
        "epoch_num_accounts": 0,
        "pk_epoch_num_blocks": 877,
        "epoch_num_blocks": 10350
      }
    ]
  }
}

There is something strange happening with the about query because the following returns correctly:
query Foobar {
  stakes(query: {epoch: 1}) {
    pk
    balance
    nonce
    chainId
    epoch_num_blocks
  }
}

I just tried, and it seems to be working for me...

It would work, or not, depending on whether prod is ingesting or not ingesting.


Requests with multipart form body keep loading

Expected Behavior
The request sends
Actual Behavior
When attempting to send a request it keeps loading indefinitely.
Reproduction Steps

Create any request (the actual url doesn't matter)
Change the body to "Multipart Form" (contents don't matter)
Attempt to send the request

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
2021.7.1
What operating system are you using?
macOS
Operating System Version
macOS  Monterey 12.0.1
Installation method
download from insomnia.rest
Last Known Working Insomnia version
2021.6.0

me too2021.6.0

I concur, same issue happening with me (on Windows,  Insomnia 2021.7.1 -- was working on prior versions)
app log shows error for any multipart form request:
[2021-12-09 10:45:36.918] [error] [catcher] Unhandled rejection: TypeError: Cannot read property 'app' of undefined
    at C:\Users\jeff\AppData\Local\insomnia\app-2021.7.1\resources\app.asar\main.min.js:120624:80
    at new Promise (<anonymous>)
    at buildMultipart (C:\Users\jeff\AppData\Local\insomnia\app-2021.7.1\resources\app.asar\main.min.js:120623:10)
    at C:\Users\jeff\AppData\Local\insomnia\app-2021.7.1\resources\app.asar\main.min.js:121253:19

My too on both (windows and mac)

Same happening on Windows. Request works fine when sending as JSON, loads indefinitely when sending as multipart

Can reproduce this. It was working on 2021.6.0.

This has been fixed after latest update (version 2021.7.2).  Can you kindly confirm it is the case for you folks?
cc @Gration1 / @screamndigit / @danielfalsetti / @Volksfeld

2021.7.2 seems to have fixed it.

@abbluiz the flatpak release is being setup by community efforts outside our team (e.g. by @WhyNotHugo).
I've reached out to @WhyNotHugo so we can also get permissions to update the flatpak release on https://github.com/flathub/rest.insomnia.Insomnia


DR readiness discovery

The task is to understand the current state of data stores (single AZ, multi-AZ) and create a list of TF resources associated with their deployment.  Document and analyze the gap between current state and desired state of readiness for DR.

Related to 1181

There is a skeleton entry in https://github.com/CDCgov/prime-reportstream/blob/master/prime-router/docs/docs-deprecated/playbooks/platform_restore.md
It refers to moving from Azure's useast to uswest by changing location = "eastus" to location = "westus"

Draft document (under development) is here: https://docs.google.com/document/d/18z1HLzTPkVQ7G0CiOVJDRH00ididrzhywVE8LacgatE/edit

The BC DR doc is in the External shared Gdrive. I sent Patrick Stoneburner a note to let him know that this is part of the current sprint and he has Commenter access. I also mentioned that it is a work in progress and I expect to build it out more when we have access to Sharepoint and Azure.


vlt package manager is not recognized by turborepo

Verify canary release

[X] I verified that the issue exists in the latest Turborepo canary release.

Link to code that reproduces this issue
https://github.com/hamlim/vlt-monorepo
What package manager are you using / does the bug impact?
npm
vlt
What operating system are you using?
Mac
Which canary version will you have in your reproduction?
2.2.3
Describe the Bug
The new vlt package manager doesn't seem to be recognized yet.
Expected Behavior
The vlt package manager should be recognized by turborepo
To Reproduce

Clone the reproduction repo: https://github.com/hamlim/vlt-monorepo
Run vlt install (install vlt via something like bun add -g vlt@latest or npm install -g vlt@latest, etc)
Run vlt run dev and see turborepo error

Additional context
No response

Note - the issue template also only allows me to select a set list of package managers that doesn't list vlt yet either

HI, we're not looking to support any new package managers right now. Maybe in the future!

Is there documentation on why there's a strict allowlist of package managers that turborepo works with as opposed to being agnostic of the package manager?

We parse lockfiles to get knowledge about the package and task graph, letting us create all the optimizations that we do. Since every package manager has a different lockfile format, we have to write fresh code for every one.

Going to move this do a Discussion as that's where we handle our feature requests.


add an option to check_enabled

Some services are not ment to be enabled but may be monitored as well.
An option "check_enabled" (default: true) that would skip the "service.is_enabled()" case would be of great help

This is solved in a fork on https://github.com/Penbase/check-systemd-service
The specific commit is https://github.com/Penbase/check-systemd-service/commit/1b55debcc9e3385fad0e41dafc5a608417dfc2df

This is solved in a fork on https://github.com/Penbase/check-systemd-service
The specific commit is https://github.com/Penbase/check-systemd-service/commit/1b55debcc9e3385fad0e41dafc5a608417dfc2df


Update notebooks

Just update notebooks.
The test is under the environmental variables setting:
# export XLA_PYTHON_CLIENT_PREALLOCATE=false
unset XLA_PYTHON_CLIENT_PREALLOCATE
# export XLA_PYTHON_CLIENT_ALLOCATOR=platform
unset XLA_PYTHON_CLIENT_ALLOCATOR
# export DO_NOT_USE_APPROX_IN_BINOM=1
unset DO_NOT_USE_APPROX_IN_BINOM

unset DO_NOT_USE_APPROX_IN_BINOM will use Normal as an approximation of Binomial which will greatly increase run speed.
For XLA_PYTHON_CLIENT_PREALLOCATE and XLA_PYTHON_CLIENT_ALLOCATOR, one can check reference: https://jax.readthedocs.io/en/latest/gpu_memory_allocation.html

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1818

Relevant Lines:
2268

💛  - Coveralls


support mustache?

There are dozens of us out here still using mustache templates. I've tried several different alternatives that all nearly support formatting it consistently for me, but nothing quite works. It seems like something this project would potentially be able to support?

Sorry, I don't have time to support new syntax, but let's keep this issue open.

Of course, totally understandable. I see djlint is receiving updates again, so maybe I'll give that one another try, but last time it wasn't working very well for us, and some devs complained about the complexity of managing python environments. So if anyone who comes across this thread has any recommendations, let me know.


PowerShell script not working

I am trying to execute the PowerShell script in this page, however I get the following exception (where "/201-web-app-redis-cache-sql-database" is the name of the folder at which the ARM template is placed)
For some reason the step at which the $template is being retrieved doesn't seem to find any templates, I have removed the "Where-Object" clause and all the returned objects, none of them is of type "Microsoft.DevTestLab/labs/artifactSources/armTemplates" as specified in the template!
Do I miss something?
Here is the powershell exception when I tried to execute the above powershell script.
Unable to find template /201-web-app-redis-cache-sql-database in lab DevTestLab001.
At C:\Users\amelbakr\OneDrive - Microsoft\AmrFolder\Demos\DevTestLab\DevTestLabCreation.ps1:81 char:28

... eq $null) { throw "Unable to find template $TemplateName in lab $LabN ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (Unable to find ... DevTestLab001.:String) [], RuntimeException
FullyQualifiedErrorId : Unable to find template 201-web-app-redis-cache-sql-database in lab DevTestLab001.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: aa4fa642-a91e-3dbf-ea97-5f5d59038435
Version Independent ID: 83dc54ff-9ac9-ce0f-b38e-270fca1638a4
Content: Create multi-VM environments and PaaS resources with Azure Resource Manager templates
Content Source: articles/lab-services/devtest-lab-create-environment-from-arm.md
Service: lab-services
GitHub Login: @spelluru
Microsoft Alias: spelluru

@amrelbakry Just wanted to confirm if you added https://github.com/Azure/azure-quickstart-templates.git to the repositories in your Dev Test Lab?

We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

@AmanGarg-MSFT
Yes, I have added the repository as indicated and I am having the same error.
The script is retrieving the Repository, then from there retrieving the ARM templates.
the repository is retrieved successfully, however the ARM template is not found!
I have even modified the script to look like this
$templates = Get-AzResource -ResourceGroupName $lab.ResourceGroupName     # -ResourceType "Microsoft.DevTestLab/labs/artifactSources/armTemplates"
-ResourceName "$LabName/$($repository.Name)" `
-ApiVersion 2016-05-15
ForEach ($template in $templates) {
$template
Write-Output "name: '$($template.Name)', Display Name: '$($template.Properties)'"
}
and none of the retrieved resources are templates!
am I missing something here?

@amrelbakry Can you try copying the 201-web-app-redis-cache-sql-database folder to your personal repo with fewer ARM templates and then try again?


Update tools-ui-components.md

Hello @miteshtagadiya!
Unfortunately, the page you edit in this PR no longer exists nor is maintained in the later docs, so I will close this as outdated.
If you would still like to contribute please feel invited to create new PRs for other open issues!


When custom port is specified, don't overwrite when connection type is set

The UX would be improved if the value of "Port" in "Server Setup" was only changed if the user hasn't edited the input yet. There has been a few times in which I set my port first, then change my connection type - the port is set to 21, and I need to re-put it back to custom value.
Acceptance criteria:

If the "Port" input hasn't been "dirtied"/changed by a human, then change it when "Type" is changed
If the "Port" input has been "dirtied"/changed by a human, then leave it alone

The fix has been released and It's available since version 1.3.2.


Double open event

Reopen of #19249
Fix double event issue in https://github.com/envoyproxy/envoy/issues/19232
When using the starttls filter, it breaks the expectation that connections should only raise the connected event once. This expectation is relied on for proper operation of core envoy functions like the connection pooler.
Risk Level: Low
Testing: Added regression test and tests for new code.

Oh dear that's not right

Just going to close this so everyone isn't notified even after I fix the commits. Sorry again.


Blueprint test should error if targeting an undefined stage

Right now it's easy to target the wrong stage. If the environment variable is set to a non-existent stage, we should error out.
We should also consider aliasing some of the old stage names:

converge -> apply
setup -> init

fixed with old stage mappings in CLI
https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/blob/9353691e4d74c65d2226d66c3ebe7f91e9673d6b/cli/bptest/stages.go#L5-L12


Have ScratchTrie create block and asynchronously send to LinearChain before applying to trie

Closing as this was the basis for "working set" and was intended as a portion of it.


targetSdkVersion mismatch

Hi,
Thanks for providing good sample app!
When installing with $ adb install -t Application-debug.apk" on Pixel (OPR1.170321.003),
Err "Requires development platform android-O (current platform is any of [O])]" occurred.
I've tried "O" for targetSdkVersion and solved the problem.
Could you please check?
(PS; icon for app is missing and build is failed...)

I signed it!

This sample has been deprecated/archived (check README for more information on newer samples related to this technology).
As recommended by GitHub, we are closing all issues and pull requests.


Checking in IO protocols and readers

IO protocols
Basic raster based reader to read test data

Checking in Delaunay triangulation

Sorry, can you revert this commit and put it in a separate PR? If you're comfortable with git rebase, you can chain the PR branches on top of each other, then rebase the newer PRs on top of the main branch as the older PRs get merged.
I concede that that's a lot more burdensome on you when the PRs have dependencies on each other, but it makes my life as a reviewer a lot easier. I can get through a bunch of smaller PRs much more efficiently than one big PR.
No worries if reverting the commit is a problem this time, but going forward let's try to figure out a workflow that involves chaining multiple smaller PRs, since that's the only way that I'll be able to continue reviewing them.

Ignore the Delaunay files for now. Moving forward, I will daisy chain the PRs. The next PR will be Delaunay related and we can discuss that in more detail then.


Add failure stats to test output

Codecov Report
Attention: 14 lines in your changes are missing coverage. Please review.

Comparison is base (d108620) 5.71% compared to head (74edd93) 5.56%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##           master     #25      +/-   ##
=========================================
- Coverage    5.71%   5.56%   -0.15%
=========================================
  Files           6       6
  Lines         490     503      +13
=========================================
  Hits           28      28
- Misses        460     473      +13
  Partials        2       2

Flag
Coverage Δ

unittests
5.56% <0.00%> (-0.15%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

app/main.go
0.00% <0.00%> (ø)

app/report.go
6.42% <0.00%> (-0.38%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Flaky test for show action

Situation
Sometimes CI fails for the .show action. Mainly deletion of intermediate data seems to be happening before show action.
Acceptance Criteria

The issue is investigated and fixed

Not valid anymore, is not possible to reproduce. We will address it if we observe same issue again.


[main] Format Go code

Cron -knative-prow-robot
/cc knative-sandbox/eventing-writers knative-sandbox/serving-writers
/assign knative-sandbox/eventing-writers knative-sandbox/serving-writers
Produced by: knative-sandbox/knobots/actions/update-gofmt
Details:

/lgtm
/approve


Using DVDI Module with Docker Containers

Greetings,
Wondering if someone can supply an example of a JSON file where Docker container would mount DVDI provided volume ? Let's say I want to  elasticsearch container to mount a volume called elkvol001 as the /data provided by DVDI module. Something along the lines of this:
{
	"id": "elasticsearch",
	"container": {
		"type": "DOCKER",
		"docker": {
			"image": "elasticsearch",
			"network": "BRIDGE"
		}
	},
	"volumes": [{
		"containerPath": "/data",
		"hostPath": "/var/lib/rexray/volumes/dbdata",
		"mode": "RW"
	}],
	"cpus": 0.2,
	"mem": 512.0,
	"env": {
		"DVDI_VOLUME_NAME": "elkvol001",
		"DVDI_VOLUME_DRIVER": "rexray",
		"DVDI_VOLUME_OPTS": "size=5,iops=150,volumetype=io1,newfstype=xfs,overwritefs=true"
	},
	"instances": 1
}

The DVDI isolator is not used (or needed) for external volume mounts from Docker containers.
The DVDI isolator is for use with tasks utilizing the Mesos containerizer instead of the Docker containerizer. For various reasons, many Mesos users elect to use the MEsos containerizer instead of using Docker.
If you want to use Docker with external volume mounts, this works, it just doesn't use the DVDI isolator.
This blog post covers external volume mounts from a Docker container on Mesos.
http://blog.emccode.com/2015/08/28/run-your-stateful-apps-with-mesos-and-docker/

Steve, Thanks so much!!!


Issue with script editor in visual editor for sign in options

Describe the issue:

When try to edit content on script editor, the toggle is automatically on/off.
https://github.com/wso2/product-is/assets/37938529/42e28eaf-2d43-400e-aebf-102fd1916552
How to reproduce:

Login to console.
Go to an application > sign in options > visual editor.
Turn on the conditional authentication toggle and try to edit the script (Hit enter).


Disable __DEBUG__ for browser build

This PR fixes the issue described in https://github.com/petkaantonov/bluebird/issues/897
When using Bluebird with webpack or browserify, the ./js/browser/bluebird.js file is used, which is currently build with __DEBUG__ set to true.
This has a major performance impact on Bluebird in a production environment.
This PR sets __DEBUG__ to false by default, except if build.js is invoked with --debug.
Please let me know if there are any changes you would like me to make.

:+1:

Is there any way to enable __DEBUG__ when e.g. browserify is invoked with --debug ?

@spion, you can run node tools/build.js --debug --release --browser --minify to have __DEBUG__ set to true to get a DEBUG build, but you can't set it from browserify since the built artifacts are bundled with the npm package and accessed via the   "browser": "./js/browser/bluebird.js" setting in bluebird's package.json, not generated on the fly

@suprememoocow sure; but one of the things that bluebird makes easy is getting started with it: the defaults are developer-friendly (long stack traces when NODE_ENV=development and in the browser, logging possibly unhandled rejections...)
So I was wondering if it may be possible to both keep that property and not require custom builds. (e.g. if browserify injected a process.browserify.debug = true global, we could use that...

Thats a great point, but I'm not sure whether it's in the scope of this pull-request as there are additional things to take into account if you wanted to allow that:
For example, relying on process.browserify.debug would break in webpack. You could possibly use process.env which webpack and browserify will shim for you but it would need further discussion.
This PR is just making the browser build consistent with other builds:
Current situation

Build
Long Stack Traces
Possible to disable long stack traces?

node
Off by default, can be enabled via env vars
Yes

browser minified
Off by default
Yes

browserify/webpack
On by default
No

This PR

Build
Long Stack Traces
Possible to disable long stack traces?

node
Off by default, can be enabled via env vars
Yes

browser minified
Off by default
Yes

browserify/webpack
Off by default
Yes

Technically, we could fix things so you can turn off LST by using e.g. envify with browserify.
browserify index.js -t [ envify --BLUEBIRD_LONG_STACK_TRACES 0 ]

Right now that doesn't work due to the fact that a helper is used to read environment vars.

Ah, now I see: "Possible to disable long stack traces?" - "No". Yeah, thats definitely a real problem.

Yeah, perhaps I haven't been explaining the problem in the right way :)

So the only problem is longStackTraces: false not disabling long stack traces? That should be fixed instead I think

@petkaantonov in the webpack build, yes, that's correct. Can you suggest another approach?

It's not straight forward so I just did it myself

Thanks for the quick response @petkaantonov


[USERSNAP] Improve Search

Taken at: https://gov.tools/drep_directory
Fields:

Name of the idea or new feature: Improve Search
Please summarize your idea or new feature: Remembering the Drep ID is completely impossible. Please can we search via the username. For example my name is set as zkdid but you cannot search any of the Dreps from their name. That would make sense because sadly right now from my mobile phone and I’m not able to even find my Drep ID from this website.

Thanks 🙏
Browser: Safari 17 (iOS)
Screen size: 428x926
Open #75 in Usersnap
Powered by Usersnap.

duplicate https://github.com/IntersectMBO/govtool/issues/1893


Incorrectly setting __config_persisted

In config.py we incorrectly set __config_persisted, resulting in database initialization and config write happening twice.  The following snippet is from config.py:
r = db.singletons.replace_one({'_id': 'config'}, __config, upsert=True)
__config_persisted = bool(r.modified_count)

on upsert, modified_count will be equal to 0.  The mongo response object has a property called "upserted_id" which will have the id of the upserted document.  Thus the first time we save the config, r.modified_count is 0 and when we cast that to a boolean it returns False, meaning __config_persisted is set to False.

Nice catch, @coltonlw.
Seems to me that we don't actually care whether an insert or an update took place. Since any problem with the Mongo operation would raise an exception and stop the local control flow, we should be fine simply setting __config_persisted to True. Right?

Yeap that sounds correct to me


FGR-223 inclusion fails with TypeError: Cannot convert undefined or null to object

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?
NO, my problem is NOT within Home Assistant or the ZWave JS integration
Is your problem within Z-Wave JS UI (formerly ZwaveJS2MQTT)?
NO, my problem is NOT within Z-Wave JS UI
Checklist

[X] I have checked the troubleshooting section and my problem is not described there.

[X] I have read the changelog and my problem was not mentioned there.

Describe the bug
What causes the bug?
Inclusion of FGR-223
What do you observe?
Trying to include two FGR-223 shutter module. They both start inclusion, ask for pin, but are then stuck at "OverwriteConfig". I use latest zwave js version. I've tried to re-interview them. Everything seems good except last part:
17:56:24.521 DRIVER   Dropping message because it could not be deserialized: The command class Appli
                      cation Status is not implemented (ZW0303)
17:56:24.522 SERIAL » [ACK]                                                                   (0x06)
17:56:24.996 CNTRLR   [Node 056] [translateValueEvent: value added]
                        commandClass: Central Scene
                        endpoint:     0
                        property:     interviewComplete
                        propertyKey:  undefined
                        internal:     true
                        secret:       false
                        event source: undefined
17:56:25.001 CNTRLR   [Node 056] [+] [Central Scene] interviewComplete: true [Endpoint 0] [internal]
2023-08-21 17:56:25.004 INFO Z-WAVE: [Node 056] Interview stage COMMANDCLASSES completed
17:56:25.008 CNTRLR   [Node 056] Interview stage completed: CommandClasses
2023-08-21 17:56:25.013 INFO Z-WAVE: [Node 056] Interview stage OVERWRITECONFIG completed
17:56:25.016 CNTRLR   [Node 056] Interview stage completed: OverwriteConfig
2023-08-21 17:56:25.032 ERROR APP: Unhandled Rejection, reason: TypeError: Cannot convert undefined or null to object
17:56:25.104 DRIVER   all queues idle

I attach the node's json. Firmware version is 5.1
Device information
Manufacturer: Fibaro
Model name: FGR-223
Node ID in your network: 56
How are you using node-zwave-js?

[X] zwave-js-ui (formerly zwavejs2mqtt) Docker image (latest)
[ ] zwave-js-ui (formerly zwavejs2mqtt) Docker image (dev)
[ ] zwave-js-ui (formerly zwavejs2mqtt) Docker manually built (please specify branches)
[ ] ioBroker.zwave2 adapter (please specify version)
[ ] HomeAssistant zwave_js integration  (please specify version)
[ ] pkg
[ ] node-red-contrib-zwave-js (please specify version, double click node to find out)
[ ] Manually built from GitHub (please specify branch)
[ ] Other (please describe)

Which branches or versions?
zwave-js-ui branch: latest
Did you change anything?
no
If yes, what did you change?
No response
Did this work before?
Don't know, this is a new device
If yes, where did it work?
No response
Attach Driver Logfile
zwavejs_2023-08-21.log
Node's json :
{
  "id": 56,
  "name": "",
  "loc": "",
  "values": [],
  "groups": [],
  "neighbors": [],
  "ready": false,
  "available": true,
  "hassDevices": {},
  "failed": false,
  "inited": false,
  "eventsQueue": [
    {
      "time": "2023-08-21T17:48:01.946Z",
      "event": "alive",
      "args": [
        0
      ]
    }
  ],
  "status": "Alive",
  "interviewStage": "OverwriteConfig",
  "priorityReturnRoute": {},
  "customReturnRoute": {},
  "customSUCReturnRoutes": [],
  "applicationRoute": false,
  "hexId": "0x010f-0x0303-0x1000",
  "dbLink": "https://devices.zwave-js.io/?jumpTo=0x010f:0x0303:0x1000:0.0",
  "manufacturerId": 271,
  "productId": 4096,
  "productType": 771,
  "deviceConfig": {
    "filename": "/usr/src/app/store/.config-db/devices/0x010f/fgr223.json",
    "isEmbedded": true,
    "manufacturer": "Fibargroup",
    "manufacturerId": 271,
    "label": "FGR223",
    "description": "Roller Shutter 3",
    "devices": [
      {
        "productType": 771,
        "productId": 4096
      },
      {
        "productType": 771,
        "productId": 12288
      },
      {
        "productType": 771,
        "productId": 16384
      }
    ],
    "firmwareVersion": {
      "min": "0.0",
      "max": "255.255"
    },
    "preferred": false,
    "paramInformation": {
      "_map": {}
    },
    "proprietary": {
      "fibaroCCs": [
        38
      ]
    }
  },
  "productLabel": "FGR223",
  "productDescription": "Roller Shutter 3",
  "manufacturer": "Fibargroup",
  "firmwareVersion": "5.1",
  "protocolVersion": 3,
  "zwavePlusVersion": 1,
  "zwavePlusNodeType": 0,
  "zwavePlusRoleType": 5,
  "nodeType": 1,
  "endpointsCount": 2,
  "endpoints": [
    {
      "index": 0,
      "label": "Root Endpoint"
    },
    {
      "index": 1,
      "label": "Endpoint 1"
    },
    {
      "index": 2,
      "label": "Endpoint 2"
    }
  ],
  "isSecure": true,
  "security": "S2_Authenticated",
  "supportsSecurity": false,
  "supportsBeaming": true,
  "isControllerNode": false,
  "isListening": true,
  "isFrequentListening": false,
  "isRouting": true,
  "keepAwake": false,
  "maxDataRate": 100000,
  "deviceClass": {
    "basic": 4,
    "generic": 17,
    "specific": 6
  },
  "lastActive":<PHONE_NUMBER>943,
  "firmwareCapabilities": {
    "firmwareUpgradable": false
  },
  "deviceId": "271-4096-771",
  "statistics": {
    "commandsTX": 1,
    "commandsRX": 0,
    "commandsDroppedRX": 0,
    "commandsDroppedTX": 0,
    "timeoutResponse": 0,
    "rtt": 55.1,
    "lastSeen": "2023-08-21T17:48:01.943Z"
  },
  "_name": "NodeID_56",
  "prioritySUCReturnRoute": false
}

Interesting fact : it worked downgrading to 8.15

I'm able to reproduce. Fix is on the way.

Z-Wave JS UI v8.23.1 is releasing now.

Just for your information, the issue is/was not only for FGR-223, but also at the FGR-222 shutters. (Fibaro Shutter 2 [FGR-222] and Fibaro Shutter 3 [FGR-223].

Same root issue, same solution


Degree Of Separation off by 1

Make direct follows 1st degree separation instead of N/A separation (No text displayed). Makes more sense, otherwise they are trusted the same as individuals with no relation currently.

Alternatively, this bug could be fixed by displaying degree 0 on direct follows if we prefer doing it that way.

Turns out, the function itself has "degree 0" implemented, it was just the current templates that did not make use of it. #111 addresses this


Stack [Python]

https://en.wikipedia.org/wiki/Stack_(abstract_data_type)
Require following functionalities -

Print
Push
Pop
Peek
Get size

Create a class and add example usage in single file.

@aviaryan @Monal5031 @yashLadha @jsroyal : Does this looks good?

Looks Good to me.

Can we define some functionalities should be implemented in all data structure in contribute.md
or just define yourself to implement functionalities.

@jsroyal Individual will be better, the existing can be updated later if we want to add some more functionality

Okay. Let's see :)

Taking this up.

@singhpratyush Please assign yourself sir, if you are doing this?

I am waiting for #65 to be resolved as I am currently assigned to #41. Sadly, I see no activity on the pull request.


Size of custom language model is very small compared to common_crawl_00.prune01111.trie.klm

Hi,
I've created a language model using kenlm on common crawl data. The size of resultant output arpa is only 51Mb. The trained language model that you have provided is close to 8Gb in size which is also trained on common crawl dataset (common_crawl_00.prune01111.trie.klm). Could you tell why this could happen?
Command used to create the language model:
bin/lmplz -o 5 --prune 01111 -T /datadrive/ -S 50% --text en.00.deduped --arpa en.00.deduped.arpa
Output:
=== 1/5 Counting and sorting n-grams ===
Reading /datadrive/speechexperiments/paddle/datasets/en.00.deduped.sadguru.processed
----5---10---15---20---25---30---35---40---45---50---55---60---65---70---75---80---85---90---95--100
****************************************************************************************************
Unigram tokens<PHONE_NUMBER> types 109380831
=== 2/5 Calculating and sorting adjusted counts ===
Chain sizes: 1:1312569972 2:6413600768 3:12025502720 4:19240802304 5:28059504640
Statistics:
1 116081/109380831 D1=0.832246 D2=1.08381 D3+=1.25936
2 641408/699725052 D1=0.817398 D2=1.02413 D3+=1.20664
3 525936/2424418023 D1=0.843848 D2=1.08604 D3+=1.26471
4 241171/4531878860 D1=0.894573 D2=1.15259 D3+=1.2933
5 152819/5933461247 D1=0.898007 D2=1.2717 D3+=1.32749
Memory estimate for binary LM:
type       kB
probing 38645 assuming -p 1.5
probing 47352 assuming -r models -p 1.5
trie    20622 without quantization
trie    12119 assuming -q 8 -b 8 quantization
trie    18450 assuming -a 22 array pointer compression
trie     9947 assuming -a 22 -q 8 -b 8 array pointer compression and quantization
=== 3/5 Calculating and sorting initial probabilities ===
Chain sizes: 1:1392972 2:10262528 3:10518720 4:5788104 5:4278932
=== 4/5 Calculating and writing order-interpolated probabilities ===
Chain sizes: 1:1392972 2:10262528 3:10518720 4:5788104 5:4278932
=== 5/5 Writing ARPA model ===
Name:lmplz	VmPeak:67678176 kB	VmRSS:85464 kB	RSSMax:67560500 kB	user:12830.9	sys:2520.67	CPU:15351.5	real:15451.3

Seems that you have done some processing for en.00, maybe that is why.

Following is the preprocessing I'm doing:

Characters not in [A-Za-z0-9\s'] (\s represents whitespace characters) are removed and Arabic numbers are converted to English numbers like 1000 to one thousand.
Repeated whitespace characters are squeezed to one and the beginning whitespace characters are removed. Notice that all transcriptions are lowercase, so all characters are converted to lowercase.

After this preprocessing, the size of the en.00... is around 55GB, but the output is only 51MB

The problem is solved, the issue was in the command.
I had given --prune 01111, it should be --prune 0 1 1 1 1
bin/lmplz -o 5 --prune 0 1 1 1 1 -T /datadrive/ -S 50% --text en.00.deduped --arpa en.00.deduped.arpa

Hello - Where do I get the lmplz from?
And can I get the common_crawl_00.prune01111.trie.klm from the above mentioned speech model corpus using the commands mentioned on this thread?


Support in linux kernel 5.4

Hi :)
I'm using your ZYDIS in my LKM and therefore zycore-c submodule, and since kernel 5.4 (using gcc >7), I'm receiving the following error:
./include/linux/compiler_attributes.h:200:41: error: expected ‘)’ before ‘__attribute__’
  200 | # define fallthrough                    __attribute__((__fallthrough__))
      |                                         ^~~~~~~~~~~~~
/home/chameleon/mlp.main/mlp.kernel/main-module/include/Zycore/Defines.h:246:45: note: in expansion of macro ‘fallthrough’
  246 | #   define ZYAN_FALLTHROUGH __attribute__ ((fallthrough))
      |                                             ^~~~~~~~~~~
/home/chameleon/mlp.main/mlp.kernel/main-module/repos/zydis/src/Decoder.c:2617:17: note: in expansion of macro ‘ZYAN_FALLTHROUGH’
 2617 |                 ZYAN_FALLTHROUGH;
      |                 ^~~~~~~~~~~~~~~~

Now, after some research it seems that the problem was due the fact that in kernel 5.4 this commit inserted a fallthrough define.
the same issue was at vbox
And after testing, the solution was replace the
#   define ZYAN_FALLTHROUGH __attribute__ ((fallthrough))

to
#   define ZYAN_FALLTHROUGH __attribute__ ((__fallthrough__))

Can you apply this change?
Many thanks

Hey there,
I will have a look. Need to check if this change does have any side effects with different compiler versions.

Thanks for answer :)
I've tried this with gcc version 7,8,9 and seems to work fine.
Is there any other tests I can do to validate?
If you want I can make a PR on this change

That sounds promising :-) A PR is always appreciated!

I checked this on godbolt.org earlier today, and interestingly enough, despite there being no documentation on it, the variant with the additional underscores seems to work fine for all GCC versions in our ifdef filter (>= 7), so we should be good to go with that. :-)
On a related side note, C23 adds [[fallthrough]] to the standard, so we might be able to just use that in a few years.

Thanks for the PR. I just merged it and will bump the zycore-c dependency of zydis later today.
Solved with https://github.com/zyantific/zycore-c/pull/26


Sankey Flow Diagram, closes #196

Currently breaking on products like petroleum gas and uranium fuel cells.


Handle case when struct abis don't match up exactly. Version bump to v0.0.46

Codecov Report
Attention: Patch coverage is 9.09091% with 10 lines in your changes missing coverage. Please review.

Project coverage is 52.57%. Comparing base (ab3d0bc) to head (5647ae0).

Files with missing lines
Patch %
Lines

pypechain/render/contract.py
0.00%
7 Missing :warning:

pypechain/utilities/abi.py
33.33%
2 Missing :warning:

pypechain/render/types.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #146      +/-   ##
==========================================
- Coverage   52.63%   52.57%   -0.06%
==========================================
  Files          80       80
  Lines        5605     5611       +6
==========================================
  Hits         2950     2950
- Misses       2655     2661       +6

Flag
Coverage Δ

unittests
52.57% <9.09%> (-0.06%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


feat: update deps and improve failure testing (#31)

This pull request updates the dependencies including an update of the go-version and the Makefile. It also improves (fixes) the failure testing of the test isolation framework adding a default expectation in case of failures to simplify setup.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 7 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
939

Relevant Lines:
939

💛  - Coveralls


Variant-analysis: feedback from the embedded feedback form

@galaxyproject/training-variant-analysis
Notes for all tutorials in this topic (16.09.2019):

Details per tutorials:

Calling variants in diploid systems

Pro:

cartoons are nice (04/03/2019)

Con:

I'm still in the middle of it.  But when doing snpEFF there is no refernece genome available.  This tutorial does not say to download one.  Other variant analyses say to download hg19.  this is not available in the shared data folders.  When using an hg19 fasta from another source freeBayes fails because it isn't indexed.  Not sure how to index it.  SnpEFF calls for a database.  I downloaded one and am trying it now.  But it would be good to include this step in the tutorial (04/11/2018)
There is a problem in SnpEff step. There is no option for "Locally installed reference genome" when clicking Genome resource causing the next step cannot be proceed (15/04/2019)

Calling variants in non-diploid systems

Pro:

I like the idea (04/11/2018)

Con:

After running the FreeBayes I ended up with 2 variants instead of 30.  And so got stuck there.  Also it's not clear why the filtering and subsequent steps would be done on the markduplicates output, rather than the left align output.  Why do the leftalign if we don't use that output again?  I did both and got the same results. Overall I am finding the galaxy experience to be rather frustrating.  I'm trying to decide whether to incorporate these tutorials in my class, but often I can't find the necessary input data, get unexplained errors, or get output different from the tutorial (04/11/2018)

Exome sequencing data analysis

Pro:

I like the idea of it (04/11/2018)

Con:

It seems impossible to complete.  The training data is not available in the shared data link.  If it is, it's too hard to find.  A more general point would be to explain how the tutorial could be used on other data sets.  or ref genomes.  For this I tried using an hg19 fasta file from another data set, but got an error about indexing.  So I guess, explaining how the training data sets are pre-formatted would help when trying to use sets from other sources (04/11/2018)
unfortunately in compare with others tutorial, this is very unclear and o'm not satisfied please improve this very important tutorial with clear and accurate dive. (26/12/2018)

Exome sequencing data analysis for diagnosing a genetic disease

Pro:

looking at genes (26/07/2019)
Even simple steps were explained. (26/07/2019)
How the tuturial is complete (26/07/2019)

Con:

Remember to list the softwares that is in galaxy because Gemini ins't there (26/07/2019)

Identification of somatic and germline variants from tumor and normal sample pairs

Mapping and molecular identification of phenotype-causing mutations

Pro:

all (09/04/2019)

Con:

more screenshots (09/04/2019)

Microbial Variant Calling

Pro:

This was very straight forward and ran well (once it was clear that it only runs on the eu site).  listing the file types was useful (04/11/2018)
the tutorial is quiet clear, plus the images and screenshots are really helpful (11/09/2019)
Hands on work (11/09/2019)

Con:

maybe link to descriptions of the various file type generated/used (04/11/2018)
The step "Select all outputs" in "Hands-on: Run Snippy" is not obvious what to do exactly.  (24/07/2019)
Good experience using Galaxy (11/09/2019)

Now moved here: https://training.galaxyproject.org/feedback


Can this be used now?

Sorry if this is not the right place to post this question, but can this be used to import FamilySearch Trees to Gramps, now? Is it ready to be used? If so, how do I use it?   Didn't see anything on the wiki..
Thanks.

Please get in touch with “EderaMevans” to learn the status of these products.
FamilySearch-gramps
Familysearch-python-sample-apps
FamilySearch-python-sdk-opensource
These products are not familysearch nor supported by familysearch.
Gordon
From: shanghaiknight<EMAIL_ADDRESS>Sent: Friday, May 26, 2017 8:56 AM
To: elderamevans/familysearch-gramps<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [elderamevans/familysearch-gramps] Can this be used now? (#10)
Sorry if this is not the right place to post this question, but can this be used to import FamilySearch Trees to Gramps, now? Is it ready to be used? If so, how do I use it? Didn't see anything on the wiki..
Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_elderamevans_familysearch-2Dgramps_issues_10&d=DwMCaQ&c=z0adcvxXWKG6LAMN6dVEqQ&r=7fHR0xh9JmJl6y-H9ULa_ojTWCIFdDJCoUjMh7C6uWQ&m=vG4ClXBKE_sfmc51_QP1iyF8Wfs91MPP6aGhnCPAsdE&s=x-0RGt9ksVzl_AETM0oD30U386x9IUhoM2pLWy8ADsI&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AAzLfYT4JiwhFvwdAZzKapBisELMqkCcks5r9ufzgaJpZM4NnsyN&d=DwMCaQ&c=z0adcvxXWKG6LAMN6dVEqQ&r=7fHR0xh9JmJl6y-H9ULa_ojTWCIFdDJCoUjMh7C6uWQ&m=vG4ClXBKE_sfmc51_QP1iyF8Wfs91MPP6aGhnCPAsdE&s=FwzGtH5nugTqGJyCDA5E9OzrCfzS2cl8HdYf4CKCi7Q&e=.


Export LottieProps

How would you feel about exporting the LottieProps type?
I think it would be handy in scenarios where we may want to offload some logic that determine what props to pass in to the player so that it's outside of the main component that renders the player. As an over-simplified example:
const getLottieProps = (componentProps: MyProps): LottieProps => {
  // Some logic that determines the props
  const play = true;
  const goTo = 123;
  return { play, goTo };
}

const MyComponent = (props: MyProps) => {
  return (

      <AnimationHeader/>
        <Lottie {...getLottieProps(props)}/>
      <AnimationFooter/>

    );
}

Yeah, that's one option. Although sometimes its desirable to have a pure stateless component.
There's definitely a few options to get around the example I gave, but I feel it would be beneficial to allow consumers of this library to access the LottieProps type. I dont really see any reason to keep it private.
Another use case would be extending the type. Perhaps my component is a simple wrapper around the lottie player component and I need a few extra properties, eg.
  type MyProps = LottieProps & {
    myHeaderText: string;
    myFooterText: string
  };

  const MyComponent = (props: MyProps) => {
    return (

        <AnimationHeader text={props.myHeaderText}/>
          <Lottie {...props}/>
        <AnimationFooter text={props.myFooterText}/>

    );
  }

Oops sorry, I think I misread. You're only talking about a typescript change here, right? TBH I'm not so proficient in TS, but if it's just a change to index.d.ts that doesn't break anything, then I'm open for a PR!

i think your issue might be a duplicate of #46 ?

Yeah, just a little TS change - should be fairly unobtrusive.
My ticket is indeed a dup, sorry about that. I'll close this one off and I'll have a look at opening a PR for that change this week.


Mjmlconfig

@kmcb777 @iRyusa Following up on our discussion from #1131
I included a couple enhancements that seemed to make sense:

you can specify community components installed via npm by their standard requirable package names/paths (i.e. "mjml-chart" rather than "./node_modules/mjml-chart/lib/index.js"). I handle scenarios where this fails to resolve correctly.
if paths in the packages array resolve to a non-callable object whose properties are components (like mjml-accordion does), I iterate over the keys of the object and register the callable component classes instead

I'm going to create a simple test project and make sure this will work with mjml-charts and an additional custom component and then link the repo here so you guys can verify

Hey !
That looks really great for me ! Just I would stick to current --config syntax over adding a new option

Here's my test repo: https://github.com/jskrzypek/mjml-config-test-project
I'll fix the config option thing now

@iRyusa if there's any other changes you'd like me to make on this pr,  just let me know 😄

HI @jskrzypek
We just freeze our dev on 4.1 so it will be merged for a 4.1.1 👌

@jskrzypek nice PR, i'll merge it right now. I'll just change the option name configPath into mjmlConfigPath and the function registerCustomComponents into handleMjmlConfig because i also made a PR to make custom components work on the app and that's the names i used ;)


Pattern: Change the Card section - simple and Card section with images patterns for the Cards Group

User Story

As a [user role below]:
IBM.com Library developer

I need to:
change the Card section - simple and Card section - with images patterns`

so that I can:
provide the ibm.com adopter developers patterns/components they can use to build ibm.com web pages

Additional information

Story within Storybook with corresponding knobs
Utilize Carbon
Both the Card section - simple and Card section - with images patterns are used in the Cards group
Functional specs issues (#2252 ) and (#2028)
Card section - simple Box folder link
Card section - with images box folder link

Acceptance criteria

[ ] Include README for the react pattern and corresponding styles
[ ] Minimum 80% unit test coverage

@kennylam Jeff assigned this issue to you because it is related to the Cards group development that you volunteered to do.


HA Production "ready"

Hi, i have used microk8s 1.19 and 1.20 with 3 and 10 hosts, i don't do any exceptional thing to use it but if i read on the microk8s site, microk8s is a production-grade and supports HA with dqlite, but (this things are experiences using it):

One of my nodes has network intermittent issues and the cluster break, i tried to force delete the node but the cluster never get stable again, i was waiting that dqlite supports this, but at the end the cluster get inconsistent, for example, if i list the daemonset, calico are showing that 3 pods are running and ready, but when list the pods, it's never appear, so the information on database was completely inconsistent, that's why i say about dqlite.

One of the HA things that i expect is that if a node lost the connection, the theory is that dqlite select another master and all come to stable again, my experience with this was bad, lost a node and all the cluster get inconsistent like the first point.

The thing is that never i get a real HA recover from a issue, so how is tested this HA things of microk8s??? Sorry now i moved to rke2 with embedded etcd and tested the network issues, the lost of a node and everything works, the cluster come to stable again....

Hi @lfdominguez, I am sorry to see you leave and I appreciate letting us know the reasons.
The case you describe should be handled. Can you extend on two points that would help me reproduce the issue?

How can I simulate a node is lost? You perform a forced power off not leaving any time for the cluster to do a graceful shutdown?
What workloads do you have? Anything special I should be aware of?

Thank you.

Thanks for response, really i liked microk8s (the snap autorefresh is insane but i worked with the download and install --dangerous jeje). The simulation is removing completly a node (are VMs), but this was for rke2, for microk8s was in production, on hetznet cloud, their lost the connection to one of the DC and then i lost the network with one of the nodes and the cluster get inconsistent.
The real thing is that the cluster get running, but, the information showed by kubectl was wrong, like i say, for example, if you show the deployments or daemonsets, they show the RUNNING and READY columns as the pods was running, but if you get the pods, where is not there because i need to delete because lost, is like the information get from dqlite was wrong or something else. A real example, i found an screenshoot that i take:

you can see there how the ingress ds is with 1 and there 3 pods "running" (really there is nothing running when i use ctr to show containers running in containerd), and the calico ds show 3 running, but there is not calico pods. How one can solve this in this situation? if we can solve or known how to solve this i can go back, really i like the microk8s.

Because that behavior is that if you remove for example, all the ingress, is never redeploy because the deployment or ds show has is deployed.


Refactor GridComponents to enable use outside VmDetails

Changes:

Move GridComponents.js from src/components/VmDetails to src/components/Grid
Added Grid/index.js in the same style as patternfly-react
Adjust existing import statements to match the new location

Pre-work for PR #987, Issue #923

@bond95 - I added an offset to the Col along the same lines as what you pointed to.  I even added some fancy validation functions for the PropTypes that check ranges on cols and offset so they make sense, at least on a single Col.


Host key verification failed. fatal: Could not read from remote repository.

error/failure when doing a "make nb" on macos 10.15.7
here's the end of the debug.log:
3875 verbose stack Error: exited with error code: 128
3875 verbose stack     at ChildProcess. (/usr/local/lib/node_modules/npm/node_modules/pacote/lib/util/finished.js:12:19)
3875 verbose stack     at ChildProcess.emit (events.js:182:13)
3875 verbose stack     at maybeClose (internal/child_process.js:962:16)
3875 verbose stack     at Socket.stream.socket.on (internal/child_process.js:381:11)
3875 verbose stack     at Socket.emit (events.js:182:13)
3875 verbose stack     at Pipe._handle.close (net.js:610:12)
3876 verbose cwd /Users/luxiouronimo/Documents/NewsBlur/node
3877 verbose Darwin 19.6.0
3878 verbose argv "/usr/local/bin/node" "/usr/local/bin/npm" "install"
3879 verbose node v10.14.2
3880 verbose npm  v6.9.0
3881 error Error while executing:
3881 error /usr/bin/git ls-remote -h -t<EMAIL_ADDRESS>3881 error
3881 error Host key verification failed.
3881 error fatal: Could not read from remote repository.
3881 error
3881 error Please make sure you have the correct access rights
3881 error and the repository exists.
3881 error
3881 error exited with error code: 128
3882 verbose exit [ 1, true ]

after fresh git clone, make nb:
npm ERR! Error while executing: npm ERR! /usr/bin/git ls-remote -h -t<EMAIL_ADDRESS>npm ERR! npm ERR! Host key verification failed. npm ERR! fatal: Could not read from remote repository. npm ERR! npm ERR! Please make sure you have the correct access rights npm ERR! and the repository exists. npm ERR! npm ERR! exited with error code: 128
npm ERR!  /Users/luxiouronimo/.npm/_logs/2021-10-06T13_53_02_905Z-debug.log
npm install @postlight/mercury-parser
that appears to have worked, despite claims of package.json not existing:
npm WARN saveError ENOENT: no such file or directory, open '/Users/luxiouronimo/Documents/NewsBlur/package.json' npm notice created a lockfile as package-lock.json. You should commit this file. npm WARN<EMAIL_ADDRESS>had bundled packages that do not match the required version(s). They have been replaced with non-bundled versions. npm WARN enoent ENOENT: no such file or directory, open '/Users/luxiouronimo/Documents/NewsBlur/package.json' npm WARN NewsBlur No description npm WARN NewsBlur No repository field. npm WARN NewsBlur No README data npm WARN NewsBlur No license field. npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/@postlight/mercury-parser/node_modules/http-headers): npm WARN optional SKIPPING OPTIONAL DEPENDENCY: Cannot read property 'type' of undefined

<EMAIL_ADDRESS>added 165 packages from 246 contributors and audited 165 packages in 8.938s
found 2 vulnerabilities (1 low, 1 moderate)
run npm audit fix to fix them, or npm audit for details

make nb:
npm ERR! Error while executing: npm ERR! /usr/bin/git ls-remote -h -t<EMAIL_ADDRESS>npm ERR! npm ERR! Host key verification failed. npm ERR! fatal: Could not read from remote repository. npm ERR! npm ERR! Please make sure you have the correct access rights npm ERR! and the repository exists. npm ERR! npm ERR! exited with error code: 128
npm ERR!  /Users/luxiouronimo/.npm/_logs/2021-10-06T14_00_41_003Z-debug.log

How about manually execute npm install postlight/difflib ? Maybe this will work @luxiouronimo

oki this is no longer an issue; it now builds past that part, and i'm having a different issue noted in #1558


Sharing Status on a company page

Hi,
I'm trying to use the API to post updates to a company page, I can do it for a person/profile but I don't see any API to allow posting to a company page.
Can this be added please? :-)

Any progress on this?

There is this pull request. https://github.com/spring-projects/spring-social-linkedin/pull/62
Also did an it before finding that pull request. This project isn't really useful without it!


Mechanism to open BottomSheet on top of a navigation bar

Hello!
We are using Apple's TabView and are triggering BottomSheet from within the view.
Is there a straightforward way to open BottomSheet above the TabView's navigation bar? As of now, the navigation bar is still present. Perhaps there would be a suggestion to open this modally, i.e. above the tab view?
Thank you!
Example implementation:
@State private var bottomSheetPosition: BottomSheetPosition = .hidden
      VStack {
          Button {
              if self.bottomSheetPosition != .hidden{
                  self.bottomSheetPosition = .hidden
              }else{
                  self.bottomSheetPosition = .middle
              }
          } label: {
              Image(systemName: "ellipsis.circle").font(.system(size: 20))
          }
      }
        .bottomSheet(bottomSheetPosition: self.$bottomSheetPosition, options: [.swipeToDismiss,.noBottomPosition, .showCloseButton(action: {
...
}

Does the following work for you?
        TabView(selection: $selectedView) {
            Text("First View")
            .tabItem {
                Label("First", systemImage: "1.circle")
            }
            .tag(1)

            Text("Second View")
            .tabItem {
                Label("Second", systemImage: "2.circle")
            }
            .tag(2)
        }
        .bottomSheet(...)

yep - that works for me


[Mac] feat: Support screen capture for macOS. (#24)

feat: Add screen capture support for macOS.

chore: rename files.

chore: update.

chore: Implement DesktopSource, DesktopMediaList ojbc interface.

chore: add license.

fix: capture_state.

update.

update.

Hide private delegate declaration.

Re-generate thumbnail for all sources when call UpdateSourceList.

fix: Fix window resize issue.

update.

update.

fix for intel mac.

cleanup.

update.

Add Window/Screen change listener.

Add force reload for MediaListUpdate and capture cursor.

fix: Fix the screen freeze caused by the desktop frame not setting the timestamp.

update.

fix issue for screen sharing.

Fixed H264 profile-level-id limitation that could not encode desktop frames with resolutions higher than 720p.

There are no conflicts and the tests are ok, so I'll merge it.


What's a good way to deal with deeply nested cursors?

I have a tree like this:
const tree = new Baobab({
  lanes: []
}, {
  validate: {
    lanes: [{
      id: 'number',
      name: 'string',
      notes: [
        {
          id: 'number',
          task: 'string',
        }
      ]
    }]
  }
});

It is possible the tree will grow further as I add new concepts to the system (maybe boards etc.).
So far the decorator syntax has worked just fine. I feel it gets a little cumbersome on the lower levels of the hierarchy, though. To point at notes I would have to do something like this:
@branch({
  cursors: function() {
    return {
      note: ['lanes', this.props.laneIndex, 'notes', this.props.noteIndex],
    };
  }
})
...

Maybe the cursor of previous level could be passed instead of root to the child instead? Then I could do:
// XXX: this should point to lanes cursor, not root to work! Needs to be passed as a prop?
@branch({
  cursors: function() {
    return {
      note: ['notes', this.props.noteIndex],
    };
  }
})
...

If I could do this, I wouldn't have to care about the whole hierarchy, just the immediate parent.
How would you deal with deep nesting like this?

This is an interesting use case. I am think of a possibility to do something like this through props or context but need to think this a little bit more because cases when this would not be as simple. What you can do in the meantime is to pass the base of the path as a prop, for instance.
On a side note, you seem to be using<EMAIL_ADDRESS>here whereas this repo deals with<EMAIL_ADDRESS>version. There is a<EMAIL_ADDRESS>available on npm and I will soon release v1.

@Yomguithereal Cool. I'll go with props.

@Yomguithereal I've been thinking this a bit more. It seems to me that in my case the smartest move would be to flatten the structure (notes and lanes and root level) and provide facets.
Related: https://github.com/Yomguithereal/baobab/issues/225.

I was thinking about this lately but I guess I'll see how cerebral fix the problem.

@Yomguithereal Yeah, apparently monkeys solve this neatly. There's a WIP demo.

Cool. Will check this out asap.


Extend motor option to beanie

Add the possibility to install optional packages from motor.
As an example using the the aws option allow to connect with an IAM role.

Any chance to have more reviews or other reviewers @07pepa ? or should I go to discord in order to discuss about this PR ?

@Dudesons  well i asked on discord and you are welcomed on discord
but i requested usual active reviewers already...

@adeelsohailahmed  ok, I just added it in the getting_started but not sure if it's the best place or should I put it in the README or both

@adeelsohailahmed ok, I just added it in the getting_started but not sure if it's the best place or should I put it in the README or both

Thank you for adding the documentation.
getting_started seems right to me. Might not be a bad idea to include a condensed version in the README too. For condensed version, we could mention the supported dependencies, and provide an example or two.

Thank you for adding the documentation.

@adeelsohailahmed ok, I just added it in the getting_started but not sure if it's the best place or should I put it in the README or both

getting_started seems right to me. Might not be a bad idea to include a condensed version in the README too. For condensed version, we could mention the supported dependencies, and provide an example or two.

I added the reference from README to getting started


Fix #6210 - define semantics for pagemodels

Now a pagemodel requires a [PageModel] somewhere in it's hierarchy. We
don't do a guess at whether or not your model class is a PageModel.

Follow up in https://github.com/aspnet/Mvc/pull/6465


1006/1007 - Report fixes

 Download a report on utilisation

 Download a report on all bookings, voids and turnaround times


bug: stream.chunk() don't working

I don't know why this happens, it worked months ago but suddenly it stopped working;
Code:

Logs:

I tested using only stream and stream with ffmpg, I didn't check the download but as it uses the stream methods, I imagine you have the same problem;
The chunks in the while let are all returning 0

Crate 0.7.3 has not yet been updated in crates.io! You can use like that
rusty_ytdl = { git = "https://github.com/Mithronn/rusty_ytdl/", branch = "main"}

Hum... Don't worked

see https://github.com/Mithronn/rusty_ytdl/issues/31#issuecomment-2265577102

This error happened because I had outdated my tokio, when I returned it to version 1.9, it fixed the problem. The stream method works, but the one I need is stream_with_ffmpeg
But why with stream_with_ffmpeg dont work?

I take back what I said, the problem wasn't in my tokio, you're right, I'll be waiting for this problem to be fixed

Yeah as I said. Youtube sometimes giving error and sometimes not


Release: White labeled for Pangea, Developer Console started with login and App List page

Need to add UNITED Wallet links for Google Play

vercel is failing because yarn version is not correct.
I have seen this before with our Communication service, and the fix was to add the yarn executable to the repository. It's a bit of a hack so if we can avoid this it would be great.
https://github.com/Tonomy-Foundation/Tonomy-Communication/tree/development/.yarn/releases
Please check what has changed since last PR that has caused the problem.
One thing to check out is how I did the branches. Instead of creating a PR from testnet to master, I created a release branch from testnet, rebased from master and then made a PR. Perhaps this made some difference as it does not have the correct code... Worth checking...

resolved with

https://github.com/Tonomy-Foundation/Tonomy-App-Websites/commit/fe8237cc3a1cf7f6aa512689d2bd5c78a484cb1f
https://github.com/Tonomy-Foundation/Tonomy-App-Websites/commit/5540cd902ff30efaa595fbb85a16c752e820f5ca


Error npm install in coreUI laravel

I have error
> npm ERR! code 1
> npm ERR! path /var/www/coreuicuba
> npm ERR! command failed
> npm ERR! command sh -c cross-env NODE_ENV=development node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js
>
> npm ERR! A complete log of this run can be found in:
> npm ERR!     /home/laradock/.npm/_logs/2020-12-22T10_50_41_923Z-debug.log
> npm ERR! code 1
> npm ERR! path /var/www/coreui
> npm ERR! command failed
> npm ERR! command sh -c npm run development
>
> npm ERR! A complete log of this run can be found in:
> npm ERR!     /home/laradock/.npm/_logs/2020-12-22T10_50_41_970Z-debug.log

I have done run npm audit fix command and I have an error
> braces  <2.3.1
> Regular Expression Denial of Service - https://npmjs.com/advisories/786
> fix available via `npm audit fix`
> node_modules/sync-glob/node_modules/braces
>   micromatch  0.2.0 - 2.3.11
>   Depends on vulnerable versions of braces
>   node_modules/sync-glob/node_modules/micromatch
>     anymatch  1.2.0 - 1.3.2
>     Depends on vulnerable versions of micromatch
>     node_modules/sync-glob/node_modules/anymatch
>       chokidar  1.3.0 - 1.7.0
>       Depends on vulnerable versions of anymatch
>       node_modules/sync-glob/node_modules/chokidar
>
> yargs-parser  <=13.1.1 || 14.0.0 - 15.0.0 || 16.0.0 - 18.1.1
> Prototype Pollution - https://npmjs.com/advisories/1500
> No fix available
> node_modules/sync-glob/node_modules/yargs-parser
>   yargs  4.0.0-alpha1 - 12.0.5 || 14.1.0 || 15.0.0 - 15.2.0
>   Depends on vulnerable versions of yargs-parser
>   node_modules/sync-glob/node_modules/yargs
>     sync-glob  *
>     Depends on vulnerable versions of yargs
>     node_modules/sync-glob
>
> 7 low severity vulnerabilities
>

I have done follow in https://github.com/JeffreyWay/laravel-mix/issues/2389#issuecomment-646919340 . but still failed.
I don't know how to fix it. there may be a suggestion to improve it.

hope you have solved it by now
what happens if you rm -rf node_modules?
if you look in your package.json, is cross-env there?
Solve it step by step, let's make a good npm run dev first.


catch TemplateNameParser exceptions

AsseticTokenParser attempts to determine the bundle name from the template file name by parsing it with the TemplateNameParser. However, when the file name is already resolved, and is a real file name, containing .., this throws an exception (Template name "%s" contains invalid characters.).
This PR silently catches this exception, since it's not critical.
I my case the exception was happening when extraction translations with JMSTranslationBundle's translation:extract command.

I would prefer only the one line that throws the exception we're catching be wrapped in the try statement.

done


IAmazonRDS is missing method for rds:ModifyDBSnapshotAttribute

Amazon.RDS.IAmazonRDS is missing a method for ModifyDBSnapshotAttribute as per http://docs.aws.amazon.com/AmazonRDS/latest/APIReference/API_ModifyDBSnapshotAttribute.html

Thanks for letting us know. I'll track down why the operation is missing.

We just released version 3.1.3 of the RDS NuGet package that contains this support.
https://www.nuget.org/packages/AWSSDK.RDS/3.1.3


Access to model google/gemma-2-2b-it is restricted.

I'm sure this has been asked before but I can't find clear instructions on what I need to do to get this to auto download the gemma model for Sana use. I tried putting HF_TOKEN=hf_etc... in a .env in my comfy root but that didn't work. ideas? thanks.

Ok, I did this:
huggingface-cli login
Pasted token, said yes to "add token as git credential"
worked after that.

Ok, I did this:
huggingface-cli login
Pasted token, said yes to "add token as git credential"
worked after that.


[NFC] Eliminate use of SymbolTable::getSymbolVisibility()

This is in preparation of upstream MLIR changes.

@ThomasRaoux @nicolasvasilache @MaheshRavishankar please review.

@GMNGeoffrey, any idea when this be merged (intended for the main branch).


Linux Build for PSScriptAnalyzer Module

Since PowerShell is largely cross-platform.   Have you all thought about or seriously considered creating a Linux build for this, as well?  Even though it might not be 100% inclusive, even basic functionality is a win for the community.

@kirkpabk this module is cross-platform...are you hitting any issues or is there something specific you are looking for? Thanks!

Correct. The product can be used on any non-Windows platform where PowerShell 7 is supported on. The build for PowerShell 7 is using the cross platform version of .NET, therefore you can use this module . You will find that we the test the build continuously on Ubuntu and MacOs. The build itself happens on Windows so we can build the Windows PowerShell binaries at the same point and build the product itself on a Linux server does not make any difference to the consumer.


One small thing before the release ...

Sorry, just another small question: I noticed that the title of this dataset (CLDF dataset accompanying Daniel et al.'s "Lingua Franca as Lexical Donors: Evidence from Daghestan" from 2021) deviates from our usual pattern (CLDF dataset derived from Author et al ...). Was this intentional or should this be changed @LinguList and @ilchec?

I can't say much about this, as far as I remember, I simply took this sentence from a different lexibank dataset and changed the title of the paper and the authors. @LinguList?

I can't say much about this, as far as I remember, I simply took this sentence from a different lexibank dataset and changed the title of the paper and the authors. @LinguList?

Yes, intentional, since this is a supplemented CLDF dataset that is
published along with a paper, not a retro-standardized dataset. So I'd
say: the "CLDF Datsets Accompanying" should be our title for the future
for datasts published ALONG with a study.

Yes, intentional, since this is a supplemented CLDF dataset that is
published along with a paper, not a retro-standardized dataset. So I'd
say: the "CLDF Datsets Accompanying" should be our title for the future
for datasts published ALONG with a study.

Thanks for clarifying!

Thanks for clarifying!


Update to latest template

This PR brings in several updates from the Cookbook template, including links to the new Pythia binder.
It should fix the current link-checker error.

strange... I can't reproduce
@sandhujasmine 2.0_Preprocessing.ipynb still works for you?

I believe if you have the latest versions, the crs is nested under rio(xarray): landsat_8_da.rio.crs

I have pinned some packages to avoid this, and a few other issues.

It seems there are more issues with open_rasterio... I'll try come back to this when I have time but please go ahead if anyone else wants to take a swing at it.
It also looks like there are issues towards the end of the spectral clustering notebook