Several Divide by zero exceptions

Hi,
Working against 3.1;
after much digging I have found that if a price hasn't moved within a given period of time the CCI will cause a DivideByZero exception @ line 29 because the mean deviation is 0.
To reproduce:
`[TestMethod]
public void Expect_CCI_Exception()
{
var candles = new List();
for(int i = 0; i <= 100; i++)
{
candles.Add(new Candle()
{
Open = 0.03m,
Close = 0.03m
});
}
Assert.ThrowsException(() => candles.Cci(20));
    }`

Same with the ADX. Still tracing the source of that one...
[TestMethod]         public void Expect_ADX_Exception()         {             var candles = new List<Candle>();             for (int i = 0; i <= 100; i++)             {                 candles.Add(new Candle()                 {                     Open = 0.03m,                     Close = 0.03m                 });             }             Assert.ThrowsException<DivideByZeroException>(() => candles.Adx(20));         }

And DMI
[TestMethod]         public void Expect_DMI_Exception()         {             var candles = new List<Candle>();             for (int i = 0; i <= 100; i++)             {                 candles.Add(new Candle()                 {                     Open = 0.03m,                     Close = 0.03m                 });             }             Assert.ThrowsException<DivideByZeroException>(() => candles.Dmi(20));         }

I dare say there may be others, but I am running out of time as I am about to go away for work.
Could refactor any divisions to something like;
static public decimal SafeDivision(this decimal numerator, decimal denominator)         {             (numerator == 0) ?? return 0;             return (denominator == 0) ? 0 : numerator / denominator;         }

@pahlot thanks for pointing out. I need some time to check all the indicator for the division.

@lppkarl that's understandable. I was thinking about how to best test this along with expending on how we can use the indicators.
I haven't got the time in the next few weeks but what I was thinking is to have a mechanism of discovery of and registration of indicators.
something (very roughly like);
IDiscoverableIndcator {    string Key => $"nameOf(theIndecator)_{ConfigValue}";    string FriendlyName;   int DefaultConfigValue = 20;   int ConfigValue = 20; }
And then a discovery service to list each. This will allow us to have a common discoverable interface for custom implementations, and also register an indicator with a custom config value.
So we could have something like;
indicator.GetByName(name);
indicator.GetInstancesOf();
etc
This would then also allow us to run any new indicators against a common test, such as making sure there is no DivideByZero issue in one foul swoop.
Just an idea.

@pahlot Thanks for the idea, but i just want to make something clear. Do you mean we introduce an IDiscoverableIndicator, and make all Analyzables (the base of all indicators) implement this, and we create a test case to discover all these indicators and test the case? I am a bit confuse.


Branch2

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


Impossible to specify ticks that are different from xaxis data labels

Often I have many more points on the x axis than what I would want to show on the plot. It is currently impossible to specify a set of xticks different from the whole set of x data.

Scatter plots would be nice :-)

@mseri try the XY chart introduced in v1.0.7, it supports

scatter plots
tick number customization

Works perfectly, thanks!


[Bug]

Describe the Bug
I updated to the latest version if DiscordCanary and as of the latest update (0.0.124), it won't start due to a JavaScript error:
To Reproduce
Update to version 0.0.124 and start DiscordCanary with BetterDiscord installed
Expected Behavior
It should have started properly
Screenshots

Discord Version
Canary
Additional Context
I am on Arch Linux, although this should have no affect.

duplicate of #821


♻ improve Fotoware integration error logging (#1686)

This should prevent the Fotoware integration of getting stuck on "Retrieving access token..." when there is an error - making it more clear to both the users and us what happened. See: https://github.com/equinor/energyvision/issues/1686

@padms @millianapia @fernandolucchesi had to close my original PR (#1687) because I messed up the history
Last 4 commits here are new - made the error handling more generic and updated the error handling for the image export function as well.


Fix incorrect bower package name

Existing "name" in bower.json points to wrong repository.  Doing a bower install failed.
Doing a bower search pulls back the following results :
bower search reconnecting-websocket
Search results:
angular-reconnecting-websocket git://github.com/adieu/angular-reconnecting-websocket.git
reconnectingWebsocket git://github.com/joewalnes/reconnecting-websocket.git
reconnecting-websocket git://github.com/firstopinion/reconnecting-websocket.git

Merged. Thanks

Thanks Joe :-)
Best regards,
Craig
On 5 February 2015 at 14:39, Joe Walnes<EMAIL_ADDRESS>wrote:

Merged. Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/joewalnes/reconnecting-websocket/pull/39#issuecomment-73056023
.


fix pilot crash issue if there is zero listeners and err is nil

if len(listeners) == 0 and err is nil,pilot will be crash because of err.Error()
This will happen when servers is nil in gateway as below:

apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
creationTimestamp: 2018-08-02T07:39:05Z
name: istio-gateway
namespace: istio-system
spec:
selector:
istio: ingressgateway
servers: []

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: zackbutcher
If they are not already assigned, you can assign the PR to them by writing /assign @zackbutcher in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/pkg/networking/OWNERS

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

Hi @x15zhang. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ZackButcher
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/pkg/networking/OWNERS [ZackButcher]

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

/ok-to-test

@x15zhang: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
a54c7aa63d426173637dcd468d06d635ed3f3831
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@x15zhang: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
a54c7aa63d426173637dcd468d06d635ed3f3831
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[chore]: add solution for challenge 23

Read and understand challenge 23 and find a solution to be used in answering the question

Can I write the solution in javascript?

pure javascript ...

Okay

You can assign me this then ...


Deploying ASP.NET Core from git with kudu fails because of locked pdb file

Moved from https://github.com/dotnet/core/issues/1721 - opened by @ellern. PTAL @pakrym. cc @muratg

I got a couple of web apps running .NET Core 2.1.0 on different Azure service plans running on Windows.
Lately I've started getting a file locked error when deploying from git on Azure. Almost every single time I push, it errors out telling me the file (Site.Views.pdb) is in use. Restart app, redeploy works every time, but it's annoying. I found out if I push two changes within a short timespan then there is no lock on the pdb file and the deployment works as expected.
Remote debugging is disabled for the web apps, so I have no idea why there is a lock on the pdb file.
I can't remember having any issues before I updated my web apps to .NET Core 2.0.0
KuduSync.NET from: 'D:\local\Temp\8d5d80c09bb9d1c' to: 'D:\home\site\wwwroot'
Error: Failed to change file that is currently being used "D:\home\site\wwwroot\Site.Web.Views.pdb"
Copying file: 'Site.Web.deps.json'
Copying file: 'Site.Web.runtimeconfig.json'
Copying file: 'Site.Web.Views.dll'
Copying file: 'Site.Web.Views.pdb'
Failed exitCode=1, command="kudusync" -v 50 -f "D:\local\Temp\8d5d80c09bb9d1c" -t "D:\home\site\wwwroot" -n "D:\home\site\deployments\c764c8a77ef871df3f13953049f5bc51088cac1f\manifest" -p "D:\home\site\deployments\bd8a3c2f17acba1e45b1c14276d9530f24e5a649\manifest" -i ".git;.hg;.deployment;deploy.cmd"
An error has occurred during web site deployment.
Error: Failed to change file that is currently being used "D:\home\site\wwwroot\Site.Web.Views.pdb"\r\nD:\Program Files (x86)\SiteExtensions\Kudu\74.10611.3437\bin\Scripts\starter.cmd "D:\home\site\deployments\tools\deploy.cmd"

This issue is copied from projectkudu/kudu#2788

The same issue.
Here is the deployment log:
[
{
    "log_time": "2018-06-26T22:26:33.317431Z",
    "id": "",
    "message": "Command: \"D:\\home\\site\\deployments\\tools\\deploy.cmd\"",
    "type": 0,
    "details_url": null
},
{
    "log_time": "2018-06-26T22:26:34.3799913Z",
    "id": "",
    "message": "Handling Basic Web Site deployment.",
    "type": 0,
    "details_url": null
},
{
    "log_time": "2018-06-26T22:26:35.7237958Z",
    "id": "",
    "message": "Cannot removed the 'to be deleted' directory, ignoring",
    "type": 0,
    "details_url": null
},
{
    "log_time": "2018-06-26T22:26:35.7237958Z",
    "id": "",
    "message": "Error: Failed to change file that is currently being used \"D:\\home\\site\\wwwroot\\CurlToCSharp.pdb\"",
    "type": 1,
    "details_url": null
},
{
    "log_time": "2018-06-26T22:26:35.7237958Z",
    "id": "",
    "message": "KuduSync.NET from: 'D:\\local\\Temp\\zipdeploy\\extracted' to: 'D:\\home\\site\\wwwroot'",
    "type": 0,
    "details_url": null
},
{
    "log_time": "2018-06-26T22:26:35.7394195Z",
    "id": "",
    "message": "Copying file: 'appsettings.Development.json'",
    "type": 0,
    "details_url": null
},
{
    "log_time": "2018-06-26T22:26:35.7550254Z",
    "id": "",
    "message": "Copying file: 'appsettings.json'",
    "type": 0,
    "details_url": null
},
{
    "log_time": "2018-06-26T22:26:35.7550254Z",
    "id": "",
    "message": "Copying file: 'CurlToCSharp.deps.json'",
    "type": 0,
    "details_url": null
},
{
    "log_time": "2018-06-26T22:26:35.770675Z",
    "id": "",
    "message": "Copying file: 'CurlToCSharp.dll'",
    "type": 0,
    "details_url": null
},
{
    "log_time": "2018-06-26T22:26:35.7862697Z",
    "id": "",
    "message": "Copying file: 'CurlToCSharp.pdb'",
    "type": 0,
    "details_url": null
},
{
    "log_time": "2018-06-26T22:26:35.7862697Z",
    "id": "",
    "message": "Failed exitCode=1, command=\"kudusync\" -v 50  -f \"D:\\local\\Temp\\zipdeploy\\extracted\" -t \"D:\\home\\site\\wwwroot\" -n \"D:\\home\\site\\deployments\\010d093fca7b44479e9d0eac2b3c723d\\manifest\" -p \"D:\\home\\site\\deployments\\ba4b36117f9942d1b1813f5b7dc9c32b\\manifest\" -i \".git;.hg;.deployment;deploy.cmd\"",
    "type": 0,
    "details_url": null
},
{
    "log_time": "2018-06-26T22:26:35.8018911Z",
    "id": "",
    "message": "An error has occurred during web site deployment.",
    "type": 0,
    "details_url": null
},
{
    "log_time": "2018-06-26T22:26:35.8175451Z",
    "id": "",
    "message": "Error: Failed to change file that is currently being used \"D:\\home\\site\\wwwroot\\CurlToCSharp.pdb\"\\r\\nD:\\Program Files (x86)\\SiteExtensions\\Kudu\\74.10620.3448\\bin\\Scripts\\starter.cmd \"D:\\home\\site\\deployments\\tools\\deploy.cmd\"",
    "type": 2,
    "details_url": null
}]

cc @Petermarcu @rynowak @richlander

Forgot to mention, I'm using this method for the deployments.
https://www.appveyor.com/docs/deployment/azure-app-service-zip-push-deploy/
Here is the repo with appveyor.yml
https://github.com/olsh/curl-to-csharp

I am hitting the same issue with Git deploy. Currently tracked here https://github.com/projectkudu/kudu/issues/2788. The App Service is currently in the broken state, so if any of you want to try to do some debugging on it, feel free. If you need credentials or something to access the App Service, just ping me (hajek.j@hotmail.com).

All - having same issues with Functions running on .NET Core 2.1.302.  Any updates on what is locking the files?
As a workaround, I've changed the release scripts to stop shipping PDB's, but this doesn't help dev deploys.

Any news on this? Is it best to revert to good ol' WebDeploy?

+1, trying out Azure Function App deployment from Visual Studio Code and getting the same error:
Error: Failed to change file that is currently being used "D:\home\site\wwwroot\bin\azure-functions.dll"

Adding @SergeyKanzhelev here. Sergey, any thoughts on this?

I believe we had an email thread about this. The .NET runtime will load/lock the PDB when an exception is thrown (and the call stack evaluated).

Anybody, who has a repro - follow up on Application Insights suspicion. Is Application Insights extension is installed on web app or Application Insights SDK in the deployed app?

I had this problem today after updating my app to .NETCore 2.1.4 (from 2.0.3)
Removing Application Insights from my App Service and restarting the service fixed the problem.
Note that I did not try to restart it before removing Application Insights (my AI setup was broken anyways), so a simple restart may have fixed the problem without removing Application Insights, but I can't confirm this.

@SergeyKanzhelev Yes, App Insights SDK in my case

@severisv so no App Insights extension there?

Than it seems like an issue with exception stack traces calculations. which @rynowak mentioned

That's right, no AppInsights extension there :)

We're experiencing the same problem here... ASP.NET Core 2.1. We're pushing to the site using git. No custom deploy.cmd script, but it seems like the standard deploy.cmd is putting an app_offline.htm file there, however, something is still locking the PDB:
2018-11-01T12:29:51.8546477Z,Updating branch 'master'.,cfb14059-486a-480e-b68b-a056e35b5131,0
2018-11-01T12:29:55.5734310Z,Updating submodules.,5f3b6c05-f55e-4210-ac90-4ff2002484fd,0
2018-11-01T12:29:55.6359017Z,Preparing deployment for commit id '043f87ea5f'.,169a2a4e-11a6-4780-852a-ffddb566b1ca,0
2018-11-01T12:29:55.9952781Z,Generating deployment script.,aaf73c8a-b872-41f5-8460-f8c4d94a1bf8,0
	2018-11-01T12:29:56.0421521Z,Using cached version of deployment script (command: 'azure -y --no-dot-deployment -r "D:\home\site\repository" -o "D:\home\site\deployments\tools" --basic --sitePath "D:\home\site\repository"').,,0
2018-11-01T12:29:56.1046810Z,Running deployment command...,b0c63d75-e1a1-41ac-9523-85564d367572,0
	2018-11-01T12:29:56.1202853Z,Command: "D:\home\site\deployments\tools\deploy.cmd",,0
	2018-11-01T12:29:57.0109319Z,Handling Basic Web Site deployment.,,0
	2018-11-01T12:30:02.2609728Z,Creating app_offline.htm,,0
	2018-11-01T12:30:02.2609728Z,Error: Failed to change file that is currently being used "D:\home\site\wwwroot\MyApp.pdb",,1
	2018-11-01T12:30:02.2765211Z,Cannot removed the 'to be deleted' directory, ignoring,,0
	2018-11-01T12:30:02.2765211Z,KuduSync.NET from: 'D:\home\site\repository' to: 'D:\home\site\wwwroot',,0
	2018-11-01T12:30:02.2921499Z,Copying file: 'MyApp.dll',,0
	2018-11-01T12:30:02.3077806Z,Copying file: 'MyApp.pdb',,0
	2018-11-01T12:30:02.3258034Z,Failed exitCode=1, command="kudusync" -v 50  -f "D:\home\site\repository" -t "D:\home\site\wwwroot" -n "D:\home\site\deployments\043f87ea5f9afcbc4b380bb93e718cdaa172ced5\manifest" -p "D:\home\site\deployments\db5371a2c2b14fbe552e90023a26032769dec7d0\manifest" -i ".git;.hg;.deployment;deploy.cmd",,0
	2018-11-01T12:30:02.3258034Z,An error has occurred during web site deployment.,,0
	2018-11-01T12:30:02.3546532Z,Error: Failed to change file that is currently being used "D:\home\site\wwwroot\MyApp.pdb"\r\nD:\Program Files (x86)\SiteExtensions\Kudu\78.11022.3613\bin\Scripts\starter.cmd "D:\home\site\deployments\tools\deploy.cmd",,2

It happens occasionally on some of our sites, but not all. Is there anything I can do to help troubleshoot the issue? Is it a Kudu issue or a .net core issue or an Azure WebSite issue?

@erikbra do you have App Insights extension or SDK installed on your site? We've seen people hitting this issue with that (but not everyone.)
In your case (deploying with kudu/git), you're building on the server which is an Azure App Service VM so it may possibly be related to that. (Perhaps runtime holding onto the PDBs to keep the stack trace.)

We don't have application insights installed on those instances. We're running into so much issues with that, so we skipped it.
Actually, we're only deploying binaries with git, not the source code itself. It is first built on the build server, and the pushed to the Azure site with git. That part is OK. It's the deployment with Kudu, from the deployment folder to the wwwroot that is the problem.

@erikbra Thanks for the information! Does your app have Razor views? (is there any runtime compilation?)

No, they are REST APIs.

Thanks!
@pakrym, @davidfowl any thoughts on @erikbra's case?  I'm a bit out of ideas at this point.

@erikbra I'm not terribly familiar with the deploy.cmd script used, but taking a look at the timestamps in your logs, it just looks like you haven't given the application sufficient time to shutdown and ergo the files are locked.
The default shutdown timelimit is 10 seconds, so I would recommend introducing a retry loop in your deploy script for at least as long as the shutdown time limit

I haven't created a deploy.cmd myself. This is just what comes out-of-the-box if you don't create one yourself, @shirhatti. It's the standard deploy.cmd for kudu.

This started happening to me as well a few days ago, any ideas how to work around it? I am using ApplicationInsights and would much like to continue so.
It is very inconsistent, some deploys work, others don't

I'm also seeing this issue fairly regularly and would love any recommendations.  I suspected it was related to using a BackgroundTaskQueue pattern and there being a backlog of operations to complete.. but even as I've removed those I'm still encountering the issue with the .pdb being locked.

We also ran into this problem.
.NET Core 2.2.105, remote debugging disabled, no custom deploy.cmd -> failure rate 95%+ on a simple service that isn't even used yet.
And please let me express my amazement at the state of this issue. 9 months+ old, no one assigned, no milestone and only workaround is to stop the service before deployment, start it afterwards and hope it manages to finish in time. That's something I remember doing cca 6 years ago on Azure Web Roles back when they were a thing.

Is anyone here using app insights?

Yes

30 apr. 2019 kl. 19:57 skrev David Fowler<EMAIL_ADDRESS>Is anyone here using app insights?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm having the same issue in Azure DevOps release pipeline while deploying a Function App.
2019-05-17T05:52:32.9082281Z Error: Failed to change file that is currently being used "D:\home\site\wwwroot\bin\MyFunctionApp.pdb"\r\nD:\Program Files (x86)\SiteExtensions\Kudu\81.10329.3844\bin\Scripts\starter.cmd "D:\home\site\deployments\tools\deploy.cmd" 2019-05-17T05:52:32.9136091Z ##[error]Failed to deploy web package to App Service. 2019-05-17T05:52:32.9145167Z ##[error]Error: Package deployment using ZIP Deploy failed. Refer logs for more details.
The previous task is stopping of my Function App but apparently it doesn't help here.

We have the same issue deploy function apps

The Kudu deployment logs also have a strange situation here:
{
    ....
    "last_success_end_time": null,
    "complete": true,
    ....
}

Not exactly sure what "last_success_end_time" means, bit of an odd name, but seems logically to assume that "complete" should not be true if "last_success_end_time" is null.

I'm hitting this problem regularly, self-hosting on IIS and using kudu to deploy (git push).  My "workaround" is to log onto the IIS server, find the relevant Application Pool and recycle it just before trying to deploy again.  Most of the time this works.
This is only a problem with .net core (2.2) - it doesn't happen at all with full framework applications on the same server.
The failure is always trying to deploy one of my own .pdb files: Error: Failed to change file that is currently being used
I guess the relevant question is: why is something locking the .pdb file?  If it needs to open the .pdb can't it get what it needs then close it again?!  Or better still open in a way that doesn't lock the existing file on disk?

@steveglen you're seeing PDB locking on your local IIS? Can you reproduce that behavior on demand?

@davidfowl not 100% of the time, more like 50% chance of hitting it on any given deploy attempt.  When it's hit deployments consistently fail until I recycle the IIS app pool.
I'm running on Windows Server 2016, Kudu from git but I last updated 31st May 2018 so it's not up to date.  I've seen no suggestion this issue is improved in newer Kudu - please let me know if this is something that is expected to improve with newer code.

OK I think I've identified the PDB locking issue. Stay tuned...

We've identified the issue and are looking at backporting the fix to older versions of .NET Core (possibly 2.1 and 2.2). More information here https://github.com/dotnet/corefx/issues/38212

@davidfowl We're running into this issue when deploying Azure Functions apps as they use .NET Core 2.x. How can I track the progress/status of this fix being backported to 2.1/2.2?

@MGudgin I opened a PR for the backport here https://github.com/dotnet/corefx/pull/38291

@davidfowl is there ASP.NET Core work here or is the change localized to corefx? I'd like to get this off our dispatch queue :).

@anurse It's all on corefx and it's in the pipeline for consideration for the 2.2 patch.

Cool, closing this now then. For the others on this thread, you can track the backport in the issue linked above (https://github.com/dotnet/corefx/pull/38291) and if you're still having a problem after that patch is deployed, feel free to file something new!


Implement functional: admin home page - 'Add Sessions' button

Please do all work for this task in a branch named issue-23.
Implement functional aspects of the admin home page.
Specifically:

'Add Sessions' button

Issue resolved within issue-22 branch.


Updating configuration files on startup

This is a best practice question.
I have a Dockerfile that uses dunglas/frankenphp:sha-7454826-php8.2.15-alpine as the base image, installs all of the PHP extensions I need, and installs the email server MSMTP.
I have to add a configuration file for MSMTP (/etc/msmtp) that has a few details like host / user / password. These values are being passed to my service in ENV vars, as they are dynamic depending on the service that is using this container.
It seems I can't use ENV vars in my /etc/msmtp config file like '{$MSMTP_RELAY_SERVER_HOST}` - PLEASE CORRECT ME IF I AM WRONG - this is the ideal solution.
So now I am including a docker-configure file, using it as the ENTRYPOINT, then calling the parent containers ENTRYPOINT at the end via exec /usr/local/bin/docker-php-entrypoint "@". I am also including the CMD below my ENTRYPOINT in my Dockerfile like CMD [ "--config", "/etc/caddy/Caddyfile", "--adapter", "caddyfile"].
Is this the best way to handle injecting ENV vars into config files? Should I be able to drop ENV vars in the config file directly, like {$FRANKENPHP_CONFIG} for example in the base Caddfile?
Here is my Dockerfile for reference. Any recommendations to improve are welcome!

Ultimately, it is up to you as to how the environment gets configured. You can use environment variables, mount a configuration file, or whatever makes the most sense for your application.


boards/arm/nucleo_l476rg: Fix usart1 node

Fix usart1 node in boards dts file
Signed-off-by: Yannis Damigos<EMAIL_ADDRESS>

Codecov Report

Merging #6348 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6348   +/-   ##
=======================================
  Coverage   53.08%   53.08%
=======================================
  Files         430      430
  Lines       41040    41040
  Branches     7908     7908
=======================================
  Hits        21785    21785
  Misses      16032    16032
  Partials     3223     3223

Continue to review full report at Codecov.

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


I cant post comment on kata...

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected Behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Contexts
Please complete the following information:

Codewars username (if account related): [e.g., jhoffner]
OS: [e.g. macOS]
Browser: [e.g. chrome, safari]
Browser Version: [e.g. 22]

Have you confirmed your email address? Someone had similar issue https://github.com/Codewars/codewars.com/issues/1814#issuecomment-490755232

Thanks solved..;)


Does TGI inference save data remotely without telling the user

We are planning to use TGI inference for our llama2 models .
All the required model and data are hosted locally and no data data can be transmitted anywhere else.
If anyone can help to understand if TGI only keeps the data in local environment or can collect some data for remote reportries will be of great help.

We do not do any type or sort of telemetry. TGI can run in an air gapped environment as long as you download the model ahead of time.

Thanks a lot  for the response.


[RFC][NI] Change locale translate query

Why?
We all know we've had problems using this with language code google does not recognize (things like pl-pl and nl-nl). I was tired of this.
What this does
This changes the util that generates the google translate formula so we are compliant with google translate api specifications.
How was this working before?
It happens that google translate api is very permissive and accepts some common locale identifiers that are not according to the standard
Context
Google translate API uses ISO693-1, this means it uses language identifiers and does not
distinguist language variants. pt-br works but is the same as pt-pt, same happens for english, as this standard only cares about the mother language (except a few cases that are handled).
More details: https://cloud.google.com/translate/docs/languages


Investigate more efficient repository layouts

We currently store all objects as loose objects on the remote. This means that there's no delta compression.
I haven't thought of a storage format and purely client-side protocol that allows for packing / delta compression similar to git's on-disk format. It would be super cool if it is possible to do this, because that'll mean that repositories will consume a lot less space on Dropbox.

Closing due to age.


[GL] C3 should be fulfilled if one combination exists instead of all of them

[ ] One combination is sufficient to fulfill the guideline
[ ] pushed to develop

This also means that there are no longer uncovered transitions, but uncovered input states and uncovered output states.


Improve symbol repr

This improves the repr for symbol, making it valid Typst code to produce the exact same (optionally modified) symbol.
Before this PR, the repr was simply that of a string containing the character that would be inserted if using the symbol as is.

Symbol
Previous repr
New repr

sym.amp
"&"
symbol("&", ("inv", "⅋"))

sym.amp.inv
"⅋"
symbol("&", ("inv", "⅋")).inv

sym.smash
"⨳"
symbol("⨳")

This may be a really dumb question, but why not make it become sym.amp, sym.amp.inv, etc.?

Make sense, thanks!

As I understand, repr will be pretty big for symbols, which have multiple variants? Like arrow.r.double.long.bar. Also it will probably include other variants like arrow.r.bar?

I had the same concern reading the description. If we want it to be valid Typst code, maybe symbol("the symbol") is enough, or just symbol("parent symbol", ("variant", "variant symbol")).variant?
To be honest, maybe it should be just like the old repr, but with symbol( ) around it to avoid confusion with strings.

The proposed repr can indeed become quite large for symbols with many variants. But this is similar to how element functions, arrays, or dicts reprs can quickly grow in size when they have many fields. I think displaying all available variants can be beneficial because it provides a simple way to know the list of available variants, the same way most people expect repr for dict to display all available fields.

I had the same concern reading the description.

Same here. It feels a bit different from a dict, since a dict really is just the data in it, while a symbol from a user PoV is just the single letter, and it happens to also support field access.  What's also a bit odd, is that after you've entered a few variants, it would still display all the variants, even those that are unreachable.
At the same time, I can see the argument that it can be useful for debugging and understanding symbols and that's the main point of repr.

Maybe in th case where some modifiers are applied, the unreachable variants could be removed from the repr. But appart from that, I mostly expect repr to show me everything there is to know about a value.

Maybe add a parameter to repr like full: true or similar?

Maybe add a parameter to repr like full: true or similar?

I think that would be unnecessary complication.

Maybe in th case where some modifiers are applied, the unreachable variants could be removed from the repr.

That could be a good middle ground.

Maybe in th case where some modifiers are applied, the unreachable variants could be removed from the repr.

I was thinking that we could do something like this then:
Given
#let envelope = symbol(
  "🖂",
  ("stamped", "🖃"),
  ("stamped.pen", "🖆"),
  ("lightning", "🖄"),
  ("fly", "🖅"),
)

We would have

Symbol
Previous repr
New repr

envelope
"🖂"
symbol("🖂", ("stamped", "🖃"), ("stamped.pen", "🖆"), ("lightning", "🖄"), ("fly", "🖅"))

envelope.stamped
"🖃"
symbol("🖃", ("pen", "🖆"))

envelope.stamped.pen
"🖆"
symbol("🖆")

envelope.lightning
"🖄"
symbol("🖄")

envelope.fly
"🖅"
symbol("🖅")

We would perhaps lose some information, but I'm ok with that as it seems less confusing to me, and where the symbol came from doesn't really matter anyways.
Also, it's worth noting that this will enable listing the variants of a symbol. Currently, this doesn't seem to be possible, so we should probably have a follow-up PR with a method to return the symbol's variants as an array of arrays, exactly like you'd specify them to the symbol constructor. Otherwise, the only way to obtain this information will be through repr parsing.

Sounds good

Thanks!


serial and parallel test support

The idea is to have the option to run the tests in serially to force those tests not to run parallely.
The idea is from ava framework(https://openbase.io/js/ava-tf/documentation)
By adding .serial after the tests you can enforce it
import vest, { test } from 'vest';
import enforce from 'vest/enforceExtended';

export default vest.create('user_form', (data = {}, currentField) => {
  vest.only(currentField);

  test.serial('username', 'Username is required', () => {
    enforce(data.username).isNotEmpty();
  });

  test.serial('username', 'Username is too short', () => {
    enforce(data.username).longerThanOrEquals(3);
  });

  test.serial('password', 'Password is required', () => {
    enforce(data.password).isNotEmpty();
  });

  test.serial('password', 'Password already used', async () => {
    const response = await axios.post('http://localhost:3000/auth')
    enforce(response.code).matches(/200/);
  });

  if (data.password) {
    test.serial('confirm_password', 'Passwords do not match', () => {
      enforce(data.confirm_password).equals(data.password);
    });
  }

  test.serial('email', 'Email Address is not valid', () => {
    enforce(data.email).isEmail();
  });

  test.serial('tos', () => {
    enforce(data.tos).isTruthy();
  });
});

What do you think?
Thanks,
Ziv

Hey Ziv, sorry for taking a couple of days to reply.
That's an interesting suggestion. You suggest to have al tests async by default, which could help us with long-operations. I am wondering about the output interface though.
At the moment vest.create returns a function that when run, it returns a result object
const validate = vest.create('name', () => {/*...*/}) // [Function Validate]
const res = validate(data) // [Object object]
res.hasErrors()
res.getErrors()
/*...*/

Changing the default to async tests means that the result object will be pretty useless, because it will never have the validation results, unless explicitly .serialed, and in most of the cases, tests are sync.
I am wondering if it would be useful to introduce a parallel mode that would run tests as async by default, unless requested otherwise:
const v = vest.create()
v.async() // run tests as async

Could be interesting to look into.


Whereis config.toml located?

I install with
tar --no-overwrite-dir -C / -xzf cri-containerd-cni-1.3.4.linux-amd64.tar.gz
but /etc/containerd/config.toml does not exist, whereis is it located?

hey @hillbun , I am afraid that tarfile doesn't contain config file. For default config, you can generate default config by containerd config default. Hope it can help!

It won't come by default if you download the binary, I usually run this "/usr/bin/containerd config default > /etc/containerd/config.toml" pointing it to wherever the container binary is . It will produce a default config.toml @ /etc/containerd/config.toml


provisioning woolf

[x] fix up hostname + mailname

[ ] Config rng-tools

[ ] Set MySQL

[x] defaults for UTF8/16
[ ] to slave necessary bits from sql.no2id.net

[ ] mail
[ ] wordpress
[ ] erm…

[x] Further lock down SSH (key only, even with ssh firewall rules)

[ ] Config bind to slave from ns0.no2id.net, using AXFR

[ ] Config SpamAssassin, ClamAV, spamc/pd (updated quite a bit)

[ ] Config Exim (replicate goddard set-up)

[x] Config HAProxy (replicate goddard set-up)

[ ] Config Varnish (replicate goddard set-up)

[ ] Config Nagios

[ ] Config Mailman (copy from goddard)

[ ] Config nginx for

[ ] Mailman http://wiki.nginx.org/Mailman
[ ] Wordpress https://codex.wordpress.org/Nginx
[ ] A git viewer? http://gitlist.org/
[ ] Nagios http://johan.cc/2012/02/06/nagios-nginx/

[ ] ?? post-commit hooks to mail a commit list ??

[ ] Reboot and check things

[ ] set MOTD, banner etc.

[ ] Provision users + ssh keys

[ ] rsync data from goddard

[ ] Config mailman

WIP


Conformance based on feature groups

What would you like to be added:
At the beginning, this API was focused on HTTP with plans to include other protocols.
Then L4 protocols were added in.
Overtime, we have added support for Service Mesh like use cases.
Add "conformance profile" to the API so that an implementation can support only a subset of the API. For example, an HTTP gateway, L4 gateway or a mesh gateway.
Why this is needed:
A single implementation may wish to focus on specific feature group and not support the entire API. This is not possible today.

Some not-quite-ready-for-a-GEP thoughts from me:
Roughly, it seems like we've got three main classes of thing wanting to use this API right now:

Ingress controllers (layer 7 load balancers/application gateways that route mainly HTTP traffic)
Load Balancers (layer 4 load balancers that route TCP and UDP, maybe also handling some TLS)
Mesh controllers (that provide a gateway between the inside and outside of a service mesh)

Does that sound about right to everyone?
If that is the case, we need to explain what a conformance profile would actually do. In my mind, there are a few advantages to having conformance profiles:

Creates easy-to-understand use cases for users to know if they can use the API
Routes should be able to move between Gateways of the same conformance profile with minimal changes
Defining what a conformance profile means allows us to define conformance tests that implementations can use to certify that they meet the contract.
The list is not exhaustive, we can always add more profiles later if other use cases arrive.

So, what should be in a conformance profile? I think we can actually do this in a reasonably straightforward way to begin with, and define a conformance profile as the support of certain objects in the API, something like this (YAML used because my brain is now warped into thinking in it):
conformance:
  - ingress-controller:
      core:
      - GatewayClass
      - Gateway
      - ReferencePolicy
      - HTTPRoute
      extended:
      - TLSRoute
  - l4-lb:
      core:
      - GatewayClass
      - Gateway
      - ReferencePolicy
      - TCPRoute
      - UDPRoute
      extended:
      - TLSRoute
  - mesh:
      core:
      - GatewayClass
      - Gateway
      - ReferencePolicy
      - HTTPRoute?

The point here is that the profiles are additive; an implementation may support one or more, so can have an implementation that provides layer 4 and ingress-controller functions.
Obviously, there will need to be things defined for what support of a particular object means, like we've already done for HTTP, but I think this goes a long way to getting us started.

The names of the categories might want to change (like make it L7, L4, mesh); but the general direction +1 from me.

I believe this was fixed by @youngnick's work on GEP #917. Closing this one out, feel free to reopen if I missed anything.
/close


Adds LICENSE to built tarball

adds setup.cfg with license_files

Hi Stewart. Thanks for opening this pull request. Let me have a look. :)
I like the simplicity of this solution. :+1:
I think that it's possible to simplify it even more and add the following to setup.py:
license_files=["LICENSE"]

This way, we avoid another file in the root of the repo. See https://stackoverflow.com/a/66443941/554283 for reference.

Thanks Frederick.
That seems to do the job as well.

Looks good to me. :) Thank you for your contribution to asyncio-mqtt. :+1:


SFLOW-1: loopback sub-interface index and handling of source-address when configuring sflow collector

Parametrized the loopback sub-interface index; in Juniper's case, loopback sub-interface 0 is typically already configured/present on the device, so reusing that number. For other vendors, left the index as 1 (no change from before).

Removed the defer func where a part of the config used in the test was cleaned up (the NetworkInstance config block). Cleaning up just a part of the config ends up leaving unresolved dependencies behind, causing errors. Also, cleanup of config is typically not done by the tests anyway.

Added support in sflow.go func NewSFlowCollector() to take v4 and v6 source addresses as arguments, to be used when creating the collector instead of picking the address from the loopback interface For vendors that don't support update of source address, picking the source address from the loopback interface is used (no change from before).

Pull Request Functional Test Report for #3445 / 3550ed1e0a22e4cdff542c875404a076a8ccc3d9
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

SFLOW-1: sFlow Configuration and Sampling

Cisco 8000E

SFLOW-1: sFlow Configuration and Sampling

Cisco XRd

SFLOW-1: sFlow Configuration and Sampling

Juniper ncPTX

SFLOW-1: sFlow Configuration and Sampling

Nokia SR Linux

SFLOW-1: sFlow Configuration and Sampling

Openconfig Lemming

SFLOW-1: sFlow Configuration and Sampling

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

SFLOW-1: sFlow Configuration and Sampling

Cisco 8808

SFLOW-1: sFlow Configuration and Sampling

Juniper PTX10008

SFLOW-1: sFlow Configuration and Sampling

Nokia 7250 IXR-10e

SFLOW-1: sFlow Configuration and Sampling

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
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 55.314%

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
1983

Relevant Lines:
3585

💛  - Coveralls

/fptest virtual


[Jenkins-source] description field missing in Jenkins Builds table

Issue
For the jenkins-source connector
The builds table is currently having the below columns:

"result"
"duration"
"number"
"displayname"
"fulldisplayname"
"id"
"_class"
"actions"
"url"
building"
"timestamp"
"_airbyte_ab_id"
"_airbyte_emitted_at"
"_airbyte_normalized_at"
"_airbyte_builds_hashid"

description field is missing, which has the information like parameters used to execute the build job.
This is one of the important fields needed.
The description field is available in the API response.
https://your.jenkins.url/job/$JOB_NAME/$BUILD_NUMBER/api/json?pretty=true
This can be be added to Jenkins connector by modifying the below files to include description.

sources/jenkins-source/resources/schemas/builds.json
sources/jenkins-source/src/jenkins.ts

PR submitted for the issue
https://github.com/faros-ai/airbyte-connectors/pull/1010

PR merged
https://github.com/faros-ai/airbyte-connectors/pull/1010


fix(godeltaprof): fix mutex tests on gotip

Fixes https://github.com/grafana/pyroscope-go/issues/103
This PR fixes an issue when a heap/mutex profile has a record with equal stack.
The issue was found in tests for mutex(golang started to produce two profile records for the same mutex ( see the linked issue for more info))
I'm not sure if it is possible to trigger the issue for heap.
The PR turned up big because of refactored test helpers for reuse. I can split PR into 2 or 3 smaller ones if needed.
This PR also drops support for go16, go17 as I wanted to use generics in prof/map.go
TODO(korniltsev): add benchmark results in the PR description

yey, green light \M/

goos: darwin
goarch: arm64
pkg: github.com/grafana/pyroscope-go/godeltaprof/compat
                                      │   old.txt   │              new.txt               │
                                      │   sec/op    │   sec/op     vs base               │
HeapCompression-8                       715.6µ ± 1%   570.0µ ± 1%  -20.35% (p=0.002 n=6)
MutexCompression/ScalerMutexProfile-8   676.0µ ± 0%   544.5µ ± 1%  -19.45% (p=0.002 n=6)
MutexCompression/ScalerBlockProfile-8   681.8µ ± 1%   543.8µ ± 1%  -20.25% (p=0.002 n=6)
HeapDelta-8                             78.44µ ± 1%   97.20µ ± 2%  +23.92% (p=0.002 n=6)
MutexDelta/ScalerMutexProfile-8         54.43µ ± 1%   80.91µ ± 1%  +48.65% (p=0.002 n=6)
MutexDelta/ScalerBlockProfile-8         54.02µ ± 1%   81.12µ ± 2%  +50.16% (p=0.002 n=6)
geomean                                 205.8µ        218.1µ        +5.96%

                                      │   old.txt    │               new.txt               │
                                      │     B/op     │     B/op      vs base               │
HeapCompression-8                       965.7Ki ± 0%   969.5Ki ± 0%   +0.39% (p=0.002 n=6)
MutexCompression/ScalerMutexProfile-8   965.7Ki ± 0%   969.1Ki ± 0%   +0.35% (p=0.002 n=6)
MutexCompression/ScalerBlockProfile-8   965.6Ki ± 0%   969.1Ki ± 0%   +0.36% (p=0.002 n=6)
HeapDelta-8                              333.00 ± 0%     50.00 ± 2%  -84.98% (p=0.002 n=6)
MutexDelta/ScalerMutexProfile-8          313.00 ± 0%     30.00 ± 0%  -90.42% (p=0.002 n=6)
MutexDelta/ScalerBlockProfile-8          313.00 ± 0%     30.00 ± 0%  -90.42% (p=0.002 n=6)
geomean                                 17.36Ki        5.802Ki       -66.57%

                                      │  old.txt   │              new.txt              │
                                      │ allocs/op  │ allocs/op   vs base               │
HeapCompression-8                       788.0 ± 0%   782.0 ± 0%   -0.76% (p=0.002 n=6)
MutexCompression/ScalerMutexProfile-8   788.0 ± 0%   782.0 ± 0%   -0.76% (p=0.002 n=6)
MutexCompression/ScalerBlockProfile-8   788.0 ± 0%   782.0 ± 0%   -0.76% (p=0.002 n=6)
HeapDelta-8                             2.000 ± 0%   1.000 ± 0%  -50.00% (p=0.002 n=6)
MutexDelta/ScalerMutexProfile-8         2.000 ± 0%   1.000 ± 0%  -50.00% (p=0.002 n=6)
MutexDelta/ScalerBlockProfile-8         2.000 ± 0%   1.000 ± 0%  -50.00% (p=0.002 n=6)
geomean                                 39.70        27.96       -29.56%

The delta bench became slower by 20-50%
What I don't understand is why compression test became faster 20% :shrug:

What I don't understand is why compression test became faster 20% 🤷

Ok, I understand now: the benchmarks were using different randomized testdata. Fixed.
goos: darwin
goarch: arm64
pkg: github.com/grafana/pyroscope-go/godeltaprof/compat
                                      │   old.txt   │               new.txt                │
                                      │   sec/op    │    sec/op      vs base               │
HeapCompression-8                       691.1µ ± 0%    729.8µ ± 65%   +5.59% (p=0.002 n=6)
MutexCompression/ScalerMutexProfile-8   640.6µ ± 1%    693.8µ ±  9%   +8.30% (p=0.002 n=6)
MutexCompression/ScalerBlockProfile-8   643.2µ ± 1%    690.3µ ±  1%   +7.33% (p=0.002 n=6)
HeapDelta-8                             76.26µ ± 1%   113.92µ ±  2%  +49.38% (p=0.002 n=6)
MutexDelta/ScalerMutexProfile-8         48.98µ ± 1%    81.11µ ±  2%  +65.59% (p=0.002 n=6)
MutexDelta/ScalerBlockProfile-8         49.03µ ± 1%    81.14µ ±  1%  +65.49% (p=0.002 n=6)
geomean                                 193.3µ         253.0µ        +30.87%

                                      │   old.txt    │               new.txt               │
                                      │     B/op     │     B/op      vs base               │
HeapCompression-8                       974.1Ki ± 0%   973.9Ki ± 0%   -0.03% (p=0.002 n=6)
MutexCompression/ScalerMutexProfile-8   974.6Ki ± 0%   974.3Ki ± 0%   -0.02% (p=0.002 n=6)
MutexCompression/ScalerBlockProfile-8   974.6Ki ± 0%   974.3Ki ± 0%   -0.03% (p=0.002 n=6)
HeapDelta-8                              333.00 ± 0%     53.00 ± 0%  -84.08% (p=0.002 n=6)
MutexDelta/ScalerMutexProfile-8          312.00 ± 0%     30.00 ± 0%  -90.38% (p=0.002 n=6)
MutexDelta/ScalerBlockProfile-8          312.00 ± 0%     30.00 ± 0%  -90.38% (p=0.002 n=6)
geomean                                 17.42Ki        5.874Ki       -66.28%

                                      │  old.txt   │              new.txt              │
                                      │ allocs/op  │ allocs/op   vs base               │
HeapCompression-8                       788.0 ± 0%   787.0 ± 0%   -0.13% (p=0.002 n=6)
MutexCompression/ScalerMutexProfile-8   788.0 ± 0%   787.0 ± 0%   -0.13% (p=0.002 n=6)
MutexCompression/ScalerBlockProfile-8   788.0 ± 0%   787.0 ± 0%   -0.13% (p=0.002 n=6)
HeapDelta-8                             2.000 ± 0%   1.000 ± 0%  -50.00% (p=0.002 n=6)
MutexDelta/ScalerMutexProfile-8         2.000 ± 0%   1.000 ± 0%  -50.00% (p=0.002 n=6)
MutexDelta/ScalerBlockProfile-8         2.000 ± 0%   1.000 ± 0%  -50.00% (p=0.002 n=6)
geomean                                 39.70        28.05       -29.33%

So it looks like the cost of dediplication is about 5%, which is not that bad.


Problem with a YT short site

Checklist

[x ] I'm reporting a broken site support issue
[ x] I've verified that I'm running youtube-dl version 2021.12.17
[ x] I've checked that all provided URLs are alive and playable in a browser
[ x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x ] I've searched the bugtracker for similar bug reports including closed ones
[ x] I've read bugs section in FAQ

Verbose log

$ ./youtube-dl.exe --verbose https://youtube.com/shorts/aCYFDPdNM50
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://youtube.com/shorts/aCYFDPdNM50']
[debug] Encodings: locale cp1252, fs mbcs, out cp1252, pref cp1252
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.4.4 (CPython) - Windows-7-6.1.7601-SP1
[debug] exe versions: none
[debug] Proxy map: {}
[youtube:tab] shorts: Downloading webpage
ERROR: Unable to recognize tab page; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 815, in wrapper
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 836, in __extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 534, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\youtube.py", line 2862, in _real_extract
youtube_dl.utils.ExtractorError: Unable to recognize tab page; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

A YT Short video site is not recognized.

You can use video ID as fallback:
youtube-dl aCYFDPdNM50

Fixed in git master.


DFF with constant set or reset non removal

I compiled attached file with yosys 0.9+2406 and yosys 0.9+3521. It seems 3521 does not remove DFFs with constant set or reset inputs. On the other hand, 2406 removes DFFs with constant set or reset input. I have also attached log files and final synthesis results for both version. Same ys script is used. My operating system is Centos 7.
dffissue.zip

Thanks for quick response


Add logo keyword to Documenter

Closes #221
I think this does the right thing, at least.

Nice! Thanks! This PR looks perfect for the assumption there's a single logo (andis sufficient for my use-case), however Documenter actually supports have two logos (light theme and dark theme) - how should that case be handled?

Documenter checks for the existence of assets/logo.{svg,png,webp,gif,jpg,jpeg}, in this order. The first one it finds gets displayed at the top of the navigation sidebar. It will also check for assets/logo-dark.{svg,png,webp,gif,jpg,jpeg} and use that for dark themes.

ah shucks
another kw logo_dark I guess?

another kw logo_dark I guess?

🤷♂
sure 😆

I feel like 826aa536adbea87f7ddbc382a3ba5cffcf9733e6 is maybe a bit better... thoughts?


[Bug] Azure exits with exit code 1 after authentication

Windows Terminal version
1.15.3466.0
Windows build number
10.0.22621.963
Other Software
No response
Steps to reproduce

Open the Terminal ;
Open a new tab (Azure Cloud Shell) ;
Use the login code to login at https://microsoft.com/devicelogin ;
The Terminal outputs that I have been authenticated and Azure exits suddenly with exit code 1 .

This is the output in Chinese below (Can't upload screenshots) :
若要登录，请使用 Web 浏览器打开页 https://microsoft.com/devicelogin，然后输入代码 I7[Private]8 进行身份验证。
此代码将在 15 分钟后过期。
已经过身份验证。

[已退出进程，代码为 1 (0x00000001)]

Expected Behavior
No response
Actual Behavior
Azure exits suddenly with exit code 1(0x00000001).

Btw, I also tried to run Terminal as Administrator, and it isn't fixed.

Since you authenticated but didn't get a prompt to select a tenant, it seems that either

the process is exiting while attempting to populate your list of tenants
OR you only have 1 tenant, and the process is exiting after getting that tenant but before you get the prompt asking you if you would like to store your credentials (this feels unlikely as there isn't much going on here)

Are you able to use the Azure Cloud Shell directly in your browser via this link https://shell.azure.com/ ?

Hi,
I have just noticed that I didn't have Azure account yet. Maybe this would cause the problem?
The link reminds me that I have not signed up and paid for it.

I have just noticed that I didn't have Azure account yet. Maybe this would cause the problem?

Very possibly! Once you are able to use the Azure Cloud Shell in your browser (via that link), try again in Terminal and see if it works

It works with a valid az shell account:


question: Native way to hide the 'Contact support' link on Login page?

A clear and concise description of what you want to accomplish.

On the Login page there is a "Contact support" link to /auth/help.
To hide the link we could inject custom css/js (custom css/js path ...) OR provide a custom login template (template login ...).
However, is there a native way to hide or edit this link without css/js/template customisations?

However, is there a native way to hide or edit this link without css/js/template customisations?

@iamjrock, this is a feature request. I am actually in the process of redesigning some of the templates. So ... will add an option to hide the link.

will add an option to hide the link

Hey that's awesome Paul, thx for your time and awesome caddy module.

@iamjrock , I did not get to it yet. Your ask involves modifying sandbox template. Right now I want to focus on settings and build towards user management.
As a shortcut, I suggest adding custom JS that would remove the link programmatically. Here, you don't modify a template. Rather, you inject custom JS https://authp.github.io/docs/authenticate/ui-features#custom-javascript

OK understood.  We will use custom js or custom css for now.
Would you like me to leave this issue open as a placeholder for the feature?

Would you like me to leave this issue open as a placeholder for the feature?

@iamjrock , yes. I will get to it 😄
FYI, I already added the necessary plumbing - see https://github.com/authp/authp.github.io/issues/23
What is left is the template.

You move faaaast! 👍

Starting working on sandbox template. That's where the links show up.

@iamjrock , please test with the latest release. The contact support link will not be visible during authentication.

Ok thx @greenpau! Will test. My pc died last night :-( so may take a few days.

+1 for hiding the 'Contact support' ability. And related to https://github.com/greenpau/caddy-security/issues/127.

@greenpau whilst testing with v1.1.13 I am getting this error:
run: adapting config using caddyfile: parsing caddyfile tokens for 'security': /                                                                              app/conf/caddy/Caddyfile:79 - Error during parsing: unsupported subdirective for                                                                               security.authentication.portal.ui: password_recovery_enabled
I know this is not related to the ui changes in this issue, but was this directive removed since v1.1.7?

I know this is not related to the ui changes in this issue, but was this directive removed since v1.1.7?

The password_recovery_enabled in the UI was removed in favor of enable password recovery under a specific identity store. Please see https://github.com/authp/authp.github.io/issues/23

Some more testing:
I commented password_recovery_enabled out of the Caddyfile and then caddy worked.
However, on the Sign In page:

The "Contact Support" link still shows
Now the "Forgot Username" field is also showing (because password_recovery_enabled was commented out....)
The "Register" link is showing. This is using a local json IdP file and we don't want people to be able to register.  I'm pretty sure that the Register link wasn't showing before (but maybe I'm wrong?).

So in summary - Contact Support is still visible (sorry!) and I'm trying to work out how to hide all 3 links in this new version.
Thanks again for your time on this.
Screen shot:

The "Contact Support" link still shows

@iamjrock , I need to fix the use case where there is only 1 authentication provider/store. Will tackle it in the next release.

The password_recovery_enabled in the UI was removed in favor of enable password recovery under a specific identity store. Please see authp/authp.github.io#23

Ah I see. So to disable these do we just omit the following from the identity store definition:
enable password recovery
enable username recovery
enable contact support
Or do we use something like:
disable password recovery
disable username recovery
disable contact support

The "Contact Support" link still shows

@iamjrock , I need to fix the use case where there is only 1 authentication provider/store. Will tackle it in the next release.

OK thx.  Is it the same situation for all 3?  Ie. All of these will not work with a single IdP/store until the next release:
enable password recovery
enable username recovery
enable contact support

OK thx. Is it the same situation for all 3? Ie. All of these will not work with a single IdP/store until the next release:

Register link: if you disable registration, then it will not show
Username recovery: if not enabled, disabled (the feature is actually not implemented)
Contact support, if not enabled, disabled

@iamjrock , could you please write a testimonial for your use of the project? Please see https://docs.authcrunch.com/help-needed for details.


[bbc] Failed to parse JSON error

[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--proxy', u'<IP_ADDRESS>:3128', u'--verbose', u'http://www.bbc.co.uk/iplayer/episode/b071fyfr']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.02.13
[debug] Python version 2.7.4 - Linux-2.6.32-5-openvz-686-i686-with-debian-6.0.10
[debug] exe versions: avconv 0.8.6-6, avprobe 0.8.6-6, ffmpeg 0.8.6-6, ffprobe 0.8.6-6, rtmpdump 2.3
[debug] Proxy map: {u'http': u'<IP_ADDRESS>:3128', u'https': u'<IP_ADDRESS>:3128'}
[bbc.co.uk] b071fyfr: Downloading video page
[bbc.co.uk] b071fyfr: Downloading playlist JSON
ERROR: b071fyfr: Failed to parse JSON  (caused by ValueError('No JSON object could be decoded',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/home/steve/personal/python/youtube-dl.py/youtube_dl/extractor/common.py", line 509, in _parse_json
return json.loads(json_string)
File "/usr/local/lib/python2.7/json/init.py", line 338, in loads
return _default_decoder.decode(s)
File "/usr/local/lib/python2.7/json/decoder.py", line 365, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/local/lib/python2.7/json/decoder.py", line 383, in raw_decode
raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded
Traceback (most recent call last):
File "/home/steve/personal/python/youtube-dl.py/youtube_dl/YoutubeDL.py", line 666, in extract_info
ie_result = ie.extract(url)
File "/home/steve/personal/python/youtube-dl.py/youtube_dl/extractor/common.py", line 315, in extract
return self._real_extract(url)
File "/home/steve/personal/python/youtube-dl.py/youtube_dl/extractor/bbc.py", line 507, in _real_extract
programme_id, title, description, duration, formats, subtitles = self._download_playlist(group_id)
File "/home/steve/personal/python/youtube-dl.py/youtube_dl/extractor/bbc.py", line 396, in _download_playlist
playlist_id, 'Downloading playlist JSON')
File "/home/steve/personal/python/youtube-dl.py/youtube_dl/extractor/common.py", line 503, in _download_json
json_string, video_id, transform_source=transform_source, fatal=fatal)
File "/home/steve/personal/python/youtube-dl.py/youtube_dl/extractor/common.py", line 513, in _parse_json
raise ExtractorError(errmsg, cause=ve)
ExtractorError: b071fyfr: Failed to parse JSON  (caused by ValueError('No JSON object could be decoded',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Your proxy transfers all requests to TellyPass.com.

On Sun, Feb 21, 2016 at 01:56:44AM -0800, Sergey M. wrote:

Your proxy transfers all requests to TellyPass.com.

I'm not sure why that should stop the download from happening, but even
if it should, then perhaps a better error message would be helpful?
Something like "your proxy is doing bad things, change the proxy".
Steven

There is no notion of a bad thing. You direct request to proxy, proxy returns you an HTML. It's impossible to guess whether it's bad or not.


Add code change to 'Account.java'

Failed Tests:

testAddToBalance() from io.diffblue.corebanking.account.AccountDiffblueTest
expectedount10  Balance20   Transactionstatbutwasount10 Balance0    Transactionstat
testExecuteTransaction() from io.diffblue.corebanking.transaction.CashTransactionDiffblueTest
expectedount10  Balance20   Transactionstatbutwasount10 Balance0    Transactionstat

Tests run: 46, Failures: 2, Errors: 0, Skipped: 0


gRPC code "Unavailable" is being reported as "Unknown" by temporal SDK Client

Expected Behavior
Correct gRPC code should be recorded & returned.
Actual Behavior
gRPC code "Unavailable" is being reported as "Unknown"
Steps to Reproduce the Problem

Run a temporal server that returns Http 503 Unavailable gRPC code
Verify that the correct code is being received via another client. In my case, I verified in python, a generic gRPC client & with postman. They all confirmed the "Unavailable" code.
Sample code:

func testRawGrpc() {
	var grpcClient *grpc.ClientConn
	var err error
	creds := credentials.NewTLS(&tls.Config{})
	grpcClient, err = grpc.Dial("<host:port>", grpc.WithTransportCredentials(creds), grpc.WithMaxMsgSize(1*1024*1024), grpc.WithTimeout(10*time.Second))

	defer grpcClient.Close()

	if err != nil {
		log.Printf("unable to create client: %#v\n", err)
		return
	}

	req := workflowservice.ListNamespacesRequest{
		PageSize:        100,
		NamespaceFilter: &namespace.NamespaceFilter{IncludeDeleted: true},
	}

	resp := workflowservice.ListNamespacesResponse{}

	err = grpcClient.Invoke(context.Background(), "/temporal.api.workflowservice.v1.WorkflowService/ListNamespaces", &req, &resp)
	if err != nil {
		log.Printf("unable to ListNamespaces: %#v\n", err)
		grpcErr, _ := status.FromError(err)
		log.Printf("grpcMessage=%v, grpcCode=%v\n", grpcErr.Message(), grpcErr.Code())
		return
	}
	log.Println("ListNamespaces result:", &resp)
}

Notice in the logs:
...
grpcMessage=no healthy upstream, grpcCode=Unavailable
...

Now, try the same with the sdk client

func testTemporalClient() {
	temporalConnectionOpts := client.ConnectionOptions{
		TLS: &tls.Config{},
	}
	temporalClientOpts := client.Options{
		HostPort:          "<host:port>",
		Namespace:         "default",
		ConnectionOptions: temporalConnectionOpts,
	}
	c, err := client.Dial(temporalClientOpts)
	if err != nil {
		grpcErr, _ := status.FromError(err)
		fmt.Printf("grpcMessage=%v, grpcCode=%v\n", grpcErr.Message(), grpcErr.Code())
		log.Println("Failed to create temporal client:", err)
		return
	}
	defer c.Close()

	req := workflowservice.ListNamespacesRequest{
		PageSize:        100,
		NamespaceFilter: &namespace.NamespaceFilter{IncludeDeleted: true},
	}

	resp, err := c.WorkflowService().ListNamespaces(context.Background(), &req)
	if err != nil {
		log.Println("Unable to ListNamespaces", err)
		grpcErr, _ := status.FromError(err)
		log.Printf("grpcMessage=%v, grpcCode=%v\n", grpcErr.Message(), grpcErr.Code())
		return
	}
	log.Println("ListNamespaces result:", resp)
}

Notice in the logs:
...
grpcMessage=failed reaching server: no healthy upstream, grpcCode=Unknown
...

Specifications

Version: Observed in go-SDK versions v1.24.0, v1.25.0 & v1.28.1
Platform: Linux

In your second example is the error coming from c, err := client.Dial(temporalClientOpts)?

In your second example is the error coming from c, err := client.Dial(temporalClientOpts)?

Yes. That is when the System Health Check is getting triggered & hence the error.

The Go SDK is not changing the status code of any errors. status.FromError(err) is returning UNKNOWN because it encounters an incompatible error type. If you checked the second parameter you would see OK is false indicating the error is not compatible with status. The Go SDK does not return raw grpc errors, it returns serviceerrors if you want to get the underlying status you can use ToStatus

Good point. My bad for missing that.
func testTemporalClient() {
	temporalConnectionOpts := client.ConnectionOptions{
		TLS: &tls.Config{},
	}
	temporalClientOpts := client.Options{
		HostPort:          "<host:port>",
		Namespace:         "default",
		ConnectionOptions: temporalConnectionOpts,
	}
	c, err := client.Dial(temporalClientOpts)
	if err != nil {
		grpcErr := serviceerror.ToStatus(err)
		fmt.Printf("grpcMessage=%s, grpcCode=%s\n", grpcErr.Message(), grpcErr.Code())
		fmt.Println("Failed to create temporal client:", err)
		return
	}
	defer c.Close()
}

This still prints:
...
grpcMessage=failed reaching server: no healthy upstream, grpcCode=Unknown
Failed to create temporal client: failed reaching server: no healthy upstream
...

What am I missing? Am I wrong to expect "Unavailable" is the logs above?

@Quinn-With-Two-Ns can you please reopen this?

Am I wrong to expect "Unavailable" is the logs above?

Maybe so. I think not even be able to establish connection is not a gRPC RPC error, it's a network error, so I wouldn't expect it to have a gRPC status code.

An HTTP status code of 503 unavailable is not the same as a gRPC status code of unavailable (gRPC status coeds do not have HTTP numeric status codes). Can you show what type err is in your snippet? Also, can you confirm what https://pkg.go.dev/google.golang.org/grpc/status#FromError returns for this error?

An HTTP status code of 503 unavailable is not the same as a gRPC status code of unavailable (gRPC status codes do not have HTTP numeric status codes). Granted some clients in some languages may reuse the gRPC errors pre-gRPC connectivity.

I don't quite follow what you mean here. Here are the codes: https://pkg.go.dev/google.golang.org/grpc/codes#Code .

Can you show what type err is in your snippet?

c, err := client.Dial(temporalClientOpts)
vs
grpcErr := serviceerror.ToStatus(err)

Does that answer your question?

https://pkg.go.dev/google.golang.org/grpc/status#FromError returns for this error?

@cretz please see the function: testRawGrpc in the description of this issue.
I can confirm that FromError returns the "Unavailable" code as expected.

We want to make sure we match what Go gRPC does here, so maybe we're wrapping incorrectly compared to pure gRPC use.

Agreed. Exactly what this issue is about.

I can confirm the Go SDK is correctly wrapping the error. I can confirm if a grpc server returns Unavailable for GetSystemInfo the SDK client will return the correct status.
func TestUnavailable(t *testing.T) {
	// Start a server that always returns an error on get system info
	srv, err := startTestGRPCServer()
	require.NoError(t, err)
	defer srv.Stop()
	srv.getSystemInfoResponseError = status.New(codes.Unavailable, "some server failure").Err()

	// Confirm DialClient returns unavailable
	_, err = DialClient(context.Background(), ClientOptions{HostPort: srv.addr})
	status := serviceerror.ToStatus(err)
	require.Equal(t, codes.Unavailable, status.Code())
}

func TestBadHostUnavailable(t *testing.T) {
	// Confirm DialClient returns unavailable
	_, err := DialClient(context.Background(), ClientOptions{HostPort: "localhost:12345"})
	status := serviceerror.ToStatus(err)
	require.Equal(t, codes.Unavailable, status.Code())
}

@Quinn-With-Two-Ns your tests fail in v1.24.0 (please see screenshot). I did not test against other released versions but they succeed against the master branch of this repo.

Since it worked against the latest code and that nothing in the diff stood out to me as what may have "fixed" the issue. , I am happy to leave this issue closed & wait for the next release.
As always, appreciate the quick responses & your time!


Write a function in JS that gets a country name and returns the exchange rate of the currency to GBP

Completed the task of fetching the currency rate of baseCurrency and targetCurrency. added the code in script.js


Fix unique reference obtained from shared reference

slice.as_ptr goes through a &[T], which only gives it permission to read and not write. Fix it by using as_mut_ptr instead.
Also add miri with MIRIFLAGS=-Zmiri-strict-provenance to CI.

Thanks for the catch and PR!


Node count on cluster view gets out of sync.

What kind of request is this (question/bug/enhancement/feature request):
bug, possible enhancement
Steps to reproduce (least amount of steps as possible):
When using cluster autoscaler after having a few large scaling events, the node count in the UI would become out of sync. After a few hours the node count reflected accurately but this misrepresentation could cause confusion.
Not sure if we can make the resync for this more aggressive or introduce a trigger to manually induce a sync.

Based on @aemneina offline comment, the cluster type is EC2 RKE, custom.

I think this was originally spotted in v2.0.6.


Delay in monitor status - on purpose? Service calls seems strange, no entity?

Version of the custom_component

1.4.1
Configuration

Describe the bug
When I turn the monitor on or off, there is a delay on the status shown in Lovelace.
Is it 'as designed' or is it a fluke?
The services set up in the integration doesn't show any entity requirement when looking in 'developer tools', which seems like a problem :-)

P.S. I am SUPER excited that you made this integration, it's simply fantastic to not have to set up MQTTBridge like I've had for years now.

Ahh, ok, I can see that you are removing the special service calls, so that part is at least making sense, is the removal of 'binary_sensor' part the reason for the delay in status, it reads the status of the mirror, and with the new switch entity you are going to make it 'optimistic' in the sense that the switch will hold the status itself, and just hope the mirror agrees?

I'm sure there are some optimizations possible to make the switch more responsive. I will take a look at it. I wrote the integration without much experience, but in theory it should fetch new state after POST, so it might just be the delay from the next status-fetch. Making it optimistic might be the best way out if there are no other glaring issues.
Will at it to my internal todo

And also most of the service calls will be deprecated to be replaced by buttons per MagicMirror. For whatever service calls are left I might need a entity, hadn't though of that. Not sure how the integration handles multiple magicmirrors to be honest.

Well, the optimistic approach will of course give faster status updates, as it doesn't check the real status, problem is though, that if the monitor is turned on or off locally, the switch will not reflect that if it's optimistic, so it's a bit damned if you do, and damned if you don't 😆

For sure, I'll try to do some tweaks and see what works best. Maybe we can do optimistic but update if it fails or something?

Yes, that would probably work well if it's doable.

I have released a pre-release v1.2.0b0, which contains optimistic switch as well as some other improvements. All the services have been removed as well, now replaced with button-entities and notify-service. Again not 100% sure how the integration would handle multiple instances, but I'll tackle that at some later time.
I like to test the new releases for a few days before I properly release them, but you can update it to try if you want.

Should be solved by release 1.2.0


Gateway: availability of DirectQuery with Enterprise mode

According to the release notes of the On-premises gateway, the DirectQuery mode is now supported for both personal and enterprise modes. See at https://powerbi.microsoft.com/en-us/blog/on-premises-data-gateway-august-update-is-now-available/

We will be deprecating these docs shortly (I should have already done it but I've been very busy) and keeping it for samples, in favor of the new ones at https://docs.microsoft.com/en-us/power-query/


Accept Header Validation Is Too Restrictive

Description
The validation of the Accept request header for the /export endpoint has become too restrictive in version 6.9.0 (due to this pr). Requests with the header set to values indicating that multiple formats are accepted (ike application/json, application/ndjson) now cause a 400: Bad Request response:
{
  "statusCode": 400,
  "code": "FST_ERR_VALIDATION",
  "error": "Bad Request",
  "message": "headers/accept must be equal to one of the allowed values"
}

Environment
I'm using CRUD Service version 6.9.0
Minimal Reproduction
curl \
--location 'http://crud-service/collection/export' \
--header 'Accept: application/json, */*'

@danibix95 we could it fix by removing:
"enum"
and adding
"patterm": "(\*\/\*|application\/ndjson)"?

New service version v6.9.1 has been released, which fixes this issue


Yarn mint won't post to IPFS

Where do We go to change the files to assets we want to try posting?

looks like you need to call the transferOwnership(your frontend address) in the deploy or the constructor of the contract then you will be owner and able to mint

@DylanCkawalec were you able to resolve this so I can close this issue?


Migrate to python 3, closes #425

Refactor all python utils
Add type annotations for functions
Reformat files in accordance with PEP8
Update minimum required language version from 2.7 to 3.5

Well, I suppose I'll fix it a bit later.

Ok, seems like there are some issues with how CMake looks for Python - let's try some hacky way: PYTHON_ROOT_DIR may work.
 - cmd: cmake -G "Visual Studio 14 2015 Win64"  -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTS=On

to
 - cmd: cmake -G "Visual Studio 14 2015 Win64"  -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTS=On -DPYTHON_ROOT_DIR=%PYTHON%

Ok, seems like everything on CI tools is good.
I will check scripts overall a little bit later (not all scripts are covered by tests or build in any way) and merge if everything will be ok.
Thanks!

I supposed that this will be squash merge so I felt free to create a bunch of commits with temporary description. Feel pretty sorry for now.

Nah, it's ok.


Changed webpack.entry to be an object

This PR changes entry to always be an object, except for the case when the user manually sets a function instead.
A warn is now shown if entry is a string or an array of strings.
closes #6947

@timneutkens I had to do a weird workaround with one of the tests that was launching a dev server, using entry as an object was stopping this line from getting called. The issue is probably with webpack-[dev/hot]-server


[ensmallen] VCPKG Version Out of Date

Library name:   ensmallen
New version number:    2.14.2
Other information that may be useful (release notes, etc...)

Hi @Blakjak88 ,The latest version of  ensmallen is 2.14.2, So it's your need?

Hi, yes. The current version has missing features.

@Blakjak88 ,Thansk for posting this issue.
I have submitted pr to upgrade ensmallen to the latest version 2.17.0. When pr is merged, you can update vcpkg to test.


anchors only have one radius

Hi,
I am having an issue where when I place down an anchor and break it I can still fly in the area it was in.
Thanks!

Same problem here
In addition, when you fly without the anchors, you may take fall damage when landing


feat/enable_logging: enable core logging

Coverage decreased (-0.1%) to 84.008% when pulling c1e3c24c3b289f36aa8e92343f31bf5d33f068aa on shankar2105:log into 8ecb773d11eae309ca968cf90fa81cb31111e37c on maidsafe:master.


Unify posts naming to YYYY-MM-DD-COMPONENT-X-Y-Z.md

See https://trello.com/c/DJXPIONO/26-news-tidy-up
this PR should review all of the divergent naming schemes used under _posts to use a single naming scheme. OME Files C++ and OMERO.mtools are called ome-files-cpp and mtools respectively.
/cc @jburel

name changes look good
Merging


Submission to join

What is your username and discriminator?
j.erwin#1449
Why did you join Discord Linux?
Looking for support with Discord on Linux.
Where did you hear about Discord Linux?
Other official servers like the discord testing one
What is the passphrase that was assigned to you by the bot?
sudo -i empty-determined-occur-last

From Ethan Weegee#9302 :
Looks good.  I've given you a temporary role so that you can enter the passphrase in the welcome channel.


Disable Do you want the application “Electron.app” to accept incoming network connections?

On OS X, when running something spectron (via eg. tape), I always get this dialog box. I see that for final applications, there's a parent.plist that should be added to, as part of publishing. What should I do to prevent this dialog box during local test running, which is many? What am I doing wrong, since, based on searching around on the net or among the issues, it doesn't look like a frequently asked question?

@monfera I am having the same problem. It'll disappear once you disable your Mac's firewall inside the System Preferences though.


[ENH] - Increase h1 margins

Feature description
Increasing the top and bottom margins for h1 will be nicer in the Login & Server options page
Currently the text seems very close to the top navbar:

Value and/or benefit
Visual nicety
Anything else?
I think the relevant section to be updated is:
https://github.com/nebari-dev/jhub-apps/blob/43fcc8c0cf1086b2251a77387b80a595eefba499/jhub_apps/templates/style.css#L54-L65
In the old theme, 30px was set, ref: https://github.com/nebari-dev/nebari-jupyterhub-theme/blob/dbfa10e9ca071ed0c5e5015442faaba4ee3c1285/nebari_jupyterhub_theme/templates/style.css#L74-L78

This is issue should be resolved, and I believe was related to the Bootstrap upgrade. It looks like you're running an older version of the app launcher based on a few things. This is the change that fixed it: https://github.com/nebari-dev/jhub-apps/commit/9c70cbf6abfe08b5a10a513ef4197364e3aa86a8#diff-323f0362761d127917ab6d95526774b20c29df50ce9c25da35cffa14b28d0ed0R374
Here is what it looks like now.


Improve docs for balenaEngine devs

Describe the project organization.
Include a list of commands commonly used during development.
Update the documentation on how to merge upstream changes, based on my
experience following the instructions for the first time.

Hey, thanks for the comments!

For cross compiling, do you do this on your host machine or on the container you are put in.

On the host. I added a note explaining that all commands are to be run on the host (unless otherwise is mentioned in the description).

Then do you scp the binary into /mnt/data?

Yes. Or, if I just flashed balenaOS, I copy the binary directly to right partition on the SD card.

In cases the device is full, it would be good to understand which host this can happen.

Not sure I understand your question. But to be honest, I always use the /mnt/data + symlink trick because it always works and allows to easily switch between different Engine versions for tests/comparisons.

For unit tests, it'd be nice to have estimated time these should run. Looks like it takes under 2 minutes for unit tests.

Yup, good call. I updated the docs with this info. (Takes two minutes to me. :slightly_smiling_face:)

Merge time, then! Thanks! :-)


223 data base account follow up

Hello!
I have added code to the init_db.py file that adds an API user and changed the location string in the yml file that the api container uses to use this account information. I also made a script that we can run to create admin/researcher accounts, I think that the plan for user accounts was to add manually so that's what that's for, just to make it easier.
Let me know if anything looks strange and please try to test, I used the api with the new permissions to pull inputs and it did so successfully with the permissions i assigned. To check what the permissions are in PG Admin you can use the PSQL tool and the command \z to check the permissions of each user on all of the tables.

Tissot: Bad Practice: We don't want to concatenate strings with input variables, it leaves the function open to SQL injection attacks.

We also need to see how we can keep the api account across deployments, since we don't want to run init_db again once the database has been up for a while


Fatal errors during initial sync

What version of Music Assistant has the issue?
2022.7 beta 0
The problem
At the end of the initial sync, I get the following error in the log and I have no music in the library:

How to reproduce
Initial sync
Relevant log output
Logger: music_assistant
Source: /usr/local/lib/python3.10/site-packages/music_assistant/mass.py:245
First occurred: 6:23:48 PM (6 occurrences)
Last logged: 6:23:58 PM
Job [Library sync for provider Spotify] failed with error no such table: artists.
Job [Cleanup removed items from database] failed with error no such table: cache.
Job [Library sync for provider Filesystem] failed with error no such table: cache.
Traceback (most recent call last):
File "/usr/local/lib/python3.10/site-packages/music_assistant/models/music_provider.py", line 307, in sync_library
for db_item in await controller.db_items(True):
File "/usr/local/lib/python3.10/site-packages/music_assistant/models/media_controller.py", line 76, in db_items
return await self.get_db_items_by_query(
File "/usr/local/lib/python3.10/site-packages/music_assistant/models/media_controller.py", line 262, in get_db_items_by_query
for db_row in await self.mass.database.get_rows_from_query(
File "/usr/local/lib/python3.10/site-packages/music_assistant/helpers/database.py", line 107, in get_rows_from_query
return await self._db.fetch_all(query, params)
File "/usr/local/lib/python3.10/site-packages/databases/core.py", line 142, in fetch_all
return await connection.fetch_all(query, values)
File "/usr/local/lib/python3.10/site-packages/databases/core.py", line 250, in fetch_all
return await self._connection.fetch_all(built_query)
File "/usr/local/lib/python3.10/site-packages/databases/backends/sqlite.py", line 98, in fetch_all
async with self._connection.execute(query_str, args) as cursor:
File "/usr/local/lib/python3.10/site-packages/aiosqlite/context.py", line 41, in aenter
self._obj = await self._coro
File "/usr/local/lib/python3.10/site-packages/aiosqlite/core.py", line 184, in execute
cursor = await self._execute(self._conn.execute, sql, parameters)
File "/usr/local/lib/python3.10/site-packages/aiosqlite/core.py", line 129, in _execute
return await future
File "/usr/local/lib/python3.10/site-packages/aiosqlite/core.py", line 102, in run
result = function()
sqlite3.OperationalError: no such table: artists
Additional information
No response
What version of Home Assistant Core are your running
latest
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Windows

I have similar errors, except Spotify (don't have that), and no media scraped.
I can browse and play media via the browse and radio buttons.
I'm on HA core 2022.7.
and MA 2022.7 beta 0

strange, can you try to restart HA or reload the integration ?

Yes I tried that - I even nuked db and resynced with same errors

How is that possible... it is trying to sync without the db being initialized ?!

Found the issue. Fix in next version

published beta 2 with the fix


Tutorial

Hello Lohani,
How can i get the tutorial to this repo

@Mangielijah Not sure what you mean?

Here check this tutorial
https://www.youtube.com/watch?v=a0c15BpgeH8


Credit scene for the game over

Right now if we finish the game (win or lose), it result to a crash because there is no credits scene.
I'll try to do one soon.

I added a ui/credits.tscn file, but I suppose there is a credits.gd file missing to make it work in the game...
I don't know how to do that...
@punto- ?

Thanks @punto-  !
But it crash for me while playing and finishing the game ...
hud got tooltip text 	Utiliser inv_bad_bamboo_2.tooltip avec bamboo_god_2.tooltip
hud got tooltip text
set state 	bamboo_god_2	default	angry	False
has animation	True
playing animation 	angry
item set speaking! 	bamboo_god_2	True	angry
 ** loading speech 	res://audio/speech/en/bamboo_god_2_use_2.spx
ERROR: set_file: Condition ' err != OK ' is true.

ERROR: get_tree: Condition ' !data.tree ' is true. returned: __null
   At: scene/main/node.h:209.
ERROR: load_interactive: Condition ' err!=OK ' is true. returned: Ref<ResourceInteractiveLoader>()
   At: core/io/resource_format_binary.cpp:1081.
ERROR: load: Failed loading resource: res://game/globals/scene_main.scn
   At: core/io/resource_loader.cpp:218.
ERROR: get_tree: Condition ' !data.tree ' is true. returned: __null
   At: scene/main/node.h:209.
ERROR: get_tree: Condition ' !data.tree ' is true. returned: __null
   At: scene/main/node.h:209.

For the translations, your are right for UI_CREDITS vs CREDITS, and I think everything can be at the same place... no need to create and code credits_en.txt, credits_fr.txt I think, it's quite short in our case. I'll correct the general.csv file ;)
We are almost finish...

Fixed :)
On 12 September 2016 at 10:06, Camille Bissuel<EMAIL_ADDRESS>wrote:

Thanks @punto- https://github.com/punto- !
But it crash for me while playing and finishing the game ...
hud got tooltip text    Utiliser inv_bad_bamboo_2.tooltip avec bamboo_god_2.tooltip
hud got tooltip text
set state   bamboo_god_2    default angry   False
has animation   True
playing animation   angry
item set speaking!  bamboo_god_2    True    angry
** loading speech  res://audio/speech/en/bamboo_god_2_use_2.spx
ERROR: set_file: Condition ' err != OK ' is true.

ERROR: get_tree: Condition ' !data.tree ' is true. returned: __null
At: scene/main/node.h:209.
ERROR: load_interactive: Condition ' err!=OK ' is true. returned: Ref()
At: core/io/resource_format_binary.cpp:1081.
ERROR: load: Failed loading resource: res://game/globals/scene_main.scn
At: core/io/resource_loader.cpp:218.
ERROR: get_tree: Condition ' !data.tree ' is true. returned: __null
At: scene/main/node.h:209.
ERROR: get_tree: Condition ' !data.tree ' is true. returned: __null
At: scene/main/node.h:209.
For the translations, your are right for UI_CREDITS vs CREDITS, and I
think everything can be at the same place... no need to create and code
credits_en.txt, credits_fr.txt I think, it's quite short in our case. I'll
correct the general.csv file ;)
We are almost finish...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flossmanualsfr/escoria/issues/4#issuecomment-246340966,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVmPd1hGggK6ValpjMI9hQKUlGEvx_fks5qpU5OgaJpZM4JwQUc
.

Great ! Just tested, it works nicely in every cases ;)
Thanks Ariel !


Username of Wamp Pusher

Since Wamp Pusher is using onPublish like browser client, is there a way in the onPublish method of TopicHandler to find out that it is the server who is sending the message? With sharing session, I find that anon-xxx user is the user pushing from server; basically I can't differentiate whether the user is the server or browser client.
Example use case where it is useful to find out if the user is the server:

All browser clients subscribe to server_notification channel. But only the server is allowed to message (push) on that channel to all subscribe clients. If a browser client publish on that channel, the server will simply ignore.

How to do this with this bundle?

You can create an identity for the server (a secret, a jwt that say 'hey 'im the server) and filter all incoming message to let only your server broadcast messages.


Aspire 9 app - azd provision  / up via Visual Studio Publish

Is there an existing issue for this?

[x] I have searched the existing issues

Describe the bug
After updating to Aspire 9, and running  the VS Publish process (as per logs below)
26/11/2024 11:03:45 AM: Info
26/11/2024 11:03:45 AM: Info Provisioning Azure resources (azd provision)
26/11/2024 11:03:45 AM: Info Provisioning Azure resources can take some time.
26/11/2024 11:03:45 AM: Info
26/11/2024 11:03:47 AM: Info Analyzing Aspire Application (this might take a moment...)
26/11/2024 11:04:08 AM: Info Initialize bicep provider
26/11/2024 11:04:14 AM: Info Reading subscription and location from environment...
26/11/2024 11:04:16 AM: Info Subscription: CSP Production (0c5b0241-2f0c-419c-adea-51f5be95eea1)
26/11/2024 11:04:16 AM: Info Location: UK West
26/11/2024 11:04:16 AM: Info
26/11/2024 11:04:16 AM: Info Comparing deployment state
26/11/2024 11:04:16 AM: Info Comparing deployment state
26/11/2024 11:04:17 AM: Info Creating/Updating resources
26/11/2024 11:04:20 AM: Info   You can view detailed progress in the Azure Portal:
26/11/2024 11:04:20 AM: Info   https://portal.azure.com/#view/HubsExtension/DeploymentDetailsBlade/~/overview/id/%2Fsubscriptions%2F0c5b0241-2f0c-419c-adea-51f5be95eea1%2Fproviders%2FMicrosoft.Resources%2Fdeployments%2Fsiteconnect-XXXXXXXXXXX
26/11/2024 11:04:20 AM: Info
26/11/2024 11:04:21 AM: Info   (✓) Done: Resource group: rg-siteconnect
26/11/2024 11:04:21 AM: Info   (✓) Done: Log Analytics workspace: law-vdgz464uprlhm
26/11/2024 11:04:21 AM: Info   (✓) Done: Key Vault: kv44144fb3vdgz464uprlhm
26/11/2024 11:04:21 AM: Info   (✓) Done: Container Registry: acrvdgz464uprlhm
26/11/2024 11:04:21 AM: Info   (✓) Done: Container Apps Environment: cae-vdgz464uprlhm
26/11/2024 11:06:35 AM: Info   (✓) Done: Azure Cosmos DB: cosmosdb-vdgz464uprlhm
26/11/2024 11:07:21 AM: Info
26/11/2024 11:07:21 AM: Info Deploying services (azd deploy)
26/11/2024 11:07:21 AM: Info
26/11/2024 11:07:21 AM: Info Deploying service siteconnect-api
26/11/2024 11:07:21 AM: Info Deploying service siteconnect-api (Logging in to registry)
26/11/2024 11:07:21 AM: Info Deploying service siteconnect-api (Pushing container image)
26/11/2024 11:07:47 AM: Info Deploying service siteconnect-api (Updating container app)
26/11/2024 11:08:04 AM: Info Deploying service siteconnect-api (Fetching endpoints for container app service)
26/11/2024 11:08:04 AM: Info   (✓) Done: Deploying service siteconnect-api
26/11/2024 11:08:04 AM: Info   - Endpoint: https://siteconnect-api.internal.delightfulwater-XXXXXXXXX.ukwest.azurecontainerapps.io/
26/11/2024 11:08:04 AM: Info
26/11/2024 11:08:04 AM: Info   Aspire Dashboard: https://aspire-dashboard.ext.delightfulwater-XXXXXXXXXXXXXX.ukwest.azurecontainerapps.io
26/11/2024 11:08:06 AM: Info Initialize bicep provider
26/11/2024 11:08:11 AM: Info Loading latest deployment information
26/11/2024 11:08:11 AM: Info Loading server resources
26/11/2024 11:08:12 AM: Info Loading server resources for service siteconnect-api
Once deployed, I could see the ACA new revision with a status of "activating" - After some time the revision activation failed:
Running status
Activation failed
Status details
Deployment Progress Deadline Exceeded. 0/1 replicas ready.
The original revision remains active
Expected Behavior
Expected behavior is activation is completed, and the new revision becomes active and the old revision is marked inactive.
Steps To Reproduce
No response
Exceptions (if any)
No errors or exceptions, during the actual deployment
.NET Version info
8.0.400 [C:\Program Files\dotnet\sdk]
9.0.100 [C:\Program Files\dotnet\sdk]
Anything else?
    <PackageReference Include="Aspire.Hosting.AppHost" Version="9.0.0" />
    <PackageReference Include="Aspire.Hosting.Azure.CosmosDB" Version="9.0.0" />

net8.0

Can you  share your apphost?

Appreciate the quick response @davidfowl   - assume you mean csproj
<Project Sdk="Microsoft.NET.Sdk">
<Sdk Name="Aspire.AppHost.Sdk" Version="9.0.0" />
	<PropertyGroup>
        <OutputType>Exe</OutputType>
        <TargetFramework>net9.0</TargetFramework>
        <ImplicitUsings>enable</ImplicitUsings>
        <Nullable>enable</Nullable>
        <IsAspireHost>true</IsAspireHost>
        <UserSecretsId>xxxx</UserSecretsId>
    </PropertyGroup>

    <ItemGroup>
        <PackageReference Include="Aspire.Hosting.AppHost" Version="9.0.0" />
        <PackageReference Include="Aspire.Hosting.Azure.CosmosDB" Version="9.0.0" />
    </ItemGroup>

    <ItemGroup>
        <ProjectReference Include="..\SiteConnect.API\SiteConnect.API.csproj" />
    </ItemGroup>

</Project>

Just to clarify the version I tried to deployed targeted .net 8.0
Haven't been able to try net9.0 yet.

No I mean the code in the app host.

Right on - I'll try that and revert

To circle back - switching builder.Environment.IsDevelopment() to builder.ExecutionContext.IsRunMode seems to have worked.  I assume that api.WaitFor(cosmos) call was just waiting in vain?

Was this an update between 8.x and 9.x?

it was
Matt Evans
Senior Business Developer
T (direct): +27 21 702 7514 | T:  +27 21 702 7500
P O Box 30417, Tokai, 7966, South Africa
Block H, The Terraces, Steenberg Office Park,
1 Silverwood Close, Tokai, Cape Town, South Africa

From: David Fowler @.>
Sent: Thursday, November 28, 2024 7:30:01 PM
To: dotnet/aspire @.>
Cc: Matt Evans - Wilderness @.>; Author @.>
Subject: Re: [dotnet/aspire] Aspire 9 app - azd provision / up via Visual Studio Publish (Issue #6810)
This message was sent from outside of Wilderness. Please do not reply, click links or open attachments unless you recognise the source of this email and know the content is safe.
Was this an update between 8.x and 9.x?
—
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/aspire/issues/6810#issuecomment-2506557530, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BKIALME6UIOGF6WAPO3F4TT2C5HJTAVCNFSM6AAAAABSP72ATSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBWGU2TONJTGA.
You are receiving this because you authored the thread.Message ID: @.***>
Disclaimer
The information contained in this communication from the sender is confidential. It is intended solely for use by the recipient and others authorized to receive it. If you are not the recipient, you are hereby notified that any disclosure, copying, distribution or taking action in relation of the contents of this information is strictly prohibited and may be unlawful.
This email has been scanned for viruses and malware, and automatically archived by Mimecast SA (Pty) Ltd, an innovator in Software as a Service (SaaS) for business.  Mimecast Unified Email Management (UEM) offers email continuity, security, archiving and compliance with all current legislation.  To find out more, visit http://www.mimecast.co.za/uem.

https://learn.microsoft.com/en-us/dotnet/aspire/compatibility/9.0/azure-resource-name-scheme

Right - more or less what I thought - but something of a breaking change IMO.  Anyone upgrading from 8 - 9 and redeploying will see this behavior.


Add service adapter for l7policies and l7rules.

OpenStack Release
Starting with mitaka
Description
In order to support L7 Content Switching in the agent, we must be able to adapt the policies and rules in the service object to conform to the bigip's view of these objects. This is the realm of the service adapter. This adapter is a class, instead of a class method as before, that inherits from ServiceModelAdapter and manages the translation.

Closed by PR #409


Bounty Account Refill Request - 26/06/2021 - FrontEnd

Amount Requested
250 for Bug Bounty - https://github.com/thenewboston-developers/Website/issues/1884 - 18c07b4227cb0993fa7a1db834834e42461a0d9f61b2b157dfbfee625bbf3de6
250 for PR Reward - https://github.com/thenewboston-developers/Website/issues/1884 - 94cf53921e9c5ddf48dc5589b7cdf111f7acdc93dc3073b94880416d4676b9ae
Bounty Account Number
FE Wallet - 1345375a4d25452ab0e702df2467b1fd575bcaae7d7fce49423dfce4811e1580

@angle943, is this for 252?

@tspearing from now on Chrisha will directly send those bounty coins to contributors.

Ok, so will this one be paid directly or through Justin?

Ok, so will this one be paid directly or through Justin?

Directly. This will reduce extra responsibility for team leads to do bounty payments. The team lead just need to make a Bounty Payout Request.

@tspearing from now on Chrisha will directly send those bounty coins to contributors.

Alright


fix: move saving logic outside captor package

Fix #21

PR and commit message talk about "captore" while it should be  capture

PR and commit message talk about "captore" while it should be capture

This was intentionally named captor since is the package name, where the Capture function is implemented. Not sure this is the right name for describing "someone who captures something" but this was the expected meaning.

Captor then, but it's connoted I think, maybe catcher then?

PR and commit message talk about "captore" while it should be  capture

I think I get what you didn't get, and that created confusion.
I was not reporting than captor was a bad name at first.
I reported you used captorE with a trailing "e" in PR and commit message

PR and commit message talk about "captore" while it should be  capture

I think I get what you didn't get, and that created confusion.
I was not reporting than captor was a bad name at first.
I reported you used captorE with a trailing "e" in PR and commit message

I see! thanks for clarifying ;)

Thanks for the review @ccoVeille.
I learned something new about the difference between Run and RunE :)

Yes, you should take a look at other files and refactor them too.